slice bounds out of range on  --status localhost:6060

In the 1.3.0 docker image --status localhost:6060 results in a panic.
The docker command I run is:
server --listen <IP_ADDRESS>:27018 --target localhost:27017 --keystore /var/keys/outputCert.p12 --cacert /var/keys/CA.crt  --allow-all --status localhost:6060
 panic: runtime error: slice bounds out of range

goroutine 1 [running]:
github.com/square/ghostunnel/vendor/github.com/kavu/go_reuseport.getTCPSockaddr(0x9b5d5f, 0x3, 0x7ffe70ffd17c, 0xe, 0x9b5d5f, 0x3, 0x7ffe70ffd17c, 0xe, 0xa4af80)
	/go/src/github.com/square/ghostunnel/vendor/github.com/kavu/go_reuseport/tcp.go:43 +0x364
github.com/square/ghostunnel/vendor/github.com/kavu/go_reuseport.getSockaddr(0x9b5d5f, 0x3, 0x7ffe70ffd17c, 0xe, 0xc0001f9a10, 0x8911e3, 0x9b5d5f, 0x3, 0x7ffe70ffd17c)
	/go/src/github.com/square/ghostunnel/vendor/github.com/kavu/go_reuseport/reuseport.go:30 +0x72
github.com/square/ghostunnel/vendor/github.com/kavu/go_reuseport.NewReusablePortListener(0x9b5d5f, 0x3, 0x7ffe70ffd17c, 0xe, 0x7ffe70ffd17c, 0xe, 0x7ffe70ffd17c, 0x9)
	/go/src/github.com/square/ghostunnel/vendor/github.com/kavu/go_reuseport/tcp.go:99 +0x67
main.(*Context).serveStatus(0xc000068100, 0xc00000c800, 0x2540be400)
	/go/src/github.com/square/ghostunnel/main.go:547 +0x228
main.serverListen(0xc000068100, 0x9de010, 0xc000068100)
	/go/src/github.com/square/ghostunnel/main.go:440 +0x703
main.run(0xc0000a0010, 0xd, 0xd, 0xc0001a8a50, 0xc000076058)
	/go/src/github.com/square/ghostunnel/main.go:352 +0x1348
main.main()
	/go/src/github.com/square/ghostunnel/main.go:272 +0x66

Also thank you for your work on this tool.

That's certainly odd.
Unfortunately I wasn't able to reproduce the issue on my system, but I believe what's happening is:

parseUnixOrTCPAddress in main.go resolves "localhost:6060" to ("tcp", "<IP_ADDRESS>", "localhost", nil)
NewReuseablePortListener is called with ("tcp", "<IP_ADDRESS>") as arguments
getTCPSockaddr in kavu/go_reuseport/tcp.go calls net.ResolveTCPAddr returns a 4-byte IP addr
determineTCPProto maps this to "tcp4" since ip.IP.To4() determines that this fits into an IPv4
kavu/go_reuseport/tcp.go:43 (falsely) assumes that the IP must be 16-bytes, leading to a panic

I think the confusion comes from the fact that net.ResolveTCPAddr can return a 4-byte and 16-byte slice for the IP, but a 16-byte slice can still represent an IPv4. Do you happen to have IPv6 disabled on your system by any chance?

Thanks for the amazingly quick patch. This does indeed fix the issue.
FYI I was running all of this via docker on box a gke k8 cluster and locally on my mac.

Fix has been released in version 1.3.1, see https://github.com/square/ghostunnel/releases and https://hub.docker.com/r/squareup/ghostunnel/tags

Thank you!


Add command to insert into FSTAB

Hello, team.  wondering if we want to add a command to this page to help them insert their UUID entry into the FSTAB file.  just to be a little more convenient for users.  Perhaps something like the commands below?  Just a thought.  Thanks.
#Automatically gets the UUID and inserts the rest of specifications into FSTAB file
echo " $(blkid | grep /dev/sdc1 | awk '{print $2}') /datadrive xfs defaults,nofail 1 2" >> /etc/fstab
#Automatically gets the UUID and the file system type and inserts the rest of specifications into FSTAB file
#I've seen people accidentally specify the wrong file system type before, which can cause a boot issue if the nofail option isn't included.  This command just gets it for them and inserts.
echo " $(blkid | grep /dev/sdc1 | awk '{print $2}') /datadrive $(df -hT | grep /dev/sdc1 | awk '{print $2}') defaults,nofail 1 2" >> /etc/fstab

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

ID: 6fde1939-787d-2aa5-4b29-d303f8ed49cb
Version Independent ID: c2838cb5-c34c-e0ff-b280-b86f29453cad
Content: Attach a data disk to a Linux VM - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/attach-disk-portal.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @cynthn
Microsoft Alias: cynthn

@mbiver Thank you for the feedback!  I've assigned this to the document author for further review.

#reassign:mattmcinnes

#reassign:roygara

Thank you for your dedication to our documentation.
We sincerely apologize for the delayed response. After a careful review, we are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close

Thanks for your dedication to our documentation.
We sincerely apologize for the delayed response. The requested updates have not been made since the creation of this issue, and the timeline for resolution may vary due to various constraints. Therefore, we've created an internal work item to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close


APIView - ability to rebase (reupload first revision)

While working on long running branch (Storage STG73 branch to be precise) we ended up in situation where:

lot of feedback has been accumulated and we didn't want to lose it
some changes in master got released in the meanwhile and APIViews became not 100% accurate confusing some reviewers a little bit

Today we can either keep accumulating feedback and let review become slightly inaccurate OR create brand new based on fresh master.
Could we have some solution that would let us change the base with fresh master but retain historical feedback so that we don't have to cross reference multiple APIViews if same question is being asked?
FYI @kurtzeborn

This should no longer be an issue since APIView comments are persisted across different revisions of the same package


Fix typo in readme

Fix typo in readme (development -> development).

Thanks for the merge @Tyriar!


Year setter typo

I've noticed that there's a typo in the year setter, preventing the ability to customize the format for the year (especially when using the datetime format) - it affects the day format instead.
One test (unrelated) also expected its output to omit the date because of the default behavior when the current year is the same as the date's year, so the second commit here addresses that failing test.

:wave: Hello and thanks for pinging us! This issue or PR has been added to our inbox and a Design Infrastructure first responder will review it soon.

:art: If this is a PR that includes a visual change, please make sure to add screenshots in the description or deploy this code to a lab machine with instructions for how to test.
:fast_forward: If this is a PR that includes changes to an interaction, please include a video recording in the description.
:warning: If this is urgent, please visit us in #primer on Slack and tag the first responders listed in the channel topic.


ValueError: mutable default <class 'fairseq.dataclass.configs.CommonConfig'> for field common is not allowed: use default_factory

Environment

OS: Ventura 13.3
Python Version: Python 3.9.6 (default, May  7 2023, 23:32:44)
Hardware: Apple Silicon M2

When running sh ./go-applio.sh and then selecting [1] NVDIDIA/AMD this happens:
Issue Description:
Traceback (most recent call last):
File "/Users/alvarom2/Documents/RVC/applio/Applio-RVC-Fork/infer-web.py", line 26, in
import fairseq
File "/Users/alvarom2/Documents/RVC/applio/Applio-RVC-Fork/.venv/lib/python3.11/site-packages/fairseq/init.py", line 20, in
from fairseq.distributed import utils as distributed_utils
File "/Users/alvarom2/Documents/RVC/applio/Applio-RVC-Fork/.venv/lib/python3.11/site-packages/fairseq/distributed/init.py", line 7, in
from .fully_sharded_data_parallel import (
File "/Users/alvarom2/Documents/RVC/applio/Applio-RVC-Fork/.venv/lib/python3.11/site-packages/fairseq/distributed/fully_sharded_data_parallel.py", line 10, in
from fairseq.dataclass.configs import DistributedTrainingConfig
File "/Users/alvarom2/Documents/RVC/applio/Applio-RVC-Fork/.venv/lib/python3.11/site-packages/fairseq/dataclass/init.py", line 6, in
from .configs import FairseqDataclass
File "/Users/alvarom2/Documents/RVC/applio/Applio-RVC-Fork/.venv/lib/python3.11/site-packages/fairseq/dataclass/configs.py", line 1104, in
@dataclass
^^^^^^^^^
File<EMAIL_ADDRESS>line 1230, in dataclass
return wrap(cls)
^^^^^^^^^
File<EMAIL_ADDRESS>line 1220, in wrap
return _process_class(cls, init, repr, eq, order, unsafe_hash,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File<EMAIL_ADDRESS>line 958, in _process_class
cls_fields.append(_get_field(cls, name, type, kw_only))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File<EMAIL_ADDRESS>line 815, in _get_field
raise ValueError(f'mutable default {type(f.default)} for field '
ValueError: mutable default <class 'fairseq.dataclass.configs.CommonConfig'> for field common is not allowed: use default_factory
Press Enter to continue...

We do not have MacOS equipment so we cannot fully determine a solution for your error.

We do not have MacOS equipment so we cannot fully determine a solution for your error.

yeah i figured, still wanted to post it though. Might be someone who found a fix for it.

Downgrade to 3.9

Downgrade to 3.9

do you have the command on hand? ain't familiar with venv


Add types to entity synonyms

Description of Problem:
I don't see a way to scope synonyms by type. This is important in making calls to knowledge bases that are typed.
For instance, let's say I know how to recognize two entities types, date.month and finance.term. To make sure the extracted entity values match the way I need to look those things up in a knowledge base, I use an EntitySynonymMapper.
But the following example shows the current shortcoming:
"The most recent report was from APR 2019"
->
{
   "end": ...,
   "entity": "date.month",
   "start": ...,
  "value": "April"
}

But then we'd want
"The APR on that option is 4.3%"
->
{
   "end": ...,
   "entity": "finance.term",
   "start": ...,
  "value": "AppliedPercentageRate"
}

I don't think this is possible with the existing implementation, which would map "APR" to a single synonym, regardless of the type of the entity.
Overview of the Solution:
Allow the EntitySynonymMapper to be type aware. In "entity_synonyms" we could add in types.
Examples (if relevant):
"entity_synonyms": [
      {
        "value": "April",
        "types": ["date.month"]
        "synonyms": ["APR", "04", "april"]
      },
      {
        "value": "AppliedPercentageRate",
        "types": ["finance.term"]
        "synonyms": ["APR", "applied percentage rate"]
      },

Definition of Done:

EntitySynonymMapper can enforce optional type checking
A single surface form like "APR" can map to separate synonyms based on extracted entity type.

I imagine this as being an optional, additional entry. So if you had an entry in your synonyms that didn't have a type on it, it would still work. And if you had a typed entry and an untyped entry in your synonyms, the untyped would catch all the types you didn't have explicit entries for.

Perhaps a more universal example is locations (in the U.S., states) colliding with short utterances
"OK" + "location.statecode" -> "Oklahoma"
vs
"OK" + "confirmation" -> "Okay"

Thanks for submitting this feature request 🚀 @degiz will get back to you about it soon! ✨

Any thoughts on this? If it's desired functionality I could probably get a basic PR for it, but I don't want to implement it if it's unlikely to be supported

@degiz any thoughts?

is this what you need? https://forum.rasa.com/t/introducing-entity-roles-and-groups/28156

@amn41 I don't think it quite solves the problem (though it's very cool functionality, I was excited when I saw it in the release notes), but correct me if I'm wrong.
I think that even if I had two separate roles for my entities, the EntitySynonymMapper would still synonym-ize them the same way.

yes, you're quite right that it would. But you could build a custom synonym mapper component to do that part

Gotcha yeah. I was just thinking it would be cool to have that as an optional feature of the default EntitySynonymMapper instead of writing a custom component.
I think it would be pretty easy to keep the same basic functionality by default, and just let the synonym mapper first check to see if the type of your entity matches the type of the synonym entry. If there's no type on it, it will process the example. If there is a type and it doesn't match, it'll leave the example alone.


IE11 support fix

I've ran the tests in IE11 and everything works fine (you can run npm run test-browser and open test/runner.html in your web-browser).
Are you sure that problem happens with IE version 11 and not the older ones? In the latter case those browsers are no longer supported by their vendors and we don't aim to be compatible with them.
As a side note you shouldn't modify files in the dist/ directory, they are only changed on version bump

Ok, I'll try to figure out later why it's working diferently in test environment.
For now I released v0.1.3 on bower.

Thanks.


Css output filter

Issue description:
I want to change the css background for my header only on one page on others leave it as it is from main options
Version used:
latest from wp.org
Using theme_mods or options?
theme_mods
How can I get this result ? I also want to get the answer asap please

/**
 * Filtering Kirki 4 output arguments.
 *
 * @param array $current_output The generated output.
 * @param mixed $value The value.
 * @param array $kirki_output The Kirki output set on field.
 * @param array $kirki_field The Kirki field.
 *
 * @return array
 */
function sample_filter_output_args( $current_output, $value, $kirki_output, $kirki_field ) {

  // modify the $current_output here

  return $current_output;

}

add_filter( 'kirki_output_item_args', 'sample_filter_output_args', 10, 4 );

This should do it in the upcoming version 4. Props @contactjavas for the code-snippet :)


Switch to the community operator version

The current image of kube-rbac-proxy is not multi-arch. Switch to a version that is.

fixed by #105


Faster Double-Precision Math

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


Add type to runReadQuery RPC response columns contained object

User Story
A simple read query returns the columns and rows for a particular table. However, for each column, it is not clear what the type on the column is. Namely, if I create a column of id, there's no way to directly know what this type should be without some specific SQL knowledge.
As an example, I would have to run select sql from sqlite_schema where name='rest_api_5_85' (see query here), which returns a row of CREATE TABLE rest_api_5_85 (id INT) STRICT. Then, I'd have to parse that response etc. to get the actual type.
Requirements
Add type to each object returned in the runReadQuery RPC response's columns array:
{
  "columns": [
    {
      "name": "id",
      "type": "int"
    }
  ],
  "rows": [
    [
      0
    ]
  ]
}

hey @dtbuchholz Exploring this a bit.
This is something folks discussed in the past, but don't quite remember the outcomes of those conversations. I agree that having a type is handy in a lot of situations.
Can you elaborate more on what is trying to be accomplished with that new field and how it helps? Just trying to get the full picture and see if we are not missing any other potential solution.

Just want to mention another nuance about this. There are two cases to be differentiated:

the type of the columns of a CREATE statement
the type of a SELECT column result

If we do SELECT * FROM rest_api_5_85, it's obvious it should return type int for id because CREATE TABLE rest_api_5_85 (id INT). That's case 1.
However, what if I do SELECT count(1) FROM rest_api_5_85? That's case 2. count(1) is not a column with a type, it's calculated on the fly. What type should it be? It seems reasonable to return int for that as well.
And what if we have SELECT 3.1415 FROM rest_api_5_85? What type is 3.1415? Should it be Go's float, JavaScript's number, or SQLite's real?
We also work with JSON. For example, SELECT json_object(...) FROM ... that returns a JSON and not a string (because of the NFT use case). Should we create the type json for that SELECT result?

If we want a response like:
{
  "columns": [
    {
      "name": "id",
      "type": "int"
    }
  ],
  "rows": [
    [
      0
    ]
  ]
}

makes sense to treat everything as case 2 and only care about the type of SELECT column result (and not the type defined in the CREATE statement). That's a bit more complicated because we have to infer the type of things like count(1), 3.1415, json_obect(...), or any other arbitrary expr. And possibly creates new types such as json. That means we'd have two different kinds of types: the ones from a CREATE statement (text, int, any, blob, ...) and the types of a SELECT column result.
On the other hand, if we just want to provide a way for users to discover the types of the columns of a table, we can create an endpoint that returns the schema of a table.


Add pprof support

Add pprof support. Expose it only to localhost, so you have to port-forward to get access to it.

Lower priority - really only will matter unless we identify a memory leak or are looking to improve the performance of Ark for large clusters.  Refs #94


\dot{x} dot is too small

In my opinion, the dot produced by \dot{x} is too small. Is there a way to make it bolder? Perhaps it would be possible to add a class to the dot element during rendering, so that it could be styled using CSS.

Fixed by 3cd85d9. Dot was being set in superscript size.


Increase node wait timeout to 5mins to allow for Cloud provisioned VMs to come online

Orka nodes can take over a minute or so to come online when none are currently online, this could apply to any Jenkins Cloud provisioned node request.
Extending timeout to 5mins
Fixes https://github.com/adoptium/ci-jenkins-pipelines/issues/900

run tests


xr.open_dataset(... decode_coords='all') fails for certain CF conform grid_mapping values

What happened?
Reading of CF-conform projection information and converting it automatically to coordinates with the decode_coords='all' argument of xr.open_dataset() failed for certain dataset.
What did you expect to happen?
To do the detection of coordinates, xarray relies on the grid_mapping attribute that is set per variable. The CF-conventions allow for various different grid_mapping formats:

as a single variable name (e.g. Example 5.6):variables:
  float T(lev,rlat,rlon) ;
    T:coordinates = "lon lat" ;
    T:grid_mapping = "rotated_pole" ;

as mapping (e.g. example 5.12):variables:
   float temp(y, x) ;
     temp:coordinates = "lat lon" ;
     temp:grid_mapping = "crs: x y" ;

as a combination of both (e.g. example 5.13):variables:
  float temp(y, x) ;
    temp:coordinates = "lat lon" ;
    temp:grid_mapping = "crs_osgb: x y crs_wgs84: latitude longitude" ;

I expect that all formats are supported.
Minimal Complete Verifiable Example
import xarray as xr
import numpy as np

dimensions = {
    "lat": 648,
    "lon": 648,
    "y": 18,
    "x": 36
}

x_coords = np.arange(dimensions["x"])
y_coords = np.arange(dimensions["y"])

temp_data = np.random.rand(dimensions["y"], dimensions["x"])

# WORKS
ds = xr.Dataset(
    {
        "temp": (("y", "x"), temp_data, {
            "long_name": "temperature",
            "units": "K",
            "coordinates": "lat lon",
            "grid_mapping": "crs"
        }),
        "x": (("x"), x_coords, {
            "standard_name": "projection_x_coordinate",
            "units": "m"
        }),
        "y": (("y"), y_coords, {
            "standard_name": "projection_y_coordinate",
            "units": "m"
        }),
        "crs": xr.DataArray(
            data=np.array([]),
            attrs={
                "grid_mapping_name": "transverse_mercator",
                "longitude_of_central_meridian": -2.0,
            }
        )
    },
)

ds.to_netcdf("grid_mapping_str.nc")
xr.open_dataset("grid_mapping_str.nc", decode_coords="all")
# <xarray.Dataset> Size: 6kB
# Dimensions:  (y: 18, x: 36, dim_0: 0)
# Coordinates:
#   * x        (x) int64 288B 0 1 2 3 4 5 6 7 8 9 ... 27 28 29 30 31 32 33 34 35
#   * y        (y) int64 144B 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17
#     crs      (dim_0) float64 0B ...
# Dimensions without coordinates: dim_0
# Data variables:
#     temp     (y, x) float64 5kB ...

# FAILS
ds = xr.Dataset(
    {
        "temp": (("y", "x"), temp_data, {
            "long_name": "temperature",
            "units": "K",
            "coordinates": "lat lon",
            "grid_mapping": "crs: x y"
        }),
        "x": (("x"), x_coords, {
            "standard_name": "projection_x_coordinate",
            "units": "m"
        }),
        "y": (("y"), y_coords, {
            "standard_name": "projection_y_coordinate",
            "units": "m"
        }),
        "crs": xr.DataArray(
            data=np.array([]),
            attrs={
                "grid_mapping_name": "transverse_mercator",
                "longitude_of_central_meridian": -2.0,
            }
        )
    },
)

ds.to_netcdf("grid_mapping_dict.nc")
xr.open_dataset("grid_mapping_dict.nc", decode_coords="all")
# <ipython-input-4-23281f6f2619>:1: UserWarning: Variable(s) referenced in grid_mapping not in variables: ['crs:']
#  xr.open_dataset("grid_mapping_dict.nc", decode_coords='all')

# FAILS
ds = xr.Dataset(
    {
        "temp": (("y", "x"), temp_data, {
            "long_name": "temperature",
            "units": "K",
            "coordinates": "lat lon",
            "grid_mapping": "crsOSGB: x y crsWGS84: lat lon"
        }),
        "x": (("x"), x_coords, {
            "standard_name": "projection_x_coordinate",
            "units": "m"
        }),
        "y": (("y"), y_coords, {
            "standard_name": "projection_y_coordinate",
            "units": "m"
        }),
        "crsOSGB": xr.DataArray(
            data=np.array([]),
            attrs={
                "grid_mapping_name": "transverse_mercator",
                "longitude_of_central_meridian": -2.0,
            }
        ),
        "crsWGS84": xr.DataArray(
            data=np.array([]),
            attrs={
                "grid_mapping_name": "latitude_longitude",
                "longitude_of_prime_meridian": 0.0,
            }
        )
    },
)
ds.to_netcdf("grid_mapping_list.nc")
xr.open_dataset("grid_mapping_list.nc", decode_coords='all')
# <ipython-input-8-8e2c0fa99fa2>:2: UserWarning: Variable(s) referenced in grid_mapping not in variables: ['crsOSGB:', 'crsWGS84:', 'lat', 'lon']
#  xr.open_dataset("grid_mapping_list.nc", decode_coords='all')

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.
[X] Recent environment — the issue occurs with the latest version of xarray and its dependencies.

Relevant log output
No response
Anything else we need to know?
The issue seems to be that currently CF attributes can either be a string/list or a key-value pair depending whether the cf-attribute is listed in CF_RELATED_DATA or also listed in CF_RELATED_DATA_NEEDS_PARSING.
https://github.com/pydata/xarray/blob/main/xarray/conventions.py#L476-L480
grid_mapping is currently only part of CF_RELATED_DATA, but adding it to CF_RELATED_DATA_NEEDS_PARSING as well would cause the now working case to fail.
 A companion PR to cf-xarray would also be appreciated :)

@observingClouds It seems, that there are variables (str, dict, list) and coordinate variables (list) missing from those files. At least my cf-checker complains here, but good chance I'm missing something.

Oh yes @kmuehlbauer, I have simplified these example datasets to reduce the code section. They might no longer be CF-conform. Adding the missing variables and coordinates will not affect the decoding of the grid_mapping attribute in mind.

@observingClouds No worries. At least the UserWarning's are emitted when decoding the grid_mapping. Anyway, this whole block might need a refactoring.

@kmuehlbauer, I just updated my MVCE to be cf complient (except of a missing CF-convention attribute). The error remains the same.

@observingClouds Not sure if you're already working on a PR. I gave it a try in #9765.
This should now work for the two additional grid_mapping layouts. Would be great if you could test this against your workflows.
I've also added some todos for testing things, if you have any suggestions on that, please let me know.


How do you write Android code in Java or iOS code in objective-c for a project you've created

I have a project that was created by checking two boxes in the red box above. Now I'm going to call the native code.
How to write Android code in Java, or iOS code in objective-c.
I don't want to recreate Java and objective-c enabled projects, which is a bit of a slow process.

You would need to use whatever mechanism exists for interop in the selected language. You could also regenerate the project to use Java and Objective C.

@jonahwilliams in other words it's not possible in any easy way.
I will close the issue then.

in other words it's not possible in any easy way.

I guess that is a matter of opinion? I would say its fairly straightforward. At any rate, it has little to do with Flutter itself, so I agree this issue should be closed.


dialog version 5.0.0 not working with latest typescript version (5.4.5)

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
We are getting compile errors for the ngneat/dialog when trying to update to the latest typescript (5.4.5) version:

Expected behavior
dialog working as perfectly as with typescript 5.3.x
Minimal reproduction of the problem with instructions
What is the motivation / use case for changing the behavior?
To avoid security warnings, we need to update our projects and their dependencies regularly. Since we love using the ngneat/dialog, it would be awesome to get a fix for this issue.
Environment

Angular version: 17.3.4

Browser:
- [x] Chrome (desktop) version 123.0.6312.107
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: v20.9.0
- Platform:  Mac

Others:

Same issue


Conway changes not visible in PDF

Description
This closes issue #536.

Checklist

[X] Commit sequence broadly makes sense and commits have useful messages
[X] Code is formatted according to CONTRIBUTING.md
[X] Self-reviewed the diff

This PR adds the new figure (pictured below) showing some changes that were missing from the Conway pdf.

The figure shown below is not new, but this PR includes it in the Conway pdf, along with a brief explanation. The \begin{NoConway}...\end{NoConway} tags that had surrounded the figure were removed.


Fe: do not let app scroll leak

Right now if you scroll the inner app iframe, scroll events leak outwards.
The way to fix this is to track the inner pointer lock and set outer overflow hidden in that case.

Looks like its completely fixed! Closing issue.


Change AudioHandle::Config to an aggregate type

This changes AudioHandle::Config to be an aggregate type, maintaining the default values but replacing the explicit constructor.
This allows for aggregate initialization in addition to the standard constructor. Usage of the standard (default) constructor is still possible, with the same parameter ordering.
No changes to current code are required.

Looks great! Thanks again for the contribution 👍


[POAE7-2540] PlanNode validation for cider plan validator

What changes were proposed in this pull request?
Add plan validator module for cider, with single plan node validation as first step
Why are the changes needed?
provide plan validation capability for Cider
Does this PR introduce any user-facing change?
Yes.
Public APIs:
static bool isSupported(const substrait::Plan& plan, const std::string& from_platform);
static PlanSlice getCiderSupportedSlice(substrait::Plan& plan, std::string& from_platform);
How was this patch tested?
UT
Which label does this PR belong to?

@winningsix @jikunshang @haiweizh Please help take a review. Thanks.


Fix cadantine blood potion, nihil dust on mixables

Description:
Fixed error with mixing cadantine blood potion and nihil dust
Changes:
Added missing aliases for Cadantine blood potion (unf) and Nihil dust
Other checks:

[x] I have tested all my changes thoroughly.
Fixes https://github.com/oldschoolgg/oldschoolbot/issues/4482

Instead of this, why don't you fix the /mix command so it checks name and alias, for consistency.
	const mixableItem = Herblore.Mixables.find(
			item =>
				stringMatches(item.name, options.name) || item.aliases.some(alias => stringMatches(alias, options.name))
		);

Hey @themrrobert changes done, thanks for suggestion


ENH: Use tz-aware dtype for timestamp columns

Adds a table documenting the current behavior, including that pandas 0.24.0 stores as time zone aware dtype and earlier versions store naive. I could not figure out how to make 0.24.0+ store as a naive dtype (https://github.com/pydata/pandas-gbq/issues/261), nor could I figure out how to make earlier versions use time zone aware (https://github.com/pydata/pandas-gbq/pull/263)

Just when I gave up, I figured out how to make it work on old versions. It's not all that bad to call localize on the older versions and this way we don't have to bump the min version.


This isn't the illustration you're looking for

The picture here in this article doesn't answer to the description in the paragraph below.

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

ID: 958fc635-d7d3-8ac2-bfbb-d723c27d54b0
Version Independent ID: 5ea85a53-54a5-3bd2-1623-c4d14d2e5e71
Content: Assembly contents
Content Source: docs/standard/assembly/contents.md
Product: dotnet
GitHub Login: @gewarren
Microsoft Alias: dotnetcontent

Thank you @ruvor. You are right, and I'll get this fixed.

Note to self: https://github.com/dotnet/docs/blob/69f825f8b65a5c7eb67d552328ecca072a84924d/docs/framework/app-domains/assembly-contents.md is the original file.


[bug] azure package is deprecated

Describe the bug
The azure package is deprecated. Need to install individual packages instead.
To Reproduce
Steps to reproduce the behavior (please include the full Skyplane command you ran):

Run 'pip install azure'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
% pip install azure
Collecting azure
Downloading azure-5.0.0.zip (4.6 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [25 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "/private/var/folders/72/tm0y9z7s5ps7czwbhk3b3rwm0000gn/T/pip-install-9bbpxnbf/azure_e8b7dcef3e364884b5509ff05e7da165/setup.py", line 60, in
raise RuntimeError(message)
RuntimeError:
  Starting with v5.0.0, the 'azure' meta-package is deprecated and cannot be installed anymore.
  Please install the service specific packages prefixed by `azure` needed for your application.

  The complete list of available packages can be found at:
  https://aka.ms/azsdk/python/all

  Here's a non-exhaustive list of common packages:

  -  azure-mgmt-compute (https://pypi.python.org/pypi/azure-mgmt-compute) : Management of Virtual Machines, etc.
  -  azure-mgmt-storage (https://pypi.python.org/pypi/azure-mgmt-storage) : Management of storage accounts.
  -  azure-mgmt-resource (https://pypi.python.org/pypi/azure-mgmt-resource) : Generic package about Azure Resource Management (ARM)
  -  azure-keyvault-secrets (https://pypi.python.org/pypi/azure-keyvault-secrets) : Access to secrets in Key Vault
  -  azure-storage-blob (https://pypi.python.org/pypi/azure-storage-blob) : Access to blobs in storage accounts

  A more comprehensive discussion of the rationale for this decision can be found in the following issue:
  https://github.com/Azure/azure-sdk-for-python/issues/10646

Transfer client log
None.
Environment info (please complete the following information):

OS: Mac OS
Python version: 3.11.2
Skyplane version : 0.3.2

Additional context
Needed only when using Azure.

Note that this fix is not enough - https://github.com/skyplane-project/skyplane/commit/2aab284a80943871c8c4cc9e25ccde7eec063831
This is because the same should be fixed in the main README.


Explore setting volume permission and ownership at cephcsi level

https://kubernetes.io/docs/tasks/configure-pod-container/security-context/#delegating-volume-permission-and-ownership-change-to-csi-driver mentioned that a csi driver can hable volume permission and the ownership on this own, this can improve the pod startup time and avoid the recursive chmod/chown at kubelet level

/assign

Few tasks I can think of to complete this work

Test read/write operation using pod
Enable external test for cephfs/rbd https://github.com/ceph/ceph-csi/blob/66f71aea3a42e0ca022225cb027738ddb9caf08c/scripts/k8s-storage/driver-cephfs.yaml#L37 https://github.com/kubernetes/kubernetes/blob/d9c54f69d4bb7ae1bb655e1a2a50297d615025b5/test/e2e/storage/framework/testdriver.go#L154-L155C25
Upgrade test
setting groups at different levels in pod yaml (one is at pod level and another is at container level)
Test it with openshift
Add E2E tests and validate the group
Add it to the Pending release note


Unsupported visuals in EmailSubscription pipeline

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

#sign-off


fastly: 10.6.4 -> 10.8.0

(cherry picked from commit bf398b457e349291d9bab3b6b4901f71f71fb170)
Description of changes
Backport of https://github.com/NixOS/nixpkgs/pull/281632

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

darwin build is failing due to running out of disk space.


Python Refactoring Move to file/to new file

I do not find any extension to move a function, a class or a selected text to a file/a new file, it should be a common instruction that I made every day in other ides. Did I miss a plugin or this function in python is not implemented?
The common use is for move into a specific file (for example extract this into function and after move function into existing file utils.py) or  move to a new file (for example you create a class and you want to extract to a new file).

/extPython

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!

Thank you, I open the issue here: https://github.com/microsoft/vscode-python/issues/19270


More awesome OCR tools

Add

Camelot
PD3F
Parsr

They help you to extract information (even tables) from PDFs.
Checklist

[x] I have read and understood the contribution guidelines.
[ ] Table of contents has been updated (if a section is added / removed).
[ ] Contents have been sorted alphabetically.

Does it make sense for us to maintain a section for OCR tools? There is an Awesome OCR repository for that...

I guess it doesn't. We should add a link to the list instead.


DVTF: Server side clustering

The clustering feature within Mapbox is only supported for local GeoJSON sources; clustering locations provided by a vector source is currently not possible (at least via Mapbox).
Exploration into server-side clustering, either by GeoServer styling or some other transformation, needs to take place at some point.
(low priority, long term)

@gpeb and I have looked into a number of solutions here, this turns out to be a non-trivial issue. The goal is to host clustered, tiled, vector-data; but we can only achieve 2 of 3 without a large infrastructure change.

You can cluster vector data using the SLD style format...

However, you cannot serve this via GeoServer as it assumes all vector data will be styled client side and does not apply styles to served vector data.
Neither Cesium nor Mapbox appear to provide a way to style with SLD files on the client side.

You can cluster data directly returned from PostGIS...

However this can only be done statically as there's not way to update the SQL View that does this with the current zoom level in the client map

For the foreseeable future, clustering will only be possible in Mapbox visualisations, when sources are loaded using the geojson type (note that you can load geojson data directly from GeoServer via a  WFS endpoint).

In the future, there may be avenues to explore this by using a mapping library that can support client-side SLD styles (e.g. OpenLayers), but for the time being, the geojson approach is the only viable solution.


How to specify tags for algorithms, both search and assignment

comments on how to ideally specify the tags in the json calls are welcome, I can adapt the code accordingly

Don't we already have a tags field in the execution?

yes, we do, but that did not seem to be sufficient. Example: Suppose you want to choose all documents having the tag "ssoar" and apply all patterns having the tag "some_subset_of_ssoar" on them. Problem: how to specify which tags to use for which purpose if the execution has only one set?
I now implemented it according to the example: json objects are now supported for tags, like this: "tags": {
"InfolisPattern": ["tag1", "tag2"],
"InfolisFile": ["tag2", "tag3"]
}
If the key translates to the InfolisPattern class, its tag set is used to set uris for setPatternUris(). If it translates to the InfolisFile class, its tag set is used to set uris for setInputFiles(). If it translates to Entity or something else: I actually don't know in which cases that could be useful, so it doesn't do anything right now ;) But including additional cases should be straight-forward.
Execution still has its old tags field: that set of tags describes the Execution in case we'd like to identify it with them. The actual tags for selecting files, patterns or other things are now stored in the additional field tagMap. <- should probably rename it to something more self-explanatory like selectionTags or something like that.

The approach is sound and I can see why just "tags" isn't enough. It's just a matter of organizing the code in a way to avoid any map-like structures. Converting the Multimap<String,String> to a Java class and representing it in our data model will make serialisation easier and conformant with RDF limitations.
I'd then argue for making it clear in the documentation and our code that Execution.tags is for assigning tags to some resource or resources and Execution.tagMap for selecting resources by their tag.

Good point, thanks. Done.
Yes, we absolutely have to make that clear!

entered the wrong issue in my commit message, sry -.- didn't notice it was in a different location. Anyway, it's fixed in https://github.com/infolis/infoLink/commit/830b7278b57403a1ae30a926206c93eb4e25508c

opened a new issue about the documentation at https://github.com/infolis/infolis-web/issues/31.
This issue remains open until the json command has been successfully tested.

works, see https://github.com/infolis/infolis-web/issues/20#


Batch posting rules works invalid if BatchPositingLimit and BatchPostingPeriod are combined

Describe the bug
If both BatchPositingLimit and BatchPostingPeriod are settled a butch of logs delivered only when batch filled (BatchPositingLimit condition).
To Reproduce

Set batch posting limit as 5
Set batch posting period as 1 min
Generate 4 log entries that have to be logged
See that batch is not sent
Generate one more log entry
See that batch sent

Expected behavior
Logs batch sent on any of conditions reached (BatchPositingLimit or BatchPostingPeriod)
Additional context

Issue occurs for versions 1.0.0

Workarounds
Use only one of rules which suites more

Resolved #14


Merge master into dev to make semantic-release happy

A merge to the master branch of this repo yielded the following error:
$ yarn run semantic-release
yarn run v1.3.2
$ semantic-release pre && npm publish && semantic-release post
[Travis Deploy Once]: Success at attempt 1. All 1 jobs passed.
semantic-release ERR! commits The commit the last release of this package was derived from is not in the direct history of the "master" branch.
semantic-release ERR! commits This means semantic-release can not extract the commits between now and then.
semantic-release ERR! commits This is usually caused by force pushing, releasing from an unrelated branch, or using an already existing package name.
semantic-release ERR! commits You can recover from this error by publishing manually or restoring the commit "b4163e29dc35cbcaa099e48732d64d7e64e37489".
semantic-release ERR! pre Failed to determine new version.
semantic-release ERR! pre ENOTINHISTORY Commit not in history
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

It looks like b4163e29dc35cbcaa099e48732d64d7e64e37489 is only contained in the master branch, so perhaps we need to merge master back into dev so that semantic-release will run successfully. @evansiroky, is this the correct interpretation?

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


Fix a crash bug of piped when generate diff from empty manifest on Cloud Run

What this PR does / why we need it:
Fix crashes when generate plan preview from empty service manifest on Cloud Run.
Stack Trace:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x13a953b]
goroutine 207 [running]:
github.com/pipe-cd/pipecd/pkg/app/piped/platformprovider/cloudrun.Diff({{0x0?, 0x0?}, 0x0?}, {{0xc000b49500?, 0x0?}, 0xc00011c2f8?}, {0xc0009f3740?, 0xc00063f600?, 0xb?})
    /home/runner/work/pipecd/pipecd/pkg/app/piped/platformprovider/cloudrun/diff.go:42 +0x5b
github.com/pipe-cd/pipecd/pkg/app/piped/planpreview.(*builder).cloudrundiff(0xc0007b2180, {0x26e1080, 0xc0000b9c80}, 0xc00077e2d0, {0x26d3140?, 0xc0007dc620?}, {0x0, 0x0}, 0xc00063f640?)
    /home/runner/work/pipecd/pipecd/pkg/app/piped/planpreview/cloudrundiff.go:61 +0x4f7
github.com/pipe-cd/pipecd/pkg/app/piped/planpreview.(*builder).buildApp(0xc0007b2180, {0x26e1080, 0xc0000b9c80}, 0x1?, {0xc000a50f30?, 0xc00052eec8?}, 0xc00077e2d0, {0x26eaea8?, 0xc000768780}, {0xc000ae3578, ...})
    /home/runner/work/pipecd/pipecd/pkg/app/piped/planpreview/builder.go:254 +0xabd
github.com/pipe-cd/pipecd/pkg/app/piped/planpreview.(*builder).build.func1(0x1)
    /home/runner/work/pipecd/pipecd/pkg/app/piped/planpreview/builder.go:185 +0x205
created by github.com/pipe-cd/pipecd/pkg/app/piped/planpreview.(*builder).build
    /home/runner/work/pipecd/pipecd/pkg/app/piped/planpreview/builder.go:182 +0xc55"
timestamp: "2023-01-01T05:48:57.613492Z

Does this PR introduce a user-facing change?:
Fix a crash bug of piped when generate diff from empty manifest on Cloud Run.

@kurochan
Thank you for your good report.
I investigated this issue and addressed it by https://github.com/pipe-cd/pipecd/pull/4100.
Hence let me close this PR, and this issue will be resolved at v0.41.2.


Improve SD Factory Toggle

Overview

Explain the topic in 2 sentences

As a CX-Operator, we want to have an override button at the SD Factory step, similar to override Clearing House step.
What's the benefit?

We could selectively skip SD factory step for countries which are not supported by Gaia-X Clearing House.
What are the Risks/Dependencies ?

No dependency

Detailed explanation

Current implementation
The current toggle is global, apply to everyone or no one.
Proposed improvements

Make toggle based on override button

Feature Team
Contributor

@dhiren-singh-007
@tfjanjua

Committer

User Stories

Issue 1, linked to specific repository
Issue 2, linked to another specific repository

Acceptance Criteria

[ ] Criteria 1
[ ] Criteria 2
[ ] Criteria 3

Test Cases

Test Case 1

Steps

Do something
Click something
Add something

Expected Result

Expectation
Expectation
Expectation

Architectural Relevance

The following items are ensured (answer: yes) after this issue is implemented:

[ ] This feature aligns with our current architectural guidelines

Data Sovereignty: All data sharing activities across company boundaries follow the Catena-X Regulatory Framework, in particular the Data Exchange Governance, and the Dataspace Protocol via a compliant Connector (like the tractusx-edc or similar, see Connector KIT)

[ ] CX-0010 Business Partner Number
[ ] CX-0013 Identity of Member Companies
[ ] CX-0018 Data Space Connectivity (EDC)
[ ] CX-0049 DID Document Schema
[ ] CX-0050 Framework Agreement Credential
[ ] CX-0149 Verified Company Identity

Interoperability: Digital Twins are used (compliant to the Digital Twin KIT and the Industry Core KIT)

[ ] CX-0001 EDC Discovery API
[ ] CX-0002 Digital Twins in Catena-X
[ ] CX-0018 Data Space Connectivity (EDC)

Data Format:

[ ] The data model is based on a published Semantic Model

[ ] The impact on the overall system architecture has been assessed. The Feature does not require changes to the architecture or any existing standard? Please have a look here on the overarching architecture
[ ] Potential risks or conflicts with existing architecture has been assessed

Justification: (Fill this out, if at least one of the checkboxes above cannot be ticked. Contact the Architecture Management Committee to get an approval for the justification)
Additional information

[ ] I am aware that my request may not be developed if no developer can be found for it. I'll try to contribute a developer (bring your own developer)

Some hints from Release Management (@ther3sa) and Tractus-X Project Lead (@stephanbcbauer)

Status currently in Inbox. ⇾ Only features with status backlog are considered in open planning
Please add missing sections from the feature template, or fill them out
Please add contributor and committer

Committers:
@oyo
@ntruchsess
@Phil91
@evegufy


Update to 1.16.1

Maybe you update this instrument on new minecraft version for testing my protect?

It already supports 1.16, it's just that the new version is not in the releases, it needs to be compiled


Unrelated clusters overlap

Two separate groups of connected nodes are rendered with a significant overlap, confusing the view.
Attached archive contains the JSON report and an SVG showing an example.
archive.zip

Here is the SVG rendered:

@pidster could u draw your ideal diagram?

Ideally the two groups should be distinct.
Edges from group 1 may intersect with edges from group 1, but should not intersect with edges from group 2.
In the example above, the edges from the swarm/consul group intersect with edges from the unconnected/unrelated weave/scope group. There are 4 weave nodes on the upper left that are far located far away from the rest of the connected group.

I believe this is a dupe of #361


Using different SVG's but loading the same one

I'm loading 4 different SVG icons dynamically in the component constructor.

When I use them in the HTML the first one is the only well rendered.
In this example, I set those four and right after I put the same ones but using an  tag:

The result is:

I've not been able to reproduce this. In the demo app, what I've done:
  constructor(private registry: SvgIconRegistryService) {
    this.registry.loadSvg('assets/images/eye.svg', 'eye.icn');
    this.registry.loadSvg('assets/images/moon-o.svg', 'moon.icn');
  }

And in the template:
<div class="portal">
    <svg-icon name="eye.icn"></svg-icon>
    <svg-icon name="moon.icn"></svg-icon>
</div>

And both the eye svg and moon svg load as expected.

I found the problem. All the SVGs I was provided have the same ids (they're around 50):

So the problem is when trying to load multiple ones with the same ID.
Not sure if it's your business though 😅
Thank you 👍


[New?] Pay Respects Command

only a meme would do this
TODO :

[x] command for it obviously
[x] memes
[x] even more memes

that was quick


update env_logger to 0.11

https://github.com/rust-cli/env_logger/blob/HEAD/CHANGELOG.md#0110---2024-01-19

Thanks

@d-e-s-o it'd be great to have a release with this, so that we can update env_logger in our applications.

@d-e-s-o it'd be great to have a release with this, so that we can update env_logger in our applications.

I'd rather wait some more. The crate doesn't have a formal MSRV policy right now, but N-5 is the bare minimum from my point of view. I'd prefer N-6 or "higher". Why is it so important that this crate uses env_logger 0.11?

I had this error when trying to update env_logger:
Command failed: cargo update --config net.git-fetch-with-cli=true --manifest-path Cargo.toml --package<EMAIL_ADDRESS>--precise 0.11.1
    Updating crates.io index
error: failed to select a version for the requirement `env_logger = "^0.10"`
candidate versions found which didn't match: 0.11.1
location searched: crates.io index
required by package `test-log v0.2.14`
    ... which satisfies dependency `test-log = "^0.2"` (locked to 0.2.14) of package `reference_service v0.1.0 (/tmp/renovate/repos/github/commercetools/reference_service)`
perhaps a crate was updated and forgotten to be re-vendored?

It's not completely blocking as I can still update the dependency though.


Custom argument names are not case insensitive in 1.0 preview

The option --target=foo works as expected but not --Target=foo.

It comes to Cake.Frosting too. Looks like it is because it using Spectre.Console to parse command-line arguments


[SEDONA-3] First release: [WIP] change path names and file headers

Is this PR related to a proposed Issue?
Prepare the first Sedona release: 1.0.0-incubator
What changes were proposed in this PR?
Change all paths to org.apache.sedona
Scala and Java:
The new path of sub-modules is as follows:
Core: org.apache.sedona.core
SQL: org.apache.sedona.sql
VIZ: org.apache.sedona.viz

Classpath inside Spark SQL kernal
SQL: org.apache.spark.sql.sedona_sql
VIZ: org.apache.spark.sql.sedona_viz

All class files which have the name prefix "GeoSpark" have been changed to either the prefix "Sedona" or no prefix
All comments which have "GeoSpark" have been updated to "Sedona" accordingly.
The code has been restructured, re-formatted and cleaned.
Python
To be done by @Imbruced
How was this patch tested?
Did this PR include necessary documentation updates?
Will be handled in a separate PR

@jiayuasu I can see that only viz tests are failing, so I started to work on Python version. I should finish soon.

@jiayuasu Python version is almost there, I need to combine dependencies to one package and fix one test, but I found out that one of geospark core tests failing due to invalid records number.
Failed tests:
  CRSTransformationTest.testPolygonDistanceJoinWithCRSTransformation:375 expected:<5467> but was:<5472>

@Imbruced This is because when I cleaned up the code, somehow messed up the test data files. The error in Viz is also caused by this.
I was very busy in the past week so didn't fix it. But now I get sometime, I will fix it in two or three days. If I cannot fix the test data file, I will redo this PR from scratch. Please stay tuned.

@Imbruced I have fixed the failed Viz test cases. But the core test does not fail on my machine: CRSTransformationTest.testPolygonDistanceJoinWithCRSTransformation:375 expected:<5467> but was:<5472>
All test cases in Scala and Java API of core, sql and viz have passed. You can also verify this in the Travis CI build.
You can focus on the Python side

@jiayuasu Hello Jia, hope you are doing well.
Is there anything I can help with?

@Imbruced Any updates?

@jiayuasu I am almost there, today I will create PR.


Background Task possible with this plugin? Complety closed App

Hello,i have a short question about this plugin. Is i possible to detect bluetooth device if an app is complety closed? Does this library works with the internal API from apple?

https://github.com/dotintent/react-native-ble-plx/wiki/Background-mode-(iOS)


Review/71

@ccoVeille :saluting_face:

You are not in a hurry to merge things you know 😅

@ccoVeille :grimacing:


We care what you said

adds the file caption to the reportback permalink page.
remove default vars from theme array, because :information_desk_person:

Code looks good, but would be nice to use the existing ReportbackFileEntity class to keep things DRY

@aaronschachter updated

Just fix stray space and ready to ship :+1:


new feature: help add proxy

hi, due to region limitations, some ppl have to visit tiktok with a network proxy,
could you help add proxy while navigating tiktok
thanx

Implemented in v0.1.8 by specifying launch options in kwargs of TikTokAPI or the AsyncTikTokAPI.
Full details of the launch options can be found in Playwright's Documentation.
Example for using a proxy:
from tiktokapiypy.api import TikTokAPI

with TikTokAPI(proxy={
  "server": "http://myproxy.com:3128",
  "username": "usr",
  "password": "pwd"
}) as api:
    ...

Related to #29 - Proxy setup should be working now as of 0.1.11.

hey this is not working waiting until "load"


Failed device setup not retried

I've had this issue several times now, it seems that if Tuya Local cannot find a device at startup or for some time after startup, it will stop looking for it and it will stay in "Failed to set up" state.
I've had this happen in 2 different situations:

When my power of the whole house goes out and I restart power, HA boots before my Wifi access point boots, to which the Tuya devices connect (probably with even some extra delay for connection retry)
I had my HA OS boot without a network connection for a few minutes, and after connecting the network, the Tuya devices are still not found, while everything else in HA is working.

If I manually reload the failed device in HA admin, it starts working immediately.
This is different to some other integrations I used (e.g. HomeKit Device), which never "give up" on a device and will keep trying to find it (successfully when it eventually becomes available). Especially since this plugin is only making local connections, there is even more reason to keep retrying.

If there is a bug in the integration preventing the device from being initialised by Home Assistant, there should be an error message in the Home Assistant log, which will have a stack trace pinpointing the issue.
Without that, it is not even clear the issue is in the integration and not Home Assistant itself.

@make-all here is the relevant log entries from the last occurence of this issue:
2024-02-25 15:51:26.622 ERROR (MainThread) [custom_components.tuya_local.device] <class 'Exception'>: Unable to find device on network (specify IP address) while initialising device 1a2a3a4a5....
2024-02-25 15:51:26.622 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Vticnica Dnevna Prvo Nadstropje for tuya_local
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 444, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/tuya_local/__init__.py", line 314, in async_setup_entry
    setup_device(hass, config)
  File "/config/custom_components/tuya_local/device.py", line 635, in setup_device
    device = TuyaLocalDevice(
             ^^^^^^^^^^^^^^^^
  File "/config/custom_components/tuya_local/device.py", line 86, in __init__
    raise e
  File "/config/custom_components/tuya_local/device.py", line 77, in __init__
    self._api = tinytuya.Device(dev_id, address, local_key)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/tinytuya/core.py", line 941, in __init__
    raise Exception("Unable to find device on network (specify IP address)")
Exception: Unable to find device on network (specify IP address)
2024-02-25 15:51:44.753 ERROR (MainThread) [custom_components.tuya_local.device] <class 'Exception'>: Unable to find device on network (specify IP address) while initialising device 2a2a3a4a5....
2024-02-25 15:51:44.753 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Mansarda Luc Tuya for tuya_local
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 444, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/tuya_local/__init__.py", line 314, in async_setup_entry
    setup_device(hass, config)
  File "/config/custom_components/tuya_local/device.py", line 635, in setup_device
    device = TuyaLocalDevice(
             ^^^^^^^^^^^^^^^^
  File "/config/custom_components/tuya_local/device.py", line 86, in __init__
    raise e
  File "/config/custom_components/tuya_local/device.py", line 77, in __init__
    self._api = tinytuya.Device(dev_id, address, local_key)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/tinytuya/core.py", line 941, in __init__
    raise Exception("Unable to find device on network (specify IP address)")
Exception: Unable to find device on network (specify IP address)
2024-02-25 15:52:02.780 ERROR (MainThread) [custom_components.tuya_local.device] <class 'Exception'>: Unable to find device on network (specify IP address) while initialising device 3a2a3a4a5....
2024-02-25 15:52:02.781 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Lucka Spalnica Levo Tuya for tuya_local
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 444, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/tuya_local/__init__.py", line 314, in async_setup_entry
    setup_device(hass, config)
  File "/config/custom_components/tuya_local/device.py", line 635, in setup_device
    device = TuyaLocalDevice(
             ^^^^^^^^^^^^^^^^
  File "/config/custom_components/tuya_local/device.py", line 86, in __init__
    raise e
  File "/config/custom_components/tuya_local/device.py", line 77, in __init__
    self._api = tinytuya.Device(dev_id, address, local_key)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/tinytuya/core.py", line 941, in __init__
    raise Exception("Unable to find device on network (specify IP address)")
Exception: Unable to find device on network (specify IP address)
2024-02-25 15:52:13.126 WARNING (SyncWorker_1) [custom_components.tuya_local.device] Terrace Dimmable light protocol error 901: Network Error: Unable to Connect
2024-02-25 15:52:52.697 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:52:57.700 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:53:02.705 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:53:07.711 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:53:12.717 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:53:17.721 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:53:22.723 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:53:27.727 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:53:32.729 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
2024-02-25 15:53:37.734 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
...message repeating...
2024-02-25 16:13:34.314 ERROR (MainThread) [custom_components.tuya_local.device] Failed to fetch device status for Terrace Dimmable light
...message stops...
2024-02-25 16:13:49.931 WARNING (MainThread) [homeassistant.helpers.service] Referenced entities light.mansarda_luc_tuya are missing or not currently available
2024-02-25 16:13:49.934 WARNING (MainThread) [homeassistant.helpers.service] Referenced entities switch.lucka_spalnica_levo_tuya are missing or not currently available


report the error "Immediately connect to the internet to check for updates" when input "PASSIVE=0 NOSENSOR=1 USE_WEBCAM=1 ./manager.py

Describe the bug
I use the webcam tool to run openpilot on PC. We refer to the guideline and connected the PC to two web cameras and Panda. When input "PASSIVE=0 NOSENSOR=1 USE_WEBCAM=1 ./manager.py", report "Immediately connect to the internet to check for updates",  details please refer to below picture. How to solve it ? Thanks.

Expected behavior
Openpilot can run successfully
Additional context

Operating system: [e.g. Ubuntu 【20.04]

Have you tried running updated on PC? This can be solved by clearing your params folder (~/.comma/params)

Fixed in 29ac94b71997303fdd2cae137a1a3ccd016868e7. After clearing your params, you shouldn't see this again.


Please add the Elmah SQL SPROCs to this repo

As a first time Elmah SQL logger, I was unaware that I needed to create the secret SPROCs on the database first.. can you please add them to the repo/add a note about them?

Same problem, and the first I found was not functionnal

This one works 👍
``
/*
ELMAH - Error Logging Modules and Handlers for ASP.NET
Copyright (c) 2004-9 Atif Aziz. All rights reserved.
Author(s):

    Atif Aziz, http://www.raboof.com
    Phil Haacked, http://haacked.com

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
  http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
*/
-- ELMAH DDL script for Microsoft SQL Server 2000 or later.
-- $Id: SQLServer.sql 677 2009-09-29 18:02:39Z azizatif $
DECLARE @DBCompatibilityLevel INT
DECLARE @DBCompatibilityLevelMajor INT
DECLARE @DBCompatibilityLevelMinor INT
SELECT
@DBCompatibilityLevel = cmptlevel
FROM
master.dbo.sysdatabases
WHERE
name = DB_NAME()
IF @DBCompatibilityLevel <> 80
BEGIN
SELECT @DBCompatibilityLevelMajor = @DBCompatibilityLevel / 10,
       @DBCompatibilityLevelMinor = @DBCompatibilityLevel % 10

PRINT N'
===========================================================================
WARNING!
---------------------------------------------------------------------------

This script is designed for Microsoft SQL Server 2000 (8.0) but your
database is set up for compatibility with version '
+ CAST(@DBCompatibilityLevelMajor AS NVARCHAR(80))
+ N'.'
+ CAST(@DBCompatibilityLevelMinor AS NVARCHAR(80))
+ N'. Although
the script should work with later versions of Microsoft SQL Server,
you can ensure compatibility by executing the following statement:

ALTER DATABASE ['
+ DB_NAME()
+ N']
SET COMPATIBILITY_LEVEL = 80

If you are hosting ELMAH in the same database as your application
database and do not wish to change the compatibility option then you
should create a separate database to host ELMAH where you can set the
compatibility level more freely.


Upvote Standard in this Stack Overflow question

http://stackoverflow.com/a/28954080/359072
Perhaps also a mention of this in the readme, would that be agreeable?
Also, the answer was posted by me so we could change it if we like.

I've just noticed that the question is closed. I'm not sure if that means no more upvoting.

Upvoted 👍

Upvoted 👍

The top answer has 110 upvotes. I bet we can beat that.
Can we put something in the readme, perhaps?

Going to close this now, since it's not an issue with standard. We can just share the link on twitter if we want people to see it.


Minor: [Doc] Fix wrong heading level of Single-object encoding in the doc

What is the purpose of the change
This PR fixes the heading level of Single-object encoding in the doc.
Currently, Single-object encoding is under JSON Encoding but it seems wrong.
Verifying this change
Confirmed the layout is fixed with the following command.
docker run --rm -v $(pwd):/src -p 1313:1313 jakejarvis/hugo-extended:latest --source doc/ server     --buildDrafts --buildFuture --bind <IP_ADDRESS> --navigateToChanged

Then, we can find Single-object encoding in the table of contents.

Documentation

Does this pull request introduce a new feature? (no)

Thank you, @sarutak !


fixed autocorr + Lempel Ziv Complexity + Some failing tests

The issue is that in lazy Polars, slice behaves differently. The problem lies somewhere in its length calculation. Try running the following code, which will give us error in the lazy case. Since in our original code, we are just returning one expression at the end, the error didn't occur, but something inside was wrong. You can see the issue here: https://github.com/pola-rs/polars/issues/11594#issue-1931924184
df = pl.DataFrame({
    "a": [1,2,1,2,1,2],
    "b": [None, None, 1,1,1,1]
})

df.select(
    pl.col("a").slice(offset = 0, length=pl.count() - 2).alias("1"),
    pl.col("a").slice(offset = 1, length=pl.count() - 2).alias("2")
)

df.lazy().select(
    pl.col("a").slice(offset = 0, length=pl.count() - 2).alias("1"),
    pl.col("a").slice(offset = 1, length=pl.count() - 2).alias("2")
).collect()

I fixed the bug by reverting back to using shift, which will be slower than .head() / tail() / slice(), but will avoid using slice for now. We actually do not need to drop nulls after shifting, because dot product will null will just result in 0.

@MathieuCayssol  Autocorrelation should be correct now. Please take a look

Sure, I will have a look. For the unit tests, the things you've updated, I already did it in the branch #66


[Bug]: Notebook title format issue

Describe the bug
https://microsoft.github.io/autogen/docs/notebooks/agentchat_groupchat_finite_state_machine
The title looks polluted. There may be a bug in the notebook processing.
Steps to reproduce
No response
Model Used
No response
Expected Behavior
No response
Screenshots and logs
No response
Additional Information
No response

This is fixed now


Windows 10: Reveal in Files Explorer doesn't work

Bug Report
What is the current behavior?
Right clicking a file in the Files Tree view, and selecting "Reveal in Files Explorer" does nothing.
What is the expected behavior?
Doing the above should open Windows Explorer to the location where the file is located, preferably with the file highlighted/selected.
For GitBook Editor

Operating System: Windows 10 Pro x64
(Desktop) Editor version: 6.2.1
Was the bug present in previous versions: I'm not sure, I haven't used Gitbook Editor since before this feature was implemented

I am still seeing this bug in Editor version 6.3.0-beta.1.

In 7.0.2, Reveal in Files Explorer still does not work.

In 7.0.12 + Window 7, "Reveal in Files Explorer" doesn't work :-(

@Deozaan @jimmy-profit @john20xdoe Can you confirm if this happens for all files, even committed files ?
I understand that it wouldn't work for uncommitted files that haven't yet been written to disk.

@AaronO I can confirm in 7.0.12 on Windows 10 "Reveal in Files Explorer" doesn't do anything for any file I attempt it on.


Add semgrep post about enforcing origin checks on postMessage

@clintgibler would you like to double check this new post? I feel it can be improved. :)

@bernardoamc thanks for sharing! 🙏  Today was quite busy, will try to take a look tomorrow.

@clintgibler My pleasure, thanks for taking a look at it. No rush! :)

@clintgibler What an amazing review, thank you! <3 I will slowly adjust the post based on your feedback, you raised a lot of interesting points! 🎉

No problem, happy to help :)

Adjusted for most of the feedback, the post is looking much better. :)


App: new setRegions method

What's done

Introduced new setRegions method, accepting multiple Region names
Existing setRegion still exists, and calls setRegions internally.

Note: not a breaking change. done mainly to be more in line with the app.manifest file, which stores the region names as an array under targetContainers key.

Coverage increased (+0.1%) to 90.299% when pulling f031cd63548c7b0f31ab8d94c545f38ff2fd5ff9 on set-regions into 51f79ee3e80a2b518a489ba8c382d4b728fbe793 on master.


Add Color-by-attribute to 3d Network Nodes

Resolved awhile ago...

Also, duplicate of #7. Manage your issues, folks!


graveler allows merge with no changes

graveler should return error in case of merge that has no changes
same as MVCC return ErrNoDifferenceWasFound

Also, a revert with no changes should return a similar error

Also, a revert with no changes should return a similar error

Why is this a release blocker?
Being able to merge repeatedly causes long-term damage to the user repository.  It is an easy mistake to make, and hard to recover from.

Why is this a release blocker?
Being able to merge repeatedly causes long-term damage to the user repository.  It is an easy mistake to make, and hard to recover from.


Artefact in Ineichen clearsky model for sun close to horizon

Consider following example:
import pandas as pd
import pvlib
times = pd.date_range('2018-02-16', '2018-02-17', freq='min')
location = pvlib.location.Location(latitude=60., longitude=0.)
for model in ['ineichen', 'haurwitz', 'simplified_solis']:
    location.get_clearsky(times, model=model)['ghi'].plot(label=model, legend=True)

which results in following figure:

The (default) 'ineichen' clearsky model seems erratic at sunrise and sunset. The effect is especially visible for high latitudes during Winter when the sun slowly approaches the horizon.
Upon closer look in the code, I found the following comment:
https://github.com/pvlib/pvlib-python/blob/04bd7532f54b550313d84309c3d3d19818694ccc/pvlib/clearsky.py#L98-L103
which refers to the formula:
https://github.com/pvlib/pvlib-python/blob/04bd7532f54b550313d84309c3d3d19818694ccc/pvlib/clearsky.py#L122-L126
However, apart from the typo mentioned, there is another important difference between both formula of Solar Energy 73, and that's the np.exp(0.01*airmass_absolute**1.8) factor, which is only present in the formula of pg 311, but not in the formula of pg 156!
It seems like the artefact is exactly a result of that np.exp(0.01*airmass_absolute**1.8) factor. This factor slowly increases from 1.01 at solar zenith to 1.95 at a zenith angle of 85°, but then shoots to 1000 when the sun reaches the horizon.
So, maybe there was an error also in the formula of Solar Energy 73, pg 311, and that factor wasn't supposed to be there?
Nevertheless, in reference [3] (Reno et al., 2012), this model was found to excel for US sites with the same formula as used by pvlib. The US sites were all below the 49th parallel. I wonder whether this model is still recommendable for higher latitudes?

There is actually another version of the paper of Ineichen, which doesn't contain the typo nor the factor causing the artefact: http://citeseerx.ist.psu.edu/viewdoc/download?doi=<IP_ADDRESS>5.4248&rep=rep1&type=pdf.

Hi Karel!  I saw this effect too but simply didn't have the time to investigate and report--so thank you for this!  One hypothesis I had was that this might have worked ok for hourly averages.

Hi Anton, The effect indeed mostly disappears in hourly averages.
Unless you go even further north. Then the whole curve gets really biased. See e.g. the result for 70° latitude:

Thanks for the detailed report. This artifact and the discrepancies among the papers and code is part of what motivated me to add the simplified solis model to the library.

Time to write an email to Pierre Ineichen?

Hi @kdebrab thanks for catching this. How does Ineichen compare to haurwitz and solis when the mystery factor np.exp(0.01*airmass_absolute**1.8) is removed? And how do they compare to Bird?

Pierre will retire in May, so don't wait too long to ask!

Gasp!

I checked with Pierre, and he explained that Richard Perez added the exponential term in the later paper based on further analysis of measurements at 10 continental US sites.  I guess no one checked the effect on locations outside this geographic area.
I would suggest that function be modified to accept a flag to indicate whether the Perez enhancement is to be used or not, with a default value of False.

Thanks for checking. I support your proposed solution. I'm not going to get to it anytime soon, though, and would welcome a pull request.

I concur with @adriesse proposal.  I would add a comment to the function signature warning that the Perez enhancement may produce biased results at high latitude, and link the False/True condition for the flag to the appropriate reference.

One way to remove the artifacts while introducing less bias is to clip the airmass_absolute values to the point where the exponential correction becomes problematic. Something like:
airmass_critical = (cg2*(fh1+fh2*(tl-1))/0.018).^1.25
airmass_clipped = np.fmin(airmass_absolute, airmass_critical)
ghi = (np.exp(-cg2*airmass_clipped*(fh1 + fh2*(tl - 1)) + 0.01*airmass_clipped**1.8))
...
ghi = cg1 * dni_extra * cos_zenith * tl / tl * np.fmax(ghi, 0)
airmass_critical is a minimum for the combined exponential terms, which shoot up afterwards. Loosely:
airmass_critical = argmin(f(x)), for f = exp(-cg2*x*(fh1 + fh2*(tl - 1)) + 0.01*x^1.8))
This threshold starts at AM = 2.6 (for turbidity = 1 and altitude = 0), equivalent to ~22.5° solar elevation, and goes up to O(10) for higher turbidities and altitudes. For most cases, it corresponds to solar elevation angles below the 4° cutting threshold of the CIE 1994 quality control guidelines that Ineichen & Perez seem to have used, i.e. they probably didn't see beyond it.
Clipping at the minimum makes the adjustment smooth (C1), and leaves the results of the model untouched for most operating conditions.

@martinherrerias your proposal makes a lot of sense to me, now that I better understand the the role that correction plays. Rather than continuing in this old issue, would you like to open a new issue? If you are also inclined a pull request is welcome.

I would suggest controlling this new feature by the same flag or a new one. It should be possible to run the model as published.
This reminds me of some discussions about using models outside of the range of inputs for which they were developed and/or validated.

Are there any references for this new feature? @martinherrerias any plans to present or publish this idea? I think that would be useful. Thanks!


Remove count of projects in each category

Removes project count entirely per https://github.com/gratipay/gratipay.com/issues/4315

I guess this info is still available since we enumerate the items. This moves us towards making these more like filters than tabs. And it's a PR from @clone1018! 💃 😄

Rebased and clean up whitespace, was bbd47689da0699b42f76f1f8e82f8bd1358939a1.

Merging when green ...

Gosh I've restarted https://travis-ci.org/gratipay/gratipay.com/builds/214939176 five times already. #4391! 😡

Now Travis won't load so I can restart? 😕

Gonna top up https://github.com/gratipay/gratipay.com/pull/4391 and then revisit here ...


Combine search survey results into sample.csv

Codecov Report

Merging #698 (b2458cd) into search-survey (bf6c630) will decrease coverage by 0.59%.
The diff coverage is 71.68%.

@@                Coverage Diff                @@
##           search-survey     #698      +/-   ##
=================================================
- Coverage          85.11%   84.51%   -0.60%
=================================================
  Files                 63       65       +2
  Lines               2425     2538     +113
  Branches             362      381      +19
=================================================
+ Hits                2064     2145      +81
- Misses               300      330      +30
- Partials              61       63       +2

Impacted Files
Coverage Δ

CreeDictionary/search_quality/sample.py
50.00% <ø> (ø)

CreeDictionary/search_quality/combine_samples.py
60.97% <60.97%> (ø)

...eDictionary/search_quality/combine_samples_test.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Skip tacotron2 and unskip vision_maskrcnn in torchbench.py

See #82 and #101

Not sure why vision_maskrcnn is unskipped. Is it passing now? (if yes, should we close the other issue?)

It seems to work for me... (at least for forward)


Migrate to webpacker (all necessary deps)

Migration to webpacker has apparently been done and we should now move on to more contemporary solutions for Rails 7, see #454


Basic publishing settings

Set up basic publishing support
Be able to publish to maven local folder
Be able to run demos using version from this published folder

@TWiStErRob, could you please review this PR?

Not sure if you need all the junk I hoarded up recently :) For example sorting the nodes.

You know.. most of this junk makes total sense :)
I'm not sure about configuration for dokka task itself, but it contains an important link to an issue to remove trace logging. Function similar to withDokkaJar for sourceJar and javadocJar I met with an issue with sourceJar I had recently, and it's way easier to use than it was before.
All publication fields are mandatory and.. it's hard to work with a POM which is not sorted or sorted in a way, that artifact id on the top, version in the middle and group at the bottom (I saw such projects at work)

also https://github.com/jsonschema2dataclass/js2d-gradle/pull/723#discussion_r1133141394

don't block it on me though, it's Green, and it looks reasonable :)

it's always worth a while for me to look on the code from another perspective


Restore old tag lookup behaviour

If versioning is disabled and a PR is open, there
will likely not be a tag on the HEAD commit.
However we still want to use something to tag
our artifacts so use git descibe.
Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

I self-certify!


GE: fix template builder widget definition bug

Critical Changes
Changes

Fixes a bug that causes the GAU Allocation widget to throw a de-reference error on new form templates.

Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

**lurch: attach W-041468

Tracking W-041468


Sharing source between instances

Hello,
first, thanks a lot for the component, it's slick.
Like the other fella in #9 I come from a library where pages are displayed one by one. So here I've use a v-for around the component with the :page prop. So far so good. I'm doing that because I need to decorate each page (checkboxes, etc. around every page, not sure I'm doing it the best way possible though).
However, because the loading is done inside the component, I have to have another instance, with v-show="false" just to be able to get the number of pages to feed to the v-for through a ref. Other components load the pdf outside and so we can get the number of pages without ref to the component. But here it needs to be rendered to be ref'd, and so we need to have the number of pages for that, so it needs to be rendered and ref'd, so it's a loop.
Now my question: is the source shared by all component instances? (I see it's loaded with a worker but I'm not sure of the behaviour). Just to be sure I'm not multiplying the pdf in memory.
Cheers!

Hey, thanks for you response.
Here is what I mean:

As you can see, each page is decorated with checkboxes and a button. It's not just styling but actual elements.
In order to do that, I iterate across all the pages and display them 1 by 1 with the checkboxes.

  <q-checkbox label="Cover" />
  ...
  <vue-pdf-embed :source="src" disableAnnotationLayer disableTextLayer :page="i" />

Problem 1: as you can see, I'm multiplying the instances of the component, one per page.
Problem 2: this pages variable can contain a number only once the document has been loaded. But it can be loaded only if we execute the v-for which requires pages to contain a number (that's a deadlock).
So to solve that, I've put yet another instance at the very top, that loads but is not displayed:
<vue-pdf-embed :source="src" disableAnnotationLayer disableTextLayer @rendered="loaded" v-show="false" />

When that instance emits rendered, I can populate pages that in turns triggers the v-for and displays the individual pages.
So going back to the initial issue and the one in #9.
For problem 1, it's actually not a problem if the underlying resource (the PDF) is loaded only once in memory and shared.
For problem 2, this is due to the fact that the loading of the PDF is done inside the VuePdfEmbed component, and not outside like in, e.g., vue-pdf.
I'm not bothered that much by the workaround, this part is not client-facing, but I was wondering if there is a better way of doing it.
Cheers!

I see the problem, thanks for the details.
Could you address the question about memory usage (if it is loaded once for the same sources) in the PDF.js repo? At first glance it doesn't seem that way, but I can be wrong.

That's amazing, thanks a lot! I'll check that out soon. Cheers 👍

Changes from @mweimerskirch have been included in 1.1.2
In this release, a getDocument function was also attached to the exported module. So that you can load the PDF document outside the component as follows:
<template>

      <vue-pdf-embed :source="pdfSource" :page="1" />
      <vue-pdf-embed :source="pdfSource" :page="3" />

</template>

Hope this helps

We've tested on our end, and we're not exactly sure about this bit:
https://github.com/hrynko/vue-pdf-embed/blob/c1e20be7a2a45e1ccd0dcc0450b4c71ad65eac25/src/vue-pdf-embed.vue#L165
When packed, all the class names are supposed to be minimised, so I don't think "PDFDocumentProxy" actually exists. When we run
(await VuePdfEmbed.getDocument('<PDF_SOURCE>').promise).constructor.name

it gives us e, the minimised name, so it seems we never end up in the true side of the if.

@mweimerskirch could you take a look at that?


Switch camera output for nice animation

Some users might not want to have a camera view of themselves all the time and the highlights might be off putting. We should allow the use to replace their face with something else so they feel more comfortable. Whatever they replace themselves with should still be able to show blinks.
I have in my head something similar to Apple Memojis

What would a first version of this look like?
Would something like a character based on the logo work?

Hi @sogen - thanks for the interest. We aren’t sure. But yeah - I guess something on the logo would be a start. It’s a bit square but it’s not gender specific and the eyes are nice and clear.


Added item to Web Development

Added FreeCodeCamp to Web Dev section. Free Code Camp is a massive community. Offers free courses to learn coding on its website. I learnt most of the Front End from it

Free Code Camp is already in the Resources with certificates options section. I also almost added it to Web Development in my PR until I realized it was in this section already. I am not sure if items such as this should be listed twice, because Free Code Camp can easily fit in both categories.

Free Code Camp is already in the Resources with certificates options section. I also almost added it to Web Development in my PR until I realized it was in this section already. I am not sure if items such as this should be listed twice, because Free Code Camp can easily fit in both categories.

This is a good point thanks, on the one hand I don't want to duplicate on the other hand I would rather duplicate & be sure people got the information they need. We added in a certificates section as part of some feedback given in this issue #6 .I've merged for now, but feel free to open an issue for this & we can see how to improve.


fix: install inetutils on Arch Linux

This provides the hostname command used in the install.sh script, which is not available by default on Arch Linux.
The hostname command is used here:
https://github.com/coollabsio/coolify/blob/ef56797c7d592658a6f5fc1dde7568f23d8b82c4/scripts/install.sh#L545
Some relevant resources:

https://bbs.archlinux.org/viewtopic.php?id=125308
https://archlinux.org/news/deprecation-of-net-tools/
https://man.archlinux.org/man/hostname.1.en

Changes
From:
https://github.com/coollabsio/coolify/blob/ef56797c7d592658a6f5fc1dde7568f23d8b82c4/scripts/install.sh#L161-L164
The inetutils package was added to the pacman install command for Arch Linux
Issues
Arch Linux does not ship with the hostname command by default (provided by inetutils). For example:

Reproduction command logs
$ sudo docker run -it --rm archlinux:latest
Unable to find image 'archlinux:latest' locally
latest: Pulling from library/archlinux
7fa4bb7a964d: Pull complete
f8d609407a8e: Pull complete
Digest: sha256:901cf83a14f09d9ba70b219e22f67abd4d6346cb6d3f0c048cd08f22fb9a7425
Status: Downloaded newer image for archlinux:latest
[root@13a16f1a73a0 /]# hostname
bash: hostname: command not found
[root@13a16f1a73a0 /]# pacman -Sy --noconfirm --needed inetutils
:: Synchronizing package databases...
 core downloading...
 extra downloading...
error: restricting filesystem access failed because the landlock ruleset could not be applied!
resolving dependencies...
looking for conflicting packages...

Package (1)     New Version  Net Change  Download Size

core/inetutils  2.5-1          1.07 MiB       0.30 MiB

Total Download Size:   0.30 MiB
Total Installed Size:  1.07 MiB

:: Proceed with installation? [Y/n]
:: Retrieving packages...
 inetutils-2.5-1-x86_64 downloading...
error: restricting filesystem access failed because the landlock ruleset could not be applied!
checking keyring...
checking package integrity...
loading package files...
checking for file conflicts...
:: Processing package changes...
installing inetutils...
:: Running post-transaction hooks...
(1/3) Creating system user accounts...
Creating group '_talkd' with GID 972.
Creating user '_talkd' (User for legacy talkd server) with UID 972 and GID 972.
(2/3) Reloading system manager configuration...
  Skipped: Current root is not booted.
(3/3) Arming ConditionNeedsUpdate...
[root@13a16f1a73a0 /]# hostname
13a16f1a73a0

As an aside, I also tested the other OS-es, and most come with the hostname command available by default, except Arch and Opensuse, at least based on their docker images.

I think a better approach would be to use ip or another arch specific command if the first one fails instead of adding another package, can you please change this.

Thanks @peaklabs-dev - I also realised that even with inetutils providing hostname for arch, the -I flag isn't supported.
I've changed the PRIVATE_IPS to:
hostname -I 2>/dev/null || ip -o addr show scope global | awk '{print $4}' | cut -d/ -f1

where both the hostname + ip command gives the same addresses (different order) on a Debian/Ubuntu host., and fallback to the ip command on Arch Linux.


Fix key_value_store path in Getting Started guide

Hi! I came across this issue while going through the Getting Started guide.
Apify.getInput() returned null even though I had an INPUT.json file at the specified path. Turns out this is because this path is meant to be snake_case, not kebab-case.
INFO  System info {"apifyVersion":"0.20.4","apifyClientVersion":"0.6.0","osType":"Windows_NT","nodeVersion":"v12.12.0"}

Thanks a lot for pointing this out!


Explore overheads of CcdCallbacks / idle tracing

Building AMPI-only with "-DCSD_NO_IDLE_TRACING=1 -DCSD_NO_PERIODIC=1" and running bench/ampi/pingpong shows an overhead of ~200ns on my Mac for a multicore build with +p2 +vp2. We don't yet know whether the overhead is from calling timers or from calling them too often or from poor locality in the scheduler / CcdCallback data structures.

Justin and myself looked into this and merged optimizations to address the overheads. The biggest problem was calling timers unconditionally everytime CcdCallbacks were executed or a condition was raised. This in combination with the sched loop conditional led to high timer overheads.


Fix github action for dependabot

This patch fixes the current github action for dependabot.
/cc @kvmware @dprotaso

I verified by https://github.com/nak3/pkg/pull/4

/lgtm


bump the Go image to 1.20

Proposed changes
This PR bumps Go to 1.20. Go version 1.16 is old, unsupported and has known CVEs. These known vulnerabilities have been addressed in newer versions of Go.
Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[x] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[X] I have read the CONTRIBUTING doc
[X] I have signed the commit for DCO to be passed.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Dependency
n/a
Special notes for your reviewer:

Thanks for the raise PR. But the Upgrade version of Go has lots of side effects. And I know that the Litmus team has a long-term plan for this topic. So how about resolving other issues first?

@namkyu1999: Perhaps you could provide an example of such an incompatibility between this code base and a newer version of Go? It's a very bad idea to use old and unsupported versions of Go. Bugs which have long been fixed in the compiler are still present in Go 1.16. This isn't related to generating slower code. This is related to having correct Go code compiled to the wrong machine instructions.
I've sent this PR while evaluating litmus for a deployment. I understand if compiler bugs and security don't represent a concern for this project.

hey @unclejack,
I fully understand that upgrading version of Go is really important.
But If you want to migrate successfully, you need to change more files than you do now.
For example, you also need to change files like go.mod, each backend component(graphql-server, authentication server …), and relevant libraries.
Here’s a sample comment: https://github.com/litmuschaos/litmus/pull/3923#discussion_r1160668942
Also, You need to make sure that the changes you make don't affect the current service.
The topic of raising the language version came up at the Litmus contributor meetup yesterday. If you'd like to address this issue, you can join the next meetup and share your thoughts, or you can join the litmus channel on Slack and say you're available for this issue. Since you haven't contributed to litmus yet, I'd recommend doing a little PR first.

@namkyu1999: It's not that important that I haven't contributed to your project before. I'm familiar with Go.
Your initial reply left me with the impression that you speak in the name of all the maintainers and that such a change is undesirable. Based on what you've written in your second reply, it seems that it's perceived more as a technical issue.
I'll check what steps need to be taken to test the various components. If it's reasonable, I'll go through that process. Updating go.mod isn't an issue as far as I'm concerned. It's just a task.
In the end, it's not something I insist on doing at all costs.


chore(deploy): don't name project in firebase deploy

Naming the project would sometimes cause gulp to report An unexpected error has occurred with exit code 2.

LGTM. Thanks for the fix!


bump: Playwright to 0.16.0

Maybe it makes sense to exclude playwright entirely from the project as we did in jest-playwright? So only playwright-core will be used for types.
So the user can decide which browser / npm package he wants to install in the end to save bandwidth. (playwright vs. playwright-chromium etc.)
Disadvantage for that is definitely that sometimes it can lead to issues if playwright-runner is not compatible with the latest Playwright version.

So the user can decide which browser / npm package he wants to install in the end to save bandwidth. (playwright vs. playwright-chromium etc.)

I've been thinking about depending on playwright-core, and then downloading browsers the first time a test requires them.
I don't want to support old versions of playwright in new versions of the runner.

Thanks for the PR!


Is there a way to generate the video without reference and source images?

Is there any parameter to switch off and generate the synthesis video only?

Currently, there is only the fused video (source + reference + synthesis). if you only want the synthesis video, you can make it
with single-frame images by using FFmpeg

Currently, there is only the fused video (source + reference + synthesis). if you only want the synthesis video, you can make it
with single-frame images by using FFmpeg

Good idea, I'll use the PNG images in my editing software directly.

Good idea, I'll use the PNG images in my editing software directly.


Rename project

It is good to give better name of project (repo) other than "how_to". Maybe "send_over_tcp" or similiar but not simply "how_to" - it is equivalent to "my_project_1" = no information.

This repository was created by me as a repository for examples which can be useful for me in future (SimpleTcpServer / send_mail etc). "How To" - it's a good name for it, to my mind.
Thank you for your comment.

If this project is for your own needs, then yes - the name must be useful from your viewpoint. I decided that you share it and for that reason was the recommendation.