Getting Server process terminated with exit code 143. See jetty.log for further details.

I'm using the 1.1.8 plugin and trying to run my project with the vaadinRun task, with the following configuration:
vaadin {
    version '8.0.5'
    logToConsole = true
    vaadinRun {
        server = 'jetty'
    }
}

Yet, for some reason the JVM dies out of nowhere, with the following error message: Server process terminated with exit code 143. See jetty.log for further details..

There is no jetty.log file anywhere on my machine. Please modify the error message to print the full path to the jetty.log file instead.
I am using ProcessBuilder (commons-exec) to run sub-process (simple scripts) - could it be possible that after those scripts finish running, Gradle will kill the JVM with my server?
Is it possible to debug this thing?

I have found out that log.error("foo") will kill the JVM for reasons that are beyond me. I am not able to debug the Jetty process in Intellij (since it's probably a subprocess of Gradle launched by Vaadin-Gradle-Plugin) - Intellij won't stop at breakpoints so I can't even find out what's going on. For the time being I will avoid using vaadinRun for development purposes.

Further testing:

System.err.println("foo") will correctly print foo and will not kill JVM.

Also slf4j complains that
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/home/mavi/.gradle/caches/modules-2/files-2.1/org.slf4j/slf4j-simple/1.7.12/42db62298b899818ff17352cbc00050e940bbfb0/slf4j-simple-1.7.12.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/home/mavi/.gradle/caches/modules-2/files-2.1/ch.qos.logback/logback-classic/1.1.8/e54f49a227611976642f81cf2106b81c1ece8bf5/logback-classic-1.1.8.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [org.slf4j.impl.SimpleLoggerFactory]

Yet I don't have slf4j-simple on my classpath nor in my Gradle dependencies. Maybe it's hooking into Jetty's logger under the hood, raising all sorts of havoc?

System.err.println("foo") will correctly print foo and will not kill JVM.

So, no problem then(?).

 Filed as #369

Already closed #369 as it works as expected. Check my answer there to get the sysout print-outs.

Also slf4j complains that Class path contains multiple SLF4J bindings.

I'm not seeing that by default, no logging framework is added by default. Maybe some of your dependencies are pulling in the the logging frameworks? You can check your dependency hierarchy and exclude where necessary.

Is it possible to debug this thing?

You can attach a remote debugger. A very old video how to do it (https://youtu.be/opjN3xlQ9Xk?t=1m39s) but most is still valid (except the devmode thing).

Regarding the slf4j: created a bug rgd this: #371
Regarding the original bug report: "process terminated with exit code 143": attached a simple project which demonstrates this immediate java process kill. Could you please check whether this is happening on your machine as well?
vaadingradletest.zip

Updating to Vaadin Gradle plugin 1.1.9 will even terminate JVM immediately, without any slf4j logging:
./gradlew vaadinRun
:compileJava
:vaadinUpdateWidgetset SKIPPED
:processResources NO-SOURCE
:classes
:vaadinClassPathJar SKIPPED
:vaadinCompile UP-TO-DATE
:vaadinUpdateAddonStyles UP-TO-DATE
:vaadinThemeCompile UP-TO-DATE
:vaadinThemeCompress UP-TO-DATE
:vaadinRun
SLF4J: The requested version 1.7.16 by your slf4j binding is not compatible with [1.6]
SLF4J: See http://www.slf4j.org/codes.html#version_mismatch for further details.
Apr 24, 2017 12:34:54 PM fi.jasoft.plugin.JettyServerRunner$1 lifeCycleStarting
INFO: Jetty starting
Server process terminated with exit code 143. See jetty.log for further details.

BUILD SUCCESSFUL

Total time: 1.647 secs

I'll post a link to the project shortly.

https://gitlab.com/vyzivus/simple-vaadin-gradle-app
Tag: vaadin-gradle-bug-368


setup confusion

I think there is a step missing between this setup instruction below. I don't know what "create a flow run " means in this context. Were we supposed to open a webpage, if so which one.
* Deploy test flow and deployment
    ```
    make register-test-flow
    ```
    A flow named `my-docker-flow` and a `my-docker-flow/docker-example` deployment should have been created
* Create a flow run from the `Deployments` screen by selecting the deployment and clicking on `Run`

Hello!
The step before the first one you posted describes how to access the UI
* Open http://prefect-server:4200 in your browser and verify Prefect UI is up and running
The screenshot below shows how to run a flow from the deployment screen
Create a flow run from the Deployments screen by selecting the deployment and clicking on Run

@fraibacas Thank you for your help on that.


Android: memory leak issue fixed

added stopPlayback() method for android.
use this method to release memory
the player to avoid a potential memory leak

Playback control is controlled by play: boolean props, so I don't want to add a method about playback.
I think the same effect can be obtained by calling stopPlayback on switching videos at loadVideo.
https://github.com/manse/react-native-brightcove-player/blob/d3a3d3be6b5928b3edf40ae3741342fa38b902af/android/src/main/java/jp/manse/BrightcovePlayerView.java#L307

@manse Yes, I agreed, We can call stopPlayback on switching videos at loadVideo.
but in my opinion, it's not the proper way to handle memory leak. We suppose to expose method for it. so one can handle it according to their cases.

@manse : I think instead of adding stopPlayback in loadvideo method. The separate method will be more convenient for library users. Users will free to use where ever they need to stopPlayback.

OK then, could you please implement stopping playback (with releasing memory) in iOS?
https://github.com/ronit-kadwane/react-native-brightcove-player/commit/d3a3d3be6b5928b3edf40ae3741342fa38b902af#diff-0a799f5d684ca54dc6b30f07ec2a41acR115

@manse Thanks for agreeing with the thought process.
Let me inform you first stopPlayback method is only for Android SDK, it is not for iOS SDK.
So, we don't need to worry about iOS.
I have already tested stopPlayback method for both the platform and it is working seamlessly. So you can merge it.
Thank you.

@manse
You will be more clear if You read BrightCove player documentation and official example on GitHub.
They are not able to handle the memory leak issue internally on the Android platform that's why
they have provided stopPlayback method just for the Android platform.
For iOS its handle by internal SDK that's why they don't have stopPlayback method for iOS.
As far as the plugin method is concerned, I have developed only for Android as it's needed in the Android only.
Just to look similar I have created a method in iOS as well but it is an empty function as iOS does not have stopPlayback method in SDK.
#39 0a799f5d684ca54dc6b30f07ec2a41acR115
I hope now you would be bit clear.

Thanks @ronit-kadwane your code saved my project. I believe this pull request should ve been merged

@ronit-kadwane  I am not understand where to call the function stopPlayback() in my react-native code

@ronit-kadwane where should I call stopPlayback? when unmounting?can I simply out it in the componentWillUnmount metgod here?
componentWillUnmount = Platform.select({
ios: function() {
NativeModules.BrightcovePlayerManager.dispose(
ReactNative.findNodeHandle(this)
);
},
android: function() {}
});


Showing Terraform outputs as "global" variables

Outputs specified in the Terraform file are passed to the Ansible dynamic inventory JSON accessible to all hosts.

Good idea, thanks for submitting this. Would you mind adding a test case to parser_test.go? (I know it's strange that all of the tests are in there; I intend to split them up.) I'd like to be sure that I understand what's going on here.

Hey @adammck , I've commited the parser test.

@adammck Fixed the compatibility issue with the #31, able to merge :+1:

Looks good to me:
adammck@imac5k [git df49397]
~/code/src/github.com/lazartravica/terraform-inventory$ TF_STATE=fixtures/example.tfstate terraform-inventory --list | jq .
{
  "all": {
    "datacenter": "mydc"
  },
  "alpha": [
    "<IP_ADDRESS>",
    "<IP_ADDRESS>"
  ],
  "alpha.0": [
    "<IP_ADDRESS>"
  ],
  "alpha.1": [
    "<IP_ADDRESS>"
  ],
  "beta": [
    "<IP_ADDRESS>"
  ],
  "beta.0": [
    "<IP_ADDRESS>"
  ],
  "gamma": [
    "<IP_ADDRESS>"
  ],
  "gamma.0": [
    "<IP_ADDRESS>"
  ],
  "role_web": [
    "<IP_ADDRESS>",
    "<IP_ADDRESS>"
  ],
  "role_worker": [
    "<IP_ADDRESS>"
  ],
  "type_aws_instance": [
    "<IP_ADDRESS>",
    "<IP_ADDRESS>",
    "<IP_ADDRESS>"
  ],
  "type_digitalocean_droplet": [
    "<IP_ADDRESS>"
  ]
}

This is fantastic, thank you!


💄 Add global information

Added a landing view with a catchphrase
The number of members in the organisation are now displayed
The number of repositories are also displayed
Updated styles to follow the BEM convention

(Styles are a work in progress, the goal being to display the data)
Github Insights must be updated to retrieve the number of PRs made by members of the organisation in order to move forward. (See https://github.com/Zenika/github-insights/issues/8)

:+1:

Nice design ! 👍

Thank you ! It's far from what the final product will be, I just threw it together to avoid having something empty for now. We'll need to agree on a final design eventually but until then the MVP will probably stay simple.

Is this still work in progress @christopherkade :smile: ?

Is this still work in progress @christopherkade 😄 ?

It is since we are missing the number of PRs done by our members (https://github.com/zenika-open-source/insights-website/issues/4). It would involve updating the Github Insights project first and then updating the current view.

Maybe we could postpone this part ? If so could I review this PR ?

Sounds good to me, of course, go ahead ! 😄


Update dependencies

Qs is a major version update, the breaking change is allowDots
in qs4:
Qs.parse('a.b=c', { allowDots: true });
//  {a: {b: 'c'}}

in qs5 by default
Qs.parse('a.b=c', { allowDots: true });
//  {'a.b': 'c'}

I don't think we use dots in our cookies, so this should be a safe update. I've also promoted 0.0.1 to 0.1.0

:+1:


[FLINK-8106] [cep] Optimize the timer logic in AbstractKeyedCEPPatternOperator

What is the purpose of the change
This pull request optimize the performance of AbstractKeyedCEPPatternOperator
Verifying this change

Existing tests

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Close this PR for the same reason as #5063


Support for PriceOracles

High Level Overview of Change
This PR adds client library support for Price Oracle amendment.
Please refer to the spec: https://github.com/XRPLF/XRPL-Standards/discussions/129
CPP Implementation: https://github.com/XRPLF/rippled/pull/4789

Context of Change
Support for transactions OracleSet and OracleDelete has been added. GetAggregatePrice request is included in the client library.

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (non-breaking change that only restructures code)
[x] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Release

Did you update CHANGELOG.md?

[x] Yes
[ ] No, this change does not impact library users

Test Plan

Unit, Integration and snippet tests have been included. There is some overlap between integration and snippet tests, perhaps they could be streamlined better.

@khancode  Since I've already written the snippet tests, I feel we can let it stay in the codebase. I'll be mindful about writing them for the future features.
If you feel otherwise, I can remove them too. I don't strongly about it.

@khancode Since I've already written the snippet tests, I feel we can let it stay in the codebase. I'll be mindful about writing them for the future features.
If you feel otherwise, I can remove them too. I don't strongly about it.

The snippets would still need to be reviewed, since they're intended to be public-facing - they're not tests, they're sample code. If they were just tests I'd agree, but since it's sample code, I do think it'll cause some clutter for devs.

@khancode Since I've already written the snippet tests, I feel we can let it stay in the codebase. I'll be mindful about writing them for the future features.
If you feel otherwise, I can remove them too. I don't strongly about it.

The snippets would still need to be reviewed, since they're intended to be public-facing - they're not tests, they're sample code. If they were just tests I'd agree, but since it's sample code, I do think it'll cause some clutter for devs.

I agree with @mvadari . I'd rather have them removed.

alrighty

c'mon guys, please approve the PR soon! we've had enough back and forth :)

I have resolved those comments which I could address -- If any of the reviewers prefer to resolve the comments yourself, please let me know.

thanks to all the reviewers for patiently working through this PR.
If there are any other comments, please let me know. I'll create a separate PR.


First click on the bottom bar tab doesn't behave correctly. But after that, everything is fine.

Every click after the first click is showing a nice ripple effect. However, the first click instead of ripple effect, it shows a weird and not very pleasant animation.
Anyone notice this as well? I know it's a small issue. I expect it possibly is an easy fix as well. Anyone with any clue about it? Thanks!

I'm experiencing the same issue, I don't know exactly what's going on.. I don't have a spare time to look on it right now, but I believe someone already got an easy workaround for this at all

I'm experiencing the same issue, I don't know exactly what's going on.. I don't have a spare time to look on it right now :/ Hope someone already got an easy workaround for this..
My 2 cents is that it have something to do with the support for background color changes and circular reveals introduced in this commit https://github.com/roughike/BottomBar/commit/916e8797b2608e7c85280639769ab07d170a11d7
I do think that defaultBackgroundColor field should be assigned to currentBackgroundColor to handle the cases where only the default ripple effect is wanted and the circular reveal animation is not necessary.

Anyone who is looking for a workaround solution, simply put below code after your bottom bar initialization.
final int DEFAULT_CURRENT_COLOR = 0;

for (int i = 0; i < bottomBar.getTabCount(); i++) {
     bottomBar.getTabAtPosition(i).setBarColorWhenSelected(DEFAULT_CURRENT_COLOR);
}

In current library, it's "handleBackgroundColorChange()" causing the weird animation. My temporary workaround is programmatically set the bar selected color equals to currentBackgroundColor to prevent this method from executing. (Setting values to 0 in XML didn't work)
Quote the code below
    private void handleBackgroundColorChange(BottomBarTab tab, boolean animate) {
        int newColor = tab.getBarColorWhenSelected();

        if (currentBackgroundColor == newColor) {
            return;
        }

        if (!animate) {
            outerContainer.setBackgroundColor(newColor);
            return;
        }

        View clickedView = tab;

        if (tab.hasActiveBadge()) {
            clickedView = tab.getOuterView();
        }

        animateBGColorChange(clickedView, newColor);
        currentBackgroundColor = newColor;
    }

However, if you need this bar color change feature, this solution maybe not ideal for you.
When I have time, I will try to look into it and try to find a more proper solution.


Screen Rotation

Description
What steps will reproduce the problem?

What steps will reproduce the problem?

Hi,
I have a laptop which can be used in tablet mode as well with laptop lid opening all the way to other side. Also I can keep my laptop like this /\ where screen is on one side and keyboard on other. In this configuration the screen is inverted and laptop automatically rotatest the display such that program windows are right side up and not upside down. But as soon as I open Spyder it inverts the screen and the screen becomes upside down. If I put focus on other program window the screen automatically rotates to right side up, but as soon as focus is back on Spyder, it again rotates it back to upside down. Very frustrating. My laptop model is Dell P57G...
Versions

Spyder version: 3.3.2
Python version: 3.7.1
Qt version: 5.9.6
PyQt5 version: 5.9.2
Operating System: Windows 10

Dependencies
pyflakes >=0.6.0  :  2.0.0 (OK)
pycodestyle >=2.3 :  2.4.0 (OK)
pygments >=2.0    :  2.3.1 (OK)
sphinx >=0.6.6    :  1.8.2 (OK)
rope >=0.9.4      :  0.11.0 (OK)
jedi >=0.9.0      :  0.13.2 (OK)
nbconvert >=4.0   :  5.4.0 (OK)
pandas >=0.13.1   :  0.23.4 (OK)
numpy >=1.7       :  1.15.4 (OK)
sympy >=0.7.3     :  1.3 (OK)
cython >=0.21     :  0.29.2 (OK)
qtconsole >=4.2.0 :  4.4.3 (OK)
IPython >=4.0     :  7.2.0 (OK)
matplotlib >=2.0.0:  3.0.2 (OK)
pylint >=0.25     :  2.2.2 (OK)

Thanks for reporting. I really don't know how we could fix this.
Any ideas @goanpeca?

Hmm not at the moment. Will investigate

@xanatzar if you disable automatic rotation on your laptop and configure this manually, the problem still persists?


A bit more stable observations

This change fixes a very small bug (I suppose) in the aiocoap_api class
where the flag whether to parse the output as json was not there in the
observe method. Hopefully this makes this part of the API a bit more stable.
Also I've added an observation callback as a global variable in the
ApiResource class. When tracking this value for an object, it is possible
to invoke the callback manually. This way the observation can also be
invoked when the user makes a manual change. Thus don't having the need to
call out the observation.
Thirdly I've added an explicit error callback to the ApiResource observation
which can be used to diagnose issues easily with the use of the print function.
Finally I've also added a check to check whether or not the incoming result has
even changed from the already existing result. Thus eliminating the need to
continiously update the value. This can hopefully make some parts of the
library a little more stable, because it will limit callback calls.
In conclusion, these are very minor changes, but hopefully will make the
library and in turn the applications using this library a little bit more
stable.

@lwis I've added a cancellation event for observations as per your suggestion. This would make it easier for integration into something like Home Assistant.
Today I've encountered the error I described in that home-assistant/home-assistant#18497 again. The only clue I've got so far is that the API is only returning decrypt_verify(): found 8 bytes cleartext and nothing more. I can also still control my lights, but the state is not being reflected in the interface anymore. I am still looking to trace down the root cause of this bug. And I am currently unsure whether it is a Home Assistant or pytradfri issue.
But this change should already (and probably) make somewhat of a difference.

Closing in favor of #214


Added a comment for a memory efficient loop.

Unfortunately, now some tests in this repo is broken, but the tests and the cs-check are passing locally for datastore/api directory. I'm going to merge without running the test on travis (job cancelled).

LGTM


fix(freebsd): Initialize state in check_proc_stopped.

Technically this is a warning, but warnings treated as errors.

Thanks!


behavior of "fetch prebuilt packages"

I have the suspicion that fetch prebuilt packages doesn't take all dependencies and profile into full account (correct me if I'm wrong).
This is my /usr/local/etc/synth/LiveSystem-make.conf
WITH_OPENSSL_PORT=	yes
OPENSSL_PORT=           security/libressl

This is my /usr/local/etc/synth/jail_ports.txt
databases/mariadb100-server
databases/mariadb100-client
databases/php56-redis
databases/py-MySQLdb
databases/redis
devel/git
devel/pear
devel/php-libawl
devel/py-pip
editors/nano
ftp/curl
ftp/php56-curl
ftp/py-pycurl
ftp/wget
lang/php56
lang/php56-extensions
lang/python
lang/python2
lang/python3
mail/dovecot2
mail/dovecot2-pigeonhole
mail/dovecot2-antispam-plugin
mail/opendkim
mail/opendmarc
mail/postfix-current
mail/rmilter
mail/rspamd
mail/ssmtp
security/ca_root_nss
security/clamav
security/libressl
security/openssh-portable
security/openssl
security/openvpn
security/php56-openssl
security/py-letsencrypt
security/py-pycrypto
security/py-cryptography
security/tor
shells/zsh
www/uwsgi
www/nginx
www/nginx-devel

This is the output with Fetch prebuilt packages   false
~ # ❯❯❯ synth status && synth status /usr/local/etc/synth/jail_ports.txt
Querying system about current package installations.
Stand by, comparing installed packages against the ports tree.
Scanning existing packages.
These are the ports that would be built:
Total packages that would be built: 0
The complete build list can also be found at:
/tmp/synth_status_results.txt
Scanning existing packages.
These are the ports that would be built:
  => security/py-openssl
  => devel/py-funcsigs
  => devel/py-pbr
  => www/py-requests
  => www/py-werkzeug
  => net/py-ndg_httpsclient
  => devel/py-mock
  => devel/py-pyrfc3339
  => devel/py-zope.event
  => devel/py-zope.interface
  => security/py-acme
  => devel/py-configargparse
  => devel/py-configobj
  => devel/py-parsedatetime
  => devel/py-python2-pythondialog
  => devel/py-zope.component
  => sysutils/py-psutil
  => security/py-letsencrypt
  => databases/redis
Total packages that would be built: 19
The complete build list can also be found at:
/tmp/synth_status_results.txt

This is the output with Fetch prebuilt packages   *true
~ # ❯❯❯ synth status && synth status /usr/local/etc/synth/jail_ports.txt
Querying system about current package installations.
Stand by, comparing installed packages against the ports tree.
Stand by, updating external repository catalogs ... done.
Scanning existing packages.
These are the ports that would be built:
Total packages that would be built: 0
The complete build list can also be found at:
/tmp/synth_status_results.txt
Stand by, updating external repository catalogs ... done.
Scanning existing packages.
These are the packages that would be fetched:
  => py27-openssl-0.15.1.txz (security/py-openssl)
  => py27-funcsigs-0.4.txz (devel/py-funcsigs)
  => py27-pbr-1.8.1.txz (devel/py-pbr)
  => py27-ndg_httpsclient-0.4.0_2.txz (net/py-ndg_httpsclient)
  => py27-mock-1.3.0_1.txz (devel/py-mock)
  => py27-pyrfc3339-0.2.txz (devel/py-pyrfc3339)
  => py27-zope.event-4.1.0.txz (devel/py-zope.event)
  => py27-zope.interface-4.1.3.txz (devel/py-zope.interface)
  => py27-configargparse-0.10.0.txz (devel/py-configargparse)
  => py27-configobj-5.0.6_1.txz (devel/py-configobj)
  => py27-parsedatetime-1.4.txz (devel/py-parsedatetime)
  => py27-python2-pythondialog-3.3.0.txz (devel/py-python2-pythondialog)
  => py27-zope.component-4.2.2.txz (devel/py-zope.component)
  => py27-psutil-3.3.0.txz (sysutils/py-psutil)
Total packages that would be fetched: 14
The complete build list can also be found at:
/tmp/synth_prefetch_list.txt
These are the ports that would be built:
  => www/py-requests
  => www/py-werkzeug
  => security/py-acme
  => security/py-letsencrypt
  => databases/redis
Total packages that would be built: 5
The complete build list can also be found at:
/tmp/synth_status_results.txt

I assume that at also py27-opensslshould be built if I choose to use libressl? Or is this a false assumption?

I have the suspicion that fetch prebuilt packages doesn't take all dependencies and profile into full account (correct me if I'm wrong).

it does.
if it doesn't pull something, it is because 1) it doesn't need updating 2) what's in external repo is not usuable.  It must think py27-openssl is usuable.
for something major like switching SSL, you probably shouldn't use prefetch anyway.  prefetch comes with a risk and it's off by default on purpose.  If in doubt, rebuild from source.

thanks

i think we can close this one.

Yes, thanks!


FrozenError when calling .new on STI model with state_machine in Rails 6.1

Calling .new on an ActiveRecord Single Table Inheritance model with a state_machine defined results in a "FrozenError: can't modify frozen Hash" since Rails 6.1.
Example, suppose we have the following model:
class Car < ApplicationRecord
  state_machine do
  end
end

which has a state and type column (the latter enabling STI) and some subclasses extending from Car. We would expect Car.new to return a new Car instance, but instead it crashes with "FrozenError: can't modify frozen Hash:".
Rails version: <IP_ADDRESS>
Ruby version: 2.7.3
State machine version: 1.2.0
This did work in Rails 6.0 and before.

Same here from Rails 6.0.5 to 6.1.6. Any advice here?

Seeing the same thing here. We make use of column_defaults and you can see that as of Rails 6.1, it was changed to freeze the returned Hash:
https://github.com/rails/rails/commit/c9c293e19464c9ce37316f2d917ac123d7b654e6
A number of Hashes were frozen in this commit. Still not sure of a workaround yet but at least we know where it came from and maybe even why.

FYI, since this gem hasn't been updated in 10 years, we updated to the state_machines gem (notice the plural "s" there). Specifically, we used the ActiveRecord integration gem called state_machines-activerecord.
What's super nice is that it's a drop-in replacement.
We're still conducting testing but it seems to fix our issue on Rails 6.1.

Seeing the same thing here. We make use of column_defaults and you can see that as of Rails 6.1, it was changed to freeze the returned Hash:
rails/rails@c9c293e
A number of Hashes were frozen in this commit. Still not sure of a workaround yet but at least we know where it came from and maybe even why.

Just tried this. Swapped in state_machines-activerecord. All my tests that were failing due to this issue are now passing.


Add entry for Azure Service Health

Hi team,
Can we please add an entry for Azure Service Health to this page?
Suggestion:
================================
Azure Service Health
Service Health keeps you informed about the health of your environment. It provides a personalized view of the status of your Azure services and regions, includes information about planned maintenance and current incidents and offers richer functionality, including alerting.
For more information, see your Azure Service Health dashboard.
Thanks.
Gr. Rutger

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

ID: 7282cce0-0a76-99d1-26a7-f2fa78d7df8c
Version Independent ID: e1c4eb4c-7c94-b41f-71fd-6b1d0f8bbfcf
Content: Ways to monitor Azure NetApp Files
Content Source: articles/azure-netapp-files/monitor-azure-netapp-files.md
Service: azure-netapp-files
GitHub Login: @b-hchen
Microsoft Alias: anfdocs

Thanks for reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

@RutgerKosters  -- Thanks as always!
@b-ahibbard - Andrew, would you mind running the suggested addition by PM and then implement this change?  Thanks!

#assign: @b-ahibbard

The requested update was published yesterday.
#please-close


Admin option no longer available

When I log in with privileged account admin option is no longer available. The first time I logged in it was there, and I added a couple of spaces, now it is not there, and I can no longer add spaces.

Hi @thudgen, nice to meet you!
When you say "privileged account", do you mean in the Azure sense, or do you mean that you explicitly added this user as an Admin within the IPAM UI?
The way Azure IPAM handles Admins is by remember the Object ID's of users and/or principals that were added as Admins. When you first login to the tool and no Admins are defined, everyone is essentially an Admin. Once you define Admins, only those users can access the Admin features.
If you believe that somehow you were accidentally locked out (for whatever reason), one solution could be to go into the Cosmos DB database and remove the Admins, setting it back to the "everyone is an Admin" mode. I'm happy to provide some screenshots and guidance around this process, or I'm also happy to setup a quick Teams meeting to walk you through the process myself.
What works best for you? We're here to help!

Good morning @thudgen, as it's been a while since this case was opened and I haven't heard back from you I'm going to close this issue for now.
If you're still experiencing this issue, let us know and we are more than happy to assist you however we can. Please do not hesitate to reach out again via GitHub, or you can also contact me directly via my email at<EMAIL_ADDRESS>if that is more comfortable for you. We can absolutely re-open this issue in the future if required.
I hope you have a great experience with the Azure IPAM project, we're always here if there's ever any other assistance you may need!

I re-installed and it is working.
Terrence K. Hudgen
Deputy Chief Technology Officer
Office of the Chief Information Officer
Small Business Administration
Office<PHONE_NUMBER>
Cell<PHONE_NUMBER>
E-Mail @.***
Non avere peli sulla lingua
From: DCMattyG @.>
Sent: Monday, September 11, 2023 10:48 AM
To: Azure/ipam @.>
Cc: Hudgen, Terrence K. @.>; Mention @.>
Subject: Re: [Azure/ipam] Admin option no longer available (Issue #154)
CAUTION - The sender of this message is external to the SBA network. Please use care when clicking on links and responding with sensitive information. Send suspicious email to<EMAIL_ADDRESS>Good morning @thudgenhttps://github.com/thudgen, as it's been a while since this case was opened and I haven't heard back from you I'm going to close this issue for now.
If you're still experiencing this issue, let us know and we are more than happy to assist you however we can. Please do not hesitate to reach out again via GitHub, or you can also contact me directly via my email at<EMAIL_ADDRESS>if that is more comfortable for you. We can absolutely re-open this issue in the future if required.
I hope you have a great experience with the Azure IPAM project, we're always here if there's ever any other assistance you may need!

Reply to this email directly, view it on GitHubhttps://github.com/Azure/ipam/issues/154#issuecomment-1714042560, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGLLHYMH4HVPTXEW464CFP3XZ4QDDANCNFSM6AAAAAA3UXAG6E.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Great to hear @thudgen, thank you so much for following-up with me on that. Have a great day!


Dexter.java avaliable on gradle is diferent of Dexter.java avaliable on github repository

The class Dexter.java avaliable by gradle doesn't contain the method isRequestOngoing and the class Dexter.java avaliable in the github repository does. Can you guys publish another version of Dexter to use by gradle that contains isRequestOngoing method?

Hi @jackmiras we are going to publish a new version this week. We are waiting for a PR review :)

Hi @jackmiras
This will happen from time to time as we are using master for backwards-compatible changes in order to keep the repository simple. We are sorry for the inconveniences this flow might cause but we already discussed it and we think the size of the project is not worth a more complex solution.
Anyways, the latest version has been already released in github and will be available in maven central in a few hours :3
Thank you for your feedback!


Support Reflected Type Conversion for DynamoDBEntryConversion

Description
This PR promotes ConvertFromEntry, TryConvertFromEntry, ConvertToEntry, and TryConvertToEntry from internal to public.
Motivation and Context
The current public API for DynamoDBEntryConversion exposes Generic accessors to type conversions making it difficult for runtime reflection to occur if needed.  These public methods call existing internal versions that accept Type parameters to function allowing information obtained from PropertyInfo and other reflection primitives that obtain Type objects to be supplied to DynamoDBEntryConversion and operate within the native SDK without additional complexity.
Current work around involved a complex Reflection off MethodInfo, maintaining a fork of the SDK, or large duplication of what already exists in the SDK (such as layering on a converter map or a large conditional list of types to convert).
Testing
No logic was altered in this PR.  No tests were created or modified to support.  All tests continued to pass.
Types of changes

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

Checklist

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have read the README document
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

License

[x] I confirm that this pull request can be released under the Apache 2 license

This was manually merged in https://github.com/aws/aws-sdk-net/commit/8c1e98b824c9ff1bb473c851f9bba7f2b5207744
Thanks @Habikki for your contribution!


please lower spec dependency to ios 8

we still need to support ios 8 devices, there shouldn't be anything which prevents this running on ios 8 right ?

Alamofire no longer supports iOS 8. Since this library is an Alamofire serializer, it is dependent upon that unfortunately.

but can't you use alamofire 3.5 ?

Not for Swift 3. Alamofire 3.5 was pre Swift 3 from my understanding. If that assumption is wrong, then we can consider it.
Otherwise, if this is something you need, you're going to need to fork Alamofire 3.5 and convert it to Swift 3 then you can fork this repo and build a version that supports 8.0

Can this issue be reopened and the library updated to support iOS 8 again for a 1.0.5 release? As of Alamofire 4.1.0, iOS 8 is now supported again: https://github.com/Alamofire/Alamofire/issues?utf8=✓&q=milestone%3A4.1.0

Yes that seems doable. I am very ill right now, so I don't know when I'll get to this. But if you need it soon, I would be willing to take a look at a PR and push a new version.

@jettf If you're looking for more github contributions this should be a simple one! ;)

Went ahead and setup a PR on the develop branch since it was a simple adjustment: https://github.com/AnthonyMDev/AmazonS3RequestManager/pull/56

1.0.5 has been pushed with iOS 8.0 support!

Thanks for your help on this @u10int


Website resources page: Broken links to all projects

All of the project links at https://opentk.github.io/resources/ are at the old opentk.com site, which evidently does not exist any more. It would be helpful to conveniently review some example projects to evaluate OpenTK, and fixing these links would help.
Also having this page with all those broken links makes OpenTK seem abandoned.

Ill take care of this after I'm finished with #9

Just an update, I'm about half-way through the list getting updates to the projects or marking them as abandoned. I hope to have a PR up between this weekend and the next.

Fixed in the new site


fix(TDI-43723): override beanutils version from spring-boot-starter

What is the current behavior? (You can also link to an open issue here)
Additional fix for https://jira.talendforge.org/browse/TDI-43723
What is the new behavior?
Override beanutils version from spring-boot-starter-parent
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:
Sonar results

Pull request test result: success :star:
Sonar results


Password component

Closes: #...
Checklist

[ ] Made good Password Component

Changelog

Made a Password Component in Storybook
Extra features:

Eye-color: change eye-color depending on color-theme

Error message displayed under input-field

Info message displayed above input-field

Also going to ping @kharann in regards to icon library - Iconify?

Replaced in favor of https://github.com/dotkom/monoweb/pull/302


Remove xterm.clearActiveSearchDecoration

fix #194098

Looked into this, we actually need this in xterm.js so we need to bring it back. Example bug without it:

Find in terminal with multiple results
Press escape

Hi @Tyriar.
It will be like this?? :flushed:

However, I think losing focus doesn't require cleaning up the decorations.

We should probably be calling clearDecorations on blur too, but I suspect that's not actually a regression actually. Just doing the low risk thing here and bringing it back as it was accidentally removed


Database.BeginTransaction() throwing on InMemory Repository

I have a working code for EF7 beta6 with the SQL CE 4.0 data-provider, which uses some transactions inside.
Now I want to unit-test these things. However, when I switch the context options to be in-memory, I get the following exception:

"No service for type 'Microsoft.Data.Entity.Storage.IRelationalConnection' has been registered."
Stack:
at Microsoft.Framework.DependencyInjection.ServiceProviderExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)
at Microsoft.Framework.DependencyInjection.ServiceProviderExtensions.GetRequiredService[T](IServiceProvider provider)
at Microsoft.Data.Entity.Infrastructure.AccessorExtensions.GetService[TService](IAccessor`1 accessor)
at Microsoft.Data.Entity.RelationalDatabaseFacadeExtensions.GetRelationalConnection(DatabaseFacade databaseFacade)
at Microsoft.Data.Entity.RelationalDatabaseFacadeExtensions.BeginTransaction(DatabaseFacade databaseFacade)

Basically, the code to reproduce:
var options = new DbContextOptionsBuilder<BloggingContext>();
options.UseInMemoryDatabase(persist: true);
//  options.UseSqlCe(@"Data Source=Blogging.sdf"); --> works fine

using (var x = new BloggingContext(entityOptions))
{
    var tx = x.Database.BeginTransaction(); // Fails, when using InMemoryDatabase
    x.Blogs.Add(new Blog() {Url = "URL-" + (i*10)});
   ...
}

What am I doing wrong?
Am I missing some context-options?
I would have thought, that these things work out of the box.
Cheers,
Harald

@hkoestin This is early beta software so do not expect anything to Work "out of the box" :smile:
Have you tried removing the use of Transactions?
And why do you need Transactions in the first place, SaveChanges is transactional already ?

Well, if I remove the Transactions, it is working - even "out of the box" :smile:
However, that is not really the plan.
I need the transactions in place, because we have repositories (= DbContext) per aggregate. As long as we only touch one aggregate in one command, it is totally fine to just have "no" transaction there, because, as you said, SaveChanges anyhow pulls up a transaction.
Unfortunately, the world is not ideal all the time, and in some cases we need to touch two aggregates, meaning, we need to touch two Repositories (= DbContexts). To make things transactional then, I would like to "share" the DB connection (and thus the transaction) for the two DbContexts. That is why I would like to use the transactions.

This is by-design because the transaction APIs are specific to relational data stores. That said, the exception is not at all helpful so we opened #2879 to provide an exception that tells you why the API does not work.

@rowanmiller thanks for you reply and your thoughts. I still don't see a reason, why I can't use the BeginTransaction method on an InMemory store. It could just ignore it - or even implement some of the ACID properties (namely ACI). As a user of the whole thing, I don't mind the actual implementation behind (may it be SQL or Azure Doc Store or InMemory). I want to start a Transaction and then perform actions.
Reason for me is exchangeability for unit testing. I do not want to build in additional logic for this. I just want to configure my context options and pass them in.
I would just love to have it like:
// for production ...
var contextOptions = new DbContextOptionsBuilder().UseSqlServer("....").Options;

// OR for testing ...
var contextOptions = new DbContextOptionsBuilder().UseInMemory().Options;

// inject them and use them, no matter which kind of store it may be
var context = new DbContext(contextOptions);

I see no need for other variations here.
Do you see my point?
It would be a ease of testing all over!

@hkoestin
One of EF7s goals is to remove any magic and general abstraction over different datastores and rather make all operation which cannot generally be abstracted explicit. I think this is a good principle because it is impossible to reasonable abstract/hide all the different behaviors and features of the various datastores ef7 will support.
In your concrete example,  Database.BeginTransaction is an extension method from the EntityFramework.Relational package and cannot work with other none-relational datastores since its explicit for relational stores and uses the System.Data.IsolationLevel enumeration for further configuration of the transaction behavior. (For example some of the isolation levels may not be supported in Azure Doc store).
I think you would have to create your version of BeginTransaction() if you want to hide the fact that you are either using an InMemory store or a relation store. For example you could create your own extension method which checks if the the context is running in relational mode and then conditionally begin a transaction. Your unit tests may then not cover the bugs which are specific to your relational store.

@hkoestin we did talk about having a flag that you could set to make the relational specific APIs a no-op when called on a non-relational store. So you would try and begin a transaction on in-memory... it would throw saying "that's not supported but change this flag if you want it to just no-op". Would that work for you?

@rowanmiller yes, this would already help a lot. Still, I would favor a real transaction (even in memory) over a no-op, but it is better than nothing.
Background: I want to be able to run basic unit (and integration) tests really fast by using the in-memory features of EF7. Setting up a whole context there would just cost to much. And I am talking about roughly 5000 tests. It is a big difference, if setting up the db-context takes 20ms or is almost instant.
For tests, which need a real transaction, e.g. they are testing rollback features or similar, I still can configure an ordinary SQLCE provider. That is ok.

@hkoestin yep your scenario makes total sense. BTW we have talked about promoting the concept of transactions to core (rather than relational). Not all data stores have transactions... but the vast majority do.

@rowanmiller Love to hear that the concept of transactions may be known to more (and not only relational) databases :smile:

Excellent, support for transactions using the in memory store will make for easier testing of unit of work patterns. Any update on this?

@hkoestin I am in the same boat for sure. I absolutely love what I can do with an inmemory database, test server, and some tinkering with code. I have nearly all of what would have been scripted or manual tests as xunit tests and it has been fantastic. However, not being able to test methods with transaction scope is a huge letdown right now. I don't care if the transaction doesn't work, but right now I can't even run that code.

@Yablargo we promoted the transaction APIs to core in our working code base. So when RC2 release you'll be able to use them with InMemory (https://github.com/aspnet/EntityFramework/issues/2891)

@rowanmiller Thanks, after trying and failing for a day to wire this in rc1, I just wrote a helper in my base dbContext that calls the transaction only when not in memory. Looking forward with the new version eagerly.

Is this already available in EF Core 1.1.0?

Yes, it was actually included in the 1.0.0 release

So if I use InMemory option I should be able to test methods that uses BeginTransactions? I'm using 1.0.0 core and I am experiencing a runtime error in using InMemory DB when testing methods with BeginTransaction. Thanks for your response!

@kimwandev can you post the exception that you are getting?

I think the exception message tells you everything you need to know 😄...

Warning as error exception for warning 'InMemoryEventId.TransactionIgnoredWarning': Transactions are not supported by the in-memory store. See http://go.microsoft.com/fwlink/?LinkId=800142. To suppress this Exception use the DbContextOptionsBuilder.ConfigureWarnings API. ConfigureWarnings can be used when overriding the DbContext.OnConfiguring method or using AddDbContext on the application service provider.

So here is what your configuration code would look like...
options
    .UseInMemoryDatabase()
    .ConfigureWarnings(w => w.Ignore(InMemoryEventId.TransactionIgnoredWarning))

@rowanmiller Thank you so much for pointing that out. I have tested it and it worked like a charm. I am going to my eye doctor after my shift and get a pair of glasses so that I can read the exceptions carefully next time 😄
Again, Thanks for your help!
Regards,
Kimwan Ogot

@rowanmiller I am a little confused about this issue. The inmemorydb is that now mainly for testing?
ms testing in-memory
But if we cannot make transactions, how can we use it for tessting e.g. our repositories that accept a transactions.
Thanks for your time

@generik0 It depends what you are trying to test. If the code is largely database agnostic and just needs to get data from somewhere, then in-memory is a good choice. If the code depends on relational concepts, like transactions, but is largely agnostic to the specific relational database engine, then SQLite in-memory might be better. If the code depends on specific functionality of the database engine, then the only option is to test with that specific engine.

@ajcvickers
Yes you are right, it depends on what is being tested!.
The issue with SQLite it that the migrations in Ef7 are database secific. So it you want to use SQLite but are actually running with e.g. MySQL you need to maintain 2 sets of migrations.
This is why the in memory is good. No migrations. The mssqllocaldb we normally use for integration tests as then we only have one migration.
The way we normally do architecture is -> cqrs handler -> repository-> dbcontext/dbaets.
This fits very good with testing.
The majority of the repository tests are fine as memory because the repository method calls have a common text or a uow passed to them. The uow we can mock so the transaction is not created, but make the mock return the dbcontext for the dbset calls.
In the cqrs handler, yes we create the uow/transaction. But just need to mock all interfaces her, so no issue here either.
My problem was,. I was doing integration tests in the cqrs handler, where a transaction was being created. The in memory is not a candidate for integration tests on this level, as transactions are not supported. It would have been nice, much faster tests, but technically not possible.

And here's a great intro on how to use SQLite in-memory for testing: Testing EF Core in Memory using SQLite.


External Player settings have no effect on embedded players

Any settings I try doesn't affect the embedded player. It's been this way for a while, at least 20 builds I'd say.
Firefox 34
YTC 456

I can confirm that the settings for the external player only work with the Userscript and not with the extension.
Tested both with
Firefox 36
YTC 471

Any fix on this?

Sorry for bumping. But it's weird this important bug didn't get any attention. Considering it's a regression I suppose it should be easy to fix?

Only if the regression was caused by YePpHa which is unlikely IMHO.
The regression was rather caused by code changes by YouTube.
I don't experience this anymore/don't know whether this bug is still in, because I switched to the userscript version -> auto update (dev version) saves time :)


Timezone Issue: DST cause items on calendar to not show in correct time slot

Do you want to request a feature or report a bug?
Bug
What's the current behavior?
If you are going to create an event via the onSelectSlot functionality on Sunday November 4th, 2018 you are going to see a 2 hour offset error.  The reason for this is because on November 4th the timezone will change from PDT to PST (daylight savings).

What's the expected behavior?
What is expected is that BigCalendar should automatically detect that on November 4th it should know to adjust from PDT to PST so it does not have a 2 hour offset.

Image:

As you can see from the example as I am creating an event right now on Sunday November 4th it will have a 2 hour offset.  The calendar is showing my 7AM - 9AM event on the 9AM - 11AM grid.

Seems like this is related to #1028

@quyphan have you seen #1044 and corresponding PR #1046? I tried to run examples with that fix and event time is the same as slot time on the left.

@anna-kartynnik Thanks for fixing this! I showed the before and after in your PR #1046


compatibility updates for sequelize >=6.2

Types compatibility updates for sequelize >=6.2

[x] Build OK
[x] Tests OK

This updates type changes introduced in sequelize >=6.2 (https://github.com/sequelize/sequelize/commit/871157bc07ccdbf2445581b8becd297ede887c22)

Pleaseeeeeeeeee @RobinBuschmann  🐒🙏🏼

@RobinBuschmann please merge if all ok...

@RobinBuschmann can you merge for next beta to let people test?
If all OK, happy to prepare PR to update README to reflect new sequelize changes and probably remote old-style scopes. They were already deprecated since v1, so I think it could be removed for v2 and sequelize v6.

 Have you considered transfer ownership to someone?
Lots of nice guys are trying to keep this project updated.

@kamilmysliwiec @BrunnerLivio @fwoelffel @jmcdo29 @johnbiundo @MarkPieszak @nartc @ThomRick  @valorkin Hello guys 👋🏼  Do you think about a fork of this project ? This package it's a big part of a good integration between NestJs and Sequelize for typescript usage 🧐 I think @RobinBuschmann no longer follows this project 🤔 There is no release since this summer (moreover it's a beta). 😞

@paolocattani  @Benjamin-Gardien I think he is still active in github, I tried reaching out through another repo...

@RobinBuschmann thank you 🎉

Sorry guy, this year was a really tough one for me and I'm afraid I won't have much time in the near future. But I'm still highly interested in keeping this project up. So help is very much appreciated. I would really like to add collaborators to this project. If anyone is interested and can show some references in open-source development..? I know that collaborators won't be enough, since sequelize-typescript runs under my npm account. So that the release process requires to be automated via github as well.

I've just released a new beta... also available with @next<EMAIL_ADDRESS>Any feedback would be very nice
Thank you @lukashroch very much for implementing. Nice work 👍

@QuanticPotatoes and me are interested to help you to maintain this project but we don't really have experience in Open source projet 😞  Only @QuanticPotatoes participated 3 years ago to an open source project. Usually, we built application for private company 🤷🏼♂  Maybe it's time for us to try ! It will be a pleasure to try to help you 😃

@QuanticPotatoes and me are interested to help you to maintain this project but we don't really have experience in Open source projet 😞 Only @QuanticPotatoes participated 3 years ago to an open source project. Usually, we built application for private company 🤷🏼♂ Maybe it's time for us to try ! It will be a pleasure to try to help you 😃

I'm also interested. I have a little of experience with open source but we are really interested in helping in the project.

@goalia @luiz290788 thanks. I think any advanced contributions to this project would be a good reference :)
@lukashroch, what about you? Are you interested in being added as a collaborator to this project?

@goalia @luiz290788 thanks. I think any advanced contributions to this project would be a good reference :)
@lukashroch, what about you? Are you interested in being added as a collaborator to this project?

@RobinBuschmann maybe we can make a call with @luiz290788 and @lukashroch and also with my co founder @QuanticPotatoes ? 😊

@RobinBuschmann maybe we can make a call with @luiz290788 and @lukashroch and also with my co founder @QuanticPotatoes ? 😊

@RobinBuschmann sure, happy to help out! 👍

@RobinBuschmann sure, happy to help out! 👍


Deploy multiple power workers on the same machine.

Is your feature request related to a problem? Please describe.
 public class OmsWorkerFileUtils {

    private static final String USER_HOME = System.getProperty("user.home", "powerjob");
    private static final String WORKER_DIR = USER_HOME + "/powerjob/";

    public static String getScriptDir() {
        return WORKER_DIR + "script/";
    }

    public static String getContainerDir() {
        return WORKER_DIR + "container/";
    }

    public static String getH2Dir() {
        return WORKER_DIR + "h2/";
    }
}

The working directory of the project is set to ${user.home}/powerjob/h2，and the database file name is always powerjob_worker_db.mv.db. So I am wondering if I deploy multiple workers for different works on the same machine, would any conflicts occur?
Describe the solution you'd like
Powerjob worker automatically reads spring.application.name from application.properties?
Describe alternatives you've considered
Generate different paths for different projects, or read path from application.properties, or set different names for different databases?
Additional context
Many thanks!

通过接入的应用名来隔离运行时文件是一个非常好的 idea！
下个版本 worker 的工作路径会改成 ~/powerjob/$appName/UUID/powerjob/h2
感谢支持，敬请期待～
（预计本周末发布新版本

UUID改成port是不是更好点： ~/powerjob/$appName/port/powerjob/h2

since v3.2.2


Add integration testing on AWS

Description
This can be done in few ways:

Use cloudtest as in monorepo https://github.com/networkservicemesh/networkservicemesh/tree/master/.cloudtest/aws.yaml. Estimation: 4d
Use github actions doing steps from https://github.com/networkservicemesh/networkservicemesh/tree/master/.cloudtest/aws.yaml. Estimation: 5d

@edwarnicke
Could you please add AWS access key?

@glazychev-art I believe I did yesterday in environment 'EKS' :)

@glazychev-art secret names:
AWS_ACCESS_KEY_ID
AWS_SECRET_ACCESS_KEY

@glazychev-art How is this going?  Is there anything blocking you?

@edwarnicke There is PR, but integration-tests failed there. The same problem as on AKS

@artem-belov Which problem?  What do we need to solve to get unblocked?

@edwarnicke I see that some tests don't work properly. It would be interesting to see aws-cluster logs.

@glazychev-art Check your email, you should now have access to the aws-cluster logs :)


Return Types Missing in "Getting Started" Sample Code

Nice project.
I was able to run the sample code in the "Getting Started" section of the ReadMe. but it has a slight bug. The two methods I need to override, OnInitialize and OnRender, need to be declared with void return types. The example as actually written won't compile.
Cheers,
Stevens

Hey, thanks for reporting this. Not sure how I missed that lol. Will fix it now.


Flask hard

Pull Request Type
What kind of change does this PR introduce?

[x] Solved an issue/task
[x] Feature

What is the current behavior?
Added blog-app to flask-hard with db configuration
Issue Number: https://github.com/nikohoffren/fork-commit-merge/issues/1050
#hacksquad

This PR contains all the code for a blog-app made using Flask

This PR contains all the code for a blog-app made using Flask

This repository has only tasks that contributors can solve, we don't really add any example code here. Most of the tasks are as simple as possible, since this repository is mostly for beginners or people who are new in open source in general. You can create a new task if you like or try to solve some of our 120 different tasks that we have so far.

Closed for extra large size and dependency review failing.


Add cask Opera Neon

Opera Neon is a new concept browser by Opera Software.

Please submit this to caskroom/homebrew-versions.

https://github.com/caskroom/homebrew-versions/pull/3129.


How to use specific property names

Hi,
According to documentation, we need to configure mongodb access with this pattern (in cypress.config.ts):
{
    module.exports = defineConfig({
        env: {
            mongodb: {
                uri: "mongodb://login:password@url",
                database: "database_name",
                collection: "collection_name"
            }
        }
    });
}

What if I want/need to use these properties which are set by CI when launching tests?
    env: {
      MONGODB_SERVER: '<url>',
      MONGODB_BASE: '<database_name>',
      MONGODB_LOGIN: '<login>',
      MONGODB_PASS: 'passW0rd',
      MONGO_COLLECTION: 'collection',

=>
      mongodb: {
        uri: 'mongodb://MONGODB_LOGIN:MONGODB_PASS@MONGODB_SERVER',
        database: 'MONGODB_BASE',
        collection: 'MONGO_COLLECTION',
      },

Thanks for your help!

You can do something like this in your cypress.config.ts file, inside the setupNodeEvents function:
setupNodeEvents(on, config) {
  config.env.mongodb = {
     uri: config.env.MONGODB_SERVER,
     database: config.env.MONGODB_BASE,
     collection: config.env.MONGO_COLLECTION
   }
  // other stuff
  return config; // don't forget to return the config object at the end
}

Adjust the uri however you want to add username and password to get the valid mongodb connection string.

Thanks @Zaista that's a good idea!


Double clicking function names no longer opens disassembly graph

In previous versions, I could open the disassembly graph for a function in linear view if I just double clicked on it.
This now no longer works. In very old versions, there used to be a context menu entry to open in disassembly graph.
I really hope this comes back, as it makes my experience worse, since my workflow is always interrupted switching views.
Binary Ninja Version: 3.0.3233 Personal, e250d0a3

So you want the graph to be the default view of you navigate no matter what view you started in? Double clicking is (and always has been) the only way to navigate around any symbols. I don't actually remember older versions switching views as we would have considered that a bug.
I don't know that we will ever implement this as I understand it in the core product but I can write a small plugin that every time you navigate if you changes locations it will automatically switch you to graph view if that's the behavior you are asking about.

What I would really like to have back is the context menu entry that just allows me to jump from linear view to graph view (or that switching the views would synchronize locations).

About how it worked previously:

You could double click the header to jump to the matching graph view previously. This replaced the context menu entry that did the same thing in version 1.0.

Oh! I understand. Just the header entry but not a function name in the listing itself. It's early for me still. Gotcha. Hmm, I would consider that. I honestly don't even remember that behavior. 😂
Thanks for clarifying.

You can also hit "space" to do the same thing but adding a context menu is definitely a good idea and  adding back double click header might be reasonable.

After some discussion we're not planning on adding double click back as a way to switch graph/linear view. I suspect if we change it to anything, it would be the "edit function properties" dialog. Hopefully the space hotkey works for you,
If you're interested in a context menu (besides the file / view/graph), let me know and I can re-open this to track that.
Alternatively, if you really want the double click feature, it's possible to build a UI plugin that adds a click handler for the graph and flowgraph views and implements that behavior yourself but it would be a bit hacky and not officially supported.

I would be totally happy with a context menu version too :)

Perhaps this is fixed in 3.0.3390-dev. If you navigate to linear view from graph view via data ref or feature map, you can scroll and double click a function header to navigate to that function in graph view. Before this behavior was present but you were required to double click the function name token. Now you can double click anywhere in the function header.

Also noteworthy, navigating to Linear/Graph view via the drop down selection in the top left pane did not used to change the graph/linear view preference. This changed around 3 months ago so that this preference is changed, similar to pressing the space bar.


mosquitto: 1.6.1 -> 1.6.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/mosquitto/versions.
meta.description for mosquitto is: '"An open source MQTT v3.1/3.1.1 broker"'.
meta.homepage for mosquitto is: '"http://mosquitto.org/"
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.6.2 with grep in /nix/store/9x2s5b1b8d5d7y64k1hdnwxziv7p64b8-mosquitto-1.6.2
directory tree listing: https://gist.github.com/4820a737f6ee561a13ddc5065bf26a1c
du listing: https://gist.github.com/a821fd6d1859c6dec9fee0b1fa56da50

Rebuild report (if merged into master) (click to expand)

14 total rebuild path(s)
4 package rebuild(s)
4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
collectd
collectd-data
haskellPackages.c-mosquitto
mosquitto

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9x2s5b1b8d5d7y64k1hdnwxziv7p64b8-mosquitto-1.6.2 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A mosquitto https://github.com/r-ryantm/nixpkgs/archive/0efd03e0defb4314547f602dec255aa0de0a3b6e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9x2s5b1b8d5d7y64k1hdnwxziv7p64b8-mosquitto-1.6.2
ls -la /nix/store/9x2s5b1b8d5d7y64k1hdnwxziv7p64b8-mosquitto-1.6.2/bin

cc @peterhoeg for testing.

@GrahamcOfBorg test mosquitto

This is a security release: http://mosquitto.org/blog/2019/04/version-1-6-2-released/


out of memory

RuntimeError: CUDA out of memory. Tried to allocate 856.00 MiB (GPU 0; 8.00 GiB total capacity; 2.96 GiB already allocated; 2.60 GiB free; 3.13 GiB reserved in total by PyTorch)
Although there is available VRAM, the system consistently displays "out of memory" error while running on Windows.

Please try reducing the batch size or image resolution, or add the code to empty the cache periodically.

Looks like an env error rather than OOM, as it was raised upon import. Please make sure your env has no problem.

I installed torch first: pip install torch==1.9.0+cu111 torchvision==0.10.0+cu111 torchaudio==0.9.0 -f https://download.pytorch.org/whl/torch_stable.html
Then proceed to install the libraries in the requirements.txt file. Not sure what's wrong with the environment.

I seem to have found the cause of the problem, because I am running under windows and can train by changing dataloader_num_worker: 5 in the configuration file to 0. But there are always warnings appearing.

This is okay, it will not affect training.


[Question] Share video from local source file or remote url instead of youtube?

Hi,
I use jitsi to talk and share screen, but I would need to share a non public video. Video could be placed at a private server or local client and should be streamed to members in the room.
Tried to do it with screensharing but chrome browser @ windows 10 died...
Second try video runs really slow.
Is it possible to stream a video by url  or from disk instead of youtube?

Please when you have problems or questions use the community forum before opening new issue.
The repository you used is the legacy Desktop client and has nothing to do with jitsi-meet.
You can open the file to be played in one chrome tabs by dropping it there and share that tab with audio.


List billing projects

Fixes #1182
Used a table per Lou's suggestion and showed it to Jerome. The long cards from groups looked odd as billing doesn't have as much information to fill up the card as the groups do.

Oh shoot--didn't realize merging Isaac's changes into mine would show us as commits/changes too...
Things of concern are adding the button to the side-nav in Top Bar and a new BillingProjectList page

I don't think it actually does anything different from isDataLoaded--I'll get rid of it. I think it might be useful later when we create/delete billing projects, but for now it isn't serving any purpose. Thanks for the catch!

You should definitely notify Comms & @bradtaylor that this is going out today, in 40 minutes!


KAFKA-5099: Replica Deletion Regression from KIP-101

Replica deletion regressed from KIP-101. Replica deletion happens when a broker receives a StopReplicaRequest with delete=true. Ever since KAFKA-1911, replica deletion has been async, meaning the broker responds with a StopReplicaResponse simply after marking the replica directory as staged for deletion. This marking happens by moving a data log directory and its contents such as /tmp/kafka-logs1/t1-0 to a marked directory like /tmp/kafka-logs1/t1-0.8c9c4c0c61c44cc59ebeb00075a2a07f-delete, acting as a soft-delete. A scheduled thread later actually deletes the data. It appears that the regression occurs while the scheduled thread is actually trying to delete the data, which means the controller considers operations such as partition reassignment and topic deletion complete. But if you look at the log4j logs and data logs, you'll find that the soft-deleted data logs actually won't get deleted.
The bug is that upon log deletion, we attempt to flush the LeaderEpochFileCache to the original file location instead of the moved file location. Restarting the broker actually allows for the soft-deleted directories to get deleted.
This patch avoids the issue by simply not flushing the LeaderEpochFileCache upon log deletion.

Thanks for the PR, can we add a test for this? This bug showed an important gap in our tests.

@onurkaraman : Thanks for the patch. LGTM


run command

Please mention how to run this project.

@kakhilkrishnan, please take a look at the documentation.


example for updating trigger

Hi all,
I'm trying to enable/disable a specific trigger for one task. Does anybody have an example how to do this?
TIA
Originally posted: 2012-02-22T19:11:56

using (TaskService ts = new TaskService())
{
Task t = ts.GetTask(taskName);
td = t.Definition;
td.Triggers[0].Enabled = false;
t.RegisterChanges();
}
Originally posted: 2012-02-22T23:12:49

thanks.
Originally posted: 2012-02-23T06:32:48


iPython REPL will not start

I am new to Python and Visual Studio and having issues getting IPython to work (I need to be able interactively plot data using matplotlib).
I've gone into Python Tools\Interactive Windows\Interactive Mode and set it to IPython.  Once I start typing in the REPL I get the following error:
Traceback (most recent call last):
File "C:\Program Files (x86)\Microsoft Visual Studio 12.0\Common7\IDE\Extensions\Microsoft\Python Tools for Visual Studio\2.2\visualstudio_py_repl.py", line 1370, in
_run_repl()
File "C:\Program Files (x86)\Microsoft Visual Studio 12.0\Common7\IDE\Extensions\Microsoft\Python Tools for Visual Studio\2.2\visualstudio_py_repl.py", line 1366, in _run_repl
BACKEND.execution_loop()
File "C:\Program Files (x86)\Microsoft Visual Studio 12.0\Common7\IDE\Extensions\Microsoft\Python Tools for Visual Studio\2.2\visualstudio_ipython_repl.py", line 308, in execution_loop
self.execute_file_as_main(self.launch_file, None)
File "C:\Program Files (x86)\Microsoft Visual Studio 12.0\Common7\IDE\Extensions\Microsoft\Python Tools for Visual Studio\2.2\visualstudio_ipython_repl.py", line 291, in execute_file_as_main
contents = f.read().replace("\r\n", "\n")
TypeError: expected bytes, bytearray or buffer compatible object
The Python REPL process has exited
I'm using Visual Studio Pro 2013 with PTVS 2.2
Python 3.4
IPtython 3.0
Any insight would be appreciated.

Wow, this bug has been here for years... thanks for finding it!
Our next release (coming very soon) isn't going to have the fix, but once I've written one up you'll be able to apply it manually to C:\Program Files (x86)\Microsoft Visual Studio 12.0\Common7\IDE\Extensions\Microsoft\Python Tools for Visual Studio\2.2\visualstudio_ipython_repl.py.

Just to confirm - in Interactive Options, what value (or maybe just spaces?) do you have for Startup Script? The error you're hitting is for when a startup script is specified, which is apparently not a very common thing for people to do, but I get a different error when I put a Python file there.

I do use a startup script (something I'm used to coming from MATLAB).  Maybe I shouldn't?  This is what was in there.
C:\Users\vu062605\Documents\Visual Studio 2013\Projects\TunnelTools\TunnelTools\setupinteractive.py
I removed the startup script and sure enough, the REPL seems to work in IPython mode.  I was hoping switching to IPython mode would fix my actual issue (which is wanting to be able to make plots from matplotlib without each plot window hanging up and "Not Responding").  But that's a question for a different thread I  think.

Not using the startup script is a workaround, as the bug is in our code that loads the startup script. I'd rather fix the bug than tell you not to use it though.
The matplotlib issue should be helped by changing to IPython mode. The main difference is that matplotlib will use a different backend:
In [2]: matplotlib.get_backend()
Out[2]: 'module://IPython.kernel.zmq.pylab.backend_inline'

vs normal mode:
>>> matplotlib.get_backend()
'TkAgg'

The TkAgg backend is the one that pops up a separate window. You can specify the backend using matplotlib.use(), but I don't think we'll display plots properly unless you're in IPython mode.

So if you click through to the commit linked above, you'll see the changes you can make to your own install to get the fix right away.

Thanks!  That fixed the REPL issue.  I still can't get matplotlib to create a window though.
In [1]: plt.get_backend()
Out[1]: 'Qt4Agg'

In [2]: plt.ion()

In [3]: plt.plot([1, 1])
Out[3]: [<matplotlib.lines.Line2D at 0x8142b50>]

I get the same backend in standard mode
>>> plt.get_backend()
'Qt4Agg'

Should that be the case?

I figured out if I set Interactive Mode to IPython w/o Pylab, then enter
>>> %matplotlib

as the first thing I do (before the REPL has started), everything works as expected.  I'm not sure what that does exactly but I'm running now.
Thanks again for your help and looking at this.  I was beginning to start looking at other IDEs.


Issues Implementing $Metadata in Get-SecretInfo

When trying to call the following in the extension I am developing
$vault_metadata= $(Invoke-VaultAPIQuery -VaultName $VaultName -SecretName $PSItem).data.metadata
$Metadata = [Ordered]@{}
$vault_metadata.psobject.properties | ForEach-Object { $Metadata[$PSItem.Name] = $PSItem.Value }

[Microsoft.PowerShell.SecretManagement.SecretInformation]::new(
    "$PSItem",
    "String",
    $VaultName,
    $Metadata)

I receive the following error:
Exception             : System.Management.Automation.MethodException: Cannot find an overload for "new" and the argument count: "4".
                         at CallSite.Target(Closure , CallSite , Type , String , String , Object , Object )
                         at System.Dynamic.UpdateDelegates.UpdateAndExecute5[T0,T1,T2,T3,T4,TRet](CallSite site, T0 arg0, T1 arg1, T2 arg2, T3 arg3, T4 arg4)
                         at System.Management.Automation.Interpreter.DynamicInstruction`6.Run(InterpretedFrame frame)
                         at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
TargetObject          :
CategoryInfo          : NotSpecified: (:) [], MethodException
FullyQualifiedErrorId : MethodCountCouldNotFindBest
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo

Reading the code here in SecretManagement.KeePass.Extension from @JustinGrote it appears that I need to do a bit more with the Hashtable before passing it in as Metadata.

I have a convertto-readonlydictionary function in my modules that you can use for the hash table conversion

Thanks. I saw that. Was just hoping that adding the metadata would be a little easier to implement, and the error made me think that it didn't take 4 arguments.

@joshcorr you can always do new without the parens to see all the potential constructors.
I agree this is not ideal and have asked @PaulHigin to adopt a hashtable constructor in https://github.com/PowerShell/SecretManagement/issues/108

Hopefully this changes. For the time being I went ahead and used your code here and here

Yes, I agree and will add the new constructor.  I'll close this as a duplicate to #108 .


Zooming map?

Is it possible to zoom a map?
Ive tried playing with map_scale parameter, setting it to 1.5 or 2 but it doesnt zoom.

map_scale should zoom. Could you paste your whole configuration? Maybe you didn't indent it correctly?

You're right, map_scale was intended to crop section. Thanks.
On another subject if i may, i have adjusted margins with crop top, left and map scale, but im getting a lot of white space underneath. When im trying to cut it off by increasing crop bottom, the whole image moves top and left, instead of whitespace getting cut.

Yeah my cropping code kinda sucks, need someone to help improve that once :(

Duplicate of https://github.com/TheLastProject/lovelace-valetudo-map-card/issues/3 i believe?

Depends, do you also use rotation? If so, yes


enable/disable inertia in runtime

I haven't found a way to enable/disable inertia (and other options) in runtime.
I suppose something like
map.inertia.enable()
map.inertia.disable()
Is this easy to be added?

Hi and thanks for bringing this question up. It seems reasonable that inertia should be possible to enable or disable at runtime.
However, you've opened a pull request, which is something you do if you want to contribute code (ask us to pull your code into our code). You should open an issue instead. You can of course also open a pull request that actually fixes this.
Closing this for now.


[Improvement] Prompt configuration when starting engine timeout

Code of Conduct

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

Search before asking

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

What would you like to be improved?
Now Spark Engine start timeout, usually in the YARN UI will be prompted configuration, but the user see is the server throws an exception without prompting configuration.
https://github.com/apache/incubator-kyuubi/blob/3bc299d01e1399d6e82cad00419b1c647f9c1915/externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/SparkSQLEngine.scala#L270-L272
https://github.com/apache/incubator-kyuubi/blob/3bc299d01e1399d6e82cad00419b1c647f9c1915/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala#L208-L210
How should we improve?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

I want to implement it please assign me

I want to implement it please assign me

Thanks, assigned.


Email notification doesn't work

Email notifications aren't being sent, and I'm seeing this in the logs:
2016-03-23 21:26:03,261 ERROR: u'email-notification' [in /opt/lemur/lemur/common/utils.py:60]
Traceback (most recent call last):
  File "/opt/lemur/lemur/common/utils.py", line 46, in wrapper
    resp = f(*args, **kwargs)
  File "/opt/lemur/lemur/plugins/views.py", line 129, in get
    return plugins.get(name)
  File "/opt/lemur/lemur/plugins/base/manager.py", line 37, in get
    raise KeyError(slug)
KeyError: u'email-notification'

Are you on the latest version? Typically this line would ensure that the email plugin is at least available.
https://github.com/Netflix/lemur/blob/master/setup.py#L167

Using revision 0a58e106b5318ded0da7e3007205e4183e387bb3. Think upgrading will fix this?

I am not sure, that is the latest release (0.2.2). Do you have that line in your setup.py? You could also try running python setup.py install again. Basically Lemur can't find that plugin for some reason, make sure it exists under lemur/plugins/lemur_email.

That line does exist in my setup.py and lemur/plugins/lemur_email exists as well.

The other thing that I have seen cause this is that somehow the plugin is unable to load because of syntax error, when Lemur starts up do you get any messages about "Failed to load plugin X"?
 Right under those lines there should be a stack-trace explaining whats going on.

You really know your stuff. The stack traces revealed that future wasn't installed. Thanks!


Controll Power Mode

I use the Grainfather for distilling, is there a command to control the power mode i.e increase/decrease the power?
I know you can switch to power mode.
Thanks for the commands I've been looking for this for ages

Hi, I don't think what you're asking is physically possible. I believe the grainfather controller simply turns the power to the pump/heating element on or off.
What would you like to achieve by manually controlling the 'power'?
If it's to regulate the temperature you could just write a simple program that reads the current temperature and turn the heating element based on/off that input.
If it's to slow the pump down, I don't think that's possible unfortunately.
Do correct me if I'm wrong or misunderstood your question. It's still a mystery to me what the developers mean by 'power mode' tbh, I can only guess.


UI enhancements to the login page

If the gradient icon logo does not look good on the background, use the white version of the logo.

2. Clicking on an input fields displaces it by a pixel (due to getting a border). Fix that.
3. Add a hover state to the login button, currently it does not respond to it.

@shivankacker I want to work on this?


rename cli

rename cli to tdbpy
closes #310

The original issue was https://github.com/terminusdb/terminusdb-client-python/issues/242.

@Neelterminusdb will review this.

Looks good to me @spl I tried using the tdbpy locally but was unable to do it as i didnt know how one can debug python client locally using pip.

Yes i have a old version of python client thats why i was not able to use tdbpy. Great that you have checked it and is working


test deserialize exact Theta sketch from Java

added missing test

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.253%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2203

Relevant Lines:
2388

💛  - Coveralls


Error on repeated Sortable.create call

First I fill my ul with li elements and make them sortable by calling createSortable();
function createSortable() {
        sortableFrom = Sortable.create($('#nav-from-ul')[0], {
            ghostClass: "ghost",
            onStart: function () { ...
            },
            onEnd: function (evt:any) { ...
            }
        });
}

Everything works as expected.
When I don't need the list I get rid of it by calling first
sortableFrom.destroy();

then
$('#nav-from-ul').empty();

Then when I need another sortable list I fill the ul with another li elements and call createSortable() I get this error:
Uncaught TypeError: Cannot set property 'Sortable1436963789329' of null
Sortable.js 843

Line 843 is el[expando] = null;
Help me, Obi-Wan Kenobi, you are my only hope.
Thanks

PS: version 1.2.0

Create example on jsbin.com (draft)

I had this problem too. For me, I was calling
Sortable.create( someElement )
where someElement was actually null. I did a check for null before making the Sortable.create() call and it's fixed


I want to make the reference easier to read

API on one page.

:tada: This issue has been resolved in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Snyk] Security upgrade zipp from 3.15.0 to 3.19.1

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

requirements.txt

⚠ Warning
```
pyOpenSSL 23.2.0 has requirement cryptography!=40.0.0,!=40.0.1,<42,>=38.0.0, but you have cryptography 42.0.8.
</details>

---

> [!IMPORTANT]
>
> - Check the changes in this PR to ensure they won't cause issues with your project.
> - Max score is 1000. Note that the real score may have changed since the PR was raised.
> - This PR was automatically created by Snyk using the credentials of a real user.
> - Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

---

**Note:** _You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs._

For more information: <img src="https://api.segment.io/v1/pixel/track?data=eyJ3cml0ZUtleSI6InJyWmxZcEdHY2RyTHZsb0lYd0dUcVg4WkFRTnNCOUEwIiwiYW5vbnltb3VzSWQiOiIyOTFmZjE0ZC1jYzE1LTRiNDctYjBhZi1hMjFhMGM1MmNjYjYiLCJldmVudCI6IlBSIHZpZXdlZCIsInByb3BlcnRpZXMiOnsicHJJZCI6IjI5MWZmMTRkLWNjMTUtNGI0Ny1iMGFmLWEyMWEwYzUyY2NiNiJ9fQ==" width="0" height="0"/>
🧐 [View latest project report](https://app.snyk.io/org/sam.stepanyan/project/2595930d-09d3-4614-ace4-800cf431e230?utm_source=github&utm_medium=referral&page=fix-pr)
📜 [Customise PR templates](https://docs.snyk.io/scan-using-snyk/pull-requests/snyk-fix-pull-or-merge-requests/customize-pr-templates)
🛠 [Adjust project settings](https://app.snyk.io/org/sam.stepanyan/project/2595930d-09d3-4614-ace4-800cf431e230?utm_source=github&utm_medium=referral&page=fix-pr/settings)
📚 [Read about Snyk's upgrade logic](https://support.snyk.io/hc/en-us/articles/360003891078-Snyk-patches-to-fix-vulnerabilities)

---

**Learn how to fix vulnerabilities with free interactive lessons:**

🦉 [Learn about vulnerability in an interactive lesson of Snyk Learn.](https://learn.snyk.io/?loc=fix-pr)

[//]: # 'snyk:metadata:{"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"zipp","from":"3.15.0","to":"3.19.1"}],"env":"prod","issuesToFix":[{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-ZIPP-7430899","priority_score":738,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.9","score":345},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Infinite loop"}],"prId":"291ff14d-cc15-4b47-b0af-a21a0c52ccb6","prPublicId":"291ff14d-cc15-4b47-b0af-a21a0c52ccb6","packageManager":"pip","priorityScoreList":[738],"projectPublicId":"2595930d-09d3-4614-ace4-800cf431e230","projectUrl":"https://app.snyk.io/org/sam.stepanyan/project/2595930d-09d3-4614-ace4-800cf431e230?utm_source=github&utm_medium=referral&page=fix-pr","prType":"fix","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":["updated-fix-title","pr-warning-shown","priorityScore"],"type":"auto","upgrade":[],"vulns":["SNYK-PYTHON-ZIPP-7430899"],"patch":[],"isBreakingChange":false,"remediationStrategy":"vuln"}'

Fixed via poetry dependencies.


add argument to Toggle ON the "Automatically hide the taskbar" setting

add argument to Toggle ON the "Automatically hide the taskbar" setting like -v,-x,-h

hi @direrFlood is this different from #2?


Auto fetch competition start points

Suggestion:
Automatically fetch starting data points for all members at the beginning of a competition to ensure proper tracking. This fetch would bypass the 1h update cooldown so that everybody has a starting data point during the competition.

Duplicate of #126


Easy way to have the last step appear at end of stepper?

The above illustrates what I'd like to achieve. I'd like the third step to be at the end of the stepper line, and the second step re-centered to be in the middle. Any easy way to customize the stepper to achieve this?

I'm afraid not, but that will change soon. I'm moving to provide/inject pattern which allows much better composition.

Looks like this wasn't too hard with scss:
.v-step{
  //reposition last step to be at the end
  &:last-child{
    flex-grow: 0.2;

    label{
      width:0;
    }
  }
}

I figured what CSS changes have to made to achieve the look you were after, but I didn't suggest them since the CSS is scoped by design. If you think about it, CSS is an implementation detail that can break on any version update. Therefore, I scoped it and planned to allow overriding it by API (props in this case).
It seems however you are in luck, since my CSS is flat (no nesting) and specificity is very low, thus easy to override.
.v-step {
  flex-grow: 0;

  .divider {
    display: none;
  }
}

That should be the best solution. If you set width: 0 to label, it collapses its width and it becomes weird to follow when you inspect elements.

@adi518 thanks for the tip! Yes, the divider display:none works better-


Smart/Motion Smoothing ratio doesn't "recover" as easily as with Steam VR.

I've noted that the threshold for returning to 1:2 motion smoothing doesn't occur as easily with PimaxXR. This may be due to increased overhead. But for example if I pause a scene at 42 FPS will go to 25 FPS (1:3 MS); then it will return to 1:3 only when my frametimes are well above the 1:2 threshold (somewhere between 50-55FPS); then it releases to 1:2 MS (37.5FPS), whereas with SteamVR I can do that same at 42FPS, and although it first goes to 1:3 for a few seconds (which I would think it shouldn't), it recovers after 1-3 seconds to 1:2. On the other hand, SteamVR will get tripped back to 1:3 every so often as it only takes a single frame-time below the 37.5 fps equivalent to trip back into 1:3 mode for a few seconds. (this is a "bouncing" MS issue I have reported to Pimax, requesting a way to disable 1:3 as was true in their old versions of Pitool before .271)

Futhermore, the 1:3 motion smoothing appears to get tripped as well to 1:3 at a threshold of ~50, such that it's a difference in thresholds altogether it appears.

After some more testing It appears that in 90Hz mode the threshold to recover/fall from/to 1:3 to 1:2 is at 60FPS. So there's a bit of trend where MS thresholds are at 2/3 Hz. I initially thought it was MS overhead, but perhaps it's a miscalculation?
This begs a question: Everytime MS is tripped in either SteamVR and PimaxXR. it goes to 1:3 MS first. Can that be avoided?

I submit frames as fast as the Pimax API and the game allow me to. I have no control over any of the things you've described unfortunately.
I'm going to continue looking into why we're getting 1/3 when we should be getting 1/2, but I'm not super hopeful that this is something I can control.

Ok Probably is a Pitool error, which I've reported to them. It's so nuanced and complex that I'm sure it won't be taken seriously hahah. But I gotta try. Meanhwile I'm awaiting a 120hz update so that at least if I do get 1:3 it's at 40 fps instead of 25.

The Smart Smoothing issues will be tracked under #12.


Viewport following the current line when stepping

Usually when stepping, the viewport doesn't shift unless it needs to, like when the current line runs off the bottom. But when stepping through some scripts, it shifts on every step. I think this is for scripts that vscode doesn't have on disk (that had their source provided by the debug adapter) but I don't know why that would make a difference.
Good:

Bad:

This is a blast from the past. I remember I was looking into this, but forgot what is the root cause - setting to september to investigate.

This is happening because the reference of those source files changed between steps. Namely the debug adapter sends a brand new source reference between steps. Due to that on the vscode side we think those files are different and we do this 'bad stepping'.
Assigning to @weinand to investigate if node adapter could preserve the reference between steps for internal modules since we use those as an identifier on the vsocde side

is this with node-debug2?

I see it with node-debug.
If it is also happening with node-debug2 @roblourens can comment

Oh that's right, I'm surprised I never noticed that before...

+1, makes it super hard to follow steps in an internal node module.

This has a more serious reprocution of #13753, due to that assigning to October

Fixed in node-debug2

It would be awesome if we fix this also in node-debug as it makes the debug experience in internal modules quite poor.


when toggle and config?

Please just tell us if you're busy right now and can't do make the toggle because right now it seems like we're being ignored

added


Remove readme-symlink.md.

This was causing lots of warnings during the sphinx build. Adding a title didn't have the desired effect, either.
zulip\docs\index.rst:26: WARNING: toctree contains
reference to document 'readme-symlink' that doesn't have a title: no link will be ge
nerated

https://zulip.readthedocs.io/en/latest/ shows the same behaviour.

Automated message from Dropbox CLA bot
@rbcarson, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

So the intent of this symlink setup is to make the root README.md displayed on GitHub be the same as the README page on ReadTheDocs; I think if we remove this, we would break that functionality.  Would love to find another way to fix the warnings, though!  It does appear to work as  http://zulip.readthedocs.io/en/latest/readme-symlink.html right now...

I haven't figured out how to make the ToC link appear. Switching the symlink to RST and providing a title works correctly, except that it looks like README.md is then interpreted as RST. Trying to provide a title for the Markdown symlink file breaks the symlink behaviour.

@rbcarson I think #905 fixed the main issue we were having with with that symlink.  Is there any other problem you see other than warnings in the output of the makefile?

I just updated, and I think the problem is that on Windows, the file is literally a text file containing '../README.md', which is taken literally by Sphinx. I haven't been able to find a solution, unfortunately.


[Bug] SelectableNavigator - shouldShowMenuForSelection is not disabling "Copy Link with Highlight" and "Writing Tools"

Describe the bug
Hey just want to first say I really appreciate Readium and all the support you provide for it!
In iOS 18.2, highlighting a section in the book still shows the "Copy Link with Highlight" and "Writing Tools" even when shouldShowMenuForSelection is set to false:
func navigator(_ navigator: SelectableNavigator, shouldShowMenuForSelection selection: Selection) -> Bool {
    return false
}

Any help would be greatly appreciated.
How to reproduce?

Open book
Highlight selection

Readium version
3.0.0-beta.1
OS version
18.2
Testing device
iPad Pro (M1)
Environment
macOS: 14.5
platform: arm64
carthage: N/A
Xcode 16.1
Build version 16B40

Additional context
No response

I believe this is an issue in iOS 18.2 beta, as we cannot disable this action by overriding WKWebAction.canPerformAction(). I opened a bug report issue on https://feedbackassistant.apple.com/ and I urge anybody reading to do the same to make sure they address this issue before iOS 18.2 is released.

Hi Mickael,
It seems despite the bug reports we submitted, we are still unable to override the actions.
Is there any workaround to disable these actions?
Thanks

I got unexpectedly hit with this after updating to the released iOS 18.2. What I found after a day of digging into this is that the problem is that on iOS 18.2, the function buildMenu(with builder: UIMenuBuilder) is straight up not getting called anymore, and this is ultimately what is responsible for supressing these other edit actions. However it seems that if you implement the same method on the WebView itself, it will get called, so simply adding the following to WebView seems to fix the issue for me:
    @available(iOS 13.0, *)
    override func buildMenu(with builder: UIMenuBuilder) {
        editingActions.buildMenu(with: builder)
        super.buildMenu(with: builder)
    }

@DetachedHeadState That fixed it for me.
Thank you so much for the insight!

@DetachedHeadState Thanks for investigating that, is it really all you changed? It doesn't work for me, and editingActions.buildMenu() is actually only removing specific actions, so that makes sense.
https://github.com/readium/swift-toolkit/blob/36f4151af090fe879e015213a9975fb13dde81bb/Sources/Navigator/EditingAction.swift#L155-L165
It does remove the item if we don't call super.buildMenu(), but that might have other unintended consequences.

I believe that Apple is adding this item with a generated identifier com.apple.menu.dynamic.EF7A91EE-DD02-42FF-A27E-42455C8C2F1A, which is... really unfortunate. Maybe the answer is to really prevent super in this case.
(lldb) po builder.menu(for: .root)?.children
▿ Optional<Array<UIMenuElement>>
  ▿ some : 9 elements
    - 0 : <UIMenu: 0x600001dd1540; identifier = com.apple.menu.standard-edit; options = (Inline); preferredElementSize = medium; children = <NSArray: 0x600003693c80>>
    - 1 : <UIMenu: 0x600001dd19a0; identifier = com.apple.menu.replace; options = (Inline); children = <NSArray: 0x600003692f00>>
    - 2 : <UIMenu: 0x600001dd1a40; identifier = com.apple.menu.open; options = (Inline); children = <NSArray: 0x104392eb0>>
    - 3 : <UIMenu: 0x600001dd1c20; title = Format; identifier = com.apple.menu.format; image = <UIImage: 0x600001e957a0; symbol = (bold.italic.underline)>; children = <NSArray: 0x6000023ca040>>
    - 4 : <UIMenu: 0x600001dd1cc0; identifier = com.apple.menu.lookup; options = (Inline); children = <NSArray: 0x6000023ca100>>
    - 5 : <UIMenu: 0x600001dd1d60; identifier = com.apple.menu.learn; options = (Inline); children = <NSArray: 0x600002e87dc0>>
    - 6 : <UIMenu: 0x600001dd1e00; title = Speech; identifier = com.apple.command.speech; options = (Inline); children = <NSArray: 0x6000023ca0d0>>
    - 7 : <UIMenu: 0x600001d94000; identifier = com.apple.menu.dynamic.EF7A91EE-DD02-42FF-A27E-42455C8C2F1A; options = (Inline); children = <NSArray: 0x600002e9c1e0>>
    - 8 : <UIMenu: 0x600001dd1ea0; identifier = com.apple.menu.share; options = (Inline); children = <NSArray: 0x600002e87ce0>>

I do have some other modifications to the EditingActions.buildMenu code but I don't think they're related, if I comment them all out that menu item still doesn't show up. I also built a small sample app that just uses a webview directly and was able to reproduce the behavior there. From what I can see, the line that gets rid of it is builder.remove(menu: .share). I guess if you have other items that you want from that menu then you might still be in trouble. For me at least I don't want any of the built in actions and just want my own actions, so this fix works for me.

Right, I confirm that the "Copy with Highlight" is removed with .share. This is unfortunate as you might want to keep "Share..." but not the "Copy with Highlight" item.
For now, I think I'll add the fix in WebView and indicate that "Share" and "Copy with Highlight" are synonymous.