too_many_write_operations

Hi, I am trying to upload almost 7K+ images to dropbox.
I have 7K files written on disk, and then I start session against each file and save it in a file such as
 "AAAAAABvLzFovHr07N4kIw 748431 /Construction2/trave-tvjeu/5939/2020-05-15T04:59:16+01:00.jpg",
 "AAAAAABvLzIhJHULASATsw 739442 /Construction2/trave-tvjeu/5939/2020-05-15T04:58:01+01:00.jpg",
 "AAAAAABvLzMWMtkKBARH7w 757474 /Construction2/trave-tvjeu/5939/2020-05-15T05:00:01+01:00.jpg",

when all the sessions have been started, you can say almost 7K sessions have been started,
then I divide all entries in file into 1000 parts and commit 1000, 7 times in a row
First 1000 session then next 1000 and next so on..

First I was doing single upload one at a time but I got too many write operations, now I am getting this error on upload session as well.
do each session commit need some time? or a break between?

Hello, could you provide a little more information about your case? What version of the SDK are you using, what version of JS are you using? What endpoint are you using to upload these images?

Also, apologies for the delay! It looks like we weren't notified of this post for some reason.
If you haven't already, I recommend reading the Performance Guide, as it has useful information on this:
https://www.dropbox.com/lp/developers/reference/dbx-performance-guide
Anyway, I'll close this out since this is old, but please feel free to re-open it if you still need help with this of course.


Webtorrent stuck at fetching metadata from 0 peers

Have been testing webtorrent for about a week now, and it suddenly stopped working correctly for me. After launching webtorrent from the commandline as always, it is stuck displaying ‘Fetching torrent metadata from 0 peers’.
Using a torrent file works and starts the download immediately. Weird thing is that I installed peerflix and that’s also stuck at fetching metadata. This gives me the impression that something on my system must be blocking something about the magnet links. Anybody got any idea on how to even start debugging this?
When I use the DEBUG=* prefix everything seems to run fine, until it just stops at a message something like this:
bittorrent-dht [956b745] got unexpected message from <IP_ADDRESS>:6881 {"r":{"id":{"type":"Buffer","data":[222,60,114,94,121, etc...

Ok seems like this is fixed. Was testing a magnet link that had a capitalised hash:
72F242DB89E763B6CE390F25D576195C2169B149
When I changed this to lowercase everything seems to work fine!


Can't satisfy CONNECT request - operation was canceled

After 2 hours using the proxy it stops working with these errors:

Would it be possible to automatically reconnect if this is happening?

Hello!
I just checked and I see no issue after 2 hours running hola-proxy. Also I see server specified in logs on your screenshot right now responds properly.
Two things could have happened:

Chosen server temporarily went down for maintenance. However, I checked it right away when you created an issue and it seems it was not the case.
Server is getting blocked by filters in your country after some detection period.

First case does not happen too often. But if second one takes place, probably it will be more reasonable to consider some other censorship circumvention solution, this one is always on the brim of failure and countermeasures won't be very efficient.
Detection of such situations from the code is kind of possible, but

Observing network errors passively is not very reliable, because there may be plenty of other reasons for connection timeout.
Active checks of some well-known URL could work, but it makes this solution have very distinct behavior from other hola clients and may be prone to detection and bans.

We could rotate server endpoint proactively, just like we do with credentials, but it will make exit IP rotate frequently as well, which is not very desirable behavior.
For now I suggest just periodic restarts. Let's observe this issue a little bit longer and see if there any way around it. Please let me know how often such disruptions happen, what is your ISP and if restarts always help.


[Security] device-virtual: 502 Bad Gateway

🐞 Bug Report
Affected Services

 The issue is located in:
device-virtual
Is this a regression?

 -
Description and Minimal Reproduction

run nightly-build security enabled docker-compose
https://github.com/edgexfoundry/developer-scripts/blob/master/releases/nightly-build/compose-files/docker-compose-nexus-mongo.yml
get TOKEN

OT=$(docker-compose -f docker-compose-nexus-mongo.yml run --rm --entrypoint /edgex/security-proxy-setup edgex-proxy --init=false --useradd=jerry --group=admin | grep '^the access token for')
export TOKEN=$( echo $OT | sed 's/.*: \([^.]*\.[^.]*\.[^.]*\).*/\1/')

GET virtual device service

curl -ki -H"Authorization: Bearer $TOKEN" https://localhost:8443/virtualdevice/api/v1/ping

🔥 Exception or Error

HTTP/2 502
date: Tue, 14 Apr 2020 08:29:12 GMT
content-type: text/plain; charset=utf-8
server: kong/2.0.1
content-length: 58
x-kong-upstream-latency: 2
x-kong-proxy-latency: 16
via: kong/2.0.1

An invalid response was received from the upstream server

The log of kong:
2020/04/14 08:29:12 [error] 29#0: *158801 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "GET /virtualdevice/api/v1/ping HTTP/2.0", upstream: "http://<IP_ADDRESS>:49990/api/v1/ping", host: "localhost:8443"

🌍  Your Environment
Deployment Environment:
EdgeX Version:
Master
Anything else relevant?
similar to https://github.com/edgexfoundry/blackbox-testing/issues/424

This issue is fixed. Closed this issue.


Fix bug with chat api

dup of https://github.com/keybase/client/pull/11530

closed since CI is working


pexpect is not support on windows machine

uvicorn opendevin.server.listen:app --port 3000
C:\Users\techs\miniconda3\envs\od\lib\site-packages\pydantic_internal_fields.py:151: UserWarning: Field "model_list" has conflict with protected namespace "model_".
You may be able to resolve this warning by setting model_config['protected_namespaces'] = ().
warnings.warn(
C:\Users\techs\miniconda3\envs\od\lib\site-packages\pydantic_internal_fields.py:151: UserWarning: Field "model_name" has conflict with protected namespace "model_".
You may be able to resolve this warning by setting model_config['protected_namespaces'] = ().
warnings.warn(
C:\Users\techs\miniconda3\envs\od\lib\site-packages\pydantic_internal_fields.py:151: UserWarning: Field "model_group_alias" has conflict with protected namespace "model_".
You may be able to resolve this warning by setting model_config['protected_namespaces'] = ().
warnings.warn(
C:\Users\techs\miniconda3\envs\od\lib\site-packages\pydantic_internal_fields.py:151: UserWarning: Field "model_info" has conflict with protected namespace "model_".
You may be able to resolve this warning by setting model_config['protected_namespaces'] = ().
warnings.warn(
C:\Users\techs\miniconda3\envs\od\lib\site-packages\pydantic_internal_fields.py:151: UserWarning: Field "model_id" has conflict with protected namespace "model_".
You may be able to resolve this warning by setting model_config['protected_namespaces'] = ().
warnings.warn(
ERROR:root:  File "C:\Users\techs\miniconda3\envs\od\lib\runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\techs\miniconda3\envs\od\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "C:\Users\techs\miniconda3\envs\od\Scripts\uvicorn.exe_main.py", line 7, in
sys.exit(main())
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\click\core.py", line 1157, in call
return self.main(*args, **kwargs)
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\click\core.py", line 1078, in main
rv = self.invoke(ctx)
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\click\core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\click\core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\uvicorn\main.py", line 409, in main
run(
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\uvicorn\main.py", line 575, in run
server.run()
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\uvicorn\server.py", line 65, in run
return asyncio.run(self.serve(sockets=sockets))
File "C:\Users\techs\miniconda3\envs\od\lib\asyncio\runners.py", line 44, in run
return loop.run_until_complete(main)
File "C:\Users\techs\miniconda3\envs\od\lib\asyncio\base_events.py", line 649, in run_until_complete
return future.result()
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\uvicorn\server.py", line 69, in serve
await self._serve(sockets)
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\uvicorn\server.py", line 76, in serve
config.load()
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\uvicorn\config.py", line 433, in load
self.loaded_app = import_from_string(self.app)
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\uvicorn\importer.py", line 19, in import_from_string
module = importlib.import_module(module_str)
File "C:\Users\techs\miniconda3\envs\od\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in find_and_load_unlocked
File "", line 688, in load_unlocked
File "", line 883, in exec_module
File "", line 241, in call_with_frames_removed
File "C:\devin\git\OpenDevin\opendevin\server\listen.py", line 12, in
import agenthub  # noqa F401 (we import this to get the agents registered)
File "C:\devin\git\OpenDevin\agenthub_init.py", line 7, in
from . import planner_agent # noqa: E402
File "C:\devin\git\OpenDevin\agenthub\planner_agent_init.py", line 2, in
from .agent import PlannerAgent
File "C:\devin\git\OpenDevin\agenthub\planner_agent\agent.py", line 2, in
from .prompt import get_prompt, parse_response
File "C:\devin\git\OpenDevin\agenthub\planner_agent\prompt.py", line 4, in
from opendevin.controller.agent_controller import print_with_color
File "C:\devin\git\OpenDevin\opendevin\controller_init.py", line 1, in
from .agent_controller import AgentController
File "C:\devin\git\OpenDevin\opendevin\controller\agent_controller.py", line 22, in
from .command_manager import CommandManager
File "C:\devin\git\OpenDevin\opendevin\controller\command_manager.py", line 3, in
from opendevin.sandbox.sandbox import DockerInteractive
File "C:\devin\git\OpenDevin\opendevin\sandbox\sandbox.py", line 7, in
from pexpect import pxssh
File "C:\Users\techs\miniconda3\envs\od\lib\site-packages\pexpect\pxssh.py", line 23, in
from pexpect import ExceptionPexpect, TIMEOUT, EOF, spawn
ERROR:root:<class 'ImportError'>: cannot import name 'spawn' from 'pexpect' (C:\Users\techs\miniconda3\envs\od\lib\site-packages\pexpect_init_.py)
(od) C:\devin\git\OpenDevin>pip install pexpect
Requirement already satisfied: pexpect in c:\users\techs\miniconda3\envs\od\lib\site-packages (4.9.0)
Requirement already satisfied: ptyprocess>=0.5 in c:\users\techs\miniconda3\envs\od\lib\site-packages (from pexpect) (0.7.0)
(od) C:\devin\git\OpenDevin>

For now, we don't support Windows, so you can run the project on WSL. If you face any issues let me know

Thanks for reporting the issue! I think wexpect could be a potential solution for this, but there's probably no easy available for now. PRs are welcomed if you have any ideas to workaround this!

Should be fixed with the new docker installation method!

Should be fixed with the new docker installation method!

I did try the new docker installation method and still got the same error:
10:05:47 - opendevin:INFO: ssh_box.py:161 - Connecting to<EMAIL_ADDRESS>via ssh. If you encounter any issues, you can try ssh -v -p 38873<EMAIL_ADDRESS>the password '558203f6-353e-45d1-88d4-521a788e9db1' and report the issue on GitHub. 10:05:47 - opendevin:ERROR: agent.py:123 - Error creating controller: Could not establish connection to host Traceback (most recent call last):   File "/app/opendevin/server/agent/agent.py", line 114, in create_controller     self.controller = AgentController(                       ^^^^^^^^^^^^^^^^   File "/app/opendevin/controller/agent_controller.py", line 47, in __init__     self.action_manager = ActionManager(self.id, container_image)                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/app/opendevin/controller/action_manager.py", line 36, in __init__     self.shell = DockerSSHBox(                  ^^^^^^^^^^^^^   File "/app/opendevin/sandbox/ssh_box.py", line 90, in __init__     self.start_ssh_session()   File "/app/opendevin/sandbox/ssh_box.py", line 164, in start_ssh_session     self.ssh.login(hostname, username, self._ssh_password,   File "/app/.venv/lib/python3.12/site-packages/pexpect/pxssh.py", line 424, in login     raise ExceptionPxssh('Could not establish connection to host') pexpect.pxssh.ExceptionPxssh: Could not establish connection to host


vmTools: set msize to 128KiB

500KiB was causing issues https://github.com/NixOS/nixpkgs/pull/142577

Motivation for this change
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

currently taking times..

This still results in the same issue with the hibernate test. The memorySize still has to be increased.

hibernate test fails with virtualisation.memorySize = 1024;
but with
virtualisation.memorySize = 2048;
and adding
documentation.enable = false;
documentation.nixos.enable = false;

(i noticed that the manual was being built when i changed the msize)
it takes
2:26,41(135s)  withdefault9PMsizeBytes = 8 * 1024;
2:08,42(124s) with default9PMsizeBytes = 128 * 1024;

also, might be the time to change the default value of virtualisation.memorySize which has been 384M since 2009

Yep, 1024M seems like a more reasonable size today. A quick search for some different sizes revealed 37 instances of memorySize being changed to 1024.

rg --pcre2 'memorySize' | rg --pcre2 '\d+(\d+)' --only-matching | sort | uniq -c
43 1024
4 128
1 1300
3 1536
3 2000
2 2046
10 2047
15 2048
5 256
1 3000
1 386
5 4096
2 500
10 512
1 5120
1 64
1 686
8 768

@GrahamcOfBorg test hibernate

i'll make a separate pr bumping the default size and removing the explicit 1024 and less sizes

i did a eval locally and ran the test too.

https://github.com/NixOS/nixpkgs/pull/146804

Closely related regression: https://github.com/NixOS/nixpkgs/pull/146810

Closely related regression: #146810

https://github.com/NixOS/nixpkgs/pull/146804#issuecomment-974715358

another lowering https://github.com/NixOS/nixpkgs/pull/146816 hopefully this will be enough


Move filer-dialogs into filerjs org, clean up deps

We have a great start at reusable dialogs for Filer in https://github.com/ScottDowne/filer-dialogs thanks to @ScottDowne.  However, it should be moved into the filerjs org, and we should blow away the dependency on bower (I've tried using these in various scenarios, and that dep makes it a pain).

Agreed.


releng: Bump promoter to v3.4.10

What type of PR is this?
/kind feature
What this PR does / why we need it:
Bump promoter to v3.4.10. Based on #676.
Which issue(s) this PR fixes:
xref #682
Does this PR introduce a user-facing change?
NONE

/assign @puerco
cc @kubernetes-sigs/release-engineering

/lgtm


[BUG]

Describe the bug.
InvalidArgument was removed (very recently) in discord.py v2.0

List the steps.

I updated the version of discord.py

What you expected.
For there to be no issues.
What you saw.

What version of the library did you use?
release
Version specification
No response
Code of Conduct

[X] I agree to follow the contribution requirements.

interactions.py v3 and below are no longer supported. Furthermore, those versions only worked with discord.py v1.7.X.


sqldb: add blinded path tables

This commit adds two new tables: blinded_paths and blinded_path_hops.
A single invoice can have multiple blinded_path entries and a single
blinded_path can have multiple blinded_paths.
The commit also adds various queries for the two new tables.

note to self: check that sqlite/postgres replacements are working. blob/byte


add ability to style linke as button in tool-bar

it seems good idea - you have buttons in toolbars - but technically it's just link ie.: to form to create new object

Does the <button> class not work for you? https://missing.style/docs/utils/#masquerades

Yes, it does!
I haven't notice that docs, sorry for taking your time!


Implement User Account Recovery V2 API

Is your suggestion related to an experience ? Please describe.
We have encountered following issues with the current recovery V1 API [1] we have.

When the user submits an OTP in the recovery flow, the OTP itself is used to correlate the user, which is not a correct behaviour.
The resend code can be used and refreshed indefinitely to keep resending OTPs to the user/ flooding the OTP services.

Considering the above, we need to provide a new V2 API based on the following approach,

There will be a unique identifier(UUID) for a particular password recovery attempt which is bound to a particular user.
This identifier will be generated at the recovery flow initiation, and it will not be changed in the rest of the flow.
The generated OTP will be bound to the above unique identifier.
When the password is reset, the API call should include the above UUID, OTP and the new password.
Validation happens for the submitted OTP against the recovery context data retrieved from the UUID in the request.
To mitigate possible brute force attacks, the UUID will be invalidated when a certain number of failed OTP validations is exceeded.

[1] https://is.docs.wso2.com/en/latest/guides/identity-lifecycles/extend-username-and-password-recovery/#with-internal-notification-management

Effort estimation for the backend implementation of the proposed solution: https://docs.google.com/spreadsheets/d/1DRES_pMEXXDgdG4T2hOodqi7db-Y6opAXlA0y95Kp14/edit#gid=0

We have decided to disable the current recovery service along with this improvement. A config is introduced to enable/disable the service.
Related PRs:

https://github.com/wso2/carbon-identity-framework/pull/4878
https://github.com/wso2-extensions/identity-governance/pull/750
https://github.com/wso2/identity-api-user/pull/178

The following config should be added to the deployment.toml file to enable the service.
[identity_mgt.recovery]
enable_v1_api = true

Recovery V2 API is implemented with:

https://github.com/wso2/carbon-identity-framework/pull/4883
https://github.com/wso2-extensions/identity-governance/pull/751
https://github.com/wso2/identity-api-user/pull/179

DB migration is tracked with https://github.com/wso2/product-is/issues/16576

There's an issue in Recovery V2 /confirm API when the channel is EXTERNAL. It will be fixed with https://github.com/wso2/product-is/issues/16884


Support for Next.js draft mode

The new App Router feature of Next.js 13.4 doesn't work with the old Preview Mode API. Instead Next.js apps that use the App directory need to use the new Draft Mode feature.
Here's a blog post mentioning the Draft Mode feature (under "other improvements") included with Next.js 13.4: https://nextjs.org/blog/next-13-4#other-improvements
And here is Next.js' Draft Mode documentation
This request is related to #442, but that issue specifically mentions the Server Components part of the App Router feature and Draft Mode isn't related to Server Components.

Draft mode is also recommended for pages too https://nextjs.org/docs/pages/building-your-application/configuring/draft-mode but the implementations are slightly different which would need to be taken into account.
Also there doesn't seem to be a replacement for setPreviewData so revisions will need a rethink.

@dottodot Thanks so much for the link!
Somehow I had missed that Draft Mode was recommended for pages router. That simplifies what next-drupal needs to support when a website is only partially migrated from pages router to app router.

Fixed with #670


List of Recipes

List of Recipes open a separate UI when button is clicked and that's making a annoying bug

For the reference: the Recipe GUI overlays with the List GUI in logic programmer


  [Composer\Repository\InvalidRepositoryException]
  No valid composer.json was found in any branch or tag of https://github.com/WondrousLLC/fractal-twig-drupal-adapter, could not load a pack
  age from it.

@cristinallamas thanks for reporting. But this is not a composer package. Please use npm or yarn to install this into your node_module folder as stated in the readme.
$ npm install --save @wondrousllc/fractal-twig-drupal-adapter


Problems with stubs

phpstan.neon
parameters:
    paths:
        - freemius-stubs.php
    level: 5
    tipsOfTheDay: false
    ignoreErrors:
        - '#but return statement is missing\.$#'
        - '#has an unused parameter#'
        - '#^(Property|Static property|Method|Static method) \S+ is unused\.$#'
        - '#is never read, only written\.$#'
        - '#has invalid (return )?type (WP_Error|WP_Customize_Manager)\.#'

@mralaminahamed There are 66 problems.
I recommend adding constants manually in a separate file.
🚀

@szepeviktor, thank you for running PHPStan and sharing the analysis results. I appreciate you taking the time to review the codebase and provide that feedback. Currently, I am working on addressing these issues.
Please let me know if you have any other specific concerns or suggestions as I prioritize improving this part of the codebase based on PHPStan's output.

bootstrapFiles:
    - freemius-constants-stubs.php

Constants are not static stubs. define-s must be executed.
But their values could be generic, like \define('WP_FS__DEBUG_SDK', false);
or \define('FS_SDK__USER_AGENT', 'fs-php-1.0.0');

all issues are fixed. let me know if have any feedback.


add validation timed out message

Explain your changes:
This PR adds logs for validation timed out
Explain how you tested your changes:
*
Checklist:

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me


English translation of manual

It would be great if the package owner could review my translation!

Sorry for the late reply. Thank you very much for your translation. It's fine with me.


Exposing API Calling URL Links in Data Views

Would it be useful to provide a link to an API query URL for views over data generated in the search interface for each dataset?
For example, if I search a particular dataset for a particular term and then select a few filters/facets, if we provided a link to the corresponding API call it would allow users to use openspending.org to build up a query/data view, and then grab a link that they could use to pull the data directly into another application.
This approach makes the API available to users who may be confident in consuming the data, or plugging an API calling URL into a library that can handle that data, but who struggles to make sense of the API documentation (or can't be bothered to read it...)

Closing as it has been a long time, and this is probably a question for the forums anyway: http://discuss.okfn.org/c/openspending


The manual isn't building correctly / updating online

Original issue: https://charm.cs.illinois.edu/redmine/issues/1785

The Charm++ manual and other manuals are not building correctly since 10 days ago according to Jenkins. This means that the online versions are not automatically updating.
http://localhost:8080/job/charm-documentation/

Original date: 2018-01-30 20:36:47

The build error is:
make[1]: Leaving directory '/scratch/jenkins/workspace/charm-documentation/doc/charj'
rm -rf /www/manuals/backup
mkdir -p /www/manuals/backup && \
mv /www/manuals/pdf /www/manuals/html /www/manuals/backup
mkdir: cannot create directory ‘/www’: File exists
Makefile:29: recipe for target 'web' failed
make: *** [web] Error 1
Build step 'Execute shell' marked build as failure
Finished: FAILURE

On January 19th (=10 days ago), the filesystem on the build machine (panache) was somehow changed so that /www is now a symlink to /web/www, which results in the mkdir failure shown above:
mdiener`panache $ ls -l /www
lrwxrwxrwx 1 root root 8 Jan 19 16:21 /www -> /web/www
mdiener`panache $ mkdir -p /www
mkdir: cannot create directory ‘/www’: File exists

Original date: 2018-01-30 21:15:22

Untested patch here: https://charm.cs.illinois.edu/gerrit/#/c/3591/

Original date: 2018-01-31 21:18:10

The issue has been fixed on panache. The above patch was not necessary.

Original date: 2018-01-31 21:37:53

This was caused by /web on panache not being mounted properly for some reason. Running sudo mount /web fixed it.

Original date: 2018-02-01 14:27:30

Marking 'Closed' so redmine will remove it from the 'Open' category


AST: Fix ReplaceOpaqueTypesWithUnderlyingTypes to consider the curren…

…t resilience expansion in non resilient mode
When we decide which access level we can allow for types we need to consider the resilience expansion context. 'internal' visibility cannot be allowed to be looked through in inlineable code as an external module does not have visiblity to such a symbol.
rdar://119725428
(Cherry picked from commit 8d649a252505d3132ffae47985d005cc2e1d070e)

@swift-ci please test

Explanation: Ensure that private types are not leaked across inlinable boundaries.
Scope: Fixes the inlining behaviour of opaque types but maintains source compatibility. It fixes an ABI issue where internal types may be leaked.
Issue:
Risk: The fix impacts inlinability. It may reduce inlinability but would correctly prevent internal types from being leaked.
Testing: Built a large private codebase with the change applied.
Reviewer: @aschwaighofer

CC: @DougGregor

Original PR is https://github.com/apple/swift/pull/70770


Ability to Create Engine From an Existing BigQuery Client

In situations where a user already has a google.cloud.bigquery.Client object, it'd be helpful to be able to create an sqlalchemy engine either using that already existing BigQuery client object or the same credentials used by that client.
I'm poking around and one option would be something like
bq_client: bigquery.Client
engine = sqlalchemy.engine.create_engine(
    'bigquery://project/?credentials_user_info_json=%s' % bq_client._credentials.to_json())

to create a client with the same credentials. This is unfortunately accessing a private _credentials attribute on the client. We might look to adding a public interface for this on the client so callers don't need to access _credentials.
I was also looking at something like
bq_client: bigquery.Client
engine = sqlalchemy.engine.create_engine(
  'bigquery://project', connect_args={'client': bq_client})

using the connect_args kwarg
The problem is that the engine will still try to create the client itself even if it will end up being replaced by the one from connnect_args. That attempt to create the client by the engine may fail (we're assuming the big query client is created by some machinery that's opaque to the user here.) I'm not seeing a clean way to avoid this.
Any other suggestions? I'm more than happy to work on this if it seems like a reasonable thing to support!

What about something like:
engine = sqlalchemy.engine.create_engine(
    # do not let sqlalchemy_bigquery attempt to create a client itself
    'bigquery://project/?create_client=False',
    # instead we supply one of our own
    connect_args={'client': bq_client},
)

?
More than happy to put together a pull request for this!


Package name is incorrect

Believe this statement:
sudo apt-get install curl apt-transport-https lsb-release gpg
Should be
sudo apt-get install curl apt-transport-https lsb-release gnupg
The gpg package name is incorrect.

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

ID: fe1526a9-7f22-504e-484c-0f3edc11b178
Version Independent ID: c5dec108-efcd-aa77-04b5-803b6b894dba
Content: Install the Azure CLI on Linux with apt
Content Source: docs-ref-conceptual/install-azure-cli-apt.md
Product: azure
Technology: azure-cli
GitHub Login: @sptramer
Microsoft Alias: sttramer

@swieder The gpg package is available on the versions of Ubuntu that I tested - which distribution are you using that requires gnupg?

This is a duplicate of #1397 which includes information on the distribution, closing.


ARC-43 Control Scripts PT7: Add Frequency plots for fixed rate publishers

PR Details
Description
This PR adds a script to analyze the fixed rate publishers' frequencies over time from ROS2 tracing (mostly interested ones are arbitrator, plan_delegator, trajectory_executor, pure_pursuit, and trajectory_follower_wrapper, but it does for everything as long as it says timer)
Convert selected nodes’ mean latency callback into frequency.
Also convert the duration over time into frequency plot.

Design document for this task is here: https://leidoscorpus.sharepoint.us/:w:/r/sites/STR/TODevelopmentLibrary/22-256 CDA Design %26 Architecture/Work-In-Progress/Task 7 Architecture Improvement Implementation/CARMA Platform Controls Improvements/Identified Core Performance Metrics And Analysis Design for CARMA Platform Control.docx?d=w40615c2fbfb74d439215cf62c5e5c4ca&csf=1&web=1&e=NXYkMT
This implemented 8. and 9. from the design

Related GitHub Issue
NA

Related Jira Key
https://usdot-carma.atlassian.net/browse/ARC-54

Motivation and Context
Control improvements under CDA DnA needed control performance analysis scripts.

How Has This Been Tested?
To get the callback duration data from ros2_trace for each nodes run:
python3 carma_analyze_callback_durations.py /workspaces/carma/carma_ws_ros2/src/analysis-data
Example output:

Callback,Mean Freq (Hz),Std Dev Freq,Min Freq (Hz),Max Freq (Hz)
lci_strategic_plugin Timer Callback period: 500 ms,1.9946396526341914,0.1178082570454688,0.3846936549782627,2.496797856748741
lci_strategic_plugin Timer Callback period: 100 ms,10.32943098817453,10.853643085545098,0.41715299753629437,359.8416696653072

Types of changes

[ ] Defect fix (non-breaking change that fixes an issue)
[X] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that cause existing functionality to change)

Checklist:

[X] I have added any new packages to the sonar-scanner.properties file
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

Generally looks good. Only comment is that the title of the plots are confusing, maybe is better to rephrase, or add a hyphen or comma so its not misleading.


1640 fix choice item reset in paginated questionnaire

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #1640
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

LGTM


1/ternoa client

Structure / outlayer of new cli commands
NOTE: This is not the final PR - ist just for showing the structure of the new cli calls. Some more clean up is necessary.

@brenzi would you mind to take a look at the folder / file structure  and give some feedback if that's acceptable for you?


Would it be possible to add support for wit.ai? #question

Would it be possible to add support for wit.ai by adding a new dialog that implements the IntentDialog interface? The implementation could be very similar to the LuisDialog and would allow users to build  bots that could use both LUIS and Wit.ai.

Yes. I just haven't had time to write it but totally possible. I'm actually planning to deprecate the IntentDialog class and replace it with just the LuisDialog class. There are a few reasons for this so I'd wait and look at that class for inspiration on building a Wit.ai based dialog.

Moved to UserVoice for tracking and voting.
https://feedback.botframework.com/forums/354222-bot-builder-sdk/suggestions/13494474--nocejs-add-a-wit-ai-dialog-class-similar-to-luis

Is it Possible to integrate wit.ai with Mircrosoft bot framework? Is there any documents for this?

@venkat345 There is a prototype of LUIS implementation for Wit.Ai (C#) here.  It implements the ILuisService for Wit.Ai.  Sample will be added to the repo, asap.  It's available on NuGet here.
Hope it helps.

Here is nice NPM module for Bot Framework for Node.js : https://github.com/sebsylvester/botbuilder-wit

@Stevenic Anything new on this one? Or better, Swedish, Norwegian & Danish support in Luis :)


Location input field are output with a long label.

Describe the Bug with repro steps

Open a flow which contains actions has long placeholder text
In Power Automate, one example is the Excel 'delete a row' action.
Observe placeholder label.

Actual Result:
Location input fields are output by the screen reader with a long label, such as the "Select from the drop-down or specify one of the following: - "me" - "SharePoint Site URL" - "users/someone's UPN" - "groups/group Id" - "sites/SharePoint Site URL:/teams/team name:" (the colons are required)" This means that blind users have to wait for a long output to be able to perceive all the information.
Expected Result:
Additional information should be short and meaningful. If short wording is not possible, it should be moved out to a help icon. This icon should:
be placed in front of the input field,
receive the focus with the tab key,
display the hint when focus is received, and
be output by the screen reader.
What type of Logic App Is this happening in?
Consumption (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos

Browser
Edge
Additional context
Related internal bug: https://msazure.visualstudio.com/One/_workitems/edit/13921602
AB#25622474

This is because the description field being sent for the Location Field is entirely too long, this should be fixed in the manifest, not change how the UI works because of it.
I also agree that we don't necessarily need to have screen readers read the placeholder value of the text box but this seems to be fluentUI's behavior, not ours.


View account in explorer redirects to an inactive URL

Selecting view account in explorer leads to "https://explorer.iota.org/mainnet/address/iotaxxxx" but nothing shows up for me.

When I search from the explorer site for my iota address, the explorer site uses the url "https://explorer.iota.org/mainnet/addr/iotaxxxx" which does show my history.
Might be the the explorer site updated their URL and Tanglepay just needs to update the redirect link.
Using chrome extension in Brave.

Hi Brbubba,
Thanks for the catch
after verifying  that we will fix it ASAP

测试通过


tools: (mostly) enforcing flag alpha order

Risk Level: n/a (tooling only)
Testing: manual
Docs Changes: n/a
Release Notes: n/a

Looks like some formatting issues that python linter is complaining about.

/wait


Workspaces: option to restore opened application windows, instead of launch new instances

Description of the new feature / enhancement
A per-workspace switch that, if enabled, makes the workspace restore opened application windows to the saved layout. The workspace should launch an application instance only if the application is not already open.
Scenario when this would be used?
My workflow involves applications like vscode, browser, word, etc, using FancyZones to arrange them. Usually I need to move them across monitors or into different layouts, but to restore them I have to manually drag them back to FancyZones grids. It would be benefitial if Workspaces can restore the windows into my favorite layout, instead of open new empty windows (like an empty vscode workspace).
Supporting information
No response

@gomico please retest with the new 0.87 PT version and if the issue persists send a /bugreport. Thanks!

@gomico please retest with the new 0.87 PT version and if the issue persists send a /bugreport. Thanks!

The new version works for my workflow. Thanks for implementing the feature.


close non-closed file descriptor will resolve : #38, #43, #150, #164, #221 - shutdown will work properly

This PR introduces changes which resolve these issues (#38, #43, #150, #164, #221).
the file descriptor will stay open at the end of the Listener and Epoll life-time, there for the shutdown will not work properly, and may get the Bad file descriptor in multi threading and also Too many open files.
it just close the listener port and FD and close FD in epoll deconstructor.

Codecov Report

Merging #650 into master will increase coverage by 0.52%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #650      +/-   ##
==========================================
+ Coverage   70.43%   70.96%   +0.52%
==========================================
  Files          49       50       +1
  Lines        6958     7032      +74
==========================================
+ Hits         4901     4990      +89
+ Misses       2057     2042      -15

Impacted Files
Coverage Δ

include/pistache/os.h
100% <ø> (ø)
:arrow_up:

src/common/os.cc
57.59% <100%> (+1.1%)
:arrow_up:

src/server/listener.cc
70.43% <100%> (-0.11%)
:arrow_down:

include/pistache/http_header.h
89.54% <0%> (-0.26%)
:arrow_down:

src/client/client.cc
78.81% <0%> (-0.2%)
:arrow_down:

src/common/stream.cc
91.83% <0%> (-0.06%)
:arrow_down:

include/pistache/tcp.h
80% <0%> (ø)
:arrow_up:

include/pistache/mime.h
95.45% <0%> (ø)
:arrow_up:

include/pistache/router.h
100% <0%> (ø)
:arrow_up:

... and 12 more

Continue to review full report at Codecov.

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

Thanks @mohsenomidi. I think @dennisjenkins75 might be away on holidays right now. But once this is reviewed and merged it looks like a great addition because you've solved multiple problems with a fairly simple patch.


Scope down the usage message when typing docker-machine create -d driver to only list flags relevant to that driver

Right now, docker-machine create -d openstack spits outs 113 lines of usage message, 104 lines of which are flags. Only 18 of these lines are actually relevant to openstack.
It's a pretty minor papercut, but if there's a trivial way to scope it down, that'd be real slick.

@hairyhenderson I agree, we definitely need to change this. I'm unsure what the best UX would be for this.

@sthulb maybe something like:
$ docker-machine create -d openstack
NAME:
   create - Create a machine

USAGE:
   command create [command options] [arguments...]

OPTIONS:
   --openstack-auth-url 												OpenStack authentication URL [$OS_AUTH_URL]
   --openstack-endpoint-type 												OpenStack endpoint type (adminURL, internalURL or publicURL) [$OS_ENDPOINT_TYPE]
   --openstack-flavor-id 												OpenStack flavor id to use for the instance
   --openstack-flavor-name 												OpenStack flavor name to use for the instance
   --openstack-floatingip-pool 												OpenStack floating IP pool to get an IP from to assign to the instance
   --openstack-image-id 												OpenStack image id to use for the instance
   --openstack-image-name 												OpenStack image name to use for the instance
   --openstack-insecure													Disable TLS credential checking.
   --openstack-net-id 													OpenStack network id the machine will be connected on
   --openstack-net-name 												OpenStack network name the machine will be connected on
   --openstack-password 												OpenStack password [$OS_PASSWORD]
   --openstack-region 													OpenStack region name [$OS_REGION_NAME]
   --openstack-sec-groups 												OpenStack comma separated security groups for the machine
   --openstack-ssh-port "22"												OpenStack SSH port
   --openstack-ssh-user "root"												OpenStack SSH user
   --openstack-tenant-id 												OpenStack tenant id [$OS_TENANT_ID]
   --openstack-tenant-name 												OpenStack tenant name [$OS_TENANT_NAME]
   --openstack-username 												OpenStack username [$OS_USERNAME]
   --swarm														Configure Machine with Swarm
   --swarm-master													Configure Machine to be a Swarm master
   --swarm-discovery 													Discovery service to use with Swarm
   --swarm-host "tcp://<IP_ADDRESS>:3376"											ip/socket to listen on for Swarm master
   --swarm-addr 													addr to advertise for Swarm (default: detect and use the machine IP)

Run 'docker-machine create --help' for all options.

FATA[0000] You must specify a machine name

After prototyping a bit, this might be way more trouble than it's worth. If drivers were subcommands (instead of flags) it'd be easier, but since they're flags it turns into a huge mess pretty quickly.

#dibs

+1

Fixed by #901 :tada:


Verify domain w/GitHub

Prove we're legit by verifying our domain w/a GitHub TXT record: instructions

done — thanks @netlify


key_tools for gorc

NOTE: Tracking scope/todos in key_tools/src/lib.rs for now. Planning to make that a checklist in the PR description as we firm up what we're doing. eg: optimistically, key_tools becomes a stand-alone crate we can re-use in other projects.

Closing this PR (good learnings but not destined to be merged)


Clean all at the end of tests

👍

Coverage increased (+0.02%) to 77.29% when pulling 307b178c871ee9b57fb8388b2853b18aa72e262b on sni/better-rule into d3c4b3b947c3a170a7cef67bee22b45b64d514e1 on master.


"cannot read property 'firstChild' of undefined" error?

Hi there -
I was following along with the video tutorial but I couldn't get the space to populate.  Here's my SPGo.json file:
{
    "sourceDirectory": "src",
    "sharePointSiteUrl": "https://xxx.sharepoint.com/teams/xxx",
    "workspaceRoot": "c:\\Users\\alainna\\site",
    "publishWorkspaceOptions": {
        "destinationFolder": "/",
        "globPattern": c:\\Users\\alainna\\site\\src\\**\\*.*",
        "localRoot": "c:\\Users\\alainna\\site\\src"
    },
    "publishingScope": "SaveOnly",
    "authenticationType": "Digest"
}

Here's the output (snipped some "==" so it wouldn't initiate a scroll in the issue report):
[5/19/2020, 8:49:23 AM] =============     ERROR  =================

[5/19/2020, 8:49:23 AM] Cannot read property 'firstChild' of undefined
[5/19/2020, 8:49:23 AM] Error Detail:
[5/19/2020, 8:49:23 AM] ----------------------
[5/19/2020, 8:49:23 AM] {}
[5/19/2020, 8:49:23 AM] ===============

Any thoughts? Thank you!

I suspect this may be because it's a /teams/ and not /sites/.

Hello @alainnaw
A couple notes/questions:

From your config file, it looks like you are configuring SPGo's advanced publishing features to publish files to sharepoint without mapping them to your local folder - for instance if you are building/deploying a React, Angular, or Vue-based solution. Is this the case? If so, there is a syntax errors in your config: You need to add an opening quote " to the "globPattern" propety.
If you are not using advanced publishing, and just want to synchronize files directly, I would recommend using the remoteFolders property instead  Can you try the following changes:

{
    "sourceDirectory": "src",
    "sharePointSiteUrl": "https://xxx.sharepoint.com/teams/xxx",
    "publishingScope": "SaveOnly",
    "authenticationType": "Digest"
    "remoteFolders" : [
        "/SiteAssets/**/*",
    ]
}

Of course, you would replace xxx and SiteAssets with the name of your team, and the folder within the site that you want to replicate. You can find documentation on the Populate Workspace command here.

If you are still having problems, can you describe which files you want to synchronize locally and I'll help you get your config file set up correctly.
Thanks for using SPGo!

Hey Chris - Thanks for the quick reply.  I did as asked, but still no joy.  I'm just a site collection owner that wants to do some customizing to her SharePoint site.  I would ideally like to populate the whole tree so I have extra flexibility (and extra dangerousness ;))
{
    "sourceDirectory": "src",
    "sharePointSiteUrl": "https://xxx.sharepoint.com/teams/xxx",
    "publishingScope": "SaveOnly",
    "authenticationType": "Digest",
    "remoteFolders" : [
        "/SiteAssets/**/*"
    ]
}

Thanks for the help!
Alainna

@alainnaw - Thanks for the update.
I'm going to try to replicate the issue using team sites. Do you know if this site was created via the SP UI, or via the Teams app?
Can you do a quick test with a site in your tenant that is on the /sites managed path just to make sure the issue isn't related to authentication or something else?

Separately - SPGo does support working with full Site Collection hierarchies (trees), so once we get this initial issue resolved, I can help you configure you spgo.json file to pull down all the files you need.

Hey Chris -
I don't have any sites on the /sites path unfortunately.

Hello @alainnaw - I've been trying to test this a number of ways, and I am, so far, unable to reproduce the bug. I've tried creating a team site via teams, as well as team sites via the Site Collection Manager.
A couple question for you:

when you made the changes to your spgo.json file did you get the same error or a different error?
do any other commands work? Are you able to use the Retrieve Folder command or are you able to create a new file via VSCode?
Are you using any sort of custom authentication, e.g. ADFS, or does your site use SPO/Digest?

Thanks for working with me on this. Hopefully we can get this resolved soon.

Hey @chrishasz,
I got the same error with each spgo.json file iteration I tried.
I tried the retrieve folder command and I received the same error.
I don't think we're using any kind of unique authentication.
Are there any debugging flags I can add in to increase verbosity of the output?
Thank you for taking the time to help me out here.  I appreciate it.
-AW

Hello @alainnaw -
Unfortunately VSCode is still preventing me from supporting O365 MFA natively. Are you able to define an app password for SPGo on your site?
If so, here is how to configure SPGo with App passwords:
https://www.chrishasz.com/spgo/authentication/two-factor-authentication
Here is the documentation for configuring App Passwords:
https://docs.microsoft.com/en-us/azure/active-directory/user-help/multi-factor-authentication-end-user-app-passwords#create-new-app-passwords

Hello @alainnaw -
I am in the process of adding "Addin Only" authentication to SPGo. Maybe this will work for you. Are you able to create an Addin registration on your team site(s)?

Hello @alainnaw - SPGo supports Addin-only authentication as of version 1.7.0. You can find documentation on how to configure Addin-only authentication here.


Authentication Failed (sometimes)

When I join a game, sometimes it pops up with the error message below instead of Roblox, and there's a higher chance if I join a server from "Other Servers".

closing this because at least i've found the cause, the fix is proably something on my end though.


savetobinary io module, issue #3

Test script below to address issue#3:
from peakinvestigator.io import save_binary

headers = ['This is a header\n','it should contain meta data\n','There is no limit to number of lines, and is optional.\n']
data_points = [[1234.5,67.89],[1234.56,78.9],[1234.567,89.0]]
with open('Test.bin', 'wb') as file:
    save_binary(headers, data_points,handle=file)

Comments are addressed in latest commit.

One last thing: is the test actually run via Travis? I didn't see a change in the associated script.

Travis tests passed on updated test_binary.py.
https://github.com/Veritomyx/PeakInvestigator-Python-SDK/pull/21/commits/4ee21a9853777a8d1f6f60f89878988ab523561c

But you haven't made any changes so that Travis automatically runs the test_binary.py file: https://github.com/jct197/PeakInvestigator-Python-SDK/blob/binaryiomodule/travis/run_travis_tests.sh

Updated file looks to have passed this morning.
Should I make a new PR for verification?


Posting an empty array in a Form errors with 422

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
# python3 -m pip install fastapi pytest
# python3 -m pytest tests/post.py

from typing import Optional

from fastapi import Form, FastAPI, Query
from fastapi.testclient import TestClient

app = FastAPI()

@app.post("/list")
def read_query(q: list[Optional[str]] = Form()):
    return q

client = TestClient(app)

def test_full_list_valid():
    response = client.post("/list", data={"q": ["hi", "there"]})
    assert response.status_code == 200
    assert response.json() == ["hi", "there"]

def test_empty_list_valid():
    response = client.post("/list", data={"q": []})
    assert response.status_code == 200
    assert response.json() == []

Description
I would expect all tests to pass
Operating System
macOS
Operating System Details
No response
FastAPI Version
0.85.2
Python Version
Python 3.10.9
Additional Context
No response

This only somewhat overlaps with https://github.com/tiangolo/fastapi/issues/3506


Inject GRUB customizations to support CoreOS Ignition

Add support to CoreOS GRUB configuration to support bare-metal Ignition templating support.

ok to test

test this please

I've pushed some changes to this branch per feedback from @yyscamper in the paired PR: https://github.com/RackHD/on-tasks/pull/377
I've tested them and found them to be functioning as intended. Please review & test as necessary prior to merge.
Thanks!

Thanks @tgelter , this is perfect.


Invalid Authorization while trying SSRF attack

while trying the first objective, on step 3, no URL comes back as response, but "{"body": "Invalid Authorization"}" appears

@BarakAgranov, we updated the code for the JWT token because GCP Cloud Functions Gen 1 no longer accepts header keys with underscores. It's working well now. Thanks for bringing this to our attention!


Investigate bulk export

Take a look and see if Superset data export is sufficient or whether we need to build additional tooling to handle medium/large scale data export.

This has already been explored and it's enough for most workloads. Closing it now


Pass in len(values) to get_window_bounds & fix existing flake8/black errors

xref https://github.com/twosigma/pandas/pull/5#discussion_r309916743
len(values) is the only component required for get_window_bounds in VariableWindowIndexer and FixedWindowIndexer and is less complex than having the method accept values directly.
Also address some black and flake8 errors discovered by azure

The current tests I have added don't hit this case. Moreover, this aligns how the current Indexers utilize values i.e. num_values is passed instead of values

great so rebase this and good?

The last of the linting errors are related to files this project isn't touching, and the 3.7 builds seems to be an unrelated failure.
Odd the linting errors are not showing up in pandas/master
Linting code-blocks in .rst documentation
##[error]doc/source/getting_started/basics.rst(765,8): error F821: undefined name 'f'
##[error]doc/source/getting_started/basics.rst(765,10): error F821: undefined name 'g'
##[error]doc/source/getting_started/basics.rst(765,12): error F821: undefined name 'h'
##[error]doc/source/getting_started/basics.rst(771,17): error F821: undefined name 'h'
##[error]doc/source/getting_started/basics.rst(772,17): error F821: undefined name 'g'
##[error]doc/source/getting_started/basics.rst(773,17): error F821: undefined name 'f'
##[error]doc/source/development/contributing_docstring.rst(50,9): error F821: undefined name 'add'
##[error]doc/source/development/contributing_docstring.rst(52,9): error F821: undefined name 'add'
##[error]doc/source/development/contributing_docstring.rst(54,9): error F821: undefined name 'add'
##[error]doc/source/development/contributing_docstring.rst(190,5): error F811: redefinition of unused 'astype' from line 37
##[error]doc/source/development/contributing_docstring.rst(200,5): error F811: redefinition of unused 'astype' from line 44
##[error]doc/source/development/contributing_docstring.rst(210,5): error F811: redefinition of unused 'astype' from line 51
##[error]doc/source/development/contributing_docstring.rst(220,5): error F811: redefinition of unused 'astype' from line 58
##[error]doc/source/development/contributing_docstring.rst(325,5): error F811: redefinition of unused 'Series' from line 86
##[error]doc/source/development/contributing_docstring.rst(563,5): error F811: redefinition of unused 'Series' from line 105
##[error]doc/source/development/contributing_docstring.rst(914,5): error F811: redefinition of unused 'Series' from line 176
##[error]doc/source/development/contributing_docstring.rst(958,9): error F821: undefined name 'Substitution'
##[error]doc/source/development/contributing_docstring.rst(959,9): error F821: undefined name 'Appender'
##[error]doc/source/development/contributing_docstring.rst(965,9): error F821: undefined name 'Substitution'
##[error]doc/source/development/contributing_docstring.rst(966,9): error F821: undefined name 'Appender'
##[error]doc/source/development/contributing_docstring.rst(995,5): error F821: undefined name 'Appender'
##[error]doc/source/development/contributing_docstring.rst(995,14): error F821: undefined name 'template'
##[error]doc/source/development/contributing_docstring.rst(995,25): error F821: undefined name '_shared_doc_kwargs'

@mroeschke and this is after a merge master right?

Correct this is after the merge.

ok, let's just merge for now then and come back and see can make all green at some opoint.


Properly handle error responses with null or not present body

When a HTTP response contain no body or the body field is None, botocore crashes with this error :
'NoneType' object has no attribute 'strip'
Traceback (most recent call last):
[ ...]
  File "/[...]/dist-packages/botocore/parsers.py", line 237, in parse
    if self._is_generic_error_response(response):
  File "/[...]/dist-packages/botocore/parsers.py", line 270, in _is_generic_error_response
    body = response['body'].strip()

The typical trigger for this error is a timeout that generate an internal http 599

Codecov Report

Merging #1568 into develop will decrease coverage by 0.04%.
The diff coverage is 75%.

@@             Coverage Diff             @@
##           develop    #1568      +/-   ##
===========================================
- Coverage    91.95%   91.91%   -0.05%
===========================================
  Files           50       50
  Lines         9035     9037       +2
===========================================
- Hits          8308     8306       -2
- Misses         727      731       +4

Impacted Files
Coverage Δ

botocore/parsers.py
99.56% <75%> (-0.22%)
:arrow_down:

botocore/credentials.py
98.4% <0%> (-0.4%)
:arrow_down:

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 06bef45...f97daf6. Read the comment docs.

https://github.com/boto/botocore/issues/1570
I opened an issue yesterday for this, thank you for the pr! :D

@SeleniumK You're welcome ! I've removed the useless else:)

@theredcat Do you have any more context on when this happens?
What service is returning this?
Does the API being called have a large request or is otherwise complex?
Just trying to understand the underlying cause here a little more.
The PR looks good to me though.

@joguSD I use https://github.com/thumbor/thumbor with an S3 backend and few weeks ago, AWS had trouble on the eu-west-1 region that impacted my thumbor instances. S3 api calls were hiting timeouts and the error in my first message start popping up in my logs.
I have no proxy.

@theredcat After some discussion in the other thread I've tracked this down to the tornado-botocore package. The unfortunate thing is the way that's implemented is via patching out botocore internals to use an async HTTP client and thus has very strict botocore requirements. To pick up this patch you'll need to get the following repos also updated to use newer versions of botocore that include this patch.
See:
https://github.com/thumbor-community/aws/blob/7b40255be6c846c6ed27a2ab037ed74d6c8d2c84/setup.py#L60-L61
https://github.com/nanvel/tornado-botocore/blob/07542ddc032c6fcf49e819e48c543d5a3ad953b2/requirements.txt#L1


Readme updates

Just noting the README needs to be updated with new flags/options, examples such as how to alias filetypes, and that the statusline function needs to be updated.

I'm thinking of taking the table of options away from the README, and linking to the Vim help file for that. Then we can put the important common points of configuration into an FAQ, and move the Installation section higher up.
It will probably make more sense in a pull request. I'll get to it sooner or later.


WorldGuard plugin breaks claiming land

When i add the worldguard plugin to my server i cannot claim land. I can enter the bell menu for claiming but i cannot confirm it. is this expected behavior or an actual bug?

Never tested with WorldGuard, so I'll have to see what kind of conflicts are happening here.

I am not experiencing any WorldGuard conflicts. Can you show me any errors in the console so I can see what's going on? Make sure you're running the latest release version too.

Ill try this in a bit, might have been an version issue indeed. Ill get back to you when ive tried


Apply patches Rendering of accent symbols and Loading of predefined formulas

Thanks to Marc Palmans.
Rendering of accent symbols

Accent symbols are rendered separately instead of above other characters.
The following patch appears to fix this issue but someone with more knowledge of the source code and latex should check it.

Loading of predefined formulas

I fixed some issues regarding the loading of predefinedformulas.xml Everything should work except \left and \right, but I think we should use TexFormulaHelper.AddEmbraced to do this. I've temporarily disabled those in the xml file.

Example:
before patches

after patches

#3

Thanks both to you and to Marc Palmans for the patches.


Need new release

Could you make a new release for 1.17+?

1.18.2 and 1.19 have now released
Please release a new version of the mod @emilcarr


Add Harbor build process

Coverage remained the same at 56.767% when pulling 0dfcc4a43eece20f3f70d46cca8c36eb95daad18 on wy65701436:drone-test into c6198dd68e448cccdc1a7dcae6f43d8b8253203d on vmware:drone.

Coverage decreased (-0.04%) to 56.722% when pulling 55916e1354dad3c4360fd7d5129fdf7910c4ed52 on wy65701436:drone-test into c6198dd68e448cccdc1a7dcae6f43d8b8253203d on vmware:drone.

Coverage decreased (-0.04%) to 56.722% when pulling 55916e1354dad3c4360fd7d5129fdf7910c4ed52 on wy65701436:drone-test into c6198dd68e448cccdc1a7dcae6f43d8b8253203d on vmware:drone.

Coverage decreased (-0.04%) to 56.722% when pulling a3e4d44b5ed9bb5d689b9fcc00204fe5b0146e72 on wy65701436:drone-test into c6198dd68e448cccdc1a7dcae6f43d8b8253203d on vmware:drone.

Coverage decreased (-0.04%) to 56.722% when pulling 453138ed84d09b8a6fc0ae3824a77455aa5b854d on wy65701436:drone-test into c6198dd68e448cccdc1a7dcae6f43d8b8253203d on vmware:drone.

Coverage decreased (-0.04%) to 56.722% when pulling 1696fadbace3870f41b9801ae84792547fb0266f on wy65701436:drone-test into c6198dd68e448cccdc1a7dcae6f43d8b8253203d on vmware:drone.