can't ignore unavailable video from playlist

DO NOT DELETE THIS! Please take the time to fill this out properly. I am not able to help you if I do not know what you are executing and what error messages you are getting. If you are having problems with a specific video make sure to include the video id.
To Reproduce
Steps to reproduce the behavior:
What code / cli command are you executing?
I am running:
URL = "https://www.youtube.com/playlist?list=PLPTV0NXA_ZShaln9GfiHO_c0HzOSqLOGv"
laylist_info = ydl.extract_info(URL, download=True)

Which Python version are you using?
Python 3.11
Which version of youtube-transcript-api are you using?
2024.7.16
Expected behavior
the playlist have 16 video that video "bbWTBp8ER4M" is not from them.
but the above code error that ERROR: [youtube] bbWTBp8ER4M: Video unavailable. This video has been removed by the uploader
Actual behaviour
it connote ignore this video and go to download other videos.
For example: Instead I received the following error message:
Removing cache dir /root/.cache/yt-dlp ...
[youtube:tab] Extracting URL: https://www.youtube.com/playlist?list=PLPTV0NXA_ZShaln9GfiHO_c0HzOSqLOGv
[youtube:tab] PLPTV0NXA_ZShaln9GfiHO_c0HzOSqLOGv: Downloading webpage
WARNING: [youtube:tab] YouTube said: INFO - 1 unavailable video is hidden
[youtube:tab] PLPTV0NXA_ZShaln9GfiHO_c0HzOSqLOGv: Redownloading playlist API JSON with unavailable videos
[download] Downloading playlist: ML project based course: Learn Explainable AI (XAI) and publish reesarch paper
[youtube:tab] PLPTV0NXA_ZShaln9GfiHO_c0HzOSqLOGv page 1: Downloading API JSON
WARNING: [youtube:tab] Incomplete data received. Retrying (1/3)...
[youtube:tab] PLPTV0NXA_ZShaln9GfiHO_c0HzOSqLOGv page 1: Downloading API JSON
WARNING: [youtube:tab] Incomplete data received. Retrying (2/3)...
[youtube:tab] PLPTV0NXA_ZShaln9GfiHO_c0HzOSqLOGv page 1: Downloading API JSON
WARNING: [youtube:tab] Incomplete data received. Retrying (3/3)...
[youtube:tab] PLPTV0NXA_ZShaln9GfiHO_c0HzOSqLOGv page 1: Downloading API JSON
WARNING: [youtube:tab] Incomplete data received. Giving up after 3 retries
[info] Playlist description is already present
[youtube:tab] Playlist ML project based course: Learn Explainable AI (XAI) and publish reesarch paper: Downloading 16 items of 16
[download] Downloading item 1 of 16
[youtube] Extracting URL: https://www.youtube.com/watch?v=bbWTBp8ER4M
[youtube] bbWTBp8ER4M: Downloading webpage
[youtube] bbWTBp8ER4M: Downloading ios player API JSON
ERROR: [youtube] bbWTBp8ER4M: Video unavailable. This video has been removed by the uploader
---------------------------------------------------------------------------
ExtractorError                            Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in wrapper(self, *args, **kwargs)
   1625                 try:
-> 1626                     return func(self, *args, **kwargs)
   1627                 except (DownloadCancelled, LazyList.IndexError, PagedList.IndexError):

17 frames
[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in __extract_info(self, url, ie, download, extra_info, process)
   1760         try:
-> 1761             ie_result = ie.extract(url)
   1762         except UserNotLive as e:

[/usr/local/lib/python3.10/dist-packages/yt_dlp/extractor/common.py](https://localhost:8080/#) in extract(self, url)
    739                         url if self.get_param('verbose') else truncate_string(url, 100, 20)))
--> 740                     ie_result = self._real_extract(url)
    741                     if ie_result is None:

[/usr/local/lib/python3.10/dist-packages/yt_dlp/extractor/youtube.py](https://localhost:8080/#) in _real_extract(self, url)
   4266             if reason:
-> 4267                 self.raise_no_formats(reason, expected=True)
   4268

[/usr/local/lib/python3.10/dist-packages/yt_dlp/extractor/common.py](https://localhost:8080/#) in raise_no_formats(self, msg, expected, video_id)
   1262         else:
-> 1263             raise ExtractorError(msg, expected=expected, video_id=video_id)
   1264

ExtractorError: [youtube] bbWTBp8ER4M: Video unavailable. This video has been removed by the uploader

During handling of the above exception, another exception occurred:

DownloadError                             Traceback (most recent call last)
[<ipython-input-34-a1c0062d6a1e>](https://localhost:8080/#) in <cell line: 3>()
     16         print('-' * 50, '\n')
     17
---> 18         download_video(url, base_path,
     19                        playlist_start=playlist_start,
     20                        skip_download=True,

[<ipython-input-32-3da3d5b558ba>](https://localhost:8080/#) in download_video(URL, base_path, playlist_start, skip_download, subtitleslangs, max_resolution)
     36            with yt_dlp.YoutubeDL(youtube_dl_options) as ydl:
     37                ydl.cache.remove() # Clear cached information
---> 38                playlist_info = ydl.extract_info(URL, download=True) # Get video metadata
     39                print('*'*50)
     40                ydl.prepare_filename(playlist_info) # Prepare filename based on metadata

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in extract_info(self, url, download, ie_key, extra_info, process, force_generic_extractor)
   1613                     raise ExistingVideoReached
   1614                 break
-> 1615             return self.__extract_info(url, self.get_info_extractor(key), download, extra_info, process)
   1616         else:
   1617             extractors_restricted = self.params.get('allowed_extractors') not in (None, ['default'])

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in wrapper(self, *args, **kwargs)
   1624             while True:
   1625                 try:
-> 1626                     return func(self, *args, **kwargs)
   1627                 except (DownloadCancelled, LazyList.IndexError, PagedList.IndexError):
   1628                     raise

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in __extract_info(self, url, ie, download, extra_info, process)
   1780         if process:
   1781             self._wait_for_video(ie_result)
-> 1782             return self.process_ie_result(ie_result, download, extra_info)
   1783         else:
   1784             return ie_result

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in process_ie_result(self, ie_result, download, extra_info)
   1909             self._sanitize_thumbnails(ie_result)
   1910             try:
-> 1911                 return self.__process_playlist(ie_result, download)
   1912             finally:
   1913                 self._playlist_level -= 1

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in __process_playlist(self, ie_result, download)
   2056                 f'of {self._format_screen(n_entries, self.Styles.EMPHASIS)}')
   2057
-> 2058             entry_result = self.__process_iterable_entry(entry, download, collections.ChainMap({
   2059                 'playlist_index': playlist_index,
   2060                 'playlist_autonumber': i + 1,

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in wrapper(self, *args, **kwargs)
   1624             while True:
   1625                 try:
-> 1626                     return func(self, *args, **kwargs)
   1627                 except (DownloadCancelled, LazyList.IndexError, PagedList.IndexError):
   1628                     raise

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in __process_iterable_entry(self, entry, download, extra_info)
   2088     @_handle_extraction_exceptions
   2089     def __process_iterable_entry(self, entry, download, extra_info):
-> 2090         return self.process_ie_result(
   2091             entry, download=download, extra_info=extra_info)
   2092

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in process_ie_result(self, ie_result, download, extra_info)
   1859             # We have to add extra_info to the results because it may be
   1860             # contained in a playlist
-> 1861             return self.extract_info(
   1862                 ie_result['url'], download,
   1863                 ie_key=ie_result.get('ie_key'),

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in extract_info(self, url, download, ie_key, extra_info, process, force_generic_extractor)
   1613                     raise ExistingVideoReached
   1614                 break
-> 1615             return self.__extract_info(url, self.get_info_extractor(key), download, extra_info, process)
   1616         else:
   1617             extractors_restricted = self.params.get('allowed_extractors') not in (None, ['default'])

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in wrapper(self, *args, **kwargs)
   1642                     self.report_error(msg)
   1643                 except ExtractorError as e:  # An error we somewhat expected
-> 1644                     self.report_error(str(e), e.format_traceback())
   1645                 except Exception as e:
   1646                     if self.params.get('ignoreerrors'):

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in report_error(self, message, *args, **kwargs)
   1090         in red if stderr is a tty file.
   1091         """
-> 1092         self.trouble(f'{self._format_err("ERROR:", self.Styles.ERROR)} {message}', *args, **kwargs)
   1093
   1094     def write_debug(self, message, only_once=False):

[/usr/local/lib/python3.10/dist-packages/yt_dlp/YoutubeDL.py](https://localhost:8080/#) in trouble(self, message, tb, is_error)
   1029             else:
   1030                 exc_info = sys.exc_info()
-> 1031             raise DownloadError(message, exc_info)
   1032         self._download_retcode = 1
   1033

DownloadError: ERROR: [youtube] bbWTBp8ER4M: Video unavailable. This video has been removed by the uploader

Hello @MammadTavakoli,
it looks like you're using yt-dlp, not youtube-transcript-api. Please open your issue there.


Azkaban multi web server configuration for HA

We observed that if multiple web servers are used in Azakaban. Schedules gets executed multiple times.Is it because of config issue? Also Azkaban there is not proper documentation for HA configuration. Can someone redirect to good documentation?

HA web server is not supported in Azkaban


In a single-tab-mode, add option "Show statistics on another tab"

I suggest to add a new option "Show statistics on another tab (for single tab mode only)" in Preferences.
If it is true AND the option "Show results in a single/multiple tab" = single, then:

show Statistics in additinal tab
othewise - leave it as it is now (current behavior).

Here on the screen - the panel highlighted with yellow color will be moved to additional tab highlighted with red color.

Why?
Some users don't care a lot about the rows affected and other statistic info.
Some users need more space for data with results of the queries.
If users want to see the statistics, they still can do that on this separate tab.
Additional scrolling of a scrollbar (where it can be avoided) is not convenient for a user.
I don't treat it as a violation of principle of a single tab, since it is done as additional option in Preferences (everyone can choose what option is more convenient for himself).

Thanks for your report.


Why is the behavior of aspect different in different scenarios

Why is the behavior of the aspect config option in different scenarios? It's not intuitive and it's confusing also. https://github.com/JonasKruckenberg/imagetools/blob/main/packages/core/src/transforms/resize.ts#L60

The idea of the aspect directive is to resize the source image to fit a given aspect ratio, while removing as little from the original image as possible.
This is why we need to determine which sides to resize based on the source aspect ratio.

I am not sure if it's only me but this behavior felt very confusing to me. Instead of an image with an incorrect aspect ratio (it's relative), I expected imagetools to return an enlarged image.

And I had to correct this behavior in astro-imagetools as it confused multiple users.


No error message displayed , if unregistered email  is given for password recovery

Issue: No error message displayed , if unregistered email  is given for password recovery
steps1: go to https://staging.fightpandemics.work/auth/login
2. click on "Forgot password"
3. Enter email which is NOT registered with FP
4. click on 'Email Recovery'
Actual: No error message, redirected to recovery message page
Expected: Error message:similar ' We couldn't find your account with that email''

Checked with @joshmorel . Closing this as he will be working with rate limiter.
https://postmarkapp.com/guides/password-reset-email-best-practices#how-do-i-make-sure-my-password-reset-emails-are-secure


⚠ Fortnite has degraded performance

In 79abc81, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1550 ms

Resolved: Fortnite performance has improved in 4599ea2.


Prepare for release 0.3

Description

Draft release for 0.3
Changelog: https://github.com/autonomousapps/gradle-script-grammar/pull/1

@autonomousapps tagging for review

Version 0.3 released and these changes on main. Thanks!


Single-Cell Mass Cytometry Analysis of the Human Endocrine Pancreas

Publication:
Single-Cell Mass Cytometry Analysis of the Human Endocrine Pancreas
Priority:
2

Published in the DCP already


[Skill Calculator] Set Hallowed Sepulchre Floor Experience to Cumulative.

Except for floor 1, repeating a single floor in the Hallowed Sepulchre is impossible. Namely, each floor must be completed to continue to the next. Therefore, the required laps per floor shown on the skill calculator are not intuitive.
As a solution, this PR lists the floor experience cumulatively. All starting at floor 1, the entries now denote the total experience after completing each floor. Finally, the word Floor is replaced by Lvl because of the maximum activity name size (Hallowed Sepulchre Floor 1-4 would show as Hallowed Sepulchre Floor ...).
Before

After

I think the original intent was to shift-click to select multiple floors when running a specific floor in the sepulchre. I'm not really sure which way would be less confusing at the moment

I think the original intent was to shift-click to select multiple floors when running a specific floor in the sepulchre. I'm not really sure which way would be less confusing at the moment

I wasn't aware of the shift-click functionality. In that case, this change isn't necessary indeed.


This is awesome. Can we use yarn?

Hi, I've been shopping around for a SSR starter kit with everything Apollo for some time and I'm very excited to see if this works for me. I tried using yarn start to begin the project but it looks like it's dependent on being initiated via npm start. The following failure returns:
yarn start v0.16.1
$ if [ "$NODE_ENV" = production ]; then node --harmony $npm_package_app_backendBuildDir; else babel-node tools/webpack.run watch; fi
/Users/jb/Sites/clients/DiazGill/dg_apollo/tools/webpack.config.js:137
module.exports = process.env.npm_lifecycle_script.indexOf('mocha-webpack') >= 0 ? serverConfig : [serverConfig, clientConfig, dllConfig];
                                                 ^

TypeError: Cannot read property 'indexOf' of undefined
    at Object.<anonymous> (/Users/jb/Sites/clients/DiazGill/dg_apollo/tools/webpack.config.js:158:3)
    at Module._compile (module.js:541:32)
    at loader (/Users/jb/Sites/clients/DiazGill/dg_apollo/node_modules/babel-register/lib/node.js:144:5)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/jb/Sites/clients/DiazGill/dg_apollo/node_modules/babel-register/lib/node.js:154:7)
    at Module.load (module.js:456:32)
    at tryModuleLoad (module.js:415:12)
    at Function.Module._load (module.js:407:3)
    at Module.require (module.js:466:17)
    at require (internal/module.js:20:19)
error Command failed with exit code 1.

Not a deal breaker for me since I'm still able to install, add, remove, and all the other operations explained in the README with yarn...the only thing that fails is npm start but I thought I'd open up an issue and see if you have any guidance. Thank you very much!

@josephdburdick Hi, I think it can be fixed pretty easily. I will look and report back.

Here you go, please update via git pull and test

Works for me, thank you @vlasenko. 👍


Add Conditions of Sale link to the footer

finishes https://artsyproduct.atlassian.net/browse/PURCHASE-449
Screenshot
Desktop

Mobile

🎨


Add TTL for WX10ZM

Adding a TTL of 1440 minutes (1 day) prevents the entities from becoming Unavailable, at which point the device couldn't be controlled anymore. https://github.com/AlexxIT/XiaomiGateway3/issues/804

Thanks!


fix some typos found by codespell

Reference Issues/PRs

What does this implement/fix?

Any other comments?

src/primihub/data_store/csv/csv_driver.cc
src/primihub/data_store/mysql/mysql_driver.cc
src/primihub/data_store/sqlite/sqlite_driver.h
src/primihub/kernel/pir/operator/keyword_pir.cc
test/primihub/algorithm/aby3_div_test.cc


posibility to not auto populate examples in parameters

See my comment here (ne reason to duplicate it here).
My current "fix" is to use examples with first example having empty value.
parameters:
  - in: query
    name: with
    schema:
    type: string
    examples:
      a:
        value: ## this has no value - so it will not be auto populated and sent with request
      b:
        value: file-info
        summary: show file info

I used this but I often find myself manually clearing all examples or a few of them so I could run the request without them.
parameters:
  - in: query
    name: with
    schema:
      type: string
      example: file-info

It would be useful to add a clear button for each field (in: query or in: param).

closing this as we will track it with #686


Clarify language around QUIC extensions

Remove bogus language about an example QUIC extension.
Closes #5

I like what spencer has proposed here.

lgtm

Raised at interim meeting and no objections so merge it


add dx methods

Before
const bwd = Diffractor.PrimeDerivativeBack
bwd(bwd(x->5))

failed.

Needs tests

Codecov Report

Merging #77 (080a188) into main (be3b62d) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
- Coverage   52.67%   52.62%   -0.05%
==========================================
  Files          21       21
  Lines        2170     2172       +2
==========================================
  Hits         1143     1143
- Misses       1027     1029       +2

Impacted Files
Coverage Δ

src/interface.jl
69.09% <0.00%> (-2.61%)
:arrow_down:

Continue to review full report at Codecov.

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

this is ready to merge.


range slider should update itself when props are updated

Bug: Range Slider, right now its not setting the new range provided through props and also not updating the view (after component is loaded).
Range Slider should update itself when props are updated after component is loaded.

Thank you for the patch!


Verify w5500 chip properly (IDFGH-9158)

Datasheet says that 0x4 is only value returned. We should verify that we got correct one.

Note: Currently trying to make W5500 work using esp-idf-svc in rust. SPI returns garbage data, but no clear error message is returned. So would be nice if we failed fast here.

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

Thanks for the update. Could you also please squash the commits into one to follow contribution rules?

sha=bb2f346e55f484edc50f68cb26d164d8a79bcf39

Hi @reinismu, please, I have the issue with ethernet MAC W5500 on the latest master (after this PR). I have board with this SPI ethernet chip and it is working right. But after updated IDF to latest master, it fails when initilization - read chip ID returns 0x0 instead of 0x4. Please, could you help me, where can be a problem?

Hi @espzav,
Interesting. Datasheet does say that it will always return 0x4.
I wonder if there are different W5500 chips. What is your w5500 part number?

Hi @reinismu, I bought this: https://www.tme.eu/cz/en/details/w5500/ethernet-interfaces-integrated-circuits/wiznet/ (I can try another board later, which was assmbled in JLCPCB)

They link to the same datasheet where it mentions that 0x4 is the returned version.
You could try writing simple program that just reads the version register and see what happens. Maybe commands before corrupt it.

Thank you, I will try it and check the reset state.

Hi @reinismu,
I tried reset and still returns 0x0. But when I put 100ms delay before ID check, it works!


scene: consider re-introducing double-buffered state

This would be useful for minimizing damage when performing more involved operations on the scene. For instance when adding a surface to the scene at a given position, first the scene is damaged at (0, 0) then it is damaged at the new position.

That would be very useful for performance, especially on slow hardware or with software rendering.


Allow rendering of polymorphic slots

Polymorphic slots are an experimental feature, but I've found them to be quite useful so I'm pulling for their stability.
https://viewcomponent.org/guide/slots.html#polymorphic-slots-experimental
I've started a patch that allows their use in ViewComponent::Storybook, but it requires a bit of TLC. It is built on the support for nested components from #109.
Here's the patch:
https://github.com/dark-panda/view_component_storybook/commit/07de5e95c1136e798e65c6603766a8e6ce446ca9
Not the nicest looking patch and it could stand to be cleaned up, but it appears to be working at the moment.

Thanks for the contribution. Can you perhaps create the pull request where we have a few more tools to discuss and look at the code?

The patch you linked too is certainly pretty expansive. I'm curious about why it's so complex. I remember similar kind of logic in this repo that was ultimately much simplified by adding slot_type to view_component: https://github.com/github/view_component/pull/1009
I wonder would the logic simplify here if we added Polymorphic support to slot_type there too?

It's been a while since I worked on the code, but I think the basic gist of it had something to do with the way that polymorphic slot names are generated, and how the nesting in polymorphic slots work. It's been a few months now but that's more or less what I recall. The patch could likely be simplified a bit, of course, but it does presently work as far as I can tell.

Circling back to this conversation I'm curious if you have thoughts on the discussion in #150

fixed in v1.0.0


Cannot sort Channel

Hej,
I'm seeing this warning, while trying to make a test at the workflow level, any idea on how to deal with it?
Warning: Cannot sort channel, order not deterministic. Unsupported objects types: class org.apache.groovy.json.internal.LazyMap vs. class org.apache.groovy.json.internal.LazyMap

yes, I'm currently trying to use nf-test to assess workflows in the fetchngs pipeline cf https://github.com/nf-core/fetchngs/pull/191

Thanks! I will look into this. I think it depends on the nested elements of the output channel.
I noticed you disabled autoSort in nf-test.config for all testcases. If you add for exampple process.out to a snapshot, this could lead to failed tests, because the order of elements is not deterministic without sorting. Thus, the order in the snapshot could be different as the order of the current test run.

made a more tiny reproducible example: https://github.com/maxulysse/nf-test_109

Thanks, this helps us a lot! 🙌
A first try with this fix seems to solve the auto-sort problem.But I will check it again tomorrow 👍
However, I noticed that the zip files (e.g. test_pair_1_fastqc.zip) have a different md5 on each run. Thus, the snapshot matching failes :/

yeah, but no worries about that, my plan was to catch the snapshot of the versions, not the zip.

PR #115 should fix this issues. Same here, I'm leaving the issue open until we create a new release and you confirm its resolution.

We have just released RC1 of version 0.8.0 :tada:
This is a pre-release and can be installed using the following command:
curl -fsSL https://code.askimed.com/install/nf-test | bash -s 0.8.0-rc1

If everything works as expected, please feel free to close this issue. Thanks for your support! :pray:

Still having the issue as well: https://github.com/nf-core/fetchngs/pull/192

Hi guys, I was unfortunately very busy the last few weeks 🤯 I'll see that I have a release ready by the beginning of next week. Maybe I will find some time on the weekend 🚀

I have just built 0.8.0-rc2. You can install it as follows:
curl -fsSL https://code.askimed.com/install/nf-test | bash -s 0.8.0-rc2

The snapshots have to be recreated and then it should now work with the versions channel.
If this fixes your issues, I will build and release 0.8.0 👍

I'll test it right away

looking good for me: https://github.com/nf-core/fetchngs/pull/201


possibly bug fix

What is the purpose of the change
im not sure if is a bug
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #3460 into master will increase coverage by 0.05%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #3460      +/-   ##
============================================
+ Coverage     64.01%   64.06%   +0.05%
  Complexity       71       71
============================================
  Files           664      664
  Lines         28608    28608
  Branches       4772     4772
============================================
+ Hits          18313    18328      +15
+ Misses         8086     8073      -13
+ Partials       2209     2207       -2

Impacted Files
Coverage Δ
Complexity Δ

...mon/src/main/java/org/apache/dubbo/common/URL.java
49.22% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
61.9% <0%> (-9.53%)
0% <0%> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
51.42% <0%> (-5.72%)
0% <0%> (ø)

.../org/apache/dubbo/remoting/ExecutionException.java
15.78% <0%> (-5.27%)
0% <0%> (ø)

.../dubbo/remoting/transport/netty4/NettyChannel.java
60% <0%> (-4.71%)
0% <0%> (ø)

...bo/rpc/cluster/support/FailbackClusterInvoker.java
67.21% <0%> (-3.28%)
0% <0%> (ø)

...exchange/support/header/HeaderExchangeHandler.java
55.81% <0%> (-2.33%)
0% <0%> (ø)

...dubbo/remoting/exchange/support/DefaultFuture.java
72.97% <0%> (-2.03%)
0% <0%> (ø)

.../java/org/apache/dubbo/config/ReferenceConfig.java
57.51% <0%> (-0.38%)
0% <0%> (ø)

...pache/dubbo/registry/support/AbstractRegistry.java
81.49% <0%> (+2.36%)
0% <0%> (ø)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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

LGTM

good catch.


fix(migrate): change the connectionTimeoutMillis to 10000 from 10_000

What kind of change does this PR introduce?
Bug fix
What is the current behavior?
Database connect timeout is not a valid number

Hello @key7men thanks for the contribution, in JS using an underscore makes it a valid number.
It is the same as 1000.
10_000 === 10000 // true

So it can also be like this, learn a new point of knowledge. Thank U.


Add Enapter repo with KeyDB multimaster setup PoC inside.

Signed-off-by: Andrey Voronkov<EMAIL_ADDRESS>

@Antiarchitect Can you please rebase from master to fix the conflict.

@mattfarina Yeah, sure!


Improve PySB/cupSODA interface; move to separate repo

Currently, a user must construct the 'MX_0' and 'c_matrix' arrays manually and pass them to CupSODASolver.run() or run_cupsoda(). I'd like to change that so that these are created internally based on the initial conditions and parameter values of the model, also allowing for user-defined values in the method call.

As per recent group meeting, cupSODA will be moved to a separate git repo.

cupSODA also needs unit tests, but that will be handled in the separate repo. Hence, I'll close that ticket off in the PySB repo.

Issue moved to pysb-cupsoda repo


[Android] Attempt to invoke virtual method 'int android.graphics.Bitmap.getWidth()' on a null object reference

Using "react-native-image-picker": "^0.18.3"
I try to select an image from the device library, when selecting an image - the app crashes. This only happens on the device and not in the simulator.
Here is the error:
Fatal Exception: java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=2, result=-1, data=Intent { dat=content: flg=0x1 }} to activity {com.myapp/com.myapp.MainActivity}: java.lang.NullPointerException: Attempt to invoke virtual method 'int android.graphics.Bitmap.getWidth()' on a null object reference
       at android.app.ActivityThread.deliverResults(ActivityThread.java:3881)
       at android.app.ActivityThread.handleSendResult(ActivityThread.java:3931)
       at android.app.ActivityThread.access$1300(ActivityThread.java:144)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1408)
       at android.os.Handler.dispatchMessage(Handler.java:102)
       at android.os.Looper.loop(Looper.java:155)
       at android.app.ActivityThread.main(ActivityThread.java:5696)
       at java.lang.reflect.Method.invoke(Method.java)
       at java.lang.reflect.Method.invoke(Method.java:372)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1028)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:823)
Caused by java.lang.NullPointerException: Attempt to invoke virtual method 'int android.graphics.Bitmap.getWidth()' on a null object reference
       at com.imagepicker.ImagePickerModule.getResizedImage(ImagePickerModule.java:504)
       at com.imagepicker.ImagePickerModule.onActivityResult(ImagePickerModule.java:388)
       at com.facebook.react.bridge.ReactContext.onActivityResult(ReactContext.java:173)
       at com.facebook.react.ReactInstanceManagerImpl.onActivityResult(ReactInstanceManagerImpl.java:526)
       at com.facebook.react.ReactActivity.onActivityResult(ReactActivity.java:180)
       at com.myapp.MainActivity.onActivityResult(MainActivity.java:105)
       at android.app.Activity.dispatchActivityResult(Activity.java:6160)
       at android.app.ActivityThread.deliverResults(ActivityThread.java:3877)
       at android.app.ActivityThread.handleSendResult(ActivityThread.java:3931)
       at android.app.ActivityThread.access$1300(ActivityThread.java:144)
       at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1408)
       at android.os.Handler.dispatchMessage(Handler.java:102)
       at android.os.Looper.loop(Looper.java:155)
       at android.app.ActivityThread.main(ActivityThread.java:5696)
       at java.lang.reflect.Method.invoke(Method.java)
       at java.lang.reflect.Method.invoke(Method.java:372)
       at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1028)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:823)

Another test with "react-native-image-picker": "^0.18.6" gives me the same error...

+1

What kind of image ?

@yfuks Seems to be any photo based on the stacktraces we are seeing.  Hard to tell exactly since the path is something like content://media/external/images/media/7396. I don't have an Android device to test different file types at the moment though.
If I can help in any way let me know.  We just did a soft-launch of our Android app yesterday, and have already seen this crash 92 times for 45 different users on a variety of devices and Android versions:

A fast work around is to not set the folowingoptions :
angle
maxWidth
maxHeight
quality

I'm actually working on a fix :)
Can you guys tell me if you get the same issue with the example ? It could help me.

I can't test the example at the moment, but I just published a new APK with the workaround you suggested (we used quality), so hopefully that will work for now.

Make sure you add the folowing permission
<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/>

Since i'm not able to reproduce i will add robustness code to the project to avoid crash.

I just push robustness code to fix this issue that appears in the case you can't read the selected image.
https://github.com/marcshilling/react-native-image-picker/commit/f134b9158cb29144fc12f9112f690c0bdef8ea79#diff-e2ac0feb8ce151bd872a7c4ab6b4801dR400
Hope this will work.

@yfuks well, this will prevent the crash...but it's not going to help these guys actually get the image. Do we know the root cause?

Well the code that i push will just prevent resizing if we cannot read the selected image. I feel like it could be helpfull since it don't prevent from getting response infos (base64...)
The crash can only appear when we can't read the selected photo.

@yfuks ohhh I see, I didn't see that it avoids the resize if we can't read it. Cool.
I'm going to publish a new release to npm right now...@CodeTitanian @anativ @dhrrgn can you guys see if this fixes your issue?

@marcshilling if you can just wait a couple of minutes :)
I'm on @jeremyeaton89 PR actually. Probably going to merge soon

@yfuks oops, too late! v0.18.9 is live with the robustness changes for this issue. I'll leave it to you to publish the next patch!

@yfuks I did have the   <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/> permission.
I installed module version 0.18.10 and now the app does not crash but also does not really do anything...
Removing the options you suggested:
angle
maxWidth
maxHeight
quality
Does not help because the response data I get is empty string.

@yfuks that's what I was talking about ^. You fixed the crash but that root cause is that the photo doesn't exist.

@CodeTitanian

does not really do anything

What you mean exactly ? Nothing is returned to RN ?
@marcshilling

the photo doesn't exist

I'm not sure about that. And avoid crash is still an enhancement :)

@CodeTitanian can you tell me what kind of image you pick in your gallery ? Image from facebook ? Camera ? .. ?

@yfuks When I select a photo from "Gallery" it works and brings up the cropping tool (with allowsEditing option). When I select from the Photos app it does not work and does not open the cropping tool (even with allowsEditing option enabled).
Same photo as far as I can tell ... Just matters which picture app I go thru.

@rpastorelle can you open an other issue ? :)

@yfuks He was just giving you the information you asked for.  It is the same issue.

He was just giving you the information you asked for. It is the same issue.

ok.
So now that we have something to work with i can investigate further and try to reproduce.

Good new i'm finally able to reproduce the issue ! I also know what can i do to fix it. I'm on it. :)

fix in 0.18.11
So i fix the issue by using the initial image if the crop don't perform. I still need to find out why crop don't work on some app.

@yfuks Can confirm this has fixed the issue for us.  Thanks for the quick turn-around on this. 👍

@yfuks the new version works good. Thanks for fixing it!

I'm getting this exact same error now on version 0.19.2. Any idea what could be causing it?

@smwoods nothing appears to have changed with the code that was added to fix this, and I can't reproduce. Let me know if you can track it down.

Hm. I'm running on a galaxy s6, react native version 0.26.3, I've tried uninstalling and reinstalling multiple times, and I'm running the example exactly as written. Every time I select a photo, the app crashes. Any advice on how to begin debugging this at least?

Reinstalled Android Studio, finally got it working. Still crashes with allowsEditing on but is working otherwise.


Implement CurrentValues, OriginalValues, and GetDatabaseValues on EntityEntry

when will this feacture be implemented? I need it for a update function in an update method in a service (N-Tire architecture).

Currently planned for 1.1 - see https://blogs.msdn.microsoft.com/dotnet/2016/07/29/entity-framework-core-1-1-plans/ for details.

I see 1.1 was just released.  Did this make it into the release?

@dstep2007 - Yes.


Need to figure out standard method for error reporting in oort

From<EMAIL_ADDRESS>on January 25, 2013 08:32:05
oort is often run via jenkins, and configured such that the console is sent to users / a list if there is an issue.
one strategy is to just run oort. if there are issues then info is generated on stderr/stdout/console, and this goes at the end of the report, and is sent to the user if there is an error. this works mostly fine.
however, the drawback is that there is no file for the overall report, it's not a build artifact, and there is no report that can be post-processed with grep.
we've tried redirecting the output and exiting if there is an error. But this is awkward and problematic. E.g. see issue here, which is buried midway in the console.
<a href="http://build.berkeleybop.org/job/build-phenoscape/190/consoleFull" rel="nofollow">http://build.berkeleybop.org/job/build-phenoscape/190/consoleFull

What would be best would be if there was an option such that output could be simultaneously generated on the console and to a file. This may be as simple as a log4j setting - or it may be harder. investigate.
Original issue: http://code.google.com/p/owltools/issues/detail?id=46

From<EMAIL_ADDRESS>on February 07, 2013 09:51:22
OORT has now an updated reporting mechanism. One can now create separate log files for errors and the full trace: The new CLI options:
--error-report [error-report.txt]
--trace-report [trace-report.txt]
Both options canbe used as a boolean flag or append a file name to overwrite the default output file.
Status: Fixed


Feature/compatible with mvc 3

Make Zend\Navigation compatible with Zend\Mvc and Zend\Router

@RalfEggert
Don't forget the unit tests! (See Travis)

Fixed the tests

@RalfEggert
Some more:

We have to check the the dependency to Zend\Mvc. Maybe we can remove it and add only Zend\Router to composer.json.
I think, at the moment we must move Zend\Permissions from the "suggest" part to "require" in composer.json. (See Zend\Navigation\Page\AbstractPage)

@RalfEggert you can look at https://github.com/zendframework/zend-view/blob/master/src/Helper/Url.php
It consumes both Zend\Mvc\Router and Zend\Router.

@vaclavvanik
Please move this problem to the Zend\View issue tracker. Thanks!

@froschdesign I meant @RalfEggert can look at Zend\View\Helper\Url because this helper could be used with both Zend\Mvc 2.0 and 3.0. Ralfs changes are Mvc 3.0 imho.

@vaclavvanik
You are right. I thought you mean the Zend\Navigation view helpers. :grin:

It consumes both Zend\Mvc\Router and Zend\Router.

This is an option for a next minor release in version 2 of Zend\Navigation. For version 3 with can remove Zend\Mvc – after a deeper look into the code base.

The only dependency to Zend\Mvc left is in https://github.com/zendframework/zend-navigation/blob/develop/src/Page/Mvc.php  It uses the two constants ModuleRouteListener::ORIGINAL_CONTROLLER and ModuleRouteListener::MODULE_NAMESPACE.
Not sure about the Zend\Permissions. It is used in https://github.com/zendframework/zend-navigation/blob/develop/src/Page/AbstractPage.php and already added to the require-dev section in composer.json.

@RalfEggert I noticed that Travis was reporting test failures for zend-mvc v2 (and servicemanager, etc.). I took the liberty of pulling locally and figuring out how to get things running. This included:

adding zend-router to the development requirements
keeping zend-mvc as a development requirement, but upping it to allow either v2 or v3.
updating the Mvc page type to inline the zend-mvc ModuleRouteListener constants, which removes the concrete dependency on zend-mvc. It's still needed for tests, as we have some integration tests with that class.
updating the suggestions to remove zend-mvc and add zend-router, and make them more descriptive of when and why you'd need them, as well as what versions are supported.
updating the Mvc page type tests to vary what type of router artifacts to use based on what version of zend-mvc is installed; if zend-mvc v2 is installed, it uses Zend\Mvc\Router\* classes, while for v3 it uses Zend\Router\* classes.

I've tested locally with both an updated composer.lock, as well as running --prefer-lowest, and it appears to work well! I'm opening a competing PR that incorporates your work just to ensure that tests pass against travis; if they do, I'll issue a 2.8.0 release.
Thanks for doing the heavy-lifting, @RalfEggert !

:+1:


404

SERVLET:	org.eclipse.jetty.servlet.ServletHandler$Default404Servlet-5f8edcc5

Since helm-dashboard does not use any Java technology, I conclude that this is caused by something in your browser redirecting the traffic.


Current Date Issue

So I set up the project with the updated code. I'm looping through all the days and putting the appropriate color on that day... however, it keeps crashing. IMHO it looks as if the bug is assassinated with setting the the color on the current date.
I was able to test this by checking if the date I want to set the color for is today and skipping it... No crash.
To reproduce set up an expandable calendar view and attempt to set set the mark style to leftbar and a color...
Thanks!

Hmmm...this looks like something went wrong in the exp calendar view. Could you send me some trace back from the android console? That my help me to locate the bug.
:)
Thanks.
Bob.
Sent from my iPad

在 2016年2月26日，下午11:18，Jason H Caruso<EMAIL_ADDRESS>写道：
So I set up the project with the updated code. I'm looping through all the days and putting the appropriate color on that day... however, it keeps crashing. IMHO it looks as if the bug is assassinated with setting the the color on the current date.
I was able to test this by checking if the date I want to set the color for is today and skipping it... No crash.
To reproduce set up an expandable calendar view and attempt to set set the mark style to leftbar and a color...
Thanks!
—
Reply to this email directly or view it on GitHub.

Fixed. Please pull the latest commit and verify. Thanks.

Thanks Bob, I haven't had time to reproduce it.

You are welcome~

BTW if you guys decide to use this lib, could you tell me which app are you building when you finished? I want to add your app name & link to my readme file. You know, just to show everybody this lib is battlefield testified.
:)
Thanks.

Absolutely! I'm in the middle of a redesign of an app is currently on the market. I hope to have it out by the end of the month!

Glad to hear that! Thanks! :+1: :+1: :+1: :+1:


Fix triple counting of All England measure numbers

Some of the All England totals are generated by aggregating CCG level data from
the MeasureValues table. It's not possible to select CCG level data by
selecting only rows where pct_id is not NULL because practice level
data also has the pct_id set. So we have to instead select where
practice_id is NULL.
Previously, including the extra condition that pct_id be not NULL was
redundant as this was true for all rows. However with the inclusion of
STP and Regional Team data this stopped being true and our query was now
returning CCG, STP and Regional Team data which resulting in a triple
counting of some of the All England numbers.

I should have thought about this more when I added STPs and Regional Teams to the measures.

Oh yes, the manager would be a great place for this. Something like filter_by_org_type?

Some notes for the rest of the team


Add support for joining by library ID

Closes #8
This PR add support for merging RNAseq and feature runs based on the new library_id. It also changes the output location (publishDir) to be nested by sample and library id, for an organization that will be closer to the final output we expect to want.
The other major change is how metadata is passed around within processes. Each process now expects a metadata map (dictionary) as the first element of the main channel input it receives. It then returns this as the first element of the output tuple in all cases.
This should make it easier for us to add or remove metadata that might be used by processes and pass them along with less fuss. It did make joining by library a bit more involved, but I think the tradeoff is probably worth it.
One nice extension (not currently implemented, but I'm pretty sure this is possible) is that a process could add to the metadata before returning. This would be done by adding a line like
meta.foo = "bar"

within any process script: block (before or after the shell script portion). Adding something like this would not affect any other script.
At the moment, I have not included in the metadata map the slide information for spatial data. Since we have not yet implemented processing those data, I thought that could wait until we needed it. Adding it should not affect any existing process!

This all seems reasonable to me and looks good. To make sure I have the directory structure right for a sample that has CITE-seq, there will be two output folders that are published:
out_directory/sample_id/library_id/run_id-rna
out_directory/sample_id/library_id/run_id-feature
And then we will have the same overall structure to house the RDS files.

Yes, though I kind of assume that we will at that point stop publishing the alevin folders, so we will will have something like:
out_directory/sample_id/library_id/library_id_qcreport.html
out_directory/sample_id/library_id/library_id_filtered.rds
out_directory/sample_id/library_id/library_id_unfiltered.rds
as the final outputs. run_id will go away as features get merged.


Add the materialized views

Hello,
Since the introduction of refresh concurrently (9.4+) mode on materialized views (9.3), these objects can become bloated.
The following PR, perhaps a little too simplistic, but normally functional, adds the materialized views as part of the measurement of bloated.
The name of the columns in the query 'table_bloat.sql' remains unchanged. There is no particular code which would be specific to versions 9.4 and higher, which are the only versions to be potentially affected by bloated materialized views.

Thank you!


Create dimension(updated).dart

const marginSmall = 12.0;
const paddingText = 14.0;
const sizedBoxHeight = 60.0;

We already have files for dimension and colors, and these are not the colors that are specified in the issues for the app.
I will have to close this PR as invalid.


Jest does not honor "mute" package when running multiple test suites

Do you want to request a feature or report a bug?
Perhaps a feature, but trying to figure out if the behavior is expected?
What is the current behavior?
If you have a couple test suites and at least one of those uses mute, it will successfully "mute" when running a single test suite (e.g., jest test/suite1.spec.js) but will not mute if running all tests (jest). In the repo below, there's a loud and quiet function that are tested in two different suites. I'm using mute to silence the loud function in my test as an example.
If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can yarn install and yarn test.
Example repo: https://github.com/robballou/jest-mute-issue
There's a screenshot in that repo as well.
What is the expected behavior?
Running a suite directly and with multiple suites would ideally behave the same. This is mostly an issue where I'm just trying to keep same behavior in both environments. Adding mute is helpful for me in cases where I know something is going to output something, but don't really care to have it show up when running tests. If mute doesn't play well with jest, no worries. Just trying to figure out if this is expected. Thank you for taking a look :)
Please provide your exact Jest configuration and mention your Jest, node,
yarn/npm version and operating system.

Not sure if this is something we want to support. Why not just do &> /dev/null?

--runInBand works the same as far as I can tell. Jest has a --silent option which I think would be the same as redirecting the output which is great in it's way, but that turns off all console output.
My example repo is kind a poor example of what I'm doing, but basically:

I have tests that are supposed to error and talk with stderr (for example, "when the API calls fails, throw an error")
In those cases, I would love to not have them actually write to stderr because I know they're going to output there
But if another test fails suddenly and talks with stderr unexpectedly, I would want to see that output. This is why I like the mute package, I can tell it to be quiet in places where I expect it to make noise, but if noise suddenly comes from somewhere else, that's something I'm going to want to fix.

And I can understand if this ends up as "we can't/don't want to support" realm... just trying to see if this is expected and if it maybe can work this? 😁

Can you just mock out process.stdout.write or something in the tests?

mute is supposed to basically noop that write to a stream, but my guess is that Jest is doing something with the streams in the case of the multi-suite test runs. I haven't looked into what jest is doing yet, but here is what mute is doing:
https://github.com/shannonmoeller/mute/blob/master/index.js
So maybe process.stdout and process.stderr aren't what mute expects in those cases?

FWIW, I believe this approach is going to work for this use case: https://www.npmjs.com/package/jest-mock-console and it's a drop-in for mute, basically.

In order to mute console output, the idiomatic way to do that in Jest is to use --silent. Issue tracking that for process.stdout is #6524


Find a way to update this per era and cache automatically

https://github.com/James-Sangalli/dot-tools-server


Update travis config

Coverage remained the same at 100.0% when pulling fff31dcd90d3d99c3ffa5d98de443d2ad2258606 on dev into 5ebf4692bcdf6af48fc434fe75e3de548f8b3838 on master.


Gh 1178 user object walkthrough

Codecov Report

Merging #1206 into develop will increase coverage by 0.54%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #1206      +/-   ##
=============================================
+ Coverage       64.2%   64.75%   +0.54%
- Complexity      1397     1444      +47
=============================================
  Files            645      645
  Lines          20355    20355
  Branches        2497     2497
=============================================
+ Hits           13069    13180     +111
+ Misses          6242     6123     -119
- Partials        1044     1052       +8

Impacted Files
Coverage Δ
Complexity Δ

...onutil/iterable/LimitedInMemorySortedIterable.java
50% <0%> (+2.77%)
16% <0%> (+1%)
:arrow_up:

...uk/gov/gchq/gaffer/proxystore/ProxyProperties.java
45.09% <0%> (+45.09%)
14% <0%> (+14%)
:arrow_up:

...java/uk/gov/gchq/gaffer/proxystore/ProxyStore.java
71.07% <0%> (+71.07%)
32% <0%> (+32%)
:arrow_up:

Continue to review full report at Codecov.

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


Document Title does not update as expected

Bug
I noticed as I go through the various pages of the app, the Document Title (observed in the tab) does not update, with the exception of the About page:

Expected behavior
The Document Title will update according to which page the user is on.  For example:

Home page => Challenge Registry
About page => About - Challenge Registry
Search page => Search - Challenge Registry
User profile page =>  <first name> <last name> - <username> (or some other pattern)

Alternatively: not even update the Document Title at all (therefore, removing the title update from the About page).

For better viewing:

This bug should be addressed by #560. The task is loosely related to search engine optimization (SEO).

Gotcha - sorry for the redundant ticket.  I only checked the tickets labeled with bug.

This ticket and description of the expected behavior is useful! This ticket is not referenced in #560 so we can retrieve it easily.


stdArray starts at 1 instead of 0

Description
In the current implementation of the stdArray it starts at 1 instead of 0, however in vba and javascript the array starts at 0 is this a planned design decision or is it designed to be that way?
Example
    Dim arr As stdArray
    Set arr = stdArray.Create(1, 2, 3, 4, 5, 6, 7, 8, 9, 10)
    Debug.Print (arr(0)) 'throws an error
    Debug.Print (arr(1))

Error
Runtime error: 9
Index out of range
Expectation
stdArray starts at 0

It is designed to be this way, however happy to add a feature for rebasing the array to start at 0 (or any integer for that matter)

It is fine that the array starts a 1.
However it should be mentioned that it starts at 1 instead of zero I haven't read any info which says that it starts at 1.
Maybe you should mention it somewhere
But it would be nicer if it use 0 as start point for the following reason that it is easier to migrate existing vba code without this lib. However that would break backwards compatibility with existing users of the lib

However it should be mentioned that it starts at 1 instead of zero I haven't read any info which says that it starts at 1.
Maybe you should mention it somewhere

Yes the documentation for stdArray is currently not written. However it is present in both the tests and the code.

But it would be nicer if it use 0 as start point for the following reason that it is easier to migrate existing vba code without this lib.

Indeed, we can make a global option on stdArray for defaultBase, similar to what we do for stdLambda 's  function lookup.


Bug: registerUpdateListener gives stale state after clearing formatting

Lexical version: 0.6.4
Steps To Reproduce

make bold non-first word
click on clear
editorState still gives bold format

Link to code example:
https://codesandbox.io/s/mutable-browser-tsk7n2?file=/src/App.tsx

The current behavior
Bold state is not reset after clearing formatting
The expected behavior
Bold state is set to false after clearing formatting

@TrySound Remove the useCallback around updateToolbar and then it works as expected, or make updateToolbar accept a selection argument:
 const updateToolbar = useCallback((selection) => {
    if ($isRangeSelection(selection)) {
      setIsBold(selection.hasFormat("bold"));
    }
  }, []);

The problem is the callback is being memoized with the same stale selection, so doesn't correctly get the new selection when you call editor.read.
Tested this out in your CodeSandbox and verified that the toolbar button updates as expected

Hi @thegreatercurve. I removed useCallback and inlined the code instead. The issue still reproduced. I updated codesandbox.


operation could not be completed when disabling/enabling sync

Seems to happen sometimes, not always. Akin thinks it's related to trying to set a default account that's already default.

@tzarebczan we think it is SDK related, please re-open if continuing to experience.


Possible world: infinite intersection world

If we want to prove that P(A)=P(B) implies A=B, one way of doing it would be to make infinite intersection world.
Let $F$ be a set of subsets of $X$ (it doesn't have to be a filter).
What we want is the intersection $T$ of all the elements $S$ of $F$.
So in other words, $x\in T$ if and only if $\forall S\in F, x \in S$.
You'll need to know the notation for intersection $T$ of the elements of $F$.
You need the proof that $x\in T\iff \forall S \in F, x \in S$.
You need that $\forall S \in F, T \subseteq S$.
And you need that $T$ is the \emph{greatest lower bound} of all the $S$s,
i.e., ∀ U : Set 𝓧, (∀ S ∈ 𝓕, U ⊆ S) → U ⊆ T.
This should be enough to prove that if P(A)=P(B) then A=B.
However, there is another way. If you prove that A \in P(A), which should be a lemma about principal filters, then you can deduce A \in P(B) if they're equal, and hence B \subseteq A. Similarly A \subseteq B so you can use antisymmetry, thus completely avoiding infinite intersections.
Either way, we should have this result in the game really.

Yael pointed out to me that in fact this was Filter.ker so one should develop that instead. The API is in Game/KernelLevels.lean .


New package: Dedispersion v0.0.1

Registering package: Dedispersion
Repository: https://github.com/astrogewgaw/Dedispersion.jl
Created by: @astrogewgaw
Version: v0.0.1
Commit: 8aeb094d3dec1172e969545c9be4bdb9e7dcb9e5
Reviewed by: @astrogewgaw
Reference: https://github.com/astrogewgaw/Dedispersion.jl/commit/8aeb094d3dec1172e969545c9be4bdb9e7dcb9e5#commitcomment-140190092
Description: Dedispersion algorithms.

Thank you for contributing your package! Please make sure to add a little bit of documentation before registering, though. At a minimum, this would be an explanation in the README what "Dedispersion" is and a usage example.

Thank you for contributing your package! Please make sure to add a little bit of documentation before registering, though. At a minimum, this would be an explanation in the README what "Dedispersion" is and a usage example.

Hi @goerz! Yes, no worries, the documentation is forthcoming.

Thank you for contributing your package! Please make sure to add a little bit of documentation before registering, though. At a minimum, this would be an explanation in the README what "Dedispersion" is and a usage example.

Also, @goerz, I think your comment might be blocking this PR from being merged. Could you add a [noblock] to it?

[noblock] Sorry, I don’t see that you added the missing documentation.

the documentation is forthcoming

The registration can’t proceed until the package has a reasonable minimum amount of documentation that makes it useable to the general public.

[noblock]

[noblock] Sorry, I don’t see that you added the missing documentation.

the documentation is forthcoming

The registration can’t proceed until the package has a reasonable minimum amount of documentation that makes it useable to the general public.

@goerz First of all, this package is a meta-package. All of its constituent packages have their documentation. Second of all, there is no requirement for a "minimum" amount of documentation in Julia's docs about making and registering new packages. I have already passed all the criteria for auto-merging, and I should be allowed to do so. When I said the documentation is forth-coming, I meant I will take up it up when I have more time. In the meantime, I would like other people to be able to use this package, and so would like it to make it to the General registry without any further ado.

Second of all, there is no requirement for a "minimum" amount of documentation in Julia's docs about making and registering new packages.

Sorry, that is not true: https://github.com/JuliaRegistries/General?tab=readme-ov-file#should-i-register-my-package-now
This guideline is not checked automatically, because there's some judgement required that a bot can't easily do, but that doesn't mean it's not enforced.
See also https://discourse.julialang.org/t/when-does-it-make-sense-to-post-a-new-package-to-the-julia-repository/108701/4
Please note that this is a very low bar. All that is required is for the README to give some idea about the purpose of the package and for how people might be able to use it. It should not take you more than 15 or 20 minutes to write a README. A "full" documentation (with Documenter or something like that) is absolutely something that can come after registration.
Packages without any meaningful documentation are not yet ready for registration and will not be merged.
[noblock]

@goerz My apologies for getting back so late. I agree with you that adding a README is indeed a very low bar. At the time, I was terribly busy, and thought it might be a good idea to push for the registration of the package first, and worry about the documentation later. However, in retrospect, I agree that a README should have been added nonetheless. I finally found some time, and added more content to the README as suggested. Since this is a meta-package, there is no usage specified, since for that users can just go to the individual component packages. Let me know if this up to snuff, and if so, do unblock the PR since otherwise it will close in 7 days. Cheers!

[noblock] Thanks! That looks good! You’ll have to add [noblock] to your own comment as well

[noblock] Thanks @goerz!


Background tasks are considered as scenarios

In my cucumber feature files, there is a Background  scenario in most cases, which runs before each scenario (similar to @BeforeMethod in TestNG).
The  report generated by multiple-cucumber-html-reporter considers these tasks as scenarios as well, so the combined report shows a much higher total scenario count than there actually is.
Is there any way to exclude the  Background  scenarios?

Hi @lahiruchandima
Can you provide an example JSON file with the background steps? Because I think you will see that the background steps are shown as scenario steps because that is how Cucumber works.
This report only processes the JSON-files and calculates the failures/passed and so on

I have the same issue, here is my json ( I removed irrelevant information)
[
  {
    "uri": "/test/test.feature",
    "id": "test",
    "keyword": "Feature",
    "name": "Test",
    "description": "",
    "line": 2,
    "tags": [],
    "elements": [
      {
        "keyword": "Background",
        "name": "",
        "description": "",
        "line": 4,
        "type": "background",
        "steps": []
      },
      {
        "id": "test-scenario",
        "keyword": "Scenario",
        "name": "Test Scenario",
        "description": "",
        "line": 11,
        "type": "scenario",
        "tags": [],
        "steps": []
      }
    ]
  }
]

Here is a screenshot

Hmm,
I need to dive into this. The background was always created like this "isBackground": true, in a step and not as a separate scenario.
I'll set up some examples and try to fix this

@AlexEzzeddine
I've been checking this with CucumberJS 5 and I can't find the "keyword": "Background",. Can you provide:

the version and language of Cucumber  you are using
a better demo example of the background (with some steps and a feature file)
explain what you expect to happen with the Background steps in the report

Tnx

I am actually using cucumber for ruby. Version is 2.4.0
I expect:

I expect background elements to show up in report as "Background:", not "Scenario:".
I expect them to not be included in any scenario counts. Example: I have 6 Background elements and 6 Scenarios. Total scenario count should be 6 and not 12.

Sorry about this one.
Can you please provide a sample feature file and a sample json output so I can dig into this?

@wswebcreation
{
    "line": 2,
    "elements": [
        {
            "line": 4,
            "name": "",
            "description": "",
            "type": "background",
            "keyword": "Background",
            "steps": [
                {
                    "result": {
                        "duration":<PHONE_NUMBER>,
                        "status": "passed"
                    },
                    "line": 5,
                    "name": "I am logged in",
                    "match": {
                        "location": "iLoggedIn()"
                    },
                    "keyword": "Given "
                }
            ]
        },
        {
            "start_timestamp": "2019-08-12T16:22:35.079Z",
            "before": [],
            "line": 8,
            "name": "Scenario 01",
            "description": "",
            "id": "scenario01",
            "after": [],
            "type": "scenario",
            "keyword": "Scenario",
            "steps": [],
            "tags": []
        },
    ],
    "name": "Feature 01",
    "description": "",
    "id": "feature01",
    "keyword": "Feature",
    "uri": "file:features/feature01.feature",
    "tags": []
}

any solution for this error?


Bug fix in JedisCluster scan

Closes #1901

LGTM!>

Merged into master, 2.10, 2.9.

Just realize this doesn't have a test. Shouldn't we add one?

Yes, I've seen those but maybe in this case it's work to add a JedisCluster test specifically to test that condition.


Updated README.md, install scripts for CentOS

@allyshaban5 @litlfred @lduncan-intrah
This PR includes the following for DATIM:

Updated README for CentOS, macOS, backup/restore, and OIM in production.
New generic install script in resources/scripts. This is tested on CentOS, macOS. It should work on any Unix-like system.
New install script for additional libraries with simple CLI. This could be cleaner, for example, do stored definitions need to be copied over before doing the basex repo install?
Ansible-playbook for meeting requirements of install (but not doing the install).

Please have a look.

Merging in the CentOS docs and new install scripts.


Module generator incorrect src path

Hi,
I am switching from Ceedling 0.31.0 to 1.0.0 (currently using 1.0.0-9fc30fb).
Using module generator pluggin it seems the source files are created in another place than the path I set in my project.yml
Header and C files are created in src/bsp/ inner directory instead of src/ directory.
Is something wrong with my configuration or is this a bug ?
This is my configuration :
# =========================================================================
#   Ceedling - Test-Centered Build System for C
#   ThrowTheSwitch.org
#   Copyright (c) 2010-24 Mike Karlesky, Mark VanderVoord, & Greg Williams
#   SPDX-License-Identifier: MIT
# =========================================================================

---
:project:
  # how to use ceedling. If you're not sure, leave this as `gem` and `?`
  :which_ceedling: gem #vendor/ceedling
  :ceedling_version: 1.0.0

  # optional features. If you don't need them, keep them turned off for performance
  :use_mocks: TRUE
  :use_test_preprocessor: :all
  :use_backtrace: :simple
  :use_decorators: :auto #Decorate Ceedling's output text. Your options are :auto, :all, or :none

  # tweak the way ceedling handles automatic tasks
  :build_root: build_test
  :test_file_prefix: test_
  :default_tasks:
    - test:all

  # performance options. If your tools start giving mysterious errors, consider
  # dropping this to 1 to force single-tasking
  :test_threads: 8
  :compile_threads: 8

  # enable release build (more details in release_build section below)
  :release_build: FALSE

# Specify where to find mixins and any that should be enabled automatically
:mixins:
  :enabled: []
  :load_paths: []

# further details to configure the way Ceedling handles test code
:test_build:
  :use_assembly: FALSE

# further details to configure the way Ceedling handles release code
:release_build:
  :output: MyApp.out
  :use_assembly: FALSE
  :artifacts: []

# Plugins are optional Ceedling features which can be enabled. Ceedling supports
# a variety of plugins which may effect the way things are compiled, reported,
# or may provide new command options. Refer to the readme in each plugin for
# details on how to use it.
:plugins:
  :load_paths: []
  :enabled:
    #- beep                           # beeps when finished, so you don't waste time waiting for ceedling
    - module_generator                # handy for quickly creating source, header, and test templates
    #- gcov                           # test coverage using gcov. Requires gcc, gcov, and a coverage analyzer like gcovr
    #- bullseye                       # test coverage using bullseye. Requires bullseye for your platform
    #- command_hooks                  # write custom actions to be called at different points during the build process
    #- compile_commands_json_db          # generate a compile_commands.json file
    #- dependencies                   # automatically fetch 3rd party libraries, etc.
    #- subprojects                    # managing builds and test for static libraries
    #- fake_function_framework        # use FFF instead of CMock

    # Report options (You'll want to choose one stdout option, but may choose multiple stored options if desired)
    #- report_build_warnings_log
    #- report_tests_gtestlike_stdout
    #- report_tests_ide_stdout
    #- report_tests_log_factory
    - report_tests_pretty_stdout
    - report_tests_raw_output_log
    #- report_tests_teamcity_stdout

# override the default extensions for your system and toolchain
:extension:
  #:header: .h
  #:source: .c
  #:assembly: .s
  #:dependencies: .d
  #:object: .o
  :executable: .hex
  #:testpass: .pass
  #:testfail: .fail
  #:subprojects: .a

# This is where Ceedling should look for your source and test files.
# see documentation for the many options for specifying this.
:paths:
  :test:
    - +:test/**
  :source:
    - src/**
  :include:
    - src/** # In simple projects, this entry often duplicates :source
  :libraries: []

# You can even specify specific files to add or remove from your test
# and release collections. Usually it's better to use paths and let
# Ceedling do the work for you!
:files:
  :test: []
  :source: []

# Compilation symbols to be injected into builds
# See documentation for advanced options:
#  - Test name matchers for different symbols per test executable build
#  - Referencing symbols in multiple lists using advanced YAML
#  - Specifiying symbols used during test preprocessing
:defines:
  :commmon: &common_defines
    - UNITY_INT_WIDTH=16
    - CMOCK_MEM_INDEX_TYPE=uint16_t
    # - CMOCK_MEM_PTR_AS_INT=uint16_t # segmentation fault avec GCC dès l'utilisation de ExpectAndReturn si spécifié
    - CMOCK_MEM_ALIGN=1
    - CMOCK_MEM_SIZE=4096
  :test:
    - *common_defines
    - TEST # Simple list option to add symbol 'TEST' to compilation of all files in all test executables
    # - __eds__=
  :test_preprocess:
    - *common_defines
    - TEST
  :release: []

  # Enable to inject name of a test as a unique compilation symbol into its respective executable build.
  :use_test_definition: FALSE

# Configuration Options specific to CMock. See CMock docs for details
:cmock:
  # Core conffiguration
  :plugins:                        # What plugins should be used by CMock?
    - :ignore
    - :callback
  :verbosity:  2                   # the options being 0 errors only, 1 warnings and errors, 2 normal info, 3 verbose
  :when_no_prototypes:  :warn      # the options being :ignore, :warn, or :erro

  # File configuration
  :skeleton_path:  ''              # Subdirectory to store stubs when generated (default: '')
  :mock_prefix:  'mock_'           # Prefix to append to filenames for mocks
  :mock_suffix:  ''                # Suffix to append to filenames for mocks

  # Parser configuration
  :strippables:  ['(?:__attribute__\s*\([ (]*.*?[ )]*\)+)']
  :attributes:
     - __ramfunc
     - __irq
     - __fiq
     - register
     - extern
  :c_calling_conventions:
     - __stdcall
     - __cdecl
     - __fastcall
  :treat_externs:  :exclude        # the options being :include or :exclude
  :treat_inlines:  :include        # the options being :include or :exclude

  # Type handling configuration
  #:unity_helper_path: ''          # specify a string of where to find a unity_helper.h file to discover custom type assertions
  :treat_as:                       # optionally add additional types to map custom types
    uint8:    HEX8
    uint16:   HEX16
    uint32:   UINT32
    uint64:   UINT64
    int8:     INT8
    bool:     UINT8
  #:treat_as_array:  {}            # hint to cmock that these types are pointers to something
  #:treat_as_void:  []             # hint to cmock that these types are actually aliases of void
  :memcmp_if_unknown:  true        # allow cmock to use the memory comparison assertions for unknown types
  :when_ptr:  :compare_data        # hint to cmock how to handle pointers in general, the options being :compare_ptr, :compare_data, or :smart

  # Mock generation configuration
  :weak:  ''                       # Symbol to use to declare weak functions
  :enforce_strict_ordering: true   # Do we want cmock to enforce ordering of all function calls?
  :fail_on_unexpected_calls: true  # Do we want cmock to fail when it encounters a function call that wasn't expected?
  :callback_include_count: true    # Do we want cmock to include the number of calls to this callback, when using callbacks?
  :callback_after_arg_check: false # Do we want cmock to enforce an argument check first when using a callback?
  #:includes: []                   # You can add additional includes here, or specify the location with the options below
  #:includes_h_pre_orig_header: []
  #:includes_h_post_orig_header: []
  #:includes_c_pre_header:  []
  #:includes_c_post_header:  []
  #:array_size_type:  []            # Specify a type or types that should be used for array lengths
  #:array_size_name:  'size|len'    # Specify a name or names that CMock might automatically recognize as the length of an array
  :exclude_setjmp_h:  false        # Don't use setjmp when running CMock. Note that this might result in late reporting or out-of-order failures.

# Configuration options specific to Unity.
:unity:
  :defines:
    - UNITY_EXCLUDE_FLOAT

# You can optionally have ceedling create environment variables for you before
# performing the rest of its tasks.
:environment: []
# :environment:
#   # List enforces order allowing later to reference earlier with inline Ruby substitution
#   - :var1: value
#   - :var2: another value
#   - :path:            # Special PATH handling with platform-specific path separators
#     - #{ENV['PATH']}  # Environment variables can use inline Ruby substitution
#     - /another/path/to/include

# LIBRARIES
# These libraries are automatically injected into the build process. Those specified as
# common will be used in all types of builds. Otherwise, libraries can be injected in just
# tests or releases. These options are MERGED with the options in supplemental yaml files.
:libraries:
  :placement: :end
  :flag: "-l${1}"
  :path_flag: "-L ${1}"
  :system: []    # for example, you might list 'm' to grab the math library
  :test: []
  :release: []

:module_generator:
  :project_root: ./
  :source_root: src
  :inc_root: src
  :test_root: test
  :naming: :snake #options: :bumpy, :camel, :caps, or :snake
  :includes:
    :tst: []
    :src: []
...

The test file is also created inside the first subdirectory inside test/

Both of those things make sense.
The module_generator plugin puts the files in the first folder in the list, unless something else is specified when you call it. You're using the splat operator as the first item, so it's searching the /src/** folder and listing all the subdirectories... and (annoyingly enough) that operator returns src as the last item on that list.
You can fix the default by updating your paths like so:
:paths:
  :test:
    - test/
    - +:test/**
  :source:
    - src/
    - src/**
  :include:
    - src/
    - src/** # In simple projects, this entry often duplicates :source
  :libraries: []

You can also specify which path you want the module generator to use when you call it, like so:
ceedling module:create[src:filename]

Thank you very much Mark

Those paths are not used anymore by the plugin ?
:module_generator:
  :project_root: ./
  :source_root: src/
  :inc_root: src/
  :test_root: test/

Oh. Hm? Good point. I'll take a look. Perhaps we changed the naming to be more consistent?

It appears those features are called path_src, path_inc and path_tst but I'm not 100% sure that they aren't clobbered by ceedling every time? I'd need to run some experiments.

I tested those and they are clobbered by Ceedling

It would be nice if we can keep those options in case it is a simple fix but it is not an important feature, I could specify the directory on the call as you described it.

I think I see a straightforward solution... but I'll need to test it. :)

Great thank you ! Please let me know if I can help to test it

This should be fixed in the latest pre-release #60c24c2

@mvandervoord It works great, thank you !

Awesome! Thanks for checking @JuPrgn !


Add Move

This command is a bit tricky because it is the first to take in two paths. Generally, we only take a single path in. Internally to CircuitPython we have a single static packet buffer that takes in the header and the first path. This buffer size therefore, limits the length of the input path. If we add a second path to the command, we then would halve the amount of space per path.
An alternative would be to have some dynamic memory for the second path but that is not simple.


Indexers never expire LIVE snapshots

Describe the bug
Indexers currently never expire live snapshots. For long running indexers, this can cause disk saturation issues and also causes failures to delete once these reach the snapshot deletion service time.
Starting delete of snapshot SnapshotMetadata{name='LIVE_log_1689833167_dc25ce42-abbf-41ca-955e-e0888a66fe75', snapshotPath='LIVE', snapshotId='LIVE_log_1689833167_dc25ce42-abbf-41ca-955e-e0888a66fe75', startTimeEpochMs=1689228850484, endTimeEpochMs=1689837644041, maxOffset=45729812334, partitionId='83', indexType=LOGS_LUCENE9}

Exception deleting snapshot
error_root_cause_stack_trace:
    java.lang.IllegalArgumentException: Parameter 'Bucket' must not be null at software.amazon.awssdk.protocols.xml.internal.marshall.SimpleTypePathMarshaller.lambda$static$0(SimpleTypePathMarshaller.java:46) at software.amazon.awssdk.protocols.xml.internal.marshall.XmlProtocolMarshaller.doMarshall(XmlProtocolMarshaller.java:104) at software.amazon.awssdk.protocols.xml.internal.marshall.XmlProtocolMarshaller.marshall(XmlProtocolMarshaller.java:80) at software.amazon.awssdk.protocols.xml.internal.marshall.XmlProtocolMarshaller.marshall(XmlProtocolMarshaller.java:49) at software.amazon.awssdk.services.s3.transform.ListObjectsV2RequestMarshaller.marshall(ListObjectsV2RequestMarshaller.java:51) at software.amazon.awssdk.services.s3.transform.Li

To Reproduce
Long running indexers will show high stale snapshot counts on reboots. Also an increase in exception attempting to delete LIVE snapshots.
Deleting 38 stale snapshots: [SnapshotMetadata{name='LIVE_log_1690035728_cf55...

Expected behavior
Indexers should expire LIVE snapshots after a configurable time, or once cache node(s) start serving the content.

this fixes the symptom - https://github.com/slackhq/kaldb/pull/650
Probably not going to go with this

@kyle-sammons was digging on this today, and after we discussed his findings it appears that we do delete the chunks using the ChunkCleanerService (for indexers - not recovery). However, it appears that we may not be deleting the snapshots from ZK.
We should check to see if we leave the ZK nodes around - maybe also what happens when we have a ZK node that is searched but then doesn't exist on a node - why didn't that throw an exception? Or did it and we just didn't notice?
If the recovery nodes delete their chunks immediately after indexing in the RecoveryChunkManager indexer we may want to reconsider using a ChunkCleanerService here at all, and move this into the IndexingChunkManager for consistency.
In general, we should audit the recovery and indexer nodes - ensure both are cleaning up the disk appropriately, as well as ZK. And likely simplify some of the existing logic as it's currently a bit obtuse as to who cleans up what, when.


Provide ability to authenticate with service account PAT

To extend the functionality in https://github.com/zitadel/terraform-provider-zitadel/issues/27, it would be helpful to authenticate with a PAT of a service account instead of a jwt profile. PATs are easier to deliver in CI environments and serve the same purpose.
provider "zitadel" {
  domain                = local.zitadel_issuer
  personal_access_token = var.access_token
}

A validator can then check if one of the authentication methods is set.

Would love to have this feature, so we can start using Terraform at the instance level instead of at project level(using jwt profile token)


[TACKLE-107] Backend endpoint to return Assessment 'Risk' data for the Review screen

The Review screen contains the Assessment Summary table based upon questions answered during the Assessment.
The this issue covers the implementation of the API to return that data for a particular Application Id.

There is also a risk doughnut chart that will requires assessment data (please create a separate issue to cover this task if that is he most practical way to implement these API calls).

DOUBTS

This information is already returned in the Get Assessment operation #12 and the UI already has this information at this point, so my point is that this endpoint is not necessary.
Only showing Question,Answer and Rank ( risk ) is making lose Category context, so it can make not understandable the question.
Questions can be very long ( > 160 chars ) like this one , what could make the screen to show not correctly
"Does the application have any legal compliance requirements? e.g. PCI, HIPAA etc. Does the application have any licensing requirements? e.g. per core licensing
Answers can also be very long (> 120 chars ) like this one, what could make the screen to show not correctly
"Uses proprietary discovery technologies that are not kubernetes suitable e.g. hardcoded ip addresses, custom cluster manager"


Local copy of latest stm32-rs required to build

Waiting on an upstream release.

@richardeoin @astraw @hargoniX do you have pending breaking changes for stm32-rs that you would like to land before 0.8 or can we try pushing for that release to get this out of the way? I don't have anything pending for 0.8.

I don't have done anything related to stm32-rs yet so that's a clear no from me

Also nothing pending from me.

@jordens I've got two PRs open
https://github.com/stm32-rs/stm32-rs/pull/241
https://github.com/stm32-rs/stm32-rs/pull/246
Would be good to land them for 0.8


feat: 🎸 get treasury account

JIRA Link
https://polymesh.atlassian.net/browse/DA-64
Changelog / Description
adds GET /accounts/treasury method to retrieve treasury account information
Checklist -

[x] New Feature ?
[x] Updated swagger annotation (if API structure is changed) ?
[x] Unit Test (if possible) ?
[x] Updated the Readme.md (if required) ?

@polymath-eric shouldn't this get merged in alpha instead of master?

though - the alpha branch should be updated first - otherwise there is a huge list of changes that are in master but are not in alpha?

https://github.com/PolymeshAssociation/polymesh-rest-api/pull/143 I have created a PR for the same

:tada: This PR is included in version 2.2.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


delete_policy=no leads to division by zero in check_set_integrity

Due to a race condition the count of current document can become 0 for sources that pre-delete all documents to keep track of deletions (where delete_policy=no). Probably an all deletes Set should lead to PendingHarvesterDocument exception or similar to retry. But that needs to be tested thoroughly and that's not feasible before the Christmas break.

Commit to revert: 4838b51c9fbdf76e9409e6be0304aa855c47f053

Sentry error: https://surf-eduservices.sentry.io/issues/4725032389/?project=5318021&query=is%3Aunresolved+division&referrer=issue-stream&statsPeriod=90d&stream_index=1

I'm actually now suspecting that something is wrong with the logging instead of some arcane race condition. I'm going to leave the code as is and add some logging that indicates whether replacement occurs or not. That should give some insight into how this part of the code is functioning.


refactor(ES6): RuleSet.test.js

What kind of change does this PR introduce?
refactor, ES5 => ES6

Did you add tests for your changes?
All existing tests pass

If relevant, link to documentation update:
N/A

Summary
Refactor RuleSet.test.js to ES6

Does this PR introduce a breaking change?
No

Thanks


chore: update caip and type packages to fix build

closing as dupe of https://github.com/shapeshift/lib/pull/661


Improve Benchmark Test Utility

This functionality

[ ] should not rely on modifying a list buffer (make it functional)
[x] Not rely on string comparisons
[x] Not rely on specific running directories (This is modified int the refactor the build PR)
[x] Us standard spark CSV saving and loading
[x] Hook into scalatest in a meaningful way either through a specific type of test, or a mixin that applied to all of the cases in our library

#284


Is:iss is is:open by the provided example, tested, very easy to use, can this version be applied to production?

is:issue is:open通过提供的example，测试了，非常好用，目前这个版本可以应用生产吗

@zwmnhao1980 Thank you very much for your feedback. What version did you test?

0.61

@slievrly 0.6.1版本的

0.61

Yes, version 0.6.1 already has the ability for production, but there still are something we're working on it.So, I suggest you wait until after 0.7.0.


[BE] Apply Kotlin validation in INPUT requests DTOs

use Kotlin validation in input request DTOs, refactor Java validation intro Kotlin by example
like in UserAccountCreateRequest: @field@NotEmpty
instead of simple @NotBlank  as in Java(as it is not work in Kotlin)

StudyHistoryDto


Deployment notifications

Still a WIP but I wanted to publish this for discussion.
Sample output
$ emp deploy remind101/acme-inc:master
master: Pulling from remind101/acme-inc

420890c9e918: Already exists
a3ed95caeb02: Already exists
1e14aab0083d: Already exists
4393485f2bb3: Already exists
a2a051cf14f7: Already exists
1f9177248efa: Already exists
Digest: sha256:d05682347b56191a950f55ab06f11a6c6e34041921af32281b01f4a6a9a0f75f
Status: Image is up to date for remind101/acme-inc:master
Status: Created new release v64 for acme-inc
Status: Updating stack "acme-inc"
Status: Stack updated
Status: Deployment in progress for web
Status: Deployment in progress for worker
Status: Deployment stable for web
Status: Deployment stable for worker
Status: Closing event stream for release v64

There are issues with these tests: github.com/remind101/empire/tests/api. Relying on the Done channel to be closed via the EventsChan receiving an Event{Close: true} is really brittle.

This is looking good so far. Do mind splitting this into 2 PR's?

Adds the event stream.
Adds waiting for deployments to stabilize.

They're mostly orthogonal to each other and it'll make it easier to review.

This is handled by other PRs


[enhancement] Requirement for Legacy Editor to be disabled

Please consider possibly building in a setting/option to make v1 of outliner and zoom plugin work with Obsidian legacy editor and v2 to work with the new version.  Right now it says I have to manually install v1 to work with legacy editor so I can no longer auto update all my plugins.  Would love to see an option or to release themn as separate plugins even like a seperate plugin for legacy editor?

@jasondavis thank you for your feedback!
I understand your problem, but I don't want to do that because of several reasons:

I don't want to support Legacy Editor because I don't have time to properly support a modern editor
any public release will be considered relevant and supported, and people will create new issues
I just found a workaround

The workaround:

install v1 of the plugin
rename the folder .obsidian/plugins/obsidian-outliner to .obsidian/plugins/obsidian-outliner-legacy
rename the plugin in .obsidian/plugins/obsidian-outliner-legacy/manifest.json

That's exactly what I did (even before searching for a solution online), but somehow once you modify the manifest.json, Obsidian no longer wants to show the plugin in the list.
Manually installing v1 works, but as soon as I try to rename the plugin in manifest.json in order to stop updates, it doesn't show in the list anymore.
Maybe Obsidian blocks modified files...?
I think you could fix the issue by updating the latest v1 release, with the already-renamed manifest?

I created a repository with the renamed plugin. You can easily install it with BRAT by specifying URL to the repository https://github.com/vslinko/obsidian-outliner-legacy.

Thanks, it works perfectly!


Where is my multiselection?

VSCode Version: Code 1.17.2 (b813d12980308015bcd2b3a2f6efa5c810c33ba5, 2017-10-16T13:59:46.104Z)
OS Version: Windows_NT x64 6.3.9600
Extensions:

Extension
Author (truncated)
Version

csharp
ms-vscode
1.12.1

Steps to Reproduce:

Open VS Code
Open any file
Press Ctrl+Alt+Any Arrow Button
?????
PROFIT!!!!!111

Reproduces without extensions: Yes
P.S. On Win7 this bug isn't seen. Only on Win8 and Win8.1.

@kirusyaga Please check that you don't have any drivers or other software that "steals" those keybindings. I know there are some GPU drivers that steal those keys.
Search online for ctrl+alt+arrow not working.

@alexandrudima It seems that the Intel Graphics driver is to blame for this.


[Feature]: Global Admin Page for Appsmith

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Global admin has no clean UI to check how many users are using the instance or to understand the basic setting like https://docs.appsmith.com/setup/instance-configuration/disable-user-signup
would be super helpful if we cud create a simple page to list all of these features there.
Why should this be worked on?
This will enhance self hosted hosts of appsmith to have a better idea of the global level settings and metrics of their hosted applications.

@marks0351 We have global admin settings page under a feature flag now. Though it doesn't have user details, it lists all the global settings there and you can modify them from UI itself.


Create config.py

There is a reason this was not made. The plan is to eventually make tclib a Pypi package, and in that case it simply would not work with having a config file, located at C:\python3.7\Lib\site-packages (on windows) This is also the reason a client was not made.
I do appreciate your time and effort, but i hope you understand this decision.

Yeah I meant to close this with the other client I posted.
On Thu, Mar 14, 2019, 12:49 PM nortxort<EMAIL_ADDRESS>wrote:

There is a reason this was not made. The plan is to eventually make tclib
a Pypi package, and in that case it simply would not work with having a
config file, located at C:\python3.7\Lib\site-packages (on windows) This
is also the reason a client was not made.
I do appreciate your time and effort, but i hope you understand this
decision.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nortxort/tclib/pull/5#issuecomment-473032504, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AotEriWFJKp2LJ_b4Ojo6utjwmi4cqyDks5vWqesgaJpZM4bmfEJ
.


Lucio has ow2 healing penalty on crossfade/amp

lucio currently has a 60% reduction received from heal aura in normal crossfade and amp it up

he receives full healing from amp as of the latest patch he just doesn't receive it from regular crossfade

The math to fix this is a bit complicated, so don't expect this to be fixed any time soon.


Considering an update to support custom snowballs and other projectiles?

Maybe it's a good choice to take to make grenades or other interesting gadgets

This plug-in doesn't allow any 'real' projectiles because they suffer from several problems:

Their velocity is hard to control
They tend to change direction when they have in-flight explosion effects
Their gravity can't be controlled properly

Why do you need to use snowballs anyway? You can configure the speed and gravity of custom projectiles, which allows you to give them the same flight directory as a snowball.

I will close this for now due to inactivity. Anyone is free to reopen it when they explain why this feature would be needed.


request for built in analytics

Hello,
I'd like to know how you guys think about built-in analytics like GA or similar. Do you recommend using VueAnalytics or is there a plan to add it to the roadmap?
Cheers

Hi,
For using Google Analytics in Nuxt.js we use it as a plugin.
You can see how we use it on our documentation.


Changing bg color from grey to black doesn't cause test to fail

Overview
I get this with versions 5.0.4 and 5.0.3, unknown if on previous version
See my PR to demonstrate the issue here: https://github.com/syntapy/superformula_clone/pull/93
Before / After images
No diff is generated from these two images
I did set the error threshold to 0
Fyi this is a clone website I'm working on for practice, not the actual Superformula website

bg color = '#1a1a1a'
bg color = 'black'

Seems to be an issue as far back as 3.0.0, since in my temporary PR the mobile tests pass even with the bg color changed

The reason for this is the sensitivity option for pixelMatch, currently it is set by default to 0.1 (https://github.com/mapbox/pixelmatch?tab=readme-ov-file#pixelmatchimg1-img2-output-width-height-options).
Since this difference is subtle and almost invisible to the naked eye, I wouldn't change the implementation.
What we could do though is expose this parameter so you can change it to 0 in your setup to hopefully achieve desired effect

Could I give a at making the PR for this issue?

Please do

Hey @syntapy let's agree on how to continue, I'd like to be aware of the progress so that I can jump in if things don't work out. I see that your forked branch is quite out of date, so just let me know where/how you will be working on it.
Appreciate it 🙂

Released in v5.2.0, there is a possibility now to change pixelmatch threshold sensitivity (or other options)


corrected permission requirement

@sravanisaluru
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Please also update the part for SQL Database:
the new permission is VIEW DATABASE SECURITY AUDIT

Replaced by internal PR.


Check Permission is working backwords

Hi,
It seems the check permissions is working reverse of what it should do.
If I check "no" the email to check permissions is sent. (& the user is added in the mailchimp list only if I confirm from the email)
If I check "yes" the email is not sent and the user is not added in the mailchimp list.
Can you please help with this?
Thanks,

I'm not sure this is a bug @raullukacs, as Mailchimp does the opt-in if you haven't sent a check_permission field.
Can you please show me your mailchimp.yaml file?

Do you know where I can find the mailchimp.yaml file ?
I can confirm that double-opt in is not checked on the list.
Thanks,

It in site/settings/addons

Here you go: ( I removed the keys)
mailchimp_key: XXXXXXXXXXXXXXXXXXXXXXXXX
add_new_users: false
forms:
mailchimp_list_id: XXXXXXX
form: become-ambassador

mailchimp_list_id: XXXXXXX
form: newsletter
license_key: ""

Right so under those settings, I would expect Mailchimp to send the opt-in email as you are not intentionally bypassing it (and you shouldn't) and you are not sending check permissions data.

@edalzell And if I want to bypass it, is there a setting like check_permission: false I can put in the yaml file or somewhere else?
Thanks,

Like it says in the docs Not recommended, but you can disable the Double Opt In.

@edalzell Ok . Thanks for all your help.