Remove --hard flag

Background
--hard: Replaces package references within project files
          even if they are not yet adhering to the Paket's conventions
          (and hence considered manually managed).

Sample
in default mode paket 2.0 will not change references to libraries if they are not already controlled by paket. A common case is "file/ enw F# project" creates a fsproj file that already references FSharp.Core. If you then add FSharp.Core as nuget package, then Paket will not replace the reference unless you specify --hard parameter.
Suggestion
I suggest we remove the --hard flag in paket v3 and thread all calls as if --hard would have been set.

[x] removed flag from all commands
[x] fixed tests
[x] update docs

We will not create a --soft flag to restore old behaviour, but if you really need to keep that existing reference you can manually add a
<paket>false</paket>

This is obviously breaking, but it was a big source for confusion.
I'm only suggesting this for v3 so I'm relaxed about it like this guy:

This is good, it's what I think people expect anyway as the default.

:+1:

It fine to remove it in my opinion. But what would happen if  I try to use it ? Will I gen an error or just it will be ignored ? I prefer not to have an error this way I don't need to update my scripts.

With this pr we would throw an error: unknown parameter --hard
On Apr 4, 2016 8:28 PM, "tsibelman"<EMAIL_ADDRESS>wrote:

It fine to remove it in my opinion. But what would happen if I try to use
it ? Will I gen an error or just it will be ignored ? I prefer not to have
an error this way I don't need to update my scripts.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/pull/1567#issuecomment-205436453

Can this be a warning instead ?

Since this is a major release change I would prefer not to take that debt.
On Apr 4, 2016 8:43 PM, "tsibelman"<EMAIL_ADDRESS>wrote:

Can this be a warning instead ?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/pull/1567#issuecomment-205442721

Agree with @forki - this a major release with a breaking change that should be documented as part of the release.


Equalizing requirements.txt versions

Equalizing requirements.txt versions.
Error dependencies.
I`m using python 3.9 and 3.10

After update requirements.txt


Feature request: Support OCIO color conversions when viewing QuickTimes

It appears that the use of OCIO configs to control the colorspace configurations only works for image sequences. In my testing (2.0.8, Windows 10, x64) OCIO colorspace settings were ignored when playing back QuickTime files.
It would be very useful for us to be able to apply OCIO colorspace correction to QuickTimes, during playback. We often output QuickTimes in a client requested colorspace which differs from the display colorspace of our artist workstations. Thus, it would be beneficial to be able to view those files correctly on the artist workstations (which we cannot do, currently).
Thanks!

Hi, I just tried here with a ProRes .mov file and it seemed to be working OK. You have color spaces set for both the display and image? In the image section of the "Color Space" window you can specify the color space either as the "Default" or specifically for the "FFmpeg" plugin.

Thanks for the quick reply. That's interesting that it works for you.
I'm having a hard time getting consistent results.
I've tried using the "Nuke Default" config, and my own custom OCIO config (which works perfectly with image sequences), and both ProRes 4444 and DNxHD formats. I'm setting both the image and the display colorspaces.
In my initial test it did not work (hence submitting the issue). However, as it worked for you, I re-tested it. In some of those tests, it worked as expected, and as you described. Other times, it appeared that my colorspace settings were ignored. Changing them does not update the display. Having said that, I've also just encountered an issue where DJV stops playing back the QT (I click play, the time-slider moves, but the image hangs on the first frame), so now I'm not sure if that is the problem I sometimes am encountering (the colorspace was changing internally but the display was not refreshing), as opposed to an OCIO colorspace problem.
I wish I could diagnose the exact steps that are causing the issue, to provide you with a useful bug report, but it's behaving inconsistently for me. Each time I test it, I get different results. I'm thinking that unless this is an issue for other people, we maybe just ignore that it is an issue for me? We have a work around (which we've used up until this point, anyway), so it isn't a show stopper for us.
Thanks,
Steve

I did some more testing and still haven't been able to reproduce what you are seeing, but I'm actually working on the color space code right now and have been fixing up some parts that looked a bit dodgy. I'll have a new version available in a couple weeks, if you have some time to test it out that would be great.
For the QT file, is it video only (no audio tracks)? I just noticed that movie files without audio tracks don't playback properly, usually getting stuck on the first frame. I've made a separate issue for that:
https://github.com/darbyjohnston/DJV/issues/331

Hi. Yeah, absolutely, I'll definitely test out the new version when released. If it isn't too much to ask, can you please reply here when it's released? I might miss it, otherwise.
The QTs I'm using are video only, so audio isn't the cause of the issue for me, unfortunately. When the issue occurs I am sometimes able to scrub to a different frame, and eventually the view might update to that frame, but then hangs on that frame instead. I thought maybe my machine was struggling to keep up with the processing - but it's a fairly powerful workstation (6 core i7, 32GB RAM, NVIDIA 770GTX). It seemed to be aggravated by higher bitrate codecs (i.e. the ProRes 4444 file was worse than the DNxHD file - both the same footage, but the ProRes file is about 250MB vs 50MB for the DNxHD)

Thanks, I'll let you know when the new version is available.
Sorry, I meant that QTs without audio tracks are causing the playback issues, I hope to have that fixed in the next version as well.

I'm going to add this to the next milestone and close it as "fixed", we can re-open it later if your testing still shows the problem exists.


Update .envrc

Added tmp directory

Hey @Arunkumar7540, thanks for the PR! As the reddbot said, you don't have a CLA yet. If you could do that, and then reopen the issue, that would be great.


When using query raw SQL with scan, will no error be returned when the record does not exist？

Your Question
        sql := `select * tests_tb where id=?`
	err := db.Raw(sql, -1).Scan(&obj).Error
	if err != nil {
		return err
	}

id=-1 not exists, but err is nil, can't use IsRecordNotFoundError
Expected answer
Can't use this method to judge errors when using raw scan？

We only raise ErrRecordNotFound error for First, Last, Take

how to judge no record?

This seems to be a big difference compared with v1 (which does raise ErrRecordNotFound when using db.Raw).
Can this be reconsidered to make migration from version 1 to version 2 easier?

This seems to be a big difference compared with v1 (which does raise ErrRecordNotFound when using db.Raw).
Can this be reconsidered to make migration from version 1 to version 2 easier?


BUG: GET templates for a service returns 500 if no templates are on that service

Description
When exercising the API GET /service/{service-id}/template route, if that service has no templates, then the API returns a 500 Internal Server Error and a very long stacktrace is thrown in the logs.

[ ] Ticket is understood, and QA has been contacted (if the ticket has a QA label).

Steps to Reproduce

Use Postman to call the GET /service/{service-id}/template route where no templates exist on that service
Observe that a 500 is returned
View the logs for a TypeError (or one of the other logs you see in the screenshot below)

Workaround
Use the Portal to look things up
Impact/Urgency

Low, an annoyance and poor API behavior.
Expected Behavior

[ ] When I use Postman to call the GET templates for a service route, if there are no templates on that service then a 200 response and an empty array ([ ]) is returned

QA Considerations

[ ] Validate and Automate (needs the ability to switch to a different service in the QA Suite)

Additional Info & Resources

Logs (truncated)

closing - wrong understanding of the bug.


nixos/nginx: forced change permissions

Motivation for this change
Fixed this error - https://github.com/NixOS/nixpkgs/pull/56255#issuecomment-566547096
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

As described in https://github.com/NixOS/nixpkgs/pull/56255#issuecomment-568277115, please use

z ${cfg.stateDir}/logs/* 6?? ...`

We don't want to make log files executable.
We should however have a discussion about ?? (the permissions for group and other), as well as about how to handle ${cfg.stateDir}/*_temp (and contents in there).

Thanks, fixed.

I've merged this as nginx service is broken without this. Discussion about *_temp is still welcome

@danbst: the tmpfiles rule is still wrong - it should be a lowercase z, and a glob pattern matching files - please see https://github.com/NixOS/nixpkgs/pull/76174#issuecomment-568287028
Can you post a fixup PR which does that?

@flokli yes

@flokli I'm testing z (small-z) and it doesn't change ownership. But big-Z does change ownership:
With
    systemd.tmpfiles.rules = [
      "z '${config.services.nginx.stateDir}/logs' 0750 nginx nginx - -"
    ];

[root@s1:~]# chown root:root /var/spool/nginx/logs/somefile

[root@s1:~]# ls -la /var/spool/nginx/logs
total 16
drwxr-x--- 2 nginx nginx 4096 Dec 26 12:00 .
drwxr-x--- 8 nginx nginx 4096 Dec 26 12:00 ..
-rw-r--r-- 1 nginx nginx   88 Dec 26 12:05 access.log
-rw-r--r-- 1 nginx nginx  380 Dec 26 12:00 error.log
-rw-r--r-- 1 root  root     0 Dec 26 12:00 somefile

[root@s1:~]# nixos-rebuild switch
building Nix...
building the system configuration...
updating GRUB 2 menu...
activating the configuration...
setting up /etc...
reloading user units for root...
setting up tmpfiles

[root@s1:~]# ls -la /var/spool/nginx/logs
total 16
drwxr-x--- 2 nginx nginx 4096 Dec 26 12:00 .
drwxr-x--- 8 nginx nginx 4096 Dec 26 12:00 ..
-rw-r--r-- 1 nginx nginx   88 Dec 26 12:05 access.log
-rw-r--r-- 1 nginx nginx  380 Dec 26 12:00 error.log
-rw-r--r-- 1 root  root     0 Dec 26 12:00 somefile

With
    systemd.tmpfiles.rules = [
      "Z '${config.services.nginx.stateDir}/logs' 0750 nginx nginx - -"
    ];

[root@s1:~]# chown root:root /var/spool/nginx/logs/somefile

[root@s1:~]# ls -la /var/spool/nginx/logs
total 16
drwxr-x--- 2 nginx nginx 4096 Dec 26 12:00 .
drwxr-x--- 8 nginx nginx 4096 Dec 26 12:00 ..
-rwxr-x--- 1 nginx nginx   88 Dec 26 12:05 access.log
-rwxr-x--- 1 nginx nginx  380 Dec 26 12:00 error.log
-rwxr-x--- 1 root  root     0 Dec 26 12:00 somefile

[root@s1:~]# nixos-rebuild switch
building Nix...
building the system configuration...
updating GRUB 2 menu...
activating the configuration...
setting up /etc...
reloading user units for root...
setting up tmpfiles

[root@s1:~]# ls -la /var/spool/nginx/logs
total 16
drwxr-x--- 2 nginx nginx 4096 Dec 26 12:00 .
drwxr-x--- 8 nginx nginx 4096 Dec 26 12:00 ..
-rwxr-x--- 1 nginx nginx   88 Dec 26 12:05 access.log
-rwxr-x--- 1 nginx nginx  380 Dec 26 12:00 error.log
-rwxr-x--- 1 nginx nginx    0 Dec 26 12:00 somefile

@danbst doesn’t work for me either

@Izorkin it is because little-z expects glob-pattern, like ${stateDir}/logs/*. In the end I went with big-Z but without changing mode

@danbst correct worked with:
Z '/var/spool/nginx/logs/*.log' 0650 nginx nginx - -


Unable to extract title

➜  /tmp youtube-dl -v "https://www.youtube.com/watch?v=8-7M1--bbOM&list=PLEZm9Qpp2gIOUhkPjL9SwVp1Lex5Wuyd1"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=8-7M1--bbOM&list=PLEZm9Qpp2gIOUhkPjL9SwVp1Lex5Wuyd1']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.02.22
[debug] Python version 2.7.12 - Linux-4.4.0-93-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: ffmpeg 2.8.11-0ubuntu<IP_ADDRESS>, ffprobe 2.8.11-0ubuntu<IP_ADDRESS>, rtmpdump 2.4
[debug] Proxy map: {}
[youtube:playlist] Downloading playlist PLEZm9Qpp2gIOUhkPjL9SwVp1Lex5Wuyd1 - add --no-playlist to just download video 8-7M1--bbOM
[youtube:playlist] PLEZm9Qpp2gIOUhkPjL9SwVp1Lex5Wuyd1: Downloading webpage
ERROR: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 666, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 316, in extract
    return self._real_extract(url)
  File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1730, in _real_extract
    return self._extract_playlist(playlist_id)
  File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1700, in _extract_playlist
    page, 'title')
  File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 620, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 611, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

➜  /tmp

Hi there, I dont know if I must say any detail or not

Version 2016.02.22 is way too old. Please check https://github.com/rg3/youtube-dl/blob/master/README.md#how-do-i-update-youtube-dl.


multiple trival modifications

Signed-off-by: kevin xu<EMAIL_ADDRESS>

Current coverage is 44.78% (diff: 0.00%)

Merging #130 into master will increase coverage by 0.20%

@@             master       #130   diff @@
==========================================
  Files            16         16
  Lines          8907       8905     -2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3971       3988    +17
+ Misses         3911       3879    -32
- Partials       1025       1038    +13

Powered by Codecov. Last update ee65507...2a80ac0


install the enviroment failed

pip install torch-scatter==1.1.0 -f https://pytorch-geometric.com/whl/torch-1.4.0.html
pip install torch-sparse==0.4.4 -f https://pytorch-geometric.com/whl/torch-1.4.0.html
pip install torch-cluster==1.4.5 -f https://pytorch-geometric.com/whl/torch-1.4.0.html
pip install torch-spline-conv==1.1.0 -f https://pytorch-geometric.com/whl/torch-1.4.0.html
pip install torch-vision==0.5.0
I am trying to install these but all failed, I am asking for help if there is someone who successfully installed these pakeges? Thank you.

Yes, you can construct ur dataset by KNN.

I use KNN for each data point to construct N subgraph or just use one time to construct a graph? hua zheng @.*** 签名由 网易邮箱大师 定制 On 06/30/2021 02:07, Yuning You wrote: Yes, you can construct ur dataset by KNN. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

Looks like either work for me.

So I should construct the graph data same format like the TU dataset? the raw data have many documents as far as I know. Thank you for your kind help! hua zheng @.*** 签名由 网易邮箱大师 定制 On 06/30/2021 09:28, Yuning You wrote: Looks like either work for me. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

Yes replace tu_dataset is one way to plugin ur own data.


162 판매꿈 검색

#162

Codecov Report

Merging #165 (9de37e9) into main (8c30376) will decrease coverage by 1.11%.
The diff coverage is 5.12%.

@@             Coverage Diff              @@
##               main     #165      +/-   ##
============================================
- Coverage     62.65%   61.53%   -1.12%
- Complexity      412      413       +1
============================================
  Files           123      124       +1
  Lines          1478     1508      +30
  Branches         10       10
============================================
+ Hits            926      928       +2
- Misses          547      575      +28
  Partials          5        5

Impacted Files
Coverage Δ

...g/zamong/controller/dream/SellDreamController.java
7.69% <0.00%> (-0.65%)
:arrow_down:

.../zamong/controller/dream/ShareDreamController.java
27.27% <0.00%> (ø)

...to/response/dream/selldream/SellDreamResponse.java
0.00% <0.00%> (ø)

...vice/dream/sell/find/SellDreamFindServiceImpl.java
1.96% <0.00%> (-0.75%)
:arrow_down:

...ce/dream/share/find/ShareDreamFindServiceImpl.java
64.06% <ø> (ø)

...elldream/search/SellDreamSearchRepositoryImpl.java
20.00% <20.00%> (ø)

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 8c30376...9de37e9. Read the comment docs.


Update documentation (readme.md file)

THe documentation (readme.md file) is incomplete and needs to be update.

I approved and merged your PR. It looks like there will need to be a couple more updates but the last changes are live in staging so we can review them.


performance issues when embedding processing.js sketches inside inspire.js steps

I'm not sure if anybody else has tried this, but I'm happy to report that we can embed processing.js sketches inside inspire.js steps. Here is a demo you might enjoy:
http://bar.utoronto.ca/~jwaese/zoom/zoomBrowserTest.html#/m
But here's the challenge -- since the processing sketches are all running at the same time, everything runs a lot slower than it should. Also if you open your Activity Monitor, you'll see some crazy memory requirements for the browser, and it keeps going up the longer the page is open. And these are really simple processing sketches. I fear it would be prohibitively slow if they were any more complex.
Anyhow, I'm wondering if you can suggest a technique to pull some slight of hand here, and only execute/display the processing sketches just before they should become visible? Or, do you have any other suggestions for improving the step level performance of an impress.js page? I'm hoping to use it as a framework for a set of interconnected data visualization tools, which means somewhat more complex processing sketches than my demo above!
Thanks,
Jamie

As from the comment above and due to the lack of feedback from the OP, I will assume the issue was fixed, if not please comment in this issue to reopen.


Translations update from Weblate

Translations update from Weblate for mattermost-languages-shipped/mattermost-webapp_release-5.25.

Blocking this because this will/should be changed to v5.26 branch.


Update n98-magerun.phar / phpcs.phar

Especially n98-magerun would be import as the version in this repo is not compatible with PHP 7.

Thanks, closed by 5bfd664db0a0a50c5c0aa3641c16c497e1ff06a7


Update repositories.txt

I am moving the HeliOS library from my personal GitHub account (https://github.com/mannypeterson) to a GitHub organization (https://github.com/heliosproj) which will remain the home for the project (library) for the foreseeable future. Thank you, Manny Peterson

Hello @MannyPeterson. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests. They will merge this pull request when it is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page. Information about that here: https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

Thank you @per1234

Done:
2023/03/22 16:53:56 Loaded 5648 libraries from DB
Changing URL of library HeliOS from https://github.com/MannyPeterson/HeliOS.git to https://github.com/heliosproj/HeliOS.git
Success!


Add filters validation

https://projects.engineering.redhat.com/browse/SSP-315

I feel we may just want to update the api spec rather than validate in the code. Return an empty list and code 200 seem fine to me.

@hsong-rh per our discussion, please clean up openopi.json that we don't specifically list allowed filtered attributes for requests because the filtering is handled in a common way for listing any resources. As a result returning code 200 with an empty array is valid.

@bzwei Cleanup on openapi.json is done, please review.


ansible-lint: 6.14.0 -> 6.14.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ansible/ansible-lint/releases.
meta.description for ansible-lint is: Best practices checker for Ansible
meta.homepage for ansible-lint is: https://github.com/ansible/ansible-lint
meta.changelog for ansible-lint is: https://github.com/ansible/ansible-lint/releases/tag/v6.14.2
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.14.2 with grep in /nix/store/bgf6agvyzp1m8ywsdsmdxzkhc33sb5pi-python3.10-ansible-lint-6.14.2
found 6.14.2 in filename of file in /nix/store/bgf6agvyzp1m8ywsdsmdxzkhc33sb5pi-python3.10-ansible-lint-6.14.2

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
ansible-lint
python310Packages.ansible-lint
python311Packages.ansible-lint

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bgf6agvyzp1m8ywsdsmdxzkhc33sb5pi-python3.10-ansible-lint-6.14.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bgf6agvyzp1m8ywsdsmdxzkhc33sb5pi-python3.10-ansible-lint-6.14.2
ls -la /nix/store/bgf6agvyzp1m8ywsdsmdxzkhc33sb5pi-python3.10-ansible-lint-6.14.2/bin

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

2 packages built:

ansible-lint (python310Packages.ansible-lint)
python311Packages.ansible-lint

Maintainer pings
cc @sengaya for testing.

Result of nixpkgs-review pr 220686 run on aarch64-darwin 1

2 packages built:

ansible-lint (python310Packages.ansible-lint)
python311Packages.ansible-lint


Android.graphics.pdf.PdfRenderer.finalize() timed out after 10

I have already implemented open pdf file from SD card, but sometimes it crashes.
EXCEPTION:
08-10 15:11:03.952 1617-1635/com.notviz.android E/AndroidRuntime: FATAL EXCEPTION: FinalizerWatchdogDaemon
Process: com.notviz.android, PID: 1617
java.util.concurrent.TimeoutException: android.graphics.pdf.PdfRenderer.finalize() timed out after 10 seconds
at android.graphics.pdf.PdfRenderer.nativeClose(Native Method)
at android.graphics.pdf.PdfRenderer.doClose(PdfRenderer.java:223)
at android.graphics.pdf.PdfRenderer.finalize(PdfRenderer.java:212)
at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:191)
at java.lang.Daemons$FinalizerDaemon.run(Daemons.java:174)
at java.lang.Thread.run(Thread.java:818)
08-10 15:11:13.739 1617-1638/com.notviz.android D/ANRAppManager: !!! It is not under singleton mode, U can't use it. !!!
08-10 15:11:13.740 1617-1638/com.notviz.android W/Binder: Caught a RuntimeException from the binder stub implementation.
java.lang.NullPointerException: Attempt to invoke interface method 'android.os.IBinder com.mediatek.anrappmanager.IFrameworks.serviceManagerGetService(java.lang.String)' on a null object reference
at com.mediatek.anrappmanager.ANRManagerNative$1.b(SourceFile:77)
at com.mediatek.anrappmanager.ANRManagerNative$1.c(SourceFile:75)
at ...

Hi,
when does this happen? when opening the PDF file? or when closing the app?
Looks like a PdfRenderer issue. I'll have a look.

I have problem for this as well:
Devices are : Samsung SM-T805 	5.0.2, SM-T805 5.0.2, LG-D855 5.0
Fatal Exception: java.util.concurrent.TimeoutException: android.graphics.pdf.PdfRenderer.finalize() timed out after 10 seconds
       at android.graphics.pdf.PdfRenderer.nativeClose(PdfRenderer.java)
       at android.graphics.pdf.PdfRenderer.doClose(PdfRenderer.java:219)
       at android.graphics.pdf.PdfRenderer.finalize(PdfRenderer.java:208)
       at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:209)
       at java.lang.Daemons$FinalizerDaemon.run(Daemons.java:192)
       at java.lang.Thread.run(Thread.java:818)

Hi again,
could you send a link to a PDF where this error appears?
I'd never seen it, so I'd like to test it in my devices and see what's happening.

Hi voghDev, I have the same problem too!

Hello,
In search for deeper information about this issue, this answer is very helpful. The error is related to GC while Android turns the phone into a "Low battery consumption" mode. A proposed solution to mitigate the error is to acquire a WAKE_LOCK while the user is reading the PDF document. I haven't tried that yet.
Good to have a PDF that causes the crash. We will try to reproduce your case and post any news.


"no info on this car ID" on status - but all other functions work!

Hi there!
Any idea what I could be doing wrong? I've run through all your various query tools and all have worked - e.g. http://xxx.xxx.xxx.xxx:8080/api/v1/cars/1/charges show me all the charges. But http://xxx.xxx.xxx.xxx:8080/api/v1/cars/1/status yields the error "no info on this car ID"
Any idea at all what the issue could be?
Thanks!

Hi @chamsters,
My initial thought is that it's probably the configuration of the MQTT setting that is either missing or faulty.
Are you seeing any entry in the log of MQTT and/or TeslaMateApi when restarting the TeslaMateApi container?
Regards,
Tobias

So, oddly - it started working after my first charge. I'm guessing that I had hooked this up to a new instance of teslamate which was still seeding a lot of initial data, hence the error?

Thanks for the help!


Previous Versions URLs should preserve the language chosen

We always will switch to an English version. That is annoying for other languages.
This would look like improving the logic in https://github.com/Qiskit/qiskit_sphinx_theme/pull/267 to handle when locale/bn_BN is in the URL, e.g. https://qiskit.org/documentation/locale/bn_BN/index.html. The change should use test driven development by modifying our Jest tests.

This should be much easier thanks to https://github.com/Qiskit/qiskit_sphinx_theme/pull/321.

@coruscating had a great suggestion based on Python docs:

They will show an English page for an untranslated older version page.
We'd have to change our workflow to render all pages for different languages even if they're not translated though

We don't version translations. https://qiskit.org/documentation/locale/fr_FR/stable/0.43/index.html and https://qiskit.org/documentation/stable/locale/fr_FR/0.43/index.html don't exist. So this is not even relevant.
It would be good to deploy translations for previous versions of the docs. But given that the docs are moving, this should be part of that new infrastructure rather than done via qiskit-sphinx-theme.


[BugFix][VTA] Fix vta_conv2d crash issue after change vta_config.json

configuration.
Issue:
Once change LOG_BLOCK_IN or LOG_BLOCK_OUT into > 4 value, when run vta
“Simple Matrix Multiply” or load vta, vta would crash at vta_conv2d.py.
Analysis:
This issue caused by resnet18 logic of vta_conv2d.py which have
in_filter minmum size that is 16. > 4 value would cause such in_filter
check failed then make xfer_size be empty and find_schedules function
return a empty list finally cause crash.
Solution:
add the empty list check.
Thanks for contributing to TVM!   Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers.

@tqchen @tmoreau89 could you help to review this patch?
Thanks
Hua

@tmoreau89 , the change request already addressed, please kindly let me know if you have any other comments.


Temporarily pin scipy < 1.9

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR is a workaround  for #1142

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

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] HISTORY.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added

[x] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

Pins scipy below v1.9 until we can implement a workaround for lmoments3 tests

Does this PR introduce a breaking change?
Yes. Dependencies have been affected.
Other information:

Coverage remained the same at 91.433% when pulling 15ff7937b7a2aae9764f3e4b452d425d6a1af423 on pin_scipy into 6133c4686143a6735bc4a2ed0aa8924ef7eba9a3 on master.


feat: query retry on race condition

Issue being fixed or feature implemented
Query Tx could have a race condition where we would execute a query between the time that the state would be saved and the transaction committed. This would lead to invalid proofs. A few solutions were possible here, however since we need the main thread to never need to wait on reader threads using a RWLock is problematic as the writer would at least need to wait on readers on a fair strategy. While this isn't terrible a better solution is to allow the queries and proof generation to fail very rarely and instead have a mechanism that detects such failures.
What was done?
We introduce a block height guard in the platform struct that is used to verify that the commit of the transaction happened on the main thread. Any queries that happened between the state save and the commit are restarted.
How Has This Been Tested?
We ran load test on thousands of transactions, the error no longer happens.
Breaking Changes
Not a breaking change.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have added "!" to the title and described breaking changes in the corresponding section if my code contains any
[x] I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

This has approval by Ivan to be merged.


feat: integrating linter

Description

Integrating eslint
Removing errors and warning from the lint.

Related issue(s)

Resolves #29

@Souvikns Tip for the future: feat: and fix: should be only used when you improve the code that can be consumed by the package's user. In your case you only fix the indentations/linter errors so it should be treated as chore: - checks our doc https://github.com/asyncapi/asyncapi-react/blob/master/CONTRIBUTING.md#conventional-commits
Also please fix the all errors that are throw by linter. If you think that someone linter's suggestions are unnecessary, please change the config for linter.

@Souvikns Tip for the future: feat: and fix: should be only used when you improve the code that can be consumed by the package's user. In your case you only fix the indentations/linter errors so it should be treated as chore: - checks our doc https://github.com/asyncapi/asyncapi-react/blob/master/CONTRIBUTING.md#conventional-commits

Oh sorry, I am updating this thanks for the tip.

Also please fix the all errors that are throw by linter. If you think that someone linter's suggestions are unnecessary, please change the config for linter.

Yeah, there is a lot of issues, I am fixing them one by one. Before changing any linter config I will discuss here.

@magicmatatjahu
for spec.ts files no-undef is turned off but still the linter is throwing error have i done something wrong is setting up ?
"overrides": [
    {
      "files": "*.spec.ts",
      "rules": {
        "no-undef": "off",
        "security/detect-non-literal-fs-filename": "off",
        "sonarjs/no-duplicate-string": "off",
        "security/detect-object-injection": "off",
        "max-nested-callbacks": "off",
        "sonarjs/no-identical-functions": "off",
        "@typescript-eslint/no-non-null-assertion": "off",
        "@typescript-eslint/no-unused-vars": "off"
      }
    }
  ]

ss

Also the linter is throwing error for no-underscore-dangle should I turn this off or it is requried. _ is used for private members of classes.

@Souvikns You have defined only ts files and your tests have tsx extension :)

Also the linter is throwing error for no-underscore-dangle should I turn this off or it is requried. _ is used for private members of classes.

You should disable it in the config.


Create and enable nodemanager service for dynamic cluster

Refer https://github.com/wls-eng/arm-oraclelinux-wls-dynamic-cluster/blob/develop/arm-oraclelinux-wls-dynamic-cluster/src/main/scripts/setupDynamicClusterDomain.sh
Following needs to be added under admin server setup.
create_nodemanager_service
enabledAndStartNodeManagerService
Once this is done it needs to be enabled for https://github.com/wls-eng/arm-oraclelinux-wls-dynamic-cluster/blob/develop/.github/workflows/build.yml

PR created
https://github.com/wls-eng/arm-oraclelinux-wls-dynamic-cluster/pull/50


carousel is not working for upgraded flutter in my project.Do anyone can help me......?

URL: https://pub.dartlang.org/packages/carousel
I upgraded my flutter today and got an issue
The current Dart SDK version is 2.1.0-dev.1.0.flutter-ccb16f7282.
Because breadfruit depends on carousel >=0.1.0 which requires SDK version <2.0.0, version solving failed.

This issue was moved to gbrvalerio/carousel#11


libfido2: 1.4.0 -> 1.5.0

Motivation for this change
Version update
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build libfido2

please rebase it onto staging
@GrahamcOfBorg eval

Changed the branch to staging


Add text and markdown components

What's new?

Text component
Markdown component

Screenshots

Text
Responsive centered plain text. Background is white if the user's system prefers white, or the background is black if the dark theme is preferred. It adjusts automatically.

Markdown
Beautiful just like Github's markdown

Hey @RotemDl! Changes already here and waiting your review. 🫨😀
PR in dataloop-ai/components:
feat(DlTypography): add markdown and plaintext #863
Cheers!


Catch redundant sleep/wake commands, and notify the user that Talon is already in the requested mode

This commit adds some commands to catch redundant sleep/wake commands, and notifies the user.
The main example is, if Talon is already awake, the "talon wake" and "wake up" commands aren't active. So if the user is unsure of their current mode and says "wake up", Talon interprets the command as something else ("page up", for example).
These new commands are active in the mode the user is trying to activate. They catch the command, and trigger an app.notify to let the user know they're already in that mode.

Thanks @phillco. I agree with your suggestions. I've commited the changes :)


Do timestamp production closer to event instrumentation sites

By producing timestamps (if not provided by the user) closer to the instrumentation site we incur the cost of a little extra work on the instrumented thread (to produce the system time), for the benefit of more precise timestamps that aren't offset due to the asynchronous nature of the ingest worker thread.
The first commit also contains some alternate variants on the monitored_pipeline example demonstrating different ways of mixing nonce and timestamp based interaction annotation.  Removed in the second commit to keep the example clutter down.

My initial gut reaction is that I'd like to see the timestamp capture mirror (or replace) what is currently done for tick: https://github.com/auxoncorp/modality-tracing-rs/blob/master/tracing-modality/src/common/layer.rs#L61
I have to imagine that just always copying the current time into the struct would be faster than the hash lookup to check if we should and, at least IMO, is cleaner.
Thoughts?

I disagree about the  "unconditionally clobbering the timestamp field" portion.  If the user has decided to provide a custom value for the timestamp, we should respect it, since they are in possession of richer context and understanding of the system being instrumented.

That's not what I'm suggesting.
I'm suggesting that we unconditionally fill a struct field on the trace'd thread, like tick: https://github.com/auxoncorp/modality-tracing-rs/blob/master/tracing-modality/src/common/layer.rs#L61
Then we do the implicit/explicit value choice along with all the other metadata that is already happening in the ingest thread, like tick: https://github.com/auxoncorp/modality-tracing-rs/blob/master/tracing-modality/src/common/ingest.rs#L449-L456

Hmm, tick is for some reason not actually checking for an explicit value on the ingest side, look at any of the other ones: Edit: like anything besides tick, I guess: https://github.com/auxoncorp/modality-tracing-rs/blob/master/tracing-modality/src/common/ingest.rs#L578-L586

Okay, thank you for the clarification on clobbering.
Pros of unconditionally producing a timestamp as suggested:

Central location of timestamp generation
Remove one hash-table lookup

Cons:

Do system-time calls per instrumentation activity even if that time info is definitely going to get ignored:

When the message is not getting turned into an event
When the message already contains a user-provided timestamp

The central location is a little further away from the point of instrumentation

I think that one hash table lookup in a small, in-cache table for some messages is probably a bit cheaper than system-timestamp generation all the time.  This is in no small part because getting the system time likely involves a syscall, and (even an efficient syscall that elides mode switches in some cases) is far from a free copy.

That said, it's probably pretty close and there's something to be said for being cleaner.

I think that one hash table lookup in a small, in-cache table for some messages is probably a bit cheaper than system-timestamp generation all the time. This is in no small part because getting the system time likely involves a syscall, and (even an efficient syscall that elides mode switches in some cases) is far from a free copy.

I was assuming that getting time would be faster than the hash lookup. Though I'd probably trust your intuition on that over mine.

However, I don't think we can trivially replace the existing tick which uses Instant with a SystemTime based timestamp since SystemTime does not have the property of being monotonic, and we require monotonic ticks for event ordering in the ingest protocol.

Ah, that's something I definitely overlooked. I didn't really take note of the fact that this was using SystemTime and was assuming we could compute both tick and timestamp on the other end with a single value from the traced thread.

That said, it's probably pretty close on perf and there's something to be said for being cleaner.

I don't have a super strong opinion one way or the other, especially if the unconditional grabbing of time isn't faster. I'll leave the choice up to you.

Ended up doing the unconditional time-grab for simplicity's sake.


ARROW-2395: [Python] Fix flake8 warnings outside of pyarrow/ directory. Check in CI

+1, will merge on passing build

Codecov Report

Merging #2137 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2137      +/-   ##
==========================================
- Coverage   86.42%   86.41%   -0.01%
==========================================
  Files         242      242
  Lines       41536    41536
==========================================
- Hits        35896    35894       -2
- Misses       5640     5642       +2

Impacted Files
Coverage Δ

cpp/src/plasma/store.cc
88.27% <0%> (-0.45%)
: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 d4755e4...d9e6e9f. Read the comment docs.


IndicatorColor property missing from IndicatorView

The IndicatorView should have two colour properties, indicatorColor and selectedIndicatorColor.
The selectedIndicatorColor property is there, but indicatorColor is missing.
You can see the property used here in the XF docs:
https://docs.microsoft.com/en-us/xamarin/xamarin-forms/user-interface/indicatorview#create-an-indicatorview

@Ryan-Palmer Thanks for the report.
This is likely an oversight when we mapped the IndicatorView control.
This can be easily fixed by adding a mapping for IndicatorColor in this file:
https://github.com/fsprojects/Fabulous/blob/fb4f251ce5e7cd3f9755099dcd6f12991bcce794/Fabulous.XamarinForms/src/Fabulous.XamarinForms/Xamarin.Forms.Core.json#L2775-L2804

Awesome, thanks Tim! Really enjoyed your Xamarin vids with Don the other day. Just been giving Fabulous a go for the first time this week, I hope to get involved and start contributing soon :)


Winget Install broken

Describe the bug
The winget installation on a new system won't work for me, will always error out and if tried out won't work
To Reproduce
Steps to reproduce the behavior:

Go to "Install"
Click on "install anything"
See error

Expected behavior
should install winget correctly
Screenshots

My Suggestion
I would install choco using:
Set-ExecutionPolicy Bypass -Scope Process -Force; [System.Net.ServicePointManager]::SecurityProtocol = [System.Net.ServicePointManager]::SecurityProtocol -bor 3072; iex ((New-Object System.Net.WebClient).DownloadString('https://community.chocolatey.org/install.ps1'))

then using choco i would do a choco install winget, after successfull installation I would uninstall choco and cleanup the stuff it left behind

The chocolatey method is already included in install-winutilchoco.ps1 and should run automatically of the direct approach fails.
The chocolatey way in the past also produced some errors for me.


Add junit reports

Which problem is this PR solving?

Add jUnit generation report to the E2E tests

Short description of the changes

Add undeploy-cert-manager make rule to undeploy cert manager
Remove not existing undeploy-es-operator make dependency from clean rule
Create run-e2e-tests.sh script to handle the logic to run the E2E easier
Add integration with junitcli to fix the jUnit report output provided by KUTTL

@rubenvp8510 @frzifus please, could you take a look to this PR? Thanks!


Metadata integration tests

This PR adds integration tests for reading the table metadata from files. Some of the tests are designed to fail. With the current design of the serialization/deserialization the error doesn't specify which field is missing. So I couldn't do a precise check for certain tests.

@liurenjie1024, @Xuanwo , @Fokko it would be great if you could take a look.


Cleanup/remove legacy soap connection and object model

Supersedes https://github.com/solidify/jira-azuredevops-migrator/pull/631

Integration tests:  https://dev.azure.com/solidify/Internal/_build/results?buildId=9894&view=results


switching the output to the second hdmi port?

Hello,
I am using two displays, each connected to an HDMI port from my RPI CM4. But the program is only displayed on HDMI port 0. Is it possible to show the output on the second HDMI port? What do I have to change in the code for this or is it possible at all?

Maybe that's possible with the --device option, .e.g. -D /dev/dri/..., using the DRM device that corresponds to your second HDMI display.

What's the output of ls /sys/class/drm?

Thanks for your help, I found the solution :)

Great, let me close this 👍🏼.


BTRX-216 and BTRX-222 - Add Project Title on DAR related pages

[ ] Submitter: Include the JIRA issue number in the PR description
[ ] Submitter: Make sure Swagger is updated if API changes
[ ] Submitter: Check documentation and code comments. Add explanatory PR comments if helpful.
[ ] Submitter: JIRA ticket checks:

Acceptance criteria exists and is met
Note any changes to implementation from the description
Add notes on what you've tested

[ ] Submitter: Update RC_XXX release ticket with any config or environment changes necessary
[ ] Submitter: Update FISMA documentation if changes to:

Authentication
Authorization
Encryption
Audit trails

[ ] Submitter: If you're adding new libraries, sign us up to security updates for them
[ ] Tell the tech lead (TL) that the PR exists if they wants to look at it
[ ] Anoint a lead reviewer (LR). Assign PR to LR

Review cycle:

LR reviews
Rest of team may comment on PR at will
LR assigns to submitter for feedback fixes
Submitter rebases to develop again if necessary
Submitter makes further commits. DO NOT SQUASH
Submitter updates documentation as needed
Submitter reassigns to LR for further feedback

[ ] TL sign off
[ ] LR sign off
[ ] Product Owner sign off
[ ] Assign to submitter to finalize
[ ] Submitter: Verify all tests go green, including CI tests
[ ] Submitter: Squash commits and merge to develop
[ ] Submitter: Delete branch after merge
[ ] Submitter: Test this change works on dev environment after deployment. YOU own getting it fixed if dev isn't working for ANY reason!
[ ] Submitter: Verify swagger UI on dev environment still works after deployment
[ ] Submitter: Inform other teams of any API changes via hipchat and/or email
[ ] Submitter: Mark JIRA issue as resolved once this checklist is completed

Coverage increased (+0.006%) to 61.53% when pulling 5d53e89884ecf6d557908904277122953c8aae65 on BTRX-222-ProjectTitle into f40d02e2dad20e456a97bedda2d833c42f5dc06f on develop.


DRILL-4560 - ZKClusterCoordinator fix

Fix for DRILL-4560 - ZKClusterCoordinator does not call DrillbitStatusListener.drillbitRegistered for new bits. While this functionality is not needed in Drill itself, it is needed for tools that monitor Drill (such as Drill on YARN.) Adding this code to Drill avoids the need to clone the file for other tools.
Also cleaned up logging and comments a bit.

Please fix the commit message. Maybe "DRILL-4560: When new bits register, invoke listeners  in ZKClusterCoordinator"?

Changed PR title.

Not the PR title, the commit message :)

Sorry, the commit message quotes the JIRA entry title. I'll change the message, but quote the JIRA title in the body.

+1, pending minor comment
Can you squash the changes as well?


installer.sh fails to install

Environment

OS / version:  Every one,  apart from Fedora
Processor architecture: X
TPM Manufacturer: X
Keylime version: latest

Description
The installer.sh is outdated and failed to install Keylime on most of the systems I was able to test.
I am already working on the fix in my fork and will create pull request as soon as it'll be discussed in this issue.
I am creating this issue, because according to documentation it's required before contributing
Steps to reproduce problem

Run installer.sh on any OS apart from Fedora

Which distributions did you try exactly?
Can you check if this PR https://github.com/keylime/keylime/pull/1665 fixes it for you?

Thank you, this is actually great. I fixed the installer for RHEL 9, working on 8.
I tested Ubuntu, Debian, Fedora and Rhel8/9. Fedora was the only one that installed.
I'll take the changes you made and will implement mine.
Will it be okay to create pull request in the coming days?

Definitely! Contributions to improve the installer are very welcome.

This should fix RHEL8, didn't test RHEL7 yet, but I think there was still problem somewhere.
Worked on this for little while so also took this as opportunity to organize the code.
Could not test distributions anolis and alinux as I didn't find installer for them.


[Feature Request] Auto Create Default config.yml?

Any plans on creating a default config.yml file if the file isn't found at start? Would make this a bit more intuitive (and reduces the work I need for my Helm chart).

Sadly we cannot create a "default" config file as rewriting rules and API tokens are must-haves for the config file.


Add placeholder images for Settings and Permission tabs

I also needed to include jQuery in the header because bootstrap.js requires jQuery to be loaded previously.  I'm not sure how Bootstrap is supposed to work with Morpheus at all??? Morpheus loads jQuery in the footer, while the tools load javascript inline or in the  (like this Wicket app).  Do any other tools use the Bootstrap javascript?  I'm getting the feeling the entire framework is mutually exclusive to the Morpheus setup.

Can we just get Morpheus to load it in the header? Seems like a pretty valid use case.
I don't know if others use it, I just thought it would be cool to use the styles, we can do whatever we want so open to suggestions for sure.

Hm yeah.. it's tricky because I know it'd be great to have all of the scripts at the end of the page (page load time performance etc).  If this is the standard practice for Morpheus and Sakai going forward, perhaps it's the tool's responsibility to ensure their Javascript is loaded as a part of that block i.e. below jQuery and other plugins at the bottom of the page.  Do you know if there's a Wicket / Sakai handle to drop things at the bottom of the <body>?

We definitely can get wicket to load them at the bottom. See positioning of contributions on this page: http://wicketinaction.com/2012/07/wicket-6-resource-management/ If you want, send me the URLs we want to load at the bottom and I'll whack it in.


remove excluded files from flake8 docstring tests

https://github.com/hpe-container-platform-community/hpecp-python-library/blob/master/.travis.yml#L39

Done


Azure App Service tasks - reduce time in L0 tests

Kinda stumbled across this PR on accident and had some thoughts:

Thank you for looking into this. Reducing the time for these tests would be a big win, I've noticed they eat up a significant amount of time any time we have to queue a manual build of all tasks.

We should special case test changes in filter tasks so that it doesn't fail if versions are bumped. @stephenmichaelf for visibility.

For now if you want you can get around this restriction by queueing a manual build which skips this part of the filter step. That way you still get confirmation that the CI is passing.

@damccorm , thanks for looking into this.
Unfortunately, I dont have queue permission on PR CI.
How do I bypass the build failures & merge the PR? Will the CI break if I merge it?

Right sorry, forgot about permissions restrictions. I'll queue a manual build. Assuming that passes mergint the PR won't break CI - the common check is only for PRs.
Queued a Linux build, a Mac build, and a Windows build


Moving sharp ib v2

Add the capacity to simulate moving IB particles using an implicit Euler shame. Add modification to the linear and non-linear solver to make the process more robust.

@Luckabarbeau You need to resolve the comments above when they have been resolved. Otherwise it is difficult to keep track of what has or has not been done.


crash and can't open same games like Brwal Stars , CLASH OF CLANS

[Enter feedback here]

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

ID: 31f9a801-c5b5-94e7-41b9-1b2b4a1d27de
Version Independent ID: 31f9a801-c5b5-94e7-41b9-1b2b4a1d27de
Content: Windows Subsystem for Android™
Content Source: hub/android/wsa/index.md
Product: dev-environment
Technology: windows-android
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

Thanks for filing this issue @IG-BADAL. This sounds more like a support issue... whereas this repo is specifically for updates needed to the docs. I'll investigate to see what I can find out about this though and report back.

I'm sorry, we're not able to assist in this instance. I would reach out to the publishers of these games directly to help troubleshoot your problem.


Add Xcode 8 support: rename "OS X" to "macOS"

Most (but not all) of these changes are cosmetic. Xcode 8 changes the platform string to "macOS" (when used in a destination specification) and also uses macOS as PLATFORM_DISPLAY_NAME.

Tested with both Xcode 7.3.1 and 8.
Xcode 7.3.1:
+----------------------+-----------------------+
|            Summary for gym 1.6.4             |
+----------------------+-----------------------+
| project              | ./test.xcodeproj      |
| scheme               | test                  |
| destination          | generic/platform=OS X |
| output_name          | test                  |
| clean                | false                 |
| output_directory     | .                     |
| silent               | false                 |
| use_legacy_build_api | false                 |
| buildlog_path        | ~/Library/Logs/gym    |
+----------------------+-----------------------+

Xcode 8:
+----------------------+------------------------+
|             Summary for gym 1.6.4             |
+----------------------+------------------------+
| project              | ./test.xcodeproj       |
| scheme               | test                   |
| destination          | generic/platform=macOS |
| output_name          | test                   |
| clean                | false                  |
| output_directory     | .                      |
| silent               | false                  |
| use_legacy_build_api | false                  |
| buildlog_path        | ~/Library/Logs/gym     |
+----------------------+------------------------+

Perfect, thanks for updating the PR and posting those 2 tables, looks good to me 👍

Alright, I'm done with updating the PR. Feel free to merge.

Rebased on current HEAD

Perfect, thanks for your contribution @IngmarStein 👍


Handle error 500 when creating a new repository

When creating a new TeamForCapella repository, a 500 internal server error can occur.
The repository does still get created on the server side. We should therefore check if the repository is created and return success if it is the case.

Done with https://github.com/DSD-DBS/capella-collab-manager/pull/286


Declaration requests for patients with age<14

In case patient age<14 confidant_person should be mandatory in declaration request
If person age < 14 check existence of confidant_person
 - in case error return 422 - msg "Confidant person is mandatory for children"

@svetavedmed patient age<14 still can make declaration request without confidant person in case this patient has DS and tax_id.
For combined method (when person and declaration are in one request) tested and done

on POST http://ehealth.edenlabllc.com/api/cabinet/declaration_requests
check persons age.
If person age < 14 check existence of confidant_person

in case error return 422 - msg "Confidant person is mandatory for children"


Meetme Plugin Error -  No metrics to send to NewRelic this interval

We have installed the Meetme plugin and it is not started. Received error below. Appreciate any suggestion to resolve the issue.
Thank you,
JP
INFO       2018-05-23 13:44:13,255 33     MainProcess     MainThread newrelic_plugin_agent.agent                   init                  L55    : Agent v1.3.0 initialized, Linux-4.4.0-43-Microsoft-x86_64-with-glibc2.7 (Ubuntu 16.04 xenial) CPython v2.7.12
INFO       2018-05-23 13:44:13,256 33     MainProcess     MainThread helper.controller                             run                       L251   : newrelic-plugin-agent v2.4.2 started
WARNING    2018-05-23 13:44:13,256 33     MainProcess     MainThread newrelic_plugin_agent.agent                   send_components           L217   : No metrics to send to NewRelic this interval
INFO       2018-05-23 13:44:13,256 33     MainProcess     MainThread newrelic_plugin_agent.agent                   process                   L133   : Stats processed in 0.00 seconds, next wake in 59 seconds
WARNING    2018-05-23 13:45:13,257 33     MainProcess     MainThread newrelic_plugin_agent.agent                   send_components           L217   : No metrics to send to NewRelic this interval
INFO       2018-05-23 13:45:13,260 33     MainProcess     MainThread newrelic_plugin_agent.agent                   process                   L133   : Stats processed in 0.00 seconds, next wake in 59 seconds

@Jackpham6407 try setting the cfg log-level to DEBUG for requests and the agent. I had a similar issue where my license key had a typo and I didn't know until I turned on debug.

I have a similar issue (running Ubuntu 18.04). Rising the log levels to debug doesn't provide any additional information. My conf is pretty simple. The conf URLs (http:/<IP_ADDRESS>:80/nginx_status , http:/<IP_ADDRESS>:80/status-php56 and http:/<IP_ADDRESS>:80/status-php72 all respond)
nginx:
    name: my.hostname.here
    scheme: http
    host: <IP_ADDRESS>
    port: 80
    path: /nginx_status

php_fpm:
  - name: fpm56-pool
    scheme: http
    host: <IP_ADDRESS>
    port: 80
    path: /status-php56
    query: json
  - name: fpm72-pool
    scheme: http
    host: <IP_ADDRESS>
    port: 80
    path: /status-php72
    query: json


Fix < 0  comparison for an unsigned long long

line 43 should also be changed.
Otherwise if countChars is nil, nextEventID will be max ulll

Done.


PlistHttpMessageConverter

Please review my implementation of a PlistHttpMessageConverter. Do you see a chance to publish your work to maven?

Thanks again, finally merged to master.


Can not paginate RSS feed

Issue Summary
I need to get access to all my posts via RSS. I would like to paginate my RSS feed so that I can import all my posts elsewhere. However, I have tried mydomain.com/rss/2 to get the second page and it doesn't work. This is what I have found for how to paginate on ghost forums, so I'm filing a bug report.
Steps to Reproduce

Navigate to Ghost URL + /rss/2
Redirects to  404 page

Ghost Version
4.x
Node.js Version
unknown
How did you install Ghost?
Hosted
Database type
Other
Browser & OS version
No response
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Hi there @djbusstop, the RSS pagination behaviour was deliberately removed a some major versions ago. Instead Ghost has an extensive API & related tooling that you can use to get all of your content. If you need some help using it, reach out via the forums and we'll give you a hand.


duck-type values responding to #to_hash or #to_ary

I work with JSON data wrapped in classes which behave like Hash and Array, which respond to #to_hash / #to_ary to support implicit conversion (as described in https://docs.ruby-lang.org/en/master/doc/implicit_conversion_rdoc.html )
this PR lets jmespath be used with these implicitly convertible objects. it is in draft because I have updated no tests or documentation, and may be incomplete in other ways. if an active maintainer can tell me it might be merged, I will bring it to completion.

Thanks for opening up a draft PR. I think what you're trying to do makes sense. What is the experience without any changes? - i.e. for your JSON data wrapped classes, are you able to call to_a or to_h on them and pass those raw converted values into jmespath?

What is the experience without any changes? - i.e. for your JSON data wrapped classes, are you able to call to_a or to_h on them and pass those raw converted values into jmespath?

I can pass the JSON that my classes wrap to jmespath, yes, but then lose the functionality that is added by the wrapper I have implemented. my goal is to search the wrapped document and get wrapped children out as the search result(s).
the wrapping I work with is associating JSON data with JSON schemas (per the specification at https://json-schema.org/) describing those data. so if I search within a document described by a schema, the resulting children get returned associated with subschemas that describe them. the library implementing this is here, in case that is of any interest: https://github.com/notEthan/jsi
if this is merged and released I'll add a method something like #jmespath_search(expression) to the wrapper class, to help with searching documents.

I think I have the code in a good state for review, and have added a couple tests

Thanks. We'll review this shortly.

I've added CI back to this project and merged those changes in. I've also ported the compliance tests into the implicit conversion test cases - there were a few failures in functions which I've fixed.
There are a few other failures in the compliance tests w/ implicit conversion that I'm not positive how to address (they may be issues in my spec code as well) - could you take a look at those cases?

thanks for the additions to the tests, that was definitely needed. I added and fixed a couple more to_arys, and added a Util.as_json for recursive comparison (this made redundant the unwrap method you'd added to the tests, so I removed that). added some test cases to the compliance json to cover code changes.
also rebased against main.

glad to hear. thanks very much for your reviews and additions. I am quite happy to find active maintainers here, the project looked very quiet. also, aside, this is among the best-written libraries I've worked with or contributed to.
looking forward to integrating support when this is merged and gets released.

@notEthan - Thanks again for a great contribution. I've released this as version 1.5.0.

For people searching google:
Any developers using the AWS SDK for Ruby v2 that has a dependency on jmespath, Aws::EC2::Instance.block_device_mappings returns an array of InstanceBlockDeviceMapping that are now converted from Structs to Hashes implicitly.
To correct errors, update dot operator to bracket notation for value access.
Old way:
block_devices = Aws::EC2::Instance.new(instance_id, client: EC2_CLIENT_OBJECT).block_device_mappings
data_block_device = block_devices.find { |bd| bd.device_name == "/dev/xvdb" }
data_block_device.ebs.volume_id

New way:
block_devices = Aws::EC2::Instance.new(instance_id, client: EC2_CLIENT_OBJECT).block_device_mappings
data_block_device = block_devices.find { |bd| bd[:device_name] == "/dev/xvdb" }
data_block_device[:ebs][:volume_id]

That seems wrong. It should be a struct. We will investigate.

Thanks for reporting that - I agree with @mullermp  - it should be a struct there.  I have confirmed this does NOT impact V3 of the SDK (note: V2 of the SDK is no longer supported).


Conncurant Connections Is Now Limited To Just 2

FYI
There's been a change to the streaming API - https://forum.aws.chdev.org/t/streaming-api-concurrent-connections/4912
As your code connects to all 6 at once, it will cause problems :(

Hi Paul,
Thanks for letting me know.
I have updated my code to work with this limitation that they have imposed.
What I've done is created 3 API keys on Companies House website, and then use each key for only 2 streams, and that way I can connect to all 6 at the same time.
I created a little helper class to manage the key usage to make sure I don't connect more than twice with the same key. You can see it in https://github.com/mrbrianevans/companies-house-stream/blob/master/server/src/utils/KeyHolder.ts if you're interested.
I'm running the project locally and its working again, showing events from all 6 streams.
Kind regards,
Brian

Nice one Brian I'll have a nosey later.
Hopefully using multiple keys from the same IP doesn't fall foul of their fair use policy


Registering multiple transforms with the same values of attributes should fail

Current Behavior
Open the attached project: TransformWithSameAttributes.zip

It has the following transform setup (expand to see)
import org.gradle.api.artifacts.type.ArtifactTypeDefinition.ARTIFACT_TYPE_ATTRIBUTE

plugins {
    java
}

dependencies {
    implementation("com.google.guava:listenablefuture:1.0")
}

abstract class ExampleTransform : TransformAction<ExampleTransform.Parameters> {

    interface Parameters : TransformParameters {
        @get:Input
        var minSdkVersion: Int
    }

    @get:InputArtifact
    abstract val inputArtifact: Provider<FileSystemLocation>

    override
    fun transform(outputs: TransformOutputs) {
        println("Running transform with minSdk = ${parameters.minSdkVersion} for artifact ${inputArtifact.get().asFile.path}")
        outputs.file("transformed-minSdk-${parameters.minSdkVersion}-${inputArtifact.get().asFile.name}").createNewFile()
    }
}

val minSdkAttribute = Attribute.of("minSdkAttribute", String::class.java)

// NOTE: We will register 2 transforms and 2 tasks for the 2 minSdk values below.
for (minSdk in listOf(21, 24)) {
    dependencies {
        registerTransform(ExampleTransform::class) {
            parameters.minSdkVersion = minSdk
            // BUG: If we forget to set the minSdkAttribute below, Gradle should fail because 2
            // transforms are registered with the same set of attributes, but currently Gradle
            // doesn't fail.
//            from.attribute(minSdkAttribute, minSdk.toString())
//            to.attribute(minSdkAttribute, minSdk.toString())
            from.attribute(ARTIFACT_TYPE_ATTRIBUTE, "jar")
            to.attribute(ARTIFACT_TYPE_ATTRIBUTE, "transformed")
        }
    }
    tasks.register("printArtifacts$minSdk") {
        val artifactFiles = configurations.getByName("runtimeClasspath").incoming.artifactView {
            attributes {
                attribute(minSdkAttribute, minSdk.toString())
                attribute(ARTIFACT_TYPE_ATTRIBUTE, "transformed")
            }
        }.artifacts.artifactFiles
        inputs.files(artifactFiles)
        doLast {
            println("Transformed artifact: ${artifactFiles.files}")
        }
    }
}

The idea is that we register 2 transforms with the same set of attributes (by accident), but Gradle doesn't fail, therefore causing latent errors that are hard to debug.
For example: Running ./gradlew --stop -Dgradle.user.home=gradle-user-home && rm -rf gradle-user-home/caches/transforms-3 && ./gradlew printArtifacts21 printArtifacts24 -Dgradle.user.home=gradle-user-home will produce the following log:
> Task :printArtifacts21
Running transform with minSdk = 24 for artifact TransformWithSameAttributes/gradle-user-home/caches/modules-2/files-2.1/com.google.guava/listenablefuture/1.0/c949a840a6acbc5268d088e47b04177bf90b3cad/listenablefuture-1.0.jar
Transformed artifact: [TransformWithSameAttributes/gradle-user-home/caches/transforms-3/12cf933ebe09a55ab0718d0a63694f35/transformed/transformed-minSdk-24-listenablefuture-1.0.jar]

> Task :printArtifacts24
Transformed artifact: [TransformWithSameAttributes/gradle-user-home/caches/transforms-3/12cf933ebe09a55ab0718d0a63694f35/transformed/transformed-minSdk-24-listenablefuture-1.0.jar]

It shows that the transform for minSdk = 21 didn't run, and the task :printArtifacts21 is supposed to get the transformed artifact with minSdk = 21 but it gets the transformed artifact with minSdk = 24 instead.
Expected Behavior
It's easy for the users to forget to set the necessary attributes. One could say it's the responsibility of the users. But Gradle could easily help users avoid that mistake by failing the build when multiple transforms are registered with the same values of attributes.
Context (optional)
This is a simplified example of a real bug that we have in the Android Gradle plugin (https://issuetracker.google.com/293206305 - "NoClassDefFoundError with AGP 8.1.0, desugaring and minify"). When setting up L8DesugarLibTransform, we forgot to set up the minSdkVersion attribute even though it is an input to the transform.
Steps to Reproduce
(See above)
Gradle version
8.4
Build scan URL (optional)
No response
Your Environment (optional)
No response

Note that the new behavior will be a breaking change: It will fail the build for user projects if they use plugins / plugin versions where the transforms are not registered properly.
This belongs to the group of Gradle's validation problems, so I'm guessing there is already a process for introducing such a breaking change.

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team, but this area of Gradle is currently not in focus. It might take a while before it gets implemented.

I ran into the same problem, see https://github.com/gradle/gradle/issues/28960 for more details.


fix: Premature delivery stream close

Just pushed some more tests, it's not easy to test completeness, but we can test all scenarios and check if nothing unexpected was thrown.
And I did reproduce the issue and the checked the fix on a real device.


Bug: VK_ABNT_C1 and VK_ABNT_C2 are the wrong type

Which crate is this about?
windows, windows-sys
Crate version
0.44
Summary
All virtual keys (like VK_ESCAPE, VK_NUMPAD1, ect...) have VIRTUAL_KEY as their type.
Similarly, in the windows-sys crate they are u16.
But VK_ABNT_C1 and VK_ABNT_C2 are defined as u32 in both crates.
Ideally they should be the same types.
Toolchain version/configuration
No response
Reproducible example
No response
Crate manifest
No response
Expected behavior
No response
Actual behavior
No response
Additional comments
No response

Thanks for reporting. This looks like an issue with the Win32 metadata definitions for these constants. I'll transfer it to their repo for resolution.


Support 'const' keyword

Hi, great work with this amazing tool. I've tested some examples and some self code and i've seen that you support 'let' beside 'var' keyword. Have you any plans to support 'const' keyword soon? I've made some tests and is not very complicated, i'm not a C++ dev but i know that it support const variables and this is my suggestion.
You use babel as js parser and in code genration process of ast there is no check for VariableDeclaration kind property, it means that you can put whatever you want as kind and it will be generated.
This is the test:
compiler/native/src/macro.h
#define __NJS_CONST const NJS::VAR

and in
compiler/native/visitor/plugin/VariableDeclaration.js
if (_path.node.declarations) {
    if (_path.node.kind === 'const') {
      _path.node.kind = '__NJS_CONST';
    }
    ...

I tested this solution and it works very well. You could, of course, use eslint to check constant reasigments but, this is just a suggestion.
Again great work, i'm glad that somebody put your effors in somthing like this idea.

Hi @InDIOS ,
Thank you very much for your message and for your tip and test, i will implement it right now.

Added in 2f2759d
and published in v0.7.56
Thx !


Aggregate incremental and diff storage

The aggregate must be updated with diff process

a relatively simple solution to diff process could be the following:

the add or insert action of an aggregate never has a diff, the complete body of the data always be added to DB.
when adding an aggregate, a Snapshot is created with the complete data structure
the Actions that need to get the diff like Update, must call the Snapshot of the Aggregate, convert into a dict and use DeepDiff


Add Unit Test cases for Authentication Microservice.

The main purpose of this task is to add unit test cases to Authentication Microservice.

Completed jUnit Testing for this microservice.


Improve findability of zkvm update instructions

I just went to run the zkVM for the first time in a while and found it challenging to find the update instructions.
My Intuitive Click Path
I went to Quick Start. This describes first-time installation instructions but doesn't offer any guidance on what to do if you're looking to update. There's no link to the installation page, and the installation page isn't super obvious on the sidebar.
Suggested changes

[ ] Add a link from the quick start page to the installation page.
[x] [Optional] Review sidebar organization

hmm @SchmErik was there a PR you wanted me to review for this?

#1511 wasn't directly related, but I'm going to go ahead and close this as I think it's no longer an issue. This was a pain point based on the previous sidebar organization, which doesn't seem like a pain point anymore.


read/write stream incompatibility

As of<EMAIL_ADDRESS>this fails:
var db = level(__dirname + '/db');
db.createReadStream()
  .pipe(db.createWriteStream())
  .on('close', console.log.bind(console, 'close'));

Nothing is logged. It doesn't depend on whether the db is empty or not. The snippet works with<EMAIL_ADDRESS>

@juliangruber Can we close this or do you want to keep it open?

since there's no write stream any more, let's close :)


Java code cleanup [phab:D1434]

Created by jrtom at 2017-03-18 00:01:27:
Remove the isLocalAnalysis parameter, as it was only ever set to false.
Add default cases for non-exhaustive switch statements.
Add a parameter to the relevant method in JavacOptionsUtils to enable grabbing classpath information from the JavaDetails.  (This is a prequel to the changes needed to address #1805.

salguarnieri commented inline (context lost during migration) at 2017-03-20 16:40:42:
Throw another illegal state exception then? That way we are covered in case line 456 changes.

salguarnieri commented inline (context lost during migration) at 2017-03-20 16:40:42:
Don't use bug uids in open source, create a phab ticket, todo note that ticket here.

salguarnieri commented inline (context lost during migration) at 2017-03-20 16:40:42:
Same as above for todo style

jrtom wrote at 2017-03-22 18:17:28:

removed references to internal bug tracker, fixed indentation, added exception

jrtom wrote at 2017-03-22 18:21:35:
PTAL

salguarnieri commented inline (context lost during migration) at 2017-03-22 18:28:06:
nit: indentation?

salguarnieri accepted this revision at 2017-03-22 18:28:06

jrtom wrote at 2017-03-22 18:34:10:

fix indentation

jrtom closed via commit 0ce0cc5180133fcc186fe5c14903609ea1100258 at 2017-03-22 18:35:53

salguarnieri wrote at 2017-03-22 18:36:09:
Thanks!


Fixed memory leak of implementation data in FileLock

I noticed this leak when I was testing my own block

Nice! I changed the patch to delete the pointer in the C++ files. FileLock wasnt meant to have references copied around, so that should fix it cleanly in the implementation. Mostly I was worried to change the ABI with the shared pointer in the header file.


ParameterMarkerStrategy markers are 1 based

The position argument in ParameterMarkerStrategy#createMarker is 1-based, but StandardTemporaryTableExporter#getSqlTruncateCommand calls it passing a 0.
Hibernate Reactive is affected by this when running queries on CockroachDB. Hibernate ORM generates queries using ? as marker and it's not affected by it unless somebody implements the service.

https://hibernate.atlassian.net/browse/HHH-17428

Shoudl I merge it? It seems that something fails with the CI build. It doesn't seem related though

I think the failures are not related to your PR.

Merged https://github.com/hibernate/hibernate-orm/pull/7548, thanks


Mongo Language Server output channel keeps popping up

The Mongo Language Server output channel is opened quite frequently when using the extension. I simply connected to a cosmos mongodb and then this pops open

I don't know if it is coming from the extension or the languageserver module, it seems like the latter. For end users, none of this is very actionable, so not forcing it open would be the best experience.

That's related to #28 and fixed in #29

Thanks for filing the issue though :) Better safe than sorry

my point is that the output channel keeps coming up.  is that part fixed, or the error itself (which is goodness) ?

I'm not quite sure I understand what you're asking. The error itself is fixed, so the output channel shouldn't pop up (unless some other error is occurring). I just merged the fix, so you can try it from master close this issue as you see fit

ok, i see the fix in for the error. my point was the fact that the output pane keeps popping to the foreground when i dont think it should.  i'll reopen with a better repro.


Incorrect link model for collection metadata

In package-30.rnc, the content model for the metadata is this:
opf.collection.metadata.content = opf.collection.dc.elems & opf.collection.meta* & opf.link*
But it should be referencing opf.collection.link.

Just noticed this issue. I believe the schema is correct: the collection section refines the link element when it is a child of collection only ; in other words, when a link element is used in collection metadata, it is specced by the same rules as links in the publication's metadata, and has the same content model.
Please reopen if you disagree.


feat(clerk-js): Allow skipping invitation screen after CreateOrganiza…

…tion
Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

Description

[ ] npm test runs as expected.
[ ] npm run build runs as expected.

This PR introduces the skipInvitationScreen prop on <CreateOrganization />. The new prop will enable developers to choose whether the want to allow invites to be send right after the creation of an organization.
Tests are added as well :)

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Fix/testingcenter

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Fork choice updates for v0.9.2

What was wrong?
Part of #1321
How was it fixed?
Add the fork choice updates.
NOTE: this is based off the refactor in #1297. The only new change here is the last commit 1b0db5b and I can rebase after #1297 is merged.
Cute Animal Picture

looks like i need to go fix the LMD GHOST tests here

the LMD GHOST tests have been fixed but it seems that the length of test_store_get_latest_attestation has increased significantly as we mock out all of the blocks used by attestations in that test

Close in favor of #1360.


Doesn't work when nested inside another scrollview

I'm trying to use invertible scroll view inside another scrollview(https://github.com/brentvatne/react-native-scrollable-tab-view) and nothing gets inverted. Everything works fine if I don't nest inside another scrollview. Do you have any idea what could be the potential issue here?

I'm not sure why this could be (nested ScrollViews in RN are buggy anyway). Invertible Scroll View is simply using view transforms.

Thanks for the quick reply! Will have to skip nested scrollviews, till issue is resolved in RN core.


Please update the dependency  "jackson-mapper-asl" associated with Rest-assured 4.4.0.

Hi,
https://mvnrepository.com/artifact/io.rest-assured/rest-assured/4.4.0.  showing the 1 vulnerability for "jackson-mapper-asl" dependency.
Please update it.

This is still an issue in 5.1.1, same vulnerability in the same dependency.


[improve][pip] PIP-360 Add admin API to display Schema metadata

Motivation
Adding support for showing metadata of schema in readable format.
Modifications
Adding admin API and cli to read schema metadata.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

+1

this PIP is approved with 2 binding and 1 non-binding


Add Fluent support for CALL subquery

CALL subquery (https://neo4j.com/docs/cypher-manual/current/clauses/call-subquery/) does not appear to be supported by ICypherFluentQuery. Furthermore, I can't find anyway to extend CypherFluentQuery to add support myself.
I can hack it by doing the following:
public static ICypherFluentQuery AggregationSubquery(this ICypherFluentQuery query)
{
    return query
        .Call("{") // Abuse CALL proc
        .Unwind(Enumerable.Range(1, 5), "i")
        .Create("(c:Clone)")
        .Return<int>("count(c) AS numberOfClones }"); // Abuse RETURN clause
}

Is there an easier was to get this working?
Ideally, the API would be something that would take a lambda to build the subquery:
public static ICypherFluentQuery AggregationSubquery(this ICypherFluentQuery query)
{
    return query
        .Call(query => query
          .Unwind(Enumerable.Range(1, 5), "i")
          .Create("(c:Clone)")
          . Return<int>("count(c) AS numberOfClones"));
}

Ah, ok, so what you want is the Fluent querying to be available inside a .Call.
The lambda approach is quite nice, I can have a think about it.
I'll change the title, as subquery calls are supported, it's just via plain text as opposed to the fluent interface.

Good shout, yes it's just the fluent interface I was referring to. Awesome, thanks!

This should have been resolved with PR #411
Let me know!
(Sorry it took the time it did!)

I may have jumped the gun slightly, as the build server is complaining about tests which all pass on my machines... So I'm investigating now!

Finally in 4.1.14


Correction for raw affiliation Cerema, Research Team ENDSUM, 23 avenue de l'Amiral Chauvin, 49136 Les Ponts de Cé, France

Correction needed for raw affiliation Cerema, Research Team ENDSUM, 23 avenue de l'Amiral Chauvin, 49136 Les Ponts de Cé, France
raw_affiliation_name: Cerema, Research Team ENDSUM, 23 avenue de l'Amiral Chauvin, 49136 Les Ponts de Cé, France
new_rors: 05k0nvr25
previous_rors: 05k0nvr25;02nwvxz07
works_examples: W4283314441
contact: 8aba637816fc738cde51f7283d98c55f:a344557ab13b2dbb911faa99886a96 @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


bau: Simplify and rename CardAuthoriseBaseService

It was clear from https://github.com/alphagov/pay-connector/pull/2673 that
there is some refactoring that could be done to remove code duplication between
WalletAuthoriseService and CardAuthoriseService. As there are still slight
authorisation implementation differences between the two it is envisaged there
will be a common AuthoriseService. However looking at the current code there is
a plethora of authorise services:
CardAuthoriseService
CardAuthoriseBaseService
Card3dsResponseAuthService
WalletAuthoriseService
The term "BaseService" is quite annoying in the sense that it doesn't really mean anything,
so is a candidate for simplification and renaming. This would be a good first
step before unifying the WalletAuthoriseService and CardAuthoriseService.

I was trying to avoid AuthorisationService as that's I planned to name and up-and-coming class that contains common methods between WalletAuthoriseService and CardAuthoriseService. How about ExecuteAuthorise?

Bah! Might it be the case that AuthorisationService includes the functionality of AuthorisationExecutor in the end? The division of responsibilities seems a bit arbitrary at the moment. That said, I’d prefer us to have more small classes than fewer large ones. Maybe just ChargeAuthoriser or something?

You know, let me try and put everything in an AuthorisationService (in another PR). I think it can be done.


Configuration of trusted_proxies for deployment behind a reverse-proxy

Is there a way to pass customize the configuration.yaml that is deployed in the chart to include values for trusted_proxies? This would be very useful for setups that are using a reverse-proxy.
If possible, adding an additional configuration file that is auto filled with appropriate values if a reverse proxy is in use would be an excellent addition. This file could be included from the base configuration.yaml file.

Hi,
In theory, I could add a default configuration to the Helm chart, but it would only apply once, before the initial start of Home Assistant. Editing the configuration of an already running Home Assistant via Helm chart upgrade is not a good idea. It could potentially cause problems with Home Assistant.
The second issue is related to the values within trusted_proxies. It should be the IP address of the reverse proxy, but obtaining this value during the Helm chart installation is not feasible. Installing a reverse proxy is beyond the scope of this Helm chart.

Hi, I've identified two issues:
Firstly, try mounting only one file:
additionalVolumeMounts:
- name: home-assistant-config
   mountPath: /config/configuration.yaml
   subPath: configuration.yaml

The existing /config mount is in place to share the same settings with the VSCode extension. However, I believe it's not a best practice to mount configuration.yaml directly into a running Home Assistant instance. When mounted from a ConfigMap, the configuration.yaml file becomes read-only, making it impossible to edit, even in VSCode.
Secondly, the trusted_proxies value should include the IP addresses of your Nginx controller. In a Kubernetes environment, this can be challenging to determine; however, when using k3s, I typically set the entire pod CIDR range to <IP_ADDRESS>/16 (as per the k3s documentation on networking). For added security, I also implement a network policy, although this is optional.

Thanks for the helm chart. I ran into this as well.
My recommendation is for someone setting up the first time is to make use of port forwarding for the first time setup.
If you just need port forwarding on localhost:
kubectl -n homeassistant port-forward pod/helm-homeassistant-home-assistant-0 8123 8123

But in most cases,  your k3s/k8s cluster is remote, so you need to listen all addresses:
kubectl -n homeassistant port-forward pod/helm-homeassistant-home-assistant-0 --address <IP_ADDRESS> 8123 8123

`

@pajikos  I also run into this. I am not sure if it would be a good idea to manage the /config/configuration.yaml mount and config map directly into the chart?  that would allow you to at least have a "deploy" and works. On the downside, making backups is harder as just snapshotting the pvc is not enough. I'm a bit torn into this one.

@Starkie  Yeah, that is of course an option, but the idea of making the chart provide a working configuration I think is the best solution.

@pajikos  IMO the golden solution for this have an init container that updates the config when a certain .Values.X is enabled probably with yq -i '.http.use_x_forwarded_for = true' configuration.yaml
I have a couple of complicated months ahead, I can't commit to fix this, but I can help review if you jump into it.

@jlpedrosa ok, I will suggest such a solution, but I think,  http.trusted_proxies is required as well and that is the problem, each cluster has its own IP ranges. Are you using http.use_x_forwarded_for only?

maybe something like this:

in values file we have

configmap:
    configuration.yaml: | -
       stuff here

as part of the chart
if config = true create the default configuration.yaml and mount it.
This way if you want to use vanilla version you leave out and if you want custom config you can override.
happy to do the pr

maybe something like this:

in values file we have

configmap:
    configuration.yaml: | -
       stuff here

as part of the chart if config = true create the default configuration.yaml and mount it.
This way if you want to use vanilla version you leave out and if you want custom config you can override.
happy to do the pr

Be careful here, mounting the configuration.yaml is not a problem, but then it is not possible to edit this file. It would be necessary to mount this file into an init container and then copy it from there into the target configuration.yaml. I have already implemented a similar pattern on another project, I will try to outline it.

I've got same issue today and eventually I managed to solve that by changing service type to LoadBalancer:
service:
  # Service type (ClusterIP, NodePort, LoadBalancer, or ExternalName)
  type: LoadBalancer

With this small change I've got no more connection errors to this service and probably no need for config change in files.

Loadbancer may work as long as you don't use an ingress (depends on the loadbalancer), but with in an ingress it will always be L7 Proxy, and it will have the issue.
@RLutsch I agree with @pajikos, We need to be very careful so we don't override other settings managed by the user. That is why I was sugesting to do it item by item in yq.
yq -i '.http.use_x_forwarded_for = true' configuration.yaml
yq -i '.http.trusted_proxies = [  "<IP_ADDRESS>/16" ] configuration.yaml
this ensures that only those two keys are modified.

https://github.com/pajikos/home-assistant-helm-chart/pull/54

Yes, I got to the point where most of request were working but still some post and background ajax queries returned error. It was configured with traefik, but still I need to modify several things to make it working as well as on port forwarding (no problem here, but no ssl).

Hey, maybe this is incredibly dumb so feel free to ignore.
But would it not make sense to (if at all possible) add an include in configuration.yaml to some file... and have the helm-chart write said file, either making it just [] because it is empty or write whatever is found under a particular key in values.yaml like @RLutsch suggested above?
That way there would be a way to configure anything in home assistant by putting the relevant yaml in your own values for the chart... well... almost anything. You would still have problems if you wanted to remove something I guess.
I don't know, I just thought it would be "neat" to use an include file. Mostly because you would keep the changes to configuration.yaml as minimal as possible.

Hi, I have developed a general approach for configuring Home Assistant, including the use of a reverse proxy. See #55 for details. @jlpedrosa or others, it's quite a significant change, so it would be fine to test it before merging. Can you help me with this? I'm particularly concerned about using securityContext at any level. Many thanks.

I can if you can wait a bit, I am out of town until the weekend.
tis 14 maj 2024 kl. 10:01 skrev Pavel Sklenář @.***>:

Hi, I have developed a general approach for configuring Home Assistant,
including the use of a reverse proxy. See #55
https://github.com/pajikos/home-assistant-helm-chart/pull/55 for
details. @jlpedrosa https://github.com/jlpedrosa or others, it's quite
a significant change, so it would be fine to test it before merging. Can
you help me with this? I'm particularly concerned about using
securityContext at any level. Many thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/pajikos/home-assistant-helm-chart/issues/30#issuecomment-2109531917,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAASHYRTSUXQNQDOC7RBMQTZCHAHLAVCNFSM6AAAAABDCEZDPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBZGUZTCOJRG4
.
You are receiving this because you commented.Message ID:
@.***>

I was having an issue where my trusted proxies weren't getting added to the configuration.yaml, and then realised that of course it only happens if the value ingress.enable = true, which in my case it is not because I'm using Gateway API and an HTTPRoute instead of an "old fashioned" ingress. I'm not sure what support you'd like to consider for the Kubernetes Gateway API given that's the way things are going, and I had the fairly easy workaround of removing the line {{- if .Values.ingress.enabled }}, but I thought it was worth mentioning in case others want to go down the gateway route.

Hey team, did not have time to test it, IMO it felt a bit dangerous.....

I was having an issue where my trusted proxies weren't getting added to the configuration.yaml, and then realised that of course it only happens if the value ingress.enable = true, which in my case it is not because I'm using Gateway API and an HTTPRoute instead of an "old fashioned" ingress. I'm not sure what support you'd like to consider for the Kubernetes Gateway API given that's the way things are going, and I had the fairly easy workaround of removing the line {{- if .Values.ingress.enabled }}, but I thought it was worth mentioning in case others want to go down the gateway route.

Ok, good point, I am thinking about adding configuration property externalIngress = "true|false" to be able to include trusted_proxies in a case of using custom reverse proxy


reduce 14 moniker ranges.

See related #21600

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

ID: 04eafd1b-403a-4aa3-48bc-78fed5b4d05e
Version Independent ID: f35451f4-7c74-3ada-e969-b769428b94f4
Content: ASP.NET Core Web Host
Content Source: aspnetcore/fundamentals/host/web-host.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Closing since Generic host is now used.


Remove pipes from table text values

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

@LowlyDBA : Thanks for your contribution! The author, @rothja, has been notified to review your proposed change.

@LowlyDBA : Thanks for your contribution! The author, @rothja, has been notified to review your proposed change.

@LowlyDBA Thank you for this change. That formatting was really bad. I think it happened when we converted from MSDN to the new docs.microsoft.com. But your change is clear. I fixed the headers as well and pushed one more commit. I hope that is OK. Appreciate your help in improving the docs!
#sign-off