Add GptTagger code

This PR adds a GptTagger class.
For a quick overview, please see below the example I extracted from the readme.
A few notes:

I tried to keep the surface of the API as tight as possible
The main user facing classes are GptTagger, TaskConfig and LabelledNode. It's probably a good
idea to spend the most attention there
I added a lot of tests, which should help with evolving the code
I made a lot of assumptions. Adding actual use cases will help to iron them out
The code has a couple of integration tests, which are currently excluded from
the build (have to be run manually locally, and require a working API Key)

This PR is pretty large, appologies.
Not sure how you guys usually deal with large PRs:

I can break the PR down if you have suggestions
In previous companies, we'd do a live PR walkthrough (on Zoom) to lower the overhead. It's definitely an option if you folks are interested.

Thank you

Example
Note: An API key needs to be set. See below.
from pprint import pprint

from nxontology_ml.efo import get_efo_otar_slim
from nxontology_ml.gpt_tagger import TaskConfig, GptTagger
from nxontology_ml.utils import ROOT_DIR

# Create a config for EFO nodes labelling
config = TaskConfig(
    name="precision",
    prompt_path=ROOT_DIR / "prompts/precision_v1.txt",
    openai_model_name="gpt-4",
    node_attributes=["efo_id", "efo_label", "efo_definition"],
)

# Get a few EFO nodes
nxo = get_efo_otar_slim()
nodes = (nxo.node_info(node) for node in sorted(nxo.graph)[:20])

# Get their labels
tagger = GptTagger.from_config(config)
for ln in tagger.fetch_labels(nodes):
    print(f"{ln.node_efo_id}: {ln.label}")

# Inspect metrics
print("\nTagger metrics:")
pprint(tagger.get_metrics())

You should get an output similar to:
DOID:0050890: medium
DOID:10113: low
DOID:10718: low
DOID:13406: medium
DOID:1947: low
DOID:7551: low
EFO:0000094: high
EFO:0000095: high
EFO:0000096: medium
EFO:0000174: medium
EFO:0000178: medium
EFO:0000180: medium
EFO:0000181: medium
EFO:0000182: medium
EFO:0000183: medium
EFO:0000186: medium
EFO:0000191: medium
EFO:0000195: low
EFO:0000196: medium
EFO:0000197: medium

Tagger metrics:
Counter({'ChatCompletion/total_tokens': 3187,
         'ChatCompletion/prompt_tokens': 3009,
         'ChatCompletion/completion_tokens': 178,
         'Cache/get': 20,
         'Cache/misses': 20,
         'ChatCompletion/records_processed': 20,
         'Cache/set': 20,
         'ChatCompletion/create_requests': 1})

Note that the requests are cached in .cache/precision_v1.ldb, so if you run the same code again you will
see the same labels, but different metrics:
Tagger metrics:
Counter({'Cache/get': 20, 'Cache/hits': 20})

@dhimmel I'll late you take a pass at the comments before merging 🙏

Closing #3 for now. We should create new tickets for improvements.


data.load rewrite, Node support improvements

NodeJS: Added vg.config.defaultProtocol for protocol-relative URLs
//example.org/data.csv  =>  http://example.org/data.csv

Rewrote vg.data.load.sanitizeUrl(url) to

properly join baseUrl with the url
handle relative protocol
on Node ensures only 'http' and 'https' protocols

Switched to 'request' NPM module to handle HTTP downloading
Current implementation has frequently crashed - better use a lib

Removed all global vg_load_* functions

Handle SVG's URLs in loadImage() via sanitizeUrl()

Added a global utility function vg.startsWith()
There is a number of inefficient code doing str.indexOf(search) === 0
that needs to be fixed. The new code uses it, but other changes
are out of scope for this patch, and should be changed separatelly.

Merged into v1.5.0 as of commit cca509e86737ef8202be4b29f609a99fa73274c3.


⚠ Bofrost.fr has degraded performance

In 10cfbdb, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1006 ms

Resolved: Bofrost.fr performance has improved in 961ae63 after 6 minutes.


No SDKs were found.

I have tried several times in my Ubuntu 22.04, still can not get the SDK. >_<
dotnet --info
Host:
Version:      7.0.10
Architecture: x64
Commit:       a6dbb800a4
.NET SDKs installed:
No SDKs were found.
.NET runtimes installed:
Microsoft.AspNetCore.App 7.0.10 [/usr/lib/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 7.0.10 [/usr/lib/dotnet/shared/Microsoft.NETCore.App]
Other architectures found:
None
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

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

ID: 5ac16736-f225-12d0-2de2-efded54a505a
Version Independent ID: 909b29bd-2729-3b59-beb5-20a7281344d0
Content: Install .NET on Ubuntu 22.04 - .NET
Content Source: docs/core/install/linux-ubuntu-2204.md
Product: dotnet-fundamentals
Technology: dotnet-install
GitHub Login: @adegeo
Microsoft Alias: adegeo

That is what I did. Still cannot find the SDK.....
Thx
sudo apt-get install -y dotnet-sdk-7.0
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
dotnet-sdk-7.0 is already the newest version (7.0.400-1).
The following packages were automatically installed and are no longer required:
cuda-12-1 cuda-cccl-12-1 cuda-command-line-tools-12-1 cuda-compiler-12-1
cuda-cudart-12-1 cuda-cudart-dev-12-1 cuda-cuobjdump-12-1 cuda-cupti-12-1
cuda-cupti-dev-12-1 cuda-cuxxfilt-12-1 cuda-demo-suite-12-1
cuda-documentation-12-1 cuda-driver-dev-12-1 cuda-gdb-12-1
cuda-libraries-12-1 cuda-libraries-dev-12-1 cuda-nsight-12-1
cuda-nsight-compute-12-1 cuda-nsight-systems-12-1 cuda-nvcc-12-1
cuda-nvdisasm-12-1 cuda-nvml-dev-12-1 cuda-nvprof-12-1 cuda-nvprune-12-1
cuda-nvrtc-12-1 cuda-nvrtc-dev-12-1 cuda-nvtx-12-1 cuda-nvvp-12-1
cuda-opencl-12-1 cuda-opencl-dev-12-1 cuda-profiler-api-12-1
cuda-runtime-12-1 cuda-sanitizer-12-1 cuda-toolkit-12-1
cuda-toolkit-12-1-config-common cuda-tools-12-1 cuda-visual-tools-12-1
gds-tools-12-1 gir1.2-rb-3.0 libcublas-12-1 libcublas-dev-12-1 libcufft-12-1
libcufft-dev-12-1 libcufile-12-1 libcufile-dev-12-1 libcurand-12-1
libcurand-dev-12-1 libcusolver-12-1 libcusolver-dev-12-1 libcusparse-12-1
libcusparse-dev-12-1 libdmapsharing-3.0-2 libfuse2 liblirc-client0
libnpp-12-1 libnpp-dev-12-1 libnvidia-common-530 libnvjitlink-12-1
libnvjitlink-dev-12-1 libnvjpeg-12-1 libnvjpeg-dev-12-1 libnvvm-samples-12-1
libsgutils2-2 linux-image-5.19.0-50-generic linux-modules-5.19.0-50-generic
linux-modules-extra-5.19.0-50-generic nsight-compute-2023.1.1
nsight-compute-2023.2.0 nsight-compute-2023.2.1 nsight-systems-2023.1.2
python3-mako python3-markupsafe
Use 'sudo apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

That is what I did. Still cannot find the SDK.....
Thx
sudo apt-get install -y dotnet-sdk-7.0
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
dotnet-sdk-7.0 is already the newest version (7.0.400-1).
The following packages were automatically installed and are no longer required:
cuda-12-1 cuda-cccl-12-1 cuda-command-line-tools-12-1 cuda-compiler-12-1
cuda-cudart-12-1 cuda-cudart-dev-12-1 cuda-cuobjdump-12-1 cuda-cupti-12-1
cuda-cupti-dev-12-1 cuda-cuxxfilt-12-1 cuda-demo-suite-12-1
cuda-documentation-12-1 cuda-driver-dev-12-1 cuda-gdb-12-1
cuda-libraries-12-1 cuda-libraries-dev-12-1 cuda-nsight-12-1
cuda-nsight-compute-12-1 cuda-nsight-systems-12-1 cuda-nvcc-12-1
cuda-nvdisasm-12-1 cuda-nvml-dev-12-1 cuda-nvprof-12-1 cuda-nvprune-12-1
cuda-nvrtc-12-1 cuda-nvrtc-dev-12-1 cuda-nvtx-12-1 cuda-nvvp-12-1
cuda-opencl-12-1 cuda-opencl-dev-12-1 cuda-profiler-api-12-1
cuda-runtime-12-1 cuda-sanitizer-12-1 cuda-toolkit-12-1
cuda-toolkit-12-1-config-common cuda-tools-12-1 cuda-visual-tools-12-1
gds-tools-12-1 gir1.2-rb-3.0 libcublas-12-1 libcublas-dev-12-1 libcufft-12-1
libcufft-dev-12-1 libcufile-12-1 libcufile-dev-12-1 libcurand-12-1
libcurand-dev-12-1 libcusolver-12-1 libcusolver-dev-12-1 libcusparse-12-1
libcusparse-dev-12-1 libdmapsharing-3.0-2 libfuse2 liblirc-client0
libnpp-12-1 libnpp-dev-12-1 libnvidia-common-530 libnvjitlink-12-1
libnvjitlink-dev-12-1 libnvjpeg-12-1 libnvjpeg-dev-12-1 libnvvm-samples-12-1
libsgutils2-2 linux-image-5.19.0-50-generic linux-modules-5.19.0-50-generic
linux-modules-extra-5.19.0-50-generic nsight-compute-2023.1.1
nsight-compute-2023.2.0 nsight-compute-2023.2.1 nsight-systems-2023.1.2
python3-mako python3-markupsafe
Use 'sudo apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
sudo apt install zlib1g

solved by following this link. Took me the whole night.... >_<
https://learn.microsoft.com/en-us/dotnet/core/install/linux-package-mixup?pivots=os-linux-ubuntu#solutions

Hi @AK51 sorry that you had problems. Do you have an idea of how you got into this state? And how did you find the solution? What kind of search terms did you use that didn't lead you to the correct solution? What search terms found the solution?
Understanding the whole cycle of this problem may help me improve the documentation.


Fix Power-Diffing

Most of the differentation works (see UnitTests)
But the power is simply wrong.
Doublecheck Prof. Stroetmanns Code and maybe look for additional sources on this topic.
There has to be a common scheme regarding power-differentation.

Can't find any error, tests look fine but are not ... to verbose. Lol.


Add My Project kwdb

My project kwdb is a REST wrapper for LevelDB

Thanks for sharing your work! ❤
I'm wondering about something: does it wrap LevelDB or RocksDB? Because I noticed rocksdb in the dependencies.

The v1.1.0 wraps RocksDB, other version wraps LevelDB because I had found out that rocksDB is too big
@vweevers

@vweevers

I'll get to this, I promise. There's other work with higher priority.

OK, I know, thanks

Added in e8c23b211386e7b92e19f068863f99beb38e959d. Instead of saying "kwdb is newer" I updated the description of level-rest to say it's incompatible with latest levelup.
Thanks!


URLSession must be instantiated once for entire lifespan of Connectivity instance

In the current v5.0.0 version, URLSession is instantiated as a local variable on every checkConnectivity method call and not retained till finishing of the dataTask. This may lead to crashes.
Much better approach would be defining it as a lazy var.

Thanks for reporting this - although I haven't heard of any reports of this causing crashes so far I've made changes to address this potential problem: https://github.com/rwbutler/Connectivity/commit/f136a51c1d4724b07e3c3c43cc6b1e29206cf56d


Issue with restricted permissions

I tried digging into WHERE the actual issue is but couldn't find it. So I'll describe the issue here.
I am part of an EKS cluster with restricted permissions when I try to auto complete (just about anything) but specifically kubectl edit deployment <tab>
I get an error printed out, but the the auto complete is still correct. Not sure what the issue is
 Error from server (Forbidden): customresourcedefinitions.apiextensions.k8s.io is forbidden: User "jschober" cannot list customresourcedefinitions.apiextensions.k8s.io at the cluster scope
 Error from server (Forbidden): customresourcedefinitions.apiextensions.k8s.io is forbidden: User "jschober" cannot list customresourcedefinitions.apiextensions.k8s.io at the cluster scope

You should be able to set -Ux FISH_KUBECTL_COMPLETION_COMPLETE_CRDS 0 and then try again with the latest. The issue was that we tried to complete crds, but if you don't have access to list them, then we can't complete them. Please feel free to re-open if the latest doesn't work. Thanks!


Move schema creator assets to fanstatic

From @amercader

We include both versions, and then fanstatic will pick the .min.js when running CKAN in prod mode

Fixed in #25


Comment out codes in _bootstrap-variables.scss are written in sass syntax(missing semicolon).

I installed bootstrap-sass and try to change some variables in _bootstrap-variables.scss.
I have found all lines missed semicolon.
This is not big problem, but bit troublesome to override variables.
I hope this will be fixed.
I guess just replacing templates/project/_bootstrap-variables.sass with _bootstrap-variables.scss written in scss syntax.
// Override Bootstrap variables here (defaults from bootstrap-sass v3.3.3):

//
// Variables
// --------------------------------------------------

//== Colors
//
//## Gray and brand colors for use across Bootstrap.

// $gray-base:              #000
// $gray-darker:            lighten($gray-base, 13.5%) // #222
// $gray-dark:              lighten($gray-base, 20%)   // #333
// $gray:                   lighten($gray-base, 33.5%) // #555
// $gray-light:             lighten($gray-base, 46.7%) // #777
// $gray-lighter:           lighten($gray-base, 93.5%) // #eee
[snip]

_bootstrap-variables.scss
$ compass install bootstrap -r bootstrap-sass --force
directory public/assets/fonts/bootstrap/
directory public/assets/js/bootstrap/
directory public/assets/sass/
   create public/assets/sass/styles.scss
   create public/assets/sass/_bootstrap-variables.scss
   create public/assets/js/bootstrap/affix.js
   create public/assets/js/bootstrap/alert.js
   create public/assets/js/bootstrap/button.js
[snip]

I opened _bootstrap-variables.scss just after compass install.

@chriseppstein Is there a way to provide the template in the right syntax?

I've just discovered the same thing – not a huge deal but a bit annoying and possibly confusing for people new to sass. I see it's in the roadmap to fix, which is great, thanks!

@graemeblackwood It's in the roadmap but unfortunately I have no idea how to tell compass to provide the template based on the user's preferred syntax

@glebm how do you normally use the template file / override variables then? I understood this semicolon thing would be an extra possibility only, or? Thanks!

@glebm Where are we at on this? Perhaps we should WontFix this?

@cvrebert There is still no way to provide the template in the matching syntax AFAIK. Closing as won't fix.

I just sat and fixed this file as a result of having to poach somethings from a formerly config'd LESS file. Just change the extension on it.
_bootstrap-variables.txt


Remove version specification for Runes

We don't actually need to be so prescriptive about what version of Runes
we're using anymore. Any version after we turned on extension
compatibility will be fine for our needs.

:+1:


Add GitHub Actions to meta-ros

Adding GitHub Actions to meta-ros would be useful for automating checks and builds.

@ajlennon Yocto Layer CI Build and Test with GitHub Action by Alex Lennon (https://youtu.be/TsAcxd_acJI?si=HqRCqZ9M-f0T-Y-3) his work is here: https://github.com/DynamicDevices/meta-mono/blob/master/.github/workflows/CI_github.yml
@razr did some work integrating meta-ros CI with GitHub Actions (https://github.com/razr/meta-ros/blob/hardknott-ur5-gh-action/.github/workflows/two-jobs.yaml)
@Allegra42 is also giving a talk at the Yocto Summit 2023 entitled "Building a Yocto CI Pipeline with KAS, GitHub Actions and AWS - What could possibly go wrong?" (https://summit.yoctoproject.org/yocto-project-summit-2023-11/talk/RNHAPH/)

@moto-timo Also pointed me to the work done for CROPS: https://github.com/crops/poky-container/blob/master/.github/workflows/build-test-deploy.yml

See ROS 2 Tooling Working Group for existing ROS tooling:

Design: GitHub Action Release Process
action-ros-ci
ROS 2 GitHub Action CI Template
https://github.com/ros-tooling/setup-ros
https://github.com/ros-tooling/setup-ros-docker/tree/master
https://github.com/osrf/docker_images


Why are packages being removed/disappearing from the Registry?

Hi
I'm the owner of LibreTiny platform package on the PIO registry.
The project was formerly called LibreTuya, and such package was also published in the registry.
What I don't understand is, why is the previous package gone, even though I didn't request it to be unpublished?
Also, the new package currently has only one version available (1.1.0), even though versions 1.0.1 and 1.0.2 have been published and available before. Again, I didn't request them to be removed. The same issue was happening to the previous LibreTuya platform (old versions were disappearing).
Is there something wrong on my end? (i.e. some setting in the CI publishing workflow, or some account setting?)

Hi, what runtime API and dependencies do you mean? Were the versions removed manually by the administration team, or is it an automated process? Can I request all versions to be kept for compatibility purposes in some way? (projects might require a specific version of the platform)

what runtime API and dependencies do you mean?
https://github.com/kuba2k2/libretiny/blob/master/platform.py

Were the versions removed manually by the administration team

Yes, we restored them.

Can I request all versions to be kept for compatibility purposes in some way?

We will not touch further release if this is the final stable development platform. We saw duplicated platforms from your account before. Even the current version looks like an experimental demo. The link is broken to the repository https://registry.platformio.org/platforms/kuba2k2/libretiny

The duplicated platform was because of the project rename (which was necessary due to possible copyright issues). That was done along with a bigger refactor, so I don't need the old platform to be available anymore.
The current version is stable, working, and used by many people on hundreds of devices. I'm not sure what makes it look like an experimental demo; the broken repository link is just a little thing that I've missed (it's only visible on PIO registry anyway). The fact that it evolves quickly doesn't necessarily mean that current versions aren't stable.
The project name is final, but that also doesn't mean the platform will not get newer versions than 1.1.0 - it certainly will. Even then, keeping old versions is just a good practice, and shouldn't mean that any released version makes old versions unstable.
To summarize, I'd like to request all previous (and future) versions of LibreTiny not to be removed from the registry, because it may be very problematic and break projects like ESPHome (which depend on a specific version). Let me know if that's possible, thanks.

Thanks for the updates! No new versions will be deleted in the future.

break projects like ESPHome (which depend on a specific version).

Could you point me to the source of code on which version ESPHome depends?

Here:
https://github.com/kuba2k2/libretiny-esphome/blob/platform/libretiny-prerelease/esphome/components/libretiny/__init__.py#L159
ESPHome support hasn't been merged into upstream yet, but we're working with the ESPHome team towards doing that soon. The same version constraint is there for esp32 or esp8266 support.


Change log is not added for 4.11.0

There is no change log for 4.11.0 version https://square.github.io/okhttp/changelogs/changelog_4x/

Please read: https://github.com/square/okhttp/issues/7849#issuecomment-1561123598

Thank you.


unicode or img option

Hi There,
I added an additional option to get just the unicode representation of the emoji instead of an image element.
~Nick

In my opinion it's better to add unicode support as a separated function i.e. unicodify.
Because if you make it as part of "emojify" other options don't make any sense.
I can grab your PR and make my own with that change, hopefully with passing tests.
Will you merge that PR @banyan ?

@hamczu you are welcome to make any changes to my pull request that you see fit. Let me know if there is anything you would like me to do.


Added a mode to AStar/AStar2D for ignore disabled points feature

Added a new boolean property called point_disabling_mode (which is enabled by default so it's not a breaking change). It's sometimes useful to generate a path regardless of obstacles generated by set_point_disabled, without recreating the whole AStar. The example is a fog of war in strategy games - you don't know which object is under that fog and assume that there is no obstacle on the path to generating a simple way, but internally they should be marked as disabled points in AStar, so when they become visible the same AStar can be used with point_disabled_mode toggled on to generate a path across the obstacle.

Ah, nvm, I think it's a bad idea - since the visible tiles are affected too..


Word game

Feature ✅
Description
For each input word find its occurrence.
Example
First line is number of test cases.
In output first print number of distinct words followed by word occurrence.
Input:
4
bcdef
abcdefg
bcde
bcdef
Output:
3
2 1 1
Contributors are supposed to mention their coding language while asking for assignment

can i do this in c++ ?

Please assign this to me! @pratik-choudhari

Can I do in java as a part of hactoberfest?

I'd like to do this in C++ too. @pratik-choudhari

I would like to do in python. Please assign it to me.

@vanya-24 Language?

I can do in C++,C,Python! @pratik-choudhari

@vanya-24 c++
@shubha028 java
@nikhil05-code python


Unblurred images and info lost after next refresh(s)

It seems that after the first unblur() and the next (or one after that) updateUserInfos all profiles are shown.

Tested with Version 4.0

I have 14+ likes
When i open "likes" page, i see unblured images and 2,3 seconds later all of them gone and i have empty page.
Refresh doesnt help.

That would require the api response to change though, I’ll test a few things later.

@tajnymag #29 should give us more info

@gavracone please check if the issue persists with the latest dev version: https://github.com/tajnymag/tinder-deblur/raw/develop/tinder.user.js

@gavracone please check if the issue persists with the latest dev version: https://github.com/tajnymag/tinder-deblur/raw/develop/tinder.user.js

On FF all thumbs are blured and all gone after 1,2 seconds but one stay with messege "invalid"
On CHrome, thumbs are still blured and all gone after 1,2 seconds.

yup, doesnt work well

Firefox 113.0.1 (64bit)
The new version works way better for me.
Temporary video links (deleted after 5d)

https://tempclip.com/CMOMkV71rhATdUf/watch
with console open: https://tempclip.com/B4paCfAlbZX04Er/watch

Looks similar to me @n00b42 .
Here's how it looks to me with both Firefox (version 113.0.1) and Chrome (Brave v1.51.118): https://streamable.com/v81n4u
@imongithub123 @gavracone @naoliv could you please clear your browser data or disable cache via DevTools and try again with this alpha version: https://gist.github.com/JustixDevelopment/dcb9766e8bc5f7a4d0b3a14c85079cea/raw/05c2ec4333191a51ad7b48bdd1d39794b44d6f42/tinder.user.js

Yep, it did work with cache disabled.
And I could even match or discard even without having Tinder gold.
Fantastic!

Just one doubt: after discarding/matching some contacts, it only stays like this:

And it doesn't load anything else anymore.
It means that basically all that 17 likes are from fake Tinder profiles?

yeah and it doesnt seem to be loading different people like it used to. i feel like im seeing the same faces.

yeah and it doesnt seem to be loading different people like it used to. i feel like im seeing the same faces.

This is because of Tinder, they changed their API so that it always returns the same 10 teasers now.
Also, I have noticed that I have more likes than the app/website tells me since I once had a match with someone that was not in the teasers and I had exactly 10 likes at that point.

Just one doubt: after discarding/matching some contacts, it only stays like this:

And it doesn't load anything else anymore. It means that basically all that 17 likes are from fake Tinder profiles?
Edit:
Sometimes it loads like this:

Okay so I'll show a different message if there are no likes left. Like I already explained in the comment above, even if you disliked/liked all 10 teasers, you will most likely see the same teasers over and over again.

@naoliv @imongithub123 here would be the new version with the "no likes available" message: https://gist.github.com/JustixDevelopment/29747f682a30377f2e2f9d9cd869aae5/raw/600d6ee3c4adeb9a8951c785578d834b5f329eda/tinder.user.js

thanks, unfortunately it's not as good as it was a few days ago

@imongithub123 could you explain what is not working as expected?

At the moment, I still see one "invalid" on my screen

Though, it's great to be able to match with the person you see now just by clicking the heart. Unfortunately once the you do everyone on the page, even if you have more likes, it won't show anyone new.

This is the only issue for me now

Justix came up with a new batch of fixes, could you please test it and let us know?
https://github.com/tajnymag/tinder-deblur/raw/develop/tinder.user.js

Still have the same problems, sadly. Just one showing that is 'invalid'

Still have the same problems, sadly. Just one showing that is 'invalid'

Even after cleaning page data+cache?

Though, it's great to be able to match with the person you see now just by clicking the heart. Unfortunately once the you do everyone on the page, even if you have more likes, it won't show anyone new.

Well as I already said, we can’t do anything about that since the script relies on the Android API. They sadly changed it so that it only returns the first 10 likes, even if you disliked them. The „invalid“ tag means that the teaser has a url which cannot be unblurred since it doesn’t contain the user id.

Clearing cache shows me the same faces like you said. Any chance I'll see different people and how long does that take?

I can’t give you any information about that because only Tinder knows how the teasers are chosen, sorry.

All good, thank you

Screen is stuck on loading unfortunately.

https://github.com/tajnymag/tinder-deblur/raw/develop/tinder.user.js

Yesterday I had the impression it still worked.

I have been using my own php tinder script, to show recommendations, likes, view profiles, and everything works except for getting the user profile returning 403 Forbidden as mentioned above, trying to figure it out with no success, the same url works when accessing from tinder.com, I guess something wrong with headers?

After further investigation I can confirm that the API was changed, the platform header no longer has any effect and we only get fake user ids back. It's been a great time, thank you all for your loyality to the script and especially @tajnymag for making this possible at all!

Are you sure? I get valid user ids on "/v2/fast-match/teasers" the only problem is "/user/_ID" returning 403 Forbidden for me.

@JustixDevelopment https://github.com/ecole96/TinderUnblur/tree/master
refer to this repo, it is working

i dont think the api was changed

@JustixDevelopment https://github.com/ecole96/TinderUnblur/tree/master refer to this repo, it is working

well this is because this is using a different approach to unblur the images, this will only work for 1 image though, I could implement that using the following code:
teaserEl.style.backgroundImage = https://preview.gotinder.com/${teaser.user._id}/original_${teaser.user.photos[0].id}.jpeg`;`

We might have to go back to the roots

Are you sure? I get valid user ids on "/v2/fast-match/teasers" the only problem is "/user/_ID" returning 403 Forbidden for me.
By inspecting network on tinder.com it retrieves the user, when I replace the user id and resend it returns 403, only thing that seems to change is from Sec-Fetch-Mode: cors to Sec-Fetch-Mode: no-cors if that matters..

The user id is valid, but it is not the real user id.
Have a look at this, I show the result using a real user id from a match and the one returned from the Android api: https://streamable.com/fjc7l7

i dont think the api was changed

as you can see in my comment above, it was

We might have to go back to the roots

yup, we have to remove like 80% of the script sadly

Are you sure? I get valid user ids on "/v2/fast-match/teasers" the only problem is "/user/_ID" returning 403 Forbidden for me.
By inspecting network on tinder.com it retrieves the user, when I replace the user id and resend it returns 403, only thing that seems to change is from Sec-Fetch-Mode: cors to Sec-Fetch-Mode: no-cors if that matters..

The user id is valid, but it is not the real user id. Have a look at this, I show the result using a real user id from a match and the one returned from the Android api: https://streamable.com/fjc7l7

the user id is the real id, you arent getting any result because the /user/_ID endpoint is returning 403 on any user id you fetch

Are you sure? I get valid user ids on "/v2/fast-match/teasers" the only problem is "/user/_ID" returning 403 Forbidden for me.
By inspecting network on tinder.com it retrieves the user, when I replace the user id and resend it returns 403, only thing that seems to change is from Sec-Fetch-Mode: cors to Sec-Fetch-Mode: no-cors if that matters..

The user id is valid, but it is not the real user id. Have a look at this, I show the result using a real user id from a match and the one returned from the Android api: https://streamable.com/fjc7l7

the user id is the real id, you arent getting any result because the /user/_ID endpoint is returning 403 on any user id you fetch

i literally just showed in the video that it doesn't?!

Omg, try fetching a "REAL" id on https://api.gotinder.com/user/, you GET 403 for any id, except when when you fetch it via tinder.com

Well since we do fetch it from tinder.com with this script, it doesn't matter if the referer must be tinder.com. Fetching real ids works fine for me, just not the ones from the teasers.

I have logged real ids from swipes from few days ago, and they dont work, I dont know anymore

I have logged real ids from swipes from few days ago, and they dont work, I dont know anymore

I can confirm that swipe ids also don't work, it looks like they restrict the user endpoint for matches.

Yeah I just figured that out now, I can view only my matched profiles via /user/_ID, the ids are real just restricted to matches.

@JustixDevelopment https://github.com/ecole96/TinderUnblur/tree/master refer to this repo, it is working

well this is because this is using a different approach to unblur the images, this will only work for 1 image though, I could implement that using the following code: teaserEl.style.backgroundImage = https://preview.gotinder.com/${teaser.user.id}/original${teaser.user.photos[0].id}.jpeg;

I tried that extension and it unblurred all the images that were in my Liked section

@JustixDevelopment https://github.com/ecole96/TinderUnblur/tree/master refer to this repo, it is working

well this is because this is using a different approach to unblur the images, this will only work for 1 image though, I could implement that using the following code: teaserEl.style.backgroundImage = https://preview.gotinder.com/${teaser.user.id}/original${teaser.user.photos[0].id}.jpeg;

I tried that extension and it unblurred all the images that were in my Liked section

yes that is exactly what I meant, it unblurres 1 photo per user (like), but you cannot see the name, other photos etc.

@JustixDevelopment https://github.com/orknist/tinder-deblur-gold/blob/main/tinder.user.js
Found this script where clicking the heart emoji matches automatically. Is there any way to get the username from this implementation?

@anilabhadatta no, but I made a workaround for the API changes, though we just have one unblurred image and the like/pass functions now, the name, age, distance etc. are gone for now.

Guys, can you try the development version again? Justix came up with a workaround for the API changes.
https://raw.githubusercontent.com/tajnymag/tinder-deblur/develop/tinder.user.js

Guys, can you try the development version again? Justix came up with a workaround for the API changes.
https://raw.githubusercontent.com/tajnymag/tinder-deblur/develop/tinder.user.js

It's displaying the pictures again, but dismissing/matching isn't working.

It's displaying the pictures again, but dismissing/matching isn't working.

@naoliv that will work again once #36 was merged

Merged

Cleared cache and cookies.
I have 9 likes.
Today i received one like. I cant  see that picture, i get "invalid" error
I see rest of likes, i dismiss them but upon next refresh, i see them again.

I see the first photo of my likes. That’s about it. Good stuff though
either way guys
On Sun, May 21, 2023 at 20:02 gavracone @.***> wrote:

Cleared cache and cookies.
I have 9 likes.
Today i received one like. I cant see that picture, i get "invalid" error
I see rest of likes, i dismiss them but upon next refresh, i see them
again.
—
Reply to this email directly, view it on GitHub
https://github.com/tajnymag/tinder-deblur/issues/27#issuecomment-1556225941,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2J4S5ZZCVMZXU5PC6HTBSLXHJDD5ANCNFSM6AAAAAAYCK4L6A
.
You are receiving this because you were mentioned.Message ID:
@.***>

Cleared cache and cookies.
I have 9 likes.
Today i received one like. I cant see that picture, i get "invalid" error
I see rest of likes, i dismiss them but upon next refresh, i see them again.

the second issue should be fixed with the current file, the "invalid" image simply means that it cannot be unblurred because the image url doesn't contain enough information, you can send the image url in here though and I'll try to find a way to unblur it

I think this issue can also be closed since the initial issue is resolved. @tajnymag


Could not send to Todoist with Logseq URI link.

I see an option to Append Logseq URI. I opened this option but see no Logseq URI link in Todoist after sending tasks. Is that normal?
And I see #37 and thought that would be awesome. While, now I could not send to Todoist with Logseq URI link.

Should be solved in latest release.

Thanks for your great work! But I still could not get the Logseq URL in tasks sended to Todoist.
And the tags showed as numbers.

Can you screenshot your entire settings page  please?

It looks like you are not at the latest version. Can you update the plugin please?

I am also seeing behavior this on v5.1.1

I tried it showed: up to date

Hmm it should be on v5.1.1, and not 5.2.7. Could you uninstall the plugin, quit Logseq, and install it again?
The settings in the latest version does not have Todoist Sync at the bottom.

Thanks v5.1.3 worked nice!


Add support for overriding .pypirc location

Add support for overriding .pypirc location via an environment variable PYPI_CONFIG_FILE. I've added tests as well. Happy to address any review feedback. This fixes #954

Closing per discussion in #954.


[graphql] The resolvers array is empty

After installing the package, I need to manually change this provider in adonisrc.ts.
() => import('@foadonis/graphql/graphql_provider') to
() => import('@foadonis/graphql/providers/graphql_provider')
Also, when starting the application,  it says that:
Error: Empty 'resolvers' array property found in 'buildSchema' options!
Even though I have registered the resolver here.
import graphql from '@foadonis/graphql/services/main'

graphql.resolvers([() => import('#graphql/resolvers/user_resolver')])

After installing the package, I need to manually change this provider in adonisrc.ts. () => import('@foadonis/graphql/graphql_provider') to () => import('@foadonis/graphql/providers/graphql_provider')
Also, when starting the application, it says that:
Error: Empty 'resolvers' array property found in 'buildSchema' options!
Even though I have registered the resolver here.
import graphql from '@foadonis/graphql/services/main'

graphql.resolvers([() => import('#graphql/resolvers/user_resolver')])

Facing the same issue.

After installing the package, I need to manually change this provider in adonisrc.ts. () => import('@foadonis/graphql/graphql_provider') to () => import('@foadonis/graphql/providers/graphql_provider')
Also, when starting the application, it says that:
Error: Empty 'resolvers' array property found in 'buildSchema' options!
Even though I have registered the resolver here.
import graphql from '@foadonis/graphql/services/main'

graphql.resolvers([() => import('#graphql/resolvers/user_resolver')])

Facing the same issue. and getting the message tried to access Apollo Server when not initialized
when requesting on the navigator /graphql

I have resolve the issue here. The start/graphql.ts must be created by using the command node ace make:preload graphql or manually add () => import('#start/graphql') under preloads in adonisrc.ts

Thanks @ajagsantiago,
But after adding the start/graphql
I am facing this new problem.
TypeError: Cannot read properties of
undefined (reading 'resolvers')
`
import graphql from '@foadonis/graphql/services/main'
graphql.resolvers([() => import('#graphql/resolvers/user_resolver')])

TypeError: Cannot read properties of
undefined (reading 'resolvers')
*
`


Should not fail if target to be deleted is already gone

BuildpacksCacheController#delete removes all entries of the buildpack_cache. With
a DavClient this will create an issue as soon as an app is deleted, because the
delete_all_in_path method (called when the buildpack_cache for the app is removed)
will raise after not finding the app guid in the first place.
This is not an issue with FogClient, which just silently returns. We believe this
is the correct behavior because the target state (app_guid and children are gone)
has already been reached.
BuildpacksCacheController#delete removes all entries of the buildpack_cache. With
fog_client.delete_all_in_path and dav_client.delete_all_in_path behave
differently:

fog_client will not attempt to delete anything if the given path does not exist.
It will not fail or raise an exeption.
dav_client will raise an exception when to path to be deleted does not exist.

Note: This is only an issue when using the CC v3 API.
This commit brings the behavior of DavClient in line with FogClient when attempting
to delete a resource that already is gone.
thx,
@suhlig and @MarcSchunk

Looks great! Thanks!


running error

DO NOT ASK FOR PHISHLETS.
DO NOT ASK FOR HELP CREATING PHISHLETS.
DO NOT ASK TO FIX PHISHLETS.
DO NOT ADVERTISE OR TRY TO SELL PHISHLETS.
EXPECT A BAN OTHERWISE. THANK YOU!
REPORT ONLY BUGS OR FEATURE SUGGESTIONS.
when running
goroutine 26867 [running]
github.com/inconshreveable/go-vhost.readClientHello(0xadfd00?0xc0001c2780?}
/rootevilginx2/vendor/github.com/inconshreveable/go-vost/tos.go:259+0x2f5
...
..
..

telegram itsdayztar


OpenCV not Found

Although opencv is installed and its path is included in the env variable, the make process shows that OpenCV not installed.
F:\Technology\Learning\dlib\dlib\examples\build>cmake ..
-- C++11 activated (compiler doesn't have full C++11 support).
-- Enabling SSE2 instructions
OpenCV not found, so we won't build the webcam_face_pose_ex example.
-- Configuring done
-- Generating done
-- Build files have been written to: F:/Technology/Learning/dlib/dlib/examples/build

Apparently it's not set or installed correctly :)
That CMake file uses CMake's built in find opencv support.  So if it doesn't work then either you installed opencv wrong or (unlikely) you have a broken version of CMake.

How to set it? What should be the path variable?

Modify the cmake script so the opencv find command isn't quiet.  The output
of cmake will then tell you what to do.

F:\Technology\Learning\dlib\dlib\examples\build>cmake ..
-- C++11 activated (compiler doesn't have full C++11 support).
-- Enabling SSE2 instructions
-- OpenCV ARCH: x86
-- OpenCV RUNTIME: vc14
-- OpenCV STATIC: OFF
CMake Warning at C:/opencv/build/OpenCVConfig.cmake:163 (message):
  Found OpenCV Windows Pack but it has not binaries compatible with your
  configuration.

  You should manually point CMake variable OpenCV_DIR to your build of OpenCV
  library.
Call Stack (most recent call first):
  CMakeLists.txt:125 (find_package)

CMake Warning at CMakeLists.txt:125 (find_package):
  Found package configuration file:

    C:/opencv/build/OpenCVConfig.cmake

  but it set OpenCV_FOUND to FALSE so package "OpenCV" is considered to be
  NOT FOUND.

OpenCV not found, so we won't build the webcam_face_pose_ex example.
-- Configuring done
-- Generating done
-- Build files have been written to: F:/Technology/Learning/dlib/dlib/examples/build

F:\Technology\Learning\dlib\dlib\examples\build>

'''
OPENCV_DIR  C:\opencv\build\x64\vc11\bin

You need to add the OpenCV directory in your Path as well as create a variable called OpenCV_DIR. add the bin directory to your path for example mine is located at H:\lib\opencv\build\install\x64\vc14\bin and then point to the root of your opencv folder in your OpenCV_DIR variable mine is H:\lib\opencv\build\install. I have built OpenCV from source. If you install it using a binary distribution it will look different.

The simplest way is usually to set CMAKE_PREFIX_PATH to wherever you put opencv.  For example
cd /wherever/you/put/opencv/
mkdir build
cd build
cmake ..
# build opencv
make

cd /wherever/you/put/dlib/examples/build
cmake .. -DCMAKE_PREFIX_PATH=/wherever/you/put/opencv/build


[sigh] fix resigning of OnDemandResources when app has space in name

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
If the app path has a space in it, the call to dirname here results in the following error, which is silently ignored
usage: dirname path

The following line of code then checks if /OnDemandResources is a directory, which never will be, and this code is never executed.
Description
Just wrapping the $APP_PATH variable in double quotes to ensure it will work for paths with spaces in them.
Testing Steps

@lyndsey-ferguson This is a simple change but I think you are still more of an expert here than me 😛 Can I borrow your 👀?

@lyndsey-ferguson Thank you so much!!

Thanks! 👍


[PE-D][Tester A] The error message displayed is not useful when the user accidentally puts comma in between the index

command used:  event -n Get Together Party -sd 2023-10-25 -ed 2023-10-27 -l MBS -a 1, 2, 3

Labels: severity.Low type.FeatureFlaw
original: Elinengu/ped#15

Perhaps we should make the invalid index message more specific - like Format of attendee indices is invalid. Proper format: ...


[DRAFT] Add eval and explain

Description of changes:
Add query evaluation and Explain to the playground using partiql-lang-rust 0.2.0 new features.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
Base: 1.56% // Head: 1.09% // Decreases project coverage by -0.47% :warning:

Coverage data is based on head (5e1285a) compared to base (964e61e).
Patch coverage: 3.57% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff            @@
##            main      #6      +/-   ##
========================================
- Coverage   1.56%   1.09%   -0.47%
========================================
  Files          1       1
  Lines         64      91      +27
========================================
  Hits           1       1
- Misses        63      90      +27

Impacted Files
Coverage Δ

src/lib.rs
1.09% <3.57%> (-0.47%)
:arrow_down:

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

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


Overall story: Rolling release system

For the rolling release system, there's currently few things to be done

[ ] make executable binaries, WIP
[ ] let our CI upload those binaries to GitHub
[ ] test and merge #399
[ ] add a graphical user interface to install inexor-game/core npm packages to flex

[ ] make the flex extensions framework working
[ ] make the TOML extension running, see #397

[ ] add extensive user documentation on how to install Inexor with the new system
[ ] (optionally) fix the layout of @inexor-game/site

This will also close milestone 0.9.0-alpha (Speed Edition)

Why aren't you using #37?

Because this is the overall story for 0.9.0 release, which all tickets should be closed before release.

I've also cleaned up the milestones on GitHub, please consider the current list at https://github.com/inexor-game/code/milestone/10 as locked. All the tasks should be done to a level where we can say they are acceptable as an experimental stage. Then go, release.

Aaah okay: this issue is meant to be a milestone actually :)
We should make this page https://github.com/inexor-game/code/milestone/10
contain the info we have in this issue.
So we need to create issues for the points you mentioned (and tag them).

I don't believe we can deliver a rolling release system with 0.9.0. A lot of tasks have to be done until we get to this point. So this topic shouldn't the main focus of 0.9.0.
Until now the main focus were Flex, CEF and RPC/Tree, which are big goals already and all of them still needs work to be completed. I would suggest to focus on these and bring them into a stable state first before targeting another huge goal.

1. Updating
So Inexor Flex is planned to update Inexor core.
Flex can maybe get updated via a .sh/.bat file.
How are we updating Node.js?
NOT updating Node.js or expecting the users to do it manually is unacceptable.
2.  Packaging problem
This doesn't fix the packaging problem for Linux or Windows (at all?).
Quote Wiki: "built for Ubuntu 16.04 and Windows only for now"
So it's highly limited on which Linux Distributations you can run it and we might get non producable  "but it's working on my machine" bugs because of different Ubuntu versions AND we exclude all other Linux users which aren't Ubuntu users. Doesn't look like a solution. We are back were we started on Linux: Building for each and every distro binaries (not do-able) or exclude people (not acceptable). And if we are starting to build specific binaries, we could host distro specific files/repositories in the first place. Remember where we started this entire discussion?
Snapcraft as proposed by Fohlen previously or Flatpak (as favored personally be me #395) or another container-alike packaging format seems to be the only sophisticated, long-lasting solution on Linux. Everything else might improve a few things here and there, but is not solving it at the root of the problem, plus creating likely new problems on every step of the way. Snapcraft/Flatpak also having their updating services, solving the update problem as well. Also solving it in a widely adopted and tested way.
I'm sure we would find an good updating solution on Windows as well if we don't rush it. We are mostly a 4-core-team-member show and I feel sometimes like we want to do too much in parallel. Concentrating on doing it from the ground up might be better.  What we are doing right now in parallel is getting it build for some, have it complicated and error prone to run for everyone, and working on it to get it packaged for some.
3.  End-user dependency
On Windows we seem to settle even more on Node.js/npm as an end-user dependency, right? (#364) How else should this setup work without a pre-installed Node.js/npm? And if we are starting again to ship Node.js/npm as we previously did with our platform repository (https://github.com/inexor-game-obsolete/platform) how are we supposed to update these via npm? (Back at problem 1.)

@aschaeffer
I don't think going rolling release is actually that far fetched.
Yes, sure we had different plans for 0.9.0 but CEF, Flex and RPC/Tree are all more high-level features than the rolling release system is.
Since we're working bottom-up it would make sense imo to tackle this first.
So the ordering is:

make the build system more stable
make testing / distribution easy (rolling release)
add features and systems (CEF, flex, ..)

This issue is a duplicate, close.
Everything else -> #37


[CONSOLE] Fix real-time balance on accounts

Fixes #66
Solution?
Balances are first queried from Subgraph and then from the RPC as soon as possible. The RPC calls are automatically batched/throttled into a multicall.

@kasparkallas Events tab is not showing any events , thanks Cypress

For testing, you can use: 0x7bda037dfdf9cd9ad261d27f489924aebbce71ac which is an account with balances not in sync on Subgraph.

Haven't done it for "Balances" panel yet.

@kasparkallas Events tab is not showing any events , thanks Cypress

Yep, that stems from the dev build of the sdk-core. Will push another fix to the sdk-core...

I will let you know when 2 and 3 are done.

Should be good now, @elvijsTDL


Handle serving from a sub-path

Hi there! 👋
I took a shot at letting wastebin work when served from a sub-path (e.g. https://example.com/wastsebin/). It's a WIP, but I thought of getting this out early and discuss 1. if useful; 2. improvements.
The idea is to re-use the WASTEBIN_BASE_URL variable and get its path component (if any).
Right now:

[X] Existing tests pass.
[X] It works™.
[x] If required, clean-up.
[x] Add tests for the sub-path implementation.

What's the best approach to add the test? Duplicate them? Run them in a for loop with / without the sub-path?
Also, I am not convinced of storing the base path in the metadata as a static variable. Are there other approaches?

First of all, the feature is certainly welcome, thank you :+1:

Also, I am not convinced of storing the base path in the metadata as a static variable. Are there other approaches?

I also don't think it's the best approach and rather compute it at startup and store it in a OnceCell. Also, it could make sense to abstract the BaseUrl in a Url newtype and avoid the manual joins in the templates and elsewhere.
I will have a closer look and more comments by the end of the week.

I also don't think it's the best approach and rather compute it at startup and store it in a OnceCell. Also, it could make sense to abstract the BaseUrl in a Url newtype and avoid the manual joins in the templates and elsewhere.

https://github.com/matze/wastebin/pull/48/commits/cbb71a1a28c0a0a21af786e25e8705d131aed86f gives a shot at addressing this. I have added a newtype for BasePath and used OnceLock to initialise it. I am not a super fan of handling all errors there, but raising them would require a bigger change. Happy to hear thoughts about it.
Overall kept things pretty simple, but the code looks cleaner now by using BasePath::join. Maybe BasePath could live somewhere else than env, but that's easy to change.
I considered adding a newtype for the BaseUrl too, but eventually decided against it as it doesn't add much value.
Also, open to comments about the best way to test this. I could think of duplicating the tests and/or the client, but maybe there are better approaches.

Also, open to comments about the best way to test this.

Up to you, so far things look good to me. If you think it's ready, let me know.

Done! I have considered using parametrised tests, but since we use static variables things get complicated pretty fast.
In the end I have opted for simplicity: the tests respect BASE_URL and the CI runs both on / and on /wastebin.
Ready from my point of view!

Thanks, I will let it simmer for a bit and try to make a new release soon enough.

Thank you! When you do release, could you also update the README? I forgot to do that.


Extend supported models in TimmFeatureExtractor

Description
This PR extends supported backbones inside TimmFeatureExtractor. I refactored the function that maps layer names to feature IDs, so it now also supports models that don't have layer names inside named_children dict.

Fixes #1442

As requested in #1144 I think that documentation would need to be extended with some more info on backbones.
Changes

Describe the changes you made

[ ] Bug fix (non-breaking change which fixes an issue)
[x] Refactor (non-breaking change which refactors the code base)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Checklist

Ensure that you followed the following

[ ] I have added a summary of my changes to the CHANGELOG (not for minor changes, docs and tests).
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas (If applicable)
[ ] I have made corresponding changes to the documentation (If applicable)
[ ] I have added tests that prove my fix is effective or that my feature works (If applicable)

Hi @blaz-r thanks for this PR! It helped me test with a few backbones that I couldn't do previously.
I have a small question. I'm trying to use ViT base from timm as a feature extractor for patchcore/other models. Since these models don't support features_only flag while using the create_model function, this PR would fail in that case. I know it might be out of scope for the current PR, but just wanted to bring this up in case we can make it more extendable. I'm working on it and will update if I find something that works for transformer models too.

Hi @saifvazir, nice to see you find this useful. Thanks for pointing out the problem with transformer based models.
There are some used in the Fastflow model if you want to see how to use them. As you said features_only is omitted as well as return_indices. Maybe we could add an option inside the layers of config for something like final_layer which would indicate to avoid using the two parameters above.

Yes, I took a look at the the Fastflow model and I found that we've sort of hardcoded the allowed transformer model feature extractors in torch_model.py. I feel that this is kind of restricting given the amount of ViT models available in timm. Maybe requires a separate discussion thread on how to handle transformer models as feature extractors. However, thanks for putting up this PR and replying :)

Glad to see that you find this PR useful. While we maybe could use a separate discussion thread, I think it's still a possibility to extend this PR as it technically covers this exact thing.
Maybe we could specify a special parameter for layers in config, that would indicate that the final layer should be used.
@ashwinvaidya17 is this worth implementing, since timm is supposed to be deprecated?

@blaz-r I think there is a bit of confusion due to my last PR. We are deprecating the class FeatureExtractor in favour of explicit TimmFeatureExtractor. This is because we have both TorchFX and Timm. Actually, if it isn't too much work can you target this PR to the v1 branch? The CI in main is failing due to which a lot of PRs are now outstanding.

Okay. I'll change this to v1.

Well that was a bit messy, but it should be ready for v1.0 now.

The question we discussed with @saifvazir still remains. What should we do with ViT like models?

Hi @blaz-r , I might be wrong here, but can we use torchfx to load the backbone weights directly and then pass in the layer names to extract the relevant features? I see that TorchFx is just being used in the CSFlowModel as of now with the Multi Scale extractor. I think we could make it more generalizable and use it directly by specifying something in the config file? @ashwinvaidya17 since you were the author for TorchFx, what are your comments on this idea?

@saifvazir sorry for the late reply. There is TorchFX feature extractor implemented, so new models can already use that. But many of the current models use Timm. Having this as part of the config would probably introduce some problems, and I think it just unnecessary complexion to code and usage of library.

@blaz-r is this pr still up to date and relevant? if yes, I'll merge it

@samet-akcay yes, it is still relevant. I just merged the new additions of v1, if all the checks pass it should be ready to merge.


Download as txt presents partial information in the report

For example - Pods (it happens in all entities, looks like there's some limitation on number of columns presented)

Its txt report:

@miq-bot add_label ui, reporting

@gtanzillo this issue is open for more than a year, is there any ETA for a fix?

also, is this the right repo to have this bug in or it should move to the ui repo?

@miq-bot assign @yrudman

@miq-bot close_issue


USHIFT-2063: Make fetch_tools.sh more robust when parsing function names

Previous parsing failed occasionally on aarch64 platform with the following error. Since the awscli parameter is correct, it's likely that the parsing failed to "find" it in the function list, depending on the environment.
Script for downloading various tools
Usage: fetch_tools.sh <all | specific-tool...>
Destination can be changed using DEST_DIR environmental variable. Default: '/home/ec2-user/microshift/_output/bin'
Tools:
 - awscli
 - controller-gen
 - golangci-lint
 - gomplate
 - govulncheck
 - hadolint
 - kuttl
 - lichen
 - robotframework
 - shellcheck
 - yq
ERROR: Unknown tool: "awscli"

See this error as an example.

/test microshift-metal-cache-arm

/lgtm

/cherry-pick ?


Customize font and color?

Like UIDatePickerView, this is backed by a UIPickerView which does not have public methods to change the font or colour.


CORS on localhost

This is indirectly connected with Webpack as we are using Angular.
The issue is that after updating Angular 12 to Angular 13 we are no longer able to access our app served on localhost from outside of it, the request is being blocked by CORS. Basically when switching from Webpack 4 to Webpack 5.
this is what worked in Angular 12
ng s --project app-search --disable-host-check --port 4207 --deploy-url https://localhost:4207/ --ssl --public-host https://localhost:4207/
and stopped working in Angular 13.
We were looking into the docs around Webpack 4 and Webpack 5 and we can see that the disableHostCheck flag is removed from webpack 5 doc but is in webpack 4 docs. We tried playing around with disable host check and allowed hosts passing it to ng serve per official docs (https://angular.io/cli/serve) but with no luck.
Not quite sure if this is a bug or do the flags have to be passed in a different way to enable the same behavior that we had before.
Happy to provide more info if needed.
Any help or info is much appreciated.

as I see disableHostCheck was removed you need to set allowedHosts: "all"

Thanks for the prompt response.

for anyone stumbling across this seems the issue is in webpack-dev-server: https://github.com/webpack/webpack-dev-server/issues/4180


refactor(jenkins/pipelines/ci): add retry for fetch groovy lib

add options:

--retry 3: retry 3 times
--retry-delay 5: delay 5 seconds between retries
--retry-connrefused: retry when connection refused
--fail: return non-zero exit code when fetched failed.

test link https://ci.pingcap.net/job/tidb_ghpr_build/74792/

curl option --retry-connrefused need version greater curl 7.52.0.

master node(jenkins server) is ok


Adding implementation details to tags in globalFlux

What is the change?
This is adding details of the implementation in the .. impl:: tags within globalFlux. Most of them are adding paragraphs. A couple of them I simply put the Notes section in the doctring under the tag to avoid duplicate information.
Why is the change being made?
This is part of on-going QA work.

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify that the new/changed code works.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[x] The release notes (location doc/release/0.X.rst) are up-to-date with any important changes.
[x] The documentation is still up-to-date in the doc folder.
[ ] If any requirements were affected, mention it in the release notes.
[x] The dependencies are still up-to-date in pyproject.toml.

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


Script timesout (No such file or directory) while running with bash command

System: fedora 35
Trying to run the script without +x execution permission using bash command.
bash wait-for-it.sh google.com:80
timeout: failed to run command ‘wait-for-it.sh’: No such file or directory
wait-for-it.sh: timeout occurred after waiting 15 seconds for google.com:80

(The error message is same for above case even if execution permission is given)
But it works when directly executed given +x.
./wait-for-it.sh google.com:80
wait-for-it.sh: waiting 15 seconds for google.com:80
wait-for-it.sh: google.com:80 is available after 1 seconds

Any way I can use the script without giving it execution permission?

It works when you give +x permissions and use ./ because that is the intended behavior by the system.
Check the accepted answer for details: Why do I need to type ./ before executing a program in the current directory?

I get that, about the path issue. Turns out the script calls itself during execution, for that it needs the +x permission.


Create reusable Test Cases / Test Case Template.

Is your feature request related to a problem? Please describe.
Data Quality.
Currently I have to create a test case for each table or column even if I am testing for the same things. For example:

Email or Date format with Regex
Columns across different systems and tables only containing values of valid customer types.
To have to recreate the same test logic for the identical test case is time consuming.

Describe the solution you'd like
It would be very convenient to be able to set Template Test Cases and then simply chose like

must - instead of selecting a test definition one can select a predefined test case template "email format" or "customer group".
should - from a test case template - choose columns/tables.
nice -  select  tables and columns based on their tags. For example tag columns with "email" and then one is able to run test case template across all columns tagged - "email" or columns tagged with a glossary term "customer group".

One added benefit  of this  would be being able to track the DQ of the same logical attribute across systems. For example: to see the quality of "customer group" attribute, I could look into the quality results for the template test cases, rather than individual test cases. Even if they are done as part of different Test Suites. Really good view for Data Stewards.
Describe alternatives you've considered
Just an idea
Additional context
Add any other context or screenshots about the feature request here.

+1
In my case we are reconciliating 2 different sources for a machine learning model training and inference respectively. The two tables will have different lineages, but we want them to conform to the same suite of checks / tests / constraints.
Having reusable test cases - or being able to reflect the tests of one table onto another - would be a big helper for us.


fix: default pulumi to last known working version.

Codecov Report
Base: 28.49% // Head: 28.49% // No change to project coverage :thumbsup:

Coverage data is based on head (f891121) compared to base (9aa5f1d).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #452   +/-   ##
========================================
  Coverage    28.49%   28.49%
========================================
  Files           54       54
  Lines         4944     4944
========================================
  Hits          1409     1409
  Misses        3402     3402
  Partials       133      133

Flag
Coverage Δ

unittests
28.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Add isNullish and isNonNullish Utility Functions

Summary
This pull request introduces two new utility functions, isNullish and isNonNullish, to the Radash library. These functions are intended to provide a more intuitive and expressive way to handle nullish values in JavaScript, complementing the existing utility functions in Radash.
Motivation
In JavaScript, handling nullish values (null and undefined) can often be verbose and error-prone, especially when dealing with deep object structures or complex logic. The isNullish function provides a clear, concise way to check for nullish values, while isNonNullish allows for the inverse, checking for values that are not nullish. These additions aim to improve developer experience and code readability, aligning with Radash's goal of offering practical and straightforward utility functions.
Details

isNullish(value: any): boolean: Returns true if the value is null or undefined, otherwise returns false.
isNonNullish(value: any): boolean: Returns true if the value is not null and not undefined, otherwise returns false.

Implementation
The implementation of these functions is straightforward, relying on strict equality checks to determine if a value is nullish or not. This approach ensures maximum performance and minimal overhead, in line with the efficiency goals of Radash.
Use Cases
These functions can be particularly useful in scenarios where distinguishing between "no value" and falsy values like 0, '', or false is important. For example, in configuration objects, function parameters, or API response parsing, where a nullish value might signify the absence of a meaningful value.
Tests
Comprehensive unit tests have been added to ensure the reliability and correctness of these functions under various scenarios.
Changes Made

Added the isNullish function along with its unit tests.
Added the isNonNullish function along with its unit tests.
Updated the documentation to include these new functions, providing examples of their use.

How to Test

Pull this branch locally.
Run npm install to install any new dependencies (if any).
Run npm run test to execute the unit tests for isNullish and isNonNullish. All tests should pass.
Optionally, integrate these functions into a sample project to see them in action.

Potential Impact
These changes are additive and fully backward compatible. They introduce no breaking changes to existing functionality in Radash.
I believe these additions will make working with nullish values in JavaScript more intuitive and expressive for Radash users. I look forward to your feedback and any suggestions for improvement!

Summary
This pull request introduces a new utility function, isKeyOf, to the Radash library. This function enhances type safety and developer experience by providing a more robust way to check for the existence of keys in objects, especially useful in TypeScript for narrowing down types and ensuring compile-time safety.
Motivation
Current JavaScript operations like key in object or Reflect.has(object, key) merely return a boolean indicating whether a key exists in an object, without any type narrowing. This can be limiting in TypeScript, especially when working with enums and needing to ensure a string is a valid key of a specific object type. The isKeyOf function addresses this by not only checking for the existence of a key but also narrowing down the type, thereby enhancing type safety and providing more utility in scenarios involving enums and type-checking.
Implementation

isKeyOf<T>(obj: T, key: PropertyKey): key is keyof T: The function takes an object obj of type T and a key, and returns true if the key exists in the object, with TypeScript being able to infer the narrowed type.

For example, using isKeyOf with an enum:
enum Values {
   A = 'a',
   B = 'b',
}
const result = isKeyOf(Values, 'A'); // result is true, with 'A' being recognized as a valid key of `Values`.

Use Cases
isKeyOf is particularly useful for validating and narrowing types when dealing with enums or when keys of an object must be validated in a type-safe manner. It's an excellent tool for TypeScript developers looking to leverage the language's type system more effectively.
Tests
Comprehensive tests have been added to verify the functionality of isKeyOf across various scenarios, ensuring its reliability and type-safety in practical use cases.
Changes Made

Implemented the isKeyOf function, designed to check for key existence and narrow down the type in TypeScript.
Added unit tests for isKeyOf covering various use cases and ensuring its effectiveness with enums and object types.
Updated documentation to include isKeyOf, illustrating its benefits and how to integrate it into TypeScript projects.

How to Test

Fetch this branch and run npm install to ensure all dependencies are up to date.
Execute npm run test to run the suite of unit tests for the isKeyOf function. All tests should pass, demonstrating its reliability.
For a hands-on test, integrate isKeyOf into a TypeScript project and use it for type checking and narrowing with enums or object keys.

Potential Impact
This addition is non-breaking and complements the existing suite of utility functions in Radash. It introduces a type-safe way to check for the presence of keys in objects, specifically designed to leverage TypeScript's capabilities.
This function aligns with Radash's goal of providing practical, straightforward utilities that enhance developer productivity and code safety. I'm excited to see how isKeyOf can benefit TypeScript users in the Radash community and look forward to any feedback or suggestions for improvement!

Hey @shtse8, I appreciate your thoroughness here in explaining your PR! Would you be interested in joining us over at the Radashi fork, where we're bringing Radash to new heights? I see you now have your own utility library called XDash, but it'd be great to combine our skills. We already have a team of about 6 people, and it'd be great to have you with us. We've made a lot of improvements already. 😄
P.S. You can read my post to learn more about the differences between Radashi and its predecessor.


Edit note

ACs

Edit by option menu in file manager
Edit by notebook editor
Edit by note previewer action menu

[ ] Edit by option menu in file manager
[ ] Edit by notebook editor
[ ] Edit by note previewer action menu


Xeno is not working

What I need to do?

I fix it by launching in compatible with Windows 8

Uhh, now it not working again, I mean there is no Clients :(

it works but idk when it inject and when not QwQ


installed beta8 ( + win10 ) -> no settings save

When i start the application (admin or not admin privs both broken), the settings menu doesn't save for the apply button.
If you make a settings file (like in the zip), and give the settings.exe admin rights to run, its do the job.
Othervise its greatly works on win10 too I think its just a small packing problem.

I'll get Windows 10 installed so I can check it out... Thanks for letting me know!

Hey tg44, if you navigate to %APPDATA% in the file explorer, has a 3RVX entry been created? (should be a folder named "3RVX" with Settings.xml inside). I just upgraded to Windows 10 and I'm not seeing this bug, but it might've been because of the upgrade...
When you hit 'Apply' in the settings app, does the program reload (show the volume OSD?)

There is an xml with language settings, and if I rename the xml in the pf its now write the xml in the app data when I hit apply.
Possible other problems: my username in this computer contains 'ö'
And I tought its fixed itself but the problem is not the writeing. Its not loading back. When I hit apply the xml get a new timestamp every time. If I close the window and open it again it didn't preload the beforely checked options, and when I hit apply again its save a nearly empty (but proper) settings file.
The Clog doesn't write to the cmd output :( How coud I force to see the clog outputs?
I checked the source I have no idea why it can write but couldn't read. I give myself an hour to check this :)

The problem is definitely the "ö" in the user name. See the pull req and the link in it. I think it will fix the problem and will not generate new ones :)

Hey Gergő,
Thank you so much for tracking this down and figuring out what was going on. Reading the XML files should not have been using those StringUtils functions anyway, so I updated the code to open and read the files properly now (sometimes unicode is hard, I guess :-))
Would you mind pulling in the changes I made and testing it out (I will as well, shortly)? We need to keep the UTF-8 code page in Narrow() for reading XML data (which is actually formatted as UTF-8, unlike the string I was passing to it earlier)

It's working here.

Awesome, thank you!


Modification of CP3.0 Install Script for Separation of Permissions

Removed all cluster-admin actions from setup_tenant.sh

Namespace and RBAC creation will be handled by a different script (e.g authorize-ns.sh) or olm-utils command
Checks CSV instead of sub for cert manager (since cert manager is installed with all-namespace mode)

Enhanced error checking and reporting
setup_singleton.sh can now check, instead of install, cert manager and licensing service

/lgtm


Where can I find the AzureAD module source code?

I've been trying to find the Azure AD module source code, but links from the PowerShell Gallery take me to the docs, and the docs repo doesn't mention where the source code can be found. Is Azure AD still closed source?

I'd like to know the answer to this as well.

Thank you for your issue.  The AzureAD module source code is currently not public.
Thank you.


Passport not recognizing session as authorized only when multiple workers

Hi I'm back ONE last time!! I've successful moved everything over to sc and changed everything that needs to be changed to pub/sub channels. However, I've encountered an error that I simply cannot solve. I spent the past day googling and trying things to fix it before making another issue, but I simply couldn't get it. The system I had worked out was great until I started more than 1 worker. It seems like passport isnt succesfully latching the user data to the session through the serialize/deserialize funcitons (I cant remember which one does that rn)
Here is how the passport workflow should go (I can link to a better visual one if you need it):
User goes to homepage, the app checks if they are authenticated (by checking if req.user exists or not) if not, it sends them to steam, you login on steam, if it fails you are taken right back to the steam page to relogin (which is what happens every time I log in with more than 1 worker), if it's successful it runs the serialize function then takes you to url/steam/return, there another cookie is set so i can utilize it from socket clusters, and then its redirected to the homepage now successfully authenticated and the deserialize function runs in between one of these last few steps.
This works great when it's just one worker, but as soon as I start 2 workers it stops working. Where it stops working is when they go to steam to login, afters successfully logging in, the session is serialized, but the user is still sent back to the failure url. I know this is half a passport issue and not really your domain, but its also half a socket clusters issue because it only happens when multiple workers exist.
I can get around this SOMETIMES, its hardly reproducible, by deleting the session cookie while on the steam login page, right before I log in. But sometimes this doesn't work, and when I launch 4 workers, it never works.
Here is my code, any help or advice you can give is greatly appreciated, but I understand if this simply isn't your alley.
var app = express();

var httpServer = worker.httpServer;
var io = worker.scServer;

healthChecker.attach(worker, app);
httpServer.on('request', app);

passport.serializeUser(function(user, done) {
	console.log(user.id + ' serialize');
	done(null, user);
});

passport.deserializeUser(function(user, done) {
	console.log('DESERIALIZE');
	done(null, user);
});

passport.use(new SteamStrategy({
	returnURL: 'http://1xxxxxx/auth/steam/return',
	realm: 'http://xxxxxx/',
	apiKey: 'xxxxxxxxxxxxxxxxxxxxxxxxxx'},
	function(identifier, profile, done) {
		process.nextTick(function (){
			return done(null, profile);
		});
	}
));

app.use(session({
	secret: 'secret',
	resave: true,
	saveUninitialized: true,
}));

app.use(passport.initialize());
app.use(passport.session());

app.use('/public', express.static('public'));

app.get('/auth/steam', passport.authenticate('steam', {failureRedirect: '/login'}), function(req, res) {});

app.get('/auth/steam/return', passport.authenticate('steam', { failureRedirect: '/' }), function(req, res) {
	var cookie = {
		'id':req.user.id,
		'displayName':req.user.displayName
	};
	worker.auth.signToken(cookie, worker.options.authKey, null, function (err, token) {
		res.cookie('GameID', token);
		res.redirect('/');
	});
});

app.get('/login', function(req, res){});

app.get('/', ensureAuthenticated, function (req, res) {
	res.sendFile(__dirname + '/public/index.html');
});

function ensureAuthenticated(req, res, next) {
	console.log("checking Auth");
	if (req.user){
		return next();
	} else {
		res.redirect('/auth/steam');
	}
}

@FantomHD That's because passport isn't designed to scale across multiple processes/servers by default.
I can think of two options:

Use a JWT solution instead of passport for authentication over HTTP.
Store the passport session data in an external process (e.g. Redis) instead of storing it in memory.

Each worker process has independent memory so if you store data in one worker, it will now show up in the other worker - Data that you need to share between multiple workers needs to be stored in a database/datastore.
Or maybe there is a way to make passport work better with JWT.
This articles says that you can set the session feature to false: https://blog.hyphe.me/token-based-authentication-with-node/
I found this article about storing password sessions in Redis: https://blog.risingstack.com/node-hero-node-js-authentication-passport-js/
The ideal way would be to not store any session data on the backend (just use the signed JWT token for all your authentication needs) - That is the simplest and most scalable approach by far - But if passport doesn't allow you to use that approach, then you may have to use Redis. Or maybe you can find another way to to steam authentication without using passport at all (just handle it yourself inside express and create the JWT token yourself).

Okay, I'm switching to using JWT to authenticate. How can I verify the sc token I set earlier with jwt?
I have this to sign the token i pull from socket.authToken:
worker.auth.signToken(cookie, worker.options.authKey, null, function (err, token) {
		res.cookie('GameID', token);
		res.redirect('/');
	});

inside my ensure authenticated function, im trying to use jwt to verify that cookie, how can I go about that? Or do I have to set a whole other cookie for this?

never mind. I had this: jwt.verify(req.cookies.GameID, worker.options.authKey, function(err, cookie){} to try and verify the token and it didn't work the first time, but now it does. So i guess that's it! This was super simple, dunno why i didn't use this to begin with. Now inside of my ensureAuthenticated instead of checking if req.user exists I just check if this cookie is legit lol.

Yep, or you can also use worker.auth.verifyToken(signedToken, key, options, callback) if you want to be consistent. https://github.com/SocketCluster/sc-auth/blob/master/index.js#L9


Fix installing required file bundles

Fixes https://github.com/meltano/meltano/issues/6046
Even though required was set, meltano add orchestrator airflow would look for plugins with the same namespace in discovery.yml, find the airflow file bundle there, and try to add it. Then during adding that plugin, inside the disallow_discovery_yaml context, Meltano would look for that plugin on the Hub, and not find it there (as it had been renamed in https://github.com/meltano/hub/pull/538) and fail with the error shown in https://github.com/meltano/meltano/issues/6046.
In other words: to find related plugins, Meltano only looks at discovery, and then to add those related plugins, it only looks at the Hub. So if a plugin exists in the one but not the other, things break.
Now, if required is set, namespace-based related plugins are ignored. For some reason, add_required doesn't seem to have been called at all before.

@edgarrmondragon points out this may be a dupe of https://github.com/meltano/meltano/pull/5999.

@edgarrmondragon Do you think anything from this  PR is worth saving now that #5999 is merged?

@DouweM I went ahead and removed lingering related code paths and would like to save some of the updates from this PR. I'll merge once tests pass.


[ADR] Enhance existing ADRs

I have been doing a bit of archeology...

Update the status if needed
Indicate whether it had been implemented or not
Link the JIRA ticket (and thus the PRs as they should be references in the JIRA)
Link the pull request discussing the very ADR
If possible, link mailing list discussions

Globally, we should try to link better sources of information together.

thanks a lot for these updates !

Merged


Add SandboxPurchase() in client

Finally, I know the reason of my issue - the test payment and sandbox environment.
Because I used a test user to do payment, so every payment is a test payment. The purchase query API should be /public/sandbox/purchase.
I add  SandboxPurchase() method in client.

Awesome!


Initialize timeout when creating a RethinkDB connection

Right now the connection_timeout configuration parameter is used only by MongoDBConnection. The RethinkDB driver supports a similar option called timeout, but we are not using it.

What is the expected timeout ?

is this right way of testing?
py.test -s test_connection.py
if yes 3 attempts to connect port 28015 fails after 5000ms.

@lavinasachdev3: sorry for now replying earlier, I missed the notification on GitHub
@tdsgit: AFAIK we are not testing if the timeout parameter is honored by the driver. The time should be cumulative, so if it's 3 tries, the total time is 15000ms. To run the tests you should do pytest tests/backend/rethinkdb/test_connection.py.

@vrde
tried testing mongodb connection to see what's going right with mongodb first.
but bigchaindb/backend/connection.py retaining old config values and trying to connect to rethinkdb!
tried clearing tests/pycache
the only file which has old keypairs was tests/conftest.py .

update (april 20):
the mongodb test need to be run as
pytest -q tests/backend/mongodb/test_connection.py --database-backend="mongodb"
and not as
pytest -q tests/backend/mongodb/test_connection.py
this snippet assigns tiemout to serverselectiontimeoutms
          client = pymongo.MongoClient(self.host,                                          self.port,                                          replicaset=self.replicaset,                                          serverselectiontimeoutms=self.connection_timeout,                                          ssl=self.ssl)
however in rethinkdb the parameter to be passed is timeout ( not connection_timeout)
r.connect(host=self.host, port=self.port, db=self.dbname, timeout=5000 )
mock_connect.assert_called_once_with may assert
original_connect also should pass timeout
this concludes the test scripts passing timeout to rethinkdb socket.


Use of new dataset

What dataset you need?

I want to use my dataset for my analysis. it is just a .csv file regarding Alberta's cases.
How will you use the dataset for your analysis?

(Optional) Example datasets

Thank you for your issue.
We can create JHUData instance  with local CSV files via CountryData.
https://lisphilar.github.io/covid19-sir/markdown/INSTALLATION.html#use-a-local-csv-file-which-has-the-number-of-cases
Could you share the column names and some lines of your dataset?

Hello Hirokazu
Thank you for your email. Sure, here is the data I am using: "
https://opendata.arcgis.com/datasets/3afa9ce11b8842cb889714611e6f3076_0.csv".
I will use it to create the SEIR and SEIRD modelling.
Best regards,
Ali.
On Fri, May 28, 2021 at 7:16 AM Hirokazu Takaya @.***>
wrote:

Thank you for your issue.
We can create JHUData instance with local CSV files via CountryData.
https://lisphilar.github.io/covid19-sir/markdown/INSTALLATION.html#use-a-local-csv-file-which-has-the-number-of-cases
Could you share the column names and some lines of your dataset?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lisphilar/covid19-sir/issues/791#issuecomment-850411957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARPZ4UV5UFCBIX27JDWDOTTTP6JUNANCNFSM45UXGOUQ
.

At the latest version 2.19.2, we can do like this.
https://gist.github.com/lisphilar/a1c7f8bc03eaf9fddeda8d0dd4142e47
Note at the latest version:

It will be necessary to set population values manually. (Cell 30 of the notebook)
It will be necessary to convert time "2021-05-24 12:00:00+00:00" to "2020-01-25" manually. (Cell 23)

-> As new features, CovsirPhy may handle them internally.
FYI.
We can read all comments of this issue at https://github.com/lisphilar/covid19-sir/issues/791
E-mails are just a notification.

With #793 (only development version 2.20.2-alpha at this time), time will be rounded to date.
(i.e. in the next stable version 2.21.0, we will skip df["Date"] = df["Date"].dt.date.)
https://gist.github.com/lisphilar/fdc473719bdf005bc46e74d7208e666d

Seems staled.

Hello Hirokazu
Do you have any idea how to drow the best fit for infected cases? I tried
the graph below, but it looks different from yours.
Thank you very much in advance.
Best regards,
Ali
[image: image.png]
On Sun, Jun 13, 2021 at 6:31 AM Hirokazu Takaya @.***>
wrote:

Seems staled.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lisphilar/covid19-sir/issues/791#issuecomment-860203411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARPZ4UXHUB5BR72Z3FF2YFLTSSQJNANCNFSM45UXGOUQ
.

@aboukric ,
Thank you for reporting, but could create a new issue using your browser with the codes, outputs and version number? When we reply to GitHub notification e-mails, attached files are removed automatically.

Please use your browser when you have attached files.


Add DockerHealthcheckWaitStrategy

Adds a DockerHealthcheckWaitStrategy that relies on the built-in Docker healthcheck.
TBH I was not sure what's our current template for creating WaitStrategy tests, so I tried to keep it as simple as possible. The write_file_and_loop.sh could surely be included into the Dockerfile CMD, but this way it seemed to be a bit easier for debugging purposes.
This PR also implicitly adds a isHealthy() method to the ContainerState interface.
Regarding backwards compatibility, I'm not sure how docker-java behaves if the health state is missing. docker-java itself documents to support it since Docker version 1.12.

OMG, some cached Gradle file locks in Travis...

@rnorth You are right about the documentation, I'll try to come up with something.


cannot install via gradle

hello, im newbie on android studio.
i try to install via gradle but its always fail
this is the event log:
Information:Gradle tasks [:app:generateDebugSources, :app:generateDebugAndroidTestSources, :app:mockableAndroidJar, :app:prepareDebugUnitTestDependencies]
Error:Execution failed for task ':app:processDebugManifest'.

Manifest merger failed : Attribute meta-data#android.support.VERSION@value value=(26.0.0-alpha1) from [com.android.support:appcompat-v7:26.0.0-alpha1] AndroidManifest.xml:27:9-38
is also present at [com.android.support:recyclerview-v7:25.3.1] AndroidManifest.xml:24:9-31 value=(25.3.1).
Suggestion: add 'tools:replace="android:value"' to  element at AndroidManifest.xml:25:5-27:41 to override.
Information:BUILD FAILED
Information:Total time: 5.88 secs
Information:1 error
Information:0 warnings
Information:See complete output in console

@setiawanfredy Just try to add a string tools:replace = "android:value" to your application tag in the app's manifest. Like this
 <application         android:name=".ApplicationEx_"         android:allowBackup="true"         android:icon="@mipmap/ic_launcher"         android:label="@string/app_name"         android:supportsRtl="true"         android:theme="@style/AppTheme"         tools:replace="android:value">
Do it for all such errors you'll get (there could be a bunch of such errors in sequence).

@setiawanfredy also try to add tools:replace="android:value" to the <manifest> element instead of <application>


Update ADLINK IPC.

Update the description of ADLINK IPC.

@evshary  Hi, thank you for your update.


Python 2.6 compatibility

Tests are failing for Python 2.6 on Travis. Drop support or figure out what's wrong.

Drop support.

Unsupported since 2013-10-29
https://snarky.ca/stop-using-python-2-6/
http://www.curiousefficiency.org/posts/2015/04/stop-supporting-python26.html
http://www.python3statement.org
Current pip 9 deprecates Python 2.6 support, pip 10 won't support it (https://github.com/pypa/pip/issues/3955)
Not much PyPI traffic (June 2016) https://github.com/pypa/pip/issues/3796

@hugovk Agreed. Dropping.


RP-EF read and update related data, concurrency

Internal review URL Reading related data
Updating related data
Concurrency
fixes #4739
Fixes #4740

@ajcvickers hoping you can review this too. You're previous review was fantastic. cc @divega
No hurry. Be sure to use the Internal review URL

@ajcvickers hold off for now. I've got customer feedback I'm incorporating. It will probably be ready to review Thursday 9 Nov.

@ajcvickers let me know when you can review this  Parts 6 & 7

Read Related data
Update related data

cc @divega

@ajcvickers @divega please review
Use the links below to review.
Internal review URL Reading related data
Updating related data
Concurrency

@ajcvickers ping

@divega  address all your comments. Removed entire section on locking given the tutorial doesn't use locking.  Removed all the incorrect faster statements.

refers to EF Core as just "EF" a lot.

Ouch. I must have fixed 50 occurrences in the series.

This has a lot of overlap with conceptual topics of EF Core. There is a trade off between having all this information here and the risk of it being wrong or becoming outdated. It also makes it very long. Should we tune it down to be shorter and less deep?

This series is a copy of the EF/MVC series. I changed from MVC to Razor Pages, shortened each section and removed the last 2 tutorials. The original series had a good review from your team (MVC3/EF4) and has since been updated without much review (although we did ask for reviews :))
The series is extremely popular with customers - currently getting 125K+ views/month. The MVC5/EF6 series gets slightly better views. Once more customers migrate to .NET Core we expect readership to double.
I've never seen any tutorial MS produced get reviews as good as this series.
I'd like to:

Keep  this series for now
Work on trimming it down (and pointing to the EF docs where I trim it down).
Retire the ASP.NET Core 2 MVC/EF Core 2 series
Consider retiring this series with .NET Core 3

I've got alot of bugs to fix in the EF Core/MVC Core version from feedback on this version.

@divega Let me know if my changes work. I'm happy to trade your team 3 hours of work for every hour you put in this series.

@Rick-Anderson Diego's comments aside, this looks good to me.

although we did ask for reviews :)

If I am not mistaken, I did participate in a review of this before, once or maybe even twice. It is only every time we review it we find new stuff :smile_cat:, but I guess some things are new too.
Anyway, thanks for your patience @Rick-Anderson.


Settings panel unavailable in Idea 2016.1.1 ?

Am I missing something, or is the plugin configuration (settings) panel is not available anymore?

Not missing anything - found the issue - working on a solution

Opened a PR, can you please have a look?


Wrapper support

I'm trying to use this plugin with DFP and we often have wrappers. Can you please add support for wrappers?

Hello,
Regarding this reply
"The issue here (if you look at the network requests) is that the test player is requesting the VAST file twice, and DFP does this really fun thing where subsequent requests with the same "correlator" value lead to an empty file. So, on the second request, DFP returns an empty file, the player says "ok, well then I guess I'll just play the video"."
Please let me know if there is a solution to this.
According to this page: https://support.google.com/dfp_premium/answer/1068325?hl=en
"A random number, typically the timestamp of the page view, that's shared by multiple requests coming from the same page view. If you're not using the Google IMA SDK, see note below.* Correlator is used to implement competitive exclusions, including in cookieless environments."
Thank you


Use external UDN docker network

In my use case, I want to start several containers that handle my testing infrastructure: test drivers, logging, monitoring, alerting, etc. I want to use blockade to start and manipulate a set of nodes under test.
I want to start and destroy the blockade network between different tests. The problem is that blockade tries and fails to remove the user-defined network because my test infra is still connected to that network. Then it raises an exception and stops. Also, it creates a brand new network with the same name when I start blockade. So I've got multiple networks with the same name now.
I think one solution is to use the idea of an external network from docker-compose. I want to tell blockade to use an externally defined network, then it should not create and destroy it every time. It should just work on the containers. In docker-compose I'd do this:
networks:
  blockade:
    external:
      name: testinfra_net

I think this is a good idea. I don't have an immediate need for it, but will keep it in mind for future work in this area. In the meantime, pull requests are welcome. :-)


Set up UmpleOnline to be remotely hosted on an elastic cloud system

UmpleOnline is hosted at the University of Ottawa on cruise.uottawa.site.ca, but as it grows we need to set it up to work on a more powerful cloud platform.
One option is Amazon EC3
Another option is Elastic Beanstalk with Linode
Installing the Docker image might be the best option, but we would have to ensure it is able to run the CRUD interface (which requires a better php setup). We would also have to ensure that storage saved by the image is persistent.
Existing UmpleOnline work done by users would have to be moved over.
Links to the cruise site would have to be redirected when this goes live.

Closing this, since we are satisfactorily hosted on ComputeCanada


Fix pkgsrc text.

They prefer lowercase for pkgsrc.
Pkgsrc.se is only a search engine for pkgsrc, not the name of the
project.
They support also Linux, MacOS, Minix, etc. So, not only for BSDs.

Thank you!


fix: add renku cli to PATH in rstudio

Fixes renku not being present in the PATH of rstudio images.

This project: https://renkulab.io/projects/tasko.olevski/test-r-project-1 uses the image built in this repo. And renku cli works in both rstudio and jupyterlab.

if I'm reading the R Dockerfile correctly, we never actually install renku in there. Should we copy it from the python base image like we do the other python packages here? I guess all that is needed is something like
COPY --from renku_base /home/jovyan/.renku /home/rstudio/.renku

if I'm reading the R Dockerfile correctly, we never actually install renku in there. Should we copy it from the python base image like we do the other python packages here? I guess all that is needed is something like
COPY --from renku_base /home/jovyan/.renku /home/rstudio/.renku

Rok what you describe does happen in the Dockerfile here: https://github.com/SwissDataScienceCenter/renkulab-docker/pull/232/files#diff-0905cf205d369bd74981475a9ea32b552da07810e4010c9936058a9837a6885eR79.
Maybe I misunderstand what you mean though.

I verified that the cli is available in that image :+1:

@olevski you're totally right I missed it because of the line break 🤦 !!

@rokroskar @gavin-k-lee if you are done testing/reviewing can you approve please?


Optionally return full role arn including path during token verification

What this PR does / why we need it: When creating EKS nodegroups where the node's IAM role contains a path, the nodes fail to join the cluster. This adds the ability to look up a role's full ARN including the path and return that when verifying a token.
The reason this is needed is because the assume-role ARN that is returned by sts.get-caller-idendentity does not include the IAM role's path.
I've added a "role cache" which calles iam.list-roles and stores a mapping between the roles unique role ID and the full ARN, as the UserID returned by sts.get-caller-identity contains the roles unique ID therefore looking up the existing role's correct ARN is possible.
This does require the clusters IAM role to have the iam:ListRoles action. If the role cache gets an error when calling list roles and its not a transient error, it'll disable all future lookups and the functionality will revert to the original behaviour. I figured it would be better to make this more an opt in rather than a requirement as else it might just break nodes joining a cluster whenever its released.
Which issue(s) this PR fixes:
Fixes #268
Fixes #153

@dims / @nckturner / @nnmin-aws can we get a review on this please?

/test

/test all

Thanks @nnmin-aws the tests seem to have passed, what now?

@nnmin-aws @dims @nckturner what else is required for this PR? There's been no progress for nearly 2 months, some guidance would be appreciated.

cc @cartermckinnon

@terrycain @dims apology for the delay! i am so sorry about it. We were hesitating at the change because it requires additional invocation of aws IAM and might increase latency of authN.
There is another PR actually merged https://github.com/kubernetes-sigs/aws-iam-authenticator/pull/670/files#diff-5de38ab8d77d935766314beeb9a9e3b7165eb470e656dbb5e4fdf9bb60ae59de. which removed the path from the arn comparison.
Could you please kindly pull latest release and verify if it works for your scenario? If it doesn't, we can check this PR again.
Thank you!