Incompatibility in gitbucket migration

Description
Thank you for development.  Thank you for implementing migration from gitbucket (#16767).  It helps me a lot.  However, it doesn't work if a gitbucket repository has many issues or PRs.  I inspect it and find the source of problem, so I'm posting this bug report.
While I'm checking the log of migration, I notice there are several logs like Request get issues 49/1, but in fact get 39.  This is the source of gitbucket migration problem caused by following incompatibility

Gitea treat it, fewer issues than perPage issues, as a mark of end of information.
Gitbucket doesn't return full 49 issues if there are missing issues in the situation like #1 issue, #2 PR, #3 issue.

Easiest way to patch this is to change the migration mechanism.  For example, it is possible to change the mechanism to perform migration until an empty array of issues is returned from GetIssue() routine.
+++ b/services/migrations/migrate.go
@@ -331,6 +331,10 @@ func migrateRepository(ctx context.Context, doer *user_model.User, downloader ba

                for i := 1; ; i++ {
                        issues, isEnd, err := downloader.GetIssues(i, issueBatchSize)
+                       isEnd = false
+                       if len(issues) == 0 {
+                               break;
+                       }
                        if err != nil {
                                if !base.IsErrNotSupported(err) {
                                        return err

This works fine while the number of missing issues is small.  If there are many missing numbers more than 98 in issues like #1 issue, #2 PR, ..., #99 PR, #100 issue, this doesn't work well.  I have no idea how to improve the mechanism, so I'm just posting this bug report.
Gitea Version
1.21.10
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
Gitea in docker, gitea/gitea:1.21.10.
Database
None

But how did you know it's end if there is one middle page that contained issues that are all pull requests?

Yes.  That is the problem and I have no idea how to improve the mechanism, so I'm just posting this bug report.

Maybe, it's something what gitbucket and other github close must take care.  But, it's still good to know the reason of this kind of problems...  You know.  ;-)

Maybe we can rewrite the function `GithubDownloaderV3.GetIssues` to use this fields to confirm whether it's end.

Thank you for suggestion.  Unfortunately, gitbucket doesn't support those fields (Next Page, Last Page).
$ http_proxy= HTTP_PROXY= curl -L -H "Accept: application/vnd.github+json" -H "X-GitHub-Api-Version: 2022-11-28" "http://localhost:50856/api/v3/repos/CWB-LIB/Library/pulls?state=all&sort=created&direction=asc&page=6&per_page=49" -u USER:PASSWD -I
HTTP/1.1 200 OK
Date: Mon, 08 Apr 2024 02:48:31 GMT
Set-Cookie: JSESSIONID=node06o594avhud4rgctaefj2zi1z102.node0; Path=/; HttpOnly
Expires: Thu, 01 Jan 1970 00:00:00 GMT
Content-Type: application/json
Transfer-Encoding: chunked

github.com
$ curl -L -H "Accept: application/vnd.github+json" -H "X-GitHu
b-Api-Version: 2022-11-28" "http://api.github.com/repos/go-gitea/gitea/issues?state=all&sort=created&direction=asc&page=1&per_page=49" -I
HTTP/1.1 301 Moved Permanently
Content-Length: 0
Location: https://api.github.com/repos/go-gitea/gitea/issues?state=all&sort=created&direction=asc&page=1&per_page=49

HTTP/2 200
server: GitHub.com
date: Mon, 08 Apr 2024 03:03:52 GMT
content-type: application/json; charset=utf-8
cache-control: public, max-age=60, s-maxage=60
vary: Accept, Accept-Encoding, Accept, X-Requested-With
etag: W/"db7bdfe2b0d8646e5d376ddaef072ae2cd1d0160dfdf82de1f39758b3eb8f264"
x-github-media-type: github.v3; format=json
link: <https://api.github.com/repositories/72495579/issues?state=all&sort=created&direction=asc&page=2&per_page=49>; rel="next", <https://api.github.com/repositories/72495579/issues?state=all&sort=created&direction=asc&page=615&per_page=49>; rel="last"
x-github-api-version-selected: 2022-11-28
access-control-expose-headers: ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Resource, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, X-GitHub-SSO, X-GitHub-Request-Id, Deprecation, Sunset
access-control-allow-origin: *
strict-transport-security: max-age=31536000; includeSubdomains; preload
x-frame-options: deny
x-content-type-options: nosniff
x-xss-protection: 0
referrer-policy: origin-when-cross-origin, strict-origin-when-cross-origin
content-security-policy: default-src 'none'
x-ratelimit-limit: 60
x-ratelimit-remaining: 58
x-ratelimit-reset:<PHONE_NUMBER>
x-ratelimit-resource: core
x-ratelimit-used: 2
accept-ranges: bytes
x-github-request-id: D4D4:167ECD:1A8817B:1AA9EFD:66135E98

So maybe gitbucket needs to make some improvements to their API, otherwise, we cannot know whether the pagination is ending.

Just an update.  I've read gitbucket source code and noticed it doesn't support per_page at all.  I check again the actual message from gitea log.  It is 2024/04/09 07:40:41 ...migrations/github.go:434:GetIssues() [T] Request get issues 49/1, but in fact get 25.  The internal per_page of gitbucket is just 25.  This is the real source of this problem.  Sorry for wrong description in previous posts.
So, it may be possible to use 25 as a hard coded per_page value for only gitbucket migration until gitbucket changes it.  Is such kind of patch acceptable for gitea?  If it's acceptable, I'll try to create a PR.

A patch is merged and the problem is fixed, so I'm closing this issue.  Thank you.


iOS 14 not launching

Since I’ve updated to iOS 14 beta I can’t launch the app anymore.
Splash screen shows for a second then only a black screen.
I can see it accessing my location
iPhone 11 Pro Max iOS 14.0

Hi I am using iOS 14.0 on an iPhone XR, I experience the same behavior. I assume it has smth to do with permissions, just a wild guess tho

It is not so much a "black" screen but more of an "empty" screen (depending on your iPhone-setting).
One needs to realize, of course, that iOS 14 is in "early beta status", hence not so much a "bug" in terms of the current version of the CWQ, which is designed for iOS 13!
Having said that, I do concur that this needs to be addressed in the not so far out future (definitely not of any urgency for the moment).
Note:
The core problem is (for the time being) that the documentation of contacts (Apple iOS side) is deactivated for our (Germany) region for the moment (check "COVID-19-Kontaktprotokoll in your settings) from Apple's side.
The App-Developers will not be able to do anything about the problem, until this is "lifted" (probably with the start of the "public-Beta", maybe mid-July).
It is not even unreasonable to guess that this will solve the problem, entirely.
So IF you SERIOUSLY want to run CWA on your phone, please revert back to iOS 14 in the meantime...

Please close issue request.
See #774

@Kabulla, pls change the last iOS 14 to iOS 13 (in your comment I mean)

@Kabulla, pls change the last iOS 14 to iOS 13 (in your comment I mean)

done... "Freud at work"... sorry...

Please take a look at the FAQ before posting a new issue. This issue got poste several times in the last days, and for the devs it can get really annoying seeing the same „issue“ again and again. (On top it is not an issue that can be solved right now, because the EN framework got temporarily disabled in Beta 1)
You can close this issue🙂

Can confirm that

Apologies! Searched before I posted. Just wanted to let you guys know, not an issue for me, having a developer OS brings it’s flaws...


Aqcu 1280 - revision

Working on another task, I noticed that the stationId wasn't being correctly included into the tab in the browser (it would sometimes, but not always, come up as another number entirely). I modified the inline expression and it seems to work better now...
before

after

Coverage remained the same at 97.427% when pulling 200d1b79e905d10ca2bab776547074bcd0816093 on mhines-usgs:AQCU-1280-1 into 66156ba7d97d615926e7f5932674285f95c589de on USGS-R:master.


Updating domain to point to pekko.staged.apache.org

Fixing: https://github.com/apache/incubator-pekko-site/pull/51

Targeting asf-staging. Proceeding by merging.


Question: can functions be called normally inside a coroutine?

It was not clear from the examples whether inside the coroutine C functions can be called normally rather than via the call api.

You can call functions normally. Those call functions just let you call a function on the threads stack if need be.  You would only need to use call function if a function you would call would exceed the size of stack you chose for your coroutines.


Notebook cut/copy stack overwrites editor clipboard

Type: Bug
Sorry if this is for the jupyter extension, but it seems like notebook is internal to vscode.
When trying to cut and paste I am used to the jupyter browser style of managing two different clipboards, one for cells and one for cell contents. It appears that the VSCode editor respects the notebook clipboard but the notebook clipboard will overwrite the contents of the editor.
Here are the steps to reproduce and after I show the steps that show how the editor clipboard works as intended.

Create new jupyter notebook.
create new cell, write "foo" in cell, select "foo" and copy (ctrl+c), exit cell (escape) [or copy text from anywhere else]
create new cell, write "bar" in cell, exit cell (escape)
copy/cut cell (x/c)
paste cell (v)
edit cell (enter), paste (ctrl+v)

expect to paste: "foo"
what happens: "bar"

The editor clipboard does not overwrite which I think makes more sense. Example:

Create new jupyter notebook.
create new cell, write "bar" in cell, exit cell (escape)
copy/cut cell (x/c)
create new cell, write "foo" in cell, select "foo" and copy (ctrl+c), exit cell (escape) [or copy text from anywhere else]
paste cell (v)
edit cell (enter), paste (ctrl+v)

expect to paste: "foo"
what happens: "foo"

VS Code version: Code 1.73.0 (Universal) (8fa188b2b301d36553cbc9ce1b0a146ccb93351f, 2022-11-01T15:38:50.881Z)
OS version: Darwin arm64 21.6.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
2, 2, 3

Memory (System)
32.00GB (1.72GB free)

Process Argv
/Volumes/workplace/erevia/erevia.code-workspace

Screen Reader
no

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

vscode-sql-formatter
adp
1.4.4

TabOut
alb
0.2.2

gitlens
eam
13.0.4

shell-format
fox
7.2.2

applescript
idl
0.21.2

vsc-python-indent
Kev
1.18.0

isort
ms-
2022.4.0

python
ms-
2022.18.1

vscode-pylance
ms-
2022.11.10

jupyter
ms-
2022.11.1003141027

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.12

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

autodocstring
njp
0.6.1

vscode-sort-json
ric
1.20.0

vscode-open
san
0.1.0

intellicode-api-usage-examples
Vis
0.2.6

vscodeintellicode
Vis
1.2.29

(2 theme extensions excluded)

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

looks like this is the same issue in https://github.com/microsoft/vscode/issues/162204 ?

@timforby you are right, it's duplicate and let's track it there.


2019csv file

Now that the salary map for year-2019 has been updated, it would be more convenient if the author provided the corresponding year-2019 csv file. Thank you.

Check here
https://github.com/kiang/salary/blob/gh-pages/data/csv/2019.csv


Add periods to B.C. on Programs page

Under 'BCIC':
Add periods to B.C. in this phrase: "...the needs of industry in B.C."

done


fix: correct namespacesRequired type in serverSideTranslations

Fixes #2201
Namespace is not extending Array but ReadonlyArray so ArrayElementOrSelf<Namespace> is evaluated to (readonly string[])[] instead of string[].
Checklist

[x] only relevant code is changed (make a diff before you submit the PR)
[x] run tests npm run test
[x] tests are included
[x] commit message and code follows the Developer's Certification of Origin

thank you... it's included in v14.0.3


Bump version v4.0.11

This PR is used to bump TiKV version to v4.0.10
Release note

No release note.

The commit need to be sign-off @lilinghai

/rebuild

/run-check-release-note

/merge

/merge

/run-all-tests

/merge

/merge


how to make opening and closing the menu faster

Is there anyway to make opening and closing the menu a bit faster? I could't find anything in the tutorial.

Yes of-course.
Just override those methods in your MainVC class
https://github.com/SocialObjects-Software/AMSlideMenu/blob/master/AMSlideMenu/AMSlideMenuMainViewController.h#L121-L131


Special: Comb and Comb with repetition

Implementation of comb from scipy. Essentially it is n choose k.
Current implementation only does this exactly (which may not be the most efficient way). The binom function is defined for the entire real numbers and should agree with comb at the integers.
With and with out repetition is represented as a different function since in scipy it is a kwarg, but this could be changed to a bool in the comb function if desired.
In general, I was unsure how to match the function signature in scipy as compared to what I can do in rust given that python is dynamic typing and much of the special functions in scipy have arguments be any of: real, complex, ndarrays, ints which can then change the return type as well. For example, the comb function can accept either int or ndarray of ints. If exact is False, then it will return a float. If exact is true, it will return an int.

Thanks for your contribution! This looks good.
I will give this a closer review this weekend. I'll probably release 0.4.0 since the savgol interface changed this week.


Request to add Ref to body object

I tried with a postman collection and the generated OAS do not have a ref component as below.
Any plans to update the library? This is required for any endpoint having body parameter.

Hi @muqeed11,
At this moment the library just use object as schema type and doesn't support refs.
Best Regards.


Bug fix

remove hard coded command "rm" to avoid deleting system files accidentally.
remove  UseParNewGC collector, it's deprecated  in JDK 9 and removed in JDK 10. our current env uses JDK 11.

outdated


Added -x to fallocate command during vsc predeploy

I made the necessary changes in the RELEASE_NOTES.md file, Thank you


[C++] Unrecognized filesystem type in URI: abfss://

I am running the below commands in databricks.
When I am trying to read a file which is stored in adls using pandas:

pip install adlfs
import pandas as pd
data = pd.read_parquet("abfss://data.parquet", storage_options= {})

Then I got the below error:

File "/databricks/python/lib/python3.7/site-packages/pandas/io/parquet.py", line 310, in read_parquet
return impl.read(path, columns=columns, **kwargs)
File "/databricks/python/lib/python3.7/site-packages/pandas/io/parquet.py", line 125, in read
path, columns=columns, **kwargs
File "/databricks/python/lib/python3.7/site-packages/pyarrow/parquet.py", line 1573, in read_table
ignore_prefixes=ignore_prefixes,
File "/databricks/python/lib/python3.7/site-packages/pyarrow/parquet.py", line 1434, in __init__
ignore_prefixes=ignore_prefixes)
File "/databricks/python/lib/python3.7/site-packages/pyarrow/dataset.py", line 667, in dataset
return _filesystem_dataset(source, **kwargs)
File "/databricks/python/lib/python3.7/site-packages/pyarrow/dataset.py", line 424, in _filesystem_dataset
fs, paths_or_selector = _ensure_single_source(source, filesystem)
File "/databricks/python/lib/python3.7/site-packages/pyarrow/dataset.py", line 371, in _ensure_single_source
filesystem, path = FileSystem.from_uri(path)
File "pyarrow/_fs.pyx", line 347, in pyarrow._fs.FileSystem.from_uri
File "pyarrow/error.pxi", line 122, in pyarrow.lib.pyarrow_internal_check_status
File "pyarrow/error.pxi", line 84, in pyarrow.lib.check_status
pyarrow.lib.ArrowInvalid: Unrecognized filesystem type in URI: abfss://data.parquet

Reporter: Prakhar Sandhu
Note: This issue was originally created as ARROW-17672. Please see the migration documentation for further details.

If you really want to use adlfs this issue is definitely solvable just with changes to the user code. However, I think this will also be solved by https://github.com/apache/arrow/issues/39317. This will connect up the new C++ AzureFileSystem on the python side and provide much better performance and reliability compared to adlfs.

Let's close this issue in favor of https://github.com/apache/arrow/issues/39317


RPC qbft_getSignerMetrics returns incorrect validators after blockheader transition block

When transitioning from contract validatorselectionmode to blockheader with validator overrides, the RPC incorrectly builds the metrics based on an empty set of validators for some of the block range.
This is because the RPCs use a BlockValidatorProvider.nonForkingValidatorProvider which is not aware of the BftValidatorOverrides. Hopefully, simply using a BlockValidatorProvider.forkingValidatorProvider when constructing the RPCs will fix it.
Steps to Reproduce (Bug)

Network started in validator contract mode with no extra data in genesis block.
Transition to blockheader mode with validator overrides
Restart node before next epoch is hit
Call RPC (before next block is produced so the cache isn’t populated)
The metrics will be wrong because no validators are returned for the affected block range. The VoteTallyCache can’t be reconstructed since the previous epoch is a contract-mode block (e.g. genesis block) which has no extra data, and the RPC’s validatorProvider doesn’t have access to the validator overrides so there’s no overrides available to replace the missing cache entries (which is how it works when producing blocks).

We should also use forkingValidatorProvider for IBFT2 as well.


Outdated section Catalog Properties Dialog Box

The section Catalog Properties Dialog Box is outdated.
The view catalog.catalog_property does not exist for SQL Server 2016 anymore.
The view catalog.catalog_properties does not contain a row for VALIDATION_TIMEOUT
The dialog does not contain a setting 'Validation Timeout'.

Closing now.  However, this Issue 1230 will still be used later to communicate updated status of internal DevOps MsEng bug 1386649 back to the customer.
CC: @douglaslMS

@abbgrade cc @MightyPen I have just opened a PR to fix these issues. Thank you for bringing the issues to our attention. The updated page will be published tomorrow Tue Nov 13.


Will you support pytorch>1.7.1 ?

Can you tell me the reason and how can I mod codes to support pytorch>1.7.1 ?
For example 1.8.1 ?

I was also looking into doing this transition myself, but like the other poster, am not familiar with the fft methods in Pytorch. In 1.8 the torch.fft methods were remove and torch.fft is now a full new module.
https://github.com/pytorch/pytorch/wiki/The-torch.fft-module-in-PyTorch-1.7
Most importantly: the backing code and tensors changed to be complex tensors, and so even if you replace the call to torch.rfft():
https://github.com/EndlessSora/focal-frequency-loss/blob/27f99aa8429d890a29cd693d41598313aa1a6311/focal_frequency_loss/focal_frequency_loss.py#L48
with something like
return torch.fft.fft(y, dim=2, norm='ortho')

Then later the calls to .max() fail:
https://github.com/EndlessSora/focal-frequency-loss/blob/27f99aa8429d890a29cd693d41598313aa1a6311/focal_frequency_loss/focal_frequency_loss.py#L65-L68

Thanks all for your interest and suggestions. We have supported PyTorch > 1.7.1 in this new commit. Please upgrade/reinstall the focal-frequency-loss package through pip or use the newest code in this repo.
Please close the issue if you find the problem is resolved. Thank you. :)

thx


Prepare interceptor beans in archetype resources

This makes it easier for people to actually use the already existing package
classes for the various interceptors; they simply have to uncomment some lines
and are then ready to implement project or artifact specific interceptor
behaviour.
Please review.
/cc @buehner @dnlkoch

Nice addition! Thx!

Cheerio for the review!


Set default fold level

I'd appreciate being able to set a default fold level. A ton of my flow end up expanding all of the folds by default. It might also be possible to read VIM's native foldlevel value initially
This is possibly related to #10

not sure if this is related, but every time i open NeogitCommitView everything is expanded. I think that isnt good default (or at least should be configurable). When I open commit, first of all I wanna see files that are affected. Then hunks and their contents. Probably bind that folds everything would be sufficient. If I remember correctly, emacs have this "global folding feature" where tab/shift+tab folds everything, something like that would be amazing.


Create pull_request_template.md

Title of the PR Review is wrong according to our Wiki
The file should be inside a .github folder
The branch name is also wrong according to our wiki


Issue with lg_lvn361hv4 hysteresis  and unable to set settings

Hi I have a lg central air handler heat pump setup.  My esphome-lg-controller is working in that it turns on and enables compressor, gets pipe temp.
But I can't seem to change the fan speed like I could with the builtin controller, I also am having problems with getting the outdoor unit activated. I have to set the target_temp more than 10 degrees colder than the actual temperature in my home for it to activate.  I thought this was just a problem with the hysteresis so I made a template for the temperature sensor
        value_template: >
          {% set temperature = state_attr('climate.house_thermostat', 'current_temperature') %}
          {% set setpoint = states.climate.first_floor_heatpump_first_floor_heatpump.attributes.temperature|float %}
          {% if (not is_state('climate.first_floor_heatpump_first_floor_heatpump', 'cool')) %}
            {{ temperature | int}}
          {% else %}
            {% if setpoint  < temperature -1 %}
              {{ (temperature + 12) | int }}
            {% else %}
              {{ temperature | int}}
            {% endif %}
          {% endif%}

but even with this it didn't activate I still had to set the temperature to 60f for it to activate even when it thought temperature was in 90s.  So I am not sure exactly what is going on. Is there any debug information I could provide to hopefully solve some of this?

What's the official LG controller for your unit? Can you pastebin the output from esphome logs your-yaml-file, after running it for a few minutes?
It's possible LG uses a different protocol for these air handler devices.

The remote I use is PREMTC00U
The unit is lvn361hv4
the outdoor unit is luu350hhv
logs_first_floor_heatpump_logs.txt
some logs.  I set fan speeds with no change, i also set some temperature points in logs, as well as finally setting it at a very low setpoint which finally caused the out door unit to fire.
logs_first_floor_heatpump_logs.txt
thank you for looking into this.

It shows that your "fan mode" is set to Auto all the time. Try changing that setting, it's part of the climate entity. It should have an option for Low / Medium / High / Auto.
The "Fan Speed High" and "Fan Speed Low" values are advanced settings that only apply if you set the fan mode to "low" or "high" first.

The temperature thing is weird because what I'm seeing in the log all makes sense. It's sending a current temperature value that's a lot higher than the target value.
Does it work better if you set the "Internal Thermistor" setting to On?

i just tried playing with fan settings. and still see no change in fan speed, and can confirm that my current meter shows same 316 w draw when its just running this fan and no compressor
logs_first_floor_heatpump_logs(1).txt

Something I just thought of to check: the unit is sending its own temperature value in the CC message and that's only 20-21C (68-69F). So maybe that's the "current temperature" value it's using internally? That might explain the behavior you're seeing.
The esphome controller currently doesn't send this type of message to the AC, because it hasn't been necessary for other units. I wonder if your unit requires this.
If you want to test this, you could add the following code before this line in lg-controller.h
uint8_t buf[MsgLen] = {};
buf[0] = 0xAC;
buf[6] = uint8_t(target);
if (send_buf_[1] & 0x1) {
    buf[7] = 0x80; // settings changed
}
if (send_buf_[5] & 0x1) {
    buf[8] = 0x50; // target + 0.5C
}
buf[9] = uint8_t(temp * 2);
buf[12] = calc_checksum(buf);
ESP_LOGD(TAG, "sending %s", format_hex_pretty(buf, MsgLen).c_str());
UARTDevice::write_array(buf, MsgLen);

setting it to internal thermistor, did seem to have it activate close to what I would expect, i set it to 72 and the internal thermistor is at 75 and it kicked on.

i think this code change caused pip temperature in to change. it was reading 9c right before code deployed ,and changed to -5c after , also the pipe out temp went from 10c to 17c as well.
logs_first_floor_heatpump_run(1).txt
i changed the sending line to MOG sending so it was easier to spot in logs but otherwise no change

i think this fixed different bug with pipe temperature i thought i have had. the in and out temp were almost always the same which seemed awful, since applying this patch they seem to have diverged 10c or so

that def fixed my setpoint issue today, was hard to tell as so hot only got to temp once today

So with that code change it works as expected with Internal Thermistor set to Off?

Correct. Solved issue with temperature. Any ideas on fan speed settings?

What if you change it from Cooling to Fan Only and reset the fan speed settings by setting them all to 0. Do you then see any difference between Low and High?
Do you remember which options you had with the LG controller? It’s weird this doesn’t work.

fan only mode I measured the following. i felt and saw no change with these settings.
with 0 for all settings
auto: 318w
low: 315w
med: 316w
high: 316w

so i assume same within noise values
logs_first_floor_heatpump_logs(2).txt
now trying with 240 245 and 250 as the values for the fan speeds, capture shows me setting these values as well.
auto: 315w
low: 315w
med: 316w
high: 314w

logs_first_floor_heatpump_logs(3).txt

I was able to set fan speeds with the factory controller in the past

Your unit is likely different from the AC units that others are using with this controller. I suspect it uses the newer protocol messages more that my controller doesn't send or process.
One option would be to connect both controllers at the same time and put the ESPHome controller in 'slave' mode, so you can see what's being sent by the LG controller if you change settings.

is that just setting
is_slave_controller: "true"
and connecting the remote to same bus line?

is that just setting is_slave_controller: "true"
and connecting the remote to same bus line?

Yes.


App Crashed with No such method message.

My Android app getting crashed with below error message. Already tried with change 'com.google.android.gms:play-services-base:8.3.0' to 'com.google.android.gms:play-services-base:8.4.0' but didn't get it worked. RN version is 0.32 in windows environment.
Fatal Exception: java.lang.NoSuchMethodError
No static method zzb(Ljava/lang/Object;Ljava/lang/Object;)Ljava/lang/Object; in class Lcom/google/android/gms/common/internal/zzx; or its super classes (declaration of 'com.google.android.gms.common.internal.zzx' appears in /data/app/com.ssssss-1/base.apk)

You're probably on version 0.7.x of react-native-maps. Had same problem. Upgrade to version 0.8.2 of react-native-maps.

You should be using google play services 9.4.0, which is the default for react-native-maps 0.8.2

Hi @mauron85 @felipecsl , I have changed my code as suggested, app din't crashed, but there is a message saying that the app won't work unless update Google play services. Could not expect all the users to upgrade their play service version? How to make the app compatible with older versions of google play?

I would say, they'll be eventually forced to update anyway (by google maps or other google app that will stop working). But you can stay on version 0.7.x and google play services 0.8.4. I did it this for https://github.com/mauron85/react-native-background-geolocation-example
But keep in mind, if any of react-native module also uses play services, you'll end up with multidex errrors.
In my case it was module I'm developing myself, so fix was easy.

**Module react-native-maps v0.7.1 requires Google Play Services library version 8.4.0 (issue #420), we need to change node_modules/react-native-mauron85-background-geolocation/android/lib/build.gradle

dependencies {
    ...
    compile 'com.google.android.gms:play-services-location:8.4.0'
    ...
}

@mauron85 Thanks a ton for detailed explanation. I am updating for play 9.4.

when an app requests a version of google play services newer than the one that the user has installed, it should automatically update in the background

App loaded after adding the following items at app/build.gradle. compile "com.google.android.gms:play-services-base:8.4.0" compile 'com.google.android.gms:play-services-auth:9.4.0' compile 'com.google.android.gms:play-services-maps:9.4.0' Now the app shows UPDATE button with message "this app won't run unless you update google play services"

Anyone fixed the message "this app won't run unless you update google play services"? Can't find a solution. Runs fine on device, but not in simulator


[HH-WEB] Finalize route options in Navigation menu

https://github.com/osmandapp/OsmAnd-Issues/issues/2332


Add optional effective_caller_id parameter to vtgate_client.py.

@mberlin This small change is needed to support an upcoming effective_caller_id change in the YT code base.

You should also add tests to python_client_test.py to test this works as expected, see what vtgatecleinttest does with the passed values in callerid.go. Would test the stubby version in google3 as well.

Please refer to https://github.com/youtube/vitess/blob/master/go/cmd/vtgateclienttest/services/callerid.go
for callerid testing logic and https://github.com/youtube/vitess/blob/master/go/cmd/vtgateclienttest/ contains a mock testing vtgate for testing clients

@dumbunny I've reviewed the current state and I'm fine with that. Let me know when you added the tests and I'll review them as well.
Please note that you need to mention @michael-berlin on Github because "mberlin" is already taken :(

My bad. For some reason, @dumbunny wasn't taken when I got on.
Dean Yasuda
On Tue, Aug 18, 2015 at 6:39 PM, Michael Berlin<EMAIL_ADDRESS>wrote:

@dumbunny https://github.com/dumbunny I've reviewed the current state
and I'm fine with that. Let me know when you added the tests and I'll
review them as well.
Please note that you need to mention @michael-berlin
https://github.com/michael-berlin on Github because "mberlin" is
already taken :(
—
Reply to this email directly or view it on GitHub
https://github.com/youtube/vitess/pull/999#issuecomment-132411877.


[express-apollo-prisma] Stopping the script prisma:studio causes an error

Note
Possible duplicate of #1136

Reproduction
Stack
OS: MacOS 13.1
Shell: zsh
Node: 16.17.0
Steps

Run pnpm install
Ensure there is a valid configuration (for example, by copying .env.example to .env)
Run pnpm infrastructure:start
Run pnpm run prisma:studio
Press [ctrl] + [c]

Expected behaviour
All services started by the command should gracefully stop again. No errors should be shown in the terminal output, and the exit code should be 0.
Actual behaviour
The command stops with the message:
ELIFECYCLE Command failed.

With exit code 130

We decided not to do graceful shutdowns.


build: Bump gitpod image for DDEV v1.23.3 [skip ci]

The Issue
After release we have to bump the gitpod image for use with v1.23.3. This does it.

Tested this after implementing in ddev-gitpod-launcher and it's working OK. I'm pretty sure we can just add and apt update && apt install -y ddev to the .gitpod.yml in both places and we won't have to update for every release any more.

I'm pretty sure we can just add and apt update && apt install -y ddev to the .gitpod.yml in both places

I'm all for it.

apt-get install -y ddev is added in

https://github.com/ddev/ddev-gitpod-launcher/pull/44

We don't need to add it to the .gitpod.yml here because we build from source here. When it goes in there, we can update the developer docs here.


Appservice user events

The appservice user events (room.join, room.invite, room.leave) only easily present the roomId, the userId needs to be fetched via the event.
As the userId is commonly needed for handling those events, perhaps consider making that a parameter

Ideally I'd actually like to go the other way and push the room ID into the event. This isn't being done currently because the event object is any due to it being the JSON object received over the wire. https://github.com/turt2live/matrix-js-bot-sdk/issues/11 would address this though.


SAK-42017: Too much time while loading gradebookNG

I would like someone more test this one before merge. Thanks!

I've just updated all issues. Thanks @adrianfish !


VS Code Prettier extension version 10.1.0 conflicts with Prettier v3 in Preflight

Formatting with VS Code Prettier extension v10.1.0 (latest on Prettier VS Code extension page as of Aug 2023) causes issues with Preflight because v10.1.0 uses Prettier v2.x and Preflight uses Prettier v3.0.2
✖ Prettier
  › Prettier has not been run in the following files:
    components/styles/chips.styles.ts

    For each of the files above, open the file in your editor and save the file. This will format the file with Prettier, which will cause changes
    to appear in Git.

    In some very uncommon cases (this probably doesn't apply to you), the mismatch may come from inconsistent end of line characters. Read more
    here: https://github.com/upleveled/answers/issues/31

For a workaround, revert to Prettier VSCode Extension version v10.0.0. An update of the Prettier VSCode Extension is expected to rectify the compatibility issue. Currently, it's not possible to transition to Prettier VSCode Extension version v10.2.0 preview. Relevant information can be found in Karl's comment in the prettier-vscode PR.
The styling in the components/styles/chips.styles.ts file in next-starter-peacock portfolio is causing this issue with Preflight. VS Code Prettier extension v10.1.0 formats:
transition: transform 0.15s, color 0.15s;

Prettier v3.0.2 formats:
    transition:
      transform 0.15s,
      color 0.15s;

Created a full issue to track this problem now, since it's pretty easy to make the extension crash with an ESM config:

https://github.com/prettier/prettier-vscode/issues/3298

Decided to just install prettier during UpLeveled ESLint config install:

https://github.com/upleveled/eslint-config-upleveled/issues/332


Add ptr record support

Hi @cwood. Thanks so much for adding this. Much appreciated. Would you be able to add the new type to the tests (you should be able to just add to https://github.com/puppetlabs/puppetlabs-aws/blob/master/spec/unit/type/route53_record_spec.rb) and to the documentation in the README: https://github.com/puppetlabs/puppetlabs-aws#types and https://github.com/puppetlabs/puppetlabs-aws#type-route53.

Yea I can do that and update. Really hoping this module gets far. I want to use this over terraform

Anything else that needs to happen?

Hi @cwood. Thanks for this, apolgies, I'd been travelling back to the UK. This looks good, much appreciated.

Thanks so much!
On Sun, Oct 18, 2015 at 2:43 PM Gareth Rushgrove<EMAIL_ADDRESS>wrote:

Merged #237 https://github.com/puppetlabs/puppetlabs-aws/pull/237.
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppetlabs-aws/pull/237#event-438578848.


Bump mbedTLS version to 2.28.0 (new LTS).

Keep applying the windows entropy patch (UWP support).
Remove no longer needed padlock patch.
Update thirdparty README to reflect changes, and new source inclusion criteria.

I tested a build against distro-packaged mbedtls 2.28.0 with builtin_mbedtls=no and that seemed to work fine.
Let's give this one a spin and see how it fares on all platforms.

Thanks!

Cherry-picked for 3.5.

Cherry-picked for 3.4.5.


Better Swift compatibility for status codes

Swapped the existing enum to NS_ENUM for considerably better Swift compatibility. Now things will automagically show up as Ints instead of some nonsense with Int32, and you can use the failable Int initializer for stuff like:
guard let statusCode = HTTPStatusCode(rawValue: 404) else {
    print("not an actual status code"
    return
}

switch statusCode {
case .notFound: // or .NotFound in Swift 2.3
   print("404!")
default:
   print("Something else went wrong")
}

Bumped podspec to 2.0 despite the minute number of changes since this would break existing swift code.

@ilg I got sick of making Swift extensions on this. 😄  As far as I can tell, this should not have any adverse affect on existing Obj-C code.

@ilg post-holiday poke


Fix spelling error when starting BigQuery jobs

Fix spelling error when starting BigQuery jobs

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @chamikaramj
Please merge this


Keyboard shortcuts not bound in RMD files

Dev15, but may be in the master too

https://github.com/Microsoft/RTVS/commit/698b187013402962ac7ff35ccdf9ae4d321a1869


feat(client-debugger): Redesign debugger UI

Description
Re-think the UI for the debugger so we can separate things that are container-specific (container data, audience, state transitions) from things that are container-agnostic (e.g. telemetry).

Reviewer Guidance
So far I've only updated the inline view. Once I have buy-in from the team and we've agreed on the approach, I can expand it the changes to the devtools view. Would love to be able to reuse more thing across them in order to do that, though, and that might require tweaking the interface/communication-mechanism between the FluidDebugger registry and the main debugger view that gets rendered.
~One thing that's not working correctly, as can be seen by the end of the gif above, is that when switching between containers in the menu, the contents of the view on the right are not updating to reflect the new container data, until we switch the "inner view" (data/audience/state history).~ Fixed :).

The structure of left pane and details looks great. In terms of UI library/framework being used, do you have any insight on what Chrome/ Edge use for their debug windows? Is there any existing library that we can use to adopt color scheme/design language?

The structure of left pane and details looks great. In terms of UI library/framework being used, do you have any insight on what Chrome/ Edge use for their debug windows? Is there any existing library that we can use to adopt color scheme/design language?

I don't know if there's any standard for that; agree that it would be great to find it if it exists and leverage it.

Styling nitpick: Can we add some sort of highlighting in the container selection list to indicate which one is currently selected?

Split some changes into #14809 and #14810 .

Ok team, review away and let's merge this!

I left a few more comments. Mostly nitpicks, but I think there are a few lingering React state issues to resolve before merging.

@Josmithr addressed, all in a single commit so should be easy to look at just these changes VS the version you had seen.

Tong's first PR is merged and the second one should be soon; I'll wait for those and integrate those changes, then merge this.


cidr should be referenced as cidr_block

I have been playing with this, and it seems that best practice is to use the same variable names for inputs as aws outputs to avoid confusion. this module is using cidr as the input but aws outputs cidr_block am I missing something? Is there a reason why cidr is used and I just don't understand?

You are absolutely correct. This has been done without proper thinking. There are no plans to change this in this module though unless somebody wants to submit a PR.


Require signed commits

Is your feature request related to a problem? Please describe.
To better enhance our security, require signed commits.
Describe the solution you'd like
GPG signed commits for all interactions.
Research
Here's why one should use commit signing.

Created a personal commit signing key (ssh), associated it with my account, associated it globally with git, requiring commit signing for the main branch now.


Add internal transfer and deposit fiat abilities

Superadmin and Admin users can:

Manage InternalTransfer
Create Deposits::Fiat

Accountant, Technical, Support users can:

Read InternalTransfer

we should have the same for 2-6

we should have the same for 2-6

we should have the same for 2-6

https://github.com/openware/peatio/pull/2811

we should have the same for 2-6

https://github.com/openware/peatio/pull/2811

we will rebase master on 2-6-stable, thats why we don't need this PR


{Network} az network nic ip-config: Migrate commands to codegen v2

Related command

az network nic ip-config
Description

Total: 9 commands
Testing Guide

History Notes

[Network] BREAKING CHANGE: az network nic ip-config: Rename output property privateIpAddress to privateIPAddress to keep consistent with the name in API
[Network] BREAKING CHANGE: az network nic ip-config: Rename output property privateIpAllocationMethod to privateIPAllocationMethod to keep consistent with the name in API

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network


app.use() all routes

I can't seem to find anywhere in the documentation if you can use this for all routes by going app.use(require('express-brute)());
I really don't feel like inserting this into each route - I have hundreds..
Is this possible?

You can totally put it in app.use if you're trying to implement a global API limit or something. For something like a maximum of 1000 requests an hour it'd be something like this though:
var ExpressBrute = require('express-brute');

var store = new ExpressBrute.MemoryStore(); // stores state in memory, don't use this in production
var bruteforce = new ExpressBrute(store, {
    freeRetries: 1000, // Max API calls in a day
    refreshTimeoutOnRequest: false,
    minWait: 61*60*1000, // 1 hour 1 minute (should never reach this wait time)
    maxWait: 61*60*1000, // 1 hour 1 minute (should never reach this wait time)
    lifetime: 60*60 // 1 hour (seconds not milliseconds)
});

app.use(bruteforce.prevent); // error 429 if we hit this route too often

Fantastic Adam! Thanks for clarifying.

I want to limit the no. of call to 5 in 3 minute with the min wait of 30 min. How can I achieve the same?


Tezos signer forwarder

The last remaining piece of https://github.com/midl-dev/tezos-on-gke/ to move into tezos-k8s, tezos-signer-forwarder is a terminating pod for ssh tunnels exposing a tezos signing endpoint from an on-prem location.
This PR introduces:

a new alpine container running sshd with custom configuration: this is the tunnel termination endpoint
a sidecar for this container that exposes the metrics of the signer, complemented with a custom URL monitor
a chart.

The chart allows to define several "bakers" which correspond to one baking public key. A "baker" can have several "endpoints" corresponding to an individual hardware signer. Every "baker" deploys a service that the tezos chart can target as remote signer; this is indistinguishable from a cloud-hosted signer.
This supports HA deployments with 2 remote signers.
It works well in combination with https://github.com/midl-dev/tezos-remote-signer-os but all that's really needed is to connect to the ssh tunnel endpoint and forward the signer port.
This will be better documented later.

@harryttd long story short, I was struggling with this a lot in the past, I've now rediscovered that sshd must run as root. It's not really meant to be used in containers, but it's possible to do so as long as you don't change the user. There is still an user (signer) but sshd assumes this identity when it starts. see https://superuser.com/a/1548482
I've addressed your other comments, thanks for them. Please re-review.


aerospike: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/aerospike/aerospike-server/releases.
meta.description for aerospike is: Flash-optimized, in-memory, NoSQL database
meta.homepage for aerospike is: https://aerospike.com/
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/plfx9s2c7vq7idxkiyl82g1lb72qkf95-aerospike-server-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/plfx9s2c7vq7idxkiyl82g1lb72qkf95-aerospike-server-<IP_ADDRESS>

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

aerospike

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/plfx9s2c7vq7idxkiyl82g1lb72qkf95-aerospike-server-<IP_ADDRESS> \
  --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 aerospike https://github.com/r-ryantm/nixpkgs/archive/6a9a98298cacbe5d1e9af6c6dd283b9db6a777d5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/6a9a98298cacbe5d1e9af6c6dd283b9db6a777d5#aerospike

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/plfx9s2c7vq7idxkiyl82g1lb72qkf95-aerospike-server-<IP_ADDRESS>
ls -la /nix/store/plfx9s2c7vq7idxkiyl82g1lb72qkf95-aerospike-server-<IP_ADDRESS>/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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

aerospike

Maintainer pings
cc @kalbasit for testing.

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

1 package built:

aerospike


Usage with Normal JS Project

How do I integrate geo-three with a normal JS Project without NPM

Hi tentone, Thanks for the help !


TOOL: Permissions managed from multiple places, explore universal

User Story
“As a Developer I want have a universal way to manage roles and permissions in one space so I can get have a simpler more efficient process.”
Tasks
Steps needed to achieve this feature.
Acceptance Criteria
The checklist the product owner uses to decide whether a story is done. Objective and discrete answers (such as a yes/no) that can be tested.

Role Management is potentially going to be in future forms flow versions. This is not a very pressing issue in DJ for us right now. Cold and Flu is the only form where we have an "admin" role, and we have that process figured and works well.
So not a priority and we do not want to invest in custom developing this when it might be in future forms flow versions. Closing.


Feature/restructuring

after #61 is merged.
This PR is intended to open up a discussion about a more flexible structure. I am currently planning on adding point cloud coloring and eventually also Turcan's meshing into this repo. If we do not want an ever growing number of packages in the root directory, I propose a structure like this.
What do you guys think?

PS: let's first merge the reformatting. otherwise it is hard to spot anything

yeah, i think what should be quickly checked is the documentation, whether everything is still at the correct place


Default translation support

Hello. Thanks for great work. It's only one question. How about support default translations? It seems in regex you always seach values with key: ([.][^\1)]+)+" , but translations can be in resources/lang/en.json example and call __('someValue'). So now there no way to sync their? Some thoughts about it?

@al0mie, currently there is no support for json translation files.
Default translations were not available when this package was developed.
What is the advantage of using default translations?

@vsch, I guess for using translation as is if it not exists, so __('I love coffee') return I love coffee if not exist value in file. Using simple lang function return something like message.I love coffee.
Also it possible to store common messages (not to define specific files like message.php).
Seems, it was proposed here https://github.com/laravel/internals/issues/252
So maybe add some kind of handler for sync this?

@al0mie, I looked at the justification and in my opinion it solves nothing but a lazy approach to translations. Having duplicate short keys is a problem caused by lack of standard and coordination in the development team which is not addressed by the default translations. Nothing stops developers from creating similar but different strings for the same items.
Also lumping all translations into a single file does not seem like a good way to break up large site translations.
With LTM the typical use case I use, is to create the UI with keys without creating any files or translations. Then go into LTM web UI and use "Auto Fill" to quickly convert keys to text. Later these auto filled translations can be adjusted for correct formulation and translations to other languages can be applied.
The other issue is translation keys can have limits on characters they contain while strings used for default translations do not. This causes a lot of issues for LTM since it passes "keys" in the URL and Laravel does not handle some combinations even when URL encoded. Right now translation keys containing some combinations of characters will cause LTM not work for these. This limitation was inherited from the original package.
Being able to handle any characters in translation keys is a big rework of the package and would be pre-requisite to handling default translations. If someone wants to do this work I would welcome it.

@vsch, thanks for extended answer, I guess you are right.

Being able to handle any characters in translation keys.

Maybe it's not possible for limitation of format files, but it is not common cases.

This causes a lot of issues for LTM since it passes "keys" in the URL and Laravel does not handle some combinations even when URL encoded.

Any ideas how better do it?

@al0mie, ideally the WebUI support should be changed to use post instead of get with parameters on the URL.
Also, JS code assumes it can use the translation key for valid identifiers HTML. This is also another roadblock to making arbitrary keys. It would require changing how lookup is done for translations in JS code.

@vsch, yes, I understand you. I'll think about it. Maybe do some refactoring before it, seems it's more important.

@al0mie, I did the most work on this package when I desperately needed the functionality for a project and had no time budget for it. A lot of functionality was hacked into existing implementation with quite a bit of clean-up afterwards but never had the time to rework it to allow arbitrary keys.
If you do look into it and have questions, don't hesitate to ask.

I asked this same question in the original repository, but since Barry doesn't have time for the project, it was not worth going deeper into the question.
I would like to see this implemented, so I would be glad to help in what I can, but the one who knows better how the package works is you @vsch.
Could you indicate a strategy to follow to implement this functionality?

@gerardreches, this functionality can touch on every aspect of the translation package: import, DB, export, cache, WebUI and Translator service or at least import, export, cache, WebUI and translator.
The simplest way to do this would be to:

Parse the JSON translation files and store them in the JSON:: group with a fake short key for all translations in the DB so that DB and WebUI for these is not distinguishable from standard translations.  However, I would add a fake locale, say _ that is not editable that represents the original key string. This way you could create a translation for the primary locale for the string, changing the translation string used. Otherwise you will not be able to have a JSON translation file for the primary locale.

In export determine when the group is a JSON destination file and export these files in JSON format using the fake locale string as the translation key.

Cache should also be updated in those cases where it is enabled to override file content with cached values. Since the key is the fake locale string then JSON:: group should use this as the key not the short key. The rest should be automatic.

Translator should know how to handle JSON group translations that are cached.

Showing which values are cached in the WebUI (uses alternate background for these translations) should also be looked at in the case of JSON:: group to make sure it works as expected.
Also the fake _ locale should be excluded from all the locale editing since changing that will make all the key strings out of sync with the code.

First iteration might uncover a few kludges but should be workable.

This has been implemented in the master branch for Laravel 5.6


docs site is out of sync with npm version

Issue description

version #4.8.0
components: Popover

Steps to reproduce issue
The 4.8.0 release uses the Popover components <PopoverTitle> and <PopoverContent> but they are not in the 4.8.0 release. This issue took me quite a while to chase down, I believe the docs site should be in sync with what a normal npm install will give the user...

...

We are migrating to version 5.0.0 fairly quickly and everything was in a branch previously however now merged into master ... it shouldn't be long before 5.0.0 gets stable

#195 needs to be implemented to allow people to view docs for previous versions. Right now the docs have npm install --save reactstrap@next react react-dom indicating they works with the @next version (in one place, but needs to be update in other place), but yes, you cannot see docs for the normal version now.

Do you guys feel that Reactstrap next with Bootstrap beta is relatively production ready as is?

Speaking strictly for reactstrap. I am using reactstrap v4.x in production now (with a uikit based on bootstrap alpha.6). v5 is just as production ready (I just cannot use it in production myself because that uikit I mentioned doesn't support the bootstrap beta yet)
As far as making the update from v4.x to v5.x and the current state of the v5 alpha, I don't foresee any more breaking changes.
There are a few known issues (when using with react 16), but none are "show stoppers" (AFAIK). At worst, some animations will not work and refs within modals do not get set in the order they should (marked as a wontfix in react) both issues only with react 16.
It's not 100% drop-in replacement (even though I tried). There are some breaking changes which will require you to make some changes when upgrading.
I deprecated as much as I could (props and components) without removing anything, but some stuff (e.g. tether props) simple do not exist anymore and do not have direct a replacement or upgrade path. (For tether props specifically, I don't believe they were used as much and many of the features enabled by them are now available via various props on components when use the PopperContent)
We are trying to fix the animation issues with react 16 before cutting v5 officially though.
Give it a try, start your upgrade development using the alpha and if you run into any problems, open an issue.
As I said above, I don't foresee any more breaking changes, so any work you do to upgrade will still be valid when v5 officially releases.

I switched from react-bootstrap to reactstrap and its been amazing so far, the reason I switched is because reactstrap is updated with bootstrap 4 and everything works seamlessly (you can't use bootstrap 4 css styles with react-bootstrap you will get messed up big time.),

we should get this to become the standard in react projects and contribute to the code.

With the release of 5.0.0, the docs are more in sync, but they will still be slightly ahead as they are generated from master.


optimize.minimize(method='SLSQP') constrain violation

I have tried to run a multivariate, constrained and bounded optimization with both, optimize.minimize(method='SLSQP') and optimize.fmin_slsqp(). Indeed I get the same result:
For some initial guess for the independent variables x0, I get a converged solution where my equality constraints are satisfied (all equal to 0). However, for some initial guess values x0 I also get a converged solution ( 0 : Optimization terminated successfully) but when I check the equality constraints, they are not zero.

The constraints are satisfied only up to some numerical tolerance, given
as a parameter to the algorithm. Is that the issue?

As always, providing a self-contained example eliminates unnecessary back-and-forth:
http://sscce.org/

The constrain residuals are [-4098.2, -17809.9, -47707.2].
My code is quite complex but I am preparing a self contained case right now.

I save the residuals (equality constraints function result) in a global variable at each evaluation. I check this variable at the end of the optimization process to check if the constraints are actually met. This workaround is ok for me.

Reopening (workaround should not be necessary)

A reproducible test case would still be useful (there's not much that can be done with the information at hand).

No reply, closing.

No follow-up, and some SLSQP bugs have been fixed, so closing.


Alpha unused

https://github.com/amirgholami/adahessian/blob/fe2c5741501c6b490c100e56b6af9d630cd97c79/adahessian_tf/adahessian.py#L341
I noticed that the alpha variable calculated above is not being used. I suspect this is not intended.

Thanks a lot for pointing it out. We correct it.

Thanks a lot for pointing it out. We correct it.


Client doesn't work with runtime

I think that eshttp should use the same backend for both node and runtime, since the net module is now fully supported in runtime. It currently throws an error saying that backend.createClientHandle is not a function. The full error:
TypeError: backend.createClientHandle is not a function
    at new HttpClient (/node_modules/eshttp/lib/http-client.js:16:28)
    at module.exports (/client.js:7:18)
    at runtime.shell.setCommand (/index.js:14:22)
    at Object.exports.runCommand (/node_modules/runtimejs/js/service/shell/index.js:46:21)
    at e (/node_modules/runtime-telnet-server/index.js:40:39)
    at Socket.client.on (/node_modules/runtime-telnet-server/index.js:56:7)
    at Socket.EventEmitter.emit (/node_modules/events/events.js:77:17)
    at readableAddChunk (/node_modules/readable-stream/lib/_stream_readable.js:198:18)
    at Socket.Readable.push (/node_modules/readable-stream/lib/_stream_readable.js:157:10)
    at Socket._handle.ondata (/node_modules/runtimejs/js/modules/net.js:96:40)

@facekapow Using net here means using node streams, which, I believe, are slow. I think specialized backend for runtime that uses low level sockets API can be more efficient.
With the addition of whatwg streams to the web platform, I think, current node API is going to be less useful.

OK, that's fine, just that in order for the module to be used for runtime http, it needs to support an http client.

yep, I've been mostly focused on the server component, gotta fix client too.

#7 Fixed this


Container crash after startup with exitcode=6 when using vespa-sentinel-cmd on on-prem Podman setup

When creating and starting a container on an on-prem host , we are seeing the container health status transition from initialization to up. However, shortly after, the container crashes. It exits with an exitcode=6.
Steps to Reproduce
Create a container on an on-prem host using Podman.
Start the container using vespa-sentinel-cmd start container.
Observe the logs and notice the following :
Container health status changes from initialization to up.
"Starting cluster monitor thread" appears in the logs.
GS logs appear but do not going more then 19ms.
The container crashes.
Running the process manually results in exitcode=6.
Expected Behavior
The container should start and remain stable after the up status is reached, and the process should not exit with an error.
Actual Behavior
The container crashes shortly after the "Starting cluster monitor thread" message appears. The process exits with exitcode=6
Environment"
Podman: version 4.4.1
Vespa-> 8.301.19
On-prem host OS: Linux 8.8
Please provide guidance on interpreting exitcode=6 or steps to further diagnose the issue. Any help with potential configuration issues or log file analysis would be appreciated.

I think we need some more info to reproduce this. Thanks

Thats the issue, we need more info too but lack of logs isnt helping. Can you check if all your exit conditions are covered with adequate logging?

first, run vespa-stop-services and verify that no vespa processes are running afterwards.
Then, after vespa-start-services run vespa-logfmt -f -N -l all to tail the log.
Paste here what you see.  We don't have any exit conditions that would cause the container to exit without any log messages.

exitcode=6 is probably from this code:
        } catch (Throwable e) {
            log.log(Level.SEVERE, "JDisc exiting: Throwable caught: ", e);
            System.exit(6);

Is this an async logger? Because we havent seen this log
We have tried to do vespa-sentinel-cmd stop and start with vespa-logctl container all=on but that didnt result in any additional logs?

logging is sync, so what do you see from vespa-logfmt -l all ?

@arnej27959 we dotn have that output captured now, however, irresp, SEVERE logs are turned on by default and yet there was nothing. Hence we urge you to check if there are other exit conditions without any logs

this is the only place with exit code 6.  re-open when you reproduce it and has the output that I asked for, there's no information to work with as is.


BEWARE OF MISSING PLUGINS! webrtc-to-hls files not being created.

I followed the readme files line for line and found that it worked perfectly when running locally, but when running on server the hls files were not being created. If you have a similar issue, check your gstreamer plugins list using this command: gst-inspect-1.0 . If you do not see this plugin 'multifile:  multifilesink: Multi-File Sink' then that is your problem. Install gstreamer using the official docs https://gstreamer.freedesktop.org/documentation/installing/on-linux.html?gi-language=c

The same problem, when running the demo server getting this error:
GET http://localhost:8000/playlist.m3u8 404 (Not Found)
All gstreamer plugins installed


Add CLI --help text to README

The documentation for specifying a web seed and recursive files should be in the README.

A CLI usage section would be helpful
Maybe even an additional section on how to seed a planktos website using a webtorrent compatible torrent client since this requires manually specifying the web seed url.

I actually don't think that this is necessary, since they can access this information via the CLI.


Republish Hakai JSP data on OBIS under Hakai organisation rather than OBIS Canada

Hakai was added as a publishing organization after I had already published the Hakai JSP data under the 'Canadian Node of the OBIS'

Turns out this is not possible for data sets that have been registered with GBIF and assigned a DOI (both of which were done for the Hakai Data set). So unfortunately this can't be done (without duplicating the data.


Show name of the schema instead of Show Child Attributes

Discussed in https://github.com/scalar/scalar/discussions/1454

Originally posted by kovapatrik April 18, 2024
When a schema has a reference to another, it shows in the API reference as Show Child Attributes. But if I have a oneOf or allOf field like this, it would be nice to know which schema is which.

That’s such a great idea!


fix(logo-grid): remove links

Related Ticket(s)
#4549
Description
remove links from logo
Changelog
Removed

a wrapper
href props
:hover styles
documentation updates

Thanks @ariellalgilmore for fixing this! It seems that this is also present in the Web Components version of this (#4425). Maybe it would be good if both issues could be addressed in this PR?

Thanks @ariellalgilmore for fixing this! It seems that this is also present in the Web Components version of this (#4425). Maybe it would be good if both issues could be addressed in this PR?

oh yeah @RaphaelAmadeu actually fixed the web component version already in this PR


Reset Password

Summary

Fixes #189
Test plan

Screenshots

Checklist

[ ] My code follows the code style of this project and passes make format.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate unit test coverage.

@ankushbhardwxj how is it going with this? Need any assistance?

hey @homanp, didn't get time to get to it this week. Planning to wind this up in the weekend. Will ping if help needed. Thanks !


updated code structure with with neat file system and provider logger

Hi Team,
I observed that the written code style looks very disturbed since it has different file for each method. To achieve readability and code maintainability its important to have it properly structured. I have done this proper code structure in this repo, maintained proper exception handing terminology and added logger. Feel free to merge.
Thanks,
Prasad, Coder

Hi Team,
I observed that the written code style looks very disturbed since it has different file for each method. To achieve readability and code maintainability its important to have it properly structured. I have done this proper code structure in this repo, maintained proper exception handing terminology and added logger. Feel free to merge.
Thanks, Prasad, Coder

Hi @Prasadpoojary ,
Thanks for taking the initiative to improve the code structure. I appreciate your effort in enhancing readability and maintainability. However, after reviewing the changes, I noticed that the modifications made do not align with the SOLID principles, which are crucial for our codebase.
In order to ensure the long-term scalability and flexibility of our software, it's essential to adhere to SOLID principles. I recommend revisiting the code adjustments with these principles in mind. Once the changes align with SOLID principles, we can consider merging them into the main codebase.
Feel free to reach out if you have any questions or if you'd like further clarification on SOLID principles.
Thanks


Kernel boot logo

Steps to reproduce issue
Connect to the HDMI connected to a monitor
Boot the device
Expected result
The tux (linux) logos should not be visible on the screen
Other information
Reported by: Florin Sarbu
Test configuration: VIA VAB820
Test case number: TC08
Test run: https://resinio.testlodge.com/projects/16238/runs/230391?tab=2&run_section_id=253458&executed_case_id=10655800#executed_case_10655800

No further development is planned for this board.


Correctly parse cmdclass in setup.cfg. Fixes #2570

Summary of changes

Closes #2570
Note: I don't know if resolve_class function is a duplicate, don't know setuptools code base in deep. Also, I can't say which approach is the correct one between having cmdclass as property of options section or having a options.cmdclass section.
Pull Request Checklist

[x] Changes have tests
[x] News fragment added in changelog.d/.
(See documentation for details)

Note: I don't know if resolve_class function is a duplicate, don't know setuptools code base in deep. Also, I can't say which approach is the correct one between having cmdclass as property of options section or having a options.cmdclass section.


Deadlock in agent state processing

This was observed on a test orchestrator
2022-09-16 03:53:22.259 UTC [25779] DETAIL:  Process 25779 waits for ShareLock on transaction 189497482; blocked by process 25459.
        Process 25459 waits for ShareLock on transaction 189497484; blocked by process 25779.
        Process 25779: DELETE FROM agent WHERE environment=$1
        Process 25459: SELECT  *  FROM agent WHERE environment=$1 AND name=$2 FOR UPDATE

This is probably the environment.clear conflicting with the agent state update
# rpm -qi inmanta-service-orchestrator
Name        : inmanta-service-orchestrator
Version     : 4.4.5
Release     : 0.dev20220904021800.el7
Architecture: x86_64
Install Date: Mon 05 Sep 2022 12:00:11 AM UTC
Group       : Development/Languages
Size        : 73886834
License     : EULA
Signature   : RSA/SHA256, Sun 04 Sep 2022 02:32:35 AM UTC, Key ID 6267cd49f0157b1b
Source RPM  : inmanta-service-orchestrator-4.4.5-0.dev20220904021800.el7.src.rpm
Build Date  : Sun 04 Sep 2022 02:32:14 AM UTC
Build Host  : 173035ae59d5450c9ae8fdd020987036
Relocations : (not relocatable)
URL         : http://inmanta.com
Summary     : Inmanta Service Orchestrator
Description :

This may or may not have been solved with #4889

Lets close it then.


Better npm audit

Non duplicated report of the same vulnerability
No more use better-npm-audit
Add possibility to ignore CWE
Error on ignoring not present CVE

Example of current report: https://github.com/camptocamp/ngeo/runs/1598394702?check_suite_focus=true


GitHubSource pods fails to run: chdir to /home/nonroot permission denied

Describe the bug
I want to install and run GitHubSource on my cluster (it's OCP 4.6).  Both the controller manager and webhook pods fail to run with following error:
Error: container create failed: time="2021-03-10T17:05:04Z" level=error msg="container_linux.go:366: starting container process caused: chdir to cwd (\"/home/nonroot\") set in config.json failed: permission denied"

I worked around that by adding the respective service accounts to the anyuid scc, but the GitHubSource pod itself in my application namespace has same problem and it's running under the default service account.
Expected behavior
GitHubSource pods can run without special privileges
To Reproduce
Install GitHubSource to OCP cluster
Knative release version
0.19.0
Additional context
I have played with older releases of GitHubSource in the past and this was not an issue.

@makandre Can you try adding this to the operator and webhook cluster roles:
 apiGroups:
     - security.openshift.io
     resources:
     - securitycontextconstraints
     resourceNames:
     - anyuid
     verbs:
     - use

@matzew @lberk Do you have the same issue for the eventing controllers and webhook?

/cc @zroubalik

@makandre Can you try adding this to the operator and webhook cluster roles:
 apiGroups:
     - security.openshift.io
     resources:
     - securitycontextconstraints
     resourceNames:
     - anyuid
     verbs:
     - use

AND add this to the Deployment Spec:
   securityContext:
        runAsUser: 65532

see: https://github.com/knative-sandbox/eventing-autoscaler-keda/pull/104

@lionelvillard After adding the rules the respective cluster roles the controller and webhook pods in knative-sources namespace both ran.  However the pod that's in my application namespace still has same problem.
@zroubalik I'm not sure to which deployment spec to add runAsUser; and both controller and webhook already ran fine after adding the cluster role rules.

Hi @lionelvillard, @zroubalik, wondering what is the status of this issue?  As mentioned above the controller and webhook pods now run with the suggested fix, but the pod in my namespace still has the same problem.

I tried a new install of this and actually don't see the problem anymore, thanks @lionelvillard @zroubalik!

This is kind of odd... I have one cluster where this continues to be a problem and workarounds needed, but on another cluster it was fine. I'm not really sure what's different

This regression was introduced in a certain version of OpenShift 4.7, you should check that both of your cluster are the same version.


Add to git-credential-libsecret

Maybe, we can use secretsd to support  git-credential-libsecret on every Linux including CentOS
gnome-keyring-daemon and kwalletd are not ok for CentOS .

There's nothing to add. git-credential-libsecret doesn't have a whitelist; it'll use whatever backend is running.


AVRO-3484: Followup Check default json parsing at compile time for derive macro

Checking the json before using in the macro with dummy command.

Thank you, @jklamer !
Do you think we can improve the compilation error report somehow ?
For example, if I break the default value for array field with:
diff --git lang/rust/avro_derive/tests/derive.rs lang/rust/avro_derive/tests/derive.rs
index 0886156ed..4199144c0 100644
--- lang/rust/avro_derive/tests/derive.rs
+++ lang/rust/avro_derive/tests/derive.rs
@@ -1240,7 +1240,7 @@ mod test_derive {
             #[avro(default = r#"{"a": 1, "b": 2}"#)]
             map: HashMap<String, i32>,

-            #[avro(default = "[1, 2, 3]")]
+            #[avro(default = "[1, 2, 3")]
             array: Vec<i32>,

then the error is:

error: Invalid avro default json:
       EOF while parsing a list at line 1 column 8
    --> avro_derive/tests/derive.rs:1244:13
     |
1244 |             array: Vec,
     |             ^^^^^
error[E0599]: the function or associated item get_schema exists for struct TestBasicStructWithDefaultValues, but its trait bounds were not satisfied
--> avro_derive/tests/derive.rs:1307:88
|
1231 |         struct TestBasicStructWithDefaultValues {
|         ---------------------------------------
|         |
|         function or associated item get_schema not found for this
|         doesn't satisfy _: apache_avro::schema::AvroSchema
|         doesn't satisfy _: apache_avro::schema::derive::AvroSchemaComponent
...
1307 |         if let Schema::Record { name, fields, .. } = TestBasicStructWithDefaultValues::get_schema()
|                                                                                ^^^^^^^^^^ function or associated item cannot be called on TestBasicStructWithDefaultValues due to unsatisfied trait bounds
|
= note: the following trait bounds were not satisfied:
TestBasicStructWithDefaultValues: apache_avro::schema::derive::AvroSchemaComponent
which is required by TestBasicStructWithDefaultValues: apache_avro::schema::AvroSchema
&TestBasicStructWithDefaultValues: apache_avro::schema::derive::AvroSchemaComponent
which is required by &TestBasicStructWithDefaultValues: apache_avro::schema::AvroSchema
&mut TestBasicStructWithDefaultValues: apache_avro::schema::derive::AvroSchemaComponent
which is required by &mut TestBasicStructWithDefaultValues: apache_avro::schema::AvroSchema
note: the following trait must be implemented
--> /home/martin/git/apache/avro/lang/rust/avro/src/schema.rs:1589:5
|
1589 | /     pub trait AvroSchemaComponent {
1590 | |         fn get_schema_in_ctxt(named_schemas: &mut Names, enclosing_namespace: &Namespace)
1591 | |             -> Schema;
1592 | |     }
| |_____^
error[E0599]: the function or associated item get_schema exists for struct TestBasicStructWithDefaultValues, but its trait bounds were not satisfied
--> avro_derive/tests/derive.rs:1335:62
|
1231 |         struct TestBasicStructWithDefaultValues {
|         ---------------------------------------
|         |
|         function or associated item get_schema not found for this
|         doesn't satisfy _: apache_avro::schema::AvroSchema
|         doesn't satisfy _: apache_avro::schema::derive::AvroSchemaComponent
...
1335 |         assert_eq!(schema, TestBasicStructWithDefaultValues::get_schema());
|                                                              ^^^^^^^^^^ function or associated item cannot be called on TestBasicStructWithDefaultValues due to unsatisfied trait bounds
|
= note: the following trait bounds were not satisfied:
TestBasicStructWithDefaultValues: apache_avro::schema::derive::AvroSchemaComponent
which is required by TestBasicStructWithDefaultValues: apache_avro::schema::AvroSchema
&TestBasicStructWithDefaultValues: apache_avro::schema::derive::AvroSchemaComponent
which is required by &TestBasicStructWithDefaultValues: apache_avro::schema::AvroSchema
&mut TestBasicStructWithDefaultValues: apache_avro::schema::derive::AvroSchemaComponent
which is required by &mut TestBasicStructWithDefaultValues: apache_avro::schema::AvroSchema
note: the following trait must be implemented
--> /home/martin/git/apache/avro/lang/rust/avro/src/schema.rs:1589:5
|
1589 | /     pub trait AvroSchemaComponent {
1590 | |         fn get_schema_in_ctxt(named_schemas: &mut Names, enclosing_namespace: &Namespace)
1591 | |             -> Schema;
1592 | |     }
| |_____^
error[E0277]: the trait bound TestBasicStructWithDefaultValues: apache_avro::schema::derive::AvroSchemaComponent is not satisfied
--> avro_derive/tests/derive.rs:1337:22
|
1337 |           serde_assert(TestBasicStructWithDefaultValues {
|  ------------^
| |         |
| |         required by a bound introduced by this call
1338 | |             a: 321,
1339 | |             b: "A custom value for 'b'".to_owned(),
1340 | |             condition: false,
...    |
1347 | |             myenum: MyEnum::Bar,
1348 | |         });
| |_^ the trait apache_avro::schema::derive::AvroSchemaComponent is not implemented for TestBasicStructWithDefaultValues
|
= note: required because of the requirements on the impl of apache_avro::schema::AvroSchema for TestBasicStructWithDefaultValues
note: required by a bound in serde_assert
--> avro_derive/tests/derive.rs:43:61
|
41   |     fn serde_assert(obj: T)
|        ------------ required by a bound in this
42   |     where
43   |         T: std::fmt::Debug + Serialize + DeserializeOwned + AvroSchema + Clone + PartialEq,
|                                                             ^^^^^^^^^^ required by this bound in serde_assert
error[E0283]: type annotations needed
--> avro_derive/tests/derive.rs:1319:48
|
1319 |                     "array" => assert_eq!(Some(json!([1, 2, 3])), field.default),
|                                                ^^^^^^^^^^^^^^^^ cannot infer type for type {integer}
|
= note: multiple impls satisfying {integer}: serde::Serialize found in the serde crate:
- impl serde::Serialize for i128;
- impl serde::Serialize for i16;
- impl serde::Serialize for i32;
- impl serde::Serialize for i64;
and 8 more
= note: required because of the requirements on the impl of serde::Serialize for &{integer}
note: required by a bound in serde_json::to_value
--> /home/martin/.cargo/registry/src/github.com-1ecc6299db9ec823/serde_json-1.0.79/src/value/mod.rs:944:8
|
944  |     T: Serialize,
|        ^^^^^^^^^ required by this bound in serde_json::to_value
= note: this error originates in the macro json_internal (in Nightly builds, run with -Z macro-backtrace for more info)
error[E0283]: type annotations needed
--> avro_derive/tests/derive.rs:1321:30
|
1321 |                           Some(json!({
|  ____^
1322 | |                             "a": 1,
1323 | |                             "b": 2
1324 | |                         })),
| |^ cannot infer type for type {integer}
|
= note: multiple impls satisfying {integer}: serde::Serialize found in the serde crate:
- impl serde::Serialize for i128;
- impl serde::Serialize for i16;
- impl serde::Serialize for i32;
- impl serde::Serialize for i64;
and 8 more
= note: required because of the requirements on the impl of serde::Serialize for &{integer}
note: required by a bound in serde_json::to_value
--> /home/martin/.cargo/registry/src/github.com-1ecc6299db9ec823/serde_json-1.0.79/src/value/mod.rs:944:8
|
944  |     T: Serialize,
|        ^^^^^^^^^ required by this bound in serde_json::to_value
= note: this error originates in the macro json_internal (in Nightly builds, run with -Z macro-backtrace for more info)
Some errors have detailed explanations: E0277, E0283, E0599.
For more information about an error, try rustc --explain E0277.
error: could not compile apache-avro-derive due to 9 previous errors
warning: build failed, waiting for other jobs to finish...
error: build failed

The first error is the expected one, but then it also logs a lot more that is not really helpful and is actually hiding the real problem.

@martin-g Im not sure. Might be a weakness of macros in general. The cascading failure of the macro compiling might be worth calling out in docs? But I'm not sure of other solutions


GROUP BY CUBE generates empty results when input is empty

Bug description
GROUP BY CUBE should always generate at least one row.
assertQuery("SELECT orderkey FROM orders WHERE orderkey < 0 GROUP BY CUBE (orderkey)");

java.lang.AssertionError: For query:
 SELECT orderkey FROM orders WHERE orderkey < 0 GROUP BY CUBE (orderkey)
not equal
Actual rows (up to 100 of 0 extra rows shown, 0 rows in total):

Expected rows (up to 100 of 1 missing rows shown, 1 rows in total):
    [null]

CC: @zacw7 @aditi-pandit
System information
n/a
Relevant logs
No response

In Java, HashAggregationOperator has an extra flag (produceDefaultOutput) and a list of group IDs for empty grouping sets. This triggers logic to produce extra output when there is no input. HashAggregation operator in Velox doesn't have this logic and AggregationNode in Velox is missing fields to trigger this logic.
    private final List<Integer> globalAggregationGroupIds;
    private final boolean produceDefaultOutput;

if (!inputProcessed && produceDefaultOutput) {
    // global aggregations always generate an output row with the default aggregation output (e.g. 0 for COUNT, NULL for SUM)
    finished = true;
    return getGlobalAggregationOutput();
}

This flag is set based on whether there are "empty" grouping sets (which is the case for CUBE aggregations).
public boolean hasDefaultOutput()
{
    return hasEmptyGroupingSet() && (step.isOutputPartial() || step.equals(SINGLE));
}

@mbasmanova Interestingly we too had an issue for this, and I had reached the same conclusion as well which I was going to discuss with you.
https://github.com/prestodb/presto/issues/20911
Failing SQL
SELECT SUM(CAST(quantity AS BIGINT)) FROM lineitem WHERE quantity < 0 GROUP BY GROUPING SETS ((), ())
The HashAgg operator was handling it with special code https://github.com/prestodb/presto/blob/master/presto-main/src/main/java/com/facebook/presto/operator/HashAggregationOperator.java#L622 which we don't have in C++.

Somewhat related: https://github.com/prestodb/presto/pull/7239
The implementation is AggregationNode::hasDefaultOutput seems unnecessarily verbose.
    public boolean hasDefaultOutput()
    {
        return hasEmptyGroupingSet() && (step.isOutputPartial() || step.equals(SINGLE));
    }

It can be simplified to
    public boolean hasDefaultOutput()
    {
        return hasEmptyGroupingSet() && !step.equals(FINAL));
    }

Fixed by
https://github.com/facebookincubator/velox/pull/7112
https://github.com/facebookincubator/velox/pull/7353


Hub cluster HA solution proposal

The scenario encountered in production environment is that:

Hub cluster hc1 managed some managed clusters and many manifest works also many addons are deployed.
Hub cluster hc2 was installed as a backup.
Hub cluster encountered a critical error that could not recover for a short time.
We must join the managed clusters to Hub cluster hc2 and recovered OCM resources in Hub cluster hc2

Hub cluster HA solution proposal architecture is shown below:

Of course, OCM resources such as manifest works deployed in managed clusters  should save meta data in annotaions or somewhere else.
Whether this solution is feasible ?

In this case we do not need to unjoin and join again. If we change the bootstrapKubconfig secret, then klusterlet will connect to the second hub. However, we do not have a clustercmd command to do that yet. Maybe something like
clusteradm rejoin --hub-apiserver second-hub --token xxx

And it just update the bootstrap kubeconfig secret.
/kind feature

In this case we do not need to unjoin and join again. If we change the bootstrapKubconfig secret, then klusterlet will connect to the second hub. However, we do not have a clustercmd command to do that yet. Maybe something like
clusteradm rejoin --hub-apiserver second-hub --token xxx

And it just update the bootstrap kubeconfig secret.
/kind feature

Ok, it can achieve the same goal. And what i am concerned about is recovery of resources such as manifest work.

Right, I think we could create a solution scenario in this repo by using Velero as an example. WDYT?


ISA Extensions

Hi,
I have a question regarding ISA extension group G (IMFAD).
Will there be support for remainder ?
If not what is the suggested practice for adding support for them ?
Thank you,

Yes, there's plan to support the FAD extension. Here's preferred the way to do it:

Create a development branch for each ISA extensions
File a bug for each ISA extensions

List all the features required to support this new ISA, collaborators can participate in the development by adding feature requirements, review PR, implement features etc.

Finish all development work under development branch before merging to the mainline.

I will go ahead to create a development branch for A extension first. Regarding the complexity , I think A,D are relatively easy, may take a couple of weeks to implement. F extension will take more time, presumably a month or two.


POSTing results in error

It appears as though POSTing to the API(s) is now resulting in errors. Using the GET method results in no such error.
library(httr)

endpoint <- 'https://www.patentsview.org/api/patents/query'
body <- '{"q":{"_gte":{"patent_date":"2007-01-04"}},"f":["patent_number","patent_date"]}'

response <- POST(endpoint, body = body)
content(response, "text")
#> [1] "{\"status\":\"error\",\"payload\":{\"error\":\"'q' param is missing\",\"code\":\"RQ1\"}}"

Created on 2020-09-09 by the reprex package (v0.3.0)
Related to https://github.com/ropensci/patentsview/issues/21#issuecomment-687624142

@crew102 & @mustberuss The server of the previous api does not suppot redirects where POST data is also redirected(only GET query paramters are redirected). In any case, the redirect is a failsafe for when people do not know about the change in URL. You should update the URL on all you applications as soon as you can.
Please re-open or create new issues if you are still facing issues.


Use browser package.json field if we have jsdomify in a file

Is it possible to somehow rewrite all require calls inside a scope of jsdomify to use browser field in NPM packages instead of using node runtimes?  Since it's jsdomify - it has window injected and therefore should be using bower/webpack/browser version of packages

It looks to me that you want to live in the browser, not in Node.js.
If that's the case you should probably use jsdom directly, as jsdomify is useful if you want to run your tests in Node.js


How the multiscreen app can be design

What would be the design of the shell and core for multi screen application ??

It's quite an open ended question, could you elaborate on what you mean? An app with multiple screens which can be navigated between, or an app which runs across multiple physical screens?
What's true for both is that the view model needs to capture what's going on screen in a way the UI can easily pick up and use - its structure should follow the sructure of the UI.
If you can share details of some specific challenges you're facing, it will be easier for us to help.

I am talking about navigation in the app. My app would have multiple screens so how will the navigation happen?
Core request to the ui using render capabilty that means core decides's what to show on the device screen.
I did not understand how the navigation work with rust crux.

Have a read through the Tap to pay example https://github.com/redbadger/crux/tree/master/examples/tap_to_pay, that does multi-screen navigation.
It uses SwiftUI to turn a basic description in the view model of what screen should be shown (there's an enum for the different screens) to the UI that should be on screen - like a NavigationStack a sheet and a fullScreenCover.


Theme dropdown: On small breakpoint, make sure status doesn't get cut off

Note: This is how it currently functions on production, but would be great to fix while we're changing the dropdown.
Current:

Ideally, none of the dropdowns would get cut off:

Screenshot from safari on iOS


When using gamepad input to select a menu option, next menu option also gets selected as if submit was performed twice

Describe the bug
I am using the new input system but I'd suspect this is the case with the old one also. When an option is selected by pressing submit on the gamepad, the next set of menus are displayed which happens so fast that the currently pressed submit is selecting the next appearing options. This issue only happens when selecting a menu option leads to more menu options being selected:

The above gif demonstrates gamepad input. Here is mouse input to demonstrate expected behaviour:

I have adjusted the Input System Package's timings;

and also the repeat delays on the InputSystemUIInputModule components to no avail (i've tried 2+ secs, no difference);

I believe this issue is to do with the next menu options appearing too fast; if I place a 1 second wait at the start of the next menu options, it no longer happens, however that should not be necessary to counter this issue. Replicated in builds as well as in editor.
To Reproduce
In a scene set up to work with the new input module, set up a flowchart with 2 sets of menus to appear after one another, without say dialogs inbetween. (see example in gif above)
Enter play mode and press submit. Observe the next set of options also being selected as if submit was pressed twice in rapid succession.
Expected behavior
Submit only ever selecting 1 menu option. Perhaps the solution is to add a timer similar to the SayEndDelay feature in #895 but for menu options, but I'm not sure if there's a way to apply this delay only to cases where a menu option leads to more menu options... so perhaps another kind of solution would be better.
I suppose the issue could also be approached from an input point of view, where after a new menu option appeared, the system would check whether the submit button is already down - making it so you would have to first release the submit button and press it again to select a newly appeared option.
Perhaps there is another angle here I'm missing, but any solution to this that wouldnt require me hunting down every case like this in our project and putting a Wait command in there would be super appreciated.
Versions & Platform (please complete the following information):

Unity version 2019.4.9f1
Fungus Version 3.13.3 (plus some changes from the develop branch)
Editor OS: Win8.1

I've since ditched the new input system, and I will re-test this with a controller soon and report back on it.

This is not an issue on the old input system anymore. Closing.

I'm not sure if this may the problem, but I did have the same issue where hitting an input button would result in the action being performed twice (or 3 times). This is a bug with how the input system was programmed with its events. I was able to fix it by adding a new interaction and assigning my buttons to it. This effectively simulates Input.GetButtonDown() for a single frame. You could see if setting your submit inputs to use this interaction helps.
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.InputSystem;
using System;
using UnityEditor;

#if UNITY_EDITOR
using UnityEngine.InputSystem.Editor;

[InitializeOnLoad]
#endif
public class KeyPress : IInputInteraction
{
    public void Process(ref InputInteractionContext context)
    {
        switch (context.phase)
        {
            case InputActionPhase.Waiting:
                context.Started();
                break;

            case InputActionPhase.Started:
                context.Waiting();
                break;
        }
    }

    public void Reset()
    {
    }

    static KeyPress()
    {
        InputSystem.RegisterInteraction<KeyPress>();
    }

    [RuntimeInitializeOnLoadMethod]
    private static void Initialize()
    {
        // Will execute the static constructor as a side effect.
    }

#if UNITY_EDITOR
    public class KeyPressEditor : InputParameterEditor<KeyPress>
    {
        public override void OnGUI()
        {
            EditorGUILayout.LabelField("This interaction treats this action as a button and can only be activated. Performed and cancelled steps are skipped.");
        }
    }
#endif

}

Thanks @Arylos07 I think my issue was fixed by using Rewired in the end and reverting back to the default input system. While I no longer need a solution for this myself, this may be useful for others who arent using Rewired!


Add ability to have map initial extents centred on user

There are situations where an atlas should be centred on a user's location. Add an map auto initial bounds based on the geolocation returned by the browser.
Add ability to adjust only the latitude or the longitude.

In map.json definition:
{
	...
	,"coordinates": {
		"initialBounds": [-140,50,-54,77]
		,"autoInitialBounds": {
			"type": "mapAutoInitialBoundsCentreOnUserPosition"
			,"adjustLatitude": false
			,"adjustLongitude": true
		}
	}
}

Recall that this is already implemented for arcticbm.org, so there may be potential to reuse some code.

On Feb 21, 2017, at 2:33 PM, Jean-Pierre Fiset<EMAIL_ADDRESS>wrote:
There are situations where an atlas should be centred on a user's location. Add an map auto initial bounds based on the geolocation returned by the browser.
Add ability to adjust only the latitude or the longitude.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/GCRC/nunaliit/issues/627, or mute the thread https://github.com/notifications/unsubscribe-auth/ACXeGLNxHdnPKnP20YLk-TtwK7o7CgiAks5rezv5gaJpZM4MHwt5.

Peter, you are correct.

Currently, this feature works with Firefox. It does not work with Chrome unless the atlas is served over HTTPS. In Chrome, an error is returned:
PositionError
message: "Only secure origins are allowed (see: https://goo.gl/Y0ZkNV)."


Data Types to store

Hi ,

Can i store data types of standard types like  — String, int, Map, List, using this package , using key value pairs ?
Please answer......

Nope. Just Strings.

I think the lib should be able to store the different data types, android already does it. Are there plans for that?


Auto Generate Whitelist Data Install Script

Rather than just doing this https://github.com/rhoerr/supee-6788-toolbox/blob/186ecb2cada4913c9e8850635c15a6bb780518ce/fixSUPEE6788.php#L705 it might be nice if this created a data install script for the necessary blocks and variables (perhaps optionally). That way you can deploy the upgrade script rather than running this in production. Magento "technical details" also recommend a data install script. I'm sure everyone is busy getting this patch out, but if anyone has time (I can maybe, but probably not PR this myself).

The tech details recommendation was in regards to affected modules, and in that context it does make sense. Creating a module, you'd want to make sure any blocks needed are added to the whitelist when it gets installed.
To do that, though, you need a module to attach the data upgrade to. Don't have that in this context. You could generate everything needed, for sure, but that's getting rather involved.

Yeah, it certainly wouldn't be trivial, but would be nice...even just to get the install script only would save some time and then it would be on the developer to create the module to make sure it gets executed...

created a PR to get the upgrade scripts from log file


Objectives tick themselves without any input from the player

Scenarios I have seen this happen:
Enabling grabless/jumpless/dashless after finishing a relevant objective.
Checking the board after finishing an all collectibles objective.

Bug seems to be relating to pressing enter
https://youtu.be/eLKAJsmsNkw
https://youtu.be/JKBQfukTGPU

should there be a different default binding?