use group matched in resourceQuery

Feature request
In rule definition when using resourceQuery it's a good idea to have ability to use matched group and queryParams

What is the expected behavior?
  resourceQuery: /(color-.*)/
  generator: {
    dataUrl: (content, meta) => {
         // somehow access matched query and use given color in creating dataUrl
    }
  }

What is motivation or use case for adding/changing the behavior?
when using mini-svg-data-uri to creating dataURI for svgs you can use queryParam to give color and set it in dataURI and having mulltiple colors of a same svg icon
                    generator: {
                        dataUrl: (content, meta) => {
                            content = content.toString().replace(/currentColor/g, meta.matchedQuery /* example red or #fc2*/);
                            return svgToMiniDataURI(content);
                        },
                    },

How should this be implemented in your opinion?
Are you willing to work on this yourself?
no

You can setup this on rules level using matchedQuery

Best use a custom loader for that. Inside of the loader you can access this.resourceQuery

Answered, feel free to feedback


man: Add cmake build rule for adoc based man page

We need to generate the man page so users can run man on it.

Checklist

[ ] Language changes are updated in docs/reference_guide.md
[x] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

thanks!


KOGITO-9691 Create SonataFlow operator BDD tests for devMode

JIRA: KOGITO-9691
This is a rewrite of the devMode end-to-end test to BDD test framework. Currently it lacks 2 things:

OpenShift integration hasn't been tested yet
Automatic execution in the PR checks, this will also provide a separate Makefile for easier execution

Those will be added shortly, not necessarily in this PR, but we will see.

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

@radtriste @ricardozanini
I have exported the framework including the whole Git history into a separate module in the /bddframework directory. History is relevant just for the framework and some other parts, which I included in it.
What I did was:

Added BDD Framework.
Added kogito-operator kubernetes client which you @ricardozanini did for the Kogito operator. The framework heavily uses it so I decided to make it a part of the framework.
I decided to retain some types like KogitoJobsService or KogitoDataIndex as they may be easily reused in the future should we need them.
There is no longer a dependency on the kogito-operator repository.
Added relevant hack scripts from the kogito-operator repo which are used by the tests.
Added separate Makefiles for both the BDD tests (testbdd) and BDD Framework (bddframework).

What is remaining is to update the documentation and include this in a PR check. I have tested the scenario on Minikube and OpenShift already.
Thanks for the attention!

Marking it as unread, I'll carefully review it tomorrow. It's a waaay long PR :D


[BUG] There are 2 bugs in Contributing.md

Describe the bug

Step of  "cd [name of folder] " is missing before step 4 of adding upstream
Instead "git pull upstream main" --> "git pull upstream master" in step 5

Please assign me this issue

@keshavgbpecdelhi  Please assign me this issue

Done

Closing wrt #104


Prometheus metric is not send

The Prometheus metric is not send when backup is failed. Value 0 must to be sent.

This is a slightly different problem, if a backup job has not been initialized, it is not reflected in the metrics in any way and no metrics appear for that job at all.


Camel K 2.1 documentation still contains information on using local files for runtime resources despite it is no more supported

What happened?
While deploying with resource which are not config or secret file
the deployment is failing
with error:
Error: could not match config or secret configuration as file:/home/apupier/ws/single-file-without-extension-workspace/demo.camel.xml
The documentation is stating: The runtime resources are expected to be any resource type (text or binary content). https://camel.apache.org/camel-k/2.0.x/configuration/runtime-resources.html
Steps to reproduce

Create a Camel K integration file
Create a random file (which is not a config or secret file) like an xml file
try to deploy the integration with a command like: kamel run Demo.java --resource=file:/path/to/the/file.xml

Relevant log output
`Error: could not match config or secret configuration as file:/home/apupier/ws/single-file-without-extension-workspace/demo.camel.xml`

Camel K version
2.1.0

Hi,
The change has been made in Camel K 2.0 when deprecated kamel command have been removed : CAMEL K 2.0. I you want to provide a file you need to store it in a Configmap or Secret. This is why you no longer see any reference to "file:xxxx" in the documentation you are looking at.

I think the documentation needs a major update as it is still mentioning that the "most familiar way is to handle local file".

@squakez I will fix the doc.

Oh man. My routes use some json and velocity (.vm) files to format responses. What would be the easiest way for my Camel-K routes to reference these files?

Oh man. My routes use some json and velocity (.vm) files to format responses. What would be the easiest way for my Camel-K routes to reference these files?

Not sure to understand how this comment relates to this specific issue. Would you mind opening a separate issue with all the details about your request? thanks.


Allow passing a URL and streaming an image to the drive.

This is a major feature that needs to re-architect the code to pass a readable stream directly instead of a file name to the writer first.

Implement UI to be able to paste a URL as the first step.
Handle network error conditions gracefully.
Handle invalid URLs or other invalid condition.

In Etcher, we want to allow the user to pass a URL to an image hosted on the internet, and stream directly to the device. The scope of this project is to implement a new function in etcher-image-stream called “.getFromURL()” that sends an HTTP request to the provided endpoint and makes use of the provided response headers to determine the image size, format, compression, etc to return all the needed information.
If the image is compressed, the function should return the appropriate transform stream that can deal with it (see examples on the same modules).
This functionality should be tested by an end to end suite that starts a local HTTP server to serve example images, and uses those endpoints to write the image to a temporary location, and assert that the image was written and decompressed correctly (if needed).

For v1 we are ok with two stage download and write, right?

For v1 we are ok with two stage download and write, right?

Yeah, we'll do first download, then write, and we'll improve in the
future.
On Thu, Dec 01, 2016 at 08:08:46PM -0800, Alexandros Marinos wrote:

For v1 we are ok with two stage download and write, right?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/resin-io/etcher/issues/54#issuecomment-264368920

--
Juan Cruz Viotti
Software Engineer

Following #975 concept, would it be possible to have a widget on publisher's webpage that will preload Etcher with a specific image? For instance, on Raspberry Pi's Rasbian page, there could be 3 CTA's: Download Torrent Download Zip Flash with Etcher or Load to Etcher.

 image

i guess a widget with a dual call to action -> "download including Etcher"
and "open with etcher" (better names needed) would be the ultimate nirvana
for several of our publisher integration efforts. I think there are several
background pieces needed to be aligned, as there are binary signing issues
to sort out, but I love the idea and want to figure out what's needed to
get there.
--
Alexandros Marinos
Founder & CEO, Resin.io
+1<PHONE_NUMBER>
@alexandrosm
On Thu, Dec 29, 2016 at 11:20 AM, Konstantinos Mouzakis <
<EMAIL_ADDRESS>wrote:

Following #975 https://github.com/resin-io/etcher/issues/975 concept,
would it be possible to have a widget on publisher's webpage that will
preload Etcher with a specific image? For instance, on Raspberry Pi's
Rasbian page, there could be 3 CTA's: Download Torrent Download Zip Flash
with Etcher or Load to Etcher.
[image: screen shot 2016-12-29 at 11 16 55]
https://cloud.githubusercontent.com/assets/15323961/21542776/5a2bd9e6-cdb8-11e6-9821-f41b8947bb1a.png
image
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/resin-io/etcher/issues/54#issuecomment-269616312, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABLUCKDzqUt1_fHHMwtJBDfMouGeNL7Uks5rM5dxgaJpZM4GreEH
.

Exactly, pretty much the same way torrents and magnets work.

We should re-use https://github.com/zenflow/node-fast-download for this task:

node module and command-line program for accelerated (multiple connections) http download Stream

Given the current plans for v2, should this issue be closed?

Yeah, lets do it. Given the complexity of streaming any kind of images, we decided to only support streaming on our internal image catalog.


Add non-hand-drawn figures to Chapters 1-10

These are all the non-hand-drawn figures for Chapters 1-10. The hand-drawn ones, annoyingly, require me to upgrade to macOS 14 (what?!) but they are all drawn.
Also deleted one obsolete figure that we no longer use.

Let me also note that the way images look on the web version is currently not 100% great but I think that can be fixed in, like, a week. The images also don't look amazing in the LaTeX version but a lot of that will be up to the typesetter anyway.


Minute bundles sometimes missing first few bars

Run this unit test:
tests/exchange/test_bundle.py:457
With these parameters:
        exchange_name = 'bitfinex'
        data_frequency = 'minute'
        period = '2017-06'
        symbol = 'etc_btc'

Notice that the first minute of the bundle is missing:
  | close | high | low | open | volume
-- | -- | -- | -- | -- | --
2017-06-01 00:00:00+00:00 |   |   |   |   | 0
2017-06-01 00:01:00+00:00 | 0.09981799 | 0.09981799 | 0.09981799 | 0.09981799 | 2.25752429
2017-06-01 00:02:00+00:00 | 0.09981799 | 0.09981799 | 0.09981799 | 0.09981799 | 0
2017-06-01 00:03:00+00:00 | 0.0993 | 0.0993 | 0.0993 | 0.0993 | 1621.348101
2017-06-01 00:04:00+00:00 | 0.098705 | 0.0993 | 0.098593 | 0.0993 | 1790.962076
2017-06-01 00:05:00+00:00 | 0.099 | 0.099 | 0.098578 | 0.098705 |<PHONE_NUMBER>

I suspect that it's because there is not volume in this particular minute. Since it's at the beginning of the bundle, Pandas cannot ffill the data. A solution might be to make the make the start date earlier by a first periods, then slice out the trailing data.

The server-side scripts that process the data bundles have been updated to properly handle these cases, and data bundles have been re-done. @fredfortier would you like to test it again?
There is something wrong with the following few bundles, which are mostly empty, but that is probably a different issue. For reference:
bitfinex-minute-eth_usd-2016-05
bitfinex-minute-eth_usd-2016-06
bitfinex-minute-ltc_usd-2015-03
bitfinex-minute-ltc_usd-2016-06
bitfinex-minute-ltc_btc-2015-06
bitfinex-minute-ltc_btc-2016-06
bitfinex-minute-ltc_btc-2015-03

@lacabra I believe that this is resolve. Can you confirm?


BE: add fields in UserWithAnalyticsResponse

BE: change mock to real data in UserWithAnalyticsResponse
workDayByLastMonth - fill with real data
firstVisit - add and fill
lastVisit - add and fill

@ElenaSpb
I'd like to make sure: firstVisit means time of first login or time when user made his first exercise?

Thank you for contribution!
great start


⚠ Bofrost.nl has degraded performance

In bf25d4c, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 2463 ms

Resolved: Bofrost.nl performance has improved in 5807c8b.


Damage on full life support and ranged skills

Damage on full life support now works with ranged skills. It doesn't seem to work with the current PoB.

It does work in PoB, but you need to enable the "Are you always on Full Life?" option in the Configuration tab for it to take effect. Note that like most options, it doesn't appear unless enabling it would have some effect, which in this case requires linking Damage on Full Life to your main skill.


Commands missing from MicrosoftTeamsPowershell

Get-Command -module microsoftteams does not return Add-TeamChannelUser yet the cmdlet is listed here
https://docs.microsoft.com/en-us/powershell/module/teams/add-teamchanneluser?view=teams-ps
This is with the latest version, 2.5.0
It seems this was an issue in the past as well
https://github.com/MicrosoftDocs/office-docs-powershell/issues/6549

@HorridMutilation How are you? That cmdlet is part of the -Preview branch of the Teams PowerShell module. (the ones identified with -preview in the module version)
Hope this information helps you

@get-itips Thank you very much for the contribution and sharing this explanation. @HorridMutilation Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


to_hdf multiple nodes first node bottleneck should be avoided

when saving to multiple nodes in the same hdf file (which i feel is the best compromise between efficiency and clean output), there's a bottleneck created by all subsequent to_hdf tasks depending on the first.
It might be worth creating a fake "to_hdf" node that will not depend on any input for this case.
this assumes PR #1293 is in, which is not the case atm, but i wanted to make sure this is noted somewhere.
For example, with mode='a' or 'r+', there's no need to link anything.

FWIWI #1293 is in now.
@nirizr can you provide a small code snippet that demonstrates the issue? That will make it easier to determine when this is fixed.


Replace Crouton for Snackbar

The Crouton library is not in development anymore and Android has introduce the Snackbar (also for old versions through the support design library).
I think that replace the crouton dependency for a native solution as Snackbar is more than appropriated.

@AyoPrez this is more like a preference than an issue, closing.


Enhancement: Specify environment variable in env section of Travis configuration

This PR

[x] specifies an environment variable in the env section because that's it what it's used for

:+1:, thanks.

@gfosco
Waiting for you to merge #229 before I'm rebasing this one!

@gfosco
Rebased!

:+1:


Add more MusicBrainz tags to Metadata

Such as album artist, etc.

Currently these seem less useful than initially anticipated


Please tag version used for release 3.0.0

There is https://repo.maven.apache.org/maven2/jakarta/enterprise/jakarta.enterprise.cdi-api/3.0.0/ published.
May I ask to tag the version that was used to produce this release, please?

I have tag
tag 3.0.0
Tagger:     Scott M Stark
TaggerDate: Wed Aug 26 16:48:30 2020 -0500

Move 3.0.0 tag

for commit 5c59e423662505beecc10093710e5834b68ba2b3, which seems to be deleted in this repository, and I can't see replacement.

I have tag
tag 3.0.0
Tagger:     Scott M Stark
TaggerDate: Wed Aug 26 16:48:30 2020 -0500

Move 3.0.0 tag

for commit 5c59e423662505beecc10093710e5834b68ba2b3, which seems to be deleted in this repository, and I can't see replacement.

I don't see a reason why anyone would delete it.
@starksm64 this should be done automatically via release in Jenkins, right? Do you remember deleting some tags for any reason?

I don't see a reason why anyone would delete it.
@starksm64 this should be done automatically via release in Jenkins, right? Do you remember deleting some tags for any reason?

I don't see a reason why anyone would delete it.

Deletion happened before Mon 31 Aug 10:23:02 CEST 2020.

@starksm64 this should be done automatically via release in Jenkins, right? Do you remember deleting some tags for any reason?

It seems tags in this repo are not created by bots/Jenkins, but by humans rather.

I don't see a reason why anyone would delete it.

Deletion happened before Mon 31 Aug 10:23:02 CEST 2020.

@starksm64 this should be done automatically via release in Jenkins, right? Do you remember deleting some tags for any reason?

It seems tags in this repo are not created by bots/Jenkins, but by humans rather.

Yes, the staging release process should be creating the tag. I'll look into why it could be missing.

3.0.0 tag has been added to 5c59e423662505beecc10093710e5834b68ba2b3 to match the 3.0.0 release build:
https://ci.eclipse.org/cdi/job/CDI Release/37/


Upgrade cloudscraper to solve Challenge evaluation failed due to unexpected token

I have this error reporting from cloudscraper
2019-05-23T16:03:05.395672+00:00 app[worker.1]: ### Cloudflare may have changed their technique, or there may be a bug.
2019-05-23T16:03:05.395673+00:00 app[worker.1]: ### Bug Reports: https://github.com/codemanki/cloudscraper/issues
2019-05-23T16:03:05.395675+00:00 app[worker.1]: ### Check the detailed exception message that follows for the cause.
2019-05-23T16:03:05.395677+00:00 app[worker.1]:
2019-05-23T16:03:05.395679+00:00 app[worker.1]: SyntaxError: Challenge evaluation failed: Unexpected token |
2019-05-23T16:03:05.395680+00:00 app[worker.1]: at solveChallenge (/app/node_modules/cloudscraper/index.js:254:21)
2019-05-23T16:03:05.395681+00:00 app[worker.1]: at Timeout._onTimeout (/app/node_modules/cloudscraper/index.js:179:7)
2019-05-23T16:03:05.395683+00:00 app[worker.1]: at ontimeout (timers.js:436:11)
2019-05-23T16:03:05.395684+00:00 app[worker.1]: at tryOnTimeout (timers.js:300:5)
2019-05-23T16:03:05.395686+00:00 app[worker.1]: at listOnTimeout (timers.js:263:5)
2019-05-23T16:03:05.395687+00:00 app[worker.1]: at Timer.processTimers (timers.js:223:10)```

**Possible solutions:** Upgrade cloudscraper to the latest version fixes the error. Got that from [cloudscraper](https://github.com/codemanki/cloudscraper/issues/190) and tested it on my instance.

Cloudscraper has been upgraded in the last release (2.1.0).


Adding additional results for browsers

Broke out colspan cells into individual cells for easier updating
Added Chrome, Firefox, Edge, IE and Safari data

Don't merge this yet--I will send you (@arronei) a PR for the remainder of the file. You can merge it on your repo, then update this PR...

See: https://github.com/arronei/test-results/pull/1

:+1:


fix(telegram): polling when stops bot

在 polling 时停用适配器，http 请求未中断，响应后向下继续执行

Codecov Report

Merging #150 (e60f010) into master (a3ea0c8) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##           master     #150   +/-   ##
=======================================
  Coverage   81.83%   81.83%
=======================================
  Files           1        1
  Lines         424      424
  Branches       86       86
=======================================
  Hits          347      347
  Misses         77       77


Undef OPAQUE marco conflicting with TensorFlow symbol.

See issue #25773.
@jlebar @rmlarsen.

I think Eigen is managed as a separate repository and copy/pasted into TF (?), so the PR should be against https://bitbucket.org/eigen/eigen/ ?
But @rmlarsen I also notice that third_party/eigen3 hasn't been updated since Nov 6 2018.  Not sure if that means that adding this one-line fix is going to require a massive Eigen upgrade?

third_party/eigen3 is synced with upstream roughly weekly. Feel free to
send a PR for Eigen to fix this issue.
On Mon, Apr 15, 2019 at 8:35 AM Justin Lebar<EMAIL_ADDRESS>wrote:

I think Eigen is managed as a separate repository and copy/pasted into TF
(?), so the PR should be against https://bitbucket.org/eigen/eigen/ ?
But @rmlarsen https://github.com/rmlarsen I also notice that
third_party/eigen3 hasn't been updated since Nov 6 2018. Not sure if that
means that adding this one-line fix is going to require a massive Eigen
upgrade?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/27859#issuecomment-483301402,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQH9DphGPPIkrMysygmWNmPAtXrCl9Yfks5vhJxYgaJpZM4cvkCX
.

@rmlarsen third_party/eigen3/unsupported/Eigen/CXX11/Tensor is not the same as src/default/unsupported/Eigen/CXX11/Tensor.
There are also commits like this affecting third_party/eigen3/unsupported/Eigen/CXX11/Tensor on TensorFlow repo. @jlebar So I do not think it is managed as a separate repository.

Ah, my bad!
On Tue, Apr 16, 2019 at 4:49 AM Stepiii<EMAIL_ADDRESS>wrote:

@rmlarsen https://github.com/rmlarsen
third_party/eigen3/unsupported/Eigen/CXX11/Tensor
https://github.com/tensorflow/tensorflow/blob/master/third_party/eigen3/unsupported/Eigen/CXX11/Tensor
is not the same as src/default/unsupported/Eigen/CXX11/Tensor
https://bitbucket.org/eigen/eigen/src/default/unsupported/Eigen/CXX11/Tensor
.
There are also commits like this
https://github.com/tensorflow/tensorflow/commit/d0d975f8c3330b5402263b2356b038bc8af919a2#diff-dd67b23046f83602c7172fd9457a44c8
affecting third_party/eigen3/unsupported/Eigen/CXX11/Tensor
https://github.com/tensorflow/tensorflow/blob/master/third_party/eigen3/unsupported/Eigen/CXX11/Tensor
on TensorFlow repo. @jlebar https://github.com/jlebar So I do not think
it is managed as a separate repository.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/27859#issuecomment-483625573,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJMh2Qp1pu_cvAGvljLIh_mEgIcAPG-ks5vhbjdgaJpZM4cvkCX
.

@rmlarsen Will this PR fix the issue?

Already fixed in 0621242.


Flat Config mode doesn't work

fixed in https://github.com/lvjiaxuan/vscode-eslint-disable/commit/6b78b06098f24267380c218d1dc478ddea8f9948

refer to 1bcde1e948d8692d1c09bb9ccc37b9678434b0ef


KOGITO-8667 Reorganize GHA

https://issues.redhat.com/browse/KOGITO-8667
Related:

https://github.com/kiegroup/drools/pull/4996
https://github.com/kiegroup/kogito-runtimes/pull/2807
https://github.com/kiegroup/kogito-apps/pull/1631
https://github.com/kiegroup/kogito-examples/pull/1572

Can one of the admins verify this patch?


fix(plugin-ledger-connector-xdai): OpenAPI spec validation missing from plugin REST endpoints

Describe the bug
Right now the constraints we define for request/response objects are not checked when a request comes in.
To Reproduce
Send an invalid request to some plugin endpoint that is invalid in a certain way and watch how it crashes the plugin resulting in internal server error vs. bad request.
Expected behavior
Fail with bad request even before the plugin's back-end code is reached if a request is sent that does not comply with the open api specs defined for that endpoint.
Logs/Stack traces
N/A
Screenshots
N/A
Cloud provider or hardware configuration:
N/A
Operating system name, version, build:
N/A
Hyperledger Cactus release version or commit (git rev-parse --short HEAD):
main
Hyperledger Cactus Plugins/Connectors Used
Xdai
Additional context
The interface of the web service plugins should be extended with a getOpenApiSpecs() method that returns an OpenAPI v3 spec document (see the openapi-types package)
Then the api-server calls up this method when it's installing the web services of the plugin, obtains the open api specs and creates a validator instance from it.

Working on this!


Add an Experiment object to gather scenarios and run them in sequence

@RomainGoussault @arthurPignet what do you think of the following:

an Experiment object, which would essentially contain a list of Scenario instances (let's call it scenarios_list here)
a run() method which loops over the scenarios_list and run them one by one (or in parallel if someone knows how to parallelize computations... see #49 if you have ideas about that)
as currently done in main.py it would add one level in the folders to gather results, corresponding to the Experiment instance (e.g. ./experiments/my_experiment/scenario_n/...)
additionnally we could have:

a nb_repeats parameter, as @RomainGoussault you had introduced
a name
a pre-validation of all scenarios, as we have in main.py
a results file with all scenarios results, as we have in main.py
a notebook that offers default tables and graphs on the results

It would enable to refactor main.py properly, to be much closer to the way the library is supposed to be used in a notebook (for an individual scenario or a serie of scenarios gathered in an Experiment).

I think that is it a great idea, the benchmark of many methods will become easier.
For the pre-validation of scenario, maybe it is time to move the split of the data between the partner in the scenario's init ?
Thus the process will be : init, validate, run
This object could also be the place to implement methods to compare directly scenarios
For the parallelize scenario computations, I already made a project using multiprocessing and I see 2 issues

The first issue is about the handling of datasets, as each process needs its own memory space. Computing 2 scenarios at the same time will require twice the memory in space, so we will quickly run out of space. It could maybe work with a shared dataset, but honestly I don't really know if the partners are deepcopying the dataset...
The shared memory will implies that we compute in the same time only scenario with the same dataset.
In fact the best thing to share can be the whole scenario, dataset and partners included. (Thus, we will parallelize mpl, or repetition)
The second issue is the parallelization on GPU. I never did that, and I am wondering how it works.

I totally agree with the idea.
Actually maybe going one step further: if we implement that, do we still need to keep the old way of launching script with a config file ? What is the benefit of it compared to what you propose ? I don't see any.

The benefit is the mechanism to make the product of parameters to define the list of scenarios


Playback view slows down to a crawl

Hi loving the mod for seeing my deaths,
I'm having an issue though, I upgraded to v .06 deleted the correct .dll plugin and mod folder and migrated my data folder to the new usr/mods folder.
The issue however is that when I open a raid for playback on my chrome browser it starts out playing fine at normal speed but then it gets really laggy as if it was playing back at 10fps or lower.
What logs might you need to see?

I won't need to see any logs, I have been working on improving this.
If you check the releases on the main page of this repository, there is a 0.1.0_test release.
Did you want to give that a try see if there is a noticeable performance difference?

Gave it the test release a try and I'm even though I migrated my data it shows my profile and I can see the saved raids, however upon clicking on any raid all I see is an empty map, no name list and no events logged on the timeline, pressing play doesn't do anything

Gave it the test release a try and I'm even though I migrated my data it shows my profile and I can see the saved raids, however upon clicking on any raid all I see is an empty map, no name list and no events logged on the timeline, pressing play doesn't do anything

same

Gave it the test release a try and I'm even though I migrated my data it shows my profile and I can see the saved raids, however upon clicking on any raid all I see is an empty map, no name list and no events logged on the timeline, pressing play doesn't do anything

Okay I see - one sec I'll upload a new version with that fixed.

Hi. As mine was really slow too, I tried this, and with the old raids, I have no info and get this error

I'ma play some raids now, and Ill post if it works.
I dont mind losing old data, should I delete the data folder inside mods/raidreview0.1.0?
thank you :)

@RebeliusGaming @serranojavier @KonstAl54
Please try this, the previous version I uploaded I forgot to re-add some critical code to the Playback.
https://github.com/ekky-llc/spt-raid-review/releases/tag/v0.1.0_test_a
Sincere apologies for screwing you guys around.

Should I just extract it and overwrite if asked?

It shouldn't ask for an overwrite, it should make a new folder in your user/mods/ and you need to copy your /data folder from the old one as per install notes.
Might just be because it's a very fresh upload, however, if you don't trust, I've included a scan result in the download page.
Otherwise not sure why it's complaining 😞

Works FLAWLESS right now.
I dont know if you get access to this data, but also being able to watch bullets would be great!

Sorry
Spoke too early. Same problem again. Nice at start, but really slow after some seconds

Sorry
Spoke too early. Same problem again. Nice at start, but really slow after some seconds

Yeah, I figured that would happen, I'm overloading the browser too hard.
I tried switching the rendering approach, which broke the last build, and I need to do some more research around a better way.
I'll keep this Issue open for now.

Just tried v0.1.1 and performance is great now! Love this mod to see how I've died

Just tried v0.1.1 on 3.8.3 and performance is great now! Love this mod to see how I've died

That's great news, glad you like it man :)


feat(list): support multiple selection

Closing this since the following pr is a continuation of this one: https://github.com/wixplosives/stylable-components/pull/1080


Steam Menu not opening when in Emulated Game

When opening an emulated game (tested with Ryujinx), the steam menu and the right hand menu no longer open.
You therefore can't exit the game.

I’m experiencing the same issue.

I have the same problem with Ryujinx. However, in my case, it doesn't affect all emulators as the Steam (left) and QAM (right) menus work perfectly fine in-game while playing Yuzu and RetroArch . The problem only happens with Ryujinx, so this might be some issue with Ryujinx.
I've switched back to Yuzu for now.


Resource Integrity

Hello,
Is there any way I can easily add resource integrity metadata to link,script tags? Do you've any plan to include this feature in your package?
Regards

There is already a feature for adding metadata (HTML attributes) to the generated HTML tags.
Using ...
Assets::css(['foo' => 'bar', 'dummy']);

... will result in
<link rel="stylesheet" type="text/css" href="css/style.css" foo="bar" dummy="dummy">

Or using ...
Assets::js(['foo' => 'bar', 'dummy']);

... will result in
<script type="text/javascript" src="js/effects.js" foo="bar" dummy="dummy">

For adding the  resource integrity metadata to links you only need to use:
Assets::css(['integrity' => 'myhash', 'crossorigin' => 'anonymous']);

You have to generate the hash yourself since I don't plan to add such feature.

Hi,
I am using your package as asset manager for my main side project and Im not getting this to work.
I have tried you exact example and I dont get any output like that. In
Assets::css(['foo' => 'bar', 'dummy']);
How did you specify css/style.css? For me it looks like it would apply to all css.
So, how do you apply attributes individually? (I am using the ->add method)
Thanks in advance

@marcmascarell Could you please show us which exact code are you using (both for adding the assets and generating the html tags), which result are you getting (the actual html tags you are getting) and what do you expect to to get?

It's a very basic use case: https://gist.github.com/marcmascarell/05eacc711b8226c07ea93a013901e00e


idf.py sees LOLIN S2 mini DFU devices but cannot open them (IDFGH-9162)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
WIN10 has esp-idf v5.0 installed. I connect the WEMOS LOLIN S2 mini board and use the buttons to switch it to bootloader mode.
Then check the list of connected devices:
E:\_esp-idf\hello_world>idf.py dfu-list
Executing action: dfu-list
Running ninja in directory e:\documents\arduino\_esp-idf\hello_world\build
Executing "ninja dfu-list"...
[0/1] cmd.exe /C "cd /D E:\Documents\Arduino\_esp-idf\hello_world\build && E:\Espressif\tools\cmake\3.24.0\bin\cmake.exe -D ESP_DFU_LIST="1" -P E:/Espressif/frameworks/esp-idf-v5.0/tools/cmake/run_dfu_util.cmake"
Command list: dfu-util;--list
dfu-util 0.9

Copyright 2005-2009 Weston Schmidt, Harald Welte and OpenMoko Inc.
Copyright 2010-2016 Tormod Volden and Stefan Schmidt
This program is Free Software and has ABSOLUTELY NO WARRANTY
Please report bugs to http://sourceforge.net/p/dfu-util/tickets/

Cannot open DFU device 303a:0002
Done

If I connect two LOLIN S2 mini in download mode, I get this response:
E:\_esp-idf\hello_world>idf.py dfu-list
Executing action: dfu-list
[...]
Please report bugs to http://sourceforge.net/p/dfu-util/tickets/

Cannot open DFU device 303a:0002
Cannot open DFU device 303a:0002
Done

The documentation shows that the response should look something like this:
Found Runtime: [303a:0002] ver=0723, devnum=4, cfg=1, intf=2, path="1-10", alt=0, name="UNKNOWN", serial="0"
Found Runtime: [303a:0002] ver=0723, devnum=6, cfg=1, intf=2, path="1-2", alt=0, name="UNKNOWN", serial="0"

The above problem does not allow flashing the specified device in the esp-idf environment.

The problem was solved by updating the dfu-util version to 0.11 and installing the libusbK driver using the Zadig program.


Setting and retrieving user in same request

TL;DR
# config/initializers/warden.rb
class Warden::SessionSerializer
  def deserialize(attributes)
    User.new attributes
  end
end

# some_controller.rb
warden.set_user id: 42, name: 'Bob'
warden.user  # => A Hash, not a User.

More elaborate:
When I set the user in my SessionsController, I pass in a Hash with user attributes. A simple Warden::SessionSerializer  runs User.new(attributes) to turn that into an instance each time I access warden.user. After login I redirect the user and have the user available as an User instance. So far so good.
Now I wish to mutate the user but don't want to have to wait one request for the changes to take effect. I would like to set the user and be able to retrieve it immediately after setting it. How would I go about that?
Because if I set the user as a Hash, a Hash is what I get out (according to the code to set and retrieve a user, no immediate serialization takes place). If I set it as a User instance, it works for this request but the deserializer is going to be confused when not confronted with a Hash at the next request.
On logout, the code takes effect immediately by (understandably) cleaning out the @users variable and the session_serializer. I'm wondering why setting and retrieving the user doesn't behave similarly. Is that intentional?
Now, it does turn out that I can do this in my middleware to achieve the desired effect, but it feels really awkward:
warden.user.attributes = something: :new         # <- Affects current request
warden.session_serializer.store user.attributes  # <- Affects subsequent requests

Is it okay to do that or did I miss something? I'd appreciate any pointers.
(Maybe just to clarify my use case: I have a middleware that checks the user's single-sign-on session with a remote server and is supposed to mutate the user depending on what that SSO server says about the user.)
Thank you for your time! Warden is wonderful otherwise :)

PS: I assume I'm supposed to do this, right?
env['warden'].set_user(user, :store => false)   # <- Affects current request
env['warden'].set_user(user.attributes)         # <- Affects subsequent requests

It's not ideal, but it works.

Ok, but this doesn't work because at login I don't have a user at hand yet.
Say I have this setup:
Warden::Manager.serialize_into_session do |user_id|
  user = ::Customers.find_by_id(user_id)
  user.attributes
end

Warden::Manager.serialize_from_session do |serialized|
  ::Customer.new serialized
end

So I would normally run success! 42 and that's enough to have a Customer instance available at warden.user at the next request. But I need the instance in the same request now.
I really feel bad duplicating the serializing logic. As a workaround, is there a way to call the Warden::Manager.serialize_into_session and then the Warden::Manager.serialize_from_session logic manually so I can pass the result to success!?

OK, I found a workaround again:
user_id = warden.authenticate! :password
serialized = warden.session_serializer.serialize(user_id)
deserialized = warden.session_serializer.deserialize(serialized)


Add osvr-unity-ver.txt to unitypackage

Edit CI build to identify OSVR-Unity version.
osvr-ver.txt, rendermanager-ver.txt, and osvrUnityRenderingPlugin-ver.txt already exist.

this is now included starting with build #445


Minor performances improvements

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(android): add set hevc black list api

Signed-off-by: pingkai<EMAIL_ADDRESS>

Codecov Report

Merging #440 into develop will increase coverage by 0.14%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #440      +/-   ##
===========================================
+ Coverage    68.82%   68.96%   +0.14%
===========================================
  Files          216      216
  Lines        11775    11777       +2
===========================================
+ Hits          8104     8122      +18
+ Misses        3671     3655      -16

Impacted Files
Coverage Δ

mediaPlayer/SuperMediaPlayer.cpp
74.21% <100.00%> (-0.12%)
:arrow_down:

mediaPlayer/system_refer_clock.cpp
91.66% <0.00%> (-4.17%)
:arrow_down:

framework/codec/ActiveDecoder.cpp
85.30% <0.00%> (-0.95%)
:arrow_down:

framework/demuxer/play_list/SegmentTracker.cpp
70.81% <0.00%> (-0.55%)
:arrow_down:

framework/demuxer/play_list/HLSManager.cpp
74.20% <0.00%> (-0.40%)
:arrow_down:

framework/demuxer/play_list/HLSStream.cpp
67.26% <0.00%> (-0.39%)
:arrow_down:

framework/data_source/curl/curl_data_source.cpp
86.88% <0.00%> (+0.40%)
:arrow_up:

mediaPlayer/player_msg_control.cpp
81.74% <0.00%> (+0.79%)
:arrow_up:

framework/demuxer/avFormatDemuxer.cpp
80.85% <0.00%> (+1.14%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


Combine Windows Snap with FancyZone? Add more border targets to the screen and link them to zones? Combined window resizing like in Snap?

Windows snap (move window to top screen for full screen, to the left for 50% etc.) is a fantastic feature with a great UX and actually even faster to use than FancyZones. I use both in combination.
The only limitation in Snap since it was introduces are too few targets. 50% left/right and full screen is not enough. You have now also this 25% zone when you drag a window in the screen corner. But I don't find this too useful and a bit limited on larger screens. I often want something like 1/3 or 2/3. Or something centred with free space to the left and right. This is easy to do with FancyZones.
But the Snap UX feels so great and even faster than FancyZones. I like FancyZone to access more complicated zones possibilities. But using Snap for your most common zones would be great and for me even more efficient.
My idea is to add more targets to the screen for window Snap, and give the possibility to map these targets to zones defined in FancyZone.
For example:

Left corner is not 25% resize, but re-mapped to FancyZone e.g. zone 1.
Think also about to add additional Snap zones on the screen border. Like moving a window at the top at 0-20% left is an own target that could be used for something different than full screen. Full screen would be top/left 20%-80% for example.

In addition. When you Snap two windows 50% 50% and resize one window, you resize both windows at the same time in Windows 10. This is also a great feature. It would be great when this would be the same for directly touching zones in FancyZone too. You need to resize only one windows instead of two this way.
This is just a rough idea. Especially if and how to setup additional Snap targets needs some additional thoughts. But this would feel like the ultimate solution for me.

I would love that as well as currently, you HAVE to use the keyboard in order to snap a window to a fancy zone. It would make a huge difference if you could just drag to the left side of the screen to snap to the left zone, like you can with Windows Snap, as it could be done just with the mouse.

@Flavien

you HAVE to use the keyboard in order to snap a window to a fancy zone

Not necessarily, you can use the mouse only if you want.

Ok you can use the right click thing as you drag. Better than nothing, but still not great. Ideally it would just override the Windows Snap behaviour. Why have two competing mechanisms active at the same time?

It would also be nice, if Win+Up/Down arrows would actually for minimizing/maximising windows. Since current [Win]+[Up/Down/Left/Right Arrow] removes whole windows snapping functionality.

I'm loving FancyZones with Win + Up/Down/Left/Right move windows based on relative position, but I miss the ability to minimize a window with the keyboard. Is there currently any way to have both?

I'm loving FancyZones with Win + Up/Down/Left/Right move windows based on relative position, but I miss the ability to minimize a window with the keyboard. Is there currently any way to have both?

You can use Win + Shift + Down to minimize

Great, thanks!
Is that documented somewhere?

I don't think so. I was also looking for same thing as you and found it in one of the issues.

What about the resize alone? That would be great

I think the general sentiment of some the comments here and in a few related issues, is that users of FZ generally like the behavior enabled by the "Override Windows Snap Shortcut" option, which makes moving windows around more intuitive, especially between multiple monitors.
However, when the FZ key override is enabled, users are left with the Win + Shift + Up/Down shortcuts, and although the behavior of minimizing is the same, maximizing with this combo actually behaves as a shortcut to fullscreen a window, which of course hides any taskbars/titlebars/etc on that screen.
What most folks are missing (at least myself) is a simple short for maximizing windows by resizing them, which is the way that Win + Up works when FZ isn't overriding native Windows behavior. From what I understand there is no "easy" way to set such a maximize-by-resizing shortcut either, although I'd be happy to learn otherwise.

Actually Win + Shift + Down doesn't work to minimize all apps. E.g. Microsoft Edge (90.0.818.46).

I wanted to leave some of my comments from #10537 here since it's the only open issue related to window resizing hotkeys, see that issue for more details. @enricogior have you happened to read my comments in #10537 by chance?
I'm looking for a solution to maximize windows by resizing them, which is the way that Win + Up works when FZ isn't overriding native Windows behavior. From what I understand there is no "easy" way to set such a maximize-by-resizing shortcut once the FZ override option is enabled either, although I'd be happy to learn otherwise.
Maybe the FZ override could itself also override the Win + Shift + Up shortcut to more closely mimic the native maximize-by-resize behavior or perhaps an additional toggle option to do so?

I would love to have FancyZones mimic or replicate Windows Snap behavior as closely as possible. With overlapping zones, I can get about 85% of the way there, and while that's great it's not really as smooth as it seems like it could be.
Motivation
I recently moved from 3 monitors (1600x1200 : 1920x1200 : 1600x1200) to 1 ultra-wide (5120 x 1440, so same horizontal resolution). Windows Snap with the 3 separate monitors and Display Fusion taskbars was absolutely perfect.
Now I can still use Display Fusion to do its own splitting to get the taskbars while using FancyZones to handle splitting for the sake of window management and it's pretty good, but it's not quite great. My dream is that I am someday able to exactly replicate the window behavior of my old setup.
Issues

I have Windows Snap overridden but Windows key + up/down don't do anything
Windows key + left/right can move windows from the far-left side to the far-right side and vice versa (like the monitor is wrapped)
Dragging windows with the mouse triggers the FancyZones UI overlay even at 0 zone opacity (maybe I'll get used to this)
Using the mouse, nothing is preventing the Windows Snap "preview" border or even its behavior (disappointing but maybe not something FancyZones can change)
Using the mouse, non-resizable windows still trigger the FancyZones UI

Current Settings

Screenshot

Current Zone Layout

Screenshot

1 - Left side full
2 - Left side left half
3 - Left side right half
4 - Middle full
5 - Middle left half
6 - Middle right half
7 - Right side full
8 - Right side left half
9 - Right side right half

I may have duplicated much of this in my general shortcut post. I think we are both trying to get to a more general solution.
There are so many issues related to shortcuts that I have created a more general issue #22186 to prompt discussion for the entire topic hoping to find a more complete solution to the shortcut issue.


No "Learn" page

Describe the bug
Clicking the "Learn" on nav goes to a blank page
To Reproduce
Steps to reproduce the behavior:

Go to '.http://localhost:3000/#/.'

Click on 'LEARN'

See error

Expected behavior
Should have a UI for content of LEARN
Screenshots
If applicable, add screenshots to help explain your problem.

take a screenshot

Mac: shift + command + 3
Windows: use desired screenshot program

drag & drop screenshot image here:

System Info (please complete the following information):

Device: [e.g. Macbook Pro, iPhone6]
OS: [e.g. iOS, Android]
Browser: [e.g. chrome, firefox]
Version: [e.g. 22]

Additional context
Add any other context about the problem here.

@natsun121 I see. You were making two issues for JOBS and LEARN. I think the JOBS one needs to be edited as it says LEARN within the description.

This issue is solved and is apart of pull request #150

Thank you for this contribution. It has been merged into the master branch.


Rewrite the copr-builds part of the API using PostgresSQL instead of Redis

Completes the first checkbox of #459
This is completely back-compatible with the previous redis API, although I added more information than before because we need it for the dashboard.
Some previously used info was not stored in psql so I added fields for them and modified the code to store them.
/api/copr-builds now returns a list of objects where one item looks like:
{
    "project": "IceWreck-hello-world-9",
    "owner": "icewreck",
    "repo_name": "hello-world",
    "build_id": "1300479",
    "status": "pending",
    "chroots": [
      "fedora-31-x86_64",
      "fedora-rawhide-x86_64",
      "fedora-30-x86_64"
    ],
    "build_submitted_time": "11/03/2020 10:09:48",
    "repo_namespace": "IceWreck",
    "web_url": "https://copr.fedorainfracloud.org/coprs/icewreck/IceWreck-hello-world-9/build/1300479/"
}

And the specific copr build info returns everything we know about said build.

Also moving to postgres fixed the issue where specific copr build info was not shown (stored?) unless fedora messaging was working.

@IceWreck don't worry about the tests now, I broke the deployment, fixing it right now

tests should now be passing once again, please rebase

@TomasTomecek I updated the tests for the new columns in postgres.
Should I also do something about the new CoprBuild classmethods I created instead of using get_sa_session ?
They only make queries to fetch data like the preexisting get_by_id and get_by_build_id methods, which do not have any tests either.

Do you think that we need tests for some new code? I'll leave the answer to you :)

They only return queries so I guess we can leave them for now

About this:

git web url is not directly related to a copr build, can we move that field to the GitProject model?

This was already there before I made changes so if I change stuff then previous data in the stg/prod database will be deleted.
Maybe I can leave it there and create another one for new data but then the old data won't be retrievable.

About this:

git web url is not directly related to a copr build, can we move that field to the GitProject model?

This was already there before I made changes so if I change stuff then previous data in the stg/prod database will be deleted.
Maybe I can leave it there and create another one for new data but then the old data won't be retrievable.

I'd say that we don't have such a change in stg/prod, it was done locally only on your machine in the previous iteration, so we should be good with the way it's done right now in this PR.

Do you think that we need tests for some new code? I'll leave the answer to you :)

They only return queries so I guess we can leave them for now

...and I thought I taught you something already :(
Yes, please add new tests for the two new methods.

About this:

git web url is not directly related to a copr build, can we move that field to the GitProject model?

This was already there before I made changes so if I change stuff then previous data in the stg/prod database will be deleted.
Maybe I can leave it there and create another one for new data but then the old data won't be retrievable.

I'd say that we don't have such a change in stg/prod, it was done locally only on your machine in the previous iteration, so we should be good with the way it's done right now in this PR.

I think we are talking about different things here. You told me to move the git web url thing I placed in CoprBuilds and I changed the https_url location.
Then I misunderstood and confused it with web_url  which has been here since the beginning of this file https://github.com/packit-service/packit-service/blame/master/packit_service/models.py#L164

...and I thought I taught you something already :(

Yes, please add new tests for the two new methods.
Sheesh! Sorry, will do.

I think we are talking about different things here. You told me to move the git web url thing I placed in CoprBuilds and I changed the https_url location.
Then I misunderstood and confused it with web_url which has been here since the beginning of this file https://github.com/packit-service/packit-service/blame/master/packit_service/models.py#L164

Oh you bet, I have no idea what we're talking about here.
I think the current code is good:
op.add_column("git_projects", sa.Column("https_url", sa.String(), nullable=True))

Yes, that's what we want, to have the url in git_projects.
CoprBuild has web_url with a link to a build web UI. That change is live now.
So I think we're good here. Please let me know if there is anything I'm missing here.

@TomasTomecek
Added tests for those two methods and added another for an old method which didn't have tests.
Also did the changes you requested earlier today.


Style/RaiseArgs does not catch raise with both new and message params

Expected behavior
raise MyError.new(data: "mydata"), "some error message"

According to the documentaion

The exploded style works identically, but with the addition that it will also suggest constructing error objects when the exception is passed multiple arguments.

it should be offensive when this configuration is used:
Style/RaiseArgs:
  Enabled: true
  EnforcedStyle: exploded
  AllowedCompactTypes: []

It should be autocorrected to
raise MyError.new("some error message", data: "mydata")

Actual behavior
No offense is found
Steps to reproduce the problem
Run rubocop with this configuration:
Style/RaiseArgs:
  Enabled: true
  EnforcedStyle: exploded
  AllowedCompactTypes: []

on this code:
raise MyError.new(data: "mydata"), "some error message"

RuboCop version
1.29.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.17.0, running on ruby 3.1.0 x86_64-darwin20)

The signature of raise is below.
raise
raise(string, cause: $!)
raise(exception [, string [, array]], cause: $!)

https://ruby-doc.org/core-3.1.0/Kernel.html#method-i-raise
So this issue would be an expected behavior because raise MyError.new(data: 'mydata'), 'message' and raise MyError.new('message', data: 'mydata') are incompatible.
I'm going to close this issue. Thank you.


'numpy.int64' object has no attribute 'to_pydatetime'

Problem Description
I got the following errorn when I run create_full_tear_sheet from pyfolio. can anybody help as I cannot use Pyfolio.
The following are the packages version I have installed:
python 3.6
pandas 1.0.5
numpy 1.18.5
pyfolio 0.9.2
empyrical 0.5.3
the following is the full error I got
AttributeError                            Traceback (most recent call last)
 in
----> 1 pf.create_returns_tear_sheet(Globale)
~\Miniconda3\envs\lavoro\lib\site-packages\pyfolio\plotting.py in call_w_context(*args, **kwargs)
50         if set_context:
51             with plotting_context(), axes_style():
---> 52                 return func(*args, **kwargs)
53         else:
54             return func(*args, **kwargs)
~\Miniconda3\envs\lavoro\lib\site-packages\pyfolio\tears.py in create_returns_tear_sheet(returns, positions, transactions, live_start_date, cone_std, benchmark_rets, bootstrap, turnover_denom, header_rows, return_fig)
502                              header_rows=header_rows)
503
--> 504     plotting.show_worst_drawdown_periods(returns)
505
506     vertical_sections = 11
~\Miniconda3\envs\lavoro\lib\site-packages\pyfolio\plotting.py in show_worst_drawdown_periods(returns, top)
1662     """
1663
-> 1664     drawdown_df = timeseries.gen_drawdown_table(returns, top=top)
1665     utils.print_table(
1666         drawdown_df.sort_values('Net drawdown in %', ascending=False),
~\Miniconda3\envs\lavoro\lib\site-packages\pyfolio\timeseries.py in gen_drawdown_table(returns, top)
1006         df_drawdowns.loc[i, 'Peak date'] = (peak.to_pydatetime()
1007                                             .strftime('%Y-%m-%d'))
-> 1008         df_drawdowns.loc[i, 'Valley date'] = (valley.to_pydatetime()
1009                                               .strftime('%Y-%m-%d'))
1010         if isinstance(recovery, float):
AttributeError: 'numpy.int64' object has no attribute 'to_pydatetime'

Been having the same error for quite a while
Python     3.8.2
numpy     1.19.1
pyfolio     0.9.2
empyrical 0.5.3
pandas    1.0.5

Same issue--would be great if this could be resolved!

With @twiecki gone, maintenance for pyfolio is going to be an issue now.

fixed this by changing line 893 in file timeseries.py
valley = underwater.index[np.argmin(underwater)-1] # end of the period

In function: def get_max_drawdown_underwater(underwater):
this line return array index(as int), but not array value:
valley = np.argmin(underwater)
so we need to change return type:
return peak, valley, recovery
to
return peak, underwater[valley:].index[0], recovery

changing to valley = underwater.index[np.argmin(underwater)] didnt workout :(

neighter did return peak, underwater[valley:].index[0], recovery work out

I tried reinstalling directly from git, and that fixed this issues for me
pip uninstall pyfolio
pip install git+https://github.com/quantopian/pyfolio

I tried reinstalling directly from git, and that fixed this issues for me
pip uninstall pyfolio
pip install git+https://github.com/quantopian/pyfolio

It works, thanks ~


Removes second sort on brick shows

The opening soon and closing soon sections in the All tab had a second unnecessary sort which was distorting the order, this fixes this issue by removing it.

Merge on green


Please help me

I am confused about the version of StarGAN, because when I running it on GPU, it said:
File "/home/willamyeoh/StarGAN/solver.py", line 53, in init
self.device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
AttributeError: module 'torch' has no attribute 'device'
in the version I download to GPU, the line 53 of solver.py is: self.device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
and I found there is another version which I cloned on my own Mac is :self.dtype = torch.cuda.FloatTensor if torch.cuda.is_available() else torch.FloatTensor
Does this has anything to do with the running problem? and could you give me a hint how to solve this problem?

AttributeError: module 'torch' has no attribute 'device'
For the above issue, update your PyTorch version to 0.4.0.


Options Reloaded

Fixes https://github.com/aspnet/Options/issues/145#issuecomment-250020098
Initial shape with basic tests (want to add a few more)
Few points to consider:

The feature is always 'on', any time Configure(IConfiguration) is called, we hook up the notifications (no way to disable)
We could have the current OptionsManager implement both IOptions and IOptionsSnapshot (we'd register the class twice, singleton for IOptions, and Scoped for Snapshot, felt too weird so I didn't go that route, and copied the class, since its tiny)
Support multiple listeners in OptionsMonitor.OnChange or not?  Basically subsequent calls could either add additional callbacks, or override the first callback.  I did the more complicated list of callbacks, but I'd be glad to kill this.
(Found a nasty bug with CurrentValue in the old implementation where it wouldn't get updated if no one ever called OnChange)

@divega @Eilon @DamianEdwards @davidfowl

The feature is always 'on', any time Configure(IConfiguration) is called, we hook up the notifications (no way to disable)

At first glance it looks like the baseline cost (if noone hooks up to listen) is a singleton IOptionsChangeTokenSource. Are there other hidden costs?

Support multiple listeners in OptionsMonitor.OnChange or not? Basically subsequent calls could either add additional callbacks, or override the first callback. I did the more complicated list of callbacks, but I'd be glad to kill this.

I agree the code is a bit more involved this way, but I think it is fine, and I cannot see us getting away without it.

Yeah I guess the overhead isn't too bad if no one asks for an IOptionsMonitor since the change token hookup in the ctor won't happen

@Eilon @DamianEdwards @davidfowl last call for other name suggestions for the initial checkin...?
'

Keep IOptionsSnapshot - it conveys the nature of the object ;)

acecacff57f9399e5871216a3f5a073548700dc3

I fully expected the reason for the existence of IOptions<> being the reloading ability. Why was IOptionsSnapshot<> created instead and why not just register the options type  T directly?


fix(browser-resolve): fix regressions from refactor

This should fix #130 and #131, with tests added for each regression.

This has been updated to cover some additional usecases @mislav found. This should be very robust now I believe.I'm very confident in this PR.

released as 3.0.3, thanks!


[SUP-766] Issues with opening and previewing files uploaded through AnalysisModal

https://broadworkbench.atlassian.net/browse/SUP-766
Description
Users were unable to successfully upload a file from AnalysisModal and preview or edit the file.
Cause of issue
When uploading through AnalysisModal, .ipynb files were converted to strings instead of being kept as JSON. This caused rendering issues, and file loading issues.
Additional Updates
In addition to fixing the file uploading for .ipynb files, this PR also adds a new default .Rmd file which populates a proper header. Previously, the file was just a string literal saying "starter file". This new file default should be a much better starting ground for users.
In addition, previously, when attempting to download this default file, chrome would open a new tab due to the file type and it's contents. Now users can more easily download the file.
Testing
Ensured creating new .ipynb and .Rmd files continued to work on GCP, including previews.
Ensured uploading .ipynb and .Rmd files continued to work for GCP, including previews.
Ensured creating and uploading .ipynb files continued to work for Azure, including previews.

In response to your not testing - I did, and had Alessandro test!


X connect

relates to #161

Great job!

How do we control the case of someone using an already-used tweet from another person to connect to their X account?
We can probably do something with our backend for that. But how about an intruder who uses someone else's tweet before the original poster has been able to authenticate themselves?
@HrithikSampson

an probably do something with our backend for that. But how about an intruder who uses someone else's tweet before the original poster has been able to authenticate themselves?

I was thinking in the lines of being able to send a private dm but I cant think of anything yet, or limit the message to a group but the problem was shown in this post https://www.quora.com/How-do-I-send-a-private-tweet-to-one-person-only-that-no-one-else-can-see


Fix code scanning alert - Missing origin verification in postMessage handler

Tracking issue for:

[x] https://github.com/webrtc-org/samples/security/code-scanning/3

https://gist.github.com/gilbertalgordo


Add configuration values for the Pipelines created containers

I've tried to use my docker file template to deploy Gitness.
In that docker file, I define specific networks for the stack, and usually, I run everything without exposing ports by exposing the services with a tunnel (Cloudflare) or a proxy (Traefik)
Following this approach I have found that it is not possible to run pipelines, as they create the Drone container expecting to have access to Gitness using the 'http://host.docker.internal:3000/' address, instead of 'http://container_name:3000', accessible from the same docker network.
This results in an error on the first step of the pipeline that says 'unable to access 'http://host.docker.internal:3000/'
Changing the docker file by exposing the port makes it work.
It should be possible to specify the network for the newly created containers + where to find Gitness, as docker environment variables that can be passed on the container creation or as part of any docker file

+1, am running into the same issue. I am running Gitness behind Traefik and this is pretty annoying.

Following this approach I have found that it is not possible to run pipelines, as they create the Drone container expecting to have access to Gitness using the 'http://host.docker.internal:3000/' address

I believe this is configurable using GITNESS_URL_CONTAINER
https://github.com/harness/gitness/blob/a9895959ca593d81fc4a9da7a8e26ac77572ca97/types/config.go#L104

Hello,
any update on this?
@bradrydzewski GITNESS_URL_CONTAINER (not documented at all, Configuration) is the one I thought could work but... it does not.
Here is a simple docker-compose.yml file with just traefik and gitness
version: '3.8'

services:
  traefik:
    container_name: traefik
    image: traefik:latest
    restart: always
    ports:
      - '80:80/tcp'
    volumes:
      - ./config/traefik.yml:/etc/traefik/traefik.yml
      - ./logs/:/logs/
      - /var/run/docker.sock:/var/run/docker.sock:ro
    labels:
      - 'traefik.enable=true'
      - 'traefik.http.routers.traefik.rule=Host(`traefik.localhost`)'
      - 'traefik.http.routers.traefik.service=api@internal'
      - 'traefik.http.routers.traefik.entrypoints=web'

  gitness:
    container_name: gitness
    image: harness/gitness
    restart: always
    ports:
      - '3000:3000'
    environment:
      - GITNESS_URL_BASE=http://gitness.localhost
      - GITNESS_HTTP_PORT=3000
      - GITNESS_URL_CONTAINER=http://gitness.localhost
      - GITNESS_USER_SIGNUP_ENABLED=false
      - GITNESS_PRINCIPAL_ADMIN_PASSWORD=password
    volumes:
      - gitness_data:/data
      - /var/run/docker.sock:/var/run/docker.sock
    labels:
      - 'traefik.enable=true'
      - 'traefik.http.services.gitness.loadbalancer.server.port=3000'
      - 'traefik.http.routers.gitness.rule=Host(`gitness.localhost`)'
      - 'traefik.http.routers.gitness.entrypoints=web'

volumes:
  gitness_data:
    name: 'gitness_data'

traefik confile file is a basic one, nothing special
# API and dashboard configuration
api:
  # Dashboard
  dashboard: true
  insecure: false

# Docker configuration backend
providers:
  docker:
    watch: true
    exposedByDefault: false
    swarmMode: false

# Traefik Logging & Access Logging
log:
  level: ERROR
  filePath: /logs/traefik.log
accesslog:
  filePath: /logs/traefik_access.log

# Entrypoint
entryPoints:
  web:
    address: ':80'

With this setup I can create project/repo/pipeline, clone, commit etc. All is working fine except running the pipeline failing at the clone step.
Cloning with 0 retries Initialized empty Git repository in /gitness/.git/ + git fetch origin +refs/heads/main: fatal: unable to access 'http://gitness.localhost/git/playground/go-hello.git/': Failed to connect to gitness port 80 after 2 ms: Connection refused

I tried severals values for GITNESS_URL_CONTAINER but no way. E.g.
- GITNESS_URL_CONTAINER=http://gitness.localhost
- GITNESS_URL_CONTAINER=http://<container_name>
- GITNESS_URL_CONTAINER=http://<container_name>:3000
- GITNESS_URL_CONTAINER=http://<IP_ADDRESS>:3000
- GITNESS_URL_CONTAINER=http://<internal_container_ip>:3000

I tried to set the GITNESS_URL_INTERNAL env variable too
https://github.com/harness/gitness/blob/a9895959ca593d81fc4a9da7a8e26ac77572ca97/types/config.go#L95-L97
Who can help? Thanks

@ignaciolg @indaco @cozyGalvinism thanks for trying out Gitness and for your insight!
I think I understand the problem - currently we didn't have a way in gitness so that all created containers get added to a specific network or list of networks (this functionality does exist in drone: https://docs.drone.io/runner/docker/configuration/reference/drone-runner-networks/).
I think that small bit should solve this issue where that env variable can be added to the gitness container docker-compose (say GITNESS_CI_CONTAINER_NETWORKS=my_net) and then gitness and all created containers would run on the same network. So that, along with the set value of GITNESS_URL_CONTAINER should allow all containers to interact with gitness.
Let me know if the above makes sense - I do have a PR up for this and have done a basic sanity check but would be great to hear from you if you think there's a use case that's not being solved.  Thanks!

Hi folks, the above has been merged and will be available in the next release. We'll update the documentation with this use case. Let me know if you have any thoughts - thanks!

To fix my issue #3485 i use GITNESS_URL_CONTAINER to change route to external domain.
version: "3"
services:
  gitness:
    image: harness/gitness
    restart: unless-stopped
    volumes:
      - data/gitness:/data
      - /var/run/docker.sock:/var/run/docker.sock
    environment:
      - GITNESS_URL_BASE=https://gitness.${DOMAIN}
      - GITNESS_URL_CONTAINER=https://gitness.${DOMAIN}
    labels:
      - "traefik.enable=true"
      - "traefik.http.routers.gitness.rule=Host(`gitness.${DOMAIN}`)"
      - "traefik.http.routers.gitness.entrypoints=websecure"
      - "traefik.http.routers.gitness.service=gitness-svc"
      - "traefik.http.services.gitness-svc.loadbalancer.server.port=3000"

Gitness v3.0.0-beta.6 has been released.
Docs have been updated with GITNESS_CI_CONTAINER_NETWORKS and GITNESS_URL_CONTAINER variables.

Working on Gitness v3.0.0-beta.6
A note for podman-compose users (may apply for docker-compose ones)
The network that you create has the stack name on it, so for the GITNESS_CI_CONTAINER_NETWORKS you will need to keep that in mind, i.e
After that (under my scenario, the network is called gitness, so the value for the GITNESS_CI_CONTAINER is gitness_gitness 🤷 ) it works

Thank you all! Its great to have a self hosted CI environment integrated on the git platform.
<3

thanks for trying @ignaciolg ! I'll go ahead and close the ticket, let me know if you face any other issues.


 Not the most complex type classes out there, but these can still be useful (to @mpilquist in particular).

Wonderful! Thanks! Will merge ASAP (I'm still wavering on RC5).

There will be an RC5, so merging ... thanks again :-)


Image scaling issue on Chrome on mobile devices

Project images are not scaled properly when using Chrome on smartphones.
Firefox does not seem to be affected.
Reference image

Thanks for reporting it.
The problem was with long urls - just fixed it and released 1.0.7 version. Please update to the new version using yarn upgrade.


Slowly remove windows support

It's too much work, too much complex and it's not that useful.

Reopen because basilico need to be un-windowsed

@il-palmi There are exactly 2 IFs that add windows support, remove them and we're done asd


Reorganization

Closes #20

I don't think CANMetdata.c/.h should be in the scripts folder. Could they be moved to a CAN folder inside another directory? If it doesn't make sense then this is fine too.


Add --encryption to helm install mode

This enables the helm install to use the --encryption flag by moving the key creation into the shared pre-install hook.
This will un-block the work of bringing the cilium/cilium tests to Helm mode to test new features that require those mode like Spire and the Envoy DS.

I just now noticed there is a test matrix now for both modes, will add e2e tests

we are having an offline discussion regarding the security implications of this change. added  dont-merge/blocked label 🙏

Forgot this one existed, after long discussions we did not go with this approach. Closing it.


Player Game Information

I am new to this API and I was just wondering if there was a way to get the game information of a specific player. I know I can get all the statistics for the player but can I get the team this player played against?

Sure.  There are probably other ways, but a player's .home attribute will tell you if he was on the home team.
import nflgame
games = nflgame.games(year=2016, week=5, kind='REG')

for game in games:
    for player in game.players:
        print '{} ({}) played against {}'.format(player, player.team, game.away if player.home else game.home)

B.Gabbert (SF) played against ARI
...
J.Witten (DAL) played against CIN
...
J.Winston (TB) played against CAR


Please support angular2-meteor, thanks!

I tried to use tap:i18n in angular2-meteor project.
When I use like this:
client/home.ts
import { Component } from 'angular2/core';
@Component({
  selector: 'test-component'
  template: `<div>{{_ "hello"}}</div>`
})
export class TestComponent {}

i18n/en.i18n.json
{ "hello": "Hey there" }
It will show the error:

EXCEPTION: Template parse errors:
Parser Error: Unexpected token 'hello' at column 3 in [{{_ "hello"}}] in TestComponent

Oh, I think my direction is wrong. For angular2-meteor project, maybe ng2-translate is the right direction.


Recording as Mirror Mode

I want to record the video as mirror mode and I want to know if it has any options i can do about it.

@CherryChann you can mirror the video element during preview/recording. Add the following CSS (taken from https://stackoverflow.com/a/14455993):
video
{
    transform: rotateY(180deg);
    -webkit-transform:rotateY(180deg); /* Safari and Chrome */
    -moz-transform:rotateY(180deg); /* Firefox */
}

No feedback, closing.

@thijstriemstra are there any hooks for where to put the canvas flipping?
For example, this relies on Whammy js, which uses a canvas. But how do we manipulate pixel data? i've tried on the record canvas, but did nothing to the final blob.


Tracking: Seattle FreshBucks launch

Eng tracking for the FreshBucks launch:

[ ] General Eligibility support #3744
[x] Multiple Question Predicates #3950
[ ] Address #4051
[x] ---- Connection to Predicate system #4028

@seanmalbert can you make a Issue/tracking-Issue/Pr for the address stuff and link it above please

No longer tracked here.


Unable to access chart.print()

Unable to recreate same on react-highcharts
http://jsfiddle.net/gh/get/library/pure/highcharts/highcharts/tree/master/samples/highcharts/members/chart-print/
Can you please help?

I can't call this method too. I see example
let chart = this.refs.chart.getChart();
chart.series[0].addPoint({x: 10, y: 12});

then go to the API and see that addPoint() method is in the Series section. Fine.
I want to repeat this. So I find print() method in the Chart section in API and print this code
chart.chart[0].print();

But it's didn't work. What am I doing wrong?
See the screenshot https://prnt.sc/gncvwa

Closing and there has been no activity in this issue for a while.
if this is still relevant, please feel free to reopen


AttributeError

I'm trying to follow a tutorial by Tensorflow on image classification. I've gotten up to the step where you retrain the model. I'm trying to run this retraining script using terminal, but I'm getting the following error:

Tutorial: https://www.tensorflow.org/tutorial
Retraining script: https://github.com/tensorflow/hub/blob/master/examples/image_retraining/retrain.py
Terminal: https://github.com/llSourcell/tensorflow_image_classifier/blob/master/src/train.sh

This looks similar to the issue covered by this thread:
https://github.com/tensorflow/tensorflow/issues/18889.
On Thu, Jun 7, 2018 at 5:15 PM Shreya<EMAIL_ADDRESS>wrote:

I'm trying to follow a tutorial by Tensorflow on image classification.
I've gotten up to the step where you retrain the model. I'm trying to run
this retraining script using terminal, but I'm getting the following error:
[image: screen shot 2018-06-07 at 10 03 12 am]
https://user-images.githubusercontent.com/16789305/41109039-a3029062-6a3b-11e8-99e9-a24655fca6f3.png
Tutorial: https://www.tensorflow.org/tutorial
Retraining script:
https://github.com/tensorflow/hub/blob/master/examples/image_retraining/retrain.py
Terminal:
https://github.com/llSourcell/tensorflow_image_classifier/blob/master/src/train.sh
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/hub/issues/69, or mute the thread
https://github.com/notifications/unsubscribe-auth/AbunTGbJ-qYIXlbvnQBX2u1pgC-_i2_sks5t6UOGgaJpZM4UenCM
.


Cygwin : watir-6.18 issue with selenium-webdriver-3.142.7

Meta -
Watir Version: watir-6.18
Selenium Version: 3.142.7
Browser Version:
Browser Driver Version:
OS Version: Cygwin, windows 10
Expected Behavior -
Actual Behavior -
selenium-webdriver-3.142.7/lib/selenium/webdriver/common/platform.rb:122:in `tr': no implicit conversion of nil into String (TypeError)
issue caused by the line : "full_path = unix_path(full_path) if windows?"  (should be replaced by : full_path.tr!('\', '/') if windows?)
Steps to reproduce -

On Sat, Mar 6, 2021 at 16:17 mora-01<EMAIL_ADDRESS>wrote:

Meta -
Watir Version: watir-6.18
Selenium Version: 3.142.7
Browser Version:
Browser Driver Version:
OS Version: Cygwin, windows 10
Expected Behavior - Actual Behavior -
selenium-webdriver-3.142.7/lib/selenium/webdriver/common/platform.rb:122:in
`tr': no implicit conversion of nil into String (TypeError)
issue caused by the line : "full_path = unix_path(full_path) if windows?"
(should be replaced by : full_path.tr!('', '/') if windows?)
Steps to reproduce -
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/watir/watir/issues/915, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5RBCBXOWUYQPEGYSBUMCTTCKLXBANCNFSM4YXDVBBA
.

--
Best Regards,
Vitaly Kovrigin
1<PHONE_NUMBER>
<EMAIL_ADDRESS>

This is in the Selenium code, not Watir code.
#unix_path is doing the substitution you suggest based on what the File values are supposed to be on a windows machine.
Can you verify what File::SEPARATOR & File::ALT_SEPARATOR are on your system, and then file a Selenium bug?


(tortoisegit) Skip install when the packaged version is already installed

Description
This changeset implements an install-time version-check for tortoisegit, intended to allow package installs/upgrades to succeed when the packaged software version is already installed. This should gracefully handle both new package installs for pre-existing installations, as well as software upgrades that happen outside of Chocolatey.
Motivation and Context
Fixes #2159.
I recently found myself in this situation when TortoiseGit itself prompted me to update to v<IP_ADDRESS>. Seeing an update prompt in Chocolatey-managed software normally triggers me running choco outdated --ignore-pinned to check if an updated package is available from the Community Repository.
As no such package was available at the time, I opted to upgrade the software manually, with the understanding that I'd pick up an updated package later when it became available. When an updated package for TortoiseGit became available. I tried to upgrade and ran into the documented issue.
Implementing this for behavior consistency with some other community packages (e.g. brave, firefox, flightgear, googlechrome, inkscape, opera, thunderbird), as well as to prevent the need for workarounds (e.g. using the --skip-powershell switch, waiting for the next package release, etc.) to get the software and package versions synchronized.
How Has this Been Tested?
Environments
Dev

OS Build: Windows 11 Pro v10.0.22621.0 (64-bit)
PowerShell version: 7.3.3
Chocolatey version: 1.3.0

Chocolatey Test Environment

Vagrant Box Version: 3.0.0
VM Provider: Virtualbox (although practically Hyper-V, since I manually converted the resulting virtual disk)
OS Build: Windows Server 2019 Datacenter v10.0.17763.0 (64-bit)
PowerShell version: 5.1.17763.3770
Chocolatey version: 1.3.0

Steps
Create a Test Package

Revert tortoisegit.nuspec to fa29f721dad264ebc38ce59a20c54372cfc8cb94.
Manually change newly introduced $softwareVersion variable in tools\chocolateyInstall.ps1 to '<IP_ADDRESS>'.
Executed .\update_all.ps1 -Name tortoisegit -ForcedPackages tortoisegit to download software installers, test update.ps1 changes, and create a test package based on v<IP_ADDRESS>.

Clean Install Test

Note
TortoiseGit (both the package and software) is assumed to be currently uninstalled.

Install the test package version (i.e. choco install tortoisegit --source="'.;https://community.chocolatey.org/api/v2/'" [to prioritize local packages while sourcing dependencies from the Community Repository]).
Confirm software and package install succeeds without errors.

Existing Install Test

Uninstall the existing package (i.e. choco uninstall tortoisegit).
Manually download and execute TortoiseGit v<IP_ADDRESS>'s 64-bit installer.
Reinstall the test package version (i.e. choco install tortoisegit --source="'.'").
Confirm the package skips installing TortoiseGit, and the package install succeeds without errors.

Chocolatey-Managed Upgrade Test

Downgrade to a previous version of TortoiseGit (e.g. choco install tortoisegit --version=<IP_ADDRESS> --allow-downgrade).
Upgrade to the test package version (i.e. choco upgrade tortoisegit --source="'.'").
Confirm that both the software and package upgrade succeeds without errors.

Upgrade Catchup Test

Downgrade to a previous version of TortoiseGit (e.g. choco install tortoisegit --version=<IP_ADDRESS> --allow-downgrade).
If not already installed, install the git.install package to gain access to TortoiseGit's Settings dialog (e.g. choco install git.install).
In File Explorer/Desktop, right-click on a blank space, browse to and press the TortoiseGit > Settings context menu item.
In TortoiseGit's Settings dialog, on the General page, in the TortoiseGit groupbox, press the Check now button.
In the Check For Updates dialog, press the Download button.
When the installer download completes, press the Install button.
Manually step through the installer to completion.
Upgrade to the test package version (i.e. choco upgrade tortoisegit --source="'.'").
Confirm the package skips installing TortoiseGit, and the package upgrade succeeds without errors.

Forced Reinstallation Test

Force reinstallation of the test package in non-silent mode (i.e. choco install tortoisegit --source="'.'" --force --not-silent.
Confirm the installer is started without issues.
Browse to the next page in the installer, confirm the user is prompted to Modify/Repair/Remove the current installation.
To gracefully progress the "installation", select the Modify or Repair option. Step through the installer to completion.
Confirm that both the software and package "installation" succeeds without errors.

Screenshot (if appropriate, usually isn't needed):
N/A
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] I have updated the package description and it is less than 4000 characters.
[x] All files are up to date with the latest Contributing Guidelines
[x] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

:white_check_mark: Package verification completed without issues. PR is now pending human review

@brogers5 your changes have been merged, thanks for your contribution 👍

Just curious, does this update fix the Brave issue below, where Chocolatey does two version checks, the first one is incorrect and stale, as Brave auto-updates can't be disabled, so it will 100% of the time update outside of Chocolatey under normal use-case of daily browser usage.  However, a secondary version check is correct, but only AFTER the time, bandwidth, and disk space has been wasted on an unnecessary download.  Ideally, there should be a single, accurate version check, and only download anything if it really is necessary.  This scenario may effect other software packages, but this is the situation I encounter most frequently.
You have brave v1.49.132 installed. Version 1.50.114 is available based on your source(s).
Progress: Downloading brave 1.50.114... 100%

brave v1.50.114 [Approved]
brave package files upgrade completed. Performing other installation steps.
Checking already installed version...
WARNING: Skipping installation because version 1.50.114 is already installed.

This update is for the tortoisegit package @Leif-W

@Leif-W the first line

You have brave v1.49.132 installed. Version 1.50.114 is available based on your source(s).

is letting you know that the brave package v1.49.132 is installed, and that the package v1.50.114 is available.
Then Chocolatey CLI downloads and unpacks/installs the files inside the package. Then the chocolateyInstall.ps1 in the brave package is run. The line

WARNING: Skipping installation because version 1.50.114 is already installed.
comes from the chocolateyInstall.ps1, and is from the script check the Brave software is already up to date.

One check is to update the package, the other check is the package checking if the software is up to date.
If you want to discuss futher, I would suggest the chat, https://ch0.co/community


"Go To Definition" not working as expected.

Coming from PyCharm, Go to Defintion is not working as expected. Usually going to package root works - but not functions or constants etc. When it does work it just takes me to __init__.py of the package root.
MacOS 10.14.4
Visual Studio Code 1.34.0
PTVS: 2019.5.18875
FWIW I'm using pipenv.
What am I doing wrong? This is detrimental to productivity.

Thanks for bringing this to our attention, @BeyondEvil.  We definitely want to help resolve this.  It's unclear from your description what the problem is exactly.  Can you provide example code that demonstrates the problem?  I'd guess it would look something like this:
project/
    pkg/
        __init__.py
        spam.py
        eggs.py
    script.py

# project/pkg/__init__.py
import os.path
ROOT = os.path.dirname(__file__)
__version__ = '0.1.0a1'

# project/pkg/spam.py
from .eggs import Eggs
class Spam:
    eggs = Eggs()

# project/pkg/eggs.py
class Eggs:
    n = 42

# project/script.py
import pkg
from pkg import eggs
from pkg.spam import Spam

print(Spam.eggs.n)
print(Spam.eggs == eggs.Eggs)
print(pkg.__version__)

If I "Go To Definition" on any of the names in script.py then it does the right thing for me.

Here's an example repo
e2e/utils/database.py
None of the self.db.<function> works.
e2e/tests/conftest.py

In from api.base import Base .base and Base works. But not api. In PyCharm clicking api will highlight the directory - which is pretty useful in a large project. Same goes for all from-import imports.
.authenticate, .add, .node.get_closest_marker, item_class.create, and .find does not work. .create will in PyCharm show a list of "Multiple Implementations". Note that User.create works.

e2e/tests/api_tests/test-api.py

from api import apps, fixtures, orgs nothing works.
In orgs.Org types, orgs doesn't work. (Org does work)
.get doesn't work.

e2e/tests/api/*.py

from api.fixtures import Fixture same as in e2e/tests/conftest.py for that type.

Note: I've listed the ones that work for me in PyCharm, but not in VSC.
I hope that makes things a little clearer! Let me know if there's anything else I can provide and/or try!
Thanks!

Thanks for the info, @BeyondEvil.  It was super helpful.   After some tweaks I was only able to reproduce some of the issues.  Please let us know if you can't get the results I got, or if I missed anything (i.e. not covered by the 2 issues I opened).
To start off, I did the following:

clone the repo
run python3.7 -m pipenv --python 3.7
run python3.7 -m pipenv install
open the repo's root directory in VSCode
open e2e/utils/database.py  (the extension automatically selected the pipenv virtualenv)
right click on name
click on "Go to Definition"

At that point I was only able to check one of the items (for e2e/utils/database.py) and I didn't have any problems there.
For the rest I ran into the fact that the "e2e" directory is not on sys.path, so every import of the "api" module was an "unresolved import".  (The extension only adds the project's root directory to PYTHONPATH.)  To work around this I added a top-level .env file with PYTHONPATH=e2e in it.
Below are my final results, post-tweak.  For some of the items it worked fine for me.  I have opened 2 issues (#6312 and #6314) for the remainder.
Note that all this was using the language server (rather than Jedi).  If you are using Jedi (the default), then trying add "python.jediEnabled": false to your settings.json file.

e2e/utils/database.py
** "dadd" (of self.db.dadd(... on line 10) worked with just the initial steps
e2e/tests/conftest.py

"api" in from api.base takes me to the definition for api.base; I've opened a feature request for what you expected
this would require type inference on arbitrary parameters, which we currently do not support;  if I explicitly type-annotate the params then "Go to Definition" works; I've opened a feature request

e2e/tests/api_tests/test-api.py

"api" says "no definition found" (see #6312); "apps", "fixtures", and "orgs" all work
"orgs" took me to the import (as expected); "Org" worked
"get" says "no definition found" (see #6314)

e2e/tests/api/*.py
(assuming you meant e2e/api/orgs.py)

same as for e2e/tests/conftest.py

Can I add the following issue I'm having:
Importing modules does sometimes work, sometimes it doesn't.
Zo say I have:
module1
  __init__.py
  constants..py

and in my code I do:
from module1 import constants

LOCAL_CONSTANT = constants.CONSTANT

Then using go to definition on constants opens up the init.py file (which is empty).
Using go to definition on CONSTANT does bring me to the right file and location.

@hvdklauw, please open a separate issue.  Thanks!

Closing in favour of https://github.com/microsoft/python-language-server/issues/1412


[Bug] consecutive-empty-lines should ignore comments in the end of keyword/test

Following code:
*** Keywords ***
Keyword 1
    Step

# comment 1

    # comment 2
    # comment 3

# comment 4

Keyword 2
    Step

reports consecutive-empty-lines twice on line 6, and then on line 10 (counting 4 consecutive empty lines even if there are comments in between).

This will also fail while there are no consecutive empty lines:
Keyword 3
    Step
    # comment

    # comment

    Step

I think we should redesign empty lines rules fully.

Relates #611
The changes will also affect empty-lines-between-keywords and empty-lines-between-test-cases (due to fixed handling comments).


Get temperature from sensor

Hi, I'm currently working with a FLIR lepton 2.5 sensor.
My software is based on raspberrypi_video and works well on the Raspberry Pi 3b, I would like to know if there is any way to generate a relationship between the image and the maximum and minimum temperatures.
I currently get from the functions:
LEP_GetSysFpaTemperatureKelvin(...)

get 30511 (305.11 K)

LEP_GetSysAuxTemperatureKelvin(...)

get 30460 (304.60)

both should be expressed in Kelvin.
I look forward to suggestions and thank you in advance.
You can view my source

Hello frank1789,
i didnt test your code, but as i see, your code is in C programming language.
do you have the code for python, because i run it in raspberry pi B?

Thanks for the reply, unfortunately I have no python code available to carry out the tests.
I also forgot to specify to use branch "feature/temperature" from git repository.

i use the code from Frank
LEPTON 2.zip
But, there is little problem in my project (Lepton 2.5 with board V1.4 + Raspberry Pi 3 B)
my schematic is

CS = SPI0 CS0 (Pin 24)            # Use spidev0.0
MOSI = SPI0 MOSI (Pin 19)
MISO = SPI0 MISO (Pin 21)
CLK = SPI0 SCLK (Pin 23)
GND = GND (Pin 14)
VIN = 3.3V PWR (Pin 17)
SDA = I2C1 SDA (Pin 3)
SCL = I2C1 SCL (Pin 5)

According with the suggestion above, i make some change in ....
*** LeptonThread.cpp**

const char *LeptonThread::device = "/dev/spidev0.0";   # (can change in spidev0.1)
unsigned int LeptonThread::speed = 18000000;            # (from 16000000)
qDebug() << resets << "reset," << errors << "errors";    # (before this line is commented)
*** LeptonThread.h**
#define DEBUG_LEPTON true                                       # (before this line is false)

when i run the Lepton-CaptureRaw, the result is good like this, but...

but..., sometimes when i try 10 times run this i can video it the thermal video and when i press the button "Snapsot" it will give me the .jpg picture (1-rgb.jpg, 2-rgb.jpg) and .bin format (1-raw.bin, 2-raw.bin) that this raw format will show a temperature value (i'm not yet with this) but 2 times will fail.
The error is "Error transfering SPI packet" and the layer full of red. Can you help me with this? why this happen?

project (Lepton 2.5 with board V1.4 + Raspberry Pi 3 B)
I use the following:

Raspberry GPIO
Breakout Board

(Pin 1) +3.3V
VIN

(Pin 3) SDA
SDA

(Pin5) SCL
SCL

(Pin6) GND
GND

(Pin 19)MOSI
MOSI

(Pin 21) MISO
MISO

(Pin 23) SCLK
CLK

(Pin 24) CE0
CS

This configuration works well and avoids the red rectangle on the screen most of the time;
+I preferred a method that would automatically decide whether to use:
"/dev/spidev0.1" or "/dev/spidev0.1";
I can't explain why after red sessions I can't explain, but it is my opinion that some camera register or on the breakout board is not reset correctly and that it retains memory. Or some counter overflows. To solve this problem, just disconnect the camera from the socket and re-insert it after 10[s].


bug 59227: parse dates formatted in Japanese or Chinese.

bug 59227: parse dates formatted in Japanese or Chinese. Change javac source encoding from ASCII to UTF-8 (same as build.gradle). Patch from jzhao. This closes #48 on Github.

LGTM


Fix dataset links in README

Looks like the hs_pbmc dataset may not still be accessible at the old link. Suggestion: replace with the hs_pbmc3 dataset?
Also, more generally, let's start using these issues for 'to do' items

Updated readme, but the jupyter notebook is still using the old dataset

Updated readme, but the jupyter notebook is still using the old dataset

There's the zip containing datasets and linked in the examples section of the README, so maybe this is OK.


Configurable TimeCharacteristic for test environment

I'm trying to test some CEP with flink-spector, but I can't make it work.
My test is shown below. This test passes although it should not (the expected record should not match the actual one). I'm wondering if the CEP processing is applied correctly...
The code is in Scala, but I'm using the Java DataStream classes.
@Test
  def testWithFlinkspectorAndJavaStreams(): Unit = {
    val event1 = Event("xxx", "", "message")
    val event2 = Event("xxx", "", "web portal")
    val event3 = Event("xxx", "", "message")
    val event4 = Event("xxx", "", "message")
    val event5 = Event("xxx", "", "call")

    val input = createTestStreamWith(event1).emit(event2).emit(event3).emit(event4).emit(event5).close()

    val pattern = Pattern.begin("event0").subtype(classOf[Event]).where(new EventActionCondition("message")).next("event1").where(new EventActionCondition("call"))

    val patternStream = CEP.pattern(input, pattern)
    val alertStream: DataStream[String] = patternStream.select(new AlertPatternSelectFunction())

    val output: ExpectedRecords[String] = ExpectedRecords.create[String]("Got one !")
    assertStream(alertStream, output)
  }

With the util classes:
class EventActionCondition(action: String) extends IterativeCondition[Event] {
  override def filter(value: Event, ctx: IterativeCondition.Context[Event]): Boolean = {
    value.action == action
  }
}

class AlertPatternSelectFunction extends PatternSelectFunction[Event, String] {
  override def select(pattern: util.Map[String, util.List[Event]]): String = {
    "got it !"
  }
}

I will have a detailed look at this later. I must confess I've never tested Flinkspector with CEP because I assumed It's producing the same results as the rest of the transformations.

Ok, I found what was the problem. My events do not have a timestamp and the stream time characteristic in the test environment is set to EventTime.
This is set in the initialize() method, and can not be changed.
@org.junit.Before     public void initialize() throws Exception {         testEnv = DataStreamTestEnvironment.createTestEnvironment(1);         testEnv.setStreamTimeCharacteristic(TimeCharacteristic.EventTime);     }

Aaah, this is something which only happens when using CEP. Instead of changing the stream time charateristic you can instruct Flinkspector to provide timestamps.
Change this line:
val input = createTestStreamWith(event1).emit(event2).emit(event3).emit(event4).emit(event5).close()

to
val input = createTimedTestStreamWith(event1).emit(event2).emit(event3).emit(event4).emit(event5).close()

and Flinkspector will provide timestamps.
This works for me and the test fails as expected. But I'm not familiar with cep so I'm not sure if this solution is sufficient.

Yes, it's working fine. Thanks !

This issue has been bundled into #57


Make VectorRefIterator clonable

I want to iterate over the line segments of a contour. Preferably it can be written as
contour
    .iter()
    .zip(contour.iter().cycle().skip(1))
    .for_each(|(p1, p2)| /* do something*/)

However, unlike std::slice::Iter, VectorRefIterator has not implemented Clone trait so we can't call .cycle() on it. It would be nice to add #[derive(Clone)] to VectorRefIterator.

Hi, thank you for the suggestion! It seems like an easy fix, can you please try making a PR for that?

Hi, thank you for the suggestion! It seems like an easy fix, can you please try making a PR for that?

@twistedfall Certainly.