Adding pull request auto update

What does this do?

When a PR gets merged, the other PR gets auto updated and would trigger test pass

So what other PR gets autoupdated?  Cloud-operator?  And what test pass do we trigger?  the appliance test pass?

So what other PR gets autoupdated? Cloud-operator? And what test pass do we trigger? the appliance test pass?

Its only for this repo.
If another PR from this repo gets merged to master, you dont have to rebase and push your new change in your PR.
This would try to update and merge those changes into your PR and kick the test pass automatically.


Help with off device symbolication

I don't see any information on how to do off device symbolication. Is there any documentation on how to do this with KSCrash? We strip all symbols so on-device symbolication isn't an option.

Duplicate issue to #109


Title url changing

Source information
Rizz Comic 1.4.2
Source language
English
Source new URL
https://rizzcomic.com/
Other details
the url is still the same "https://rizzcomic.com/" but the library title don't update even if you remove it. it's like the problem with asura scans extension, but asura scans extension have the toggle in the settings to continue to use the entry even the changes in url.
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL is not updated yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

The extension already tries to find new url. Restart app if you are seeing old urls


Type checks: failure to infer type after match statement when returning from specific case statements

Environment data

Language Server version: 2024.4.1 (pyright version 1.1.356, commit 6652c4a8)
OS and version: Debian Bookworm (12.5)
Python version: OS-installed: python3 3.11.2-1+b1 (bookworm/main)

Relevant config:

python.analysis.typeCheckingMode: standard

Code Snippet
Pylance detects when types are being checked before use, so in the following snippet f1()and f2() have no type check errors, however in f3() it fails to detect the correct types as I used return instead of a default case. This is a common pattern using if statements as seen in f1().
from typing import Optional

def other(a: str, b: str) -> bool:
    return a == b

def f1(a: Optional[str], b: Optional[str]) -> bool:
    """f1"""

    if a is None and b is None:
        return True
    if a is None:
        return False
    if b is None:
        return False
    return other(a, b)

def f2(a: Optional[str], b: Optional[str]) -> bool:
    """f2"""

    match (a, b):
        case (None,None):
            return True
        case (_, None):
            return False
        case (None, _):
            return False
        case _:
            return other(a, b)

def f3(a: Optional[str], b: Optional[str]) -> bool:
    """f3"""

    match (a, b):
        case (None,None):
            return True
        case (_, None):
            return False
        case (None, _):
            return False
    return other(a, b)

In f3(), return other(a, b) is highlighted for both parameters with the following type check error:
Argument of type "str | None" cannot be assigned to parameter "a" of type "str" in function "other"
  Type "str | None" cannot be assigned to type "str"
    "None" is incompatible with "str"Pylance[reportArgumentType](https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportArgumentType)
-----
(parameter) a: str | None

Expected behavior
The f2() and f3() functions are semantically identical. Pylance should ignore the types that end up in an unconditional return after checking in match/case statements, like it's doing for if statements.
Logs
Let me know if additional logs are required.

Could someone from the pylance team please transfer this to the pyright repo since it's a core type checking issue? Thanks.


[$1000] mWeb / Safari - Chat View Shrinks When SoftKeyboard Is Opened While Scrolling

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Click On Chat
Open Soft Keyboard
Scroll Through The Chat

Expected Result:
Chat View Should Behave Normal
Actual Result:
Chat View Shrinks While Scrolling
Workaround:
unknown
Platform:

Where is this issue occurring?

Mobile Web Safari

Version Number: 1.2.37-1
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/206814873-e2c59b7d-7a04-4310-8af5-7e9f9b2ebd70.MP4
https://user-images.githubusercontent.com/43996225/206814891-7e513592-a5ff-4ea0-8e35-bc3ab81763bb.mov
Expensify/Expensify Issue URL:
Issue reported by: @syedsaroshfarrukhdot
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1670590388576729
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011564e2219ad85e68
Upwork Job ID:<PHONE_NUMBER>133781504
Last Price Increase: 2022-12-13

This is reproducible on ios16.1 safari

Yep, I too was able to reproduce it. This one seems like it can be fixed externally.

I think it's the same issue here (same root cause).
https://github.com/Expensify/App/issues/8731

This issue seems to be a weird problem with iOS Safari itself not specifically Expensify. When the keyboard is opened, it actually pushes the main view up and it weirdly adds an extra scrollbar to allow users to scroll to the top of the main view. There is no clear solution to this at this stage and it happens with most websites.

Got it, thanks for the explanation @priyeshshah11. Given that context and the fact that we closed https://github.com/Expensify/App/issues/8731 due to it being a known bug that can only be solved with a complex workaround, I'd suggest we just close this one. Any objection, @Luke9389?

I think we can keep it open maybe  Apple provided solutions.
also, why not go with a workaround since it's affected us in many cases, and for now no ideal solution!
for example, I see google and apple products are not affected by this issue.
What do you think?

and apple products are not affected by this issue. so they solve it somehow

@ahmdshrif Apple products are affected, the video attached above is apple. I think workaround can work if it doesn't have any negative side effects technically or in user experience but it's up to the team.

I mean apple websites . or google search, Gmail..etc. on safari ios.

I mean apple websites . or google search, Gmail..etc. on safari ios. I remember I tests them on the old issue. and they work well. we can retest if you don't agree on this point.

Oh ok, got what you mean 👍 but I did test gmail & apple.com and they both should the same behaviour as this issue.

I am not sure what steps you follow because there are many repro steps like what I reported before https://expensify.slack.com/archives/C01GTK53T8Q/p1655052706729669 or steps on an old issue.
here is my test on google search and I can't repro it.
https://user-images.githubusercontent.com/21364901/207885913-ecc8a223-57ed-499a-84cb-fdd6f5b0d0e9.mp4

here is my test on google search and I can't repro it.

Google search looks like a different case, because of that recommendations list. Try going to a form, I tried it by going to checkout on apple.com and I could see the extra whitespace at the bottom.
Having said that, if you think you have a possible workaround/solution then feel free to go ahead and propose it.

Having said that, if you think you have a possible workaround/solution then feel free to go ahead and propose it.

not yet. but I just say we can keep it open if other websites solve it. if not we should close I agree.

@stephanieelliott I agree, I think we should close it.


MVP rollout of Protagonist

Rollout Protagonist and associated services to existing dlcs-io estate.
This will enable us to run DDS integrations against Protagonist for some initial testing. The resources created will be completely separate from the existing DLCS, the only things that will be shared are networking resources (VPC, subnets, load-balancer etc).

Summary of resources deployed:
1.2TB scratch disk Lustre FSx for Lustre volume.
ASG/capacity provider to provision EC2 host. Launch template mounts above Lustre volume to /fsx/images
ECS cluster with the following services running on the EC2 host (as they need Lustre volume mounted):

Orchestrator - single instance
Cantaloupe using Kakadu - scale based on memory usage
Lustre-watcher

The following services run as Fargate services:

API
Engine - has Appetiser side car and slightly larger scratch disk
Fireball
Thumbs
Thumbs-resize
Specialserver - Cantaloupe using Kakadu, configured to stream directly from S3 to avoid the need to orchestrate

Separate RDS instance, secrets, parameter-store values etc.
Ingest + API Tests
These will be tested thoroughly as part of DDS integration, at this stage I just conducted some basic tests. e.g. Ingest and image/video/audio, ingest batches, request PDFs
Image Serving Load Tests
Ran various tests of image serving to check performance - there are definitely things we can do to improve but some figures. Tests ran with Locust scripts.
For comparison the current Orchestrator is averaging around 1500 requets in a 5 min period with drops to ~200 and max of ~5000 (based on last 7 days from Cloudwatch metrics for Orchestrator TargetGroup)
All test runs started by reading several thousand asset-ids and then making various requests for them. Either:

Thumbnail requests (/90,/, /!200,200/, /!400,400/)
Resized full images (/!300,300/, /^!600,600/)
Full image (/full/max/)
All possible tiles for image

Title
Requests
Failures
Median Response
Average Response
Min Response
Max Response
Req/s

5u, 5m (deliverator)
3402
310
190
429
64,5
12613
11.5

5u, 5m
5418
0
120
272
57.5
6825
18

10u 10m
11930
0
140
495.51
58.59
14485.82
19.98

50u, 15m
22928
70
250
1883.09
58.12
54247.17
25.66

50u, 15m, mem
22183
2
280.0
1928.99
58.23
43119.84
24.96

75u, 20m
30417
104
390.0
2799.02
57.6
68700.88
25.35

75u, 20m, mem
24893
8
600.0
3444.52
59.32
98930.11
20.68

Notes:
Format of title is <number-users>u, <duration-mins>m, so 20 simulated users for 10 mins would be 20u, 10m.
Most protagonist runs had Cantaloupe + Orchestrator on a single CPU-optimised instance (c6a.large), the exception being those with ,mem in title were ran with a memory-optimised instance (r5b.large)
5u, 5m + 10u, 10m tests were ran with ~3700 images in DB but all others had ~7800.

Note on various runs:

5u, 5m (deliverator) - Simulate 5 users for 5 mins on existing platform. Didn't want to run higher figures as this is production platform and currently in use. The high failure count is partly due to use of ^! size syntax which isn't understood by IIP image.
5u, 5m - Like for like with existing platform. No scaling.
10u, 10m - As above.
50u, 15m - added autoscaling to Cantaloupe + Special Server. Restarted services + wiped Lustre prior to run.

Special-server scaled to 2 instances
Cantaloupe scaled to 2 instances
All failures were /full/max/ requests.

50u, 15m, mem - as above but with memory-optimised instance.

Special-server scaled to 2 instances
Cantaloupe did not scale
1 failures due to /full/max/, 1 was a 502 for a !200,200 thumb

75u, 20m - ran immediately after 50u, 15m, without clearing caches but set all svc counts = 1.

Special-server scaled to 2 instances
Cantaloupe scaled to 2 instances
All failures were /full/max/ requests.

75u, 20m, mem - as above but with memory-optimised instance and I manually set Cantaloupe count = 2 as previous run didn't scale.

Special-server scaled to 2 instances
Cantaloupe scaled to 3.
All failures were /full/max/


different tooltips offset when hovering first time and second times

When hovering a tooltip item: http://v4-alpha.getbootstrap.com/components/tooltips/
the first time offset will be different from other times, any fixes?

Could you please paste the code from the first and second case to see it??

The screenshot is taken from official site using chrome on macOS

Confirmed on Mac Chrome 55.

Confirmed on Windows 10 Chrome 54

confirmed on windows firefox developer edition 51.0a2

I used setInterval(function() { console.log($('.bs-tether-element')[0].outerHTML) }, 1000); to pull the HTML for the tooltips.
First activation:

Tooltip on top

Second activation:

Tooltip on top

Third activation:

Tooltip on top

The most interesting exemples are the first and the second.
The difference between the second and the third is the just the tooltip's ID because when we hide the tooltip is destroyed so the generated ID will be different on the next show.
See the difference between first and second exemple : https://www.diffchecker.com/kyilr4EM

Just to note: this also happens on Safari 10 on OS X. And, strangely, it only happens with the top, left and html examples, but not the right and bottom ones.


Proper clean up in Tortoise ORM plugin example

At the moment example on Tortoise ORM plugin doesn't work properly as it doesn't close db connections on app shutdown.
Tortoise ORM states that it is very important to do clean up or otherwise there will be problems with program termination. Indeed, when you try to run the example under uvicorn it will hang on Ctrl-C (see screenshot) forcing user to kill the program.

You can check yourself with the following piece (I simply added uvicorn.run to the example code) with python test.py

test.py
import uvicorn

from typing import cast

from tortoise import Model, Tortoise, fields
from tortoise.connection import connections

from starlite import Starlite, get, post
from starlite.plugins.tortoise_orm import TortoiseORMPlugin

class Tournament(Model):
  id = fields.IntField(pk=True)
  name = fields.TextField()
  created_at = fields.DatetimeField(auto_now_add=True)
  optional = fields.TextField(null=True)
  events: fields.ReverseRelation["Event"]

  class Meta:
      ordering = ["name"]

class Event(Model):
  id = fields.IntField(pk=True)
  name = fields.TextField()
  created_at = fields.DatetimeField(auto_now_add=True)
  tournament: fields.ForeignKeyNullableRelation[Tournament] = fields.ForeignKeyField(
      "models.Tournament", related_name="events", null=True
  )
  participants: fields.ManyToManyRelation["Team"] = fields.ManyToManyField(
      "models.Team", related_name="events", through="event_team"
  )
  address: fields.OneToOneNullableRelation["Address"]

  class Meta:
      ordering = ["name"]

class Address(Model):
  city = fields.CharField(max_length=64)
  street = fields.CharField(max_length=128)
  created_at = fields.DatetimeField(auto_now_add=True)

  event: fields.OneToOneRelation[Event] = fields.OneToOneField(
      "models.Event", related_name="address", pk=True
  )

  class Meta:
      ordering = ["city"]

class Team(Model):
  id = fields.IntField(pk=True)
  name = fields.TextField()
  created_at = fields.DatetimeField(auto_now_add=True)

  events: fields.ManyToManyRelation[Event]

  class Meta:
      ordering = ["name"]

async def init_tortoise() -> None:
  await Tortoise.init(db_url="sqlite://:memory:", modules={"models": [__name__]})
  await Tortoise.generate_schemas()

async def shutdown_tortoise() -> None:
  await connections.close_all()

@get("/tournaments")
async def get_tournaments() -> list[Tournament]:
  tournaments = await Tournament.all()
  return cast("list[Tournament]", tournaments)

@get("/tournaments/{tournament_id:int}")
async def get_tournament(tournament_id: int) -> Tournament:
  tournament = await Tournament.filter(id=tournament_id).first()
  return cast("Tournament", tournament)

@post("/tournaments")
async def create_tournament(data: Tournament) -> Tournament:
  assert isinstance(data, Tournament)
  await data.save()
  await data.refresh_from_db()
  return data

app = Starlite(
  route_handlers=[get_tournament, get_tournaments, create_tournament],
  on_startup=[init_tortoise],
  plugins=[TortoiseORMPlugin()],
)

if __name__ == "__main__":
  uvicorn.run(app, host="<IP_ADDRESS>", port=8000)

Dependencies
aiosqlite==0.17.0
starlite[picologging]==1.20.0
tortoise-orm==0.19.2
uvicorn[standard]==0.18.3

Also while linked Tortoise ORM docs offers to use Tortoise.close_connections it seems that this method has been deprecated in 0.19.0 and connections.close_all should be used instead.
https://tortoise.github.io/setup.html#tortoise.Tortoise.close_connections
PR Checklist

[x] Have you followed the guidelines in CONTRIBUTING.md?
[ ] Have you got 100% test coverage on new code?
[ ] Have you updated the prose documentation?
[ ] Have you updated the reference documentation?

Sorry, I was too lazy and edited the docs in Github WebIDE. It was a bad idea :-( Had to do force pushes to fix typo.

Looks good, thank you!
@all-contributors please add @jtraub for docs


FlutterNativeView.runFromBundle should support parameters

This is important for projects who want to integrate Flutter into their existing apps. There's currently no reasonable way to treat FlutterView like a regular Android view:

It does not support different endpoints in Dart. You can specify different endpoints via runFromBundle but if you expose multiple entry points in Dart, the AOT compiler tree-shakes them.
There's no easy way of passing parameters to Dart the way you would pass parameters between Android activities via an Intent. The expected way right now is for the Dart app to run and register callbacks that are tied to a PlatformChannel and then the Android shell calls that channel. This works fine for standalone Flutter apps but for a small Flutter activity integrated into a larger Android app, it is a pain.

If we support parameters in runFromBundle, we could do something like this on Dart:
void main([List<String> args]) {
  if (args?.length != 1) return;
  switch (args[1]) {
    case 'page1':
      runApp(new Page1());
      break;
    case 'page2':
      runApp(new Page2());
      break;
  }
}

Ideally the params can be passed further down the app and the app can decide how to route pages but you get the idea.

I assume the same thing will be a problem in iOS as well.

On iOS, we could do this by specifying the launch arguments as an array (or maybe a map) on FlutterDartProject. Dart IO seems to also populate Platform.environment. Maybe putting these parameters in that map is more idiomatic.

Heart has to work around this issue in their prototype at the moment which requires them to pass data into FlutterView from their native activity at construction time. For now the workaround is to construct the view and have Flutter ask for the data via PlatformChannel once it is ready.

Does setInitialRoute address this suitably for you? It's how the design currently intends for you to pass data from the OEM side to the Dart before the Dart code starts up.

@Hixie That indeed works. I think it should be enough to cover this use case. It would be nice to be able to pass structured data but definitely low priority.

Sending structured data is indeed something that would be interesting, but we'd rather not reinvent system channels just for the initial route, so our theory is that if you want to send structured data, you'd send a sentinel that indicates to the dart code "hey, contact the OEM side to find out what the structured data is".

Thanks @Hixie. That is precisely the workaround I had recommended.
Just as an anecdote for people stumbling on this: initialRoute only works if you are in foreground and have an active FlutterView. It does not work if you are trying to call an arbitrary Dart fn in the background via FlutterNativeView. For instance, the AlarmManager plugin currently supports VoidCallback type methods to be run.
I imagine we will eventually need to pave the path to setting up PlatformChannel's in the background. These channels already allow you to pass parameters and return values from either side. That's ultimately what AlarmManager plugin needs to be able to do. Since we have no customer use case demanding this (yet), I am OK closing it.

The use case for this is firebase messaging isn't it?  Right now firebase messaging drops messages entirely on the floor for android.  It should really be able to call into the native view, and pass in the remote message, to do some useful background processing.  This makes firebase messaging a lot less useful.


redirect to desired page after login

Behavior: User clicks on link to private resource on the OSF and is instructed she must sign in to view resource. User signs in and is directed to her own dashboard and has to navigate to the resource she was trying to reach in the first place.
Desired behavior: Upon logging in, user is directed to the private resource rather than her dashboard.

That should be what's happening (and I tested it just now and it seemed to be working that way).
It will only work if you sign in on the page that's presented to you when you try to access the resource; if you land on that page then click the sign-in button (which I have done many times) it won't take you to the proper page. Otherwise it will (or seems to according to my testing and how it's supposed to work).

@saradbowman can you provide an example (perhaps not exactly the one the user reported but one that adequately mimics the issue?)?

Actually... no. I didn't realize this was intended behavior, and now I
can't replicate the error on Safari or Chrome. Gathering more information
from the user....
On Mon, Jan 26, 2015 at 11:26 AM, Jeffrey Spies<EMAIL_ADDRESS>wrote:

@saradbowman https://github.com/saradbowman can you provide an example
(perhaps not exactly the one the user reported but one that adequately
mimics the issue?)?
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/1582#issuecomment-71488558
.

@saradbowman is this still occuring or can we close this issue?

We can close this out -- we couldn't repeat the error here and the user
reported that it seemed to resolve itself.
On Sun, Feb 22, 2015 at 11:30 AM, Harry Rybacki<EMAIL_ADDRESS>wrote:

@saradbowman https://github.com/saradbowman is this still occuring or
can we close this issue?
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/1582#issuecomment-75443515
.

pinging @lbanner

Thank you both -- Closing.


refactor: reflect kernel pr 11757

Upstream change: https://github.com/risingwavelabs/risingwave/pull/11757
Draft until it is released.

Will this change affect existing users that are already running with hummock+xx://%s@%s

I think we should keep existing users using bucket@rootand new users(new clusters) using bucket only

Does the code path to delete cluster need any changes? e.g. for a cluster with @, is it OK to delete its object store by path without @.

Will this change affect existing users that are already running with hummock+xx://%s@%s

No. Helm chart is unlike the operator. It won't touch the deployment any more until the user performs another action. But the change will indeed break the compatibility between the latest chart and the older kernel versions.
Helm has provided a way to workaround this, by specifying the chart version. Users can use older chart versions once they found the latest one incompatible with their kernel versions.


SSL settings to get an A rating

This was previously vulnerable to POODLE, as shown by https://www.ssllabs.com/ssltest/analyze.html
The new settings come roughly from https://cipherli.st/, minus the X-Frame-Options header they include since we use CSP, X-Frame options are deprecated, and we have several use cases for iframe embedding.
/cc @minrk @smashwilson

Seem fine to me.


fix: collapsed editor init added

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


preprocessing subject selection old

The old file for object selection is still there. This is quite specific to a project, should we update? remove?

Hmm... @AnaTomomi, is it still there? Is this about niimpy/preprocessing/subject_selection.py?
At this point I would assume it's not used unless it's documented. Do we have similar functionality elsewhere?


AS0011 Compiler Error for Enum Extension

Starting this afternoon we get an AS0011 compiler error for an existing Enum Extension.
AL Compiler version <IP_ADDRESS>22
Our Enum values don't have a mandatory affix, therefore our app does not longer compile successfully (it did before).
enumextension 5042650 "BET IND Comment Line Table Name" extends "Comment Line Table Name"
{
    value(5042650; "Packaging Structure Definition Header") { }
    value(5042651; "Packaging Structure Definition Line") { }
}

I tried to mark those Enum values as Obsolete, but the error still occurs.
I also tried to remove the Enum Extension or to remove the invalid Enum Values, but then our build pipeline (based on the CI/CD example from @freddydk) using Run-AlPipeline shows a breaking change, since we have removed it.
error AS0083: The enum value with name 'Packaging Structure Definition Header' and ID '5042650' was found in the previous version of 'EnumExtension "BET IND Command Line Table Name"', but is missing in the current extension. This might break the upgrade of existing installations and dependent extensions.
So... I can not fix it.
My current solution is using pragma:
(I also refer to a different Enum, therefore I could build a new correct Enum Extension)
enumextension 5042650 "BET IND Comment Line Table Name" extends "Service Comment Line Type"
{
#pragma warning disable AS0011
    value(5042650; "Packaging Structure Definition Header")
    {
        ObsoleteState = Pending;
        ObsoleteReason = 'Prefix missing';
        ObsoleteTag = '31.0';
    }
    value(5042651; "Packaging Structure Definition Line")
    {
        ObsoleteState = Pending;
        ObsoleteReason = 'Prefix missing';
        ObsoleteTag = '31.0';
    }
#pragma warning restore
}

I thinks this Yammer thread covers it:
https://www.yammer.com/dynamicsnavdev/#/Threads/show?threadId=1415324861079552
The upshot was:

We discussed this internally and we won't enforce the rule on existing violations due to the manual cost and the risk that updating enum value IDs and names can be imply.
For newly introduced enum values without affixes (they don't exist in the baseline version), we will report an error with AS0011. This way we prevent more violations to be added.
The rule AS0098 will then be reporting a warning on enum values which do not have affixes and that were already defined without affixes in the baseline version. It is kept as a warning for awareness that the enum names could collide one day with the enum values defined in another app.

Btw, you'll also run into warnings and potential runtime issues with enumerator names that long; see
https://github.com/microsoft/AL/issues/6740#issuecomment-911443709

This issue should be now fixed in version 8.2.545335 of the AL Language extension.
The behavior of the affix validation is now as follows for enum values without affixes defined in enum extensions:

if you don't have a baseline set in your AppSourceCop.json, AS0098 is reported (warning)
if you have a baseline and your enum value is in the baseline without affix, AS0098 is reported (warning)
if you have a baseline and your enum value is not in the baseline, AS0011 is reported (error)


Bug: default not generated in OpenAPI schema for body as models

Description
Declaring default fields (various ways) generates OpenAPI spec with missing default declarations.
Also, when/if fields are marked into required varies a bit.
Didn't test what happens when requests are sent - are the defaults actually picked up  runtime. But at least the OpenAPI schema generation fails.
(The docs don't mention how these are supported so a bit difficult to know how it's intended.)
MCVE
import dataclasses
import json
from dataclasses import dataclass
from typing import Annotated

import msgspec
from litestar import post
from litestar.app import Litestar
from litestar.params import Parameter
from msgspec import Struct
from pydantic import BaseModel, Field

class PydanticBody(BaseModel):
    field1: Annotated[str, Field(default="dummy")]
    field2: str = Field(default="dummy")
    field3: str = "dummy"
    field4: Annotated[str, Parameter(default="dummy")]

@dataclass
class DataclassBody:
    field1: Annotated[str, Parameter(default="dummy")]  # default generated, but declared as required
    field2: Annotated[str, dataclasses.field(default="dummy")]  # no default, marked as required
    field3: str = Parameter(default="dummy")
    field4: str = "dummy"
    field5: str = dataclasses.field(default="dummy")

class MsgspecBody(Struct):
    field1: Annotated[str, Parameter(default="dummy")]  # default generated, but declared as required
    field2: Annotated[str, msgspec.field(default="dummy")]  # no default, marked as required
    field3: str = Parameter(default="dummy")
    field4: str = "dummy"
    field5: str = msgspec.field(default="dummy")

@post("/1")
async def pydantic_handler(data: PydanticBody) -> None: ...

@post("/2")
async def dataclass_handler(data: DataclassBody) -> None: ...

@post("/3")
async def msgspec_handler(data: MsgspecBody) -> None: ...

app = Litestar([pydantic_handler, dataclass_handler, msgspec_handler])

print(json.dumps(app.openapi_schema.to_schema(), indent=4))

Steps to reproduce
This generates:
    "components": {
        "schemas": {
            "DataclassBody": {
                "properties": {
                    "field1": {
                        "type": "string",
                        "default": "dummy"
                    },
                    "field2": {
                        "type": "string"
                    },
                    "field3": {
                        "type": "string"
                    },
                    "field4": {
                        "type": "string"
                    },
                    "field5": {
                        "type": "string"
                    }
                },
                "type": "object",
                "required": [
                    "field1",
                    "field2"
                ],
                "title": "DataclassBody"
            },
            "MsgspecBody": {
                "properties": {
                    "field1": {
                        "type": "string",
                        "default": "dummy"
                    },
                    "field2": {
                        "type": "string"
                    },
                    "field3": {
                        "type": "string"
                    },
                    "field4": {
                        "type": "string"
                    },
                    "field5": {
                        "type": "string"
                    }
                },
                "type": "object",
                "required": [
                    "field1",
                    "field2"
                ],
                "title": "MsgspecBody"
            },
            "PydanticBody": {
                "properties": {
                    "field1": {
                        "type": "string"
                    },
                    "field2": {
                        "type": "string"
                    },
                    "field3": {
                        "type": "string"
                    },
                    "field4": {
                        "type": "string"
                    }
                },
                "type": "object",
                "required": [],
                "title": "PydanticBody"
            }
        }

Litestar Version
2.7.0
Platform

[X] Linux
[ ] Mac
[ ] Windows
[ ] Other (Please specify in the description above)

[!NOTE]
While we are open for sponsoring on GitHub Sponsors and
OpenCollective, we also utilize Polar.sh to engage in pledge-based sponsorship.
Check out all issues funded or available for funding on our Polar.sh dashboard

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified
This, along with engagement in the community, helps us know which features are a priority to our users.

We use the APIs provided by dataclasses, msgspec, and pydantic to parse the details of a model such as the default. Both, msgspec and dataclasses, do not support specifying the field within the annotation but rather, you have to assign it whereas pydantic does (I think it does).
Also, I don't think Parameter is meant to be used for the request/response bodies. They're meant to be used to give extra details for parameters like query params, path params etc.

It seems I was mistaken since the default isn't being generated as part of the schema even when specifying the default value correctly.
from dataclasses import dataclass

from litestar import post
from litestar.app import Litestar
from litestar.openapi.spec import Schema

@dataclass
class DataclassBody:
    field4: str = "dummy"

@post("/2")
async def dataclass_handler(data: DataclassBody) -> None:
    ...

app = Litestar([dataclass_handler])

schema = app.openapi_schema.components.schemas["DataclassBody"]

assert schema.properties
schema = schema.properties["field4"]

assert isinstance(schema, Schema)
assert schema.default == "dummy" # This fails.

There's a few things going on here (some of which have also been discussed elsewhere I think):
@dataclass
class DataclassBody:
    field1: Annotated[str, Parameter(default="dummy")]  # default generated, but declared as required
    field3: str = Parameter(default="dummy")
    field4: str = "dummy"
    field5: str = dataclasses.field(default="dummy")

field1 is simply an invalid way of defining a default for a dataclass. You are essentially "lying" to Litestar by telling it "even though I have not defined a default for this field, I want you to say that it has a default value of "dummy"".
field3 is setting a default value for the field, but, according to dataclass semantics, the default value is Parameter(default="dummy"). Since this is neither an annotation, nor the default value you meant to specify, it's not surprising this isn't working
field4 and field5 should work, and it not working is a bug (and I think a regression, will have to check this out)

class MsgspecBody(Struct):
    field1: Annotated[str, Parameter(default="dummy")]  # default generated, but declared as required
    field2: Annotated[str, msgspec.field(default="dummy")]  # no default, marked as required
    field3: str = Parameter(default="dummy")
    field4: str = "dummy"
    field5: str = msgspec.field(default="dummy")

field1/field2/field3: Same as with the dataclass, this is not a valid way to define a default way for a msgspec.Struct
field4/field5: Also same as with the dataclass, this should work


Recover test upgrade

I clipped out the unspent_note test and test upgrades when I force pushed to dev.

Codecov Report
Patch coverage: 61.22% and project coverage change: +2.21 :tada:

Comparison is base (491b2f7) 56.33% compared to head (c6c8fcf) 58.55%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #263      +/-   ##
==========================================
+ Coverage   56.33%   58.55%   +2.21%
==========================================
  Files          35       35
  Lines        5119     5320     +201
==========================================
+ Hits         2884     3115     +231
+ Misses       2235     2205      -30

Impacted Files
Coverage Δ

zingo-memo/src/lib.rs
72.91% <ø> (ø)

zingocli/src/lib.rs
0.00% <0.00%> (ø)

zingocli/tests/integration_tests.rs
100.00% <ø> (ø)

zingocli/tests/utils/mod.rs
87.50% <ø> (ø)

zingolib/src/commands.rs
0.00% <ø> (ø)

zingolib/src/wallet/data.rs
39.66% <ø> (+12.00%)
:arrow_up:

zingolib/src/wallet/message.rs
77.46% <ø> (ø)

zingolib/src/wallet/keys/extended_transparent.rs
69.07% <52.27%> (-2.36%)
:arrow_down:

zingolib/src/wallet.rs
63.74% <57.57%> (+6.01%)
:arrow_up:

zingolib/src/wallet/keys/unified.rs
60.81% <59.88%> (+4.97%)
:arrow_up:

... and 23 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


AJ-1666 Report PR images to sherlock

https://broadworkbench.atlassian.net/browse/AJ-1666
Before this PR:
The publish-docker workflow:

requires the input input-tag
is only called by the tag workflow
tags the new docker image with both the input-tag sent from tag and the github short hash.

The tag workflow, which is called manually for release:

discovers the new semver tag (e.g. 0.2.117)
passes the new tag to the publish-docker workflow
passes the new tag to sherlock to report the existence of the new docker image.

After this PR:
The publish-docker workflow:

does not require the input input-tag
is called both by the tag workflow and upon PR
If input-tag is present:

tags the new docker image with both the semver input-tag sent from tag and the github short hash
reports to sherlock the existence of the new docker image with the semver tag

If input-tag is not present:

tags the new docker image only with the github short hash
reports to sherlock the existence of the new docker image using the github short hash

The tag workflow:

discovers the new semver tag
passes the new tag to the publish-docker workflow
No longer calls report-to-sherlock, as this is now accomplished in publish-docker.

Reminder:
PRs merged into main will not automatically generate a PR in https://github.com/broadinstitute/terra-helmfile to update the WDS image deployed to kubernetes - this action will need to be triggered manually by running the following github action: https://github.com/DataBiosphere/terra-workspace-data-service/actions/workflows/tag.yml. Dont forget to provide a Jira Id when triggering the manual action, if no Jira ID is provided the action will not fully succeed.
After you manually trigger the github action (and it completes with no errors), you must go to the terra-helmfile repo and verify that this generated a PR that merged successfully.
The terra-helmfile PR merge will then generate a PR in leonardo.  This will automerge if all tests pass, but if jenkins tests fail it will not; be sure to watch it to ensure it merges. To trigger jenkins retest simply comment on PR with "jenkins retest".

Thank you for the writeup describing the before/after state.  One question I still have: can you explain what the presence or absence of inputTag actually means?  Under what circumstances do we omit it or provide it, and who does the providing?

Thank you for the writeup describing the before/after state. One question I still have: can you explain what the presence or absence of inputTag actually means? Under what circumstances do we omit it or provide it, and who does the providing?

@jladieu The inputTag is the semvar generated upon calling the tag workflow for a release - e.g. 0.2.117 when previously we had 0.2.116.  So we only have this when we call the 'tag' workflow and are releasing a new version.  Whereas, on a PR, we haven't updated the version and have no new semvar to pass into the publish-docker workflow, leaving only the git commit hash to distinguish the new docker image.  Is that clearer?


Package refresh should be automatic and hidden.

Why do I have to click "refresh packages". When would I not want them refreshed? Especially annoying if you are updating individual packages, using the Show Only Packages with Updates checkbox. It appears as if the update failed because the item stays in there.

Package refresh is currently a somewhat expensive two part process, so we've tried to avoid doing it unnecessarily. Plus thanks to some code debt (sorry), the refresh isn't 100% clean. That being said, it has gotten less expensive recently, so I could look into do a semi-regular background refresh.
The second half, though sounds like a bug. Anytime you do a package operation, it should refresh the package details in the list. Is that not the behavior you're seeing? If so, could you file a bug w/ the steps to reproduce?

I can confirm this issue. When updating a package, ChocoGUI returns with the full list of packages even though Show Only Packages with Updates is ticked. When the checkbox is unticked and reticked, the recently updated package still shows with the old version. Clicking Refresh Packages does refresh the version, but again shows all packages even though Show Only Packages with Updates is ticked, requiring a further unticking / reticking.  Is this not what you see?

This should actually be fixed by 5b758857.


SPA & SEO Enhancements - Create & Add SiteMap to Google

SPA's need some configuration for web crawlers.   This task is to generate and submit a sitemap.  Please add me as a reviewer for whoever takes this task on.
https://support.google.com/webmasters/answer/183668?hl=en
Use the latest marvel designs.

PR #85 created for this.


"Fix all occurrences of a code issue..." lists different issues then in the problems tab.

Issue Description
I'm using .editorconfig to disable some rules. This works fine for the problems reported, however when I run any of the "Fix all occurrences of a code issue..." from the command palette it sees and will apply disabled rules as valid options.
Steps to Reproduce

Enable Roslyn analyzers and editor config support.
Set a dotnet_diagnostic severity to none.
Bring up an example file that should trigger the rule.
Open the command palette and run a "Fix all occurrences of a code issue..." command. It should bring up a list that includes the disabled rule.

Expected Behavior
I expect that the disabled rules do not appear or apply.
Actual Behavior
The disabled rules do appear and will apply. This is particularly annoying for "fix all issues".
Logs
OmniSharp log
N/a - but it will list that it "fixing" the disabled rules in the log.
C# log
N/a
Environment information
VSCode version: 1.50.1
C# Extension: 1.23.3
Dotnet Information
.NET Core SDK (reflecting any global.json):
 Version:   3.1.302
 Commit:    41faccf259
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19041
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.302\
Host (useful for support):
Version: 3.1.6
Commit:  3acd9b0cd1
.NET Core SDKs installed:
3.1.302 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.App 3.1.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.6 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

awesome-vhdl
puorc
0.0.1

csharp
ms-dotnettools
1.23.3

debugger-for-chrome
msjsdiag
4.12.11

EditorConfig
EditorConfig
0.15.1

gml-support
liaronce
1.7.2

markdown-mermaid
bierner
1.8.1

mermaid-markdown-syntax-highlighting
bpruitt-goddard
1.0.3

python
ms-python
2020.9.114305

unity-debug
Unity
2.7.5

vscode-eslint
dbaeumer
2.1.10

vscode-icons
vscode-icons-team
11.0.0

vscode-nuget-package-manager
jmrog
1.1.6

They also often fail with a message like:
Command 'OmniSharp: Fix all occurrences of a code issue within solution' resulted in an error ("System.OperationCanceledException: The operation was canceled.\n   at System.Threading.CancellationToken.ThrowOperationCanceledException()\n   at System.Threading.SemaphoreSlim.WaitUntilCountOrTimeoutAsync(TaskNode asyncWaiter, Int32 millisecondsTimeout, CancellationToken cancellationToken)\n   at OmniSharp.Roslyn.CSharp.Services.Diagnostics.CSharpDiagnosticWorkerWithAnalyzers.AnalyzeProjectsAsync(Project project, CancellationToken cancellationToken) in /Users/runner/work/1/s/src/OmniSharp.Roslyn.CSharp/Workers/Diagnostics/CSharpDiagnosticWorkerWithAnalyzers.cs:line 267\n   at OmniSharp.Roslyn.CSharp.Services.Refactoring.RunFixAllCodeActionService.FixAllDiagnosticProvider.GetAllDiagnosticsAsync(Project project, CancellationToken cancellationToken) in /Users/runner/work/1/s/src/OmniSharp.Roslyn.CSharp/Services/Refactoring/RunFixAllCodeActionService.cs:line 217\n   at Microsoft.CodeAnalysis.CodeFixes.FixAllContext.GetFilteredDiagnosticsAsync(Task`1 getDiagnosticsTask, ImmutableHashSet`1 diagnosticIds, Nullable`1 filterSpan)\n   at Microsoft.CodeAnalysis.CodeFixes.FixAllContext.GetProjectDiagnosticsAsync(Project project, Boolean includeAllDocumentDiagnostics)\n   at Microsoft.CodeAnalysis.CodeFixes.FixAllContextHelper.GetDocumentDiagnosticsToFixAsync(FixAllContext fixAllContext)\n   at Microsoft.CodeAnalysis.CodeFixes.FixAllContext.DiagnosticProvider.<GetDocumentDiagnosticsToFixAsync>g__GetDocumentDiagnosticsToFixWorkerAsync|3_1(FixAllContext fixAllContext)\n   at Microsoft.CodeAnalysis.CodeFixes.FixAllContext.DiagnosticProvider.GetDocumentDiagnosticsToFixAsync(FixAllContext fixAllContext)\n   at Microsoft.CodeAnalysis.CodeFixes.BatchFixAllProvider.DetermineDiagnosticsAsync(FixAllContext fixAllContext, IProgressTracker progressTracker)\n   at Microsoft.CodeAnalysis.CodeFixes.BatchFixAllProvider.FixSingleContextAsync(FixAllContext fixAllContext, IProgressTracker progressTracker, Dictionary`2 docIdToTextMerger)\n   at Microsoft.CodeAnalysis.CodeFixes.BatchFixAllProvider.FixAllContextsAsync(FixAllContext originalFixAllContext, ImmutableArray`1 fixAllContexts)\n   at Microsoft.CodeAnalysis.CodeFixesAndRefactorings.DefaultFixAllProviderHelpers.GetFixAsync[TFixAllContext](String title, TFixAllContext fixAllContext, Func`3 fixAllContextsAsync)\n   at OmniSharp.Roslyn.CSharp.Services.Refactoring.RunFixAllCodeActionService.FixSpecificDiagnosticIdAsync(Document document, String diagnosticId, FixAllScope scope, CancellationToken cancellationToken) in /Users/runner/work/1/s/src/OmniSharp.Roslyn.CSharp/Services/Refactoring/RunFixAllCodeActionService.cs:line 205\n   at OmniSharp.Roslyn.CSharp.Services.Refactoring.RunFixAllCodeActionService.Handle(RunFixAllRequest request) in /Users/runner/work/1/s/src/OmniSharp.Roslyn.CSharp/Services/Refactoring/RunFixAllCodeActionService.cs:line 110\n   at OmniSharp.Endpoint.EndpointHandler`2.AggregateResponsesFromLanguageHandlers(ExportHandler`2[] handlers, TRequest request) in /Users/runner/work/1/s/src/OmniSharp.Host/Endpoint/EndpointHandler.cs:line 191\n   at OmniSharp.Endpoint.EndpointHandler`2.HandleRequestForLanguage(String language, TRequest request, RequestPacket packet) in /Users/runner/work/1/s/src/OmniSharp.Host/Endpoint/EndpointHandler.cs:line 234\n   at OmniSharp.Endpoint.EndpointHandler`2.Process(RequestPacket packet, LanguageModel model, JToken requestObject) in /Users/runner/work/1/s/src/OmniSharp.Host/Endpoint/EndpointHandler.cs:line 143\n   at OmniSharp.Stdio.Host.HandleRequest(String json, ILogger logger) in /Users/runner/work/1/s/src/OmniSharp.Stdio/Host.cs:line 258")

This sometimes happens when a refactor/fix takes longer than the timeout. There are other instances where it fails immediately due to various logic errors.

Is there any setting to increase this timeout? or to find out these logic errors to report? Thanks.


TO: Federation mappings route (federations.json) should be per-CDN

Please make this part of the URL, for example:
/internal/api/1.2/:cdnName/federations.json
..that will allow us to simply reconfigure a parameter used by Traffic Router and avoid code changes.


The unified style

memory-request-limit.yaml
    resources:
      limits:
        memory: "200Mi"

memory-request-limit-3.yaml
    resources:
      limits:
        memory: "1000Gi"

But  memory-request-limit-2.yaml
    resources:
      requests:
        memory: 50Mi

Should be:
    resources:
      requests:
        memory: "50Mi"

This change is

Both are OK

If you not think it’s necessary, Close it, Please,


Applying font fallback on Mac OS X?

I wish to use Pomicons on iTerm2 in Mac OS X.
If understood correctly I need to copy fonts from build to ~/Library/Fonts (instead of to ~/.fonts).
But I'm not sure where to put the 10-symbols.conf, provided that Mac OS X font system can understand this fontconfig file.
For now I'm working around this issue using the "Non-ASCII Font" fallback of iTerm2 but this is not a very good solution as you can add only just one font alternative.
I know about nothing about Fonts and how Fonts works on Mac OS X, and I'm also having some difficult in understand what to search on Google to find a solution, any help will be appreciated.

I don’t think that the OS X font system works that way (I should specify in the README that the How to install section is specific to Linux freetype2)
Anyway, the magic words to search are ”OS X font fallback”, with that I found:

https://discussions.apple.com/thread/6509534
http://apple.stackexchange.com/questions/71719/how-does-the-os-x-font-fallback-system-work

As far as I understood, you can configure the fallback fonts in the Plist located at /System/Library/Frameworks/ApplicationServices.framework/Frameworks/CoreText.framework/Resources/DefaultFontFallbacks.plist
With PlistBuddy you can also automate the procedure if you want
Be mind, I don’t know if this will work but hopefully it will, otherwise it could be a starting point :smile:

You have to disable System Integrity Protection by rebooting and holding CMD+R and launching Terminal from the Utilities menu so you can run:
csrutil disable; reboot

Then you can use Xcode.app to open and edit the plist file after first copying it to ~/Desktop:
cp /System/Library/Frameworks/ApplicationServices.framework/Frameworks/CoreText.framework/Resources/DefaultFontFallbacks.plist ~/Desktop

open -a Xcode.app ~/Desktop/DefaultFontFallbacks.plist

In the plist editor, expand "monospace" then click the + icon to add a new row and type in Pomodoro. Do this three more times for FontAwesome, Octicons, and Icomoon until the plist looks like:

Now reboot into recovery mode again (CMD+R) and re-enable System Integrity Protection:
csrutil disable; reboot

Works for me in iTerm2 Build 3.0.5 👍


add IndexController Test

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #521 (9fe4339) into master (6b85577) will increase coverage by 1.25%.
The diff coverage is n/a.

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


Branch2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

34 of 34 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
34

Relevant Lines:
34

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

34 of 34 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
34

Relevant Lines:
34

💛  - Coveralls


Link to https://www.hello-matrix.net/public_servers.php or otherwise maintain a list of public servers

dup of #765

closing as dup


How to subscribe to a location of the user in flutter

I have used roam's roam.startTracking() to start tracking. I want to know how to subscribe to this location and show the location data in another flutter application.   I have tried roam.subscribeUserLocation("userId"), but it returns a bool.
How can I achieve this task of getting the location from one device and showing geo coordinates of that user, on another device in Flutter SDK?

Hey @Ilthizam Once the user if subscribed with the method Roam.subscribeUserLocation("userId"), you can listen to location from Native side of our SDK which is currently supported. Our scheduled release planned for tomorrow has direct method which will have the support to listen to these locations from the flutter side itself instead of native side.
We will update the version here once the release is live.


Add more checking

Codecov Report

Merging #154 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #154      +/-   ##
===========================================
+ Coverage     39.58%   39.6%   +0.02%
- Complexity      597     598       +1
===========================================
  Files           140     140
  Lines          4416    4416
  Branches        322     322
===========================================
+ Hits           1748    1749       +1
+ Misses         2541    2536       -5
- Partials        127     131       +4

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/com/webank/wecross/resource/Path.java
68.96% <0%> (+3.44%)
12% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


fix(electron): load react devtools extension failed

lose extensions build folder

Reminder: Storing these kinds of files is not a long-term solution.

Reminder: Storing these kinds of files is not a long-term solution.

On this point, we will discuss in https://github.com/netless-io/flat/pull/1084.

Our discussions have been completed and I will merge this PR first and later I will optimize it according to what we have discussed


update begineer projects with new codes

Can you please be more specific to the issue?
Describe what exactly has to be done so that everyone will find it easy to solve it.

 If you don't mind then please change it from begineer to beginner.

hello @ShruAgarwal! Actually the issue is to add new code(modifications) to the existing beginner projects.
thanks for pointing out the typo, I will change it

Ok! Got it!


Close Stream

Hey,
I am streaming the result of a huge select.
But Sometime I want to close the stream before the end. Is there any way of doing that? Thanks

Why not use conn.emit("end"):
conn.query("SELECT * FROM hugetable")
  .on("result", result => {
    ...
    if (needCloseConnectionOnTheWay) {
      // Close this connection!
      return conn.emit("end");
    }
  })

Closing, as implementation of this requires a higher-level extraction, with multiple connections to MySQL server, which is outside the scope of this module.


Installing brax.experimental locally

After playing around with the Braxlines Composer using this Colab, I wanted to try it locally.
However pip install git+https://github.com/google/brax.git@main doesn't install the brax.experimental part. I assume this is intentional? How does it work fine in Colab though?
I fixed it by cloning the repo and doing pip install -e . instead. Is this the right way to do it? Maybe there should be something like pip install brax[experimental]?

Hi @KarolisRam - good catch!  This is now fixed in 984855ec367e50ab3d619025ae23d0c8359e232d - you should now be able to run pip install git+https://github.com/google/brax.git@main and then import brax.experimental
Upon our next release, this will also be available via pip install brax.
Closing, but please reach out if you have any other issues.


remove platform version check from Messaging header

Also fixes the changelog.

Ignoring the keychain test failures for now since they're unrelated to this PR.


[ODSP] Implement accept header in api call in fluid binary snapshot format

Work Item
Implement accept header in api call in fluid binary snapshot format. This could also lead to merge of the old and new snapshot apis in trees latest. This is basically client informing server which snapshot format/versions it can accept and server will respond accordingly.

Required some server work. So moving to March.


Thanos-1x.github.io

[ ]

[x]  [/###](url) Is your proposal related to a problem?

(Write your answer here.)
Describe the solution you'd like

(Describe your proposed solution here.)
Describe alternatives you've considered

(Write your answer here.)
Additional context

(Write your answer here.)

Thanos-1x.github.ioUse git status to list all new or modified files that haven't yet been committed.

Thanos-1x.github.io


EsbSoapWrapperPipe missing CorrelationId header in 7.7

Describe the issue
The nl.nn.adapterframework.extensions.esb.EsbSoapWrapperPipe displays different behavior between the framework 7.5 and 7.7
7.5RC-3

7.7

7.7 doesnt have the correlationid
Reporter
Tarik

The CorrelationId header field is populated from the SoapHeader session variable, using XPathExpression MessageHeader/HeaderFields/MessageId.
Writing of the SoapHeader session variable is not seen in unwrap-phase.

It appears to me that the failing EsbSoapWrapperPipe is configured as a Pipe, and not as an InputWrapper.

In this case to achieve the intended functionality of this adapter it must go to the next pipe even if the validation fails.
In the earlier version of the framework we had  the property..... to so that, but now we need a failure forward to the next in this cases wrapper pipe for this the wrapper needs a name

If SoapWrapperPipe with direction=unwrap is used as pipe, rather than as a Wrapper, then there is not a default for the attribute soapHeaderSessionKey. I will make that clear in the documentation


Copying services metadata

Summary
We want suppliers to be able to copy (published) services from one
framework (G-Cloud 9) to the next iteration (G-Cloud 10). In order to do
this we need to define, in the new framework, which source framework
contains compatible services. We then also need to whitelist which
questions are equivalent between the two frameworks so that answers for
those can be copied across. Anything not whitelisted will be assumed to
not be compatible (either the question is no longer asked in the new
framework, or the wording has materially changed and it would be
inappropriate to copy the data across).
Related PR
https://github.com/alphagov/digitalmarketplace-content-loader/pull/56

Well, apart from that failing test! Looks like a schema needs updating.


remove DataModule

Since this is also implemented in the template (see https://github.com/ChristophAlt/pytorch-ie-hydra-template/pull/64), the datamodule is really not necessary anymore in core pytorch-ie.

This is a duplicate of #181.


KAFKA-8470: State change logs should not be in TRACE level

The StateChange logger in Kafka should not be logging its state changes in TRACE level.
We consider these changes very useful in debugging. Given that we configure that logger to log in TRACE levels by default, it seems better to change the default level to INFO and move the logs we consider useful to that level.

I tried to be selective in the logs we keep in trace in order to avoid a regression in log volume. This commit (https://github.com/apache/kafka/pull/8320/commits/fcbe35cc34341554e6846e96ef31495a6377cf0d) contains the logs I think we should leave in TRACE (+ some expensive logs in the UpdateMetadataRequest path), all others were changed to INFO.
This PR is a continuation of https://github.com/apache/kafka/pull/6878

Thanks for being patient with the review @junrao! You can tell I'm not too familiar with the code.
I believe I've addressed all of your comments. I've got one outstanding question about the per-partition debug logs in ControllerChannelManager#sendStopReplicaRequests()

@junrao apologies for having to squash all commits - I needed to rebase with master. There was a simple conflict in ReplicaManager#1381 - the parameters for the method in the  if (partition.makeLeader(partitionState, highWatermarkCheckpoints)) line
I addressed your last comments in these three commits, before squashing them - https://github.com/stanislavkozlovski/kafka/commit/11f0ed59c1f3f4d9998cd208e82db6f35be6828c, https://github.com/stanislavkozlovski/kafka/commit/09e9cf69a6d3e0cbd1d0beb5549027cbb0f103c4, https://github.com/stanislavkozlovski/kafka/commit/ccd1e38bf17f9a789d30506298c006995d1de494

ok to test

The default state-change log looks like the following.
[2020-03-26 11:38:33,075] INFO [Controller id=0 epoch=1] Sending LeaderAndIsr request to broker 1 with 0 become-leader and 1 become-follower partitions (state.change.l
ogger)
[2020-03-26 11:38:33,075] INFO [Controller id=0 epoch=1] Sending LeaderAndIsr request to broker 0 with 1 become-leader and 0 become-follower partitions (state.change.l
ogger)
[2020-03-26 11:38:33,075] INFO [Controller id=0 epoch=1] Sending UpdateMetadata request to brokers Set(0, 1) for 1 partitions (state.change.logger)
[2020-03-26 11:38:33,076] INFO [Broker id=0] Handling LeaderAndIsr request correlationId 4 from controller 0 for 1 partitions (state.change.logger)
[2020-03-26 11:38:33,078] INFO [Broker id=0] Stopped fetchers as part of LeaderAndIsr request correlationId 4 from controller 0 epoch 1 as part of the become-leader tr
ansition for 1 partitions (state.change.logger)
[2020-03-26 11:38:33,078] INFO [Broker id=0] Leader test-1 starts at leader epoch 1 from offset 0 with high watermark 0 ISR 0 addingReplicas  removingReplicas .Previous leader epoch was 0. (state.change.logger)
[2020-03-26 11:38:33,079] INFO [Controller id=0 epoch=1] Sending UpdateMetadata request to brokers Set() for 0 partitions (state.change.logger)
[2020-03-26 11:38:33,080] INFO [Controller id=0 epoch=1] Sending a stop replica request (delete = false) for 1 replicas to broker 1 (state.change.logger)
[2020-03-26 11:38:33,088] INFO [Broker id=0] Add 1 partitions and deleted 0 partitions from metadata cache in response to UpdateMetadata request sent by controller 0 epoch 1 with correlation id 5 (state.change.logger)
[2020-03-26 11:38:33,095] INFO [Controller id=0 epoch=1] Partition test-0 state changed to (Leader:0,ISR:0,LeaderEpoch:1,ControllerEpoch:1) after removing replica 1 from the ISR as part of transition to OfflineReplica (state.change.logger)
[2020-03-26 11:38:33,096] INFO [Controller id=0 epoch=1] Sending LeaderAndIsr request to broker 0 with 1 become-leader and 0 become-follower partitions (state.change.logger)
[2020-03-26 11:38:33,097] INFO [Controller id=0 epoch=1] Sending UpdateMetadata request to brokers Set(0, 1) for 1 partitions (state.change.logger)
[2020-03-26 11:38:33,097] INFO [Controller id=0 epoch=1] Sending a stop replica request (delete = false) for 1 replicas to broker 1 (state.change.logger)
[2020-03-26 11:38:33,097] INFO [Broker id=0] Handling LeaderAndIsr request correlationId 6 from controller 0 for 1 partitions (state.change.logger)
[2020-03-26 11:38:33,097] INFO [Broker id=0] Stopped fetchers as part of LeaderAndIsr request correlationId 6 from controller 0 epoch 1 as part of the become-leader transition for 1 partitions (state.change.logger)
[2020-03-26 11:38:33,098] INFO [Broker id=0] Leader test-0 starts at leader epoch 1 from offset 0 with high watermark 0 ISR 0 addingReplicas  removingReplicas .Previous leader epoch was 0. (state.change.logger)
[2020-03-26 11:38:33,099] INFO [Broker id=0] Skipped the become-leader state change after marking its partition as leader with correlation id 6 from controller 0 epoch 1 for partition test-0 (last update controller epoch 1) since it is already the leader for the partition. (state.change.logger)
[2020-03-26 11:38:33,100] INFO [Broker id=0] Add 1 partitions and deleted 0 partitions from metadata cache in response to UpdateMetadata request sent by controller 0 epoch 1 with correlation id 7 (state.change.logger)

Only one following failure. Seems unrelated.
12:29:16 org.apache.kafka.streams.processor.internals.RecordCollectorTest > shouldNotThrowStreamsExceptionOnSubsequentCallIfASendFailsWithContinueExceptionHandler FAILED
12:29:16     java.lang.AssertionError
12:29:16         at org.junit.Assert.fail(Assert.java:87)
12:29:16         at org.junit.Assert.assertTrue(Assert.java:42)
12:29:16         at org.junit.Assert.assertTrue(Assert.java:53)
12:29:16         at org.apache.kafka.streams.processor.internals.RecordCollectorTest.shouldNotThrowStreamsExceptionOnSubsequentCallIfASendFailsWithContinueExceptionHandler(RecordCollectorTest.java:519)


Unnest Answers

@lowellmower @benjamincohen1989
This branch unnest Answers from Questions. I manually tested the CRUD for Answers after the change. It also adds nested routes for comments and votes, in the same way it was done for Questions.

@shmartin @benjamincohen1989 - - thanks for doing this! I went home and did the same thing last night but also implemented the comments and moved those into partials. I'd like to merge my branch as it has the same changes plus answer comment feature. Everyone cool with that? I made a pull req

I took a look at it, that's fine with me. I won't work on anything else today. I'll work on what ever features are left over tomorrow.


largest-series-product: Do not test digits/slices

The digits and slices functions are internal implementation details and
thus the test case for the largest-series-product problem should not be
concerned with testing them. The series tests could potentially be moved
to the series exercise (which would have to be a new addition to this track)
Their presence may cause students to falsely think that their solution
has to use these two functions, instead of the alternative
implementation of only iterating through the digits once.
If it is desired to give hints on how to approach this problem (which
was one advantage of having the digits and slices tests), then consider
including a hints file and/or directory in the largest-series-product
directory.
This PR arises from discussion in
https://github.com/exercism/x-common/issues/192

What is the series exercise? I'm not seeing it in the track. This PR looks good though, thanks.

Oops. I had updated the commit text but forgot to update the PR text. The series problem would have to be a new addition to this track.

Ah, cool. Feel free to make a PR to add series to xclojure if you're so inclined.

I will put it in my todo list. There are various things there so I may not get to that for an extended time. If anyone reading this issue cares, I would not mind if I were beaten to the punch on it.


Chapter 6: Cannot seed database

I can't seed database. Can anyone help me solve this problem?     /(ㄒoㄒ)/~~

PS D:\projects\next\nextjs-dashboard> npm run seed

seed
node -r dotenv/config ./scripts/seed.js

node:events:493
throw er; // Unhandled 'error' event
^
Error: read ECONNRESET
at TLSWrap.onStreamRead (node:internal/stream_base_commons:217:20)
Emitted 'error' event on WebSocket instance at:
at emitErrorAndClose (D:\projects\next\nextjs-dashboard\node_modules\ws\lib\websocket.js:1016:13)
at ClientRequest. (D:\projects\next\nextjs-dashboard\node_modules\ws\lib\websocket.js:864:5)
at ClientRequest.emit (node:events:515:28)
at TLSSocket.socketErrorListener (node:_http_client:495:9)
at TLSSocket.emit (node:events:515:28)
at emitErrorNT (node:internal/streams/destroy:151:8)
at emitErrorCloseNT (node:internal/streams/destroy:116:3)
at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
errno: -4077,
code: 'ECONNRESET',
syscall: 'read'
}
Node.js v21.0.0

Have you resolved it? I am encountering the same issue as you, and I'm getting the same error on two different computers.

Have you resolved it? I am encountering the same issue as you, and I'm getting the same error on two different computers.

Not yet

I have the same issue when seeding. How to solve it?

Could you try Node.js 20 LTS instead of 21?

Could you try Node.js 20 LTS instead of 21?你能试试 Node.js 20 LTS 而不是 21 吗？

Even if it is changed to version 20, it still won’t work

I am also facing the same issue... ☹

Hi! For those experiencing issues with populating the table, in particular, error -4077, a solution for me was to temporarily downgrade your node version to LTS14.
nvm install 14
nvm use 14
It worked for me. Hope it helps!

Hi! For those experiencing issues with populating the table, in particular, error -4077, a solution for me was to temporarily downgrade your node version to LTS14.
nvm install 14 nvm use 14
It worked for me. Hope it helps!

It works,but why can't use npm 20 or 21

I remember darkly that TLS defaults/support in Node versions greater 14 has changed. This is an issue of your environments. I'm not savvy with networking, but suspects are OSes, VPNs and Firewalls. If you have time and if you are patient, debug/monitor networking.

try running vercel dev

https://github.com/orgs/vercel/discussions/2350#discussioncomment-5850746

I am not able to reproduce this. Please open a new issue if you are still having trouble here. Thank you!
We are also making the seeding easier in this PR by allowing you to hit a URL in the browser: https://github.com/vercel/next-learn/pull/764


Percent Sign (%) in HTML

I have a mustache template that I am using the three curly braces to render the data as HTML. In the HTML there is a percent sign. When the page renders the percent sign breaks the page. If I take out the percent, everything is good. Suggestions? Known issues?
var description = '
I am 100% positive, you should answer yes to the question.
';

{{{description}}}

That should work exactly as you intend and i just confirmed that using www.trymustache.com. Are you sure nothing else could be causing the issue. Could you provide a code sample?

I don't know what happened, now it is working. I deleted the text and re-added it to the database. I am assuming someone pasted from Word, which we all know is bad... Thanks Romanx for the reply.

You're welcome. You need to close the issue to mark it as resolved :smile:

Closing the issue. I'm glad you got it working. :)


V-38604 ypbind service in cat2.yml not properly written, breaks in ansible v2

The following:
- name: V-38604 Medium  The ypbind service must not be running
  service: name=ypbind state=stopped enabled=no
  register: ypbind_test
  when: ypbind_test.msg is defined and "'FAIL' not in ypbind_test.msg"
  tags: [ 'cat2' , 'V-38604' , 'insecure_services' , 'ypbind' ]

shouldn't work in ansible v1, but it does and it improperly skips the task every time. The register var being used in the conditional in the same task is not proper.
TODO: Refactor so that the presence of ypbind is checked in a separate task prior to attempting to stop/disabling the service.

Fixed in this commit
There is a task  in prelim.yml that enumerates all services on the system. It was not being leveraged for this task.


C86 usability

Change Summary

Changed colour of 'add to queue' button to match client request

Given card headers uniform sizes

[x] The pull request title has an issue number

[x] The code works by "Smoke testing" (in other words, you have tested it your own and it works)

[ ] The code has documentation (either comments or in main documentation platform)

Sorry I should have clarified yesterday. I ran the tests and all passed except 1, which was due to an error in detecting the correct dates on the file downloaded from the export to csv page


Issue about error after command yarn dev-server "Cannot find module '../../../../smart-contract/typechain/index' or its corresponding type declarations"

Hello,
I have a question about error, appearing when i'm trying to launch next command:
\nft-erc721-collection-2.4.1\minting-dapp> yarn dev-server
Response:
ERROR  Failed to compile with 1 errors            00:18:16
error  in C:\nft-erc721-collection-2.4.1\minting-dapp\src\scripts\lib\NftContractType.ts          00:18:16
[tsl] ERROR in C:\nft-erc721-collection-2.4.1\minting-dapp\src\scripts\lib\NftContractType.ts(5,49)
TS2307: Cannot find module '../../../../smart-contract/typechain/index' or its corresponding type declarations.
Entrypoint main [big] 7.57 MiB = main.css 89.7 KiB main.js
7.48 MiB
Entrypoint _tmp_copy (127 KiB) = 3 auxiliary assets
webpack compiled with 1 error
ERROR  Failed to compile with 1 errors            00:18:18
error  in C:\nft-erc721-collection-2.4.1\minting-dapp\src\scripts\lib\NftContractType.ts          00:18:18
[tsl] ERROR in C:\nft-erc721-collection-2.4.1\minting-dapp\src\scripts\lib\NftContractType.ts(5,49)
TS2307: Cannot find module '../../../../smart-contract/typechain/index' or its corresponding type declarations.
Entrypoint main [big] 7.57 MiB = main.css 89.7 KiB main.js
7.48 MiB main.a097f456b6f32bfe8e6d.hot-update.js 2.17 KiB
Entrypoint _tmp_copy 794 bytes (127 KiB) = _tmp_copy.a097f456b6f32bfe8e6d.hot-update.js 3 auxiliary assets
webpack compiled with 1 error
Environment - windows 10, latest versions of all the involved frameworks.
How can i solve this issue?
Thanks a lot.

Have the same Issue

@liarco I have the same issue.

Hi @Tkareder, please stick to the Issue template that we provided, without that information in that exact format I must close the issue as invalid. If you just wan't to ask questions, then you should use the Discussions Section.
Anyway, looking at the error, it seems like you are trying the run the dApp without compiling the contract. I suggest watching the tutorials in the README file (please skip the "speedrun" since it's using version 1.x so it might be confusing).
@Freska98 I hope this helps you as well.


566 ask for total boops

This changes how we calculate the total items in the form from the value in the prompt.
Feature tests not affected and still pass.
Resolves #566
Description
Instead of asking total number of items being scanned for a particular item (diaper count in total packages of the same line_item) we now ask for total number of packages.  The end result is still the total number of items, but the human interaction is simplified.
Type of change

Non-breaking change.

How Has This Been Tested?
Feature tests still pass. End result is the same. User interaction is slightly modified.
Screenshots

:100:


Fix some typos in strings and comments

This PR fixes several typos found in a few strings as well as in many comments, but it does not affect the code's functionality in any way. There are many more typos, and I will try to fix some more.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

@sndth
Let me know when you're done with all fixes.

Currently that's all!

@sndth
OK. Thanks a lot!
I'll merge the changes.


Fix(ReactVis): Stop Setting Refs On Stateless Components

00ffe61 tries to set the ref attribute on stateless components - which causes loads of console errors.  This fix only sets the ref attribute on components which were previously receiving it.
#861

@mcnuttandrew and should I overwrite the previous buggy patch or push a new one?

I like your fix! Is there a way to write a test about this so it doesn't happen again? (if not i'm not incredibly concerned, I think we'll get a similar bunch of people yelling about it)
You should always cut new patches! Our versioning isn't super fancy or branded, so in my view it's better to have something that is actively versioned rather than try to fix buggy versions. Version 1.10.2 here we come!!

@mcnuttandrew great, thanks - and sorry about breaking things!  Spying on console.error (or causing it to throw an error) during existing tests might cover this scenario and a number of other undesirable ones - let me try that - and possibly add the regression test to this PR


fix(#1814): allow form-item slotted content for helptext and error

Allow slotted content projection for error and helptext. Otherwise, continue to keep the existing fucntionality where the error and helptext show up as before in regular font type. Documentation changes will be in a separate PR https://github.com/GovAlta/ui-components-docs/pull/164.
Below is a quick reference for examples on the usage.
This is the PLAIN TEXT example:
REACT
<GoAFormItem label="First name" helpText="This help text is in regular font plain-text">
  <GoAInput name="firstName" ..... ></GoAInput>
</GoAFormItem>

ANGULAR
<goa-form-item label="First name" helptext="This help text is in regular font plain-text">
  <goa-input name="firstName" .....></goa-input>
</goa-form-item>

This is the ReactNode (custom html) example:
<GoAFormItem label="First name (slotted)"
                          helpText={<>This is a  slotted help text.</>}
>
 <GoAInput name="firstName" ..... > </GoAInput>

    This is a slottedhelp text.

</GoAFormItem>

This is the Angular SLOTTED example
<goa-form-item label="First name">
<goa-input goaValue name="item" [formControl]="itemFormCtrl" [value]="itemFormCtrl.value"></goa-input>

    This is
    slotted
    help text.

</goa-form-item>

Same goes for 'error' property.

Issue is verified on react and angular, chrome. firefox and edge

Issue is verified on react and angular, chrome. firefox and edge

Hi @lizhuomeng71 , just a quick comment. HTML is not supposed to be passed to the helptext and error proerties on , it will continue to be plain text. However, the html can be passed into the appropriate slots. It's either or. You either provide the plain text as before or use slot for custom html where italiziced text or any other custom html can be passed into.

:tada: This PR is included in version 1.17.0-alpha.54 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.17.0-alpha.27 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0-alpha.4 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error init::0x02|0x00

Is the c2c64 supposed to throw an error to the UART console when there isn't an OLED display attached?
Oled NOT detected

Error init::0x02|0x00

I have an NTSC NES attached to the board and have tried both directly to an LCD TV and through a GBS-C scaler, and am getting no video output. I will test it with a cartridge that outputs at a different framerate tomorrow since that could have something to do with it.
When testing, I also noticed that the TLV70018 was getting fed 5V to step down to 1.8V, is this normal? The voltage regulators seem to be getting warm when powered on for a while.
I'll test it with a VHS player or a camcorder later and see if the NES console could be the issue...

After tracing the error, I think 0x02 means the ADV7180 did not reply to the power on request on the I2C bus. Are there any components you would recommend for me to check for issues?

You are correct, that error signals an issue in the initialization of one of the video ICs, in this case the 7180. Things to check:

Have a look at the output of all the regulators, but it's probably not a power problem given the encoder seems to be initializing ok, as well as the MCU.
99% of the times I got this error it was because of a bad soldered IC. These little f$ckers are not easy to solder correctly. For the IC to initialize ok, you just need to have power, gnd, sda, scl and reset working. One of these is probably not well connected/soldered

Good luck debugging the issue, let me know if I can help

I went and probed around, and traced the board, and noticed that C26 is only reading 1V... I found that U6 and its capacitor, C41 are both reading 1V.
I'm not sure how the encoder chip is responding properly since one voltage regulator just isn't working

Hi,
U6 is the PLL power supply, shouldn't really affect the boot up of either the encoder of the decoder but perhaps the decoder is a bit more anal about it. Question now is what's failing, is it the regulator? Perhaps there something shorted somewhere in the board. If you have a spare just replace this regulator and measure voltages again.
Jose

Thanks for your reply! That's exactly what I am going to do. I'll have to double-check the TLV70018 datasheet to see if it is even capable of dropping to 1V without being a faulty unit

Note that if there is a short or a broken part somewhere in the rail, it may be pulling down the voltage to 1V. There's a bunch of bypass caps that connect the rail to ground directly, it could be one of these. Measure resistance between the output of the regulator and gnd, just in case.

I think I've figured out what's wrong. The enable lead of my regulator wasn't getting 5v, which lead me to figure out that it's not soldered properly... the 1V reading is likely just from the 1.5µA leakage current that it's rated for. Fixing this made the ADV7180 initialize correctly...
Now I have a new error:
Oled NOT detected

Error init::0x00|0x0A

I think this error means that there were either 5 NACK signals or it timed out twice

UPDATE: I checked the voltage regulators and after fixing them, it trips the fuse during power on.

Hi there, the 0x0A seems to be an undetected Encoder but I think you had that figured out already.
It's a bit unclear what happened next. There was another issue with the regulators that you fixed? Bypassing the fuse could be dangerous, what type of fuse are you using? what specs? The board may generate a bit of a current rush when in first powers on, but it should go pretty low pretty quick (200ma or so).
Regarding NTSC signals, some people have tested the board extensively with NTSC gear in the US and there are no specific issues with this. The ICs support most standards (PAL, NTSC, SECAM) and have the ability to autodetect and configure themselves for the input signals.
What device are you trying to visualize with the c2c? is it composite or svideo? Remember there is a button to select the input, which defaults to composite. Also, the OLED screen should display information about the signal being transcoded and there's a green led that should be on when an input signal is locked.

Hi again,
I'm still unsure if you were able to solve the 0x0a encoder init error. I assume you did. In that case, what was the problem and how did you solve it?
About the fuse, the fact that you are having to bypass it indicates a potential problem with your board, either a short somewhere or a faulty part. It shouldn't just trip.
About the signal, one thing to have in mind is that the output will be identical in frequencies to the input. I've found that my tv was not able to sync to Commodore 64 15khz output, however OSSC had no problem. In general, GBS-C is known to work with C2C without any changes. Are you using the gbs control fw or the original? I haven't tested this in a while, perhaps newer versions do not work. I'll check this afternoon.
I recommend adding an OLED screen to the c2c to simplify the debugging.

I just remembered that gbs control was unable to sync to c2c out of the box. The coasting setup in the gbs firmware is too tight for the signal generated by the encoder in the c2c. How does the c2c->gbs output look like? Could you describe it?
The solution is pretty easy, you need to tweak with the coasting controls. I am not sure if the gbs UI allows you to do that though. Back in the day I remember I had to change the coasting constants in two lines of the code. I need to have a look again, I just don't remember the details. Do you think you could flash a new fw into your gbs if I send you a tweaked version?

The output I was getting to my VGA PC CRT monitor, from my GBS-C was somewhat green while the desynchronized grayscale output from the NES was somewhat overlaid.
I tested it with an LCD TV by plugging the output of the c2c64 directly into the component input of the TV. The TV is a low-end model from a decade ago, and was sometimes getting a grayscale variant of the desynced image I got from the GBS-C to my VGA CRT, so the TV could have been improperly showing data because of something in the TV.

Suspecting that it may have something to do with data traveling between the decoder and encoder, I tested the solder joints of RN1 and found at least one joint that isn't making contact. I will fix it then try it again. Thank you for all of your advice.

I have now (hopefully) fixed RN1. The fuse isn't tripping anymore either... I think the video encoder was consuming obscene amounts of energy trying to encode garbage into component.
Now, the output I'm getting seems somewhat less like a hardware issue:
https://github.com/jsanpe/c2c-64/assets/35894769/0947d976-b606-487d-8407-70c39986a111
It still isn't syncing correctly, but it appears to be outputting some color information.
My LCD TV is also not syncing correctly, but it seems to only be displaying in grayscale, compared to the GBS-C, which is pushing some color information.

I'm still unsure if you were able to solve the 0x0a encoder init error. I assume you did. In that case, what was the problem and how did you solve it?

Yes, I traced in the code that 0x0A could be caused either by no reply from the encoder IC or a timeout for the last two commands. I checked the voltage between FB4 and GND, and read 0.8v. This low voltage seems to always mean leakage current from a disabled TLV70018. I found the solder joint on the EN pin of U3 was also somewhat loose, and I could trace the next issue (fuse) after that.
The fuse was likely caused by the video encoder consuming obscene amounts of energy trying to encode the floating inputs from the bad solder joint on RN1.

I plugged in a Sony camcorder over S-Video and saw that the "Sync" value was showing Y. I was having an issue with F1 tripping, however. Deciding that it must be caused by solder joints between the decoder and encoder, I resoldered RN1 (again), and replaced F1.
I found that the sync issues were definitely caused by the encoder chip since the output was now clean, and syncing on both my GBS-C and LCD. This is what the output looks like now:

I was testing to see if only one color channel is having issues, and found that if I disconnect Pr, I got a clean signal.

This happens on both S-Video, and Composite, on both my GBS and LCD. I suspect there is either an issue near the red channel, or one of the solder joints on RN1 is not happy, if one of the pins carries data specifically in the red channel. I will check the datasheet soon and let you know if this is also caused by RN1.

Hi there,
If the problem is limited to the red channel, I would first look at D1 and R4, which are directly connected to the Red channel output. Check if they are correctly soldered, and working correctly. Otherwise the issue could be in pad 26 of the ADV7393, perhaps it's not making good contact.

I suspect D1 is likely related to the issue. I will check it out on my oscilloscope soon and let you know what I find

AND AFTER HOURS AND HOURS OF TROUBLESHOOTING, I ATTACHED RN1 CORRECTLY...
I discovered that all of my issues were caused by RN1 still not being attached properly. Almost all the pins were connected except one specific pin. I should have probed the individual pins of that resistor array with my oscilloscope much much earlier. If I did, I would've traced this much faster. It works! This is my first major soldering project where I ordered a PCB that I didn't design myself. Quite the learning experience.

Over the course of this project, I figured out how to create color bars on the composite port of a raspberry pi zero so that I could have a consistent waveform when viewing the output signals on my analog oscilloscope.
Thank you for all of your help!

Well, you've shown a lot of determination and perseverance. I'm really happy you made it work, and I'm sure after so much frustration these past few weeks you are now feeling absolutely great. Congratulations!!


Feat testing

Thank you for contributing to SunPy
🚨 IMPORTANT 🚨
We know that working on code and submitting pull requests takes effort, and we appreciate your time.
Thank you.
Please be aware that everyone has to follow our code of conduct
Furthermore, you might need to check with your work place if you are allowed to contribute code

Please try to create an issue before creating a Pull Request.
Please use the following Git commit message style

Use the future tense ("Adds feature" not "Added feature")
Limit the first line to 72 characters or less

We have several automatic features that run when a pull request is open.
They can appear daunting but do not worry about them!
This a brief explanation of them.
↑👆 DELETE above before submitting 👆↑
PR Description

Codecov Report

Merging #75 (a37f7e0) into master (818ae4f) will increase coverage by 22.18%.
The diff coverage is 50.31%.

@@             Coverage Diff             @@
##           master      #75       +/-   ##
===========================================
+ Coverage   32.50%   54.68%   +22.18%
===========================================
  Files          18       19        +1
  Lines        3098     3114       +16
===========================================
+ Hits         1007     1703      +696
+ Misses       2091     1411      -680

Impacted Files
Coverage Δ

sunxspex/sunxspex_fitting/fitter.py
53.59% <46.30%> (+40.32%)
:arrow_up:

sunxspex/logging.py
100.00% <100.00%> (ø)

sunxspex/sunxspex_fitting/data_loader.py
17.29% <100.00%> (+4.55%)
:arrow_up:

sunxspex/sunxspex_fitting/instruments.py
21.89% <0.00%> (+3.64%)
:arrow_up:

sunxspex/sunxspex_fitting/likelihoods.py
68.57% <0.00%> (+20.00%)
:arrow_up:

sunxspex/sunxspex_fitting/parameter_handler.py
71.34% <0.00%> (+52.04%)
:arrow_up:

sunxspex/sunxspex_fitting/rainbow_text.py
82.35% <0.00%> (+70.58%)
:arrow_up:

Continue to review full report at Codecov.

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


[RFE] Trace Controller: documentation (#197 follow-up)

#197 added a new controller for the Trace custom resource but the documentation is missing:

[ ] Document how users should use the Trace custom resource. There could be several categories of users:

Users accessing custom resources directly with kubectl
The kubectl-gadget CLI
Graphical interfaces such as Headlamp plugins
Other software consuming low-level insights from Inspektor Gadget

[ ] Document how developers can implement new gadgets that are controlled by the Trace resource

Should we put it on the v0.5.0 milestone?

I believe this was fixed with #363


Formatting of atlases table

Trying to use the list atlases on my laptop, the table generated by list_atlases() in a notebook is impossible to read because everything is too squeezed (always from notebooks, and in terminal unless I widen it a lot from default).

Maybe we can drop the remote URL?

That's weird, I need to check rich docs, maybe there's a way to ensure a minimal column width. But yes in general the online path is not very useful.  Also not sure why it ends with object ...?

addressed in: https://github.com/brainglobe/brainatlas-api/pull/49
were you testing in a jupyter notebook? On which OS?
In theory the column width should match the content...

Testing in a notebook on macOS 15''

looks like this for me..

Same for me now.

Still, I would find it uninformative if there's no full name, so I would definitively try to ensure some column width

should be fixed with the latest commit in the PR

I had to add a hard-coded minimal width for the column, so we better not have atlases with more than 32 characters in the name :)

that would be annoying to type anyway :p
I was also thinking that the local path is also quite redundant, once you print the brainglobe location in the header. What about keeping the table only for the first four entries, and place outside the table the local path and the remote url?

I thought to leave it in case atlases are saved in different folders. But maybe it's unnecessary as users can get the atlas path in other ways... maybe I'll leave it as optional?

As per https://github.com/brainglobe/brainatlas-api/pull/49 , now the local path is optional and disabled by default.

@vigji  are you happy with how it looks now?

Yep, beautiful!


"Can't serialize unregistered packet" error when connecting to Fabric server with mod installed

Hi we have issues with the FabricProxy-Lite Mod for our Network usage, please have a look below:
Environment:
Minecraft version: 1.19.4
Mod version: 2.5.0
Server software: Fabric + fabric-api
Client software: Fabric
Velocity version: latest 3.3.0-SNAPSHOT
Steps to reproduce:
Install the mod on the fabric server
Start the server and client.
Attempt to connect to the server from the client.
Expected behavior:
The client should be able to connect to the server without any issues.
Actual behavior:
The client throws an error "Unable to connect" and fails to connect to the server.
Logs:
Please find the relevant log output below:
[Error message: "Internal Exception: io.netty.handler.codec.EncoderException: java.io.IOException: Can't serialize unregistered packet"]
Additional information:
We have tried different versions of all software used but still got the same error at the end with unregistered packets.
The mod is only needed server-side right?
Our velocity setup uses new forwarding.
We think its a issue with the velocity communication of packets, is this a known issue or do you might have some more info for us about that?
Thanks Guys!
Greets

1.19.4 is no longer supported. If there are still problems with the new version, please open a new issue.


lists should not contain

Why do you use

 in lists ? This is not the way Commonmark does it.
Other strange behavior: in the same document, some lists get the paragraphs, some others don't. Could you explain how it is supposed to work ?

Hi there!
According to CommonMark:

A list is loose if any of its constituent list items are separated by blank lines, or if any of its constituent list items directly contain two block-level elements with a blank line between them. Otherwise a list is tight. (The difference in HTML output is that paragraphs in a loose list are wrapped in
 tags, while paragraphs in a tight list are not.)

So:
- a
- b

- c

... will be rendered to:

a

b

c

And:
- a
- b
- c

... will be rendered to:

a

b

c

Can you attach the wrong example you're seeing?

I did revert my code to use markdown, that does not have this behavior.
But, in deed, the last block list is right, but all the ones before get paragraphs.
Example:
## heading 1

* a
* b
* c

## heading 2

* d
* e
* f

## heading 3

* g
* h
* i

Block 1 and 2 get paragraphs, and block 3 (last one) is ok.


Disable dev.openjdk dev.system from weekly

Original reverted via Adopt 573

fyi @llxia

Thanks @AdamBrousseau
Not related to this PR. We have specifically enabled dev.system on JDK11 xlinux on Semeru only: https://github.com/ibmruntimes/ci-jenkins-pipelines/pull/96/ last Oct. It seems that this change somehow got lost. Do you happen to know what happened?

Looks like it got lost when Adopt did a big syntax change and I merged it in.
https://github.com/ibmruntimes/ci-jenkins-pipelines/commit/d2615c1d173923f291aadc700c6bcbf3a5ca143f#diff-cd68afc4fbcb6c7457170bc83490eb803243e4de6cfc3841de93d569cd475a74L267


pep8 fixes

Some pep8 style fixes.

Coverage remained the same when pulling 2149a5fde6846d3a5e8c8b1a9ae24df2616eae8f on kovacsbalu:fix-pep8 into 71aeb00003d454f2f53bca56dd2a12aefb78f1f9 on randomchars:master.

Coverage remained the same when pulling 769b3b7bab1071f2d93e042b8411f1681fc56703 on kovacsbalu:fix-pep8 into 71aeb00003d454f2f53bca56dd2a12aefb78f1f9 on randomchars:master.


Appendicular_bones not showing anything

After using TotalSegmentator v2 Appendicular_bones i try to import it to 3D slicer. But nothing is shown. Please tell me what's wrong

how does your ct image look like in 3d slicer? does it contain arms and legs?

Hi! Same as @empat6, the predictions .nii.gz are empty.
Tested on this crop (see image below) of patient verse573_CT-iso from the VerSe 20 validation dataset (https://paperswithcode.com/dataset/verse-1)

Can you send me the nifti image<EMAIL_ADDRESS>? Than I can try to reproduce the issue.

Thanks for sending the data. There was indeed a bug. I just pushed a fix to the master branch. If you install from master branch (see readme for how to do this) it should work.

<EMAIL_ADDRESS>
Hi, my data looks like this. It only shows the legs, can i use TotalSegmentator for data like this?

yes

Thanks for sending the data. There was indeed a bug. I just pushed a fix to the master branch. If you install from master branch (see readme for how to do this) it should work.


sparkql_statistics.py treat incomplete date as null

Describe the bug
Very unfortunate bug that's not really our doing, but 2 of the datasets in statistics.gov.scot have incomplete date objects (only time - see screenshot), so when we pull it in, datetime convertors auto-fills today's date. Meaning the updated date constantly changes (even though we don't actually know the last date).
To Reproduce
See data/scotgov-datasets-sparkql.csv and example source. The api call comes from sparkql_statistics.py
Expected behavior
Add handling - if date cannot be determined then return as NULL
Screenshots

Hardware and software used
NA
Additional context
None

Add the fix anyway, but there's an admin to-do for me

[ ] inform the publisher of the issue also