[Notion] Major(ish) refactoring (pt. 1)

Description
This PR will deal with refactoring the following:

[x] The DatabaseProperty and PageProperty types.
[x] Refactor PagePropertyField.tsx (63b62bddc88a7e0a2c647285283409ff1081c20b - needs testing)

The DatabaseProperty and PageProperty types
I've been putting off dealing with this for a while, but it's long overdue. Notion's method of typing objects is a nightmare in typescript. I've implemented a few hacks over the years but never dealt with the larger issue. This PR should put that nightmare to rest.
I've written an unhealthy amount of Typescript over the last half-decade, but this might be the best. I've created a new type, Standardized, and a helper function, standardize.
Here's how it works:
Currently, Notion's objects (page properties in this example) look like this.
[{ type: "checkbox", checkbox: true }, { type: "url", url: "..." }]

That's fine until you want to get the value inside a loop. For example:
props.forEach(prop => {
  prop[prop.type] // No index signature with a parameter of type 'string' was found on type...
})

So here's the solution. The Standardized type and standardize function will take those Notion objects and convert them to the following format.
[{ type: "checkbox", value: true }, { type: "url", value: "..." }]

The new key doesn't have to be "value", it can be anything. Inside of DatabaseProperty, it's "config".
Here's the best part: It's recursive.
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@thomaslombart
Can you take a look at what I have so far? I want to get a second opinion before I continue.

I'm not too familiar with the original issue. I don't see any problems with your implementation (I haven't tested the extension though). Are you looking for a second opinion on something specific?

@thomaslombart
Honestly, no really. I just want to allow everyone to put their 2 cents in.

Are there any standard practices for implementing unit testing for extensions?

Not really as of now though it's something we'd like to do at some point.

@thomaslombart @thomaslombart
How realistic would it be for ray lint to run npm test if a test script exists in package.json?
Then tests would also be run when PRs are opened.
Is there a PR for this topic?

It's feasible, but we don't have a testing framework for extensions yet, so there's no real point to it as of now.

@thomaslombart This PR is ready whenever you get the chance to look at it.
I'd like to have some people beta test it for a week, but I'm not sure how we'd do that.

@mathieudutour, Would you be able to look at this, or should I just wait for @thomaslombart to return from vacation?

We are all at an offsite next week, Thomas will have a look the week after - sorry for the delay

@HenriChabrand @mathieudutour @metakirby5 @thomaslombart @erics118 @AlexIsMaking @Yakitrak @tleo19 @dariuszpaluch @zsidnam
This PR would really benefit from some beta testing, to insure nothing is broken. Would any of you be willing to run it locally for a week or so?

@bkeys818 I'm happy to be a beta tester, but would it make sense to hold off on that until the PR is approved in case changes are required that would invalidate testing?

@zsidnam
This PR only contains refactoring, so nothing should be functionally different. That being said, there are a lot of changes. Having beta tests would ensure that everything is working like normal.
I've done a decent amount of work on this extension, and @thomaslombart and I spoke about reworking the property components here: https://github.com/raycast/extensions/pull/13507#issuecomment-2267553982. As long as everything working as expected, I'm fairly confident this PR will be accepted.

I've done a decent amount of work on this extension, and @thomaslombart and I spoke about reworking the property components here: https://github.com/raycast/extensions/pull/13507#issuecomment-2267553982. As long as everything working as expected, I'm fairly confident this PR will be accepted.

@bkeys818 sounds good; I'll install it today 👍 If there is anything specific you'd like tested, just let me know, else I'll smoke test and run through my normal workflows for the next week.

@bkeys818 I've been running this for the last week and haven't encountered any issues 👍 (I've primarily been creating new database pages and searching for existing pages -- both seem to be working fine!)

@thomaslombart Could you take a look at this PR when u get a chance?

Hi @bkeys818 👋
I'll take over this PR. I'm giving it a spin until Monday and hopefully, we'll be able to merge it then 🙂

@pernielsentikaer Perfect! Thank you!

Let's keep an eye out for bugs in the next couple of days 🙂


New charts in v2.6

System charts (v1):

Name
Version
Backend Dev(s)

~gke-operator~
~1.1.100~
~@cmurphy (if not moved to rancher/charts)~ Moved to rancher/charts

~aks-operator~
~1.0.100~
~@mjura (if not moved to rancher/charts)~ Moved to rancher/charts

~eks-operator~
~1.1.100~
~@thedadams~ Moved to rancher/charts

~rancher-operator~
~TBD~
~@thedadams @StrongMonkey (if not transitioned to Rancher)~ Moved into Rancher

monitoring
v0.2.3
@aiyengar2 @jiaqiluo

rancher-cis-benchmark
0.2.0
@prachidamle

rancher-external-dns
v0.1.3
@prachidamle

rancher-istio
1.5.920
@brendarearden

rancher-logging
0.3.0
@paynejacob

Feature charts (v2):

Name
Version
Backend Dev(s)

rancher-vsphere-csi
100.0.0
@PennyScissors

Logging v2
100.0.0+up3.12.0
@nickgerace @paynejacob

Monitoring V2
100.0.0+up16.6.0
@aiyengar2 @jiaqiluo

gee-operator
100.0.0+up1.1.1
@thedadams

aks-operator
100.0.0+up1.0.1
@thedadams

eks-operator
100.0.0+up1.1.1
@thedadams

rancher-backup
2.0.0
@jiaqiluo

rancher-alerting-drivers
100.0.0
@jiaqiluo

rancher-prom2teams
100.0.0+up0.2.0
@jiaqiluo

rancher-sachet
100.0.0
@jiaqiluo

rancher-cis-benchmark
2.0.0
@prachidamle

fleet
100.0.0+up0.3.6
@nickgerace

longhorn
100.0.0+up1.1.1

rancher-gatekeeper
100.0.0+up3.3.0
@aiyengar2

rancher-istio
100.0.0+up1.10.2
@brendarearden

rancher-kiali-server
100.0.0+up1.35.0
@brendarearden

rancher-tracing
100.0.0
@brendarearden

rancher-pushprox
100.0.0
@aiyengar2

rancher-prometheus-adapter
100.0.0+up2.14.0
@aiyengar2

rancher-grafana
100.0.0+up6.11.0
@aiyengar2

rancher-windows-exporter
100.0.0
@aiyengar2

rancher-node-exporter
100.0.0+up1.18.1
@aiyengar2

rancher-kube-state-metrics
100.0.0+up3.2.0
@aiyengar2

rancher-wins-upgrader
100.0.0+up0.0.1
@aiyengar2

system-upgrade-controller
100.0.0
@aiyengar2

rancher-webhook
1.0.0+up0.2.0 (this will be beta until right before release)
@dramich @ryansann @thedadams

release-note: need to clarify why everything moved to 100

@kinarashah Can you review that these are the versions that you see in the PRs that we make to bring the dev branches into the release branches?

Some of these are wrong / need updating. Going through now.

As discussed offline with @cbron, I have opened up the PR to change v0.2.3 -> v0.3.0 for Monitoring V1. Updated the above description accordingly as well.

PR https://github.com/rancher/system-charts/pull/516 opened to update rancher-external-dns v0.1.3 to rancher-external-dns v0.2.0

Rancher version: v2.6.0-rc10 - Fresh Install - RHEL 8.4 RKE1 Nodes
v2.5.9 -> v2.6.0-rc10 - Upgrade - RHEL 8.3 RKE1 Nodes
Fresh install worked, the fluentd pods no longer show the Ruby plugin error in the logs. The logs are flowing to Elasticsearch.
Fresh install  the fluentd image version was: v0.1.30
Upgrade path worked, logs were flowing before and after upgrade.
During Upgrade the Logging v1 App was upgraded automatically without intervention as expected from 0.2.4 to 0.3.0
During Upgrade the fluentd image was upgraded from v0.1.24 to v0.1.30
rancher-images.txt is also including this new fluentd image version v0.1.30


NO-ISSUE: Changes the package name in order to allow publishing it under @openshift-assisted

Changes the lib name to @openshift-assisted/ui-lib
This PR is meant to allow publishing the lib, after the account owning openshift-assisted-ui-lib in npm got locked.
It is meant for 2.7.11-cim.
We'll need to update the pkg name and import specifiers in solostron/console after bumping the lib version in the 2.7 branch.

/lgtm


Dummy question

Hello, is a dummy question but... where can i put the properties and configure them? thanks buddy!

Hi, props can be passed like any react component, ie:
<Slider direction="vertical" autoplay duration={5000}></Slider>


Remove diseases entered as equipment

I cast remove disease on equipment.db

@silvative is Blueblisters linked in any AV journals for the premium content?

It isn't linked in AV, no. I checked.


Patch 1

All Submissions Basics:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[ ] Have you created a .py in ~/pyod/models/?
[ ] Have you created a _example.py in ~/examples/?
[ ] Have you created a test_.py in ~/pyod/test/?
[ ] Have you lint your code locally prior to submission?

Pull Request Test Coverage Report for Build 1070

14 of 20 (70.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 95.547%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/test/test_lscp.py
14
20
70.0%

Files with Coverage Reduction
New Missed Lines
%

pyod/models/lscp.py
1
91.59%

pyod/models/base.py
1
95.49%

Totals

Change from base Build 1051:
-0.2%

Covered Lines:
3991

Relevant Lines:
4177

💛  - Coveralls


Remove the meta variables feature

@adamreichold Ok, I was wrong whan I said I never used meta variables. They were essential to build NetLogo UI. The UI syntax in NetLogo files is almost impossible to handle without them. Hm, that really makes me doubt...

They were essential to build NetLogo UI. The UI syntax in NetLogo files is almost impossible to handle without them. Hm, that really makes me doubt...

I remember that discussion. I would not subscribe to "essential" though. I would rather use "convenient" as they would avoid a lot of repetitive code. I think my main issue at the time was that a literate programming tool should not be used to work around the deficiencies of the target language.
What do people building plain NetLogo models use to handle this? Does NetLogo provide any RAD-like editors for the UI parts? Is that work flow compatible with Yarner's reverse mode? If so, I would argue that using NetLogo's native design tools to basically ingest the UI code as a opaque blob, probably even hidden in the documentation output would actually be preferable than trying to make writing this stuff by hand more manageable using macros and meta variables.

I would rather use "convenient" as they would avoid a lot of repetitive code.

The problem is not the repetitive code, but the format / "syntax". Take a look at the example and you will see...

What do people building plain NetLogo models use to handle this?

NetLogo has a dedicated "Tab" for UI etc, and normally everything is created manually, probably almost never from code. Have you never seen NetLogo?
Reverse mode is compatibel with NetLogo when the logic is moved into a .nls file, as in the example. That is because NetLogo does not support comments, exept in the code part.

I also re-opened the other MR regarding this topic, with a rebased branch. #78

If so, I would argue that using NetLogo's native design tools to basically ingest the UI code as a opaque blob, probably even hidden in the documentation output would actually be preferable than trying to make writing this stuff by hand more manageable using macros and meta variables.

Maybe this is indeed an option: Mave the NetLogo file with only UI and the other ugly stuff, as well as __includes["Codenls"] in the code tab, as a copy-code file. One would then only LP the .nls file, and code-copy the .nlogo.
With reverse mode, it should then even be possible to edit the UI in the code output, as the reverse mode also works for the copy-code files (it copies them back).
🤔

I reworked the NetLogo model to give it a try.
With reverse mode, it indeed works very well:

Start with a minimal .nlogo file in a sub-directory to be copied. Put __includes["Code.nls"] as only code in the code tab
Start LP the actual code
Run yarner
Open the .nlogo in the generated code output, edit UI
Run yarner reverse
Continue to LP the actual code

LGTM after the NetLogo clarification. The other stuff is clean-up and not really related to the issue at hand.


Create seperate document to write a good term request

Right now, guidelines for writing good term requests are buried here:
https://oboacademy.github.io/obook/lesson/ontology-term-use/#7-making-term-requests-to-existing-ontologies
I think we should have a separate guide for making a great term request. Any volunteers?

I can do it!

done

I will reopen this with a few requests for changes:

move this "lesson" to howto (lesson is not diataxis and should be reserved for very macroscopic units of teaching)
Revise the content carefully to be 100% general (no Mondo specific stuff like OMIM)
Add a more clear guide on what to do before requesting a term and add that earlier in the text. This text we have now is too short and hard to understand. This should be understandable by a database curator that has never heard of ontologies! How will they check "synonym". What is a "scope"?


New release 1.0.2 is necessary (bower install)

right now, bower install fetches v1.0.1 from Feb 2
to include icons.png, a new release should be created

It has been released, thanks for the reminder @earthquakesan


Update caching to use a flag instead of preferCache boolean.

Example of using an enum: 0=OnlyFresh 1=CachedOnFailure 2=PreferCache 3=OnlyCache

@andrewshell 4 puzzles #10, #11, #8, #9 are still not solved.

@andrewshell all 4 puzzles are solved here: #10, #11, #8, #9.


cargo regression: cdylib output path changed

I use:
$ cargo --version
cargo 0.16.0-nightly (6e0c18c 2017-01-27)

Before that version cargo create cdylib in path like this:
target/release/libhelpers_py.so at now it changed to target/release/deps/libhelpers_py-4bae97ef34a93c7e.so.
The path is important for me, because of my two use cases:

Use rust code from python, to do that I place symlink to .so library
in the same directory as python script, and this works like a charm,
I run cargo build and start python script nothing more,
at now I have to recreate symlink after every new commit?
I use rust code from java on android, build tool for java gradle,
run cargo build --target=arm-linux-androideabi --release and grab
output ".so" library, at now I have to write smart gradle script to list
directory and find file by mask?

Thanks for the report! Can you check beta/nightly to see if the bug is fixed there for you?

I suspect this was fixed by https://github.com/rust-lang/cargo/pull/3478, but just want to confirm. Definitely an unintentional regression!

Can you check beta/nightly to see if the bug is fixed there for you?

yes cargo from rustup default beta:
$ cargo --version
cargo-0.17.0-nightly (dfc0f95 2017-02-01)

fix this bug for me,
is it possible to backport bugfix to stable cargo in terms of rustup?

Ok, thanks for checking!
It's certainly possible for us to make another point release (1.15.2) but it's unclear whether the issue is severe enough to do so. For now I believe that target/release/deps/libhelpers_py.so also exists, so could both locations be checked?

It's certainly possible for us to make another point release (1.15.2) but it's unclear whether the issue is severe enough to do so. For now I believe that target/release/deps/libhelpers_py.so also exists, so could both locations be checked?

Actually, after I not found ".so" in usual place, I ran find target -name "*.so" and it show only:
$ find target/ -name "*.so"
target/release/deps/libptest_py-e4bb40c243c4e2ec.so

So no, there is no libptest_py.so, may be you not remove target directory before checking,
and there were old artifacts?

Hm ok so the regression is worse than I thought :(
I think a "fix" should be to avoid the use of workspaces, but that's obviously not exactly a great fix...

At now, with rustc 1.19.0 and cargo 0.20.0 (a60d185c8 2017-07-13) all works fine.


[BUG] Memory leak on odbc 2.4.x when querying a table with columns in TEXT type of SQLite DB

Describe your system

odbc Package Version: 2.4.6
ODBC Driver: libsqlite3odbc.so (installed with cmd: apt-get install -y libsqliteodbc)
Database Name: SQLite
Database Version: 3
Database OS: Debian GNU/Linux 11
Node.js Version: 17.6
Node.js OS: Debian GNU/Linux 11 (it's a container with nodejs: base image node:17.6)

Describe the bug
We have been using odbc 2.3.6 to access our sqlite db with no issue for two years. After we upgrade to odbc 2.4.6, we can see memory leak whenever we do query on a table with TEXT columns
Expected behavior
The process memory usage should keep stable.
To Reproduce
Steps to reproduce the behavior:

in sqlite DB, CREATE TABLE testTable (myKey TEXT, name TEXT, class TEXT, description TEXT, grade TEXT);
insert hunders of records into above table
run below code snippets with odbc 2.3.6, everything looks fine
run below code snippets with odbc 2.4.6, the memory usage of the process can be very high (>10G)
if I use new query parameter {initialBufferSize:255}, 2.4.x only, it can slow down the memory increase, but still see the leak.
if there is no TEXT column in the table, odbc 2.4.6 can work well with below code snippets.

Code
var odbc = require("odbc");
async function testDB() {
	const connConfig = {
		connectionString: 'DSN=AppDB',
		connectionTimeout: 10,
		loginTimeout: 10,
		multipleStatements: true,
	};
	try{
		var tempconn = await odbc.connect(connConfig);
		for (var i=0; i<1000; i++) {
			var results = await tempconn.query("SELECT * FROM testTable"); //parameter {initialBufferSize:255}; can slow down the crash
			console.log(`query done and get result ${i} ${results.length}`);
		}
	}catch(e){
		console.log(`*******Exception*******: Connect to DB ${connConfig.connectionString} failed`);
	}
}

The offending code:
Any DEBUG information printed to the terminal: NA
Any error information returned from a function call: NA

Additional context
Have tried 2.4.0, same issue as 2.4.6. So the issue should exist for release of 2.4.x.

@kadler @markdirish can someone please quickly review and merge this PR. We're facing same problem since 2 months.


can not create react app  Unexpected end of JSON input while parsing near '...webpack-dev-middlewar'

Is this a bug report?
(write your answer here)
yes

Did you try recovering your dependencies?

(Write your answer here.)
There is actually no package.json preinstalled .
Which terms did you search for in User Guide?

(Write your answer here if relevant.)
Environment

(paste the output of the command here)
$ npx create-react-app react-spa-demo
npx: 91 安装成功，用时 46.343 秒
Creating a new React app in D:\WebstormProjects\react-spa-demo.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts...
npm ERR! Unexpected end of JSON input while parsing near '...webpack-dev-middlewar'
npm ERR!      C:\Users\lenovo\AppData\Roaming\npm-cache_logs\2019-06-22T07_18_33_539Z-debug.log
Aborting installation.
npm install --save --save-exact --loglevel error react react-dom react-scripts has failed.
Deleting generated file... package.json
Deleting react-spa-demo/ from D:\WebstormProjects
Done.
Steps to Reproduce

(Write your steps here:)

npx create-react-app react-spa-demo

Expected Behavior

(Write what you thought would happen.)
create this app successfully without exceptions
Actual Behavior

(Write what happened. Please add screenshots!)
$ npx create-react-app react-spa-demo
npx: 91 安装成功，用时 46.343 秒
Creating a new React app in D:\WebstormProjects\react-spa-demo.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts...
npm ERR! Unexpected end of JSON input while parsing near '...webpack-dev-middlewar'
npm ERR!      C:\Users\lenovo\AppData\Roaming\npm-cache_logs\2019-06-22T07_18_33_539Z-debug.log
Aborting installation.
npm install --save --save-exact --loglevel error react react-dom react-scripts has failed.
Deleting generated file... package.json
Deleting react-spa-demo/ from D:\WebstormProjects
Done.
Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Could you paste your Error log file here?

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

This worked for me. Thanks

Cleaning the cache also worked for me, thank you so much!!

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Thank You so Much. clearing Cache helped me too to get off that error.

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

thanks, it worked for me.

Clearing cache worked for me too

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Worked for me too. Thanks!!

Iam too getting same error...tried several things but nothing worked.
Error file is as follows..
Can anyone hep me..?

I think this is a local caching issue.

Thanks man. It's working now...!

npm cache clean --force

thanks, man. this is working for me.

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

This worked for me. Thanks

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

This solution is working for me. Thank you for sharing.

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Thanks man its working!!:)

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

This worked as expected. but what it does mean?

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Its working for me too.
thanks a lot

It not work for me,I have clean the cache but got the same error.
log file attached here:
2020-05-27T09_49_21_994Z-debug.log

Could you paste your Error log file here?

npm ERR!       This might take a couple of minutes.
Installing react, react-dom, and react-scripts with cra-template...
npm ERR! Unexpected end of JSON input while parsing near '...m":"62c7d5b452bbb4ef8'
npm ERR!
Aborting installation.
npm install --save --save-exact --loglevel error react react-dom react-scripts cra-template has failed.
Deleting generated file... package.json
Deleting my-app/ from C:\Users\xxxxx
Done.
Following is the log file :
2020-06-03T14_18_58_081Z-debug.log
The Fix for this was just as @mrmckeb pointed out the cache issues
After I ran this:
C:\Users\xxxxx>npm cache clean --force
npm WARN using --force I sure hope you know what you are doing.
I was able to run the create-react-app without any issues
Thank you @mrmckeb for the fix!
Kudos

i am also getting thuis error

i tried many times your mentioned procedure but it didnt work

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Thanks it worked!!

Approach 1:

npm cache clean --force
npm update

If this doesn't work, Approach 2
Delete your package-lock.json file.
Normally, npm install would work, but you can add commands from Approach 1 again too with this.

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

This did not work for me.

npm cache verify
solved my problem. try that

I got similar issue.
It got resolved after cleaning cache
npm cache clean --force

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Thanks a lot!

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

This working as expected. but what it does mean?

You might have cancelled the same process previously which might have left behind broken, incomplete packages, clearing local cache removes all cached packages, hence fetching the proper packages and installing using them.

just clear cache, its working

I had the same issues this is procedure i followed:
C:\Users\xxxxx>npx create-react-app my-app
npx: installed 98 in 78.02s
Creating a new React app in C:\Users\xxxxx\my-app.
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts with cra-template...
npm ERR! Unexpected end of JSON input while parsing near '...m":"62c7d5b452bbb4ef8'
npm ERR!
Aborting installation.
npm install --save --save-exact --loglevel error react react-dom react-scripts cra-template has failed.
Deleting generated file... package.json
Deleting my-app/ from C:\Users\xxxxx
Done.
Following is the log file :
2020-06-03T14_18_58_081Z-debug.log
The Fix for this was just as @mrmckeb pointed out the cache issues
After I ran this:
C:\Users\xxxxx>npm cache clean --force
npm WARN using --force I sure hope you know what you are doing.
I was able to run the create-react-app without any issues
Thank you @mrmckeb for the fix!
Kudos

i am still facing same issue even after clearing cache

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

thanks a lot . it worked for me . i was trying to solve this past 2 day but finally  it solved .

I, too, face this issue. The command did not help.

I, have the same problem, excecuted previusly npm cache clean --force
Installing packages. This might take a couple of minutes.
Installing react, react-dom, and react-scripts with cra-template...
npm ERR! Unexpected end of JSON input while parsing near '...st":{"shasum":"f32fb7'
npm ERR!
Deleting generated file... package.json
Deleting login/ from /home/informix/src/node
Done.
2020-08-24T13_45_21_165Z-debug.log

Cache command not work for me. Still face this issue.

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Thank @mrmckeb it works for me.

you just need to change your registry to avoid this error
here is the list of registries:

http://r.cnpmjs.org/
https://registry.npm.taobao.org/
https://registry.nodejitsu.com/

open Command Prompt as Administrator and paste following commands one by one:
note: on URL paste any of above registry.

npm config set registry URL
npm config list
note: check the registry URL, is it the as you have change know. if yes then follow net step other try step 1 again
npm cache clear --force
npm cache verify

NOW try to install OR create project Hope so its work.
Happy Coding

you just need to change your registry to avoid this error
here is the list of registries:

http://r.cnpmjs.org/
https://registry.npm.taobao.org/
https://registry.nodejitsu.com/

open Command Prompt as Administrator and paste following commands one by one:
note: on URL paste any of above registry.

npm config set registry URL
npm config list
note: check the registry URL, is it the as you have change know. if yes then follow net step other try step 1 again
npm cache clear --force
npm cache verify

NOW try to install OR create project Hope so its work.
Happy Coding

Firstly uninstall the existing npm package:
npm uninstall -g create-react-app (-g if you installed globally)
Secondly:
npm cache clean --force
Thirdly:
npm install -g create-react-app@latest and create-react-app my-app again.

@mrmckeb you saved my day! The command is useful! Many thanks!

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

Really genius. it worked for me

If nothing from above works, try cleaning the yarn cache with  yarn cache clean . This worked for me :grinning:

This problem is mainly due slow internet speed

Hi @Evan-Tsai17, can you please try running npm cache clean --force, and then try again?
I think this is a local caching issue.

This worked for me. Thanks

Hey Cool, This worked !!!

I have the same issue and tried everything above mentioned:(

I tried everything above, still got error
If I do this:
export default function() {}
I wont get an error, but just a warning
If I do this:
export default function MyFunctionName() {}
I get that error

Thank you for your help :)
On Thu, Nov 19, 2020 at 9:44 PM Nodirbek Sharipov<EMAIL_ADDRESS>wrote:

I tried everything above, still got error
If I do this:
export default function() {}
I wont get an error, but just a warning
If I do this:
export default function MyFunctionName() {}
I get that error
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/facebook/create-react-app/issues/7261#issuecomment-730655769,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ3MEZB5SAFN6PBMCF2DPJLSQWGUFANCNFSM4H2WHFEQ
.

Hey Same error, I tried cache forcely clean, but didn't work what I do now

Hey Same error, I tried cache forcely clean, but didn't work what I do now

Try all the possible ways of exporing / importing that module which is failing. Maybe try anonymous export

I'm basically just trying to install react-native using npm install -g react-native --save, from 2 days stuck with above error. Tried all solution, cleared catch, changed internet connection, made sure under proper proxy, reinstalled node multiple times with different versions, no luck. Can anyone please help me ? And issue occurred all off a sudden.
My Environment:
OS: Mac Catalina

This worked for me as well. Thanks

This worked for me as well. Thanks

Creating a new React app in E:\local. Installing packages. This might take a couple of minutes. Installing react, react-dom, and react-scripts with cra-template... npm ERR! Unexpected end of JSON input while parsing near '.../scheduler/-/schedule' npm ERR!  C:\Users\Lenovo\AppData\Roaming\npm-cache_logs\2021-02-01T04_56_44_948Z-debug.log Aborting installation. npm install --save --save-exact --loglevel error react react-dom react-scripts cra-template has failed. Deleting generated file... package.json Deleting local/ from E:\ Done.

Creating a new React app in E:\local. Installing packages. This might take a couple of minutes. Installing react, react-dom, and react-scripts with cra-template... npm ERR! Unexpected end of JSON input while parsing near '.../scheduler/-/schedule' npm ERR!  C:\Users\Lenovo\AppData\Roaming\npm-cache_logs\2021-02-01T04_56_44_948Z-debug.log Aborting installation. npm install --save --save-exact --loglevel error react react-dom react-scripts cra-template has failed. Deleting generated file... package.json Deleting local/ from E:\ Done.

npm cache verify
solved my problem. try that

Thank you it solved my problem

this command doesn't work in ubuntu 20.04! still facing the same issue! @mrmckeb can you help?

just do sudo apt remove npm

just uninstall it
sudo apt remove npm
sudo apt remove nodejs
and then install it again
sudo apt install npm
sudo apt install nodejs
this worked for me

  Sent from Mail for Windows 10 From: Sachin9822Sent: 03 مايو, 2021 05:07 مTo: facebook/create-react-appCc: SubscribedSubject: Re: [facebook/create-react-app] can not create react app Unexpected end of JSON input while parsing near '...webpack-dev-middlewar' (#7261) just uninstall itsudo apt remove npmsudo apt remove nodejsand then install it againsudo apt install npmsudo apt install nodejsthis worked for me—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

I tried all the ways recommended before separately, but they didn't work. In the end, I tried a combination of them. First, I uninstalled the npm and the node using these commands:
sudo apt remove npm
sudo apt remove nodejs
 sudo apt install npm
sudo apt install nodejs
and then the command npm cache clean --force. Then tried the command npm cache verify. This worked for me.


Fix export with empty assignment values

Export was failing when attribute's intermediary models instances (AssignedProductAttribute, AssignedPageAttribute, AssignedVariantAttribute) exist without values

Fix export for such situations
Remove corresponding models when attribute values for instance (Page, Product, ProductVariant) are removed
Fix creating attribute headers

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2859 Export function not working for Lush


Create msi installer for jdk11

There are msi installers on the AdoptOpenJDK release download page for jdk8 but not for jdk11.

Looks like the jobs might be in the pipeline but failing: https://ci.adoptopenjdk.net/job/build-scripts/job/release/job/create_installer_windows/

Failures in the https://ci.adoptopenjdk.net/job/build-scripts/job/release/job/create_installer_windows job have now been resolved for JDK11 openj9 and hotspot builds.
See JDK11 pipeline jobs https://ci.adoptopenjdk.net/job/build-scripts/job/openjdk11-pipeline/347 and https://ci.adoptopenjdk.net/job/build-scripts/job/openjdk11-pipeline/343 as examples.

should be good to go if someone pushes a new release of jdk11

Installer for JKD11 is out


37 Typescript fixes

Fixes #37
Reminder: Pls include index.d.ts in next release ;-)

Looks great man, thanks a lot. Glad I didn't have to spend my weekend learning TypeScript :)
FYI You can add js after triple backticks in markdown to get JS syntax highlighting (and jsx too), but I'll do that later.

Edit: and ```ts for typescript
declare module 'react-recollect' {
  interface Store {
    someProp?: string[];
    somethingElse?: string;
  }
}

Thanks, David, now it needs to be battle tested.


Added --enable-experimental-typescript to enable TypeScript in cpg-neo4j

Fixes #574

Can you take a look at this @anisyusof-sc?


Update conversion-rate.md

Updating based on the discussion dt: 11/9/2023 from Commown WG to incorporate these changes, which the App Ecosystem WG suggested.
Refer to the first four proposed metrics in this list: https://docs.google.com/document/d/1i7fiI5JZaoxT12qKQlkhmbuJwg0rKcNvoGY-nuBulNg/edit
Signed-off-by: Vinod K. Ahuja<EMAIL_ADDRESS>

Closing this, as we have revised this metric since this PR. Thanks @vinodkahuja ! 🙏


Include validation to Aura CreditCard

first , I enjoyed the plugin and you guys are to be congratulated . As a suggestion , you guys can put a validation for the Aura card

Thanks!
What’s Aura card?
Could you provide a source of IIN ranges for Aura cards?

@paulorudolph07 ?

Aura is Brazilian brand I think, the pattern is /^50/.

Yeah that would be great to have validation for Aura.

@douggr
/^50/ is MasterCard so I can’t use that to match Aura.
Is Aura actually a card brand (like Visa or MasterCard) or just an issuer?

@douggr Aura is a brand which belongs to the BNP Paribas French Group and also seems to be managed by another corporation called Cetelem widely active both in Brazil, Portugal and other european countries. Their portfolio is consisted of around 3,5 million credit cards – 1 million of those already carry the Mastercard brand and the rest are still issued with the Aura brand.
I've made a brief research and indeed since 2009, the French Group and the Cetelem corporation have been developing a complete migration process to Mastercard, so the Aura brand will be obsolete and out of the market soon.
As for the Aura pattern validation, I use a Brazilian payment method on my woocommerce powered website which matches Aura cards gracefully, the plugin is called Cielo WooCommerce Webservice, it relies on jquery.payment plugin which is now supported by default in woocommerce.
You can have a look at its code here: https://github.com/greguly/cielo-woocommerce/blob/master/assets/js/credit-card/checkout-webservice.js

So it basically starts with 5078?
The question is: should I add a card, which “will be out of the market soon“?

@paulorudolph07  I believe it is a valid request from a UX perspective. The migration process might take some time, so until there I see it as a UX fix in order to match the old card brand while the migration is not complete.

But 50 is a range reserved for Maestro. I’m worried that there will be collisions. Since there is no official source for Aura, I’m very hesitant to add it.

@monecchi thank you for the great explanation! Awesome.
I double checked a validator we developed back in the days, they use /^50/ for Aura. Awkwardly all UnitTests begins with 5078, actually 50786. But I'm not sure if the "6" matters.
I'd quote @PawelDecowski opinion on this:

But 50 is a range reserved for Maestro. I’m worried that there will be collisions. Since there is no
official source for Aura, I’m very hesitant to add it.

@ceasbz If you still need this, follow the solution here #82, last comment . Abraços :)

#104

#133

#33

#73

#22

@PawelDecowski


Update notification-hub-limits.md

There is a contradiction between limits and pricing page.

Hi @schwrzKtz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@schwrzKtz : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@rothja Hi Jason Please review these proposed changes. Thanks.

#sign-off


Email never appears in inbox

Hey! I really like your work and tried it out but no email ever got sent. I used DigitalOcean to host a server and installed postfix and ran the command as shown via screenshot below. However, no email was ever sent to my mailbox. Any ideas on what could be wrong? Thanks!

Hi. the same issue here. nothing received neither on inbox nor on spam folder. any ideas?

the Spam fold

i did. also waited for couple of minutes whether takes a bit to be received. bty, i used a sender id which is spoofed by now and the reporting procedures has not helped.

I’m having another issue as well, this is what it’s giving me.
Traceback (most recent call last):
File "SimpleEmailSpoofer.py", line 288, in
raise e
smtplib.SMTPSenderRefused: (530, '5.7.0 Must issue a STARTTLS command first. a8sm41746237pfo.144 - gsmtp', 'testing@*******')
Any suggestions would be great, Thank you!


Djinni generated c++ not compiling in clang-15

To be as up front as possible: I don't think this is a problem in Djinni. However, I figured I'd post here anyway in case someone else hits this problem or wants to reference it.
I wrote the following and generated c++ with it.
widget = record +c {
    test: optional<list<widget>>;
}

The code that this generates (seen in godbolt link below) builds cleanly in the latest MSVC (19.33),  GCC (12.2), as well Clang-14.
https://godbolt.org/z/q4o5P87xW
In Clang-15 and my locally compiled Clang-16, they both result in compilation errors with these error messages:
error: arithmetic on a pointer to an incomplete type 'Widget'
error: static assertion failed due to requirement 'std::__is_complete_or_unbounded(std::__type_identity<Widget>{})': template argument must be a complete class or an unbounded array
This error was specific to Clang-16:
error: invalid application of 'sizeof' to an incomplete type 'Widget'

So it seems std::optional<std::vector<Widget>>  now requires the full type definition of Widget to compile.  Is it possible to include the definition of Widget first in your compilation unit.
Extended records is a feature that exists in the original code but not used by Snap.

I had a test app that I was experimenting against where that approach works sufficiently. However, for the larger app, reordering the include ordering was infeasible due to the scope of our project. I was able to refactor the type to not require the class extension though, so it's fine.
Again, I just wanted to make this post mostly as a heads up for other Djinni consumers who are locked into Clang since I can imagine this will hit others when Apple pulls Clang-15 into a future version of XCode.


[MM-49198] Fix reported device as desktop when coming from mobile app

Summary

Manual cherry-pick as the automated one picked up only the tests for this change.
Ticket Link

https://mattermost.atlassian.net/browse/MM-49198
Related Pull Requests

https://github.com/mattermost/mattermost-webapp/pull/11854
Release Note

NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 902fd17f324af1daa9c21045bc98cec54ad7368b


How to compute F1 score for each class?

Can we compute F1 score for each class? I am using 6 classes and I want to compute F1 score for each class. Can anyone please let me know how can we calculate this.

For now, mmdetection hasn't provided the code to compute F1 score out of the box. You can refer to mmocr to adapt this function in mmdet.

MMDetection provide AR, AP, and mAP evaluation, if you only need some of them, you can modify the metric_items, which is a parameter of the val_evaluator in the config file.


Global

Source:
Azure portal
Current Translation:
Azure Portal
Suggested Translation:
Azure portal

please help provide file url.

Hi,
Please check in file:
LAB_05_Implementing_Azure-based_recovery_services.md

Run the bugfix tool and waiting for result. ICMS QUERY

fixed


Removing empty column from row

Hi All,
When I am uploading the excel file to convert into JSON data.
I am facing the problem. Its removing the empty column which having value empty..
Please help me.
Regards,
Dhiraj

I have the same issue, I am using defval like say the documentation and it does not work.

We are also experiencing this issue.  When all rows have an empty value for a particular column, it is dropping said column.  We don't want to lose that column, we need to know that there is an empty value for that column.

I have modified the code and working fine as expected. I will ccheck the code and let you know.

I have modified the code and working fine as expected. I will ccheck the code and let you know.

That would be great if you have a fix.  After you check the code, can you share the fix with us?  We'd like to get this working as well :)

When using the sheet_to_json utility, the defval option allows you to specify a default value and force the columns to appear.
If the column header is missing, it will be stored at a dedupe'd key starting with __EMPTY.
If you can share an example we can take a closer look.

Make sure you are using the latest version of js from https://cdn.sheetjs.com/ (Read "how to use") and you can refer this https://github.com/SheetJS/sheetjs/issues/159#issuecomment-403228877


chore: promote golang-http-kardasa to version 0.0.1

chore: promote golang-http-kardasa to version 0.0.1
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: kardasa
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign

/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: kardasa
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Moving AWUDS to sciencebase

When building, user will need sciencebase credentials saved to file.  Handily, there is a function in vizlab to save this for you (needs to be run if you update your password).  Just run storeSBcreds() and enter your username and password (note that password is displayed, we should fix this).

This all set?

Yup

dev deploy will again break for a bit though (needs creds)

Ok, going to merge this after mine gets merged then. Mine #143 has the legend thing working now


[SDK-750] Update sample readme

Reformatted the sample readme with the following content. Would've committed directly myself, but I don't have edit access to this repo yet.
Python SDK samples
Test the SDK by adding VAULT-ID, VAULT-URL, and SERVICE-ACCOUNT details in
the required places for each sample.
Prerequisites

A Skyflow account. If you don't have one, register for one on the
Try Skyflow page.
Python 3.7.0 or higher.

Prepare
Install the Python SDK
pip install skyflow

Create the vault

In a browser, sign in to Skyflow Studio.
Create a vault by clicking Create Vault > Start With a Template >
Quickstart vault.
Once the vault is ready, click the gear icon and select Edit Vault Details.
Note your Vault URL and Vault ID values, then click Cancel.
You'll need these later.

Create a service account

In the side navigation click, IAM > Service Accounts > New Service Account.
For Name, enter "SDK Sample". For Roles, choose Vault Editor.
Click Create. Your browser downloads a credentials.json file. Keep
this file secure, as You'll need it for each of the samples.

The samples
Get data by ID
Get data using Skyflow IDs for the desired records.
Configure
Replace the following values in the sample file:

Value
Description

<YOUR_VAULT_ID>
ID of your vault.

<YOUR_VAULT_URL>
URL of your vault.

<SKYFLOW_ID1>
Skyflow ID of the first record.

<SKYFLOW_ID2>
Skyflow ID of the second record.

<SKYFLOW_ID3>
Skyflow ID of the third record.

<YOUR_CREDENTIALS_FILE_PATH>
relative path to your service account credentials file.

<TABLE_NAME>
Name of the table to get data from.

Run the sample
python3 getByIDSample.py

Insert data
Insert data in the vault.
Configure
Replace the following values in the sample file:

Value
Description

<YOUR_VAULT_ID>
ID of your vault.

<YOUR_VAULT_URL>
URL of your vault.

<YOUR_CREDENTIALS_FILE_PATH>
relative path to your service account credentials file.

<TABLE_NAME>
Name of the table to insert data into.

<FIELD_NAME>
Name of the column to insert data into.

<VALUE>
Valid value to insert into the corresponding column.

Run the sample
python3 InsertSample.py

Detokenize data
Detokenize a data token from the vault. Make sure the specified token is for
data that exists in the vault. If you need a valid token, use
insert_sample.py to insert the data, then use this data's
token for detokenization.
Configure
Replace the following values in the sample file:

Value
Description

<YOUR_VAULT_ID>
ID of your vault.

<YOUR_VAULT_URL>
URL of your vault.

<YOUR_CREDENTIALS_FILE_PATH>
relative path to your service account credentials file.

<FIELD_NAME>
Name of the column to insert data into.

<TOKEN>
Token for the data you want to detokenize.

Run the sample
python3 detokenizeSample.py

Invoke a connection
Skyflow Connections is a gateway service that uses Skyflow's underlying
tokenization capabilities to securely connect to first-party and third-party
services. This way, your infrastructure is never directly exposed to sensitive
data, and you offload security and compliance requirements to Skyflow.
Configure
Replace the following values in the sample file:

Value
Description

<YOUR_VAULT_ID>
ID of your vault.

<YOUR_VAULT_URL>
URL of your vault.

<YOUR_CREDENTIALS_FILE_PATH>
relative path to your service account credentials file.

<YOUR_CONNECTION_URL>
URL of your connection.

<YOUR_CONNECTION_BASIC_AUTH>
Access token for your connection.

requestBody
Your request body content.

pathParams
Your path parameters.

Run the sample
python3 invokeConnectionSample.py

Service account token generation
Generates SA Token using path of credentials file.
Configure
Replace <YOUR_CREDENTIALS_FILE_PATH> with the relative path to your service account credentials file.
Run the sample
python3 SATokenSample.py

Generate Bearer Token
Generates SA Token using json content of credentials file.
Configure
Replace credentials with the content of service account credentials file.
Run the sample
python3 generateBearerTokenFromCredsSample.py


fix: anvil ts mnemonic

Resolves #
quick fix cause anvil.ts was hardcoded to use one mnemonic instead of pulling from .env

@pavlovcik merged the development branch so that the "build and test" could pass

For the future: I think rebasing is cleaner but if this solves the problem for now then ok!


Styles break when table overflows horizontally as well as vertically

Describe the bug
When configuring a table with a fixed height (for sticky headers), which also needs horizontal scrolling due to too many columns / not enough horizontal space, the table's styles break.
To Reproduce
Steps to reproduce the behavior:

Go to https://codesandbox.io/s/o7ynj9kn0z
Drag the preview window so that it is narrow enough to cause horizontal scrolling
Observe the following issues:
i. Table headers have scrollbars (both vertical and horizontal - should not have either)
ii. The table body seems to have ~10px of horizontal scrolling going on (in addition to the main table's horizontal scrollbar, which is working as expected)
iii. The vertical scrollbar for the table body is hidden until you scroll horizontally all the way to the right

Expected behavior
The above issues do not occur
Codesandbox!
https://codesandbox.io/s/o7ynj9kn0z - Forked from the "Fixed Header + Vertical Scroll" codesandbox
Screenshots

Desktop:

OS: Ubuntu 18.04
Browser Chrome
Version 69.0.3497.100 (Official Build) (64-bit)

Additional context
Horizontal scrolling works on it's own, and sticky headers/vertical scrolling seems to work on it's own, but both at once is what causes this.

Update:
https://codesandbox.io/s/jnj5z20jqv
By removing the overflow-y: 'scroll' style on the rt-thead selector in styles.css fixes the headers having overflow - not sure why this was added to this codesandbox example?
I also added a style which hides the x overflow on rt-tbody - this fixes the strange horizontal scroll on the body.
The only remaining issue is that the vertical scrollbar is hidden until the body is horizontally scrolled all the way to the right. I can't see any way of fixing this issue, hopefully someone else may have an idea.

Happens to me as well. Is there a fix?

Note how the scroll-bar on the right shifts all cells. This also creates a weird horizontal scrolling behaviour...

Happens to me as well. Is there a fix?

Note how the scroll-bar on the right shifts all cells. This also creates a weird horizontal scrolling behaviour...

OK, this fixed it for me:
overflowY: 'scroll', overflowX: 'hidden' on Tbody


What is a release

Related to #73 and #56 ...
A gitub action is now installed on this repo.  It fires with every merge into the main branch. That could be a merge from dev or a feature branch.  I believe it will also fire with commits direct to main -- I did not test this, as I understood our workflow is to always work on a separate branch for changes to main.
Upon merge, the action will bundle up the essential components that a user will need to set up their conda environment and start using hytest workflows on their own jupyter server. The quickstart instructions refer to a download of this bundle of essential information as the minimum download to start (as opposed to cloning the repo).
An archive such as this is an 'asset' associated with a package or release in the github ecosystem. It is associated with a tag, which I made to match semantic versioning standards.  The instructions for how to create a release are included in the .github/workflows folder.

@amsnyder -- do you want this information documented in the repository somewhere?  Perhaps as part of the README? Elsewhere?

Yes - could you put it in the release description for now? That way when someone clicks on the release, they will see what exactly is contained in the release and what it is used for.

Done


MAINT - Update actions

Description
There are several warnings in our CI about deprecations due to outdated actions.
This PR

bumps relevant actions to their latest major release
adds some cosmetic changes to keep workflows consistent

Pull request checklist

[x] Did you test this change locally?
[ ] Did you update the documentation (if required)?
[ ] Did you add/update relevant tests for this change (if required)?

Additional information

It should always use main for the release action since it is only triggered on tag creation which should be created off main


[TextField] Add "input" property for custom Input component

Like Select does, It'd be useful to allow TextField to receive a custom Input component.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Passing a custom Input element via the input prop.
<TextField
  label="Name"
  input={<CustomInput name="amount" id="amount-simple" />}
/>

Current Behavior
Having to stick to the default Mui Input
<TextField
  label="Name"
/>

Context
My custom Input shows a specific icon depending on the error prop
Your Environment

Tech
Version

Material-UI
1.0.0-beta.31

@oliviertassinari the suggestion you wrote doesn't work. I'm trying to attach a masked input inside the Material UI TextField, but my application doesn't even load... something like this:
<TextField
                          label="Name"
                          InputProps={{
                            inputComponent: <CurrencyInput
                              id={`timeRanges.${index}.period`}
                              type="text"
                              placeholder="00:00"
                              decimalSeparator=":"
                              ref={input => this.inputRef = input}
                              maxLength={6}
                              value={timeRange.period}
                              onChangeEvent={this.onTextChange.bind(this)} />
                          }}/>

Do this in new versions:
<TextField
  label="Name"
  InputProps={{
    input: <CustomInput name="amount" id="amount-simple" />
  }}
/>v

@oliviertassinari I have the Input component with custom styles.
How can I reuse it in the TextField? I don't want to use global styles for that.
Probably it would help me:
const InputComponent = props.InputComponent || variantComponent[variant];

@kirillgalushko Interesting, I think that if you go down using a custom Input component (not a custom TextField) you might want to compose your own TextField component. Where does the need for the component come from?

@rastegars Did you get the CustomInput to show on the screen? I couldn't get mine and also my custom Input component doesn't show an auto suggestion list.


Tracing not propagated when using SE and Jersey client

Helidon 1.1.2
When I use jersey reactive client, tracing is not automatically propagated and I need to send SpanContext as request property:
Invocation.Builder requestBuilder = webTarget.request();

        // propagate security if defined
        request.context()
                .get(SecurityContext.class)
                .ifPresent(ctx -> requestBuilder.property(ClientSecurityFeature.PROPERTY_CONTEXT, ctx));

        // propagate tracing
        requestBuilder.property(ClientTracingFilter.CURRENT_SPAN_CONTEXT_PROPERTY_NAME, request.spanContext());

        // and reactive jersey client call
        requestBuilder.rx()
                .get(String.class)
                .thenAccept(response::send)
                .exceptionally(throwable -> {
                    // process exception
                    response.status(Http.Status.INTERNAL_SERVER_ERROR_500);
                    response.send("Failed with: " + throwable);
                    return null;
                });

Depends on https://github.com/eclipse-ee4j/jersey/pull/4226

This problem is caused by the usage of rx(). This uses an executor service that cannot be easily overridden.
The PR for Jersey was an attempt to fix a behavior when explicit executor sevice (configured in client builder) is ignored and an executor service provider (registered using method register) is used instead.
We must create a Helidon client feature that would autoregister an executor service provider, so we can wrap the executor service with Contexts.wrap, as otherwise we cannot propagate context.

Fix is based on automatic feature that configures an ExecutorServiceProvider with the Jersey client.
Behavior may be overridden via JaxRsClient.defaultExecutor to use an explicit executor service, or via JaxRsClient.configureDefaults(Config) to use a configuration based approach.


Add terraform command output validations

All Submissions:

[YES] Have you followed the guidelines in our Contributing document?
[YES] Have you added an explanation of what your changes do and why you'd like us to include them?
[YES] I have updated the documentation accordingly.
[YES] I have added tests to cover my changes.
[YES] All new and existing tests passed.
[YES] My code follows the code style of this project.
[YES] I ran lint checks locally prior to submission.
[YES] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

What is the current behavior?

What is the new behavior?

Adds the ability to test custom variable validations rules (introduced in Terraform v0.13) w/ user-defined assertions on the terraform command output.
Does this introduce a breaking change?

[NO]

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)
Other information

This LGTM. Can you resolve the conflicts between your two commits and re-push? I'll also be merging a commit shortly so a re-base would be helpful.

Bonus points if you add a unit test (see latest commit in master) to cover this scenario, but it is not necessary


This is a virus

Don't recommend. 1. Sometimes doesn't detect when I go in a level 2. After recording a macro and setting the speed to 1, it glitches and plays the macro at approximately 2.175894676845675867875857475756786867867958 speed

How can you call it a virus when the source code it right there
On Tue, 3 May 2022 at 18:09, Wavar Deimsar @.***> wrote:

Don't recommend. 1. Sometimes doesn't detect when I go in a level 2. After
recording a macro and setting the speed to 1, it glitches and plays the
macro at approximately 2.175894676845675867875857475756786867867958 speed
—
Reply to this email directly, view it on GitHub
https://github.com/decoder0007/PosBor/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMASUCMDYTAM7FWBCJTHROTVIFMVNANCNFSM5U7QN23Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I have different terms of the word virus. When something looks malicious, then it's a virus. Also I have a problem after downloading this. One of the backgrounds is now about 1.5x brighter than normal. It's really distracting.

I have different terms of the word virus. When something looks malicious, then it's a virus. Also I have a problem after downloading this. One of the backgrounds is now about 1.5x brighter than normal. It's really distracting.

ok i managed to fix it

But… the source code it right there. You cannot call it a virus until you
or someone else has checked through it.
On Wed, 4 May 2022 at 19:56, Wavar Deimsar @.***> wrote:

I have different terms of the word virus. When something looks malicious,
then it's a virus. Also I have a problem after downloading this. One of the
backgrounds is now about 1.5x brighter than normal. It's really distracting.
ok i managed to fix it
—
Reply to this email directly, view it on GitHub
https://github.com/decoder0007/PosBor/issues/2#issuecomment-1117693684,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMASUCMWQIRC6RI2W7LYLO3VILB45ANCNFSM5U7QN23Q
.
You are receiving this because you commented.Message ID:
@.***>


Hotfix/remove duplicated webpack config file

Fix duplicated webpack config file

remove the old one from build folder
update package.json build and dev tasks to point to webpack config file located now inside the build folder
add CleanWebpackPlugin root accordingly
use constants to define all sensitive data
use only one webpack config object with multiple entries

As agreed, I will create a new PR with 2 separate webpack config files (one for CMP and another one for GLOBALSTORAGE)

Codecov Report

Merging #31 into feature/develop-v1.0.0 will decrease coverage by 0.69%.
The diff coverage is n/a.

@@                    Coverage Diff                    @@
##           feature/develop-v1.0.0      #31     +/-   ##
=========================================================
- Coverage                   97.16%   96.47%   -0.7%
=========================================================
  Files                          49       49
  Lines                        1799     1870     +71
  Branches                      145      150      +5
=========================================================
+ Hits                         1748     1804     +56
- Misses                         51       66     +15

Impacted Files
Coverage Δ

...p/infrastructure/container/BaseConsentContainer.js
77.24% <0%> (-5.78%)
:arrow_down:

...frastructure/repository/CookieConsentRepository.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix lims for FixedCoordProvider

We need a different way of handling xlim, ylim of coord_fixed: in addition to x,y-domen adjustments we heed to set clippath in accordance to the given limits.
@IKupriyanov-HORIS please re-open the issue.


Run update-payload-crds after update-codegen-crds

When working on #1928, noticed that the payload crds were not generated after running a make update. This is because codegen is run after update-scripts. This PR removes update-payload-crds from update-scripts and runs it after codegen.

/lgtm

/retest-required

/retest-required

/test e2e-aws-serial-techpreview

/hold cancel
/override ci/prow/e2e-aws-ovn
This PR doesn't affect the E2Es


Update javadoc plugin

It also adds a new path for the auto-labelert plugin.

@Fokko I saw your recent message about the 1.9 branch cut, can you give me just until wednesday max to do the PR (I think I will do it even if some modules are unfinished) it is mostly static analysisis cleanups. Sadly I have been a bit sick and not been able to work on it.

Take care @iemejia Probably I'll cut the branch next week if that is okay by you.


Expand coordinate only from start of string

do not expand each component separately
repeat until no shortcut matches

Fixes #16

I changed expand_coordinate:
def expand_coordinate(coordinate, shortcuts={}):
    was_changed = True
    while was_changed:
        matched_shortcut = False
        for shortcut, replacement in shortcuts.items():
            if coordinate.startswith(shortcut):
                _logger.debug("Replacing %s with %s in %s.", shortcut, replacement, coordinate)
                coordinate = coordinate.replace(shortcut, replacement)
                matched_shortcut = True
        was_changed = matched_shortcut

With ~/.jrunrc:
[shortcuts]
nij = net.imagej
imagej = nij:imagej

imagej:2.0.0-rc-67 gets expanded to net.imagej:imagej:2.0.0-rc-67
net.imagej:imagej:2.0.0-rc-67 gets expanded to net.imagej:imagej:2.0.0-rc-67 (stays the same)

I am happy with this but I have one more question: Should every shortcut be applied once at the most for each coordiante? I.e. if, in a single call to expand_coordinate, a coordinate is matched, it cannot applied a second time? This would prevent recursive shortcuts.

@ctrueden I modified the behavior such that each shortcut gets applied once at max to avoid recursive shortcuts like
some.shortcut = some.shortcut.expanded

Worst case (every single shortcut is applied) complexity is N^2 where N is the number of shortcuts but that is

unlikely
N will be pretty small (probably tens)

Thanks @hanslovsky !


Global Illumination: Fix WGSL code

I also fixed debug marker groups in WebGPU.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15670/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15670/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15670/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15670/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15670/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15670/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15670/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15670/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15670/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15670/merge/testResults/webgl2playwright/index.html


Add CI Tests

This adds a very simple CI action to confirm that nothing breaks CI.
We only have 12 tests right now so it'll need to be a major breaking change

LGTM, thank you for doing this!


get_rights vs get_by_left

pub fn get_rights(&self, left: &L) -> Option<Vec<&R>> where L: PartialEq,

vs
pub fn get_by_left(&self, left: &L) -> Option<Vec<&R>> where L: PartialEq,

My editor displays the following.
I decided to use get_rights because it is easier to understand that it takes left as an argument and returns rights.

I looked at the list of methods in the documentation and thought get_by_left was better.
get_rights and rights are confusing.


Putting functions into the appropriate modules

(This is where the code is consolidated; indeed some conflicts need to be resolved)


Fix Jitter on all Modals

The reason there's a jitter on close/open of modal is because we often don't render the modal when the state to show it's visibility is set to false.
What we do
{showModal && <ReactNativeModal isVisible={showModal} ... />}

Because of this what happens is there's a re-render when the modal is toggled and the animation in/out couldn't be played properly. This causes the experience to be a little jittery especially on slower phones.
What we should do
<ReactNativeModal isVisible={showModal} ... />

This would handle the animation as well nicely.

AmountInputModal
AppUpdateModal
CustomModal
Stablesats
EarnQuiz
ContactModal
SetLightningAddressModal
ChooseWalletModal
ModalNfc
AccountDeletionModal
UpgradeAccountModal

@sandipndev are these all the Modals to modify?

Yup, that should be all @Aditi-Singh16


Do not require db url for prepare

Continuation of #2675, since I didn't have permission to reopen that PR.
I've rebased and restored the eager connection checks when DATABASE_URL is provided.

I didn't realize that the PR author wouldn't be able to reopen it if I closed it. I wonder if that's in the settings somewhere.


Feature/minor-changes-to-readme-instructions

minor patch to update the REAME.md file to ensure users have a clear directive on what they need to check for step 4 in order to move on to step 5.

Thanks! @DHolmanCoding Please feel free to merge whenever.


Serde for key identifiers

What problem does this solve or what need does it fill?
Currently key bindings cannot be serialized or deserialized.
It's a necessary feature for storing key binding information.
What solution would you like?
Add serialization and deserialization for existing key identifiers:

MouseButton
KeyCode
GamepadButtonType

A related problem that should be also considered is serde for device type (mouse, keyboard, gamepad), where there can be multiple gamepads.

It's already implemented, but hidden under the the serialize feature.


Adds section for using Azure Managed Identity with KeyVault

Adds instructions for using Azure Managed Identity with KeyVault

@willtsai  @greenie-msft


KAFKA-4139: Reset findCoordinatorFuture when brokers are unavailable

Thanks for the PR @rajinisivaram, would it be possible to include a test? cc @hachikuji

@ijuma Have added  a unit test. Thank you.

LGTM, but I'll leave it for @hachikuji to review and merge.

LGTM. Merged to trunk.


turn off burstable t instance for now

Issue #, if available:
N/A
Description of changes:
Turn off burstable instances for now until we determine how to handle the details of cpu credits.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

So easy!


Code to generate spraggins.ome.tiff fixture?

The spraggins.ome.tif fixture is not a properly formatted OME-TIFF. Do we know where this fixture came from?

I added this script in #90


Add in useful tools

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @geraldooi, thank you for the PR! However, we do not accept PRs that only add links.
I'm going to close this PR, but feel free to open a new one with more content. Happy coding!


Unobserved Exception thrown while running cache refresh task

We are continuously addressing and improving the SDK, if possible, make sure the problem persist in the latest SDK version.

Describe the bug
User recently upgraded their service's CosmosDB SDK from v3.32.3 to v3.33.0 and is experiencing unhandled ArgumentExceptions from the CosmosDB SDK occasionally.
Logs:
[OnUnhandledException,UnhandledExceptionReporter.cs(37) <NoCorrelationId> ] An unhandled exception has occurred. The process may be in an inconsistent state. Context: TaskScheduler_UnobservedTaskException CorrelationId:
ExpInfo(fullInfo->System.AggregateException: A Task's exception(s) were not observed either by Waiting on the Task or accessing its Exception property. As a result, the unobserved exception was rethrown by the finalizer thread. (Identifier {0} does not exist. Please call start before calling end. (Parameter '7fe193c0-a931-4f29-a8b2-9af6e849b62a'))
 ---> System.ArgumentException: Identifier {0} does not exist. Please call start before calling end. (Parameter '7fe193c0-a931-4f29-a8b2-9af6e849b62a')
   at Microsoft.Azure.Cosmos.Tracing.TraceData.ClientSideRequestStatisticsTraceDatum.RecordAddressResolutionEnd(String identifier)
   at Microsoft.Azure.Cosmos.Routing.GatewayAddressCache.GetServerAddressesViaGatewayAsync(DocumentServiceRequest request, String collectionRid, IEnumerable`1 partitionKeyRangeIds, Boolean forceRefresh)
   at Microsoft.Azure.Cosmos.Routing.GatewayAddressCache.GetAddressesForRangeIdAsync(DocumentServiceRequest request, PartitionAddressInformation cachedAddresses, String collectionRid, String partitionKeyRangeId, Boolean forceRefresh)
   at Microsoft.Azure.Cosmos.AsyncCacheNonBlocking`2.AsyncLazyWithRefreshTask`1.CreateAndWaitForBackgroundRefreshTaskAsync(Func`2 createRefreshTask)
   at Microsoft.Azure.Cosmos.AsyncCacheNonBlocking`2.UpdateCacheAndGetValueFromBackgroundTaskAsync(TKey key, AsyncLazyWithRefreshTask`1 initialValue, Func`2 callbackDelegate, String operationName)
   at Microsoft.Azure.Cosmos.AsyncCacheNonBlocking`2.RefreshAsync(TKey key, Func`2 singleValueInitFunc)
   at Microsoft.Azure.Cosmos.Routing.GatewayAddressCache.<>c__DisplayClass25_0.<<TryGetAddressesAsync>b__5>d.MoveNext()
   --- End of inner exception stack trace ---)

Needs to investigate:

Root cause for unobserved task exceptions and can it be avoided.
Why is this issue prevalent in certain version v3.33 and not in other v3.32/.

Environment summary
SDK Version: v3.33
OS Version (e.g. Windows, Linux, MacOSX)
Additional context
Add any other context about the problem here (for example, complete stack traces or logs).

@aavasthy
Isn't this the same as https://github.com/Azure/azure-cosmos-dotnet-v3/issues/4010?

It was released in 3.36.0

@kundadebdatta the issue looks the same as this https://github.com/Azure/azure-cosmos-dotnet-v3/issues/4010. Please confirm if the fix is in 3.36.0.

Duplicate issue: https://github.com/Azure/azure-cosmos-dotnet-v3/issues/4010


Neofetch formulae not upgraded in Brew

Brew formulae for neofetch doesn't seem to be updated to v3.3.1. Due to this, there doesn't seem to be support for macOS High Sierra.

The latest neofetch release is 3.3.0, not 3.3.1. Therefore there are no High Sierra support yet because High Sierra is released way after Neofetch 3.3.0 is released.

Anyway, you can use the git version of neofetch for High Sierra support.


Support for Prebuilt frameworks to use with Carthage

It would be nice if for each tagged release you enabled pre built frameworks also - this would enable Carthage users to have faster build times.
See https://github.com/Carthage/Carthage#archive-prebuilt-frameworks-into-one-zip-file for details on how to implement this. (it's pretty easy)

I am not a Carthage user, but if you would like to package them up, test it, and submit a pull request, I would take a look and see about merging into master.

Not much code to add I built these from the head of the master - but I think this is close to the 0.1.1 release.

I'm closing this issue, it all seems 👍🏼.


Implement less restricted mixins (aka allow super() calls)

DEP 34 -- allowing mixins to have super() calls & superclasses other than object -- has been accepted (but has not yet gone through TC 52). That means now is a great time to start implementing it. That way we can iterate on the details if needed before they are carved into the spec.
Since this feature is not in the spec yet, the implementation should be gated behind a flag called "supermixin".
This is the main tracking bug. Each platform's specific bug is below:
Required for 1.13:

[x] VM (#23771)
[x] analyzer (#23772)
[x] Remove flag from VM (https://github.com/dart-lang/sdk/issues/24479)
[x] Update docs (https://github.com/dart-lang/www.dartlang.org/issues/1475)
[x] Add UI to enable flag for Atom (https://github.com/dart-atom/dartlang/issues/370)

Post 1.13:

[ ] DDC: https://github.com/dart-lang/sdk/issues/27344
[ ] dart2js (#23773)
[x] TC52 approval (#24262)
[ ] Add UI to enable flag for Eclipse

@munificent you have the dart2js and analyzer bug numbers swapped.

Thanks @donny-dont , I fixed it.

Thanks, homies!

fyi DDC bug is

[ ] https://github.com/dart-lang/dev_compiler/issues/306

munificient@, does this track both these two parts that are in the DEP?

Mixins can refer to super.
Mixins can have superclasses other than Object.

Updated task list.

Yes, my intent is that this bug is for the whole proposal. (Also, neither of those parts is very useful without the other.)

I see the tasks for 1.13, and those for after, in order to add support for this flag. Is there a time-frame for removing the flag though? Or is that entirely dependent on TC52?

That's entirely dependent on TC52.

Seth, can we mark this complete for 1.13?

Still waiting for docs, but that doesn't impact the branch point.

Docs bug is at dart-lang/www.dartlang.org#1475

Is this officially enabled in 1.13? Didn't see any word in the changelog.

Hi @donny-dont, yes this is in 1.13, however as it has not been approved by TC52 and as it is not yet supported in dart2js, it's behind a flag -- see https://github.com/dart-lang/sdk/issues/23772#issuecomment-143890654

analyzer:
  language:
    enableSuperMixins: true

in .analysis_options should do it (not tested)

This was approved by TC52 in December'15.
It is not yet supported in dart2js.

FWIW Workiva would benefit from this work.

Work on this was finished long ago


Temporary Destroyed Block

A block that remains destroyed while the raid is ongoing, but returns when the raid is over. Potentially remains destroyed if the raid is lost.
Useful to add creepers to the raid, without having a wave of them permanently destroy your whole village

implemented


chore: adaptations upstreaming norm_cast

Mathlib adaptations required for upstreaming norm_cast in leanprover/std4#351.
Nearly all of the changes are using the term elaborator mod_cast rather than by exact_mod_cast, which is usually just a synonym for by exact mod_cast.
(We still have the exact_mod_cast tactic, used in a few places, which somehow (?) works a little bit harder to prevent the expected type influencing the elaboration of the term. I would like to get to the bottom of this, and it will be easier once the only usages of exact_mod_cast are the ones that don't work using the term elaborator by itself.)

Closed, I've split this into #8403 and #8404.


This still an issue.

:-(


Took out icon to avoid unnessecary Manifest Merger failures

No need for an android icon on a library, and it just causes a bug so its got to go!

@CapitanRedBeard Thanks for the PR!
I think there is some confusion here... hard to say with all the extra commits in this PR.
I removed the launcher icon back in april:
https://github.com/dbachelder/CreditCardEntry/commit/c506433a5932c58286264589f608b7c3eb1e9e75
Is there anything else in this PR?

Yeah sorry, didnt mean to make a pull request. I'll try out 1.3 and get back to you, thanks for the quick response! I was on 1.2.1 btw


Active Maintenance of Cloudworker?

In the README, it says Dollar Shave Club is no longer actively maintaining Cloudworker, what does that mean?

Are you still going to be merging PR's/reviewing issues until a new owner is found?
Has Dollar Shave Club decided to move away from Cloudflare Workers? If so, I'd love to know why.

The person who was the primary maintainer of cloudworker no longer can do so. We have a few people who are contributing time here and there but it will be best-effort. The README tries to make this clear to anyone who opens an issue or bug.

Cloudflare's wrangler CLI has a dev command that supplants the need for cloudworker.
See here: https://github.com/cloudflare/wrangler#-dev

@aleclarson it's a great solution but doesn't work offline and it's not pratical to me in unit test/CI.

doesn't work offline

You sure about that? The docs say it starts a localhost server. Why wouldn't that work offline?


Grammatical error in the Farming Tracker notification

Describe the bug
I received a notification that makes no grammatical sense, Your Compost BinArdougne.
To Reproduce
I am unsure on this, I got the notification randomly while in Edgware.
Expected behavior
Produce a grammatically correct notification such as Your compost bin in Ardougne is empty., for example.
Screenshots
While in Edgware:

After visiting the patch in Ardougne:

Environment (please complete the following information):

OS: N/A
RuneLite version: 1.6.40-SNAPSHOT
Launcher version: Unknown

Additional context
I believe the code responsible for this is located here:
https://github.com/runelite/runelite/blob/master/runelite-client/src/main/java/net/runelite/client/plugins/timetracking/farming/FarmingTracker.java#L576-L603
I think adding a check to make sure the prediction.getCropState() != EMPTY should fix this, as I don't believe a notification is needed for an EMPTY state at all.

It may also be good to move this notification logic to its own method which you can then test. cc @loldudester

This commit should resolve this issue:  https://github.com/runelite/runelite/commit/525c64b8dfa239b9bba5e8e7e12ee8684ac130ef
A fall through case was added that prevents notifications from being sent when the compost bin is empty. See https://github.com/runelite/runelite/blob/1684eccd006560f984474ffea1248dd8666df78c/runelite-client/src/main/java/net/runelite/client/plugins/timetracking/farming/FarmingTracker.java#L592

Fixed by #13189


Images are not uploading while importing the site.

Describe the bug
While importing a site, the images are not being uploaded in Asset folder.
To Reproduce
Steps to reproduce the behavior:

Import a site (using Create Site).
Preview pages of the imported site.
Images are missing in html pages.

Marking this as closed as the issue is resolved.


Feature request: panBy()

Hi Masashi,
since we're now able to place content on top of the map it would be nice to recenter our markers sometimes. Instead of providing a difficult way of recalculating world's card it would be very nice to have the "panBy" function.
You can pass a X and Y value in pixels to pan the map. In my case I like to move the pan -100px by top.
In JS, this could be done with:
map.panBy(0,-100);

Here is an example:
http://jsfiddle.net/doktormolle/JsBEx/
Google:
https://developers.google.com/maps/documentation/javascript/reference
I'm don't know if this is supported by Google SDK for iOS and Android. I hope so!
Edit:
http://stackoverflow.com/questions/15625970/offseting-the-center-of-the-mapfragment-for-an-animation-moving-both-the-target
https://gist.github.com/codebutler/6925097
(but maybe old and outdated. I hope panBy is implemented in SDK)

Thanks a lot for the panBy-feature.
Is there any possibility to move the camera to Coordinates with offset in one move?
My Challenge is to focus a Marker after click - but the marker shall not be in the middle of the screen, but near the bottom. Currently I am moving the Camera to the Marker and after that I am panning the map up. The result is right, but of course not the animation before. First it moves to center the marker, afterwards it pans animated up. Looks a little weird.
Thanks for replies :)


Flask/Gunicorn server api

Hi,
Hope you are all well !
I would like to setup a flask server with a query endpoint for processing a message.
Is it possible to push an example of gpt2bot with a flask server or gunicorn ?
In fact, I am writing a multi bot project (written in Golang) and I d like to aggregate response of several type of chatbots with an agent. The projet reference is there: https://github.com/paper2code/telegram-multibot
As I am much more a gopher than a pythonista, I am requesting your kind help on that. ^^
Can you help me on that ?
Cheers,
X

You need to adapt interactive_bot.py by creating a route around the following lines https://github.com/polakowo/gpt2bot/blob/a091b7ec7b350819ca5cf3efa5675ed420890fc8/gpt2bot/interactive_bot.py#L35-L66
Having query prompt it will generate a bot message bot_message that you can send back to the user.
To create a Flask app for text generation you can look at my other project textai, for example this app. I'm currently preparing a new release of gpt2bot, so maybe after I finished that I will consider creating a Flask version.

Awesome mate :-)
When do you think you gonna release it ?
For the flask server, we managed to create one so no hurries ^^
https://github.com/paper2code/gpt2bot/blob/master/server.py
Cheers,
X

Looks good. You would also need some sort of session management and ideally a route for updating the parameters of the generator such as temperature, no need to restart the server to update the config.
I will release it the next week I guess, I'm trying to decouple it from dialogpt and generalize it to accept any gpt2 model.

Hi @polakowo
I tried to create a new flask server from your latest changes with no luck...
Do you mind to create it ? That would be so awesome and kind of you :-)
Cheers,
X


Reservation Flags

#462 implemented a bitmask for miscellaneous boolean values for reservations, which is currently used to flag requests and for an email task. This should be used to mark any issues with the reservation; for example, if it is lost or broken, the reservation should be flagged as such.
This also means adding checkboxes to the check-in form and the archive JS pop up to set these flags.
See here and here for more information on bitmasks.

This can be thought of as addressing the original impetus for #462; namely, the ability to flag reservations with different attribute (such as equipment broken) for more classification options. Note that we might want to figure out a way to generate an "ad-hoc" scope for reservations that match a particular bitmask attribute.

The fined flag also needs to be implemented.

Oh, there's also information in our wiki about the flags: see here

And we have a scope that returns reservations with a particular flag as of #462, so that is no longer necessary for this issue

Updated the issue description with a consolidated to-do list.


RANGER-5026: agents-cred module: update for code readability improvement

What changes were proposed in this pull request?
Updated agents-cred module sources to comply with the checkstyle requirements (subtask of RANGER-5017).
How was this patch tested?
Build with unit tests completed successfully.

merged in master branch.


Add progress bars on scroll reveal

Is your feature request related to a problem? Please describe.
Whenever user will scroll the screen, the progress bars will appear and when the user scrolls up back, the progress bar will disappear.

Is your feature request related to a problem? Please describe. Whenever user will scroll the screen, the progress bars will appear and when the user scrolls up back, the progress bar will disappear.

Assigning it to you


Typing issues with Board.backend

I think the issue is partially due to backend being of type Type[Backend] rather than
Backend. If the below code is used  MyPy is happy but it will almost certainly run time error when we don't provide the necessary values to the subtype's constructor.
board_type: Type['Board'] = backend().board

Converting Backend.board from a @property to a @classmethod is the only thing, other than the cast, that I can think of but that would require large changes to the codebase and Backend.board() doesn't feel particularly nice, though a rename could fix that.
Do you think it's worth changing, or should we just leave as the cast?
Originally posted by @sedders123 in https://github.com/j5api/j5/pull/406

Possibly relevant: #342


When I start notion-enhancer, infinite loading error

What is happening?
infinite loading happen occured
What should be happening?
Solve infinite loading problems
Screenshots (if applicable):

Operating system/browser name & version:
Window 10, Chrome 118.0.5993.118 / Notion-Enhanced-Setup-2.0.18-1
notion-enhancer and/or notion-repackaged version:
notion-enhancer
notion-enhancer configuration (enabled/disabled mods and their settings):

see https://github.com/notion-enhancer/notion-repackaged/issues/116

When  starting notion-enhancer（the lastest version on window）, it will appear infinite loading error，what should i do

@GRACEBL you should see https://github.com/notion-enhancer/notion-repackaged/issues/116


Add support for TagHelper/Component parsing.

Expanded the OmniSharpPlugin project to now spin up a project engine and use it to resolve TagHelpers in the context of the associated OmniSharp project.
Updated the Razor dependency of all projects, this included consuming new changes for ProjectWorkspaceState which enables us to replace TagHelpers for a given project without needing a Roslyn project.
Due to strong naming restrictions had to Assembly.Load and reflectively invoke several classes, this included the TagHelperResolver.
Since we now have minimal project engines being spun up in the OmniSharp side of the world added a LanguageServer.Common project that encompasses the minimal logic needed to build a proper project engine.
Expanded our language querying mechanism to understand TagHelpers better. If we find a non-TagHelper classified span we use it to determine the content type, otherwise we fallback to TagHelper classified spans to determine if an area should be treated as Html or not.
Added tests where I could. Because of how the OmniSharp plugin works was not able to add a TagHelper functional test to validate the correctness of TagHelpers. This is because our functional tests use the currently released OmniSharp which does not ship the appropriate OmniSharp plugin that understands TagHelpers. We also can't force the OmniSharp server to load the new plugin because we can't accurately resolve the path at configuration time to tell OmniSharp early enough which path to use for anyone who clones the Razor.VSCode repo.

#16

As a future note for testing since the currently released version of OmniSharp can't load the plugin in this PR. You need to add the following settings to your dev project when verifying:
{
    "razor.devmode": true,
    "razor.plugin.path": "C:/GitHub/Razor.VSCode/src/Microsoft.AspNetCore.Razor.OmniSharpPlugin/bin/Debug/net461/Microsoft.AspNetCore.Razor.OmniSharpPlugin.dll",
    "omnisharp.path": "1.32.9-beta.56"
}

Merging this to unblock myself, feel free to review afterwards and will address comments then.


Minimum metadata needed to operate

#31 exposes the current metadata for a Register. This opens the question of what is the minimum metadata needed to operate a register? That could be in development, discovery, production, … so it is mostly a technical question with any extra guarantees we need to have on top of that.
RFC0018 defines the context as:
type Context =
  { id : Name
  , copyright : Maybe String
  , custodian : Maybe String
  , description : Maybe String
  , hashingAlgorithm : HashingAlgorithm
  , licence : Maybe String
  , rootHash : Hash
  , schema : Schema
  , stats : Stats
  , status : Status
  , title : Maybe String
  }

This suggests that id, copyright, hashing algorithm, schema are the minimum information needed for the Register to compose this resource.
Note that root hash, stats and status are computed values.
I'll try to describe, one by one, the reason they are part of the context and my thoughts on why they need to be present.
id: The identifier of the register is still connected with the attribute name for the primary key, the domain name and the curie prefix. Once this is untangled we could reassess if this even needs to exist inside a register.
hashing algorithm: Required to know how to hash blobs, entries, proofs, etc.
schema: Required to validate the data to be stored in the register.
copyright: Technically not required. Except in development or casual deploys, I think this should be explicitly stated. Given that is not required to operate it is optional. It would be nice to find a way to ensure this is present on registers that need to be published, perhaps an external checker.
licence:  Technically not required. Except in development or casual deploys, I think this should be explicitly stated. Given that is not required to operate it is optional. It would be nice to find a way to ensure this is present on registers that need to be published, perhaps an external checker.
custodian:  Technically not required. Except in development or casual deploys, I think this should be explicitly stated. Given that is not required to operate it is optional. It would be nice to find a way to ensure this is present on registers that need to be published, perhaps an external checker.
description: Technically not required. Nice for discovery and supporting the catalogue.
title: Technically not required. Nice for discovery and supporting the catalogue.
So, with id and hashing algorithm a register can exist, empty. Once a schema is defined, then it can be used.

#31 proposes an approach for this


unfinished binary MIR

Is there any way to debug the error unfinished binary MIR given when reading a MIR module that was writen using the exact same code just couple of lines before of it?
Trying to read the code I see that there is quite a bit of encoding going on and I can't really figure out what is wrong...
This is the MIR_output that I get right before using the MIR_write function: https://pastebin.com/ffWhJLDN

debugging a bit more it seems that
my_module:	module
	import	System.Reflection.ExceptionHandlingClause
	import	System.Reflection.ExceptionHandlingClauseOptions
	endmodule

is enough to trigger it, it only happens when using the MIR api, and not the scanning methods (aka, using MIR_import), and it does not happen for any overlapping strings, for example trying:
my_module:	module
	import	aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
	import	aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
	endmodule

but for
my_module:	module
	import	aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
	import	aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaabaaaaaa
	endmodule

it did trigger it, if I had to guess, this is some strange weirdness with the string interning logic, but I am trying to dig deeper

after a bit of testing I think it is a bug in my code, closing for now


WFCORE-5800 Remove unused Charset Map Variable and Method in ManagementHttpServer class

https://issues.redhat.com/browse/WFCORE-5800

@clara0 thanks


[GUI] Template/Device screen with cards over spaced

Version:

branch: development
commit 86e19293912490387fcbe6aa59d9969d06f6d5de)

just one more information, is you click on space below the card it opens the template (it does not happen with the device).

Test OK.

Devices:

Templates:

Tested in version: 63.1-20190705


sensu server unable to connect to redis using haproxy

I want to scale sensu and using this guide to setup-file:///Docs/sensu/Highly%20Available%20Sensu%20%20%20DevOps%20in%20the%20Real%20World.html.
I have two sensu servers each running rabbitmq instance as cluster and 4 redis servers(1 master, 3 slaves).One sensu server is easliy connected to redis using haproxy while other is unable to connect ro redis server as showing logs-

Also sensu server stops after sometime. While sensu clients are able to connect to redis.Please provide me reasons as what is causing this error.Thanks in advance

@snehahaldar are both Sensu servers using the same instance of HAProxy? Is HAProxy using a backend check for Redis master?

yes both sensu servers have same instance of haproxy.Rabbitmq and sensu api load balancing is working fine with haproxy but unable to load balance redis. As once redis master goes down all connections are lost. I tried with backend also but it is also not working.Also after some time sensu api crashes and shows log like this:

@snehahaldar what does your haproxy backend configuration look like? Redis cannot be "load balanced", haproxy is only there to redirect traffic to the current master.

Below is the my haproxy backend configuration:-
defaults REDIS
mode tcp
timeout connect  4s
timeout server  30s
timeout client  30s
frontend ft_redis
bind <IP_ADDRESS>:6379 name redis
default_backend bk_redis
backend bk_redis
option tcp-check
tcp-check connect
tcp-check send PINGrn
tcp-check expect string +PONG
tcp-check send info replicationrn
tcp-check expect string role:master
tcp-check send QUITrn
tcp-check expect string +OK
server powermngr <IP_ADDRESS>:6379 check inter 1s
server sensu1 <IP_ADDRESS>:6379 check inter 1s
server rd-slave1 <IP_ADDRESS>:6379 check inter 1s
server rd-slave2 <IP_ADDRESS>:6379 check inter 1s
Using this configuration haproxy is unable to redirect traffic to current redis master.Do we have any other alternative for making redis failsafe for sensu environment?

@snehahaldar do you have the haproxy web interface enabled? Is the current redis master showing up as available in haproxy?

@snehahaldar any update on this?

Closing due to inactivity, and because HAProxy configurations are no longer required thanks to the native Redis Sentinel support released in version 0.23 (which is now also fully documented, here: https://sensuapp.org/docs/latest/redis#redis-high-availability-configuration. Please feel free to create a new issue if this matter is not resolved. #monitoringlove