町字が二重で追加される

Example Data
"愛媛県南宇和郡愛南町中塚","愛媛県南宇和郡愛南町中町塚",0.93,"machiaza_detail"
"愛知県刈谷市上重原町丸山","愛知県刈谷市小垣江町上重原町丸山",0.75,"machiaza_detail"
"愛知県刈谷市上重原町恩田","愛知県刈谷市小垣江町上重原町恩田",0.75,"machiaza_detail"
"愛知県安城市八ツ田町池谷","愛知県安城市城ケ入町八ツ田町池谷",0.75,"machiaza_detail"
"愛知県愛西市平和町城之内","愛知県愛西市川北町平和町城之内",0.8,"machiaza_detail"
"愛知県田原市芦村町アラコ","愛知県田原市芦町村町アラコ",0.93,"machiaza"
"愛知県田原市芦村町小山田","愛知県田原市芦町村町小山田",0.93,"machiaza"
"愛知県田原市芦村町柿ノ木","愛知県田原市芦町村町柿ノ木",0.93,"machiaza"
"愛知県田原市芦村町比留輪","愛知県田原市芦町村町比留輪",0.93,"machiaza"
"愛知県稲沢市竹ノ腰町中瀬","愛知県稲沢市目比町竹ノ腰町中瀬",0.8,"machiaza_detail"
"愛知県稲沢市竹ノ腰町川端","愛知県稲沢市目比町竹ノ腰町川端",0.8,"machiaza_detail"
"愛知県稲沢市竹ノ腰町江端","愛知県稲沢市目比町竹ノ腰町江端",0.8,"machiaza_detail"
"宮城県登米市豊里町新中谷岐","宮城県登米市豊里町新町中谷岐",0.93,"machiaza"
"宮城県登米市豊里町新田鏡形","宮城県登米市豊里町新田町鏡形",0.93,"machiaza"
"宮城県登米市豊里町新白鳥前","宮城県登米市豊里町新町白鳥前",0.93,"machiaza"
"宮城県登米市豊里町竹の沢沼","宮城県登米市豊里町竹ノ沢沼",0.93,"machiaza"
"愛知県刈谷市上重原町東八鳥","愛知県刈谷市小垣江町上重原町東八鳥",0.77,"machiaza_detail"
"愛知県刈谷市上重原町西八鳥","愛知県刈谷市小垣江町上重原町西八鳥",0.77,"machiaza_detail"
"愛知県刈谷市二本木町時ケ堀","愛知県刈谷市野田町二本木町時ケ堀",0.82,"machiaza_detail"
"愛知県刈谷市二本木町鉄砲湫","愛知県刈谷市野田町二本木町鉄砲湫",0.82,"machiaza_detail"
"愛知県稲沢市竹ノ腰町中瀬","愛知県稲沢市目比町竹ノ腰町中瀬",0.8,"machiaza_detail"
"愛知県稲沢市竹ノ腰町川端","愛知県稲沢市目比町竹ノ腰町川端",0.8,"machiaza_detail"
"愛知県稲沢市竹ノ腰町江端","愛知県稲沢市目比町竹ノ腰町江端",0.8,"machiaza_detail"
"愛知県田原市芦村町上田","愛知県田原市芦町村町上田",0.92,"machiaza"
"愛知県田原市芦村町前畑","愛知県田原市芦町村町前畑",0.92,"machiaza"
"愛知県田原市芦村町南台","愛知県田原市芦町村町南台",0.92,"machiaza"
"愛知県田原市芦村町向田","愛知県田原市芦町村町向田",0.92,"machiaza"
"愛知県田原市芦村町大田","愛知県田原市芦町村町大田",0.92,"machiaza"
"愛知県田原市芦村町山崎","愛知県田原市芦町村町山崎",0.92,"machiaza"
"愛知県田原市芦村町平岩","愛知県田原市芦町村町平岩",0.92,"machiaza"
"愛知県田原市芦村町平成","愛知県田原市芦町村町平成",0.92,"machiaza"
"愛知県田原市芦村町柳原","愛知県田原市芦町村町柳原",0.92,"machiaza"
"愛知県田原市芦村町清水","愛知県田原市芦町村町清水",0.92,"machiaza"
"愛知県田原市芦村町田嶋","愛知県田原市芦町村町田嶋",0.92,"machiaza"
"愛知県田原市芦村町芦西","愛知県田原市芦町村町芦西",0.92,"machiaza"
"愛知県田原市芦村町西浦","愛知県田原市芦町村町西浦",0.92,"machiaza"
"愛知県田原市芦村町道上","愛知県田原市芦町村町道上",0.92,"machiaza"
"愛知県田原市芦村町道下","愛知県田原市芦町村町道下",0.92,"machiaza"
"愛知県田原市芦村町郷津","愛知県田原市芦町村町郷津",0.92,"machiaza"

git log
% git log -1
commit 039262be6052206752f0466e803f739ddf47f420 (HEAD -> develop, origin/develop)
Author: Masashi Katsumata<EMAIL_ADDRESS>Date:   Sun Nov 10 23:25:58 2024 -0800

    wip: fixing memory leak

データ確認
% cat ./mt_town_all.csv |grep "愛知県"|grep "八鳥"
232106,0093103,3,愛知県,アイチケン,Aichi,,,,刈谷市,カリヤシ,Kariya-shi,,,,上重原町,カミシゲハラチヨウ,Kamishigeharacho,,,,西八鳥,ニシハトリ,,,0,0,0,0,0,0,0,1,1947-04-17,,0,,
232106,0093104,3,愛知県,アイチケン,Aichi,,,,刈谷市,カリヤシ,Kariya-shi,,,,上重原町,カミシゲハラチヨウ,Kamishigeharacho,,,,東八鳥,ヒガシハトリ,,,0,0,0,0,0,0,0,1,1947-04-17,,0,,
232254,0006114,3,愛知県,アイチケン,Aichi,,,,知立市,チリュウシ,Chiryu-shi,,,,上重原町,カミシゲハラチョウ,Kamishigeharacho,,,,西八鳥,ニシハツトリ,,,0,0,0,0,0,0,0,1,1947-04-17,,0,,
232254,0006125,3,愛知県,アイチケン,Aichi,,,,知立市,チリュウシ,Chiryu-shi,,,,上重原町,カミシゲハラチョウ,Kamishigeharacho,,,,東八鳥,ヒガシハツトリ,,,0,0,0,0,0,0,0,1,1947-04-17,,0,,

Example Data
input,output,others,score,match_level
"京都府京都市北区西賀茂北鎮守庵町100-2","京都府京都市北区西賀茂中島町北鎮守庵町100-2","北鎮守庵町100-2",0.88,machiaza_detail
"京都府京都市北区西賀茂鎮守庵町105","京都府京都市北区西賀茂中島町鎮守庵町105","鎮守庵町105",0.86,machiaza_detail
"京都府京都市左京区松ケ崎六ノ坪町2-10","京都府京都市左京区松ケ崎樋ノ上町六ノ坪町2-10","六ノ坪町2-10",0.84,machiaza_detail
"京都府京都市右京区京北上弓削町皆味曾谷3-2","京都府京都市右京区京北上弓削町筒江皆味曾谷3-2","皆味曾谷3-2",0.92,machiaza_detail
"京都府京都市右京区梅ケ畑槙尾町1","京都府京都市右京区梅ケ畑燧岩谷槙尾町1","槙尾町1",0.85,machiaza_detail

version
% abrg --version
2.1.2

https://github.com/digital-go-jp/abr-geocoder/tree/a40ae16253904d85255e661836c4254ccf58cb77 では修正済みです
input,output,score,match_level
"愛媛県南宇和郡愛南町中塚","愛媛県南宇和郡愛南町中塚",1,"machiaza_detail"
"愛知県刈谷市上重原町丸山","愛知県刈谷市上重原町丸山",1,"machiaza_detail"
"愛知県刈谷市上重原町恩田","愛知県刈谷市上重原町恩田",1,"machiaza_detail"
"愛知県安城市八ツ田町池谷","愛知県安城市八ツ田町池谷",1,"machiaza_detail"
"愛知県愛西市平和町城之内","愛知県愛西市平和町城之内",1,"machiaza_detail"
"愛知県田原市芦村町アラコ","愛知県田原市芦村町アラコ",1,"machiaza_detail"
"愛知県田原市芦村町小山田","愛知県田原市芦村町小山田",1,"machiaza_detail"
"愛知県田原市芦村町柿ノ木","愛知県田原市芦村町柿ノ木",1,"machiaza_detail"
"愛知県田原市芦村町比留輪","愛知県田原市芦村町比留輪",1,"machiaza_detail"
"愛知県稲沢市竹ノ腰町中瀬","愛知県稲沢市竹ノ腰町中瀬",1,"machiaza_detail"
"愛知県稲沢市竹ノ腰町川端","愛知県稲沢市竹ノ腰町川端",1,"machiaza_detail"
"愛知県稲沢市竹ノ腰町江端","愛知県稲沢市竹ノ腰町江端",1,"machiaza_detail"
"宮城県登米市豊里町新中谷岐","宮城県登米市豊里町新中谷岐",1,"machiaza_detail"
"宮城県登米市豊里町新田鏡形","宮城県登米市豊里町新田鏡形",1,"machiaza_detail"
"宮城県登米市豊里町新白鳥前","宮城県登米市豊里町新白鳥前",1,"machiaza_detail"
"宮城県登米市豊里町竹の沢沼","宮城県登米市豊里町竹ノ沢沼",0.93,"machiaza"
"愛知県刈谷市上重原町東八鳥","愛知県刈谷市上重原町東八鳥",1,"machiaza_detail"
"愛知県刈谷市上重原町西八鳥","愛知県刈谷市上重原町西八鳥",1,"machiaza_detail"
"愛知県刈谷市二本木町時ケ堀","愛知県刈谷市二本木町時ケ堀",1,"machiaza_detail"
"愛知県刈谷市二本木町鉄砲湫","愛知県刈谷市二本木町鉄砲湫",1,"machiaza_detail"
"愛知県稲沢市竹ノ腰町中瀬","愛知県稲沢市竹ノ腰町中瀬",1,"machiaza_detail"
"愛知県稲沢市竹ノ腰町川端","愛知県稲沢市竹ノ腰町川端",1,"machiaza_detail"
"愛知県稲沢市竹ノ腰町江端","愛知県稲沢市竹ノ腰町江端",1,"machiaza_detail"
"愛知県田原市芦村町上田","愛知県田原市芦村町上田",1,"machiaza_detail"
"愛知県田原市芦村町前畑","愛知県田原市芦村町前畑",1,"machiaza_detail"
"愛知県田原市芦村町南台","愛知県田原市芦村町南台",1,"machiaza_detail"
"愛知県田原市芦村町向田","愛知県田原市芦村町向田",1,"machiaza_detail"
"愛知県田原市芦村町大田","愛知県田原市芦村町大田",1,"machiaza_detail"
"愛知県田原市芦村町山崎","愛知県田原市芦村町山崎",1,"machiaza_detail"
"愛知県田原市芦村町平岩","愛知県田原市芦村町平岩",1,"machiaza_detail"
"愛知県田原市芦村町平成","愛知県田原市芦村町平成",1,"machiaza_detail"
"愛知県田原市芦村町柳原","愛知県田原市芦村町柳原",1,"machiaza_detail"
"愛知県田原市芦村町清水","愛知県田原市芦村町清水",1,"machiaza_detail"
"愛知県田原市芦村町田嶋","愛知県田原市芦村町田嶋",1,"machiaza_detail"
"愛知県田原市芦村町芦西","愛知県田原市芦村町芦西",1,"machiaza_detail"
"愛知県田原市芦村町西浦","愛知県田原市芦村町西浦",1,"machiaza_detail"
"愛知県田原市芦村町道上","愛知県田原市芦村町道上",1,"machiaza_detail"
"愛知県田原市芦村町道下","愛知県田原市芦村町道下",1,"machiaza_detail"
"愛知県田原市芦村町郷津","愛知県田原市芦村町郷津",1,"machiaza_detail"
"京都府京都市北区西賀茂北鎮守庵町100-2","京都府京都市北区西賀茂北鎮守庵町100-2",1,"machiaza"
"京都府京都市北区西賀茂鎮守庵町105","京都府京都市北区西賀茂鎮守庵町105",1,"machiaza"
"京都府京都市左京区松ケ崎六ノ坪町2-10","京都府京都市左京区松ケ崎六ノ坪町2-10",1,"parcel"
"京都府京都市右京区京北上弓削町皆味曾谷3-2","京都府京都市右京区京北上弓削町皆味曾谷3-2",1,"machiaza"
"京都府京都市右京区梅ケ畑槙尾町1","京都府京都市右京区梅ケ畑槙尾町1",1,"machiaza"
"京都府京都市右京区太秦一ノ井町10-12","京都府京都市右京区太秦一ノ井町10-12",1,"parcel"


RON -> JSON converter

It should be possible to write an automatic RON->JSON converter for cases when you have a .ron file and an app that only reads .json files.
Initial discussion on gitter: https://gitter.im/ron-rs/ron?at=598b80f551915d962af79f5d

Once RON is self-describing (#48), this is a matter of using serde-transcode.
extern crate ron;
extern crate serde_json;
extern crate serde_transcode;

fn main() {
    let data = r#"
        Scene( // class name is optional
            materials: { // this is a map
                "metal": (
                    reflectivity: 1.0,
                ),
                "plastic": (
                    reflectivity: 0.5,
                ),
            },
            entities: [ // this is an array
                (
                    name: "hero",
                    material: "metal",
                ),
                (
                    name: "monster",
                    material: "plastic",
                ),
            ],
        )
        "#;

    let mut de = ron::de::Deserializer::from_str(data);
    let mut ser = serde_json::Serializer::pretty(std::io::stdout());
    serde_transcode::transcode(&mut de, &mut ser).unwrap();
}


Secure the invoker with ssl.

Description
This PR adds SSL to the invoker to secure the ping-endpoint. In addition it tidies up some controller-ssl-settings.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[x] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[x] I updated the documentation where necessary.

Codecov Report

Merging #3968 into master will decrease coverage by 4.77%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #3968      +/-   ##
==========================================
- Coverage    85.2%   80.43%   -4.78%
==========================================
  Files         146      146
  Lines        7057     7063       +6
  Branches      413      434      +21
==========================================
- Hits         6013     5681     -332
- Misses       1044     1382     +338

Impacted Files
Coverage Δ

...a/src/main/scala/whisk/http/BasicHttpService.scala
90.16% <ø> (-3.28%)
:arrow_down:

.../main/scala/whisk/core/controller/Controller.scala
82.29% <100%> (+0.18%)
:arrow_up:

...rc/main/scala/whisk/core/controller/Triggers.scala
94.16% <100%> (+0.04%)
:arrow_up:

...er/src/main/scala/whisk/core/invoker/Invoker.scala
67.21% <71.42%> (-0.65%)
:arrow_down:

...mmon/scala/src/main/scala/whisk/common/Https.scala
77.77% <83.33%> (-16.96%)
:arrow_down:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.1%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-81.82%)
:arrow_down:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
0% <0%> (-58.83%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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

PG2#3514 🔵


update aws connection docs fro devs

update recent changes in connection procedure (using EBI SSO, etc.)

pending doc PR review

Documentation has been reviewed!


Change Icon on Calliope mini blocks

Change the icon from screenshot to the attached SVG.

![Icon_Blocks](https://github.com/calliope-edu/scratch-gui/assets/14014852/041e2aa5-28c3-4929-b0d2-d8fbef0a4367)

@haithembelhaj

The above one is an SVG, here is the PNG.

@joernalraun change was deployed. Please check it out!

Könnte nun das Icon oben links wechseln? @haithembelhaj

@joernalraun I changed the header menu icon! Please review


Lambda functions as Docker Container Images

Is this related to a new or existing Amplify category?
function
Is this related to another service?
ECS, Docker
Describe the feature you'd like to request
As an engineer,
I want to create a Lambda function that is deployed in the form of a Docker container rather than in a dedicated runtime,
so that I have more flexibility, control, and fewer restrictions on the type of code that my function executes
More information on Lambda containers: https://docs.aws.amazon.com/lambda/latest/dg/images-create.html
Fewer restrictions

File size limitations. 250MB seems like a pretty generous limit, however, when doing image processing you typically need to use dependencies that easily exceed this limit. Even basic NodeJS projects creep up to 100MB very quickly because of the various modules that one installs.

Lambdas using containers can be up to 10GB in size!

Run time limitations. Some libraries need to be installed inside a specific runtime. Because I use MacOS these get installed with the wrong binaries causing my Lambda to fail when it starts up.

Lambdas using containers get around this problem since all packages will be installed using the same runtime that will execute the application.
Describe the solution you'd like
When running amplify add function I'd like another option (i.e. provider) that lets me choose "Container Image". This will then create the right CF template with a basic Dockerfile and /src folder that I can then use to do what I need to do.
When running amplify push it will build the image, upload it to ECS with the correct tags and deploy the CF template using the new image tag.
When running amplify remove it will delete all assets locally, along with the ECS images and CF template.
Describe alternatives you've considered
I'm investigating what it would take to build a custom function provider plugin based on how the python/nodeJS ones work. I can do this but will need some guidance/pointers from the Amplify team.
I've also looked at building something custom outside my amplify app, with a custom build pipeline, etc. This is NOT ideal since it's extra work - given I'm a 1 man team.
Additional context
No response
Is this something that you'd be interested in working on?

[X] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

Thanks so much for raising it. I also hope to have this feature in amplify!

Thanks! This is a very important feature for us.

Thanks a lot for this. Would be great to have it ! It is indeed a very important feature and doesn't seem there's another way around this in amplify right now.

I tried to customize lambda cloudformation file to image type but then after doing amplify push, the cloudformation file is reset to code .zip file type.

Hey folks, any news on that? Are there any strategies how to do it right now?

@harakiro btw. you can include amplify into CDK
https://www.youtube.com/watch?v=rjiiNpJzOYk

That what I'm searching, I'm facing similar problem wrong binary is getting installed in lambda. Other people are also facing this issue.
https://github.com/chunyenHuang/hummusRecipe/issues/196

I tried to customize lambda cloudformation file to image type but then after doing amplify push, the cloudformation file is reset to code .zip file type.

@hoan-pham-duy Is it possible to try using the CloudFormation code using tools other than the Amplify Command Line Interface?

I tried to customize lambda cloudformation file to image type but then after doing amplify push, the cloudformation file is reset to code .zip file type.

@hoan-pham-duy Is it possible to try using the CloudFormation code using tools other than the Amplify Command Line Interface?

Currently, I need to set up some Lambda Container Images in "out_of_amplify" folder and then try to link with "Amplify Resource".


Add printf! and printfln! macros that uses the DebugPrintf extension

See https://github.com/EmbarkStudios/rust-gpu/issues/151.
This will probably require some work before being merged. These macros and different from the core rust print and println macros as the type of variables needs to be specified in the format string. There's probably a way to change this as mentioned here: https://github.com/EmbarkStudios/rust-gpu/issues/151#issuecomment-717584254 but for now I'm doing the simpler thing.
I have a branch here: https://github.com/EmbarkStudios/rust-gpu/compare/main...expenses:printf-working that modifies the sky shader and ash runner for testing. It requires 'Debug Printf' to be checked in the Vulkan Configuration app.
One important caveat is that using DebugPrintf seems to SEGFAULT (on Windows I get a (exit code: 0xc0000005, STATUS_ACCESS_VIOLATION)) the Vulkan program if you build a single module with multiple entry points. This is truely bizarre, no clue what's going on here. The only difference between working code and segfaulting code is this: https://github.com/expenses/rust-gpu/compare/expenses:printf-not-working...printf-working.
At first I thought this was a driver issue but I'm getting this on both an integrated intel gpu and an external amd gpu.

Fun fact: you can print emojis using these macros

we could map each format argument to the rust type - %f to f32 for example - and wrap each one in assert_is_type::<T> before using them. It's a bit gross but it would work.

I did a test of this and it actually works quite nicely:
error[E0308]: mismatched types
  --> $DIR/debug_printf.rs:41:31
   |
41 |         debug_printfln!("%f", vec);
   |                               ^^^ expected `f32`, found struct `Vec2`

Just tested this - it just prints r without a %.

what in the absolute heckity goodness, lmao, that's wild
anyway, yeah, probably fine to merge as-is and work on full validation later if you'd prefer that PR model!
as for the path test failure, I thiiink there might be a way to work around that with something like this, although that might have to be tweaked slightly


Some newlines appear to be broken

As an example, the one you see when PowerShell starts up:
bad

good

Looks like only WT...

Is this a recent regression? I would have thought it might be #3271 but that hasn't merged yet.

Very. The current selfhost build from 12/31 does not exhibit this issue. Maybe related to the audit mode refactoring?

I believe the problem is in the TerminalDispatch::CursorPosition method here:
https://github.com/microsoft/terminal/blob/d711d731d7d00259b2e0fe5847928c9f1c1121ed/src/cascadia/TerminalCore/TerminalDispatch.cpp#L40-L44
If the SizeTToShort calls succeed, then the FAILED tests are going to be false, and the RETURN_BOOL_IF_FALSE will force the method to abort.

Oof. Thanks.


1.45: mint login dies quickly with ChromeDriver/Chrome 103 (MacOS and Linux)

Running version 1.45
Parsing Amazon Orders |████████████████████████████████| x/x
Parsing Amazon Items |████████████████████████████████| y/y
Logging in to mint.com\

You may be asked for an auth code at the command line! Be sure to press ENTER after typing the 6 digit code.
Navigating to Mint homepage.
Traceback (most recent call last):
  File "/Users/cwm/Library/Python/3.8/bin/mint-amazon-tagger-cli", line 8, in <module>
    sys.exit(main())
  File "/Users/cwm/Library/Python/3.8/lib/python/site-packages/mintamazontagger/cli.py", line 114, in main
    results = tagger.create_updates(
  File "/Users/cwm/Library/Python/3.8/lib/python/site-packages/mintamazontagger/tagger.py", line 126, in create_updates
    if not mint_client.login():
  File "/Users/cwm/Library/Python/3.8/lib/python/site-packages/mintamazontagger/mintclient.py", line 60, in login
    self.logged_in = _nav_to_mint_and_login(
  File "/Users/cwm/Library/Python/3.8/lib/python/site-packages/mintamazontagger/mintclient.py", line 199, in _nav_to_mint_and_login
    webdriver.get(MINT_HOME)
  File "/Users/cwm/Library/Python/3.8/lib/python/site-packages/selenium/webdriver/remote/webdriver.py", line 442, in get
    self.execute(Command.GET, {'url': url})
  File "/Users/cwm/Library/Python/3.8/lib/python/site-packages/selenium/webdriver/remote/webdriver.py", line 430, in execute
    self.error_handler.check_response(response)
  File "/Users/cwm/Library/Python/3.8/lib/python/site-packages/selenium/webdriver/remote/errorhandler.py", line 247, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: unknown error: cannot determine loading status
from unknown error: unexpected command response
  (Session info: chrome=103.0.5060.53)

I've tried variations of --mint_user_will_login, --mint_mfa_preferred_method sms, and --mint_login_timeout 999. chrome closes before I can even clear cookies!

Chrome and ChromeDriver are both 103.0.5060.53
Following https://stackoverflow.com/questions/59489803/selenium-common-exceptions-webdriverexception-message-unknown-error-failed-to I may downgrade both to 102 next if I have time.

Strangely, Amazon had no issues...

Got this working by setting "LATEST_RELEASE_102" in https://github.com/jprouty/mint-amazon-tagger/blob/53c5715c713108a2a5c067dccbfff07942bd53ad/mintamazontagger/webdriver.py#L101

Thanks for the report. I am able to repro on Win10 as well. Ironically everything was working hunky dory for me yesterday and today I get the 103 update from chromium stable channel 👍 I'll look into the underlying issue a bit more, and consider your work around as well (thanks for that).

Okay, I think I have a better fix: leave the version logic alone (use 103) but add a workaround (introducing a 1 second sleep between getting the webdriver and starting the mint login flow. Please reopen if this doesn't work for you or you need more assistance. I'll cut release 1.46 to include this, since others will likely hit this too. Thanks for the quick report!

I'm guessing this is the underlying chromedriver issue, which sounds like it will be fixed in chromedriver 104 (aka, in 4 weeks time).

Thanks for the follow-up. I've come up with a better workaround after investigating the issue a bit deeper. I was able to get it to happen on nearly any website, when using a new chrome session. It seems to depend on the complexity of the site & if that site has been cached/visited before for the given chrome session. In any case, I've added a loop that catches the exception in question with a 1 sec sleep, allowing webdriver some time to get ready. v1.47 should work better for you. Thanks!


Terminal editors are detaching when the window is not focused

Repro:

Open a terminal in panel
Drag into editor
Click outside the editor

This is happening because editors are serialized when window blur happens, so we can't detach on serialize.


API key e-mail

I am not receiving my key in the email

Hi, Gabriel. I am sorry to hear that.
I will resend it to you (bielzeraaaw@gmail.com).

No problem, thank you.
Congratulations! your's API is very useful!

@yongghongg could you please check this email too (sae010@campus.mephi.ru), didn't receive an API key.
Thanks in advance!


Delete Meeting Minutes Template

Description
This GitHub template is a duplication of https://github.com/finos/devops-automation/blob/master/.github/ISSUE_TEMPLATE/Meeting.md and therefore not required.

I've merged this PR as no SIG risk


CheckLog reports fine, eac_logsigner reports forged

For the attached log, running CheckLog.exe returns "1. Log entry is fine!" but using eac_logsigner returns "Forged". Is this because eac_logsigner is more strict or is it a bug?
Solo.log

It seems that your script wrongly assumes that EAC log files created at or before v1.0 beta 1 can't use a log checksum.
https://github.com/puddly/eac_logsigner/blob/2156dc0f1d4917923b051b15d1e44d540092c27a/eac.py#L10
https://github.com/puddly/eac_logsigner/blob/2156dc0f1d4917923b051b15d1e44d540092c27a/eac.py#L150
This is incorrect according to the website, which states the following in the changelog for v1.0 beta 1:

+ Option for writing a checksum over the extraction log data to the log file

Yeah, it's a bug. Changing <= to < should work but I need to rework the version number parser anyways so I'll fix this in a bit.

The fix works for me with your log file.

I could be wrong but don't you want to change this to a < too?
https://github.com/puddly/eac_logsigner/blob/8d7acc049d58c597d54e2c34c457164c125d23b1/eac.py#L133

Thanks, fixed. I'll add unit tests to this project...eventually.


Noticeable Input Latency -- can something be done ?

There is a noticeable latency with JavaFX across the board in general. (Dragging nodes, animation etc etc) when compared to other frameworks / render technologies. This latency is not noticeable for the average user but for someone working with a lot of text this will be a major annoyance.
I noticed it when I was playing with the rich text demo. Can any micro-optimizations be done to make the experience rival for example this very text box that I am typing in ? (Safari, Macbook, 10.10).
I know it is possible in swing because of the lengths Jetbrains have gone to make a first-class editing experience. The UI is much more responsive than Eclipse (which I know is not Swing). It almost rivals this text box

Hi Hassan, such a question is very general for me to answer. If you can point out a concrete operation that you perceive as slow and provide detailed instructions how to reproduce it, then I can have a look and try to answer your question.


macOS build_visit issue: cgns build requires f77 compiler

Describe the bug
clang -fno-common -fexceptions -O2  -fPIC  -I.  -o  lib/cgns_error.o -c cgns_error.c
clang -fno-common -fexceptions -O2  -fPIC  -I.  -I/Users/harrison37/Work/github/visit-dav/VISIT_RELEASE/visit/build-mb-3.2.0-darwin-10.14-x86_64-release/thirdparty_shared/third_party/hdf5/1.8.14/darwin-x86_64/include -o  lib/cgns_internals.o -c cgns_internals.c
clang -fno-common -fexceptions -O2  -fPIC  -I.  -o  lib/cgns_io.o -c cgns_io.c
clang -fno-common -fexceptions -O2  -fPIC  -I.  -I/Users/harrison37/Work/github/visit-dav/VISIT_RELEASE/visit/build-mb-3.2.0-darwin-10.14-x86_64-release/thirdparty_shared/third_party/hdf5/1.8.14/darwin-x86_64/include  -o  lib/cgnslib.o -c cgnslib.c
f77 -fno-common -fexceptions  -I.  -o  lib/cgns_f.o -c cgns_f.F90
make: f77: No such file or directory
make: *** [lib/cgns_f.o] Error 1

OS and version: macOS 10.14
VisIt Version: 3.2RC

Additional context
(encountered while preping for release builds)

I can turn off fortran support.
But other things are wrong.
The hdf5 linking flags don't seem to be propagating.
That means if it's linking on some platform, there is a strong chance it is linking against system hdf5 libs :-(

actually on Linux the symbols just remain unresolved.
It's fine there, but not allowed on macOS.
[cyrush@pascal83:lib]$ ld libcgns.so
/usr/bin/ld: warning: cannot find entry symbol _start; not setting start address
libcgns.so: undefined reference to `H5T_NATIVE_SCHAR_g'
libcgns.so: undefined reference to `H5Tget_native_type'
libcgns.so: undefined reference to `H5Pset_link_creation_order'
libcgns.so: undefined reference to `H5Sget_simple_extent_npoints'
libcgns.so: undefined reference to `H5T_IEEE_F64LE_g'
libcgns.so: undefined reference to `H5Tcopy'
libcgns.so: undefined reference to `H5Fopen'
libcgns.so: undefined reference to `H5Gunlink'
libcgns.so: undefined reference to `H5Awrite'
libcgns.so: undefined reference to `H5Gmove'
libcgns.so: undefined reference to `H5Ewalk2'
libcgns.so: undefined reference to `H5T_IEEE_F32LE_g'
libcgns.so: undefined reference to `H5Pset_deflate'
libcgns.so: undefined reference to `H5Tequal'
libcgns.so: undefined reference to `H5Tset_precision'
libcgns.so: undefined reference to `H5T_IEEE_F32BE_g'
libcgns.so: undefined reference to `H5Lcreate_external
``

skipping for now, I could (again) try cmake just for the macOS case, but I fear I might just revisit:
https://github.com/visit-dav/visit/issues/5461
spack uses cmake to build cgns, so that gives me some hope.

successful build w/o cgns. I am going to work on code signing to make sure we have a handle on that.

  -I/Users/miller86/visit/masonry/build-mb-3.2.0-darwin-10.14-x86_64-release/thirdparty_shared/third_party/hdf5/1.8.14/darwin-x86_64/include -o  lib/cgns_internals.o -c cgns_internals.c
clang -fno-common -fexceptions -O2  -fPIC  -I.  -o  lib/cgns_io.o -c cgns_io.c
clang -fno-common -fexceptions -O2  -fPIC  -I.  -I/Users/miller86/visit/masonry/build-mb-3.2.0-darwin-10.14-x86_64-release/thirdparty_shared/third_party/hdf5/1.8.14/darwin-x86_64/include  -o  lib/cgnslib.o -c cgnslib.c
f2c -fno-common -fexceptions  -I.  -o  lib/cgns_f.o -c cgns_f.F90
Error on line 0: Invalid flag '-no-common'
Error on line 0: Invalid flag '-exceptions'
Error on line 0: Invalid flag '-o'
make: *** [lib/cgns_f.o] Error 1

I am back to building 3.2 on OSX. I was too spent from ECPAM to do it friday last week and I was helping in the CI Startup Tutorial through mid-day friday anyways.

@cyrush ... I keep getting an issue where VisIt tries to pass the linker -llibcgns.so instead of -lcgns so the build fails during CGNS plugin. I've tried all sorts of manual edits to CMake content and then restarting masonry build (just so I can understand where I might need to go to fix) and it still winds up getting overwritten with the wrong string.
Separabely, but perhaps related. CGNS appears to be the only lib on my macOS system that is built as a .so instead of a .dylib. Is there some extra stuff we need to do in bv_cgns when using autotools for a lib to make it build a .dylib instead of a .so? Or, is that maybe another problem with CGNS autotools build?

What doe the cngs entries in the config-site look like?
There is some build_visit logic to create a shared lib specific to macOS:
https://github.com/visit-dav/visit/blob/develop/src/tools/dev/scripts/bv_support/bv_cgns.sh
I would expect that to create a dylib -- however it might be based on old cgns builds, maybe its not working to all?


Previewer (builtin/cat/bat) errors

Info

Operating System: macOS Ventura 13.2.1 (22D68)
Shell: zsh
Terminal: Apple Terminal / VimR
nvim --version: v0.8.3 (Terminal) / v0.8.2 (VimR)
fzf --version: 0.38.0
[ ] The issue is reproducible with mini.sh

fzf-lua configuration
require('fzf-lua').setup({
  winopts = {
    preview = {
      default = "bat",
    },
  },
  grep = {
    cmd = "rg",
    file_icons = false,
    color_icons = false,
    git_icons = false,
  },
})

Description
I tested FzfLua after seeing @ibhagwan 's Reddit post, it worked well, so I added it to my Neovim setup. For some reason, none of the three previewers work, neither in a GUI (VimR) nor in a terminal (Apple Terminal).
The builtin previewer works well in Terminal, but gives a blank screen in VimR.
cat gives the following error message, both in Terminal and VimR:
cat: illegal option -- -
usage: cat [-belnstuv] [file ...]

bat gives the following error message, both in Terminal and VimR:
[bat error]: invalid digit found in string

I've only tried live_grep, live_grep_native, and help_tags. live_grep works in Terminal but fails to populate the list in VimR, hence my grep settings. (I got the idea from live_grep_native source code.) Not sure if these are enough details to identify the cause. Happy to provide more details!

cat: illegal option -- -

I'll fix this one, nobody really uses cat so it didn't come up but the default cat options are for GNU cat which isn't the case on Mac, in the meantime can you try setting the below and let me know if it works?
require("fzf-lua").setup({
  previewers = {
    cat = { cmd   = "cat", args  = "" }
  }
})

As for the bat error can you please try :FzfLua files previewer=bat_async and lmk if this works?
If so, you can set it in the meantime in your setup until we figure out why the regular bat (native) doesn't work:
require('fzf-lua').setup({
  winopts = {
    preview = {
      default = "bat_async",
    },
  },
})

Regarding this:

I tested FzfLua after seeing @ibhagwan 's Reddit post, it worked well

Just to be clear, does it work well with mini.sh and only fails in your setup?

Can you please update to the latest commit https://github.com/ibhagwan/fzf-lua/commit/da1881ad7a2d643283ef4eb364b67062e6235532 and test default cat again, just tested it on a Mac and it worked.
Awaiting further responses from you so we can fix bat and the builtin previewer on VimR

Further investigation regarding VimR shows an issue with the internal neovim build packaged within VimR, fzf-lua is dependent on being able to run neovim --headless --cmd with lua commands (in order to have process separation), this seems to segfault with VimR, you can try the below yourself:

You can find out the exact path by running :lua print(vim.v.progpath) inside VimR

❯ '</path/to/>/VimR.app/Contents/Resources/NvimView_NvimView.bundle/Contents/Resources/NvimServer' -n --headless --cmd 'lua print("test")'
zsh: segmentation fault   -n --headless --cmd 'lua print("test")

Vs a standard build:
❯ nvim -n --headless --cmd 'lua print("test\n"); os.exit(0)'
test

I opened an issue upstream https://github.com/qvacua/vimr/issues/1001, hopefully it gets fixed fast.
I'm assuming the latest commit fixes the issue with the cat previewer, that leaves us with just one issue, bat previewer which isn't working:

Can you confirm if bat_async works for you (in terminal neovim run :FzfLua files previewer=bat_async)?
Can you run :FzfLua files previewer=bat debug=true, press F3 once so we can see the wrapped lines of the debug output and post the contents (or screenshot) of the preview window debug data?

Thanks for a super quick response & update! Yeah, cat now works fine in Terminal, and shows a blank screen in VimR.
And as to VimR, I get the same zsh: segmentation fault error message. Yeah, many VimR issues seem go unanswered, so I'm also not sure how fast this will get addressed, but thanks for opening the issue.
Okay, to the last remaining issue. bat_async do work! Yeah, I can get going with that first. Just curious, is it supposed to be faster or slower than the regular bat? And here's the debug message I got for bat.
[DEBUG] debug = true
[DEBUG] function ID = 3
[DEBUG] fzf_lua_server = /var/folders/ps/z37vy3b970q22dph4z8zf11c0000gn/T/nvim._____/EatHy0/nvim.48545.1
[DEBUG] argv[1] = /opt/homebrew/Cellar/neovim/0.8.3/bin/vim
[DEBUG] argv[2] = -n
[DEBUG] argv[3] = --headless
[DEBUG] argv[4] = --clean
[DEBUG] argv[5] = --cmd
[DEBUG] argv[6] = lua loadfile([[/Users/_____/. local/share/nvim/lazy/fzf-lua/lua/fzf-lua/shell_helper.lua]])().rpc_nvim_exec_lua({fzf_lua_server=[[/var/folders/ps/z37vy3b970q22dph4z8zf11c0000gn/T/nvim._____/EatHy0/nvim.48545.1]], fnc_id=3 , debug=true})
[DEBUG] argv[7] = --
[DEBUG] argv[8] = [DEBUG]: fd --color=never --type f --hidden --follow --exclude .git
[DEBUG] arg[1] = [DEBUG]: fd --color=never --type f --hidden --follow --exclude .git
[bat error]: '[DEBUG]': No such file or directory (os error 2)

And I'm not sure which file or directory the last error message is referring to, but fzf_lua_server (nvim.48545.1) do exist in my file system.

Okay, to the last remaining issue. bat_async do work! Yeah, I can get going with that first. Just curious, is it supposed to be faster or slower than the regular bat?

No big difference TBH, with bat, fzf spawns the bat which then calls neovim to get the filename (stripped of icons, line, col, text), with bat_async fzf calls neovim which returns the bat output to stdout via a call to libuv, both are super fast and no big difference, in fact it used to be the default until not long ago and I changed it, might go back to async being the default as it’s less error prone to shell escaping woes (which is probably what’s happening here).

And I'm not sure which file or directory the last error message is referring to

That’s because you’re trying to preview the debug mode line :)
Would you be able to screen shot again while the cursor is on a file line (not the debug line)?

I just realized that the preview works for the files command. But I still get the same error message with live_grep. Debug messages for live_grep:
[DEBUG] debua = true
[DEBUG] function ID = 3
[DEBUG] ff_lua_server = /var/folders/ps/237vy3b970q22dph4z8zf11c0000gn/T/nvim._____/ru37JI/nvim.51532.1
[DEBUG] argv[1] = /opt/homebrew/Cellar/neovim/0.8.3/bin/nvim
[DEBUG] argv[2] = -n
[DEBUG] argv[3] = --headless
[DEBUG] argv[4] = --clean
[DEBUG] argv[5] = --cmd
[DEBUG] argv[6] = lua loadfile([[/Users/_____/.local/share/nvim/lazy/fzf-lua/lua/fzf-lua/shell_helper.lua]])().rpc_nvim_exec_lua({fzf_lua_server=[[/var/folders/ps/z37vy3b970q22dph4z82f11c0000gn/T/nvim._____/ru37JI/nvim.51532.1]], fnc_id=3 , debug=true})
[DEBUG] argv[7] = --
[DEBUG] argv[8] = lazy-lock.json
[DEBUG] arg[1] = lazy-lock.json
[bat error]: invalid digit found in string

I just realized that the preview works for the files command. But I still get the same error message with live_grep. Debug messages for live_grep

As I suspect it’s probably just shell escaping woes due to the matching text which also gets sent to the bat command and confuses the shell, can you post the full text in the selected line from live grep where it fails (from the main fzf window, not the preview)?
I’m suspecting some special characters in there the shell doesn’t like, I will test myself using text from lazy-lock.json which I see was used here.

I'm not sure if the content of the file matters here. I just created a simple test.txt that has 5 characters hello, and even that gives [bat error]: invalid digit found in string. 😞

I'm not sure if the content of the file matters here. I just created a simple test.txt that has 5 characters hello, and even that gives [bat error]: invalid digit found in string. 😞

Maybe it’s your home folder or system path that fails, can you cd /etc and try there, if that works can you post the output of pwd within a failed folder?
I’m going to restore bat_async as default anyways once I implement an improvement for it later today.

https://github.com/ibhagwan/fzf-lua/commit/2d4e3affe86ab1b2612ce23ad80b9d50cf842dae - this commit should "solve" the bat issues, I wrote solve in quotes as this changes the default back to bat_async (with slight improvement regarding process spawn).
It's up to you if you want to continue troubleshooting this issue (which I suspect is OS path related), the only difference is that now in order to invoke the bug you'd need to set the previewer to bat_native specifically :FzfLua live_grep previewer=bat_native.
If you're happy with how it works and don't feel inclined to try and figure this out it's ok as I plan to one day phase out the bat_native previewer as it makse no sense to keep maintining both as long as the other is working perfectly.

I just tried in /etc and every single file gives the same error there, too. I'll just close out the issue since I can use the new bat a.k.a. bat_async and it sounds like bat_native will be retired. Thanks again for prompt help!

vimr issue can be dirty fixed by change the NvimServer binary to nvim as headleass subprocess.
diff --git a/lua/fzf-lua/libuv.lua b/lua/fzf-lua/libuv.lua
index ba1ba2c..7c5e7db 100644
--- a/lua/fzf-lua/libuv.lua
+++ b/lua/fzf-lua/libuv.lua
@@ -465,7 +465,8 @@ end
 M.wrap_spawn_stdio = function(opts, fn_transform, fn_preprocess)
   assert(opts and type(opts) == "string")
   assert(not fn_transform or type(fn_transform) == "string")
-  local nvim_bin = vim.v.progpath
+  -- local nvim_bin = vim.v.progpath
+  local nvim_bin = "nvim"
   local call_args = opts
   for _, fn in ipairs({ fn_transform, fn_preprocess }) do
     if type(fn) == "string" then
diff --git a/lua/fzf-lua/shell.lua b/lua/fzf-lua/shell.lua
index dce9dc5..db26a2b 100644
--- a/lua/fzf-lua/shell.lua
+++ b/lua/fzf-lua/shell.lua
@@ -60,7 +60,8 @@ function M.raw_async_action(fn, fzf_field_expression, debug)

   -- this is for windows WSL and AppImage users, their nvim path isn't just
   -- 'nvim', it can be something else
-  local nvim_bin = vim.v.progpath
+  -- local nvim_bin = vim.v.progpath
+  local nvim_bin = "nvim"

   local call_args = ("fzf_lua_server=[[%s]], fnc_id=%d %s"):format(
     vim.g.fzf_lua_server, id, debug and ", debug=true" or "")

vimr issue can be dirty fixed by change the NvimServer binary to nvim as headleass subprocess.

Ty @SolaWing, I thought about it and opted against it, you’re not guaranteed the user has neovim binary installed and even if it is might be some unknown version with other bugs and a hell to troubleshoot, perhaps ok as a hack for someone who really wants it to work.

Thanks for this brilliant idea, @SolaWing ! I just overrode those two functions locally and it works very well. In fact, in my case, it was enough to override raw_async_action only to get the preview fixed, but it's probably safe to override both just in case.
@ibhagwan I agree this is a hack fraught with danger. Hope VimR fixes the issue soon. Thanks for this great plugin! I'm happy that I can use it in both of my environments now. :)

Thanks for this brilliant idea, @SolaWing ! I just overrode those two functions locally and it works very well. In fact, in my case, it was enough to override raw_async_action only to get the preview fixed, but it's probably safe to override both just in case.
@ibhagwan I agree this is a hack fraught with danger. Hope VimR fixes the issue soon. Thanks for this great plugin! I'm happy that I can use it in both of my environments now. :)

That's great @benyn and ty @SolaWing for bringing this to conclusion.
I really dislike users having to use hacks to make things work and this seems imporant to you so I just puhsed a much easier fix (https://github.com/ibhagwan/fzf-lua/commit/368b9980f7ddd9ee622cf84d0e7f96185a95bfc0) that you can apply that doesn't require any maintenance on your part.
Set the FZF_LUA_NVIM_BIN enviroment variable to a working terminal neovim binary and you're done!
You can also do it specifically just for neovim, you can add this to yor config:
vim.env.FZF_LUA_NVIM_BIN="nvim"

And test to see if it worked with:
:lua print(os.getenv("FZF_LUA_NVIM_BIN"))

That works very well. Thanks for going above and beyond, @ibhagwan !

This might be an off topic question, but for live_grep, is the matching line supposed to be highlighted in preview? Or jump to the matching line when selected? Neither happens, whether I use VimR or Terminal.

This might be an off topic question, but for live_grep, is the matching line supposed to be highlighted in preview? Or jump to the matching line when selected? Neither happens, whether I use VimR or Terminal.

Yes, it should work, I’m pretty sure this is related to the shell escaping stuff that also caused bat to fail on your system with ‘live_grep`, let’s open a new issue and work on it there?

Argh. So that issue is more pervasive. Okay, I'll open a new issue.


Simulator & witgen properly handle calls to contracts that don't exist

including top-level calls to contracts that don't exist!

Higher gas cost for CALL to account for bytecode stuff?


Tweaked the case and highlighting of console.log

After mentoring the Perth Node Girls Workshop on 7 July 2018 I noticed
the group ran into issues due to the case of console and the surrounding
brackets (). This PR changes the case to match JavaScript and adds
highlighting to hopefully make it clearer.

Thank you @antonjb


[Question] Are Load Balancer IPs Available Externally?

Summary
I feel like I have a bug but I am not sure at the moment. To explain: I am currently testing a single node Harvester machine with two NICs. One NIC is the management interface, the second NIC is my production/development NIC. NIC two has two VLANs attached (5 and 7). When I deploy a cluster to VLAN 7 all nodes are able to get an IP on VLAN 7 (<IP_ADDRESS>/26), then I can deploy a workload to the child cluster and get a loadblanced service with Traefik. I successfully obtain an IP from the IP pool, however I can never get the service to be accessible from out side the layer2 domain. Is this intended?
Steps to reproduce

Deploy single node Harvester cluster  (I am on v1.3.0)

Node must have 1 mgmt interface, and 1 workload interface
Workload interface should have a VLAN attached

From Rancher

Create a project named "Production" in the harvester cluster
Create a namespace in the "Production" project named "production"
Create a Cluster Network (I Named mine "cluster-net")
Add a Network Config to "cluster-net"

Select all nodes
Select the second NIC with VLAN
Mode: active-backup
Miimon: -1
MTU: 1500

Add a VM Network

Add to "production" namespace
Name: production
Type: L2VlanNetwork
VlanID: Whatever your vlan is (Mine is 7)
Cluster Network: cluster-net
Route: Your CIDR and Gateway (Mine is <IP_ADDRESS>/26 and <IP_ADDRESS>)

Add an IP Pool

Name: production
Range:

Subnet: <IP_ADDRESS>/26
Gateway: <IP_ADDRESS>
Start IP: <IP_ADDRESS>
End IP: <IP_ADDRESS>

Selector:

VM Network: production/vm
Scope:

Production All All

Deploy child cluster via Rancher into Harvester node/cluster using VLAN interface

Use a loadbalancer service to consume an IP
When you have consumed an IP confirm it can be curl'd.

In my case it can not be curl'd directly as I am deploying Traefik so I set an entry in the VMs etc/hosts file corresponding to the VIP and hostname. This succeeds. Now curl the VIP from another network and it will fail. I can confirm there should be no firewall rules preventing this as I am ssh'ing into the VMs for the initial testing. From my firewall I can see my desktop is being allowed in to the production vlan.
Notes
I did do a packet capture from the firewall interface and did find something of note:

In the snippet above from the packet capture my desktop (<IP_ADDRESS>) when reaching out to the loadbalanced IP (<IP_ADDRESS>) an ICMP response follows it with a "Destination unreachable". This is odd as I can SSH into those VMs (which are in that <IP_ADDRESS> network) and ping my desktop. So again no firewall funny business.
Any guidance would be much appreciated.

I also believe I may be using VMs with too high of an MTU, much like the MTU issue described in the docs I thought I would go through all the VMs and set their MTU values to 1400 just in case.
No change

I will close this issue. I redeployed my harvester node and all appears to work fine.
One thing to note in the future: using externalTrafficPolicy: Local on a service appears to break the load balancer. I can't verify if this was the cause of the issue originally but using this setting does prevent access into the cluster.

Hi @Daemonslayer2048 For externalTrafficPolicy: Local to work, you'll need to enable leader election per service for kube-vip. Please refer to the document for more details. kube-vip, which runs in the guest cluster, is responsible for the nitty-gritty. I tried enabling the svc_leaderelection option, but there seems to be a problem updating the lock/lease for services, so I wouldn't recommend it.
For your original issue, the setup looks fine and should work. If you happen to run into the problem again, please let us know. Thank you.


Mobile version renders incorrectly

Directory Lister Version
3.12.0
PHP Version
7.4.30
PHP Extensions
No response
Web Server
LiteSpeed V8.0.1 Cloudlinux 1.3
Describe the Bug
When using a mobile phone with Safari or Chrome the site doesn't render as it should.
Trying to render it from my desktop using developer tools and a "mobile" view it renders OK.
Screenshot from my iPhone running Chrome:

The Directory Lister demo seems to be rendering just fine across all devices I've tested it on. This seems like an issue specific to your installation or phone. Without any additional information I'm not sure what the problem might be.

Sorry... my bad, some files was probably corrupted while FTP-transfering them one by one.
Uploaded the zip and then extracted it, now it works just fine.


[Bug]: The page turning component under the project list is blocked

ODC version
ODC423
OB version
independent
What happened?
The page turning component under the project list is blocked

What did you expect to happen?
chat privately
How can we reproduce it (as minimally and precisely as possible)?
chat privately
Anything else we need to know?
No response
Cloud
No response

Fixed In Commit 5c0acb0a509bb4e4d2486e600a4cc374e6db0b0e
------
咏和
Sun Dec 10 18:02:52 2023 +0800


AttributeError: 'FigureCanvasMac' object has no attribute 'copy_from_bbox'

When starting the flagging procedure, the following problem appears on Mac:
Traceback (most recent call last):
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/artist.py", line 62, in draw_wrapper
    draw(artist, renderer, *args, **kwargs)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/figure.py", line 1165, in draw
    self.canvas.draw_event(renderer)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/backend_bases.py", line 1809, in draw_event
    self.callbacks.process(s, event)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/cbook.py", line 563, in process
    proxy(*args, **kwargs)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/cbook.py", line 430, in __call__
    return mtd(*args, **kwargs)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/matplotlib/widgets.py", line 1176, in update_background
    self.background = self.canvas.copy_from_bbox(self.ax.bbox)
AttributeError: 'FigureCanvasMac' object has no attribute 'copy_from_bbox'

Tried to reproduce this error with version 0.3.95 on a Mac OS 10.12. Failed, however... Eventually this bug was already solved with one of the earlier fixes. Will close this issue with 1619752ff2d9b37f67a348f1b95eccc74a51848c .


ratelimit decorator is not working

Hi there, ratelimit decorator is not working.

Using sequence it is working fine, but I want to rate limit for a specific api. I have removed it from the sequence.
I am using a newly created Redis data source named 'redis'. I can see keys getting created in redis when implemented rate limit through the sequence.
As mentioned in #9, windowMs is not getting overriden.

controller.ts
  @ratelimit(true, {
      max: 4
  })
  @post('/add_user', {
    responses: {
      '200': {
        description: 'Add User',
        content: {
          'application/json': {
            schema: {
              type: 'object',
              properties: {
                is_success: {
                  type: 'boolean',
                },
              },
            },
          },
        },
      },
    },
  })
  async addUser(): Promise<{is_success: boolean}> {
      return {is_success: true}
  }

application.ts
    this.component(RateLimiterComponent);
    this.bind(RateLimitSecurityBindings.CONFIG).to({
      name: 'redis',
      max: 10,
      windowMs: 900000, // #9
      message: 'Too many requests, please try again later.'
    });

package.json
  "engines": {
      "node": ">=10.16"
  },
  "dependencies": {
      "@loopback/authentication": "^7.0.1",
      "@loopback/authentication-jwt": "^0.7.1",
      "@loopback/authorization": "0.7.1",
      "@loopback/boot": "^3.0.1",
      "@loopback/core": "^2.10.1",
      "@loopback/cron": "^0.4.1",
      "@loopback/repository": "^3.0.1",
      "@loopback/rest": "^7.0.1",
      "@loopback/rest-explorer": "^3.0.1",
      "@loopback/service-proxy": "^3.0.1",
      "loopback-connector-kv-redis": "3.0.0",
      "loopback4-ratelimiter": "^2.2.0",
   }
   "devDependencies": {
      "@loopback/build": "^6.2.4",
      "@types/node": "^10.17.35",
      "typescript": "~4.0.2"
    }

@samarpan-b can you please check? Thanks in advance!

Let me check this @ruturajmore

So basically you want rate limiter only for one api and disabled for all others ? @ruturajmore

Yes, the intention behind using a decorator is to enable rate-limiting for some required APIs. But it is not working as expected. Requests are not getting blocked even after max attempts. There no error in logs. @samarpan-b

Thats because you have removed it from sequence. Decorator is not meant to do the rate limiting. Decorator is meant to provide it the config. That's what  decorator pattern means actually. Can you enable it in sequence and try again ?

This component also exposes a method decorator for cases where you want tp specify different rate limiting options at API method level. For example, you want to keep hard rate limit for unauthorized API requests and want to keep it softer for other API requests. In this case, the global config will be overwritten by the method decoration. Refer below.
Check it out in docs. Also for disabling rate limiting from apis you can just do @ratelimit(false)

Yes, I checked it's working. One last thing, default windowMs is not getting overridden even through the decorator. Thanks for your support @samarpan-b  !!!

We'll check that tomorrow @ruturajmore when @akshatdubeysf is back. Will that be fine ?

Yes, it would be great. Thanks! @samarpan-b

Observed one more weird thing when findRoute() and parseParams() are called after rateLimitAction() decorator's configs are ignored and global configs are used.
sequence.ts
    await this.rateLimitAction(request, response);

    const route = this.findRoute(request);
    const args = await this.parseParams(request, route);

Can you please check? @samarpan-b @akshatdubeysf

@ruturajmore thats how loopback sequence works. The decorator config is only read when you parsed the params. If you try to do it before that, it will not work. So better to invoke it afterwards only. Parse params and find route is responsible for identifying the controller method for the API. Hence it is needed before.

Fixed through a PR hence closing this


#[UIC-797] Enabled Cypress tests on RVF: Dev setup with e2e test cases execution

Description:

Enabled cypress on RVF with sqllab,dashboard and explore.
100 % execution of test cases for all 3 modes - dashboard, explore and sqllab

We are no longer using cypress


chore(deps): update renovate/renovate docker tag to v37.50.1

This PR contains the following updates:

Package
Update
Change

renovate/renovate
minor
37.46.0 -> 37.50.1

Release Notes

renovatebot/renovate (renovate/renovate)
v37.50.1
Compare Source
Build System

deps: update dependency @renovatebot/osv-offline to v1.3.8 (#25627) (5b3c0a4)
deps: update dependency @renovatebot/ruby-semver to v3.0.17 (#25628) (b2b99b4)

v37.49.3
Compare Source
Build System

deps: update dependency semantic-release to v22.0.7 (#25622) (fb570b5)

v37.49.1
Compare Source
Bug Fixes

deps: update ghcr.io/containerbase/sidecar docker tag to v9.23.12 (#25619) (5a5244a)

Miscellaneous Chores

deps: lock file maintenance (#25598) (a560ef3)
deps: update dependency nock to v13.3.7 (#25614) (6ff4b65)
deps: update dependency nock to v13.3.8 (#25617) (4834f6d)

v37.49.0
Compare Source
Features

kustomize: support image digest pinning in newTag (#25596) (5674887)
presets: add extractVersion to regex managers (#25591) (65dc335)

v37.48.0
Compare Source
Features

predictible json/json5 handling (#24612) (4231b1a)

Miscellaneous Chores

deps: update actions/dependency-review-action action to v3.1.1 (#25600) (0ecc133)

v37.47.0
Compare Source
Features

presets: add hickory-dns and trust-dns monorepos (#25593) (3935e2e)

Documentation

update references to renovate/renovate to v37.46.0 (#25597) (4798368)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 5.2.18 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add CDNJS version badge in readme

This will add the badge to show its version on CDNJS and also link to its page on CDNJS!

Thanks!

My pleasure


proposal: custom-element as Rules engine

Expose custom-element with actionable components as Rules Engine to be run in the context of browser and on back-end.
MVP
Rule engines are used in applications to define some of the business logic declaratively in user-friendly manner.
In the web applications it is often used to guide the user across multiple microapplications. For example, drive the user UI for the 1st time login via legal compliance, basic user preferences, and the setup for chosen services. The Rules Engine would trigger each sub-flow based on the user account properties.
Goal
Expose the custom-element as the rules engine for the most popular scenarios. Demo the showcase as for browser-driven session as for back-end web service exposed to be consumed by web apps and other web services.
In technical terms, the goal is to provide the state change management framework which

loads the default/initial/last state either from inline serialized or remote data
loads the updated/runtime state
on each state node chage trigger the rule-driven action component with preserving the updated(by trigger) state as "current"

The Rules Engine comprise:

declarative conventions based on custom-element
web app and web service SDK
UI designer. DnD graph editor.
runtime state monitor UI in web and devtools for single "process"
commercial dashboard for ^^ for multiple "process" correlation and analytics.
commercial LM-driven designer and analytics board

references

list of OSS Rule Engines
Java Rule Engine API (JSR 94)

Chllenges

the rules engine meant to be driven in runtime-modified config, better via user-friendly UI( diagram, etc. ).
While the custom-element payload is available via loading in runtime via src attribute, the additional security precautions need to be in place as in any web service APIs. Should custom-element has to extend the fetch-element request properties?


Support for uuidv7?

Thank you for building this library. May support be added for uuidv7?

I'd also love to see this feature

I'd also like this! Also, @nthState-Chris funny finding you here? Hahaha

The main issue is getting a good quality timestamp. IMO the spec is a little bit ambiguous, because the section on UUIDv7 says the timestamp should be:

the number of milliseconds since midnight 1 Jan 1970 UTC, leap seconds excluded.

While other parts of the spec say that the timestamp only needs to be monotonic (BTW, Foundation's Date() initialiser does not make that guarantee), and that

This specification makes no requirement or guarantee about how close the clock value needs to be to the actual time

So it's unclear if we need to care about leap seconds or not. Previously I thought maybe we did, but now I'm leaning towards thinking that we don't (which is easier to implement, thankfully). One of the things on my to-do list is to send some feedback about that.
There are also a number of configuration options. The spec only requires the timestamp have millisecond precision, but there is another field for additional monotonic data, which may be the sub-millisecond portion of the timestamp, a counter, or some combination of the two. I'm debating how much of this to surface in the API; I always like exposing the knobs and dials so professionals can adjust the behaviour for their environment.
I have some other things going on which take priority, but I haven't forgotten about this and hope to get to it in a week or two (more likely two).

Yeah I can imagine its a little ambiguous. Appreciate you feeding back. Take your time, I know exactly how you feel haha!


Dead Link

https://github.com/dotnet/docs/blob/master/docs/framework/data/adonet/asynchronous-programming.md links to:
https://blogs.msdn.microsoft.com/adonet/2012/04/20/using-sqldatareaders-new-async-methods-in-net-4-5/ and to
https://blogs.msdn.microsoft.com/adonet/2012/07/15/using-sqldatareaders-new-async-methods-in-net-4-5-part-2-examples/ .
However, the blog at https://blogs.msdn.microsoft.com/adonet/ has been disabled.

Thanks @briangithex. I'm going to resolve this as a duplicate of #9807, where we're tracking the work for fixing these links after the migration/exclusion of some blogs.


Flaky tests in WSL2(Windows 11, Ubuntu20.04)

Describe the bug
I ran pnpm run test three times

https://cloud.nx.app/runs/ILyyfwTp5P
https://cloud.nx.app/runs/czXaFeTO7e
https://cloud.nx.app/runs/aTK8nfP6pU

And I encountered some failures, which is flaky
It doesn't seem to fail every time
Failed : nx run @tanstack/react-query-next-experimental:test:types (Failed 2 out of 3 times)
><EMAIL_ADDRESS>test:types /home/cho/tanstack/query/packages/react-query-next-experimental
> pnpm run "/^test:types:ts[0-9]{2}$/"

. test:types:ts50$ node ../../node_modules/typescript50/lib/tsc.js
. test:types:ts51$ node ../../node_modules/typescript51/lib/tsc.js
. test:types:ts52$ node ../../node_modules/typescript52/lib/tsc.js
. test:types:ts53$ tsc
. test:types:ts53: src/HydrationStreamProvider.tsx(3,26): error TS7016: Could not find a declaration file for module '@tanstack/react-query'. '/home/cho/tanstack/query/packages/react-query/build/modern/index.js' implicitly has an 'any' type.
. test:types:ts53:   Try `npm i --save-dev @types/tanstack__react-query` if it exists or add a new declaration (.d.ts) file containing `declare module '@tanstack/react-query';`
. test:types:ts53: src/ReactQueryStreamedHydration.tsx(9,8): error TS7016: Could not find a declaration file for module '@tanstack/react-query'. '/home/cho/tanstack/query/packages/react-query/build/modern/index.js' implicitly has an 'any' type.
. test:types:ts53:   Try `npm i --save-dev @types/tanstack__react-query` if it exists or add a new declaration (.d.ts) file containing `declare module '@tanstack/react-query';`
. test:types:ts53: src/ReactQueryStreamedHydration.tsx(18,8): error TS7016: Could not find a declaration file for module '@tanstack/react-query'. '/home/cho/tanstack/query/packages/react-query/build/modern/index.js' implicitly has an 'any' type.
. test:types:ts53:   Try `npm i --save-dev @types/tanstack__react-query` if it exists or add a new declaration (.d.ts) file containing `declare module '@tanstack/react-query';`
. test:types:ts53: src/ReactQueryStreamedHydration.tsx(47,44): error TS7006: Parameter 'event' implicitly has an 'any' type.
. test:types:ts53: src/ReactQueryStreamedHydration.tsx(71,32): error TS7006: Parameter 'query' implicitly has an 'any' type.
. test:types:ts53: Failed
 ELIFECYCLE  Command failed with exit code 2.
 ELIFECYCLE  Command failed with exit code 2.

Failed: nx run @tanstack/react-query-next-experimental:test:types (Failed 2 out of 3 times)
> @tanstack/query-example-nextjs-suspense-streaming@ test:types /home/cho/tanstack/query/examples/react/nextjs-suspense-streaming
> tsc

error TS6053: File '/home/cho/tanstack/query/examples/react/nextjs-suspense-streaming/.next/types/app/api/wait/route.ts' not found.
  The file is in the program because:
    Matched by include pattern '.next/types/**/*.ts' in '/home/cho/tanstack/query/examples/react/nextjs-suspense-streaming/tsconfig.json'
error TS6053: File '/home/cho/tanstack/query/examples/react/nextjs-suspense-streaming/.next/types/app/layout.ts' not found.
  The file is in the program because:
    Matched by include pattern '.next/types/**/*.ts' in '/home/cho/tanstack/query/examples/react/nextjs-suspense-streaming/tsconfig.json'
error TS6053: File '/home/cho/tanstack/query/examples/react/nextjs-suspense-streaming/.next/types/app/page.ts' not found.
  The file is in the program because:
    Matched by include pattern '.next/types/**/*.ts' in '/home/cho/tanstack/query/examples/react/nextjs-suspense-streaming/tsconfig.json'
 ELIFECYCLE  Command failed with exit code 2.

Failed: nx run @tanstack/react-query:test:lib (Failed 1 out of 3 times)
><EMAIL_ADDRESS>test:lib /home/cho/tanstack/query/packages/react-query
> vitest

Testing types with tsc and vue-tsc is an experimental feature.
Breaking changes might not follow SemVer, please pin Vitest's version when using it.

 RUN  v2.0.5 /home/cho/tanstack/query/packages/react-query
      Coverage enabled with istanbul

 ✓ |@tanstack/react-query| src/__tests__/queryOptions.test-d.tsx  (19 tests)
 ✓ |@tanstack/react-query| src/__tests__/infiniteQueryOptions.test-d.tsx  (13 tests)
 ✓ |@tanstack/react-query| src/__tests__/useInfiniteQuery.test-d.tsx  (8 tests)
 ✓ |@tanstack/react-query| src/__tests__/useQueries.test-d.tsx  (6 tests)
 ✓ |@tanstack/react-query| src/__tests__/useQuery.test-d.tsx  (9 tests)
 ✓ |@tanstack/react-query| src/__tests__/useSuspenseQueries.test-d.tsx  (6 tests)
 ✓ |@tanstack/react-query| src/__tests__/suspense.test-d.tsx  (9 tests)
 ✓ |@tanstack/react-query| src/__tests__/prefetch.test-d.tsx  (5 tests)
 ✓ |@tanstack/react-query| src/__tests__/useQuery.test.tsx  (155 tests) 18136ms
 ✓ |@tanstack/react-query| src/__tests__/useInfiniteQuery.test.tsx  (27 tests) 4133ms
 ✓ |@tanstack/react-query| src/__tests__/useQueries.test.tsx  (21 tests) 1238ms
stdout | src/__tests__/suspense.test.tsx > useSuspenseQueries > should show error boundary even with gcTime:0 (#7853)
queryFn

 ✓ |@tanstack/react-query| src/__tests__/suspense.test.tsx  (23 tests) 11511ms
 ✓ |@tanstack/react-query| src/__tests__/useMutation.test.tsx  (23 tests) 3410ms
 ✓ |@tanstack/react-query| src/__tests__/QueryResetErrorBoundary.test.tsx  (13 tests) 3308ms
 ✓ |@tanstack/react-query| src/__tests__/prefetch.test.tsx  (9 tests) 2950ms
 ✓ |@tanstack/react-query| src/__tests__/HydrationBoundary.test.tsx  (7 tests) 951ms
 ✓ |@tanstack/react-query| src/__tests__/ssr-hydration.test.tsx  (3 tests) 394ms
 ❯ |@tanstack/react-query| src/__tests__/useMutationState.test.tsx  (7 tests | 1 failed) 2306ms
   × useIsMutating > should return the number of fetching mutations
     → expected [ +0, 1, 2, +0 ] to deeply equal [ +0, 1, 2, 1, +0 ]

Ignored nodes: comments, script, style

 ✓ |@tanstack/react-query| src/__tests__/useIsFetching.test.tsx  (5 tests) 812ms
 ✓ |@tanstack/react-query| src/__tests__/useSuspenseQueries.test.tsx  (6 tests) 903ms
 ✓ |@tanstack/react-query| src/__tests__/fine-grained-persister.test.tsx  (3 tests) 754ms
 ✓ |@tanstack/react-query| src/__tests__/ssr.test.tsx  (5 tests) 177ms
 ✓ |@tanstack/react-query| src/__tests__/QueryClientProvider.test.tsx  (4 tests) 465ms

⎯⎯⎯⎯⎯⎯⎯ Failed Tests 1 ⎯⎯⎯⎯⎯⎯⎯

 FAIL  |@tanstack/react-query| src/__tests__/useMutationState.test.tsx > useIsMutating > should return the number of fetching mutations
AssertionError: expected [ +0, 1, 2, +0 ] to deeply equal [ +0, 1, 2, 1, +0 ]

Ignored nodes: comments, script, style

- Expected
+ Received

  Array [
    0,
    1,
    2,
-   1,
    0,
  ]

 ❯ src/__tests__/useMutationState.test.tsx:62:49
     60|
     61|     renderWithClient(queryClient, <Page />)
     62|     await waitFor(() => expect(isMutatingArray).toEqual([0, 1, 2, 1, 0…
       |                                                 ^
     63|   })
     64|
 ❯ runWithExpensiveErrorDiagnosticsDisabled ../../node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@testing-library/dom/dist/config.js:47:12
 ❯ checkCallback ../../node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@testing-library/dom/dist/wait-for.js:124:77
 ❯ Timeout.checkRealTimersCallback ../../node_modules/.pnpm/@testing-library+dom@10.4.0/node_modules/@testing-library/dom/dist/wait-for.js:118:16

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/1]⎯

 Test Files  1 failed | 22 passed (23)
      Tests  1 failed | 385 passed (386)
Type Errors  no errors
   Start at  20:45:21
   Duration  178.99s (transform 9.12s, setup 29.77s, collect 15.64s, tests 51.45s, environment 51.97s, prepare 11.67s, typecheck 23.04s)

 ELIFECYCLE  Command failed with exit code 1.

Your minimal, reproducible example
Sorry but this is test failure in local environment
Steps to reproduce
pnpm install > pnpm build:all > pnpm run test
Expected behavior
All of the tests passed
How often does this bug happen?
None
Screenshots or Videos
No response
Platform
Windows 11, using WSL2 (Ubuntu 20.04)
pnpm version 9.12.1
node version 20.10.0
Tanstack Query adapter
None
TanStack Query version
v5.59.8
TypeScript version
v5.3.3
Additional context
On macOS all tests passed, but on WSL2 (Ubuntu20.04, Windows11) some tests failed. It seemed flaky, and I just wanted to point it out
Thanks

the useMutationState.test from the second run has been stabilized in a different PR.
type failures are no flakiness, it just means something hasn’t been built yet. Not sure what that can be as I’ve never seen it before.


Reachability not working correctly

I tried to use NetworkReachabilityManager as follow in the iOS example, but it doesn't work correctly:

When I just start the app, it shows:

status :reachable(Alamofire.NetworkReachabilityManager.ConnectionType.ethernetOrWiFi)
status :notReachable

And I turned the airplane mode on to disconnect the internet, it didn't respond to show me the internet is off.

When I turned the airplane mode, it shows

statuslala :reachable(Alamofire.NetworkReachabilityManager.ConnectionType.ethernetOrWiFi)
statuslala :notReachable
statuslala :reachable(Alamofire.NetworkReachabilityManager.ConnectionType.ethernetOrWiFi)
statuslala :notReachable

The result is not correct. Can you please figure it out what's the problem?

The testing profile is:
Xcode 8.2 beta
iPhone 6, iOS 10.1.1
The newest Alamofire commit: 928cdae0c54957cb53a6873ec64a109bc6a1be60
Under 3G internet
class MasterViewController: UITableViewController {

    @IBOutlet weak var titleImageView: UIImageView!

    let manager = NetworkReachabilityManager(host: "https://www.apple.com/")

    var detailViewController: DetailViewController? = nil
    var objects = NSMutableArray()

    override func awakeFromNib() {
        super.awakeFromNib()

        navigationItem.titleView = titleImageView

        weak var wSelf = self
        manager?.listener = {status in
            print("statuslala :\(status)")
            let alert = UIAlertController(title: "Alert", message: "status:\(status)", preferredStyle: .alert)
            alert.addAction(UIAlertAction(title: "Click", style: .default, handler: nil))
            wSelf?.present(alert, animated: true, completion: nil)
        }

        manager?.startListening()
    }

You need to use www.apple.com as the host. It gets all screwed up when you throw the scheme in there.
Cheers. 🍻

Awesome! Problem sovled. I hope this colsed issue helpful to others...


Updated bot  #116

Pull Request Checklist
Thank you for contributing to our project! Please ensure the following before submitting your pull request:
Description of Changes
A new feature of  bot is added Fixes #116
Type of Change

[x] Bug fix
[x] New feature
[ ] Documentation update
[ ] Refactoring
[ ] Other (please describe):

Related Issue

Fixes #116

Testing

Checklist

[x] I have tested my changes and ensured they work as expected.
[x] I have added necessary documentation (if applicable).
[x] I have reviewed my code for style and linting issues.
[x] I have ensured that any dependent changes are merged before this PR.

Additional Notes

@Vimall03 Please delete or remove the older chatbot so that the new could be visible and made into effect.


Performance issues

I have 1.5 million scheduled messages, and it's performing quite bad. I know 1.5 million is a lot, but it's constantly making a "starts with" query in Mongo to get scheduled messages, which is not very optimized.
Is it possible to make a different query that uses a direct match instead?

@ffMathy,
Thank you for your inquiry.
The challenge is that in order to make sure we dont create duplicates, we need a unique key for the meta data used by Hangfire.Core. we cannot control the keys used by Hangfire.Core and we dont necessarily have transactions available
The compromise is that we are creating a compound key to meet the requirements.
We do have a index on the fields and the regex should take advantage of those.
You can override the default behavior by overriding the "CreateSetFilter" by:
public class MyWriteOnlyTransaction : MongoWriteOnlyTransaction
{
    public MyWriteOnlyTransaction(HangfireDbContext dbContext) : base(dbContext)
    {
    }

    public override BsonDocument CreateSetFilter(string key)
    {
        return base.CreateSetFilter(key);
    }
}

public class MyFactory : MongoFactory
{
    public override MongoWriteOnlyTransaction CreateMongoWriteOnlyTransaction(HangfireDbContext dbContext)
    {
        return new MyWriteOnlyTransaction(dbContext);
    }
}

You need to use the factory in the MongoStorageOptions like so:
var options = new MongoStorageOptions
{
    Factory = new MyFactory()
};

let me know if it works out :)

Thank you for the elaboration. While the regex does indeed get hit by an index, it is far from as efficient as a direct match index.
Instead of having "recurring-job:Foo" as key (or whatever format we use here), can we string split it into two properties with values "recurring-job" and "foo" and match on those?

@ffMathy,
We could probably figure something out.
Could you please try to run the offending queries in mongodb compass eg. and check explain, it would be great if you could post the json result here. I suspect it might the sorting.
{Key:{"$regex": "recurring-jobs"}, _t: "SetDto"}

Please try with and without sorting:
{_id: 1}

Well it's not that it isn't hitting the index. It's just that a "starts with" regex index hit is not as efficient as one with two fields.
Thank you so far for your involvement 🙏

@ffMathy,
no worries.  You have good data, so its an interesting use-case. :)
When doing the "startswith" query I also use another indexed field: "_t", so I suspect the query is not that invasive.
When you schedule a job Hangfire is hitting the db a couple of time:

acquire a lock for the scheduler
insert the job
update job state and schedule info

besides all this there is a lot other traffic going on
so scheduling 1.5 million jobs generates a lot of traffic.
Would you be able to create recurrent jobs which poll for data instead? or schedule less jobs and have each job handle multiple items?

I'll report back with more data tomorrow (Monday).
Our Mongo cluster has quite heavy specs, and is hosted on Mongo Atlas.
It is an NVMe disk with 100.000+ IOPS. I can easily query in collections with 4 million items in, in less than 100 milliseconds. So something is causing this particular query to be slow.
We noticed that disk utilization is not the issue, but CPU utilization.
I believe that a direct index is less CPU intensive than a regex starts with index.
So in other words... The index is indeed helping with disk usage. It's the CPU that is a problem.
For other queries, there is no issue. Not even with larger amounts of data.

@ffMathy,
Atlas is flagging the query in the profiler? If we could narrow down which specific query is slowing the system. Its a bit intrusive to change the schema, but it should be possible

Inside the "Realtime" tab, I caught one of the queries now.

Note that it says "Index utilized: No".
Here's the full JSON output for that query:

But even if it was utilizing an index, I doubt the regex index would be very efficient.

@ffMathy,
Thanks for the info,
Could you please verify there is an index on both "Key" and "_t" in the "jobGraph" collection? I would expect both indexes to be utilized
Thanks

Both indexes exist, and they are the right sort order (ascending). I think it's the regex query that makes it not perform well, and not hit the index properly.
Also note that it's only in Mongo Atlas' realtime view that it says it's not utilizing the index. In Compass, it seems to work fine when I run an explain on it:

However, do note that it only seems to be using the Score index. Shouldn't it use _t too?
Also, have you considered using composite indices? I think they can really speed up things, but I am probably not as knowledgeable about Mongo as you are.
If you decide to go for a composite index, perhaps having _t before Score in the composite index, if _t is more specific. That has a performance difference too.
As a side-node, I noticed this morning that the amount of documents in the database had actually climbed to 3.8 million, despite only having around 1.5 million scheduled messages.
I investigated further, and found out that Hangfire Console seems to be adding individual items starting with console: into the job-graph as well - one per line of logs!
I don't know who I think is the "sinner" here, but perhaps even storing in different collections could help?
For instance, instead of storing a Key of  schedule:foo in collection jobGraph, perhaps store it in a jobGraph.schedule collection.
That way, Hangfire Console would get into its own collection, and scheduled messages etc would as well.
It would speed up such queries heavily, since the queries always know where to get their data from, right?

@ffMathy,
Could you try the query without Score? eg. only querying using _t and regex. you might want to put a limit on the result.
Its not possible to support atomicity if its not one collection. Hangfire was originally written with sql in mind. Im using the "BulkWrite" feature to ensure the needed transaction style api exposed by hangfire.
With these kinds of numbers and since you schedule them anyway, it might be better store the needed data and to poll using recurrent jobs.
I will see if I can help the queries by using the aggregation framework to force to match on _t first
Thanks!

@ffMathy,
Please also try to remove the sort
Thanks!

@gottscj ah yes. Atomicity. But can't you then use Mongo Transactions, which was introduced in Mongo 4?
Here's without the sort, and without the Score:

It looks like that's now using the Key index.

Also, generally, in my opinion, it should never take long to get the "next" item to schedule.
I'm not blaming your library here, but perhaps the overall way Hangfire was designed.
Taking the top job sorted ascending by schedule time, should not be an expensive operation, even with millions of records.
But I see how Hangfire doesn't exactly give you a lot of options here. Can we find a way to get around it somehow?
Quite curiously, just finding something based on score (and a large one even), takes "only" 3 seconds for 640,000 returned records:

@ffMathy,
Criticism is always welcome. without it I cant improve. :)
Thank you for the data. I will update you.

@ffMathy,
quick question?
There is no "limit" operator used on the slow query you found in ATLAS? eg. ALL documents are returned?

Yes, looks like it. But it does have a "less than" and "greater than" on the Score field. I assumed that's how Hangfire handles fetching just some data, or something similar.
It does indeed seem a bit funny.
By the way, thank you so much for all of this. If you sign up for GitHub Sponsors, I'll sponsor you.

Interesting. Atlas just came up with this:

And it's true - that particular index does not exist. I'll try creating it.

Note that it's also a composite index, like I suggested. While Mongo can merge individual indices, they are far from the same efficiency AFAIK.

Alright, that reduced cluster CPU usage from around 95% to 70% steadily.
However, the long queries remain. So there is an underlying architectural issue too.

@ffMathy,
I have also tried to track down the offending query, and I cannot figure out where it comes from. that particular query should come with a limit of 1000 (hardcoded in Hangfire.Core). Can you confirm the query requests ALL documents for the given query, is it an update? if you manually dropped the collection, the indexes will not be rebuild again as they are created in a migration step
I also tried to change to use a compound unique key, but difference was minimal as mongo already uses the Score index.
a note on transactions, they are only supported if you have a replicaset, im not aware of the usage of Hangfire.Mongo, but I suspect its also being used with single instances, but I dont know. that could be easily added.
Could you check the indexes on the jobGraph, it should look something like this (split in two):

They do indeed look like that. All those indices do indeed exist. Yet, creating the composite index that it suggested, worked just fine, and reduced CPU by around 15% for me, steadily.
I have never dropped the collection before, and I am 100% sure, because I am the only developer on this project.

Also, I looked closer into one of the offending queries that are still slow after the composite index. And indeed, you were right. The limit simply just isn't shown in the "Realtime" tab of queries.
Here is the full query:
{
  "type": "command",
  "ns": "reshopper_prod.Hangfire__sirius_.jobGraph",
  "command": {
    "find": "Hangfire__sirius_.jobGraph",
    "filter": {
      "_t": "SetDto",
      "Key": {
        "$regex": "^schedule",
        "$options": ""
      },
      "Score": {
        "$gte": 0,
        "$lte":<PHONE_NUMBER>
      }
    },
    "sort": {
      "Score": 1
    },
    "projection": {
      "Value": 1,
      "_id": 0
    },
    "limit": 1000,
    "$db": "reshopper_prod",
    "lsid": {
      "id": {
        "$uuid": "1ed1934e-a244-4c98-8716-9659bbcf4b36"
      }
    },
    "$clusterTime": {
      "clusterTime": {
        "$timestamp": {
          "t":<PHONE_NUMBER>,
          "i": 187
        }
      },
      "signature": {
        "hash": {
          "$binary": "v9Hdt4LZyrxm+UrXwPWNkwe/2+E=",
          "$type": "00"
        },
        "keyId": {
          "$numberLong": "6904722023574929409"
        }
      }
    }
  },
  "planSummary": "IXSCAN { Score: 1 }",
  "keysExamined": 636915,
  "docsExamined": 636915,
  "fromMultiPlanner": true,
  "replanned": true,
  "replanReason": "cached plan was less efficient than expected: expected trial execution to take 24 works but it took at least 240 works",
  "cursorExhausted": true,
  "numYields": 2558,
  "nreturned": 5,
  "queryHash": "D8B51B3F",
  "planCacheKey": "5186805B",
  "reslen": 475,
  "locks": {
    "ReplicationStateTransition": {
      "acquireCount": {
        "w": 2559
      }
    },
    "Global": {
      "acquireCount": {
        "r": 2559
      }
    },
    "Database": {
      "acquireCount": {
        "r": 2559
      }
    },
    "Collection": {
      "acquireCount": {
        "r": 2559
      }
    },
    "Mutex": {
      "acquireCount": {
        "r": 1
      }
    }
  },
  "storage": {
    "data": {
      "bytesRead": 23982887,
      "timeReadingMicros": 24430
    }
  },
  "protocol": "op_msg",
  "durationMillis": 10629
}

What I find interesting is that it only returned 5 elements, had a limit of 1000, but also read 23 megabytes before returning those documents. It really suggests to me that the internal data structure has issues.
I also find it interesting that it was "replanned" for some reason. Not sure why that happened, but that can't be good for performance.
I understand your perspective on enabling transactions. That was not very thought-through of me. It would be a shame to require multi-node clusters indeed.
Any other ideas?

@ffMathy,
Thanks, I see limit of 1000, that makes more sense. The SetDto's Scores are being updated by hangfire, but writes should not stop reads on mongo, so im not completely sure whats going on. The amount of documents thats needs to examined and sorted is quite high, and im not sure changing the schema would help that.
I think for now, your best option is to batch the required work into (alot) less workers and let the jobs run work in parallel.
Or if you can schedule the jobs at different times to split the workload.
Maybe schedule 1000 jobs which then starts 1500 Tasks in parallel.
I will need to study the schemas, and perhaps split them into separate collections. I like the idea of using transactions for systems that supports this.

Also you could set the schedule poll interval in "BackgroundServerOptions", its default value is 15 seconds. depending on your schedule you could increase this value

@ffMathy,
I just released 0.7.20. I added a field so the regex query is no longer used. I also added some optimizations when updating data.
Please let me know if it helps.
Thanks

I also added the possibility to opt out of using a tailed capped collection to notify jobs added to queue, you should probably opt out as this is primarily used for fire-and-forget and it would be less strain on your db. just a suggestion

@gottscj thank you so much for your efforts here! I'll check it out Monday and report my findings. I'll also make good use of opting out of that feature. That should mean so much.
Cheers! 🍺 🙏

Sure thing.
I hope it works out. Else we need to look at schemas and collections

Holy...!
That worked. CPU decreased from 80% stable, to varrying between 20% and 40% (still spikes once in a while, but that's due to other activity). In addition, there are no more slow Hangfire queries being reported - at all. They went from 15 seconds down to less than 200 milliseconds.
Thank you so much for this!
Oh, and we've been using this for several years. Perhaps bump to 1.0? :smirk:

great news!
if I bump to 1.0 I have no excuse :) but yes, its time.
maybe after next release where I will support transactions and $watch features.

Is there any way that I can sponsor you?
I'm not on sponsors, i did sign up though.
thanks!

Alright. Let me know when you're signed up, and I'll become your first sponsor 🥂

By the way, here's the CPU utilization after the update:

Hmmm, side note. All of our scheduled messages were cleared as part of the update 😦
Maybe that was why performance improved. I'll let you know in a month or so, when we reach the same level of scheduled messages again.

I needed to create a migration. If your migration strategy is drop,then it sounds about right. 😬
Yes please let me know! 👍

Oh my god, the strategy was drop for some reason indeed. Not sure why we chose that. We must have had issues with migrate in the past.
I think it's because we do a partial rollout on all of our horizontally-scaled nodes.
I think the regular migrate strategy caused issues sometimes if something has been deployed to node A, but not node B yet?
But I re-enabled it now. I'll let you know if that causes problems in the future :+1:
My bad! Sorry about that.

@ffMathy,
Any chance you got some insights? no news is good news I hope?

@gottscj definitely. However, the load is still higher than we'd like. Not sure if that's due to this package though. I think that's just how Hangfire is built up.
We're also switching Hangfire over to a new PostgreSQL cluster soon either way, to offload the load from our main server.

@ffMathy,
Thanks for the feedback. Good luck!


Fix various bugs

Based on #26

57c15ed03273f236a4ab9167398560884bb514f0 fixes trampoline with non-void ret
9b7dd4cd9c5f2f12b200e0e8b18974a975e198e5 fixes duplicated function body; and memset returning the inner ret instead of call's resut.

superseded by https://github.com/numba/pixie/pull/28


feat: WASM integration

#6
TODO

[x] Basic assembly script
[ ] fix the assembly script lol
[x] GCounter?
[x] Frontend
[ ] Better imports and stucture pipeline from writing to complie and execute in web.
[ ] auto compile pipeline? ts => asmscript => wasm

Overview
This is a replica of the canvas in wasm. objects in the assembly dir get compiled to build and are exposed by the src code, specifically the wasmLoader to hopfuly be used by the front end in a seemless fashion.
issues
The objects arn't correctly coded out tbh. currently fixing

Changed up the crdt-wasm pkgs for it to work under assembly script. No testing is done but wdyt of this design pattern @d-roak ?
I'm also planning on adding some tests at the pkg level for all the crdts to let us quickly test them. Can't be bothered to build up another from end or manually test it every time.

the asc build will run on the ci, so it will be tested there. We also started having unit tests for the crdt's, they might need to be re-written, but no need to new tests (?)
regarding design pattern, we need some more discussion

closed in favor of #101


point to alpine tarfile; update to docker-gen 0.7.3

docker-gen should reference the alpine-linux-amd64 binary. Using the i386 binary fails on a strictly amd64 alpine linux host.

Thanks!


Datepicker version 1.6.1 stops jQuery 2.2.x

Working fine with jQuery 2.1.x version. When jQuery was upgraded to 2.2.x, when we call datepicker function, jQuery stops showing error
Error is: View is null
Error comes from jQuery, not from bootstrap-datepicker.js

Can you provide (as the issue template suggests) a jsfiddle example to show the issue? (I can't reproduce)

hi!
Here an example for this behavior on Firefox 47.0 and Chrome 51.0
https://jsfiddle.net/sx49tcjx/1/
please see console for following error:

SyntaxError: expected expression, got '.' bootstrap-datepicker.css:8:0
TypeError: $(...).datepicker is not a function


Setup a new way to compose containers

Changes Unknown when pulling f6bcac481855f0eeb71d13af7c758bb36ac155cd on pedrommone:cuducos-elm-to-nginx into ** on datasciencebr:cuducos-elm-to-nginx**.

Changes Unknown when pulling ca2052f09f194d6b5a83114b20fa4ad9c628f334 on pedrommone:cuducos-elm-to-nginx into ** on datasciencebr:cuducos-elm-to-nginx**.

Changes Unknown when pulling ca2052f09f194d6b5a83114b20fa4ad9c628f334 on pedrommone:cuducos-elm-to-nginx into ** on datasciencebr:cuducos-elm-to-nginx**.

@gomex and @petriuslima — Pedro's PR fixes some bug we had in the current Docker environment (compiled files from Elm container was missing for nginx container). However it changes a bit the way things work. For production the docker-compose.yml is the base, and you load the docker-develop.yml that adds certain configuration that should make it a proper development environment. Does that make sense, can we merge it?

@cuducos IMHO development opinion should be easy to use. In that case docker-compose.yml should be for dev.
About that Prod vs Dev env I already sent this PR #129

Ok… we have three scenarios here:

master was supposed to be focused on dev, but it was broken
#129 introduced Docker for production, but as it was based on master development was still broken
Now we have a third approach (this PR) that attempts to fix both

I do agree that development should be easy to use, and that production should be made via Docker too — this is our target. I'm just not experienced enough to fix all Docker issues and to ponder on these three scenarios.
Today I'll try #129 for dev and prod, and this PR #137 for dev and prod  on a fresh machine and gonna update you all.

Hi all,
I tested this PR 3 times for dev 3 times for prod, on a fresh VM. No good news — and I have no idea what we did yesterday that make it work.
Production
django.db.utils.OperationalError: could not connect to server: Connection refused
	Is the server running on host "db" (<IP_ADDRESS>) and accepting
	TCP/IP connections on port 5432?

But the PostgreSQL container was up:
$ docker-compose ps
     Name                    Command               State                Ports
-----------------------------------------------------------------------------------------
jarbas_cache_1    docker-entrypoint.sh memcached   Up       11211/tcp
jarbas_db_1       /docker-entrypoint.sh postgres   Up       5432/tcp
jarbas_elm_1      /bin/sh -c yarn assets           Exit 0
jarbas_jarbas_1   gunicorn jarbas.wsgi:appli ...   Up
jarbas_nginx_1    nginx -g daemon off;             Up       443/tcp, <IP_ADDRESS>:8000->80/tcp

This test routine was as follows:
git clone https://github.com/datasciencebr/jarbas.git && \
cd jarbas && \
git pull https://github.com/pedrommone/jarbas.git cuducos-elm-to-nginx && \
sed 's/<IP_ADDRESS>/<IP_ADDRESS>/g' contrib/.env.sample > .env && \
docker-compose up -d --build && \
docker-compose run --rm jarbas python manage.py migrate && \
docker-compose run --rm jarbas python manage.py ceapdatasets && \
docker-compose run --rm jarbas python manage.py collectstatic --no-input

I've seen this database situation a couple of times in Docker and I know that usually re-running stuff fixes it. However in my three tests I ended up there.
Development
The three times I trie to build a development environment the database issue was gone. But it failed to serve the static folder:
$ curl -I localhost:8000/static/app.js
HTTP/1.1 404 Not Found
Server: nginx/1.11.5
Date: Thu, 19 Jan 2017 15:02:15 GMT
Content-Type: text/html
Content-Length: 169
Connection: keep-alive

This test routine was as follow:
git clone https://github.com/datasciencebr/jarbas.git && \
cd jarbas && \
git pull https://github.com/pedrommone/jarbas.git cuducos-elm-to-nginx && \
sed 's/DEBUG=False/DEBUG=True/g' contrib/.env.sample > .env && \
sed 's/<IP_ADDRESS>/<IP_ADDRESS>/g' .env > .env && \
docker-compose -f docker-compose.yml -f docker-develop.yml up -d --build && \
docker-compose run --rm jarbas python manage.py migrate && \
docker-compose run --rm jarbas python manage.py ceapdatasets && \
docker-compose run --rm jarbas python manage.py collectstatic --no-input

Closing by a) inactivity and to b) focus on the active Docker-related PR #163


fix(side-nav): override side nav visited link styles

Description
The recent work to update the Canopy links caused some unwanted visited styles on the side nav. This PR fixes those to make them the desired states.
Checklist:

[x] The commit messages follow the convention for this project

@aaronBery @elenagarrone I have updated this OR, please can you give it another once over? 🙏


Actual heating status is not provided by this integration

Good day Thank for  a very nice integration. I need to get the heater status (on or off) from the integration. I don't currently see it as a property. Any plans or idea on how I can get it?
Thanks

Link to the Home Assistant docs concerning the difference between hvac_mode and hvac_action.
The hvac_mode is provided by the state of the climate entity, which would return 'heat' if the spa is in the heat mode.
The hvac_action is not provided by the spa controller, hence this integration doesn't provide it.
As I personally would love to gather this info, I looked again (and again!) at the packets exchanged with the spa... But they haven't changed since last time and the value has not magically appeared (too bad for us!).

Well... I won't just close this issue, but mention this limitation into the documentation.

Thanks @mathieu-mp, appreciate it.


Update PR template

Overview
This is super meta but I getting really annoyed always having to delete the intro on how to make a good PR title. I think everyone here already knows how to do this...
Basically everytime I delete it, Github detects the markdown as the start of an issue:

Changes Made

Remove redundant descriptions (like who doesn't understand this stuff after reading the titles??)
Add an HTML tag template for adding images so that we can resize and that is actually useful instead of having to click on a dropdown and seeing some image swallow up your whole screen

reminders never hurt and will help new contributors

I think the point is that the reminders are a paper cut, and so do hurt, even if just a little bit.
The new maintainers thing is a good point. I think adding a CONTRIBUTING.md could solve that issue.

reminders never hurt and will help new contributors

I think the point is that the reminders are a paper cut, and so do hurt, even if just a little bit.
The new maintainers thing is a good point. I think adding a CONTRIBUTING.md could solve that issue.

sure, having that file would do the job

thanks for the comments guys, bump on this @aliciaxw @whoiswillma
made a CONTRIBUTING.md! @whoiswillma does it suffice to put this new doc in .github?


feat: create new subdomain (DB resolver)

Feature Request
Describe the Feature Request
Implement the flow for registering a subdomain and add records within the registration flow. The components and prototype are available here.

Constraints

This should only work for domains that use our new ENSIP for offchain resolvers

  const name = normalize('edu.blockful.eth')
  const node = namehash(name)
  const signer = privateKeyToAccount(privateKey as Hex)

  if (!resolver) {
    resolver = getChainContractAddress({
      chain: client.chain,
      contract: 'ensUniversalResolver',
    })
  }

  const [resolverAddr] = (await client.readContract({
    address: resolver as Hex,
    functionName: 'findResolver',
    abi: uAbi,
    args: [toHex(packetToBytes(name))],
  })) as Hash[]

  // REGISTER NEW DOMAIN
  try {
    await client.simulateContract({
      functionName: 'register',
      abi: dbAbi,
      args: [toHex(name), 300, signer.address],
      account: signer.address,
      address: resolverAddr,
    })
  } catch (err) {
    const data = getRevertErrorData(err)
    if (data?.errorName === 'StorageHandledByOffChainDatabase') {
      const [domain, url, message] = data.args as [
        DomainData,
        string,
        MessageData,
      ]
      await handleDBStorage({ domain, url, message, signer })
    } else {
      console.error('writing failed: ', { err })
    }
  }

@alextnetto

Can you please review this issue's specs and let us know if anything?
Should it be possible to register subdomains for a domain that does not use Database Resolver?

subdomains should not be created through SubGraph

SubGraph is a tool indexing data from the blockchain, in context of ENS is used for getting domain data. So there is no write operation possible, for creating a subdomain. That's done directly in ENS contracts.

Blockchain transaction statuses are correctly reflecting in user's Ui;

In this case is not a blockchain transaction, it's a signature.
Besides that all looks good!

Great question! At this moment, we'll only register a domain through the database resolver. In a future moment where we implement an L2 infrastructure, then we'll do a blockchain transaction in the L2 (instead of the signature), following the exact same structure.


Check if systemd is available for ceph-volume

When using the init system different from systemd:
-->  FileNotFoundError: [Errno 2] No such file or directory: 'systemctl': 'systemctl'
Run ceph-volume with --no-systemd option if there is no systemd

Can one of the admins verify this patch?


Syntax highlighting still disabled

The repository already has ha highlights.scm file, but the highlighings are still disabled.
Steps to reproduce:

Follow the instructions in the Adding to neovim section
Execute in neovim :TSInstallFromGrammar fsharp
:TSModuleInfo shows an 'x' instead of a checkmark for 'fsharp' and 'highlights'

Hi,
This is an artefact of how the Neovim tree sitter integration works. Neovim won't import the highlights from the parser repository. So there is not much that can be done about this.


Hack Camp is now a school option

This makes it so hackers from Hack Camp can choose a relevant option when submitting their projects.

@zachlatta Would you please review this?

Schweet. I'll merge this back when I'm at a computer.
--
Zach Latta
http://zachlatta.com
https://github.com/zachlatta
https://twitter.com/zachlatta
On Aug 18, 2015, 11:55 PM -0700, Max<EMAIL_ADDRESS>wrote:

Assigned#38(https://github.com/hackedu/shipit/pull/38)to@zachlatta(https://github.com/zachlatta).
—
Reply to this email directly orview it on GitHub(https://github.com/hackedu/shipit/pull/38#event-386153399).


Trying Docker Compose on MacOS and getting an error db not being used when trying to add a service after running Kong

curl -i -X POST
--url http://localhost:8001/services/
--data 'name=example-service'
--data 'url=http://mockbin.org'
HTTP/1.1 405 Not Allowed
Date: Thu, 21 Jul 2022 18:32:18 GMT
Content-Type: application/json; charset=utf-8
Connection: keep-alive
Access-Control-Allow-Origin: *
Content-Length: 114
X-Kong-Admin-Latency: 156
Server: kong/2.8.1
{"name":"operation unsupported","message":"cannot create 'services' entities when not using a database","code":12}

Same problem, on docker-compose u can see this ${KONG_DATABASE:-off} u need change this

The default is to spin up a db-less Kong -- check this out.
If you want to spin up a Kong + Postgres, you can do:
  $ KONG_DATABASE=postgres docker-compose --profile database up


Add a command line argument --version to bin.sh

Issue #, if available:
Closes #443
Description of changes:

Add a command line argument --version to bin.sh.
Use the specified version as the tag name for git clone.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM!