Unable to connect to an authless remote jupyter server

Environment data

VS Code version: 1.37.1
Extension version (available under the Extensions sidebar): 3.7.4
OS and version: macOS Mojave 10.14.6
Python version (& distribution if applicable, e.g. Anaconda): Python 3.7.4, Anaconda 4.7.11
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: N/A
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): false

Expected behaviour
Open a remote notebook on an authless remote server. Since the remote server does not use a token or password, the Jupyter server URI was specified with an empty token as -
http://host:port/?token=
Note the empty token. Also tried without the ?token=, same result. Code prompts for a server password, but since there is no server password, hitting enter results in the error below.
Actual behaviour
Failed to connect to remote Jupyter notebook.
Check that the Jupyter Server URI setting has a valid running server specified.
http://host:port/
Error: Failed to connect to password protected server. Check that password is correct.

Steps to reproduce:

Setup a jupyter server with no token or password
Try to connect to it

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
Starting Microsoft Python language server.
> conda --version
> pyenv root
> python3.7 -c "import sys;print(sys.executable)"
> python3.6 -c "import sys;print(sys.executable)"
> python3 -c "import sys;print(sys.executable)"
> python2 -c "import sys;print(sys.executable)"
> python -c "import sys;print(sys.executable)"
> /usr/local/anaconda3/bin/python -c "import sys;print(sys.executable)"
> conda info --json
> conda env list
> conda env list
> /usr/local/anaconda3/bin/python -m pylint ...
> conda info --json
> conda --version
> conda env list
> /usr/local/anaconda3/bin/python -m jupyter notebook --version

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
Same error reported under `Actual behaviour` above.

This might be a regression. We will look into this.

I am experiencing the same issue. Connecting to a remote server that manually has both the token and password set to nothing with the command line args to jupyter --NotebookApp.token='' --NotebookApp.password=''.  When I try to connect to it in vs code I get the error Error: Failed to connect to password protected server. Check that password is correct.

I just added a fix for this. The no token situation was regressed when password support was added. However one note, you do need the setting c.NotebookApp.disable_check_xsrf = True as well as setting the token to ''. We use the jupyter lab services to connect and they won't do a tokenless connection with xsrf check on.

Validated


Batch command not reading users / not working

Fill in this template completely. Issues not following this template will be closed and ignored.
Check the boxes below by filling [ ] with an x so it looks like [x].
Use the Preview button to ensure the template is filled in correctly.

[x] I am using the latest version of PyInstaLive: 3.2.3.
[x] I have installed either Python 2.7.x or 3.5+: 3.6.9
[x] I have read the README.
[x] I have read the FAQ.
[x] I have checked all existing issues, none of which could solve my issue.

I am using:

[ ] Windows 10
[ ] Windows 8.1
[ ] Windows 8
[ ] Windows 7
[x] Linux (distribution: Linux ubuntu 4.15.0-109-generic #110-Ubuntu SMP Tue Jun 23 02:39:32 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux )
[ ] macOS (version: )
[ ] Other (device & OS name: )

To report a bug, fill in the information below.
Required files
Please attach the log file of the user you were trying to download (if applicable) and the pyinstalive.default.log log file.
If your issue is related to assembling segment files please also include the JSON file and a zipped segment files directory.
If any of these files exceed the Github upload size limit in size please use WeTransfer or a similar service to upload these files.
PyInstaLive information
root@ubuntu:~/instalive# pyinstalive -i
[I] PYINSTALIVE (SCRIPT V3.2.3 - PYTHON V3.6.9) - 07-05-2020 02:02:34 PM
[I] To see all the available arguments, use the -h argument.
[I] PyInstaLive version:        3.2.3
[I] Python version:             3.6.9
[I] FFmpeg framework:           Available
[I] Cookie files:               1 (removed.json matches config user)
[I] CLI supports color:         Yes
[I] Command to run at start:    None
[I] Command to run at finish:   None
[I] Config file contents:
[pyinstalive]
username = removed
password = removed
download_path = /mnt/instalive
download_lives = True
download_replays = True
download_comments = true
show_cookie_expiry = True
log_to_file = True
ffmpeg_path =
run_at_start =
run_at_finish =
use_locks = True
clear_temp_files = False
do_heartbeat = True
proxy =
skip_merge = False

[I] End of PyInstaLive information screen.
Example:
$ pyinstalive --info
---------------------------------------------------------------------------
[I] PYINSTALIVE (SCRIPT V3.2.3 - PYTHON V3.6.3) - 01-02-2019 07:00:17 PM
---------------------------------------------------------------------------
[I] To see all the available arguments, use the -h argument.

[I] PyInstaLive version:        3.2.3
[I] Python version:             3.6.3
[I] FFmpeg framework:           Available
[I] Cookie files:               2 (johndoe.json matches config user)
[I] CLI supports color:         Yes
[I] Command to run at start:    None
[I] Command to run at finish:   None
[I] Config file contents:

    [pyinstalive]
    username = *removed*
    password = *removed*
    download_path = \path\to\downloads
    download_lives = True
    download_replays = True
    download_comments = True
    show_cookie_expiry = False
    ffmpeg_path =
    log_to_file = True
    run_at_start =
    run_at_finish =
    use_locks = True
    clear_temp_files = False

[I] End of PyInstaLive information screen.
---------------------------------------------------------------------------

Command used
Paste the command here that you are running. Don't forget to omit your username and password.
pyinstalive -u removed -b /mnt/instalive/users.txt -cp pyinstalive-batch.ini -o
Behavior
I am using the command above in loop with interval of 60s, expecting pyinstalive reads the users in users.txt and download the live; however it has never happened in batch mode.
Custom CP file:
root@ubuntu:~/instalive# cat pyinstalive-batch.ini
[pyinstalive]
username = removed
password = removed
download_path = /mnt/instalive
download_lives = True
download_replays = True
download_comments = true
show_cookie_expiry = True
log_to_file = True
ffmpeg_path =
run_at_start =
run_at_finish =
use_locks = True
clear_temp_files = False
do_heartbeat = True
proxy =
skip_merge = False
Loop bash file :
root@ubuntu:~/instalive# cat loop-script-batch.sh
while true
do
pyinstalive -u removed -b /mnt/instalive/users.txt -cp pyinstalive-batch.ini -o
sleep 60
done
################
The content of users.txt files
root@ubuntu:~/instalive# cat /mnt/instalive/users.txt
user1
user2
user3
user4

Downloading using a batch file works, you just can't use --organize together with any other commands.
Omit -o and it should work fine

Downloading using a batch file works, you just can't use --organize together with any other commands.
Omit -o and it should work fine

Thanks, I am not an expert in Linux. I want to run the batch file as a background task, generally, in this situation, I put "&" at the end of the bash script command; however, it doesn't work when I do Ctrl+c.
user1@ubuntu:~/instalive$ sudo ./loop-script-batch.sh &
[3] 1753
user1@ubuntu:~/instalive$ ---------------------------------------------------------------------------
[I] PYINSTALIVE (SCRIPT V3.2.3 - PYTHON V3.6.9) - 07-06-2020 01:19:40 AM
[I] Downloading 5 users from batch file.
^C
[3]+  Stopped                 sudo ./loop-script-batch.sh

Make sure the job is in the foreground before sending ctrl+c to it
Quick test on debian reveals no issues for me so i can't reproduce the problem you're experiencing

Even if it is a bit out of date, I do not recommend you to pass the parameters -b or —batch-file, as PyInstaLive monitors each individual person, which wastes time when the last person in the list is live but the first  Person still needs to be checked.  I recommend using -df —download-following instead.  PyInstaLive notices who is live at lightning speed, starts a background service and downloads all of them at the same time.


Added new learning platform - https://www.codewell.cc/

Added a new learning platform

@TheFlorz There are a lot of changes in the file. You can check it here: https://github.com/markodenic/web-development-resources/pull/352/files.
Please, start a new PR, but go step by step following this tutorial: https://markodenic.com/make-your-first-open-source-contribution/
Thanks!


EnchantGiver Visually Levels Up Enchantments, but According to IntelliJ, the Level is Still Whatever is Set in the Config

Upon a user doing some testing with MCDW, they found that some of the enchantments were not leveling up when using books or combining items. However, when using the commands within EnchantGiver, this does work. We did some more testing and found that this is the case with all innate enchantments set via EnchantGiver.
We also suspect that the reason for this is that in the EnchantmentHelperMixin, there is a point at which the return value is always set to the string value that it is in the config, though we might be misunderstanding the way that EnchantGiver is functioning. If there is anything that we can do to help with the resolution of this issue, please let Kluzzio or I know and we'll be as helpful as we can be.
Kluzzio thinks that a potential solution to the issue could be something along the line of what is below. Please disregard the red underlining, it is because it was written where IntelliJ was throwing a fit because of it:

Please forgive my write up if this is the wrong location for the code, but the general idea should stand wherever necessary to resolve this. I unfortunately did not do any testing to see if this code would work. Please let me know if you need anything from me, and thanks for your time!


Make vtables non-zero-size to fix a rustc warning.

warning: found non-foreign-function-safe member in struct marked #[repr(C)]: found zero-size struct in foreign module, consider adding a member to this struct

Emilio said on IRC:

the empty vtable means that we don't care of figuring out the proper vtable layout, so we create an empty struct

Sounds like all that matters is to have a pointer, we don’t look at the data behind it. Using c_void seems appropriate, then.

r=me

This fails with:
   Compiling tests_expectations v0.1.0 (file:///home/simon/projects/servo-deps/rust-bindgen/tests/expectations)
error[E0277]: the trait bound `std::os::raw::c_void: std::default::Default` is not satisfied
  --> tests/vtable_recursive_sig.rs:27:33
   |
27 | pub struct Base__bindgen_vtable(::std::os::raw::c_void);
   |                                 ^^^^^^^^^^^^^^^^^^^^^^^ the trait `std::default::Default` is not implemented for `std::os::raw::c_void`
   |
   = note: required by `std::default::Default::default`

I think it’s some CanDeriveDefault impl incorrectly returning true? I don’t really understand these.

Seems like that would prevent us from deriving Default for every type with virtual functions.
What about making it Vtable(*const c_void) instead? That would have a similar effect, and also grant proper alignment in the vtable struct (as useless as it is in the current state of affairs though).

How was Default derived before? What was the vtable pointer set to?

Specifically, the code generated before this PR was:
#[repr(C)]
#[derive(Default)]
pub struct Base__bindgen_vtable {
}
#[repr(C)]
#[derive(Debug, Copy)]
pub struct Base {
    pub vtable_: *const Base__bindgen_vtable,
}

Base does not derive Default, and can’t because *const _ does not implement Default. Why is Default for Base__bindgen_vtable useful?

It's null. The utility of Default is very debatable, that's why it's not default (no pun intended).

Base__bindgen_vtable: Default is not needed for Base::vtable_ to be set to null (however wrong that it), right?

Right, but it's needed to derive Default in Base.

Is it? That would require Default for *const Base__bindgen_vtable, not for Base__bindgen_vtable

@bors-servo r=fitzgen,emilio
Thanks!

:pushpin: Commit f67967a has been approved by fitzgen,emilio

:hourglass: Testing commit f67967a2930082ef54cde03d26dc3c0fe8f388cd with merge b4e3be9d25c583cb24a4b4d4587f292170c25b0d...

:sunny: Test successful - status-travis
Approved by: fitzgen,emilio
Pushing b4e3be9d25c583cb24a4b4d4587f292170c25b0d to master...


Chromium does not display full page.

I'm new with puppeteer and I want to know how can set my browser in order to display the full page because I set the size but does not matter how big I set the browser still display a big blank rectangle.

I don't want that display that blank space.

@qianx1992 Hi, when I use fullpage option, it's not working, it only screenshots the top part of the webpage, not the full page(from top and bottom).
const puppeteer = require('puppeteer');
const VIEWPORT = { width: 1920, height: 1080 };

(async () => {
    const browser = await puppeteer.launch({
        executablePath: './chromium/chrome.exe',
        args: ["--ash-host-window-bounds=1920x1080", "--window-size=1920,1048", "--window-position=0,0"],
        headless: false
    });
    const page = await browser.newPage();
    await page.setViewport(VIEWPORT);
    await page.goto('http://music.163.com/');
    await page.screenshot({ path: "test.png", fullPage: true });
    await browser.close();
})();


The Discord chat link in the docs is invalid

Invite's expired or something.
https://github.com/jonathandturner/rhai/blob/master/doc/src/about/related.md

Ouch!  I'm new to this Discord thing...  I was wondering why nobody joined!
:-)
Do you know how I can get a link for everybody that doesn't expire?

How about this?
https://discord.com/channels/767611025456889857/
Is this the right thing?

That works, thanks! Apparently the default invite links expire after a day, there's a small option at the bottom of the link creation dialogue box to extend the limit.

I'll update a new link with a new PR.


Removing redirection of cerr and cout messages

This redirection forces cout messages onto qDebug() but float values are badly interpreted and special characters appear

Concerning these lines, we already work on that here and on medInria, and it was solved. If there is still a problem, it should be solve on every OS, and on medInria and our fork. I am searching the linked PR from that problem

At one moment it has been removed in https://github.com/medInria/medInria-public/pull/703. The PR where @mjuhoor suggested to remove the 2 lines: https://github.com/Inria-Asclepios/medInria-public/pull/600. The current file with the 2 lines on medInria: https://github.com/medInria/medInria-public/blob/master/src/app/medInria/medLogger.cpp on master, and on medInria3.1.x: https://github.com/medInria/medInria-public/blob/medInria3.1.x/src/app/medInria/medLogger.cpp.
It seems that the medLogger from 3.1.x and master are not the same. We need to investigate indeed. But this should be on medInria.

What should we do? Create a new PR on medInria 3.2.x?

You could talk to Florent Leray (and I, since i worked also on it, if you want) about this problem which have already been looked at few months ago. If the problem remains, it should be solved, but not hide removing some of the logs from the log files or the terminal, i think

With the redirection mmg output info are shwon once the app is closed which is not good

Kill it.

(do not merge this PR, this will be handled on medInria)

https://github.com/medInria/medInria-public/pulls has been done, merged, and will be merged here. You can close this PR @paulineMig


[4.0] Scaffold for polyrepo generates tsconfig.build.json but looks for tsconfig.mjs.json

To reproduce, scaffold a polyrepo then run packemon pack --declaration
packemon pack --declaration

 ERROR

 Command failed with exit code 1: tsc --declaration --declarationDir mjs --declarationMap --emitDeclarationOnly
 --project tsconfig.mjs.json
 error TS5058: The specified path does not exist: 'tsconfig.mjs.json'.

 STACK TRACE

    at makeError (file:///Users/svallory/Library/pnpm/global/5/.pnpm/execa@8.0.1/node_modules/execa/lib/error.js:60:11)
    at handlePromise (file:///Users/svallory/Library/pnpm/global/5/.pnpm/execa@8.0.1/node_modules/execa/index.js:124:26)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Promise.all (index 0)
    at async Artifact.buildTypes (file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>e_modules/packemon/mjs/Artifact.mjs:134:5)
    at async Promise.all (index 1)
    at async Artifact.build (file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>ules/packemon/mjs/Artifact.mjs:59:5)
    at async file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>s/Package.mjs:48:9
    at async Promise.all (index 1)
    at async Package.build (file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>les/packemon/mjs/Package.mjs:44:5)
    at async Packemon.build (file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>ules/packemon/mjs/Packemon.mjs:29:7)
    at async PackCommand.build (file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>modules/packemon/mjs/commands/Build.mjs:24:5)
    at async PackCommand.pack (file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>odules/packemon/mjs/commands/Build.mjs:67:5)
    at async PackCommand.run (file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>dules/packemon/mjs/commands/Pack.mjs:9:5)
    at async Program.runAndRender (file:///Users/svallory/Library/pnpm/global/5/.pnpm/@boost+cli@5.0.0_ink@4.4.1_react@1
8.2.0_typescript@5.3.3/node_modules/@boost/cli/mjs/Program.mjs:464:24)
    at async Program.run (file:///Users/svallory/Library/pnpm/global/5/.pnpm/@boost+cli@5.0.0_ink<EMAIL_ADDRESS>escript@5.3.3/node_modules/@boost/cli/mjs/Program.mjs:288:18)
    at async Program.runAndExit (file:///Users/svallory/Library/pnpm/global/5/.pnpm/@boost+cli@5.0.0_ink@4.4.1_react@18.
2.0_typescript@5.3.3/node_modules/@boost/cli/mjs/Program.mjs:307:22)
    at async run (file:///Users/svallory/Library/pnpm/global/5/.pnpm/packemon<EMAIL_ADDRESS>on/mjs/bin.mjs:39:3)

Fixed


AttributeError: 'str' object has no attribute 'get'

Steps to Reproduce
in config.json set "hatch_eggs": true
Other Information
OS: Debian
Git Commit: 6c6fc59de9ea02a2079cb7548f80937f65894088
Remark
The bot run normally when i change hatch_eggs to false

it seem the outdate python package (pgoapi) here.
https://github.com/PokemonGoF/PokemonGo-Bot/issues/1248#issuecomment-236026985
run su pip install --upgrade -r requirements.txt also not working
i just rm -rf /usr/local/lib/python2.7/dist-packages/pgoapi* and then run su pip install --upgrade -r requirements.txt again, it work for me now :laughing:


Prevent bots/analyze.dart crashing on circular dependencies

The bot rolling packages is failing here:
https://github.com/flutter/flutter/pull/123350#issuecomment-1490722596.

Exhausted heap space, trying to allocate<PHONE_NUMBER>4 bytes.
Unhandled exception:
Out of Memory

It appears that there's a circular dependency between test_api and matcher which this script gets stuck following until it runs out of memory.
This change preferences adding packages that have already been added, and prevents the crash in my local machine.

It appears that there's a circular dependency between test_api and matcher which this script gets stuck following until it runs out of memory.

Will that circular dependency cause any other trouble? It seems like a code smell, but I don't know what (negative) effects it could have.
If circular dependencies like that are undesirable, the script should probably fail gracefully instead of OOMing or just ignoring this.

Will that circular dependency cause any other trouble? It seems like a smell, but I don't know what (negative) effects it could have.

Mostly it causes pain while publishing those packages. There are potential impacts on modular builds, but we are sure with these packages that we never introduce module cycles.

@natebosch said on the other issue:

It is expected temporarily. We will break the cycle in the next breaking version of test_api, but I'll first have to clean up a bunch of places in flutter/flutter where test_api was imported against the deprecation warning.

@natebosch Do you have an issue for breaking that cycle that we can link here to remove this workaround and instead fail the check gracefully if a cycle is detected?

Do you have an issue for breaking that cycle

I can file, one... https://github.com/dart-lang/test/issues/1979

and instead fail the check gracefully if a cycle is detected?

It's not clear to me why we'd specifically want to fail. Package cycles are not intended to cause pain for anyone outside of the publisher.


Make sublimetext generator create import paths for DKit

Simplifies project setup by setting include_paths for DKit and therefore DCD.

LGTM. Need to fix/disable that unrelated failing test...


Filter API fetch using the mapping scheme to mood

good luck

Created mood mapping to the api moods and then have a function to return the moods from the api


Support for php 8 and mysql 8 on ubuntu 20.04

I create a simple docker file to support PHP8, mysql 8 on ubuntu 20.04 based on your code.
I'm using your container for my project and created this for laravel 8 projects.

Thanks for your contribution @droni-co, #123 fixed this with the above mentioned tests included so I will be closing this PR for now
Thanks,
Matt


about NativeMailerHandler

Hello .. Good Night
I'm testing the following process but is not firing e-mail
How should I proceed?

use Monolog\Logger;
use Monolog\Handler\StreamHandler;
// create a log channel
$log = new Logger('name');
//
$from =<EMAIL_ADDRESS>$to =<EMAIL_ADDRESS>$subject = "Logger \Monolog\Handler\NativeMailerHandler";
$body = 'body';
$nativeMailerHandler = new \Monolog\Handler\NativeMailerHandler($to, $subject, $from);
$log->pushHandler($nativeMailerHandler);
// add records to the log
$log->addWarning($body);

By default the mailer handler uses the ERROR level to avoid sending to much emails for nothing, you can change that with $nativeMailerHandler = new \Monolog\Handler\NativeMailerHandler($to, $subject, $from, Logger::DEBUG); if you like.


GPTBot IP ranges download error

➜  bash openai/downloader.sh
+ curl -s https://openai.com/gptbot-ranges.txt

➜  echo $?
1

When downloading from the IP GitHub runners we get an error from CloudFlare (need to pass captcha). And both links do not work:

https://openai.com/gptbot-ranges.txt
https://openai.com/gptbot.json

What to do — still not clear. Perhaps we need to write to OpenAI, so that they unblock access to GitHub runners.
But where to write?

Temporary hotfix: https://github.com/lord-alfred/ipranges/commit/5f88342b2434cd313c2a062a389d9bc2c71d3d75


[WIP] Add missing loop arguments.

This pull requests adds the loop argument in places #17 forgot to add them. Please do not merge this yet as I want to add tests that run this code before finishing the pull request.

All commits in this branch are also in #20 so I'm closing this one.


[GLUTEN-3676][CH] Enable TPCH Deicmal Test

What changes were proposed in this pull request?
(Fixes: #3676)
How was this patch tested?
Test by ut

Let's create an issue for traceing ignored ut. We ignore a Decimal test in the PR of fixing decimal.

Let's create an issue for traceing ignored ut. We ignore a Decimal test in the PR of fixing decimal.

follow by issue #4026 .

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_3677_time.csv
log/native_master_12_12_2023_8c6f1643e_time.csv
difference
percentage

q1
33.98
33.75
-0.236
99.30%

q2
24.68
24.88
0.201
100.81%

q3
38.28
37.71
-0.573
98.50%

q4
38.86
37.86
-0.999
97.43%

q5
71.89
72.83
0.939
101.31%

q6
7.09
7.05
-0.042
99.41%

q7
86.76
86.50
-0.264
99.70%

q8
88.13
86.24
-1.887
97.86%

q9
123.41
125.50
2.092
101.70%

q10
45.01
44.68
-0.329
99.27%

q11
20.60
20.21
-0.385
98.13%

q12
24.60
25.06
0.464
101.89%

q13
47.02
46.58
-0.443
99.06%

q14
14.36
19.99
5.626
139.17%

q15
28.18
27.55
-0.638
97.74%

q16
15.55
15.86
0.306
101.96%

q17
104.08
103.27
-0.808
99.22%

q18
151.71
151.33
-0.377
99.75%

q19
12.81
13.89
1.083
108.46%

q20
29.79
28.68
-1.101
96.30%

q21
229.00
229.12
0.126
100.06%

q22
15.14
13.95
-1.192
92.13%

total
1250.93
1252.49
1.561
100.12%


Profile Image moves with scrolling

Profile image comes down as I'm scrolling through the pages

The header was purposefully remove on a8403f5fe, this is not a bug.

What about the profile image?
Is it supposed to come down as I'm scrolling through the page?

It shouldn't. I tried but wasn't able to reproduce. Can you post browser version and extension version so I can take a look? I'll re-open this until the image profile issue is cleared

Seems like the bug has been patched. Never mind.

Seems like it's still here.

It might not be common, but it is a bug.

I'm going to need some time to reproduce the bug, don't close this issue in the mean time.

No problem, we'll wait! It's just that I wasn't able to reproduce using my extensions, so maybe something is conflicting. I'll investigate it anyway while you do that! Thank you @caxvis.

I've disabled all of my extensions except for Refined Twitter, all the twitter pages we're loaded pretty fast. The bug never occurred.
It probably has some conflict with DuckDuckGo or one of my extensions, because when I enable all of my extensions again, the twitter pages we're being loaded pretty slow.
So no, this is not an issue with Refined Twitter, it is an issue with one of my extensions. This issue will now be closed.


Is it possible drag and drop?

I have visited https://github.com/jquense/react-big-calendar and with the addon option of the drop-down menu it is possible to use drag and drop.
Is it currently possible to do this in Obsidian Big Calendar?

Of course! But now I have to make the plugin pass the review from Licat, so will be available in  0.3.0

Great!


backcompat implying properties

according to
http://microformats.org/wiki/microformats2-parsing#parsing_for_implied_properties
Imply properties only on explicit h-x class name root microformat element (no backcompat roots):
https://github.com/microformats/tests/blob/master/tests/microformats-v1/hcard/single.json
imply's a name: for the h-geo

https://github.com/microformats/tests/blob/master/tests/microformats-v1/hnews/all.json
here as well, same issue


[os] descriptor from string defaults to default descriptor

What does this PR do?
Default descriptor to a valid descriptor instead of to an os flavor
Which scenarios this will impact?
docker and vm
Motivation
The docker run function is currently broken, as it panics if a valid os descriptor is not defined on the config map
Additional Notes

/merge

@pducolin This PR has broken usage of inv docker command. If you input ::arm64, it will always use Ubuntu regardless of the default flavor set in scenarios/aws/ec2/vm_run.go


Ensure IP is not already registred on other farms before adding it

Which package/s are you suggesting this feature for?
Dashboard
Is your feature request related to a problem? Please describe
https://github.com/threefoldtech/tfchain_graphql/issues/157
Describe the solution you'd like
Add a validation on add ip form to ensure IP is not already registred on other farms before adding it

Verified on Dev
Version: ac03039
The validation that checks for an existing IP has been added successfully.

verified on devnet
version : c112478

testcase


Add a subject field for appointments

The subject should probably be an enum with Math, Science and English. It should also probably be optional.

Closed by #106


Support rolling with numbagg

Is your feature request related to a problem?
We can do plain reductions, and groupby reductions with numbagg. Rolling is the last one left!
I don't think coarsen will benefit since it's basically a reshape and reduce on that view, so it should already be accelerated. There may be small gains in handling the boundary conditions but that's probably it.

Yes, we can definitely do this! Need to write the routines (only have move_mean atm).
But tbh that's the easy part (maybe the first huge benefit I've found from LLMs, fwiw)

Done in numbagg for all the easy ones — https://github.com/numbagg/numbagg/pull/174. So much faster than pandas. I can't actually test bottleneck locally, since it doesn't seem to compile on ARM... I can try and get it running on GHA (though the dev loops will be frustrating...)
Others like move_nanmax are actually not trivial, particularly with numba, which doesn't have the data structures to implement this; e.g. a dequeue. I'll think about it some more, ideas welcome!
@dcherian should we push this through flox? Or add to xarray directly from numbagg?

This is done!


Persona's

Persona's created based on user interviews conducted in Term 1.
Figma file:
Persona 1
Persona 2

Thanks @avantikajain289!
We've just migrated the CNCF landscape to v2, so I'll close this one for now.
Please feel free to reopen if needed 🙂


Need help debugging authentication issue

Hello,
I am trying to use keystoneauth-oidc plugin to authenticate to our Openstack, mod_auth_openidc setup.  We are using Globus as OpenID client.  Any suggestions on how to troubleshoot this would be very much appreciated.  - Augustine Abaris
openstack -v --debug --os-auth-url https://XXXXX.fas.harvard.edu:13000/v3 \
      --os-auth-type v3oidccode \
      --os-identity-provider nercglobusidp \
      --os-protocol openid `\`
      --os-identity-api-version 3 \
      --os-project-id "XXXXXXXXXXXXXXXXXXXXXXXXX" \
      --os-discovery-endpoint "https://auth.globus.org/.well-known/openid-configuration" \
      --os-client-id "XXXXXXXXXXXXXXXXXXXXXXXXXXXX" \
      --os-client-secret "XXXXXXXXXXXXXXXXXXXXXXXXXXXX=" \
      --os-openid-scope "openid profile email" \
      token issue

I have verified a working globus client, and retrieved client-id and cilent-secret from the config.  I can confirm that globus issues an access token when running "openstack token issue" command.
{
  "access_token": "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
  "scope": "email openid profile",
  "expires_in": 172800,
  "token_type": "Bearer",
  "resource_server": "auth.globus.org",
  "other_tokens": [],
  "id_token": "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX"
}

The error I'm getting is:
REQ: curl -g -i -X POST https://XXXXX.fas.harvard.edu:13000/v3/OS-FEDERATION/identity_providers/nercglobusidp/protocols/openid/auth -H "Authorization: {SHA256}XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX" -H "User-Agent: openstacksdk/0.55.0 keystoneauth1/4.3.1 python-requests/2.25.1 CPython/3.7.9"
Starting new HTTPS connection (1): XXXXX.fas.harvard.edu:13000
https://XXXXX.fas.harvard.edu:13000 "POST /v3/OS-FEDERATION/identity_providers/nercglobusidp/protocols/openid/auth HTTP/1.1" 401 381
RESP: [401] Content-Length: 381 Content-Type: text/html; charset=iso-8859-1 Date: Thu, 08 Apr 2021 16:43:37 GMT Server: Apache WWW-Authenticate: Bearer error="invalid_token", error_description="Reference token could not be introspected"
RESP BODY: Omitted, Content-Type is set to text/html; charset=iso-8859-1. Only application/json responses have their bodies logged.
Request returned failure status: 401
Unauthorized (HTTP 401)

On mod_auth_openidc side, I'm seeing:
[Wed Apr 07 17:01:53.687162 2021] [auth_openidc:error] [pid 108663] [client X.X.X.X:47892] oidc_util_json_string_print: oidc_util_check_json_error: response contained an "error" entry with value: ""unsupported_grant_type""

no longer relevant


Using Flexbox to create a responsive comment section

Proposed article introduction
Flexbox is a new way for building layouts in most websites. It is supported by most websites in our today world.
The layout of weight is very important as it can tell you more about your company. When people will be navigating a website with good layout it would be easier for them to follow.
Here I will be showing you how to create responsive comment section using flexbox.
Key Takeaways
At the end of the article one should be able to:
•	Use flexbox
•	Creating various layouts
•	One can create a responsive comment section
Quality of the article
This article will help developers learn how to use flexbox without challenges. It is easier to understand and follow the tutorial.

This topic appears oversaturated on other blog sites and official docs. Feel free to suggest another topic. Thanks


Can I call swapSuggestions method with no animation?

When I want to update suggestions list with the new result set and call swapSuggestions method, the list performs an animation when is showing the new results, but with my large suggestions list I want to use no animation.
Is there a way to set up the floating search view to do the swap suggestions with no animation?
Thanks in advance,

Hi @juancitogo ,
You can simply set the animation duration to 0 in code: mSearchView.setSuggestionsAnimDuration(0); or
in xml: app:floatingSearch_suggestionsListAnimDuration="0"
Regards,
Ari

Hi @arimorty ,
thanks for your help.
I test it with both methods but animation on suggestion list is still running.
In xml:
<com.arlib.floatingsearchview.FloatingSearchView
        android:id="@+id/floating_search_view"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        app:floatingSearch_dismissOnOutsideTouch="true"
        app:floatingSearch_leftActionMode="showHome"
        app:floatingSearch_searchBarMarginLeft="@dimen/search_view_inset"
        app:floatingSearch_searchBarMarginRight="@dimen/search_view_inset"
        app:floatingSearch_searchBarMarginTop="@dimen/search_view_inset"
        app:floatingSearch_searchHint="Search..."
        app:floatingSearch_showSearchKey="false"
        app:floatingSearch_menu="@menu/menu_search_view"
        app:floatingSearch_suggestionsListAnimDuration="0"/>

and using method:
mSearchView = (FloatingSearchView) findViewById(R.id.floating_search_view);         mSearchView.setSuggestionsAnimDuration(0);
Is there any other attribute that I need to set?
Thanks,

+1, disable animation not work

Hi @arimorty
setting the animation duration to 0 works on android version superior to 5.1.1
do you plan some fix for versions 5.1.1 and lower?
thanks,

@juancitogo The animation your are referring to is the a result of a list scroll. The next version of the library has significant changes put in place in order to avoid that scroll.
Thanks for contributing!
Best,
Ari


Safari iOS9 broken

I apologize if you are already aware of this, but there appears to be an error that was introduced in iOS 9’s version of safari. It is causing the entire globe texture and tile layer to not render leaving the globe blank. I have found a couple other WebGL based API’s that have run into this same error and found the changes they did to work around this error.
https://github.com/NASAWorldWind/WebWorldWind/commit/ce5d1f2560abf1e3e9e0fbcc14338013c8938c29?diff=split
https://github.com/xml3d/xml3d.js/commit/00f8f3018cf37ecb7bc7ddd265b21d1000bc4ebb?diff=split
In the 1.14 version of Cesium they also mentioned in their changelog that they have created a fix for this by “hard-coding constants instead of accessing them through ‘WebGLRenderingContext.’”
http://cesiumjs.org/2015/10/01/Cesium-version-1.14-released/
I do not have the background or full knowledge of how to fix this error myself so I am hoping that this helps you and your team in creating a fix instead.
I am very impressed and thankful for all the work you’ve done and hope to see this issue resolved soon.
Thank you again,
Mitchell Adams.

@klokan Can you please confirm this is fixed now? It seems to be working okay on iPad with iOS 9.0.2.

Yes this issue is now fixed from what I can tell, thank you.

Tested. Thanks @petrsloup


OnConnectAuthenticate cannot specify an error code (like Client Identifier not valid)

The conclusion of issue #398 was that the Server shouldn't require usernames to match in a session takeover; instead an OnConnectAuthenticate hook should do this if desired, i.e. rejecting the login if the clientID matches a persistent session but the username does not.
However, the OnConnectAuthenticate hook cannot specify an error code; all it can do is return false, in which case the client is sent a "Not authorized" error. This is misleading: the error code implies the client credentials are incorrect, but instead it's the client ID that's incorrect. This could be really confusing for a developer, especially since it would likely be an intermittent problem.
The proper error code would be "Client Identifier not valid" or "Connection Refused, identifier rejected" -- if the client got that error instead, it would know to generate a different session ID and retry. Or at least the developer would have a better idea what's going on and how to fix it.
It seems the OnConnectAuthenticate hook should be extended so it can return different error codes. But changing the return type would break API compatibility. I don't know what approach would be best. Maybe define a new hook method OnConnectAuthenticate2 with a different signature, and have Server call that one instead if it's provided?

@snej According to the MQTT v5.0 spec, each Reason Code has a specific meaning. Please refer to MQTT v5.0 Section 2.4 Reason Code.
| -- | -- | -- | -- |
| 32 | 0x84 | Unsupported Protocol Version | CONNACK |
| 133 | 0x85 | Client Identifier not valid | CONNACK |
| 134 | 0x86 | Bad User Name or Password | CONNACK |
If we add an OnConnectAuthenticateWithErr() hook, the error returned by the hook must correspond to one of the Reason Codes described in the spec. However, it's difficult to ensure that users of this hook will fully understand and return the correct Reason Codes.


Add discord invite to README

Here's the link: https://discord.gg/ea74aBray2

Hello! I would like to contribute to this issue :)


netkvm: 32 queues boot time take much longer

Hi,
We found that when multi-queue was enabled, the more the number of queues the longer the windows boot time will take. If set the queue number to 32, the boot time will take about 3 min 30 sec.
I check the boot log and found that the most time is take in function 'ParaNdis_VirtIONetInit'. There is a for loop:
    for (i = 0; i < pContext->nPathBundles; i++)
    {
        if (!pContext->pPathBundles[i].rxPath.Create(pContext, i * 2))
        {
            DPrintf(0, ("%s: CParaNdisRX creation failed\n", __FUNCTION__));
            return status;
        }
        pContext->pPathBundles[i].rxCreated = true;

        if (!pContext->pPathBundles[i].txPath.Create(pContext, i * 2 + 1))
        {
            DPrintf(0, ("%s: CParaNdisTX creation failed\n", __FUNCTION__));
            return status;
        }
        pContext->pPathBundles[i].txCreated = true;
    }

This loop will take minutes to finish initialize the nic, I wonder is there any way to improve this? Thanks!
The boot time is list below:

Queue num
First boot
Normal boot

1
50sec
30sec

2
50sec
30sec

4
50sec
30sec

8
1min5sec
35sec

12
1min10sec
40sec

16
1min20sec
50sec

20
1min50sec
1min5sec

24
2min15sec
1min30sec

28
2min50sec
2min

32
3min10sec
2min30sec

@oyzw0720  - do you have any updates regarding the patches that you tried?
Thanks,
Yan.

@YanVugenfirer
We checked our source code and found that  it has already applied that patch, and the issue is not caused by this bug.
But we also found that if we set the memory much larger (32 GB for instance), the boot time will take much shorter. So the more queues you set, the more memory you need to keep the boot time normal.

Thank you for the update.

My pleasure~

I am closing this issue, please reopen if you believe this has not been resolved.
Thanks for reporting :)


✈ Implement claim/unclaim/reclaim packages

✈  This is the flight deck for the Claim/unclaim/reclaim packages project. ✈

🗻  That project is part of the Make it easier for companies to fund open source epic. 🗻

Specs
See https://github.com/gratipay/gratipay.com/issues/4297#issuecomment-273900610.
Mocks
From https://github.com/gratipay/gratipay.com/issues/4297#issuecomment-273854209 (click for full-size):

Prereqs

[x] make room on the /on/npm/foo/ page for the claiming workflow—move the name to the banner h1, and replace the wombat with the description: #4304
[x] new email subflooring: #4343 #4344 #4345 #4365 #4350 #4355 #4346 #4347 #4348 #4349 #4383
[ ] pull emails from maintainers only, not author (https://github.com/gratipay/gratipay.com/pull/4305#issuecomment-278429627)

[ ] also dedupe email addresses

[ ] set up npm syncing—besides just needing to stay in sync, people are going to want to update the email address on file in their package.json release a new version and then "immediately" see the new email show up to continue the flow

Todo (in merge order)
Numbers are workflow steps.

[x] (1), (2) add sign in w/ copy to /on/npm/foo/ for anon: #4306 #4307 #4308 #4323
[x] stub out Package to hold Python API: #4155
[x] (3)—(6) add confirmation form to /on/npm/foo/ for auth when unclaimed: #4309
[ ] (7), (8) implement Package.send_confirmation_email: #4335
[ ] (9) add /on/npm/react/claim endpoint to receive email link: #4397
[ ] (9.ii) add claim to the Python API
[ ] (10)—(11) implement /on/npm/foo/ as project page when claimed
[ ] (12)—(16) implement /on/npm/foo/edit as project edit page when claimed
[ ] (16.i) add unclaim to the Python API

Also:

[ ] make sure package projects can be reviewed and rejected
[ ] highlight/preselect any already-verified email(s) on the confirmation form
[ ] think through merging/takeover—anything special here? like, start verification workflow, then merge accounts, then finish verification.
[ ] style the confirmation form so it's not so plain
[ ] change base back to master

✈  This is the flight deck for the Claim/unclaim/reclaim packages project. ✈

Could you share the plan of this project?
Besides, I guess you haven't started paying for gomockingbird? Shall we apply to balsamiq earlier than later?

Could you share the plan of this project?

Not sure what you mean. I updated the todo in the ticket description. Not enough?

Besides, I guess you haven't started paying for gomockingbird?

Nope.

Shall we apply to balsamiq earlier than later?

Not a priority for me. a) Balsamiq appears to be desktop-only, which is of limited utility for us since we can't share links, b) I think we have what we need from Mockingbird for now, and c) I think we can hack Mockingbird by deleting cookies/localstorage.

Not sure what you mean. I updated the todo in the ticket description. Not enough?

Maybe just me, but I don't find a easy way to tell which part a dev (me perhaps) can pick up without worrying about stepping on your toe.

a) Balsamiq appears to be desktop-only, which is of limited utility for us since we can't share links,

they offer choices in application including web version:

c) I think we can hack Mockingbird by deleting cookies/localstorage.

Seriously?

Maybe just me, but I don't find a easy way to tell which part a dev (me perhaps) can pick up without worrying about stepping on your toe.

I think we should get you unblocked and focusing on https://github.com/gratipay/finances/pull/35. I'm happy to do the bulk of the work here but I will need you @mattbk @JessaWitzel etc. for review. Let's work with who we have.

they offer choices in application including web version:

Go for it!

Seriously?

😊

@nobodxbodon at https://github.com/gratipay/gratipay.com/issues/4297#issuecomment-275050681:

When there are multiple contact emails for one package, indicating there are more than one devs, do we send notification emails out to all of them in case the package is claimed by one of the email?

Good idea!

We'll also need to account for the case where an email is attached to a different Gratipay account. They won't be able to use it in that case.

Todo reordered to reflect merge order.

Picking up from https://github.com/gratipay/gratipay.com/issues/4297#issuecomment-275438361:

I was wondering the same thing. Which email/person do we allow to claim the package.

We pull email addresses from the maintainers and authors in package.json on npm.

In case you missed https://github.com/gratipay/gratipay.com/issues/4297#issuecomment-275050681, shall we notify all the contacts after step 8 of spec, and states so to the claimer in the email something like "we'll notify all the other coauthors of the package in this email list after you confirm claiming"?

Is any one of the Co-authors of a package allowed to claim the packages?
What if the co-author is a new-comer claims the package before the real starter of the package gets a chance to.

That was really what I was trying to ask ealier

What's up with the failures here? I seem to recall expecting this ...

Still looking for the perfect process here. I keep switching the todo order. Currently working stepwise through the workflow.
I've got the confirmation flow (3-8) half roughed-in. I think at this point I'm going to go back over and clean up and merge https://github.com/gratipay/gratipay.com/pull/4155 so that we have a Package object from which to hang API, and then pick up again with #4309. At some point we also could/should rebase this on master to pick up recent changes there.

…

Is any one of the Co-authors of a package allowed to claim the packages?

No (under):

The thing that seems the most canonical representation from what I've seen around probably is whatever the owners are of the actual npm package vs. what's in the package.json.

Sent a follow-up in private email to clarify:

Looking at the registry, I see three options:

_npmUser—scoped to version
maintainers—scoped to package
author—scoped to package

My hunch based on our convo is that the _npmUser for the latest version is the best one to use for linking (though the _ suggests that this is a private API, and a spot check shows it is missing for some versions, maybe very old ones?). Does that sound right to you?

Alright, based on that and considerations at https://github.com/gratipay/gratipay.com/pull/4309#issuecomment-278430672 I've added another prereq:

update schema for a single (nullable) email & sync from the latest version's _npmUser

Okay! Shelving work on #4309. I had thought of two PRs right after that one (implementing .send_confirmation_email), and then confirmation link post-back). But now it seems I should dial back to working on npm syncing prereqs.

Scratch that! 💃

Taking a look at a few packages, it is definitely the maintainers field that I had in mind, as the corresponds to the users who have the rights to publish the package, which is the most likely to align to the folks who would manage any funds, imo

But we do want to drop authors.

/me adds that to prereqs ...

Back into #4309!

Squashed and rebased onto #4343, was 5bab99a967a36374a4e8272b20f81db5f96f7f1c.

Base changed to #4343.

+1 from https://twitter.com/_developit/status/836413846067810304.

an API to look up the opencollective for a given npm module, or the reverse

... and with that, we've come full circle. ;-)

automated gittip based on npm installs / git clones. I say I want to give $100/month and it divides it automatically.

https://github.com/OpenCollective/OpenCollective/issues/178

I've got a local rebase which slightly changes Checkpoint 1 (9ea1b0cfe4ab48e71db3c08b6a5ee83cdf00c419). It's failing in the ttw tests, however. Investigating ...

Still investigating ...

Ah! local.env misconfig.

Rebased, was 9ea1b0cfe4ab48e71db3c08b6a5ee83cdf00c419.

I am getting the sense that we may need to have me self-merge some PRs if we're to keep up our momentum here. I propose to do so tomorrow unless someone objects.

Okay! Looong email pre-req chain is in! Back to #4335! 💃 👍

Rebased onto #4391 in anticipation of its merge. Enough wasting time restarting Travis! 😖

The marketing announcement once we land this feature could/should frame it as an improved application process for npm packages. "One click."

I think we may need to implement bulk claiming here though. I'm building it out for such under the hood, easy enough. And I don't think we'll want to return to this once we've moved on, we'll want to focus on demand-side concerns (invoicing, email auth, etc.).

Needs to be rebased against master, yh?

Done! Was e17ea8152ed98a3180e84c4ad650f9c966df233a.

@mattbk in Slack:

Is there a review process for npm? On your other screenshot, I noticed it had the "in review" tag. If npm can act as a voucher for projects, do we need to put them through our own (7-day) review?

I think we should still put package-projects through our review process, because it's actually pretty trivial to create an npm package. Hopefully though this will make it easier to apply for multiple projects at once, and we can review all together vs. separately. Basically this is a streamlined way to apply for a project, but otherwise the process is the same. Eh?

That sounds fine to me. We can leave grouped reviewing as a manual process for now. Being swamped with reviews sounds like a good problem to have.

I'm pretty close on review url adaptations. Will pick up tomorrow ...

@dowski I didn't make any more progress on https://github.com/gratipay/gratipay.com/pull/4397 today, and I want to keep you topped up in case you have any spare cycles tonight and tomorrow. I'm putting this whole PR into Review. The whole chain under "Todo (in merge order)" in the description is ready for a first pass. 🙇

@whit537 Roger that. Going to review some more this evening.

#4405 from the checklist looks merged already. Should it be checked in the list?

Ah, good catch. That was actually merged into #4404 (not here), so I've just removed it from the checklist.

Reticketed select improvements as https://github.com/gratipay/gratipay.com/issues/4415.

bulk-add from /on/npm/ based on verified email addresses

Reticketed as https://github.com/gratipay/gratipay.com/issues/4416.

add packages to fake data

Reticketed as https://github.com/gratipay/gratipay.com/issues/4417.

Rebased on latest master, primarily to pick up #4414; was baf8248083660c70cdd64ae9cb84012b47f7333c.

Okay! Remaining open PRs in the chain are topped up, rebased, and ready for review! I'm going to forge ahead with new work.

think through PayPal requirement—are we accounting for that somewhere here that I'm not seeing?

No, we're not. And while I'm fine with #1383 in principle, I'm not ready to make a change there until we can afford some proper fintech counsel (cf. https://github.com/gratipay/inside.gratipay.com/issues/842). We need to enforce the PayPal constraint here.

consider notifying other package maintainers when someone claims a package

This is complicated because our current infrastructure assumes that an email address is linked to a participant. Either we need to create stub participants for all relevant emails (we do something similar for social accounts, but then I guess we want to be able to "promote" the stubs? does that get us into #1052?) or undo that assumption so we can mail without having a participant.

consider notifying other package maintainers when someone claims a package

Isn't it the package maintainers' responsibility to trust each other? At MVP stage, shouldn't we just trust that they all trust each other?

Isn't it the package maintainers' responsibility to trust each other? At MVP stage, shouldn't we just trust that they all trust each other?

I guess so. You okay with that, @kaguillera?

Talked with @kaguillera IRL, agreed to bump these two:

consider notifying other package maintainers when someone claims a package
indicate to a participant when they have a claim open

Reticketed as #4425 and #4426 respectively.

Text overflows poorly at low width.

Flight deck shifted to https://github.com/gratipay/gratipay.com/issues/4427, since we want to land this but the Claim packages project is likely at least a bit wider in scope than what we want to include here.

Alright! I am done making new PRs on here. I made #4435 for low-hanging fruit on low-width display. There are deeper interaction negligencies on mobile that I've bumped to #4432. Given that this PR doesn't even include any discover mechanisms apart from direct URL access, I don't think we should load this up further with non-essential fixes. Let's get this integrated, merged, and out the door!
I'm going to consolidate recent merges into another checkpoint and then check it out locally. That'll leave us with two big commits and then the three remaining PRs. 👍

Rebased, was<PHONE_NUMBER>81ce7512cfda27ea41217cb9954e25.

Alright, this rebase doesn't look right.
364634355781ce7512cfda27ea41217cb9954e25 (old) has a pretty complex PackageLinking.check method.
771c40ffdea51be52de987f901d26d5902db78f2 (new) does not.

Zoiks! Lost it locally? 😳
[gratipay] $ co<PHONE_NUMBER>81ce7512cfda27ea41217cb9954e25
fatal: reference is not a tree:<PHONE_NUMBER>81ce7512cfda27ea41217cb9954e25
[gratipay] $

Not encouraging:

Anyone who doesn't have C can't recover C at all (even if it's in origin's object store).

Alright, yeah, looks like I effed up. Reading through my reflog, I'm pretty sure what happened is that I didn't update my local projects/claim-packages branch before rebasing it on master. I rebased f50cf4efd098d317573d31585c1475a8ec0c0cd1, which is old. Therefore, when I force-pushed, I pushed off all of the lovely merge commits that @dowski had made. I still have the important parents of the merge commits locally, and GitHub still has the now-orphaned merge commits themselves. But it appears that there is no way for me to get recover those commits that are orphaned at GitHub back into my local repo.
Sorry. 😞
I asked @kaguillera and he doesn't have them. I believe that @dowski is just reading code on GitHub and doesn't have a local working copy. It's possible that @mattbk or someone else has them—possible, but unlikely.
I think what I'm going to do for now is redo the last merge commit and push that up here, and then try again. If someone comes up with<PHONE_NUMBER>81ce7512cfda27ea41217cb9954e25 locally we can revisit.

@mattbk and @clone1018 don't have it. [slack]
Given that all these commits are getting squashed anyway (and the merge commits are simply tossed away in the process), it's not actually that big a deal.

New plan!

Next, use Github’s Refs API to create a new branch pointing to that commit:

😍
Status: 201 Created
[...]

{
  "ref": "refs/heads/recovery",
  "url": "https://api.github.com/repos/gratipay/gratipay.com/git/refs/heads/recovery",
  "object": {
    "sha": "364634355781ce7512cfda27ea41217cb9954e25",
    "type": "commit",
    "url": "https://api.github.com/repos/gratipay/gratipay.com/git/commits/364634355781ce7512cfda27ea41217cb9954e25"
  }
}

#!/bin/sh
curl -i \
     -H "Accept: application/json" \
     -H "Content-Type: application/json" \
     -X POST \
     -d '{"ref":"refs/heads/recovery", "sha":"364634355781ce7512cfda27ea41217cb9954e25"}' \
<EMAIL_ADDRESS>

Oh heck yeah.
[gratipay] $ gf
remote: Counting objects: 8, done.
remote: Total 8 (delta 0), reused 0 (delta 0), pack-reused 8
Unpacking objects: 100% (8/8), done.
From github.com:gratipay/gratipay.com
 * [new branch]      recovery   -> origin/recovery
[gratipay] $

Back on track! 💃

Okay! Have cleaned up locally—and made a tmp branch for recovery!—and am ready to try again ...

Yeah, this looks better. Now rebasing 68 commits, was 34 previously.

Okay, pushing ...
[gratipay] $ git branch -v
  backfill-remaining-exchanges 063a8cf Start looking at the exchange side
  master                       8f6dac5 Bump version to 2071
* project/claim-packages       eb5921f [ahead 8, behind 76] Checkpoint 2
  tmp                          3646343 Merge pull request #4423 from gratipay/project/claim-packages-heart
[gratipay] $

Alright, rebasing remaining PRs went off without a hitch (conflicts there are what tipped me off to the problem initially).

Awesome. All PRs are in!
I guess we should spend some effort on syncing npm before we deploy this, though. :-/

The snapshot of npm we have in the Gratipay database is six months old. We need to update that and also build in a mechanism for constantly updating it.

Rebased and squashed, was 70b391e5c241558d3658bb52d3e6729c9108ee4f.

Okay, friends! I believe we are ready to land this sucker!

@clone1018 and I are going to merge and deploy this.

Rebased, was 8a2e258d07d399d6152f59057120d4c0a1624818.

Travis is running super-slow.

Finally green! Ready!?


Revise CenterCrop

The CenterCrop might be problematic for datasets where:

the spinal cord is absolutely not at the center (eg the edge)
there is more than 288 axial slices, because of this hardcoded param:
https://github.com/sct-pipeline/contrast-agnostic-softseg-spinalcord/blob/1976cec36162f8dd1ce200cee13a3bf5493cbd9c/config_templates/meanGT_soft.json#L126

Suggestions:

for updated dataset, find max(Z)

@ebadrian @louisfb01 pls see how this was done for sct_deepseg_sc and sct_deepseg_gm -- document everything you find (link to training script, preprocessing, etc)

@louisfb01 @jcohenadad Here are the script lines related to the cropping strategy for the different version of sct_deepseg:

Crop size in sct_deepseg_sc
Crop size in sct_deepseg_gm

Based on the current version of the model, here are the updates:

Cropping is still done but the crop patch size is updated to 64x192x320. Larger RL and AP crop sizes ensure that we deal with cases with high curvature. As for SC not being in the center --> this could still be a problem, but aggressive data augmentation tries to deal with this. We also had a decent result on a scoliotic spine segmentation (result shared on slack)
The 320 is the max size of all images in the training set in S-I direction so there is no cropping here

For updated dataset, find max(Z)-- or maybe not required because at inference, the CenterCrop parameter can be removed (to make sure the ENTIRE image is segmented)

Yes, for inference, we're currently leaving the Z to be -1 meaning that the model takes the original size in the S-I dim. (whatever it might be). Removing CenterCrop did not result as good results as the one with cropping. The errors were mainly spurious segmentations outsdie the SC (which can be removed in postprocessing)
Based on these (final) updates, closing this issue


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_b3c42b69-43fd-49a4-a472-8d8ab1a26463/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_b3c42b69-43fd-49a4-a472-8d8ab1a26463.git

Docs Build status updates of commit 2ca02f0:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_main_2022-02-22-01-34-40 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Don't discard video processor code lines containing unbalanced quotes when copying FX chains

Fixes #1857

:white_check_mark: Build sws 1090-master completed (commit https://github.com/reaper-oss/sws/commit/f8170134c1 by @cfillion)

Windows 64-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.dll
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-code-chunk-unbalanced-quotes-f8170134.exe
build/sws_python64.py

Windows 32-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.dll
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-code-chunk-unbalanced-quotes-f8170134.exe
build/sws_python32.py

macOS x86 64-bit

build/reaper_sws-x86_64.dylib
build/sws-<IP_ADDRESS>-Darwin-x86_64-code-chunk-unbalanced-quotes-f8170134.dmg

macOS x86 32-bit

build/sws-<IP_ADDRESS>-Darwin-i386-code-chunk-unbalanced-quotes-f8170134.dmg
build/reaper_sws-i386.dylib

macOS ARM 64-bit

build/sws-<IP_ADDRESS>-Darwin-arm64-code-chunk-unbalanced-quotes-f8170134.dmg
build/reaper_sws-arm64.dylib

Linux x86 64-bit

build/reaper_sws-x86_64.so
build/sws-<IP_ADDRESS>-Linux-x86_64-code-chunk-unbalanced-quotes-f8170134.tar.xz

Linux x86 32-bit

build/reaper_sws-i686.so
build/sws-<IP_ADDRESS>-Linux-i686-code-chunk-unbalanced-quotes-f8170134.tar.xz

Linux ARM 64-bit

build/sws-<IP_ADDRESS>-Linux-aarch64-code-chunk-unbalanced-quotes-f8170134.tar.xz
build/reaper_sws-aarch64.so

Linux ARM 32-bit

build/sws-<IP_ADDRESS>-Linux-armv7l-code-chunk-unbalanced-quotes-f8170134.tar.xz
build/reaper_sws-armv7l.so


Expose 'input lifetime where it is the backing lifetime.

Broken out of #92 as it should be uncontroversial and that PR will most likely not land due to its ergonomic costs.

Thanks.

Thank you!


Way to use the scripts as stand-alone library

Thanks for this great repo!
Would it be possible to build the script such that the compiled javascript file can be used as a stand alone library on the client side?
Best,
Ge

Client meaning frontend I presume?
While not currently in this package.json, we can create a browserified version of the javascript:
$ npm install -g browserify
$ mkdir dist
$ browserify lib/index.js --outfile dist/services.js --standalone services
$ ls dist
services.js

Because we're using commonjs here from node land though, you can re-package for webpack, requirejs, etc.
Where would you be using this?

I actually ended up writing my own last night! I am using my own notebook for my research at the moment, and I wanted to be able to run some python script from the text editing area.
In the future though, it would make more sense to have this repo being the official client-side javascript library for ipython, so that other projects can just use it. It is also nice because it will be kept up to-date. Granted the 5.0 protocol is already stable so it shouldn't need to be changes all the time.

hi guys trying out the services js as a standalone part but running into some troubles.
when already having loaded the services.js
and i have a jupyter notebook server on 8888 running acceptinig connections from everywhere.
var sessionOptionsTest = {
  notebookPath: "test.ipynb",
  kernelName: "python3",
  baseUrl: "/localtest/",
  wsUrl: "ws://localhost:8888"
}
var sessionTest = new services.NotebookSession(sessionOptionsTest);
sessionTest.start()

then a kernel is started but when trying to connect to the websocket the jupyter notebook gives a 404 error. when i look into the headers i find that:
ws://.../api/kernel/e2c60ccc-d809-4a16-990e-7968cedbf411/channels?session_id=6FB78E54322240AA8FE656629E1DCF66

in the api description the path should be kernels ; when i change the var KERNEL_SERVICE_URL = 'api/kernel'; in your code to kernels then i get the connection started but half a second later i get disconnected again and the jupyter notebook throws me an interesting error:
Uncaught exception in /localtest/api/kernels/034a67b2-8496-4a3b-a8d2-427eff8b5386/channels
    Traceback (most recent call last):
      File "/usr/local/lib/python3.4/dist-packages/tornado/websocket.py", line 415, in _run_callback
        callback(*args, **kwargs)
      File "/usr/local/lib/python3.4/dist-packages/notebook/services/kernels/handlers.py", line 250, in on_message
        self.session.send(stream, msg)
      File "/usr/local/lib/python3.4/dist-packages/jupyter_client/session.py", line 665, in send
        to_send = self.serialize(msg, ident)
      File "/usr/local/lib/python3.4/dist-packages/jupyter_client/session.py", line 579, in serialize
        self.pack(msg['parent_header']),
    KeyError: 'parent_header'

help would be appreciated :-)

Hi @hedata, this library is still a work in progress, but thank you for the feedback.  We are currently refactoring the Kernel and the Session in #20, but I will try your example before we merge it.

Hi @hedata, you should be able to run the code in the Readme to create a notebook session now.

I'm calling this one complete, since we have working integration tests, feel free to reopen @episodeyang.

@blink1073 Thanks for the work! I guess the decision to write my own bought me about a month and half:)


open cp1251.map: no such file or directory

When using mdtopdf.WithUnicodeTranslator("cp1251") as library I'm getting this error:
open cp1251.map: no such file or directory

The README has this example: https://github.com/mandolyte/mdtopdf?tab=readme-ov-file#using-non-ascii-glyphsfonts
@jessp01 Any thoughts?

correct, I found out that that example is outdated, even it doesn't include the theme param.
fpdf doesn't include cp1251.map and that's why it says file not found. but probably the biggest problem is that this library doesn't allow to set the Unicode translator directly.

Hi @dcu ,

correct, I found out that that example is outdated, even it doesn't include the theme param.
You do not have to specify a theme, if --theme isn't passed, the default is light; from the usage message:

  -theme string
    	[light|dark] (default "light")

fpdf doesn't include cp1251.map and that's why it says file not found.
cp1251.map is actually included in this repo, see https://github.com/mandolyte/mdtopdf/blob/master/cmd/md2pdf/cp1251.map

but probably the biggest problem is that this library doesn't allow to set the Unicode translator directly.
Can you elaborate as to what you mean by this?

The example @mandolyte pointed to actually works just fine for me. If you clone the repo (or download the full source archive) and:
$ cd cmd/md2pdf
$ go build
$ ./md2pdf -i russian.md -o russian.pdf --unicode-encoding cp1251 --font-file helvetica_1251.json --font-name Helvetica_1251

You'll get russian.pdf with:
Heading One
Here are lines of Russian:

Молоко и творог
Съешь же ещё этих мягких французских булок, да выпей чаю

Or at least, as they say, it works for me:) If the above fails for you, please let me know.

Closing for now but feel free to reopen with more info.


Leaflet.PM not working on Chrome for Android

Hi
I noticed Leaflet.PM does not work on Chrome (61) for Android (7) (at least the demo).
To reproduce:

Open https://leafletpm.now.sh/#/ in Chrome on an Android device
Click/select any editing tool (line, marker...)
Click on map: nothing is drawn

Actually I noticed similar issue with Leaflet.Editable, might be an issue with Leaflet event management.
Any idea on how to fix or workaround this problem?
Thanks

hmm 🤔   - it works fine for me.

Are there any JS errors in the console for you?

ah sorry, Android. Hmm I don't have an Android Device but I just tried it on a Nexus 5 emulator (Chrome Dev Tools) and it worked too.
Are you able to debug on your Device? E.g. DevTools? JS errors would be good to see

Hello
There is nothing displayed in the console, it stays blank.
As the code is obfuscated I do not know how to investigate further.

It works on my android,chrome,only dragging isn't handled.

Works fine on iPhone - including dragging 🤔 🤔 .
Can you guys reproduce a test case with the Chrome DevTools emulator and leafletpm.now.sh?

After extra tests, I managed to draw on Chrome Android, but I have to do VERY quick taps, usual taps being considered as map dragging instead of map clicks. Is there some timer to configure to be more permissive on the tap duration?

mhh not in leaflet.pm and nothing that I know of in leaflet.
These are the leaflet events on a map:
http://leafletjs.com/reference-1.2.0.html#map-click
They don't have dedicated mobile events afaik 🤔

Ok, I agree this issue is not clear enough, I let you close it if you want.


Sema: Fix request cycle involving protocol typealiases in preparation for RequirementMachine support

The RequirementMachine eagerly resolves protocol typealiases while building a rewrite system. This was causing a request cycle because structural type resolution would request the generic signature of a type unnecessarily, and that generic signature could then reference the protocol itself.
Here is an example (my next PR actually adds an explicit test case, but there are a bunch of examples in the test suite already that boil down to this and fail with -requirement-machine-protocol-signatures=on):
protocol P where A == B {
  associatedtype A
  typealias B = Foo<Self>
}

struct Foo<T : P> {}

While building the requirement signature of P we resolve the structural type of B, which would previously trigger the computation of the generic signature of Foo, which would ask for the requirement signature of P. However structural type resolution doesn't check generic requirements on the bound generic type, so it need the generic signature except for in a couple of specific cases (which will still cause a cycle with an example like the above, but that's okay).

@swift-ci Please smoke test

@swift-ci Please test source compatibility

@swift-ci Please smoke test

@swift-ci Please smoke test macOS


GH doesn't respect construction tag?

http://wiki.openstreetmap.org/wiki/Key:construction
I don't find GH parse construction.

As we don't want to route on highway=construction there shouldn't be a problem or do you mean #374?

there are way with construction=yes.

Do you have an example or even the count? As the wiki says: "construction=* must be used together with tags such as highway=construction, railway=construction, landuse=construction, and building=construction"

yeah, not too much.
http://www.openstreetmap.org/way/83051200

In my mind, highway= is the road classification.
construction=yes/no is the status.
Shouldn't combine them.
:)

The wiki is often confusing but in this case very clear, I would see this as a tagging mistake. Unless there are several hundreds/thausand occurrences. Or maybe we wait a bit what others say.

there are way with construction=yes.

It is a terrible tagging scheme that should be retagged to highway=construction, construction=highway_type
See https://josm.openstreetmap.de/ticket/11232 and https://josm.openstreetmap.de/ticket/11146 for custom validator rules for JOSM allowing detection and/or fixing this problem.
highway=construction is about 20 times more popular, supporting construction=yes is encouraging bad tagging schemes.

Thanks for the input! So we should indeed just support conditional #374?

Yes, supporting highway=construction and #374 should be enough enough. Supporting construction=yes gives really minor gain at cost of increased complexity of program and harming quality of OSM data in long run.

Okay closing here in favour of #374


login to mainnet fails or times out but to testnet succeeds

Using Firefox 100.0.2 64 bit on ubuntu-based Pop!_OS
With my wallet after a long time it logs out, then I have noticed I cannot log back in on mainnet with my saved password, and for extended periods of time.  The login spinner just turns for a long time (> 60 seconds to minutes).
Then after failing/quitting, I tried logging into testnet and it suceeded very fast - say 10 seconds.
Then I was able to switch to mainnet.  This problem is very annoying as it happens quite often.  How can I debug this?  Is there a way to turn on logging or something to see what is going on?... to see what is the delayed or blocked process here?
edit: I just locked the wallet and tried to log back in. I waited in excess of 4 minutes and it failed to login.
Then I switched to testnet and it logged in in 6 seconds, and then I was able to switch to mainnet.

Is there logging happening anywhere I can see what's going on during the login process?
How to debug this?
Should there be a way to enable debug mode - say a UI button or env var that can be set to enable seeing logging?

If you encounter this problem again, can you try restarting Firefox completely and seeing if this helps.
no restarting doesn't help. New browser start with no other open tabs.

I wonder if the mainnet/testnet switching has any impact at all, and the change in behaviour was simply coincidence. Afaict there's no sign in logic that changes based on network. We've had many reports of slow sign ins with Firefox. Out of curiosity, what are the specs of your machine?

it's repeatable. Not a coincidence.
My notebook is a Core i5 quad-core 8 threads with 20GB RAM (4gb used)

There is no logging available for this unless you create a build and add it yourself.

well testing it a lot it's very random. Sometimes restarting the browser works sometimes not.
I wonder if there is some server-side resource with variable load that could be causing this?

I wonder if there is some server-side resource with variable load that could be causing this?

You wonder correctly. The @stacks/wallet-sdk library interacts with Gaia, so it's possible that causes the issue (though, anecdotal reports also suggest it's predominantly a Firefox problem)

We've recently made improvements that have hopefully fixed this issue. Please let us know if you encounter it again.


[UI] Filters: Action Enhancements

Current Behavior
No current ability to deploy, bulk/individual delete filters from the Filter UI
Desired Behavior
Users need the ability to deploy, bulk/individual delete filters from the Filter UI from each row of the table
Implementation

Add the "Deploy" button (use the Play icon or Lighting Bolt icon) to invoke the Pattern "deploy" endpoint.
Change "..." elipse icon to Pencil icon
Update the name of the column from "Details" to "Actions"
Bulk/individual delete filters options from the UI (checkboxes)
Reference: https://docs.google.com/document/d/1gFVMlJtOjCaBbwnpB7KyfnQGoRGWtM-oZrpVd9oYSAQ/edit#heading=h.blih70a9hxp

Contributor Guide

@leecalcote  Could you please add the relevant labels and assign me. 😊

My bad @Chadha93 Assigned Myself by mistake. I Have assigned it to you now 🎉

@Chadha93 if we don't have the same functionality for Patterns, will you add these to Patterns as well?

@Chadha93 if we don't have the same functionality for Patterns, will you add these to Patterns as well?

@leecalcote We do have this functionality in patterns, I have already added(merged) and demoed on meshery  call : )


HBASE-23379: Clean Up FSUtil getRegionLocalityMappingFromFS

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 34s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 27s
master passed

+1 :green_heart:
compile
0m 54s
master passed

+1 :green_heart:
checkstyle
1m 21s
master passed

+1 :green_heart:
shadedjars
4m 35s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
master passed

+0 :ok:
spotbugs
4m 11s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
4m 10s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m  0s
the patch passed

+1 :green_heart:
compile
0m 54s
the patch passed

+1 :green_heart:
javac
0m 54s
the patch passed

-1 :x:
checkstyle
1m 20s
hbase-server: The patch generated 1 new + 74 unchanged - 0 fixed = 75 total (was 74)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 35s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
15m 42s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 36s
the patch passed

+1 :green_heart:
findbugs
4m 15s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
145m 51s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 35s
The patch does not generate ASF License warnings.

202m 37s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-911/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/911

JIRA Issue
HBASE-23379

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 5138d2b847f6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-911/out/precommit/personality/provided.sh

git revision
master / 9c82a65b29

Default Java
1.8.0_181

checkstyle
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-911/1/artifact/out/diff-checkstyle-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-911/1/testReport/

Max. process+thread count
4148 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-911/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@HorizonNet Thanks for the review!!

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 33s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 26s
master passed

+1 :green_heart:
compile
0m 55s
master passed

+1 :green_heart:
checkstyle
1m 18s
master passed

+1 :green_heart:
shadedjars
4m 32s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 39s
master passed

+0 :ok:
spotbugs
4m  1s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 58s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 58s
the patch passed

+1 :green_heart:
compile
0m 55s
the patch passed

+1 :green_heart:
javac
0m 55s
the patch passed

+1 :green_heart:
checkstyle
1m 21s
the patch passed

+1 :green_heart:
whitespace
0m  1s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 39s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
16m 15s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 36s
the patch passed

+1 :green_heart:
findbugs
4m 20s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
154m  7s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 36s
The patch does not generate ASF License warnings.

212m 27s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-911/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/911

JIRA Issue
HBASE-23379

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux c7642b41e5cf 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-911/out/precommit/personality/provided.sh

git revision
master / 82e155eb26

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-911/2/testReport/

Max. process+thread count
4061 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-911/2/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Any other thoughts @HorizonNet ? Thanks!


Add ProviderManager, make provider deletion generic

Relates to #2845
Change the ProviderFactory to ProviderManager. Add new method for deleting provider instances. This hides provider-specific cleanup behind an appropriate interface, and removes some duplication of logic (with slightly different implementations) in different parts of the codebase.
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 49.358% (-0.4%) from 49.759%
when pulling 9036544ae438b2d757947e558b5d95d4fe65336b on provider-manager
into e841cbb0f6b902bf845246973eafde55973fa30f on main.

This is nice! I haven't tested manually, but the code is looking good. Is there anything in particular you're looking for to take this out of draft?

@eleftherias I need to add some test coverage, and also do some manual verification of the changes.


SyntaxError: Unexpected token 'export'

Describe the bug
I get the following error when trying to import the package like so import { downloadZip } from "client-zip";:
SyntaxError: Unexpected token 'export'
    at compileFunction (<anonymous>)
    at Object.compileFunction (node:vm:352:18)
    at wrapSafe (node:internal/modules/cjs/loader:1031:15)
    at Module._compile (node:internal/modules/cjs/loader:1065:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:94:18)
    at Object.client-zip (/Users/ericamshukov/Projects/dt_bridge_pipetrekker_spa/.next/server/pages/index.js:20570:18) {
  page: '/'
}

To Reproduce
Steps to reproduce the behavior:

Import in TypeScript file
npm run
SyntaxError: Unexpected token 'export'

Additional context
I am developing a project in NextJS and TypeScript, but everything else is working properly.

Hi. client-zip is meant to be used in a browser (there are faster alternatives in Node.js) and is therefore released as an ES6 module only. Node.js supports the ES6 module syntax, but you have to ask nicely and you'll also be missing a few Web API globals that client-zip rely on.

My point is, I suspect your project is configured to only understand CommonJS imports and probably also to output ES5-compatible code (a shame, really, but those big projects tend to value backwards compatibility way too much…). One way around it would be to exclude the client-zip import from bundling — import it only at runtime in the browser.

Any luck fixing your module problem, @FullyFerret ? it's a bit off-topic, but it could still be useful to other users with a similar setup if you explained how you fixed it before closing the issue.

@Touffy I was helping a friend out this weekend with this.
Unfortunately with nextjs the solution isn't simple.
There are two things that can be done in nextjs to use this:
You can use a nextjs dynamic component which is only rendered client side:
const FileAttachmentLink = dynamic<Attachment>(
  () => import('./file-attachment-link').then((m) => m.FileAttachmentLink),
  {
    ssr: false
  }
)

Then, in that component, you can use a dynamic import to avoid it being included in the browser. For example in a callback, you could do the following:
export function DownloadFileLlink() {
  async function onDownload() {
    // import the library dynamically at runtime
    const { downloadZip } = await import('client-zip')

    const code = await fetch(
      'https://raw.githubusercontent.com/Touffy/client-zip/master/src/index.ts'
    )
    const intro = {
      name: 'intro.txt',
      lastModified: new Date(),
      input: 'Hello. This is the client-zip library.'
    }

    // get the ZIP stream in a Blob
    const blob = await downloadZip([intro, code]).blob()

    // make and click a temporary link to download the Blob
    const link = document.createElement('a')
    link.href = URL.createObjectURL(blob)
    link.download = 'test.zip'
    link.click()
    link.remove()
  }

  return (
    <a onClick={onDownload}>
      {filename}
    </a>
  )
}

Right. I was afraid you'd have to do something like this. Obviously we don't want client-zip on the server-side. It won't work, and it's not like Next would understand "server-side rendering" the Zip file anyway even if it could parse the code.
This method also has a little performance drawback because you import client-zip just when you need to create the Zip file (the first time, anyway), so it will add a bit of latency. That can be fixed easily by moving the import() call outside the onDownload function so it's started as soon as you render the component, storing just the Promise, and awaiting that Promise in onDownload.

Thanks @venkatd for the code example.
@Touffy thanks for the good work on this package. I came hunting for this question. It may seem silly given the package name is literally "client-zip" but it wasn't immediately evident to me that this only works in a client environment. I was also trying to integrate with nextJS. I imagine other folks will try to do similar things given the popularity of frameworks like nextjs right now, maybe it is worth elevating the discussion of using this package with SSR to the docs somewhere?

Forgot to do it before the update… oh well, at least it's on GitHub now, will be pushed to NPM along with the next version… probably when I complete Content-Length prediction.

(there are faster alternatives in Node.js)
@Touffy could you list some alternatives?

The faster alternative in Node is to call a (fast) native Zip program. I am not an expert on those, and I'm sure there are already performance reviews you can find elsewhere. I don't mean to be rude, I just don't want to give you poor advice from a lack of research, and I don't see why I should do the research. Zipping on the server-side is off-topic for this project. It happens to work in Deno, yes, but that wasn't even intended.
However, there is a general explanation I can give you. Native programs can use CPU features that are not available to client-zip (because they're not in JavaScript or WebAssembly), such as a hardware CRC32 primitive, and 256-bit vector operations. Also, Node and native programs have efficient random access to the local filesystem, which means they don't have to use streaming mode if zipping from there, and they can copy data from one part of the filesystem to another with efficient kernel calls instead of moving it through userspace buffers. As you can see on my benchmark, the native Zip program I used is much faster than client-zip and other JavaScript alternatives. However, I can't say if it's the fastest, or the best suited for integration with Node.

@Touffy I hit the same issue while running tests with jest. Do you have any workaround in mind for using jest with React components that use client-zip?

The problem is not React or jest, it's their configuration. You can either make them understand ESM imports, or skip the whole thing by making the import dynamic at runtime. The details will depend on your existing setup.

I just realized that for tests I can probably mock client-zip because I don't really need to test the library iteslf

That's right. You probably don't need to run downloadZip at all as part of your test (so you can use an empty mock) but even if you have to, you can easily mock downloadZip with a one-liner that returns new Response(new Blob()).


Prepare for release v30.0.0

Once #1451 is merged, we are ready for a major release with many breaking changes.
This PR can be merged after #1451, and then a new v30.0.0 release can be created and tagged.

LGTM

@wesleimp it would be great to have this merged in order to unblock v30

Thank you, @wesleimp and @joebowbeer!
Merging and then releasing v30.0.0.


Added limits for integer types

For https://github.com/zyantific/zydis/pull/254

Looks good! Merged, thanks!


Trusted domain goes offline when trying to authenticate, kerberos policy error

Version: <IP_ADDRESS>4
OS/Distro: Mac OS X 10.11.6
Issue/Impact: Trusted domain goes offline when trying to authenticate, kerberos error
Output/Error:
lsass debug output:
20170906123112:DEBUG:lsass:LsaLdapOpenDirectoryWithReaffinity():lsass/server/auth-providers/ad-open-provider/join/lsaldap.c:152: Using DC 'auth-dc-w2.auth.domain' for domain 'auth.domain' (affinitization attempt 0)
20170906123112:DEBUG:LwLdapOpenDirectoryServerSingleAttempt():lwadvapi/threaded/lwldap.c:343: SASL_MAXBUFSIZE 16777215
20170906123112:ERROR:LwKrb5CheckInitiatorCreds():lwadvapi/threaded/lwkrb5.c:1657: GSS API error calling gss_init_sec_context(): majorStatus = 0x000d0000 (Unspecified GSS failure.  Minor code may provide more information), minorStatus = 0x96c73a0c (KDC policy rejects request)
20170906123112:DEBUG:LwKrb5CheckInitiatorCreds():lwadvapi/threaded/lwkrb5.c:1685: [LwKrb5CheckInitiatorCreds() /builder/src-git/Platform/src/linux/lwadvapi/threaded/lwkrb5.c:1685] GSS API error: majorStatus = 0x000d0000, minorStatus = 0x96c73a0c
20170906123112:DEBUG:LwLdapBindDirectory():lwadvapi/threaded/lwldap.c:526: [LwLdapBindDirectory() /builder/src-git/Platform/src/linux/lwadvapi/threaded/lwldap.c:526] Error code: 40048 (symbol: <null>)
20170906123112:DEBUG:LwLdapOpenDirectoryServerSingleAttempt():lwadvapi/threaded/lwldap.c:377: [LwLdapOpenDirectoryServerSingleAttempt() /builder/src-git/Platform/src/linux/lwadvapi/threaded/lwldap.c:377] Error code: 40048 (symbol: <null>)
20170906123112:DEBUG:LwLdapOpenDirectoryServer():lwadvapi/threaded/lwldap.c:453: [LwLdapOpenDirectoryServer() /builder/src-git/Platform/src/linux/lwadvapi/threaded/lwldap.c:453] Error code: 40048 (symbol: <null>)
20170906123112:DEBUG:lsass:LsaLdapOpenDirectoryWithReaffinity():lsass/server/auth-providers/ad-open-provider/join/lsaldap.c:168: Ldap open failed for domain 'auth.domain' (dwError = 40048 (symbol: LW_ERROR_GSS_CALL_FAILED))

^^^ Above repeats for the 4 DCs it discovers for the AUTH domain.
20170906123112:DEBUG:netlogon:LWNetGetDCNameExt():netlogon/client/dcinfo.c:133: Error: 9500
20170906123112:DEBUG:lsass:LsaLdapOpenDirectoryWithReaffinity():lsass/server/auth-providers/ad-open-provider/join/lsaldap.c:147: Error code: 9500 (symbol: ERROR_DOMAIN_BLACKLISTED)
20170906123112:DEBUG:lsass:LsaDmpLdapReconnectCallback():lsass/server/auth-providers/ad-open-provider/lsadm.c:3143: Error code: 9500 (symbol: ERROR_DOMAIN_BLACKLISTED)
20170906123112:DEBUG:lsass:LsaDmConnectDomain():lsass/server/auth-providers/ad-open-provider/lsadm.c:3790: Error code: 9500 (symbol: ERROR_DOMAIN_BLACKLISTED)
20170906123112:ALWAYS:lsass:LsaDmpModifyDomainFlagsByName():lsass/server/auth-providers/ad-open-provider/lsadm.c:2374: Domain 'auth.domain' is now offline

pbis-status output:

Compiled daemon version: <IP_ADDRESS>4
Packaged product version: 8.5.334.8
Uptime:        0 days 0 hours 0 minutes 51 seconds

[Authentication provider: lsa-activedirectory-provider]

	Status:        Online
	Mode:          Un-provisioned
	Domain:        COMPUTERS.DOMAIN
	Domain SID:    S-1-5-21-256988196-913931312-1652426489
	Forest:        computers.domain
	Site:          Open-Access-Labs
	Online check interval:  300 seconds
	[Trusted Domains: 4]

	[Domain: COMPUTERS]

		DNS Domain:       computers.domain
		Netbios name:     COMPUTERS
		Forest name:      computers.domain
		Trustee DNS name:
		Client site name: ComputersSite
		Domain SID:       S-1-5-21-256988196-913931312-1652426489
		Domain GUID:      43B9A09B-6D8F-0B46-9EB6-6B8843F03399
		Trust Flags:      [0x001d]
		                  [0x0001 - In forest]
		                  [0x0004 - Tree root]
		                  [0x0008 - Primary]
		                  [0x0010 - Native]
		Trust type:       Up Level
		Trust Attributes: [0x0000]
		Trust Direction:  Primary Domain
		Trust Mode:       In my forest Trust (MFT)
		Domain flags:     [0x0001]
		                  [0x0001 - Primary]

		[Domain Controller (DC) Information]

			DC Name:              dc1.computers.domain
			DC Address:           x.x.x.x
			DC Site:              ComputersSite
			DC Flags:             [0x0000f1fc]
			DC Is PDC:            no
			DC is time server:    yes
			DC has writeable DS:  yes
			DC is Global Catalog: yes
			DC is running KDC:    yes

	[Domain: AUTH]

		DNS Domain:       auth.domain
		Netbios name:     AUTH
		Forest name:      auth.domain
		Trustee DNS name: COMPUTERS.DOMAIN
		Client site name: Site
		Domain SID:       S-1-5-21-1167378736-2199707310-2242153877
		Domain GUID:      00000000-0000-0000-0000-000000000000
		Trust Flags:      [0x0022]
		                  [0x0002 - Outbound]
		                  [0x0020 - Inbound]
		Trust type:       Up Level
		Trust Attributes: [0x0008]
		                  [0x0008 - Forest transitive]
		Trust Direction:  Twoway Trust
		Trust Mode:       In other forest Trust (OFT)
		Domain flags:     [0x0000]

		[Domain Controller (DC) Information]

			DC Name:              dc.auth.domain
			DC Address:           x.x.x.x
			DC Site:              Site
			DC Flags:             [0x0001f1fc]
			DC Is PDC:            no
			DC is time server:    yes
			DC has writeable DS:  yes
			DC is Global Catalog: yes
			DC is running KDC:    yes

Packet capture synopsis:
client --> AUTH DC  KRB5   TGS-REQ
AUTH DC --> client  KRB5  KRB Error: KRB5KDC_ERR_POLICY NT Status: Unknown error code 0xc0000413

domainjoin-cli:
tried both
domainjoin-cli join --assumeDefaultDomain yes --userDomainPrefix AUTH computers.domain <user> <pass>
domainjoin-cli join --assumeDefaultDomain yes --userDomainPrefix auth.domain computers.domain <user> <pass>

A google search for 0xc0000413 turns up this technet: https://support.microsoft.com/en-us/help/2959395/a-tgs-request-for-the-krbtgt-account-fails-with-kdc-err-policy-and-an
Unfortunately I do not think I am going to be able to get the AD team to make any modifications, as their AUTH realm provides services to 10's of 1000's of production machines.
Any insite into this from devs or hardcore directory admins?

So your joined to computers.domain and want to authenticate to auth.domain?
Did you configure assumeDefaultDomain with userDomainPrefix?

no response


Support Datetime function LOCAL TIME

Backgrounds

Kotlin JDSL does not yet support a datetime function LOCAL TIME.

Objects

Support a datetime function LOCAL TIME.

Requirements

Creates localTime() function in com.linecorp.kotlinjdsl.dsl.jpql.Jpql for supporting a datetime function LOCAL TIME.

Contribution

Please check if there is an assignee in this issue. And if it don't have an assignee, please leave comment to assign.
Please check the commit convention through the commitlint.config.js file. The commit  convention also described in the PR template.
When you use intellij, please set the code style through the .editorconfig file.
Please create the PR for develop as the base branch.
If you have any questions, please leave comment in Kotlin-JDSL Discord Channel in LINE Open Source

Hello, could I be assigned to this issue? I would like to contribute.

Hi TheAndreiUrsu!
Yes, I changed the assignee to you! Thank you for your help!


[not_landed] Should not needinfo about backed out patches

Example: https://bugzilla.mozilla.org/show_bug.cgi?id=1790765#c7
Reported by @smaug----

The "fix" is to mark patches as "Changes Planned". Perhaps we could do that automatically when a patch is backed out.


EscapeAnalysis: Convert a if to an assert.

because it should always be true
It's a NFC.

@swift-ci smoke test and merge


Add shareProcessNamespace option to podtemplate

This change relates to https://github.com/traefik/traefik/issues/3489. Currently if you're using Kubernetes with a sidecar to ship logs there is no way to rotate logs and signal Traefik to close/re-open its log files.
The addition of the shareProcessNamespace pod spec flag with a logrotate sidecar including the SYS_PTRACE securityContext.capabilities  allows one to send the USR1 signal to the traefik process and trigger it to close/re-open its logs
There are a number of ways the same result could be achieved, so I would be open to alternatives if one is suggested.

Hey @emourant,
thanks for that contribution. First time I encountered that flag, but I have to agree I think it makes sense :)
Can we get a quick unit test for that change? Also, its a new minor version rather then a patch version.