Ban some (copyrighted) custom background

Description
We have this issue in czech community: some begginers are using copyrighted custom background when editing OSM.
In the whole database, there is around 11000 edits so far. Of course members of community are educating those users and reverting the changes, but we would like to ask, if it could be somehow banned in iD itself. It is becoming a little tedious :-)

Proposed change: when custom background hostname contains "mapy.cz", warn the user and disable the Save button.

.
Here is our thread in czech: https://openstreetmap.cz/talkcz/c3793 (translator)
Screenshots
No response

iD has a validation rule for warning about the use of some common copyrighted sources. mapy.cz can be added to this object:
https://github.com/openstreetmap/iD/blob/446133d1a468611cd374cdfa95ab080217c85db0/modules/validations/incompatible_source.js#L8-L22

@pnorman is right: The proper place to add this restriction is at the imagery policy of the OSM API. Please open a ticket for that on the openstreetmap-website.

Filled openstreetmap/openstreetmap-website#4085 for this.

perfect. :+1:
Just a small nit-pick:

they do not have a way to fix it on their side

this is not quite right, we could fix it on iD's side, but it does not make sense: is much better to do this using the existing functionality of the OSM API, as it will then have an effect in all (properly implemented) OSM editors, not just in iD. :blush:


WFLY-10222 Don't propagate exception if reading the resource fails

https://issues.jboss.org/browse/WFLY-10222

Retest this please

Retest this please

@bstansberry does this look ok now? In particular is the use of context.addResponseWarning an appropriate way of dealing with this situation?

@stuartwdouglas I think this is ok if it is a critical issue. I'd like to hear your thoughts on https://issues.jboss.org/browse/WFCORE-3754 though.
@jmesnil FYI.

It is not a critical issue, if there are plans to solve this in a more general fashion then this can wait.

@stuartwdouglas @bstansberry adding the hold label until you decide what to do with this

@stuartwdouglas  I think it's fine to merge this, so long as on the JIRA there's a big obvious comment saying that this is a workaround fix and it's not a regression if we do WFCORE-3754 and then remove this, breaking people who don't set the param discussed in WFCORE-3754. Expectation would be HAL would set the param. I believe HAL is the typical use case for reading an entire deployment resource tree and hitting this problem.
I have no idea when we'd get to WFCORE-3754 so it seems silly to leave people broken.


Turn off use of compute credits for PR builds.

Merged: even though the build status says it's "red", Cirrus reports it as having succeeded.


Copy over monitor datasources config from alerting to common utils

Signed-off-by: Surya Sashank Nistala<EMAIL_ADDRESS>Description
Copy over monitor datasources config from alerting to common utils
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #247 (42f06a2) into 2.x (587d9cb) will decrease coverage by 0.26%.
The diff coverage is 64.00%.

@@             Coverage Diff              @@
##                2.x     #247      +/-   ##
============================================
- Coverage     76.25%   75.99%   -0.27%
- Complexity      591      595       +4
============================================
  Files            91       92       +1
  Lines          3706     3778      +72
  Branches        527      544      +17
============================================
+ Hits           2826     2871      +45
- Misses          662      679      +17
- Partials        218      228      +10

Impacted Files
Coverage Δ

...g/opensearch/commons/alerting/model/DataSources.kt
62.29% <62.29%> (ø)

...n/org/opensearch/commons/alerting/model/Monitor.kt
80.97% <71.42%> (-1.11%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Revert Needed: Java Chatbot Auth

Auth-related changes from https://github.com/mixer/developer-docs/commit/fa76b0848ea332b7ac6250373353d59d03c3877f to pages/01.guides/02.chat/02.ChatBot/doc.md needs to be reverted.
Lines 326 and 400 should go back to MixerAPI mixer = new MixerAPI("Click here to get your Client ID!", "Click here to get your Token!");
...oopsie...

Thanks, feel free to PR this. Its a simple enough change to go through.

There was some discussion on the community discord, and it sounds like there are some differences in the argument order between the GitHub version and what's on Maven. So depending on where you pull from, it either will work as doc'd or won't.


Added Titanic to Python folder

Hi @PraveenAllam93 . Make sure to add your name in contributors.md file. Thanks! :-)


v0.39.0 fail on startup with: No module named 'custom_components.smartthinq_sensors.wideq.devices.range'

Describe the bug
After upgrade to v0.39.0 version the integration fails on startup with error:
Setup failed for custom integration 'smartthinq_sensors': Unable to import component: No module named 'custom_components.smartthinq_sensors.wideq.devices.range'
Expected behavior
If applicable, a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment details:
Environment (HASSIO, Raspbian, etc): HAOS
Home Assistant version installed: 2024.3.1
Component version installed: v0.39.0
Last know working version: v0.38.7
LG device type and model with issue: NA
LG devices connected (list): NA
Output of HA logs
2024-04-03 20:35:00.082 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration 'smartthinq_sensors': Unable to import component: No module named 'custom_components.smartthinq_sensors.wideq.devices.range'
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/setup.py", line 296, in _async_setup_component
    component = await integration.async_get_component()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 894, in async_get_component
    comp = self.get_component()
           ^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 921, in get_component
    ComponentProtocol, importlib.import_module(self.pkg_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/config/custom_components/smartthinq_sensors/__init__.py", line 50, in <module>
    from .wideq import (
  File "/config/custom_components/smartthinq_sensors/wideq/__init__.py", line 6, in <module>
    from .factory import get_lge_device
  File "/config/custom_components/smartthinq_sensors/wideq/factory.py", line 23, in <module>
    from .devices.range import RangeDevice
ModuleNotFoundError: No module named 'custom_components.smartthinq_sensors.wideq.devices.range'

Additional context
Add any other context about the problem here.

I found a temporary work-around by doing the following:

Make a new directory in custom_components named "smartthinq_sensors.wideq"
Copied the contents of the "wideq" directory to the root of that new directory.
Restarted and everything is working with no error.

Seems to be some type of new way how the custom component loading is handled, since this has a nested structure in it.

In my case, I saw the problem after upgrading HA to 2024.4.  It refused to load the component and failed with the error like listed above.  I did the work-around above and things were all happy.  Not sure what changed in HA to cause this, but I saw it right after the upgrade.


Add __check_auth wasm test vectors and tests

What
Add new test vectors containing simple approve and decline __check_auth impls, and tests that use those wasm contracts.
Why
To run some of the same tests that already exist, but with the contracts loaded in their WASM form, to ensure consistency across the handling of auth with natively loaded and wasm loaded contracts.
Originally these tests were written while debugging:

https://github.com/stellar/rs-soroban-sdk/issues/1332
https://github.com/stellar/rs-soroban-sdk/pull/1333

Tests will pass after the following change merges:

#1333


Input_embeds support

Motivation
Adding input_embeds support for sglang, motivated and inspired from
https://github.com/sgl-project/sglang/issues/745
Modifications
Changes in /manager to support new io_struct
Changes in /model_executor to support new io_struct
test cases under test/srt to demonstrate usage
In this implementation, curl request has to be made to use input_embeds.
This implementation DOES NOT have ARGS made to compliment input_embeds (I don't know where to make this edit),
When serving, to use input embeds please add
--disable-radix tag
i.e.
python3 -m sglang.launch_server --model-path Qwen/Qwen2.5-0.5B --port 30000 --disable-radix
test_input_embeddings.py is built and tested on Qwen2.5-0.5B, please edit the model name according to the model that you'd like to test
input_embeddings.json is my test case embeddings for Qwen2.5-0.5B model and comparison_response.txt is a comparison between text based input and input_embeds input.
Checklist

[x] Format your code according to the Contributor Guide.
[ ] Add unit tests as outlined in the Contributor Guide.
[ ] Update documentation as needed, including docstrings or example tutorials.

Thanks for the contribution. There is a related PR recently. Can you take a review on that? #2082

Thanks for the contribution. There is a related PR recently. Can you take a review on that? #2082

It seems that this PR update the documents and did more alteration with batch in mind. If their implementation doesn't require radix disabled then it'd be the perfect one among the 2 pr. My changes follow the guide in that open issue and is pretty minimal, with test to try out the feature.
Also can you please launch the checks again, I fixed some conditions since the last check you triggered, thank you.

Can you fix the conflicts?
I added you as a co-author in this merged PR so you are not a first-time contributor anymore. The CI will be triggered automatically for any of your future commits. https://github.com/sgl-project/sglang/pull/2122

I apologize for the slow reply.
Will get to it as soon as I can. Expect it done in a week!


Modified Response descriptions for DomainTools Iris Investigate and Iris Enrich connectors

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @v-EgorKozhadei ,
Thanks for the feedback, I have attended all the modifications requested. Please kindly review and provide your feedback.

Hello @v-EgorKozhadei , Thanks again for your feedback, I have fixed the issues , Please check and let me know all is fixed.

Hello @v-EgorKozhadei , Thanks a lot for the feedback. I have attended the latest 3 issues raised. Please kindly review.

Hi @RamboV!
Your PR is approved and merged. Please update your submission in ISV Studio by using the latest commit ID.
Let me know if you have any questions!

Hi @RamboV!
Your PR is approved and merged. Please update your submission in ISV Studio by using the latest commit ID.
Let me know if you have any questions!

Hello @v-EgorKozhadei  @Amjed-Ayoub
Thanks, In this pull request we missed the properties files update, I have raised one more PR,  https://github.com/microsoft/PowerPlatformConnectors/pull/1779, Please kindly review this and merge. Thanks in Advance. Please let me know if any issues.


FreeNAS Jail - Will not start after Homebridge Config UI X v4.7.0 update

Describe Your Problem:
After installing the Homebridge Config UI X v4.7.0 update homebridge fails to start and continually tries to restart unsuccessfully. I am running homebridge in a Freenas Jail as detailed https://github.com/nfarina/homebridge/wiki/FreeNAS-11-Installation-using-a-Jail. and using pm2.
I've tried reinstalling Homebridge. Resetting Homebridge and using just a very simple default json.config and the problem still persists. Perhaps thsi problem is now related to the new UI displaying the cpu usage because the logs mention cpu.js?
Logs:

Show the Homebridge / Homebridge Config UI X logs here.

0|homebridge  | [1/11/2020, 11:18:34 AM] [Config] Homebridge Config UI X v4.7.0 is listening on :: port 8080
0|homebridge  | TypeError: Cannot read property 'replace' of undefined
0|homebridge  |     at /usr/local/lib/node_modules/homebridge-config-ui-x/node_modules/systeminformation/lib/cpu.js:817:50
0|homebridge  |     at Array.forEach ()
0|homebridge  |     at /usr/local/lib/node_modules/homebridge-config-ui-x/node_modules/systeminformation/lib/cpu.js:814:19
0|homebridge  |     at ChildProcess.exithandler (child_process.js:292:7)
0|homebridge  |     at ChildProcess.emit (events.js:219:5)
0|homebridge  |     at maybeClose (internal/child_process.js:1027:16)
0|homebridge  |     at Socket. (internal/child_process.js:442:11)
0|homebridge  |     at Socket.emit (events.js:219:5)
0|homebridge  |     at Pipe. (net.js:666:12)
Homebridge Config:
    "platforms": [
        {
            "name": "Config",
            "port": 8080,
            "auth": "form",
            "theme": "auto",
            "tempUnits": "c",
            "sudo": true,
            "ssl": {
                "key": "/usr/local/etc/letsencrypt/live/xxx/privkey.pem",
                "cert": "/usr/local/etc/letsencrypt/live/xxx/cert.pem"
            },
            "platform": "config"
        },

Screenshots:

Environment:

Node.js Version: Node.js v13.3.0
NPM Version: 6.13.6
Homebridge Version: 0.4.50
Homebridge Config UI X Version:v4.7.0
Operating System: FreeNAS-11.2-U7
Process Supervisor: freeBSD jail

I'll check this out soon. I haven't tested shells - your intact the first person I've come across who's setup Homebridge this way.
You can roll back to 4.6.7 for now.
sudo npm install -g --unsafe-perm<EMAIL_ADDRESS>
Does your version of FreeNAS support Docker?

Thank you for your quick reply and for letting me know how to roll back. I'm up and running for now. Freenas does support docker after first installing a linux vm so not very elegant. I look forward to hearing back from you.

I'll push an update soon to not crash on cpu stats failure / add the ability to turn it off.
You might not get CPU usage info on your dashboard anymore, but we'll see what I can do.

@simonlock can you let me know what platform Node.js reports you as being on?
You can do this at an interactive Node prompt.
node
os.platform()

It displays 'freebsd'.
On Sat, 11 Jan 2020 at 12:55, oznu<EMAIL_ADDRESS>wrote:

@simonlock https://github.com/simonlock can you let me know what
platform Node.js reports you as being on?
You can do this at an interactive Node prompt.
node
os.platform()
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oznu/homebridge-config-ui-x/issues/461?email_source=notifications&email_token=ACUGGG367CCGNK63AT33WDTQ5G6VBA5CNFSM4KFRWY42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIWBMLQ#issuecomment-573314606,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUGGG3ZYUO5KUJSZ2ESTI3Q5G6VBANCNFSM4KFRWY4Q
.

Perfect, thanks. The systeminformation module I'm now using does not support the "current load" command on FreeBSD. I'll find an alternative.

FreeBSD 12 seems to be ok.

Same with 11.2:

I couldn't replicate this in FreeBSD, I wasn't using a jailed shell though so that may have been the difference.
Either way, I've published a @test version for you to try that uses the same method to get CPU load as v4.6.7 did on freebsd only.
You can install it like this:
sudo npm install -g --unsafe-perm homebridge-config-ui-x@test

Let me know if that works for you.

Thanks for your excellent support on this. I've installed your latest test update but Homebridge still continually restarts and I never get to see the status page.
The logs report:
0|homebridge  | [1/12/2020, 11:24:48 PM] Homebridge is running on port 51780.
0|homebridge  | TypeError: Cannot read property 'replace' of undefined
0|homebridge  |     at /usr/local/lib/node_modules/homebridge-config-ui-x/node_modules/systeminformation/lib/cpu.js:817:50
0|homebridge  |     at Array.forEach ()
0|homebridge  |     at /usr/local/lib/node_modules/homebridge-config-ui-x/node_modules/systeminformation/lib/cpu.js:814:19
0|homebridge  |     at ChildProcess.exithandler (child_process.js:292:7)
0|homebridge  |     at ChildProcess.emit (events.js:219:5)
0|homebridge  |     at maybeClose (internal/child_process.js:1027:16)
0|homebridge  |     at Process.ChildProcess._handle.onexit (internal/child_process.js:285:5)
0|homebridge  | [1/12/2020, 11:24:48 PM] Got SIGTERM, shutting down Homebridge...
I've now moved on to using Homebridge on a Raspberry PI running docker and I'm finding it to be a big improvement over the FreeNAS jail version with great visible logging; which was something that was very much lacking using pm2 on freeNAS. I'm happy to continue testing freeNAS if this is something you'd like to solve?

Thanks @simonlock,
If you could try one more time with the latest @test I think I have found all the issues:
sudo npm install -g --unsafe-perm homebridge-config-ui-x@test

Let me know if that works for you.

I've just tried it out and yes you've solved it.  Thanks for your support. The Raspberry Pi version is more feature-rich but this version now works and functions correctly.

Great; thanks for testing this out. It saves me having to setup and figure out FreeNAS 😄!

I don't mean to reopen the issue itself, more to point out to @simonlock that there are logs when running homebridge (and homebridge-config-ui-x), managed with pm2, through a FreeNAS jail too. The logs are, what I believe, automatically created at /<user pm2 is run as>/.pm2/logs/<pm2 service name>/ as <service name>-[out|error].log so in my case I've got two logs at:
/root/.pm2/logs/homebridge-out.log
/root/.pm2/logs/homebridge-error.log

I've chosen to show the logs within homebridge-config-ui-x by going into the settings for the plugin and under Log Viewer Setup I've chosen Custom Command and for the command:
tail -f /root/.pm2/logs/homebridge-out.log /root/.pm2/logs/homebridge-error.log

This solution makes new log entries show up through the logging web interface that homebridge-config-ui-x provides.
Sorry for the long explanation (and possible reopening) of the issue.

Thank you Ted. I did struggle trying to get these pm2 logs to show up in
Homebridge within a FreeNAS jail. However, I've now moved onto using
Homebridge running on docker within a Raspberry Pi along with my other IOT
related containers. Your instructions will certainly help if I decide to
return to using Homebridge in a FreeNAS jail again in the future. Moreover,
hopefully, your example given above will help others who might stumble
across this.
On Mon, 13 Jan 2020 at 14:03, Ted Jangius<EMAIL_ADDRESS>wrote:

I don't mean to reopen the issue itself, more to point out to @simonlock
https://github.com/simonlock that there are logs when running
homebridge (and homebridge-config-ui-x), managed with pm2, through a
FreeNAS jail too. The logs are, what I believe, automatically created at //.pm2/logs// as -[out|error].log so in my case I've got two logs at:
/root/.pm2/logs/homebridge-out.log
/root/.pm2/logs/homebridge-error.log
I've chosen to show the logs within homebridge-config-ui-x by going into
the settings for the plugin and under Log Viewer Setup I've chosen Custom
Command and for the command:
tail -f /root/.pm2/logs/homebridge-out.log /root/.pm2/logs/homebridge-error.log
This solution makes new log entries show up through the logging web
interface that homebridge-config-ui-x provides.
Sorry for the long explanation (and possible reopening) of the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oznu/homebridge-config-ui-x/issues/461?email_source=notifications&email_token=ACUGGG4ROW4QGUGBZZEW24DQ5RYEJA5CNFSM4KFRWY42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIY2ARA#issuecomment-573677636,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUGGGZJTN4VTSH3SN4PPGLQ5RYEJANCNFSM4KFRWY4Q
.


Lazy value map

This removes #transform and provides a more composable #map:
lv = Nanoc::Int::LazyValue.new(-> { sleep 1; 'Hello' })
lv_uppercase = lv.map(&:upcase)
p lv_uppercase.value
p lv_uppercase.value

This eliminates the need for subclassing LazyValue to specify custom transformations, and it allows chaining transformations.
Next up: updating the filesystem data source to make use of these new lazy values.
CC @RubenVerborgh — this will likely break your customisations since I believe they depend on #transform

@ddfreyne Thanks for notifying me, but transform was only used in  LazyAttributesValue of Document within Nanoc itself, not in my code.

Good—that’s part of a private API anyway ;)


Add python 3.12 to CI tests

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d1739ab) 100.00% compared to head (ac45456) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #41   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          290       290
  Branches        61        61
=========================================
  Hits           290       290

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update vspacecode vscode-extensions

Motivation for this change
two simple vscode-extensions update
leaving this one day open in case anybody wants to take a look
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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

[ ] Fits CONTRIBUTING.md.

@ofborg eval


[Bug] Exception in uploading jar package to Linux ECS, unable to upload

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The upload of jar package to the Linux ECS is abnormal and cannot be uploaded. It is tested that :
this method is the cause。
StreamPark Version
2.0.0
Java Version
1.8
Flink Version
1.14.6
Scala Version of Flink
2.12.17
Error Exception
Caused by: java.io.IOException: UT000128: Remote peer closed connection before all data could be read
	at io.undertow.conduits.FixedLengthStreamSourceConduit.exitRead(FixedLengthStreamSourceConduit.java:364)
	at io.undertow.conduits.FixedLengthStreamSourceConduit.read(FixedLengthStreamSourceConduit.java:264)
	at org.xnio.conduits.ConduitStreamSourceChannel.read(ConduitStreamSourceChannel.java:127)
	at io.undertow.channels.DetachableStreamSourceChannel.read(DetachableStreamSourceChannel.java:209)
	at io.undertow.server.HttpServerExchange$ReadDispatchChannel.read(HttpServerExchange.java:2430)
	at org.xnio.channels.Channels.readBlocking(Channels.java:344)
	at io.undertow.servlet.spec.ServletInputStreamImpl.readIntoBuffer(ServletInputStreamImpl.java:201)
	at io.undertow.servlet.spec.ServletInputStreamImpl.read(ServletInputStreamImpl.java:176)
	at io.undertow.server.handlers.form.MultiPartParserDefinition$MultiPartUploadHandler.parseBlocking(MultiPartParserDefinition.java:226)
	at io.undertow.servlet.spec.HttpServletRequestImpl.parseFormData(HttpServletRequestImpl.java:857)
	... 86 common frames omitted

Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please check your network connection is ok and request is timed out?

After testing, it is not the reason for this network. There is no problem only when you upload the zip package in the server test, but only when you upload the jar package.


switch to emojidex

*Most of this was done by @R-Y
Switch emoji over to emojidex. Benefits include:

Higher quality emoji
More emoji (including candidate emoji)
Updated much more often
No need to bundle assets within the gem (assets are found in the emojidex-rasters gem)
No need to run a rake task when assets are updated (updating the emojidex-rasters gem will update automatically)

Please take a look and tell us if anything needs to be refined before a merge.

+1 👍👍👍

Sorry for late response and thank you for your suggestion.
However, i released v3.0 which include switch to Twemoji.
https://github.com/hidakatsuya/prawn-emoji/blob/master/CHANGELOG.md
So i closed this PR. Please feel free to reopen if you have any comment.


auto detect color / invert certain label

Description
my bg is dynamic. and sometimes the bg is too bright/light that i cant see the clock anymore...

Could be really usefull, I want that too!


Would you like to support text tweet, not just only media?

Thanks

There is an option to includes non-media tweet. Is this the function you need?

Got it. Thanks a lot.

Maybe the option should be named as 'text without media' ?


Remove broken observer

This PR removes part of the rating button observers - they appear to be broken with the current code from Google.  I've just removed them here as I'm not sure if they're necessary anymore and if so I'm not sure how to reimplement.  I will say though that when combined with #389, it appears to restore LastFM scrobbling and growl notifications.

:+1: Last.fm seems to be working again, and I'd not even noticed that notification centre notifications had stopped, but they seem to be visible now. Nice work!
What did you use to debug this, out of interest? I spent a while on it, and couldn't trigger any sort of error so was mostly feeling around in the dark.

For debugging, I enabled the WebKit debugger on the webview (so you can right-click and get to the dev tools) by editing Preferences.plist and adding a boolean key WebKitDeveloperExtras of value YES.

Some useful info there, cheers. I've made a note for the next time the JS starts acting weird.
Anyways, here's a LGTM / :+1: from me. All seems good from what I can see.

@radiant-player/radiant-player-mac Could we get some eyes on this PR when you get a chance? Would be good to get this merged in asap

I'm going to go ahead and merge with the LGTM from @chrismou - feel free to leave comments for review later and we can get them sorted.

Removing broken code? :+1: :shipit:

@jacobwgillespie I love your commit stats on this repo
Added Lines: 3
Deleted Lines: 7457
:wink:

Great change, I'll be looking into everything in this repo later tonight.
On Mon, Nov 9, 2015 at 3:55 PM, Chris Chrisostomou<EMAIL_ADDRESS>
wrote:

@jacobwgillespie https://github.com/jacobwgillespie I love your commit
stats on this repo
Added Lines: 3
Deleted Lines: 7457
[image: :wink:]
—
Reply to this email directly or view it on GitHub
https://github.com/radiant-player/radiant-player-mac/pull/400#issuecomment-155225203
.

@kbhomes awesome!
@chrismou yeah, gotta love removing lots of unnecessary code :wink:


need for ember-browerify

This isn't an issue so much as a question ... I notice both redux and the redux-think middleware are being included as part of the normal NPM build process but wasn't sure why you're using browserify and your import statements are "npm:redux", etc.
Second part of this would be ... would using SystemJS be a reasonable alternative or do you prefer Browserify?
Sorry for the silly question.

I thought this addon (ember-browserify) was the secret to using redux in ember with ES6 but it's possible I'm out of date or touch :)
Do you have example that shows this working without any additional hacks in ember? I'd love to reduce the deps required to use redux with ember-cli

As far as I know, ember-browserify is the most straight-forward way to bring in npm dependencies.  I was just talking about this on Slack yesterday and everyone seems to agree.  @ksnyde have you used SystemJS successfully in an addon?  I have not seen this approach and am curious to know more about how well it works with broccoli and ember-cli.
My understanding is that the thinking on managing JS dependencies (better support for npm JS deps, dropping bower, etc..) is currently tied up with other considerations for things like engines.
Here is an open ember-cli issue that touches on it: https://github.com/ember-cli/ember-cli/issues/4211

For packages distributed with UMD/globals builds, a number of addons are using ember-cli-node-assets. The addon is basically just sugar for creating a broccoli tree for an npm package and then app.importing stuff from it, without all the extra build effort and runtime machinery that Browserify requires.
To support stuff like import redux from 'redux'; you could either package a vendor-shim or try out the fancy new anonymous AMD support that landed in this ember-cli PR.

@dustinfarris there was a great video from a guy in Germany about his experience with SystemJS and Ember (and I know he was talking to the core team at that point). I know that a lot of people believe SystemJS is better solution outside the Ember landscape but I'm really not an expert on either. Let me see if I can dig up a link for that video though ... here it is:
SystemJS, JSPM, and Ember
Came out in late 2004 so it's a bit dated but I think many of his points are still worthy of a watch if you're interested.

@dfreeman how do vendor-shims and/or anonymous AMD support enable you to import from node_modules?

oh, i think i see what you mean.  do the ember-cli-node-assets first, then do a shim on top of it?

@dustinfarris Exactly — ember-cli-node-assets gets the actual libraries into the build, and then the shim just exposes the relevant bits in a way that loader.js can find them.

@ksnyde so ... did the question morph into a set of actionable steps or no? I'm all for someone taking a look into ember-cli-node-assets but keep in mind that I'm focused on having the addon work with ember-twiddle (thanks to @dustinfarris for making this happen), the ember-install story remains just as simple/easy as it is today and most importantly :) we keep the addon stable for those using it. I can't prioritize this task myself because other parts of the redux + ember stack need my attn at the moment, and this isn't truly causing me any grief personally.

sure @toranb I completely understand; really glad your focus has delivered so much value already. I'm also a bit low on bandwidth and this "issue" was more of an inquisitive ask so I think unless there's a reason to keep it open we should just close this issue.


expose helm exit code when returning

this should expose the helm error code when returning, addressing https://github.com/futuresimple/helm-secrets/issues/84

Pleeeeaaaasseeeee

Thx


Add OWNERS

/assign @radoslawc

asked @aravind254 because I don't have r/w here, OWNERS added, this can be closed


Command to fully restart TestItemRunner.jl

Sometimes TestItemRunner falls out of sync with a project. Is there a command to restart TestItemRunner without restarting the whole VSCode application?
Examples:

TestItemRunner will detect new @testitems, but it appears that it won't remove missing ones. Is there a command to force it to rescan a project? This becomes particularly obvious when switching between different git branches with different sets of @testitems.
In most cases, TestItemRunner correctly detects when types have changed and the Julia process needs to restart. But sometimes this detection fails, and the TestItemRunner process falls out of date. For example, in some cases when a function is removed, TestItemRunner will report that tests are passing when in fact that they should fail due to the missing function. (If this is not an already-known bug, let me know if you'd like a simple reproducer.).

My question/request: Is there a VSCode command (1) to restart the TestItemRunner process or (2) to force TestItemRunner to re-scan for @testitems? Thanks in advance.

You can kill test runner processes on the Julia Workspace, they show up there.
All the things you describe sound like bugs to me, we should fix those!

I found a situation which seems to be reproducible enough:
This minimal package has the issue: https://github.com/lmiq/TestTests.jl
If you run the tests there, Aqua will fail because of an unbound type parameter, which happens in this line:
f(;g::Union{H,Nothing}=nothing) where {H<:Function} = g(1)
#f(;g) = g(1)

If the first line there is commented and the second is uncommented, the error should disappear, but it doesn't, the test still fails with the same error message.
Killing the TestServer in the workspace solves the problem with that test.


[Request for External] table 77 "Report Selections" - GetEmailAddressIgnoringLayout

You we please get acces to the procedure GetEmailAddressIgnoringLayout table 77 "Report Selections" ?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Add a reload button to the editor

The context file can be modified simultaneously by multiple people either through the GUIs or through e.g. Jupyterhub or a terminal editor, and when this happens there's a danger of a user with an old version of the context file accidentally saving the old version. Ignoring the can of worms around merge conflicts, we should at least add a reload button to the editor and notify the user that the file has changed. Partially implemented in e119354.

Closing because this was merged in #119.


Onboarding Request

Your Twitch username
REXXNITRO

I checked and don't see an email from when you signed up. I need the email associated with your Spotify account in order to add you to my list of allowed users. If you aren't comfortable putting it here, you can DM me on Discord - SaxyPandaBear#0443

I DM'ed you on Discord

Worked with them and confirmed it works. Thanks


Update gen.py to comply with python-3.7

The function IterNext had a parameter named async but this is a reserved keyword in python-3.7. Just change async to is_async

This is the only modification needed to install pykaldi on python-3.7. I had a SyntaxError in the tools/install_clif.sh script during installation. After that little modification, all is working as expected.

Thanks!


Support for Web Socket

Does the App Gateway ingress controller support Web Socket traffic?

I believe Application Gateway v2 does support web sockets. @akshaysngupta could you confirm?
@rahulvmarathe  however don't think we expose any specific configurations for web sockets?

Hi @asridharan, thanks for the update per this https://docs.microsoft.com/en-us/azure/application-gateway/application-gateway-websocket, App Gateway does support Web Sockets.
Wanted to know if there is anything specific I need to for it to work with the AKS Ingress.

@rahulvmarathe I tested App Gateway v2 and the K8s Ingress. As expected - App Gwy has native support for websockets.  And I did not have to do anything special to expose my Golang websockets server.
Here is my Deployment:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: websocket-repeater
spec:
  selector:
    matchLabels:
      app: wsrepeater
  replicas: 2
  template:
    metadata:
      labels:
        app: wsrepeater
    spec:
      containers:
        - name: websocket-repeater
          imagePullPolicy: Always
          image: abc.azurecr.io/websockets-repeater
          ports:
            - containerPort: 8888
      imagePullSecrets:
        - name: acr-creds

---

apiVersion: v1
kind: Service
metadata:
  name: websocket-repeater
spec:
  selector:
    app: wsrepeater
  ports:
  - protocol: TCP
    port: 8888
    targetPort: 8888

---

apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: websocket-repeater
  annotations:
    kubernetes.io/ingress.class: azure/application-gateway
spec:
  rules:
    - host: ws.mis.li
      http:
        paths:
          - backend:
              serviceName: websocket-repeater
              servicePort: 8888

One small caveat though: my particular websocket server implementation is fairly strict - requires the Sec-Websocket-Version header etc.  So a default Health Check doing an HTTP GET on /ws and expecting a 200 did not work.  As a result my pods were marked as unhealthy by App Gateway. Eventually that results in a 502.
Here is what a health ywebsocket call would look like:
curl -i -N -H "Connection: Upgrade" \
        -H "Upgrade: websocket" \
        -H "Host: localhost" \
        -H "Origin: http://localhost" \
        -H "Sec-Websocket-Version: 13" \
        -H "Sec-WebSocket-Key: 123" \
        http://<IP_ADDRESS>:8888/ws

So because we don't have a websocket specific health check and my implementation is strict - I had to add an endpoint /health just for monitoring.
Another workaround would have been to make custom health probes in App Gwy that accept error code 400, which is what my server returns when the websocket specific headers are missing.
Please reach out if I can further assist you with this.  I'll consider this question answered (and we'll augment our documentation with this blurb here to make it clear that the ingress and app gwy v2 support websocktes)

@draychev We should document this.

@draychev @akshaysngupta We have some issues in successfully implementing our websocket enabled application via AGIC and the AppGW. The websocket connection drops after 30 seconds, which is the default http timeout. When changing the timeout vault, the ws connection drops at the selected value. We submitted a ticket at Azure support, but their suggestion was the following:

The backend should be available to work on both HTTP and HTTPS. I can see that your app gateway backend pool only has port 80 enabled.

But the documentation states that the Listeners must be setup correctly:

You would need both HTTP and HTTPS listeners to support WebSocket and secure WebSocket traffic.

I find it hard to believe that HTTPS is mandatory for websocket. Your example above here does not use HTTPS, also your example in the documentation on Websocket does not use HTTPS. I hope you can shed a light on the issue we are experiencing.
We tried the following setup:
apiVersion: v1
kind: Pod
metadata:
  name: ws
  labels:
    app: ws
spec:
  containers:
    - name: ws
      image: ********.azurecr.io/ws:latest
    - name: nginx
      image: ********.azurecr.io/ws-nginx:latest
      ports:
        - containerPort: 80
        - containerPort: 443
      volumeMounts:
      - name: cert
        mountPath: "/certs"
        readOnly: true
  volumes:
  - name: cert
    secret:
      secretName: cert-1
---
apiVersion: v1
kind: Service
metadata:
  name: ws
spec:
  selector:
    app: ws
  ports:
    - protocol: TCP
      port: 80
      targetPort: 80
      name: http
    - protocol: TCP
      port: 443
      targetPort: 443
      name: https
---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: ws
  labels:
    app: ws
  annotations:
    kubernetes.io/ingress.class: azure/application-gateway
    appgw.ingress.kubernetes.io/ssl-redirect: "true"
spec:
  rules:
  - host: ws.********.tech
    http:
      paths:
      - backend:
          serviceName: ws
          servicePort: 80
      - backend:
          serviceName: ws
          servicePort: 443
  tls:
  - secretName: cert-1

This deploys a small NodeJS application, inlcuding and Nginx in front, which exposes 80 and 443, and also loads the SSL certificates.
The ws service exposes these two ports. The ingress includes two backend ports: 80 and 443, but this fails at azure-ingress-pod-level:
`Processing event failed:network.ApplicationGatewaysClient#CreateOrUpdate: Failure sending request: StatusCode=400 – Original Error: Code="ApplicationGatewayPortNotValidForProtocol" Message="Port 443 is not valid for protocol Http in backendHttpSettingsCollection /subscriptions/********-****-****-****-************/resourceGroups/d-weu-rg-shared-dev/providers/Microsoft.Network/applicationGateways/d-app-ag-dev/backendHttpSettingsCollection/bp-default-ws-443-443-ws." Details=[]

Removing the backend with port 443 works, but still the WS connection drops after 30 sec.
We tested using npm i -g ws and store the following script in a file (e.g. client.js):
const WebSocket = require('ws');

const ws = new WebSocket('wss://ws.*****.tech', { rejectUnauthorized: false });

ws.on('open', function open() {
  console.log('connected')
});

ws.on('close', function close() {
  console.log('disconnected');
});

After running node client.js we notice that the connection drops after 30 sec.
Conclusion: Its still unclear whats the issue is, all requirements seem to be fulfilled.
We now ended up using a Keep-Alive ping, which is sent from the server to the client to keep the connection up. But I believe that this should not be necessary.


Add HD-World.cc

  {
    "s": "HD-World",
    "d": "hd-world.cc",
    "t": "hdworld",
    "u": "https://hd-world.cc/?s={{{s}}}",
    "c": "Multimedia",
    "sc": "Video"
  },

Example: https://hd-world.cc/?s=test

Thank you for your contribution. Ultimately, I think this would work best as a custom bang if you're using Kagi.

On what criteria are you making this decision? An explanation would help for future contributions.


Attaching to containerized JVM

Attaching to a JVM running in a container requires a bit of trickery. Simply running jattach $PID doesn't work, of course, because jattach tries to write to the current directory of the containerized process, which is in a different mount namespace, and everything goes downhill from there.
However, assuming that /tmp is mapped to the same directory on the host and the container (e.g. using docker -v /tmp:/tmp), the following works:
$ sudo nsenter -t $PID -m touch /proc/1/cwd/.attach_pid1
$ sudo kill -SIGQUIT $PID
$ sudo jattach 1 <command>

The /tmp mapping is needed because the UNIX domain socket is in /tmp/.java_pid1. If the process in the container is not running as root, sudo needs to use the proper user id (because IIRC the JVM checks the other side of the socket to see that it's the same uid).
Can we somehow automate this process within the tool? E.g. if jattach calls setns to enter the target's mount namespace, and parses /proc/$PID/status to figure out the proper PID (in the container's terms) for formatting the .attach_pid and .java_pid paths, then everything would just work, I think. We wouldn't even need the /tmp volume mapping.
The alternative is to tell people to put jattach inside the container, but that's not always possible.

Makes sense.
BTW, if the target JVM is launched with -XX:+StartAttachListener, there is no need to create .attach_pid file and to send SIGQUIT (JVM is already listening to a socket for incoming connections).


[Adam] Tests for SAMRecordConverter.scala

Added unit tests for mapped and unmapped SAMRecord to AlignmentRecord conversions.

Can one of the admins verify this patch?

Jenkins, add to whitelist and test this please.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/657/Build result: FAILUREGitHub pull request #633 of commit e4bc94bb713ab3c33ab7063bf0a7e28af1472e93 automatically merged.Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos) in workspace /home/jenkins/workspace/ADAM-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/633/merge^{commit} # timeout=10 > git branch -a --contains 67b8bde13b6b4442536fc32152771e69ffd75ac8 # timeout=10 > git rev-parse remotes/origin/pr/633/merge^{commit} # timeout=10Checking out Revision 67b8bde13b6b4442536fc32152771e69ffd75ac8 (origin/pr/633/merge) > git config core.sparsecheckout # timeout=10 > git checkout -f 67b8bde13b6b4442536fc32152771e69ffd75ac8First time build. Skipping changelog.Triggering ADAM-prb ? 1.0.4,centosTriggering ADAM-prb ? 2.2.0,centosTriggering ADAM-prb ? 2.3.0,centosADAM-prb ? 1.0.4,centos completed with result FAILUREADAM-prb ? 2.2.0,centos completed with result FAILUREADAM-prb ? 2.3.0,centos completed with result FAILURENotifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'
Test FAILed.

@fnothaft Wanna talk about this on Monday?

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/667/
Test PASSed.

Merged! Thanks @Ibracadabra05!


bounding box restrictions

I am trying to troubleshoot bounding box restrictions Minecraft Cauldron/kCauldron

We're not Cauldron or any derivative thereof. We do not provide support for anything but Sponge. Nor could we help anyway, because you've provided next to no information about what your problem is.
If you need help with Sponge, please visit us on our forums, IRC or Discord as they are more appropriate places for support. Cauldron is dead, if you need support for any Cauldron derivative, please go ask them.


Add "Update from Callbook"-Botton in QSO-Details and Edit-QSO

What Feature is Missing?
Please add an "Update from Callbook"-Button in QSO-Details and Edit QSO:

Why is This Feature Important?
Because many of us are using third-party-software (for example N1MM for contesting or WSJT-X for FTx-QSOs), we don't have detailed data on name, QTH and so on and with this button (that is already available in advanced logbook) one can fetch missing data from callbook-services.
This is especially of interrest, if you are viewing the QSOs in the QSL-print-list or if you search for single callsigns via search function to have this function on single-QSO-base to not have the necessarity to switch over to advanced log to do this.
Is This Feature Personal or Beneficial to Others as Well?
I think this feature would be a benefit for all users.

Which fields should be filled? Same as LBA? What about already existing, i.e. logged fields?

I think it would be a good start to have it exactly like the function in advanced logbook. Using the same backend-function but only for the one specific qso.

@dg9vh
Hello Kim
I started with a draft for the button. Imho we don't need this button in two places ("QSO Details" AND "QSO Edit"). I implemented a basic button in QSO edit which fills all fields which are available from callbook response. Also it shows green borders of this fields (Todo here: only add border to changed fields)
Currently it overwrites the data. Maybe it should only fill fields which are empty. Need to change that.
What do you think about this draft?

@dg9vh
Your button is merged to dev branch. Will be in the next release. Feel free to close this Feature Request if this button works as expected :)

Works (also with hamqth).

@dg9vh
Can we close this? :)

Yes, this function is tested by me in 1.8.8. sucessfully.


Feature Request - Installation guide for the non-technically literate

Hi Amy,
Love the project.  I wrote an implementation guide for the mambo fpv for those less familiar with computers.  I have finished the windows guide (below) and will be working on one for linux soon.
Please use it if you find it useful:
*** Start your download of Visual Studio and include the Visual C++ module
this is required (and free) software for the windows installation
Special Install Python
Special install Python, click add python to path (this will clear up some command line call issues)
type py to open python3 on windows
(type python3 on linux)
check installation by typing py in the windows cmd line, this should enter python
type: quit()
this should exist python
type:python
this should enter python
type:quit()
(windows) type: pip
this should produce a rather large list of options
***the pip command will not work inside of python
Update pip installation with
(Windows) python -m pip install -U pip
(Linux/Mac) pip install -U pip
pip install --upgrade pip
Alternatively: download the pip file then run the pip file
in python open the pip file
-- from cmd, cd Desktop, dir, py "pip file name", enter
Use Pip for Installing
in cmd line type: pip install untangle
this will install the "untangle" package
in cmd line type:
pip install pyparrot
in cmd line type:
pip install zeroconf
****visual studio c++14.0 is a requirement
***This is a large download (3gigs) and should be done beforehand
Connect via wifi
wifi connection
If you are using the wifi (e.g. Mambo FPV), you need to connect your controlling device (laptop, computer, etc) to the wifi for the drone. Look for a wifi network named Mambo_number where number changes for each drone.
After connection to your drone its time to run code !!!!
Run code by cd'ing down to the directory (the folder your python code is in) and running the desired python file from the cmd line
example:
open command line
type: cd desktop
this will Change your Directory to the desktop
type:dir
this will display a list of all the folders (directories) on the desktop
type: cd yourFolderNameHere
type: dir
this will display all the files and folders in the directory
type: py TheNameOfTheFileYouWantToRun.py
*when you click enter the file will begin to run, if you are using the demo scripts you should see lots of nice feedback as it changes states
**you can use the arrow keys to go through your history of commands which can save you lots of time if your file names are long =)
***if you have several connects and disconnects try restarting your computer or resetting your ip (for the more technically inclined)
**** if you have crashes where the drone is flipping to one side when it shouldn't check the blades and bumpers.
The bumpers can shift after a crash and prevent the blades from spinning, or slow down their spin, which causes unintended flips

Working on a major update today (adding in @Victor804 's parrot swing code) and I'll get this added to the docs also


Swagger spec for Power BI Embedded

Added swagger spec for Power BI Embedded resource provider implementation.

Hi @wbreza, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (wabreza). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

I'm taking over for @wbreza to finish this so I will close this pull request and opened new request as #300 to continue on.

sounds good 👍


mmap-backed persistent ZFP arrays

It would be nice to be able to save to/restore compressed ZFP arrays from mmapped files. At present a crude workaround has been applied to zfparray.h:
added three extra variables to the array class:
std::string storage; //persistent storage via mmap
bool is_mmapped;
size_t storage_size;
and then alloc() / free() have been changed too:
// allocate memory for compressed data
void alloc(bool clear = true)
{
bytes = blocks * blkbits / CHAR_BIT;
printf("zfp::array3::alloc<clear:%d, storage:%s, bytes:%zu>\n", clear, storage.c_str(), bytes);
if (bytes > 1 && storage != "")
{
//try mmap first
int fd = open(storage.c_str(), O_RDWR | O_CREAT, (mode_t)0600);
if (fd != -1)
{
int stat = ftruncate64(fd, bytes);
if (!stat)
{
void *buffer = mmap(0, bytes, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);
      if (buffer != MAP_FAILED)
      {
        data = (uchar *)buffer;
        is_mmapped = true;
        storage_size = bytes;
        printf("zfp::array3::alloc<mmap OK>.\n");
      }
      else
        perror("mmap");
    }
    else
    {
      perror("ftruncate64");
    }
    close(fd);
  }
}

if (!is_mmapped)
{
  reallocate(data, bytes, 0x100u);
  if (clear)
    std::fill(data, data + bytes, 0);
}
stream_close(zfp->stream);
zfp_stream_set_bit_stream(zfp, stream_open(data, bytes));
clear_cache();

}
// free memory associated with compressed data
void free()
{
nx = ny = nz = 0;
bx = by = bz = 0;
blocks = 0;
stream_close(zfp->stream);
zfp_stream_set_bit_stream(zfp, 0);
bytes = 0;
if (!is_mmapped)
deallocate(data);
else
{
if (data != NULL)
{
if (munmap(data, storage_size) == -1)
perror("un-mapping error");
else
printf("zfp::array3::free.\n");
    storage_size = 0;
    is_mmapped = false;
  };
};
data = 0;
deallocate(shape);
shape = 0;

}
This is a "quick&dirty" workaround. A more elegant ZFP-wide solution would be much nicer.
the array3 constructor in zfparray3.h has been modified slightly too:
array3(uint nx, uint ny, uint nz, double rate, const Scalar* p = 0, size_t csize = 0, std::string _storage = std::string("")) :
array(3, Codec::type),
cache(lines(csize, nx, ny, nz))
{
storage = _storage;
is_mmapped = false;
storage_size = 0;
set_rate(rate);
resize(nx, ny, nz, p == 0);
if (p)
set(p);
}

@jvo203 Thanks for the suggestion and code.  I think this capability is not mmap specific but would be generally useful when the caller has already allocated and initialized a buffer with compressed data.  I think a constructor that takes such a pre-allocated buffer would be the right approach.  I'd rather have the caller than zfp deal with memory mapping, which requires platform specific code.
This ought to be a simple fix that we can implement on a feature branch.

One thing to bear in mind: the reason for putting mmap inside the array class is the need to know the exact size of the buffer that needs to be allocated via mmap. At present this is calculated inside array::alloc():
bytes = blocks * blkbits / CHAR_BIT;
once blocks and blkbits are known.
To work-around C++ allocating/deallocating zero-sized ZFP arrays during some C++-specific internal copy operations (copy constructors etc), mmap is only used if the requested size > 0 bytes.
Since even after ZFP compression there is no guarantee that a compressed ZFP stream will fit entirely in RAM, I have been forced to mmap-allocate the data buffers even during the initial creattion of ZFP compressed arrays. The initial compressing of data already uses mmapped RAM. The Linux kernel is then left to manage how much data (and when) is shifted from mmapped-RAM to the underlying disk file, depending on the actual pressure on RAM. And the persistent disk file helps avoid having to compress data again upon future read-only accesses to the data.
I guess different people would have different use scenarios in mind... Mine is just one example. After some careful consideration an elegant solution can surely be found!

@jvo203 You make a good point.  We could add a static function that returns the compressed size of an array of given dimensions, scalar type, and rate.  Another option is to add an optional Boolean to the constructors (or as a template parameter) to tell zfp not to allocate memory.  Yet another option is to template the zfp arrays on an allocator class.  Or to make the alloc function virtual.
We'll consider these options and others to support the more general case of allocating the compressed data outside of zfp.  We have to consider what to do when resize is called and how external allocation will interact with zfp's views, etc.  I'm hesitant to include mmap and MapViewOfFile solutions directly in zfp if there are reasonable alternatives, as there appear to be.

@lindstro  At the very least making all the memory-related functions like alloc/free virtual would be most welcome (and easiest for you!). At first I tried to extend zfp by adding a custom class with the mmap logic hidden inside but because alloc()/free() were protected but not virtual, C++ would not let me override these crucial methods. So making alloc()/free() override-friendly would be at the top of my personal wish list! Many thanks.


No scrollbar in popup for IE11

Library

[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Description
I'm using popup flow for my application and I can't scroll to submit button on IE11. It's becouse scrollbars are deactivated in IE11 by default when calling window.open().
Is there any reason, why you don't add scrollbars='yes' parameter when opening popup?
Here is the code:
const popupWindow = window.open(urlNavigate, title, "width=" + popUpWidth + ", height=" + popUpHeight + ", top=" + top + ", left=" + left);
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/lib/msal-core/src/UserAgentApplication.ts#L770
From MDN:

scrollbars
For accessibility reasons, it is strongly encouraged to set this feature always to yes.

https://developer.mozilla.org/pl/docs/Web/API/Window/open

@Morkowski I am concerned that the submit button isn't showing up without scrollbars.  Can you add a screenshot?  That is an issue in itself.

We are integrating with our client and that's how theirs form looks:

Black boxes are logo and title.
Becouse of described issue I can't reach submit submit button.
pr for this issue: https://github.com/AzureAD/microsoft-authentication-library-for-js/pull/1426


Fix: install curl explicitly in tools-deps images

...and use it instead of wget to save on installing that since the tools-deps install script needs curl anyway.
Some upstream images inadvertently removed curl causing this build error: https://github.com/docker-library/official-images/actions/runs/8633451962/job/23666676411?pr=16579
This ensures that curl is installed either way.

@Quantisan I'll go ahead and merge this to unblock our official-images PR, but let me know if you have any review feedback and I can address in a follow-up PR.


Geniev5

Bring in some updates from Genie5

Wanted to check, but it's not worth pulling, will abort


Sync mater into dashboard multi branch

This will bring all the changes done in mater (bug fixes and split contests) into dashboard multi

We should do the other way around..


XEMM -  Unable to use autocomplete for taker_connector field

XEMM -  Unable to use autocomplete for taker_connector field.
Autocomplete currently has only AMM or Exchange value instead of exchanges (binance, huobi etc.)
Currently you could manually type in the connector you want and it should work fine but not when pressing TAB
Steps To Reproduce
Clone latest development branch
create a cross exchange market making strategy
at the Enter your taker connector try to autocomplete using TAB
Actual behavior:
Bot showed Exchange, AMM for autocomplete, and if you chose one of them returns an error Invalid connector please choose one from dict_keys
You could manually type in the connector you want
Expected behavior:
Bot showed connectors available for taker_connectors

Currently you could manually type in the connector you want and it should work fine but not when pressing TAB

Release version
development 1.8.0

Severity: p3
Bounty: 10000 (https://hummingbot.org/maintenance/bugs/)
Rationale: Bug related to a component

Can I take this bounty?

Issue assigned to @leastchaos. Thank you for your contribution!


Bug: getTextContent Removes a New Line on Rich Text Editor

Lexical version: 0.8.0
Steps To Reproduce
Recorded a video https://share.getcloudapp.com/Z4unbb1B
Basically the issue is that in rootNode.getTextContent() there's a point where two of the same exact editor states result in two different text contents
Seems to be easy to reproduce by
1. Having at least two lines with text
Row 1
Row 2

The text content is Row1\n\nRow2
2. Add a new line between them
Row 1

Row 2

The text content is Row1\n\n\n\nRow2
3. On the 2nd line type anything
Row 1

dRow 2

The text content is Row1\n\ndRow2
Where did the two \n's go?
Link to code example:
CodeSandbox
The current behavior
It's possible to get two different text contents for the same editor state and it seems to remove some new lines
The expected behavior
An editor state should always return the same text content

Whoops, hadn't realized you had just upgraded last week. That being said. I just tired on 0.8.0 and the issue still persists
Updated the CodeSandbox and the initial comment to be 0.8.0 from 0.7.6

My hunch is that we're missing some logic. I'm no longer working on Lexical but maybe I can help you dig into this locally and maybe that can help you fix your problem and potentially put up a PR?
https://github.com/facebook/lexical/blob/main/packages/lexical/src/LexicalReconciler.ts#L289
https://github.com/facebook/lexical/blob/main/packages/lexical/src/LexicalReconciler.ts#L619-L620
Seems like we might just be missing updating the editorTextContent?

Would love some help digging in to it to be able to put up a PR
Will poke around the LexicalReconciler, hadn't looked in there. Although I do think that the editorTextContent is updated? Not sure if you did a quick look at the video I recorded but the text content does change every time just not to what's expected

To be clear this seems to only happen when using the RichTextPlugin but it works as expected with the PlainTextPlugin

I am running into the same issue. In my case it is manifesting in PlainTextPlugin though. Initially, the Lexical content is populated correctly. I am using the OnChangePlugin to monitor the changes to $getRoot().getTextContent() and the first call always returns the expected text. After making any sort of a change though, all of the subsequent calls loose all of the \n\ns in the entire text content. Visually though, in the editor, everything is still displayed in paragraphs as expected.

@TomasHubelbauer good catch just uploaded a video of the bug to GitHub directly
Interestingly enough I'm able to get it to work perfectly fine with the PlainTextPlugin and only run in to this issue with the RichTextEditor

I'm trying to isolate a repro right now, hopefully I'll manage to get one.

Ah, nevermind, my Lexical issue happens in 0.7.5 (what I am using in my app) but works in latest (0.8.0) so I need to find a way to update: https://codesandbox.io/s/priceless-rubin-x35v51

This still repros in 0.11.1

Any updates on this issue?

useEffect(() => {
        return editor.registerCommand(
            KEY_ENTER_COMMAND,
            event => {
                if (event?.key == 'Enter') {
                    event?.preventDefault()
                    editor.dispatchCommand(INSERT_LINE_BREAK_COMMAND, false)
                    return true
                }
                return false
            },
            COMMAND_PRIORITY_HIGH
        )
    }, [editor, onSubmit, withShift])

I think we can take over Enter to fix this problem....

I can't reproduce this

@etrepum i wish this issue wasn't closed, because it is still a problem over here.
The @jackple code did fix the issue, and I think it shows 2 things:

The issue is real and reproduzable
The fix should be something native from the library, since having two \n markers when it should be a single one is a bug in my opinion. If we need to take over the enter command and override the event function, something is wrong.

useEffect(() => {
        return editor.registerCommand(
            KEY_ENTER_COMMAND,
            event => {
                if (event?.key == 'Enter') {
                    event?.preventDefault()
                    editor.dispatchCommand(INSERT_LINE_BREAK_COMMAND, false)
                    return true
                }
                return false
            },
            COMMAND_PRIORITY_HIGH
        )
    }, [editor, onSubmit, withShift])

I think we can take over Enter to fix this problem....

If you think this is still an issue please create a new one with a clear reproduction that uses the current lexical build.
That workaround looks like it changes enter to use line breaks instead of creating a new paragraph, which is not intended behavior for the rich text editor.


Repositories to include

We need to add:

add https://github.com/helio-ecosystem
add morph-kgc.
add termitup.
add widoco (original)

vocablite is missing! https://github.com/dgarijo/vocabLite/

somef is missing

Missing new repos like library, helio, https://github.com/ODRE-Framework/ODRE-Framework, etc.


Replace Day2x with growth rate in main chart

Since cases will take a long time to double, switching trending to growth rate will better describe situation.
Verify the math with US total case:
4/10: 508297 (+7.61%)
4/9: 472331 (+8%)
4/8: 437358 (+8.16%)
4/7: 404333

actually, can we try how having both looks like? People really like the doubling metric. Maybe one on top of another? maybe color it grey so it's more subtle?

How about this

How about this

reverse it,  9 days to double (+8% daily)?
On Sun, Apr 12, 2020 at 2:28 PM Shawn W<EMAIL_ADDRESS>wrote:

How about this
[image: Screen Shot 2020-04-12 at 2 27 05 PM]
https://user-images.githubusercontent.com/3301683/79080448-c8f52080-7cc9-11ea-9568-d935d5cfd33e.png
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/pull/75#issuecomment-612678421, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF2TIOFVJU3MKQZV4CDRMIW6VANCNFSM4MGR6UKA
.

This feels better

I don't know how to make text grey. I thought I could use custom renderer using "content" property but the custom label isn't showing up. Will pause on this since it's not as important as data  https://recharts.org/en-US/api/Label#content

Thanks. I’ll take care of the rest. :)
Shawn W<EMAIL_ADDRESS>2020年4月12日 週日，下午2:44寫道：

I don't know how to make text grey. I thought I could use custom renderer
using "content" property but the custom label isn't showing up. Will pause
on this since it's not as important as data
https://recharts.org/en-US/api/Label#content
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/pull/75#issuecomment-612680449, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF5HUGC67B25ISFUCALRMIY3FANCNFSM4MGR6UKA
.


Make the code generic - branch Volunteer file: volunteer.py

In this function in Volunteer.Py Count==2 is hard coded (to mean the number of search terms in the config py must match with those in the tweet). Remove this and all values hard codes and pull values from config.py only and change it to len of the search terms array
def checkSearchTerms(searchterms,tags):
......
......
return (count == 2)  #change this to length of tagschecklist, picked from config.py

Hi, @a2un The issue seems very poorly made with no conclusive points. Please state what you are referring to and which branch of the project, what needs to be fixed.

The issue isn't for public. Thanks for looking at it but I am just keeping a reminder for me to make some changes.

Updates comments and description hope it seems more conclusive now

Updated the title and comments hope it is more conclusive now

Hey, since this is a short script , why are you making it complicated by adding another file with bunch of functions and then using them in main.py. @a2un

You're welcome to make changes that seem better to be made on this.


Mockclient calls to Docker mockserver really slow

Calls to mockserver in my tests (i.e. reset and setting up expectations) each have an overhead of about 4.5 seconds.
Calls to the mockserver server instance (started in a Docker container) from my SUT are fast.

using mockserver library: 3.10.4
server started in Docker with docker-compose:

version: '3.3'

services:
  mockserver:
    image: jamesdbloom/mockserver:latest
    ports:
    - 1080:1080
    restart: unless-stopped
    logging:
      driver: json-file
      options:
        max-size: "10m"
        max-file: "10"

What am I doing wrong?

I don't believe this is any thing to do with 'docker', 'docker-compose' or 'MockServer'.  When I run the exact same 'docker-compose' file I get the following command line timings from curl:
$ time curl -v -H 'Content-Type: application/json' -X PUT -d '{"httpRequest":{"method":"GET","path":"/simple"},"httpResponse":{"statusCode":200,"body":"some response"},"times":{"remainingTimes":1,"unlimited":false}}' 'http://localhost:1080/expectation'
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 1080 (#0)
> PUT /expectation HTTP/1.1
> Host: localhost:1080
> User-Agent: curl/7.54.0
> Accept: */*
> Content-Type: application/json
> Content-Length: 153
>
* upload completely sent off: 153 out of 153 bytes
< HTTP/1.1 201 Created
< Access-Control-Allow-Origin: *
< Access-Control-Allow-Methods: CONNECT, DELETE, GET, HEAD, OPTIONS, POST, PUT, TRACE
< Access-Control-Allow-Headers: Allow, Content-Encoding, Content-Length, Content-Type, ETag, Expires, Last-Modified, Location, Server, Vary
< Access-Control-Expose-Headers: Allow, Content-Encoding, Content-Length, Content-Type, ETag, Expires, Last-Modified, Location, Server, Vary
< Access-Control-Max-Age: 1
< X-CORS: MockServer CORS support enabled by default, to disable ConfigurationProperties.enableCORSForAPI(false) or -Dmockserver.disableCORS=false
< connection: keep-alive
< content-length: 0
<
* Connection #0 to host localhost left intact

real	0m0.022s
user	0m0.004s
sys	0m0.006s

$ time curl -v 'http://localhost:1080/simple'
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 1080 (#0)
> GET /simple HTTP/1.1
> Host: localhost:1080
> User-Agent: curl/7.54.0
> Accept: */*
>
< HTTP/1.1 200 OK
< connection: keep-alive
< content-length: 13
<
* Connection #0 to host localhost left intact
some response
real	0m0.031s
user	0m0.004s
sys	0m0.006s

$ time curl -v -X PUT "https://localhost:1080/clear"
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 1080 (#0)
* TLS 1.2 connection using TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384
* Server certificate: localhost
* Server certificate: www.mockserver.com
> PUT /clear HTTP/1.1
> Host: localhost:1080
> User-Agent: curl/7.54.0
> Accept: */*
>
< HTTP/1.1 202 Accepted
< Access-Control-Allow-Origin: *
< Access-Control-Allow-Methods: CONNECT, DELETE, GET, HEAD, OPTIONS, POST, PUT, TRACE
< Access-Control-Allow-Headers: Allow, Content-Encoding, Content-Length, Content-Type, ETag, Expires, Last-Modified, Location, Server, Vary
< Access-Control-Expose-Headers: Allow, Content-Encoding, Content-Length, Content-Type, ETag, Expires, Last-Modified, Location, Server, Vary
< Access-Control-Max-Age: 1
< X-CORS: MockServer CORS support enabled by default, to disable ConfigurationProperties.enableCORSForAPI(false) or -Dmockserver.disableCORS=false
< connection: keep-alive
< content-length: 0
<
* Connection #0 to host localhost left intact

real	0m0.066s
user	0m0.012s
sys	0m0.010s

All three of these show at most 66 millisecond response time.
I would try running the same test from you command line.
Then investigating what you test is doing perhaps it is just the test being slow or the way you are recording the figures is not valid.
If its neither of those perhaps the networking for docker on your machine is some how not functioning correctly, although that doesn't seem that likely.


Use Aqua.jl

Codecov Report

Merging #100 into master will increase coverage by 0.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #100      +/-   ##
==========================================
+ Coverage   88.88%   89.27%   +0.38%
==========================================
  Files          17       17
  Lines        1107     1072      -35
==========================================
- Hits          984      957      -27
+ Misses        123      115       -8

Impacted Files
Coverage Δ

src/processes.jl
90.78% <0%> (-0.71%)
:arrow_down:

src/core.jl
74.17% <0%> (-0.34%)
:arrow_down:

src/dreduce.jl
94.44% <0%> (-0.3%)
:arrow_down:

src/show.jl
94.87% <0%> (-0.05%)
:arrow_down:

src/library.jl
93.5% <0%> (-0.02%)
:arrow_down:

src/reduce.jl
97.67% <0%> (+2.51%)
:arrow_up:

src/progress.jl
92.59% <0%> (+3.82%)
:arrow_up:

src/comprehensions.jl
90% <0%> (+20.76%)
:arrow_up:

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 fea4774...c189d16. Read the comment docs.


Can we adapt the BLM model on source code?

Hi everybody,
I'm trying to adapt the blank language model on source code since in my opinion, there are many code-related tasks where such a model con gives us astonishing results. When we deal with source code, one of the main limitations is the open-vocabulry problem.
Do you think it is doable to plug in a fully-fledged BLP/Sentencepiece model to cope with such a problem without altering the under-the-hood network?
Just for the record, I tried to increase the vocabulary size but, the number of UNK tokens I get, hinder the learning process by a lot.
Thank you  :)

Hi,
Absolutely. You can preprocess the input with your favorite tokenizer and run the code as is.
For instance, using Google's SentencePiece:
spm_encode --model=<model_file> --output_format=piece < train.txt > train.sp
spm_encode --model=<model_file> --output_format=piece < valid.txt > valid.sp
python train.py --train.sp --valid valid.sp --root_dir <checkpoint_root> --vocab_size 10000 -- model_type blm

In our experiments, we used BPE, to preprocess our data.
The calculation of the perplexity is handled by carefully keeping track of the true number of words (https://github.com/Varal7/blank_language_model/blob/release/dataset.py#L13).

@Varal7 Thank you very much for your detailed explanation, nothing more to add.
Have a good day :)


Set print width

Hey,
i would like to set print width by limiting to 40 characters per line (using font B).
It work perfectly fine until I want to align to center - the printer just prints it thinking i'm using 80mm paper (i want to use 58mm instead) so the message is cut in half...
Any solutions?
Thanks x

Which commands are you using to align text to center?

Which commands are you using to align text to center?

e.CenterAlign()

From EPSON's handbook:

In red I believe is the why you are getting your current behavior

Then how do i solve it? Can i specify the number of characters in one line when justifying the text?

But if you know your line is always gonna be 40 chars long, it is probably easier to build an internal method that pads empty space in order to center align your text.
quick search in the internet, and found this for example
static string center(string text)
{
	int paperWidth = 40;
	int stringWidth = text.Length;
	int spaces = (paperWidth / 2) + (stringWidth / 2);

	return message.PadLeft(spaces);
}


Fix node agent error processing and improve logging

Cherrypick the change https://github.com/istio/istio/pull/19341 to 1.4.

@myidpt: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioctl-k8s-tests_istio_release-1.4
a4008da0f45e0d47e47defd149490e75c802ee4b
link
/test integ-istioctl-k8s-tests_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@myidpt: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-istioctl-k8s-tests_istio_release-1.4
a4008da0f45e0d47e47defd149490e75c802ee4b
link
/test integ-istioctl-k8s-tests_istio_release-1.4

lint_istio_release-1.4
a4008da0f45e0d47e47defd149490e75c802ee4b
link
/test lint_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

John, this is a blocker for us. I can chat with you tomorrow.

this just can't be merged until after 1.4.2 or it makes things complicated with the private patches

I see. Thanks.

Can we get this in? @howardjohn


test Github is using correct images

Purpose
Changes

Checklist

[ ] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram
[ ] Good unit test/integration test coverage

Ticket
Fixes #

Tested — correct images are being used


Requires mediaType althouth it's said to be optional

System.ArgumentException: The value cannot be null or empty. (Parameter 'mediaType')
   at System.Net.Http.Headers.MediaTypeHeaderValue.CheckMediaTypeFormat(String mediaType, String parameterName)
   at System.Net.Http.StringContent..ctor(String content, Encoding encoding, String mediaType)
   at Seq.App.HttpRequest.HttpRequestMessageFactory.FromEvent(LogEvent evt) in C:\projects\seq-app-httprequest\src\Seq.App.HttpRequest\HttpRequestMessageFactory.cs:line 57
   at Seq.App.HttpRequest.HttpApp.OnAsync(Event`1 evt) in C:\projects\seq-app-httprequest\src\Seq.App.HttpRequest\HttpApp.cs:line 84
   at SeqCli.Apps.Hosting.AppContainer.SendTypedEventAsync(String clef) in C:\projects\seqcli\src\SeqCli\Apps\Hosting\AppContainer.cs:line 121

Hi @m-ocean-it
Thanks for the report. Media Type is optional for a HttpRequest app instance because it is not required unless a body is specified. If a body is specified then a media type is required as per the HTTP specification.
Without a media type the receiving server has no way to interpret the content of the body and is instructed to treat the body as an octet stream which is probably not what is required.
If you want to send a text body you can use text/plain. For JSON its application/json. Or you can use any one of the valid mime types.
I will update the help text to make it clear that media type is required if a body is specified.

Thanks! Feel free to close the issue


docs: improve README

Closes #19.
Summary of Changes
See the issue for more details.

🦙 MegaLinter status: ✅ SUCCESS

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ MARKDOWN
markdown-link-check
1

0
2.81s

✅ REPOSITORY
git_diff
yes

no
0.02s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

:tada: This PR is included in version 0.3.0 :tada:
The release is available on:

v0.3.0
GitHub release

Your semantic-release bot :package::rocket:


Conditional initialization of attributes

While trying to use attrs with sqlalchemy, I came across an issue that in some cases it's needed to initialize an attribute only if it was passed to constructor.
Basically, this code
@attr.attrs
class A:
   a = attr.ib(initialize_if_given=True)

should translate to this
CONDITIONAL_INIT = object()

class A:
    def __init__(a=CONDITIONAL_INIT):
        if a is not CONDITIONAL_INIT:
            self.a = a

Aren’t you asking for a magic default value? Uninitialized attributes are basically time bombs.

@roganov can you state the actual use case instead of the proposed solution?

I this the described behavior would have no attribute a on class A if no corresponding argument is given to the initializer.  I don't think that's feasible, since we have the class attribute, and I also don't think that attrs should support every odd thing you can do in Python, so I think I'm with Hynek here.
@RonnyPfannschmidt is right though; without explaining the use case, it's hard to know whether this is an oversight of some sort.

@hynek  It is already possible to create a 'time bomb':
@attr.s
class A:
    a = attr.ib(init=False)

a = A()
a.a  # AttributeError

Or is it a bug?

I believe what @hynek was trying to express was that if you have an uninitialized attribute after you initialize your instance, that this is bad juju.  So I'd say your example above is a bug in the program; I do think it's generally unsafe to have init=False and no default unless you are doing something in, say, __attrs_post_init__ to rectify that.

so the basic gist is that you have pairs of attributes that are managed by a 3rd party library,
and whats really needed is not conditional initialization, but something more like sqlalchemy mapper awareness
aka a attrs_sqlalchemy addon

@hynek this use case also directly shows a weakness of the codegen approach, at mapper declaration time the class declaration is done and one is unable to fix the generated code by then

You call it a weakness, I call it the biggest strength. :)
And yes, this calls for an add-on that wraps @attr.s, sets unconditionally init=False and then generates the necessary __init__.
That said: I don’t know the ORM bits of SQLAlchemy well enough, but I’m gonna claim that it would be cleaner/nicer to be able to write:
UNSET = object()

@attr.attrs(cmp=False, auto_attribs=True)
class Child:
    id: int
    parent_id: t.Optional[int] = UNSET
    parent: t.Optional[Parent] = UNSET

Than doing voodoo with getattr/hasattr. That gives you much better introspection.
Are you sure it’s impossible?

@hynek correct mapper handling needs integration with real descriptors for stuff like lazy loading of dependencies or deferred loading of properties - i dare to claim that attrs biggest "strength" makes it fundamentally incompatible with reasonable orm's

@hynek
Could you clarify what this UNSET thing helps to achieve? Also it won't typecheck.


Idempotent pipenv?

Greetings and thank you for this sweet tool, Kenneth!
I have a question as I try to integrate pipenv into our make environment... Is there a way to make pipenv act w/idempotence?
Ex: With our current setup, we know that venv/bin/activate depends on setup.py, requirements.txt, etc, and we have a well-constructed Makefile such that if those files have not been updated, then re-creating the virtualenv is a no-op (make: Nothing to be done for 'venv'.). The virtual environment will only get constructed if venv/bin/activate does not exist, or if any of those course files are newer.
I can't seem to figure out how to get pipenv to behave this way. ie. Don't do anything if the virtualenv is already in-line with Pipfile.lock. pipenv install takes a good amount of time to return, and pipenv update does a purge before reinstalling everything.
Describe your environment

MacOS Sierra
Python 3.6.3
pipenv, version 8.3.2

Expected result
A super-fast command, where pipenv can check what is in Pipfile.lock and compare with the existing virtualenv.
Actual result
I don't know of a way to do this.
Steps to replicate
Run pipenv install repeatedly. If it's a big project each invocation can take some time.

It might be sufficient, for the described purpose, if pipenv --venv could output the location where the venv would be irrespective of whether it exists.

+1 for this feature.  I also need a way for my makefiles to have idempotent behavior, as do many others.  I created a similar issue and proposed an implementation (see below).
Kenneth said he'd like to add the feature but for now it was only a "cool idea to ponder".  I would argue its a required feature for a tool like this.  Perhaps now that it's part of PyPA it can be reconsidered.
https://github.com/pypa/pipenv/issues/1080

@tehranian we have a command that does a check but not quite what you want: pipenv update —dry-run which I’m assuming you are referring to when you mention the slow update command. Per @cs01 there was discussion of adding this as a part of pipenv check, but it just hasn’t been done yet.

let's add it!

we can communicate if it actually exists or not with exit codes

I couldn't find any way to just "check that Pipfile.lock and what is in the virtualenv match". Is this closed because there is now some way to do this or because it is a "won't fix" item?


Changing  is not persisting at all

Hello,
I'm trying to persiste some modifications on FITID, but when I open the file (*.ofx) nothing happens.
This my code:
import ofxparse
import datetime as dt

with open('/home/jovani/Downloads/ofx.ofx') as file:
    ofx = ofxparse.OfxParser.parse(file)
    transactions = ofx.account.statement.transactions
    for t in transactions:
        id = t.id
        id = dt.datetime.now().strftime("%Y%m%d%f")

Some help?

Did you find a way to fix this?


comment on transmembrane transport

In the discussion of #10968 MP uracil import into cell, David O-S raised the issue of entities taken up by some form of endocytosis. Those entities certainly change from an extracellular location to an intracellular one without crossing a membrane, but I can't think of a case* where an intravesicular entity gets into the cytosol without crossing a membrane, either that of the initial import vesicle or of some other vesicle formed from it (e.g. a lysosome or late endosome). If that's true, then aside from the argument about whether the inside of an uptake vesicle is topologically equivalent to the outside of the cell housing the vesicle, there's still the requirement for an additional molecular function to make the contained entities accessible to the rest of intracellular space.
Bottom line - perhaps until a *case of the sort I can't think of arises, the family relationship between uptake and transport remains OK.
Reported by: deustp01
Original Ticket: geneontology/ontology-requests/10997

The pattern is clear and easy to apply, but doing this completely would mean a relatively large number of new terms.  This is therefore a candidate for bulk addition using design pattern docs.

Endocytosis isn't transmembrane transport. The recent and ongoing changes to transport and transporter activity should make that clear.


VLC download URL does not work

Using "curl -fs http://update.videolan.org/vlc/sparkle/vlc-intel64.xml" shows that it's 2 versions behind the latest build, so there appears to be a delay in it getting updated. Running the full command of "curl -fs http://update.videolan.org/vlc/sparkle/vlc-intel64.xml | xpath '//rss/channel/item[last()]/enclosure/@url' 2>/dev/null | cut -d '"' -f 2" returns nothing, which is also a bit odd as well.

It looks like there is a problem with sparkle feeds in general. The VLC app updater shows 3.0.14 as the latest version, where the website indicates 3.0.16 being the latest one.
I just found out that the same behaviour exists for the Brave Browser, where the sparkle feed does not show the most recent version either.
I'll try to find out where to report those issues.

Here we go: https://code.videolan.org/videolan/vlc/-/issues/25861

@darklink87 According to the reply I got from a VLC dev, this is expected behaviour. The sparkle feed will be updated next week.

The sparkle feeds have been updated now.

This can technically be marked as "fixed" for now, but it's going to happen again whenever there's another version update.
Does anyone know if it's possible to tap into the same feed that Jamf uses to track VLC versions? They seem to have expanded their catalog greatly over the last few months, and they're fast to update whenever there's a new release.

I think that in regards to Installomator this IS fixed. We cannot handle not-updated web sites.
If there is a temporary software title or version you need to install, there is also the valuesfromarguments label where things can be set manually for these rare cases.

Bad sparkle feed


added line 440: app.set('view engine', 'jade');

corrects the following throw new Error('No default engine was specified and no extension was provided.');
Otherwise the node.js version of the tutorial does not work at step 6.

Hi @kaona, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@mimig @ryancrawcour to review

ack

#sign-off @rmca14


[SPARK-38829][SQL] Introduce conf spark.sql.parquet.inferTimestampNTZ.enabled for TimestampNTZ inference on Parquet

What changes were proposed in this pull request?

Introduces conf spark.sql.parquet.inferTimestampNTZ.enabled for TimestampNTZ inference on Parquet. When enabled, Parquet timestamp columns with annotation isAdjustedToUTC = false are inferred as TIMESTAMP_NTZ type during schema inference. Otherwise, all the Parquet timestamp columns are inferred as TIMESTAMP_LTZ types. Note that Spark writes the output schema into Parquet's footer metadata on file writing and leverages it on file reading. Thus this configuration only affects the schema inference on Parquet files which are not written by Spark.
This PR also removes the configuration spark.sql.parquet.timestampNTZ.enabled. The configuration is "heavy". When false, Spark can't write TimestampNTZ columns to Parquet files.
Why are the changes needed?

The only compatibility issue for supporting TimestampNTZ on the Parquet data source is about schema inference on the files which are not written by Spark. Thus we can simply have a flag about it.
Does this PR introduce any user-facing change?

No, TimestampNTZ is not released yet.
How was this patch tested?

UT

cc @sadikovi

I'm assuming this aims for Spark 3.4 release? should we add [FOLLOWUP] in the PR title?

Yes it aims for Spark 3.4.
This is a new JIRA. I left a comment in https://github.com/apache/spark/pull/36158#issuecomment-1412925457 for reference.

This is a new JIRA. I left a comment in https://github.com/apache/spark/pull/36158#issuecomment-1412925457 for reference.

Ah OK. I'm confused a bit since I saw SPARK-38829 is already resolved in Spark 3.3

Do we want to reuse spark.sql.inferTimestampNTZInDataSources.enabled or parquet needs a dedicated conf?

Do we want to reuse spark.sql.inferTimestampNTZInDataSources.enabled or parquet needs a dedicated conf?

For Parquet, the TimestampNTZ inference is on by default. As for spark.sql.inferTimestampNTZInDataSources.enabled, it is disabled by default to avoid behavior change on JSON/CSV/Partition Discovery. I will change spark.sql.inferTimestampNTZInDataSources.enabled to spark.sql.inferStringAsTimestampNTZ.enabled(or similar one).

The test failure doesn't seem to be related.
@sadikovi @sunchao @cloud-fan Thanks for the review.
Merging to master/3.4


fix syntax error from indentation

File layers.py contains a minor syntax error preventing it from being usable.
The error was introduced a couple of hours ago by
https://github.com/microsoft/LoRA/commit/2e08c961e1f347f7c74de4c575c22ec4c3a0c4b3

Thanks!


imagesToLoad & imagesLoaded are always empty

This is a:

[x] bug

Swiper Version: 4.3.5

Platform/Target and Browser Versions: Chrome

What you did
Configuration
        const mySwiper = new SwiperMod(node, {
          centeredSlides: true,
          grabCursor: true,
          loop: false,
          preventInteractionOnTransition: true,
          slideToClickedSlide: true,
          lazy: {
              loadOnTransitionStart: true,
              loadPrevNext: true,
              loadPrevNextAmount: 2,
          },
          preloadImages: false,
          zoom: true,
          on: {
              lazyImageReady: () => {
                // This can also simply be a selector
                const swiperNode = this.swiperRef.current;
                console.log(swiperNode.swiper.imagesToLoad) // []
                console.log(swiperNode.swiper.imagesLoaded) // 0
              },
          }
        });

Expected Behavior
In the lazyImageReady callback I expect the imagesToLoad & imagesLoaded to correctly reflect the number of images that are outstanding and the ones that have been loaded.
Actual Behavior
The numbers never update.
P.S. Remember, an issue is not the place to ask questions. You can use Stack Overflow
for that.
Before you open an issue, please check if a similar issue already exists or has been closed before.

These properties are used only when preloadImages: true, not with lazy loading. And they are not documented and not intended to be used outside of core


How can I submit an improvement for build/dev-server.js?

The mentioned file doesn't appear in the templates directory. Is it fetched from somewhere else?

I merged #975 into develop yesterday - so it doesn't exist anymore in that branch, and will be gone in master as well after the weekend - hopefully.


M3-1434: add tags list story

Adds tags list story with couple UI versions

Coverage decreased (-0.009%) to 28.813% when pulling e6b64541c3e65ec17f309f599d2ff88dcf540382 on rulikrulit:feature/M3-1434-story into 791f4605f1af2c2e26c8ec826828e0bc1128909f on linode:feature/M3-1433-to-M3-1437-domain-tags.

The original branch is merged. Will recreate the PR against develop


IIncrementalGenerator vs ISourceGenerator

I maintain a list of Roslyn Source Code Generators at ignatandrei.github.io/RSCG_Examples/v2/docs/List-of-RSCG
However, the source generators have been deprecated in favor of the new incremental generators , that allows better performance
See<EMAIL_ADDRESS>However this means passing from ISourceGenerator to IIncrementalGenerator
Could you do that ?
Thanks
Andrei

Hi @ignatandrei, sorry for the late reply. It looks like it would basically require a rewrite for the most part to use the new incremental generator API with its different semantics. I'll keep trying to figure it out and work on it, but feel free to give it a try yourself as well. Thank you for bringing this to my attention!


i18n(de-DE): Update editor-setup.md

What kind of changes does this PR include?

New or updated content
Translated content

Description

Updates the German version to incorporate the latest changes to the English page.

You're on fire! Thanks for all your reviews! 🙌


IPv6 for product.connect.nginx.com

Describe the feature you'd like to add to nginx
Add IPv6 capability to your AWS loadbalancer for product.connect.nginx.com
Describe the problem this feature solves
Connect to the mandatory product.connect.nginx.com for IPv6-only setups.
Additional context
The ngx_mgmt_module module enables NGINX Plus license verification and usage reporting.
This is mandatory for each nginx/1.27.2 (nginx-plus-r33) instance.
usage_report endpoint=product.connect.nginx.com interval=1h;
product.connect.nginx.com => this is IPv4 only and not reachable for IPv6-only setups

Similar remark for the NGINX Instance Manager, it that one also has no direct connection.


Set correct capitalization of JSMin

It's not JsMin.
https://github.com/rgrove/jsmin-php/blob/master/jsmin.php

Thanks :+1:

Thanks :+1: