Ch.2 webserver cluster instance duplicate error

Hello @brikis98
While wrapping up Ch.2, I have encountered the following error
`Error applying plan:
1 error(s) occurred:

aws_security_group.instance: Error creating Security Group: InvalidGroup.Duplicate: The security group 'terraform-example-instance' already exists for VPC '<vpc_id>
status code: 400, request id: a7232801-b51d-4263-b69a-2ed4f17d08ea

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.
`
I am not sure if this related to the version of terraform(see below) or a typo within my main.tf(which I have compared to your examples here)
terraform -v
Terraform v0.7.10
Your version of Terraform is out of date! The latest version
is 0.10.1. You can update by downloading from www.terraform.io

I updated terraform to 0.9.x per the requirements highlighted within the book, the 0.7.x version was installed previously due to a linux academy
terraform -v
Terraform v0.9.11
Sorry about that

I should be all set. Towards the end of the chapter, i performed the terraform destroy and re-applied and I seem to be ok,


Upgrade to Quarkus 2.2.1.Final

The HTTP test is passing for me locally in native mode.
The CI failure may have to do with random port generator choosing the same port for https-test-port and http.test-port:
-Dcamel.netty-http.https-test-port=35671, -Dcamel.netty-http.test-port=35671


How do I perform semantic backdoor attack?

I saw that there are different attacks via the syntehsizer.py, but I cannot find any code related semantic backdoor attack. Is it implemented in the code?

Also, I am unable to run the attack against ImageNet because of this problem.

you need to download Imagenet yourself. For the semantic attack on images, I don't have it here yet, but for NLP case as in the blind paper you can refer to this modified transformers repo: https://github.com/ebagdasa/backdoored_transformers

from the code, I see that only pattern synthesizer works? What does a pattern synthesizer do? Is it a pixel pattern backdoor as mentioned in your paper, which introduces a series of pixels as a backdoor into the image?

yeah

Thank you!


update node js to 16

GitHub is deprecating 12 for some reason.  Hopefully this just works.

@ppremk Can you take a look ? Thanks.


Small modifications to some of the exercises

Thank you so much.  I will take a look at these in about 10 days!

If you are interested in this and #873 in the future, let me know and I will re-open.


Modify error hander name

Function "page_not_found" is already defined and overwrite this.
Modify error handler name "page_not_found" to "internal_server_error".

Same as #14.


Mid V1.4 Review

Hi all, I have reviewed your Developer Guide in your PR. Kindly read through the comments and make changes if needed. Please be aware that for next Friday's (Week 13) Practical Exam, you need to be physically present unless you have an official excuse. Thanks.

Noted. Thank you!


Stop testing against JDK 20

It EOL'd on Sept 19th, 2023.
See https://endoflife.date/oracle-jdk
Also, we already test against the next version (21).

Right, good catch. Done.


Rest v2 updates

Pull Request Test Coverage Report for Build 993

0 of 0   changed or added relevant lines in 0 files are covered.
98 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.8%) to 73.23%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/blockchain.js
98
55.09%

Totals

Change from base Build 986:
-0.8%

Covered Lines:
2515

Relevant Lines:
3266

💛  - Coveralls


Update non-local-defns regarding parameterized traits and types

The existing RFC does not give sufficient attention to parametrized types and traits.

Rendered
Original RFC
Tracking issue

Agree that this is under specified. I am not sure that spelling this out explicitly in the RFC text works --- its not clear whether the specified rules have any loopholes.
My current belief is that the rule we want here is exactly equivalent to coherence. So the RFC should say something along to the effect of "function bodies are treated as downstream crates for the purposes of trait checking and type inference".
As this (most excellent) example by QuineDot demonstrates
https://github.com/rust-lang/rfcs/pull/3373#issuecomment-1416529034
it is important to restrict both:

which impls are allowed inside
which inside impls are visible from the outside, for the purpose of type inference reasoning that "I know of a single impl for this trait, so _ must be that type", where "that type" might end up being a type from a function's body.

Ah, I missed this extra bit of context: https://github.com/rust-lang/rust/issues/121621, where it is suggested that we want more restritive rules than coherence here specifically to avoid fixind point two from above.

@tmandry Is this something you can review (or direct to someone who can)?
I can't tell if this is changing the RFC in a meaningful way, or just clarifying it. If it is a change from before, it would probably be best to start a new RFC.

From a quick look this is in line with the original RFC, and if @joshtriplett is willing to consider it a "friendly amendment" I think it can be merged.

Made a few editorial changes, and made one comment about moving the example out of the motivation section. With that change made, LGTM; no objections.

Isn't this going against T-types solution ? (which is being implemented in https://github.com/rust-lang/rust/pull/122747)
cc @lcnr

@Urgau Thanks for catching that. @lcnr, can you help resolve the discrepancy here?

I'd personally just keep the RFC as "we lint if the existance of the impl is observable outside of its containing item", I don't think there is much value in explaining the exact algorithm used for that in an RFC. I also thought we generally expect the reference/guidelevel explanation of RFCs to get out of date and do not bother updating them, instead referring to the reference/docs.
The idea being #122747 is as follows:
Lint impls inside of bodies unless:

the body is of an unnamed const
the self type is also local to the body
after replacing all generic params and types local to the body with inference variables, there are at least 2 global impls which could also apply for that goal.

This should mean that in all cases where the local impl may apply outside of the body, we already fail with ambiguity even when ignoring the impl

@lcnr I'm deferring entirely to the types team here for whether you want to make changes/suggestions to this or close it.
It'd be nice if the rules we have are documented somewhere. Updating the RFC is one possible option; others could work as well.

whether you want to make changes/suggestions to this or close it

My only feelings is that it is often hard to tell the status and intended behaviour of in-development features. Whether or not updating RFCs is appropriate I don't know... and now we're getting off-topic.


Add the possibility to trigger 'isFinal' or make the silence threshold configurable

I would like to trigger manually the 'isFinal' response from google.
I need to recognize a list of words like this, said pretty quickly :
banana
cake
apple
...

Unfortunately, the isFinal event is only triggered after around 2-3 seconds of silence, hence google interpret the audio input as if it's the same sentence and can change the recognized words to try to make a sentence which make sense.
It's working fine if I just end and re-start the recognition stream, but since google pricing rounded all the query to 15 seconds, i better use the same query for all those words.
Would it be possible to make a silence threshold configurable when starting the stream ? Or manually trigger the isFinal event ?
Thank you

@MadyAkira Would you mind sharing your code snippet used for this?
Second question: Have you tried changing single_utterance flag to see if you can get quicker response?

Due to inactivity, Im closing this issue.
Feel free to reopen


Flask 2.0.0 Error Handler not called when Blueprint Name Contains a Period

When you have a blueprint, and it's name contains a . and it has error handlers registered, they are never called.
We have been using blueprints to help organize our application, and we have used . to help namespace the blueprint names. And we have registered error handlers on the blueprints. However, when an error occurs, they are never called.
Below is a simple example where you might have an app that has a UI and an API, and for UI errors you may have a custom error view, and for API errors you may want to return JSON.
from flask import Blueprint, Flask, jsonify

app = Flask(__name__)
ui = Blueprint("app.ui", __name__, url_prefix="/ui")
api = Blueprint("app.api", __name__, url_prefix="/api")

@ui.errorhandler(Exception)
def ui_error_handler(error):
    print("ui error handler called")
    return str(error)

@ui.get("/")
def ui_index():
    print("ui index called")
    raise Exception("this is an unhandled exception from the ui")

@api.errorhandler(Exception)
def api_error_handler(error):
    print("api error handler called")
    return jsonify(error=str(error))

@api.get("/")
def api_index():
    print("api index called")
    raise Exception("this is an unhandled exception from the api")

app.register_blueprint(ui)
app.register_blueprint(api)

Then when you run and do a get on one of the index endpoints you get:
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: off
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)
api index called
[2021-05-13 13:14:48,849] ERROR in app: Exception on /api/ [GET]
Traceback (most recent call last):
  File "/lib/python3.9/site-packages/flask/app.py", line 2051, in wsgi_app
    response = self.full_dispatch_request()
  File "/lib/python3.9/site-packages/flask/app.py", line 1501, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/lib/python3.9/site-packages/flask/app.py", line 1499, in full_dispatch_request
    rv = self.dispatch_request()
  File "/lib/python3.9/site-packages/flask/app.py", line 1485, in dispatch_request
    return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)
  File "app.py", line 29, in api_index
    raise Exception("this is an unhandled exception from the api")
Exception: this is an unhandled exception from the api
<IP_ADDRESS> - - [13/May/2021 13:14:48] "GET /api/ HTTP/1.1" 500 -

You can see the api index called printed, but not the api error handler called printed which shows the error handler is never called.
Environment:

Python version: 3.9
Flask version: 2.0.0

Blueprint names should not contain dots. I thought this was already raising an error.

We won't be changing this at this point, as dots are reserved for separating blueprint names (which can be nested now) and endpoint names.

Blueprint names should not contain dots. I thought this was already raising an error.

According to the changelog, this has been the case since 1.0.

@davidism if you are referring to #2450 that is only restricting the view function name to not have a dot in it. AS far as I know, there has not been any restriction on the actual blueprint name to not have a dot in it.

Seems to have been an oversight than. Regardless, you'll need to update your code, the most we'll do is make this an explicit error.

We have some monitoring/alerting setup that uses the view_func name, and the view_func name is prefixed by the blueprint name. Changing the blueprint names will mean we will need to change all the alerts that we have configured. Doable, but I was hoping that this function here could just be modified slightly to also check:
handler_map = self.error_handler_spec[_request_ctx_stack.top.request.blueprint]

Or something like that.

We could do that, technically, but I don't plan to. Every . separates a blueprint name now, due to implementing nested blueprints. Sorry for the inconvenience, but you'll need to do a big search and replace on your code to avoid using dots in names.

Closing for #4041


Fix Issue 6748 - New cloned customized models are not saving their System Prompt the first time

Fix for Issue 6748

Appreciate the PR, model editor is being overhauled entirely atm.


VSCODE flickers in full screen mode

Version 1.19.2
Fixation 490ef761b76b3f3b3832eff7a588aac891e5fe80
Date 2018-01-10T15: 55: 03.538Z
Shell 1.7.9
Drawer 58.0.3029.110
Host 7.9.0
Architecture x64
Steps to Reproduce:

press f11
coding....

there are unpleasant flickering, orange in full screen mode

I'm experiencing the same flickers on Ubuntu 17.10. Very annoying

same here in ubuntu!! I've been experiencing the issue for like 2-3 months. It's not a new issue

Same on Ubuntu 17.10 with external monitor (in full screen mode only)

I'm running Debian Sid, kernel 4.16.0-1-amd64, with Intel HD Graphics 630 & NVIDIA GeForce GTX 1050 Mobile, and was able to remove all flickering in vsascode 1.23.1 with Bumblebee by not using --disable-gpu, and running optirun code.

Same on Ubuntu 18.04 LTS with GTX1080Ti with removed stock drivers and installed latest nvidia drivers. --disable-gpu fixed problem.

Same on Ubuntu 18.04 with a GeForce GT 610.
However, for other GNOME users here's a workaround that seems to behave fine for me:

Open Settings > Keyboard
Search for "Toggle fullscreen mode" and assign a shortcut to it (I've chosen Super+F11)
Instead of using F11 or the application's own fullscreen mode, use GNOME's via your shortcut

This works without flickering for me, though I imagine it might not bring with it any GPU fullscreen optimisations (on that front it is probably better, performance-wise, than disabling GPU acceleration all together!).

Hi, I'm experiencing the same problem on Manjaro with Gnome. Using Super+F11 to enter gnome's "fullscreen" mode does not help, while "--disable-gpu" fixed it for me.

Same problem with vscode 1.26.1, ubuntu Ubuntu 18.04.1
--disable-gpu solves the problem.

Question: Is there anybody who does not have this issue? :D

Yes,  my windows machines don't have this issue.

I dont have this issue on windows too.

The issue is just on Linux-based systems.

@radutomy not entirely true, soveckyFonarik the issue creator has windows 10. The issue is more likely to be some error caused by electron or the driver/hardware constellation.
Apparently most or maybe all nvidia-driver-* packages on linux are affected which explains the high density of this issue on Linux based systems. The strange part is it works on my gnome ubuntu 16.04 without any issues but my Debian installation still flickers on the same machine (Dualboot).

Ubuntu 18.04 same issue, maybe it's related to Intel HD Graphics 520 (Skylake GT2) video card

Win10 Latest + VSCode 1.27.2 still has this issue.

Same issue.
Ubuntu 18.04
GPU: Intel Corporation HD Graphics 5500

Same Issue, Windows 10. Happens when switching tabs or switching apps.
Vscode 1.27.2
AtI R9 280

Just to add on. I've also experienced this on Git Kraken on fullscreen. and I come to understand that both VsCode and Git Kraken are built on electron. I have a feeling this is an issue with electron itself?

I have the same problem - with Ubuntu 18.04.1. I see the flickering

with and without --disable-gpu flag
on fullscreen and windowed mode

vscode was working perfectly. However, after update 1.28.1 I get a persistent flicker in the vscode window making it unusable.
OS: Centos 7.5
Graphics: AMD RX 470

Same issue as @tkuraku described
VS Code Version: 1.28.2
Ubuntu: 16.04
Grpahics: Integrated

I have a similar issue I think. I'm currently running Feodora 29, but also noticed this issue on ubuntu 18.04 on the same hardware setup described below.
For all my notes below I had started VS Code with the disable gpu flag.
Specifically I'm getting flicker when in full screen mode primarily when a tool tip pops in the explorer bar and my mouse runs through it (the cursors passes through the tool tip). This also seems to happen for tool tips in the tab bar as well, however not an issue for completion pop ups etc.
Oddly enough if I click to on a second monitor to put focus on what ever is on that monitor and then move my mouse back to VS Code (not clicking to bring focus back) and hover over items in the explorer that pop tool tips the flicker goes away. However, simply click to bring focus back to VS Code the flicker returns.
A few other things that I have noted:

Only happens for me in full screen mode
Seems to only be triggered by mousing over tool tip or clicking into the VS Code from another monitor to gain focus on the window.
Sometimes it clears up (e.g. I didn't restart the app) if there are fullscreen apps behind it (I had terminal app on the same workspace), but this doesn't always happen - bear in mind I'm also moving the window between workspaces when this clears up so that could be a factor as well. Not entirely sure how the window is handled as it moves between workspaces.
It's not 100% clear what starts the flicker, its not something that is there from the start of the app. Just a moment ago it started after I clicked "open folder" from the explorer bar. Typically persist until I restart VS Code with the exception of what I noted about about how sometimes it clears up if another full screen app is behind it.
The flicker for the most part is really the entire window disappearing exposing the desktop then the screen snaps back (it's really fast, but I can see the desktop as well as other apps behind the window).
I get the flicker on both monitors (see xrandr output below), which are different resolutions

System Info
code --version
1.28.2
7f3ce96ff4729c91352ae6def877e59c561f4850
x64

uname -a
Linux localhost.localdomain 4.18.16-300.fc29.x86_64 #1 SMP Sat Oct 20 23:24:08 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

lspci -k | grep -EA3 VGA
01:00.0 VGA compatible controller: NVIDIA Corporation GP106 [GeForce GTX 1060 6GB] (rev a1)
	Subsystem: Gigabyte Technology Co., Ltd Device 371a
	Kernel driver in use: nvidia
	Kernel modules: nouveau, nvidia_drm, nvidia

cat /proc/driver/nvidia/version
NVRM version: NVIDIA UNIX x86_64 Kernel Module  410.66  Wed Oct 10 12:01:53 CDT 2018
GCC version:  gcc version 8.2.1 20180801 (Red Hat 8.2.1-2) (GCC)

xrandr
Screen 0: minimum 8 x 8, current 4160 x 1080, maximum 32767 x 32767
DVI-D-0 disconnected (normal left inverted right x axis y axis)
HDMI-0 connected primary 2560x1080+0+0 (normal left inverted right x axis y axis) 798mm x 334mm
   2560x1080     60.00*+
   1920x1080     60.00    59.94    50.00    29.97    60.05    60.00    50.04
   1680x1050     59.95
   1600x900      60.00
   1280x1024     75.02    60.02
   1280x720      60.00    59.94    50.00
   1152x864      75.00
   1024x768      75.03    60.00
   800x600       75.00    60.32
   720x576       50.00
   720x480       59.94
   640x480       75.00    59.94    59.93
DP-0 disconnected (normal left inverted right x axis y axis)
DP-1 disconnected (normal left inverted right x axis y axis)
DVI-D-1 connected 1600x900+2560+0 (normal left inverted right x axis y axis) 442mm x 249mm
   1600x900      60.00*+
   1280x800      59.81
   1280x720      60.00
   1152x864      75.00
   1024x768      75.03    70.07    60.00
   800x600       75.00    72.19    60.32    56.25
   640x480       75.00    72.81    59.94

cat /usr/share/gnome/gnome-version.xml
<?xml version="1.0"?>
<gnome-version>
 <platform>3</platform>
 <minor>30</minor>
 <micro>1</micro>
 <distributor>GNOME.Org</distributor>
 <date>2018-09-26</date>
</gnome-version>

ubuntu 18.04 - same issue
Switched to wayland - no flickering
--disable-gpu is really not an option.
Guys, let's try to fix it.
Thanks!

Having this issue on Ubuntu 18.10 on a Librem13 laptop.

Got this issue in Ubuntu 18.10 with VSCode 1.28.2, please fix this soon.

Upgraded my Dell XPS 13 9370 to Ubuntu 18.04 from 16.04 and started having this problem.
$ code --version
1.29.0
5f24c93878bd4bc645a4a17c620e2487b11005f9
x64

$ lsb_release -irc
Distributor ID:	Ubuntu
Release:	18.04
Codename:	bionic

I believe this is related to the version of Electron that is shipped with VSCode by default, which I believe is still 1.x? I first ran into this issue a few months back, when I switched to an Ubuntu 18.04 Dell XPS laptop. I noticed the same issues in another app, Slack, which also uses Electron 1.x. Same flickering, only on Linux, and only when the apps are fullscreen (or maximized) on an external 4K monitor. After trying a bunch of things, I switched to VSCode Insiders, which uses Electron 2.x, and the flickering went away. I know Insiders isn't for everyone, but maybe this will help someone until VSCode ships Electron 3.x, which seems to be in the works.
For VSCode devs, some symptoms:

"Flickering" isn't the perfect word to describe the effect. It actually looks like some DOM layers are either not rendered, or rendered partially.
Not unique to VSCode. Slack on Electron 1.x has the same flickering issues.
Only occurs when the editor window is larger than a certain size. When I have VSCode take up half the width of a 4K screen, flickering goes away, but as I stretch the window closer and closer to full width, the effect get worse and worse. I can repro this in VSCode and in Slack.
I have never seen this come up on macOS or Windows, despite using those VSCode builds heavily.

My hardware:
Ubuntu 18.04
Dell XPS 13 2018 9370
i7-8550U Processor
Intel® UHD Graphics 620

I had been experiencing the same problem intermittently for a couple weeks after setting "window.titleBarStyle": "custom". Setting it back to "native" as referred to by @mofax fixed it for me.
code --version
1.29.1

Distributor ID: Debian
Release:        9.6
Codename:       stretch

Using Dell 24" Ultrasharp UHD monitor and an AOC 27" UHD with onboard Intel graphics on an ASRock K6 Z370 mboard.

Weird. Ubuntu 18.04 and setting title bar style to custom fixed it for me.
Else the entire window when full screen would flicker in brightness...

Okay switching back to native cured it there too.

@eldarshamukhamedov

"window.titleBarStyle": "native"

adding this configuration literally solves the problem for me. I remember that the last time I updated VSCode on my Mint Linux machine, the window title bar style was changed to Microsoft Windows style, and after that the flickering problem started to happen. The solution was to force the editor back to the native styled title bar.

"window.titleBarStyle": "native"

Had the same problem, solves also for me.
OS: Debian 9.5
Kernel: Linux 4.19.0-rc4-amd64
Videocard: ASUSTeK Computer Inc. Baffin [Radeon RX 460/560D / Pro 450/455/460/555/560] [1043:054d]

another discovery I made on a different computer
if you set

"window.titleBarStyle": "native"

but the flikering continues ... also add this

"terminal.integrated.rendererType": "dom",

completely solved the flickers on all my machines, on both regular and insider releases

--disable-gpu - did trick for me :)

On a fresh install and machine, changing my display resolution back to my 'recommended' setting did it for me it seems.

For me it was "Dynamic Brightness" on my dell xps 13. Why they though this was a good idea I don't know. When sections on the screen got bright enough, it'd bounce off my shirt, triggering the light sensor, causing the brightness to adjust. Scrolling would cause a flicker as brighter areas of the screen would scroll into/out of view.
So dumb.
If you own a fancy laptop, check in the bios for "Dynamic Brightness" and turn it off.

--disable-gpu seems to solve for me as well. Now, how can I make this permanent?

"window.titleBarStyle": "native" . fixed it for me on my MacOS Mojave 10.14

Same for me: :sleeping:
https://youtu.be/Wedcs-Pu-gw
Version: 1.34.20-insider
Commit: e83e24a0c9225becd7341e56952177a20d5d4629
Date: 2019-05-14T16:08:09.042Z
Electron: 3.1.8
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Linux x64 4.15.0-30deepin-generic

On a fresh install and machine, changing my display resolution back to my 'recommended' setting did it for me it seems.

Are you saying toggling to recommended and back to the setting you actually wanted, or just leaving it on Recommended? Because that's not really a solution.

I get this on windows as well. --disable-gpu seems to fix it.
When I press F11 or when I enter Zen-Mode my screen flashes black and certain elements would vibrate around as well. I also notice that I get a shadowplay notification when going full-screen. That only tends to happen when a game is ran or something. I am on a 2080ti with driver version 425.31. This seems to happen to others on Intel and AMD gpus though and might just be a rendering issue with Electron like some of the other tickets say.
I hope the --disable-gpu mitigation doesn't have to be required anymore.

This can be configured now. Closing as duplicate of https://github.com/microsoft/vscode/issues/15211


issue found

Before creating an issue, first upgrade cfscrape with pip install -U cfscrape and see if you're still experiencing the problem. Please also confirm your Node version (node --version or nodejs --version) is version 10 or higher.
Make sure the website you're having issues with is actually using anti-bot protection by Cloudflare and not a competitor like Imperva Incapsula or Sucuri. And if you're using an anonymizing proxy, a VPN, or Tor, Cloudflare often flags those IPs and may block you or present you with a captcha as a result.
Please confirm the following statements and check the boxes before creating an issue:

[ YES ] I've upgraded cfscrape with pip install -U cfscrape

[ YES ] I'm using Node version 10 or higher
ii  nodejs                                        10.19.0~dfsg1-1                     amd64

[ YES ] The site protection I'm having issues with is from Cloudflare

[ NO ] I'm not using Tor, a VPN, or an anonymizing proxy

Python version number
Run python --version and paste the output below:
root@balder:~# python --version
Python 2.7.16
root@balder:~#

cfscrape version number
Run pip show cfscrape and paste the output below:
root@balder:~# pip show cfscrape
Name: cfscrape
Version: 2.1.1
Summary: A simple Python module to bypass Cloudflare's anti-bot page. See https://github.com/Anorov/cloudflare-scrape for more information.
Home-page: https://github.com/Anorov/cloudflare-scrape
Author: Anorov
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /usr/local/lib/python2.7/dist-packages
Requires: requests
Required-by:
root@balder:~#

Code snippet involved with the issue
#!/usr/bin/env python

import csv
import os
import sys

import cfscrape

scraper = cfscrape.create_scraper()

filename = 'psn.csv'
with open(filename, 'rb') as f:
    reader = csv.reader(f)
    try:
        for row in reader:
            if 'http' in row[0]:
                reverse = row[0][::-1]
                i = reverse.index('/')
                tmp = reverse[0:i]
                cfurl = scraper.get(row[0]).content
                if not os.path.exists("./"+tmp[::-1]):
                    with open(tmp[::-1], 'wb') as f:
                        f.write(cfurl)
                        f.close()
                else:
                    print("file: ", tmp[::-1], "already exists")
    except csv.Error as e:
        sys.exit('file %s, line %d: %s' % (filename, reader.line_num, e))

Complete exception and traceback
root@balder:~# ./grab.py
Traceback (most recent call last):
File "./grab.py", line 20, in
cfurl = scraper.get(row[0]).content
File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 543, in get
return self.request('GET', url, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/cfscrape/init.py", line 129, in request
resp = self.solve_cf_challenge(resp, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/cfscrape/init.py", line 204, in solve_cf_challenge
answer, delay = self.solve_challenge(body, domain)
File "/usr/local/lib/python2.7/dist-packages/cfscrape/init.py", line 292, in solve_challenge
% BUG_REPORT
ValueError: Unable to identify Cloudflare IUAM Javascript on website. Cloudflare may have changed their technique, or there may be a bug in the script.
Please read https://github.com/Anorov/cloudflare-scrape#updates, then file a bug report at https://github.com/Anorov/cloudflare-scrape/issues."
(If the problem doesn't involve an exception being raised, leave this blank)

URL of the Cloudflare-protected page
cant provide if needed later
URL of Pastebin/Gist with HTML source of protected page
no idea what this is

same here (and it's my first use of the package)


Why SentencePieceTokenizer can't save vocab file

I want to use vocab file in PairedDataloader, but the the save_vocab function of SentencePieceTokenizer only save the model file.
The model file can't be load by Dataloader because of decoding error.
In sentencepiece_tokenizer.py, I saw you delete the vocab file.

We deleted sentencepiece vocab file because sentencepiece mode file is purely self-contained, but vocab file is never used in the tokenizer. To the best of my knowledge, the vocab file itself is not very useful. Here is a simple vocab file:
<unk>	0
<s>	0
</s>	0
,	-3.39764
.	-3.53133
▁the	-3.56031
s	-3.70819
▁	-3.82609
▁I	-3.90308
▁to	-4.04041
▁a	-4.08637
ed	-4.16661
▁and	-4.26836
▁of	-4.27461
t	-4.31782
e	-4.43336
d	-4.44333
ing	-4.46929
a	-4.53839
▁in	-4.64852
o	-4.71318
▁was	-4.77909
▁"	-4.81017
i	-4.86229
...

@gpengzhi
But how to use the model file in PairedTextData?
The model file seems only can be used to restore a tokenizer, so I created my own "PairedTextData" with two DataSource to use SentencePieceTokenizer in my project.
Is there anyway more simple to use?

Could you write down how you integrate tokenizer with pairedtextdata? There is another related issue #256 I think we should provide the interface to use tokenizer instead of vocab. Do you think if you can contribute to this feature enhancement? A feature enhancement pull request is welcome!


[Placeholder] DebugAppView host events are not rethrown

Need to add a regression test, fix, and then close this issue.

It turns out this is a ~duplicate of https://github.com/dart-lang/angular/issues/631.

https://github.com/dart-lang/angular/blob/e81df5fd6733e5cec371b24857ed09ca5b05746e/angular/lib/src/core/linker/app_view.dart#L609-L618
In at least some cases, you are already in the zone...
...so handler() will not trigger the ExceptionHandler. This seems like a serious bug.


Introduce couch_db:normalize_dbname

Move duplicated logic into couch_db:normalize_dbname. We could use this
helper function anywhere we need to extract dbname from shard path.
COUCHDB-2715
We would need this PR for COUCHDB-2635

+1


New features & bug fixes

This PR adds new features and fixes a couple bugs, though at the price of breaking the API (which is why I've upped the version to 0.3.0). This was made to accomodate for a better config migrator in Ly, though I've tried to make it as generic as possible. Here is the complete list of changes:

Backport Zig 0.12.0 compilation support from v0.2.2
Renamed XwithMap functions to just X and removed their "shorthand" versions (they didn't accomplish much, really)
Added support for writing all fields from a struct via a new parameter in writeFromStruct
Fixed writing null or booleans from a struct
Added support for a tryMapField function as a parameter in readToStruct to map keys, namespaces and convert values where appropriate. This also removes the previous field string map
Added support for checking if any field has been actually mapped by tryMapField as a parameter in readToStruct

Changes to the writer look fine.
But I believe the changes to the reader could be implemented in a cleaner way.
Check out the following gist and let me know if these changes to reader.zig fulfill your requirements.
https://gist.github.com/Kawaii-Ash/85137b50a267c6069c049d0e7281c440

@Kawaii-Ash Your changes don't handle nested fields, e.g.:
other=33
[Nested Config]
other=12

Here, it'd only handle the first, top-level other, not the nested one.

I am able to change the key and value of a nested field like this:
fn mapFields(_: std.mem.Allocator, field: IniField) ?IniField {
    var mapped_field = field;
    if (std.mem.eql(u8, field.header, "Colors")) {
        if (std.mem.eql(u8, field.key, "tg")) {
            mapped_field.key = "bg";
            mapped_field.value = "123";
        }
    }

    return mapped_field;
}

I am able to change the key and value of a nested field like this:
fn mapFields(_: std.mem.Allocator, field: IniField) ?IniField {
    var mapped_field = field;
    if (std.mem.eql(u8, field.header, "Colors")) {
        if (std.mem.eql(u8, field.key, "tg")) {
            mapped_field.key = "bg";
            mapped_field.value = "123";
        }
    }

    return mapped_field;
}

Yeah, I suppose that can work. I think having a flattened structure makes it easier to handle nested fields, but then your solution makes it more flexible. I think I'll go with yours since it's way more simple to implement.


Build errors in typescript-webpack project when running build:dev

Running the skeleton-navigation/typescript-webpack project, I'm seeing these errors from the 'build:dev' task -
npm ERR! Darwin 15.5.0 npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "webpack" "--" "--progress" "--profile" npm ERR! node v4.4.7 npm ERR! npm  v3.10.3 npm ERR! code ELIFECYCLE npm ERR<EMAIL_ADDRESS>webpack: cross-env ./node_modules/.bin/webpack "--progress" "--profile"`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>webpack script 'cross-env ./node_modules/.bin/webpack "--progress" "--profile"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the aurelia-skeleton-navigation-webpack package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cross-env ./node_modules/.bin/webpack "--progress" "--profile"
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs aurelia-skeleton-navigation-webpack
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls aurelia-skeleton-navigation-webpack
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/.../Documents/gc-2/npm-debug.log
npm ERR! Darwin 15.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "build:dev"
npm ERR! node v4.4.7
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build:dev: cross-env NODE_ENV=development npm run webpack -- --progress --profile
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:dev script 'cross-env NODE_ENV=development npm run webpack -- --progress --profile'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the aurelia-skeleton-navigation-webpack package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cross-env NODE_ENV=development npm run webpack -- --progress --profile
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs aurelia-skeleton-navigation-webpack
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls aurelia-skeleton-navigation-webpack
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/.../Documents/gc-2/npm-debug.log`
I've attached the debug log, although it doesn't look too informative.
Also, as a side question -
If an application developer is building an application that was based on a skeleton project - like the typescript-webpack skeleton - what is the expected development process for incorporating changes made to the skeleton project?
I've tried to do this once with the recent beta.3.0.1 version, and all I could come up with was running diff on the entire folder structure and manually merging changes.  Since the app developer is not doing a Git checkout of the project, there isn't a Git merge that can be done, correct?
Regards,
Don
npm-debug.log.txt

aurelia-skeleton-webpack
Getting started
Before you start, make sure you have a recent version of NodeJS environment >=4.0 with NPM 3.

What task is that? Can you post the output of the command, not only the error? I see you're using an older version of the skeleton, is the same problem occurring with 1.0.0?

The task I'm referring to is 'npm run build:dev'.
Regarding the skeleton, yes, I am using beta3.0.1 version, which I had just updated to yesterday, and I see now that on this same day the version was updated to 1.0.0.  That is also why I had asked the question about how developers are expected to manage changes to skeleton versions -

Also, as a side question -
If an application developer is building an application that was based on a skeleton project - like the typescript-webpack skeleton - what is the expected development process for incorporating changes made to the skeleton project?

I've tried to do this once with the recent beta.3.0.1 version, and all I could come up with was running diff on the entire folder structure and manually merging changes. Since the app developer is not doing a Git checkout of the project, there isn't a Git merge that can be done, correct?

tried npm start?

Of course - but npm start is for running live in development, not for building.
Is there some reason to think that npm start should work, but npm run build:dev should not?  I imagine that there are a number of tasks being executed in 'build' that are skipped when running in the live dev server mode.

no idea i just know that npm start worked for me adn npm run server:dev didn't

@doktordirk npm run server:dev does not work for you too then, correct?

i mixed that up somewhat
npm run server:dev or npm start -> as expected
npm run watch:dev -> stalls at some stage seemingly
i didn't make many experiments though. that was with ann npm prune && npm iof de2ae65 skeleton-es2015-webpack:

@niieani
I continue to have this issue.  THis is a major problem for me, because it means I essentially cannot produce a bundled build - I can only run in dev mode.
I don't see anything changing with this.
Again, this occurs in the typescript-webpack skeleton, when I run the command:
npm run build:dev
The build runs, but at the end it fails with a non-informative error:
npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "webpack" "--" "--progress" "--profile"
npm ERR! node v4.4.7
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>webpack: `cross-env ./node_modules/.bin/webpack "--progress" "--profile"`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>webpack script 'cross-env ./node_modules/.bin/webpack "--progress" "--profile"'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the aurelia-skeleton-navigation-webpack package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cross-env ./node_modules/.bin/webpack "--progress" "--profile"
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs aurelia-skeleton-navigation-webpack
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls aurelia-skeleton-navigation-webpack
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/donpeterkofsky/Documents/gc-2/npm-debug.log

npm ERR! Darwin 15.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "build:dev"
npm ERR! node v4.4.7
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build:dev: `cross-env NODE_ENV=development npm run webpack -- --progress --profile`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:dev script 'cross-env NODE_ENV=development npm run webpack -- --progress --profile'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the aurelia-skeleton-navigation-webpack package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cross-env NODE_ENV=development npm run webpack -- --progress --profile
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs aurelia-skeleton-navigation-webpack
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls aurelia-skeleton-navigation-webpack
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/donpeterkofsky/Documents/gc-2/npm-debug.log

I am also attaching the npm debug log, although I can't find anything helpful in it.
Can you please take a look at this, and let me know what to provide you with to reproduce it?
Thanks,
Don
npm-debug.log.txt

@don-bluelinegrid so does a fresh install of skeleton-navigation not work, or are you having problems migrating to it? Not sure how I can reproduce your issue, the log is uninformative as you say.

@niieani
I am using a project based on the latest skeleton - 1.0.0.  I have already migrated to it.
As I mentioned in the original report - once a project begins that is based on a skeleton, it becomes problematic to debug of course, because a project will build on/deviate from the base skeleton.  It is unrealistic to expect that a commercial project will not deviate from the skeleton.  So, we need assistance debugging problems in projects that are based on skeletons.

@niieani
I had previously debugged this, and found that it was related to aurelia-i18n.  Please see this comment:
https://github.com/aurelia/i18n/issues/56#issuecomment-236010123
However, it seems that this occurs only for the typescript-webpack skeleton.  Can you please take a look at the comment, and see if you have some suggestion on how to make this work?  Or, does this require a fix from the i18n module?  If so, how does that get fixed?
Thanks,
Don

We'll have a way to migrate configs in the future so this doesn't happen again, plus we don't expect major changes there either (it was pre-1.0.0 to 1.0.0 migration that was larger).
I've read the comment however, and I don't see why that would be influencing the builds in such a way. Could you prepare a simple repo that reproduces the issue?
We do our best in the free time to debug problems such as the one you're experiencing, however for debugging commercial projects, we do have paid consulting options available and I can refer you to @EisenbergEffect regarding details and pricing.

@niieani
This could be reproduced by using the current 1.0.0 typescript-webpack skeleton, with the aurelia-i18n module added.  As per above, that is what causes the problem.
If you can't reproduce it that way, I can create a repo that essentially does this.
Don

@niieani
After updating to aurelia-18n v1.1.1, this is now failing:
default] Checking started in a separate process...
[default] /Users/donpeterkofsky/Documents/gc-2/node_modules/aurelia-i18n/dist/aurelia-i18n.d.ts:42:33
    Cannot find namespace 'i18next'.
[default] /Users/donpeterkofsky/Documents/gc-2/node_modules/aurelia-i18n/dist/aurelia-i18n.d.ts:43:27
    Cannot find namespace 'i18next'.

Seems like a typings bug in aurelia-i18n. Shouldn't affect the output though.

Is this still an issue?

@EisenbergEffect I have this issue using i18next. If I use import * as i18next then i get errors in console when running npm start

@EisenbergEffect I've been having this issue (using version 1.0.2 of the nav skeleton), but I think I found the problem, at least in my case. While builds were failing, when running the dev server, TypeScript was complaining about the aurelia-protractor custom typings. It complained that it couldn't find the Locator type and that the declaration of ProtractorBy in the actual protractor package was conflicting with the one in the custom typings.  Not really a permanent solution, but does shed light on what's wrong.

@AshleyGrant Can you investigate what our best course of action would be for the protractor stuff with respect to TypeScript?

I'm having this same issue - I've commented out the aurelia-protractor.d.ts contents as a workaround (thanks @Ixonal for suggesting that) but obviously it's not a long term solution. Would be awesome to get this fixed.

Fixed by https://github.com/aurelia/skeleton-navigation/pull/714


Fixes bug with "this" context

Without this fix, the "this" keywords refers to the "createClient" function instead of the "client" defined at line 16.

Hmm, I don't see how this could refer to anything meaningful before or after. Assuming you can test this, can you try setting winner = client?

If a arrow function is used, this should refer to the forEach body function, which also should be wrong, so please test and change the PR to winner = client.

@silverwind I have updated the pull request with the winner = client code

Thanks, I'll push a patch release with this.
BTW, I see that ldapjs is broken again on macOS 10.14 unless one deletes the dtrace-provider dependency. Might wanna check out https://github.com/zont/ldapjs-client which seems like a maintained alternative to ldapjs. I suppose support for multiple servers there could be contributed.


New- FREE Open-Source Music Instrument Code:

Hey everyone,
We just pushed a new repo. It's an open-source music instrument app. I would be honored if you wanted to check it out:

https://github.com/AudioKit/ROMPlayer
App Store Example:
FM Player
Thank you.
@fethica Also has a cool new repo he is working on. I'll let him tell you about it when he is ready. Stay tuned!

Hey everyone,
I'm currently working on a new radio player based on AVPlayer : https://github.com/fethica/FRadioPlayer and it will be added to the SwiftRadioPro app soon, I already started implementing it in a separate branch: https://github.com/swiftcodex/Swift-Radio-Pro/tree/dev
Everything still in development, but feel free to test them out!
Cheers!
Thanks @swiftcodex and congrats for your new amazing app 👏

@fethica's code is very exciting. We are all very lucky that he is contributing to Swift Radio and making it better with his awesome work. Make sure you check out his repo and give him a star! ⭐

@fethica ,
            // Turn off network indicator in status bar
            UIApplication.shared.isNetworkActivityIndicatorVisible = false

must be used from main thread.
public extension String {
    public var length: Int { return self.characters.count }

    public func toURL() -> NSURL? {
        return NSURL(string: self)
    }
}

        let scanner = Scanner(string: hex)
        var hexValue: CUnsignedLongLong = 0
        if scanner.scanHexInt64(&hexValue) {
            switch (hex.characters.count) {

characters is deprecated, use String or SubString instead.
            UIApplication.shared.openURL(url)

is deprecated.
        let albumArtwork = MPMediaItemArtwork(image: track.artworkImage!)

is deprecated.

Hello @iDevelopper !
Thanks for the feedback, All warning/deprecated code will be fixed in the next version (dev branch)!

My newest open-source project:
https://audiokitpro.com/synth


AnimatedCollapseArrow extended

The CollapseArrow is now a own widget in the productPage and the preferences.
Added animation to the preferences.

@monsieurtanuki no problem, this pr can wait


Add ElastiCache support

What problem are you facing?
Would like add support for ElastiCache resource
How could Crossplane help solve your problem?
Enables creation of ElastiCache resources
References

https://crossplane.io/docs/v0.9/api/crossplane/provider-aws/cache-aws-crossplane-io-v1beta1.html

We already have support for AWS ElastiCache ReplicationGroups. I don't think adding cache cluster support would add much value; I believe you can do everything you can do with a cache cluster using a replication group. We'll still want cache clusters eventually, but it's not a high priority.

CacheCluster is implemented<EMAIL_ADDRESS>also a lot of other resources in Cache Group


[Bug]: Bootstrap misspelled incorrect

Bug Title
[Bug]: Bootstrap misspelled incorrect
Description of BUG
tailwind bootsrap instead of bootstrap in follow section us section
Reporduction URL of bug
https://tailwindcsscomponents.vercel.app/
Bug Issue Type
Frontend
Screenshot of the issue if in UI/ Document

Are you taking this issue fix
YES

@munyanezaarmel go ahead with fix

@jsvigneshkanna done check it


allow pulling ssh_keys from a url

#411
Signed-off-by: Ben Abrams<EMAIL_ADDRESS>Description
Often people expose ssh public keys via a url such as https://github.com/majormoses.keys which has a line separated list of keys. Basically this allows using static and dynamic ssh keys.
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

I am not sure why the travis builds are failing but the kitchen tests pass fine for me locally:
$ kitchen verify default-ubuntu-1604
-----> Starting Kitchen (v1.16.0)
WARN: Unresolved specs during Gem::Specification.reset:
      artifactory (>= 0)
      octokit (~> 4.0)
      unf_ext (>= 0)
      winrm (~> 2.0)
      winrm-fs (~> 1.0)
      docker-api (~> 1.26)
      rainbow (~> 2)
      rspec (~> 3)
      diff-lcs (< 2.0, >= 1.2.0)
WARN: Clearing out unresolved specs.
Please report a bug if this causes problems.
-----> Verifying <default-ubuntu-1604>...
       Loaded tests from {:path=>"/home/babrams/projects/personal/chef-cookbooks/users/test/integration/default"}

Profile: tests from {:path=>"/home/babrams/projects/personal/chef-cookbooks/users/test/integration/default"}
Version: (not specified)
Target:  ssh://vagrant@<IP_ADDRESS>:2222

  User test_user
     ✔  should exist
     ✔  uid should eq 9001
     ✔  groups should eq ["test_user", "testgroup", "nfsgroup"]
     ✔  shell should eq "/bin/bash"
  Group testgroup
     ✔  should exist
     ✔  gid should eq 3000
  Group nfsgroup
     ✔  should exist
     ✔  gid should eq 4000
  User test_user_keys_from_url
     ✔  should exist
     ✔  uid should eq 9002
     ✔  groups should eq ["test_user_keys_from_url", "testgroup", "nfsgroup"]
     ✔  shell should eq "/bin/bash"
  File /home/test_user_keys_from_url/.ssh/authorized_keys
     ✔  content should include "ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAACAQC4j6wHVxbs1FQXxV4DaG6RyXMgHfxTSB0tb2uA2Z8kNy9mwZ14zvT9kUCyniXw...SpjyZqQI0tNCBUd8kE0ZR3Zn7OxXUvcanh5cgYRHiSeK10TKSd7BkzNUponUZTymZmHyeYLBZ8U1i83VJpQxQGOWbItJdD6MQ=="
     ✔  content should include "ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAACAQCy3cbPekJYHAIa8J1fOr2iIqpx/7pl4giJYAG7HCfsunRRUq3dY1KhVw1BlmMG...lvkgzT2h4Ie6lyRgNv5QnUyAnW43O5FdBnPk/XZ3LA462VU3uOfr0AQtEJzPccpFC6OCFYWdGwZQA/r1EZQES0yRfJLpx+uZQ=="

Test Summary: 14 successful, 0 failures, 0 skipped
       Finished verifying <default-ubuntu-1604> (0m0.76s).
-----> Kitchen is finished. (0m41.60s)

maybe the tests failing last night were caused by this? https://www.traviscistatus.com/incidents/4vdl52d28hz3

Nice addition. Thanks


vendor: Update for virtcontainers, logrus and sys/unix.

This change is primarily for [1] to support the introduction of
"runtime list".
--
[1] - https://github.com/containers/virtcontainers/pull/254.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

Coverage remained the same at 35.74% when pulling 9be82d4c581924d7d4b465496c21e1c2e4832185 on jodh-intel:revendor into f0609685c94f4377ef2b8c4ec0f0877fbd9feb33 on clearcontainers:master.

lgtm

LGTM

Hi @sameo - Since the only other reviewer apart from myself is not around for a few days, could you disable the pullapprove hook temporarily to allow this (and then #132) to be merged?
We then need to ensure we have atleast 3 reviewers / project to avoid this scenario :smile:

lgtm


Perform post-release operations after 1.0.1

Codecov Report

Merging #150 (94cc0af) into main (1b1d0c0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #150   +/-   ##
=========================================
  Coverage     90.40%   90.40%
  Complexity      542      542
=========================================
  Files            94       94
  Lines          1657     1657
  Branches        111      111
=========================================
  Hits           1498     1498
  Misses          143      143
  Partials         16       16

Continue to review full report at Codecov.

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

Force pushed to retrigger CLA check.


Add CLI options for base and max_range

The API rewrite gives a few more options for clients to select. Make these available as CLI flags.

resolved via #10


while executing Ansible-playbook  facing the below issue could anyonehelp

here is the inventory

here is the playbookfile

Make sure ansible user have sudo permisions on remote host
edit /etc/sudoers file add entry for anisble user

i have added user to sudo group  by executing the sudo usermod -aG sudo ansible

By adding user to sudo group using usermod user will ask for password when we try to become sudo. so we need user which has sudo permissions and password should not be asked.
add <user> ALL:ALL NOPASSWD:ALL entry in /etc/sudoers file

ok thank you  it's working   ALL=(ALL:ALL) NOPASSWD:ALL


Broken link

For the blue and gold committee, the link to their site is giving an error, as it is sent to https://blueandgold.skuleca/ instead of https://blueandgold.skule.ca/

Thank you so much for the issue, this has now been fixed! :)


Remove Spinner releated settings

🎯 Aim
As part of the fix for the related issue we removed the ora dependency and the spinner/loader. The aim now is to remove spinner related configuration.

🔗 Related
Based on the discussion from this issue: #6117

Hi @Adam-it, you are including this issue in #6117 right?

#6117 only removes the spinner itself. We keep the settings for backwards compatibility. Since we're releasing v8 out of schedule, I suggest that we remove them in v9 to give everyone time to test and update their code

Hi @Adam-it, you are including this issue in #6117 right?

TBH I was planning to, but what @waldekmastykarz suggests it's true and we need to give folks some time to adapt


Security Log Profile

Added Bigip Security Log
Added unittest and functional test for the same

f5/bigip/tm/security/test/functional/test_log.py ......

======================================= 6 passed in 0.54 seconds =======================================

f5/bigip/tm/security/test/unit/test_log.py ...
======================================= 3 passed in 0.07 seconds =======================================

@caphrim007

@hemantsk I've been working on this PR today and have one incoming.

@hemantsk I wont merge this, as I have my own that will be merged.


CM doesn't refresh token used to authenticate against Vault

🤦♂ looks like this was fixed in #61, which was submitted shortly after your own PR... Thanks though 😄


Prisma 5 Upgrade Guide

Resolves #4848
This is the upgrade guide for Prisma 4 -> Prisma 5. The last few sections are stubs, but the majority of the breaking changes have been documented (with the exception of rejectOnNotFound).
Putting this up as a draft to collect feedback.

I've wanted to take a look at this and probably went about it differently than most of you, and ended up on https://docs-git-jharrell-issue4848-prisma.vercel.app/guides/upgrade-guides that left me wondering why I wouldn't see Prisma 5 there.
Should we make the upgrade guide more easily discoverable on that page? Would you expect that users land there?

I've wanted to take a look at this and probably went about it differently than most of you, and ended up on docs-git-jharrell-issue4848-prisma.vercel.app/guides/upgrade-guides that left me wondering why I wouldn't see Prisma 5 there.

Should we make the upgrade guide more easily discoverable on that page? Would you expect that users land there?

Excellent idea and suggestion!
These links are auto generated with a component (which shows "subapges of a page") and I am not sure if we have multiple levels there. We probably should ahve this, would improve quite a lot of these pages.


warn about cluster autoscaler and upgrade

Non managed autoscalere and upgrade can fight. Let people know.

@paulgmiller : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Resolved the merge conflict.
@mlearned - Could you review the pull request? Thanks.


[submodule-sync] bot-submodule-sync-branch-22.10 to branch-22.10 [skip ci] [bot]

submodule-sync to create a PR keeping thirdparty/cudf up-to-date.
 If failed, it will remain open until test pass or manually fix.


Swift 5 migration. Warnings fixed.

@tidwall any idea when you can merge this?

+1, would love to see this hit master sooner rather than later.

Thanks for the PR!

@tidwall Can you update the podspec?

@tidwall pickettjd How can I use it in pod file?

just replace dep in your Podfile:
pod 'SwiftWebSocket', :git => 'https://github.com/tidwall/SwiftWebSocket.git', :tag => 'v2.8.0'


vv error

Error detected while processing function 69_GrepCurrentWord..69_DoGrep..69_HasTargetsTahtMatch...
E118: Too many arguments for function : glob
E15: Invalid expression : glob(p,0,1)
E121: Undefined variable: fileList
vim version is 7.3   OS: win7

What version of Vim are you using and what platform are you running on?
The glob() function has three arguments; that's what the code is passing in. I can see from your error message that p is a '*', which is one of the valid wildcards to glob.
Can you experiment with calling glob() on your own to see why it's failing for you?

I also get this error, just installed it. Version 7.3 on OSX yosemite

I just defaulted the option that appears to be causing the problem to zero. Can you try again?


Framerate issues after upgrading to evdi 1.5.0

Running ArchLinux 4.13.6-1 - the issue began when upgrading from kernel 4.12 to 4.13, and evdi 1.4.1 to 1.5.0. Getting a very low framerate, and a flood of errors in my dmesg:
[  799.610105] evdi: [W] collapse_dirty_rects:112 Not enough space for clip rects! Rects will be collapsed
(This appears to be a similar issue to #100)

I experience the same behaviour running Fedora 26. It seems worse with Chrome and Electron based applications.
displaylink version: 1.4
evdi version: 1.5.0
Kernel version: 4.13.5

I'll also add that Java apps (PHPStorm) cause the framerate to go dog-slow while they load. Takes a good while for it to settle out.

I can confirm with Java apps, IntelliJ and Android Studio, I have noticeable lag.

This is not reproducible with Ubuntu 17.10 using Kernel 4.13 and PageFlip set to off in modesetting options.

Same behavior running Slackware 14.2:
Kernel: 4.4.88
evdi: 1.5.0
displaylink: 1.4.210

Please raise these on DisplayLink's Linux forum and attach logs which are specific to this particular evdi client (DisplayLinkManager). So far we have not been able to reproduce this - maybe attaching a video captured when you experience this would help, too. I'll close this as it doesn't seem to be a generic evdi problem so far.

I have seen the same issues with 17.04, kernel 4.10, with the intel drivers (I can't get modesetting to work even with pageflip off; black screen). It's not a consistent frame rate issue. There are short periods when the framerate is normal.


KeyError:Cipher, how to fix

first go in extract.py  in the package and go to line 301 then replace this code:
cipher_url = [                     parse_qs(formats[i]["cipher"]) for i, data in enumerate(formats)                 ]
with:
try:                 cipher_url = [                     parse_qs(formats[i]["signatureCipher"]) for i, data in enumerate(formats)                 ]             except Exception:                 cipher_url = [                     parse_qs(formats[i]["cipher"]) for i, data in enumerate(formats)                 ]

I get KeyError: 'assets' here: /pytube/extract.py", line 143, in js_url .....

Possible duplicate of #641 solved for me by #644, try it :)

I have the same problem in some videos [in CLI]:

Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/pytube/extract.py", line 297, in apply_descrambler
for format_item in formats
File "/usr/local/lib/python3.6/dist-packages/pytube/extract.py", line 297, in
for format_item in formats
KeyError: 'url'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/bin/pytube3", line 11, in
sys.exit(main())
File "/usr/local/lib/python3.6/dist-packages/pytube/cli.py", line 48, in main
youtube = YouTube(args.url)
File "/usr/local/lib/python3.6/dist-packages/pytube/main.py", line 92, in init
self.descramble()
File "/usr/local/lib/python3.6/dist-packages/pytube/main.py", line 132, in descramble
apply_descrambler(self.player_config_args, fmt)
File "/usr/local/lib/python3.6/dist-packages/pytube/extract.py", line 301, in apply_descrambler
parse_qs(formats[i]["cipher"]) for i, data in enumerate(formats)
File "/usr/local/lib/python3.6/dist-packages/pytube/extract.py", line 301, in
parse_qs(formats[i]["cipher"]) for i, data in enumerate(formats)
KeyError: 'cipher'

I have the same problem in some videos [in CLI]:

Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/pytube/extract.py", line 297, in apply_descrambler
for format_item in formats
File "/usr/local/lib/python3.6/dist-packages/pytube/extract.py", line 297, in
for format_item in formats
KeyError: 'url'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/bin/pytube3", line 11, in
sys.exit(main())
File "/usr/local/lib/python3.6/dist-packages/pytube/cli.py", line 48, in main
youtube = YouTube(args.url)
File "/usr/local/lib/python3.6/dist-packages/pytube/main.py", line 92, in init
self.descramble()
File "/usr/local/lib/python3.6/dist-packages/pytube/main.py", line 132, in descramble
apply_descrambler(self.player_config_args, fmt)
File "/usr/local/lib/python3.6/dist-packages/pytube/extract.py", line 301, in apply_descrambler
parse_qs(formats[i]["cipher"]) for i, data in enumerate(formats)
File "/usr/local/lib/python3.6/dist-packages/pytube/extract.py", line 301, in
parse_qs(formats[i]["cipher"]) for i, data in enumerate(formats)
KeyError: 'cipher'

#644

Can we get a fix for this issue please.

facing the same issue

I get KeyError: 'assets' here: /pytube/extract.py", line 143, in js_url .....

just replace
base_js = get_ytplayer_config(html)["assets"]["js"] return "https://youtube.com" + base_js
with
base_js = get_ytplayer_config(html)["assets"]["js"] return "https://youtube.com" + get_ytplayer_config(html)["assets"]["js"]

Did you had a solution for "KeyError: 'assets'"??

Same issue with this error msg:
Traceback (most recent call last):   File "youtube.py", line 6, in <module>     yt = YouTube('https://www.youtube.com/watch?v=3NCyD3XoJgM')   File "C:\Users\test\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\pytube\__main__.py", line 91, in __init__     self.prefetch()   File "C:\Users\test\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\pytube\__main__.py", line 183, in prefetch     self.js_url = extract.js_url(self.watch_html)   File "C:\Users\test\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\pytube\extract.py", line 143, in js_url     base_js = get_ytplayer_config(html)["assets"]["js"] KeyError: 'assets'

Same issue with this error msg:
Traceback (most recent call last): File "youtube.py", line 6, in <module> yt = YouTube('https://www.youtube.com/watch?v=3NCyD3XoJgM') File "C:\Users\test\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\pytube\__main__.py", line 91, in __init__ self.prefetch() File "C:\Users\test\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\pytube\__main__.py", line 183, in prefetch self.js_url = extract.js_url(self.watch_html) File "C:\Users\test\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.8_qbz5n2kfra8p0\LocalCache\local-packages\Python38\site-packages\pytube\extract.py", line 143, in js_url base_js = get_ytplayer_config(html)["assets"]["js"] KeyError: 'assets'

that issues is open here...
https://github.com/nficano/pytube/issues/763

You can use this comand:
python -m pip install git+https://github.com/nficano/pytube

All,
Believe this is fixed in most recent version of Pytube
Let me know if this still doesnt work -- thanks


Specify supported Node.js versions / transpile to es5

Hi,
Wanted to use this great package in our app which has to run on node.js v5.10.
Bumped into run-time errors due to unsupported ES6 features (destructuring).
Could you transpile it to ES5 prior to release?
In any case, it is recommended to specify the supported node versions in package.json.
https://stackoverflow.com/questions/29349684/how-can-i-specify-the-required-node-js-version-in-packages-json
Best regards,
Vasili

Node v5 is no longer maintained, see https://github.com/nodejs/Release
You may use proper-lockfile@^2 which should support node > 4.


Fix docs for resumeAll parameter in AsyncTrigger

resumeAll is doing the opposite of what the description says.

@compujuckel , thanks for your contribution!


Feature/vagrant integration test on jenkins

Following changes are done to make sure functional test run successfully on Vagrant VM and on Jenkins:

Increase Cassandra connection & request timeout values
Increased resources (memory & cpu) for vagrant VM

Also, had to update JDK location...

Coverage decreased (-0.03%) to 75.793% when pulling 598e3779eb6dd451ca6580c05645f423e5af3109 on feature/vagrant-integration-test-on-jenkins into da9a100616a738bd4b32eeebe2c7f3934d5147e6 on develop.

Looks good :shipit:


Disabling and reenabling camera fails.

Results in:
Error: unable to restart an unpublished track.

@davidliu any plan in fixing this issue soon? Thanks.

Yeah, I'll be taking a look at this next.

Thanks. Curious to me that the latest tag in github is v1.0.0, where can I get the code for 1.1.9?


Add ability to store workflow run results

Task to do
Allow the download of workflow results
Reason
Allows the user to access their results
Result
The user can download the results

Download graphic shows up after a successful run in area highlighted


Fix: Missing import for except block criteria.

Missing import statement causing another exception to be raised when the second except clause was caught (reproducible with an open API schema URL that would return a 404 Not Found error), due to missing imports.

Hey @moisesbenzan, thanks for the contribution! Actually, it looks like this import is unnecessary since there is no general except block. I would suggest changing it to smth like that:
try:
    logger.debug(f"Resolving specification file")
    self._resolver.parse()
    return self._resolver.specification
except prance.ValidationError as error:
    raise ParserError(f"OpenAPI validation error: {error}")
except Exception as error:
    raise ParserError(f"OpenAPI file parsing error: {error}")

and remove the import of course 🙂

Fixed by #52


[BUG] Empty template name in template version view page

To Reproduce
Steps to reproduce the behavior:

Click View Config

Result

Still can be encountered.

The template version created by vm is missing the version number

when click the VM template actions it has no response

Fix in master-0008ce84-head version


Update to ROBOT build procedure

We really should have automated the build procedure properly, and this ticket covers the conversion to a ROBOT Makefile.

The new makefile builds working merged OWL files and a simple release. We may still want to have it do more exciting things, but working well so far.

ROBOT working well. Can close this now.


Add zoom level per-layout

I often need to click the "fit" button when working and switching between tiles, boards, and cards, so I added the zoom level to project state, per-layout. That way I can continue working more seamlessly between sessions.
This maybe a bit opinionated a change, but I think view state is something that is okay to keep in the save file for tools which work primarily as visual editors.

This is a good idea. I wonder if anyone would actually have an issue with it... I'll consider an application level setting to not use the per layout zoom.

I you'd like me to add it in this PR, shouldn't be a lot of work.

Go for it.

I got swamped with other stuff, so I didn't find the time to work on this.
Glad to see how it worked out.


[test] Add scalable regression testing framework

Proposed changes
Add scalable regression testing framework(#7584)
contains

Test framework making by groovy, and support built-in readable DSL named as Action
Demo exists in ${DORIS_HOME}/regression-test/data/demo
Chinese doc exist in ${DORIS_HOME}/docs/zh-CN/developer-guide/regression-testing.md

English document coming soon
Problem Summary:
Add regression test framework
Checklist(Required)

Does it affect the original behavior: No
Has unit tests been added: No
Has document been added or modified: Yes
Does it need to update dependencies: No
Are there any changes that cannot be rolled back: No

Further comments

Great~~~~~~~~~~

wonderful!

Maybe we could just merge this PR and improve it later?

Maybe we could just merge this PR and improve it later?

Yes, this is a relatively independent module. I think we can quickly merge it first, so that everyone can participate in and iterate quickly.
So it LGTM.
And @924060929 , Could you please send a email to dev@doris. The main content is to briefly introduce the motivation and purpose of this testing framework. so that we can discuss the follow-up in the mailing list?

okey


Import Thr files

Maybe an alternate input should be reading thr files. There is a great community of sisyphus creators out there that share their drawings on the sand tables and there is even some decent software for making patterns. Maybe it makes sense to read in those files, and output in gcode, or maybe allow some rudamentary editing for other thr tables.

Done.


1d100 adds the d10 when result is 10

When rolling 1d100 on a result of 10 on the d10 it is added to the d100 result.
Using the code based on the React Demo. Includes dicebox + parser.
Percentile dice = consist of two d10s, one labeled with every single-digit number 0-9, (Ones dice - or unit die) the other labeled for the tens place, so 00-90 (Tens dice - or ten die).
Expected behavior is the d10 is treated as a 0 on a roll of 10 when rolled as part of a d100. So 10 + 60 = 60, not 70.

I found the issue. In colliderFaceMap, the  d10 has two sides that map to 10 instead of 0.

Yup. You're right on that one. I've been using the 'dice or rolling' theme so much lately that I didn't notice this issue on the default dice. On the 'dice of rolling' theme, the numbers are correctly set.
I'll be sure to fix this. Thanks for bringing it up.

@frankieali We commissioned a new D10 model. I will send it to you. :)


[0.20] [cryptsetup] services between luks prompt and login do not run

0.19.1 works fine. I'm running gentoo full unstable.
Let me know if you need more info.

Are you sure they're not running, versus not printing to the console?  The output of rc-status -a and the contents of /etc/inittab would be helpful.

Something tells me your issue is fixed by d4c7207; could you try openrc-9999?

Thank you for having a look.
I will try later on today and report back.

0.20.1 indeed solves the issue.
Thank you


error: bufio.Scanner: token too long

Hello,
I am seeing an issue for a base64 encoded cert value.
The error message is:
error: bufio.Scanner: token too long
Is there a solution to this?

Also having this issue - running 0.7.0
error: bufio.Scanner: token too long node:events:505       throw er; // Unhandled 'error' event       ^ Error: write EPIPE     at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:94:16) Emitted 'error' event on Socket instance at:     at emitErrorNT (node:internal/streams/destroy:157:8)     at emitErrorCloseNT (node:internal/streams/destroy:122:3)     at processTicksAndRejections (node:internal/process/task_queues:83:21) ***   errno: -32,   code: 'EPIPE',   syscall: 'write'


Neurite is incompatible with python 3.11

Hi! I am trying to use neurite with python 3.11 and I noticed the following error:
attrs, varargs, varkw, defaults = inspect.getargspec(func)
which is cause by an incompatibility of inspect and python 3.11.
The issue can be resolved by changing the above line to:
attrs, varargs, varkw, defaults, _, _, _ = inspect.getfullargspec(func)
It would be nice to make neurite compatible with the newer python versions.
Thanks a lot!

It would be nice to push this update to pypi :)


remove omegaConf

now that omegaConf is not used much, maybe we can remove its use and remove it as a depenency

@joamatab This would be great, as omegaconf requires antlr4-python3-runtime version 4.9.*, and this causes dependency issues in our IIC-OSIC-TOOLS image.


CDAP-15822 Dynamic-schema-sinks plugin validation

JIRA: https://issues.cask.co/browse/CDAP-15822
WIKI: https://wiki.cask.co/display/CE/Guidelines+on+Plugin+Validation
In scope of this PR:

integrated with new validation API

@albertshau could you, please, take a look.

@albertshau I addressed all your comments and squashed commits, could you, please, merge.


R Shiny: Analyze Saccade Peak Velocity, Fixation Duration & Saccade Duration

In eye tracking, from SPV (Saccade Peak Velocity), FD (Fixation Duration) and SD (Saccade Duration), it is possible to monitor user fatigue. This would be useful to analyze from the data gathered via the accuracy test.
In Unity Dashboard:

Show mini-timelines of how SPV, FD and SD changed over time for the user during the test.

In R Shiny:

Have a tab for analyzing Fatigue using SPV, FD and SD for detailed analysis.

The exact UI will need some concrete design. But first of all, we need to investigate whether SPV, FD, SD is provided by the Pupil Labs API? Or is it something we need to calculate on our own?

an implementation of these measures can be found here:
https://link.springer.com/content/pdf/10.3758%2FBRM.42.1.188.pdf


[#5014] feat: add the basic module of MaxCompute

What changes were proposed in this pull request?
To add the basic module of MaxCompute, including catalog, operations, converter and so on.
Why are the changes needed?
Fix: #5014
Does this PR introduce any user-facing change?
No, only the basic capabilities to connect to MC are provided.
How was this patch tested?
It can only be achieved by configuring the environment variables(username/password/url) and then running

plz rebase the latest main branch


Create a repository for AskScript Playground

Blocked on: Use Monaco Editor in AskScript Playground

The AskScript Playground will live here https://github.com/xFAANG/askql-playground

Done

@czerwinskilukasz1 how are you planning on assuring the quality of this code when you commit it into a separate repo. We already have measure in this one such us prettier, linter and tests to assure quality. I'm also concerned that by adding another repo it will be more difficult for us to manage the entire project and its deployments. I'd prefer to continue going in the direction of monorepo here.

@mhagmajer Playground frontend is connecting AskQL backend and askql.org frontend. Both of them are separate repos, also AskQL backend repo is, well, backend, so I thought keeping Playground frontend separately would prevent introducing too much complexity in one repo.
What are your thoughts?

AskQL.org is a website where we would like to use the playground, yes. That does not mean that the code for playground frontend should go into that repo or its own repo. In my opinion it should go into the main project repo because of the arguments I listed in the previous comment.
I'm not sure what you are referring to as "AskQL backend repo". We have this one monorepo with all the quality measures in place for all software that is AskQL related.

Ok, thanks, I will add it to the main repo then.


Editing dataset to change database it points on

We use several Timescale DB databases.
Some tables have been migrated from one DB (DB1) to another (DB2).
There are datasets pointing on tables from DB1, that we would like to edit to point on DB2.
Expected results
The Dataset editing window seemingly allows us to edit the Database, schema and table, so that we could edit a dataset to point on a table from DB2 rather than from DB1.

Actual results
But when we click save on that window, nothing happens and the dataset is unchanged, it still points to the same table from DB1. We tried syncing columns from datasource before saving, the result is still the same.
It doesn't seem to be linked to roles or permissions, as it happens even with admin role.
To clarify, table name and table schema are the same in DB1 and DB2, the only part that we need to change is the Database.
Environment
(please complete the following information):

superset version: 1.0.1

Is it possible to edit a dataset to point on a table from a different database? If it is not possible, it should be made clearer in the editing window.

hey @junlincc if I squint, I believe this issue was mentioned / fixed in the last month or 2 right? Or am I mis-remembering

I am experiencing the same issue in the dockerized version of superset. Is there any fix to the docker version?

@srinify i can't find any other mention of this in the issue list.

I just experienced this as well On Superset 1.2.0. The UI reports that the changes were saved successfully, but nothing actually changes. I checked the logs and I see no errors.

Same issue on dockered version 1.2.0

I have same issue, both in New and LegacyDataEditor.

@cccs-tom / @hughhhh are either of you able to confirm/deny whether or not #17265 does indeed effectively close this, or if there are more nuances that need to be dealt with here?

@rusackas I just re-tested this and it seems to work as expected. Unfortunately, I can't remember the exact scenario I encountered (in July 2021) which prompted me to add my +1 to this bug report! But it looks fixed to me!

Well that's great news! Closing it out then, but happy to revisit if needed.


Add build validation via GitHub Actions

Adds support for building, packing, and publishing artifacts in both PRs and on merges. For the most part, I used a combination of the commands in the README and buildrelease.bat, with some minor changes to the cmake pattern. I also found a dependency I needed to install for Linux that wasn't in the README, so I added it.
A successful test run can be found at https://github.com/keveleigh/openxr-explorer/actions/runs/2481916369 (since it looks like this PR won't run it, since it's not my repo, which makes sense!!)

Woah, neat! That's way simpler than I was thinking it would be, thanks a bunch for this :D


dismissible: false does not work

when I set this option it not works, modal close when you click outside of the modal!
$('.modal-trigger').leanModal({dismissible: false})

See CONTRIBUTING.md and make a code pen illustrating your issue.

I tested it and it works.


Indicate Solaris / Illumos has mmap support

After bumping into #98 on SmartOS (Illumos derivative) I added Solaris to the list of OS'es with mmap support.

According to https://sourceforge.net/p/predef/wiki/OperatingSystems/ this seems like a reasonable way to do this. I'll run some tests and double check some things, but conceptually this seems fine to merge.
Do note that this isn't an OS we officially support - it's going to be best-effort.

Great, thanks!

Thanks for your patience, and apologies in the delay in getting this merged. Internal testing shows that the CI errors here are unrelated.


Unable to set correct AWS LB Protocoll

Hi,
I'm trying to set up an AWS Loadbalancer, but the protocolls for jnlp and ssh are set to HTTP instead of TCP. And I don't know how to change them.
service:
  type: LoadBalancer
  annotations:
    service.beta.kubernetes.io/aws-load-balancer-ssl-cert: arn:aws:acm:eu-central-1:xxx
    service.beta.kubernetes.io/aws-load-balancer-backend-protocol: http
    service.beta.kubernetes.io/aws-load-balancer-ssl-ports: "443"
  httpPort: 443

Thanks for help!

this is how I configured mine...
service:
  type: LoadBalancer
  httpPort: 443
  externalTrafficPolicy: Local
  annotations:
    service.beta.kubernetes.io/aws-load-balancer-type: "elb"
    service.beta.kubernetes.io/aws-load-balancer-backend-protocol: tcp
    service.beta.kubernetes.io/aws-load-balancer-connection-idle-timeout: "4000"
    service.beta.kubernetes.io/aws-load-balancer-internal: <IP_ADDRESS>/0
    service.beta.kubernetes.io/aws-load-balancer-healthcheck-interval: "60"
    service.beta.kubernetes.io/aws-load-balancer-healthcheck-timeout: "10"
    service.beta.kubernetes.io/aws-load-balancer-ssl-ports: "http"
    service.beta.kubernetes.io/aws-load-balancer-ssl-cert: "arn:aws:acm:<YOUR Certiticate ARN>"

As a concept, if the agents are running with k8s plugin within the same cluster, then you don't really need a load balancer, only ingress for the http[s] UI/API


[DOC] Add comments/basic docstrings to testing functions

Describe the issue linked to the documentation
pydocstyle is deprecated, the recommended replacement is ruff.
A lot of testing functions are currently missing the new pre-commit requirements introduced by this change.
Suggest a potential alternative/fix
Add comments describing the test functions. These can be basic and do not have to all be done in the same PR.
See #1055 for current PR swapping the pre-commit hook and the failing pre-commit action on the PR (this at the time of posting) for functions which require fixing.

@MatthewMiddlehurst
I ran ruff and found some inconsistencies with the  PEP 257 docstring conventions, should i raise a PR while addressing the same?

That would be great. I already have done some fixes in #1055, I will put them in as a separate PR right now so there is no overlap.

sure thing!


allow sending directly to a channel or content address

PR Checklist

Please check all that apply to this PR using "x":

[X] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: #5985
What is the current behavior?
Just wallet address:

What is the new behavior?
Wallet address + channel or content handle:

Great feature!
Check out the tip modal and maybe use two tabs: [ address ] [ lbry:// ] for the different modes,
Then it would be good to show a preview of the selected if you're in url mode as shown here:
https://odysee.com/$/repost?to=jsojsjjjjjj
Check out how this was done in https://github.com/lbryio/lbry-desktop/blob/master/ui/page/repost/view.jsx
If you're feeling awesome, make pull down autocomplete suggestions :)

@jessopb thanks for the feedback! something like this?

Great job - I think it looks a bit better with ClaimPreview type="small" in this case - it hides the tags and such. Will take a look at actual code today.

Nice.

it doesn't currently work yet - send stays grayed out.
It doesn't clear when switching between modes.
Might have to change the preview onClick so it doesn't navigate.
We are trying to move tab buttons to the card body rather than the title area (I know the tip modal isn't that way yet).
i found the input a bit buggy, either refusing to let me edit, etc
no error info is being shown during entry
feels weird to fill in part of the field, and click send, so the field is more of a search field than the actual entry.
-- maybe put a search field on top, the preview in the middle, and the amount and actual recipient url at the bottom?

check it out now

I think I would use claim preview type small in the confirm too.
And the placeholder text is wrong.
Anything else you think has bad visual hierarchy or spacing, but looks pretty good.

@jessopb What about this?

Anything else you think has bad visual hierarchy or spacing, but looks pretty good.

I didn't understand this part


BlockSniper not compatible with latest build of pmmp

Description

BlockSniper does not work on PocketMine-MP build #1537 on poggit. Throws error when tap/hold with golden carrot.
Versions

PocketMine version: 1.6.2dev build #1537
BlockSniper version: build #231
Game version: W10 (PE untested)

Crashdump and/or console errors

[18:36:20] [Server thread/CRITICAL]: ErrorException: "Undefined index: ray" (EXCEPTION) in "BlockSniper.phar/src/BlockHorizons/BlockSniper/sessions/SessionManager" at line 63
[18:36:40] [Server thread/CRITICAL]: Could not execute task BlockHorizons\BlockSniper\tasks\UndoDiminishTask: Undefined index: ray
[18:36:40] [Server thread/CRITICAL]: ErrorException: "Undefined index: ray" (EXCEPTION) in "BlockSniper.phar/src/BlockHorizons/BlockSniper/sessions/SessionManager" at line 63
[18:37:00] [Server thread/CRITICAL]: Could not execute task BlockHorizons\BlockSniper\tasks\UndoDiminishTask: Undefined index: ray
[18:37:00] [Server thread/CRITICAL]: ErrorException: "Undefined index: ray" (EXCEPTION) in "BlockSniper.phar/src/BlockHorizons/BlockSniper/sessions/SessionManager" at line 63
[18:37:20] [Server thread/CRITICAL]: Could not execute task BlockHorizons\BlockSniper\tasks\UndoDiminishTask: Undefined index: ray
[18:37:20] [Server thread/CRITICAL]: ErrorException: "Undefined index: ray" (EXCEPTION) in "BlockSniper.phar/src/BlockHorizons/BlockSniper/sessions/SessionManager" at line 63
[18:37:40] [Server thread/CRITICAL]: Could not execute task BlockHorizons\BlockSniper\tasks\UndoDiminishTask: Undefined index: ray
[18:37:40] [Server thread/CRITICAL]: ErrorException: "Undefined index: ray" (EXCEPTION) in "BlockSniper.phar/src/BlockHorizons/BlockSniper/sessions/SessionManager" at line 63

And none of the commands work.
An unexpected error occurred while performing this command

Yeah I'm aware of it. Currently there's no very stable version of BlockSniper. There's an indev branch, but it's aimed to work with 1.2, rather than 1.1.

Please try the latest version, or for ALPHA10 the 2.3.0 branch.


:f[ile][!] prints nothing when 'shortmess' contains 'F'

I am assuming that this is a bug as CTRL-G still works the same way (prints the file information) when 'shortmess' contains F, the documentation suggests that the message is only suppressed when going to edit another file, and :file! should never have it's output affected by 'shortmess'
To reproduce with vim -Nu NONE
set shortmess=F

" All of
file | file! | f | f!
"print no output

" CTRL-G is not affected, the following will print information about the current file:
execute "normal! \<c-g>"

Note: :f can still change buffer names, only output appears to be surpressed
Tested on Vim version 8.1 patches 1-22, huge version with gtk3 gui (OS is Arch Linux)

This can be fixed with the following patch:
diff --git a/src/ex_cmds.c b/src/ex_cmds.c
index 53334342c..3f7cb7a60 100644
--- a/src/ex_cmds.c
+++ b/src/ex_cmds.c
@@ -3095,9 +3095,7 @@ ex_file(exarg_T *eap)
        if (rename_buffer(eap->arg) == FAIL)
            return;
     }
-    /* print full file name if :cd used */
-    if (!shortmess(SHM_FILEINFO))
-       fileinfo(FALSE, FALSE, eap->forceit);
+    fileinfo(FALSE, FALSE, eap->forceit);
     redraw_tabline = TRUE;
 }

diff --git a/src/testdir/test_options.vim b/src/testdir/test_options.vim
index 167cb1ce0..e4e64be2f 100644
--- a/src/testdir/test_options.vim
+++ b/src/testdir/test_options.vim
@@ -400,3 +400,12 @@ func Test_copy_winopt()
     call assert_equal(4,&numberwidth)
     bw!
 endfunc
+
+function! Test_shortmess()
+  " This used to cause Vim to access uninitialized memory.
+  new
+  set shortmess=F
+  call assert_match("\[No file\]", execute('file'))
+  bw
+  set shortmess&
+endfunction

Christian wrote:

This can be fixed with the following patch:
diff --git a/src/ex_cmds.c b/src/ex_cmds.c
index 53334342c..3f7cb7a60 100644
--- a/src/ex_cmds.c
+++ b/src/ex_cmds.c
@@ -3095,9 +3095,7 @@ ex_file(exarg_T *eap)
        if (rename_buffer(eap->arg) == FAIL)
            return;
     }
-    /* print full file name if :cd used */
-    if (!shortmess(SHM_FILEINFO))
-       fileinfo(FALSE, FALSE, eap->forceit);
+    fileinfo(FALSE, FALSE, eap->forceit);
     redraw_tabline = TRUE;
 }

diff --git a/src/testdir/test_options.vim b/src/testdir/test_options.vim
index 167cb1ce0..e4e64be2f 100644
--- a/src/testdir/test_options.vim
+++ b/src/testdir/test_options.vim
@@ -400,3 +400,12 @@ func Test_copy_winopt()
     call assert_equal(4,&numberwidth)
     bw!
 endfunc
+
+function! Test_shortmess()
+  " This used to cause Vim to access uninitialized memory.
+  new
+  set shortmess=F
+  call assert_match("\[No file\]", execute('file'))
+  bw
+  set shortmess&
+endfunction

Hmm, not quite right.  The 'F' flag should apply when changing the file
name, then there is no need to display it.
Looks like you copy-pasted that test :-).  I'll add some more.
--
Contrary to popular belief, Unix is user friendly.
It just happens to be selective about who it makes friends with.
-- Dave Parnas
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Add Instance level privileges/tokens

Proposal:
I'd love to see instance-level permission management.
This is what I mean by Instance-level
- Instance
  - Organizations
    - Buckets

As of now, It's not possible to access InfluxDB with a ReadOnly/WriteOnly token that is valid for multiple organizations, as tokens
exception made for the "operator token", the equivalent of sysadmin, which should not be used for applications.
Current behavior:
There is no way to create a token that allows access to more than one Organization
Desired behavior:
Create and manage tokens that allow access to more than one Organization
Alternatives considered:
I've tried with the v1 authorization (for backward compatibility), but the created user is still organization scoped. There is no way around it.
Use case:
My personal use case is to have a single token have a token with write-only permission on multiple Organizations, I have a centralized system, in which all the data pass through a single Telegraf gateway.
Using InflxDB1 I've got no problems, as I just need a user with Write permission on all databases... by translating this into InfluxDB2, the best fit is to turn a DB into an Organization, but I can't have a token to access them all... which means the routing (made by the telegraf gateway) becomes hell.
The whole idea started from a telegraf feature request https://github.com/influxdata/telegraf/issues/10778, you may find some more details about my use case there.

@Trovalo usually databases in V1 would convert into buckets, not organizations. Why do you want to have the data in separate organizations?

I'm aware that the "default" migration path should turn DB+RP into a bucket, in my case, the "Bucket per customer" and "Organization per customer" designs  are both valid, the latest is cleaner but it suffers due to the auth token limitation.
As a reference, here is my current structure:

and this is the result of the "default" conversion (bucket per customer)

here is the other design (Org per customer):

Organization per customer design
The org per customer design makes the query parametrization easier, as only the high-level connection (Organization) needs to change, and all the underlying structure is constant.
On the practical side a query defined as SELECT ___ FROM standard.cpu or SELECT ___ FROM long.cpu will work on any Organization, only the connection string needs to be changed. (I know that's InfluxQL and not flux, it's just to give the idea)
bucket per customer design
here the parametrization is more complex... as the bucket name is not fixed, therefore just changing the connection string will not suffice
A query defined as SELECT ___ FROM customer1/standard.cpu or SELECT ___ FROM customer1/long.cpu won't work "for another customer" as the query code itself needs to be changed. Everything might work smoothly as long as you have only one bucket per customer because in that case, you can avoid it entirely and the query will use the default one, but if you have more than one bucket the name must change dynamically.
I'm using Grafana as the frontend tool, and it is not easy to make the bucket dynamic out of the box, the v2 connection is defined by: Org, Token, Default bucket.

@lesam, by coding everything is possible, but not every tool allows you to do that, for example, if you write data via Telegraf the token is static, therefore you can't implement a get the right token logic, not nicely at least, it's doable by duplicating the output and filtering it properly, which leads to continuous maintenance/edits.
For Telegraf the logic must be implemented in the config files, but the point is that the mapping Org-Token has to be managed somewhere, be it a table on a DB, or in the code itself, while I think this should not be the case, I expect this to be solved by the security part of the DBMS itself.
I'd normally expect to be able to create a security object (user/token) that has whatever permission is needed by the application, be it sysadmin ("operator token"), read-only on all, or just on a subset of objects (orgs, buckets) or any other possible combination.
Instance-level permission, or more generally permission management on the main system levels (Instance,DB,Objects for a SQL DB or Instance,DB,container for NoSQL DBs) is something I've always found in all DBMS and I'd like to see it also here as I think it will allow for higher flexibility.

@Trovalo if you write via telegraf the org would be static as well - I don't believe telegraf allows configuring the org separate from the token, or configuring the org dynamically based on incoming data.

@Trovalo if you write via telegraf the org would be static as well - I don't believe telegraf allows configuring the org separate from the token, or configuring the org dynamically based on incoming data.

I know that perfectly, that's exactly where my journey began... I saw there was https://github.com/influxdata/telegraf/issues/10778, proposed it and learned that it's pointless as there is no way to authenticate with permission on multiple organizations...
So here I am... proposing a feature to expand permission/token management.

@samhld FYI

I suppose that @Trovalo 's point can be summarized like this:
A multi-tenant application with several Grafana dashboards was totally possible in InfluxDB v1 using a separate database for each tenant, without having to change the InfluxQL queries and using Telegraf to route the data to the correct database.
With v2, in the "one org for all" model, this is possible only with the compatibility endpoints (setting up mappings for database + RP = bucket), because in the v2 endpoints every dashboard would have to be changed to point to the correct buckets (it has to be static in flux).
The "one tenant - one org" model would make it even harder, because the current implementation of the write APIs and the security model (no user can access multiple orgs) make it impossible using a Telegraf gateway.
Changing the security model to allow access to multiple orgs would solve the problem in v2, both for "one org for all" and "one tenant - one org" models, without the need for compatibility endpoints.

Any update/consideration on this one?
I know having instance level permission management might not be that easy as the system has been structured differently (with organization scoped security), but I'd like to know if this point has been considered/rejected/whatever
Thanks

@Trovalo we are considering ways to make this experience better in the coming months. I don't have a good answer for you yet but we will link to this issue on any work that starts!


object: use bufpool for obs and webdav

Signed-off-by: shenwenjie<EMAIL_ADDRESS>

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