Documented FormData support in .send()

I wanted to use superagent to send an HTML form, because of some <input type="file"> fields. Browser support section in the master branch's README hints that FormData is somehow supported, but I couldn't find any mentions of it in the documentation website. After blindly trying using .field() and .append(), I found out that it works with .send().

Thank you


Java 23.3: Generic Functional Expressions

Description
Add audio, documentation, and video regarding the Generic Functional Expressions
Location of File to be added The files should be added inside the Generic_Functional_Expressions which will be inside Lambda Expressions folder which will be inside of Java folder. If the folder is not present, create the folder with this exact name.

The documentation file can be in the .md/.py/.ipynb format.
Audio should be in .mp3 format and to be submitted through a Google drive link.
Video can be in any format and should be submitted through a Google Drive link.

Note:
You are required to comment with \assign CONTENT_TYPE, so that other content types can be created at the same time. Failing to comment in the above manner will lead to the removal of assignment from the issue even if the bot assigns. You are required to do a PR within 7 days otherwise you will be unassigned and others will be assigned. Issues will be assigned on a first come first serve basis.
Domain
Java_Domain
Type of Contribution
Audio, Video, Documentation
Code of Conduct

[x]    I follow Contributing Guidelines & Code of conduct of this project.

/assign documentation


schema: urls optimization

Signed-off-by: zhouhao<EMAIL_ADDRESS>

LGTM

Needs a rebase.

@jbouzane, I see you approved this PR, but PullApprove doesn't count that as a LGTM unless you use their v2 config (which we aren't).  If folks want to switch to a v2 config, I've taken a stab at one in opencontainers/project-template#29 and am happy to land any pieces of that in whatever repository folks want if they don't want to land it in project-template.  Until that happens, you should probably stick to LGTM/Rejected/etc. comments if you want PullApprove to notice.

Needs a rebase (likely because #539 has landed, so we want to drop this).

updatede,PTAL

lgtm

LGTM


update to bundler 2.0 for testing

2.0 includes a deprecation that is relevant to automate and for the purposes of testing I wanted to see what fails with this update.
https://github.com/rubygems/rubygems/commit/71b8454307e636f28b514d4553268c8f9b0d14e2 is the commit that includes the deprecation

@simaishi Can you comment here?  I know the bundler restriction was partly for build purposes and some relating to downstream.

@simaishi Can you comment here? I know the bundler restriction was partly for build purposes and some relating to downstream.

Satoe commented in the related PR, here: https://github.com/ManageIQ/manageiq-automation_engine/pull/421#issuecomment-609875875
At this point, I think we just need to merge the two PRs early in the morning eastern time so we can fix any fallout.


Ionic 3 demo broken due to version 2.4.6 -> 2.6.2 incompatibility in cordova-plugin-googlemaps

I'm submitting a ...  (check one with "x")

[ ] question
[x] any problem or bug report

OS:  (check one with "x")

[x] Android
[ ] iOS
[ ] Browser

cordova information:  (run $> cordova plugin list)
cordova-plugin-device 1.1.4 "Device"
cordova-plugin-googlemaps 2.4.6 "cordova-plugin-googlemaps"
cordova-plugin-ionic-webview 1.1.16 "cordova-plugin-ionic-webview"
cordova-plugin-splashscreen 4.1.0 "Splashscreen"
cordova-plugin-whitelist 1.3.3 "Whitelist"
ionic-plugin-keyboard 2.2.1 "Keyboard"

If you use @ionic-native/google-maps, please tell the package.json (only @ionic-native/core and @ionic-native/google-maps are fine mostly)
"@ionic-native/core": "4.14.0",
    "@ionic-native/google-maps": "4.14.0",

Current behavior:
Steps to reproduce:

Clone https://github.com/mapsplugin/ionic-googlemaps-quickdemo.git
Run npm install
Run ionic cordova prepare
Run ionic cordova build android

Result:
FAILURE: Build failed with an exception.

* Where:
Script '<redacted>\ionic-googlemaps-quickdemo\platforms\android\cordova-plugin-googlemaps\quickdemo-apikey.gradle' line: 54

* What went wrong:
A problem occurred evaluating script.
> Please add '<preference name="GOOGLE_MAPS_ANDROID_API_KEY" />' to the config.xml file.

Expected behavior:
Project in demo repository should build successfully without any manual configuration changes
Additional information
It seems that somewhere between version 2.4.6 and 2.6.2 of the plugin, the requirement was added for '' in the config.xml file. Because the demo has a dependency on "cordova-plugin-googlemaps": "^2.4.6", it resolves to version 2.6.2, but doesn't have the required preference in config.xml

https://github.com/mapsplugin/cordova-plugin-googlemaps/blob/master/README.md#api-key-android-and-ios-platforms

Hi, thanks for the reply. I understand what configuration is needed. This ticket is to point out that the ionic 3 demo at https://github.com/mapsplugin/ionic-googlemaps-quickdemo is broken in its current form. Anyway, if that doesn't matter, feel free to close this ticket.

Thank you. I updated the repo.


keyboard teleoperation updated to last version

Basic Info

Info
Please fill out this column

Ticket(s) this addresses
()

ROS2 version tested on
(Humble)

Aerial platform tested on
(Ignition)

Description of contribution in a few bullet points

Now compiles in Humble
Works with new drone_interface
Works for multidrone
Localization window for multi drone (wip)
Behavior control frontend

Description of documentation updates required from your changes

Future work that may be required in bullet points

Implement backend for behavior control (wip)
Improve localization window design (wip)
Linting and comment more code (wip)

Use t
$ as2 test keyboard teleoperation -v

before trying to merge


Animation documentation is not complete

Tooltip animation configuration is not documented.
The defaults are in Chart.defaults.tooltips.animation.
The default config is:
	animation: {
		duration: 400,
		easing: 'easeOutQuart',
		numbers: {
			type: 'number',
			properties: ['x', 'y', 'width', 'height'],
		},
		opacity: {
			easing: 'linear',
			duration: 200
		}
	},

Originally posted by @benmccann in https://github.com/chartjs/Chart.js/pull/7084#issuecomment-590159737

Some more ideas of things to add here: https://github.com/chartjs/Chart.js/issues/7223

I think this is fixed now: https://www.chartjs.org/docs/master/configuration/animations/


Added information about the data charges of AWS Simple Email Service.

AWS Simple Email Service charges for data in addition to the count of emails. If we include this in the README, people can know it beforehand.

Hi, @beawolf
The current iteration of the project isn't being maintained. We will only fix security issues on this version but we are not going to make any other changes.
Please see issue #329 where we will take feature requests for the new version of Mail for Good that we are going to start on.


fix(tooltip): close all other known dialogs on hover trigger

Closes #1023

Looks like it's unsubscribing from the last open dialog subscription, not the one actually being closed, so it closes the dialog, and kills the subscription for the one that's left open.

Closing in favour of #1048


[BUG] Does not work with anyio 4

Checklist

[x] I checked the FAQ section of the documentation
[x] I looked for similar issues in the issue tracker
[x] I am using the latest version of Schemathesis

Describe the bug
Schemathesis does not start when anyio 4 is installed.
It seems there is a dependency on starlette-testclient which is the cause of the problem.
To Reproduce
from schemathesis import from_asgi, experimental

from starlette.applications import Starlette

schema = from_asgi("/openapi.json", Starlette())

@schema.parametrize()
def test_api(case):
    response = case.call_asgi()
    case.validate_response(response)

Expected behavior
Works
Environment
- OS: [e.g. Linux or Windows]
- Python version: [e.g. 3.11]
- Schemathesis version: [e.g. 3.20.0]
- Spec version: [e.g. Open API 3.0.2]

Additional context
../../../../../Library/Caches/pypoetry/virtualenvs/bla-Ms7_xa4C-py3.9/lib/python3.9/site-packages/starlette_testclient/_testclient.py:450: in _portal_factory
    with anyio.start_blocking_portal(**self.async_backend) as portal:
E   AttributeError: module 'anyio' has no attribute 'start_blocking_portal'

Indeed, Schemathesis pins anyio to <4, exactly because of starlette-testclient
There is a plan to switch from starlette-testclient completely, but I didn't get to it just yet.

@Stranger6667 Is supporting anyio>=4 currently still planned (be it through switching away from starlette-testclient or some other way) and if so, can you estimate when that would be? Can I support you somehow in the migration?

Hi @wolfskaempf !
thank you for pinging. At this point I refactored the transport logic and now it should be easier to migrate that logic to httpx.
There are two main ways to get this through. The first one is to migrate ASGI transport to httpx, including sync-async bridge (the ASGI adapter in httpx is async). The other option is to add support for newer anyio versions to starlette-testclient and update it in Schemathesis.
the latter could be simpler, but I’d prefer the former one as it is cleaner and reduce our dependency on legacy tools (in a sense that starlette moved to httpx and starlette-testclient is a backport of its old code).
In terms of support I am thinking about a few ways:

work on the implementation, I’d be happy to assist
Test pre-release version when it’s ready
If your company uses Schemathesis, they might take a look at sponsorship options.

Other than that I am currently looking more into reworking all the jsonschema usage by integrating the next version of my ‘jsonschema’ Rust crate which aims to resolve long standing performance and compatibility issues in this area. It will take a few months, so I am not sure if I’d have the bandwidth to dive deeper into transports.

Tagging @Kludex for visibility. Maybe adjusting things on the starlette-testclient could be the easiest way to go for now.
@Kludex - would you be open to accepting anyio 4 support if I or somebody else will open a PR in the starlette-testclient repo?

@Stranger6667 I gave you maintainer rights on starlette-testclient. Feel free to do what you need. 👍

@Stranger6667 I gave you maintainer rights on starlette-testclient. Feel free to do what you need. 👍

Thanks!

The fix is available in 3.27.1


Annotate source code in PRs

Fixes #18
Fixes #21
This PR adds source code annotations to the GitHub action. For an example, see the checks on this PR.

I am seeing this fetch error in the Analyze step:

/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/core/node_modules/@octokit/request/dist-node/index.js:57
    throw new Error(
^
Error: fetch is not set. Please pass a fetch implementation as new Octokit({ request: { fetch }}). Learn more at https://github.com/octokit/octokit.js/#fetch-missing
    at fetchWrapper (/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/core/node_modules/@octokit/request/dist-node/index.js:57:1)
    at request2 (/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/core/node_modules/@octokit/request/dist-node/index.js:172:1)
    at hook (/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/auth-token/dist-node/index.js:58:1)
    at /home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/plugin-request-log/dist-node/index.js:37:1
    at /home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/before-after-hook/lib/register.js:25:1
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at Object.next (/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/plugin-paginate-rest/dist-node/index.js:76:1)```

This error disappears when using getOctokit instead of new Octokit, so I changed the code here.

I am seeing this fetch error in the Analyze step:

/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/core/node_modules/@octokit/request/dist-node/index.js:57
    throw new Error(
^
Error: fetch is not set. Please pass a fetch implementation as new Octokit({ request: { fetch }}). Learn more at https://github.com/octokit/octokit.js/#fetch-missing
    at fetchWrapper (/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/core/node_modules/@octokit/request/dist-node/index.js:57:1)
    at request2 (/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/core/node_modules/@octokit/request/dist-node/index.js:172:1)
    at hook (/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/auth-token/dist-node/index.js:58:1)
    at /home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/plugin-request-log/dist-node/index.js:37:1
    at /home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/before-after-hook/lib/register.js:25:1
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at Object.next (/home/runner/work/analyze-action/analyze-action/dist/webpack:/preflight-appmap-action/node_modules/@octokit/plugin-paginate-rest/dist-node/index.js:76:1)```

This error disappears when using getOctokit instead of new Octokit, so I changed the code here.

Weird. Good job finding the fix.


Update dg user case

Codecov Report

Merging #30 (b79067e) into master (dea5431) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #30   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 de60eae...b79067e. Read the comment docs.


add support for weighted records in ns1

Signed-off-by: Jayadeep KM<EMAIL_ADDRESS>add support for weighted records in ns1

See This PR


Error when using no seed sorting according to base algorithm

The customization of prioritizing seeds throws an error when implementing no sorting in the function following the base algorithm as implemented here:
The code I am using:
def sorting_criterion(seed):
    # do not perform seed sorting
    return lambda seed: 0.0

class MySorting(py4dgeo.RegionGrowingAlgorithm):
    def seed_sorting_scorefunction(self):
        return sorting_criterion

The error: TypeError: '<' not supported between instances of 'function' and 'function'
As the customization works with the example of random sorting, this may be a problem with the lambda function? Or some change in usage that needs to be considered - how would I implement no sorting?

You built in one level of indirection too much. You are already defining a separate function sorting_criterion - it should return the score for the seed directly. In other words: The function already takes the role of the lambda.
You can fix this with one of the following (untested):
def sorting_criterion(seed):
    # do not perform seed sorting
    return 0.0

class MySorting(py4dgeo.RegionGrowingAlgorithm):
    def seed_sorting_scorefunction(self):
        return sorting_criterion

or
class MySorting(py4dgeo.RegionGrowingAlgorithm):
    def seed_sorting_scorefunction(self):
        return lambda seed: 0.0

Your version mixes these two into something where you end up sorting by functions.

Understood - and your solution works perfectly. Thanks a lot!!


vim override file location is vim/settings/*.vim

I don't see files in vim/after/ directory are being sourced.
Should the user overrides be stored in vim/settings directory instead?

The existing customize arrangement works. Please discard this request.


fix outdated link.

Original URL was unable to access. Add one new link to vim cheat sheet.

 You should sign our Contributor License Agreement in order to get your pull request merged.


query params routing

this seems to be proposed in RR4
https://react-router.now.sh/query-params
is there any way to use query params currently ? rather then /:param routing
something like
https://mysite.com/products?size=medium&color=red
initialVariables: {
   color: null,
   size: null
},
// ...
products(first: 100, size: $size, color:$color) {
//...
}

Im not sure if there using the router but i saw a really good example of query param routing with relay https://www.verbling.com/find-teachers?language=zh&price[]=5&price[]=80&sort=popular

this library only handles data fetching, not routing


Update README.md

readme updated

That heading is intentionally "plz" as a stylistic choice, but thanks for the suggestion.

thanks for considering


I can't install all of plugins and connect to clojure is slow

I'm chinese and i'm in beijing. my company's network can't connect to pluginsServer. when i connect clojure, is slow.

It sounds like you're behind a proxy. Set your proxy information with these variables. Closing but can reopen if you're still having trouble and provide reproducible steps


AttributeError: 'NoneType' object has no attribute 'account_set'

https://sentry.io/krzysztof-szumny/steemprojectscom/issues/385495186/
AttributeError: 'NoneType' object has no attribute 'account_set'
  File "package/views.py", line 423, in package_detail
    ac.pk for profile in [ac.profile for ac in package.team_members.all()]
  File "package/views.py", line 424, in <listcomp>
    for ac in profile.account_set.all()

AttributeError: 'NoneType' object has no attribute 'account_set'

fixed by d42f1d8


agateau/refactor get files from path

Context
core.scan.file.get_files_from_path() does too many things, and gets in the way of some cleaning I plan to do on ggshield UI. This PR reduces its scope.
What has been done
core.scan.file.get_files_from_path() scope has been reduced. Most of the work is done in the 1st and 2nd commit. Look at their commit messages for details.
iac scan and sca scan no longer list skipped binary files. This has been discussed upfront and decided to be OK.
Output of secret scan path, secret scan archive and secret scan pypi is a bit nicer: it now makes use of headings:

Validation
Tests still pass. UI looks a bit nicer. Code is nicer.

Run


Keyword cipher in java

💥 Proposal
(A clear and concise description of what the proposal is.)
Keyword cipher in java
Have you read the Contributing Guidelines on Pull Requests?
yes
(Write your answer here.)

/assign


Selected class not being applied

Context
Selected class isn't correctly applied, when changing month by clicking on a day of the next month. It automaticaly fixes itself when changing month back and forth using < and >, or when .render() is triggered.
Actual Behavior

Steps to Reproduce

Setup new calendar with

trackSelectedDate: true,
showAdjacentMonths: true,
adjacentDaysChangeMonth: true

Find a month that doesn't end on Saturday (for 'en' locale)
Select day of next month

See https://jsfiddle.net/CookieMichal/0xv07zqz/3/
Your Environment

jQuery 3.1.1
underscore 1.8.3
moment.js 2.17.1
clndr 1.4.7
Google Chrome 55.0.2883.87

Thanks @CookieMichal this is an interesting one. It probably has something to do with either the events not triggering / not in the right order, or the render from changing month happening after the selected class is applied.
If you're able to find this one and submit a PR I would appreciate it. Otherwise I'll have to check it out over this or next weekend.

This is now fixed and deployed in 1.4.9!


Fix api

PR Description
Closes: #10
Closes: #23
Address comment: https://github.com/adam2392/mne-icalabel/pull/31#discussion_r863584227
@mscheltienne  is this what you had in mind?
Merge checklist
Maintainer, please confirm the following before merging:

[ ] All comments resolved
[ ] This is not your own PR
[ ] All CIs are happy
[ ] PR title starts with [MRG]
[ ] whats_new.rst is updated
[ ] PR description includes phrase "closes <#issue-number>"

FYI: I migrated all the testing datasets out into https://github.com/adam2392/mne-testing-icalabel-data

Nice!
Do we need all the stuff in mne_icalabel/conftest.py? Especially the matplotlib or pyvista/3D stuff?

Possibly not the pyvista stuff, but maybe good for future-proofing in case someone does like... 3D ica?
We can remove though.
I just copy-pasted it real quick :p.

yes, I figured 😆
Let's maybe clean it up and keep only what we need, but no rush ;)


must run with sudo?  running as root should also work

Please apply this:
$ git diff
diff --git a/ibeacon b/ibeacon
index 5f5da1a..5cf7a05 100755
--- a/ibeacon
+++ b/ibeacon
@@ -74,6 +74,9 @@ def process_command(c):
 def check_for_sudo():
   if 'SUDO_UID' in os.environ.keys():
     return 1
+  if 'USER' in os.environ.keys():
+    if os.environ['USER'] == 'root':
+      return 1
   else:
     print "Error: this script requires superuser privileges.  Please re-run with `sudo.'"
     return 0

+1


Copy elements/nodes

Is there a way to make a (deep?) copy of an element/node? I can't find any fast way to do this.
Copying element content and reparsing seems not the best performance option...

Hi @trikko ,
There are currently no cloning features. This is not complicated functionality and I think it is necessary to add it. I'll do it soon.
Thank you for the report!

@trikko
Added in https://github.com/lexbor/lexbor/commit/0c4010b10a922a45513d67beec0ddd29c2951f85.