[Feature]: XWorkspaces does not get urgent flag from XMonad

Checklist

[X] I have read the appropriate section in the contributing guidelines
[X] I believe this issue is a problem with polybar itself and not a misconfiguration on my part
[X] I have searched for other open and closed issues that may have already reported this problem
[X] I have checked the known issues page for this problem.
[X] I have followed the debugging guide to narrow down the problem to a minimal config.

Steps to reproduce

polybar -c config
wmctrl -r "qutebrowser" -b add,demands_attention

Minimal config
[bar/mybar]
modules-left = ewmh
bottom=true
background = #3f3f3f
foreground = #dcdccc
font-0 = fontawesome:pixelsize=12;3

[module/ewmh]
type = internal/xworkspaces
format = <label-state>
icon-0 = default;
icon-1 = chat;
icon-2 = coding;
icon-3 = internets;
icon-4 = todo;

label-active = %icon%
label-active-foreground = #5f7f5f
label-active-background = #3f3f3f

label-occupied = %icon%
label-occupied-foreground = #f5deb3

label-urgent = %icon%
label-urgent-foreground = #ff0000

label-empty = %icon%
label-monitor = %icon%

Polybar log
notice: Parsing config file: config
notice: Loading module 'ewmh' of type 'internal/xworkspaces'
notice: Loaded 1 modules
notice: Loaded font "fontawesome:pixelsize=12" (name=FontAwesome, offset=3, file=/usr/share/fonts/fontawesome/FontAwesome.otf)

Expected behavior
The icon for the workspace containing qutebrowser becomes red.
Actual behavior
The icon for the workspace containing qutebrowser keeps having "occupied" color.
Window Manager and Version
XMonad 0.17.0
Linux Distribution
Gentoo
Polybar version
polybar 3.7.1

Features: +alsa +curl +i3 -mpd -network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: RelWithDebInfo
Compiler: /usr/bin/x86_64-pc-linux-gnu-g++
Compiler flags: -march=native -mtune=native -O2 -fomit-frame-pointer -pipe -ggdb  -Wall -Wextra -Wpedantic -Wdeprecated-copy-dtor -Wsuggest-override
Linker flags: -Wl,-O1 -Wl,--as-needed -Wall -Wextra -Wpedantic -Wdeprecated-copy-dtor -Wsuggest-override  -Wall -Wextra -Wpedantic -Wdeprecated-copy-dtor -Wsuggest-override

Additional Context / Screenshots
The window in question does get the red frame as per XMonad config, so this appears to be unrelated to XMonad itself:
$ xprop -spy | grep -i demand
_NET_WM_STATE(ATOM) = _NET_WM_STATE_DEMANDS_ATTENTION

The xworkspaces module does not look at _NET_WM_STATE at all. It only uses the urgency bit (in WM_HINTS).
My WM (herbstluftwm) seems to set the urgency bit when _NET_WM_STATE_DEMANDS_ATTENTION is set, and yours probably doesn't you can confirm with the following command:
xprop -spy | grep -i urgency

If this is just about setting the urgency hint properly, you can try this:
xdotool search --name firefox set_window --urgency 1 %@

I wouldn't really consider this a bug, but more of a missing feature. PRs are of course welcome :)

Thanks a lot for thorough explanation Patrick! Your hunch was indeed right, XMonad does not set WM_HINTS when it comes to urgency. I'm too unfamiliar with polybar codebase, so I guess I'll just dance around getWMHints in XMonad config.

@patrick96  i have looked into the issue. I've been able to experiencing the same behavior with Polybar and XMonad regarding the urgency state not reflecting properly in the xworkspaces module.
It seems like adding a check for _NET_WM_STATE_DEMANDS_ATTENTION would really help with this, especially since it's clear that different window managers handle the urgency bit in their own ways.
i know c++ done some projects based on algorithm design and many more. since this is mine first time contributing its really helpful to know the community

@sahishnu111 Thanks for taking an interest. I think this is something that can be achieved even if it's your first time contributing to polybar. Some pointers:

The rebuild_urgent_hints in xworkspaces.cpp contains the logic for gathering the urgent state for each desktop based on its windows
There's already get_wm_state in ewmh.cpp where you can load _NET_WM_STATE.
You might need to add additional atom definitions in atoms.cpp

@patrick96 I understand how frustrating this can be, and I would like to help you resolve it.
I am currently testing the issue on my own system to reproduce the behavior.I’ll verify whether adjusting the Polybar config or updating to the latest version of Polybar/XMonad makes a difference.
In X11, windows set with the demands_attention flag should trigger a notification or event that Polybar can respond to.
When Polybar detects an urgent window (via the X11 event), it should update the label-urgent setting for the corresponding workspace.(I think so )??
If I find any solutions or workarounds, I’ll update you here.


Change the monitor "scheduler" from setTimeout to Croner + Queue

Croner:

Supports using cron expressions for the heartbeat interval.
Ensures the check accuracy based on the scheduled time.

Queue:

Allows configuring the concurrency number.

A higher number means more monitors can be checked simultaneously, but it may result in increased CPU usage and higher ping.
A lower number may cause some monitors to be delayed, but it reduces interference from other monitors. (Potential issue: the queue could become congested)

@louislam I want to work on a bigger issue/feature do you think i can pick this up or is there any other high priority project that needs help?

This PR-proposal would resolve these issues (could you add the following to the description?)
Resolves https://github.com/louislam/uptime-kuma/issues/1482
Resolves https://github.com/louislam/uptime-kuma/issues/2444
Resolves https://github.com/louislam/uptime-kuma/issues/2317 (even though this can be resolved via maintanance as well)

@CommanderStorm
It's a good feature. When is it going to start development & release?
do you have any ruff idea?

Things in open source get done when they get done.
Since we are all volunteering here, we don't give out timelines.
Chroner is a node library.


[Enhancement]: Adding two modes options light/dark mode

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
Hello, Kunj. I am a student at Nirma University from Kapadwanj. I am starting my contribution journey in GSSoC '24. I have created many web development projects and now want to contribute to various projects.
Describe the solution you'd like
I'll add a mode indicator picture in the top right corner, using JavaScript for dynamic updates and HTML/CSS for fixed positioning, enhancing user experience by providing clear mode information.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @zalabhavy !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @zalabhavy ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


OLD - chore : fix failed unit tests

Fix failed 4 unit tests
Remove dedicated header load test code in HttpResponseMessage constructor

Only first commit will be included in this PR.
I pull another PR for the feature

It would be great to separate the test fix from the feature work. I stills see it all together here.

I used the same github branch.
I push two PR with fix tests and feature this day.

Closedfor a new PR #177


[KYUUBI #1056] [BUG] Log error if no current spark sql engine is created.

Issue:  https://github.com/apache/incubator-kyuubi/issues/1056
Why are the changes needed?
Log an error message if no current spark sql engine is created.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #1423 (f00a5a9) into master (21cc217) will decrease coverage by 0.01%.
The diff coverage is 65.00%.

:exclamation: Current head f00a5a9 differs from pull request most recent head a8d0443. Consider uploading reports for the commit a8d0443 to get more accurate results

@@             Coverage Diff              @@
##             master    #1423      +/-   ##
============================================
- Coverage     57.36%   57.35%   -0.02%
  Complexity      154      154
============================================
  Files           224      224
  Lines         11449    11451       +2
  Branches       1408     1410       +2
============================================
  Hits           6568     6568
- Misses         4338     4339       +1
- Partials        543      544       +1

Impacted Files
Coverage Δ

...rg/apache/kyuubi/engine/spark/SparkSQLEngine.scala
63.63% <65.00%> (-1.70%)
: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 21cc217...a8d0443. Read the comment docs.


Remove certificate already exists warning

Do you want to request a feature or report a bug?

Bug

What did you do?

Repeated warning message when a new Ingress object is created with secret name that already exists
What did you expect to see?
Avoid repeated warnings as it floods the logs
What did you see instead?
time="2019-10-19T03:12:01Z" level=warning msg="Skipping addition of certificate for domain(s) \"mydomain.com,*.mydomain.com\", to EntryPoint default, as it already exists for this Entrypoint.
time="2019-10-19T03:12:02Z" level=warning msg="Skipping addition of certificate for domain(s) \"mydomain.com,*.mydomain.com\", to EntryPoint default, as it already exists for this Entrypoint.

Output of traefik version: (What version of Traefik are you using?)
2.0.2

time="2019-10-19T03:12:01Z" level=warning msg="Skipping addition of certificate for domain(s) \"mydomain.com,*.mydomain.com\", to EntryPoint default, as it already exists for this Entrypoint.
time="2019-10-19T03:12:02Z" level=warning msg="Skipping addition of certificate for domain(s) \"mydomain.com,*.mydomain.com\", to EntryPoint default, as it already exists for this Entrypoint.

What is your environment & configuration (arguments, toml, provider, platform, ...)?
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: my-web-ui
  namespace: default
  annotations:
    kubernetes.io/ingress.class: traefik
spec:
  tls:
    - hosts:
        - "*.mydomain.com"
      secretName: existing-system-cert
  rules:
    - host: web.mydomain.com

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)

Duplicate of #5622


Make "share to menu" entries optional

EDIT:
Is it Android or K9 behaviour that inserts contacts into Android "share to" menu? If it is k9:
It can be annoying and also a waste of screen space having those contacts appear always whenever I want to share something, which I do very frequently.
Please make share menu entries optional, thank you.
K-9 Mail version:
latest on f-droid as of today
Android 8.1

Nevermind, I just removed email adresses from contacts.


Add wallet functions

This PR moves the address generation and wallet formatting logic out of blockstack-browser and into a BlockstackWallet class. The documented interface for this class defines the following functions:
class BlockstackWallet {
  static fromBase58(keychain: string): BlockstackWallet
  static fromSeedBuffer(seed: Buffer): BlockstackWallet
  getIdentitySalt(): string
  getBitcoinAddress(addressIndex: number): string
  getBitcoinPrivateKey(addressIndex: number): string
  getBitcoinPublicKeychain(): string
  getIdentityPublicKeychain(): string
  static getAddressFromBitcoinKeychain(keychainBase58: string, addressIndex: number,
                                       chainType: string = EXTERNAL_ADDRESS): string
  static getAppPrivateKey(appsNodeKey: string, salt: string, appDomain: string): string
  static getLegacyAppPrivateKey(appsNodeKey: string, salt: string, appDomain: string): string
  getIdentityKeyPair(addressIndex: number, alwaysUncompressed: ?boolean = false): IdentityKeyPair
}

This interface was designed to match the interface currently being used in the browser. We could instead try to use a slightly more consistent interface, though that would necessitate more changes in the browser.
~You can test this in the browser, by linking this branch: https://github.com/blockstack/blockstack-browser/tree/feature/refactor-wallet-functions against it, and using the browser as normal.~
Test this using the branch in this browser PR: https://github.com/blockstack/blockstack-browser/pull/1496
There are also included unit tests (unitTestsWallet.js) which confirm that it generates the same data as the current version of the browser.
The getLegacyAppPrivateKey function returns an app private key based on the single hash code.
getAppPrivateKey derives a path based on all of the bytes from the sha256 hash. This slices off three bytes at a time from the hash -- resulting in a path with 11 nested children.

I'm going to add support for the longer derivation paths to this PR.

Just noticed you marked this as a work in progress. Assume the test failure is a result of the key derivation changes that my not be complete. Will hold off on this until I get the go ahead from @kantai that it's ready to review.

Yep, the test failure is due to the key derivation path still being in decision limbo:
See the discussion at the bottom here:
https://github.com/blockstack/blockstack-browser/issues/1367
This branch currently has both of the proposed paths in that issue implemented -- once we get consensus on which path we want to use, I'll update the tests and delete the other path.

@kantai where are we on this? From the linked thread above it looks like we were benchmarking the longer derivation path. I recall a discussion from somewhere deciding that we would simply go with the longer derivation path.
Is this something that can make it in to this sprint's release?

Yeah -- I think we concluded that we should (a) strongly prefer to use the longer path, unless (b) the benchmarking was really bad. The benchmarks seemed fine to me. Will update this PR to include just the longer derivation path, and then we should be able to merge this in.

Sounds good. We'll also have to update the corresponding browser branch. Once that's ready, I'll test it and if all is good we can merge! Thanks for the quick commits!

I opened a PR on browser updated to use the correct app-key derivation paths from this file.
That PR doesn't refactor out the other wallet functions, though it certainly could.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: wbobeirne:white_check_mark: kantai:x: larrysalibraYou have signed the CLA already but the status is still pending? Let us recheck it.

@wbobeirne It would be awesome if you could take another look at this and try it out with the related browser PR
https://github.com/blockstack/blockstack-browser/pull/1496 when you have a chance!

Looks like a lot broke with the switch from HDNode -> BIP32. It's good you marked all of this as private APIs, because I'd really like to see us come up with a better interface than the amount of mucking around that browser will have to do with these primitives. Ideally we have a very developer friendly set of functions that do things like get addresses and such.
I'll try to fix up https://github.com/blockstack/blockstack-browser/pull/1496 without making any more changes to this PR though, in the mean time.

I've been grinding through changes between bitcoinjs-lib 3 and 4 in blockstack-browser for about an hour now, and I'm really of the mind that the approach of moving the account-utils functions as they are is a useless pursuit.

I'm okay with that, but it is still important that we address https://github.com/blockstack/blockstack-browser/issues/1367

Maybe the play here is that we roll back all of my work and commit as-is. That doesn't bother me at all, my only issue is I think some of the changes Larry made to bring this up to speed with bitcoinjs-lib are still incompatible with browser. But I think if we're going to clean all of that up, it'd be better with out my added changes.


Apex Legends: Crash within 5 minutes of entering game

Apex Legends crashes within 5 minutes once entering a game. Logs give err:   DxvkSubmissionQueue: Failed to sync fence: VK_ERROR_DEVICE_LOST err:   DxvkDevice: waitForIdle: Operation failed.
Running sudo dmesg after crash gives NVRM: Xid (PCI:0000:01:00): 31, pid=13799, name=R5Apex.exe, Ch 00000063, intr 10000000. MMU Fault: ENGINE GRAPHICS GPCCLIENT_T1_1 faulted @ 0x0_bb3ad000. Fault is of type FAULT_PDE ACCESS_TYPE_READ
System information

GPU: GTX 1060 6G
Driver: NVIDIA 520.56.06
Wine version: Proton 7.0-4
DXVK version: 1.10.3

Log files

d3d11.log: r5apex_d3d11.log
dxgi.log: r5apex_dxgi.log
apex log : steam-1172470.log

@git-iso Could you also give proton experimental a spin just for good measure? 🙂

@Blisto91 Same crash within a few minutes of entering a game.
steam-1172470.log

Probably some sort of GPU driver/Pascal GPU combination problem or similar. Have you checked if you maybe running out of VRAM on your card?
Works without issues here with the 520.56.06 driver and RTX 3080. Can play hours without issues.

@xpander69 GPU memory hovers around 20%, GPU usage around 98%

@xpander69 GPU memory hovers around 20%, GPU usage around 98%
Does it also crash like that with older nvidia drivers or vulkan betas?. It surely sounds more like some sort of hardware issue, OC going wrong. RAM being broken or whatnot. There sould be alot of pascal gpu users and apex players around. Havent heard any other talk about having this issue.
I remember i had that Xid31 issue on my 1080Ti with Dirt Rally 2.0  way back also, but it turned out to be my unstable RAM OC. Not trying to say its exactly same thing for you, but yeah...

@xpander69 I only had occasional crashes with previous NVIDIA drivers but 520.56.06 seems to always crash. I don't have any OC on my hardware and other Vulkan games seem fine. If I had to guess it has something to do with the shaders, I'll try to experiment and see if I can fix it.

After much experimentation and testing withDXVK_HUD I was able to stop my crashing. In the first few minutes of loading a game, there seems to be a stutter in the frametime that sometimes goes as high as 1200. By limiting my fps  with +fps_max 60 I was able to stop the stutter from instantly crashing my game. Once in a game, if my GPU usage goes too high (above 70%) my game crashes, so keeping the framelimiter on is necessary. I also let the shaders compile in the menu fully instead of in game.
If I had to guess it has something to do with shader compilation, and since it only started after the latest NVIDIA driver it may be that VK_EXT_graphics_pipeline_library that was introduced in it makes the shader process too heavy on my GPU.

If I had to guess it has something to do with shader compilation, and since it only started after the latest NVIDIA driver it may be that VK_EXT_graphics_pipeline_library that was introduced in it makes the shader process too heavy on my GPU.

Shader compilation happens entirely on the CPU and if the generated shaders crash the GPU despite being valid, that would be a major driver bug.

@git-iso Can you try again with the new Nvidia driver and latest dxvk? :slightly_smiling_face:

i have never had this happen to me, since 520 and onwards.

@git-iso Does it still crash on newer (520+) drivers?

Running DXVK 2.0 on Proton Experimental with driver 520.56.06 still  quickly crashes above 60 fps in-game. It can also occasionally crash with 60 fps limit if GPU usage goes too high.

if GPU usage goes too high.

Make sure you're not running an unstable OC.

@git-iso Friendly ping

@Blisto91 I upgraded to a 3060 TI recently and it seems fine, maybe Pascal GPU wasn't working well. I'll close the thread.

Coolio :+1:
Good luck with your new GPU :slightly_smiling_face:


Reading rutine for rinex nav is skipping the first epoch of ephemerides

There's a bug in the reading routine for RINEX navigation files. It's skipping the first epoch with ephemerides.

Fixed in latest release


Private tasks for "dependsOn"

I, like many people, use vscode's task and launch functionality to entirely abstract away the compilation process of my programs. For complex programs like servers this can get quite messy and I make liberal use of the "dependsOn" feature to chain different compilation programs together.
However, this can get quite messy when I or my team go to run a task and tens of, what should be, intermediate tasks pop up before the actual public important ones.
It would be cool if there was a "private" attribute that can be set such that it removes it from the task menu but still allows it to take part in a "dependsOn".
..This might be a duplicate

https://github.com/microsoft/vscode/issues/149095#issuecomment-1195711551


Is possible to use mongoose-autopopulate?

HI there
Is possible to use mongoose-autopopulate to auto populate the relationships? If not, exist other way to auto populate relationships?

You can define your schema by just redefining the
static get schema () {
    return {

    }
  }

https://github.com/juampi92/adonis-mongoose-model/blob/master/instructions.md#defining-schema


Move larger binary files to LFS

What has to be done
We have to

enable git LFS for our repo
transfer all the larger files (PCB stuff, in particular) to LFS
scrub repo of the large files so they are not in repo proper
document this thoroughly and make it easy for non-software-nerds to use, i.e. electrical and hardware engineers. This last point is super important!

How do you know it has to be done
Repo is taking too long to clone. It is now nearing 2GB.
We already knew this was going to be a problem before Covent.
Curtis has negotiated a free LFS package from github for us as a non-profit, so we should take advantage of it.
Code pointers
https://git-lfs.github.com/
https://docs.github.com/en/github/managing-large-files/configuring-git-large-file-storage
Don't forget

[x] Put the issue in a milestone
[x] Assign it to one of the projects
[x] Add appropriate labels
[x] Mention dependent/blocked issues if any

What is done:

Most PCB and CAD files have been migrated.
Entire repo history has been rewritten to reflect that.
The github wiki pages has been updated with instructions for how to work with LFS.
Cloning appears to be faster, and it will be also possible to clone without LFS. Let's see if CI speeds up.

What remains:

The non-LFS part of the repo is still about 700MB
It appears there are some JPEGs upwards of 5MB, and given that we have a lot of hardware assembly instructions and renderings, they might be adding up. Might need to do another pass.

The meh:

Rewriting git history has invalidated some PRs, but nothing is lost that cannot be fixed with an old-fashioned rebase.

This is a list of issues and PRs were force-closed during the migration. Checklist to mark resolved. Resolved can mean

if issue, repeopened
if PR, branch rebased and new PR made

[ ] #845
[ ] #769
[ ] #763
[ ] #515
[ ] #185
[ ] #439
[ ] #117

If PR has to be reopened, please tag the old PR in the new one.

The procedure for migrating repo to LFS entails first fetching and pulling all branches from remote with
git branch -r | grep -v '\->' | while read remote; do git branch --track "${remote#origin/}" "$remote"; done
git fetch --all
git pull --all

Then if, for example, as I am about to do, you want to move all jpeg files to LFS you would:
git lfs migrate import --everything --include="*.jpeg,*.jpg"

This will rewrite the repo history from the beginning, and now you will have to do a force push:
git push -f

Remember to unblock master from being committed to and also enable force push on master in the repo settings.

After moving JPEGs now total non-LFS repo is about 800MB. Still hefty. Looks like the most likely culprit is data files, some of which ware 6MB. They are text files, so they do not belong in LFS. However, they are not really anything that deserved versioning, just archiving. They should likely not be text files at all, but in fact binary files, which would be orders of magnitude smaller. Likely candidate format would be something like the self-descriptive hdf5 (hierarchical data format) used by science facilities.

Two chores remain:

configure CI to clone without LFS
clean up PCB directory

Answering my own questions.

CI does not need any changes. It clones only the pertinent branch and not the whole repo, which is fast anyways
Cleaning up PCB directory is a separate ticket #873

Looks like png files might have to also be migrated to LFS.

On this day, performed one more migration for the following binary types:

png
adicht
stl
dxf

On this day, adding *.idw

On this day, adding *.idw

Additional casualties of this round:
#948
#978
#936
#945
#904
#899
#973

Additional casualties of this round:
#948
#978
#936
#945
#904
#899
#973


Embedding Static

I thing the way this code implements Static Embedding is wrong.  Am I right?
This code, uses x=variable(x) when wants to make the embedding static, while it should be something like: self.embed.weight.requires_grad=False

Using Variable also works. You can check by yourself.

It seems that the word embedding are kept static during training.
How to make the embedding changeable in backpropagation?


Color differences between first and last LED

I'm assuming this is a hardware issue but I'm wondering if anyone has come across this before.
Essentially, the further around the LED strip you go, the more they take on a different color.  I've attached a photo below showing the issue.  This is using a fresh config file, and I'm connected using the IOS remote with the color set to pure white, 100% brightness.  Other colors aren't affected anywhere near as much, but if I really look hard I can spot a subtle difference.
(it's difficult to capture with a camera, but really noticeable by eye).
Everything (Pi, LEDs (APA102), HDMI splitter, HDMI>AV etc) is running of a 5V 20A PSU, so that's well within spec.  I'm actually using a USB cable to drive the LEDs.  I've got the filter PCB connected to a USB header on the back of my box and the LED strip is soldered to a USB port (sugru'd to the back of my TV).  Could it possibly be a cable issue?  The USB is wired up so that the outer pins are +V and GND with the inner pins handling data.  Is the wire in a USB cable a thick enough gauge to drive the LEDs properly?
Reducing the brightness to somewhere around 30%-40% appears to solve the issue, but I'd like to figure out what's actually causing it.
I'd not actually noticed this until tonight after installing the beta and attempting to run through the color calibration guide (which for the record I can't get to work, but that's probably for another topic).

Reducing the brightness to somewhere around 30%-40% appears to solve the issue, but I'd like to figure out what's actually causing it.

It looks like you just input the voltage at the beginning, also attach +5v and gnd at least at the end.
If you still have this problem input again at the middle of your tv at the top.

Ok, it's late here now so I'll give this a go tomorrow.  Thanks for the advice.

Powering from both ends has made a massive difference.  I still think there's an ever-so-slight color variation, but it's nowhere near as noticeable.

Yep - try using 600 LEDs without extra feeds.
Not pretty !

To verify the voltage on both end and even at the middle use a multimeter to measure your voltage.
This is the best way to verify a voltage dif

the "best" solution is to power from your psu after every 50 led
how many leds do you have?

@brindosch I'll get the multimeter out at some point and do some testing.
@tpmodding Really?  Is that from personal experience, or do you have a data sheet on APA102 LEDs?  I'm running close to 180 of them by the way.

only personal experience, didnt read it in any data sheet... than you can be sure every led has engouh power... :) if you look at your last pic, you can see that the right side isnt as shiny as left side :)

If I use ws2812's as an example, each led can draw 60mA. Not only is that a lot of current when you grab 100 of them, the copper traces on the strips are pretty small by comparison.
I had 600 running for xmas outside and needed an extra power lead every 150.
My tv has about 100 and it could do with having power added at the far end.
In case you're wondering about the color change - blue LEDs have a higher forward voltage and so suffer from lower strip voltage first (then green, then red)

Wow, everything is said - the amount of voltage controls the brightness - your indication to tune this to your liking. :+1:

Soo, exactly the same problem hear yesterday... my leds (190s) looked exaclty like sgtriggs.
After I added a second power connect at the end of the strips the problem was gone....

This is part of the FAQ now
/closed


Update Professionalism.md with handling criticism and feedback section

This section will provide specific practical advice on how to handle receiving criticism and feedback from a co-worker, client, senior or boss so that they leave with a more positive impression of you.

Made changes by rewording various sentences according to the feedback Andy provided. Thank you, Andy!

Small nit I just realized: Perhaps we should have your topic added to the README list as well


RuboCop changes.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew tests with your changes locally?

Further RuboCop fixes. Also updated the configuration to include more of Cask's rules.

NO.

:-1:

@zmwangx @ilovezfs Those comments are extremely unhelpful. Please provide reasons for disagreement or stay silent. It's bad enough when comments like this come from users but I expect better from our maintainers.

I suspect if the prior breaking changes are fixed first, more useful reviews may follow.

I suspect if the prior breaking changes are fixed first, more useful reviews may follow.

I suspect if someone had created an issue or, dare I say it, a pull-request then the prior breaking changes would have been fixed already.

I suspect if someone had created an issue or, dare I say it, a pull-request then the prior breaking changes would have been fixed already.

Already suggested a diff on slack hours ago and explained why I didn't open a PR for it. But sure.

The multi-space alignments are so laughable I expect anyone who actually review the changes to immediately roll their eyes.
HOMEBREW_INTERNAL_COMMAND_ALIASES = {
  "ls"          => "list",
  "homepage"    => "home",
  "-S"          => "search",
  "up"          => "update",
  "ln"          => "link",
  "instal"      => "install", # gem does the same
  "rm"          => "uninstall",
  "remove"      => "uninstall",
  "configure"   => "diy",
  "abv"         => "info",
  "dr"          => "doctor",
  "--repo"      => "--repository",
  "environment" => "--env",
  "--config"    => "config",
}.freeze

      SYMBOLS = {
        sierra:        "10.12",
        el_capitan:    "10.11",
        yosemite:      "10.10",
        mavericks:     "10.9",
        mountain_lion: "10.8",
        lion:          "10.7",
        snow_leopard:  "10.6",
        leopard:       "10.5",
        tiger:         "10.4",
      }.freeze

Now, to add or remove one key, you potentially need to manually realign all of the values. Seriously?

You may not agree with the point but they are not pointless. They are investment in a consistent style which makes it easier to automatically review contributed PRs and lets people know what our style is rather than it being arbitrary.

More importantly, who decided that we should follow a "consistent style" like
   Style/AlignHash:
   EnforcedHashRocketStyle: table
   EnforcedColonStyle: table
   Exclude:
     - 'Taps/**/*'

Did anyone ask for this (sorry, maybe I missed the discussion)? Or is this just @reitermarkus's personal preference somehow?
And if I didn't call it out (although I called it out in an unhelpful way, at least in this thread, for which I apologize) this would have been merged very soon like every other of these RuboCop PRs, and suddenly the entire team had to conform to some random style decisions made by @reitermarkus? Not to mention some claimed style changes are not only stylistic? Rather than hastily merging these after getting one approval, shouldn't we need the consensus of the team before moving forward on things like multi-space alignments?

Did anyone ask for this (sorry, maybe I missed the discussion)? Or is this just @reitermarkus's personal preference somehow?

I am trying to merge Homebrew's style with Cask's, so naturally I'm trying to incorporate some of Cask's rules into Homebrew's style, and vice-versa, and not completely eliminate one of the them.
Also, I would argue that having an ugly diff is better than having hard-to read code.

The multi-space alignments are so laughable I expect anyone who actually review the changes to immediately roll their eyes.

Hyperbole does not help. Plenty of Ruby projects use this style. I also agree we should not take this particular style change (my review comment from earlier got eaten, somehow) as I also agree that requiring the indentation to be consistent causes diff-inflation.

Did anyone ask for this (sorry, maybe I missed the discussion)? Or is this just @reitermarkus's personal preference somehow?

Even if it is his personal preference the correct way is to disagree with that preference and explain why rather than to attack either the person or the higher-level project (getting our Ruby style to be consistent with Rubocop).

Rather than hastily merging these after getting one approval, shouldn't we need the consensus of the team before moving forward on things like multi-space alignments?

We don't need consensus for things that can be trivially changed back to how they were. I'm happy to get more views on these but previously I've been the only person providing any review, pre or post merge, on these Homebrew/brew PRs. If you'd like to be more involved and would like us to take more time before merging I'm happy to have your review and thoughts on these and let me know what an acceptable amount of time is to give you to review before merging.

Already suggested a diff on slack hours ago and explained why I didn't open a PR for it. But sure.

Sorry about that. This is my fault as I've not been explicit about it before but: there's too much chat in Slack and it's not publicly available so it's better to assume messages there have not been read and to cross-post thoughts into issues/PRs/etc.

I do think we need to be careful with these mass changes, given our test coverage is still a work in progress. Introducing real breakage through style fixes certainly isn't a desired side-effect, and if tests aren't catching that the tests obviously aren't comprehensive enough.
I don't think any comment here or in https://github.com/Homebrew/brew/pull/1125 were intending to set fire to @reitermarkus personally. The writing style was a little blunt, but I think it mostly stemmed from frustration that style changes are introducing breakage, style changes which to be fair haven't really been explained or discussed all that much, rather than setting out to "get" anyone. Almost the entire team, myself very much included, are guilty of using frustrated voices at times. We can all do better there.
It would be useful if the changes were explained a little better, so people don't have to go hunting around Rubocop upstream for examples on the before/after.

style changes are introducing breakage, style changes which to be fair haven't really been explained or discussed all that much

Just to be more explicit about these: we have a .rubocop_todo.yml file and Cask has it's own .rubocop.yml file and they should both be worked on until we have no TODOs and a single, shared file with shared style between Homebrew, Cask and (where appropriate and not excluded) Formulae and Taps. This is useful because it makes it easier to contribute and read code that's using a single style and almost all Rubocop checks are to make Ruby code more readable and thus easier to work with. Readability is the foundation on which security, performance, correctness etc. are built.
There will be bugs along the way and the best ways to combat this are for me and/or @reitermarkus to not merge these PRs too soon before a tag or us going out/to bed to that we can jump to fix issues when needed.


Mud data grid check box reset not working

Bug type
Component
Component name
MudDataGrid
What happened?
I am using MudDataGrid  with multi selection. When I select the row and open a Dialog process everything in it and when I close the Dialog the the checkbox is now unchecked which is expected but actually while it seems to be unchecked the values of selected rows are not changing.
Expected behavior
when the Dilog closes the checkbox should get unchecked and as well as the selected values should also be reset
Reproduction link
https://try.mudblazor.com/snippet/wumRPQYvwexEeRVz
Reproduction steps

select 2 or more elements.
open the dialog with button named "Open Simple Dialog".
close the dialog.
you will notice there two buttons in reproduction link one is named "Open Simple Dialog" and "button" the button named button will get disabled if there are more than one selected elements.
but after closing the dialog although the checkboxes are now unchecked still the button is disabled .

Relevant log output
No response
Version (bug)
6.11.1
Version (working)
No response
What browsers are you seeing the problem on?
Firefox, Chrome, Microsoft Edge, Safari
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Looks like there is a bug in your code. When you select two elements and deselect them, the button still stays deactivated. You never reset isDialogButtonDisabled.
In SelectedItemsChanged do either isDialogButtonDisabled = selectedElements.Count > 0 or declare the variable directly as read only property (isDialogButtonDisabled => selectedElements.Count > 0)

But wait, it gets worse, you have declared two list variables: Elements and selectedElements. In your code, with every changed element, you will duplicated the current selection!
Add @selectedElements.Count as your button label and watch your count exploding.
Elements gets arelady updated by MudDataGrid, why do you copy manually the elements?

Please ping a team member if you still believe it's a bug and we'll re-open the issue.


Review bastion shutdown and startup routines

A call was raised to ask for a bastion to be started up and when I checked (just after 20:00) it was in a terminated state. It was needed because there was an issue which required bastion access to resolve.
After checking the terraform code (running a plan etc) I found the service had been rebuilt and would be available within a few minutes.
We need to check the stop/start process to ensure the stop only takes place overnight (not at 20:00) and make sure the user community is aware of when this will take place.

Guidance has now been added which explains how to restart instances at night so closing this - https://user-guide.modernisation-platform.service.justice.gov.uk/concepts/networking/instance-access-and-bastions.html#accessing-instances


If attribute is named "data", an additional attribute called "data-" is added.  Bug?

This is odd, if I have an attribute called "data", two items are added to the list of choices: "data" and "data-".  This is the only attribute which I've encountered any issues (our schema has > 1000 attribute names).

You can replicate using:
{
"version": 1,
"tags": [{
"name": "my-button",
"description": "My button",
"attributes": [{
"name": "data",
"description": "My button type",
"values": [
{ "name": "alert-one" },
{ "name": "alert-two" }
]
}]
}]}
Then activate the above as you will, and type:
<my-button |
You will see list of choices including:
data
data-
This seems like a bug to me.

This is as-designed. VS Code prompts you for data- as a completion item. This happens even if you don't have any custom data.


Update configuring-two-factor-authentication.md with 1 line: the step…

… to click 'Done' button to complete process
I found when enabling 2FA on my account that I could go through the whole process (TOTP app, download recovery codes, etc) and even get the confirmation screen (with confetti), however, 2FA was not enabled. My conclusion (when going through the process a 2nd time) was that it was necessary to click 'Done' button on the confirmation/confetti page. I don't think i bothered to do this the first time, as it wasn't obvious that it was necessary (thought it was just to go back to home screen or something of convenience). Clicked 'Done' the second time, got a confirmation email, and 2FA is successfully enabled.  This is the quickest 'help' I could think of for others, as changing the process to not depend on a 'Done' button is a bigger task. Feel free to edit or put this in another .md, but I think it needs to be noted somewhere for users.

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@JenniferWhelan Thanks so much for opening a PR! I'll get this triaged for review ✨

@JenniferWhelan Hello! 👋 Thank you for your patience while our team reviewed! We will need to edit the reusable containing that set of steps to include clicking the "Done" button 💛 I've filed an issue internally to have this addressed, and will close out this issue. Thank you for raising a flag on this! ✨


Custom sources for green-deps

This merge proposes to add support for custom sources for core dependencies. Also, the GIT_TAG for each dependency can be different from GREEN_RELEASE.
The changes do not break existing functionality, and are helpful in debugging.

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

Project coverage is 95.00%. Comparing base (887c865) to head (1e78b6b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage   95.00%   95.00%
=======================================
  Files          20       20
  Lines        1542     1542
=======================================
  Hits         1465     1465
  Misses         77       77

Flag
Coverage Δ

unittests
95.00% <ø> (ø)

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

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

Rationale is to make it easier to debug the dependencies for the MBPT code.
The green-mbpt code has several dependencies. At the moment, all the dependencies are obtained using FetchContent_Declare command in CMake, with a hardwired GreenPhys github URL.
The proposed change will make it easier to provide custom local paths and GIT_TAG separately for each dependency. If no such arguments are provided, we keep the default functionality, i.e., look for the dependency in the GreenPhys github organization.

The green-mbpt code has several dependencies. At the moment, all the dependencies are obtained using FetchContent_Declare command in CMake, with a hardwired GreenPhys github URL.
The proposed change will make it easier to provide custom local paths and GIT_TAG separately for each dependency. If no such arguments are provided, we keep the default functionality, i.e., look for the dependency in the GreenPhys github organization.

@iskakoff quick reminder to get active here and approve or request changes.


driver: modem: Fix mux device name comparison

Correct always false condition used to test uart muxing usage
CONFIG_UART_MUX_DEVICE_NAME is used as a prefix for the uart muxes's name
pointer comparison will always return false
Fixes #39774

@rerickson1 should this be backported to v2.7?

Only if #41078 is also backported


Does not display PRs from submodules

Issue Type: Bug

Clone a repository with a submodule, recursively.
Open the root repo. Open pull requests in the submodule do not appear in the list. (they do appear when opening the submodule's folder directly)

Extension version: 0.1.2
VS Code version: Code 1.27.1 (5944e81f3c46a3938a82c701f96d7a59b074cfdc, 2018-09-06T09:21:18.328Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3770S CPU @ 3.10GHz (8 x 3093)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.89GB (5.80GB free)

Screen Reader
no

VM
0%

I have the same scenario. It display's PR's from my main root repo. The root repo has a bunch of submodules and I never see any PR's from these.

This should be fixed with the change for https://github.com/microsoft/vscode-pull-request-github/issues/1936, submodules should now show up as a separate entry in the tree. Here I added pullrequest-demo as a submodule in this repo:


[Xaml] Unable to resolve symbol 'ValueConverters'

Release Type: GitHub
Version: master
https://github.com/stride3d/stride/blob/c6fd21b1bc974fd3fed0f6089df31bb356b3e574/sources/presentation/Stride.Core.Presentation.Graph/Themes/generic.xaml#L8
Assembly should be specified or namespace should be removed as not used.

Fixed through #1822


panicks test blank_node::tests::test_size_and_alignment on i386

Building v0.4.0-alpha.7 on Debian build-daemons panicks in checking the testsuite on i386 architecture:
thread 'blank_node::tests::test_size_and_alignment' panicked at lib/oxrdf/src/blank_node.rs:412:9:
assertion `left == right` failed
  left: 52
 right: 56
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: core::panicking::assert_failed_inner
   3: core::panicking::assert_failed
   4: core::ops::function::FnOnce::call_once
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Should be fixed by #918

Yes, I can confirm that that fix works: I cherry-picked the (identical) earlier commit ba32a0d and status page for oxigraph indicates that it might finally enter Debian testing within a few days.

Whoops, I thought this was about #913 - I will know in few hours how this one fares (but seems so simple that even I can follow what the patch does, so looks promising)

Yes, succesfully builds also for 32-bit arches. Thanks!


about Uni-Fold bibtex

Can you post a bibtex to cite?

Thank you for your interest in Uni-Fold. Currently we have not published papers about Uni-Fold, so perhaps you may acknowledge Uni-Fold by citing the repo's URL.

We have now added the citation in Readme. Refer to our preprint for more details :)


[VS Code] Starting collaboration session: Already connected to a workspace.

Error:
Starting collaboration session: Already connected to a workspace.
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.470

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.34.0

platformName
MacOS

platformVersion
18.6.0

We are aware of the issue and working on a fix. Closing as a duplicate of #2359. Thanks for using LiveShare!

We published a new release of the extension to the marketplace which contains a fix for this issue. The version number is 1.0.471. Please update your extension in Visual Studio Code. Thanks everyone for the feedback!


Fix multi-threading crash

Resolves #188

(Hopefully) fixes multi-threading crash in PalindromeProducts tests.
Simplify the method for reversing a String
Minor clean-up

@masters3d I’ve run the test many times without seeing the crash again, but it’s pretty difficult to prove it could never happen, so let me know if you do still see it.

I think we can avoid the current data race by creating a read/write thread for the  results array. See comments.

More context. https://twitter.com/cocoaphony/status/793885415237361664


Dispatch manage --enable-bootstrap-mode doesn't encode public key

Subcommand manage doesn't encode public key when enabling bootstrap mode. It will cause jwt validation failure on identity-manager server.

[x] Bug
[ ] Feature
[ ] Enhancement

Complexity

[x] Low - Simple enhancement or bug fix, no architectural changes or refactoring
[ ] Medium - Change requires some thought, but is relatively isolated
[ ] High - Significant architectural change or large refactor

Impact

[ ] Low - Annoyance, but does not impact business or functionality
[x] Medium - Issue can be worked around, but is causing pain
[ ] High - Blocker

Your Environment

Dispatch CLI version (or git commit):
Dispatch Chart version (or image[s] tag):
Operating System and version:
Kubernetes version (and distribution):
Etc (any other useful environmental information):

fixed in #384


fix: false positives while reading issue/PR numbers

Hi @ldelossa, I've been using your plugin for a couple of days.
And I've noticed I get a Failed to fetch issue. when my cursor is
on top of a comment.

The expression used to match the number was using an asterisk, which
matches zero or more, instead of one or more.
We're also moving away from scanning the line in favor of expanding the
current WORD under the cursor, and extracting the number with lua,
instead of the internal vim functions.

LGTM! Thanks for contribution

@davidmh I'm noticing with your change when the only string in the message body is a reference to an issue (.e.g #53) it does this does not work.

#53

You can use the above message as a demonstration.
Open gh.nvim and do a "GHOpenIssue 55" and notice how no preview is displayed for the above reference. since its the only string in the message.
Its most likely parsing the separator symbols at the start. Care to fix this small bug?

Its most likely parsing the separator symbols at the start. Care to fix this small bug?

That sounds about right. I'll try to look into it later today or tomorrow.


Nuxt-Typescript 'npm run build --target static' command runs in ~/static directory context

Versions

nuxt: v2.13.0 Integrated with Typescript Repo
node: v12.16.3

Reproduction
Running npm run build --target static caused:
error TS5058: The specified path does not exist: '/home/tajpouria/.pro/fun/stub/packages/client/static/tsconfig.json'.

  error TS5058: The specified path does not exist: 'static/tsconfig.json'.

  at createTSError (node_modules/ts-node/src/index.ts:434:12)
  at reportTSError (node_modules/ts-node/src/index.ts:438:19)
  at create (node_modules/ts-node/src/index.ts:449:36)
  at Object.register (node_modules/ts-node/src/index.ts:349:19)
  at this (node_modules/@nuxt/typescript-runtime/dist/index.js:9:19)
  at console (node_modules/hable/src/hable.js:68:11)
  at name (node_modules/hable/src/hable.js:4:25)
  at NuxtCommand.callHook (node_modules/hable/src/hable.js:65:11)
  at NuxtCommand.run (node_modules/@nuxt/cli/dist/cli-index.js:2775:5)

✖ Nuxt Fatal Error

      TSError: ⨯ Unable to compile TypeScript:
      error TS5058: The specified path does not exist:
      '/home/tajpouria/.pro/fun/stub/packages/client/static/tsconfig.json'.

However, when I used yarn instead of npm works just fine: yarn run build  --target static
I don't think this is an os specific problem because I get the same result in Netlify build process:
Using npm build logs | Using yarn build logs
Not that bad but it take some time to figure out what is the problem!

Hi @tajpouria. If developing a website for static target, it is highly recommended to use target: 'static' in nuxt.config (here) and also nuxt generate (not nuxt build) this way not only this issue will happen, but also the development stage will be adjusted to be consistent with prod (like disabling query params)
General comment why npm is not working: You may need npm run build -- --target static to pass args into command. Still yarn is also recommanded more as we do all tests against yarn.


Fix stack() example in manual

This example has not been updated after [1:4] stopped returning a Vector{Int}
in Julia 0.5.
Fixes https://github.com/JuliaStats/DataFrames.jl/issues/1162.

This fix is actually already included in DataTables via a2ae0ca1.


SAP Fiori tools - Service Modeler (Extension) / An unexpected error ocurred.

SAP Fiori tools - Service modeler cannot be executed when using in VS code Version 1.46.1

Thank you for your feedback. Please name the tutorial where you are facing this issue.

Hi Michael, the tutorial is Configure a List Report Page Using SAP Fiori Tools. This is the link
https://developers.sap.com/tutorials/fiori-tools-configure-lrop.html

Hi Michael, btw I'm using Northwind V2 OData service due to issue:
https://github.com/SAPDocuments/Tutorials/issues/5358#issuecomment-653057516
I don't know if probably Service Modeler don't work with Nothwind

@juanfelipe82193 Let me get back to you. I had checked Northwind service few weeks back and it seemed to work.

Thanks @theStiggySpd, yes Northwind is working fine when gining entry point as Products. But what's not working is the Service Modelerer (Extension). I couldn't try with another OData service because I'm also facing the issue "btoa is not defined".
Thanks.

Hi @juanfelipe82193
We are looking into this. Thanks for reporting this. I will let you know as soon as the fix it out.
Meanwhile, would you be ok to close this issue as its not directly replated to the SAP tutorials repository and tutorial.
Regards,
Sushant (SAP)

Yes, were you'll be tracking it?
Thanks.

HI @juanfelipe82193
I would also recommend that you create a ticket in BCP using component "CA-UX-IDE". Will that be ok with you?

Please provide a step by step to raise it.
Thanks.


New pixel

Checklist

[ x ] I ran npm test locally and it passed without errors.

[ x ] I only edited the _data/pixels.json file.

[ x ] I entered the username in the pixels.json that I'm also using to create this pull request.
[ x ] I acknowledge that all my contributions will be made under the project's license.

This PR touches more than one file and has to be reviewed manually. @rmkubik

I'ts my package-lock.json, I will make a new PR.


getMessage does not work anymore with UID

I have recently updated the package from v1.4 -> v.2.2
In v1.4 I had this code to get a certain message:
$message = $oClient->getFolder($data->folder)->getMessage($data->muid, $fetch_body=false, $fetch_flags=false);
I have updated the syntax for v2
$message = $oClient->getFolder($data->folder)->query()->getMessage($msgn=$data->muid, $fetch_body=false, $fetch_flags=false);
But I get always an error: unique id not found
The UID is the position in the current folder. I check that the folder is correct. In my case $data->folder was INBOX.
I tried using the message id, but this also did not work. I tried getting the number getMessageNo, but it is null for all of my messages.
While I am here asking, what is the difference between UID and Msgn?

Hi @SDCRoman ,
thanks for your report. Unfortunately I can't reproduce the described problem. Therefor a few questions:
1.) Is your message available, if you fetch all messages?
$client->getFolder($data->folder)->query()->all()->get()->each(function($message) use($uid_to_be_found){
    if ($message->uid == $uid_to_be_found){
        var_dump($message);
    }
});

2.) Assuming you are using imap and not legacy-imap as protocol. If you fetch an uid overview, is your uid included?
var_dump($client->getFolder($data->folder)->query()->getConnection()->getUid());

The difference between UID and Msgn isn't clear and depends on your mail provider / server. Sometimes they are identical, sometimes the UID is unique and never changes, sometimes it is not and changes with every expunged change. However in general I believe it is save to say, that the UID stands for unique id and Msgn for the message "index" number. For example Msgn=10 if it's the tenth message...
Best regrards,

From my experience, Uid is not working because of a wrong loop inside ImapProtocol.php, function getUid().
public function getUid($id = null) {
	$uids = $this->fetch('UID', 1, INF);
	if ($id == null) {
		return $uids;
	}

	foreach ($uids as $k => $v) {
		if ($k == $id) {
			return $v;
		}
	}

	throw new RuntimeException('unique id not found');
}

if you dump $uids, you get something like this:
array:61 [▼
1 => "20674"
2 => "20675"
3 => "20676"
4 => "20677"
5 => "20678"
6 => "20679"
7 => "20680"
8 => "20681"
9 => "20682"
10 => "20683"
11 => "20684"
12 => "20685"
13 => "20686"
14 => "20687"
15 => "20688"
16 => "20689"
...
but the loop that searches for the uid is this
	foreach ($uids as $k => $v) {
		if ($k == $id) {
			return $v;
		}
	}

and should be
	foreach ($uids as $k => $v) {
		if ($v == $id) {
			return $v;
		}
	}

this way it finds the id from the list of uids. But then I get another error "no headers found".
Unfortunately I'm experiencing many problems with this new release of laravel-imap/php-imap and I think I'll go back to version 1 :(
Another one is the sort order of messages inside a folder. Maybe I'll open another issue.
Thanks anyway for your work.
Best regards

@mynamespace I was able to adjust the sort order by going into vendor->webklex->laravel-imap->src->config->imap.php and setting the options there: 'fetch_order' => 'desc',

@SDCRoman thanks, I've already tried that (clearing config cache) without success :(

@Webklex Thanks again for the fast reply.
This way of getting the message seems to work. I tried to use this to change flags, but the behaviour is weird. It will change the flag, but when I try to change the flag of a second message it will unflag all other messages. For example I have 3 seen messages. I mark msg 1 as unseen. Now I have 2 seen and 1 unseen. When I try to mark the next msg I will have again 2 seen and 1 unseen, but I should have 2 unseen and 1 seen.
$oClient->getFolder($data->folder)->query()->all()->get()->each(function($message) use($uid_to_be_found, $read_status) {
                if ($message->uid == $uid_to_be_found) {
                    if ($read_status == 0) {
                        $message->setFlag('Seen');
                    } else {
                        $message->unsetFlag('Seen');
                    }

                    return true;
                }
            });

Also executing var_dump($client->getFolder($data->folder)->query()->getConnection()->getUid()); gives me:
Method Webklex\PHPIMAP\Query\WhereQuery::whereGetConnection() is not supported
I used composer show webklex/php-imap & composer show webklex/laravel-imap to verify that I am using the newest versions: 2.2.2 & 2.2.0
Maybe I messed up here? That is how I create the Client:
use Webklex\IMAP\Facades\Client;

protected $oClient;

public function __construct() {
    $this->oClient = Client::make([
        'host'          => env('MAIL_HOST_IN'),
        'port'          => env('MAIL_PORT_IN'),
        'encryption'    => env('MAIL_ENCRYPTION_IN'),
        'validate_cert' => true,
        'username'      => env('MAIL_USERNAME'),
        'password'      => env('MAIL_PASSWORD'),
        'protocol'      => 'imap',
        'authentication' => null
    ]);
    $this->oClient->connect();
}

Hi @SDCRoman ,
you are correct. I messed up my example - I wrote Query::getConnection() instead of Query::getClient()->getConnection().
I'm a bit out of ideas. The fact that the uid can be found if you extend the search criteria is beyond my understanding. I'm also not sure if this is a bug or some other strange behavior. Please feel free add any new insights.
Best regards,
P.s.: is the used account used by any other application / client at the same time? Could they mess up the session (very unlikely - but currently my best guess).

Thanks for the reply webklex. I am currently happy with my workaround. Actually it is even better to build your own id.
I will update the topic if I find out why the issue happened. Maybe I messed something up.


FE: Implement rate component for recipes

Waiting for BE to implement rating data in GET /recipes, then done


[FLINK-31091][Table SQL/Client] Add Ser/de for Interval types

What is the purpose of the change
The adds support for serialization/deserialization of interval types on sql gateway level to make queries
SELECT INTERVAL '1' DAY;
SELECT INTERVAL '2' YEAR;

working again.
Also it hardens modifiers for tests since in case of junit5 package private is enough
Verifying this change
There were added tests for month year and date time intervals
Does this pull request potentially affect one of the following parts:

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

Documentation

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

Could you add an ITCase about the case in the jira to verify we solve the problem?

done

@flinkbot run azure


Wrong version of the installer on the main page

Describe the bug
The link to the latest version on the main page is wrong.
To Reproduce

Go to https://github.com/ViGEm/ViGEmBus
Click on https://github.com/ViGEm/ViGEmBus/releases/latest in the installation chapter
The version 1.16.115 is displayed instead of 1.16.116

Expected behavior
Display the good version of the installer.
System details (please complete the following information):

Any OS and web browser

Indeed! Wonder how that happened, I fixed it by marking it as a pre-release and un-marking again, thanks for reporting!


1.61 Compatability

There seems to be compatiblity issues with this mod and the new patch 1.61

Running the latest build and "mod settings" does not appear in my pause menu

still not working from 0.0.8

I have double-checked the install, but I don't have mod settings appear.
I can't see any issues in the log files

After the v.0.09, it's working fine for me!

Seconded, 0.0.9 is working for me.


websocket 0.2.1 ; object has no attribute 'create_connection'

If pypi package 'websocket' is installed, a backtrace results:
File "build/bdist.linux-i686/egg/socketIO_client/init.py", line 156, in init
self.define(Namespace)
File "build/bdist.linux-i686/egg/socketIO_client/init.py", line 174, in define
namespace = Namespace(self._transport, path)
File "build/bdist.linux-i686/egg/socketIO_client/init.py", line 263, in _transport
self.__transport = self._get_transport()
File "build/bdist.linux-i686/egg/socketIO_client/init.py", line 287, in _get_transport
self.is_secure, self.base_url, **self.kw)
File "build/bdist.linux-i686/egg/socketIO_client/transports.py", line 295, in _negotiate_transport
}[supported_transport](session, is_secure, base_url, **kw)
File "build/bdist.linux-i686/egg/socketIO_client/transports.py", line 128, in init
self._connection = websocket.create_connection(url)
AttributeError: 'module' object has no attribute 'create_connection'
this is https://pypi.python.org/pypi/websocket, by @denik

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

We are using https://github.com/liris/websocket-client by lyris.
Do you need to have denik's version as well?
I'm not sure how to distinguish between two packages that share the same module name.

Im not sure why I had denik's websocket installed.  It would be possible to use hasattr(websocket, 'create_connection') to that the websocket module is lyris', and bail out with a slightly less nasty error if it is another module with the same name.

Upstream has an issue about this: https://github.com/liris/websocket-client/issues/60


Documentation for name resolution (DNS) within landing zones

Benefit/Result/Outcome
So that IT Administrators have guidance on implementing DNS as it applies to their specific scenario.
Description
We considered options for auto-deployment for at least one DNS scenario, however, we believe that the best name resolution solution will depend on how external traffic is configured and connected to the landing zone.  Therefore, we believe the best option for MLZ customers is to document the most likely scenarios.
Acceptance Criteria

A markdown document exists (or a section within an existing document) that describes the most likely name resolution scenarios customers will encounter, and how to provide DNS within each scenario.
Incorporate the scenario described in #392 so that customers can customize their deployments in that way

Potentially related to this question that was recently asked: #674

This can be resolved by https://github.com/Azure/missionlz/pull/746
Current implementation proposed in the following example for IaaS DNS Forwarding gives customers a good overview of the challenges posed by integrating their MLZ deployment with DNS.
Regarding the referenced question, (https://github.com/Azure/missionlz/issues/392) this has already been closed out.


Bug: Dialog cannot resize on mac env

dialog or overlaydialog?

dialog or overlaydialog?

Window Dialog

related: https://github.com/AvaloniaUI/Avalonia/issues/17295


PR 110 redux

@aturner's PR #110 with merge conflict resolved and a test added
As previously:  Adds the option to set CATS_CONFIG_FILE environment variable to specify config file location. Config file location precedence becomes:

Command line option
Location specified in CATS_CONFIG_FILE
./config.yaml

All looked okay in PR #110 - could somebody review and merge?


Add contiguous scope modifier

Could be used for example targeting multiple contiguous line comments.
"comment slurp comments fine"
// foo
// bar

Potential solution
I was thinking that this could be implemented around the existing scope handlers similar to how the every modifier works and just join scopes/targets based on leading and trailing delimiter ranges.

Get all scopes

const scopes = scopeHandler.generateScopes(...);

Iterate scopes and join contiguous targets

const leadingRange = leadingTarget.trailingDelimiterRange ?? leadingTarget.contentRange;
const trailingRange = trailingTarget.leadingDelimiterRange ?? trailingTarget.contentRange;
if (leadingRange.intersects(trailingRange) != null) {
    newContentRange = leadingTarget.contentRange.union(trailingTarget.contentRange);
}

Thinks to discuss

One problem is that most leading and trailing delimiter ranges doesn't contain the eol. Eg with comment the leading range would end with the previous line and the trailing range would start with the next line. We could of course say that that is valid for merging, but it would probably make "take slurp tokens" join all tokens in the file and therefore take the entire file.
How do we handle nested scopes?
Do we want to support "every slurp <scope>"?

FWIW, in the Go ast, contiguous blocks of comments are called comment groups.
I too would like to be able to  operate on contiguous blocks of groups.
General purpose slurp is an interesting idea. It would be useful to think of other scopes that it might apply to / other use cases where it is distinct from “every”.

If slurp does not ignore commenst, I guess it could be used for things like taking items between one comment and the next comment, which sounds like a reasonably common task, for example, in a list of the test cases, organized into groups, separated by comments

Nailing down the definition of contiguous may prove to be hard. Exactly which types of other scopes can break up contiguity? And which ones don’t?

I've already defined that in my code examples. I propose to define it as when two instances of the same scope have intersecting trailing/leading ranges.


Auto-detect NAT

NAT settings make certain types of configurations impossible.
For example, if you have two clusters on-prem, behind a nat router, and then , another public cluster, you need to enable NAT to enable communication between on-prem and AWS clusters,
But , between the on-prem clusters submariner will detect the same external IP and will try to use that one.
This is also a performance problem, when a local network can be used, with potentially bigger MTU (jumboframes) if nat is enabled, an external network will be favored instead, forcing smaller <=1500 MTU and lower performance.
We need a mechanism that allows detection of the best path, probably by using an extra port (TCP or UDP) that submariner engines can use to detect connectivity to each other. Something as simple as a UDP ping, or HTTP response, where we can expose the cluster ID for example.

This issue is still valid

bump

This is still relevant.

This is still relevant.

Mentioned on this thread here:
https://groups.google.com/g/submariner-users/c/mSyku-V5ork/m/BPqLI8xHBAAJ?pli=1


⚠ Bofrost.be has degraded performance

In ff93653, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1643 ms

Resolved: Bofrost.be performance has improved in 8ff3ed6 after 13 minutes.


Breaking change in ripped:* tagging

Hey!
In previous versions of karton-reporter the parent was tagged as ripped:<family>, however right now it is tagged as ripped: <family> with a space. This means that (1) searching ripped tags will be affected and (2) samples that already have ripped:* tag can get two different tags like so:

Fixed, sorry about that!


Run flutter plugin On  iOS Simulator  Failed

Launching lib/main.dart on iPhone Xʀ in debug mode... Xcode build done.                                            6.2s ProcessException: Process "/usr/bin/xcrun" exited abnormally: An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=22): Failed to install the requested application The application's Info.plist does not contain CFBundleVersion. Ensure your bundle contains a CFBundleVersion with a valid semantic version number.   Command: /usr/bin/xcrun simctl install 14971B94-145F-4433-BB5E-6A9DF21A64B5 /xxxxxxx/example/build/ios/iphonesimulator/Runner.app
Run flutter doctor
`
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.14.5 18F132,
locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version
28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
[!] Android Studio (not installed)
[✓] VS Code (version 1.35.1)
[✓] Connected device (1 available)
`

What does "Run flutter plugin" mean?  Can you include the commands you ran before you saw that error?

And can you attach ios/Runner/Info.plist?

Facing the same Issue, adding log file below:
Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter run
exception
ProcessException: ProcessException: Process "/usr/bin/xcrun" exited abnormally:
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=22):
Failed to install the requested application
The application's Info.plist does not contain CFBundleVersion.
Ensure your bundle contains a CFBundleVersion with a valid semantic version number.
Command: /usr/bin/xcrun simctl install A956381E-24D8-448B-8C60-4DFBE8AF2B42 /private/tmp/fl_plugin/example/build/ios/iphonesimulator/Runner.app
#0      runCheckedAsync (package:flutter_tools/src/base/process.dart:255:5)
<asynchronous suspension>
#1      SimControl.install (package:flutter_tools/src/ios/simulators.dart:125:16)
#2      IOSSimulator._setupUpdatedApplicationBundle (package:flutter_tools/src/ios/simulators.dart:400:31)
<asynchronous suspension>
#3      IOSSimulator.startApp (package:flutter_tools/src/ios/simulators.dart:307:15)
<asynchronous suspension>
#4      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:364:54)
#5      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:77:64)
#6      _rootRunUnary (dart:async/zone.dart:1132:38)
#7      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#8      _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#9      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#10     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#11     Future._complete (dart:async/future_impl.dart:473:7)
#12     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#13     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:28:18)
#14     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:294:13)
#15     ApplicationPackageFactory.getPackageForPlatform (package:flutter_tools/src/application_package.dart)
#16     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:77:64)
#17     _rootRunUnary (dart:async/zone.dart:1132:38)
#18     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#19     _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#20     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#21     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#22     Future._complete (dart:async/future_impl.dart:473:7)
#23     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#24     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:28:18)
#25     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:294:13)
#26     FlutterProject.fromDirectory (package:flutter_tools/src/project.dart)
#27     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:77:64)
#28     _rootRunUnary (dart:async/zone.dart:1132:38)
#29     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#30     _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#31     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#32     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#33     Future._complete (dart:async/future_impl.dart:473:7)
#34     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#35     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:28:18)
#36     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:294:13)
#37     FlutterProject._readManifest (package:flutter_tools/src/project.dart)
#38     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:77:64)
#39     _rootRunUnary (dart:async/zone.dart:1132:38)
#40     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#41     _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#42     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#43     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#44     Future._complete (dart:async/future_impl.dart:473:7)
#45     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#46     _rootRunUnary (dart:async/zone.dart:1132:38)
#47     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#48     _FutureListener.handleValue (dart:async/future_impl.dart:126:18)
#49     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:639:45)
#50     Future._propagateToListeners (dart:async/future_impl.dart:668:32)
#51     Future._complete (dart:async/future_impl.dart:473:7)
#52     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#53     _AsyncAwaitCompleter.complete.<anonymous closure> (dart:async-patch/async_patch.dart:33:20)
#54     _rootRun (dart:async/zone.dart:1124:13)
#55     _CustomZone.run (dart:async/zone.dart:1021:19)
#56     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#57     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#58     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#59     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:115:13)
#60     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:5)

flutter doctor
[✓] Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.14.5 18F132, locale en-IN)
    • Flutter version 1.5.4-hotfix.2 at /Users/Cognera/Developer/flutter
    • Framework revision 7a4c33425d (8 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
    • Android SDK at /Users/Cognera/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[!] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 36.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] VS Code (version 1.35.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.1.0

[✓] Connected device (1 available)
    • iPhone Xʀ • A956381E-24D8-448B-8C60-4DFBE8AF2B42 • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-2 (simulator)

! Doctor found issues in 1 category.

@itsyashjoshi I'm trying to figure out what the invalid or missing CFBundleVersion value is in the plist since I wasn't able to reproduce.
Can you attach the following two files, and clearly mark which is which:

<Your Flutter project path>/ios/Runner/Info.plist
The Info.plist in the product app bundle.  Open <Your Flutter project path>ios/Runner.xcworkspace in Xcode and then build via Product > Build against an iPhone simulator.  The Info.plist path will be in your DerivedData directory somewhere like ~/Library/Developer/DerivedData/Runner-durhoqwtricbbcetrszcjktqoabc/Build/Products/Debug-iphonesimulator/Runner.app/Info.plist where the durhoqwtricbbcetrszcjktqoabc is random.

I did the following things to solve the problem
add version: 1.0.0+1 to example  .yaml。 @itsyashjoshi

I ran into this as well.  In ios/Runner/Info.plist, I had:
...
	<key>CFBundleVersion</key>
	<string>$(FLUTTER_BUILD_NUMBER)</string>
...

So I guess somehow FLUTTER_BUILD_NUMBER isn't getting set?  I do have a normal version: 0.0.1 in my pubspec.yaml.
Manually replacing $(FLUTTER_BUILD_NUMBER) with 0.0.1 fixed it for me, but that's not ideal.

@justinmc FLUTTER_BUILD_NUMBER should be set in the Generated.xcconfig based on the pubspec.yaml "version".  What is your version there, maybe it's being parsed incorrectly by the tool?

Sorry, I see now that you said it's version: 0.0.1.

But still, would like to run your pubspec.yaml through the parser.  Can you paste the whole thing, @justinmc?

I talked to @jmagman offline and we found that a version number that starts with a "0" doesn't get parsed correctly.  So as @One1Light mentioned, changing your version number in pubspec.yaml could be a solution to this until the parsing is fixed.

It looks like any version in the pubspec.yaml without a + doesn't get a FLUTTER_BUILD_NUMBER, and even if it does, it gets set to the number after the plus.
https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/flutter_manifest.dart#L109

Temporary workaround in pubspec:
version: 0.0.1+0.0.1

In xcode's build, you need two Numbers, build number and version number. Version: 1.0.0+1 in pubspec.yml is split into the two Numbers you need by the compiled script. This is the logic of the flutter script build. In addition, I found that in some cases it was possible to compile without these two Numbers, but it still failed. Finally, even with these two Numbers set, you need to clear the installed APP and clean the build. To take effect

@One1Light
How do you "clear the installed app"
I change my version in the pubspec to:
version: 0.0.1+0.0.1
then did:
flutter clean
flutter run --debug --build -d 'iPhone SE'
But i still got the error.

same issue here

Similar error found using xCode 11 and flutter create --template=plugin test-plugin
In this case I found useful this issue page, but is just an actual template issue.
Test application pubspec.yaml lack of version information. So easy workaround adding it back asversion: 0.0.1

I added the following string to example/pubspec.yaml:
version: 0.0.1+0.0.1

and run flutter build ios after that.
It fixed the issue.

@One1Light thanks a lotttttttttt. You saved my weekend.

As of https://github.com/flutter/flutter/pull/40611 you should get a warning and instructions of how to fix, not a runtime error.

As of #40611 you should get a warning and instructions of how to fix, not a runtime error.

After creating several plugins recently, I find the cause is missing of version: 0.0.1+1 in example/pubspec.yaml
As far as I remember, flutter-1.7.8 works fine. May some problem in plugin template via flutter-1.9.1

https://github.com/flutter/flutter/pull/40611 was fixed in v1.10.5, which has made it to the dev and beta channel, but not stable yet.

@jmagman I'm getting this error, even after hard-coding the version and build number in xcode.

Only affects new projects, created with --template=plugin.
Older projects still work, even if they didn't have an explicit version in example/pubspec.yaml !

Full Ouput
exception
ProcessException: ProcessException: Process "/usr/bin/xcrun" exited abnormally:
An error was encountered processing the command (domain=IXUserPresentableErrorDomain, code=1):
This app could not be installed at this time.
Could not hardlink copy /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Containers/Bundle/Application/55BF7ED0-B261-49A8-95FF-BA4190C05C2F/Runner.app to /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Library/Caches/com.apple.mobile.installd.staging/temp.yIC0oY/extracted/Payload/Runner.app with manifest /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Library/Caches/com.apple.mobile.installd.staging/temp.yIC0oY/extracted/com.apple.deltainstallcommands.com.scientifichackers.awsIotExample
Could not hardlink copy /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Containers/Bundle/Application/55BF7ED0-B261-49A8-95FF-BA4190C05C2F/Runner.app to /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Library/Caches/com.apple.mobile.installd.staging/temp.yIC0oY/extracted/Payload/Runner.app with manifest /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Library/Caches/com.apple.mobile.installd.staging/temp.yIC0oY/extracted/com.apple.deltainstallcommands.com.scientifichackers.awsIotExample
Underlying error (domain=MIInstallerErrorDomain, code=8):
	Could not hardlink copy /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Containers/Bundle/Application/55BF7ED0-B261-49A8-95FF-BA4190C05C2F/Runner.app to /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Library/Caches/com.apple.mobile.installd.staging/temp.yIC0oY/extracted/Payload/Runner.app with manifest /Users/dev/Library/Developer/CoreSimulator/Devices/B0274C37-FA4A-4292-89E2-AB45C4ED7BB6/data/Library/Caches/com.apple.mobile.installd.staging/temp.yIC0oY/extracted/com.apple.deltainstallcommands.com.scientifichackers.awsIotExample
  Command: /usr/bin/xcrun simctl install B0274C37-FA4A-4292-89E2-AB45C4ED7BB6 /Users/dev/FlutterProjects/aws_iot_new/example/build/ios/iphonesimulator/Runner.app
#0      runCheckedAsync (package:flutter_tools/src/base/process.dart:259:7)
<asynchronous suspension>
#1      SimControl.install (package:flutter_tools/src/ios/simulators.dart:145:16)
#2      IOSSimulator._setupUpdatedApplicationBundle (package:flutter_tools/src/ios/simulators.dart:437:31)
<asynchronous suspension>
#3      IOSSimulator.startApp (package:flutter_tools/src/ios/simulators.dart:339:15)
<asynchronous suspension>
#4      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:378:54)
#5      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#6      _rootRunUnary (dart:async/zone.dart:1132:38)
#7      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#8      _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#9      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#10     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#11     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#12     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:552:7)
#13     _rootRun (dart:async/zone.dart:1124:13)
#14     _CustomZone.run (dart:async/zone.dart:1021:19)
#15     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#16     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#17     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#18     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#19     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:116:13)
#20     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:173:5)

flutter doctor
[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.14.6 18G103, locale en)
    • Flutter version 1.9.1+hotfix.6 at /Users/dev/Tools/flutter
    • Framework revision 68587a0916 (3 months ago), 2019-09-13 19:46:58 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/dev/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Users/dev/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/191.5977832/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 10.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.3, Build version 10G8
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Users/dev/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/191.5977832/Android Studio.app/Contents
    • Flutter plugin version 41.0.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] IntelliJ IDEA Ultimate Edition (version 2019.2.4)
    • IntelliJ at /Users/dev/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 41.0.3
    • Dart plugin version 192.7402

[✓] VS Code (version 1.40.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.6.0

[✓] Connected device (1 available)
    • iPhone Xʀ • B0274C37-FA4A-4292-89E2-AB45C4ED7BB6 • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-4 (simulator)

! Doctor found issues in 1 category.

mark

I am using version like this inside pubspec.yaml ,
version: 1.0.7+12
open Your xCode and let it index for a while you will you will see the exact values.
So in my case, You have to set CFBundleVersion and CFBundleShortVersionString values like this . Here is How info.plist will look after change.

You can use below source code too
      <key>CFBundleShortVersionString</key>  	<string>$(FLUTTER_BUILD_NAME)</string>         <key>CFBundleVersion</key> 	<string>$(FLUTTER_BUILD_NUMBER)</string>
Please paste it inside your info.plist file.


[NEW FEATURE] Add the possibility to skip CCDs in the analysis

We would like to investigate the impact of the lack of stars in the corner of the focal plane.
To do this we would need to discard some particular CCDs from the entire analysis. We first need to check if MCCD can handle an "incomplete" single exposure. Probably @tobias-liaudat have the answer to that. Then we can, for example, add a check in SETools that output an empty star catalogue for the flagged CCDs. Normally the rest of the pipeline should automatically deal with the missing PSF.

@aguinot yes, MCCD can handle an "incomplete" exposure. It is actually the case right now, not all the exposures are complete, for example if a CCD has less than the minimum number of stars it will not be used and the exposure will be incomplete. If the input star catalogue received by MCCD does not include the problematic CCD, MCCD should work without problems.


Add 80-bit ISA to lib

Add the 80-bit vector instruction set to instructions.yml in config/VC4/

Renamed due to architectural changes


Predicates always take effect in reverse

@Component
public class TimeBetweenRoutePredicateFactory extends AbstractRoutePredicateFactory<TimeBetweenRoutePredicateFactory.TimeBetweenConfig> {
public TimeBetweenRoutePredicateFactory() {
    super(TimeBetweenConfig.class);
}

@Override
public Predicate<ServerWebExchange> apply(TimeBetweenConfig config) {
    LocalTime start = config.getStart();
    LocalTime end = config.getEnd();
    return (GatewayPredicate) serverWebExchange -> {
        LocalTime now = LocalTime.now();
        return now.isAfter(start) && now.isBefore(end);
    };
}

@Override
public List<String> shortcutFieldOrder() {
    return Arrays.asList("start","end");
}

@Data
public static class TimeBetweenConfig{

    private LocalTime start;

    private LocalTime end;
}

}
spring:
application:
name: cloud-gateway
cloud:
nacos:
discovery:
server-addr: localhost:8848
gateway:
routes:
- id: user_service
uri: lb://user-service
predicates:
- TimeBetween=上午9:00,下午11:00
The predicate returns the true gateway to jump 404 and the false gateway to jump to the real microservice。
My version is Hoxton.SR1

Not sure what to say here. If you're saying that there's a bug in gateway, we have an almost identical filter Between that works and is unit tested.

gateway:
routes:
- id: user_service
uri: lb://user-service
predicates:
- After=2019-03-27T00:27:50.595+08:00[Asia/Shanghai]
This year is 2020, I use Shanghai 2019 specially, the result returns 404. But if I use before filter, it turns out to be a real microservice.

@Bean
public DiscoveryClientRouteDefinitionLocator discoveryClientRouteDefinitionLocator(){
    return new DiscoveryClientRouteDefinitionLocator(discoveryClient,discoveryLocatorProperties);
}

I injected him and caused misunderstanding


Don't run BMCLoopsTest twice

When working on the performance script for Kontrol proofs which we use in the haskell backend, I've noticed that we're running BMCLoopsTest twice: once as part of test_foundry_prove, and then  again in test_foundry_bmc. This PR makes it so we only run it as part of test_foundry_bmc.

Thanks for catching and fixing @geo2a!


[Web] Tall font characters extremities are not fully displayed after release build

Steps to Reproduce

The use of google font IBMPlexSans makes that the upper and lower parts of a tall letter are hidden (see images below). I think this will happen on different fonts, not only this one, I have not tried though.
This only happens when compiling the code for release. On debug works just fine and on mobile too.
Expected results:
--- Debug text display ---

Actual results:
--- Release text display ---

(The g lower pixels are hidden)

Logs

[✓] Flutter (Channel beta, v1.14.6, on Linux, locale es_ES.utf8)
    • Flutter version 1.14.6 at /flutter
    • Framework revision fabeb2a16f (6 weeks ago), 2020-01-28 07:56:51 -0800
    • Engine revision c4229bfbba
    • Dart version 2.8.0 (build 2.8.0-dev.5.0 fc3af737c7)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /development/android-sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /development/android-sdk
    • Java binary at: /opt/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Android Studio (version 3.5)
    • Android Studio at /opt/android-studio
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.42.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.8.1

[✓] Connected device (2 available)
    • Chrome     • chrome     • web-javascript • Google Chrome 80.0.3987.132
    • Web Server • web-server • web-javascript • Flutter Tools

• No issues found!

Duplicate of #40037


Added Savings Screen

Related Issue
#20
Proposed Changes

Added icons
Added route of savings screen in routes.dart
Added a menu item in menu to navigate to savings screen
Added awesome_circular_chart dependency to pubspec.yaml
Added savings_page.dart in savings_page/  and its component in savings_page/components/
chart_data.dart contain dummy data to show in doughnut chart for savings
doughnut_chart.dart create a chart form the dummy data
devices_list.dart create a list of devices mapped in the chart

Additional Info

Any additional information or context

Checklist

[X] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the documentation.
[x] 🎀 I have updated the documentation accordingly.
[x] 👀 I have read the CONTRIBUTING document.
[ ] ✨ I have added tests to cover my changes.
[ ] 🚩 All new and existing tests passed.
[ ] 🌟 ed the repo

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

Output Screenshots

Screenshot #1
Screenshot #2

Savings Screen
Updated Menu with savings option

Demo
https://user-images.githubusercontent.com/99464268/161613837-4bc7a136-61d3-4cb5-9e46-b51b4dab25dd.mp4

Your UI looks clean , good work ✨
@chandan0000 will verify your code and other things and guide you from here
and at last if everything seems ok i'll merge your pr

Okay 😇😇

@chandan0000  ??

It doesn't match the figma design at all
Padding issue

@Chitrakshi2531 plz fix ui ...?

@all-contributors please add @Chitrakshi2531  code ,

@Chitrakshi2531 plz fix ui ...?

Can you please explain what changes you want.
Padding of which widget weekly savings one or the list of the devices one...
And as this PR is closed should I raise another PR for the changes?


replace manager exceptions with debug.logerror and return

The manager should not throw exceptions. They are expensive and handled automatically, so they aren't much more useful than a simple (and quicker) debug message
Changes

The manager no longer throws
Error debug log messages are displayed
Safe (null, empty collections, etc) values are returned where an exception would have been thrown

trying to get checks to resolve....

Yup. Likely Github is trying to recover still.


Add integration tests on env-export commands

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:b0853cab7fc6d70a1c279cd40798c52b993a0bc4

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:209e93147cdd35f92522d9a7a668b4ee0a48583f

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:11f3a24dde48a6201d492a0f9d79b584d1e37a3c

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:1ddf73be92a023ac7e9baa1a5e0755e31a8597c4

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:d3f47d2c665c609ef9d179a6b9bec11cde6ef748

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:e83038e7e5b4eee28aa523dc3a65ce6ef8de2028

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:759564eae283291347cc5599ac97dde848f422d5