[BUG] Not supported to Nvidia 4090

Please use Discussions to ask questions.
Describe the bug
When I install torchdistill==0.3.3 and torchvision==0.13.1, it did not support my cuda version (12.0).
I assume the problem is the torch library is supporting my cuda version.
Environment (please complete the following information):

OS: Ubuntu 20.04 LTS
Python ver: 3.7
torchdistill ver: v0.3.3

Additional context
Is it possible if you can update some of the libraries so that those who are using the new GPU can also interact with your repo directly? Thank you.

Hi @allent4n
To use torchdistill with some recent GPUs right now, you have two options:

Install torchdistill from the repository (Do not use scripts/configs under legacy/ in this case)
Create a conda environment, install older version of CUDA that is compatible with  torchvision==0.13.1, and then pip3 install torchdistill

As you can see in commit logs after v0.3.3, the next release of torchdistill is almost ready, but the documentation is work in progress and I cannot complete it soon because it's a very busy season for submitting/reviewing papers.
If either the 1st or 2nd options does not work for you, feel free to reopen this issue and share more details.


feat(Delete Image): delete photos in gallery

Functionality:
Implemented Delete Image feature for admin
Solution:
Implemented Delete Image feature for admin
Risk level:

[ ] high
[X] medium
[ ] low

How to test:
Delete any uploaded image in the admin panel

Change the Target branch from Main to deploy

Change the Target branch from Main to deploy

Done !!


AVVideoMaxKeyFrameIntervalKey property is not about key frames

Looking at how AVVideoMaxKeyFrameIntervalKey is used, it's about frame rate and not key frames: https://github.com/rs/SDAVAssetExportSession/blob/master/SDAVAssetExportSession.m#L291-L295

Fixed with https://github.com/rs/SDAVAssetExportSession/pull/62


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.19 to before v5.4.180",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=948d4f214fde43743c57aae0c708bff44f6345f2",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=c99e66350ca01d2e7836d80f6f451c0e648bcbe0"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "948d4f214fde43743c57aae0c708bff44f6345f2",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "c99e66350ca01d2e7836d80f6f451c0e648bcbe0",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "veth: fix races around rq->rx_notify_masked\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.4.180 by commit c99e66350ca01d2e7836d80f6f451c0e648bcbe0, it was introduced in version v4.19 by commit 948d4f214fde43743c57aae0c708bff44f6345f2. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000341


Tree Structure for Inverted Index, first draft

[ ] Implement a radix tree
[ ] Implement data insertion on tree (Insert TokenFilterOutput)
[ ] Print tree (Visualize)

PrintTrie() Result.

Trie Root: b
Parent: b, Children: e, Child DocumentIDs: []
Parent: e, Children: t, Child DocumentIDs: []
Parent: t, Children: a, Child DocumentIDs: [2:Document2]
Parent: b, Children: l, Child DocumentIDs: []
Parent: l, Children: u, Child DocumentIDs: []
Parent: u, Children: e, Child DocumentIDs: [2:Document2]

Trie Root: t
Parent: t, Children: e, Child DocumentIDs: []
Parent: e, Children: t, Child DocumentIDs: []
Parent: t, Children: a, Child DocumentIDs: [2:Document2]

Trie Root: g
Parent: g, Children: a, Child DocumentIDs: []
Parent: a, Children: m, Child DocumentIDs: []
Parent: m, Children: a, Child DocumentIDs: [2:Document2]
PASS
ok      github.com/manorie/gosse-core   0.012s


ignoreLASTEXITCODE - in the arguments section this is ignore$LASTEXITCODE (causes the build to fail)

trying out this option and don't have it working yet but noticed the item in the arguments section is not spelled correctly.
Should be: ignoreLASTEXITCODE
Currently: ignore$LASTEXITCODE

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

ID: a0e444c2-69c6-4d71-6fd0-94d0f4097d3a
Version Independent ID: 3a727833-1cc3-c2fb-c5b3-22605eee2721
Content: PowerShell task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/powershell.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

@craig-martin Thank you for fixing that.

This is fixed and should be live soon.

Thanks!


Generating count histogram from the raw data

With this issue, the raw data that is returned from the backend is taken as the source to calculate the count histogram (instead of the probabilities histogram). Also the memory data field is filled with the raw data values.
A special case is when number_of_shots = 1. In this case the backend doesn't return raw_data.
With number_of_shots = 1, there must be one value in the counts histogram (with count = 1), and the same single value in memory data. To determine this value a random float is generated in the range [0, 1). With this random number the value from the probabilities histogram is taken where the added probabilities is greater this random number.
Also the unit tests are adjusted because the single shot unit test now delivers valid data.
Some minor docstring adjustments for the other part of the file.

Coverage remained the same at 100.0% when pulling 1ee3f37aa302ac4a3a558da9845b738994ef9bf8 on bug/DEM-766/histogram_counts into 2bf7c882bdca69918a3dd87ca3b722947a0a5cb9 on dev.

The OrderedDict of project parameter I'll have to do another time when type hinting it together with Api


net: call WSAGetLastError on Windows

Winsock API calls don't set errno, instead you call WSAGetLastError(). It looks like Go doesn't. Have I missed something?
https://docs.microsoft.com/en-us/windows/win32/winsock/error-codes-errno-h-errno-and-wsagetlasterror-2
This affects net.isConnError() and a fix for #35131.
func isConnError(err error) bool {
        if se, ok := err.(syscall.Errno); ok {
                return se == syscall.WSAECONNRESET || se == syscall.WSAECONNABORTED
        }
        return false

cc @bcmills @ianlancetaylor @iWdGo @alexbrainman @zx2c4 @mattn @jstarks
@gopherbot add OS-Windows

WSAGetLastError is just a synonym for GetLastError.  At least on amd64 the Go runtime cheats by inlining GetLastError to just load directly from %gs.  See asmstdcall and getlasterror in runtime/sys_windows_amd64.s.
If we get this wrong somewhere lets look at that specific case, but I'm going to close this general issue because there shouldn't be a general problem here.

Ah, sorry; that's not mentioned in the MS docs, but described in
https://stackoverflow.com/questions/15586224/is-wsagetlasterror-just-an-alias-for-getlasterror


gstreamer1.0-plugins-nvvideo4linux2: fix assertion on nvv4l2decoder a…

gstreamer1.0-plugins-nvvideo4linux2: fix assertion on nvv4l2decoder allocation query
Avoid run the allocation query on the decoder source pad if the caps has not yet been negotiated.
Runs the the allocation query with a null caps causes an assert and this patch fix this.
It is intended to fix the following:
export G_DEBUG=fatal-criticals GST_DEBUG=7 GST_DEBUG_FILE=/tmp/gst.log
gst-launch-1.0 --gst-debug-no-color videotestsrc ! nvvidconv ! nvv4l2h264enc ! h264parse ! nvv4l2decoder ! nvvidconv ! fakesink
(gst-launch-1.0:4720): GStreamer-CRITICAL **: 16:00:27.748: gst_mini_object_unref: assertion 'mini_object != NULL' failed
Trace/breakpoint trap
Signed-off-by: Jose Quaresma<EMAIL_ADDRESS>

Thanks!


173 | Binary Search Tree Iterator | C++

Title - 173. Binary Search Tree Iterator
What will change - adding a new file with a solution for 173. Binary Search Tree Iterator in C++
Type of Issue -

[x] Adding New Code

Programming Language

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

please assign this issue to me


Remove redundant code related to content type

Related Issues

closes #5081
Short Description and Why It's Useful

Remove redundant code related to content-type while calling TaskQueue.execute/queue method. Still if anyone want to override contentType can pass content-type header while calling TaskQueue method
Screenshots of Visual Changes before/after (if There Are Any)

Which Environment This Relates To
Check your case. In case of any doubts please read about Release Cycle

[x] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[ ] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing Vue Storefront sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.7 out of 12 committers have signed the CLA.:white_check_mark: gibkigonzo:white_check_mark: dixitdeepak:white_check_mark: Fifciu:white_check_mark: filrak:white_check_mark: cewald:white_check_mark: andrzejewsky:white_check_mark: ymaheshwari1:x: didkan:x: ianaya89:x: michasik:x: Jimmerz28:x: jahvimichasik 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.

sounds good, I am also thinking to add Accept header as well in this PR

So would you like to add Accept: application/json to them? I am not sure if it is a good idea.

Please tell me why would you like to add it?
What are the benefits & possible disadvantages?
Do you see any potential future problems caused by Accept?

Please tell me why would you like to add it?

While modifying code I found some reference where accept header value is set, eg. UserServices.ts, ReviewServicse.ts
As we are already using the Accept header at some place, so to make it consistent its good to have it at common place

What are the benefits & possible disadvantages?

The Accept request-header field can be used to specify certain media types which are acceptable for the response. It tells the server which object types the client can handle
Server side api may return response based on request accept header type as well.
https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html

Do you see any potential future problems caused by Accept?
I can't imagine any potential future problem caused by Accept.

Ok, then let's implement this @dixitdeepak

@Fifciu Done

@Fifciu sounds good, Will update the PR with method = 'GET'

@Fifciu Done, I did git rebase, Please let me know if its looks good to you.

Closing the PR, as git rebase modified lots of files.
Created new PR
https://github.com/DivanteLtd/vue-storefront/pull/5209


Shelley address example/test

I'm trying to use this library to duplicate the output of the https://github.com/input-output-hk/cardano-addresses module for a Fund5 project (https://github.com/reaster/cardano_wallet_sdk). Any chance you could help me get this working? I could setup a test using, say a shelley_keys.json file, for example.

I eventually fumbled my way through this, but the March 13th commits were still helpful. Fantastic library! I couldn't have completed my Fund5 project without it.


OCSP verification failed for a lot of domains

mc.yandex.ru
www.google-analytics.com
www.googletagmanager.com

Verification doesn't fail on Linux, we may add code for loading certs from keystore to proxy's verifier.

Merged, core/pull-requests/61


Capitalisation of xAPI terms and concepts in documentation

Some words in the documentation have a specific xAPI meaning, for example "verb", "object", and so on. Sometimes it is not crystal clear in the documentation that the word is being used with that specific technical meaning.
We propose to capitalise the relevant xAPI terms and concept throughout v1.0, so that the documentation is clearer. This can be done pretty quickly.
For example: "The significance of data in a Statement can be more clearly understood if we describe the circumstances surrounding the Statement. These circumstances are described in a Statement's Context." I believe this is much clearer with Statement and Context capitalised, than without.
Does anyone wish to raise any objections?

Completed.


Loss function of acoustic model goes to nan with nit-song070 data-set on Google Colaboratory

When I try to use NNSVS with NIT-SONG070 data-set on Google Colaboratory, I get RuntimeWarning from NumPy on Stage 1(acoustic feature generation). On stage 4, loss function of acoustic model goes to nan and fail to train acoustic model.
When I disable use_harvest feature in nnsvs/bin/conf/prepare_features/config.yaml[1], NNSVS seems to work fine(and there is a minor bug in nnsvs/data/data_source.py).
I uploaded the failed ipynb to gist.github.com[2] so please see it for detail.

https://gist.github.com/taroushirani/dbcd4a69bc6372e173854846918c9906
https://gist.github.com/taroushirani/ed14e3e0228f52f4eecac68402bf34b8

https://github.com/mmorise/World/pull/100 There were two memory issues in Harvest C++ implementation. I submitted a PR to WORLD and hope it will fix the issue.

Thank you for your rapid response. I built pyworld with World-cmake/harvest-memory-fixes and confirmed the problem is solved.  I hope this patch is merged into upstream.

Glad to know it worked! I’ll make a PR for pyworld after my fixes get merged to upstream.

The new pyworld v0.2.10 has been released and this should be solved by the new verison. Please reopn if the problem persists.


Revise the current API

Some points that have been raised:

max of a column is not a number, but a column (I understand why that is so, but it is not nice to work with)
 The map over a column works in a way that gets me totally lost. I need to specify the type of the column: but I don't know it in the first place. That's a big win for pandas, where I can map like I know it.

maybe describe method should be for whole Dataframe and describeColumn sholud take columnName and do what current describe does? @piotrMocz @kustosz ?

Case: user has a table, wants to fill nulls in each column with a mean value for that column

The thing I have very strong feelings about:
the Table class should expose the read<Format> method. Please keep in mind that we're not targetting expert programmers or even hobbyist programmers, but "data profesionals".
Table.readCSV is a method that gives you a table from a CSV file. Table.readXLS is a method that gives you a table from an Excel file. And so on. Accidentally, this is exactly how pandas does it. Advantages:

single import for Table read and for Table processing
"ideological" compatibility with pandas
hides the implementation details and makes the API dead simple

Currently you write CSVParser.readFile. Let's look at the drawbacks:

when processing data, you need another entity: some parser. I wouldn't want to have to know about some parsers when all I want to do is to read a dataframe from file
one more concept to keep track of
one more import

All in all, I suggest we at least add the Table.readCSV = CSVParser.readFile alias.
Like, really, please 😅

All non-user facing functions should have _ prefix

@piotrMocz what do you rthink about https://github.com/luna/Dataframes/issues/57 in context of reading/writing files?

Moreover, the toInt method should be named toEnum

Table.fromFile should have better name (more descriptive)

removeColumn -> removeByIndex

keep as opposite to remove

corr -> correlations
corrWith -> correlationsWith

change generated names from CORR_WITH_foo to corr_with_foo

sort -> sortMultiple
sort Descending by default
sort config (?)

@piotrMocz what do you rthink about #57 in context of reading/writing files?

I left a comment there ;)

plotVerticalLayout [Plot] - to remove when lazy vis appears


useConditionalTimeout removed?

In #239 the useConditionalTimeout hook seems to also have been removed. Curious if this was due to an issue with the hook? I am relying on the hook for one of my projects.

@mganjoo
Yes it has been removed, please roll back to the previous version for the moment, I'm going to recover it from the next version


Update chart height to accommodate the increased participation for 2016

Probably a number of affected charts now, but it's already very cramped with 150 entries (and it's likely to be ~250 shortly).

Or, heck, 2017 for that matter.


[Japanese Club][text is wrong]

Japanese Club (and respective students
button text is wrong now
What We Should Do to Fix it

[x] change Japanese Club buttons
[x] change Student buttons for each chosen student

Copy And Paste-able Text
Approach...
James, talking with friends?
Grayce, playing piano?
Marie, putting up posters?
NEXT
Approach...
James: Mary, Jimmie's friend?
Grayce: Toshio and Kei, Grayce's siblings?
Marie:  George, historian and cartoonist?

gonna put this on hold till we get marie's input

its done, though i think it kinda messes with the consistency, tho we'll talk about it more tomorrow.

oops didnt mean to close

looks like nobody had any complaints during the presentation on this, ill close it now.


Step interpolation with boolean data steps do not allign with points.

Steps to reproduce:

Get boolean data in time series and go to data explorer
Trend with:
from(bucket: "my_bucket") |> range(start: v.timeRangeStart, stop: v.timeRangeStop) |> filter(fn: (r) => r["host"] == "thing1") |> filter(fn: (r) => r["_field"] == "valve1") |> toFloat()<-- to change data into 1 or 0!
View raw data as graph
customise -> options -> interpolation = step

Expected behavior:
See data charted with digital steps occurring where valve1 moves from true to false or vice versa (as with linear or smooth interpolation)
Actual behavior:
Data charted, but steps do not align with data occurrence times (see image)
NOTE: Data aligns just fine with linear or smooth interpolation.
In this image, see where data point is vs where the steps are:

In this image, see where more datapoints are (couldn't screen grab all at the same time, so you get red arrows!)

Environment info:
hosted by influx

I am also having this issue.
A point indicates a change in a value for most IoT cases (data is only updated on significant change)
Therefore for boolean values changing between 0 and 1, an engineer troubleshooting an issue would expect that the rising or falling edge indicates the time of the change.
Influx have not implemented this interpolation = step in a way that's consistent with an engineering understanding of timeseries data.

Is each step rendered half way between the two changing data points?
In other words, if:
12:00:00 - True
12:00:10 - False
12:00:05 - Datapoint rendered?
That seems to hold true for all datapoints on this chart...

see also https://github.com/influxdata/ui/issues/4

This is especially an issue in histogram plot type where it is more misleading.

Is this ever going to be fixed?


enable overide alertRelabelConfigs by local values file

without the '{}' it's not possible as helm need that it will be dict .
coalesce.go:165: warning: skipped value for alertRelabelConfigs: Not a table

@eyalleshem
Could you sign DCO and bump the chart to latest version please? For instruction please see the bottom of this PR page (click each "Details").

/hold
Let’s hold PRs to stable prometheus charts for a short time until prometheus-community/community#28 (comment) is resolved. If you want to help with that effort, to test etc, that would be great!

🚧 This chart is now deprecated.
Further development has moved to prometheus-community/helm-charts.
If applicable, please re-open this PR there. Thanks!


xh: 0.9.2 -> 0.1.0

Motivation for this change
Update version, contains fix for https://github.com/ducaale/xh/issues/135.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123738 run on x86_64-linux 1

1 package built:

xh (ht-rust)

xh author here, can I request a couple of things before this gets merged?

We have recently updated the tagline for xh to Friendly and fast tool for sending HTTP requests. Could we do that here as well?
xh supports xhs symlink which will default to HTTPS requests. Is this something that could be supported in nix?

I am more than happy to link to this in xh's README.md once those get addressed. Thanks.

@ducaale your first request was easy so went ahead and did that.
I'll look into 2.

I have something that seems to work, but I am not sure whether it's the right way to do it. Sandro may have some feedback there.

Result of nixpkgs-review pr 123738 at e1bafdbe run on x86_64-linux 1

2 packages built successfully:

tests.trivial
xh

1 suggestion:

error: no-flags-array
checkFlagsArray is only intended to be used in Bash, not as a Nix attribute.
Near pkgs/tools/networking/xh/default.nix:24:3:
   |
24 |   checkFlagsArray = [ "--skip=basic_options" ];
   |   ^

Also added shell completions, although they only seem to work for Bash (and presumably Fish, but I didn't test). @ducaale is the _xh completion intended to be used with ZSH?

Actually I see that's indeed how Arch uses it at least.
So not sure why it doesn't work with ZSH, since I end up with the _xh file in the output: /nix/store/6n91rd9cssnj6kjigmlfcq6yq9q22dim-xh-0.10.0/share/zsh/site-functions/_xh

Changing the scope of this since https://github.com/NixOS/nixpkgs/pull/124034 was merged.

Shell completion and xhs are both working at this point, ready for review.

Result of nixpkgs-review pr 123738 run on x86_64-linux 1

1 package built:

xh


Only build the app at build-time, not run-time

This is to remove the building of the meditor_app image in production.
To test:

On your local machine, remove the packages/app/.next folder as well as the meditor_app Docker image (docker image rm CONTAINER_ID)
make sure your packages/app/.env file is using NODE_ENV=production
docker compose build
docker compose up

mEditor should startup without issue

This is to remove the building of the meditor_app image in production.
To test:

On your local machine, remove the packages/app/.next folder as well as the meditor_app Docker image (docker image rm CONTAINER_ID)
make sure your packages/app/.env file is using NODE_ENV=production
docker compose build
docker compose up

mEditor should startup without issue

For me, my .env was in the root of the directory. The packages/app directory only contained Next.js's .env.development, etc.

This is to remove the building of the meditor_app image in production.
To test:

On your local machine, remove the packages/app/.next folder as well as the meditor_app Docker image (docker image rm CONTAINER_ID)
make sure your packages/app/.env file is using NODE_ENV=production
docker compose build
docker compose up

mEditor should startup without issue

For me, my .env was in the root of the directory. The packages/app directory only contained Next.js's .env.development, etc.

Whoops, you're right, wrong directory. Updating the description for posterity


redirect($namedRoute) missing helpful exception

Passing an invalid named route to redirect() causes a type error:
SYSTEMPATH/Router/RouteCollection.php at line 1117
ltrim() expects parameter 1 to be string, object given
...
1115                 // Lose any namespace slash at beginning of strings
1116                 // to ensure more consistent match.
1117                 $to     = ltrim($to, '\\');

This should be throwing something helpful, like HTTPException::forInvalidRedirectRoute. Tested with and without custom route definitions.

PHP: 7.2.16
CodeIgniter: 4.0.0-beta.2

It seems to be choking on Closures, trying to use ltrim() on them. Since this section is explicitly looking for a string match to $search  I think it would be find to skip anything else:
			foreach ($collection as $route)
			{
				$from = key($route['route']);
				$to   = $route['route'][$from];
				if (! is_string($to)
				{
				             continue;
				}


Parts of the game become invisible

Tested in
Minecraft: 1.15.2 and 20w14infinite
SleepBackground: 3.6
SpeedrunIGT: 11.1 and 10.17
Bug
Entities, particles, block breaking animations and the players hand become invisible
How to reproduce the issue
Open a Minecraft Instance with a Minecraft version before 1.16 with SleepBackground 3.6 and SpeedrunIGT, create a new world and unfocus the window during world creation

https://youtu.be/mJQvTbWdXqk
Minecraft: 1.15.2
SleepBackground: 3.6
SpeedrunIGT: 11.1
Fabric: 0.14.6
sleepbg config
speedrunigt config
minecraft options.txt
a log of when i performed the bug (not the session in the video)


[python][scikit-learn] Support for multiple evaluation metrics

This PR:

train, cv: Allows for multiple custom evaluation metrics
scikit-learn API: Allows multiple custom evaluation metrics
scikit-learn API: Allows using a mix of custom evaluation functions and LGBM's evaluation metrics as defined here
scikit-learn API: Fixes a bug that prevented the use of a list of LGBM's evaluation metrics in LGBMClassifier

The training and cv APIs allow the use of multiple LGBM's evaluation metrics defined here but allow only one custom evaluation function. With this changes the user will be able to monitor multiple custom evaluation functions.
A similar situation happens in the scikit-learn API. The eval_metric parameter in the fit function accepts a list of LGBM's evaluation metrics or one callable. With these changes the user will be able to monitor multiple LGBM's evaluation metrics, custom metrics, or a mix of both.

@gramirezespinoza can you merge the latest master branch to pass CI?
@StrikerRUS what is your opinion for this PR?

@StrikerRUS what is your opinion for this PR?

@gramirezespinoza  Can you please clarify the intent of this PR? Is it a refactoring? Because all points listed in the starting comment for this PR, like multiple custom metrics and the mix of custom and built-in metrics, are already supported in LightGBM.

hey @StrikerRUS, @guolinke thanks for having a look at the PR!
The idea of the PR is to have total flexibility on the number and types of metrics that the user can monitor during training. To the best of my knowledge, the functionality on this PR is not yet implemented.
Happy to answer more questions if needed.
All details
Will start with the most important: the PR fixes a small bug in the LGBMClassifier class. The code as-is breaks when eval_metric is a list of strings. This is meant to work (as referenced in the documentation) and does work on other derived classes of LGBMModel except LGBMClassifier. The issue is in the following code in the master branch:
https://github.com/microsoft/LightGBM/blob/1e2013a306c0ca719fa076a733796a6372e5b77e/python-package/lightgbm/sklearn.py#L785-L798
with the fix the following line works as expected (see tests in PR):
gbm = lgb.LGBMClassifier(**params).fit(eval_metric=['fair', 'error'], **params_fit)
The second, is added functionality to allow mix of "string" metrics and custom metrics in the sklearn API. Now, it is possible to do this (see tests in PR):
gbm = lgb.LGBMClassifier(**params).fit(eval_metric=[custom_recall, custom_precision, "fair"], **params_fit)
where custom_recall, and custom_precision are callables.
Finally, the train and cv python APIs are modified to allow multiple feval like this:
model = lgb.train(
    ...
    feval=[custom_recall, custom_precision],
    ...)

For any other examples, please see the tests in the PR.

@gramirezespinoza can you merge the latest master branch to pass CI?

done

@gramirezespinoza OK, seems I got it! This PR is a bug fix for

Will start with the most important: the PR fixes a small bug in the LGBMClassifier class. The code as-is breaks when eval_metric is a list of strings.

and a refactoring for better usage of mix of custom and built-in metrics in one parameter (namely, feval/eval_metric), because right now one should use different arguments to achieve the goal: https://github.com/microsoft/LightGBM/issues/2182#issuecomment-493452818.
Will it make sense to split this PR into two?

@StrikerRUS sure will split this into two PRs

Bugfix of LGBMClassifier in this PR

Second part of this PR implemented in 3254.
Closing this PR.


Add image & fix header padding

I added an Image here is the link
and I fixed the header padding to look almost exactly like the one in theBoringCompany website

@abedafr Thank you for your contribution!


Fix output argument and avoid writing .smb files when running as JIT

It seems I did not cherry pick the correct commits here so that pull request #20 is included.


Add logging

cytoscape currently does not throw erros and logs to the console instead.
Figure out if there is a way to check if there was an error in cytoscape an notify the user about that.

We have quite good logging by now.


Added review section and a feedback form

Thank you for your contribution to Bloody-Buddy 🤩🤩

Please mention the following in order to get PR merged  🙌🏻🙌🏻

Kind of change this PR introduces A new section for displaying reviews and a feedback form.
Issue it resolves: #70
GitHub Page Link: https://sanjanabharadwaj25.github.io/Blood-Buddy/
Screenshots :

I have made the requested changes. Please take a look.


Crosscompile semanticdb/ to Scala Native

This is a follow-up from https://github.com/scalameta/scalameta/pull/1243 that is limited to just the crosscompilable projects in semanticdb/.

CI started failing on master after merging https://travis-ci.org/scalameta/scalameta/builds/338096651
Looks like failure to resolve dependencies
ownloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.jsuereth/sbt-pgp/scala_2.10/sbt_0.13/1.1.0/jars/sbt-pgp.jar
Downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-doge/scala_2.10/sbt_0.13/0.1.5/jars/sbt-doge.jar
coursier.ResolutionException: 8 download error
    Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-assembly/scala_2.10/sbt_0.13/0.14.6/jars/sbt-assembly.jar
    Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-buildinfo/scala_2.10/sbt_0.13/0.7.0/jars/sbt-buildinfo.jar
    Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-doge/scala_2.10/sbt_0.13/0.1.5/jars/sbt-doge.jar
    Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-unidoc/scala_2.10/sbt_0.13/0.3.3/jars/sbt-unidoc.jar
    Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.jsuereth/sbt-pgp/scala_2.10/sbt_0.13/1.1.0/jars/sbt-pgp.jar
    Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.typesafe/sbt-mima-plugin/scala_2.10/sbt_0.13/0.1.18/jars/sbt-mima-plugin.jar
    Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/org.scala-js/sbt-scalajs/scala_2.10/sbt_0.13/0.6.21/jars/sbt-scalajs.jar
    Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/pl.project13.scala/sbt-jmh/scala_2.10/sbt_0.13/0.2.27/jars/sbt-jmh.jar
[error] (*:update) coursier.ResolutionException: 8 download error
[error]     Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-assembly/scala_2.10/sbt_0.13/0.14.6/jars/sbt-assembly.jar
[error]     Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-buildinfo/scala_2.10/sbt_0.13/0.7.0/jars/sbt-buildinfo.jar
[error]     Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-doge/scala_2.10/sbt_0.13/0.1.5/jars/sbt-doge.jar
[error]     Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.eed3si9n/sbt-unidoc/scala_2.10/sbt_0.13/0.3.3/jars/sbt-unidoc.jar
[error]     Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.jsuereth/sbt-pgp/scala_2.10/sbt_0.13/1.1.0/jars/sbt-pgp.jar
[error]     Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.typesafe/sbt-mima-plugin/scala_2.10/sbt_0.13/0.1.18/jars/sbt-mima-plugin.jar
[error]     Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/org.scala-js/sbt-scalajs/scala_2.10/sbt_0.13/0.6.21/jars/sbt-scalajs.jar
[error]     Caught java.net.UnknownHostException: repo.scala-sbt.org (repo.scala-sbt.org) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/pl.project13.scala/sbt-jmh/scala_2.10/sbt_0.13/0.2.27/jars/sbt-jmh.jar
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?

@olafurpg Looks spurious. Isn't this sbt downloading the plugins that we're requesting in plugins.sbt?

Looks like a temporary outage: https://twitter.com/scala_sbt/status/960928916750917632.


Missing documentation on supported props for text styles

In https://docs.microsoft.com/en-us/power-bi/create-reports/desktop-report-themes#setting-formatted-text-defaults
the documentation states that those are the supported properties:
  largeTitle: {
    fontFace: 'Arial'
    fontSize: 12,
    color: '#000000',
  },

But it looks like fontWeight work too here.
  largeTitle: {
    fontFace: 'Arial'
    fontSize: 12,
    fontWeight: '700'
    color: '#000000',
  },

Could you add all the supported properties (fontWeight, text styles etc) in the documentation please?
Thank you

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

ID: 8d9a8511-6ff9-78c6-5a8c-3f5ac68224d0
Version Independent ID: f4fd28c7-71fa-a776-7154-e76eb9bd3649
Content: Use report themes in Power BI Desktop - Power BI
Content Source: powerbi-docs/create-reports/desktop-report-themes.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

There is also nothing specified for line height... can we set the line height directly in this style object?

There is also NO document on how to style textBoxe
see https://community.powerbi.com/t5/Desktop/JSON-Theme-Code-for-Text-Box/m-p/437973

Thanks for your comment, @phil-lgr. This issue has been assigned to the doc author for review.

Thanks for your comment, @phil-lgr.
In the future we may document all the JSON team properties. Until then I suggest that you look for a solution in the Power BI Community or our support page.
Thanks,
Kesem

Hi, @phil-lgr -- I just wanted to add, I've been working with the engineers to get answers to your questions. It's taking longer than expected...

we may document all the JSON team properties.

We def need PowerBI's theming feature to be fully documented, IMO, without that, it makes it much harder to create complete themes. Please consider prioritizing this. Thanks.


Fix typos (helpers.entity)

Description:
Fix two typos.
Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

You can merge these PRs yourself 😉


Fixes CORS error when downloading attachments.

Fixes #1235
Checklist

[ ] ~I have made corresponding changes to the documentation.~
[ ] ~I have updated the types definition of modified exports.~
[x] I have verified the functionality in some of the neeto web-apps.
[x] I have added the necessary label (patch/minor/major - If package publish
is required).

Reviewers

@AbhayVAshokan _a please review.


Enable bitrise

We should consider building on bitrise as well as travis and circleci (#23).

Not actionable. Travis is working fine and I don't see a need to provide alternatives at the moment.


Add gitignore.

Thank you for your contribution! We are still figuring out the tools and workflow for how to properly integrate code from github pull requests. Once we do, we will come back to take a look at merging this pull request. Expect this to be at lease one week, possibly several. Apologies for the delay.

I am using this PR to test the automatic cla annotation.

I am using this PR to test the automatic cla annotation.


[QUERY] Add index query support to the client

Changes Unknown when pulling 98b02b663d6ea3a69ef92f9236ae9d4a61e395d7 on query-support into ** on master**.

Changes Unknown when pulling 58490bf33e1674834787e05e2574b97d163fced9 on query-support into ** on master**.


test case to support #943

This is test only PR, so travis is expected to report failure.

Merged locally, thank you! :heart:


Added comment to Decoder.finalize

Hi. I'm Korean student, recently studying tensorflow.
At first, I'm sorry about my poor english.
I found a curious thing that only finalize method in contrib.seq2seq.Decoder doesn't have any comment in documents.
So, I tried to write some comments on it, usually copying from comments from other methods.

I signed the indivisual CLA!

I changed comment of sequence_lengths

Nagging Reviewer @ebrevdo, @lmthang: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @ebrevdo, @lmthang: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 29 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @ebrevdo, @lmthang: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 44 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @ebrevdo, @lmthang: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 59 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @ebrevdo, @lmthang: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 74 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @ebrevdo, @lmthang: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

@smilu97 Hi, could you please look into the changes requested by ebrevdo ?

Oh.. I already commited new one as ebrevdo requested, but I forgot to write some comment about this.
or did I missed something?

This failed pylint at C0301[line-too-long], so I fixed some


v6: Add upgrade guidance. Closes #3481

Adds v6 upgrade guidance. Closes #3481

ready to be merged 👍 I will try to do it today night. Just two comments for me before I go:

[ ] recheck typo and correct it when merging
[ ] when we add --confirm option to roleinheritance reset/break commands I should add also some info to this guide as well

aaa.. I almost forgot.
awesome work @martinlingstuyl  👍👍 you rock 🤩

merged manually. thanks for your awesome work 🤩


fit the directory to nvim lua plugin structure

Hey it's brilliant piece of code and it works really well!
I created this PR to change the directory structure as below
defer.lua → lua/throttle-debounce/init.lua which makes it possible to install via package managers such as packer.
And one can import and use this via require('throttle-debounce').throttle_leading(..., x).
You can verify that this works via my fork.
-- e.g. with packer
use 'ryuheechul/throttle-debounce.nvim'

-- ...

print(vim.inpect(require 'throttle-debounce'))

besides the moving the file to another directory, made these two minor changes:

made two ms to not share their names in td_validate
format changes picked by stylua

Hey, thanks a lot for the PR! Hadn't even considered publishing this as a plugin, so this is a very welcome addition. <3


[Feature][UI Next][V1.0.0-Beta] Front-end optimization.

Optimization

[ ] [Feature][UI Next][V1.0.0-Beta] After an error message appears in the form verification, click the switch language button, the error message is not updated to the error message of the corresponding language
[ ] [Feature][UI Next][V1.0.0-Beta] The scroll bar on the right side of the task component popup box is optimized, using the scroll bar of Naive-UI
[ ] [Feature][UI Next][V1.0.0-Beta] Optimize the code and id parameter passing methods in API , no longer use the object form
[ ] [Feature][UI Next][V1.0.0-Beta] Add form search component to replace all form search functions in the project
[ ] [Feature][UI Next][V1.0.0-Beta] Unified use of Naive-UI Card , delete the Card packaged in the project
[ ] [Feature][UI Next][V1.0.0-Beta] The method of downloading the file is stripped from the requested package file to form a separate tool file
[ ] [Feature][UI Next][V1.0.0-Beta] All processes in the project are replaced by workflow, and the concept of process no longer exists
[ ] [Feature][UI Next][V1.0.0-Beta] Supplementary data types returned by the table interface
[ ] [Feature][UI Next][V1.0.0-Beta] Types of supplemental table row data
[ ] [Feature][UI Next][V1.0.0-Beta] Security -  Tenant Manage added blur and input for form validation
[ ] [Feature][UI Next][V1.0.0-Beta] Resource provide and inject optimizations in File Manage
[ ] [Feature][UI Next][V1.0.0-Beta] The default field in props uses arrow functions for arrays and objects
[ ] [Feature][UI Next][V1.0.0-Beta] Size of table component in unity project
[ ] [Feature][UI Next][V1.0.0-Beta] Size of form components in unity project
[ ] [Feature][UI Next][V1.0.0-Beta] Unify the styles and icons of table action buttons in a project
[ ] [Feature][UI Next][V1.0.0-Beta] Style the log component in a unified project
[ ] [Feature][UI Next][V1.0.0-Beta] Modify the Chinese comments in the project
[ ] [Feature][UI Next][V1.0.0-Beta] Digital flopper font color consistency problem
[ ] [Feature][UI Next][V1.0.0-Beta] The color matching of the chart is optimized into a blue gradient decreasing form
[ ] [Feature][UI Next][V1.0.0-Beta] Unified handling of the problem of positive integer input boxes in projects
[ ] [Feature][UI Next][V1.0.0-Beta] Optimize the handling of header-extra in the Modal component, changing it to a slot
[ ] #9493
[x] #9510
[ ] #9520

i will fix remove spaces from all input boxes

i will handle Monaco Editor supports theme switching function.

i will handle style the log component in a unified project

I will remove the method of downloading files from the file in the request package and add a separate tool file


[Improve][Connector-V2] Support schema evolution for mysql-cdc and mysql-jdbc

Purpose of this pull request

Support schema evolution for mysql-cdc

Support add column
Support drop column
Support modify data type of column
Support rename name of column

Support schema evolution for mysql jdbc sink

Support add column
Support drop column
Support modify data type of column
Support rename name of column

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

subtask of https://github.com/apache/seatunnel/issues/4672

中文：
这块是否有方案文档，对这块有兴趣，想了解一下是如何实现点的，想看看能不能实现动态加表的特性，如果可以，想看看我能不能实现
English：
Is there a solution document for this? I am interested in this and would like to understand how it is implemented. I want to see if it is possible to achieve the feature of dynamically adding tables. If so, I would like to see if I can implement it.

Has it already been completed?

Has it already been completed?

Yes, I have tested both locally and using e2e several times.

cc @EricJoy2048 @Hisoka-X

Has it already been completed?

Yes, I have tested both locally and using e2e several times.

Hi @dailai , could you add some doc? I found users even not know this feature so that can not use it at now.

OK


The organisation has asked if it is possible to provide the number of people who speak the following the following languages from greatest number to smallest, including the percentage of the world population:

Chinese.
English.
Hindi.
Spanish.
Arabic.

Changed task style

Matches ticket 9 so will close this.


Should indicate which port are exposed by default

https://hub.docker.com/_/memcached
Should indicate which port are exposed by default. For example: https://hub.docker.com/_/redis

I would rather go the other direction and delete any mention of EXPOSE in the official images documentation. The image itself documents what ports are "exposed" and can be seen with a docker image inspect memcached.
$ docker image inspect memcached | jq '.[].ContainerConfig.ExposedPorts'
{
  "11211/tcp": {}
}

Indeed, I'd tend to agree, especially since 11211 is the official well-known memcached default port. :+1:


Installation without webhook not working

I attempted to install cert-manager without webhooks as outlined here on GKE with Kubernetes 1.13: https://docs.cert-manager.io/en/latest/getting-started/webhook.html. Unfortunately by installing kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v0.9.1/cert-manager-no-webhook.yaml.
we get:
Internal error occurred: failed calling webhook "webhook.certmanager.k8s.io": the server could not find the requested  resource

In the past we had version 0.5.2 installed. First we tried to setup with webhooks. But this resulted in similar errors already during installation. webhook itself did subsequently not start because of missing TLS certificates. The namespace was properly labeled to disable validation.
I would have expected this error to disappear when not using webhooks.

interestingly I attempted to downgrade to 0.5.2 again and still see the same issue. Fairly certain everything v0.9.1 related is gone.

Any update on this issue? Facing a similar problem.

To my part I test the 0.9.1 without webhook it failed but 0.8.1 works

We've had the same issue when upgrading from 0.6.x to 0.9.1 and got around it by manually removing the MutatingWebhookConfiguration from the admission controller:
kubectl delete MutatingWebhookConfiguration cert-manager-webhook
After that I ran into the problem that the ingress-shim flags in the Helm chart are seemingly not supported anymore (--default-issuer-name etc), but that's a different issue altogether.

Yep, it looks like you've not properly cleaned up the webhook resources you created when you first tried to install with the webhook enabled.
I'm going to close this as it isn't really a bug, rather a misconfiguration caused by a borked upgrade/downgrade 😄


Use the correct http service in chaise

We should not use ConfigUtils.getContextJSON().server directly and instead we should use the appropriate ConfigUtils.getHTTPService() function for getting the http service.
In the case of static pages, if default config is undefined, we're not going to attach server to the context object and getHTTPService is handling this case.
This was causing terminal errors when navbar was used on a static page. The logClientAction function is using .server API and it was throwing an error. We should make sure we're not doing this in other places as well.

This has been done as part of log changes PR.


Create Configure-AzureFirewall_Policy-AzurePowerShell

This document helps configuring following, using Azure PowerShell:
Create Azure Firewall Policy
Create Firewall Policy Network Rule Collection group
Create Azure firewall Policy Network Rule collection with new network rules
Create an Azure Firewall Policy Network Rules in existing Rule Collection
Create Firewall Policy Application Rule Collection group
Create Azure Firewall  Policy Application Rule collection with new application rules
Create an Azure Firewall Policy Application Rule in an existing Rule collection

@vhorne Do you want me to copy the raw contents of the article in this PR and paste it into the article (with any edits for blocking items per the PR quality criteria) in the existing private repo PR?

#please-close
This article was published via a separate PR.


Redis async/blocking poll?

I stumbled upon: https://convore.com/celery/celery-with-redis-get_connection-error-polling/
It says that redis with kombu/celery no longer uses polling. Well, with CELERY_RESULT_BACKEND = 'redis' and BROKER_TRANSPORT='redis' I still get 0.5s+ latency without specifying interval.
Tried kombu and celery from github tip as well, same thing. I only get good results with interval = 0.001 or less, but this worries me a little especially with situations when redis communicates through WAN/LAN.
Or maybe I am misinterpreting? Is it that tasks are polled with 0.5s interval (unless faster polling is requested) and results are immediate?
Still, even though redis got me 50-70% better results than amqp when it comes to latency, this just doesn't feel right. It has async capabilities, why use sync with timeout?

Misread the issue.  Celery results doesn't use Kombu, and you are right in that they are polling.  A new non-polling Celery result backend would have to be written.

I see but what about BROKER_TRANSPORT=redis? It seems there is 0.5 delay before celery even gets task queued for redis. Is it fixed with your commit?
Is there an easy way to write non-polling Celery result backend? Is it something planned or maybe easily created without forking (with possible inclusion in main version later on)?

This issue isn't fixed. Tried kombu from git. Redis transport backend from kombu in conjunction with celery still has ~1s delay between subsequent tasks polling.
With prefetch=0 and 5 workers, it is getting 5 tasks per second, no matter the queue size.
Isn't anyone using any other backend than amqp that it is emerging now?...

Well. Have a look here: https://github.com/23doors/celery_test
Tested on OSX and Ubuntu. Same results. As you look in celery logs you can see that last task was executed with 11 seconds delay (they are executed in 10 batch due to 10 concurrency, every second polling).
Tested with tip version of kombu and celery.

Try setting CELERY_DISABLE_RATE_LIMITS=True

Ok, so basically redis RESULT BACKEND for celery is pretty much useless without low interval. It would be great if it would be remade.
CELERY_DISABLE_RATE_LIMITS=True did it. Still result backend is horrible, with default interval: latency is +0.5s in some cases unless I run with 0.001 interval (which is a bad solution), but transport is behaving normally: queueing 100 tasks with no prefetch results in: task delay between queue and run is 1.6ms-271ms (using multiprocess pool) BUT 1.7-3ms using gevent and a 2.7-3.5ms using eventlet. Increasing redis max connections and pool size for multiprocess doesnt seem to make a difference.
Still, main question is. Why disabling rate limits helps here? According to celery docs, rate limit's default value is unlimited. This task also doesnt specify any rate limits. So what's going on? This still looks like a bug...

http://docs.celeryproject.org/en/latest/userguide/optimizing.html#rate-limits
Default rate limit is unlimited, but rate limits can be changed at runtime so the subsystem responsible for enforcing the limits is enabled.  CELERY_DISABLE_RATE_LIMITS disables lots of machinery.  Why it introduces as much as a one second delay I'm not sure of at this point, but it may be a performance regression.
It is useless to instrument join latency when the result backend is polling, as it is limited by design to not cause
a busy loop. Using result.get() and join() is usually an anti-pattern and so these cases are so far only optimized in the amqp.  Redis did not have the mechanisms available at the time when I wrote its result backend, compared to today (blpop, pubsub). You are very welcome to contribute a non-polling Redis backend, I have been championing for someone to do this for some time.


Update libssh2

Please update to the latest libssh2!  Thanks!

Please update to the latest libssh2! Thanks!

Hi,
libssh2 will be updated to 1.11.1 (while the v1.11.0 has new features and ciphers, apparently it also brings in some problems and few regressions.) The libss2 team is very active nowadays to deliver the v1.11.1. So we're waiting for the v1.11.1. Good news is that it is around the corner.
Thank you for your patience.

Very glad to hear this!  The current Core/SSH refuses to connect to my server.  I can manually set up my tunnel until 1.11.1 arrives.  :-)

Very glad to hear this! The current Core/SSH refuses to connect to my server. I can manually set up my tunnel until 1.11.1 arrives. :-)

Hi, I have pushed the initial public version of libssh2 (ver 1.11.0) to my fork of U++ (see the ssh_upgrade branch). This will not be merged with U++ until the official point release (1.11.1) is available. But in the meantime, you can test it of course. :)
I'll be grateful for any feedbacks.
Best regards,

libssh2 version 1.11.1 has been released.
https://github.com/libssh2/libssh2/releases/tag/libssh2-1.11.1

libssh2 version 1.11.1 has been released. https://github.com/libssh2/libssh2/releases/tag/libssh2-1.11.1

Hi, the pull request has already been made, we are preparing for 2024.1 release of U++. After that, hopefully it will be merged.
Thank you for your patience.


NTR: Zinc administered, caregiver report

For new term requests, please provide the following information:
Term Label [required]
Zinc administered, caregiver report
Synonyms [optional]
Definition [required except for organism name]
The definition should be understandable even for non-specialists. Include a PubMed ID or DOI to refer to any relevant article that provides additional information about the suggested term.
Whether the caregiver for a participant reported giving the child zinc as treatment for a diarrheal episode
Suggested Parent Term [required]
Please look in the ontology hierarchy in a browser such as OLS or choose one from following list:

[ ] Material (independent of other entities and can be maintained through time, such as person, cell, sample)
[ ] Process (has temporal proper parts and for some time depends on some material entity, such as life cycle, cell division, heart beating, sleeping)
[ ] Information (generically dependent on some artifact and stands in relation of aboutness to some entity, such as data, identifier, document)
[ ] Characteristic/Quality (inheres in or is borne by other entities, such as height, color, disease)

Information
Examples [optional]
Contributor [optional]
If you would like to be a contributor of the term or anyone you think that contributes to the term, such as label, definition, please provide the name and ORCID ID.
Requester [optional]
Please indicate which resource/database/project/ request the term.
ClinEpiDB

EUPATH:0060122
label: indicator of caregiver reporting zinc administered
definition: Treatment information that specifies whether a caregiver of a participant under investigation reported administering zinc to the participant.
parent: treatment information
@SheenaTomko

Looks good to me.


[ClangImporter] NFC: extract Clang include path mapping logic into a separate file

This logic is going to be extended to inject the modulemap and the header of Glibc in addition to libstdc++. It doesn't depend on the rest of ClangImporter, so let's extract it to a separate file to make incremental builds and IDE features faster.

@swift-ci please smoke test

@swift-ci please smoke test


Extend the Image/Video players to support additional services.

With the current implementation, Players can be called to queue file-paths and then play, pause, stop, and play next .  Each file-path is pulled off the queue, in turn, and played automatically or when the client calls play next.
Want to update the behavior to not remove the file-paths from the queue, so that the Player can play previous.... meaning the queue will become a list instead. Additional services will be added to

clear the current list of file paths
add an entire folder of files to the Player's list of files to play

not important at this time


https://weaveworks.github.io/flux is not a valid chart repository

Running the following command fails:
helm repo add fluxcd https://weaveworks.github.io/flux

Error:
Error: Looks like "https://weaveworks.github.io/flux" is not a valid chart repository or cannot be reached: Failed to fetch https://weaveworks.github.io/flux/index.yaml : 404 Not Found

According to this the repo has moved to https://fluxcd.github.io/flux. Using this URL works for me, but the README needs to be updated.

Thanks @jwenz723 for the fix.


Add Fit and Translate

Google Fit  and Translate are missing, is that possible to add them?

Yup they can be added, but you can also use them right now by going to the Search app and searching for "Google Fit" for example and clicking the webapp link. Google Fit does not seem to work as a webapp though.

Thanks.

Please add translate.

And you can remove Google+.

Added translate, but fit just links to app on play store.


RangeSlider Doesn't Appear On iOS

IMPORTANT
RangeSlider Doesn't Appear On iOS but Appear in android
If you are reporting a bug and want it to be fixed as soon as posible don't forget to provide a sample project.

No information provided.


Cloud Functions updated only if source URL path ends with a slash

When deploying Cloud Functions from a subfolder in source repository, functions are not updated on every pulumi up op unless the source_repository url path ends with a slash / .
For example:
{repo}/moveable-aliases/master/paths/function_subfolder will create the function no problem on first run, but on subsequent runs will not update it.
{repo}/moveable-aliases/master/paths/function_subfolder/ on the other hand will update the function ([diff: ~sourceRepository]) on every Pulumi run.
Not sure if a bug or a feature, as it can actually save needless updates if there's a 'force update' flag available to be set.

It looks like this has been fixed since 2020. I tried the following program:
const functionPython = new gcp.cloudfunctions.Function("python-func", {
    runtime: "python37",
    entryPoint: "handler",
    sourceRepository: {
        url: "https://source.developers.google.com/projects/myorganisation/repos/bitbucket_myrepo/moveable-aliases/myb/paths/myfunction",
    },
    region: "us-central1",
    triggerHttp: true,
    availableMemoryMb: 128,
});

then I do get diffs on changing the repository path:
~ sourceRepository: {
  ~ url: "https://source.developers.google.com/projects/myorganisation/repos/bitbucket_myrepo/moveable-aliases/myb/paths/myfunction" => "https://source.developers.google.com/projects/myorganisation/repos/bitbucket_myrepo/moveable-aliases/myb/paths/myfunction2"
}


umap takes 10 seconds to import

First off, great work with umap, it's a very cool tool.
I am trying to make a GUI that performs some mapping for a user. My issue is, it takes 10 seconds to import umap, which means the GUI start up is very slow. No other dependencies take this long, so I was wondering if there is some way to speed it up?
I have other features in the GUI, so ideally the user could perform other actions while the umap is created with cross threading. It's just the slow import that concerns me.

I'm not actually sure what takes up time in the import phase. At least some of it may be related to numba compilation, but it could be any number of things. I can't offer any immediate solutions I am afraid.

That's okay, I found I could import/run it in a PySide2 worker thread in the background and the user can still operate the GUI while it's loading. I appreciate the quick response however!

It seems most of the time is taken by pynndescent
$ time python -c "import umap"
real	0m13.641s
user	0m13.729s
sys	0m0.518s

$ time python -c "import pynndescent"
real	0m13.315s
user	0m13.435s
sys	0m0.469s

Sorry to comment on this closed issue, but is there a way to speed up the import?
The PySide2 workaround is not applicable to my case.
UMAP gives very good results for my application and embedding calculation fast enough. However, we can't put it into production because the import is exceedingly long (cloud deployment).
Thank you!

Is the user uploading data that will be incorporated into the map? If not, you could premake the map and save the embedding, then just load the embedding to display.

The embedding has to be calculated for each user/session separately, so we can't use a cached one (and due to the system architecture, the library has to be imported at request time).
The users are non-technical and can't wait 10 seconds for the results...
At the moment I've reverted to Kernel PCA (which is as expected not as good, but usable).

Can you add a loading bar to the interface? 10 seconds is slow but not so slow that most people would just leave if you have a loading bar.


[5.2] Bug with task scheduling when using emailOutputTo()

Ever since i've updated to 5.2 from 5.1 I get an error when my task runs saying that emailOutputTo can't find the file I'm writing to with sendOutputTo.
live.ERROR: exception 'ErrorException' with message 'file_get_contents('/home/mysite/site/storage/logs/cron_deletetempfiles_2015_12_25.txt'): failed to open stream: No such file or directory' in /home/mysite/site/vendor/laravel/framework/src/Illuminate/Console/Scheduling/Event.php:710

Code I'm using in app/Console/Kernel.php is
$schedule->command('deletetempfiles')
		->daily()
		->sendOutputTo(storage_path().'/logs/cron_deletetempfiles_'.date('Y_m_d').'.txt')
		->emailOutputTo('myemail');

I checked and the file exists and has the proper info in it.

That's probably cause if the change in sendOutputTo.
I'll look into this tomorrow

Thank you very much for the report @neamtua. @arrilot Thanks. :snowman:

Ever since i've updated to 5.2 from 5.1

Actually, this bug is present on 5.1 too. You must have just been using an older version of 5.1.


add support for datetime serialization

Other db-api implementations (e.g. sqlite3) support datetime instances
in execute calls.
E.g. it is possible to do something like:
c.execute('insert into t (dt) values (?)', (datetime.utcnow(),))

This commit adds the same functionality.

ack


Type convert in $match for specific _id of collection

As on mongo 4.2.x and Grafana 6.x, I ran into an issue where Grafana Variables passed into a query are usually cast as type String.
MongoDB prohibits server side conversion to ObjectId or Date within a  "$match" statement:
https://docs.mongodb.com/manual/reference/operator/aggregation/toObjectId/
db.fails.aggregate ( [
{ "$match" :    { 'myID': { "$convert": { "input": "5e5987f66f89a972dc7d4854", "to": "objectId" }}, "isDeleted":false }...
While OK in a mongo shell, this next test is also not valid  syntax with the ObjectId() in grafana:
db.wotks.aggregate ( [ { $match :    {    "projectID":  ObjectId( "5e5987f66f89a972dc7d4854" )  , "isDeleted":false }}...
It seems rather trivial, but is there an example on how to create ObjectId and raw date numeric types from passed Grafana variables for "$match" in mongodb-grafana?
Cheers,
J

_id

Any solution for this?


Patch to fix issue #697

Fix to solve issue #697 (Mouse up outside map will cause "canvas-drag" event to stick).
Also includes improvements to achieve smoother panning with the following options enabled:
homeFillsViewer: true
constrainDuringPan: true
visibilityRatio: 1

Nice! So, this fixes https://github.com/openseadragon/openseadragon/issues/753 as well?

The tests were failing so I restarted the build to see if it was just a fluke, but they are still failing. You might try running the tests locally to see what the issue is.

I did some of the changes, but not all because some of them sound quite complex.
Regarding tests, the failing checks are the following:

enter-press-move-exit-move-release (drag, release outside tracked element):  dragHandler event count matches. Expected: 15  Result: 10  Diff: 150

enter-press-move-exit-move-release (drag, release outside tracked element):  dragEndHandler event count matches expected (1) Expected:  1  Result: 	0

enter-press-move-exit-move-release (drag, release outside tracked element):  releaseHandler event.insideElementPressed matches expected (true) Expected: true  Result: false

enter-press-move-exit-move-release (drag, release outside tracked element):  releaseHandler event.insideElementReleased matches expected (false) Expected: false  Result: true

enter-press-move-exit-move-release (drag, release outside tracked element):  Remaining tracked pointer count matches expected (0) Expected: 0  Result: 1

I think that this is because my patch stops and restarts all active mousetrackers when the mouse moves out of the viewer, that is different from the previous behaviour. I hope it helps.

Hmm... that makes me realize a side effect of this patch that I hadn't previously connected with: now, even when not in an iframe, if your cursor leaves the viewer while you're dragging, the drag ends. This is unfortunate from a usability standpoint; now you have to pay attention to how close your cursor is to the edge of the viewer, rather than just looking at the image.
I know there's been discussion about detecting whether you're in an iframe before doing the reset... can we add some detection like that in here? When you're not in an iframe, the mouse capture seems to do a fine job; my understanding is the mouse capture just doesn't work with iframes, so that's what we should be fixing.
All the changes in your recent commit look good. Looks like what's left (besides the iframe detection) is making the pan constraint code not have side effects. I think the key to that would be a new lower level (probably private, i.e. not part of the public API) viewport function that'll give you a bounds given a panX, panY, and zoom. That way you can find the bounds and constrainedBounds without actually modifying the viewport's variables. Does that sound reasonable?
@msalsbery You might want to take a look at this patch too, if you haven't.

As willbuckingham pointed out, this could be achieved, for instance, with the following code:
var isInIframe = (window.location != window.parent.location) ? true : false;
Although this check could work well when the viewer fully covers the iframe area, what if the viewer fills only a part of the iframe? Nonetheless, I think that this behaviour is better than getting the mouse down event stick.

Yeah, I think it's fine to engage for all iframes, regardless of whether it only partially covers the window.

Well, after adding the in-iframe check, all tests are OK.

Looking good!
I think all that's left is to refactor the constrained pan to avoid side effects (specifically getConstrainedBounds calling zoomTo). I've taken a look at the code to see if I can come up with a suggestion, and I'm actually getting a little confused about how the fix works. Can you explain how the new technique is working better than the old one? I mean, when I run it it certainly looks good, but I'd like to understand what's going on under the hood.
Btw, it seems the iframe drag issue portion of this patch is ready to land, but since it's in the same patch as the constrained pan fix, it'll have to wait until they're both ready.

Ok, have a look at the following sketch, which represents what happens without patching anything.

Previous viewport position in blue.
Next viewport position without constraints in dashed red.
Constrained, and finally used viewport position in yellow.

This method causes an undesired effect when panning close to image borders, and it can be improved using the following approach:

My patch tryes to achieve something like that, as I am going to describe in the following lines:
First, the patch computes red-dashed rectangle and then check if it is out of the boundaries. If it is the case, my code disables the movement in the axis that causes the viewport to get out.
The main problem here is computing the red-dashed rectangle. In order to do that, I simulate the panning movement (passing immediately=true) without constraints, not only in x and y, but also in z (and this is why I use zoomTo, just in case the user had used the mouse wheel); and then, I revert the movement to leave everything in his original state.
The result is a smoother panning close to the boundaries of the image.
I hope it helps.

Yes, the diagrams are very helpful! I'm still not sure how the old method (applyConstraints) causes the behavior in the top diagram, though. It looks like _applyBoundaryConstraints, which applyConstraints calls, operates on the x and y independently, so it should theoretically have the same results as your second diagram.
At any rate, it seems applyConstraints itself should be fixed rather than introducing an additional set of constraint code.
Thank you for digging into this stuff. :)

@josmilmed Are you able to continue pursuing this patch? Any thoughts about fixing applyConstraints?

Would be great to get this merged at some point. This patch solves a lot of the jankiness that I've had with constrainDuringPan = true.

@patrickgalbraith Agreed! Unfortunately it seems @josmilmed has lost interest, and it's still not ready to land. Any interest in picking it up?

Hey, sorry for not writing earlier, I have been veeery busy.
I removed the call to zoom methods without any problems, achieving the same result. With this change, perhaps this pull request could be ok, right?

@josmilmed Wonderful, thank you for finding some time for this! The patch looks good with that change.
@LarissaSmith since you're working in the mouse tracker, would you mind taking a quick look at this patch, specifically the mouse tracker change and the onCanvasExit in the viewer code (that's the part that address #697; the other code is for a different issue). Does it look good to you, and is it compatible with the work you're doing?

@josmilmed I'm just noticing there are merge conflicts... can you sort those out?

Done!

It looks good to me, and there shouldn't be compatibility issues with my changes.

@josmilmed Thank you!
@LarissaSmith Great, thanks for taking a look at it :)

@josmilmed Thanks for sticking with this patch to the end, btw! Next time let's tackle one issue per patch :)

@iangilman, thanks to you and your team for developing OpenSeadragon. It has been a pleasure for me to collaborate somehow.

 And the team is all of us, you included, now :)


MDS names may not start with a numeric digit.

Just spotted this:
deprecation warning: MDS id 'mds.7cf73bf1fd74' is invalid and will be forbidden in a future version.  MDS names may not start with a numeric digit.
An easy work around could be to do something like this on the MDS_NAME variable: : ${MDS_NAME:=a$(hostname -s)}

I would slightly prefer a more mnemonic "mds-" prefix, but otherwise, that sounds like a good idea.
One thing to note, though, is that metadata servers are not machine-restricted.  They are not bound by their data directories and can move around the cluster.   As a result, I generally give mine labels like "a", "b", "c".
In this case, obviously, we cannot do that, but I wonder if adding and removing arbitrarily-named metadata servers screws with the Ceph database.

I like the "daemon type" prefix and I agree about the statement of volatile MDSs in general.
Perhaps we should encourage users to use MDS_NAME and justify this with your explanations.
Let's start by fixing this and we can always add a bit of doc later.


Issue with xrdcp and xrootd python client while using xrdcl-authz-plugin at coffea-casa

The user reported that at CMS coffea-casa AF while using xrdcp to copy files, we see "Operation is not implemented" error:
 xrdcp -f root://xcache//store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000/ffNtuple_1.root /dev/null
[0B/0B][100%][==================================================][0B/s]
Run: [ERROR] Operation is not implemented:  (source)

as well there is a segfault while using xrootd python API:
>>> from XRootD import client
>>> xrd = client.FileSystem("root://xcache//store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000/ffNtuple_1.root")
Segmentation fault (core dumped)

Current repository with plugin: https://github.com/jthiltges/xrdcl-authz-plugin/tree/xcache
cc @jthiltges

Hi Oksana, can you confirm that the plugin is being built of the xcache branch? Some of the strings suggest it's coming from master. At least for hub.opensciencegrid.org/coffea-casa/cc-ubuntu:2023.03.17.

Hi @oshadura and @jthiltges, I'm "the user" in Oksana's original post, and I thought it might be helpful to give a little context. The main functionality I'm looking for is to be able to list files on LPC EOS like I would with xrdfs root://cmseos.fnal.gov ls. Of course fixing this such that xrootd works in general would be great, but if you know another good way to do this from coffea-casa, I'd happily do that instead :)

Now since we have deployed @jthiltges plugin Segmentation fault (core dumped) is fixed, but still some functionality, such as xrdfs is missing:

# the following command works on lxplus
$ xrdfs root://cmseos.fnal.gov// ls /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000/

# but the equivalent command hangs on coffea-casa
$ xrdfs root://xcache// ls /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000/

# even though the same command works on coffea-casa if I specify one specific file
$ xrdfs root://xcache// ls /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000/ffNtuple_1.root

Interesting result. This appears to partially be an issue with our xcache (running in docker).
The xcache tells the client to contact <IP_ADDRESS>, which is a private IP of the xcache container. And as expected, the client cannot connect.
$ xrdfs red-xcache1.unl.edu:1094 locate '*'
[::<IP_ADDRESS>]:1094 Server ReadWrite
$ xrdfs xcache:1094 locate '*'
[::<IP_ADDRESS>]:1094 Server ReadWrite

For now, I switched the red-xcache container over to host-mode networking (network_mode: host) and the ls proceeds to fail differently
$ xrdfs red-xcache1.unl.edu ls /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000
[ERROR] Server responded with an error: [3005] Unable to open directory /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000; too many levels of symbolic links

On the xcache server side:
230411 17:53:14 543 scitokens_Access: Grant authorization based on scopes for operation=dir, path=/store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000
[2023-04-11 17:53:18.077326 +0000][Warning][XRootD            ] [u26@cms-xrd-global.cern.ch:1094] Redirect limit has been reached for message kXR_dirlist (path: /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000), the last known error is: [ERROR] Error response: no such file or directory
230411 17:53:18 543 ofs_opendir<EMAIL_ADDRESS>Unable to open directory /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000; too many levels of symbolic links
230411 17:53:18 543<EMAIL_ADDRESS>Xrootd_Response: sending err 3005: Unable to open directory /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000; too many levels of symbolic links
[2023-04-11 17:53:18.077682 +0000][Warning][XRootD            ] Redirect trace-back:
[2023-04-11 17:53:18.077682 +0000][Warning][XRootD            ]         0. Redirected from: root://cmsxrootd.fnal.gov:1094//store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000 to: root://cms-xrd-global.cern.ch:1094/
[2023-04-11 17:53:18.077682 +0000][Warning][XRootD            ]         1. Redirected from: root://cms-xrd-global.cern.ch:1094/ to: root://cms-xrd-transit.cern.ch:1094/
[2023-04-11 17:53:18.077682 +0000][Warning][XRootD            ]         2. Retrying: root://cms-xrd-global.cern.ch:1094/
...
[2023-04-11 17:53:18.077682 +0000][Warning][XRootD            ]         29. Redirected from: root://cms-xrd-global.cern.ch:1094/ to: root://cms-xrd-transit.cern.ch:1094/
[2023-04-11 17:53:18.077682 +0000][Warning][XRootD            ]         30. Retrying: root://cms-xrd-global.cern.ch:1094/
230411 17:53:18 543 XrdTLS<EMAIL_ADDRESS>TLS error rc=0 ec=6 (zero_return) errno=0.
230411 17:53:18 543 XrootdXeq<EMAIL_ADDRESS>disc 0:00:04

I suspect that listing directory contents will be painfully slow if the request doesn't go directly to the target server/cluster. Otherwise, I'm guessing it will result in a search of the entire hierarchy.

Update, the second query now end up showing too many levels of symbolic links error:
cms-jovyan@jupyter-oksana-2eshadura-40cern-2ech:~$ xrdfs root://xcache// ls /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000/
[ERROR] Server responded with an error: [3005] Unable to open directory /store/group/lpcmetx/SIDM/ffNtupleV4/2018/SIDM_XXTo2ATo2Mu2E_mXX-100_mA-1p2_ctau-9p6_TuneCP5_13TeV-madgraph-pythia8/RunIIAutumn18DRPremix-102X_upgrade2018_realistic_v15-v1/210326_161703/0000/; too many levels of symbolic links


Change username

What and why?
How to test?
Trello

/deploy arroya configBranch=add-account-name-change-notify-template

/deploy arroya


Group contact icons

Not sure if this is possible. Might be on Big Sur, but definitely not on High Sierra or anything semi-old

Nah I just meant like the one that looks like imessage with the contacts stacked. I'm gonna give it a shot

Done


How to debug "X" not auto-importing / suggesting a place to auto-import from?

I have mixed results with auto-importing -- probably depending on a variety of things, for example:
@action here has no correct suggestion

even though, the commented import line is clearly recognized by typescript:

Here is an example of what I expect to happen (in the same file even, but with a different import):

So, how do you debug this sort of behaviour?

It's known issue of tsserver, https://github.com/microsoft/TypeScript/issues/39048


Static method 'uploadFile' can't be accessed through an instance

Hi. I was looking around for solution to upload images to S3 and found your package interesting. I opened your example and received the error message "Static method 'uploadFile' can't be accessed through an instance" at line 107 of your main.dart, before I can run and test it. Could you advice on this? Thanks.

Hi please check the version you are on it should be 0.3.2. If you're using an older version then use the example of that version your example and plugin version is mismatching.

got it working thanks.


[NG] Backport Feature/issue 3464

Backporting ...

Please add a better description, and link to the issue you are backporting.