Support using GIT_SSH environment variable

I use several development environments, including msys2 and WSL. The default behavior of this project to always set core.sshCommand isn't necessarily what I want since using Win32-OpenSSH from msys2 doesn't work.
In my POSH profile, I set GIT_SSH myself to Win32-OpenSSH and do not set it using core.sshCommand in the following manner, before running Start-SshAgent.
function Get-NativeSshAgent {
    # $IsWindows is defined in PS Core.
    if (($PSVersionTable.PSVersion.Major -lt 6) -or $IsWindows) {
        # The ssh.exe binary version must include "OpenSSH"
        # The windows ssh-agent service must exist
        $service = Get-Service ssh-agent -ErrorAction Ignore
        $executableMatches = Get-Command ssh.exe | ForEach-Object FileVersionInfo | Where-Object ProductVersion -match OpenSSH
        $valid = $service -and $executableMatches
        if ($valid) {
            return $service;
        }
    }
}

$nugetModule = Get-Module -Name NuGet
$isInteractiveDesktop = [bool]((Test-Path env:\SESSIONNAME) -And ($env:SESSIONNAME -eq 'Console') -Or (Test-Path env:\WT_SESSION))
$isNonInteractive = [bool]([Environment]::GetCommandLineArgs() -Like '*NonInteractive*')
if (($nugetModule -eq $null) -And ($isInteractiveDesktop -eq $true) -And ($isNonInteractive -eq $false))
{
    # https://github.com/dahlbyk/posh-git/wiki/Customizing-Your-PowerShell-Prompt
    Import-Module 'C:\Users\lbakken\development\dahlbyk\posh-sshell\posh-sshell.psd1'
    Import-Module 'C:\Users\lbakken\development\dahlbyk\posh-git\src\posh-git.psd1'
    $GitPromptSettings.WindowTitle = [psobject]{param($GitStatus, [bool]$IsAdmin) "$(if ($IsAdmin) {'Admin: '})$(if ($GitStatus) {"$($GitStatus.RepoName) [$($GitStatus.Branch)]"} else {Get-PromptPath})"}
    $env:TERM = 'xterm-256color'
    $env:LESS = 'FRSX'

    $service = Get-NativeSshAgent
    if ($service) {
        $serviceName = $service.DisplayName
        Write-Host "Using native ssh agent service: $serviceName"
        $ssh = (Get-Command ssh.exe -ErrorAction Ignore | Select-Object -ExpandProperty Path).Replace("\", "/")
        Write-Host "Setting GIT_SSH to $ssh"
        $env:GIT_SSH = $ssh
    }

    Start-SshAgent
}

Thanks for the suggestion. Could you help me understand your use case a bit better? If you're setting GIT_SSH directly to the path of the native ssh client, then wouldn't you just omit the call to Start-SshAgent entirely?  Ie:
 $service = Get-NativeSshAgent
    if ($service) {
        $serviceName = $service.DisplayName
        Write-Host "Using native ssh agent service: $serviceName"
        $ssh = (Get-Command ssh.exe -ErrorAction Ignore | Select-Object -ExpandProperty Path).Replace("\", "/")
        Write-Host "Setting GIT_SSH to $ssh"
        $env:GIT_SSH = $ssh
    }
    else {
      Start-SshAgent
    }

Start-SshAgent isn't really useful if you're using the native client as once the service has been set to auto-start, so there's no need to keep running if you're taking that path.

Ah ignore that comment - I misread what you were doing, as with my suggestion you'd still end up setting the path to the windows service even when when you don't want to.
I'm happy with your change - could you add a test for it in Ssh.Tests.ps1, and then I can merge it.

@JeremySkinner - test added, sorry it took so long. Thanks for the awesome libraries.


Update User Guide

Codecov Report

Merging #84 (f62216b) into master (cc7da3a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #84   +/-   ##
=========================================
  Coverage     72.18%   72.18%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1233     1233
  Branches        125      125
=========================================
  Hits            890      890
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


some assets are missing

some assets are missing from the “performance test” “component” and “gameobject” samples

Thanks I missed that! Fixed now.


List detected feeds in BrowserAction context menu

As title says, possibly make PageAction optional (leave it on for people having it already installed?)

decided to remove the PageAction altogether, it was causing too much mess in the code to handle it with Chromium shenanigans


Add metric for FQDN regex LRU access and document existing FQDN metrics

See commit msgs, main commit is copied here for visibility.

This commit enables visibility into the FQDN regex compilation LRU by
showing "hit" accesses (cache hit) and "miss" accesses (cache miss).
To enable, the user must enable the metric via --metrics +cilium_fqdn_regex_lru_accesses_total.
Signed-off-by: Chris Tarazi<EMAIL_ADDRESS>

/test

Nominating for v1.11 and v1.10 backports due to usefulness, especially considering that https://github.com/cilium/cilium/pull/19809 amdhttps://github.com/cilium/cilium/pull/19857 have been backported.

FYI @ungureanuvladvictor opened https://github.com/cilium/cilium/issues/20235 to further track the rest of the metrics reference updates for telling which are enabled by default.

Rebase with master will help with IngressConformance failure https://github.com/cilium/cilium/runs/6936167896?check_suite_focus=true.

Hey! I think this metric will be very interesting! I'm however a bit worried that these metrics won't really be too useful for deciding on the lru cache size to use. We got really bit by the change from 128 lru entries to 1024 by default, since that made some environment constantly OOMing without anything very interesting in the pprofs.
I feel like the lru cache might not be the perfect solution, although that is the topic of another discussion I assume. I would at least think that having an in memory index of the compiled patters we keep in memory, so that we can reuse them without touching an lru cache. I did a quick initial example implementation to show it here; https://github.com/cilium/cilium/pull/20247/files. The commit message has more info. The short thing is I suspect that most of the lru cache misses are due to patterns we have never seen before, aka. they have never been a part of the lru cache. Also, I assume most cache hits are for patters we already use elsewhere anyways, so we probably don't need an lru cache to find those. Having a separate code path for checking that "in use mapping" and having a separate metric for it, would be very interesting.
Right now, to reduce memory consumption, we have to reduce the cache size. If we reduce it, the chance for having to recompute/recompile a value/pattern again, even though its already used somewhere else, since it might have been evicted from the lru cache; while still being in memory since used elsewhere. On the other hand, when expanding the lru cache size, we end up with more dangling regexes that are still no longer in use, taking a lot of extra memory. A good portion of them, we will most likely never use again, but they will still hang in that lru for quite a while.
I think that having an in index of the already in memory regexes in a separate index, and then having an configurable lru for the rest, aka. those no longer in use; but that we might want later. I think that will help improve both memory consumption and cpu usage.
(on another thread, related to this, I'm also looking into making the regexes smaller, faster and better here; https://github.com/cilium/cilium/pull/20246. In essence they are much faster, much smaller and should work the same. If CI pass, we can maybe test it in a real environment to see the difference!)

Converting to draft because I agree with @odinuge that we can rebase this PR (and make corresponding changes) depending on how https://github.com/cilium/cilium/pull/20396 and https://github.com/cilium/cilium/pull/20246.


Typescript Inlay Hints with literals messes up argument names if variables are used

Type: Bug
With the following setting:
"typescript.inlayHints.parameterNames.enabled": "literals"
And the following code in a typescript file:
function checkInlays() {
    const first = true;
    const second = false;
    const third = false;
    inlayHintCheck(true, false, false)
    inlayHintCheck(first, false, false)
    inlayHintCheck(first, second, false)
    inlayHintCheck(first, second, third)

  }

  function inlayHintCheck(firstArgument: boolean, secondArgument: boolean, thirdArgument: boolean) {
    console.log(`${firstArgument} ${secondArgument} ${thirdArgument}`);
  }

I would expect the hints to show like:
inlayHintCheck(firstArgument: true, secondArgument: false, thirdArgument: false)
inlayHintCheck(first, secondArgument: false, thirdArgument: false)
inlayHintCheck(first, second, thirdArgument: false)

or

(created with having the setting  "typescript.inlayHints.parameterNames.enabled": "all" )
But it becomes:

Issues:

Would have expected it to still show the hint when adding a variable that doesn't match in name
It should display the correct argument name when skipping over a previous argument

VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:51:52.184Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz (8 x 2419)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.69GB (2.35GB free)

Process Argv
--folder-uri file:///c%3A/src/administration-frontend

Screen Reader
no

VM
0%

Extensions: none

/duplicate of https://github.com/microsoft/TypeScript/issues/55866
Already fixed in latest TS version

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


CSP compliant build

Basically use a different exp-parser that uses Esprima. Since size doesn't matter that much in CSP-enforcing environments, the extra code is an acceptable tradeoff.

Just tried to use CSP version in Chrome App and I still get
Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src 'self' chrome-extension-resource:".


Addresses #13264 where setting saveFoldStates preference to false caused folding problems

This PR properly initialises the _lineFolds cache to the empty object when the saveFoldStates preferences is set to false. Previously, this was not being done and it caused issues down stream, i.e., when rendering gutters as the active editor changed.
Also updated the test cases to accommodate the requirement.
cc @lkcampbell @ficristo

@thehogfather, the changes look good to me. Initialize the uninitialized property with an empty object and get rid of the undefined/null error message down the road.
Is _codeMirror._lineFolds where you are storing all of the fold state information for each instance of CodeMirror opened in brackets?
The only problem I see is the Travis build is failing. @ficristo or @petetnt any ideas what might be causing the build fail?

@lkcampbell

Is _codeMirror._lineFolds where you are storing all of the fold state information for each instance of CodeMirror opened in brackets?

Yes that is correct.
With respect to Travis failing, I'll pull upstream and push to trigger the build again. The error messages show it did not get very far into the build process.

That did no seem to help. Let me know if there is anything I can do.

Restarted the CI by hand, I'll look into it if it still fails

I was just trying out with the OAuth token and couldn't exceed the limit in 60 attempts. Not sure why this one failed!

Fixes #13264 (adding this comment so GitHub creates a link to the issue)

Okay, I check that the fix works and ran the unit tests: no errors.
Merging.


Update strict compile numbers

This PR fixes #

Pull Request Test Coverage Report for Build 233423485

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

Totals

Change from base Build 233402036:
0.0%

Covered Lines:
18908

Relevant Lines:
44262

💛  - Coveralls


Practice App Research: Research on Docker

What's up?
As a part of our Practice App assignment, after completing the implementation we have to dockerize our application and test that each member is able to deploy it on their personal development environment.
To Do
Each member should perform an individual research on Docker and become familiar with the concept. Please check the boxes below after making your research to let the others know.

[ ] @ooodogodogodogo
[ ] @serdarakol
[ ] @kostanya
[ ] @BElifb
[ ] @simsekhebun
[ ] @KarahanS
[ ] @sinemKocoglu
[ ] @mumcusena
[ ] @mustafa-cihan
[ ] @dundarmete

Members assigned to the issue must perform the following tasks:

[ ] Create a wiki page related to the Research on Docker.
[ ] Contribute to the wiki page. You contributions may include text, images, links to educational materials etc. Try to answer some basic questions such as:

What is Docker? What is a Docker image and why is it necessary?
How to get started with Docker?

[ ] Attach the wiki page to the sidebar.

Deadline
Deadline: 01.05.2022 @20.00
Additional Information
No response
Reviewers
Reviewer for the research and documentation: @BElifb

Created research on docker page, added it to sidebar.
Added some useful links and visuals.

Added a brief definition of Docker and container.
Added a link to a video tutorial on Docker for beginners.

Research procedure is done. No action is taken.

Added basic instructions to dockerize an application including:

How to write a Dockerfile.
How to build the image.
How to run the container.


Has anyone been able to get this to run under AWS Lambda

Alternatively is it available as a Lambda Layer?

Making Layout Parser a service is on our roadmap, and feel free to contribute to our repo if you've made any progress! Thanks!

I've tried and failed to get LP on run on AWS Lambda. Firstly the size of the dependencies make it near impossible to deploy a zip file to Lambda(cause all zip files should be under >265MB).
I also tried the container approach but it was really hard to install the dependencies on an Amazon Linux container. This is cause of missing gc++, versioning issues, etc. I managed to finally get everything installed but for some reason I couldn't run inference inside the container at all.
Given that I'm fairly new to lambdas, there might be some stupid mistakes on my side but if this was an existing layer then it'll be super easy to work with.

Any update on this?

If anyone can help here..
I have prepared a set of commands that installs and run layoutparser without any error on aws ubuntu18.06 image.
sudo apt-get update -y
sudo apt-get install python3.7-dev -y
sudo apt install python3-pip -y
sudo update-alternatives --install /usr/bin/python3 python3 /usr/bin/python3.7 1
python3 -m pip install --upgrade pip
sudo apt-get install poppler-utils -y
sudo apt install tesseract-ocr -y
pip3 install scikit-build
pip3 install opencv-python-headless==<IP_ADDRESS>
pip3 install layoutparser[ocr]
pip3 install layoutparser torchvision && pip3 install "detectron2<EMAIL_ADDRESS>
I am trying to deploy this to lambda.. however, the size is becoming an issue here. After running all above command, it generates around 4gb folder which exceeds lambda size for layers, or zip

@lolipopshock I was able to run this under AWS Lambda. I am not sure about the license whether I can it publically available.


Multitenancy without accessing ThreadPoolManager

Fixes #754
Does this make sense?
It should also be backward compatible to when we are going to remove ThreadPoolManager.

Added a commit to remove the Handlers class

Ci seems to have some trouble downloading dependencies. I will wait until CI pass before merging this

@yaakov-berkovitch Is this going to solve your issue?

@DavideD Looks good even more. Thanks. Can I use your branch or better to wait for the merge to main ?

I'm going to merge it tomorrow. I don't think there are going to be differences between main and my branch

Merged!
Thanks

@DavideD after pulling this I get a test failure when running the tests w/o Docker.
(They work fine when running on Docker.)

What kind of error? I doubt it's related

Gradle Test Executor 3 > org.hibernate.reactive.ReactiveMultitenantTest > testTenantSelection FAILED
    java.util.concurrent.CompletionException: io.vertx.pgclient.PgException: { "message": "database \"dbtenant1\" does not exist", "severity": "FATAL", "code": "3D000", "file": "postinit.c", "line": "877", "routine": "InitPostgres" }
        at java.util.concurrent.CompletableFuture.encodeThrowable(CompletableFuture.java:292)
        at java.util.concurrent.CompletableFuture.completeThrowable(CompletableFuture.java:308)
        at java.util.concurrent.CompletableFuture.uniApply(CompletableFuture.java:607)

Pretty sure it's related :-P

Maybe something goes wrong during the creation of the additional databases?
If you set SHOW_SQL to true in the base class, you should see this:
INFO SchemaCreatorImpl [vert.x-worker-thread-0] HHH000476: Executing import script 'file:/home/ddalto/workspace/hibernate-reactive/hibernate-reactive-core/build/resources/test/multitenancy-test.sql'
[Hibernate] drop table if exists Pig cascade
[2021-05-18 10:10:03] WARNING io.vertx.sqlclient.impl.SocketConnectionBase            Backend notice: severity='NOTICE', code='00000', message='table "pig" does not exist, skipping', detail='null', hint='null', position='null', internalPosition='null', internalQuery='null', where='null', file='tablecmds.c', line='1217', routine='DropErrorMsgNonExistent', schema='null', table='null', column='null', dataType='null', constraint='null'
[Hibernate] create table Pig (id int4 not null, name varchar(255), version int4 not null, primary key (id))
[Hibernate] CREATE DATABASE dbtenant1
[Hibernate] CREATE DATABASE dbtenant2
[Hibernate] select current_database()
[Hibernate] select current_database()
[Hibernate] insert into Pig (name, version, id) values ($1, $2, $3)
[Hibernate] select reactivemu0_.id as id1_0_0_, reactivemu0_.name as name2_0_0_, reactivemu0_.version as version3_0_0_ from Pig reactivemu0_ where reactivemu0_.id=$1
[Hibernate] select reactivemu0_.id as id1_0_0_, reactivemu0_.name as name2_0_0_, reactivemu0_.version as version3_0_0_ from Pig reactivemu0_ where reactivemu0_.id=$1
[Hibernate] delete from Pig where id=$1 and version=$2

I don't see why using docker should make a difference. When you say that you don't use docker, how do you start posgres?

Well the user hreact doesn't have permissions to create a database. Because we don't grant it that permission in the README instructions.

Could you try this additional step?
ALTER USER hreact CREATEDB

If it works I will update the instructions in the README

Yes, that works. I was trying (and failing) to figure out how to do that with grant.

Also, it would probably be more elegant if the test cleaned up the database it creates when it finishes.

I thought about it, but in the end I didn't see any actual benefit  in adding the drop queries.

Anyway, I will send a PR shortly

Hi @DavideD, I started integrating with multi-tenancy, but I faced an issue.
According to the doc, the hibernate.tenant_identifier_resolver is optional, if I specify the tenant-id explicitly when opening a session. So I didn't define a tenant identifier resolver class.
My test failed anyway with the error "SessionFactory configured for multi-tenancy, but no tenant identifier specified".
So, something wrong with the doc or the code  ?

Mmh... no, the documentation should be correct. Maybe the error happens during schema generation? If you are creating the database with Hibernate Reactive, I don't think you can specify a schema, so without a schema resolver it doesn't know which tenant to use.

it doesn't occur during schema creation, but during tests. Schema creation is currently not part of my tests.

OK, I will have a look. If you could share your test, that would help

Nevermind. I've found the issue

Good, I just stand to send you my test :-)

Hi @DavideD, did you already push a fix for the tenant identifier resolver ?


Improve fixed scroll with animation

https://github.com/lcrojano/Giphy_Explorer/assets/18175528/d1b818a2-da1a-43f6-95c2-34fbfe6db3af

Do I need to do something with these css lines?


HTTP.sys WS config+example updates

Fixes #10164
Internal Review Topic (Configure Windows Server section)
@marqdouj, you won't be able to see the built internal topic ... it's only visible to staff. You can flip over to the diff to see the proposed updates. If you get a chance to look at this, let me know if you feel all of the bases are covered here.
TL;DR the list of little enhancements I'm going for here. However, you can go straight to the diff ... it's all there.

References to "certificate hash" should be changed to "thumbprint" because it's immediately recognizable and reported that way in cert properties.
Note installing the cert for the Local Machine in the Personal store.
Note that the appid is informational and generated by the dev for the add sslcert command.
For the example code and commands, use an IP example ... e.g., options.UrlPrefixes.Add("https://<IP_ADDRESS>:443");.
In addition to opening required ports in the WS firewall, we may as well remind readers using Azure VM's to open the port(s) in their NSG.
For reference, show the commands for delete urlacl and delete sslcert.
Update the sample app to 2.2 (and let's drop the static assets while we're at it).
Add note that for an SCD no installs are required on the server.
Update the .NET Core download links.
Re-organize a bit ... currently, server activities are split across the app steps. Consolidate the server steps and app steps.
Show commands with placeholders and example entries.

I'll add a note for this on the next commit ...
services.AddAuthentication(Microsoft.AspNetCore.Server.HttpSys.HttpSysDefaults.AuthenticationScheme);

https://stackoverflow.com/questions/45818095/asp-net-core-2-0-httpsys-windows-authentication-fails-with-authorize-attribute


Typo in Question 24

readiness  not readyiness

Fixed in https://github.com/redhat-cop/pathfinder/commit/d184ea60019c9e495f4ef04b78b8928a4c866e9d


Update filters.txt

While queryset is mentioned in the **kwargs section, it is in relation to "the accompanying form Field".  I was not using the forms for my project (REST API) so did not realise that a queryset kwarg was a hard requirement for ModelChoiceFilter and ModelMultipleChoiceFilter.

In relation to #146.  I haven't used whatever markdown the docs are in before so apologies if my formatting needs fixing.  Also it's a while since I had the issue and I don't really fully understand the package so probably good to double check if my changes are actually correct...

👍


Pyspark Kernel is not loading

Issue Type: Bug
Right click on BDC connection --> New Notebook --> Change kernel to Pyspark
It hungs and show an error.
Azure Data Studio version: azuredatastudio 1.25.1 (e7eacc32c0d20bd4da843320c3bfac231b308c98, 2020-12-11T01:51:41.708Z)
OS version: Linux x64 5.8.0-31-generic

More info:
I tried with sparkmagic 0.15 and 0.17.1. Same behavior.
It just gets stuck in "Changing Kernel..." and fails with this error fisrt:
Failed to change kernel due to error: Canceled
And if I try again, it fails with:
Failed to change kernel. Kernel pysparkkernel will be used. Error was: Invalid response: 404 Not Found
I would appreciate your help.

@VasuBhog could you help here please?

Hi @emacastelli, I am sorry to hear that you're having trouble with the pyspark kernel. I was not able to repro this with my Mac and Windows Machines. This may be a Linux based issue. One thing I recommend is uninstalling sparkmagic and reinstalling it to the latest.
Once you open the notebook via the BDC connection, does it set the kernel to SQL?

Python Version: 3.6
sparkmagic: 0.17.1
ADS Version: 1.25.1

Hi! I tried what you suggested but the problems its still there.
When I open notebook from BDC it goes to SQL  kernel.
I think I could have some issue with python local installation. I have this:

What installation do you suggest me to choose?

Yes, it must be a sparkmagic and python compatibility issue. I am currently using Python 3.6 on both machines and it seems to work with the newest version. Can you please try to download and use Python 3.6?

Hello,
I could fix the issue. I was uninstalling / installing sparkmagic manually, from terminal. I did it again from Packages Manage and it worked.
Thank you.

Resolved by unistalling /installing sparkmagic from Packages Manager (ADS)

No problem :). Great to hear!


Escape "-", don't escape "}"

"-": this is a control within character sets. Escaping content for use within character sets is already fully supported except for this character, including escaping "]", which doesn't need escaping except within character sets.
"}": this isn't a control anywhere except within quantifiers. Within a quantifier, escaping this results in an invalid regex. escape-string-regexp doesn't, and can't possibly, support escaping strings meant for a quantifier context.
If you approve of the change, I'll make a PR.

It's been eight months with no response, please give this issue some attention.
At the very least the README should be updated to state that character sets aren't supported.

It's been eight months with no response, please give this issue some attention.
At the very least the README should be updated to state that character sets aren't supported.

@twhb Considering what you said this should be the correct solution
var matchOperatorsRe = /[|\\{()[^$+*?.-]/g;

E.g.:
var matchOperatorsRe = /[|\\{()[^$+*?.-]/g;
'the characters {, (, ), [, ^, ?, +, *, . and - are to be escaped'
  .replace(matchOperatorsRe, '\\$&');
// 'the characters \\{, \\(, \\), \\[, \\^, \\?, \\+, \\*, \\. and \\- are to be escaped'

Lodash version does not escape "-":
https://github.com/lodash/lodash/blob/master/escapeRegExp.js
In the other hand, bobince's version does:
https://stackoverflow.com/a/3561711

@jbruni I take back what I said. The character "-" need to be escaped only if inside a [ and ] block. These are escaped already so there is no need to escape "-".
var matchOperatorsRe = /[|\\{()[^$+*?.]/g;
var text = 'a-z [a-z]'
var escaped_text = text.replace(matchOperatorsRe, '\\$&')
var regexp = new RegExp(escaped_text)
regexp.test('a-z a') // false
regexp.test('a-z [a-z]') // true

i have same problem.
try this sql pattern:

SELECT * FROM members WHERE username = 'admin'--' AND password = 'password' - --

= and - does not escaped.

i have same problem.
try this sql pattern:
SELECT * FROM members WHERE username = 'admin'--' AND password = 'password' - --
= and - does not escaped.

I have decided to add - so this module can be used to escape a string that will be inserted in the middle of a regex. I don't see the point of removing }. Better to leave it for the same reason as previously mentioned.

@ujikit = doesn't need to be escaped.

Thanks for the update. What reason are you referring to when you say “as previously mentioned”, “so this module can be used to escape a string that will be inserted in the middle of a regex”? If that’s the case, then escaping “}” does not forward that goal: an escaped “}” is not valid anywhere that an unescaped “}” would not be interpreted literally.

@twhb You might be right, but there's a reason it was added in the first place and why all other similar libraries escape it. I don't really have the time to prove you wrong. I'm happy to remove it if you manage to convince https://github.com/lodash/lodash/blob/master/escapeRegExp.js to remove it.

FYI this breaks Unicode regexes:

@steadicat Could you please create an issue for that?


BUG: heatmap dimensions are distorted

Bug Description

Steps to reproduce the behavior
generate a heatmap with q2-feature-classifier 2019.7 (older releases were not affected)
Expected behavior
heatmap should be square and top/bottom rows should not be truncated.
Comments
I am fairly certain this is not isolated to q2-sample-classifier. We should see if q2-feature-table and any other plugins that use seaborn.heatmap or seaborn.clustermap are affected.

@nbokulich , does this look right to you? I generated this heatmap after installing matplotlib 3.1.0 to a fresh 2019.7 environment. I did not roll back matplotlib-base, and just wanted to make sure I was getting the expected results before opening a PR to pin this.

Yes! That looks right. Thanks!

Hey @ChrisKeefe --- feature-table heatmap isn't a good test for this bug, it is built in a different manner that doesn't suffer from the problem reported above - can you double check the heatmaps in this plugin?

Thanks for the eyes, @thermokarst . This was built with sample-classifier heatmap, which does wrap feature-table heatmap but appears impacted in @nbokulich 's link above.
Maybe I misunderstood the conversation here, but regardless, I generated a clean confusion matrix using the same environment: 19.7, matplotlib 3.1.0(rolled back), and matplotlib-base 3.1.1 (unmodified). GH is misbehaving right now, but I can slack it to you if you like.
Does pinning only matplotlib feel good to you?

This was built with sample-classifier heatmap

Thanks for clarifying!

I was able to generate a clean confusion matrix using the same environment

Perfecto.

In this context, does pinning only matplotlib (and not matplotlib-base) feel good to you?

This seems a little strange to me --- my understanding is that these two packages (matplotlib and matplotlib-base) should be pinned in lockstep. Did you encounter any issues when you pinned to matplotlib-base 3.1.0?
FWIW - it looks like MPL 3.2 might be landing any day now - maybe we should hold off on making a decision until we get closer to our 2019.10 release day? @ChrisKeefe, do you think you could double-check the confusion matrix results in an env with mpl 3.2.0rc1? Thanks!

Hey @ChrisKeefe --- feature-table heatmap isn't a good test for this bug, it is built in a different manner that doesn't suffer from the problem reported above - can you double check the heatmaps in this plugin?

Oops! I was confusing this issue for the issue described in https://github.com/qiime2/q2-sample-classifier/issues/143 --- my mistake! What I wrote above holds true, I was barking up the wrong tree regarding the feature-table heatmap business. Thanks!

Downgraded matplotlib-base to 3.1.0 in both 19.7 and current dev environments - solving didn't affect any additional libraries, and it produces nice clean heatmaps.
If matplotlib 3.2 doesn't launch before release, I think we're good to pin both to 3.1.0.


Add tests for formatters

@joshreisner The tests were getting a little hard to parse all being in one file so I ended up splitting them out, let me know what you think. We might have to do the same with data.js 👀

looks great! def makes sense to break up format and data.
think the previous committee caused a small conflict in one file


Door of castle should end the level

Right now Mario has to traverse some distance after door of castle for level to change, this should be avoided

I´ll take a look.

I'll go and see if I can fix this

Thanks


feat: rename columns

Description
I'd like to be able to rename columns using ALTER TABLE, but this appears to not be implemented.
I tried
alter table nyc_sales rename column "ease-ment" to "easement";
alter table nyc_sales rename "ease-ment" to "easement";

And I get
Query failed to run.

SQL statement currently unsupported: RENAME COLUMN "ease-ment" TO "easement"

Let me see if I can make this happen quickly.

I did add the functionality. We get the schema from delta snapshot which causes discrepancy. Delta or any other backend would maintain its own schema which might be a bit harder to deal with right now 🤔

FWIW, deltalake does plan on supporting RENAME COLUMN and it's even in the official implementation, but it doesn't look like it's made it's way into delta-rs yet.
It's currently marked as experimental, so it'll likely be a while before delta-rs adopts it.

reclassifying as "feature" as it's technically not a bug, but just missing functionality.


Parsing error on TSX block comments

Came across this parsing error on JSX comment's between curly braces (ex. {/* comment */})
const Basic = (

    {/* comment */}
    Cookie

);

On token Cookie

OCX playground
TypeScript playground

The following works:
const Basic = (

    Cookie
    {/* comment */}

);

So it appears to be an issue allowing additional jsx_child after consuming a jsx_expression_container

Thanks for the super quick fix!


Fix document replacing when content hasn't changed

Codecov Report

Merging #3 (fc00030) into main (2716b02) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #3   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           74        74
  Branches        12        12
=========================================
  Hits            74        74

Flag
Coverage Δ

integration_tests
94.59% <0.00%> (ø)

total
100.00% <0.00%> (ø)

unit_tests
60.81% <0.00%> (ø)

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

Continue to review full report at Codecov.

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


Awesome books classes

In this project we :

Using Classes instead of Objects.
Add Clear Input field Function.
Update page styling.

Hi @prolajumokeoni, Thanks a lot for your efforts in reviewing our code. 😄


Explorer Testing Placeholder

Began reviewing the staging explorer at https://explorer-silk.now.sh/
Currently there is an issue where somethings don't work because of https mixed content. Functionality is limited but I can connect to API http://crashy-stacky.zone117x.com/tx and return info for some of the tx_id but no other data points.
The URL below returns valid data
https://explorer-silk.now.sh/txid/0xeaf47b0d9b4a7439ff46943ac313637e3484c2393729da49fceac0d04d4d1ce0
@kyranjamie will let me know when the explorer is functional and ready for test.

Any given transaction is very ephemeral for now and should not be expected to exist for more than an hour. This is due to instability in the stack that requires wiping all data.
This situation will be improved as core-node stabilizes. Track relevant issues:
https://github.com/blockstack/stacks-blockchain/issues/1130
https://github.com/blockstack/stacks-blockchain/issues/1402
https://github.com/blockstack/stacks-blockchain/issues/1418

Thanks for the info @zone117x I am not writing any bugs at the current moment, as the explorer is not is a testable state. Just wanted to get an issue up for communication as we move towards the QA phase.
Thanks for linking the related blockchain issues as soon as the core node is up and running I will give it a first pass.

Hey guys just following up, are there any updates on testable functionality that I can get started with this week?

ok third time is the charm :)
@timstackblock I will write up a document today outlining how to properly test the explorer and send it over to you once it's ready

Thanks @aulneau I will use that to finalize test cases and let you know if I have any questions?

Twitter meta data does not show the blockstack logo image

Just storing the output from the google schema test run this is not a bug


【SCU】【PPSCI Export&Infer No.10】 cfdgcn

PR types
Others
PR changes
Others
Describe
为cfdgcn算子添加export和inference功能

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Got error when compile the code

I ran into the following error after cloning from github.
make
first make sure we have godep
go get github.com/tools/godep
/Users/stevesui/go_work:/Users/stevesui/go_work/httpbeat/bin/godep go build
make: /Users/stevesui/go_work:/Users/stevesui/go_work/httpbeat/bin/godep: No such file or directory
make: *** [httpbeat] Error 1
Basically, I am not sure when the bin directory will be created during the make process.
Thanks.

I tried to use go install command and got the following error:
go install
github.com/christiangalsterer/httpbeat/beat
beat/httpbeat.go:34: b.Events undefined (type *beat.Beat has no field or method Events)
Please advise on how to resolve the issue.
Thanks.

Same error here when attempting to build on windows.
# github.com/christiangalsterer/httpbeat/beat
beat\httpbeat.go:34: b.events undefined (type *beat.Beat has no field or method events)

I had the same problem. To solve this quickly, you have to go to the beats library (should be downloaded in \src\github.com\elastic\beats) and checkout to v.1.2.0 tag.

I get following error can some someone help me please?
godep: [WARNING]: godep should only be used inside a valid go package directory and
godep: [WARNING]: may not function correctly. You are probably outside of your $GOPATH.
godep: [WARNING]:	Current Directory: ../Tools/httpbeat-master
godep: [WARNING]:	$GOPATH: ../Tools/httpbeat-master
godep: WARNING: Go version (go1.6) & $GO15VENDOREXPERIMENT= wants to enable the vendor experiment, but disabling because a Godep workspace (Godeps/_workspace) exists
main.go:5:2: cannot find package "github.com/christiangalsterer/httpbeat/beat" in any of:
/usr/local/go/src/github.com/christiangalsterer/httpbeat/beat (from $GOROOT)
../Tools/httpbeat-master/Godeps/_workspace/src/github.com/christiangalsterer/httpbeat/beat (from $GOPATH)
.../Tools/httpbeat-master/src/github.com/christiangalsterer/httpbeat/beat
godep: go exit status 1
make: *** [httpbeat] Error 1

Not sure if this makes a difference but it seems that you are using Go 1.6. I used 1.5 so far make this makes a difference.
Can you also check that your GOPATH env variable is set correctly? I have set it to the parent directory of all my Go projects.

Could it be that this is related to different versions of libbeat used? Golang 1.6 as GOVENDOREXPERIMENT enabled by default and we use it in libbeat.

I think so too.
I will update a new version soon, using Glide for better dependency management/version handling.

Version 1.2.0 now available using glide

Thanks @christiangalsterer ，
your work helps me a lot, while I was trying to use the httpbeat. First I install gvm,then I install golang1.4 and golang1.6.3,then glide,then I cd $GOROOT/src and I git clone httpbeat,after I make httpbeat ,these errors appears, can you help me? thanks
main.go:5:2: cannot find package "github.com/christiangalsterer/httpbeat/beat" in any of:
/home/ubuntu/.gvm/gos/go1.6.3/src/httpbeat/vendor/github.com/christiangalsterer/httpbeat/beat (vendor tree)
/home/ubuntu/.gvm/gos/go1.6.3/src/vendor/github.com/christiangalsterer/httpbeat/beat
/home/ubuntu/.gvm/gos/go1.6.3/src/github.com/christiangalsterer/httpbeat/beat (from $GOROOT)
/home/ubuntu/.gvm/pkgsets/go1.6.3/global/src/github.com/christiangalsterer/httpbeat/beat (from $GOPATH)
make: *** [httpbeat] Error 1

Where is the $GOPATH variable pointing too? It should point to the root of all your go projects.


revert new token set button sticky

makes token set button no longer sticky

Merging because of 1-liner


Not really a bug but can I use the library without typescript?

If yes then can I get someone to help me understand what is done here please and how that would look like without ts
<UploadDropzone<OurFileRouter>
Cheers friends

Sure - if it's JavaScript just omit the generic
<UploadDropzone {{ props }} />

Thnx. Ah o it's just a prop, nice1


check fileReader->init() has failed

While running apra pipes UTs FileReader module throws error.
/home/ubuntu/src/ApraPipes/base/test/nvjpeg_combo_tests.cpp(116): error: in "nvjpeg_combo_tests/decode_resize_cc_raw_1920x960": check fileReader->init() has failed

@Vinayak-YB please work on this issue


UPnP IGD & PCP: Remove Interface column in status, as it shows the WAN

and document limitations and wording improvements

Regarding:

Set the maximum upload speed in kbit/s, informational only.

What is meant by informational only?

Thank you for explaining. I think we can leave the interface column since it provides a quick way to know which WAN interface the mapping was set up for (e.g. in case of failover gateways).
That max speed description is certainly misleading. Perhaps we can clarify that more by saying "Report a maximum upload speed (kbit/s)."

Looks good. Remove the draft flag from the PR and perhaps simplify the commit message a bit - like "UPnP IGD & PCP: improve descriptions". Should be good to merge after.

How about What is my maximum download speed in kbit/s?
otherwise Report maximum download speed in kbit/s as

I suggested "report" because we're reporting to clients of the max speed. That also implies it's informational only.


[PRODX-24162]: added tests for catalogEntities

👀

👀

@AntonNos Should this PR be fixed and merged, or closed? It's for a ticket we have since marked as done.

@AntonNos Should this PR be fixed and merged, or closed? It's for a ticket we have since marked as done.

I fixed comments, check please

toBe

Okay, understand. Will use toBe() first in the future tests


Editor does not add colon and autoindent when I enter e.g. if a==3 # comment

From<EMAIL_ADDRESS>on 2013-05-31T11:58:25Z
What steps will reproduce the problem?

in editor enter if a==3 # comment

What is the expected output?
if a==3: # comment
(indentation)
What do you see instead?
if a==3 # comment
(no  indentation)
What version of the product are you using?
2.3.0 Dev 1
On what operating system?
Win XP
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1416

From ccordoba12 on 2015-01-02T18:58:26Z
Labels: Cat-Editor MS-v2.4

Automatic colons are inserted only at the end of the line.
However, indentation work if you write
if a == 3:<Enter> # comment

I think it's the best that we can do :-)

It should be doable: strip comments, check if colon needs to be added, readd comment, and voilà ! (same for indentation).

I think it's hard to get it right in all situations. But if you want to try it, please submit a PR :-)

Yeah, I can see some corner cases:
if c == "#": # comment


Updating the code to use kubemci instead of mci

Updating the code to use kubemci instead of mci.
This involves using kubemci in command description strings and updating the code directory and files to be named kubemci instead of mci.
Note that the prefix for GCP resources is still "mci1".
cc @G-Harmon @csbell @madhusudancs

The first commit is from https://github.com/GoogleCloudPlatform/k8s-multicluster-ingress/pull/14 and hence should be ignored while reviewing this PR.

LGTM
Can you commit description to be a little clearer that this is "renaming the tool kubemci" or something?

Rebased and updated the commit message.
Thanks for the review @G-Harmon

/lgtm
/approve


Issue Template Updates for Figma link

Current Behavior
The project's current issue templates are missing an open invitation link where new contributors can join the community's Figma team and view user interface designs and other UX projects.
Desired Situation
Each template that has a reference to Figma in its resources section should an invite link added.
Implementation
- 🎨 Wireframes and [designs for Layer5 site](https://www.figma.com/file/5ZwEkSJwUPitURD59YHMEN/Layer5-Designs) in Figma [(open invite)](https://www.figma.com/team_invite/redeem/qJy1c95qirjgWQODApilR9)

Acceptance Tests
All references to Figma include the "open invite" link.

Resources

Alternatives / Additional Context

Hey, I can try to work on this.

Current Behavior
The project's current issue templates are missing an open invitation link where new contributors can join the community's Figma team and view user interface designs and other UX projects.
Desired Situation
Each template that has a reference to Figma in its resources section should an invite link added.
Implementation
- 🎨 Wireframes and [designs for Meshery UI](https://www.figma.com/file/SMP3zxOjZztdOLtgN4dS2W/Meshery-UI) in Figma [(open invite)](https://www.figma.com/team_invite/redeem/qJy1c95qirjgWQODApilR9)

Acceptance Tests All references to Figma include the "open invite" link.
Resources
Alternatives / Additional Context

I think here the implementation link is from meshery ui but it should be of layer5 design.

@Sankhadip-Roy, you're right, I've updated the issue description. Thank you for pointing out this
@depleur, go ahead 🚀

Hey, I don't see a reference to Figma in this codebase.

I want to work on it. Can u assign me? @thisiskaransgit

@depleur, updates?

@thisiskaransgit assign me will do the fixes immediately!

@thisiskaransgit as @depleur bhai removed his assignment i have made PR for the issue!


Add Gloo Edge, and Gloo Mesh.

Solo.io is an EKS anywhere partner.
https://aws.amazon.com/eks/eks-anywhere/partners/
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: murphye
To complete the pull request process, please assign chrisnegus
You can assign the PR to them by writing /assign @chrisnegus in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

docs/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @murphye. Thanks for your PR.
I'm waiting for a aws member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

Holding this while we discuss internally the best place in the docs to include partner integrations.
/hold

Solo.io is an EKS-A partner, so it meets the criteria for inclusion here.
/lgtm

@murphye We will be adding a docs contributor page to provide guidelines. In the mean time, this meets the criteria we discussed. Thanks for the PR and sorry it took so long to approve.

/remove hold

/retest

/retest

/ok-to-test

/retest

/lgtm

/approve

/retest

/lgtm

/retest

/override lint

/restest

/retest

/joke

I think this needs a rebase
If not one is on it, I can help

/lgtm


Apply patch error

this happens when I try to apply the patch on fedora 31.
[malik@localhost wine-tkg-git]$ git apply roblox-wine-staging-v2.patch
error: patch failed: wine-tkg-git/customization.cfg:94
error: wine-tkg-git/customization.cfg: patch does not apply

it worked about a month or two ago.

I will look up this issue on the weekend, in the meantime, alternatives are to either use old wine-tkg-git commits or use the prebuilt


feat: Manage runnable dependencies by HTTP server and improve DB health check

Context
Our current DB health check does not take domain rules into consideration that can infer the health of cardano-db-sync :
const HEALTH_CHECK_QUERY = 'SELECT 1';
Also currently each provider handles and manages the lifecycle methods of its dependencies, in most cases we use one and the same instance of that dependency (memoized), ex. OgmiosCardanoNode dependency now becomes a dependency of all DbSync providers, the same as DB.
Ideally, we would have one single place (HttpServer) to instantiate all runnable dependencies and manage them through lifecycle methods.
Proposed Solution
Prior Refactoring (ADP-2296):

move RunnableModule class out of cardano-services package to commonplace (a util package is a good option) and reuse it by multiple packages where is needed
refactor OgmiosCardanoNode to extend RunnableModule as a runnable dependency of HttpServer
extend HttpServerDependencies with runnableDependencies list to enable HttpServer to manage all runnable common dependencies across services in one single place through lifecycle methods (initialize, start , shutdown etc.)

Health check improvement (ADP-2186):

Add CardanoNode as a dependency in DbSyncProvider
Consider the impact of all providers now being dependent on CardanoNode (Ogmios)
Compose the health check result based on the following:

query db-sync for the latest block height (number)
fetch lastKnownTip via stateQueryClient and extract the block no from the node
compare the results with a tolerance of 5 blocks behind (db-sync could be out of sync slightly)

Commits are squashed, CI build passed, and ready for merge once passed again.


custom metaObjectReplacement object

Thanks for this nice transformer.
I'm wondering if you could share a simple test case where the metaObjectReplacement object is defined in a test, not in the config file.

Thanks @Chris55 !
Sadly, as ts-jest' (jest) is setup and bootstrapped before executing any tests, the provided configuration (for our case, specifically the optionsproperty of the astTransformer) cannot be dynamic and computed on a "per test" basis. So, it is currently impossible to define themetaObjectReplacement` value dynamically from within the tests themselves. It is also indeed not possible to mock any imports that are used within the jest configuration module, which could have been kind of cool...
With that being said, I'm currently preparing a new minor version of the package, that will allow the metaObjectReplacement property (or any of his sub properties) to be an object factory function, to offer a little more flexibility and more dynamism for you to define more complexe mocked values. Examples of that will be available in the README. as well as implemented in the project jest.config.ts
Hope that helps, don't be shy to give a little star to the repo 😃


Failed to upgrade dbeaver from v.7.2.5 to v.21.1.4

System information:

Operating system (distribution) and version: Fedora linux 27
DBeaver version: current: Community Edition 7.2.5, update to ==>  21.1.4

Describe the problem you're observing:
When attempt to update from v. 7.2.5 to v. 21.1.4
sudo dpkg -i --force-overwrite dbeaver-ce_21.1.4_amd64.deb
I'm having this error:
Preparing to unpack dbeaver-ce_21.1.4_amd64.deb ...
dpkg (subprocess): cannot set security execution context for maintainer script: Invalid argument
dpkg: error processing archive dbeaver-ce_21.1.4_amd64.deb (--install):
 subprocess new pre-installation script returned error exit status 2
Errors were encountered while processing:
 dbeaver-ce_21.1.4_amd64.deb

This may be related to this issue 47

@SupervisionT This can be related to the fact that 7.2.5 is pretty old and has different dependencies on top of other differences. Have you tried uninstalling current version and then clean installing the new one?

there is no update on the issue for a long time.
If it is still actual for you - feel free to reopen an issue.


Implement "Add JDBC Driver" for Middleware

@miq-bot add_label providers/hawkular, ui

cc @theute

@mtho11 Can you please check
@miq-bot assign @mtho11

Ah crap Mike is afk the whole week
@miq-bot assign @Jiri-Kremser
Jirka, please check.

@miq-bot assign chessbyte

@martinpovolny please review and recommend some tests if you can, as I see none in this PR.  Thx, Dan

Thanks for the review @himdel. Fixed accordingly.

@ammendonca thanks :)
As for specs, nothing really jumps up to me, but maybe at least something like trigger the mwAddJdbcDriverEvent event and see if a POST request with the right data was made, and maybe check that a flash message appears when that request fails. So that we at least test the main functionality and basic failure handling.

@miq-bot assign @chessbyte
As per discussion, @ammendonca will supply the specs in a followup PR.

Allright, merging, @ammendonca please ping me in the followup PR :)
Is this supposed to go to euwe? Tentatively putting yes..

Euwe Backport details:
$ git log
commit 795caae3127994189e25bb88ec7654d5f9c02986
Author: Martin Hradil<EMAIL_ADDRESS>Date:   Mon Oct 3 14:28:37 2016 +0200

    Merge pull request #11218 from ammendonca/MIQ-MW-ADD-JDBC-DRIVER

    Implement "Add JDBC Driver" for Middleware
    (cherry picked from commit 0a7e365583ec56f614d6392dd2fe696fda7a3906)


feat: add rename and no_sum_check flags for kcl mod command

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

feat: add rename and no_sum_check flags for kcl mod command

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
128

Relevant Lines:
335

💛  - Coveralls


How to debug StackOverflowException

I'm getting this error while moving a site from ASP.NET Core 1.1 on Mono to ASP.NET Core 2.0 on .NET Core 2.0:
dbug: Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker[2]
      Executed action method Daniel15.Web.Controllers.ShortUrlController.Index (Daniel15.Web), returned result Microsoft.AspNetCore.Mvc.ContentResult.
Process is terminating due to StackOverflowException.
[1]    12976 abort      LD_LIBRARY_PATH=/tmp/ssltest ASPNETCORE_ENVIRONMENT=Development =

How do I get a full stack trace for the StackOverflowException to determine where it's coming from?

Issue moved to dotnet/coreclr #14720 via ZenHub


The Alpine instructions to install dotnet-sdk are incomplete

Reading the documentation, and following it, it just lists a bunch of dependencies and no actual install like it does on Ubuntu.

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

ID: b5e950a9-28c4-91f3-919b-d12f4e029fca
Version Independent ID: f066a748-0a7d-7506-3b18-c422fef13293
Content: Install .NET on Alpine - .NET
Content Source: docs/core/install/linux-alpine.md
Product: dotnet
Technology: dotnet-install
GitHub Login: @adegeo
Microsoft Alias: adegeo

Hi @FilBot3
The Install section calls this out. While Alpine is an officially supported distro by .NET, there is no package manager integration like other distros.

I'm going to close this issue as I don't think there is anything to update in the docs. Please feel free to respond if there is something else I can do to the docs. Cheers!


Better error handling of push behavior in Authroing api.

Authoring api currently push commits and check afterwards if the head commit is the same in local and remote repo, in not, it reset the local repo to remote head and throws an exception.
We should improve the behavior to support multiple authoring instances by comparing:
local head (before write), local head (after write), remote head (after push).
if local head after write is different then remote head after push, and local head before write is different then remote head after push, we should reset the local repo and reattempt the write+push.

We should consider using 'simple-git' for push as we already using it for revision history and it might not hide validation errors.

The approach we took to solve this issue was to use SimpleGit (git cli wrapper), it seems that in the latest version of nodegit it might work (https://github.com/nodegit/nodegit/pull/1386)
https://github.com/nodegit/nodegit/blob/6ff67a883c66a1cee23feb53b5f82c36c8a4df04/CHANGELOG.md#v0203-2017-10-18


operator-image is missing certificates

(ISO-)image download by elemental-operator might fail with self-signed certificates:
level=error msg="Failed to dial steve aggregation server: tls: failed to verify certificate: x509: certificate signed by unknown authority"

Rebuilding the operator-image with the ca-certificate packages fixes this error

Did a successful end-to-end test with the customer for registration, cluster creation, and upgrade :rocket:


Download link

In readme, I remember a download link, where did it go？

I'm not posting download links anymore - you can get the library from several CDNs such as unpkg: https://unpkg.com/<EMAIL_ADDRESS>


limiting selection for 3 images and 1 video

is there a way to limit the selection depending on asset type ? for ex. I need to limit user selection to be 3 images and 1 video at maximum.

@zhangao0086 do you have any solution for this problem? Thanks in advance!

Hi @murad1981 @SagarUnagar
There is an alternative way:

Make a subclass of DKAssetGroupDetailVC then override the collectionView:shouldSelectItemAt: function.
Make a subclass of DKImagePickerController then return the custom subclass of DKAssetGroupDetailVC in the makeRootVC function.

"Overriding non-open instance method outside of its defining module "
its giving an error while making subclass from  DKAssetGroupDetailVC


Fix #228

This is an updated version of https://github.com/letmaik/rawpy/pull/76 with a doc string and a test. Unfortunately when I run this version I'm currently getting a segfault on the test and I can't really debug it, help appreciated!

@eroux I believe that the segfault is happening because you're creating a Python callback and then attempting to run it inside a nogil context.  As a result, a Python interpreter is not available for running the callback properly.
I solved the issue using an alternate method in #229; your test data was particularly useful in verifying that the method works.  Thank you!

wonderful, thanks!

@eroux - #236 is merged to main so corrupt files are now handled.
@letmaik - #236 took care of this PR's intent; I believe we can close this one if that's ok with @eroux.

Thanks for that! Yes let's close


[Wait for #1963] [multi head attention] added unittest

 - Added layer/model unittest for multi head attention

Signed-off-by: hyeonseok lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1964. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @lhs8928, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1964-202208301709450.83279991149902-5a7b92775ebd779e67688b9fdc1275784ee1c6f5/.

:octocat: cibot: @lhs8928, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1964-202209011649180.37418103218079-39ef970b39062695a8508b07a1b6a81ccad060d1/.

:octocat: cibot: @lhs8928, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1964-202209021335130.15086698532104-500ab4f52784a7d1f86ceab4e9188d176bf3a39a/.


undefined is not an object ('evaluating 'getPrototypeOf(sourceCompontent)')

Why I'm seeing this?
when I bulid the IOS app (app.js) it gives me this error undefined is not an object ('evaluating 'getPrototypeOf(sourceCompontent)')

Other information:
**OS, device, application version **
OS: macOS, iPhone 7 simulator

It seems that you are not giving a real source target to Hoist Non React function, also not sure if that component is supported for react-native. Anyway, just check if the target source is imported in your js file.

@axd i am also getting this error, how do i solve?

@jafar690
In most cases it means you have a problem which can be solved with @axd 's solution

just check if the source component is imported in your js file.


The cluster manager cores when tiup patch tiflash

close as we have removed the "cluster manager" in TiFlash and made it a part of TiDB since 6.0.
And for the older versions, failing to start cluster manager when tiup patch is moderate since we will try to start it again by timer.


anyway to fetch the onform image when filling the form?

I don't think it's possible with the current implementation.
Image objects in form JSON have some kind of ID (see this comment for more details), but I couldn't find a way to generate direct image links from these IDs.

Actually, I am doing something like google Forms has multi pages. The images are not in the first page. I have to complete the first few pages of the form by the bot and after that, the page will show up as an image. I would like to retrieve that shown image and OCR for that. Is it any alternative to get rig of this or any suggestion on that?

I don't really understand what you want to do with the image.
Why OCR? Do you need to copy some text from the image into a form input field? Or just extract images from all apges of the form?

I already have solutions on the remaining part, now remaining part is how to exact the image from multi section Google form.

Any possibility method that work like when image on the page either save to local files or have an url ?

Well, I guess the easiest way would be to just parse each page's HTML and extract links from img tags
Here, last_response is the page which comes after the submitted page.
E.g. if you submitted the first page, last_response.text will contain HTML for the second page of the form.

Solved in https://github.com/vvd170501/python-gforms/issues/9#issuecomment-1259320652, closing


Style review

Please describe the changes in your Pull Request using the description, ideally with a screenshot.


Beakerx not compatible with tensorflow

getting the following error when I run:
conda install beakerx
LinkError: post-link script failed for package conda-forge::beakerx-0.11.1-py36_1
running your command again with `-v` will provide additional information
location of failed script: /opt/conda/bin/.beakerx-post-link.sh
==> script messages <==
+ /opt/conda/bin/beakerx-install
Traceback (most recent call last):
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2920, in _dep_map
    return self.__dep_map
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2731, in __getattr__
    raise AttributeError(attr)
AttributeError: _DistInfoDistribution__dep_map

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2911, in _parsed_pkg_info
    return self._pkg_info
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2731, in __getattr__
    raise AttributeError(attr)
AttributeError: _pkg_info

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/conda/bin/beakerx-install", line 6, in <module>
    from pkg_resources import load_entry_point
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 3142, in <module>
    @_call_aside
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 3126, in _call_aside
    f(*args, **kwargs)
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 3155, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 664, in _build_master
    ws.require(__requires__)
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 981, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 875, in resolve
    new_requirements = dist.requires(req.extras)[::-1]
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2675, in requires
    dm = self._dep_map
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2922, in _dep_map
    self.__dep_map = self._compute_dependencies()
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2931, in _compute_dependencies
    for req in self._parsed_pkg_info.get_all('Requires-Dist') or []:
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2913, in _parsed_pkg_info
    metadata = self.get_metadata(self.PKG_INFO)
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 1486, in get_metadata
    value = self._get(self._fn(self.egg_info, name))
  File "/opt/conda/lib/python3.6/site-packages/pkg_resources/__init__.py", line 1595, in _get
    with open(path, 'rb') as stream:
FileNotFoundError: [Errno 2] No such file or directory: '/opt/conda/lib/python3.6/site-packages/bleach-2.0.0.dist-info/METADATA'

shall we close this?


[bazel] Set hello_world example visibility to public

I was playing with 'bazel cquery //...' and was unable to run any queries because of this error:
ERROR: $my_repo_path/release/BUILD:10:8: in pkg_tar_impl rule
//release:opentitan: target '//sw/device/examples/hello_world:package'
is not visible from target '//release:opentitan'. Check the visibility
declaration of the former target if you think the dependency is
legitimate
Signed-off-by: Dan McArdle<EMAIL_ADDRESS>

@drewmacrae ping :)

Why is that binary in the release in the first place?

No idea! Looks like //release also pulls in the test ROM. https://cs.opensource.google/opentitan/opentitan/+/master:release/BUILD
@cfrantz do you have any insight?

Why is that binary in the release in the first place?

No idea! Looks like //release also pulls in the test ROM. https://cs.opensource.google/opentitan/opentitan/+/master:release/BUILD
@cfrantz do you have any insight?

Its in the release because its an example program.  The the missing //visibility:public is an error and should be fixed.


Add network event stream

Close: #5

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

Coverage data is based on head (3e4398c) compared to base (3822b68).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@          Coverage Diff          @@
##            main      #8   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          5       5
  Lines        189     223   +34
=====================================
- Misses       189     223   +34

Impacted Files
Coverage Δ

src/subscribe.rs
0.00% <0.00%> (ø)

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.


⚠ Bofrost.es has degraded performance

In a9cdef3, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1530 ms

Resolved: Bofrost.es performance has improved in a8115c9 after 9 minutes.


is there any .net core console client example

hi example is very good for javascript
but i need an example from javascript html to  dotnet core console application
since,
`
const ISub = connection.stream("WatchStream", streamName).subscribe({
next: (item) => {
feed.innerHTML = item;
},
complete: () => {
feed.innerHTML = "Stream completed";
console.log("Stream is finished.");
},
error: (err) => {
feed.innerHTML = "Stream closed with an error";
console.log("Failed to watch the stream: " + err);
},
});
`
i couldn't find a way to implement
 connection.stream("WatchStream", streamName).subscribe
on dotnet core client

https://learn.microsoft.com/aspnet/core/signalr/streaming?view=aspnetcore-7.0#server-to-client-streaming-1

yeah i know it, but couldn't imagine how to implement event based .subscribe thing..
can you give a small example?

You'd have to write your own event based method.
Super rough example:
var stream = await connection.StreamAsync<int>(...);
var subscribeMethod = (int item) =>
{
};
var completeMethod = () =>
{
};
var errorMethod = (Exception ex) =>
{
};
_ = Task.Run(async () =>
{
    try
    {
        await foreach (var item in stream)
        {
            subscribeMethod(item);
        }
        completeMethod();
    }
    catch (Exception ex)
    {
        errorMethod(ex);
    }
});

ok i got the idea..
the only thing that i did not understand before, await foreach (var item in stream) in this part, when there is no stream item, what will happen if the remote part not closed it..
now i got the idea.. thanks for the example by the way..


There are problems even though I used 100 epochs models

Hi.
I use Jetson Nano.
I tried these examples,

Re-training on the Cat/Dog Dataset(Classification, Recognition)
Re-training SSD-Mobilenet(Object Detection)

But, even though I used 100 epochs models, the problems are occurred.
In case of classification, the following problem is occurred.

Not only percentage is too low, but also Jetson recognizes dog as cat.
In case of object detection, some fruits are detected, but the others are not detected. But the fruits that are detected don't always detect accurately, for example, some banana pictures are detected, but others are not detected.
Thank you for reading this issue.

Hi @SangkyunAhn, can you try deleting the *.engine files from your model's directories and try running imagenet/detectnet again?
Also, you can find my trained models for cat/dog and the fruits dataset on the tutorial pages.  Can you try those to see if there is a difference?

I solved these problems. Thank you for your help.


Updating to v1.6.2 makes all previous jobs enqueued

Description
Hi. We recently migrated from Agenda to pulse. Projects that migrated to pulse v1.6.1 seem to work fine without issues, but installing v1.6.2 seems to make nextRunAt of all jobs to be set to be now(), which makes the Pulse to lock and restart all our previously completed jobs to start all over again.
Setting resumeOnRestart: true would prevent this though.
Code example
No response
Additional context
No response

@litsynp We apologize. We have just become aware of this issue and are working on resolving it. We will address it promptly.

@litsynp Could you provide us with a sample of the data that should not be executed?

Thank you for the quick response. Here's a sample data.
{
  "_id": {
    "$oid": "66bf05cad0633e28dcd2fb4a"
  },
  "name": "SOME_TASK_NAME",
  "data": { "items": [1, 2, 3] },
  "priority": 0,
  "shouldSaveResult": false,
  "type": "normal",
  "nextRunAt": null,
  "lastModifiedBy": null,
  "lockedAt": null,
  "lastRunAt": {
    "$date": "2023-05-23T02:40:17.839Z"
  },
  "result": { "items": [2, 3, 4] },
  "lastFinishedAt": {
    "$date": "2023-05-23T03:16:33.810Z"
  }
}

The dates lastFinishedAt and lastRunAt should mean the job has already been completed, but Pulse would set like nextRunAt: { "$date": "2024-08-16T03:04:55.969Z" }, making the job to restart.

@litsynp A version that we believe has solved this issue has been deployed (v 1.6.3).
Please check it out.
npm package (@latest dist-tag)
GitHub release

@code-xhyun After the 1.6.3 update, I checked that it's going well without any issues. Thank you for your quick process.


Suggestion: Alarm for end of fish window as well as beginning

When you get focused on the fishing you don't always notice when your window for catching it is over. Would it be possible to get either another option for an end alert or just add it to the regular fish alarm?
Thank you for your time and attention.

Doesn't the fishing timer make that pretty obvious?

I suppose, but I tend to get locked on to watching the actual fish and lose track of time, but I suppose that's just a me issue. Still, just a thought, thank you for your response, and all your hard work on this great tool.


Preparations for python3.12

Updated dependencies, neeeded fixes and changes in logic.

@dnadein can you please briefly (but as detailed as you can afford)) explain the changes in redis-cache-lock? Looks like there are some logical changes, but I can't quite see the whole concept

Everything looks good, but something has to be done with this formatting

Mypy acts differently on local machine than in CI. Hopefully will be fixed now.

Test fails are not related to the changes in this PR


Immediately complete remote tasks when not leader

@hakonhall please review
Avoids edge case where set-node-state requests sent to followers would
have their response delayed indefinitely due to controller not
publishing any versions that the task's ACK barrier could be released by.

Yes, this bug impacts orchestrated upgrades in multi-controller clusters so should be part of next release.

@aressem I think we should block 6.147.14 from rolling out in the morning. We need at least 6.147.22

On the pools: I was wondering whether CC handles the filling of the pools in the best possible way. For instance, I wonder if this leads to connection refused or some sorts, while it may be better to return a proper HTTP response with description saying this and that queue is full.


Can't update

Run command:
git pull origin release && composer install --no-dev && php artisan migrate
Result:
fatal: unable to access 'https://github.com/BookStackApp/BookStack.git/': Failed to connect to github.com port 443: Время ожидания соединения истекло
What happened? How do I upgrade?

Hi @doox911,
Looks like the system you're running this on can't connect to GitHub over HTTPS. Does the system have working outbound internet access?

Привет @ doox911 ,
похоже, что система, на которой вы работаете, не может подключиться к GitHub через HTTPS. Есть ли у системы работающий выход в интернет?

Now I have error:
Could not delete /var/www/wiki/vendor/wnx/laravel-stats/composer.json:

That'll be a permissions error. The user you're running composer install --no-dev as likely doesn't have permission to manage these files.
Depending on your system, you could try running these with elevated privileges:

git pull origin release && sudo composer install --no-dev && php artisan migrate

Thank you)


Which error is thrown for Rate Limits?

I know the API itself throws a 403 and 429 response codes but does this gem catch the response and yield an actual error or should I look for the response codes?

We yield an error that contains the response code and a description of the response
https://github.com/Bandwidth/ruby-bandwidth/blob/master/lib/bandwidth/errors.rb

So the GenericError?
Do you have an example response?

  begin
    messages = Bandwidth::Message.list(client, {:fromDateTime => start_date, :toDateTime => end_date, :size => 1000})
  rescue Bandwidth::Errors::GenericError => e
    puts e

yeah, I saw that...
but what all is e made up of?
is e.code == 429 || e.code == 403 an acceptable manner of catching the rate_limit?

Basically is e.code going to have a value like 403 or message-rate-limit

Actually yeah that's right. The "code" field is set in this manner
    # Check response object and raise error if status code >= 400
    # @param response response object
    def check_response(response)
      if response.status >= 400
        parsed_body = JSON.parse(response.body)
        raise Errors::GenericError.new(parsed_body['code'], parsed_body['message'])
      end
    end

My apologies. I updated my previous comment


Moz2D does not work on Windows when MSVC is not installed

Moz2D installer should also download msvcp140.dll and vcruntime140.dll

Great!


Design improvement

Need to centre and improve design of  contact us page , assign me this issue and label it under GSSOC @singodiyashubham87
Screenshots

@SuryanshuKushwaha This is not Draw-it-out application's screenshot, it's of another project, be conscious while creating issues.


RichText Support RelativeSize TextSpan

Use case

Proposal

Please provide additional details in the body of the issue as indicated by the template. I am going to close this.

我需要Flutter SelectionText支持类似Android的RelativeSizeSpan、AlignmentSpan、LeadingMarginSpan、SuperscriptSpan、SubscriptSpan等丰富的Span来更好的解析Html
--- translate by google ---
I need Flutter SelectionText to support Android-like RelativeSizeSpan, AlignmentSpan, LeadingMarginSpan, SuperscriptSpan, SubscriptSpan and other rich Span to convert Html.


Publish 1.0.1 package in nuget?

please

Yes, this is currently a serious problem.

I don't believe 1.0.1 will ever be released as there was a problem with it.
What did you need 1.0.1 for?
I am awaiting netcore/xplat-msbuild support as currently cant use the analyser in vscode.

@mikes-gh well, there were some fixes that I wanted and I'd like a fresh build to be available from official store

I think the big blocker is netstandard support which doesn't appear to be straight forward.  Any news @sharwell

@xumix If you were waiting for fixes I don't believe there was anything in 1.0.1 that wasn't in 1.0.0
1.0.1 was an attempt at solving a problem with JSON.net on netstandard.

@mikes-gh oops, i'm sorry, I've waited for 1.1 fixes

@xumix I'm going to treat this like the title says currently. If you meant a different number, feel free to open a different issue.
The 1.0.1 release was intended to fix some compatibility issues with .NET Core users. In practice, it did not behave as we intended so I unlisted the package in NuGet. I'm still working with others on the Roslyn team to come up with a good story for analyzers in cross-platform development scenarios. I don't have news yet but hopefully soon; it's causing a big pain point and not just for user of this project.


ACC sell price currently independent from ACC chips quantity on the board

While the buying price of ACC chips raise for each new one added to the board, their resell price never rise.
This can lead to signifant investment losses when selling an ACC chips after so many others were installed, regardless how the level at which you upgraded Tramiel (and not Tamriel, as i keep writing it).
I'm not sure this is intended behavior, hence this issue creation, and-not sure either if changing the curent

No, that is a bug. Thanks for mentioning it.

and not Tamriel, as i keep writing it

that sounds Elvish, somehow ;)
Issue fixed in 3.32.


Reduce deprecated API usages

Hi,
some configuration-names have been deprecated, e.g. compile. an interim solution could be the support of the old and new name?
further could the use of custom strings like "testCompile" replaced by existing gradle API constants like JavaPlugin.TEST_COMPILE_CONFIGURATION_NAME?
best regards
Klaus

With Gradle 10, this warning deprecation warning appears as well:
The AbstractFileCollection.getBuildDependencies() method has been deprecated. This is scheduled to be removed in Gradle 5.0. org.jetbrains.kotlin.gradle.frontend.npm.NpmPackageManager$injectDependencies$1$1 extends AbstractFileCollection. Do not extend AbstractFileCollection. Use Project.files() instead.


Update tooling + fix inspections

Blocking for now pending inspection fixes.
It's come to my attention that dotnet-format hasn't been working since we switched to .NET 5. It's quite a useful tool because inspections happen very quickly (unlike inspectcode). In the process, this also requires using imports to be sorted, so long as dotnet_sort_system_directives_first = true is turned on.
It seems like Rider will sort usings when added, or via the sledgehammer option in the menu (Code -> Optimize Imports), but if this is undesired then we can just remove the .editorconfig option.

Updated dotnet-format and jetbrains.resharper.globaltools except cake (we're far behind but it's about to be deprecated anyway).
Sorted using directives to appease dotnet-format.
Fixed inspectcode not running on .NET 5 TFM.
Fixed nullref inspections that have been missed.
Changed some new inspections to hints - I think for now we still prefer using .Slice(start) instead of [start..], and no complex pattern matching.

I suggest going through the PR per-commit.

Holding back until the migration to gh-actions.


fix: declare strong type for null apikey and apisecret

Force strong type via constructor and attributes declaration. Avoid inconsitences with null values allowed in contructor

Thanks for the PR @shakaran. With this said, I might actually restructure this class to BaseServiceClass to make it easier and have less code duplication.

@p-delorme as you want it. We have other possible PR to suggest to improve unit test and code. Also using phpstan, rector, php-cs-fixer and others in default pipeline

I've updated all these types in the constructors.
I'll close this, feel free to reopen if you wish.


many sections and calculator have moved to bottom and left corner

Describe the bug

Some of the sections and calculator have also moved to the left side and at the bottom. One needs to scroll a lot to find them.

https://user-images.githubusercontent.com/61223163/115992342-d5a19980-a5ea-11eb-8764-f2609bb23123.mp4
Possible solution

I would like to solve this issue

Screenshots

If applicable, add screenshots to help explain your problem.

@gnikita432 This Issue is already made by me please check Issue Number #1367

Ohh yes , sorry for opening a duplicate issue, closing this issue.


Headset button press still opens iTunes.

I have a fairly standard headset, single button for pause and play, it only works with iTunes.
Don't know if this is a limitation of the OS or the App.

So this is possible with something like: https://github.com/jguice/mac-bt-headset-fix/tree/sierra
Would it be possible and would you consider bundling a solution like this?


Updating branch from orientation/orientation master

PT: https://www.pivotaltracker.com/story/show/126613995
Updating from https://github.com/orientation/orientation. Major new features:

Search is handled by pg_search
Rails is at version 5.0

@brupm, @benmanns, @AhmedBelal, @DanielNill and @maletor: please review.

Please have @ThomsonT give it some QA love before deploying to prod.
Ensure emails work, either in QA or optionally right after you deploy to prod.  👍
Awesome work!

No need to wait for additional CRs, if QA is good with it, go for ir.

Ready for deployment.

@SowjanyaEga deployed to production.


Unable to create ContentItemCreateModel

Hi,
I am currently working on the evaluation copy (waiting on enterprise licence) and am looking to migrate content from one KenticCloud instance to another instance (development -> Test). I found your SDK and thought this would be perfect.
I've looked at the examples and have tried to create a ContentItemCreateModel type but am getting an error (the Content Management API also gives the same error)
`
ContentManagementClient toClient = new ContentManagementClient(toOptions);
        // Defines the content item to import
        ContentItemCreateModel item = new ContentItemCreateModel()
        {
            Name = "Contact Page",
            Type = ContentTypeIdentifier.ByCodename("contact_page")
            // SitemapLocations = new[] { SitemapNodeIdentifier.ByCodename("") }
        };

        ContentItemModel response = await toClient.CreateContentItemAsync(item);

`
error as follows:
 An unhandled exception of type 'System.AggregateException' occurred in System.Private.CoreLib.dll: 'One or more errors occurred.'  Inner exceptions found, see $exception in variables window for more details.  Innermost exception 	 KenticoCloud.ContentManagement.Exceptions.ContentManagementException : The specified content type does not exist.    at KenticoCloud.ContentManagement.Modules.HttpClient.ContentManagementHttpClient.<SendAsync>d__5.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.TaskAwaiter1.GetResult()
at KenticoCloud.ContentManagement.Modules.ActionInvoker.ActionInvoker.d__62.MoveNext()    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.TaskAwaiter1.GetResult()
at KenticoCloud.ContentManagement.ContentManagementClient.d__15.MoveNext()
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter1.GetResult()    at myApp.Program.<CreateContent>d__1.MoveNext() in C:\__workspace\migration\myApp\Program.cs:line 49
My question is, is it possible to create the type objects from the sdk and if so, am I doing this correctly?
thanks in advance

Hi there!
Currently, the CM API does NOT support operations for content types. And it seems that there is no content type with codename contact_page in your project.
You need to create the content type manually in the UI. Or you could wait for a few weeks because the support for content models in being developed right now (taxonomy at this very moment).

Hi,
thanks for the quick response. I guess we'll have to manually create the content types for now but hopefully we can use the SDK if we need to in the future.


Remove usage of unstable AtomicU64 / integer_atomics

Use AtomicUsize instead. Fix #1. It behaves the same, since macOS is always 64-bit these days.
Increment the version number to 0.1.0 rather than 0.0.2 so that it is possible next time to do version number changes that Cargo does not consider breaking.

That feature stabilized so you can just remove the feature guard now. I needed a few other tweaks to make the crate build with stable Rust (1.37) however: https://github.com/akosthekiss/blurmac/pull/5


web: remove redundant style helpers

Hello @hyu,
Please review the following commits I made in branch nicks/color:
e2068eeca57901771458e8f1dfa16158818b7212 (2020-02-25 13:57:14 -0500)
web: remove redundant style helpers
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

:grimacing: fortunately, we were all working off the same color scale


Add test using tap framework

Due to request from @aqrln in #79 I open a PR for reviewing tests and choosing test framework.

@aqrln & @DzyubSpirit let's land tap ?

@nechaido Is it okay that tests fail? Is that problem because of functionality or tests?

@DzyubSpirit you can see it in Travis here: https://travis-ci.org/metarhia/MetaSync/builds/226031887
For example: https://travis-ci.org/metarhia/MetaSync/jobs/226031888

@DzyubSpirit it fails because find() function was broken at the moment of writing this test, it won't after merging new commits from master

@nechaido let's rebase and see, then :)

@aqrln already

@tshemsedinov ping

Landed in 72ed1dcbf903d0307419167e54f87d6697bba384, ec66cd6fd37f152bce8d03226dcd56d06ac150e5 and 962e76ada1c09d66271490caef46c5208ee91811


redis withHook  Why not open it up

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

It exposes go-redis, while we don't want to.
Because if we expose go-redis, when we upgrade from go-redis v8 to v9, go-zero users need to change their code.


Dev server should generate static version for url on the fly

Currently we need to build a static version + run the dev server. This is counter productive. The express server should be able to generate the page on the fly directly.

@thangngoc89 just to let you know that I am on this, and this will clean a LOT OF THINGS :)

Ok.  Good to know . I means time. I'll just blogging then

ETA: Almost ready :)

Can't wait ....

OMG it works. I finally fixed the last issue I got (bad usage of babel-plugin-webpack-loaders).
I still have to cleanup things now and will push my work as a PR so you can review it (warning: huge pr) ^^

Finally :)

Awesome. About that config, webpack 2 have a better syntax to define loader with query. no more ugly JSON.stringify

Yeah I see that.

Why don't use NODE_ENV for easier babel config ?
There is

cross-env for windows ENV
better-npm-run for a cleaner run script in package.json

Because NODE_ENV=static is used for babel-plugin-webpack-loaders and the client webpack build is run in the same process. babel-plugin-webpack-loaders does not make sense for client side code so I must disable it for client side script.

Ah. That makes sense.


state management issue in torrent_list_page

The root widget of TorrentsListPage is Consumer<GeneralFeatures>. Not mentioning all the functions inside GeneralFeatures, the two specific functions updateDiskSpace() and setListOfLabels() also notify change in the Consumer<GeneralFeatures> as usual. One of these two functions is getting called over a stream getAllAccountsTorrentList() in ApiRequests. Both the functions mentioned earlier are not utilised in the TorrentsListPage but because of these the Consumer<GeneralFeatures> is providing change very frequently in the tree which is causing widget rebuild a lot more number of times than it should actually build. The actual use of these two functions is in the Drawer. All this occurred when the torrent list was not empty.
@harchani-ritik I would like to work on this. A simple solution I thought was to create a separate class of ChangeNotifier from GeneralFeatures so that functions used inside other widgets only consume providers needed. Also mention any suggestion from your side.

A simple solution I thought was to create a separate class of ChangeNotifier from GeneralFeatures so that functions used inside other widgets only consume providers needed
This approach seems good, @rj-since-2000 . Please go ahead.

I'm closing this issue. This issue will be resolved in the new architecture in dev branch.


Demonstrate brotli in backstage example app

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Closing as stale.


Initial commit for tag subscription

Need to pull backend branch as well: https://github.com/aml-development/ozp-backend/pull/309

This doesn't handle the case where a user already has a URL from a previously shared tag that does not have the entity id in the URL.
The link still loads the results for the tags, but the subscribe button shows and does not work.

Removed the subscribe/unsubscribe link if the tag id isn't present. URL from a previously shared tag will not show the subscribe/unsubscribe

Subscribe/Unsubscribe link works.
Link does not show when the URL does not contain the ID for backwards compatibility with previously shared tag links.
:+1:

when going direct to link for a subscribed tag (eg) http://localhost:8000/dist/#/home/////tag_0/3 the text originally says (subscribe) and then switches to (unsubscribe) once data loads. Perhaps it's better not to show anything until the subscription data loads? I don't have a strong feeling one way or another though so approved if Brian doesn't think it matters :)
:+1:

Good point, @lsemesky.
@mleeBoeing, if you can stop the text from showing prior to the data loading, with reasonably little effort, I would prefer that.
The good news is that it doesn't appear you can double subscribe.

Do not merge. Found a bug while submitting a new listing.

For the subscription data being hidden until the data loads, it's going to take some effort. Will need to make the ajax call in the webapi to be synchronous. However, the async property is being deprecated on Firefox and Chrome.

I'm leaning towards you being correct that this is a bit of effort.  I haven't found an element (within props or state) that can be looked at for subscriptions having finished loading.  Looking at if element.stop exists works when there is at least one subscription  (element.stop becomes undefined), but if there are no subscriptions it never becomes undefined.

Had a bug where create/edit listing page would show [object, object] for the Tag input. Therefore, not allowing you to save or submit. Pushed fix in. Requesting review again.

see any issues with the latest @blynch11p ?

@lsemesky, nope!!!  Good stuff.


ClassNotFoundException JAXBContext in camel-core-impl

Seeing the following with OpenJDK 11 based GraalVM:
Fatal error:com.oracle.graal.pointsto.util.AnalysisError$ParsingError: Error encountered while parsing org.apache.camel.quarkus.core.FastCamelContext.createModelJAXBContextFactory()
Parsing context:
	parsing org.apache.camel.impl.engine.AbstractCamelContext.getModelJAXBContextFactory(AbstractCamelContext.java:3716)
	parsing com.oracle.svm.reflect.AbstractCamelContext_getModelJAXBContextFactory_477828eef27cc93cb9372a4eb40fd739c75455ee_1093.invoke(Unknown Source)
	parsing java.base@11.0.8-internal/java.lang.reflect.Method.invoke(Method.java:566)
...
Caused by: java.lang.NoClassDefFoundError: javax/xml/bind/JAXBContext
	at java.base/java.lang.Class.getDeclaredMethods0(Native Method)
	at java.base/java.lang.Class.privateGetDeclaredMethods(Class.java:3166)
	at java.base/java.lang.Class.getDeclaredMethods(Class.java:2309)
	at jdk.internal.vm.ci/jdk.vm.ci.hotspot.HotSpotResolvedObjectTypeImpl.getDeclaredMethods(HotSpotResolvedObjectTypeImpl.java:932)

Longer output

This should be solvable adding jaxb dependency.
Note what I said in a similar situation here. Adding such dependency has no influence in the final image, it's just to make the pointsto analysis JVMCI layer used in stock OpenJDK 11 work.

@davsclaus do you think can hide this even better ?
The culprit is https://github.com/apache/camel-quarkus/blob/master/extensions-core/core/runtime/src/main/java/org/apache/camel/quarkus/core/DisabledModelJAXBContextFactory.java#L19

@lburgazzoli yes already done
https://github.com/apache/camel/blob/master/core/camel-api/src/main/java/org/apache/camel/spi/ModelJAXBContextFactory.java
https://issues.apache.org/jira/browse/CAMEL-14829

Oh nice, great timing @davsclaus. Would a 3.3.0-SNAPSHOT camel version work with the current camel-quarkus?

We have branch  but it does not target quarkus snapshots, guess we can add a workaround for now and reference this issue to remove it when migrating to camel 3.3.x

Sounds good. I'll try to fix it with a workaround and see how it goes...

The workaround (https://github.com/galderz/camel-quarkus/commit/3da275e7560d080eab7b229869b5518198e20d48) worked. I'm letting the testsuite continue and see what else comes up.

To fix this, it should be enough to remove the workaround in https://github.com/apache/camel-quarkus/pull/1185/files#diff-93052fe9a22c86587a36da3725c66eb4R52-R55 and https://github.com/apache/camel-quarkus/pull/1185/files#diff-8fa2182cc03b4d845ad00fa7138e1d3bR59-R62 after upgrading to Camel 3.3.0

@galderz FYI - this change seems to increase the time taken to complete the integration test native builds by around 50%. Our CI build takes around twice as long to complete now. I've not dug into the specifics around why there's such a dramatic change.
Since the Camel 3.3.0 release is imminent, we can probably live with it for now, or at least work around it.

@jamesnetherton How imminent is the Camel 3.3.0 release?

@galderz It's available now and we already upgraded the master branch.

I'll give master a go and see how it goes.


Fix e2e test hanging on local runs

Description
When running cargo nextest run --test-threads 1 --package sui-graphql-rpc --test e2e_tests --test examples_validation_tests --features pg_integration locally, these tests will hang on the first one. Explicitly sending a cancellation signal and waiting for the indexer and graphql handles to spin down seems to solve the issue.
Would be nice to run these in parallel too, but punting them for when we have an embedded db setup.
Test plan
Tests pass locally

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

Thanks for fixing this @wlmyng.
Having to call cluster.cleanup_resources().await seems potentially error-prone, is there a way to avoid that step?

I concur, and did give trying to avoid this a go. The thing is the join! bit is async, so wasn't sure if that would play well if I were to implement Drop
Thinking about ts jest, there's a way to setup and spin down each test and I imagine there's probably a way to do so in rust as well

there's a way to setup and spin down each test and I imagine there's probably a way to do so in rust as well

I don't think there is, sadly :(
The fact that the test hangs without the cancellation makes me think that there is some kind of implicit join going on anyway. I wonder if we did just the call to cancel() in the Drop impl, whether that would fix things?

there's a way to setup and spin down each test and I imagine there's probably a way to do so in rust as well

I don't think there is, sadly :(
The fact that the test hangs without the cancellation makes me think that there is some kind of implicit join going on anyway. I wonder if we did just the call to cancel() in the Drop impl, whether that would fix things?

I can give it another try 👉