Switching to nvidia don't work

So i tried this and i can correctly switch btw hybrid and integrated modes but everytime i try nvidia mode computer won't boot. In my case this file did not exist.. /etc/X11/xorg.conf so the suggested fix not apply here. The grub debug just stop with the last line saying: [OK] Reached target Graphical Interface. and nothing else happens.
The ouput of lspci -k | grep -EA5 'VGA|3D' in hybrid mode is:
0000:00:02.0 VGA compatible controller: Intel Corporation Alder Lake-P Integrated Graphics Controller (rev 0c)
Subsystem: Dell Device 0b19
Kernel driver in use: i915
Kernel modules: i915
0000:00:04.0 Signal processing controller: Intel Corporation Alder Lake Innovation Platform Framework Processor Participant (rev 02)
Subsystem: Dell Device 0b19
0000:01:00.0 3D controller: NVIDIA Corporation GA107M [GeForce RTX 3050 Mobile] (rev a1)
Subsystem: Dell Device 0b19
Kernel driver in use: nvidia
Kernel modules: nouveau, nvidia_drm, nvidia
0000:a4:00.0 Unassigned class [ff00]: Realtek Semiconductor Co., Ltd. RTS5260 PCI Express Card Reader (rev 01)
Subsystem: Dell Device 0b19
That is exactly the same output i got before installing this app.
Desctop details: XPS 15 9520
CPU: 12th Gen Intel i7-12700H (20) @ 4.600GHz
GPUs: Intel Alder Lake-P and NVIDIA GeForce RTX 3050 Mobile
kernel version: 6.2.2-arch1-1
DE/WM: KDE Plasma 5.27.2
Nvidia driver version: 525.89.02-6
Steps to reproduce: just swtich to nvidia and restart
External monitor: yes
Was envycontrol working before: fist time intalation
Note: nvidia works just fine if using it like described here:
https://wiki.archlinux.org/title/NVIDIA_Optimus#Use_NVIDIA_graphics_only
So i would assume the driver is correct.
Thanks in advance!

Hello there, what DE/WM are you using?

kde

kde Plasma 5.27.2
by the way, i cloned the repo to debut a little. is there any way to execute the switch method without really changing anything? just wanna debug the files created and values without really creating anything in my system. :D
Thanks!

It doesn't support dry runs but you can switch to Nvidia mode and grab a copy of the generated Xorg config before rebooting, as well you can return to hybrid mode prior.

kde Plasma 5.27.2
by the way, i cloned the repo to debut a little. is there any way to execute the switch method without really changing anything? just wanna debug the files created and values without really creating anything in my system. :D
Thanks!

It doesn't support dry runs but you can switch to Nvidia mode and grab a copy of the generated Xorg config before rebooting, as well you can return to hybrid mode prior.

I see! thanks for that. Look, this was my setup to just use nvidia before using your scritp. I dont really understand all options and how they work. But maybe rings the bell to you.
this for my grub:
GRUB_CMDLINE_LINUX_DEFAULT='loglevel=3 quiet ibt=off i915.modeset=1
Xsetup:
xrandr --setprovideroutputsource modesetting NVIDIA-0
xrandr --output eDP-1-1 --auto --pos 0x960 --output DP-1-2-1 --auto --pos 1920x0 (this is custom crap to set my secondary screen but --auto should work)
and 10-nvidia-drm-outputclass.conf:
Section "OutputClass"
Identifier "intel"
MatchDriver "i915"
Driver "modesetting"
EndSection
Section "OutputClass"
Identifier "nvidia"
MatchDriver "nvidia-drm"
Driver "nvidia"
Option "AllowEmptyInitialConfiguration"
Option "PrimaryGPU" "yes"
ModulePath "/usr/lib/nvidia/xorg"
ModulePath "/usr/lib/xorg/modules"
EndSection

You don't need ibt=off with Nvidia 530+ drivers also i915.modeset=1 is not
needed here.
Can you remove your custom Xsetup? you can do with envycontrol
---reset-sddm also remove the other Xorg config you have.
A little bit of context but initially Envycontrol used to generate a super
simple Xorg config like yours but in some cases it didn't work so now it
contains the PCI ID of the card.
On Fri, Mar 10, 2023, 09:36 Luis @.***> wrote:

I see! thanks for that. Look, this was my setup to just use nvidia before
using your scritp. I dont really understand all options and how they work.
But maybe rings the bell to you.
this for my grub:
GRUB_CMDLINE_LINUX_DEFAULT='loglevel=3 quiet ibt=off i915.modeset=1
Xsetup:
xrandr --setprovideroutputsource modesetting NVIDIA-0
xrandr --output eDP-1-1 --auto --pos 0x960 --output DP-1-2-1 --auto --pos
1920x0 (this is custom crap to set my secondary screen but --auto should
work)
and 10-nvidia-drm-outputclass.conf:
Section "OutputClass"
Identifier "intel"
MatchDriver "i915"
Driver "modesetting"
EndSection
Section "OutputClass"
Identifier "nvidia"
MatchDriver "nvidia-drm"
Driver "nvidia"
Option "AllowEmptyInitialConfiguration"
Option "PrimaryGPU" "yes"
ModulePath "/usr/lib/nvidia/xorg"
ModulePath "/usr/lib/xorg/modules"
EndSection
—
Reply to this email directly, view it on GitHub
https://github.com/bayasdev/envycontrol/issues/89#issuecomment-1463893504,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALY7IYIYNNO57KWI7HUBTXLW3M36TANCNFSM6AAAAAAVWITZXQ
.
You are receiving this because you commented.Message ID:
@.***>

if i remove ibt=off, i just tried, my pc stop booting. so now the only param in my kernel is ibt=off

so.. running sudo envycontrol -s nvidia generates:
.few lines in the Xsetup file
.a new file under /etc/X11/
and anything else i could check?
many thanks for your time by the way

You can send the content of those files to me
On Fri, Mar 10, 2023, 09:58 Luis @.***> wrote:

so.. running sudo envycontrol -s nvidia generates:
.few lines in the Xsetup file
.a new file under /etc/X11/
and anything else i could check?
many thanks for your time by the way
—
Reply to this email directly, view it on GitHub
https://github.com/bayasdev/envycontrol/issues/89#issuecomment-1463925178,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALY7IYLWSH2XCBAUJYACTBTW3M6QZANCNFSM6AAAAAAVWITZXQ
.
You are receiving this because you commented.Message ID:
@.***>

here it is xorg.conf
https://dpaste.com/7Y8352UMY
and Xsetup
https://dpaste.com/C9K8Y9LM7

Looks good to me, try removing the other .conf files you have created
inside the /etc/X11/xorg.conf.d directory
On Fri, Mar 10, 2023, 10:02 Luis @.***> wrote:

here it is xorg.conf
https://dpaste.com/7Y8352UMY
and Xsetup
https://dpaste.com/C9K8Y9LM7
—
Reply to this email directly, view it on GitHub
https://github.com/bayasdev/envycontrol/issues/89#issuecomment-1463931628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALY7IYJNBYBU2QCQ3VX3OFLW3M7BNANCNFSM6AAAAAAVWITZXQ
.
You are receiving this because you commented.Message ID:
@.***>

yes.. i removed already... there is another config there.. but just to increment the scrool on my mouse

yes.. i removed already... there is another config there.. but just to increment the scrool on my mouse

Can you capture the Xorg logs when the black screen happens?
https://wiki.archlinux.org/title/xorg#General
Thanks in advance.

Xorg.0.log
here are the logs, i dont really get it.. why the error.. but there is an error.
Thanks sir!

I see the problem, open your /etc/X11/xorg.conf and change the Nvidia bus
ID to "PCI:1:0:0", somehow there's a bug with the regex that generates the
Xorg config.
On Sat, Mar 11, 2023, 05:13 Luis @.***> wrote:

Xorg.0.log
https://github.com/bayasdev/envycontrol/files/10948233/Xorg.0.log
here are the logs, i dont really get it.. why the error.. but there is an
error.
Thanks sir!
—
Reply to this email directly, view it on GitHub
https://github.com/bayasdev/envycontrol/issues/89#issuecomment-1464877934,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALY7IYL3KGQMLRJWSKRDVHTW3RF37ANCNFSM6AAAAAAVWITZXQ
.
You are receiving this because you commented.Message ID:
@.***>

I rewrote the get PCI function, please clone this repository and perform the following steps:
git checkout rewrite-pci
sudo python ./envycontrol -s nvidia

Please let me know if the problem has been solved or not ;)

great. will check it in an hour and let you know.

And yes! rewrite-pci branch does the job correctly. Would you latter on, merge and release it in AUR? I can survive patching the file manualy until you deploy this.
Again, many thanks for taking the time and for coding this. Is very usefull.

And yes! rewrite-pci branch does the job correctly. Would you latter on, merge and release it in AUR? I can survive patching the file manualy until you deploy this.
Again, many thanks for taking the time and for coding this. Is very usefull.

I'm happy to read the new implementation works as expected. Currently I'm working on the 3.0 version of EnvyControl which will be released soon with a handful of new CLI options and improved reliability.

@luigi370 version 3 has been released to the AUR 🎉

Yes! thanks a lot! :D


VPAIDjs not displaying video

SpotX test tag: http://search.spotxchange.com/vast/2.00/79391?VPAID=JS&player_width=640&player_height=360&content_page_url=http%3A%2F%2Fexample.org&cb=14580139120
note: in both cases below, INSECURE vpaidmode needs to be set
Using the SpotX test tag on the videojs-ima advanced test page, the ad appears to be playing, but the video is black.
Using the same test tag on the ima html5 advanced test page, the ad plays and is visible.

It looks like the issue is that the VPAID is displaying under the content video player, because the content video player is being rendered later. #193 should fix this.


Add 'All regions' option

This allows us to compare provinces/states with country level data.

Interesting. This also relates to #42

I fixed the merge conflict @aatishb


Problem with installing BridgePRS

Hello,
I am interested in running BridgePRS but the instructions on installation are a little confusing, it seems that there is no bin folder that I can run bridgePRS from: ~/BridgePRS/bin/BridgePRS.sh
Should I make a folder and put a BridgePRS.sh inside?
Thank you,
Helia

Have you tried following the instructions here
https://www.bridgeprs.net
?
On Wed, 1 May 2024 at 16:03, Helia (Helyaneh) Ziaei Jam <
@.***> wrote:

Hello,
I am interested in running BridgePRS but the instructions on installation
are a little confusing, it seems that there is no bin folder that I can run
bridgePRS from: ~/BridgePRS/bin/BridgePRS.sh
Should I make a folder and put a BridgePRS.sh inside?
Thank you,
Helia
—
Reply to this email directly, view it on GitHub
https://github.com/clivehoggart/BridgePRS/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJUD6OD3WYNVAIHW5QUJADDZAFDCJAVCNFSM6AAAAABHCPPUM2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3TIMJQHEYTSOA
.
  The shell script does not need to be put in the bin folder.  If you need any additional help please email at<EMAIL_ADDRESS>Thanks


Cannot test newly created package

When I try to run package specs, I get Error: Cannot find module fs-plus/underscore-plus. If I install those packages manually and try apm test from console, then I get;
registerProtocol API has been replaced by the register[File/Http/Buffer/String]Protocol API family, please switch to the new APIs. coming from electron/protocol.coffee. Am I testing my package wrong or something? What should I do?
OSX Yosemite
Atom 1.2.4

If I install those packages manually

How did you install those?  npm install or apm install?  If the former, please remove the node_modules folder and execute apm install instead.

@50Wliu I installed using apm install.

Any help on this? I can't test my package :disappointed:

What package are you trying to test?  Are you testing through the Atom UI or atom --test spec/apm test?

I created a package, wrote some specs and trying to test it. None of the options that you said are working. I tried it from Ubuntu, there is no problem.


QR Code Reader

The webcam was able to capture the QR Code, but in some occasions, the value was incorrect. The value contains alphanumeric text and part of it was replaced by '-'. Anyone knows why it captures differently?

Hello,
I think, in this case the decoding was wrong. (blurry image or etc.)
I recommend to you, modify the result function with double check.
Regards
András


Core: Dataframe view

Implement a way to create view of DataFrame.
View creation function should accept BitSet / list of row indices along with list of column id's.
df = DG.FromJson({'a': [1, 2, 3, 4], 'b': [5,6,7,8]})
indices = [0,2,3]
cols = ['b']
v = df.view(indices ,cols )

v => 'b'
         5
         7
         8

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-14756


Alertmanager forwarder only add fields for string values.

Motivation
I'm using falco to capture events where some of the fields values are numbers (network ports) and would like that information in the alertmanager alert fields
Feature
Currently anything not a string is ignored, for at least integer values it should be simple enough to convert to a string and add to the alert labels.
Assuming this is accepted I'm happy to make a PR for it.

Hi @alsm
I'm totally OK with that, please propose a PR or I'll do it myself if you can't.

Hi @alsm
I'm totally OK with that, please propose a PR or I'll do it myself if you can't.


Can't get trigger filters to work with Actions

Doc: https://help.github.com/en/articles/workflow-syntax-for-github-actions#on
I can't figure out the right way to set up this filtered trigger.  In all of these cases, the trigger just doesn't fire.
on:
  push:
    paths:
    - '!docs/*'

on:
  push:
    paths:
    - '*'
    - '!docs/*'

on:
  push:
    branch:
    - '*'
    paths:
    - '!docs/*'

I just removed the filter with #17.


[$1000] Payment for 16830

Creating a new upwork job for https://github.com/Expensify/App/issues/16830 to finish payment, as that post has expired
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014b7dc53f5ca56a37
Upwork Job ID:<PHONE_NUMBER>319900160
Last Price Increase: 2023-06-06

Thanks @laurenreidexpensify. Accepted the offer.

Paid! :)


[6.0] Support writing links in synthesized technology root pages

Explanation: This fixes a bug impacting article-only documentation where DocC wouldn't attempt to resolve any links in an article that was implicitly promoted to a root page. This only happens when one of the article file names match the catalog name (or when there's only a single article, but then it has nothing to link to).
Scope: Links in article-only documentation without an explicit @TechnologyRoot page.
Issue: rdar://129282701
Risk: Low.
Testing: New tests verify that the articles links are resolved and that its authored metadata is preserved.
Reviewer: @patshaughnessy
Original PR: #980

@swift-ci please test


On The Fly Typescript Compilation

Implemented usage of the Typescript Compiler API to compile typescript on the fly.

Found an arguably more native solution over in test_ts_npx. You can test via npx peterthehan/create-discord-bot#test_ts_npx.

Found a more native solution over in test_ts_npx. You can test via npx peterthehan/create-discord-bot#test_ts_npx. PR at #62.

Your solution seems to be what we tried before, requiring users to have ts-node installed globally. I ran npm uninstall ts-node -g and then I ran npx peterthehan/create-discord-bot#test_ts_npx and it gave me the error '"ts-node-script"' is not recognized as an internal or external command, operable program or batch file.

Oh shoot you're right. I should have tested elsewhere, jumped the gun there.

I think I messed something up with the relative paths. We could also use process to force the global install of ts-node and typescript.

Looking into it this seems to be a lot more complex than it should be so I will be closing this. Later, though, I will open another pull request with a more easy to understand and change solution.


External Vault Support for Carbon Configurations

Describe the improvement
This is intended to provide the support to extend the existing Secure Vault capabilities to cater for the cloud secure vault services such as AWS Secret Manager, Azure Key vault and HashiCorp Vault and Hardware Security Modules etc. As the existing implementation cannot be extended to cater for this requirement.
Link to the Document
https://docs.google.com/document/d/1BF0h8RD8ikCQ8Octovhz2b0PGOPJLTkXae7xAnbPXOM/edit?usp=sharing
Related Issues:
Core support for external Vault Support for Carbon Configurations

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


update result listing positions ncn,court,date

Changes in this PR:
Judgment title and NEUTRAL CITATION go in the first row (so Neutral Citation takes the place of Date, currently). This aids with copying and pasting (done frequently by expert users, as the layout naturally follows the accepted citation format.
COURT NAME and DATE go in the second row (taking the place of neutral citation and court in the current design). This aids quick scanning of the results by court, as the court names are ranged left flush against the margin (a more likely case than scanning by neutral citation, as we already offer the opportunity to search for a Neutral Citation directly)
Dates remain aligned with the ‘date sort’ selector above the list to make the connection between them clear.
Removing the labels “Neutral citation:”, “Date:” as all are now clear from context.
Trello card / Rollbar error (etc)
https://trello.com/c/JLGFTtWs/766-🔍-pui-browse-redesign-search-result-listing-position
Screenshots of UI changes:
Before

After

[ ] Requires env variable(s) to be updated

Codecov Report

Merging #621 (dbafcaf) into main (5f03514) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #621   +/-   ##
=======================================
  Coverage   84.13%   84.13%
=======================================
  Files          17       17
  Lines         750      750
  Branches       48       48
=======================================
  Hits          631      631
  Misses        104      104
  Partials       15       15

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Minimum kernel versions from SAP note 2812427 need to be moved to sap-preconfigure

solved in commit 55f01521de3c04534e03b866e3f01c69bbc6788b .


feat(earn): add simple stats to earn report

Closes #669.
Adds simple stats to earn report:

total earnings
earnings in last 90 days
earnings in last 30 days
earnings in last 24 hours

:camera_flash:

@editwentyone Any ideas from your side for some UI improvments? This is just the rough idea, so any inputs are highly welcome :pray:
There is no rush.

its a great start, I would prefer the containers to be larger in mobile view, to stretch out to more but still be centered
lets keep it like this, its already serving the purpose and improve with new feedback

its a great start, I would prefer the containers to be larger in mobile view, to stretch out to more but still be centered
lets keep it like this, its already serving the purpose and improve with new feedback

:camera_flash:


Drop results on service's side

To be done at least 1 month after #1033

[ ] Remove results from service completely
[ ] Serve results only on dashboard

I think we are good to go.

Yup, let's do this :))

time to delete some code :)

time to delete some code :)
edit: changed the title because I got confused about the former one

yep, I think we have two of them (issues)…


Fixed TSLint 'newline-before-return' warning in MSBot

Added a newline berofe return sentences in several files for correcting 'newline-before-return' warnings from TSLint in MSBot.

 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.:x: dfavretto sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add explainPlan in QueryFile

Current coverage is 64.02% (diff: 0.00%)

Merging #1579 into master will increase coverage by 0.85%

@@             master      #1579   diff @@
==========================================
  Files           326        329     +3
  Lines         15425      15262   -163
  Methods       14920      14986    +66
  Messages          0          0
  Branches        148        138    -10
==========================================
+ Hits           9744       9772    +28
+ Misses         5681       5490   -191
  Partials          0          0

Powered by Codecov. Last update d33594b...9975bd3

Needs a revision.md

Only a minor comment. 👍🏾. Will merge once revision.md is in place.


Remove optionality from documents that are in unions

⚠ Breaking Change: Unions with Documents will see the inner document type change from Option<Document> to Document.
I don't think it makes sense for Documents to be optional inside of unions. Consider the following:
struct Input {
   union: MyUnion
}
union MyUnion {
    doc: Document,
    notDoc: String,
}

Currently, this generates:
enum MyUnion {
    Doc(Option<Document>),
    NotDoc(String),
}

But when you consider the following:
// Invalid JSON
{"union":undefined}
{"union":{"doc":undefined}}

// Invalid: variant not specified
{"union":{}}

// Valid, but is Document::Null
{"union":{"doc":null}}

// Valid, but uses the other variant
{"union":{"notDoc":"something"}}

It doesn't seem possible to ever have None as the document value in the union.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

this is definitely an @mtdowling question. If we decide this, it should be brought into NullableIndex

Turns out the NullableIndex already states that all members in union containers should not be nullable by checking the container type and returning false if it is a union (the default case).
We were previously overriding that with the or in SymbolVisitor for reasons unknown.
I pushed another commit to the PR that simplifies handleOptionality down to just relying on the NullableIndex for this which resolves the issue.


Trigger tests

API change check
API changes are not detected in this pull request.

/azp run python - ai - ci


Android Shrinker loses field

Android shrinker uses proguard rules. But for some reasons shrinker cuts out at least one field (with proguard all fine):
Caused by: java.lang.RuntimeException: java.lang.NoSuchFieldException: once
    at java.util.concurrent.atomic.AtomicIntegerFieldUpdater$AtomicIntegerFieldUpdaterImpl.<init>(AtomicIntegerFieldUpdater.java:270)
    at java.util.concurrent.atomic.AtomicIntegerFieldUpdater.newUpdater(AtomicIntegerFieldUpdater.java:49)
    at rx.schedulers.CachedThreadScheduler$EventLoopWorker.<clinit>(CachedThreadScheduler.java:180)
    at rx.schedulers.CachedThreadScheduler.createWorker(CachedThreadScheduler.java:171)

So, I can suggest small addition:
-keep class rx.schedulers.CachedThreadScheduler$EventLoopWorker { int once; }

Looks like a bug in build tools, if you don't mind — I can report that or you can do it yourself: https://b.android.com

Ok, I'll do it.

So, here it is. But I don't think they are going to implement all proguard features)

Well, Tor already set Priority Medium, so, I guess, they'll fix at least that.
Thanks a lot, Andrey!


./dev pull-requests switch is broken

(Ignoring the bad array subscript errors which are fixed by #1727)
$ ./dev pull-requests show 8
Title:          Get admin node deploy working-ish again, and fix a few nits.
Release:        development
Last Updated:   2013-03-01T12:38:40Z
Type:           bundle
Bundle ID:      b315146a56209cc5b6a07cb05124133c40dca602
Current CI States:
    new
Repo: barclamp-crowbar
    Repo with Changes:   https://github.com/VictorLowther/barclamp-crowbar.git
    Target Account:      crowbar
    Source Account:      VictorLowther
    Branch with Changes: pull-req-master-b315146a56209cc5b6a07cb05124133c40dca602
    Local Branch:        pull-req/VictorLowther/421
    Target Branch:       master
    Pull Request URL:    https://github.com/crowbar/barclamp-crowbar/pull/421
    Builds:              master
Repo: barclamp-deployer
    Repo with Changes:   https://github.com/VictorLowther/barclamp-deployer.git
    Target Account:      crowbar
    Source Account:      VictorLowther
    Branch with Changes: pull-req-master-b315146a56209cc5b6a07cb05124133c40dca602
    Local Branch:        pull-req/VictorLowther/100
    Target Branch:       master
    Pull Request URL:    https://github.com/crowbar/barclamp-deployer/pull/100
    Builds:              master
Repo: barclamp-logging
    Repo with Changes:   https://github.com/VictorLowther/barclamp-logging.git
    Target Account:      crowbar
    Source Account:      VictorLowther
    Branch with Changes: pull-req-master-b315146a56209cc5b6a07cb05124133c40dca602
    Local Branch:        pull-req/VictorLowther/46
    Target Branch:       master
    Pull Request URL:    https://github.com/crowbar/barclamp-logging/pull/46
    Builds:              master
Repo: barclamp-network
    Repo with Changes:   https://github.com/VictorLowther/barclamp-network.git
    Target Account:      crowbar
    Source Account:      VictorLowther
    Branch with Changes: pull-req-master-b315146a56209cc5b6a07cb05124133c40dca602
    Local Branch:        pull-req/VictorLowther/151
    Target Branch:       master
    Pull Request URL:    https://github.com/crowbar/barclamp-network/pull/151
    Builds:              master
Repo: crowbar
    Repo with Changes:   https://github.com/VictorLowther/crowbar.git
    Target Account:      crowbar
    Source Account:      VictorLowther
    Branch with Changes: pull-req-master-b315146a56209cc5b6a07cb05124133c40dca602
    Local Branch:        pull-req/VictorLowther/1725
    Target Branch:       master
    Pull Request URL:    https://github.com/crowbar/crowbar/pull/1725
    Builds:              master openstack-os-build hadoop-os-build hadoop-build cloudera-os-build openstack-build cloudera-build

That looks to me like master is the only viable build, but:
$ ./dev pull-requests switch 8 master
2013-03-01 18:30:57 +0000: dell_bios is not a barclamp/.
2013-03-01 18:30:57 +0000:  is not a YML file.
2013-03-01 18:30:57 +0000: dell_branding is not a barclamp/.
2013-03-01 18:30:57 +0000:  is not a YML file.
2013-03-01 18:30:57 +0000: dell_raid is not a barclamp/.
2013-03-01 18:30:57 +0000:  is not a YML file.
2013-03-01 18:30:58 +0000: overview is not a barclamp/.
2013-03-01 18:30:58 +0000:  is not a YML file.
./dev: line 1084: build_tmp: bad array subscript
./dev: line 1084: build_tmp: bad array subscript
./dev: line 1084: build_tmp: bad array subscript
2013-03-01 18:30:58 +0000: Build master is not applicable to pull request 8.
2013-03-01 18:30:58 +0000: Run ./dev pull-requests builds 8 to see what is.

$ ./dev pull-requests builds 8
2013-03-01 18:32:04 +0000: dell_bios is not a barclamp/.
2013-03-01 18:32:04 +0000:  is not a YML file.
2013-03-01 18:32:04 +0000: dell_branding is not a barclamp/.
2013-03-01 18:32:04 +0000:  is not a YML file.
2013-03-01 18:32:04 +0000: dell_raid is not a barclamp/.
2013-03-01 18:32:04 +0000:  is not a YML file.
2013-03-01 18:32:04 +0000: overview is not a barclamp/.
2013-03-01 18:32:04 +0000:  is not a YML file.
./dev: line 1084: build_tmp: bad array subscript
./dev: line 1084: build_tmp: bad array subscript
./dev: line 1084: build_tmp: bad array subscript
development/openstack-os-build: redhat-6.2 ubuntu-11.04 ubuntu-12.04 ubuntu-11.10 ubuntu-10.10 centos-5.7 centos-6.2 redhat-5.6 redhat-5.7
development/hadoop-os-build: redhat-6.2 ubuntu-11.04 ubuntu-12.04 ubuntu-11.10 ubuntu-10.10 centos-5.7 centos-6.2 redhat-5.6 redhat-5.7
development/hadoop-build: redhat-6.2 ubuntu-11.04 ubuntu-12.04 ubuntu-11.10 ubuntu-10.10 centos-5.7 centos-6.2 redhat-5.6 redhat-5.7
development/cloudera-os-build: redhat-6.2 ubuntu-11.04 ubuntu-12.04 ubuntu-11.10 ubuntu-10.10 centos-5.7 centos-6.2 redhat-5.6 redhat-5.7
development/openstack-build: redhat-6.2 ubuntu-11.04 ubuntu-12.04 ubuntu-11.10 ubuntu-10.10 centos-5.7 centos-6.2 redhat-5.6 redhat-5.7
development/cloudera-build: redhat-6.2 ubuntu-11.04 ubuntu-12.04 ubuntu-11.10 ubuntu-10.10 centos-5.7 centos-6.2 redhat-5.6 redhat-5.7

Why isn't master listed here? and we need openSUSE listed too ...
This doesn't work either:
$ ./dev pull-requests switch 8
2013-03-01 18:33:20 +0000: dell_bios is not a barclamp/.
2013-03-01 18:33:20 +0000:  is not a YML file.
2013-03-01 18:33:20 +0000: dell_branding is not a barclamp/.
2013-03-01 18:33:20 +0000:  is not a YML file.
2013-03-01 18:33:20 +0000: dell_raid is not a barclamp/.
2013-03-01 18:33:20 +0000:  is not a YML file.
2013-03-01 18:33:20 +0000: overview is not a barclamp/.
2013-03-01 18:33:20 +0000:  is not a YML file.
./dev: line 1084: build_tmp: bad array subscript
./dev: line 1084: build_tmp: bad array subscript
./dev: line 1084: build_tmp: bad array subscript
2013-03-01 18:33:20 +0000: You must pass a build to pull-requests switch, because there is no obvious default.
2013-03-01 18:33:20 +0000: Run ./dev pull-requests builds 8 to see what is available.

WONTFIX, noone cares about ./dev any more.


Killing open, not yet retuned Promises at the end of Controller.

I noticed a funny behaviour when hitting reload on the browser very fast.
The Promises take up to a second and when I hit reload faster, they get spooled in the background.
This is a normal behaviour, since the code is still working asynchronously to get those respoonses.
When I wait another 5 seconds for a new reload I receive all those spooled promises back in the new requested page. So, this last page contains not only the regular requested Pagelets, but also many Pagelets from previous (at that time incomplete) requests.
This might be a security issue, if another user receives responses who were originally intended for another user.
A solution might be to "kill" previous Promises at the beginning/end of the Controller, but this probably has to be request or user based.
I'm still watching the behaviour and implemented some logging with timestamps in the Controller to see what really happens.

I'm not sure how to reproduce this. I hit refresh very quickly in my browser, but I only get the expected response back. I waited a few seconds and hit refresh again and got the same thing.
Which sample app and controller are you testing? Can you provide a bit more detail about the exact repro steps? Thanks!

This happens only with PageletRenderOptions.ServerSide in my application, never with PageletRenderOptions.ClientSide.
I can't reproduce it with the /serverSideRendering route and therefore I'm closing the comment again.
I probably messed up with the code too much and have many divs with the same name due to copying. If it happens after my cleanup, I will shoot a small screencast.


tests: add sysfs test case to increase code coverage number

Pull Request Test Coverage Report for Build 2214

18 of 18 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.6%) to 50.469%

Totals

Change from base Build 2201:
1.6%

Covered Lines:
2151

Relevant Lines:
4262

💛  - Coveralls


[Bug report]:

Platform
Desktop
Version
1.4.2
Description
ping code input problem
Expected behaviour
Unable to enter ping codes continuously
Actual behaviour
Unable to enter ping codes continuously
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
Unable to enter ping codes continuously
The ping code error is displayed after intermittently entering the ping code
Operating System
Windows
Diagnostics
Unable to enter ping codes continuously

Developer Profile
No
Profile Type
Software (Stronghold)
Staking
No response
Crash Status
None of the above
Error Messages
No response
Error Logs
No response
Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

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

Can you please elaborate on the exact steps you're following? Please also try in the new version, 1.5.0

@yintolerance I´m going to close this issue. Feel free to open it again if you have further issues.


#650 omit json object response check for non-json endpoints

Closes #650

Codecov Report

Merging #684 into master will decrease coverage by 0.15%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##             master     #684      +/-   ##
============================================
- Coverage     83.42%   83.27%   -0.16%
  Complexity      839      839
============================================
  Files           153      153
  Lines          2245     2242       -3
  Branches        320      322       +2
============================================
- Hits           1873     1867       -6
  Misses          229      229
- Partials        143      146       +3

Impacted Files
Coverage Δ
Complexity Δ

...ly/rule/zalando/SuccessResponseAsJsonObjectRule.kt
76.47% <71.42%> (-18.53%)
19 <18> (ø)

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 079dc7d...4f099eb. Read the comment docs.

:+1:

This is coverage from Intellij:

👍

:+1:


[1.5.1+] Need a unique file to OpenCFP for Laravel Valet Custom Driver

Since the 1.5 refactor got rid of this file:

/bin/opencfp

We no longer have a good file to check against the existence of for the Laravel Valet Custom Driver. I have updated it to use a migration temporarily but I would like something nicer and more permanent.
I would like to submit a PR for a hidden file or something like

.opencfp.keep

I don't care what its called, but it needs to be something unique to opencfp so might as well have project name in the file name.
I added instructions wiki last night only to discover today that this no longer works for 1.5.
https://github.com/opencfp/opencfp/wiki/Laravel-Valet-Custom-Driver

@libbl I would happily accept a PR to add a file like this for Laravel Valet

Thanks for the quick reply. PR inc.

@llbbl
Would you be amenable to (assuming this works for Valet):
public function serves($sitePath, $siteName, $uri)
{
    $composer = json_decode(file_get_contents($sitePath . '/composer.json'), true);

    return $composer['name'] == 'opencfp/opencfp';
}

This way, there is no need for a marker file other than composer.json to specify something OpenCFP-specific.

@mdwheele I guess that works yea. Let me check the performance impact.

@mdwheele yea its fine. (or at least something similar) i'll close the PR and update the wiki.
I think that method has to return true or false, so I just modified what you posted so it does that.


Re-enable LFS

@cfrancesco Can you enable LFS on this repository? While trying to experiment with this model, I quickly ran into the inability to pull the LFS content. Even just enabling it for a few days would permit me to complete the checkout.
https://support.github.com/ticket/personal/0/1134239
Thanks.

Hi,
You can download sample weights using:
wget https://github.com/idealo/image-super-resolution/raw/master/weights/sample_weights/rdn-C6-D20-G64-G064-x2/ArtefactCancelling/rdn-C6-D20-G64-G064-x2_ArtefactCancelling_epoch219.hdf5
wget https://github.com/idealo/image-super-resolution/raw/master/weights/sample_weights/rdn-C6-D20-G64-G064-x2/PSNR-driven/rdn-C6-D20-G64-G064-x2_PSNR_epoch086.hdf5
wget https://github.com/idealo/image-super-resolution/raw/master/weights/sample_weights/rdn-C3-D10-G64-G064-x2/PSNR-driven/rdn-C3-D10-G64-G064-x2_PSNR_epoch134.hdf5

Hi,
You can download sample weights using:
wget https://github.com/idealo/image-super-resolution/raw/master/weights/sample_weights/rdn-C6-D20-G64-G064-x2/ArtefactCancelling/rdn-C6-D20-G64-G064-x2_ArtefactCancelling_epoch219.hdf5
wget https://github.com/idealo/image-super-resolution/raw/master/weights/sample_weights/rdn-C6-D20-G64-G064-x2/PSNR-driven/rdn-C6-D20-G64-G064-x2_PSNR_epoch086.hdf5
wget https://github.com/idealo/image-super-resolution/raw/master/weights/sample_weights/rdn-C3-D10-G64-G064-x2/PSNR-driven/rdn-C3-D10-G64-G064-x2_PSNR_epoch134.hdf5

Already tried the above, doesn't work.
The below worked for me, from latest PR:
wget https://public-asai-dl-models.s3.eu-central-1.amazonaws.com/ISR/rdn-C6-D20-G64-G064-x2/PSNR-driven/rdn-C6-D20-G64-G064-x2_PSNR_epoch086.hdf5
wget https://public-asai-dl-models.s3.eu-central-1.amazonaws.com/ISR/rdn-C3-D10-G64-G064-x2/PSNR-driven/rdn-C3-D10-G64-G064-x2_PSNR_epoch134.hdf5
wget https://public-asai-dl-models.s3.eu-central-1.amazonaws.com/ISR/rdn-C6-D20-G64-G064-x2/ArtefactCancelling/rdn-C6-D20-G64-G064-x2_ArtefactCancelling_epoch219.hdf5
wget https://public-asai-dl-models.s3.eu-central-1.amazonaws.com/ISR/rrdn-C4-D3-G32-G032-T10-x4-GANS/rrdn-C4-D3-G32-G032-T10-x4_epoch299.hdf5

Closing as there is alternatives in place to git lfs


Jira import progress

Looking to migrate from Jira to spaces, any progress on the Jira importer?

@indiealexh Hi!
As I can see, the repository owners are not particularly involved in the improvements.
I am already working on an improved version of importing tasks from youtrack (for my needs).
I can't promise, but if I have some free time, I will try to make it possible to import tasks from Jira, if someone does not do it before me))

@w3cproject
still waiting for you haha

A basic version for Jira import has just landed:
https://github.com/JetBrains/space-issues-import#from-jira
We are working on expanding the import API for issues (like adding the ability to import custom fields) on Space side.
Stay tuned!


Importing Experimental predictions.mat Data into CAPTURE_demo

Hello, I am referring back to Issue #43 , predictions.mat File in CAPTURE_demo Code. It was mentioned that the predictions.mat file generated from the demo could not be imported into the CAPTURE_demo code, and I had to download another demo predictions file for the CAPTURE_demo code to run. I have looked at other pose-estimation software, but they can not accommodate the three-dimensional nature of the predictions from DANNCE. If you could work on or provide the Bash script that executes dannce-predict followed by makeStructuredDataNoMocap.py so that experimental predictions data from DANNCE can be interpreted, that would be appreciated!

In 91c6a84, I added bash script in demo/markerless_mouse_1 that runs dannce-predict followed by makeStructuredDataNoMocap.py. makeStructuredDataNoMocap.py requires a reference skeleton file, which I have also added to configs/mouse22_skeleton.mat. If you are working with a different skeleton, you will need to make your own skeleton file, which defines the names of each joint, their linkages (e.g. EarL linked to EarR), and colors for plotting each joint (used by Label3D).


Move pdf support check from viewer to renderer

We're currently not able to test pdfx viewer components in CI, as the viewer components assert that the current device has PDF support, and our CI is running linux. We've mocked out the pdf renderer, and we're supplying a MockPdfDocument to PdfView, which works fine on MacOS, but fails in CI. By moving the check to the renderer we can successfully test PdfView in CI.
I also removed PdfViewState._currentIndex. The main reason is because it was late inited, and removing the await hasPdfSupport check meant that it started being accessed before it was inited. It also had an off by one error on initialisation. Instead, we can just use the current value of the pageListenable.

@luke-stead-sonocent thanks for you contribution!


Installing the plugin

once i have a project open this is my error
I am error Error: Request failed with status code 401
at createError (g:\Intellectify\node_modules\axios\lib\core\createError.js:16:15)
at settle (g:\Intellectify\node_modules\axios\lib\core\settle.js:17:12)
at IncomingMessage.handleStreamEnd (g:\Intellectify\node_modules\axios\lib\adapters\http.js:312:11)
at IncomingMessage.emit (G:\Intellectify\lib\events.js:538:35)
at endReadableNT (G:\Intellectify\lib\internal\streams\readable.js:1345:12)
at process.processTicksAndRejections (node:internal/process/task_queues:83:21) {config: {…}, request: ClientRequest, response: {…}, isAxiosError: true, toJSON: ƒ, …}
arg1:
Error: Request failed with status code 401\n\tat createError (g:\Intellectify\node_modules\axios\lib\core\createError.js:16:15)\n\tat settle (g:\Intellectify\node_modules\axios\lib\core\settle.js:17:12)\n\tat IncomingMessage.handleStreamEnd (g:\Intellectify\node_modules\axios\lib\adapters\http.js:312:11)\n\tat IncomingMessage.emit (node:events:538:35)\n\tat endReadableNT (node:internal/streams/readable:1345:12)\n\tat process.processTicksAndRejections (node:internal/process/task_queues:83:21) {config: {…}, request: ClientRequest, response: {…}, isAxiosError: true, toJSON: ƒ, …}
f @ d:\Users\vamp\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:98:6

Hi @Vampirebat ,

You are not just limited to java programs, you can work in any programming language or framework you want.
Run "npm install", then "F5" to run the extension in the development mode.
In this development mode vs-code window you can create or open any of your projects to use the intellectify with it.
To use intellectify, Open your command pallet with ctr+shift+p, to see all the available features intellectify offers you.

Error 401 means unauthorized request, which means you didn't change the OpenAI API key, remember to change it with your own OpenAI API key (in .env file) by accessing the openai website (https://beta.openai.com/account/api-keys)
Hope it will solve your issue, ask me further if you still face any issues,
Thank you!

Being inactive for a long time, I believe this issue must've been resolved using the latest go-around I've provided.
Closing this issue now.
Thanks.


did we support bluetooth mesh ?h

Bluetooth
did we support bluetooth mesh ?

This appears to be a question about the feature itself, not a question about the sample. You can follow the instructions on Windows developer support for support options. (Click "Forums and community".)


SSL configuration is not supported

There is a bug here, SSL configuration is not supported.

You can register SSL configuration with ConnectionFactory


update dependencies

Hi @thangngoc89 can you update dependencies as i see few are old https://www.versioneye.com/nodejs/react-howler/3.4.0

@ashwin31 Thank you
@Stenerson do you have time to do the update? I suggest you use semantic versioning: howler: "^2.0.0 this would prevent us from having this kind of issue

Yes, thanks @ashwin31 & @thangngoc89!
I must have accidentally removed the caret from howler when I updated it to 2.0 a while back and it's been a while since I've updated the dev dependencies.
I'll release an update soon. I assume this just calls for a patch release -3.4.1?

@Stenerson yup. This is def a patch


For testing on Android

Hi Apple,
These are the changes I’ve used to build XCTest for Android. I don’t know if you have other plans for Package.swift as it was but I figure since there are no version tags on this repo it cannot be used anyway.
Cheers.

Hey @johnno1962, thanks for the PR!
I'd be interested in hearing more about the workflow you're using where you make use of this. In this situation, is XCTest is building against the copy of Foundation from the Swift toolchain? If so, why not take XCTest from the toolchain as well?

Hi @briancroom, I’m having to roll my own XCTest as I’m preparing a toolchain for Android (http://johnholdsworth.com/bothworlds.html) and I don’t believe the build scripts have been adapted for a cross compile of XCTest (I could be wrong there.) Once it’s built I copy the shared lib and .swiftmodule into the toolchain manually and package it up. I’m using the swift-robot-environment scritps for the build - https://github.com/gonzalolarralde/swifty-robot-environment.

@xwu's whitespace comments are fair, and it would be good to address them before merging this, but otherwise I am fine with adding this.
In the future, it would be cool if we could get the test suite to be runnable via swift test as well, possibly by bridging to lit in some way.

With #201 merged, the remaining change here is just adding the Package.swift file. There's no point in running CI on that since it isn't involved in that build anyway.

Thanks Brian 👍


client_allowed_sans is not working

What did you do?
I tried to enable client_allowed_sans in web-config.file so that in mTLS it could verify the client hostname.
What did you expect to see?
I hope this could work.
What did you see instead? Under which circumstances?
Prometheus complained error as below:
msg="Unable to parse configuration" err="yaml: unmarshal errors:\n line 8: field client_allowed_sans not found in type web.TLSStruct".
System information
Linux 4.18.0-425.19.2.el8_7.x86_64 x86_64
Prometheus version
prometheus, version 2.44.0
  build user: root@739e8181c5db
  build date: 20230514-06:18:11
  go version: go1.20.4
  platform: linux/amd64
  tags: netgo,builtinassets,stringlabels

Prometheus configuration file
web-config.yml:
tls_server_config:
  cert_file: <path-to-cert>
  key_file: <path-to-key>

  client_allowed_sans:
  - <client_hostname>

  client_auth_type: RequireAndVerifyClientCert
  client_ca_file: <path-to-ca>

Alertmanager version
N/A

Alertmanager configuration file
N/A

Logs
msg="Unable to parse configuration" err="yaml: unmarshal errors:\n line 8: field client_allowed_sans not found in type web.TLSStruct".

Should work now, as the required dependencies have been updated. This was a new field added in exporter-toolkit 0.10.0, and until recently the Prometheus dependency was at 0.9.1.

Should work now, as the required dependencies have been updated. This was a new field added in exporter-toolkit 0.10.0, and until recently the Prometheus dependency was at 0.9.1.

Thanks @rgroothuijsen , yes it is working after upgrading Prometheus to 2.45.0, any news about when Alertmanager will support this?

Hello from the bug scrub!
Closing this as it was resolved for Prometheus long ago.


Question: What's the recommended approach for implementing platform-specific views (handlers)?

I want to create a component that renders a platform-specific view. MAUI's documentation about handlers is helpful, but I'm unsure how to map these to Reactor's compatible format. Is ScaffoldAttribute recommended in this case?

Yes, you have first to create the custom control (test it in a normal MAUI application) then make a wrapper in MauiReactor using the Scaffold attribute (https://adospace.gitbook.io/mauireactor/components/wrap-3rd-party-controls)

Thanks 👍


Newly created Unit Test Projects cannot be run with the Test Execution Command Line Tool

Description
Tests inside unit test projects that target .Net Framework 4.5 and do not reference Microsoft.VisualStudio.QualityTools.UnitTestFramework.dll are not runnable with the Microsoft Test Execution Command Line Tool.
Note: I'm not sure if this is the right repo to report this in, but can't find anything better.
A complete working example can be found at https://github.com/Zymus/mstest-bug
Steps to reproduce

Build the solution (default configuration is Debug)
cd into the build folder of the ProjectWithNugetPackages (ie. ProjectWithNugetPackages/bin/Debug)
Run mstest with the test container as the built dll (mstest.exe /testcontainer:ProjectWithNugetPackages.dll)

Expected behavior
My current expectation is that, if someone creates a Unit Test Project through Visual Studio, then that project is immediately runnable through the Microsoft Test Execution Command Line Tool. However, that doesn't seem to be the case.

Build the solution (default configuration is Debug)
cd into the build folder of the ProjectWithLegacyReference (ie. ProjectWithLegacyReference/bin/Debug)
Run mstest with the test container as the built dll (mstest.exe /testcontainer:ProjectWithLegacyReference.dll)

Output should look something like:
Microsoft (R) Test Execution Command Line Tool Version 15.0.27019.1
Copyright (c) Microsoft Corporation. All rights reserved.

Loading ProjectWithLegacyReference.dll...
Starting execution...

Results               Top Level Tests
-------               ---------------
Passed                ProjectWithLegacyReference.UnitTest1.TestMethod1
1/1 test(s) Passed

Summary
-------
Test Run Completed.
  Passed  1
  ---------
  Total   1

Actual behavior
Output of running mstest against the ProjectWithNugetPackages looks like
Microsoft (R) Test Execution Command Line Tool Version 15.0.27019.1
Copyright (c) Microsoft Corporation. All rights reserved.

Loading ProjectWithNugetPackages.dll...
Starting execution...
No tests to execute.

Environment
OS: Windows 7
Versions of the Microsoft Test Execution Command Line Tool tested against:

14.0.23107.0
14.0.25420.1
15.0.27019.1
15.0.26621.2

@Zymus : We see you are using MSTestV2 to run your tests i.e. MSTest.TestFramework and MSTest.TestAdapter. Unfortunately, MSTest V2 tests are not supported with “mstest.exe”.
We recommend you to use "dotnet test" or vstest.console.exe to run your tests from CLI.

@jayaranigarg Would dotnet test or vstest.console.exe be able to run both projects that reference MSTestV2 via Nuget package, and projects that do not reference MSTestV2, but instead reference the legacy apis?

Actually, just after testing it, it seems that vstest.console is able to run both v2 and legacy.

@jayaranigarg One strange thing I've noticed is that the vstest.console.exe installed at Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow fails.
$ "D:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe" ProjectWithLegacyReference.dll
Microsoft (R) Test Execution Command Line Tool Version 14.0.25420.1
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
Passed   TestMethod1

Total tests: 1. Passed: 1. Failed: 0. Skipped: 0.
Test Run Successful.
Test execution time: 0.2686 Seconds

$ "D:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe" ProjectWithNugetPackages.dll
Microsoft (R) Test Execution Command Line Tool Version 14.0.25420.1
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
Warning: No test is available in ProjectWithNugetPackages.dll. Make sure that installed test discoverers & executors, platform & framework version settings are appropriate and try again.

Information: Additionally, you can try specifying '/UseVsixExtensions' command if the test discoverer & executor is installed on the machine as vsix extensions and your installation supports vsix extensions. Example: vstest.console.exe myTests.dll /UseVsixExtensions:true

@Zymus : In VS2015, vstest.console was picked from the path you mentioned i.e.  Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow
For running tests in VS2015 with MSTestv2, you will have to provide /testadapterpath:<Path to folder where Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.dll exists">. Your commandline will look something like this
$ "D:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow\vstest.console.exe" ProjectWithNugetPackages.dll
/testadapterpath: "C:\Users\jagarg\source\repos\ProjectWithNugetPackages\ProjectWithNugetPackages\bin\Debug\"

However, if you try to run your tests in VS2017 latest, you will observe vstest.console.exe is picked up from "C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\Common7\IDE\Extensions\TestPlatform\vstest.console.exe" and you do do not need to give any /testadapterpath to run MSTestV2 tests.
We would highly recommend you to move to latest VS2017.

@Zymus any more queries here? VS2017 should allow you to run MsTestV2 tests with minimal effort. Please move there. If this is not possible please follow the workaround specified by @jayaranigarg. Lemme know if you have any follow up questions, else will close the the issue.

@ShreyasRmsft No, that is everything I think. Using VS2017 allows us to run both legacy tests, and those using the Nuget packages.

unbelievable !  This Issue comes in many version of MS Test.
@ShreyasRmsft you could not found out the root cause ?  This is happening again in MS Test 2017
with Test Execution Command Line Tool Version 15.0.27729.1 which was updated 30 October 2018.

@usamakhalil86

Closing this issue as tracking it here.

@vagisha-nidhi I see you have closed this issue, which is separate then
https://github.com/Microsoft/testfx/issues/512
This poor user complained about using VS 2017 and creating unit test project for .NET frame 4.6.
Those are broken now.
Kindly check with VS 2017 team to use "vstest.console.exe" when creating .NET framework 4.6 unit tests as well.

Hi @usamakhalil86
How are you trying to run the tests? The IDE uses vstest.console.exe to run tests.
Are you facing any issues while running tests with VS 2017? Can you please provide us logs and screenshots or a repro project so that we can take a look?
You can collect logs in ide by enabling Tools->Options->Test->Logging Level->Diagnostic
This might help you when  collecting logs using command line.


Fixed wrong tag and added ODF

Fixed a tag that was incorrectly set to libreoffice, and Added odf tag.
Microsoft Reviewers: Open in CodeFlow

@nogajun I set the libreoffice tag because the OpenOffice and LibreOffice projects are directly related to each other.


Attempting to connect to non-default vault fails

I am attempting to create ephemeral vaults to use with automatic tests using .NET. Everything works fine when connecting to the default clients at https://localhost:8443. When I create a new client, I am unable to connect to it and get an error: "No such host is known". I even tried to connect to https://primary.localhost:8443 but still get the same error. I know I'm probably just missing something silly. How can I create two clients in parallel that point to different key vaults? Thanks!

Hi @mcolebiltd ,
thank you for using Lowkey Vault, and thank you for opening this question!
From your description, it looks like your OS layer is the one kindly informing you about the fact that the host using the primary.localhost name is unknown. You can solve this issue in at least a few ways:

If you can add the primary.localhost name as <IP_ADDRESS> to your hosts file, that should get rid of the error.
Or, you could try using the https://<IP_ADDRESS>:8443 as the second vault (it is a completely different vault than https://localhost:8443).
Or, you can use the alias feature to set your real IP address and port number as the alias of any of the vaults, so that you can reach them using your IP.
Otherwise, you need to somehow convince your network to send the packets to Lowkey Vault while using the primary.localhost:8443 value as host header, so that the app will be able to find which vault it should use. As this is the hardest, I would recommend to try the other 3 options first.

Please let me know how it goes and whether this was helpful! Thank you!

Option 2 worked like a charm. Thank you!

Hi, thank you for letting me know! Best of luck!


implements simple charge clustering class

fixes #26

Hello @richardjgowers! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/konnektor/network_tools/charge_clustering.py:

Line 10:80: E501 line too long (126 > 79 characters)


Rename common_job_properties

This PR renames common_job_properties.groovy file to use groovy's naming convention.

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

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
 I tried to run it from recent master branch (https://github.com/apache/beam/pull/5914) and it failed due to changes introduced in this PR. Standalone seed job works fine. Sorry for the trouble. Can you help with this?


out-in transition issue with async components

Version
2.3.0
Reproduction link
https://jsfiddle.net/z11fe07p/1959/
Steps to reproduce

load https://jsfiddle.net/z11fe07p/1959/
click the "B" button

What is expected?
The previous component (A) should disappear before component B appears
What is actually happening?
Components A and B are visible simultaneously

This behavior is happens only when component B is not already loaded. Next time you switch from A to B, everything is fine.

Thanks, looks like a bug. Thanks for the concise and clear repro 🙂

It seems that the async component is rendered too early

As the async component has not yet been parsed, the transition  is a comment Element

I have solved, wait a minute ...

Are you sure this solves the async component transitions?  I'm new to Vue so it's likely a bug in my code, but my repo below still exhibits the transition out-in bug where the incoming component doesn't wait for the out fade to complete.  Note I'm using the http-vue-loader named above.
Async Component Transition Bug Demo

This is indeed still failing with the same repro. The transition starts right away as @clingor said

@posva
https://cdnjs.cloudflare.com/ajax/libs/vue/2.3.4/vue.js
Looks like this version does not contain the fix code

The fix has not been released yet. We closed issues when the patch lands in the dev branch, but it might not be in a public release yet.

My bad, I just saw the commit was before the release and thought it was strange. It would have been in the release notes anyway

Sorry for the misunderstanding, and thank you for confirming the issue!

This doesn't appear to be fixed yet in 2.4.2.  Is this still pending release?
Async Component Transition Bug Demo

@clingor Are you sure? The jsfiddle works nicely

The fiddle does work correctly, but the code in my example GitHub repo still exhibits the same behavior.  It could be the http-vue-loader helper library, or maybe something in my code.  This example is simple and was created to demonstrate the issue.
@FranckFreiburger would any of these VueJS changes require an update to http-vue-loader?

if you put the issue on a jsfiddle without the loader I can take a look

Am 21.07.2017 21:16 schrieb "Chris Lingor"<EMAIL_ADDRESS>
This doesn't appear to be fixed yet in 2.4.2. Is this still pending
release?
Async Component Transition Bug Demo
https://github.com/clingor/githubIssuesFileShare/tree/master/vue-async-component-bug
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vuejs/vue/issues/5760#issuecomment-317088909, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Aclno69wDoTlZziST6eSocqxEnn8mFx-ks5sQPkKgaJpZM4Nng7j
.

The culprit is <keep-alive> !
Steps to reproduce

see https://jsfiddle.net/z11fe07p/2382/
click the "B" button

@posva, are you able to reproduce ?

@FranckFreiburger You should open a new issue for that


[HUDI-7918] Remove support of Spark 2, 3.0, 3.1, and 3.2

Change Logs
Remove support of Spark 2, 3.0, 3.1, and 3.2
Impact
Make maintaining hudi easier
Risk level (write none, low medium or high below)
medium
Documentation Update
Will need to announce this on the site?
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Also, could you update README.md and create a ticket on updating the website on Hudi Spark Support?


[Android] overflow visible doesn't work

second view {overflow: 'visible'}

Overflow is not supported on Android: https://facebook.github.io/react-native/docs/known-issues.html#the-overflow-style-property-defaults-to-hidden-and-cannot-be-changed-on-android

@ide the link is broken, is it still relevant?

Still not working. Drawing outside a view on Android is nearly impossible...

Are there any practical recommendations ?

@aymericbouzy - I managed to get something similar to this behavior using elevation. Checkout the gist.

Would it be possible to update the documentation to call this out? From what I see here in the docs, overflow: 'visible' | 'hidden' would be available on both platforms.

How is this issue closed? any solutions?

This is a real deal breaker since advanced user interfaces like this one are just impossible to create on Android, while being pretty easy on iOS.

Are there really no plans to find a solution to this matter? I find it hard to believe that Facebook is able to cope with it...

I believe the interface you are showing can be obtained with absolute positioning.

No he can't there is always a problem with zindex and ontouch methods it's very messy...

can't you abstain from using zIndex and simply put objects in the order in which they should be displayed ?

@aymericbouzy Unfortunately not, since you can't overlap views on Android. This really proves problematic for this kind of layout, particularly when soft shadows are involved. There are many issues at stake: no visible overflow (the real bummer), buggy z-indexes, absolute positioning randomness...
Let me show you another example of a UI I am trying to implement, but for which I am currently clueless.

On iOS, I would simply use negative margins or even absolute positioning with a global container; this would be pretty easy. But because of the overflow issue, neither of this classic methods will work on Android. Do you have any idea that would allow me to circumvent these limitations?

@bd-arc I have absolutely no experience building such complex UI, and I'm not saying it's simple.
The way I would go though would be the following : putting each view in a different layer, each layer being an absolute positioned view taking the full width and height of the display, and positioning the visible views inside each layer. Since each layer takes the full dimensions of the display, overflow is not a problem.
I fully agree this solution is a pain, but I believe it works. Please tell me if I'm wrong :)

On 2 Nov 2016, at 10:46, bd-arc<EMAIL_ADDRESS>wrote:
@aymericbouzy https://github.com/aymericbouzy Unfortunately not, since you can't overlap views on Android. This really proves problematic for this kind of layout, particularly when soft shadows are involved. There are many issues at stake: no visible overflow (the real bummer), buggy z-indexes, absolute positioning randomness...
Let me show you another example of a UI I am trying to implement, but for which I am currently clueless.
https://cloud.githubusercontent.com/assets/7899844/19923629/accc6622-a0e8-11e6-82b3-e7ac55c45002.png
https://cloud.githubusercontent.com/assets/7899844/19923171/7ab68200-a0e6-11e6-8e3e-07f03521133c.png
https://cloud.githubusercontent.com/assets/7899844/19923650/bfd909be-a0e8-11e6-96e6-fd67d42fff73.png
On iOS, I would simply use negative margins or even absolute positioning with a global container; this would be pretty easy. But because of the overflow issue, neither of this classic methods will work on Android. Do you have any idea that would allow me to circumvent these limitations?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/facebook/react-native/issues/6802#issuecomment-257817125, or mute the thread https://github.com/notifications/unsubscribe-auth/AHymFbxWiNuouhdMkkT7uHjvgGHMv7kgks5q6FvggaJpZM4H_jHm.

@aymericbouzy "each layer being an absolute positioned view taking the full width and height of the display" If you do that you won't be able to touch any button, or at least only the one with higher zIndex, even if you see the underlaying touchable element.

@bd-arc you're right. Then you can add touchable views on the uppermost layer upon the visible parts, but the interface is not going to be very responsive ... this is definitely not a long-term solution.

@aymericbouzy Unless I'm mistaken, you're solution is built on something that's pretty complicated: I don't see any reliable way of retrieving complex content's height, particularly before it is rendered (doing it after, assuming this would be both possible and reliable, would lead to jerkiness issues - believe me, I've been there :)
I would rather go with clever margins and paddings, since this seems easier to maintain, but it would be an incredibly tedious task, with lots of limitations. Furthermore, that still leaves no room for soft shadows. And we haven't even talked about orientation change or responsiveness...

@bd-arc indeed it is very complicated, and unmaintainable, and I wouldn't go this way on a general basis. I'm just saying this is the solution I've found for a much simpler use case than yours.

im having a similar issue, im using position absolute to draw outside a view.
everything works fine, untill i try to set onLayout property on the view, then it just clips whatever is outside the view.
any suggestions?

A property that's only supported in one OS is not very useful ... don't claim react-native supports it then.

Been bitten by this multiple times, there should be a warning in RN if overflow: 'visible' is set on Android.

 It doesn't support normal shadows, blur and even overlapping now

Any updates ..?

There is a rewrite of Android's View library called Nodes. It is still not ready yet but maybe will be later this year if it is a success. Check back later this year.

I think some updates needs to be done on the document https://facebook.github.io/react-native/docs/layout-props.html that overflow dose not work on Android currently

overflow?: enum('visible', 'hidden', 'scroll')
overflow controls how a children are measured and displayed. overflow: hidden causes views to be clipped while overflow: scroll causes views to be measured independently of their parents main axis. It works like overflow in CSS (default: visible). See https://developer.mozilla.org/en/docs/Web/CSS/overflow for more details.

Hey checkout this https://www.npmjs.com/package/react-native-zindex-views it would help you.

Just looking for a little clarity, wouldn't Android's setClipChildren provide something similar?
https://developer.android.com/reference/android/view/ViewGroup.html#setClipChildren(boolean)

@Dygerati have you found out more about this? It looks like either clipChildren or clipToPadding  control overflow on Android: https://stackoverflow.com/questions/18048997/android-view-disappearing-when-go-outside-of-parent
The Android overflow bug has been bothering me for a while now, a fix would be nice.

Can this be reopened?

Is this overflow:visible issue on Android closed by this commit - https://github.com/facebook/react-native/commit/30044fd531c22c4c5e8f1ede206fa7c2c3fd3aa8 ?

The commit referenced above (https://github.com/facebook/react-native/commit/30044fd531c22c4c5e8f1ede206fa7c2c3fd3aa8) doesn't fixes this issue when using overflow: visible with images.

AFAIK it will fix overflow:hidden in case there is border radius defined.
And about overflow:visible on android, it may be impossible to resolve because of android native views specific, but I can be wrong in this point

Found a working link to Ide's known issues article. It's a wontfix: https://web.archive.org/web/20160702114556/http://facebook.github.io:80/react-native/docs/known-issues.html#the-overflow-style-property-defaults-to-hidden-and-cannot-be-changed-on-android

RN50 overflow visible seems broken for me, even with something else than images. See https://github.com/facebook/react-native/issues/17074

Any updates on this?

Still no change?

"Overflow" feature for Android won't be supported for next a few years. So find a tricky way to implement your design.

+1 !!!

I found this workaround - https://github.com/entria/react-native-view-overflow
But note that it probably will require a bit of of restructuring of your code.


[HOTFIX][BUILD][test-maven]Remove duplicate dependency

CC @JoshRosen

Test build #49879 has started for PR 10868 at commit 09ed7a5.

I think test-maven might have to be in lowercase? Let's wait and see

right... just updated it. retest this please

Test build #49881 has started for PR 10868 at commit 09ed7a5.

Test build #49879 has finished for PR 10868 at commit 09ed7a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test PASSed.

Test build #49881 has finished for PR 10868 at commit 09ed7a5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

retest this please
On Thu, Jan 21, 2016 at 11:50 AM UCB AMPLab<EMAIL_ADDRESS>wrote:

Merged build finished. Test FAILed.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/10868#issuecomment-173688747.

retest this please

Test build #49887 has started for PR 10868 at commit 09ed7a5.

Test build #49887 has finished for PR 10868 at commit 09ed7a5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #49904 has started for PR 10868 at commit 053f061.

Test build #49904 has finished for PR 10868 at commit 053f061.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #49912 has started for PR 10868 at commit 053f061.

Test build #49912 has finished for PR 10868 at commit 053f061.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #2438 has started for PR 10868 at commit 053f061.

Test build #2438 has finished for PR 10868 at commit 053f061.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

retest this please

Test build #49920 has started for PR 10868 at commit 053f061.

Test build #49920 has finished for PR 10868 at commit 053f061.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

LGTM.

Merging to master


Enable light wallets to restore from seed

Have the light vendor scan for addresses given an extended public key (BIP44).
Since this is a resource intensive operation, be careful not to create a DoS vector.

This code needs to be ported to the hub https://github.com/byteball/byteball/blob/master/src/js/controllers/recoveryFromSeed.js#L41

available since 1.10.1


Video Conference Mute does not toggle when Destiny 2 is focused in fullscreen mode

Microsoft PowerToys version
v0.60.1
Running as admin

[X] Yes

Area(s) with issue?
Video Conference Mute
Steps to reproduce

Run PowerToys with Video Conference Mute enabled
Open Destiny 2 in fullscreen mode
Activate VCM hotkey

✔ Expected Behavior
Microphone mute expected to toggle with Destiny 2 open
❌ Actual Behavior
Microphone mute does not toggle unless Destiny 2 application is minimized
Other Software
Destiny 2 v4.1.5

Thanks for the feedback but VCM is moving to maintenance only in the 0.67 timeframe. Our team will only directly address critical bugs, security and accessibility issues.
We'll accept community PRs for enhancements.


Readme updates

[x] Docker Hub (https://hub.docker.com/repository/docker/gamedisplayer/sumo-experiments)
[ ] Github

Still under construction but better


[jsx-curly-spacing] False positive with render props

The following code is causing false positive on this rule.
tslint-react: 3.4.0
<Route
  children={({ location, history }) =>
    store.auth.isAuthenticated ? (
      <Redirect to={location.state.from} />
    ) : null
  } // Error here: There should be no space before } (jsx-curly-spacing)
/>

I might add that code is auto formatted by Prettier so I cannot write it differently :)
/cc @bolatovumar

Hi,
Can a new minor version be released that include this fix? Thanks.

+1, seeing this issue as well.

Same here.

Can we please have a release that fixes this?


kdenetwork-filesharing does not work in dolphin

Describe the bug
As described at https://apps.kde.org/kdenetwork_filesharing/ this app is supposed to enable the "share" screen of file properties in dolphin. For me this doesn't work: the "share" tab is available but it is empty.
As far as I can tell I have set up samba and usershares correctly, as I'm able to make new usershares and connect to other hosts on the network using the command line tools.
Usershares I make also show up as expected in the "samba status" app.
Steps To Reproduce
Steps to reproduce the behavior:

install the kdenetwork_filesharing package in a KDE environment
open up dolphin and right-click a folder (not a symlink)
choose "properties"
go to the "share" tab
observe it is empty

Expected behavior
I expect the share tab to contain similar options as shown at https://apps.kde.org/kdenetwork_filesharing/
Screenshots
the empty "share" tab:

the usershares showing up correctly in the "samba status" app:

Additional context
When launching dolphin from the command line, I get the following output when accessing the "file properties":
true
actions!!!
QQmlComponent: Component is not ready
qrc:/org.kde.filesharing.samba/qml/main.qml:12:1: QML StackView: push: qrc:/org.kde.filesharing.samba/qml/MissingSambaPage.qml:18 Cannot assign to non-existent property "explanation"

To me this seems to be trying to render the missing samba page and somehow failing. From looking around I noticed that the plugin has a feature to let you install samba if you don't have it, for less ehm "strict" OSes. This could mean that despite my efforts it still doesn't find samba, and that resolving that will resolve the issue.
Notify maintainers

@ttuegel
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.01 MiB download, 0.05 MiB unpacked):
  /nix/store/yq9qag5ak3w15nhcqplnhxpny6295l36-stdenv-linux
copying path '/nix/store/yq9qag5ak3w15nhcqplnhxpny6295l36-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.78, NixOS, 24.05 (Uakari), 24.05pre-git`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/store/gzf4zwcakda1nykn6h0avh45xhjhvsz4-source`

Add a :+1: reaction to issues you find important.

Correct, I've got a fix for this that I'll push to the Plasma 6 branch in a bit.

Pushed now.

Still doesn't work, but differently. Prompts for a password change, and if you enter it, nothing happens.

Also, Windows Shares tab whines on kio qt being wrong version

Could not load plugin from /nix/store/phqwx5gxccib9kzsprklrj8m8h68nwk4-kio-5.115.0/lib/qt-5.15.12/plugins/plasma/kcms/systemsettings/kcm_smb.so: The plugin '/nix/store/phqwx5gxccib9kzsprklrj8m8h68nwk4-kio-5.115.0/lib/qt-5.15.12/plugins/plasma/kcms/systemsettings/kcm_smb.so' uses incompatible Qt library. (5.15.0) [release]

This still doesn't works

Extremely unlikely. If you're still seeing this, open a new issue please.


Mocha is should not be in runtime dependencies

I want to use Jest in my project, however mocha and jest conflicts while there is mocha installed.
gdax-tt should NOT depend on mocha in runtime dependencies.

mocha and @types/mocha have been moved to dev-dependencies. That should sort you out when using GTT as a module. If not, then :man_shrugging: , since it must be packaged as part of the dev tools.
https://stackoverflow.com/questions/43541077/why-does-mocha-need-to-be-in-devdependencies


fix: Improve performance for Gravsearch queries

This is an attempt at a minimally invasive fix for some of the performance issue we face with Gravsearch.
Gravsearch queries contain FILTER NOT EXISTS blocks which are used to only retrieve resources which were not marked as deleted. For some reason this slows down the query substantially. Replacing this block with a MINUS makes simple queries faster.
For the difference between those two statements in the official SPARQL documentation:
https://www.w3.org/TR/sparql11-query/#neg-notexists-minus

NOT EXISTS and MINUS represent two ways of thinking about negation, one based on testing whether a pattern exists in the data, given the bindings already determined by the query pattern, and one based on removing matches based on the evaluation of two patterns.

Pull Request Checklist
Task Description/Number

Issue Number: DEV-2714
Basic Requirements
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] fix: represents bug fixes
[ ] refactor: represents production code refactoring
[ ] feat: represents a new feature
[ ] docs: documentation changes (no production code change)
[ ] chore: maintenance tasks (no production code change)
[ ] test: all about tests: adding, refactoring tests (no production code change)
[ ] other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No
[ ] Maybe (not 100% sure => check with FE)

Does this PR change client-test-data?

[ ] Yes (don't forget to update the JS-LIB team about the change)
[x] No

Cool! How big is the difference?

Have not measured it exactly: The query in the issue is on a resource of webern restricted to a Bibliography class with two query pattern is like on two different properties. That query times out on Fuseki. The improvement in this issue makes it work in a couple of seconds. Hope I can see the exact improvement once I set up our dashboard for the query metrics.


Don't run on normal ruby files

I've sent a PR to the other linter, but feel free to port those changes (based on NodeLinter's package.json detection) if you want.
https://github.com/pelletiermaxime/SublimeLinter-contrib-foodcritic/pull/3

If you can do a pull request I would appreciate it.
Thanks!

I had a go, but part of what makes the other one work is that it doesn't actually do the tempfile/stdin stuff.
I can port the metadata.rb detection stuff easily enough, but knowing which rules to skip is a pain.
I've raised https://github.com/acrmp/foodcritic/issues/389 to ask for a better option


HERE.com's new Geocoding & Search API

Hi @KostyaEsmukov,
Just noticed that HERE.com is no longer maintaining the "Geocoding" API but are instead re-directing folks to use the "Geocoding and Search API". I haven't used both of them extensively but it looks like the new API better handles ambiguous addresses and supports providing component level filtering (e.g. city, county etc).
Is there any interest in extending support to that API? I can make a PR if you are open to this. Both these APIs are in-compatible so it would either have to break the existing one or create a new Geocoder class (I prefer this approach).
https://developer.here.com/documentation/geocoder/dev_guide/topics/what-is.html

I think it might be useful to add a new class for the new API (e.g. HereV7). So yeah, a PR would be very welcome!
If they share a lot, then it can be implemented similarly to BaiduV3 which extends Baidu: https://github.com/geopy/geopy/blob/7e276b43fd66d603e8fd0c25313896ee324e1d49/geopy/geocoders/baidu.py#L275-L283
If they don't, it should extend the base geocoder class as the most of the other geocoders.

I think they are just different enough that it might warrant a separate class. I wrote a small wrapper that I have been using internally - I will clean up and make a PR before EOD

Fixed via #433


#first aside: change "width: 100vw" to "right: 0"

This prevents the aside element from extending beyond the width of the viewport and the ability to horizontally scroll will be removed

Great idea, thanks! :blush:


Convert identifiers to IDs for create_batch and update_batch

@michaelsauter looks good, thanks :)

Thanks for merging.
I opted for the easiest change, but I was thinking if this is a more common case and should be handled later, when the request is send? There we could do some "deep conversion" ... but I haven't seen enough of the API yet to judge whether this would be a good idea or not.


Approve Token + Create Challenge Flow UX Enhancement

User Impacted
0xc6Ac8FC10242771eA79B1Bdbb4d72f775AF9587F
Discord ID (including number)
danner.eth
Workflow Impacted
Brainstorm
Feature Impacted
Launch Challenge
Issue description
https://www.loom.com/share/d7598e6d02b34db19592ada8f589d182
Modal has two steps but doesn't make it abundantly clear.
User needs to approve token then launch challenge and transfer token.
Consider this example from velodrome.finance that shows that multiple actions will occur:

The modal shows both steps and show which step the user is on.
Blocking
no it is not
Relevant log output
No response

Prioritize redirects over this, will solve problem most likely

@mahgillis @A4x1 relevant info for the Traversing design sprint too


Jenkinsfile

https://jenkins.io/doc/pipeline/jenkinsfile/

:metal::heart::bow:

Just realised this was an issue and not a pull request. Feel like opening a PR to add this?


Test eslint

We've deliberately added a eslint failure here. Hopefully the pipeline fails...

Works now! Now the pipeline should turn red for any eslint violations


No Transactions toast is being shown although there are transactions present

Resolved in: https://github.com/Surya-06/XpensAuditor---Group-11/pull/13


[FEATURE] Reduce test data creation time in unit tests

Is your feature request related to a problem? Please describe.
Currently, our unit tests create test data on ganache (a blockchain simulator) by executing numerous blockchain requests before running test cases. These requests are executed sequentially, resulting in a significant amount of time being spent on test data creation.
Describe the solution you'd like
To reduce the time required for test data creation, inplement asynchronous execution of these requests.

The web3 module caches and retains aiohttp.Client within it. (link)
And the aiohttp.Client holds the running event loop as its private variable at the time of the object's creation. (link)
Additionally, pytest-asyncio uses different event loops at the session and function levels.
https://pytest-asyncio.readthedocs.io/en/latest/concepts.html#asyncio-event-loops
Therefore, in pytest, if async_web3 is used at the session level and subsequently at the function level, the event loops do not match, leading to improper execution of requests.
To avoid this, it is necessary to delete cache of aiohttp.Client held by web3 in the function-level fixture.


WIP: Retrieve Blockchain logs

Work towards: https://github.com/onflow/developer-grants/issues/148
Description
Provides a way to retrieve logs from the Blockchain. Also improves the look & feel for logs that come from test files.

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Logs from test files are consistent with the blockchain logs. The INF logs are the blockchain logs, which we could return to the test file


Fixes #52

Enforce NPE on InterledgerAddress#require methods.
Add unit-test coverage.
Start rename described in #53.

Signed-off-by: sappenin<EMAIL_ADDRESS>

Codecov Report

Merging #54 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #54      +/-   ##
============================================
+ Coverage     74.64%   74.65%   +0.01%
  Complexity      437      437
============================================
  Files            92       92
  Lines          2780     2782       +2
  Branches        301      303       +2
============================================
+ Hits           2075     2077       +2
  Misses          530      530
  Partials        175      175

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/org/interledger/InterledgerAddress.java
96.92% <100%> (+0.09%)
20 <4> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Remove '/' from .Hugo.Generator

See: http://www.w3.org/TR/html5/syntax.html#start-tags

Hi @luixxiul,
According to http://www.w3.org/TR/html5/syntax.html#start-tags (emphasis mine):

6. Then, if the element is one of the void elements, or if the element is a foreign element, then there may be a single "/" (U+002F) character. This character has no effect on void elements, but on foreign elements it marks the start tag as self-closing.

So, while the "/" is unnecessary in <meta name="generator" content="Hugo 0.14" /> for HTML’s "HTML Syntax", it is perfectly legal and allowed.
On the other hand, the "/" is mandatory for HTML5’s "XHTML Syntax", which I am sure some Hugo users use.  Furthermore, Hugo has the blackfriday.HTML_USE_XHTML flag enabled, so the void elements (or so-called "singleton tags" in Blackfriday source code) like img, hr and br are all closed with / >, so it is not just {{ .Hugo.Generator }} that is using / >.
That said, the HTML Syntax is more popular than the XHTML Syntax, so perhaps we should make it a user-configurable option, i.e. to enable XHTML-style close tag (/ >) or not (>).

That said, the HTML Syntax is more popular than the XHTML Syntax, so perhaps we should make it a user-configurable option, i.e. to enable XHTML-style close tag (/ >) or not (>).

No. It is easy to work around this already if people really want an alternative.

No. It is easy to work around this already if people really want an alternative.

By postprocessing with tools like https://github.com/yoniholmes/grunt-text-replace ?


performance problems in coproduct derivations

as demonstrated in
https://github.com/fommil/shapeless/commit/2ea5adfbe315b79fa082019013935faff7fb035a
i.e. adding
/** Example AST with performance problems */
package sexp.big {
  sealed trait Base
  case object Foo01 extends Base
  case object Foo02 extends Base
  case object Foo03 extends Base
  ....
}
...
  // a performance bottleneck example
  import sexp.big._
  val slow = SexpConvert[Base]

to sexp.scala we are getting very slow compilation times. Looks quadratic with a big constant factor.
@rorygraves you would be interested in the compiler CPU profile. It seems to be spending all its time recalculating hashcodes. Grab my branch of shapeless and type examples/runMain shapeless.examples.SexpExamples to kick off the compiler. You'll have plenty of time to attach visualvm. Add yourkit to .jvmopts.

FYI, I've been unable to come up with a satisfactory workaround for this. If the compile time gets too high, the only thing that can be done is to fall back to manually written marshallers :sob:

I don't have time to look into it today, but one option it might be worth investigating is getting rid of the string singleton types as early as possible. In deriveInstance you could try mapping the representation type (G) from the form,
FieldType[K1, V1] :: FieldType{K1, V2] ... :: HNil

to,
(String, V1) :: (String, V2) ... :: HNil

and similarly for coproducts. That will eliminate all the singleton typed strings previously embedded in FieldType and I think is probably safe here: you're only relying on the labels as simple values in the rest of the derivation and not things which might be used to index a type.

I'm tracking this to https://github.com/typelevel/scala/blob/2.11.x/src/reflect/scala/reflect/internal/Types.scala#L1058
which means that the compiler must be creating an excessive number of UniqueType instances.

also MethodInvoke on SingletonTypeMacros is extremely slow

this is crazy, it looks like Object.hashCode is really slow. I wonder why that would be. I thought it was basically free.

Nothing is free if you call it enough times.
Sent from my iPhone

On 11 May 2015, at 17:23, Sam Halliday<EMAIL_ADDRESS>wrote:
this is crazy, it looks like Object.hashCode is really slow. I wonder why that would be. I thought it was basically free.
—
Reply to this email directly or view it on GitHub.

could be a ghost, some people report seeing similar high CPU on Object.hashCode in java land saying that it seems false http://stackoverflow.com/questions/3124676

@milessabin (this isn't exactly 100% typesafe, I need a few more restrictions) but I've pushed a change that uses String instead of FieldType. But I have no idea how to turn the LabelledGeneric into this representation, i.e. change deriveInstance. Any ideas? https://github.com/fommil/shapeless/commit/a848f7a13d8d9d1b0f8b57a913561cd3f16e95ca

@milessabin I managed to catch @alexarchambault on gitter and he helpfully recommended using ops.record.Fields and ops.union.Fields to get the (String, V) ::/:+: Rest representations. The only problem is that we can't work out how to pass the actual value of the String down the chain... we can't use the Witness for the K anymore. Any further ideas?

I inspected the macro expansion for quadratic patterns in generated code and found a few.
Based on that, here's an improvement to the performance:
https://github.com/milessabin/shapeless/compare/master...retronym:topic/coproduct-speed
Please feel free to adapt that into a fix. I haven't submitted it as a PR because I'm not sure how you'd like things arranged.
I think there is still a problem with the implicit expansion that invokes cpZipWithKeys, but I haven't attempted to solve that.

awesome! It is indeed faster, but I'm getting a runtime error in the sexp.scala for the simpler examples, so this changes the functional behaviour somehow.

both the "wibble" example and the "less trivial example" are failing with classcast exceptions.

Yep, I just noticed it doesn't pass tests yet.

Okay, I had an off-by-one error in unsafeMakeCoproduct. Just pushed a fix.

cool stuff!

wow! thank you thank you thank you!
Please cherry pick my commit #2ea5adf so that we have a permanent test for this regression and then lets gang up on @milessabin to release this

btw, takes 10 seconds now on my machine... down from killing the compiler process after 20 minutes of eating all my CPU cycles and getting nowhere.

What! Seriously! Awesome! Thanks @retronym!

@fommil can you turn this and your test into a PR that I can merge?

The trick is apparently to accuse the scala compiler of being slow in your talk then @retronym fixes it.
Got to remember that trick ;)

:-D

now I just need to wrap up the String one I mentioned into a script that I can blame on the compiler so @retronym  can fix it and I can take the rest of the year off :wink:

I can do that, unless @retronym does it first


Adding new light materials during runtime is unstable

Easy to reproduce using the decal demo with MeshPhongMaterial.

Issue has been pinpointed to the use light sources other than AmbientLight in conjuction with materials that receive light.

Fixed in 5ff5b22355f3ce45c46aea57978f59490a3dadce


beforeRouteUpdate is called after asyncData

Version
v2.12.2
Reproduction link
https://github.com/moltar/route-hooks#readme
Steps to reproduce
git clone git@github.com:moltar/route-hooks.git
cd route-hooks
npm i
npm run dev
open http://localhost:3000/foo/1

Open browser console
Click Next Page
Observe console.log output

What is expected ?
beforeRouteUpdate should be called before asyncData
What is actually happening?
beforeRouteUpdate is called after asyncData

This bug report is available on Nuxt community (#c10613)

Relevant code chunks:
asyncData is called here:
https://github.com/nuxt/nuxt.js/blob/479e3ffefb6cca8f3194cc4661d52be8e7b4f529/packages/vue-app/template/client.js#L432
Routing happens here:
https://github.com/nuxt/nuxt.js/blob/479e3ffefb6cca8f3194cc4661d52be8e7b4f529/packages/vue-app/template/client.js#L532

Should it though? Within asyncData you can still decide to redirect to another page, so it makes sense that it runs before the nav guards for the initially intended route are triggered.

To me asyncData is associated with loading of a new page. By that logic any un-loading should happen before that.
I think all other guards come after, but I am just talking about beforeRouteUpdate.
Here's the full router nav flow for reference: https://router.vuejs.org/guide/advanced/navigation-guards.html#the-full-navigation-resolution-flow

It’s not stale. Still relevant.

This bot is extremely annoying. The issue is not stale.


(Concept Inside) Addition to the Bank Tags plugin which gives the user the ability to add custom bank tabs that when clicked on, acts similarly to the search tag: option.

If I were to tag my farming run supplies with farming, when I search in the search box in the bank interface 'tag:farming' these items would show.

This is how the Bank Tags plugin works in its current form. I am suggesting that we add an option to mark one of those items that showed up in the 'tag:farming' search as the tab icon, this would show up as a right click option under 'edit tags' and would only show when using the search tag:farming option.
If I clicked on the ranarr seeds above and chose that as my tab icon, this would appear on the left of the bank interface:

Clicking this new tab on the left would streamline the process of searching for 'tag:farming' and would only show up if you right click an item after searching your tag and selecting that item as the tab icon. The reason why you should not automatically add a tab is that a user might have a lot of different tags for different tasks/fashionscape etc. and the number of tabs being there should be up to the user.
If you have more than two tabs, it would be preferred if you could swap them above or below each other, exactly how the osrs bank tabs work.

This looks really intelligent, and would hugely increase bank usability for a lot of people. Support!


bug: prevent autocomplete completing on pressing enter

plugin version: v0.69.1
Description
When using the editor and eg typing a variable name and then pressing enter if that happens to match anything in the autocomplete dialogue then the plugin will use that. (eg typing the variable y and pressing enter keeps on outputting type ). Generally for other languages in Intellij they work by using the autocomplete if Tab is pressed and ignoring it if enter it pressed.
This makes it very hard currently to work with any reasonably short variable names as the autocomplete keeps trying to interfere and change them.

I have the same behavior in intellij idea when I use java.
it's default behavior. you can change it in settings if you uncheck 'Auto-display code completion' in Editor|General|Code Completion

perfect , thanks for explaining.

On 23 Sep 2019, at 07:30, Hervé Giraud<EMAIL_ADDRESS>wrote:

I have the same behavior in intellij idea when I use java.
it's default behavior. you can change it in settings if you uncheck 'Auto-display code completion' in Editor|General|Code Completion
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Invalid Package: expected grommet but found grommet-udacity

I'm getting this error when I run -- npm run setup:
npm ERR! Darwin 16.0.0
npm ERR! argv "/Users/karlo/.nvm/versions/node/v6.6.0/bin/node" "/Users/karlo/.nvm/versions/node/v6.6.0/bin/npm" "install"
npm ERR! node v6.6.0
npm ERR! npm  v3.10.3
npm ERR! Invalid Package: expected grommet but found grommet-udacity

Thanks, this is a recent change that I made to my Grommet fork.  Grommet can be installed from npm if needed: npm install grommet --save and then remove the reference to the other grommet, or you can change the package name to grommet-udacity with a global find and replace in the components.


Add more helper methods in utils, use cases will be split into other PRs

I've been trying to move things out of package utils since catch-all packages are an anti pattern in go.

Avoid meaningless package names. Packages named util, common, or misc provide clients with no sense of what the package contains. This makes it harder for clients to use the package and makes it harder for maintainers to keep the package focused. Over time, they accumulate dependencies that can make compilation significantly and unnecessarily slower, especially in large programs. And since such package names are generic, they are more likely to collide with other packages imported by client code, forcing clients to invent names to distinguish them

https://go.dev/blog/package-names

Are these strictly to support encoding/decoding?
What about a package codec? 🤷

Are these strictly to support encoding/decoding? What about a package codec? 🤷

happy to, was trying to follow the convention when I put it in utils.


Add national_applicability check

Adds national_applicability check to EditionBase in the migration sync checks. Part of #2715

:+1:


Cannot install with Deno 1.4.x

There are some following errors when installing denofunc with deno 1.4.x.

TS1371 [ERROR]: This import is never used as a value and must use 'import type' because the 'importsNotUsedAsValues' is set to 'error'.

As long as I know, a workaround for this is to use --no-check flag on installing (need to update README.md) or to update dependencies, especially<EMAIL_ADDRESS>Could you please look into this?

close because denofunc works on deno 1.5.x or later.