add ProtGPT2 2022 paper?

Hi,
Thanks for putting this list together. I think you could also include the following 2022 paper:
"ProtGPT2 is a deep unsupervised language model for protein design"
https://www.nature.com/articles/s41467-022-32007-7
Cheers,
Thomas

Thanks; we have listed it


Simple walker

Adds a new walking function for handling shorter walks.
Its purpose is to get in viewing range of an object to be interacted but is more limited in scope (no teleports/shortcuts).
While the normal walker does work, it will always insist on a yellow click move to finish off before returning. Which this alternate walk loop does not.
Also modifies the agility plugin to utilize this short walk function.
This PR is a draft for now I will modify the agility plugin with a config to use this new walker or not.
I have added the config option that determines whether or not to use the new walker.
See this line: Rs2Walker.java#L313

Doesn't Rs2Walker.walkFastCanvas(World point) or Rs2Walker.walkFastLocal (Local point) achieve this?

seems to be no need to have a short version of the walker


GUI applications (eg gedit and vscode) do not use same cursor theme as host

Describe the bug
When using nix GUI applications the mouse cursor is not the same as the host system.
Steps To Reproduce

Install nix on Ubuntu 20.04 via https://nixos.wiki/wiki/Nix_Installation_Guide#Single-user_install or with Ubuntu 21.10 by using apt install nix-bin
Then install eg pkgs.gnome.gedit or pkgs.vscode
Open one of the apps and notice that the cursor is not the same as the host

Expected behavior
I expected at step 3 for the mouse cursor to be Adwaita like my system.
Screenshots
Gedit - nix

Gedit - flatpak

VSCode - nix

VSCode - snap

Additional context
Do i need to configure the gtk or xcursor somewhere ? I've tried installing the apps with nix-env directly and also via home-manager. I have also then tried installing pkgs.gnome.adwaita-icon-theme and using the following configuration in home.nix but that doesn't appear to help.
  xsession.pointerCursor = {
    package = pkgs.gnome.adwaita-icon-theme;
    name = "Adwaita";
  };

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.11.0-38-generic, Ubuntu, 20.04.3 LTS (Focal Fossa)`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.16`
 - channels(andrew): `"home-manager, nixpkgs-21.11pre324835.1cab3e231b4"`
 - nixpkgs: `/home/andrew/.nix-defexpr/channels/nixpkgs`

Still an issue.


Editor: Copy session context from current tab when open new tab

It's very common to want to query something different from the same context (same catalog settings etc.) and hence opening a new tab.
But this loses all context now so one have to setup the context from scratch.
Copy the current context to the new context. (copy constructor on QuerySession?)

Moved to Queryeer


Rename start & end properties to fix conflict with RN 51

A recent commit in RN added start/end properties to RCTViewManager.
Because LinearGradient inherits from this class and has properties of
the same name, RN gets mixed up and tries to convert to a YGValue
instead of CGPoint, which results in the exception:
Error setting property 'start' of BVLinearGradient with tag #18: JSON
value '{
    x = 0;
    y = 0;
}' of type NSMutableDictionary cannot be converted to a YGValue.

This PR renames the values to avoid conflict, but keeps the JavaScript component API the same, thus preventing a breaking change.

Please accept-it, I need it !

I just did the same thing for Android here FYI:
https://github.com/react-native-community/react-native-linear-gradient/pull/242

#243

+1 Please I need it too.

Hey folks, I have updated this PR to support android as well. I know there have been several follow up PRs since my original submission and it's gotten a bit confusing -- hopefully this now takes care of everything and we can get it merged.

Can you please merge this? Thanks!

Also need this! Just updated to RN 0.51

just FYI for @kevinvangelder : you'll need to update #145 once this is merged before that PR can be merged.

Trying to yarn upgrade this to no avail... should there have been an accompanying version bump, or am I doing something wrong? 😅

But, We also need a new release for this pr.

For the impatient, just add the git url to your package.json:
"react-native-linear-gradient": "git+https://github.com/react-native-community/react-native-linear-gradient.git",

Released 2.4.0

also getting the same issue here with
└─<EMAIL_ADDRESS>└─ react-native: 0.57.7


Steam Integration

Looking for the ability to use the launcher through steam by launching the EXE as hardlink named RCT2.EXE (to trick steam). I can launch the launcher this way already, but if I launch a game through it it wont switch the steam overlay to the game. Been looking to get automatic updates working while having steam overlay working but the old third party launcher nor this launcher have had the ability to do so yet. It could work similarly to how Cyberpunk 2077, Cities: Skylines and a few other games display a launcher when opening the game in steam before fully launching the game.
Cheers

it seems like it's already working! I forgot to set this computer up to use OpenGL as drawing engine for OpenRCT2. My apologies


Minor multi gpu doc improvement

just added torch.no_grad and eval()

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Allow separators to have buttons

This is a core change only. Not planning on bringing this into extensions for now to see if this a route we want to take.
Without hover:

With hover:

When you go down and up, these items in the list are skipped unless you hit tab while the inputbox has focus, you'll focus the list where you can then use the arrows and tab to get to the buttons:

gif demo:

also I'm not including that Snippets UI in this PR... but we can consider that after this goes in.

Cool! Very promising. I know this isn't done, but dropping some early thoughts just looking at the gif:

I can help with the header styles + typography. Either in this or a separate PR.
I think we need to make it so the header doesn't show a different color on hover. It feels too much like a peer of the children items. I'm also not crazy about allowing focus on the header itself in this context. It kind of makes sense in the view title bar/toolbar example below, but only because it has a collapse/expand interaction. Is there a way we could simply tab focus right into the actions vs. the whole header item? I'll try to whip up a prototype this week to see how it feels.


How to select multiple columns to put the field

I searched a lot
But I did not find a way to adjust the multi-column
How to put multiple fields together?
There is really no way to do this

What are you trying to accomplish?

Please refer to #401, #403, and this JSFiddle

Thank


[cdoc] pylon

Coverage decreased (-19.6%) to 50.367% when pulling 825c34677ed774d26b7885803eee3bfe34161f30 on YitongFeng:yitong/pylon-cdoc into 88fc3ec7102edb25697fae88f06279d23b3d253f on Microsoft:master.


Multiple subscriptions do not work with more than 2 subscriptions

https://github.com/binance/binance-connector-typescript/blob/8d3267389f9cff96863dac09bbee77514ea153ed/src/websocketStream.ts#L23
string.replace replaces only the first occurence of a symbol, so
sub1,sub2,sub3,sub4 will incorrectly be changed to sub1/sub2,sub3,sub4
Better alternative is
 if (Array.isArray(stream)) stream = stream.join("/");

I submitted a new pull request for this in https://github.com/binance/binance-connector-typescript/pull/26


Fix post-Lollipop shared elements transition animations

Fix this issue: https://github.com/chrisbanes/PhotoView/issues/243
The main problem was that PhotoView does not override getImageMatrix
method, and when ChangeImageTransform call image.getImageMatrix() it
will receive matrix not from PhotoViewAttacher, but from PhotoView's
parent ImageView.

Ah wow, nice. I suppose I will have to take your word for it that it fixes it. Maybe I'll take some time to write a sample that shows it working. But thanks a bunch for your contribution!

Yes, I highly recommend you to test it before releasing. It is possible that I return wrong matrix in PhotoViewAttacher. It looks right and working, but better recheck it twice.

I went ahead and tested this and it looks good. Will release 1.2.8 soon with this fix

Now available as 1.3.0


gouv.fr, co.uk, com.sg, other public suffixes are not being resolved as TLDs

We're running in to an issue (cypress-io/cypress#3717) caused by an update to parse-domain causing public suffixes to not be resolved as TLDs. gouv.fr, co.uk, com.sg, and other public suffixes are not being resolved as TLDs ever since 2.1.0.
const parseDomain = require('parse-domain')

console.log(
  parseDomain('dev.classea12.beta.gouv.fr', {
    privateTlds: true,
    customTlds: /^[\d\.]+$/
  })
)

With<EMAIL_ADDRESS>this results in:
{ tld: 'gouv.fr', domain: 'beta', subdomain: 'dev.classea12' }

With<EMAIL_ADDRESS>this results in:
{ tld: 'fr', domain: 'gouv', subdomain: 'dev.classea12.beta' }

Potentially related: #64, #65

Can you try out v2.1.8? Does it fix your issue?

No, still the same issue.

Ok, I can confirm the issue. It's in combination with the privateTlds: true flag. I need to look into it.

This fix shipped with v2.3.2 🚀


[SCB-718]If auto discovery failed, will cause a dead cycle

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.002%) to 86.201% when pulling 6a0cbfd33e992c00de7e9a907351cea4100b8973 on laijianbin:cache into 6aadae0de82b3b3940f0c163b815d6675aedc2bc on apache:master.

Can you explain how this happen? It seems that you mean when service registered is successful and then query instances of service center failed?


Feature suggestion: is it possible to make it useful for non-submodule repos?

My workspace is a bit untypical. The root directory is not a git repo, but provides commands to automate git clone and make several interdependent projects with specified branch name of the repo of these projects.
For instance, when Imake ProjectC, which depends on ProjectA and ProjectB, the root repo will perform git clone and set proper CMAKE_PREFIX_PATH to compile ProjectC. However, the cloned ProjectA/B/C are not really submodules to the root repo.
At the moment, this plugins does not work in my workspace as it relies on .git and .gitmodules if I understand it correctly. I'm wondering if it's possible to make it also work for my case?

You are correct, at the moment the plugin only relies on .git and .gitmodules of the current working directory.
In your scenario we'd be looking to implement something similar to this stackoverflow question.
I suggest the below:

If the current working directory is not a git repo => try to find git repos in subdirectories
Add additional default configs

find_subdirectories = {
    enabled = false,
    depth = 1

Will this cover your use case? Do you see anything else to add?

Totally. Thanks for replying and considering my suggestion. I actually forked your repo and make it independent of .git and .gitmodules in the root directory. I'm new to lua so I didn't make it to support options of different settings.
My another suggestion, if it is not something too difficult, would be to make it async, so I don't have to wait for the entire list to be created until the telescope prompt can be displayed.

Wow, this is great feedback and suggestions. Thank yo very much.
I will slowly start working on the suggested changes in my comment above. Shouldn't take too long, but I don't have too much free time these days.

Sorry, I also removed the previewer part to make the telescope appear faster, because what I really want is to display the repos and their branches, and be able to open any of them in lazygit as fast as possible.

This makes a lot of sense. I created a branch to add a config parameter to enable/disable the previewer. I will test it tomorrow on my work machine.

Speaking of speed, my another suggestion, if it is not something too difficult, would be to make it async, so I don't have to wait for the entire list to be created until the telescope prompt can be displayed.

I like the idea, I also think the logic I wrote could be a little bit more efficient (e.g. the crazy if elseif statement) to generate the submodule list.

I created a new branch and PR #5 for this feature, @patricorgi would you mind testing/reviewing it?

Hi, many thanks for implementing this! I saw your message a week ago, but I have been too busy recently to test it. Hopefully I can find sometime this week...


Add function to check if objectory is available

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

Project coverage is 100.00%. Comparing base (24b9582) to head (c3350a6).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #10   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         5    +1
  Lines           35        48   +13
  Branches         2         3    +1
=========================================
+ Hits            35        48   +13

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


build_sdk fails

The build_sdk fails due to a check to see if the workspace is clean.  Should I remove the generated files?
https://github.com/pulumiverse/pulumi-vra/actions/runs/3207733673/jobs/5242953795#step:17:1

@ringods do you have any suggestions for how to fix this?

@schmidtw the build setup generates the SDKs and warns if there are changes to what is committed to Git. If you build locally on your machine, do you have changed SDK files? If so, commit these to git. If not, please check if you are using the same Go version as used on Github Actions.

Thank you for the suggestions @ringods the go version was it.


Override correlation

Codecov Report

Merging #885 into master will decrease coverage by 0.02%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master     #885      +/-   ##
==========================================
- Coverage   61.95%   61.92%   -0.03%
==========================================
  Files          74       74
  Lines       12934    12946      +12
  Branches     2048     2052       +4
==========================================
+ Hits         8013     8017       +4
- Misses       4263     4267       +4
- Partials      658      662       +4

Impacted Files
Coverage Δ

autobahn/wamp/protocol.py
53.83% <0%> (-0.57%)
:arrow_down:

autobahn/wamp/types.py
72.99% <100%> (+0.35%)
:arrow_up:

autobahn/_version.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Camera considers North America to be home in 2D

Cesium's camera considers "home" in SceneMode.SCENE2D to be Camera.DEFAULT_VIEW_RECTANGLE, which roughly covers North America.
@bagnell thinks this should be Rectangle.MAX_VALUE to better correspond with Columbus View, which considers a view over the entire map to be "home."

This is working as expected.  We can revisit later if we decide to make a change to the "home" behavior


Unable to view details in grafana

Hi,
I am able to get the exporter running with the following command
./mongodb_exporter-linux-amd64 -mongodb.uri "mongodb://mongodb_exporter:s3cr3tpassw0rd@localhost:27017/" -groups.enabled="asserts,durability,background_flushing,connections,global_lock,index_counters,network,memory,locks,metrics" -mongodb.collect.replset=false -mongodb.collect.oplog=false
But every time I hit the metrics URL 'http://testmongo:9001/metrics' with grafana I get

MongoDB Exporter

MongoDB Exporter
Metrics

Though a cURL to http://testmongo:9001/metrics works fine.
Even tried setting up nginx and redirecting calls from http://testmongo:80/ to http://testmongo:9001/metric but still the same.
Am I missing something? I understand that this might be a grafana issue, but I face this only when running this exporter and not while monitoring other items exposed via GKE.
Thanks in advance for the help/suggestions.
Regards,
Vikas

Same issue on Grafana 5.2.0 and MongoDB 4.0.0.

@pvikas Can you please explain the installation steps. I am trying to setup on Linux machine which has no internet connectivity.
Thanks


Add steps to Release_Process.md, to update Docker image version for k8s

When deploying BigchainDB Server in a Kubernetes cluster, they should pull the latest bigchaindb/bigchaindb image from Docker Hub. I added a step to update the version number in k8s/bigchaindb/bigchaindb-dep.yaml as part of the BigchainDB Server release process.
@krish7919 and @sbellem Do you agree with the point where I added the step to update the version number of the Docker image?
Note: I didn't add a step at the very end saying how to update all running BigchainDB Server instances. That's generally something for others to do, not us (even though we also maintain some running instances of BigchainDB Server). (Release_Process.md is for us.)

Codecov Report

Merging #1482 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1482   +/-   ##
=======================================
  Coverage   98.38%   98.38%
=======================================
  Files          56       56
  Lines        2597     2597
=======================================
  Hits         2555     2555
  Misses         42       42

I am wondering whether we could simply pull the latest image, given that we always update the version number to the latest release (in https://github.com/bigchaindb/bigchaindb/blob/master/k8s/bigchaindb/bigchaindb-dep.yaml#L21).
In the following, given that release 0.10.2 is the latest
spec:
  spec:
    containers:
    - name: bigchaindb
      image: bigchaindb/bigchaindb:0.10.2

The lines:
      image: bigchaindb/bigchaindb:0.10.2

      image: bigchaindb/bigchaindb:latest

      image: bigchaindb/bigchaindb

are all equivalent.
So if we use either one of the last two we will not need to update the yaml file each time we do a new release.

Deployments always have to be explicit in specifying version, instead of implicit. It also helps in rollback, as you cannot roll back the the latest version.
Also, some practices here: http://blog.ploeh.dk/2013/12/10/semantic-versioning-with-continuous-deployment/


fontRef Error when drawing string with ReportLab

Link to PDF file (or attach file here):
font-bug.pdf
Tried with multiple files, writing number in bottom right corner with reportlab in python using following code:
c.setFillColorRGB(255, 0, 0)
c.setFont("Helvetica", 12)
c.drawString(width - 50, 10, text)

Configuration:

Web browser and its version: Firefox 49.0.2

Steps to reproduce the problem:

open the pdf file in the browser

What is the expected behavior? (add screenshot)
Expected to see bates number in bottom right.

What went wrong? (add screenshot)
It doesn't load the text and prints to console:
Warning: fontRef not availablepdf.worker.js:2298:5
Warning: Error during font loading: Font F1 is not availablepdf.js:277:5
Warning: fontRef not available

Closing since this one is fixed now.

Closing since this one is fixed now.


Add version: LuaLS.lua-language-server version 2.6.8

Pull request has been created with Komac v1.9.1 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116133&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


android networking library - retrofit

https://square.github.io/retrofit/
http://www.truiton.com/2015/04/android-volley-vs-retrofit-better-approach/
http://vickychijwani.me/retrofit-vs-volley/

CRUD API ( GET, POST, UPDATE, DELETE ) call using retrofit library from Android and display in UI..
Material design UI (https://material.io/guidelines/material-design/introduction.html)

@prosenjitdas-technicise

CRUD API ( GET, POST, UPDATE, DELETE ) call using retrofit library from Android and display in UI..
Material design UI (https://material.io/guidelines/material-design/introduction.html)
https://github.com/sayan801/PythonSamples/issues/19

@rupak-technicise

this is what i have did. As progress will move, i will keep you updated accordingly

useing slim framework i was tried to create a API but its not working
by creating this framework i follwed this website-https://www.simplifiedcoding.net/php-rest-api-framework-tutorial-part-2/
after that i will tried to create a api without using any framework,
i created a api, that working ,CRUD API ( GET, POST, UPDATE, DELETE ) all are working on postman.
Material design UI are completed.
studied the retrofit library and understand the structure, but coding are confusing for me.this library is not cleared for me
Can i json parsing without using this library?

@prosenjitdas-technicise where is your Material design UI?

previous implemented project which one i was created using normal php api,there i am use the Material design.
ok i send that project ()but that project does not use  retrofit and slim.
today i send a project by using android networking library - retrofit.


Issue when installing

Please help me to do with this problem when installin library
Windows 7, Python 3.6

windows 10
python37(4)
cmd - pip install flatlib
i get error, seems pyswisseph will not compile. pyswisseph is outdated (pip version of pyswisseph = 2.00.00, github version of pyswisseph = 2.05.01; sweph = version 2.08
i can not compile neither of them.
is flatlib development active?
simon
s love nia

Flatlib requires a version of pyswisseph>=2.00.00-2 (as in requirements.txt), so it should install the latest pyswisseph on pypi. However you can install the latest pyswisseph from github using pip install git+https://github.com/astrorigin/pyswisseph.
As for flatlib, I update it as I need it. I've been working actively in supporting the sidereal zodiac in another branch (https://github.com/flatangle/flatlib/tree/sidereal).


Blockquotes look a bit intense

Thanks a lot for the css, I find it very helpful!
Blockquotes are not suited to quote more than a couple of words, as they are very large.
Is that by design?

@kholbekj sorry for the late reply. I didn't know people are actually looking at this. So cool to find an issue here.
Yes, the blockquote is intended for pretty much a Pull Quote from the graphic design sense, since this framework is dedicated to a typographer and graphic design in general.
Obviously, if you use this, treat it like a reset. Since I wrote everything with :where, the specificity is low, you can easily write your own CSS to change any styles.


issue #153 add results to Check title

This adds the enhancement mentioned in issue #153
It aggregates the passed, failed, and skipped counts for all TestRunResults and adds them to the output.title used in the check run.

@CoderDennis thanks for the contribution 👍
Having the number of passed/failed tests in the check run title is a good idea.
I'm just thinking about alternative phrasing of the text:

Tests passed: XX - remove the failed: 0, skipped: 0 when we have only passed tests
Tests failed: XX, passed: XX, skipped: XX - Write number of failed test first as it's the most important information

What's your take on this?

Similar already merged


Javascript challenge throws exception "Input string was not in a correct format."

On the website https://kissasian.sh/, CloudflareSolver fails with the following exception several times before being able to pass the challenge
[5:22:25.419] EXCEPTION SOLVING CLOUDFLARE -> Input string was not in a correct format. ->    at System.Number.ParseDouble(String value, NumberStyles options, NumberFormatInfo numfmt)
   at CloudflareSolverRe.Types.Javascript.JsFuck.DecodeNumber(String encodedNumber)
   at CloudflareSolverRe.Types.Javascript.JsCalculation.Solve()
   at CloudflareSolverRe.Types.Javascript.JsChallenge.ApplyCalculation(Double number, IJsCalculation calculation)
   at System.Linq.Enumerable.Aggregate[TSource,TAccumulate](IEnumerable`1 source, TAccumulate seed, Func`3 func)
   at CloudflareSolverRe.Types.Javascript.JsChallenge.Solve()
   at CloudflareSolverRe.Solvers.JsChallengeSolver.<SolveChallenge>d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CloudflareSolverRe.Solvers.JsChallengeSolver.<Solve>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CloudflareSolverRe.CloudflareSolver.<SolveJavascriptChallenge>d__35.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at CloudflareSolverRe.CloudflareSolver.<SolveWithJavascript>d__33.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CloudflareSolverRe.CloudflareSolver.<Solve>d__32.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CloudflareSolverRe.CloudflareSolver.<>c__DisplayClass28_0.<<Solve>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CloudflareSolverRe.Utilities.SemaphoreLocker.<LockAsync>d__1`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at CloudflareSolverRe.CloudflareSolver.<Solve>d__28.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at DecipherAIO.Program.<SolveCloudflare>d__2.MoveNext() in C:\Users\matti\source\repos\DecipherAIO\DecipherAIO\Program.cs:line 189

My code:
                var target = new Uri("https://kissasian.sh/");

                var cf = new CloudflareSolver
                {
                    MaxTries = 3,
                    ClearanceDelay = 5000
                };

                Logger.Warn($"SOLVING CLOUDFLARE");
                var result = await cf.Solve(siteUrl: target);

                if (result.DetectResult.ToString().Equals("NoProtection"))
                {
                    Logger.Success($"NO CLOUDFLARE DETECTED");
                }

                if (result.Success)
                {
                    Logger.Success($"SOLVED CLOUDFLARE");
                    break;
                }
                else
                {
                    Logger.Error($"ERROR SOLVING CLOUDFLARE -> {result.FailReason}");
                }`

Some here, for https://yts.unblockit.one/

I tested both URLs with #29 and they work well.


Fix display on website

Appearance of Notes and Tips column did not match expectation
This PR attempts to

Fix broken table formatting
Fix Notes and Tips column

Codecov Report
Base: 76.58% // Head: 76.58% // No change to project coverage :thumbsup:

Coverage data is based on head (1389641) compared to base (25c4993).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #168   +/-   ##
=========================================
  Coverage     76.58%   76.58%
  Complexity      832      832
=========================================
  Files           119      119
  Lines          2579     2579
  Branches        328      328
=========================================
  Hits           1975     1975
  Misses          477      477
  Partials        127      127

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Training Code Availability

Dear author,
Thank you for your excellent work and public code! However, I found no training code(including skeleton transformation code) in the current code release. Only data-preprocessing, evaluation and network backbone code are available.  I'd be very grateful if you can release the code for skeleton transformation and network training.
Thank you.

Hi @journey1234-liu , thank you for your interest in our work! I recently relocated, and part of the transformation code is currently used in a journal manuscript under review. I'll upload the skeleton transformation and training code after that. Let's keep this issue open until uploading is complete.

@reckdk Thanks for your reply! I can't wait to see the complete code release. Wish you success on the manuscript review!


Fix test cases

Test Cases:
Provide unit tests for your model classes. Provide intent tests for the requirements you support. Deliver the test code to your source repository. If you have test data files, also include those. Test data should be realistic.
Remove owner and borrower collection

I fixed NotificationTest on my branch to accommodate for the new change in the Notifications class

I have fixed the test cases of book and request, which can be seen when merging #92 with master.


Make digital traces match memmap dtypes in neo.

@zm711

This works for me once tests pass. You want it to be draft for more discussion on SI?

Na, man, your call as a neo mantainer.  I have a slight push to got for the performance improvement but I think you are right that this might better (and is easier). We should close the spikeinterface issue once we merge this.
As an aside, I never thought that it makes sense to run a test again when you mark something from draft to ready.


Updating the import orders to be consistent.

Uses the default sorting options for sort-imports.
See https://eslint.org/docs/rules/sort-imports

LGTM! Is there an autofix rule for this too? Would be nice.

(Nevermind -- I see you've added it. Thanks!)


Code Quality: Formatted csproj files with tabs

Summary
As the title.
-<PropertyGroup Condition="'$(Configuration)' == 'Debug'">
-    <DefineConstants>TRACE;DEBUG;NETFX_CORE</DefineConstants>
-</PropertyGroup>
-<PropertyGroup Condition="'$(Configuration)' != 'Debug'">
-    <DefineConstants>TRACE;RELEASE;NETFX_CORE</DefineConstants>
-    <Optimize>true</Optimize>
-</PropertyGroup>

+<DefineConstants Condition="'$(Configuration)' == 'Debug'">TRACE;DEBUG;NETFX_CORE</DefineConstants>
+<DefineConstants Condition="'$(Configuration)' != 'Debug'">TRACE;RELEASE;NETFX_CORE</DefineConstants>
+<Optimize Condition="'$(Configuration)' != 'Debug'">true</Optimize>

PR Checklist

[ ] Development: N/A
[x] Approval: Have discussed with and got approval from the team[^1]
[x] Tests: Tested accessibility on the local end
[x] Deployment: Deployed the app on the local end
[ ] Localization: Modified en-US string resources[^2]
[ ] Co-authors: N/A

Screenshots
N/A
[^1]: If the request hasn't been agreed by the team, this work might be rejected. Make sure that you get approval from the team before opening any PR related the request.
[^2]: If you removed any en-US string resources, make sure that there are no references of those resources. When you add a new en-US string resources, do not make any changes on other languages' resources; Crowdin will do that, instead.

No other changes is here.

No other change is here.

Does making a change to the project from the Visual Studio UI set it back to spaces?

At the first place, the VS format with 2 spaces, not 4.

We had this issue back when we first switched from spaces to tabs, so we decided to stick with spaces in the csproj files to stay compatible with Visual Studio. Another place with this issue is with the localization files.

Good to hear. Ok, I'm gonna close.


We need a listPicker  control/

Category
[ x] Enhancement
[ ] Bug
[ ] Question
Version
Just like we have in the Property pane controls, but to put in the webpart body, Just a adropdown on lists on the site. Would be really great to be able to have it select only lists based on a specific template, or better yet, lists that have a specific content type.

This control will be available in v1.3.0. You can test out the beta right now.


Export-PSProfileConfiguration contains machine / user specific properties

I'd like to use Export-PSProfileConfiguration to export the PSProfile config and use it across multiple machines, however within the psd1 there's multiple properties that would overwrite some environment variables when using Import-PSProfileConfiguration.
Is it possible to make the exported data not include machine or user specific information?
Also, as a side question, is it possible to configure $PSProfile.Settings.ConfigurationPath permanently on a system? Otherwise, I'd imagine the only way to achieve a consistent change to this property is via changing it in profile.ps1 and that invoking a reload of PSProfile.

Checking this out, thanks @codaamok ♥

Alright, here's a more complete response @codaamok :

Working on preventing those $env:ComputerName and $env:UserName vars from being set other than what they should be set to on the current machine
That path is set by Configuration but I was planning on adding a spot to load in a separate config from a defined/custom path to layer against it. This would let you first import whatever is at $PSProfile.Settings.ConfigurationPath, then import the custom path on top of it (whatever is layered on last should take precedence in the event there are conflicts) --- would that satisfy the ask around the side question?

Yes, that sounds like a really good solution!

Heyo @codaamok - Just pushed out v0.6.3 that should get you covered here. Let me know if you have any questions!

Hello :-) this looks very good, thank you very much.
I have noticed that when I call Add-PSProfileConfigurationPath -Path "C:\somescript.ps1" -Save, then the contents of my current $PSProfile.Settings.ConfigurationPath jump around and the indentation continuously grows for the prompt strings every time I invoke the Add-PSProfileConfigurationPath function. Not a big deal I don't think, but thought you should know as it's probably unintended.

Sorry to spam! I would also expect the same command (Add-PSProfileConfigurationPath -Path "C:\somescript.ps1" -Save) to update the contents of the configuration file but it doesn't. Unless I'm missing something?

@codaamok ::

I have noticed that when I call Add-PSProfileConfigurationPath -Path "C:\somescript.ps1" -Save, then the contents of my current $PSProfile.Settings.ConfigurationPath jump around and the indentation continuously grows for the prompt strings every time I invoke the Add-PSProfileConfigurationPath function. Not a big deal I don't think, but thought you should know as it's probably unintended.

I've seen the same and I think it's how Configuration is saving the contents when saved often? I mostly notice it with ScriptBlock's. The end result will always be the same, but if you're looking at the config file, it will be a mess of extraneous indentation. Still trying to nail that one down.

Sorry to spam! I would also expect the same command (Add-PSProfileConfigurationPath -Path "C:\somescript.ps1" -Save) to update the contents of the configuration file but it doesn't. Unless I'm missing something?

It expects a PSD1 file, not a PS1, and will semi-silently skip it if it isn't a PSD1. Ref: https://github.com/SCRT-HQ/PSProfile/blob/main/PSProfile/Public/Configuration/Add-PSProfileConfigurationPath.ps1#L37

Thanks for all your help so far.
I notice the variables under $PSProfile.Variables.Environment aren't clobbered if the PSProfile configuration is imported by leveraging the $PSProfile.ConfigurationPaths. However they are clobbered if I import using Import-PSProfileConfiguration.
It seems the benefit of using Import-PSProfileConfiguration is that the $PSProfile object is immediately updated within the user's current session. Whereas if I just modify the ConfigurationPaths property, not all the properties (if any) in $PSProfile seem to be updated immediately, even after triggering Update-PSProfileConfig and Refresh-PSProfile.
I'm trying to devise a workflow that enables quick install of my PSProfile configuration and easy update too. An ideal workflow for install and update would work like:

Download PSProfile.Configuration.psd1 from my repo/Gist to somewhere like ~
Import-PSProfileConfiguration ~\PSProfile.Configuration.psd1
Use Add-PSProfilePrompt or similar to modify $PSProfile, or modify the configuration file directly, and use Export-PSProfileConfiguration. Update my repo/Gist with newly exported psd1.
Invoke some custom function that I'll likely store as an init script e.g. Update-Profile that will pull my PSProfile.Configuration.psd1 from my repo or Gist and update ~\PSProfile.Configuration.psd1. Said function will use Import-PSProfileConfiguration once downloaded because it immediately updates $PSProfile without needing to reload.

The problem for me is in step 3, I don't want manually strip out user and machine specific properties before uploading to some location. There's more than just what's under $PSProfile.Variables.Environment which is machine and user specific too, e.g.

PluginPaths
Settings.ConfigurationPath

I have an idea which might help but I'm not sure how to create in PSProfile. What if Export-PSProfileConfiguration offered some parameter to exclude named properties? That could get a little tricky for usage for the user, e.g. how would they specify child properties such as $PSProfile.Settings.ConfigurationPath in the parameter. But it's probably the most flexible idea because if you just offered a switch instead -ExcludeMachineUserSpecificInfo (terrible name I know, but stick with me) then this means you're deciding for the user what properties they can only exclude with little flexibility.
However, maybe I shouldn't be using Import-PSProfileConfiguration after download and maybe I should just ensure $PSProfile.ConfigurationPaths includes ~\PSProfile.Configuration.psd1 and just reload my session if a new version is detected. What's your opinion?


Add KafkaIO tests to Flink runner RunnableOnService tests

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @mxm or @aljoscha
CC: @amitsela
The I/O stuff hasn't all been factored out from the core SDK yet, but I did realize that there are a number of useful tests that aren't run in any configuration.
I just added the config for KakfaIO here, mostly as an exercise & an example to see if it would be easy. I didn't look at any of the code under test or the test itself - I have suspicions that configuration changes might break it and/or make it too slow. At some point we probably want all of these to run with real clusters.
On the other hand, the whole point of the model/SDK/Runner API/Fn API/etc is that we should be able to develop these things in isolation with only selected integration tests. But for now, I'd say the more tests the better.

LGTM. Have you also looked into making the Kafka Sink tests RunnableOnService?

@rangadi
I'm surprised we have Kafka @RunnableOnService tests. I suspect that they should actually be @NeedsRunner tests. Specifically, those tests seem to use mock objects rather than pointing at a Kafka service.

What would be the difference between @RunnableOnService and @NeedsRunner? The former extends the latter but it seems to be that tests written against one or the other are interchangeable.

@dhalperi @rangadi KafkaIO also has a dependency on the DirectRunner. Should this be removed and the tests executed in the direct runner similar to the tests in java sdk and core runner?

@mxm:

@RunnableOnService means that the pipeline could be executed on any production instance of a runner (e.g., Cloud Dataflow service [hence the name, which is no longer as good], a Spark cluster, a Flink cluster, etc.).

@NeedsRunner means effectively today that a test needs the DirectRunner in order to pass. @tgroh is working on the test capability matrix to improve things.
A practical way to distinguish between the former and the latter is usually whether the test requires the the execution happen on the same-machine or even same-JVM -- if so, it's a @NeedsRunner test. Conversely, with @RunnableOnService, as long as the TestPipelineOptions are configured in the right way, the test can be made to pass running anywhere.
Examples of truly @RunnableOnService tests:

tests that do not exercise any external systems. For example, CountingInput.upTo(N) -> Count.globally() -> PAssert(count == N).
tests that exercise external systems that are hosted publicly. For example, tests that read from or write to Google Cloud Bigtable (see #505) can be run anywhere on the web as long as the execution environment can reach Google's network.
tests that exercise external systems that are configured using TestPipelineOptions. The KafkaIO tests could be written in this way if the Kafka cluster was configured via TestPipelineOptions rather than either using mocks or a same-process spun-up Kafka instance.

Seconding what @dhalperi said; there's also a distinction worth drawing between tests that can be run on a production instance and those which are worth running
Generally, RunnableOnService tests should exercise some runner functionality as it relates to the Beam model; NeedsRunner tests should exercise some PTransform functionality. As such, not all tests that can be run in production need to be marked RunnableOnService

Thanks @dhalperi & @tgroh. KafkaIOTest borrowed code and structure from CountingSourceTest, and I kept @RunnableOnService annotation as well. I didn't think much about it that time. These are clearly @NeedsRunner tests as Tom described above. Shall I send a PR?

@rangadi please do.

Updated KafkaIO tests annotation in #568


Fix compile docs

Fix https://github.com/go-gitea/gitea/pull/24530#issuecomment-1557669210
This PR fix a bug that when a PR includes docs, the compile docs workflow will not run.

As mentioned in https://github.com/go-gitea/gitea/pull/24530#issuecomment-1558762473, there is too much wrong with the changes in https://github.com/go-gitea/gitea/pull/24530, I would fully revert it.


Update to handle duplicate region names

Added suffix like "_0", "_1", etc to output filenames for nonunique region names. See Issue: https://github.com/AndGem/osm_extract_polygon/issues/10
This is not compiled or tested at all! I am hoping AndGem can do this and create a new release. I just tried to sketch out a solution. I've never coded in Rust nor have a dev environment set up.

Hi @morandd ,
Thanks a lot for this PR. I'll take a look and might make some modifications.


Capping urllib3 at 1.2.3 to prevent conflict with requests 2.18.4

@andreikeis Ready for review

Codecov Report

Merging #159 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
- Coverage   53.22%   53.19%   -0.03%
==========================================
  Files         337      337
  Lines       25817    25817
  Branches     3357     3357
==========================================
- Hits        13740    13734       -6
- Misses      11390    11394       +4
- Partials      687      689       +2

Impacted Files
Coverage Δ

lib/python/treadmill/syscall/inotify.py
79.77% <0%> (-1.69%)
:arrow_down:

lib/python/treadmill/websocket/__init__.py
79.28% <0%> (-1.08%)
:arrow_down:

Continue to review full report at Codecov.

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


[stable/jenkins] Error: YAML parse error on jenkins/templates/config.yaml: error converting YAML to JSON: yaml: line 252: could not find expected ':'

I am getting below error when I add scriptApproval:
Error: YAML parse error on jenkins/templates/config.yaml: error converting YAML to JSON: yaml: line 252: could not find expected ':'
Describe the bug
I have values.yaml file with below values:
master:
    scriptApproval:
        - "field groovy.util.Node name"

Version of Helm and Kubernetes:
Client: &version.Version{SemVer:"v2.16.9", GitCommit:"8ad7037828e5a0fca1009dabe290130da6368e39", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.16.9", GitCommit:"8ad7037828e5a0fca1009dabe290130da6368e39", GitTreeState:"clean"}
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.1", GitCommit:"d647ddbd755faf07169599a625faf302ffc34458", GitTreeState:"clean", BuildDate:"2019-10-02T17:01:15Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.11", GitCommit:"ec831747a3a5896dbdf53f259eafea2a2595217c", GitTreeState:"clean", BuildDate:"2020-05-29T19:56:10Z", GoVersion:"go1.12.17", Compiler:"gc", Platform:"linux/amd64"}
Which chart:
Jenkins
What you expected to happen:
It should deploy Jenkins successfully with scriptApproval.
How to reproduce it (as minimally and precisely as possible):
Install helm chart with given values.yaml file configs.
Anything else we need to know:
It was working fine before. Suddenly, stopped working.

@rnkhouse This should be fixed by #23175
https://github.com/helm/charts/blob/master/stable/jenkins/ci/other-values.yaml#L20-L22 should verify that it works.

@torstenwalter It's working now. Thanks!


refactor(protocol-designer): migrate i18n instances to use useTranslation hook

closes RAUT-542
Overview
This PR fixes all instances of i18n in the protocol-designer directory by using the useTranslation hook instead. This is because the previous way we were doing it wasn't properly implementing react-i18next.
Additionally, several usages of the class components + connect function were refactored in order to use the useTranslation hook properly (this is part of a chore that needs to be done anyway to remove all the class components)
Test Plan
Smoke test PD, try to go through every flow as much as possible and call out anything that doesn't work correctly or whitescreens.
Here is the sandbox to smoke test on: https://sandbox.designer.opentrons.com/pd_update-i18n/
Changelog

update i18ninit function
update the i18next and react-i18next dependencies to be the latest
update all instances of i18n in the components
fix tests
refactor several class components + connect functions

Review requests
see test plan
Risk assessment
medium-ish since it touches a bunch of code but it'll be internal

Seems that minimum (5} uL  part would have an issue.


Strange colors on android simulator

On real android and ios devices and on the ios simulator on the Mac the svg's are shown correct. On android simulators the svg's get strange colors. For example if I use a 5.2" Marsmallow (6.0.0) XXHDPI Phone (Android 6.0 -API23) as simulator and I use the svglogo.svg provided in the samples I get a real weird result. See attached screen. Colors also change per debug session. Sometimes it's shades of blue, sometimes shades of pink.

I can reproduce this issue on VS android simulators.
I'll investigate but I think this might be related to SkiaSharp.

Are you using a Hyper-V emulator (such as Visual Studio emulators) ?
It seems related to ANGLE & OpenGL rendering : https://blogs.msdn.microsoft.com/visualstudioalm/2014/11/12/introducing-visual-studios-emulator-for-android/
It works fine on AVD emulators, but this workaround is not good enough since Hyper-V is now the recommended way to go on Windows.
I currently don't have a proper solution for this issue, sorry

Yes I am using a Hyper-V emulator indeed. For testing it is a bit annoying but as long as real devices work fine it is not a major issue for me.

It is related to Skiashap version 1.57.1. I am using an other plugin to show a svg. With skiasharp 1.57.1 I have exactly the same issue. When I downgrade to version 1.56.1 it all works fine!


Implement ghost comment mitigation

Adds a config option USER_DELETE_WITH_COMMENTS_MAX_DAYS to the [service] section.
Source: https://codeberg.org/Codeberg/gitea/pulls/9
See https://codeberg.org/Codeberg/Discussion/issues/24 for the underlying issue.

Codecov Report

Merging #14349 (9f957cf) into master (bfd0c47) will decrease coverage by 0.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master   #14349      +/-   ##
==========================================
- Coverage   41.86%   41.85%   -0.02%
==========================================
  Files         744      744
  Lines       79638    79650      +12
==========================================
- Hits        33342    33339       -3
- Misses      40808    40820      +12
- Partials     5488     5491       +3

Impacted Files
Coverage Δ

models/user.go
53.68% <28.57%> (-0.18%)
:arrow_down:

routers/user/setting/account.go
25.50% <40.00%> (+0.37%)
:arrow_up:

modules/git/repo_base_nogogit.go
63.63% <0.00%> (-9.10%)
:arrow_down:

modules/git/utils.go
77.77% <0.00%> (-2.78%)
:arrow_down:

modules/log/event.go
59.90% <0.00%> (-1.89%)
:arrow_down:

modules/queue/unique_queue_disk_channel.go
53.84% <0.00%> (-1.54%)
:arrow_down:

modules/process/manager.go
75.00% <0.00%> (+2.50%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #14349 (9f957cf) into master (bfd0c47) will decrease coverage by 0.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master   #14349      +/-   ##
==========================================
- Coverage   41.86%   41.85%   -0.02%
==========================================
  Files         744      744
  Lines       79638    79650      +12
==========================================
- Hits        33342    33339       -3
- Misses      40808    40820      +12
- Partials     5488     5491       +3

Impacted Files
Coverage Δ

models/user.go
53.68% <28.57%> (-0.18%)
:arrow_down:

routers/user/setting/account.go
25.50% <40.00%> (+0.37%)
:arrow_up:

modules/git/repo_base_nogogit.go
63.63% <0.00%> (-9.10%)
:arrow_down:

modules/git/utils.go
77.77% <0.00%> (-2.78%)
:arrow_down:

modules/log/event.go
59.90% <0.00%> (-1.89%)
:arrow_down:

modules/queue/unique_queue_disk_channel.go
53.84% <0.00%> (-1.54%)
:arrow_down:

modules/process/manager.go
75.00% <0.00%> (+2.50%)
:arrow_up:

Continue to review full report at Codecov.

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


Bugfix/epg bug fixes

There are two bug fixes in this pull request:

Use click sensitivity to interpret clicks
Offset timebar to remain in sync with time indicator

I had to fix the first one in order to verify the second.
Before, expanding the channels resulted in the appearance of the time indicator shifting forward in time:

After, expanding the channels shifts the timebar in sync with the indicator:

@dominicpereira92 Is anyone looking at this?


Share to Open Food Facts should let you zoom on the picture when no barcode is detected

Share to Open Food Facts should let you zoom on the picture when no barcode is detected

we can't read the barcode. Thus we should be able to zoom it.


the 'dmos_realigned.mat' of LIVE database

I want to train the model on LIVE database, but there is an error about 'dmos_realigned.mat'.  The code is here
''dmos = scipy.io.loadmat(os.path.join(root, 'dmos_realigned.mat'))
labels = dmos['dmos_new'].astype(np.float32)''
Is it correct for me to replace  'dmos_realigned.mat' with ‘dmos.mat' and 'dmos_new' with 'dmos'(using the mat file in live database) , or have you processed the data in any other way?
Thank you

So what is the error reported?

So what is the error reported?

when I used the code you published, there is an error that No such file or directory about 'dmos_realigned.mat'. Then I replace  'dmos_realigned.mat' with ‘dmos.mat' , the 'KeyError: 'dmos_new'' is occurred.

Seems you're using a Release 1 version of LIVE dataset, see http://live.ece.utexas.edu/research/Quality/subjective.htm and get a Release 2 version, the problem should be solved.

Seems you're using a Release 1 version of LIVE dataset, see http://live.ece.utexas.edu/research/Quality/subjective.htm and get a Release 2 version, the problem should be solved.

I think I know the reason. I used the release2 but I don't download the realigned subjective quality data. thank you!

Cool


Set default output filename

Change Description
Fix to default output filename. (when not specified output filename by -o)
API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

@conr2d Please see my comment in #578.

This also fixes #582.
Object file is added to ldopts twice by compiler_options.hpp and eosio-cpp.cpp, and it causes duplicate symbol error.


Remove the extra closing bracket

Sorry, I didn't remove the extra brackets in my last edit.

Thanks@


⚠ TorrentLeech has degraded performance

In 73887ab, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 1633 ms

Resolved: TorrentLeech performance has improved in 9c8526e after 22 minutes.


Automatically increase default stretches over time

Recently the default stretches value was increased to 12. As I mentioned in the corresponding MR:
A reasonable approach to set an optimized stretches value would be manual. Every time you change a CPU you could calibrate the cost factor. E.g. you may always intend to generate an encrypted password under 250 ms. There is a nifty BCrypt::Engine.calibrate for your convenience.
Another approach for updating the BCrypt cost factor (i.e. stretches) would be automatically increasing it over time. This idea is implemented in the aforementioned PR with a comprehensive discussion.
For devise with rails it would be something like:
base_cost = 12
start_time = Time.new(2019, 1, 1)
doubling_period = 60.months
cost_increase = ((Time.now - start_time) / doubling_period).floor
config.stretches = Rails.env.test? ? 1 : base_cost + cost_increase

However it's difficult to guess a good doubling_period for the future. The rails PR uses ~5.3 years.
(copied from https://github.com/plataformatec/devise/pull/5074#issuecomment-491797257 and https://github.com/plataformatec/devise/pull/5074#issuecomment-492696360 as @mracos suggested)

In my view increasing the default stretches value manually would be a more reasonable approach. However curious what other developers think.


I hope to add a product inquiry price function, a contact form and an information form and a hidden price function!

I hope to add a product inquiry price function, a contact form and an information form and a hidden price function!
This allows us to better understand the needs of our customers and to serve them more quickly and clearly for some customized products.

You could add custom prices using this mutation https://docs.saleor.io/docs/3.x/developer/checkout#setting-custom-prices-for-checkout-items


[SPARK-28794][SQL][DOC] Documentation for Create table Command

What changes were proposed in this pull request?
Document CREATE TABLE statement in SQL Reference Guide.
Why are the changes needed?
Adding documentation for SQL reference.
Does this PR introduce any user-facing change?
yes
Before:
There was no documentation for this.
How was this patch tested?
Used jekyll build and serve to verify.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

This still has a lot of basic syntax, grammar and formatting problems. Please proofread per above.

Ping @PavithraRamachandran

kindly ping again, @PavithraRamachandran

I shall update the PR , with the necessary corrections as per the review comments.

ok to test

Test build #116731 has started for PR 26759 at commit 62502a2.

Merged build finished. Test PASSed.

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


feat(core): support catalog: protocol

Description

This PR support pnpm catalog: protocol.
Note: this pr only let lerna recognizecatalog: protocol, and lerna can not manage its version (e.g.: bumping it when publish). It could be hard to update a raw yaml files, and pnpm itself do not support bumping catalogs with pnpm up either, so I think recognizing should be enough.
Another note: This PR can be considered incomplete for the following 2 reasons:

Only pnpm-workspace.yaml under current cwd is read.
It only seems meaningful running lerna commands at root, I am not sure if we should search the ancestors' folder in case the command is running in a workspace package. If so the PR still needs to improved.

I am not sure how to update tests gracefully, in the implementation, we shall read a external file pnpm-workspace.yaml, and that's actually a bit out of scope of PackageGraph unless we provide a new constructor option indicating the file location. So it's a little hard to provide the file or the file information in current test sets. Any recommendation would be appreciated.

Motivation and Context

https://github.com/lerna-lite/lerna-lite/discussions/901
https://github.com/lerna-lite/lerna-lite/issues/902
How Has This Been Tested?

Tested locally in vuepress/ecosytem by patching the core pacakge.
Types of changes

[ ] Chore (change that has absolutely no effect on users)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

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

Thanks for the PR, the code looks ok but I can't really merge this without any unit tests in place.
There's maybe 2 ways to add tests, it could be via

package-graph spec

https://github.com/lerna-lite/lerna-lite/blob/main/packages/core/src/package-graph/__tests__/package-graph.spec.ts

or a separate spec file like this

https://github.com/lerna-lite/lerna-lite/blob/main/packages/publish/src/__tests__/publish-workspace-protocol-specifiers.spec.ts


Fix for #290

The only time the queue file has any explicit changes of ' to %27 is if
an entry in the queue file doesn't have a set download path/filename to begin
with (calling QueueLoader::get_filename()). But it doesn't do the same to the
download-path if it's configured, so if you may end up with
"foo's/bar%27s.mp3" but it will try and play "foo%27s/bar%37s.mp3"
The only time this MIGHT make a difference is if the download path is
set to a remote address (http, ftp), but so far with what testing I
could do, this change hasn't affected playback of remote files.
The switch from wrapping the command string from ' to " is because the
shell expands \' really weirdly, still counting it as a closing mark, so
you'll have to expand it 'foo'\''s' for it to work properly.

Coverage remained the same at 41.128% when pulling 62e770a15e34b62f224d4bc6e34c42fc9c88fd0f on CtrlSequence:bugfix/290 into a85f1dae4ff5b921f5e36ec55f04c5f6baf68e11 on newsboat:master.

Coverage remained the same at 41.128% when pulling 62e770a15e34b62f224d4bc6e34c42fc9c88fd0f on CtrlSequence:bugfix/290 into a85f1dae4ff5b921f5e36ec55f04c5f6baf68e11 on newsboat:master.

We can't use double quotes to wrap the URL, because they don't prevent the shell from expanding variables and stuff. I.e. mpv "$(echo hi)" won't play a file named $(echo hi)—it will run echo instead. This is precisely the vulnerability we fixed in Podboat two years ago: https://github.com/akrennmair/newsbeuter/issues/598
Looking at #290 now, I think the correct solution is to use exec* rather than system inside utils::run_interactively. The first step towards that would be: check where run_interactively is used, and cross-check with the documentation to see if changing the argument from std::string to std::vector<std::string> will break any promises we made to users.
I'm sorry to close your first PR, @CtrlSequence, but please don't feel discouraged—your contributions are still welcome! Just open another one with a different take on this when you're ready.

Ah I totally forgot about shell variable expansions!
Looking at what calls utils::run_interactively, I think key ones are View::open_in_browser, View::open_in_pager and FormAction::bookmark because you can configure more complex shell commands with them. With exec* functions, it seems far easier to run a media player since you execute the player directly and can pass args as separate strings, but using any shell built-ins like pipes or redirectors requires executing a shell and passing the whole command as an string/argument (which pretty much is the same as system).
Using std::vector<std::string> with utils::run_interactively, you could explicitly assign a shell to the element to pass as the executable and add more elements to pass as arguments to copy the current behavior, or otherwise pass what's configured as the executable. The only thing is that if someone includes extra arguments for their command in their configuration, e.g. player "mplayer -speed 1.50", you'd have to tokenize it.

I don't think tokenizing would be enough. I guess users will expect a full-blown shell syntax, so we need an actual parser, even if just to print out "XYZ not supported" when a user tries to pipe, redirect, substitute a command, or anything like that. Implementing such a parser doesn't look like a trivial task to me, and I'm pretty sure it's out of scope for Newsboat.
Getting back to #290: you said \' is expanded really weirdly. What did you mean? I tried touch 'I'\''m a test' just now, it seem to work as expected, creating a file named I'm a test. Perhaps we can replace all single quotes with '\'' rather than %27, and be done with this?


[Proposal] Unavailability Condition

This proposal defines a new #unavailable condition for negative availability checks.
Forums Thread: https://forums.swift.org/t/support-negative-availability-literals/39946
Implementation: https://github.com/apple/swift/pull/33932

Added a new Usage of deprecated APIs section due to the same problem being reporting when dealing with obsolete APIs.

It's been a while and all points have been addressed, the only thing remaining is the core team's opinion. Is it possible to get an answer or do we need more time?

@rockbruno The core team would like to run the review.  Does next week work for you?

@tkremenek Yes, it's great for me. Thanks for picking this up!

I will merge this now to stage for a review.


Bug: Empty lines when changing type of node

I am using Lexical as a rich text editor and I wasn't able to fix this, after that I found out it's in the lexical playground as well. Just copied some text from wiki, the video speaks for itself.
https://user-images.githubusercontent.com/77566262/198886272-61e325b2-a794-4aa5-8858-6dcd74992d72.mov
just in case Link for some text to try: https://en.wikipedia.org/wiki/Lion

So what I did, I created my own function instead of $wrapNodes as is here: https://github.com/facebook/lexical/blob/main/packages/lexical-playground/src/plugins/ToolbarPlugin/index.tsx#L183
I think, that this works just fine.
I am waiting for the statement of Lexical dev tem, but feel free to make PR.
import {
  $createParagraphNode,
  $getSelection,
  $isElementNode,
  $isRangeSelection,
  $isRootOrShadowRoot,
  LexicalEditor,
} from 'lexical';
import { $findMatchingParent } from '@lexical/utils';
import { $createHeadingNode, HeadingTagType } from '@lexical/rich-text';
import { $wrapNodes } from '@lexical/selection';

const changeSelectedNodeType = (
  editor: LexicalEditor,
  node?: HeadingTagType,
) => {
  editor.update(() => {
    const selection = $getSelection();
    if (!$isRangeSelection(selection)) return;
    const parent = $findMatchingParent(selection.anchor.getNode(), (e) => {
      const parent = e.getParent();
      return parent !== null && $isRootOrShadowRoot(parent);
    });
    const createdNode = node
      ? $createHeadingNode(node)
      : $createParagraphNode();
    if (parent) {
      if ($isElementNode(parent)) {
        parent.getChildren().forEach((node) => {
          createdNode.append(node);
        });

        parent.replace(createdNode);
      }
    } else {
      $wrapNodes(selection, () => createdNode);
    }
  });
};

export default changeSelectedNodeType;


Automatically append  to root app element

This should solve a lot of positioning bugs, although it will be a breaking change.
http://stackoverflow.com/questions/39857222/angular2-dynamic-component-injection-in-root

https://github.com/angular/angular/issues/9293#issuecomment-261329089


Should we have a separate error code for argument type is not assignable to the parameter type?

main() {
  foo('zzz');
}
foo(int p) {}

bin/test.dart:2:7: Error: A value of type 'dart.core::String' can't be assigned to a variable of type 'dart.core::int'.
Try changing the type of the left hand side, or casting the right hand side to 'dart.core::int'.
  foo('zzz');
      ^

In Analyzer we currently have separate StaticWarningCode.ARGUMENT_TYPE_NOT_ASSIGNABLE.

Note that this is not the only error for which analyzer has more specific contextual information and which will therefore need to be split into multiple error messages.
@stefantsov Do we want separate issues for each message, or should we have a single issue covering the general case with an explicit list?

I would prefer the single issue with the list, like those that we had for converting shadow expression and shadow statements to judgments.

Sounds good. I don't have a complete list at this point, but I'll start working on it.

This particular case is addressed by https://dart-review.googlesource.com/c/sdk/+/63960.


Nuget Restore: not adding ApiKey feeds to temporary nuget.config

Environment

We are using TFS on-premises.
We built the source code and deployed Nuget task on our server.
Task version is now 2.0.45 (see task log below)

Issue Description
Using v2 of NuGet Restore task with these options:

no path to nuget.config entered
selected 1 ApiKey outside feed
disable local cache to test the outside feed

Expected behavior
Because no path to nuget.config is entered, nuget is creating a temporary .config file. My expected behavior is that Nuget would insert the outside feed Url to the <packageSources> list in the temporary nuget.config file.
Task logs
2018-10-05T15:44:52.3247612Z ==============================================================================
2018-10-05T15:44:52.3247955Z Task         : NuGet
2018-10-05T15:44:52.3248383Z Description  : Restore, pack, or push NuGet packages, or run a NuGet command. Supports NuGet.org and authenticated feeds like Package Management and MyGet. Uses NuGet.exe and works with .NET Framework apps. For .NET Core and .NET Standard apps, use the .NET Core task.
2018-10-05T15:44:52.3248784Z Version      : 2.0.45
2018-10-05T15:44:52.3249021Z Author       : Microsoft Corporation
2018-10-05T15:44:52.3249284Z Help         : [More Information](https://go.microsoft.com/fwlink/?LinkID=613747)
2018-10-05T15:44:52.3249596Z ==============================================================================
...
2018-10-05T15:44:54.1789187Z Saving NuGet.config to a temporary config file.
2018-10-05T15:44:54.1846147Z ##[warning]No package sources were found in the NuGet.config file at D:\b\1\47\Nuget\tempNuGet_143620.config
2018-10-05T15:44:54.1849775Z [command]D:\b\1\_tool\NuGet\4.1.0\x64\nuget.exe restore D:\b\1\47\s\myproject.sln -NoCache -Verbosity Detailed -NonInteractive

My expected behavior would have been:
Saving NuGet.config to a temporary config file.
##[warning]No package sources were found in the NuGet.config file at D:\b\1\47\Nuget\tempNuGet_143620.config
[command] nuget.exe sources Add -Name "PublicNuget" -Source http://myserver/feed
[command] nuget.exe restore ...

Because of that, the Nuget Restore cannot restore packages in our internal feeds.
Anyone else having this error? Or am I not understand how to use the credential outside feeds..

Can you please run the task again with system.debug set to true and include the logs (you can censor anything you wish from the log).


Rework Peatio accounting using double entry accounting system

Operation table:

account has_many operations
operations belongs_to account
account_id as a foreign key

TODO:

[x] Fix specs
[ ] Test & fix legacy ui
[ ] Test & fix admin pannel
[ ] Prepare migrations for creating operations after removing of balance and locked
[ ] Move #payment_address from Account to AccountingService::MemberEntry
[x] AccountingService specs
[ ] AccountingService::Chart specs

Platform Accounting story
Platform owner is considered to be user with id 1

[ ] Design & Implement Accounting Chart for Platform owner
[ ] Implement AccountingService::PlatformEntry
[ ] Check existing code to make double operations on withdraw & deposit (eg. deposit causes credit for liabilities & assets accounts)

Hey this looks good.
Can we add a token id to every transaction? This will be helpful for referencing the transaction.
Instead of have credit debit columns we can have amount and type where  type can be credit or debit

@martianatwork we have reference for each operations I think it the same as token
Credit ant debit are commonly used for such kind of tables

[x] Specs for Order

[x] Validate operations system using toolbox and validating script

Single market two traders no fees
Multiple market 5 traders no fees
Single market with fees two traders
Multiple market with fees 5 traders

[x] Specs for Trade

[ ] Complex specs

[ ] Code refactor

Operation table:

account has_many operations
operations belongs_to account
account_id as a foreign key

TODO:

[ ] Fix specs
[ ] Test & fix legacy ui
[ ] Test & fix admin pannel
[ ] Prepare migrations for creating operations after removing of balance and locked
[ ] Move #payment_address from Account to AccountingService::MemberEntry
[ ] AccountingService specs
[ ] AccountingService::Chart specs

Platform Accounting story
Platform owner is considered to be user with id 1

[ ] Design & Implement Accounting Chart for Platform owner
[ ] Implement AccountingService::PlatformEntry
[ ] Check existing code to make double operations on withdraw & deposit (eg. deposit causes credit for liabilities & assets accounts)


Create flatMap.md

FEATURE
Description
Adds flatMap js function
PR Type

[x] Snippets, Tests & Tags (new snippets, updated snippets, re-tagging of snippets, added/updated tests)
[ ] Tools, Scripts & Automation (anything related to files in the scripts folder, Gatsby, website, Travis CI or Netlify)
[ ] General, Typos, Misc. & Meta (everything related to content, typos, general stuff and meta files in the repository - e.g. the issue template)
[ ] Other (please specifiy in the description above)

Guidelines

[x] I have read the guidelines in the CONTRIBUTING document.

This method is also added to ECMAScript 2019 specification. Do we want to still include it?
As a learning material, I think this should belong here as well since some interviews ask for this and it's useful to know how it's implemented.

@fejes713 The JS ecosystem is still having trouble getting to use ES6, ES2019 will be fully supported in like half a decade, so we should be good to go for at least a year with showcasing an implementation for it.

Hey, @melwinalm do you need any help regarding the @Chalarangelo feedback? I can help you figure this out and get this PR to a merge-ready state.
Let me know if you need any help. :octocat:

Closing due to lack of interest from the author.


Closed way with junction=interchange is not interpretated as area

A closed way with junction=interchange like Autobahnkreuz Stuttgart should be interpreted as area. Currently it is interpreted as closed polyline.

Hi and welcome, @dktue.
The junction tag has historically been difficult for iD to interpret since values like this are used for areas while values like junction=roundabout are used on linear highway features. As such, iD can't tell if an unknown junction tag implies area or not. See 781158874aa19cc6f5d1202e6f2ae4ca53ac722f and 6dfdec06ad0e3b64b0f47799bdad4bb05d3712ed for more info.
We could give iD the necessary info by adding presets for more junction values, but most of these tags aren't used incredibly often and the wiki documentation doesn't define what they all mean. In the meantime, you can add area=yes to any way to make iD treat it as an area.

Thanks for hinting the workaround @quincylvania : What track would you suggest to get the (in my eyes correct) solution done?

What track would you suggest to get the (in my eyes correct) solution done?

@dktue If you or others could document on the wiki what junction=interchange is and how to map it then we could probably support it in iD.


AddCommand() in perl [$5]

Let's split #198 to perl and python...
This issue will be about perl.

---
There is a **[$5 open bounty](https://www.bountysource.com/issues/7176292-addcommand-in-perl-5?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** on this issue. Add to the bounty at [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

Can you tell me where you the error is or where you want the file, Im lost.


Error: ... virtual machine instance does not have permission scopes specified.

I have a relatively straight forward cloud datastore app, deployed on appengine. I'm using Objectify 6. I am getting this error when running unit tests against the local datastore, on travis-ci.
java.io.IOException: Error code 404 trying to get security access token from Compute Engine metadata for the default service account. This may be because the virtual machine instance does not have permission scopes specified.
A few notes:

this works fine on my local osx and windows machine
it also works fine on a digital ocean ubuntu 16 instance
storage and query ops works fine if I don't use Objectify
it's not working on a ubuntu 14 instance hosted by travis (on GCE, coincidentally)

I've scraped the web looking for info on the problem, and while the problem seems scarce, the answers are even more scarce. I'm not really sure if this is a google problem, a travis problem, or an objectify problem.
Few questions:

Why, when running against the local datastore, is it querying a remote google metadata server? Compute Engine? That's not even a service we are currently using. Edit: Actually, turns out the ubuntu 14 instance is itself running on GCE, so maybe that is the link.
I've given the App Engine default service account all the seemingly relevant roles, to no avail. I do have 8 service accounts though - which one is correct?
I've also made sure that on the travis instance, I am logged in through gcloud as said default service account, before running mvn test
Is there a way I can debug using some curl calls to the metadata server? The error is just not very informative.

And finally, any ideas on what I can do to fix this? Thanks.

I eventually figured this out. Turns out, mvn was not getting the proper authentication, even though logged in with a service account. I was able to fix it by adding:

export GOOGLE_APPLICATION_CREDENTIALS=$PWD/key.json

where key.json is created in IAM -> Service Accounts ->App Engine default service account (ie. the same account being used to login with gcloud).


Fixes and improvements for the chain switcher component

[x]  Should show an error
[x]  The component needs clear boundaries.
[x]  Remove URLs from the list items.
[x]  Remove connectable via
[x]  Make the network name larger
[x]  Add clear boundaries between list items (like Anyswap)
[x]  Add radio buttons for "live", "test", "dev"

As a user, I can click to view all "live" chains and see only "live" production chains.
As a user, I can click "test" and see only the available testnet chains.
Should be improved for an 'all' or deselect.

[ ]  Live should be default

Default is test,

[x]  If you're on a "live"/"test"/"dev" chain already then it should show you those options
[ ]  Search bar for filtering, which also respects the radio buttons
[x]  Clicking outside of modal should close modal

Pictures of changes to the chain switcher:


how to let my own stage moved by using events

Describe your problem here.
i i want to know how to let my stage moved like the songs called "soul" in friday night fever by using events lua.
engine version:0.5.1
thanks!
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

eh,ok,i will take my free time to show you the video...
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2022年4月14日(星期四) 晚上11:41
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [ShadowMario/FNF-PsychEngine] how to let my own stage moved by using events (Issue #7963)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Prevent immediate validation of fields with values?

Hello,
I just updated and the changes from #350 are causing some problems for me.  Here is what is happening:

I return the old input from the server for every input field, so the field is initialized with the old value.
I set the class 'has-error' on the form group if there is a server error and render a message.
When the validator initializes, it validates the field because of the changes that resolved #350
Since the field can only be validated server side, the validation passes.
After validation passes the 'has-error' class is removed, even though I have a server error.
Since this happens on initialization, the client doesn't have a chance to see the error state.

Here is a fiddle that might help explain what I'm talking about.
What do you think about making this behavior something that can be disabled?

I have a similar issue.
I perform server-side validation, which is more thorough than the client-side validation, so in some instances, the client-side validator will think a field is valid, yet after submitting to the server, the server has determined it to be invalid, and has re-rendered the form with a form-group with 'has-error', and pre-populated error message.
Ideally, I'd like it to not run validation when the page first loads, until the field value is changed. And when it does run, to remove the server-side generated error text.

+1

I think I'll revert the changes from #350 and leave it up to people who want that behavior to trigger validation themselves. See also #393.

+1


504 errors when running behind apache proxy

Home Assistant release (hass --version): 0.41.0
Python release (python3 --version): 3.4.3
Component/platform: Web Interface / aiohttp
Description of problem:
When  running homeassistant behind an Apache proxy server, you will seen 504 errors (Gateway timeout)
in  the apache logs. Issue has been reported both for Apache as NGinx as proxy server; has been closed every time as not being a HA issue.
For example: https://github.com/home-assistant/home-assistant/issues/6676
Expected:
No 504 errors should normally occur
My info on this issue:
I have seen this issue as well; currently running HA 0.41.0
I have seen them on camera-url's on gpslogger requests, even on static content, etc.
I traced the network traffic between Apache and HA to find out who is sending the 504 and why.
I ran tcpdump on my Ubuntu server to catch the network traffic from Apache to HA (running on the same server, over the lo-adapter):

sudo tcpdump -i lo -w /tmp/tcpdump.pcap

What I see in Wireshark is the following:

Apache sends all requests to HA, event the ones we see the 504 error for in Apache log files
Apache is using a connection pool for the proxy; requests will go from apache->HA using tcp ports (for example 42635->8123)
Since Apache uses connection pooling, this port will get reused later on
in my case I have seen a port used (secs are offset to trace start):

Request 1:
0.06 sec: GET /api/camera_proxy/camera.schuur?token=966...40b&time=1490557176003
0.32 sec: 200 OK
Request 2:
30.0 sec: GET /api/camera_proxy/camera.voordeur?token=47c...5d7&time=1490557206005
30.26 sec: 200 OK
105.073 sec: 504 Gateway timeout (NOTE: no request for this response! see below for the content of this 504 sent by HA!)
Request 3:
110.05 sec: GET /api/camera_proxy/camera.schuur?token=cb2...cf0&time=1490557286013
110.05 sec: tcp port is reset by HA (8123 → 42635 [RST] Seq=64315 Win=0 Len=0)

SO: it looks like the 504-errors are related to Apache connection pooling

this is something you would never see without a proxy in front
Big question is

why is HA/aiohttp sending 504's
and why is it being sent without a request?

If I turn TCP port reuse off in apache proxy settings; i no longer get 504 errors (at least for now):

ProxyPass /api/websocket ws://localhost:8123/api/websocket disablereuse=On
ProxyPassReverse /api/websocket ws://localhost:8123/api/websocket
ProxyPass / http://localhost:8123/ disablereuse=On
ProxyPassReverse / http://localhost:8123/

This is a workaround to prevent the errors from occuring
It seems to work, for now...  But what will be the fix for this?
504 returned by HA/aiohttp without a request....

HTTP/1.1 504 Gateway Timeout\r\n
Content-Type: text/html; charset=utf-8\r\n
Content-Length: 182\r\n
Connection: close\r\n
Date: Sun, 26 Mar 2017 19:41:21 GMT\r\n
Server: Python/3.4 aiohttp/1.3.5\r\n

That's great news! I'll update my config and see if this works for me too!
Sebastian

let me know it it works for you as well. For me, I still am not seeing any 504 errors in Apache logs since yesterday....

After a short test, it's looking good so far... I'll keep you updated in case this changes.
Sebastian

Glad I saw this - I was getting the issue with my mjpeg camera, hadn't noticed it was only when I was accessing remotely via an Apache proxy. This workaround fixed it for me - the logs look clean today.

It's still looking good for me too. Thanks a lot @mjj4791 for finding that workaround!
I already created a PR to update the documentation.
Sebastian

Thanks @mjj4791 it works for me too!

Does it works with basic-auth set on Apache?
My config is working on nginx - any ideas how to copy that config there?

I can only answer your first question: yes, it works with basic auth.

This will be fixed by the aiohttp 2 upgrade

@balloob, so it was a HA (/component) issue after all?

I discovered that we could end up sending a 504 when losing connection after a connection was already used. Only happens when streaming camera's.
We will now only raise a 504 when the initial connection times out:
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/helpers/aiohttp_client.py#L82
So yes, it was an HA issue but I am still a bit confused how it can be triggered 🤔  Closing this as aiohttp 2 has been merged into develop: #6835

Just to complete the issue, for anyone searching for this error.
I upgraded to HA to 0.42.3 and removed the apache workaround I suggested above
(i.e. i removed the "disablereuse=On" from my apache config for HA)
This new config has been running for some hours now; have not yet seen any 504 error in the apache logs.

I can confirm this. I've been running 0.42.x since last Sunday and also removed the disablereuse-workaround when I update - haven't had a single 504 error since.
Sebastian


HoverCallbacks mixin

Problem to solve
When using the new flame router, hoverables seem to be still detected when you are on a different page. This can be a problem when for example you want to change the mouse cursor when hovering a component. The cursor will also change on the different page.
Proposal
@spydon suggested in the discord channel

The old event system is not aware of the router, so a HoverCallbacks mixin would have to be created

I'll create the needed mixins.

This needs to be created for V2, since Hoverable will be deprecated.

@spydon refer to one of my recent commits, 0f0db95f1f4b2461caf7ac181a80e0e12da77cd0.  Is CursorHoverCallbacks along the lines of what you're looking for to replace Hoverable?

Watching 👀
Thanks! 😄

Done by #2706


RTE Segfault: To many variables questioned in one FBSheet in CSHMI

Hi together.
If i open a FBSheet in CSHMI in my browser (firefox) and there are around 20 to 30 Functioncharts with 10 inputs and 10 outputs each, i will get a segfault for the server, that hosts the functionblocks. So i guess there is a limit, that is not catched correctly.
Bad news is, that if i leave the browser window open, the site won't stop asking for the variables. So i can't start the server, as it will have the segfault after startup again.

Thread #1 [ov_runtimeserve] 4440 [core: 3] (Suspended : Signal : SIGSEGV:Segmentation fault)
ov_element_searchchild() at ov_element.c:62 0x7ffff7ba443a
ov_path_resolve() at ov_path.c:371 0x7ffff7bb0152
ov_ksserver_getvar() at ovks_backend_getvar.c:94 0x7ffff73b9b3d
kshttp_exec_getvar() at service_getvar.c:115 0x7ffff6335e18
kshttp_httpclienthandler_generateHttpBody() at httpclienthandler.c:217 0x7ffff632791a
kshttp_httpclienthandler_HandleRequest() at httpclienthandler.c:79 0x7ffff6327160
TCPbind_TCPChannel_typemethod() at TCPChannel.c:543 0x7ffff7198faf
ksbase_RootComTask_execute() at RootComTask.c:250 0x7ffff73b3ea3
ov_scheduler_schedulenextevent() at ov_scheduler.c:259 0x7ffff7bc4884
ov_ksserver_stripped_run() at ov_runtimeserver.c:205 0x401f59
<...more frames...>

How to reproduce the error:

Load CSHMI templates
Load Bug_ToManyVariablesForCSHMI-10.fbd to Bug_ToManyVariablesForCSHMI-30.fbd
Each FBD has 10 empty functioncharts with 10 inputs and 10 outputs each.
open csHMI FBSheet on .../TechUnits/BugTest

Bug_ToManyVariablesForCSHMI-10.txt
Bug_ToManyVariablesForCSHMI-20.txt
Bug_ToManyVariablesForCSHMI-30.txt

Thanks to @LarsNt, we found the error. The KS getvar request from csHMI is bigger than the OV_MEMSTACK_SIZE of a memstack page.
In consequence the memory is overwritten in:
https://github.com/acplt/rte/blob/master/syslibs/comm/kshttp/source/service_getvar.c#L109
We will change the memstack implementation to allocate the needed memory in:
https://github.com/acplt/rte/blob/master/core/ov/source/libov/ov_memstack.c#L124

On closer inspection the memstack already supports arbitrary large pages ov_memstack.c L125.
The problem lies in the usage of the allocated memory.
In service_getvar.c L94 the right amount of memory is allocated in *addrp. But this pointer is overwritten in L107. Furthermore, the loop in L106 increases the pointer addrp which is only allocated as a single pointer in L92.
As the memstack page is almost empty at this point most requests will not cause any problems. 64 bit systems are more likely to suffer from this issue since the pointer size is doubled compared to 32 bit systems.
Further instances of this bug may exist e.g. service_setvar.c.

Furthermore, the loop in L106 increases the pointer addrp which is only allocated as a single pointer in L92.

OV_STRING is a pointer. OV_STRING* is therefor an array of pointers.

@LarsNt commit seems to fix it.

That seems to be the problem:

@FlorianPalm is that powerpoint or a fancy tool?

:smiley:  just ppt


Slow code on loops, arrays and indexing

The Crest ocean drops framerate in Unity significantly. May be it's particially by shader complexity, but still some calculations are made on CPU. I checked the cs-files and found several problems. This is obviuous for anyone, who knows how high-level language code is tranferred to machine commands. Computers become more and more powerful, but work more and more slow, because programmers don't know how their code works internally.

Explicite evaluation of array length on each iteration.
Multiple evaluation of same values (field access and indexing).
Index range checking optimization.

Explicite evaluation of array length on each iteration.

In the following fragment of code
for (int i = 0; i < patchTypes.Length; i++)
the array length is evaluated on each iteration, because code is not aware, is array changed between iteration or not, it's possible in mulithreading. This means get array reference in memory and access the length field, instead of holding the length in processor register. Such things should be replaced by
int  arrLen =  patchTypes.Length;

for (int i = 0; i < arrLen; i++)

Multiple evaluation of same values (field access and indexing).

Let's the the method QueryBase.UpdateQueryPoints. In many places two variables are used:
_segmentRegistrarRingBuffer.Current
_segmentRegistrarRingBuffer.Current._segments

which causes multiple evaluation of end token. Should be replaced by something like this
var srrbCurrent = _segmentRegistrarRingBuffer.Current;
var srrbCurrentSegments = _segmentRegistrarRingBuffer.Current._segments;

Using var is bad, I just don't know real types of variables. var is faster to write, but harder to understand, what is the real type behind. This confusing causes time wasting, trying to understand, what is it.
Next, let's check one of the loops.
for (int normi = 0; normi < countNorms; normi++)
{
    var arrIdx = segment.x + countPts + 3 * normi;

    _queryPosXZ_minGridSize[arrIdx + 0].x = queryNormals[normi].x;
    _queryPosXZ_minGridSize[arrIdx + 0].y = queryNormals[normi].z;
    _queryPosXZ_minGridSize[arrIdx + 0].z = minGridSize;

    _queryPosXZ_minGridSize[arrIdx + 1].x = queryNormals[normi].x + s_finiteDiffDx;
    _queryPosXZ_minGridSize[arrIdx + 1].y = queryNormals[normi].z;
    _queryPosXZ_minGridSize[arrIdx + 1].z = minGridSize;

    _queryPosXZ_minGridSize[arrIdx + 2].x = queryNormals[normi].x;
    _queryPosXZ_minGridSize[arrIdx + 2].y = queryNormals[normi].z + s_finiteDiffDx;
    _queryPosXZ_minGridSize[arrIdx + 2].z = minGridSize;
}

Excessive use of array indexing:
_queryPosXZ_minGridSize[arrIdx + 0] - evaluated 3 times instead of 1
_queryPosXZ_minGridSize[arrIdx + 1] - evaluated 3 times instead of 1
_queryPosXZ_minGridSize[arrIdx + 2] - evaluated 3 times instead of 1
queryNormals[normi] - evaluated 6 times instead of 1
Should be replaced by something like this:
for (int normi = 0; normi < countNorms; normi++)
{
    var arrIdx = segment.x + countPts + 3 * normi;
    Vector3 ni = queryNormals[normi];

    _queryPosXZ_minGridSize[arrIdx + 0] = new Vector3 (ni.x, ni.z, minGridSize);
    _queryPosXZ_minGridSize[arrIdx + 1] = new Vector3 (ni.x + s_finiteDiffDx, ni.z, minGridSize);
    _queryPosXZ_minGridSize[arrIdx + 2] = new Vector3 (ni.x, ni.z + s_finiteDiffDx, minGridSize);
}

And such things are all around.
var dz = -Vector3.forward * s_finiteDiffDx;
for (int i = 0; i < countNorms; i++)
{
    var p = _queryResults[firstNorm + 3 * i + 0];
    var px = dx + _queryResults[firstNorm + 3 * i + 1];
    var pz = dz + _queryResults[firstNorm + 3 * i + 2];

    normals[i] = Vector3.Cross(p - px, p - pz).normalized;
    normals[i].y *= -1f;
}

Multiple evaluation of (firstNorm + 3 * i) and multiple indexing. Should be replaced by
for (int i = 0; i < countNorms; i++)
{
    int ofs = firstNorm + 3 * i;
    var p = _queryResults[ofs+ 0];
    var px = dx + _queryResults[ofs + 1];
    var pz = dz + _queryResults[ofs + 2];

    Vector3 n = Vector3.Cross(p - px, p - pz).normalized;
    n.y = -n.y;
    normals[i] = n;
}

Index range checking optimization.

Also, index range checking is performed on each iteration, it's additional slowdown. for primitive types fixed statements and pointers should be used as much as possible
int [] someArray;
int i, arrLen = someArray.Length;

for (i = 0;   i < arrLen;   i++)
{
    // internally following checks are performed:
    // if ((i < 0)   ||   (i >= someArray.Length)
    // {
    //    throw ArgumentOutOfRangeException ();
    // }
    someArray [i] = someThing;
}

can be replaced as follows:
fixed (int* someArrayPtr = & someArray [0])
{
    for (i = 0; i < arrLen; i++)
    {
         someArrayPtr [i] = someThing;
    }
}

Hello, we replied on the discord that we did not believe that these would provide a tangible speedup. Things like this tend to either get picked up by the compiler, or pale in comparison to real perf issues (memory reads, etc). If you have evidence (data) to the contrary, please provide it (we'll get notifications from this thread even if closed).
Otherwise we welcome you to make these changes to your own fork, but we wont be merging this into our repos evidence that these actually provide a speedup.


.env configuration are not taken into consideration

I modified the variable OPENID_CONNECT_DISCOVERY_URL in the .env file, re launched npm run dev, and the value did not change. Is there a cache to clean or a parameter to pass to the command ?

.env files has been replaced by a config.json in #23


How to inject additional dependencies not listed in source repo's main?

My bower.json:
{
  "name": "client",
  "version": "0.0.0",
  "dependencies": {
    "angular": "~1.4.0",
    "angular-route": "~1.4.0",
    "admin-lte": "~2.1.1",
    "bootstrap": "~3.3.4",
    "angular-resource": "~1.4.0",
    "angular-mocks": "~1.4.0",
    "angular-locale": "~0.1.9",
    "angular-bootstrap": "~0.13.0",
    "underscore": "~1.8.3",
    "angular-ui-router": "~0.2.15",
    "moment": "~2.10.3",
    "ngCordova": "~0.1.17-alpha",
    "underscore-contrib": "~0.3.0",
    "angular-chart.js": "~0.7.1"
  },
  "overrides": {
    "admin-lte": {
      "main": [
        "dist/css/AdminLTE.css",
        "dist/css/skins/skin-blue.min.css",
        "dist/js/app.js"
      ],
      "dependencies": {
        "bootstrap": "*"
      }
    }
  }
}

In the admin-lte section, I can't seem to get the overrides to fully work.
When running:
$ wiredep --src client/index.html --bowerJson client/bower.json

I get:
<!-- bower:css -->
<link rel="stylesheet" href="bower_components/bootstrap/dist/css/bootstrap.css" />
<link rel="stylesheet" href="bower_components/admin-lte/dist/css/AdminLTE.css" />
<link rel="stylesheet" href="bower_components/angular-chart.js/dist/angular-chart.css" />
<!-- endbower -->

With no output of skin-blue.min.css.
Basically, I'm trying to pull in a CSS skin that's not listed in the source repo's main; for example, here's AdminLTE's bower.json:
{
  "name": "AdminLTE",
  "version": "2.1.2",
  "homepage": "http://almsaeedstudio.com",
  "authors": [
    "Abdullah Almsaeed<EMAIL_ADDRESS>  ],
  "description": "Admin dashboard and control panel template",
  "main": [
    "index2.html",
    "dist/css/AdminLTE.css",
    "dist/js/app.js",
    "build/less/AdminLTE.less"
  ],
  "keywords": [
    "css",
    "js",
    "html",
    "template",
    "admin",
    "bootstrap",
    "theme",
    "backend",
    "responsive"
  ],
  "license": "MIT",
  "ignore": [
    "/.*",
    "node_modules",
    "bower_components",
    "composer.json",
    "documentation"
  ]
}

Question: Is what I'm trying to do not possible with wiredep (i.e., get wiredep to inject specific dependencies not listed in the source repo's bower.json file)?
Thanks!

Thanks for the detailed report! I tried it out, and my output got skin-blue.min.css, so everything appears to be working on my end.
I did a fresh install of global wiredep (npm install -g wiredep) and used your exact bower.json. bower install, ran wiredep, and it works. Can you try making sure wiredep is up to date, and if it still fails, try creating a new directory with just the bower.json & an html file with a wiredep CSS block?

Hi @stephenplusplus, thank so much for your help, I greatly appreciate it!

so everything appears to be working on my end.

Oh my gosh, I just tested outside of my project, and everything is working just fine too!
I feel terrible for wasting your time. Sorry that I did not run an independent test before opening an issue.
On the plus side, hopefully it will help others in the future to see an example on how to override and specify specific dependencies not listed in a repo's/package's main file. :+1:
Thanks so much for your time, and awesome code, I really appreciate it!

Lol, I just discovered why it wasn't working!
I had included the skin outside of the wiredep html comment:
<link rel="stylesheet" href="bower_components/admin-lte/dist/css/skins/skin-blue.min.css" />

We were doing that because we hadn't setup overrides yet. During my tests, I left that line in the template (outside of the bower:css comment).
As soon as I removed the skin CSS call, and ran wiredep, it worked!
Awesome. Thanks again @stephenplusplus!

No problem at all, I'm glad it works now! Our next wiredep release will have that behavior removed, because of this exact issue (explained here if you're interested).
Sorry you ran into this!