[bug] -N skipping fragments?

Sometimes file will fail to download (or merge, I'm really not sure since I haven't seen any errors, though maybe I just missed them). This happened before I started using --path temp:D:\Temp\  but since using that, temporary files are getting left behind there for files that didn't completely download (in this case fragments 377-745 and respective .part and .ytdl files. I understand the default is to resume downloads with previously downloaded fragments but it should still clear up all the fragments for that respective download on completion. The skipping of download might not be a bug but I was curious as to what --fragment-retries "infinite" really does since I have it set, I assume it is regarding that fragment as unavailable and by default getting skipped but I want it to retry until it finds it instead (at least for a minute).
Command (temporary switched back to batch files since the frontend I was using didn't like some stuff):
yt-dlp -a "CR todownload - valarian.txt" -o "crunchyroll.com\%%(series)s - S%%(season_number)sE%%(episode_number)s - %%(episode)s - %%(epoch)s.%%(ext)s" --embed-subs --sub-lang en --cookies cookies-crunchyroll-com.txt --user-agent "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:89.0) Gecko/20100101 Firefox/89.0" --fragment-retries "infinite" -N 3 --no-mtime --download-archive archive.txt --path temp:D:\Temp\
Originally posted by @Jules-A in https://github.com/yt-dlp/yt-dlp/issues/359#issuecomment-853906485

@Jules-A, Can u check if the resulting file has missing fragments? "377-745" is quite a large range to have unavailable fragments. So my initial guess is that the fragments are in fact getting appended, but just not deleted from disk

Also, if you could find a verbose log of when this happened, it would be helpful

I basically just ran the batch again and it re-downloaded the file (not sure if from scratch) with a file size similar to the rest. Actually I think there's a larger issue at play, I just tried playing the file and MPV wouldn't open, MPC opened but didn't play anything. I then checked the rest of the files and none of them play, they seem to all be corrupted... This is the series I was trying to download: https://www.crunchyroll.com/valerian-and-laureline/
It's probably worth noting I tried entering the url to download the playlist but it would just return a 403 even with latest cookies so I used my greasemonkey script  to grab the urls and put in to a text file for batch.
No verbose log of when it happened, I will try again though.  I forgot to decrypt the fragment when reading from disk. I didn't notice the issue since I only tested with non-encrypted media. Will fix asap

@Jules-A Please confirm this is fixed. I don't have a crunchyroll account and hence was only able to test on trailers

@Jules-A Please confirm this is fixed. I don't have a crunchyroll account and hence was only able to test on trailers

Hmm... That's weird, in Australia pretty much everything that is older than a week (and a few exceptions for specials/ovas/uncensored versions) is available for free without an account at 1080p. Instead we get a heap of region locked content. As far as I'm aware I think it's similar except for some European countries?
Are you sure you aren't just trying to do a playlist download or trying without cookies file? The playlist download is currently broken but they are in the process of rolling out their new website update so no point fixing that yet. The cookies are required for CR and they expire fairly quickly. If you try watch for example: https://www.crunchyroll.com/welcome-to-demon-school-iruma-kun/episode-1-iruma-kun-from-demon-school-789490  does it say it requires a CR sub?
I will test it soonish and update this comment when I have but I haven't seen any left over files since I reverted to an older build.

in Australia pretty much everything that is older than a week (and a few exceptions for specials/ovas/uncensored versions) is available for free without an account at 1080p. Instead we get a heap of region locked content. As far as I'm aware I think it's similar except for some European countries?

I didn't know that... I assumed all content required a subscription

Okay so I tested just 1 show so far and it plays but I haven't watched the whole file to check if there's stuttering or anything.

If it starts playing, then there shouldn't be any other issue

I just realised that's probably my fault since the filename is changing due to my output template using epoch.

Nothing can be done about it

resuming a download probably should have a way that doesn't fully rely on filename.

What do you suggest? I cannot think of any other way that is more reliable

I want a number that is sequential for the watch order (the order I list them to download in the batch text file I send to yt-dlp) that is unique so it doesn't play a file from another season or another show entirely. It would also need to take in to account editing the batch text file

id and extractor_key together is always unique (except for the generic extractor). As for a sequential number, there is no single method that can work across all websites and can keep working when you edit the batch files. It would be possible if you are willing to put the episode numbers manually in the batch file along with the URL

so it doesn't need to download the webpage 50 times unnecessarily when I already have the files (I feel like this is an area that could really use some improvement...)

Using --download-archive should normally help with this, but I see that crunchyroll uses a non-standard regex group for getting id which is causing the pre-checking to fail. I'll fix it

What do you suggest? I cannot think of any other way that is more reliable

You could try searching for the .part file or .ytdl that is created and resuming using the filename of that file.

id and extractor_key together is always unique (except for the generic extractor). As for a sequential number, there is no single method that can work across all websites and can keep working when you edit the batch files. It would be possible if you are willing to put the episode numbers manually in the batch file along with the URL

Id isn't always sequential but I haven't tried extractor_key but I doubt that is either. Adding numbers manually would require writing some code but I'm not really that familar with batch files and I'm too lazy to start writing my own front-end right now...
If you mean the batch text file (the line separated list of urls in a txt file) that's possible is you allow optional reading of an index number I guess?

Using --download-archive should normally help with this, but I see that crunchyroll uses a non-standard regex group for getting id which is causing the pre-checking to fail. I'll fix it

Ah I see, thanks.

If you mean the batch text file (the line separated list of urls in a txt file) that's possible if you allow optional reading of an index number I guess?

Yes, I did mean the text file containing the URLs. There is no need for me to implement reading of the index. You can parse it out of the URL using --parse-metadata

Append #num at the end of the URLs. Eg: https://www.crunchyroll.com/welcome-to-demon-school-iruma-kun/episode-1-iruma-kun-from-demon-school-789490#10
Use --parse-meta "webpage_url:#%(mynum)s" to extract the number into mynum
Use that in the outtmpl like  -o "%(mynum)03d - %(title)s.%(ext)s"
Eg:

$ yt-dlp "https://www.twitch.tv/videos/1047573539#10" --get-filename --parse-meta "webpage_url:#%(mynum)s" -o "%(mynum)03d - %(title)s.%(ext)s"
010 - PARTIDO COMPLETO # Real Madrid 3-2 Real Sociedad # Primera Iberdrola.mp4

Alternatively, if you know that the URL contains the episode number already (like in the example you gave), you can directly parse it as well using --parse-metadata "webpage_url:episode-(?P<mynum>\d+)"

You could try searching for the .part file or .ytdl that is created and resuming using the filename of that file.

There would be no way to know if those files belong to the same video or not

You could try searching for the .part file or .ytdl that is created and resuming using the filename of that file.

There would be no way to know if those files belong to the same video or not

Then why not store the id in the .ytdl file?

Sorry about another unrelated question but how can I change the template so that if any of the output templates fail to return a value (I'm using 5 including the new parse-metadata value, 6 soon once I work out how to use metadata to get the domain) then simply return the title and video id instead? There's placeholder option which I can probably set to "" but I'm currently placing text in front like S and E and - .

It is not directly possible. It can (probably) be done using --parse-metadata again, but the command would be quite lengthy. What is the output template that you are using?

It is not directly possible. It can (probably) be done using --parse-metadata again, but the command would be quite lengthy. What is the output template that you are using?

Right now:
yt-dlp -a "CR todownload - Kekkaishi.txt" -o "crunchyroll.com\%%(series)s - S%%(season_number)sE%%(episode_number)s - %%(episode)s - %%(index)s.%%(ext)s" --parse-meta "webpage_url:#%(index)s" --embed-subs --sub-langs en,enUS --cookies cookies-crunchyroll-com.txt --user-agent "Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:89.0) Gecko/20100101 Firefox/89.0" --fragment-retries "infinite" -N 3 --no-mtime --download-archive archive.txt -P temp:D:\Temp\ pause

I need some way to check if mytitle contains a certain string so it can use the fallback but I'm hopeless at this....
-o "%%(extractor)s%%(mytitle|title)s - %%(myindex)s.%%(ext)s" --parse-metadata "webpage_url:#%%(myindex)s" --parse-metadata "%%(series)s - S%%(season_number)sE%%(episode_number)s:%%(mytitle)s"

You were on the right track. But there are 2 issues

Even if the fields are absent, mytitle will still be popylated with something like NA - SNAENA
The right side of | is treated literally, not as the name of a field

This is what you want:
$ yt-dlp "https://www.zee5.com/movies/details/krishna-the-birth/0-0-63098#50" "https://zee5.com/tvshows/details/krishna-balram/0-6-1871/episode-1-the-test-of-bramha/0-1-233402#20" -O "%(extractor)s\%(title)s - %(myindex)s.%(ext)s" --parse-metadata "webpage_url:#%(myindex)s" --parse-metadata "%(series)s - S%(season_number)sE%(episode_number)s:^(?P<title>.+ - S\d+E\d+)$" --output-na-placeholder ""
WARNING: Could not interpret video %(series)s - S%(season_number)sE%(episode_number)s as "^(?P<title>.+ - S\d+E\d+)$"
Zee5\Krishna - The Birth - 50.mp4
Zee5\Krishna Balram - S1E1 - 20.mp4

This will use %(series)s - S%(season_number)sE%(episode_number)s if all 3 of these fields are present, else it will use %(title)s. ofc, change -O to -o and escape the % as needed

I see, thanks a tonne! Still can't seem to wrap my head around Python regex (regex in general gives me headaches so I usually try to avoid it even in other languages...).


multiplex display and button at same time

i have got a new board, this time with 3 7 segment display and 6 button, but the button is multiplexed with the display, is it possible to do in esphome, the original mcu had 14 gpio, if we seprate the button itll be 19, which is too much for esp32

Hi, can you show schematic to of button multiplexing you mean?
If I calculate pins 8 segments + 3 digits =11 pins for display.
You have 3 remains for buttons.

You can use 2 for shift register and one for sense.
Another option use one frame of display to take button state. Connect buttons to segment pins one end and to one free pin. So when you power this extra pin from controller you can read levels from segment pins in theory.

Another option is to put between pins buttons with opposite diodes between each of 3 pins. So you can pull low 2 of pins and send third to high to read two buttons. Then cycle to read another two and so on.

And there is always analog option. You'll need one pin and 7 resistors. Connect all buttons and one resistor sequential and put resistor in parallel with each button. If you choose button resistors like 1K 2K 4K 8K 16K 32K each press combination will give you unique resistance. 7-th resistor just other hand of voltage divider.

Another option is use one display refresh cycle to reuse 8 segment pins and one extra pin. I guess that what you meant. Button with diodes connected to segments one end and extra pin another end. So we can listen segment pins one cycle of screen powering them from extra pin. This sounds doable. But you'll need a diode for each button to make sure press will not affect the screen.

the display is using only the number which is 7 segment pin and 3 digit pin with transistor, resistor is on the segment, and the 6 button is connected to 6 of the 7 segment pins, other side of button is connected through a 1k resistor together to another mcu pin, so the display and button used a total of 11 pins, plus pwm,buzzer,ir receiver its a total of 14

the button press will not effect the screen because the button is controled by gpio so when button is scanning display is off

but i ultimately gave up on the project, not because the multiplexing problem, but because the motor is not driven by pwm like i think it is
it is a digital fan which only have 3 wire to motor, but seems it is using some serial communication, it wouldnt make sense to hack it then
https://cdn.discordapp.com/attachments/1098248656697704550/1282070116732895374/VID_20240908_040343.mp4?ex=66dffe97&is=66dead17&hm=7ddf7f540f319b35737393a443f12dfdfda91da9baa9ac645939e639e8ecba3f&

the button press will not effect the screen because the button is controled by gpio so when button is scanning display is off

I see. Yous it's doable. We just need to add a button sense iteration in display cycle.

yeah next time if we meet this we can try to make it work, but id better get a new one for modding this time

For the board you showed I'd socket the IC first. So you can easily take it out, replace it or plug the esp using custom adapter.
The IC is definitely controller. So you can read microcode from it and analyse with IDA Pro or Ghidra.
You can write your own microcode for sure but looks like esp via adapter will be much easier to code.

it is otp micro controller, it is not important, it will have to be replaced with esp32 no matter what


Runaway memory usage with memory caching

Expected Behavior

Nothing works after updating... Just kidding.. A smooth stream of win and happiness.
Current Behavior

A stream that plays for a while, but eventually gets choppy and locks up the entire box
Possible Solution

I believe the issue may be related to my "zero cache" (I run a zero cache buffering to a 500gb HDD because I like smooth seekability). However, I specify a maximum amount of memory in the addon for it to cache, disabled "use advanced settings" in the torrent settings, and I set the buffer size lower as well.. It blows right past any limit I set until it locks up the box. This wasn't an issue until the latest update, since I cached to filesystem on the earlier version, but that's not working for me currently (see /issues/421)
Steps to Reproduce (for bugs)

Open Kodi
Open Elementum, select an excellent and completely public domain torrent
Watch video, smile, think of the wonders of how awesome technology is for all the cool things it can bring us
Frown when it gets choppy and stops responding to remote commands or ssh

Context

I started with Elementum a few days ago, and love the hell out of it. Best addon ever, in my opinion. But a few times it's frozen my boxes (Sunvell T95Z Max, Amazon FireTV 2nd gen). Finally had an SSH session into it running top, and until the torrent starts on the Sunvell, Elementum consumes 1.3% of my RAM. As soon as I open, it starts gobbling up RAM like a vampire looking for blood, or a zombie looking for brains. When I end the stream, RAM usage stays inflated until I disable the addon and reenable it. If I start another, it continues to climb, unless I reset between plays. I had similar issues, though not as severe, when disk caching was working.. But I think it's because I left my event log turned on... I can't confirm if this was the original issue with disk caching and memory leaking, but it seems like I'm getting leaks regardless of what I do with my FireTV/Sunvell T95Z Max (coreelec) boxes.
Your Environment

Version used: 0.1.28

Environment name and version: Uhh.. 0.1.28?

Operating System and version: OpenELEC 9.01, FireTV 2nd gen

Link to Kodi debug log: I apologize, it's late and I will totally send one as soon as I get more time if you'd like, but I wanted to get this on the radar since it may be a known issue/log may not be necessary. Otherwise, could you please let me know where in the memory filling process to save it? Because if I just let it go until it fills up, the box is unresponsive, so I need to save it at some point before it fills up the memory... Which may not be useful.. I'm not sure. Please let me know.

Log file is needed.
What is your problem in less words? Device Out of memory? Playback stops? Or what?

Sorry, im wordy. Playback gets choppy and freezes within a few minutes of starting when in memory caching playback mode. RAM usage of elementum climbs until box freezes despite settings/limits to buffer/cache sizes.
When in the process should I pull in the log file? The entire os goes unresponsive, forcing a hard reboot, so should I let it get to about 50% system ram usage or so (750mb used by elementum) and then download the log? Less? More? Please let me know.

It appears this is a duplicate of issues/412. Will close. Suggest maybe disable "high speed" as a "libtorrent settings profile" in the meantime, as it appears to ignore the memory cache settings.

@elgatito I closed this because it's a duplicate, but the issue still exists when I set "high speed" in "BitTorrent settings" (obviously the workaround is to use "normal" for now). Memory limits are ignored if playing from memory cache, caching to disk makes no difference, and even stopping playback/cancelling the download/deleting the file does not bring memory usage back down, which requires disabling/reenabling the plugin to restart the elementum process. This very well may be a libtorrent issue, but here's a log anyways in case it's not. log file here
I let it go until memory usage of the elementum process was at ~50% RAM usage, then stopped playback, cancelled download, and deleted file (using elementum dialog boxes set to "ask"). Memory usage, as I type, still sits at 53.6% fior the elementum process, and it is not currently doing anything. Worth noting: It never goes above 10% when not using "high speed", even if using memory cache.

libtorrent profiles are settled up by libtorrent.
Users reported about memory usage on highspeed profile.
I would not recommend changing from default profile.
When you select memory size - you select size for storing torrent data, not memory used by application. Release of virtual memory is done by garbage collectors, application is not controlling memory release.

Alright, thanks for the info... And for picking up this project.


Before:

Making recipe that requires both solid + reagent ingredients = literally impossible
Extra ingredients are ignored and the recipe will still be made
Unknown bugs

Now:

Extra ingredients are ignored and the recipe will still be made
Unknown bugs

Unknown bugs

they don't sound very tasty


Disable FICSMAS

And another one: allows the option to disable FICSMAS, miserable bastards ;)
Note: I've added the complete GameUserSettings.ini (as generated by the server when you use a console command that writes to this ini-file). Partial files are removed by the server for some reason upon restart, though the set values are applied regardless.
Solves #108

Thank you as always! :D

@wolveix Yw as always :) thanks a lot for the invitation!


Level Coliseu - Sino (UV)

Abertura de UV do Sino (.blend do Sino no drive: Assets Novos -> ColiseumNew)

https://drive.google.com/drive/folders/1VdZei4vCGyXZim1b9MzuPXn1PMVGhlu3


nix-prefetch-zip fails moving files up

cc @cstrahan.
Moving files one directory up (https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/fetchzip/nix-prefetch-zip#L130) may fail if one of the files has the same name as the parent directory.
E.g., applied to http://www.lix.polytechnique.fr/coq/pylons/contribs/files/ABP/v8.4/ABP.tar.gz, nix-prefetch-zip fails with:
mv: cannot move ‘/tmp/tmp.iF77nPUttb/unpacked/ABP.tar.gz/ABP/ABP’ to ‘/tmp/tmp.iF77nPUttb/unpacked/ABP.tar.gz/ABP’: Directory not empty

There seems to be a similar problem in fetchzip: https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/fetchzip/default.nix#L39


ticker: 4.5.2 -> 4.5.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/achannarasappa/ticker/releases.
meta.description for ticker is: "Terminal stock ticker with live updates and position tracking"
meta.homepage for ticker is: "https://github.com/achannarasappa/ticker"
meta.changelog for ticker is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.5.3 with grep in /nix/store/kk2d8581wz97nf1ym3gpjln9p820qg77-ticker-4.5.3
found 4.5.3 in filename of file in /nix/store/kk2d8581wz97nf1ym3gpjln9p820qg77-ticker-4.5.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ticker

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kk2d8581wz97nf1ym3gpjln9p820qg77-ticker-4.5.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ticker https://github.com/r-ryantm/nixpkgs/archive/35980a2c246c12f084dcf724d997087498725424.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kk2d8581wz97nf1ym3gpjln9p820qg77-ticker-4.5.3
ls -la /nix/store/kk2d8581wz97nf1ym3gpjln9p820qg77-ticker-4.5.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ticker

Maintainer pings
cc @siraben for testing.

Result of nixpkgs-review pr 186717 run on x86_64-linux 1

1 package built:

ticker


feat: implement header

render title and X(close) button

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[GLUTEN-2210][CH] Remove EmptyRDD.scala

What changes were proposed in this pull request?
 Please stay tuned.

Please do a rebase to include https://github.com/oap-project/gluten/pull/2212. Thanks!


Heartbeat tool stops working if the heartbeat screen is closed

When recording a recent demo, I noticed the heartbeat stopped running anytime I left the screen we use to configure the heartbeat. I suspect we have something that is only referenced in that view code that should be extracted to somewhere that runs no matter what screen you're on and whether the app is backgrounded.

Do you know if the Heartbeat stopped or just the Ditto subscription for it stopped?


pageSize.width & pageSize.height are undefined

Are you using the latest version of jsPDF?
jsPDF 1.4.1
Have you tried using jspdf.debug.js?
Yes
Steps to reproduce
Create a new jsPDF instance and try to access internal.pageSize.width/height. They are undefined/NaN.
Example: jsbin
What I saw
pageSize.width & pageSize.height are undefined
What I expected
pageSize.width & pageSize.height should contains the actual page values.
Note:

Using pageSize.getWidth()/pageSize.getHeight()works, but Typescript will marks it as an error and prevent angular 2+ from compile correctly. Example
In 1.3.5 all works correctly

You should use the release v1.3.5 to work with pageSize.height as we can see in this line: https://github.com/MrRio/jsPDF/blob/v1.3.5/jspdf.js#L1146
After release v1.4.0 they encapsulated it and change to pageSize.getHeight()

Why cant you use it with typescript?

Duplicate of #1943


Change action buttons from "Save" and "Okay" -> "Save" and "Cancel"

I feel as though it'd be more intuitive for users to see buttons that say "Save" and "Cancel" across the GUI. "Okay" sounds analogous to "Save" while actually discarding the changes/edits a user makes

I definitely agree.  This should be a relatively straightforward update, but great for usability

passed all devel tests

passed all devel tests


Fix Windows build as a result of recent TaskQueue changes

I wasn't paying attention, and added SeparateErrors to the constructor of Task.
However, there is no constructor for task taking 5 params, only 4.

Doesn't affect any of our builds, I've tested locally.


Update commons-httpclient to latest version

What kind an issue is this?

[ ] Bug report. If you’ve found a bug, please provide a code snippet or test to reproduce it below.
The easier it is to track down the bug, the faster it is solved.
[X] Feature Request. Start by telling us what problem you’re trying to solve.
Often a solution already exists! Don’t send pull requests to implement new features without
first getting our support. Sometimes we leave features out on purpose to keep the project small.

Feature description
I am working on a PR for Apache Drill to allow Drill to query ES (https://github.com/apache/drill/pull/1961).  In so doing, I found that ES-Hadoop has a dependency on commons-httpclient 3.0.1 which is causing issues with the Drill integration.  I am wondering if this could be updated to use the current version of 4.5.12. 1
I noticed that there two related issues 3 4 however, these seem to be quite complex.  I'm wondering if it would be possible to simply update the library without all the refactoring?
Additionally, there is CVE-2012-6153 out for this version of httpclient.2

This definitely relates to #1184, though I understand that issue only mentions the LLRC as the path forward for HTTP/S communication. This should be considered a reasonable fallback if the changes to LLRC needed are not all accepted. There has been some discussion about how the Hadoop client may be a special case, and thus should do things the way it expects instead of pushing those concerns into the LLRC, which is a more general client.


[Bug Report] when el-image in  a scrolling div. Left clik the image will lead to the page can't scroll

Element UI version
2.13.0
OS/Browsers version
chrome 78.0.3904.108
Vue version
2.6.10
Reproduction Link
https://codepen.io/zzxcncz/pen/LYYwOXo
Steps to reproduce
click the image ,then the page can't scroll
What is Expected?
click the image ,then the page can scroll
What is actually happening?
click the image ,then the page can scroll.
i fock the code from the element ui website.The demo of the component in the website can also lead the result. Just click the image ,and chage to other page.Then the other page can't scroll.
i found the reason is when i click the image in a scrolling div. it will add a css in .the css is over-flow: hidden.

if i don't need  preview the image, the click handler will also add a body style overflow = 'hidden'.
this will lead to i can't scroll the page.
the el-image's click handler  code.
clickHandler() {
        // prevent body scroll
        prevOverflow = document.body.style.overflow;
        document.body.style.overflow = 'hidden';
        this.showViewer = true;
      },

i think add the following code  will  handle it
    clickHandler () {
      if (this.previewSrcList.length > 0) {
        // prevent body scroll
        prevOverflow = document.body.style.overflow
        document.body.style.overflow = 'hidden'
        this.showViewer = true
      }
    },

I get this problem too. it seems to make image-viewer run. but haven't check for i had not set preview-src-list.

Also encountered this problem.

same problem

Same problem

I am finding el-image-viewer to activate when clicking simple el-images without any preview sources causing the body to have overflow: hidden. I found the change was implemented 2 months ago to fix an image-preview bug.
@luckyCao @island205 Please correct this regression. As soon as any image is clicked, we now permanently and irreversibly get a body { overflow: hidden } attribute added breaking page scrolling.... https://codepen.io/zzxcncz/pen/LYYwOXo

This PR may resolve this bug.....
https://github.com/ElemeFE/element/pull/18416

#19075

fixed

fixed


Curated projects

This is partly prompted by some thoughts after my tweet in response to the question of how to improve Scala adoption.

Trusted cohesive library ecosystem. A lot of library usage is built on trust, most folks have been bitten at one points by a lib with some gotchas. This is why I think building trust and branding of libs with shared ethos is very important.

It feels like the typelevel projects page could benefit by curating the functionality provided by the libraries in question. An example of the type of curation I think could be beneficial would be https://github.com/lauris/awesome-scala or https://github.com/rust-unofficial/awesome-rust, in that libraries are grouped based on functionality provided.
An example hierarchy could be:
"Web Server":
  - "http4s"
  - "Finch"
"HTTP Client":
  - "fetch"
  - "Hammock"
  - "http4s"
"JSON":
  - "argonaut-shapeless"
"Functional Reactive Programming":
  - "cats-effect"
  - "fs2"
  - "monix"
Database:
  - "doobie"
"Front End":
  - "OutWatch"
Configuration:
  - "Ciris"
  - "Extruder"
  - "PureConfig"
Serialization:
  - "scodec"
"Science and Data Analysis":
  - "Algebird"
  - "coulomb"
  - "Frameless"
  - "Libra"
  - "spire"
  - "squants"
"Testing":
  - "cats-scalatest"
  - "ScalaCheck"
  - "scalacheck-shapeless"
  - "Specs2"
  - "discipline"
"Linked Data":
  - "banana-rdf"
"Generic Programming":
  - "shapeless"
"Command Line":
  - "decline"
"Extensions":
  - "cats"
  - "cats-tagless"
  - "eff"
  - "kittens"
  - "monocle"
  - "mouse"
  - "refined"
  - "singleton-ops"
"Misc":
  - "algebra"
  - "paiges"
  - "scala-steward"
  - "scala-exercises"
  - "Scoverage"
"Macros and Compiler Plugins":
  - "export-hook"
  - "imp"
  - "kind-projector"
  - "Machinist"
  - "macro-compat"
  - "simulacrum"
  - "TwoTails"

Hopefully this would make it easier to navigate the typelevel ecosystem and find the correct library for the job.
The following projects have been removed, based on lacking recent commits, or being known to be unsupported:

Alleycats - redirects to cats.
shapeless-contrib - no commits in 3 years.
catalysts - can't decipher what the intended use of this project is? Is it a way of know which scala version is being used at runtime?
structures - not updated in 5 years.
tut - README suggests using mdoc tut is deprecated. Please switch to mdoc. See the migration guide..
CaseClassy - project abandoned.
Sonic - 3 years since last commit.

Notes
If there are any analytics available for the typelevel website. It would be very useful to know how much the projects page is used.

I agree that the current presentation leaves a lot to be desired, and like your approach.
shapeless-contrib and structures are both archived.  I think catalysts and machinist are both on the way out.  We could either flag these in their sections or leave a section for historical relics at the bottom.
We might also want to highlight distinguished projects.  Currently, we have "full members" and "incubator."  We might continue that distinction, or flag projects with a certain number of stars (~1k shows projects people tend to think of when they think of Typelevel).  Some sort of visual distinction within the category would be good.
I am not aware of any analytics, but that could also be a nice addition.
Thank you for this great suggestion.  Would you be interested in submitting a PR to make it happen, or let someone else run with the idea?

Thanks for the feedback @rossabaker .

Thank you for this great suggestion. Would you be interested in submitting a PR to make it happen, or let someone else run with the idea?

Yeah, I'll try and open an in progress PR in time. Hopefully I'll get round to it within the week.

I am not aware of any analytics, but that could also be a nice addition.

I don't have a huge amount of experience with this, but I expect just bunging in google analytics could be a tad controversial. So will raise under a separate issue for further discussion.

Thanks!  I agree with deferring the analytics to a different discussion.


Support Sketch 45 plugin update system

Hi there!
This is a quick note to remind you that Sketch 45 will include a plugin update system.
It would be awesome if you could add support to it (it's really easy!) before Sketch 45 comes out. Ideally, you should release an update for your plugin while we're still in Sketch 44, so your users will have a nice & easy experience when 45 is released.
For more details, please check http://sketchplugins.com/d/229-updating-plugins and http://developer.sketchapp.com/introduction/updating-plugins/
Thanks in advance!

@bomberstudios This has been done, although I am finding it difficult to prove that it has been done correctly.
Will it show there is an update only if the previously installed version had appcast included, or if latest version with appcast support is newer than any previous version?

Only if the installed version already has an appcast field.

You can test it by adding the appcast field to an old version of the plugin locally, and it should update to the latest version on the appcast.xml

Thank you for the feedback @bomberstudios.


Support for Xcode13

+1

I'll add a ticket to look into this, this is definitely something we want to get out soon!

Would we be able to use the 2.8.0 release of Swinject: https://github.com/Swinject/Swinject/releases/tag/2.8.0 ?

I'm not sure, we haven't tested that yet.

Version 2.8.1  Should be now working with Xcode 13.


[RHOAIENG-2985] Experiments - Run Integration

Closes: RHOAIENG-2985
Description
Added experiment related routes, helpers for those routes which resemble recently added route helpers for pipelines/versions/runs. The experiment helpers are used to build breadcrumbs which are conditionally used for runs pages based on the experiment feature flag and the existence of an experimentId in the URL.
Clicking on an experiment's now linked name from the experiments table should yield the runs tables/tabs where the "Active" is initially selected, where the results of the runs for each table (active/archive/schedules) are filtered by the chosen experiment's ID.
Since results are scoped to the experiment ID, the Experiment column and filter are hidden from the user and only shown if runs are accessed by other means (not via an experiment).
Screenshots (demonstrating breadcrumbs)
Runs table/tabs

Create run

Schedule run

Duplicate run

Duplicate schedule

Run details

Schedule details

cc @yannnz
How Has This Been Tested?
Added a few tests accessing runs from the experiments page and accessing breadcrumbs from experiment-scoped run pages.
Request review criteria:
Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/approve
everything works as expected


Location is used on explictit faces from sketch

# Location is used on explictit faces from sketch
with BuildPart() as part_located:
    Box(1, 1, 1)
    with BuildSketch() as sk:
        with Locations((10,0), (0, 10), (10, 10)):
            Circle(2)
        MakeHull(mode = Mode.REPLACE)
    with Locations((20, 30, 5)):
        Extrude(*sk.faces(), amount=4)

# Location is ignored on implicit pending faces from Sketch
with BuildPart() as part_implicit:
    Box(1, 1, 1)
    with BuildSketch() as sk:
        with Locations((10,0), (0, 10), (10, 10)):
            Circle(2)
        MakeHull(mode = Mode.REPLACE)
    with Locations((20, 30, 5)):
        Extrude(amount=4)

Currently this works as "expected" (i.e. with the limitations of passing a face to extrude).  Here is some updated code:
from build123d import *

# Location is used on explicit faces from sketch
with BuildPart() as part_located:
    Box(1, 1, 1)
    with BuildSketch() as sk:
        with Locations((10, 0), (0, 10), (10, 10)):
            Circle(2)
        make_hull(mode=Mode.REPLACE)
    print(sk.faces()[0].normal_at())
    with Locations((20, 30, 5)):
        extrude(sk.faces(), amount=4)

# Location is ignored on implicit pending faces from Sketch
with BuildPart() as part_implicit:
    Box(1, 1, 1)
    with BuildSketch() as sk:
        with Locations((10, 0), (0, 10), (10, 10)):
            Circle(2)
        make_hull(mode=Mode.REPLACE)
    with Locations((20, 30, 5)):
        extrude(amount=4)

show_object(part_located, name="part_located")
show_object(part_implicit, name="part_implicit")

The two versions aren't the same because the normal of the face generated by make_hull is "down" (Vector: (-0.0, 0.0, -1.0)) which causes the extrude to happen in the opposite direction to the second example where the extrude is in the direction of the default workplane (Plane.XY).  This illustrates the advantages of not passing faces to extrude but allowing the internal contexts to resolve the face and normal.


(Tianbin)Week 0 review ticket 3/3

Scrum Team GitHub pages (2 points)
Web Project: Project Description, Project Plans/Ideas/Wires, Time box
Team Roles described, Project Owner or Sponsor described, Scrum Board started
Individual GitHub pages (3 points)
GitHub page for Test Prep study, requiring 5.1, 5.2 notes and actions( See "GitHub pages actions" at the bottom of each TPT page on cspcoders)
GitHub page that documents Data Structures project, using code snippets for key learnings, and links to  GitHub for code and Replit for runtime
GitHub page that documents intended Create Task project for College Board, snippets, and links to Runtime

[x] GitHub page for Test Prep study, requiring 5.1, 5.2 notes and actions( See "GitHub pages actions" at the bottom of each TPT page)
completed notes and actions

[x]  GitHub page that documents Data Structures project, using code snippets for key learnings, and links to GitHub for code and Replit for runtime
documentation for project with code and Replit

[x]  GitHub page that documents intended Create Task project for College Board, snippets, and links to Runtime
documents Create Task with snippets, includes runtime

3/3


Updated README.md codes as per PayPal SDK 1.3.0 for Laravel 4

Please fork the L4 branch and apply your changes if it's not already fixed.
Regards


added testimonials section in faq page

added testimonials section in faq page.
kindly review this pr.

@PranavBarthwal it looks quite filled up, if you can reduce the size of the images as well as font size. Moreover, there is a lot of empty space below the 'Testimonials' heading, reducing that would make it appear much better.
Also, the FAQ & Testimonial sections appear pretty far away, reduce the distance between those sections as well.

@PranavBarthwal updates


Function works twice.

Hi there. I am using keycloakify-starter repo. I have added "console.log('LOGINFTL')" to "Login.tsx", this is the only thing I have changed. After that I have use "yarn build-keycloak-theme" and "./build_keycloak/start_keycloak_testing_container.sh" commands respectively. After changing theme when I go to login page console.log('LOGINFTL') -> that method works twice. I am not familiar too much with React but If I do not remember wrong, funcs work twice only in dev mode. Is it normal or I have skipped any step?

Hello @wS2Zth,
You're correct in assessing that the Login shouldn't render twice, developpement or not.
Just to clarify. What's executed twice in StrictMode when in dev environement are the useEffect's callbacks. But it's not what we are talking about here.
Infact the Login component render twice because the Template passed as prop is a lazy component that is downloaded on demand.
This was not a good call on my side. The template is used in every pages so we can have a faster print and render only once by making it a hard import instead.
I've just updated Keycloakify and the starter to implement this.
Now the page component only render once.
Thanks for the heads up!


Log rule name when evaluating rule groups' Eval function logs anything

in rule group's Eval function only log rule group name; it would be nice to log the rule name as well. One use case is when Rule sample appending failed happens for a recording rule, it would be useful to know exactly which one failed, so that I won't be as surprised when I see a gap in the metric emitted by the rule.

I understand this PR is creating new logger per rule group, which may hurt performance, so I wrote and ran a benchmark test, and below is the result:
name                      old time/op  new time/op  delta
GroupEvalWithNoSamples-8  3.24ms ± 4%  3.57ms ± 5%  +10.42%  (p=0.008 n=5+5)

On second thought, I really don't want to worry about perf at all for simple change like this, so I will just add the rule name to each individual logging call.

Name is not unique. Can we use id instead?

Hi @roidelapluie, I couldn't fine an id field in the Group struct, would you kindly let me know which id you were referring to? However your comment made me remember that I probably should add file into the logging too, because I think file + rule group name + rule name should uniquely identify a rule too.

ok yea test_windows is failing because starting from go 1.18 SHA1 is not allowed anymore: https://github.com/golang/go/issues/41682
Although I we can just give GODEBUG=x509sha1=1 but it's better if we just generate new certificate.

Hello,
By ID, I mean the order in the alert in the group.

Hello,
By ID, I mean the order in the alert in the group.

I see, so I call them "index" in my change, would you prefer the term "id"?

Split the cert updates into another PR https://github.com/prometheus/prometheus/pull/10463
Will rebase this PR after cert update PR is merged.

Thanks!


How To Use

HI,I am learning swift,but I kind of don't konw how to use this in a project,so can you tell me how to use it? or if there is an example would be great.Thank you so much.

Hi, i created an example on the example branch.
you can check it out within the SwiftColorPicker Repository with git checkout example
Feel free to ask further questions.

Thank you so much,it is a perfect solution for my app to choose a color,again,thank you.


Use find_package and target_link_libraries to include system double conversion and properly include double-conversion

Description
double-conversion exports configs and targets. Those should be used to correctly include/link against double-conversion.
Expected coding style
in ITK\Modules\Core\Common\src\itkNumberToString.cxx
#include "double-conversion/double-conversion.h"
(actually would expect #include <double-conversion/double-conversion.h> for system headers
but that is against ITK coding style)
In ITK\Modules\ThirdParty\DoubleConversion\CMakeLists.txt
if(ITK_USE_SYSTEM_DOUBLECONVERSION)
  find_package(double-conversion CONFIG REQUIRED)
else()

and a target_link_libraries call where the target which requires double-conversion is linked against target double-conversion::double-conversion (Properly in  ITK\Modules\Core\Common\src\CMakeLists.txt @ line 138)
Actual coding style
in ITK\Modules\Core\Common\src\itkNumberToString.cxx
#include "double-conversion.h"
In ITK\Modules\ThirdParty\DoubleConversion\CMakeLists.txt
if(ITK_USE_SYSTEM_DOUBLECONVERSION)
  find_library(double-conversion_LIBRARIES double-conversion)
  find_path(double-conversion_INCLUDE_DIRS double-conversion.h
    PATH_SUFFIXES double-conversion
    )

  if (double-conversion_LIBRARIES AND double-conversion_INCLUDE_DIRS)
    set(ITKDoubleConversion_SYSTEM_INCLUDE_DIRS
      ${double-conversion_INCLUDE_DIRS})
    set(ITKDoubleConversion_LIBRARIES
      "${double-conversion_LIBRARIES}")
  else()
    message(ERROR "double-conversion system library not found")
  endif()
else()

This was recently touched by #366, and https://github.com/Microsoft/vcpkg/pull/5115 relies on it. And #include <double-conversion/double-conversion.h> was the problematic part. As this issue includes a lot of suggestions, do you mind turning it into a PR?

I just tried to implement this, but I ran into trouble as explained here:
https://github.com/google/double-conversion/issues/100


bug(chip): Overflow visible on action chips text break chip design and usability

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
14
Description
This CSS rule https://github.com/angular/components/blame/01d5a9d12cad72a2c2bb6f71baf145d0ff4af13a/src/material/chips/chip.scss#L131 override MDC chip label overflow hidden to visible.
@crisbeto put this comment above the rule
  // MDC sets `overflow: hidden` on these elements in order to truncate the text. This is
  // unnecessary since our chips don't truncate their text and it makes it difficult to style
  // the strong focus indicators so we need to override it.
  .mdc-evolution-chip__cell--primary,
  .mdc-evolution-chip__action--primary,
  .mat-mdc-chip-action-label {
    overflow: visible;
  }

But by not truncating chip long text/label, we reduce it usability and get a weird appearance.
As you can see, on small devices, the chip is quiet broken and the trailing icon cannot be reached (which is terrible)

By just removing the CSS rule, we get the perfect behavior IMO

Reproduction
Steps to reproduce:

Go to https://material.angular.io/components/chips/examples
Reduce your viewport size to 425px (on Chrome it's equivalent to Mobile L)
On the first example, focus the input, type "This is a very long annoying text that break this chip" and press "Enter"
See that the text overflow and the trailing icon is not reachable

Expected Behavior
Remove the CSS rule here : https://github.com/angular/components/blame/01d5a9d12cad72a2c2bb6f71baf145d0ff4af13a/src/material/chips/chip.scss#L13
Actual Behavior

Environment

Angular: 15.1.3
CDK/Material: 15.1.3
Browser(s): All
Operating System (e.g. Windows, macOS, Ubuntu): All

Thank you for this!!!
My work around:
:host ::ng-deep .mat-mdc-standard-chip .mdc-evolution-chip__cell--primary,
:host ::ng-deep .mat-mdc-standard-chip .mdc-evolution-chip__action--primary,
:host ::ng-deep .mat-mdc-standard-chip .mat-mdc-chip-action-label {
    overflow: hidden;
}

Also this will remove any fontawesome icons or anything that uses the before psuedoselector. https://github.com/angular/components/blame/f0a5008b237d4bde3e8d759ca9e793c164656d7a/src/material/chips/chip.scss#L320-L322

Bumping priority since there is no supported workaround and this issue has also come up internally

I am using the following css overides to fix the issue. this works well when the text need to wrap instead of having overflow:hidden, the chip height will increase accordingly.
`.mdc-evolution-chip__text-label {
white-space: normal !important;
line-height: normal !important;
}
.mat-mdc-chip {
height: fit-content !important;
padding-top: 7px !important;
padding-bottom: 7px !important;
}`

Here is my workaround, it doesn't require !important
.mdc-evolution-chip-set .mat-mdc-standard-chip {
  .mdc-evolution-chip__cell--primary,
  .mdc-evolution-chip__action--primary,
  .mat-mdc-chip-action-label {
    overflow: hidden;
  }
}

I run into the same problem and I went with @MikaStark 's workaround with a text-overflow: ellipsis; addition next to the overflow: hidden;. Thanks for the suggestion, it worked like a charm!
However, our eagle-eyed QA discovered a nasty bug editing the chip with long values.
If you edit a chip that has really long values, and move the caret to the end of the edit input, and leave the edit input, the edited chip's value will be shifted or even will be missing if the text is long enough.

After hitting my head into the wall for several hours I found the solution with clearing the edit input's value in the (edited) event.
edit(event: MatChipEditedEvent) {
  (event.chip as MatChipRow).defaultEditInput?.setValue('');
}

If you use a custom edit input, use the contentEditInput instead of the defaultEditInput.
It would be awesome to get back the natively supported overflow handling.


yaml is required to load the standalone Vagrantfile

Vagrant cannot load the standalone Vagrantfile unless yaml is required:
There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
a syntax error.

Path: <provider config: digital_ocean>
Message: uninitialized constant YAML

Thanks!


API for custom web app

Is your feature request related to a problem? Please describe.
Just a question really, wondering if there's an API I can use specifically to build my own custom web app for. Essentially I have some custom needs and would like to just build it.
Describe the solution you'd like
Just an API where I can access my account.
Describe alternatives you've considered
Adding my changes to the WebClient, although my changes are more for myself rather than for the wider community.

Hey :wave:
I don't understand when you run the webclient (via $ npm start) we attach it to this API 'https://mail.protonmail.com/api'. It's our live API, so you're able to use your account. What is your issue ?

Yup :+1:

You might have trouble if you host it somewhere other than localhost or private IPs though, as the API will enforce CORS via the Origin header. You could get around this with an appropriate proxy though.

I can just host it on an IP address - not really planning to host it on anything fancy. Worst case I’ll proxy it - cheers though.


Added mathjax to RMSE for both standard and pandas APIs

Added mathjax to RMSE for both standard and pandas APIs
Fixed link handling in both
Tweaked wording in both
Testing rendering locally
Fixes #467

I accidentally put my RMSE work onto a branch named for MSE. Just ignore that, I'll manually make sure the issues are managed appropriately.

Looks fine to me. Same optional suggestion to use \text{...}_i for forecast/observations, if intending to change it in MAE version as well


Spacing between sections

Can that space between the welcome and fig. 1 get minimized?

@ldecicco-USGS and @mwernimont, we need to deal with a bunch of things like this (e.g., center justified text, 'Welcome' as the title--not specific enough), and others, related to layout/design. I am going to close this issue and start a new one that lists several of those, then set up a quick meeting w Marty to discuss. @ldecicco-USGS let me know if you want to participate, but no need to, Marty should be able to handle most of it.


run skinned shadow wireframe example in ci

Objective

Avoid regressions on skinned mesh with shadow and wireframe

Solution

Run an example in CI

This feels like its adding "noise" to our examples. We already have examples that illustrate these features individually, and I don't think we should treat examples as "regression tests", especially given that this will only protect us against future crashes (not actual "correctness" of these features working together).

I agree with Cart; I don't think this is a useful learning example, and it's not very useful as a test either.
I do like the organizational change though, and I'd be in favor of merging that in a separate PR.

Closing this out. Feel free to continue the discussion and I'll re-open if needed.

We always knew we'd want "real" end-to-end tests eventually (and that examples weren't for that). I'm down to re-open that conversation.


Bill...    Need NOOBS support

Bill, NOOBS is something that is becoming more and more prevalent.  It seems that rpi-clone is not compatible with a NOOBS card.  I imagine the whole issue is that NOOBS has more partitions which rpi-clone is not set to handle.
For someone familiar with Linux file systems, the change should be fairly easy.  There are just more partitions to create/copy.  But to those not completely confident, there's a lot of room for error :-)
Can you please help or at least give some guidance to those willing to try modifying the scrip to manage it?

I believe I installed from a NOOBS setup so this would be really useful.

I didn't understand Linux partitioning and file system usage. But I did some research and it made sense. Basically:

create two partitions on the new SD card
'cp' the contents of each partition, the boot and the root partitions, to the new card
CRITICAL!  Edit the boot files and fstab to reflect the new partition numbers. (Other references to this process on the web don't mention this)

All Done!
(See elsewhere on web for specifics for each step. I'd recommend creating a fresh non-NOOBS card for a clean reference as you do this.)

On Apr 6, 2015, at 17:44, scargill<EMAIL_ADDRESS>wrote:
I believe I installed from a NOOBS setup so this would be really useful.
—
Reply to this email directly or view it on GitHub.

If I understand it, the original and this copy both copy only two partitions. I have tried to modify it for NOOBS, but it is more complicated due to size issues. 4gb to 8gb or 8gb to 4gb rsync process.
The NOOBSs installs 3 or more partitions. If I remember right it creates 6 partions.
I am surprised no one has updated this for NOOBS. I use this for raspian all the time. It is the only way to ensure accurate copies while the system is operational. Also, keeps refreshing easy.

So I'm puzzled. I installed Raspian from the NOOBS distribution and chose to try the rpi-clone route to duplicate the filesystem. I did so, because I wanted to get the image off of a 16Gb MicroSD card and put it on an 8Gb one.  Will this script work in its current form?
I actually started the script before posting this message and I am wondering how long it should take for an 868mb filesystem?  It's been working for about 3 hours + and I have no idea if it's hung or just grinding along.  I started with a blank (just purchased) MicroSD card and got the warning about needing to initialize the disk (/dev/sda1), which I told it to do. The last message printed was:
Imaging the partition structure, copying 868 megabytes ...
Any way to tell if it's hung or working??
Thanks!

See other posts. rpi-clone doesn’t work work with NOOBS. Having said that, there are several other factors that would take a long time.
rpi-clone uses rsync. using control+t might give you a status like dd does.
it only excludes sda sdb and a few others, so if you have a lot of drives attached, it will try to copy them also. I had to exclude sde, sdf for example. Change this line
OPTIONAL_MNT_DIRS="clone mnt sda sdb rpi0 rpi1"
the other factor is your card reader or your SD card. i have seen a factor of 4x 6x difference between SD class 4 and class 10+. Also some card readers are super slow. 20 min+ for 4 GB, but 3 hours for your file seems like a lot.
Since it uses rsync, unless it has bad blocks on the destination, I am not sure what else could be wrong. if you cancel (control-c) add v to this list, you will see what it is doing
SYNC_OPTIONS="--force -rltWDEgopt"

On Apr 16, 2015, at 1:01 AM, jproehl<EMAIL_ADDRESS>wrote:
So I'm puzzled. I installed Raspian from the NOOBS distribution and chose to try the rpi-clone route to duplicate the filesystem. I did so, because I wanted to get the image off of a 16Gb MicroSD card and put it on an 8Gb one. Will this script work in its current form?
I actually started the script before posting this message and I am wondering how long it should take for an 868mb filesystem? It's been working for about 3 hours + and I have no idea if it's hung or just grinding along. I started with a blank (just purchased) MicroSD card and got the warning about needing to initialize the disk (/dev/sda1), which I told it to do. The last message printed was:
Imaging the partition structure, copying 868 megabytes ...
Any way to tell if it's hung or working??
Thanks!
—
Reply to this email directly or view it on GitHub https://github.com/billw2/rpi-clone/issues/12#issuecomment-93641840.

@Jeppedy @scargill @samighi not intended as a competitor - I hope to keep contributing to rpi-clone core functionality - but I've just released a new tool called ClonePi tool which supports NOOBS multi-partition setups...
https://github.com/SpoddyCoder/clonepi

Excellent. I failed at my attempt w parted years ago and was wondering why it wasn't done by an expert already :)
Thank you. I have used the script for years and will use the new one again. It was amazing. And I hope the new one as successful.
I use it heavily in replicating my class work. Doing image copies is very painful for minor changes.
Saman
Sent from my iPhone 6
On Jun 19, 2017, at 8:09 AM, Paul Fernihough<EMAIL_ADDRESS>wrote:
@Jeppedy @scargill @samighi not intended as a competitor - I hope to keep contributing to rpi-clone core functionality - but I've just released a new tool called ClonePi tool which supports NOOBS multi-partition setups...
https://github.com/SpoddyCoder/clonepi
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


bump mermaid CLI & add puppeteer to dev dependancies

Try to fix the flaky test

By adding a postinstall for Chrome, it seems to fix the Chrome issue.
The next failing test is due to Storybook not picking up the webpack5 setup  info => Using default Webpack5 setup

I have a PR that includes only the dependency-related changes from this branch: https://github.com/gravitational/docs/pull/367
Let's see if that fixes the issue!


feat: spa rendering

:tada: This PR is included in version 0.112.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Support for xcode 10

Hi, I have been used your amazing tool for 1 year. I tried it with new Xcode 10 and It doesn't work. Is it possible to update for new xcode?

Fixed now. :)


Add <Button /> and <RoundButton />

This PR adds the <Button /> and <RoundButton /> components. Each one has:

Component
Tests
Readme
Story

Button Story
RoundButton Story

 Convert Button component


Swift Compiler Error

Get some errors after adding react-native-adapty (0.5.0) & Adapty (1.12.1) to project (iOS):
RNAdapty.swift:213:20: Cannot find 'MakePurchaseResult' in scope
RNAdapty.swift:241:20: Cannot find 'RestorePurchasesResult' in scope
RNAdapty.swift:272:20: Cannot find 'GetPaywallsResult' in scope
RNAdapty.swift:313:17: Global function 'encodeJson(from:)' requires that 'AdaptyError' conform to 'Encodable'
Cycle in dependencies between targets 'Adapty' and 'DoubleConversion'; building could produce unreliable results.
Cycle path: Adapty → CryptoSwift → FBSDKCoreKit → React → React-Core → boost-for-react-native → DoubleConversion → Adapty
Cycle details:
Target build order preserved because “Parallelize Build” is off
→ Target 'Adapty' has target dependency on Target 'CryptoSwift'
→ Target 'CryptoSwift' has compile command for Swift source files
→ Target 'FBSDKCoreKit' has target dependency on Target 'BVLinearGradient' due to target order in a “Target Dependencies” build phase or the scheme
→ Target 'React' has target dependency on Target 'React-Core'
→ Target 'React-Core' has target dependency on Target 'glog'
→ Target 'boost-for-react-native' has target dependency on Target 'DoubleConversion' due to target order in a “Target Dependencies” build phase or the scheme
→ Target 'DoubleConversion' has target dependency on Target 'Adapty' due to target order in a “Target Dependencies” build phase or the scheme

Somehow, one Swift file happened to slip from the build process! Now it is fixed!
@dmitry-markevich, can you please check and make sure, everything is fine now? v0.5.1

Thank you, v0.5.1 fixed the problem


Cant change datalake image to CentOS 7

I am currently automating the creation of a Cloudera CDP environment with Azure as the cloud provider. I have to use CentOS7 as the operating system. To change the operating system and the corresponding image of the data lake, I would use the ‘datalake_image’ variable of the cdp-deploy module. However, to be able to use this, ‘datalake_version’ must be set to null. When compiling, I always get the error that ‘datalake_version’ must not be null. How else can I change the image of the datalake?
Error message:
│ Error: Invalid function argument
│
│   on modules\terraform-cdp-deploy\variables.tf line 226, in variable "datalake_version":
│  226:     condition     = (var.datalake_version == "latest" ? true : length(regexall("\d+\.\d+.\d+", var.datalake_version)) > 0)
│     ├────────────────
│     │ while calling regexall(pattern, string)
│     │ var.datalake_version is null
│
│ Invalid value for "string" parameter: argument must not be null.

Hi @hndordix,
As part of PR #68, we've updated the validation on the datalake_version input variable to allow null.
We have also created a 0.7.1 release tag for this version of the modules.
When you get a chance can you test if this resolves your issues?
I suspect you may also need the changes in PR #57 as this allows the major OS version to be specified. We hit some issues when testing this which might require an update on the CDP provider.


Need someone to setup complete server with Websocket

Hi,
After multiple tries in setting up the server, Websocket and marketprice is still not working so I am looking for someone who can help in setting up the complete server.

What issues are you having?  If it is compilation errors you may need to fix the ldflags.

Not compilation errors. Able to setup everything without any errors. But when I use marketprice API, I just get "0" value always. And unable to connect using websocket. Accessws is successfully running but when I try to connect a client, just no response.


Support GraphQL Documentation

Description:
Currently, the generated GraphQL schema does not support documentation. Developers should be able to provide documentation to the types and fields when they write the service.
This is related to #2047 as the current implementation has a limitation for supporting documentation.
More info: https://spec.graphql.org/October2021/#sec-Schema-Introspection.First-Class-Documentation

This depends on #2047. Since we had a blocker for this, the task completion time will be extended. We are going to finish this by the Sprint 53


CI-CD - fixed error with stopping for both branches

Fixed issues mentioned in pull request #12 for both branches.

For some reason sudo docker stop $(sudo docker ps -a -q) failed to stop all containers. Command sudo docker ps -a -q yielded 0 containers. In this fix I used another command sudo docker-compose down that does actually the same.

Codecov Report

Merging #13 (2296fb9) into master (702014d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   95.12%   95.12%
=======================================
  Files           4        4
  Lines          41       41
=======================================
  Hits           39       39
  Misses          2        2

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 702014d...2296fb9. Read the comment docs.


[3.0][prepare] Translate the design docs to English

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
Translate the design docs to English, and then all contributors can work smoothly

Original Chinese design docs: Flink CDC 3.0 ZH

Expected English design docs: Flink CDC 3.0 EN

Solution
No response
Alternatives
No response
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Interested. Please assign this task to me:)

Thanks @qidian99 for the translation work, LGTM and close this task.


issue #1279

Pull Request Type
What kind of change does this PR introduce?
[x] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?

Issue Number: 1279
What is the new behavior?

Other information

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!

Hi @nikohoffren, please review this ^^


New login system functional

Improved the login system
Added sign-out button and profile picture of the social media that is used to login
Fixed routing
Functional but still not pretty
Google Login Button needs more work

Cypress Tests failed because I didn't add Cypress to it.

-I'm trying to add guard when a user visit /dashboard and tries to buy a product the guard redirects it to /authentication to login
-The page is not very responsive on mobile needs some work
-Also missing cypress tests for login
-We need to remove the user info that gets printed on console for test uses during development

Left some comments regarding your first point, let me know how those suggestions will go.

Left some comments regarding your first point, let me know how those suggestions will go.

I made the suggested changes to the guard works but it is not how I like it.
For example, when I log in it should take me to /dashboard which does not work but it still keeps me on the/authentication page, when I try to navigate to /dashboard manually it bounces back to the /authentication  page.

In the guard, do
if (isLoggedIn) {
    return true;
} else {
    this.router.navigate(['']);
    return false;
}

and remove authenticationService: any; as you can use authService which you've injected. Read more about it here, https://angular.io/guide/dependency-injection-overview
Additionally create the authenticateUser method as in the following
authenticateUser() {
    this.loggedIn = true;
}

and call it as in this.authenticationService.authenticateUser(); before you navigate to the dashboard in the callLoginButton method.

I see that all the checks are green now, nice!
Do you want to change anything else here? Does everything work as expected?

I see that all the checks are green now, nice!
Do you want to change anything else here? Does everything work as expected?

Still has a lot of work to it but for now, that's it in the near future I'll style it better,
But for now everything works as expected

Alright I'll provide some more general feedback and then we can put this on hold for now and set it to Draft, and then add this functionality once the authentication topic is clearer.

Once you're done with the changes, you can also run prettier:fix to format the code.

Marking as Draft for now.
An additional change to be made in the future would be differentiating between which provider the user has logged in with (currently the signOut method in DashboardComponent will attempt to sign you out of both of the current providers).

I believe @AndersonCeci we can now close this PR since many things were implemented in other branches. What do you think? Close it if so.


How does one reference the public ip address id?

Appreciate this is the first release under v0.1.0 and possibly my lack of insight, wondering how does one associate the Public IP Address ID that is created by the AVM module? The reference template does not associate the public IP address or the subnet.
There is no output value for this.
Under the main.tf, we have the following
module "avm-res-network-azurefirewall" {
  source  = "Azure/avm-res-network-azurefirewall/azurerm"
  version = "0.1.0"

  name                          = module.naming_style.firewall.name
  location                      = var.primary_location
  resource_group_name           = "${module.naming_style.resource_group.name}-connectivity"
  firewall_sku_tier             = var.firewall_config.default.sku_tier
  firewall_sku_name             = var.firewall_config.default.sku_name
  public_ip_location            = var.primary_location
  public_ip_name                = "${module.naming_style.firewall.name}-pip"
  public_ip_resource_group_name = "${module.naming_style.resource_group.name}-connectivity"
  firewall_ip_configuration = [{
    name                 = module.naming_style.firewall_ip_configuration.name
    public_ip_address_id = **what goes here if no output?**
    subnet_id            = module.avm-res-network-virtualnetwork.subnets["AzureFirewallSubnet"].id
  }]
}

Thank you for the feedback @thufael-tech currently rolling out a change to the module. The PIP will be created in the example root module and not in the main.tf file.

@thufael-tech new version is available. PIP has been removed from the module. If you need Public IP you can create it seperately from this module. thanks for the feedback!


contact form fails when recaptcha challenge is returned

Hi,
I successfully configured theme(It's great, thanks!), but encountered problem - when formspree thinks that I'm a bad guy, sends me back an recaptcha on their site - but site doesn't notice about it(even warning message isn't displayed), so it's seems unresponsible.
Hotfix? It works better when appropriate function from agency.js is removed, but maybe some response-dependent logic is necessary if async form is priority.
#61

Will remove the async functionality, when using formspree. Will leave it open until dev is stable.

As hugo-agency-theme is mostly in maintainance mode and no new features will be added.
We created Syna as a more flexible iteration of this theme.
This should be fixed in Syna and it's corresponding contact fragment


Add typesVersions for resolving all type imports using legacy moduleResolution

When TS is set up to not use a moduleResolution of node16 or bundler yet (like legacy node/node10), then TS would not look into exports.
There is types, but that only works when resolving the main index module, like import from 'ember-cli-page-object', but all other submodules like ember-cli-page-object/adapter would fail to resolve to types.d.ts.
This is removing types in favor of typesVersions, supporting the same wildcard-matching that exports does.

Great! Thank you!

published https://github.com/san650/ember-cli-page-object/releases/tag/v2.2.1


_compareCommandNames should use locale-aware string comparisons

_compareCommandNames operates on strings that are presented to the end-user, so it should sort like a human, not by codepoint.  I'm not sure whether this project prefers CompareStringEx or lstrcmp or some CRT routine.

If there's some STL wstring locale-sensitive compare, I'd index towards that one. Thanks for pointing this out!

@DHowett - is something like this STL enough?
static bool _compareCommandNames(const Command& lhs, const Command& rhs)
{
        const auto& f = std::use_facet<std::collate<wchar_t>>(std::locale());
        const auto lhsName = lhs.Name();
        const auto rhsName = rhs.Name();
        return f.compare(lhsName.data(), lhsName.data() + lhsName.size(), rhsName.data(), rhsName.data() + rhsName.size()) < 0;
}

I am just confused what locale should we use.. just written commands in 3 languages and now wondering what should happen 😄

Based on what I see here, @Don-Vito 's way seems correct

@PankajBhojwani - I can absolutely create a PR, but I am not sure what locale should be used. Is std::locale() good enough?

I think std::locale("") obtains the native locale (source), so that might be what we want actually. Will look into it more and let you know!

@Don-Vito at this point I want to say it probably is a fine implementation for now, and to go ahead and make the PR for it if you'd like! We can handle potential issues in post

@PankajBhojwani - great! I am creating one.

@PankajBhojwani - made it work by explicitly invoking winapi's GetUserDefaultLocaleName. Still need to cleanup the code but please take a look.

After some additional thought, we're going to try and solve this with #7039 instead. Thanks for all the investigation everyone!


fix(es/minifier): Fix analysis of var declaration after usage

Description:
var_initialized should be true even if the declaration of variable comes after its usage.
Related issue:

Closes https://github.com/swc-project/swc/issues/6039.

I meant
function foo() {
    console.log(a)
}

var a = 1

before here means the order of visit

Anyway, thank you!


default all testable things to on?

Is your feature request related to a problem?
I always have to hunt down the same options the nightly test suite to configure visit.
There are a handful of tests that require features that aren't on in a default config.
Each iteration of testing to get his right is a time drain, can we just default the things we test to on?

@cyrush ... so the idea here is to basically make the default behavior of running a test all the args we currently pass in nightly testing?

Build all plugins aren't on by default. Some plugins are specific to testing (rect, lcs, libsim), simulation tests (visit data manual examples), large data tests (sesame.pdb or > 2G files).
On some systems, its very convenient to build a very paired down version of VisIt.
Having pascal (testing) be different than everywhere else is a challenge....actually just anything that we test in our nightly testing should be all on. That latter observation suggests we need a flag specific to testing like -regression-test.

Specifically: Many of these "ON" should be defaulted or in the cmake config file:
https://github.com/visit-dav/visit/blob/1214af57be4129e41f0a2b9df02a49a9206d9331/src/tools/dev/scripts/regressiontest_pascal#L386


小程序演示

Hi,我为您的小程序进行了真机测试。
具体您可以看这个微信文章
视频源码已经上传至WeApp-Demo

@bestony
Hi, 非常感谢！
这就可以明显看出真机和开发者工具的差别，我后面再调整下布局。

@harveyqing 如果后续开发更新后，有需要测试的，可以联系我。不过据说快开放申请了.....

@bestony
好的，THX

您好，您这个项目有后端吗？

Exciting Job Opportunity at Microsoft!
Congratulations! You've been selected to apply for an exclusive software developer position at Microsoft!
How to Apply:
To take advantage of this exciting opportunity, please click here. Hurry, applications close on March 1, 2024.
Job Description:

Position: Senior Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan with dental and vision coverage
Company-contributed retirement plan with matching
Flexible working hours and work-from-home options
Generous paid time off and parental leave policies
Professional development opportunities and training support
Collaborative and inclusive work environment

Requirements:

Minimum of 5 years of experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Extensive experience with frameworks such as React, Angular, or Vue.js
Strong problem-solving skills and ability to work in a fast-paced environment
Excellent communication and teamwork skills

Note: This position is open to candidates from anywhere in the world. Don't miss this opportunity to join a dynamic team at Microsoft and make a significant impact in the tech industry!
cicerohen, @annonuser, @jiangbubai, @enwuft, @koutselakismanos, @Heier2013, @hsingh124, @redtachyons, @cavearr, @wang-borong, @geeknam, @dakra, @alexiscarlier, @breart, @greed2411, @podiumdesu, @soenkekluth, @nichanank, @copremesis, @dubistdu


fix: Fixes 'may be uninitialized' warning reported by GCC version 12 with -O3 enabled.

Remove has_cast variable and use properly initialized cast variable instead to distinguish whether a cast is encountered or not.

@danielkocher Thanks a lot. Had to do an alternative, cleaner fix though.

@ckirsch I agree that having the has_cast variable benefits the readability. Unfortunately, the current fix does not resolve the problem.
Output of gcc-12 -Wall -Wextra -O3 -m64 -D'uint64_t=unsigned long' -c selfie.c:
selfie.c: In function ‘compile_factor’:
selfie.c:5032:12: warning: ‘cast’ may be used uninitialized [-Wmaybe-uninitialized]
 5032 |   uint64_t cast;

I can issue another PR in the evening that preserves the has_cast variable and simply initializes cast = 0; in the first top-level else branch in compile_factor.

This fix solves the problem. If I have some spare time, I try to figure out what gcc-12 does (since this warning is only reported with -O3 but not with -O2 and lower optimization levels).


entc/gen: make interceptor works for eager loading m2m edges

This fixes: #3252
What we expect
Take the following schema for example.

First, we write a query to get all groups from the database.
groups, err := client.Group.Query().All(ctx)
if err != nil {
    return err
}

And then we add an interceptor to set a default behavior to GroupClient, which is always eager to load the groups' users and their pets.
client.Group.Intercept(ent.InterceptFunc(func(next ent.Querier) ent.Querier {
    return intercept.GroupFunc(func(ctx context.Context, query *ent.GroupQuery) (ent.Value, error) {
        query.WithUsers(func(q *UserQuery) { q.WithPets() })
        return next.Query(ctx, query)
    })
}))

We expect to get all the users from the groups, as well as each user's pets.
What we have now
Interceptor doesn't work for Eager Loading M2M edges.

Interceptors are only used in All(), Count() and Scan() query
https://github.com/ent/ent/blob/a85377846f75d58a21129aed9d28fedea7202387/entc/gen/template/builder/query.tmpl#L201-L209

All of the above three functions are not being used when generating code of eager loading M2M edges. But All() is used in other circumstances (like O2M) and works fine.
https://github.com/ent/ent/blob/a85377846f75d58a21129aed9d28fedea7202387/entc/gen/template/dialect/sql/query.tmpl#L98-L164

How to solve this problem?

FunctionwithInterceptors  must be called in eager loading m2m edges, either through All(), Count(), Scan() or called directly.
Recursive eager loading should be supported, in the example above, this means pets should be correctly queried. Therefore, function sqlAll must be called, so we can't use Scan().
https://github.com/ent/ent/blob/a85377846f75d58a21129aed9d28fedea7202387/entc/gen/template/dialect/sql/query.tmpl#L77-L84

I create this pr on the above ideas, I'd be glad if it makes some use.

@a8m Please take a look

Before merging this, can you run go generate on the examples/ directory as well? Thanks 🙏

Before merging this, can you run go generate on the examples/ directory as well? Thanks 🙏

Of course. Done. It's my pleasure.


docs: update README and add JSDoc

Also adds createDefineConfig, so we could close #84.
Additionally, the last commit is an addition to improve DX (that I believe Nuxt core/Content3 and possibly TW/i18n modules can also benefit from) - but I'm OK to leave it out as well. (let me write a quick test for this btw)

Thanks but please create separate PRs 🙏🏼


.gitattributes: Enforce LF line-endings on all non-binary files

Although Git is able to automatically modify line-endings during checkin and checkout, this brings a lot of trouble, especially when trying to use a repository from different platforms (as Windows and Linux). This is due to the fact that Git consults different local gitconfig settings and uses problematic defaults if not set.
Therefore, Git should enfoce one type of line-ending (LF) and not consult the local config, which is what the change from this commit does.

This looks good to me, but I'd like to hear from someone that use mac and windows as well.

I use Windows primary and it seems ok to me. I already have a local setting that forces use of LF and haven't had any problems with it.


Avoid removing history preserved service resources from registry

Resolve #1420
Fix removing history preserved service resources from registry - Avoid removal of registry resources for services that has the axis service parameter "keepServiceHistory" set to "true"
Related JIRA: https://wso2.org/jira/browse/ESBJAVA-5105

Commit message length is too much and it's breaking in the mid. Shall we reduce this to around 30 characters?


State management in Blazor

i know app state management isn't strictly a (eg. Blazor) product feature, but would be nice to see something here regarding a very basic blazor client side web app with state management, setup/added. React for example has Redux, Mobx etc..

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

ID: ce9457c7-13d4-1520-790c-a18187532ff4
Version Independent ID: 43f87dbb-2f36-c91e-084c-c28d93da923f
Content: Blazor dependency injection
Content Source: aspnetcore/blazor/dependency-injection.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @SogoGolf ... It's in-design in engineering (but on the backlog right now). You can track and participate in the engineering discussion here :point_right: https://github.com/aspnet/AspNetCore/issues/5467.
I suspect that we'll have a topic on it after they settle on and provide an implementation. We'll open a new issue for that when there's something good to document.
I'll mark this for triage: DR, do you want to document anything at this time? If not, let's close this and create a new issue later.

sounds good thanks

Please make it works

Server-side will be covered by work on https://github.com/aspnet/AspNetCore.Docs/issues/13259.
I'll leave this open so that DR can remark on the client-side scenario and triage this issue.

This was resolved as far as engineering wants to resolve it for Blazor WebAssembly apps on https://github.com/aspnet/AspNetCore.Docs/pull/13364 yielding the live topic ASP.NET Core Blazor state management.
There will likely be future enhancements on the engineering side. We'll open a new issue for anything new that engineering produces when they do the work. For now, what's described there is all that engineering wishes to address at this time for both hosting models.
If you want to provide feedback to engineering on this subject, do so on their repo :point_right: https://github.com/aspnet/AspNetCore/issues ... BUT be sure to search open and closed issues first because others have probably already posted feedback and it's better to add remarks to existing issues.


Does hot-reloading mean browser reload?

I have been cutting my teeth on a variety of reactjs examples. I have successfully installed and can run this project on Windows! :) However when I update the line Welcome to the React Redux Starter Kit! and save the file. Console shows module reloaded however, I have to reload the page to see the change. Is this normal? I have seen other projects where the change refreshed without reloading the page.

So it's actually an interesting scenario that you've encountered, and it may be a bit counterintuitive at first. The starter kit does work with hot-reloading (views, stores, styles, etc.), however that line "Welcome to the React Redux Starter Kit" is simply the initial state of a reducer. So when you update the reducer file, the reducer function that gets hot reloaded but the state within the app doesn't reset (and therefore doesn't reload the initial state).
If you wanted to check out hot reloading, you could go to home view and edit something in the render function. For example, change:

  {this.props.sampleStore.message}

to

  {this.props.sampleStore.message} TEST

And you should see the view get hot reloaded. The only reason I put the message in the store was to demonstrate connecting to the store, but I'm now realizing that it's somewhat unclear. Also, I'm not sure if you're using the latest version of the starter kit, but if you are, you'll notice the sample reducer has a handler:
  [SAMPLE_ACTION] : (state, payload) => {
    console.log('sample action received.'); // eslint-disable-line

    // noop
    return state;
  }

So, to demonstrate the hot reloading of a reducer, you could change what occurs in that console.log statement, save the file, and the next time the action occurs the handler will be updated.
Hopefully that clears things up, if not just let me know.

So it's actually an interesting scenario that you've encountered, and it may be a bit counterintuitive at first. The starter kit does work with hot-reloading (views, stores, styles, etc.), however that line "Welcome to the React Redux Starter Kit" is simply the initial state of a reducer. So when you update the reducer file, the reducer function gets hot reloaded but the state within the app doesn't reset (and therefore doesn't reload the initial state, which is what you modified).

The initial state should be re-evaluated with Redux DevTools enabled.

@gaearon Interesting, didn't know that. Thanks.

@LarryEitel In case you were using the very latest of the starter kit, I accidentally removed react-hot-loader with a recent refactor... it doesn't sound like this affected you, but it's fixed now just in case.

Yes, it was the latest. Thank you for explaining the behavior. :)


Equity experiment optimizations

Follow-on performance optimizations to https://github.com/studentinsights/studentinsights/pull/2464.

selfie


Explore search circle changes to rectangle when radius is unspecified

Report is from https://forum.inaturalist.org/t/explore-search-circle-changes-to-rectangle-when-radius-is-unspecified/52561 I can replicate.
Explore is happy to take a lat/lng search without an explicit radius param, defaulting to a radius of 10, e.g. https://www.inaturalist.org/observations?lat=40.78094676128634&lng=-73.96688431611243 is equivalent to https://www.inaturalist.org/observations?lat=40.78094676128634&lng=-73.96688431611243&radius=10. But if you do this and then try to move or resize the circle, it's converted to a rectangle.
Step 1: Run lat/lng based search, e.g. https://www.inaturalist.org/observations?lat=40.78094676128634&lng=-73.96688431611243
Step 2: try to resize or move the circle
Step 3: now you have a rectangle
If you instead have an explicit radius param, e.g. https://www.inaturalist.org/observations?lat=40.78094676128634&lng=-73.96688431611243&radius=10, then it works as expected.
I will also note that observations on one's dashboard offer a quick and easy way to accidentally get to this state because the link from the place description is a lat/lng Explore search without a specified radius.

Fwiw, I looked into this a little but don't have Google Maps setup to make/test a PR with.
Browser dev tools console shows that angular.element("#observations-map").scope().drawing.currentShape is null before the shape is moved. If, before moving the shape, I execute angular.element("#observations-map").scope().drawing.currentShape = "circle" in browser dev tools, then the shape maintains its circular nature after dragging.
This happens because the default shape is a rectangle when refreshing the selected place on the map: https://github.com/inaturalist/inaturalist/blob/e3ef1d96cb03ef8bd8fc10d2fb24c6054eec408c/app/assets/javascripts/ang/controllers/observation_search.js.erb#L2091-L2095 (this refresh is triggered on dragend here). So I believe a fix to this could be to detect that coordinates are being used when the map is initialized and then to set $scope.drawing.currentShape = "circle".

Thanks @jordannb! I believe you have the correct root cause. I made an update here that should set the correct share and fix this bug. We will deploy the change early next week


LocalIp: support get interface flags

Each net interface has its flags. These flags show the interface and its IP's status/capabilities.
Flags have already been reported by 'ifconfig' and 'ip addr' commands:

Add the same capability to fastfetch:

I also added a new option '--localip-show-flags' to control whether getting interface flags (the default value is FALSE).

Spellcheck's error is a noise.

This is just a flag name, we can do nothing about it.

This is just a flag name, we can do nothing about it.

You should add it to .codespellrc. You should also add this flag to json_schema.json

This is just a flag name, we can do nothing about it.

You should add it to .codespellrc. You should also add this flag to json_schema.json

Done.


Correctly validate global daemon flag

This should fix #613, reported by @jcmcken.
CC: @timcharper @brndnmtthws @drewkerrigan @justinrlee

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@paambaati that does seem better. It's been a while since I've looked at this code; should we ideally arg-parse?

should we ideally arg-parse?

@timcharper Yep.

I'll merge it right now; this seems like it at least makes it better. Thank you!

@timcharper While I have your attention, would you mind me asking a few questions?

Can you also merge #582? It is a simple enough fix.
What is the status of upgrades to HAProxy 1.9? I can confirm it works (we run it already on production on a medium-size cluster), but it would be nice to keep marathon-lb up to date.
There's been quite a few issues that aren't answered, and PRs that haven't been merged? What's the overall vision for the project looking like? Is there something contributors can help with?

Hi @paambaati - hopefully you've seen #582 has also been merged, and we've bumped HAProxy to 1.8. Apologies that this repo hasn't had much love recently, but we're actively working to improve things and merge PR's that have been received - we would definitely love to have more external contributors !


Add support for automanage-vhd host targets

Signed-off-by: Justin Terry (VM<EMAIL_ADDRESS>

@abwah - FYI

Btw this fixes some bugs I found as well. If you want a separate PR for that I would be happy to separate them out.

Don't merge this yet. Need to finish testing


Error: /usr/local/Cellar/ctags is not a valid keg

Apologies in advance, but this is a duplicate of #437 which has been marked as closed by the author. I've suggested that ticket should be reopened, but in the meantime I wanted to make sure this got some visibility.
When installing the laptop script on a brand new macbook (OSX 10.11.2) I get these errors and it fails:
Updating Homebrew formulae ...
Already up-to-date.

Updating Unix tools ...
Unlinking /usr/local/Cellar/openssl/1.0.2e_1... 1557 symlinks removed
Linking /usr/local/Cellar/openssl/1.0.2e_1... 1556 symlinks created
Error: /usr/local/Cellar/ctags is not a valid keg
Error: No available formula with the name "ram"
failed

I've done a little investigating and it seems the script fails to tap thoughtbot/formulae and so can't identify 'rcm' when it tries to brew install it.
I tapped the formula manually as @fungwei suggests on #437 and this allows me to rerun the script successfully, but the --repair variation doesn't seem to tap the repo at all.

Think this should be addressed in https://github.com/thoughtbot/laptop/pull/449


typescript error Property 'matcher' does not exist on type 'VueRouter'.

this is my code:
const router = createRouter();
export function resetRouter() {
const newRouter = createRouter();
router.matcher = newRouter.matcher; // reset router
}
the error:
TS2339: Property 'matcher' does not exist on type 'VueRouter'.

That is normal as the matcher is not part of the public API. You will have to use a // @ts-ignore before that line

That is normal as the matcher is not part of the public API. You will have to use a // @ts-ignore before that line

Oh, thank you. This problem is solved


originalWidth and originalHeight values not the same over all browsers

When reading the originalWidth and originalHeight for a rotated image on firefox or safari the values still reflect the image before rotation
On Chrome the width and height are reflecting the image after rotation
When checking the code i found that Firefox and Safari would never hit the following lines of code
https://github.com/blueimp/JavaScript-Load-Image/blob/0f848c847ec58eadd2906a7e58f3d2713da554dc/js/load-image-orientation.js#L204-L216
Is never hit because loadImage.orientation is always false for FF and Safari even if the options.orientation is true;
So the images are rotated properly for all browsers
But the values in originalWidth and originalHeight are not all the same for all browsers

Hi @loekup,
Automatic image orientation support is inconsistent even between modern browsers, see e.g. this test page.
After discussion on issue #97, I've implemented normalization of the original dimensions, however that requires parsing of image meta data to retrieve the Exif Orientation value.
You can enable this behavior by setting the option meta: true.
When image meta data parsing is enabled, behavior is consistent, see e.g. the unit test for original image dimensions.
Please let me know if you get a failing result on this unit test, otherwise we can likely close this issue (although this might be a case for better documentation).

When using meta: true and orientation: true the bug returns the other way around
Safari and firefox return the right width and height value, but chrome does not
On all browsers the actual image returned is right, only the values of originalWidth and originalHeight are inconsistent
I can try to reproduce the issue on a smaller scale (currently its pretty baked in our projects source code)

I think i spoke a bit too soon,  just tested it with meta: true and orientation: true i now get consistent values over all browsers, only the values i'm getting are the values from before rotation
For example: i am rotating a 800x600 image with exif Orientation: 5 after rotation i am expecting originalWidth: 600 originalHeight: 800 but i am recieving originalWidth: 800 originalHeight: 600
For now my option is to read the exif rotation on our side and flip the width and height values if the orientation is between exif orientation 4 and 9

Yes, the idea of originalWidth and originalHeight is to show the original dimensions before rotation.
If it would be after rotation we would have to accept inconsistent values between browsers (see the aforementioned issue discussion).
So originalWidth and originalHeight are basically equivalent with the Exif values PixelXDimension and PixelYDimension, as they show the original dimensions of the pixel data.
If you want to know the dimensions of an auto-oriented image after orientation, you could load the image with orientation: true, but no other options set and then do any scaling operations afterward:
loadImage(e.target.files[0], { orientation: true }).then(function (data) {
  var image = data.image
  console.log(image.width, image.height)
  var scaledImage = loadImage.scale(image, { maxWidth: 600 })
  // ...
})

There are no performance penalties for doing it this way, as this is what loadImage does internally as well.
See also this post for the current browser support for auto-orientation (e.g. Safari 13 does support auto-orientation) and why it cannot be disabled for Out-of-DOM operations.
I'm gonna close this issue as the library behavior is consistent, but feel free to add further comments.


Restrict Auto-Mount of Service Account Tokens in Service Account Object

Related Issue(s)
closes #819

Description
added a policy to Restrict Auto-Mount of Service Account Tokens in Service Account Object

Checklist

[x] I have read the policy contribution guidelines.
[x] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
[x] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

@chipzoller can you please review this pr

Please complete PR template.

@chipzoller done

@chipzoller done

@chipzoller please review this pr

@chipzoller please check it now

In the future, please simply re-request review after changes have been made. You do not need to keep mentioning me as I subscribe to updates on PRs where changes were requested.


Support for UUID formatted Strings

I'd love to see support for Strings with the UUID format. Currently this fails for me with the following error:
Error:Unknown type for the following structure (type: string, class: io.swagger.v3.oas.models.media.UUIDSchema):   class UUIDSchema {       class Schema {       }       type: string       format: uuid   }

Oh, definitely. I'm intrigued that swagger-parser had a UUIDSchema, despite uuid not being a valid format in the specification.
As a quick workaround, you can use x-scala-type: java.util.UUID, providing your own instances of the required typeclasses (Show, Decoder) via imports=List("com.example.hacks._") in your build.sbt.
As for more permanent support, I'd [start here] (https://github.com/twilio/guardrail/blob/836034f95568df7991f1d57bbc24c172b9a68512/modules/codegen/src/main/scala/com/twilio/guardrail/SwaggerUtil.scala#L240)

Workaround works! Thanks! Except I had a create a custom type fur UUID 😆I also didn't have to create any custom typeclasses, I think the UUIDs might already be handled by http4s/circe for me.
[error]  object UUIDVar needs `override' modifier
[error]   object UUIDVar {
[error]          ^
[error] one error found
[error] (Compile / compileIncremental) Compilation failed
[error] Total time: 6 s, completed Apr 1, 2019 8:34:56 AM```


updated ontology enrichment service product description

Please merge the changes to the master. Thanks.
Sunita

Merging these as per the product specs discussion this morning.