PackagePlugin: make Path a wrapper around AbsolutePath

This should help the path representation be properly represented on different platforms.  This is motivated by Windows which would have the drive stripped from the path, converting to a relative path the absolute path.

@swift-ci please smoke test

@abertelrud

Because that still needs to be tested more heavily before it can be ready to use.  There are issues that have been fixed in t-s-c that haven't been done in System yet.  Furthermore, that adds another dependency before this is really ready to be removed.  Since this is @_implementationOnly we can swap it later without it impacting the user.  I'd say that this makes it no more difficult to remove the t-s-c dependency.

Seems reasonable though it's a bit concerning if FilePath still isn't ready for use.  @compnerd any idea on when that might be fixed?

@swift-ci smoke test

Seems reasonable though it's a bit concerning if FilePath still isn't ready for use. @compnerd any idea on when that might be fixed?

+1 on this, what are the issues on the System repository to track feature parity of FilePath with AbsolutePath?


Run test twice

Coverage remained the same at 100.0% when pulling 7e654d9870e04c203dd3776bf0c9f8d0855c5dd5 on run-test-twice into ff9be306571e1dd60f8d007995353da6629d1140 on master.

Coverage remained the same at 100.0% when pulling 7e654d9870e04c203dd3776bf0c9f8d0855c5dd5 on run-test-twice into ff9be306571e1dd60f8d007995353da6629d1140 on master.


[typo]The parentheses in the insert statement are not closed.

The parentheses in the insert statement are not closed.
CREATE PROCEDURE [dbo].[spUpdateInventory]
@productId int,
@locationId int,
@quantityIncrement int

WITH NATIVE_COMPILATION, SCHEMABINDING
AS
BEGIN ATOMIC WITH (TRANSACTION ISOLATION LEVEL=SNAPSHOT, LANGUAGE=N'English')
    UPDATE dbo.ProductInventory
        SET Quantity = Quantity + @quantityIncrement
            WHERE ProductId = @productId AND LocationId = @locationId

/*If zero rows were updated than this is insert of the new product for a given location*/
    IF @@rowcount = 0
        BEGIN
            IF @quantityIncrement < 0
                SET @quantityIncrement = 0
            INSERT INTO [dbo].[ProductInventory]
                (
                    [ProductId]
                    ,[LocationID]
                    ,[Quantity]
                )
                VALUES
                   (
                        @productId
                       ,@locationId
                       ,@quantityIncrement
        END
END;

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

ID: 16923b34-9fbb-86a2-071d-c4fa47e639ff
Version Independent ID: 83127f7a-a6cd-0b63-a723-ee72f7511a6a
Content: Temporal Table Usage Scenarios - SQL Server
Content Source: docs/relational-databases/tables/temporal-table-usage-scenarios.md
Product: sql
Technology: table-view-index
GitHub Login: @markingmyname
Microsoft Alias: maghan

@GCer-Hidenori -- thank you for helping improve this documentation.  I am addressing the article.


Quicksilver Insists on Opening the Preference Window on Restart

(QS B61, Mac OS 10.6.8 Snow Leopard)
To reproduce the bug, do the following:

Open QS's preference window with command-, while invoking QS.

Close this preference window (indicating no desire to see it again).

Invoke QS and press command-control-q to restart it.

Upon restart, QS will open the preference window again.  This behavior also occurs without step 2, which is probably okay since this just so happens to be the direction Apple is moving Mac apps in (probably a coincidence; I think I have been noticing this bug long before the announcement of Lion).  With step 2, this can be a considered to be minor annoyance.
Not a big deal, but fixing this bug could bring QS a significant step closer to a stable release in my eyes.

Yeah, I get this all the time too. Annoying.

This behavior also occurs without step 2, which is probably okay since this just so happens to be the direction Apple is moving Mac apps in (probably a coincidence; I think I have been noticing this bug long before the announcement of Lion)

Many apps have done things to restore their state for a long time. I think this was actually done on purpose in QS, and what we’re seeing is a bug in that behavior.

Many apps have done things to restore their state for a long time. I think this was actually done on purpose in QS, and what we’re seeing is a bug in that behavior.

I was thinking of that too, then I failed to consider the use cases where it would be useful.  Come to think of it, I guess it could be really useful for troubleshooting/experimenting with prefs.


Make exception throws in Money conversion instead of runtime error.

Let user decide on what to do  upon catching MoneyConversionException.

Done with this commit 6df8ed64d1d76f23a944b1e5f5bb7ce360a7102d


Switch to Dask

This PR switches to using dask as a distributed computing backend.

I'm going to merge this so I can get the front-end and analysis PRs going.


Path not appling correctly with the install script

I'm using the linux install script and the patch described in the readme isn't applied correctly, leading to a black screen.
Patch Gotten:
[Patch]
xpui.js_find_8008 = ,(\w+=)32,
xpui.js_repl_8008 = ,\${1}58,

Patch Expected:
[Patch]
xpui.js_find_8008 = ,(\w+=)32,
xpui.js_repl_8008 = ,${1}58,

Result:

I'll see if I can fix it myself and open a pull request

@nimsandu  Should be solved with #59, it was just a forgetfulness on #53

@javalsai yah that was even my fault
im the one who responsible to check before merge
@javalsai  can you work regarding my request on #59


[Bug][config-ui] Edit Data Scope and Scope Config fails to load with too many scopes

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I have a Devlake project with a Jenkins connection configured with 91 data scopes.

When I click on Edit Data Scope and Scope Config, navigating to https:///projects//<connection_id>, the page never stops showing the loading screen.

Upon inspecting the network requests, the page sends a total of 95 requests at once. The response times are from a couple of milliseconds to a minute (timed out by the load balancer). Most of the requests fail with a 500 status code.
On the API logs, there are several connection errors to the database.
time="2024-04-19 10:31:24" level=info msg="\x1b[31;1m/app/impls/dalgorm/dalgorm.go:215 \x1b[35;1mfailed to connect to `host=localhost user=redacted database=devlake`: dial error (dial tcp [::1]:5432: connect: cannot assign requested address)\n\x1b[0m\x1b[33m[30169.975ms] \x1b[34;1m[rows:0]\x1b[0m SELECT * FROM \"_tool_jenkins_jobs\" WHERE connection_id = '1'  AND full_name = 'redacted'  ORDER BY \"_tool_jenkins_jobs\".\"connection_id\" LIMIT 1"
time="2024-04-19 10:31:24" level=error msg="HTTP 500 error\n\tcaused by: failed to connect to `host=localhost user=redacted database=devlake`: dial error (dial tcp [::1]:5432: connect: cannot assign requested address) (500)\n\tWraps: (2) failed to connect to `host=localhost user=redacted database=devlake`: dial error (dial tcp [::1]:5432: connect: cannot assign requested address)\n\tWraps: (3) dial tcp [::1]:5432\n\tWraps: (4) connect\n\tWraps: (5) cannot assign requested address\n\tError types: (1) *hintdetail.withDetail (2) *pgconn.connectError (3) *net.OpError (4) *os.SyscallError (5) syscall.Errno"

Some notes:

The number of database connections is far from the limit
There are a lot of free compute resources in both the database and the API

What do you expect to happen
I expected the page to load the data.
How to reproduce

Create a connection
Add at least 90 data scopes to this connection
Create a project
Link the connection to the project
Load the project's page to edit data scope and scope config

Anything else
I initially noticed this problem with https://github.com/apache/incubator-devlake/releases/tag/v1.0.0-beta5 as I wanted to use the Jenkins multi-branch workflows, but even after reverting to https://github.com/apache/incubator-devlake/releases/tag/v1.0.0-beta1 the problem persists.
Version
v1.0.0-beta5
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Same behavior, same version here. Number of scopes is 1100+

Let me find out what happened.

As to the errors "cannot assign requested address" in your backend log, considering you are using docker, I think it's some limilations in docker, can you have a try at this https://stackoverflow.com/questions/66233698/cannot-assign-requested-address-localhostxxxx-docker-linux-containers ?

@d4x1 this is not running in Docker. It's in a GKE cluster and the database request goes through a cloud-sql-proxy sidecar. However, I will also look into this once I'm back from work vacation.

There is an issue that scope's details are fetched seperately, it will create 100+ requests at the same time, which I think could be improved.

Yes, this is the solution I was looking for from opening this issue. From a brief look, this seems to be handled by the code framework though, so doesn't seem that simple.

@d4x1 I fixed this by increasing the resources of the cloud-sql-proxy sidecar container. However, I still think it's valid to improve the way data is fetched.


ERROR: Unable to extract uploader id;

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.```

## Description

<!--
Provide an explanation of your issue in an arbitrary form. Please make sure the description is worded well enough to be understood, see https://github.com/ytdl-org/youtube-dl#is-the-description-of-the-issue-itself-sufficient. Provide any additional information, suggested solution and as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

WRITE DESCRIPTION HERE

Same issue for me with youtube-dl.exe on Windows 10 by trying to download all videos from a specific youtube channel:
PS F:\Documents\Save> .\youtube-dl.exe "https://www.youtube.com/@Hz-Cabana/videos" --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/@Hz-Cabana/videos', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: ffmpeg 4.2
[debug] Proxy map: {}
[youtube:tab] @Hz-Cabana: Downloading webpage
[download] Downloading playlist: Hertz's Cabana - Videos
[youtube:tab] Downloading page 1
[youtube:tab] Downloading page 2
[youtube:tab] Downloading page 3
[youtube:tab] Downloading page 4
[youtube:tab] playlist Hertz's Cabana - Videos: Downloading 148 videos
[download] Downloading video 1 of 148
[youtube] oq3HSJUYJZg: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 815, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 836, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 534, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\youtube.py", line 1794, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 1012, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

same on ubuntu

[x]  I've searched the bugtracker for similar bug reports including closed ones

Really? Learn from #30839.


clean up unused sui errors and delete AuthorityNotifier

Description
This PR cleans up unused SuiErrors and AuthorityNotifier.
Test Plan
lint

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes
Clean up unused SuiErrors and AuthorityNotifier.

Red diffs are the best diffs


Cannot add converters to the end of the list

When adding custom model converters, if you add converters via the java service mechanism they are added to the end of the converters list. This way you can have converters that are most generic at the top, and most specific at the bottom (for example unwrapping custom Containers and their parameterized values)
However, when adding a converter manually via
    public void addConverter(ModelConverter converter) {
        converters.add(0, converter);
    }

It always adds the converter to the head of the list. This means that delegating down is not possible anymore if you are the first one to find an element. Ideally you can specify to add to the end of the list so you can control the ordering better

Is this an error in your example?
If you use
converters.add(0 /* list index */, converter)

it is always inserted on the head because you specified the index 0.
Does it work, if you simply use:
converters.add(converter)

@sdoeringNew my example is showing code in swagger-core. I don't have access to the converters object otherwise I would have done so


Uncaught AssertionError [ERR_ASSERTION]: 5685 == 0

[Enter steps to reproduce:]

...
...

Atom: 1.38.2 x64
Electron: 2.0.18
OS: Unknown Windows version
Thrown From: Atom Core
Stack Trace
Uncaught AssertionError [ERR_ASSERTION]: 5685 == 0
At assert.js:42

AssertionError [ERR_ASSERTION]: 5685 == 0
    at ClientRequest.onConnect (/app.asar/node_modules/tunnel-agent/index.js:160:14)
    at Object.onceWrapper (events.js:319:30)
    at emitThree (events.js:136:13)
    at ClientRequest.emit (events.js:217:7)
    at Socket.socketOnData (_http_client.js:472:11)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at TCP.onread (net.js:594:20)

Commands
     -1:59.7.0 application:open-file (input.hidden-input)

Non-Core Packages
file-icons 2.1.31
french-menu 0.2.2

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/19407 so you can subscribe there for updates if you'd like.  If you have steps to reproduce or can confirm if you're using a proxy to connect to the internet, please let us know in https://github.com/atom/atom/issues/19407.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Incorrect sort in IE Edge for responsive objects

When using the responsive object for a style property, the order is incorrect in IE Edge.
For example, this: <Flex flexWrap={{ mobile: 'wrap', md: 'initial' }}> would output the md media query first in the CSS and the mobile media query second, therefore making it so the mobile media query rule always applies.
I created a jsbin to quickly reproduce the issue in IE Edge (I just rewrote some of the code to simplify things). The output from running this in Edge looks like:

While running this in Chrome outputs:

If i reverse the order of the properties in the object (md first then mobile) it works as expected.
Commenting out the styles.sort() fixes it for both Chrome and Edge, but I wanted to get any thoughts before proceeding.

Hmm. changing the order of the responsive styles seems to break it on other browsers (at least it seems to in my case).
Have you figured anything else out to get this to work as expected somewhat painlessly?

Unfortunately not @stinodes ... I ended up just writing regular styled-components for anything responsive.

I believe a fix for this has been published, and the changes in #513 should make this a non-issue. Feel free to reopen this or another issue if similar problems are still happening


⚠ Turnos 24x7 has degraded performance

In 42b2f23, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 996 ms

Resolved: Turnos 24x7 performance has improved in 0aa74f5.


Classify genotypes for plate 4

In this pr, two logistic regression models are trained on both the shuffled and unshuffled plate 4 data to classify nf1 genotypes. Please let me know if you have any questions or suggestions.

Thanks @MikeLippincott,  Merging now!


Feature request: Autocomplete Overflow

Currently the autocomplete shows all existing tags, the list can become very long.
It would be perfect to use autocomplete in an overflow. Perhaps even in a popover so that other elements underneath are not moved too far.
Great work and many thanks

Hey @jacquesuhlemann. Thanks for the love, I appreciate it.
If I am understanding what you are saying, then a workaround to that issue is setting a max with on the container the autocomplete input is in.
The popover feature is a good feature idea. I am currently working on a few iterations. I'll push what I have in a bit

I have tried with the Popover component and it was almost good! The only problem I noticed was when attempting to select the first tag it required two-clicks, which is not the intended behavior.
As a workaround, I tried wrapping the  element with a  component from shadcn.

@jacquesuhlemann  @eliza921 Thank you for your valuable feedback and suggestions regarding the Tag Input component.  I've gone ahead and implemented the features you requested! Try it out and let me know what you think
https://github.com/JaleelB/shadcn-tag-input/assets/78449846/ab0cbf96-8a44-4326-a759-f1111c5f073e


why zk.Connect always return no error even if zk server is down?

i found that zk.Connect return no error when zk server is error or down.
i don't know why? i think it should return error when service is unreachable or timeout.
`package main
import (
"fmt"
"time"
    "github.com/samuel/go-zookeeper/zk"

)
func main() {
c, _, err := zk.Connect([]string{"<IP_ADDRESS>"}, time.Second) //*10)
if err != nil {
panic(err)
}
fmt.Printf("reach here. no error for zk.Connect! But there is no zk service running \n")
children, stat, ch, err := c.ChildrenW("/")
if err != nil {
panic(err)
}
fmt.Printf("%+v %+v\n", children, stat)
e := <-ch
fmt.Printf("%+v\n", e)
}`

Haven't tried, but my guess is that you should check the returned channel for connection status.


Add sqlcmd open code support

Request from @yorek.  Will flesh out details of what sqlcmd open code should do in this issue

Thanks @stuartpa. Ideally it should behave similarly to that sqlcmd open ads does. Open VS Code and connect to the database defined in the active context.

This will be a good chance to refactor the tool code. We can define some command line templates indexed by tool name, with placeholders for command parameters. Eventually we hope more tools will support reading the connection info by name from the sqlcmd config file, or support the new xml-based connection file format used by SSMS 19.1+


fix: Comments not display the latest update

Just as https://github.com/JohnMcLear/ep_comments/issues/21

tnx


point plane contact springs at non 0 degree orientations

Hi @tkoziara (mention @stevebrasier),
I think there may be a bug in parmec when attempting to define a point - plane contact spring where "point1" and "point2" are not aligned with the global coordinate system axes.
I've attached two example files:
45deg_1xdirect.txt
This models a direct contact spring (non point-plane) between two particles. This model behaves as expected.
45deg_1xpntpln.txt
This models direct contact via a point-plane spring. This model is expected to behave in the same manner as "45deg_1xdirect.txt" however it doesn't. Visualising the positions of the particles and spring in Paraview suggests that the spring is not positioned correctly:

I would expect the visualisation of the blue spring to be located on the 45 degree line between the black particles.
I am using changeset version be9b202 of parmec.
What do you think?
Best regards,
Ben

Hi @tkoziara
Have you had a chance to look into this yet?
Many thanks,
Ben

Looking today and on:)
On 13 April 2018 at 17:54, bencannell<EMAIL_ADDRESS>wrote:

Hi @tkoziara https://github.com/tkoziara
Have you had a chance to look into this yet?
Many thanks,
Ben
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tkoziara/parmec/issues/6#issuecomment-381180087, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACGKNOhOY9BxD0qfdWxH54Lg7RL09DiXks5toMpIgaJpZM4S3Fq9
.

Thank you for the update :)

You can provide your csv writer class 'CsvSeries': it's not online:)  In the meantime I am working around it...

Ah sorry! I've sent you a personal email...

Ben,
I looked into this and so far it seems to be just a matter of applying the springs and setting up the model in a suitable way. Have a look at the attached two files: there are changes:)
45deg_1xdirect_tk1.zip
45deg_1xpntpln.zip
In the first file I have changes the second spring point to be (1, 0, 0) instead of (1, 1, 0); I also made
the same change for the fixed spring direction: changed it from (1, 1, 0) to (1, 0, 0); In the second file I moved the first spring point from (0, 0, 0) to (0.1, 0, 0);
In both files there are no particle restrains, but optionally one can remove all rotations;
One thing to keep in mind that a two-point spring with a fixed direction will maintain that fixed direction of the spring regardless of the motion of particles. On the other hand in the point-to-plane contact spring model the contact plane is co-rotated with the second particle. If there is some eccentricity between the spring points and particle mass centres this will make a significant difference (after the springs have been engaged) as particles will begin to rotate in different ways. This is why in the supplied files you can restrain rotations. You will see that without this restrained a small drift along dy begins to build up post-impact for the contact plane model.
I hope that this explanation and examples will shed some light onto this for you:) I can have a further and deeper look - please let me know in case that would be needed.
Best regards:)
Tomek

Thank you for the feedback @tkoziara. In hindsight, this was a poor attempt on my behalf at narrowing down the issue we were having with point-plane spring models "exploding", which you managed to fix in fb66998.
Best regards:)
Ben


Ignore color information of completely transparent pixels when resizing images

Fixes artifact reported in haydenbleasel/favicons#123
Black faint on the edges of opaque areas appears because color information of completely transparent pixels is taken into account during calculation of average color, but these pixels could be of any color. Completely transparent black completely transparent red looks identicall, so it's somewhat a common practice to replace RGB values with 0 or 255 for series of completely transparent pixels. Many encoders do this, (probably) to allow for a better compression.
Two images below illustrate what transparent areas sometimes look like if we make every pixel opaque.

This PR changes both submodules responsible for image resizing in the way that only color data of visible pixels used in average color calculation. Also fixed incorrect channels value in Resize.prototype.resizeWidthInterpolatedRGBA method (Iwasawafag/jimp@65158a9) and reduced amount of dupplicated code in /resize.js
And here's a comparison table.  6 images, every transparent pixel of which was painted in black, were scaled down from ~1000x1000 to 200x200 px using every resampling method supported. Then results from before and after patch were glued together for every image. The difference is pretty noticeable

Resized to 200x200. Before/After (click to enlarge)

default
 ~  ~   ~  ~

bezier
 ~  ~   ~  ~

bicubic
 ~  ~   ~  ~

bilinear
 ~  ~   ~  ~

hermite
 ~  ~   ~  ~

nearest
 ~  ~   ~  ~

Here's list of images from the table above in their original size:

Check mark
CP logo
Green cross thing
Cross thing with gradient fill
White cross thingy
Red cross thingy

It could be, that I broke upscaling with this.
I'll fix it if needed and reopen this PR once I'll be sure upscaling works as good as it was

Fixed. Lost incrementation of few indexes in two lines while merging simillar methods of the Resize class together.
As for the "bug" mentioned in Iwasawafag/jimp@65158a9 — it turned out to be some kind of a magic number, controlling how far image should shift vertically/horizontally. Changes introduced by this commit were reverted in master.

I kept playing with that number (1 / 3) and noticed that the farther away original and target dimensions are from one another - the more noticeable image shifts to the top-left, especially if compared to output produced by Photoshop.
I don't know if one third is a mathematically justified value or original programmer picked it at random, but I decided to make this value scale based on originalSize/targetSize proportion and I've got the results I'm happy with.
Example: (it's a 60x60 image resized to 512x512, then zoomed out to 25% in PS)

Nearest neighbor (resized in PS)
Resized with Jimp w/ original value
Resized with Jimp w/ adjusted value

These changes could be found in Iwasawafag/jimp@8c27ae5 if you want them, but they probably should not be used, since it's not backed by any actual math and the difference in output isn't too big to begin with.

Merged. Thanks and sorry for the delay. Very busy IRL. Will do a publish shortly and include this.


NPE in cleartext upgrade

Expected behavior
No null pointer exceptions
Actual behavior
When an HTTP/1.1 upgrade request comes in, we call onHttpServerUpgrade in Http2ConnectionHandler.  This makes us create a stream, which eventually calls new DefaultHttp2StreamChannel, which is fine, except that it assumes that handlerAdded has already been called on Http2MultiplexCodec . . . which is always true except when we're doing a cleartext upgrade.
Steps to reproduce
Make an HttpServerUpgradeHandler, with an Http2MultiplexCodec.  Send an h2c upgrade request.
Minimal yet complete reproducer code (or URL to code)
Try to upgrade this to netty 4.1.15.
https://github.com/twitter/finagle/blob/develop/finagle-http2/src/main/scala/com/twitter/finagle/http2/Http2CleartextServerInitializer.scala#L74
Netty version
4.1.15.Final

Digging a little deeper, the exactly problem here is because we call prepareUpgradeResponse before we call upgradeTo in Http2ServerUpgradeCodec.

I was able to get around this problem by setting two private fields in HttpMultiplexCodec (both named ctx).  @normanmaurer if I merge a fix into netty, what's the ETA on a release?

@mosesn end of next week sounds doable... so yeah please provide a pr if you already have one

@normanmaurer I don't have one yet, but I'm happy to take a stab today.  Where do you think we should fix this?  HttpServerUpgradeHandler seems pretty stable (hasn't been changed in a year) so I'm more inclined to do it in Http2ServerUpgradeCodec.
Looks like maybe we could try dumping all the code from prepareUpgradeResponse into upgradeTo?
https://github.com/netty/netty/blob/4.1/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2ServerUpgradeCodec.java#L132
What do you think?

@mosesn sounds reasonable to me.

@mosesn actually I think we need another fix... let me come back to you asap.

@mosesn actually I think the real "problem" is that Http2ConnectionHandler.on*Upgrade is called before the handler is actually added to the pipeline. IMHO this is not very consistent with everything else as I think we should only call methods on handlers after these are part of the pipeline. The catch here is that its at the moment not possible to add the Http2ConnectionHandler to the pipeline as this will trigger sendPreface(...) and so onHttp2ServerUpgrade(...) will throw an exception as its called after the preface is sent. I am not currently sure what is the best way to fix this tho, as we may still need to sent the preface sometimes when the handler is added to the pipeline, just not when we will trigger an upgrade.
@nmittler @ejona86 @carl-mastrangelo @Scottmitch @mosesn any good ideas ?

Hmm . . . we might just need to supply something that's aware on creation that it's speaking h2c.  We need to effectively do this anyway on the client-side to make sure clients start on stream id 3 instead of stream id 1, so I don't think this is too heinous.

@mosesn sounds reasonable to me... let me try to finish a pr later today or tomorrow.

@ejona86 any concerns ?

@mosesn PTAL https://github.com/netty/netty/pull/7184

Fixed by https://github.com/netty/netty/pull/7184


made taxes field customizable and fixed language error in de.json

What?
PR for customizable taxes field by language json files.
Why?
Closes: https://github.com/bigcommerce/checkout-js/issues/885
Testing / Proof
No need because few changes of strings
@bigcommerce/checkout

Hello, we're looking into the wording adjustment. Meanwhile, to pass the unit test, you need to update snapshots with this command npm run test -- -u.

Hey @pauliusinc is this still relevant?

Hey @animesh1987,
the language tag for the taxes is not relevant anymore. This will be set at another position (in the settings of the store).
However, the subtotal text is still relevant and is wrong translated to German within the checkout, the cart page, all marketing emails and for the printable invoice from the shop orders and also for the invoice that is downloadable for the customer on the shop itself.
I wrote many scripts to correct this wrong translation in all this positions. If this could be fixed I would really appreciate that.

Thanks @pauliusinc for the feedback, seems like something that we need to work with our translation team since it seems like it's just not related to this repo but a wider issue.
I will close this PR in that case. And hopefully we can fix it at the root.
Again thanks for notifying us 🙇

Hi folks. We a doc on how to process this issues: https://bc-eng-docs.herokuapp.com/technical/localization/reporting-translation-problems.html

Thanks @funivan

@animesh1987 @funivan was this ticket created and rectified?

@bc-0dp seems to have been fixed https://github.com/bigcommerce/checkout-js/blob/master/packages/locale/src/translations/de.json#L77 probably via the bot.

Hi @funivan I will close #885 for now. If this issue persists, then don't hesitate to create a new issue. ❤


Some schemas need special handling

Background
Some schemas need special handling. Like not changing owner, not clearing existing functions.
Feature

Schema public should not change owner and not clear all functions, Only those that are being created.
Schema pg_toast should be off-limit (fail if referenced)

If some other limitations come to your mind, add them. 😉

Already implemented


Include Swift extensions in zip and Carthage

v9 successor to #9445 since Xcode 13 is required
Fix #7819
Fix #9325

Based on #9480, will need to rename the CocoaPods generated umbrella (like FirebaseCore-umbrella.h) to the umbrella typically expected by Firebase Objective C samples (like FirebaseCore.h) and update the generated modulemap accordingly.


Remove or expand range of gist:conformsTo

Range is gist:Intention. Definition is "The subject conforms to the Object, e.g. meet an obligation, meet terms of an offer, adhere to a specification." gist:Intention is not a superclass of gist:Obligation. So one or the other is incorrect.

gist:Obligation is a subclass of gist:Intention, by inference.

Thanks, I didn't think to check that. Closing the issue.


how to match quality of original implementation?

I have been playing with the truck sample in both this repository and in graphdeco-inria /
gaussian-splatting

^this repo

^ reference, trained with default parameters
I can't seem to replicate the same re-construction quality with this repository, (note that the fence cannot be rendered clearly). I have tried to match the learning rates, making the following changes to the config. what is causing the difference?
--- a/config/tat_truck_every_8_test.yaml
+++ b/config/tat_truck_every_8_test.yaml
@@ -31,8 +31,8 @@ print-metrics-to-console: False
 enable_taichi_kernel_profiler: False
 log_taichi_kernel_profile_interval: 3000
 log_validation_image: False
-feature_learning_rate: 0.005
-position_learning_rateo: 0.00005
+feature_learning_rate: 0.0025
+position_learning_rate: 0.00016
 position_learning_rate_decay_rate: 0.9947
 position_learning_rate_decay_interval: 100
 loss-function-config:
@@ -45,8 +45,11 @@ rasterisation-config:
   depth-to-sort-key-scale: 10.0
   far-plane: 2000.0
   near-plane: 0.4
+  grad_s_factor: 2
+  grad_q_factor: 0.4
+  grad_alpha_factor: 20
 summary-writer-log-dir: logs/tat_truck_every_8_experiment
-output-model-dir: logs/tat_truck_every_8_experiment
+output-model-dir: logs/tat_truck_every_8_experiment_matched_lr

Based on my experience, the repo's implementation has a lot difference than the official repo, so does its rendering performance. One notable difference is its gaussian densitification strategy is much conservative than the one by official repo. Directly matching the parameters wouldn't lead to the same performance. But I had found cases that Taichi GS performs better than official one. It is really case by case.


Paket.Bootstraper.exe v5.243.0 returns 404 from GitHub

Description
Paket.Bootstraper.exe from release 5.243.0 returns 404 from GitHub.
Paket.Bootstrapper.exe seems to be missing from release 9.0.0, meaning we can't update to newest version.
Repro steps
Please provide the steps required to reproduce the problem

Run Paket.exe (Magic Mode - using bootstrapper v.5.243.0)

Error : (404) Not Found. (Github - cached (temporarily ignore updates))

unfortunately .NET 9 doesn't support net461 packaging so that the bootstrapper needs a new design.
But dotnet tool is usually the way forward

I'm not sure if this is the same thing as described by the original post, but when I install the latest tool version with dotnet tool update paket (9.0.0), I get 404 errors when I run dotnet paket update.
EXEC : The remote server returned an error : (404) Not Found. (Github - cached (temporarily ignore updates)) [D:\Developer\xxxx.xxxx.xxxx\Tests\Tests.csproj]
EXEC : The remote server returned an error : (404) Not Found. (Github - cached (temporarily ignore updates)) [D:\Developer\xxxx.xxxx.xxxx\xxxx.xxxx.xxxx.xxxx\xxxx.xxxx.xxxx.xxxx.csproj]

What confuses me the most is, that this change is a breaking change we haven't been informed of.
When I run an older paket.bootstrapper.exe in magic mode (paket.exe), I get the error 404 Not Found (Github - cached...) and it then tries to use the 9.0.0-alpha001 version to do the restore, even though it just failed with 404.
Does it successfully get that version from nuget.org instead, or what's going on?
I've looked at https://fsprojects.github.io/Paket/bootstrapper.html but there's no mention of any breaking changes regarding the bootstrapper.
It seems wrong that our old pipelines should suddenly start failing, because an older bootstrapper cannot find a suitable version anymore.
We've fixed it by adding "version 8.0.3" to our paket.dependencies files, so it still works.
Some of our pipelines are old and won't be updated, so we'd either continue fixed on 8.0.3, or remove paket as a dependency.
I don't see any information about paket.bootstrapper.exe being deprecated or removed in 9.0.0, and even if it is, I would have assumed bootstrapper would be marked as deprecated and instead be locked to versions >= 8.0.3 so it wouldn't break existing pipelines and simply download 8.0.3 as the latest version while giving a warning that bootstrapper is deprecated.
This is a sudden breaking change - without any clear notice - so I assume this is a bug, right?
A bug that breaks bootstrapper if version isn't locked to 8.0.3 or lower.

please retry

This should be fixed


pcli: add flag for reading in tendermint priv key

When configuring a validator, our docs currently say:

By default template-definition will use a random consensus key that you won’t have access to. Make sure to set the consensus_key correctly otherwise your instance of tendermint won’t be using the key expected in the validator definition. You can get the correct value for consensus_key from your tendermint configs:

There's a bit of interface tension because pcli cannot assume that it has access to the same context as pd; it may, but we don't know that for sure, thus the instruction to run a shell command manually. We can do better. Let's add an optional flag to pcli to point to the priv_validator_key.json file on disk. If that flag is set and the file exists, read the key material from it for use in the template. Otherwise, proceed with filling in a random value, as we do now.
The docs can then be update to recommend using that flag, and qualifying when it's appropriate. See related discussion in Discord.

Incidentally, implementing this would allow us to remove some boilerplate io code in the network integration tests.


Question about global_max_pooling

❓ Questions & Help

I found the func global_max_pooling use torch_scatter to implement.
I want know how to compute the max value of a feature_matrix?
Is it use absolute value or other func to select the max value of feature matrix?
Could you tell me the principle of implementation?Thank you very mach!

The maximum is taken over the reduce dimension, hence it uses the absolute value encountered for same indices:
x = [[3, 4], [5, 2], [2, 1]]
index = [0, 0, 1]
out = scatter(x, index, dim=0, reduce='max')
>>> [[5, 4], [2, 1]]

Fine,thank you for your answer.
But I can't understank how to correspond index and dim to x,in this case.
Could you tell me more about it?It troubled me a few days.
Thank you very mach!

dim denotes along which axis we want to aggregate. For an input tensor of shape [E, D], and an index tensor of shape [E], a scatter operation in dimension 0 will reduce the tensor to a shape of [N, D], where N = index.max() + 1. Basically, you need to ensure that index.size(0) == inputs.size(dim).

Fine!thank you!I know it.


Hubot overloading Slack API calls while running on PCF

We have a Hubot app on our PCF and it appears once a few commands are made via Slack it will work fine the first few times, but then something happens and it starts spamming the Slack API to the point of where Slack disconnects the bot.
If I run this locally, the issue has not presented itself, so it appears to be a PCF/Hubot issue.
I have attached copies of the main script (called dsli.txt) , the function that seems to break it the most (help.txt), and log (dsli.log). I have changed the .coffee extensions to .txt for purposes of attaching them to this post. I also changed IP and site address for security reasons.
dsli.txt
help.txt
dsli.log
Row 1 shows a GET 404 0 143 error, followed be several more 404 type errors. I cannot figure out where they are originating from, especially the "OPTIONS", but if what I read is correct, I don't need to worry about the "OPTIONS" 404. Row 24 seems to be a known issue and does not have a fix. Please let me know if I'm wrong on that. Row 31 does not make sense to me either and does seem to prelude the issue, so is that the cause? But that doesn't make sense to me that it would be the cause for spamming the API. Finally, row 53 is the start of the "Rate limited, will retry..." issue.
So far the "fix" is to restart the app, but then it only works for a handful of commands before it goes haywire again.
Other services or general info that might be useful to know:
Connected to Redis
App AutoScaler
T-Logger
Nodejs Buildpack
Any help you can offer is appreciated, thank you.

I'm wondering if PCF is killing and re deploying the instance?

I'm wondering if PCF is killing and re deploying the instance?

To the best of my knowledge, the tails do not indicate anything like that in the log file.

We have a Hubot app on our PCF and it appears once a few commands are made via Slack it will work fine the first few times, but then something happens and it starts spamming the Slack API to the point of where Slack disconnects the bot.
If I run this locally, the issue has not presented itself, so it appears to be a PCF/Hubot issue.
I have attached copies of the main script (called dsli.txt) , the function that seems to break it the most (help.txt), and log (dsli.log). I have changed the .coffee extensions to .txt for purposes of attaching them to this post. I also changed IP and site address for security reasons.
dsli.txt
help.txt
dsli.log
Row 1 shows a GET 404 0 143 error, followed be several more 404 type errors. I cannot figure out where they are originating from, especially the "OPTIONS", but if what I read is correct, I don't need to worry about the "OPTIONS" 404. Row 24 seems to be a known issue and does not have a fix. Please let me know if I'm wrong on that. Row 31 does not make sense to me either and does seem to prelude the issue, so is that the cause? But that doesn't make sense to me that it would be the cause for spamming the API. Finally, row 53 is the start of the "Rate limited, will retry..." issue.
So far the "fix" is to restart the app, but then it only works for a handful of commands before it goes haywire again.
Other services or general info that might be useful to know:
Connected to Redis
App AutoScaler
T-Logger
Nodejs Buildpack
Any help you can offer is appreciated, thank you.

I believe Cloud Foundry makes those initial requests to the App instance as part of its lifecycle. Since the app is a NodeJS app and not a Spring Boot app, it doesn't handle those URIs.
I think the warning in line 31 is because the code is doing exports.help = (robot) -> instead of exports = (robot) ->. But I haven't tried to prove that with running code. I suggest you try that change first and run it to see.
I think I ran into the retry error when I inadvertently started multiple instances of Hubot. My guess is that App AutoScaler is the actual cause of the problem. Try not using that and just start a single (1) instance of Hubot.

Gone stale. Please open a new issue if you're still having problems.


drivers: pwm: add GD32 PWM driver

This PR introduces an initial version of a PWM driver for GD32 SoCs. No PWM capture support for now.
Tested on GD32F450I-EVAL using:

samples/basic/blinky_pwm
samples/basic/fade_led

Also tested on GD32VF103V-EVAL board (not part of this PR yet) https://github.com/teslabs/zephyr/tree/pwm-gd32-vf103.
Confirmed generated waveform periods are correct using scope.

F403 status:
As is:
PWM-based blinky
Calibrating for channel 0...
Error: PWM device does not support a period at least 62500

After add prescaler = <256>
PWM-based blinky
Calibrating for channel 0...
Done calibrating; maximum/minimum periods 62500/15625 usec

samples/basic/fade_led requires at least prescaler = <64>
In both cases, no visual feedback on LED.

F403 status:
As is:
PWM-based blinky
Calibrating for channel 0...
Error: PWM device does not support a period at least 62500

After add prescaler = <256>
PWM-based blinky
Calibrating for channel 0...
Done calibrating; maximum/minimum periods 62500/15625 usec

samples/basic/fade_led requires at least prescaler = <64>
In both cases, no visual feedback on LED.

@nandojve I have tested the driver on GD32VF103V-EVAL (which also uses TIMER0_CH0) and everything seems to work fine.


chown error on provate storage

Hi,
I'm having an error when re-creating a room with private storage.
I'm mapping /home/heko/.mozilla folde to keep profile when the room is recreated I get this error:
chown /data/rooms/U00-gbrian/mozilla/firefox/p3gad2ub.default-esr/lock: no such file or directory
I think this issue is happening because lock is a linked file and that causes a problem with docker container if host and cotainer does not have the same folder struture.
I've commented out the lines below until having a better solution
at https://github.com/m1k1o/neko-rooms/blob/b183c1e0521956f55afcf37c836aab40ad90da87/internal/room/manager.go#L216

I'd say chown should only be done on creating the folder and not on recreating.
Or maybe, if that exists it should change only permission of the folder and not the whole tree structure inside. I'll look into it today.

Great, thanks! Not in a hurry.
Can you use UID and GUID from the user running neko-rooms?
	privateStorageUid   = 1000
        privateStorageGid   = 1000

I'm setting the values  of user at docker-compose.yaml to match with the owner of the private storage.
Does it make sense?
thanks

That would make sense, although the user needs to match also neko user where it is by default 1000. I don't think, that it is able to be modified right now (only during building).

@m1k1o ,
working fine. Thank you so much 👏


Profitability chart for dashboard item pop-up looks very ugly

Are you able to reproduce this consistently? Although I don't see how this could affect anything, does it persist after providing a translation for the hydrogen turbine? Does it happen with a default scenario?
If it's still broken could you provide details of the branches, scenario, and browser being used?
It looks normal to me locally and on beta:

On beta this still doesn't look ideal:


Continuous integration and testing

I suggest: https://travis-ci.org/ or https://concourse-ci.org/
I want to give it a try.

Ok.
I'll update the cmake to make it easier.

Fixed by https://github.com/Lomadriel/WaitFreeHashMap/pull/13.


BLE + HTTPS(mbed) on esp32-wroom-32 clarification

Hi,
I have encountered the not too uncommon problems on my ESP32 board with WROOM module (4MB Flash + 520KB RAM).  Is it not possible to run BLE + a fully featured Wifi connection on this board?  I think the answer is "no".  I have other items taking up heap as well (globals, arduinojson, etc.),  Insecure wifi (http) is not an option.
I have tried the following to remedy this (after reading quite a few issues), to no avail:

Enable COEX
Disable BLE client (I'm only acting as a server)
Switch to nimble instead of bluedroid
Reduce TLS message size from default 16834 to 4800 (CONFIG_MBEDTLS_SSL_MAX_CONTENT_LEN).  Unclear if this will work for my application long term, but didn't fix it anyway.
Switching BLE to core 1 (expected no change and was correct :))
Disable BT classic (off by default in the ESP-IDF menuconfig, I believe)
Moving arduinojson (~400bytes) to stack

I think the conclusion here is not really the one I want--that my use case is only possible by either:
-Adding more RAM
-Swapping back and forth between BLE and full WIFI+HTTPS usage.
Am I missing something?
Thanks!
PS I'm using vscode+platformio and adafruit huzzah32 for my application.  menuconfig changes uploaded via esp-idf

Hi,
there is no easy answer on this question. In theory it may be possible, but all depends on full requirements and code.
I can give you few suggestions, which may help you with that:

you are working with platformio, so, if you can try to use arduino + espidf framework; this way you can control some memory parameters in menuconfig
try to use cJSON instead of ArduinoJSON library; i dont have proofs but i think cJSON is light weight, only problem is it is C library
point 6) classic BT is not disabled by default in menuconfig, and you can add memory free in setup somewhere at beginning just in case
point 4) this change will works only in case arduino + espidf, with arduino only any changes in sdkconfig are ignored
point 3) switch to nimble is best option and should be fairly easy thanks to @h2zero who did great job porting it and making it almost 100% compatible with old library

Hi @jshep321, @chegewara has posted great advice already.
Just wanted to confirm that using my NimBLE port of this repo will likely free up 40-60kB of ram. I run a web server and BLE together in my application with 100kB free heap left over.

Thanks guys.   I will give it a go.  I'm not clear if my earlier menuconfig updates were staying active.  I was using the CLI esp-idf to upload changes and then moving back over to vscode+platformio to upload my app.  Will this work?
Also @h2zero, is the nimble port loaded in the default esp-idf the same as yours or do I manually pull in from here? https://github.com/h2zero/NimBLE-Arduino

@jshep321 If you're just using the Arduino platform setup in platformio then I suggest just adding lib_deps = h2zero/NimBLE-Arduino to your platformio.ini

Integrated nimble and works like a charm!
@h2zero much thanks for the migration guide at https://github.com/h2zero/NimBLE-Arduino/blob/master/docs/Migration_guide.md as it was very clear!
Here's my compile time header flag you can steal if you want:
#ifdef BLUEDROID
    #include <BLEDevice.h>
    #include <BLEServer.h>
    #include <BLEUtils.h>
    #include <BLE2902.h>
#endif
#ifdef NIMBLE
    #include <NimBLEDevice.h>
#endif

Glad to hear it and thanks for the feedback!


Lazy Join with index scans.

This is not yet quite done and will be split up into several
preparation PRs.

SKETCH of commit message:
Unter the following conditions, the operands of a join operation are not fully materialized:

one of the operands is an index scan above a certain size (TODO: find name of constant)
single-column join without UNDEF values in the join column

Then we first compute the set of blocks that are sufficient to compute the correct join results.
These blocks are then produced one after the other and buffered only as much as is necessary for computing the correct result (whn computing cross-products, we sometimes need more than one block per operand at the same time).
There is also some buffering for efficiency reasons (so that the consumers have to wait as little as possible).


SparkLinker:  Improve performance ofestimate_u_using_random_sampling

Spark does not automatically repartition (shuffle) data when it computes a Cartesian product.
Splink takes a small sample of records before computing the Cartesian product in order to estimate the u values.
There's no guarantee this sample will be distributed evenly across the cluster, and in practice we've found it's distributed poorly.
This PR performs an explicit repartition of the sampled data, resulting in big performance improvements for ofestimate_u_using_random_sampling.

Testing code that helped to isolate the cause of the problem:
import pandas as pd

from pyspark.context import SparkContext, SparkConf
from pyspark.sql import SparkSession
from pyspark.sql import Row

from string import ascii_lowercase, ascii_uppercase, ascii_uppercase

conf = SparkConf()

conf.set("spark.driver.memory", "12g")
conf.set("spark.sql.shuffle.partitions", "12")
conf.set("spark.sql.autoBroadcastJoinThreshold", "-1")  # disable auto broadcast join
conf.set("spark.sql.adaptive.enabled", "false")
sc = SparkContext.getOrCreate(conf=conf)

spark = SparkSession(sc)
spark

data = {
        "unique_id": range(26),
        "col_1": list(ascii_lowercase),
        "col_2": list(ascii_uppercase),
    }

import pandas as pd

df_pd = pd.DataFrame(data)
df_pd = df_pd.merge(df_pd, how="cross", suffixes=("_a", "_b")).merge(df_pd, how="cross",  suffixes=("_c", "_d")).merge(df_pd, how="cross")
df_pd = df_pd.reset_index()
df_pd.to_parquet("400k.parquet", index=False)
df_pd.to_parquet("400k_1.parquet", index=False)
df_pd.to_parquet("400k_2.parquet", index=False)

# df_l = spark.createDataFrame(df_pd)
# df_r = spark.createDataFrame(df_pd)
# df = spark.createDataFrame(df_pd)

repartition = True
repartition_size = 2
df_l = spark.read.parquet("400k.parquet")

df_r = spark.read.parquet("400k_1.parquet")

df = spark.read.parquet("400k_2.parquet")

if repartition:
    df_l = df_l.repartition(repartition_size)
    df_r = df_r.repartition(repartition_size)
    df = df.repartition(repartition_size)

df_l.createOrReplaceTempView("df_l")
df_r.createOrReplaceTempView("df_r")
df.createOrReplaceTempView("df")

df_pd = df.limit(2).toPandas()
# df_pd

l1 = [f"l.{c} as {c}_l" for c in df_pd.columns]
l2 = [f"r.{c} as {c}_r" for c in df_pd.columns]
select = ", ".join(l1 + l2)

sql = f"""
select {select}
from df_l as l
inner join df_r as r
on l.index < r.index
where l.index < 100 and r.index < 100
"""

result = spark.sql(sql)

result.write.mode("overwrite").parquet("temp/")


Document a few new SDK properties customers affecting Maui and xplat compilation

Help us make content visible
DisableTransitiveFrameworkReferenceDownloads
Microsoft.Maui.Essentials
EnableWindowsTargeting
NETSDK1100
Describe the new article
@tdykstra, this PR makes three changes we want documentation on: https://github.com/dotnet/sdk/pull/25358

If a customer wants to build a windows application (WPF, Winforms) on a non-windows platform, they will get NETSDK1100 today. We've changed the behavior so that if they set true, this will enable them to get unblocked. This is because we don't automatically download targeting and runtime packs on platforms that aren't supported but with this flag, we now will when cross-targeting.
If a customer has a reference to a project that uses Maui essentials but haven't set true, then they will need to. We can't flow the maui essentially dependencies automatically without breaking other scenarios but setting this will make sure that the referencing project correctly pulls in the right known framework reference for maui essentialls.
We added DisableTransitiveFrameworkReferenceDownloads. Setting this to true will let customers avoid downloading extra runtime and targeting packs they didn't need. For some Maui windows scenarios, it was trying to download aspnet and windows desktop packages just in case they were needed later and this will disable that unless directly used.

CC @dsplaisted  to review this when the documentation is ready.

@marcpopMSFT @dsplaisted For UseMauiEssentials, it seems that the namespaces are no longer named "Maui.Essentials" - https://github.com/dotnet/maui/pull/5562#issuecomment-1079255633. Did/should the property name change too?

@marcpopMSFT @dsplaisted For UseMauiEssentials, it seems that the namespaces are no longer named "Maui.Essentials" - dotnet/maui#5562 (comment). Did/should the property name change too?

@Redth @jonathanpeppers Can you comment on this?

I've found DisableTransitiveFrameworkReferenceDownloads is needed sometimes.
Errors like this for an Android project:
Unable to find package Microsoft.WindowsDesktop.App.Runtime.win-x64 with version (= 7.0.0)

Which is probably caused by stable branding/feeds changing. This was noticied on the https://aka.ms/dotnetperfstatus pipelines that build MAUI projects and measure performance.
But then a customer wouldn't hit this? Just dogfooders, and internal pipelines?


Add support for overriding daysUntilClose for pull requests

This PR adds support for overriding daysUntilClose for pull requests, in the form of an optional key called daysUntilPullRequestStale.

View rendered README.md

Is there some reason why https://github.com/probot/stale/commit/2a7abe60be4c6046508338fa67106ec6f19fd08b from https://github.com/probot/stale/issues/66 doesn't handle this use case?

Oh, I must have missed that in the documentation. Yeah, it should do what I need (and more), thanks for pointing it out!


project to preprocess spancat datasets into .spacy

This is a project that is meant to help with experimenting with the spancat and ner components. Currently it has:

ConLL (English, German, Spanish, Dutch)
WiniNeuRal (English, German, Spanish, Dutch)
Dutch Archeology data set
AnEM
Wnut 2017

For the archeology data set it creates a random train/dev/test split and for all other data sets it uses the standard splits. It has a small utility view_spans.py to show a couple of random examples on the command line from a data set and runs debug data with the a default spancat config to show the span-characteristics.
The idea is to have a home for the spancat/ner data sets to be able to easily benchmark components on them. Plus others can use these data sets perhaps for pre-training.

The project currently lives here https://github.com/explosion/spancat-datasets/, but there is no reason not to make it public and its nicer if its in the projects in any case as we discussed.


Question: Ignore SubFolders in source

We have a case where we want to copy only those files directly under Source folder to Destination folder, and ignore any subfolders under Source folder.  Is there a switch that supports this requirement?

Yup, -n (non-recursive)

Just published a new release just in case:
https://github.com/KirillOsenkov/ContentSync/releases/download/v1.6/ContentSync.exe

Thanks!
Could you please include version number in ContentSync.exe. Otherwise it is
difficult to know which version we are on currently.
Thanks
Geetha.
On Mon, Aug 20, 2018 at 7:12 PM Kirill Osenkov<EMAIL_ADDRESS>wrote:

Just published a new release just in case:
https://github.com/KirillOsenkov/ContentSync/releases/download/v1.6/ContentSync.exe
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KirillOsenkov/ContentSync/issues/3#issuecomment-414505024,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AETas0ixQIUjNuKRc_7bxkY2zy6CdErLks5uS1BjgaJpZM4WET8H
.

Will do, thanks. Filed an issue here: https://github.com/KirillOsenkov/ContentSync/issues/4


feat: add biosample index

✨ Context

🛠 What does this PR implement

🙈 Missing

🚦 Before submitting

[x] Do these changes cover one single feature (one change at a time)?
[x] Did you read the contributor guideline?
[x] Did you make sure to update the documentation with your changes?
[x] Did you make sure there is no commented out code in this PR?
[x] Did you follow conventional commits standards in PR title and commit messages?
[x] Did you make sure the branch is up-to-date with the dev branch?
[x] Did you write any new necessary tests?
[x] Did you make sure the changes pass local tests (make test)?
[x] Did you make sure the changes pass pre-commit rules (e.g poetry run pre-commit run --all-files)?

Few notes:

When joining onto eQTL catalogue three biosampleIds have no match in the biosample index: BTO_0000930  (neuroblast), EFO_0005292 (lymphoblastoid cell line), EFO_0004905 (induced pluripotent stem cell). When thinking about the other use cases for this index some thought should go into the best way to capture these biosamples.
Current no column for deprecated or not. Not super relevant for current purposes of left-joining and wasn't entirely clear how to implement (drop deprecated, flag them, etc)
dbXrefs currently a flat list of strings rather than split into ID and as done for variant index. This was partially for simplicity, partially because I was unsure how best to split (e.g. CL_1001593 -> [CL_1001593, CL], [CL_1001593, Cell Ontology], [1001593, CL], [CL_1001593, Uberon (ontology actually sourced from)] and partially because I didn't see an immediate use case unlike variant index.


Is LAMSA suitable for short reads?

Hi,
Thank you for developing the wonderful aligner LAMSA.
I have learned that when dealing with long reads, LAMSA has a good ability to handle SVs.
I have some short reads (150bp paired-end reads), and I was wondering that if LAMSA is suitable for short reads mapping and if it still performs good when handling SVs?
Best wishes，
SongtaoGui

Hi Songtao,
Thanks for your interest in LAMSA.
But unfortunately, LAMSA is not suitable for short read (neither paired nor single).
For short-read alignment and SV detection, I would recommend you to try bwa-mem or other aligners, along with SV detection tools like LUMPY.
You may find some other useful tools here.
Yan


Basic support for Remaster. Need optimized.

Music and Video playback is disabled reading from other.zzz and main.zzz. I was having issues getting the code to work reliably. Though it might be easier now because the other code has less bugs now.
I was running out of ram because I was loading multiple copies of zzz files and fs files into ram. I countered that but removing things from ram. Though increasing the complexity so it's slower. Speed is fine in 2013 as it has most of the files in a folder. My main goal was to get it functional and mostly bugfree.
If i just extract the files with demaster or another program it can run just as fast as 2013 version. Though i haven't told it to check those folders yet. To debug and improve zzz file handling. I think L4Z supports streams? I haven't experimented with that yet.
Currently the threads are disabled. They can make things faster by doing operations in parallel but they hide unoptimized code and bugs. So it made it difficult to debug or see why it'd error sometimes and not error others. I'd like this to be toggleable from a config file in the future. So users can get the faster load times.
Few more tweaks and fixes from before this project. like a null check or reorganizing sound class into 2 files instead of 1 big file.

I forgot to mention this does not support the upscale textures yet. Frame rate is slower with the new models. Might need to optimized the way we update the vertices for animations.
During battle:

In remaster

44-55 fps
The Memory graph goes up to 1 gb then back down to 425 mb or so

2013

90-120 fps
The Memory graph stays steady.


Question: how to explore a data dump?

Apologies if this is the wrong venue for questions, but:
I recently changed my read-XML-and-dump-postcard code to use quick-xml's serde integration instead of reading XML manually, using the usual serde annotations on my structs here and there instead of FromStr. The postcard dump is now 1MB larger and I don't understand why. Is there some tooling to help me understand where that MB is coming from?

Hey! I don't necessarily have an off-the-shelf tool for you, but it might be worth looking at the two experimental features, which can get you a Schema and MAX_SIZE for a given type. This might aid in your diff.
If that doesn't work, and you can give me a smaller repro case (with types used and before/after data), I can take a look!


Convert all tuples to choice classes

currently the choices are defined as list of tuples and in some places we have to reverse them as well which does not properly indicate what the value of the particular choice is, so instead we should use django textchoices and integerchoices

I would like to work on this issue.

Could you plz assign this issue to me .

@itxsoumya  I've already started working on this and partly done as well. Could you please find any other open issue to work on? Thanks

@sainak I have familiarized myself with this issue can you plz assign me if this issue is still open

@sainak can I take on this issue

Hey @sainak If no one is working, can I try this issue ?

@rithviknishad sir , I’ve created a draft PR with some initial changes to convert tuples to choice classes. Could you please review the current changes & tell me if I am doing it right ot not ?

@sainak , @rithviknishad sir, I have converted few tuples to choice classes & created a draft PR can you please just review it once & tell me if I am doing it right


Update usage of del() in example

Fixes #54

@OverZealous If you would prefer the other method mentioned in #54, I'm happy to create that pull request instead, but this seemed to be the best match with the existing documentation.

Yeah, the return makes more sense.  The original README had a return with gulp-clean, but someone wanted to change that, too.
The point of the docs is not to show how to clean or delete items, but to show how tasks are set up to handle async processes correctly.

I updated the readme.


How to use useQuery with a transform function

Thanks for this great library.
I have a question regarding the usage of useQuery with a transform function.
I'm using it like this:
<script setup lang="ts">
const { $client } = useNuxtApp()

const { data, pending } = await $client.admin.getContent.useQuery(undefined, {
  lazy: true,
  transform(value) {
    return value
  }
})
</script>

but I get a Typescript error on the transform function

what am I missing?

It's been a while and I hope you have solved your issue, but I think it should be
transform: (value) => {
  return value
}

Thanks @mlbiche


Fixes #412 - Integrate support for pFlogger

Added support to use pFlogger for logging

Command line option: --logging_config=
Each grid comp now has an associated logger with name given by the
ancestry.  E.g.,   CAP.GCM.AGCM.PHYSICS
Added warning message:
When not providing pFlogger a yaml file, a warning message is
emitted (to last_resort handler most likely).

How Has This Been Tested?

Simple run of GCM at coarse resolution
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[X] I have tested this change with a run of GEOSgcm (if non-trivial)
[X] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[X] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

I ran some tests with Intel and GCC. I ran a case with a logging.yaml and without one.
Both logging and no-logging throw FAIL messages on every PE:
pe=00001 FAIL at line=02807    MAPL_Generic.F90                         <status=1>
pe=00001 FAIL at line=00110    MAPL_CapGridComp.F90                     <status=1>

When you run without logging you also get a warning on every PE:
WARNING: MAPL: No configure file specified for logging layer.  Using defaults.

Everything seems good with this fix. Both Intel and GCC 9 seem to run. I only did Release though I imagine @tclune tested with Debug as he found that MemUtils bug.

Note. I also did a set of C48 runs of AMIP, Replay, and Replay-no-inc. They all ran without issue, which is nice. I also did gcm_regress in all three with successes. So if nothing else, we are good.


Add math.atan2

Needs a confirmation approval from @bioball (having requested changes); they look addressed to me.


[conductor] Cherrypick PR link available to release_dashboard

The conductor core should return the PR creation link back to the release_dashboard in order to be displayed in the UI.

Oops, this is already done: https://github.com/flutter/flutter/blob/master/dev/conductor/core/lib/src/globals.dart#L135


Neither a test this please comment or a change to the PR triggered the build

Using by default only username and passwords fields to provide BitBucket auth cause NullReferenceException in BitbucketRepository::getCredentials(),
because credentialsId is null.

Resolve


Don't validate invisible or gone fields

When a field is gone or invisible, it is impossible for the user to fill it.
Maybe it is possible to don't validate it ?
Validator.java; Method checkFields :
protected boolean checkFields(ValidationCallback callback)
{
        boolean result = true;
        mHasFailed = false;
        for (ValidationHolder validationHolder : mValidationHolderList)
        {
// Patch
           if (validationHolder.getEditText() != null
                && validationHolder.getEditText().getVisibility() != View.VISIBLE)
            {
                continue;
            }
// End Patch

            if (validationHolder.isRegexType())
            ...

Good question.  Would you please share a use case?  I thought a field is always visible until the validation passes (then the whole view which contains the fields to be validated will be gone).

Imagine a form with a spinner representing an identity piece type. You can select none/driver licence/id card. If the user select something different than "none", you show a field to type the serial number for exemple.

same here, sometime i passed some id to invisible edittext

Thanks @mr-day for the input, I'll start to work on this after I finish another issue with ConstraintLayout.

Nice, I'm glad to hear that :)

This feature has been added to TODO list.


update for recent GHC

Once critbit PR https://github.com/bos/critbit/pull/93 makes it to hackage, this change should allow building the package with current GHC releases. In particular, it might help people trying to build postgrest.

Meanwhile postgrest has migrated from configurator-ng to configurator-pg, so from that point of view this update is no longer necessary.


Drum Application

Description
Make a webpage having the drum ie on clicking of different keys, different sounds will be coming.
Technologies and Languages

HTML, CSS, JS

Feel free to use any other language and technology

can i take this up ?

Sure Go ahead

@bislara Can I work on this one?

sure go ahead

@hey can I work on this issue? Is it still available?

@bislara can I work on the issue? is this still valid for hacktoberfest 2021?

Yes its valid. Go ahead and start working.

@bislara  Hey, I have added my project of drum kit into the file.  Please do have a look.

Sure. I have reviewed the project. Please add the required improvements

@bislara I have made required improvements. Please do have a look


Segmentation fault when running any of the example programs

Specifically, I am trying to run bfs on the LiveJournal graph provided as the example. I successfully got the graph preprocessed, but running ./bin/bfs /LiveJournal_Grid 0 8 breaks. When I run it with GDB, I get this error message:
Program received signal SIGSEGV, Segmentation fault. _IO_vfscanf_internal (s=0x0, format=0x406b7d "%d %d %ld %d",      argptr=argptr@entry=0x7fffffffd2e8, errp=errp@entry=0x0) at vfscanf.c:347 347	vfscanf.c: No such file or directory.
Am I doing something wrong when running the program or is there an issue with the way it compiled?

Hi @tonyfleck76 , can you provide me the detailed commands you used including preprocessing?

Well, I first ran it on the graph downloaded from the link provided in the README. I then ran
./bin/preprocess -i /data/LiveJournal -o /data/LiveJournal_Grid -v 4847571 -p 4 -t 0
on it and got an error. When I looked at the issues I saw the other open one had some code to convert the LiveJournal file into binary form. I ran it through that and then ran the same prepocessing command. That appeared to have worked and gave me the LiveJournal_Grid directory that I have been passing as an argument to the bfs program.

Well, the preprocessed files are located at "/data/LiveJournal_Grid", but it seems that you passed "/LiveJournal_Grid" to the bfs program. Is this the problem (i.e. so fscanf failed to get content from the file)?

Unfortunately that is not it. Sorry, I should have changed that to show that I did not create a data directory. LiveJournal_Grid is in the main GridGraph directory.

What does the LiveJournal_Grid folder look like (e.g. sizes of the files under the folder) ?

ls LiveJournal_Grid/ block-0-0  block-1-0  block-2-0  block-3-0  column         parent block-0-1  block-1-1  block-2-1  block-3-1  column_offset  row block-0-2  block-1-2  block-2-2  block-3-2  degree         row_offset block-0-3  block-1-3  block-2-3  block-3-3  meta
Also, I worked out the segfault issue, but get this error instead.
GridGraph/core/bigvector.hpp:89: void BigVector<T>::init(std::__cxx11::string, size_t) [with T = int; std::__cxx11::string  = std::__cxx11::basic_string<char>; size_t = long unsigned int]: Assertion 'fd!= -1' failed. Aborted (core dumped)

-rw-r--r-- 1 tony tony 269334576 Jul 13 10:56 block-0-0
-rw-r--r-- 1 tony tony  51552688 Jul 13 10:56 block-0-1
-rw-r--r-- 1 tony tony  19681928 Jul 13 10:56 block-0-2
-rw-r--r-- 1 tony tony   2791952 Jul 13 10:56 block-0-3
-rw-r--r-- 1 tony tony  55155664 Jul 13 10:56 block-1-0
-rw-r--r-- 1 tony tony  50074176 Jul 13 10:56 block-1-1
-rw-r--r-- 1 tony tony  12396840 Jul 13 10:56 block-1-2
-rw-r--r-- 1 tony tony   6498848 Jul 13 10:56 block-1-3
-rw-r--r-- 1 tony tony  19986976 Jul 13 10:56 block-2-0
-rw-r--r-- 1 tony tony  12129536 Jul 13 10:56 block-2-1
-rw-r--r-- 1 tony tony  19227224 Jul 13 10:56 block-2-2
-rw-r--r-- 1 tony tony   5787824 Jul 13 10:56 block-2-3
-rw-r--r-- 1 tony tony   2751472 Jul 13 10:56 block-3-0
-rw-r--r-- 1 tony tony   6035224 Jul 13 10:56 block-3-1
-rw-r--r-- 1 tony tony   5335752 Jul 13 10:56 block-3-2
-rw-r--r-- 1 tony tony  13209504 Jul 13 10:56 block-3-3
-rw-r--r-- 1 tony tony 551950184 Jul 13 10:56 column
-rw-r--r-- 1 tony tony       136 Jul 13 10:56 column_offset
-rw-rw-r-- 1 tony tony  19390284 Jul 13 11:00 degree
-rw-rw-r-- 1 tony tony        20 Jul 13 10:56 meta
-rw-rw-r-- 1 tony tony  19390284 Jul 13 10:57 parent
-rw-r--r-- 1 tony tony 551950184 Jul 13 10:56 row
-rw-r--r-- 1 tony tony       136 Jul 13 10:56 row_offset

I thought this might be a bit more useful.

You might try inserting the following statement at Ln.89 in "core/bigvector.hpp" (before the assertion):
perror("error message: ");

It could tell us what the exact error is. Thanks.

bin/bfs LiveJournal_Grid/ 0 16
Error message: : Invalid argument
bfs: GridGraph/core/bigvector.hpp:90: void BigVector<T>::init(std::__cxx11::string, size_t) [with T = int; std::__cxx11::string = std::__cxx11::basic_string<char>; size_t = long unsigned int]: Assertion `fd!=-1' failed.
Aborted (core dumped)

It seems that O_DIRECT is not supported... What is the kernel version? And the type of the file system? I found some similar cases via Google but I am not sure what exactly causes the problem.


Improvement of Model Creation Documentation for Engine Creation Process

Is your feature request related to a problem? Please describe.
Description:
There appears to be a lack of clarity in the process of creating a new model. Specifically, it is not evident what components are required for the creation of certain models, such as the VAE or text encoder.
Actual Behavior:
At present, the model creation process appears to be somewhat ambiguous and lacking in clarity, particularly in regards to the necessary components required for certain models.
Describe the solution you'd like
It is expected that the model creation process will be clearly documented and outlined, including any necessary components or dependencies required for the successful creation of the desired model.
Describe alternatives you've considered
No response
Additional context
Steps to Reproduce:

Attempt to create a new model, such as the VAE or text encoder.
Observe the lack of clear documentation or guidance on the necessary components required for the creation of these models.

Impact:
The lack of clear guidance in the model creation process may result in confusion and frustration for users attempting to create new models. This could ultimately result in decreased usage of the platform or project.
Recommendation:
To address this issue, it is recommended that clear and detailed documentation be provided outlining the necessary components required for the creation of various models. This will enhance the user experience and facilitate the successful creation of new models.
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that asks for the same feature to avoid creating a duplicate.

There is also a problem of repos containing multiple different models. https://huggingface.co/WarriorMama777/OrangeMixs
I am not sure if it is going to download all 176 gbs of models into the docker and try to convert them all.
I guess there is supposed to be a way to pick a single model out of the repo.

There is also a problem of repos containing multiple different models. https://huggingface.co/WarriorMama777/OrangeMixs
I am not sure if it is going to download all 176 gbs of models into the docker and try to convert them all.
I guess there is supposed to be a way to pick a single model out of the repo.

You can think of it as running a plain git clone on the repo, there is a way to specify subfolders but its definitely not the best way to do any of this - there should be only one model per repo (or at least branch)
If you want a temporary fix, then create such a repo by yourself (its really easy to do) and upload just the files of one model to it, you can check other repositories like https://huggingface.co/andite/anything-v4.0 to see how the files should be structured.

After some digging, it seems like diffusers don't download everything if the repo is configured correctly.
Most of the time people download models as files such as on https://civitai.com/ and directly use these files when generating things. This is the workflow that https://github.com/AUTOMATIC1111/stable-diffusion-webui steers people towards because it doesn't read the diffuser format at all.
It seems like the current workaround to load custom models is to:

Create a git repo.
Add your safetensor, VAE, and other things into the repo.
Upload your repo to huggingface.
Download your modified repo into the docker file.

Someone experimenting with dozens of models would have to repeat these steps for each model.

The root of the problem is that the most popular UI for stable diffusion doesn't actually use the diffusers library at all. Instead, it manually loads all of the files
https://github.com/AUTOMATIC1111/stable-diffusion-webui/blob/e644b5a80beb54b6df4caa63fb19d889dd4ceff6/modules/sd_models.py#L131
This leads to a situation where people have two different workflows when dealing with the same files, automatic1111 vs diffusers library. It seems like people have a lot of confusion about what the diffuser does.
If this repo wanted to load in files, it seems like there would need to be a lot of boilerplates to reimplement logic that already exists. It doesn't seem like the diffuser library allows us to easily load files instead of repos that use the diffuser format.

I will soon add the ability to build the engine directly from a checkpoint file.


Support update conditional formatting on inserting/deleting columns/rows

PR Details

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report

Merging #1717 (221e01a) into master (134865d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##           master    #1717   +/-   ##
=======================================
  Coverage   99.03%   99.03%
=======================================
  Files          32       32
  Lines       28094    28123   +29
=======================================
+ Hits        27824    27853   +29
  Misses        179      179
  Partials       91       91

Flag
Coverage Δ

unittests
99.03% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

adjust.go
99.55% <100.00%> (+0.01%)
:arrow_up:

styles.go
100.00% <100.00%> (ø)


Create tool for comparing underlying database contents as basis for identifying app hash mismatches

Summary
When nodes encounter application hash mismatches, it is not easy to determine which data differs in order to trace the issue. A tool that offers a low-level key/value comparison of two database files (e.g. application.db) would allow the differences to be determined using only the raw database files. This would only be relevant for node data at relatively the same height, but it should work nicely in cases where there is a known issue and good/bad versions of the data at relatively similar heights.
Problem Definition
We currently do not have a method of determining the reason for an app hash mismatch since the database data is opaque unless it's being loaded from within the context of the cosmos application.
Proposal
Build a simple key/value comparison tool that notes differences between the low-level databases so they can be traced to find the cause of an app hash mismatch. The initial version will be targeted to leveldb backends (goleveldb and cleveldb), but the tm-db library will be used so that other backends may be supported as well.

For Admin Use

[x] Not duplicate issue
[x] Appropriate labels applied
[x] Appropriate contributors tagged
[x] Contributor assigned/self-assigned

Based on the results of this research it seems we will need to go a different direction.  There is simply too much change noise in the output


Refactor channel module

Channel module currently contains many functionalities : channel management, channel messaging, adapter interface definition and adapter implementations.
This module needs to be refactored into smaller ones, so that the different parts can be modified and developed independently. The refactoring can possibly done in the following steps :

[ ] Analyze the current structure and propose a structure for refactoring.
[ ] Discuss and review the proposal.
[ ] Implement the resulting structure.

Based on the current state of the channel package, I plan to refactor it by splitting it into 4 modules, organized in  the following directory structure:
channel
|--------primitives
|--------adapter
           |---------websocket

The functionalities of the individual packages will be :

channel : Manage the lifecycle of all offchain communication channels.
primitives : Provide definitions for offchain primitives, message packets.
adapter : Define a generic adapter and ReadWriteCloser interface.
websocket : Implement websocket adapter adhering to ReadWriteCloser interface. It can extend the generic adapter.

And the existing definitions in the channel module shall be split between packages as shown below:

Channel
Primitives
Adapter
Websocket

StartListenerNewChannelInstance
SessionIDMSCBaseStateMSCBaseStateSignedVPCStateVPCStateSignedVPCStateIDJsonMsgMsgIDchMsgPkt
AdapterTypeGenericChAdapterhandlerPipe
wsChannelwsStartListenerNewWsChannelwsReadHandlerwsWriteHandlerand otherwebsocket specificdefintions

This refactoring can be done in 5 individual changesets, each  constituting a Pull Request :

Move offchain primitives and message packet definitions to a separate package (primitives).

The generic adapter and all protocol specific adapter have to depend on Instance type to implement ID verification sequence. This dependency needs to be removed, by moving ID verification sequence from startListener (in adapter_factory.go) to NewSession (in manager.go).

The adapters also depend on primitives package to marshal and unmarshal message packets. To remove this dependency,

Change ReadWriteCloser interface and adapters to handle raw data (as byte array) instead of channel message packets.
Add Read, Write methods on Instance type, that will handle marshalling, unmarshalling of raw data.
The request, respond methods defined over Instance type should use the newly defined Read and Write methods.

Move generic adapter (in adapter_factory.go) into a separate package. Remaining functions (start listener, new channel) are moved to manger.go.

Move websocket adapter to a separate package.

At the end of these 5 changesets, the described structure of the channel package will be achieved.


Account activation issue

Why do we need to acitvate an account for this???
Anyhew it causes issues and prevents login. Maybe remove it?

This issue is a duplicate of closed issues #376 and #430.
If you're seeking a basic interface intended for a single user without login functionality, I recommend considering the Lite version, which can be found here.
Alternatively, if you desire additional features such as RAG and OpenAI API connection, creating a single account on the full version should suffice for you. It's important to note that authentication is local to your instance, there is no sharing of information with external parties. Currently, session timeouts are not implemented, so your account will remain logged in indefinitely until you logout.


Releases

Is there a reason you don't tag your release commits?

I tagged all recent versions of Traccar and Traccar Client.

Thank you.
On Thu, Jan 22, 2015 at 5:15 PM, Anton Tananaev<EMAIL_ADDRESS>wrote:

Closed #974 https://github.com/tananaev/traccar/issues/974.
—
Reply to this email directly or view it on GitHub
https://github.com/tananaev/traccar/issues/974#event-223349330.

--

jss


Connection problem with browser-live-reload=true on Firefox

It seems that when a form is submitted on a page with live-reload.js included then an HTTP connection may be closed while the HTTP request is being processed. As a result, the UniResponseHandler from quarkus-rest attempts to cancel the subscription of the returned Uni. In my reproducer, we have quarkus-hibernate-reactive-panache where a cancellation results in a transaction rollback (and sometimes even in inconsistent state - this is something I'd like to investigate further with someone from HR) and a broken HTTP response.
Works fine on Chromium (Version 127.0.6533.72).
I've tried to debug the code and it seems that on Firefox  (v128.0.3) the EventSource receives an onerror event after the form is submitted here:
https://github.com/quarkiverse/quarkus-web-bundler/blob/main/deployment/src/main/resources/web-bundler/live-reload.js#L51
Then the EventSource#close() is invoked and the connection is closed.
If I set a breakpoint to live-reload.js#L51 and keep it paused (i.e. avoid calling EventSource#close()) then the HTTP request is processed normally.
Reproducer
Here's the reproducer: https://github.com/mkouba/quarkus-quickstarts/tree/web-bundler-browser-live-reload
Steps to reproduce:

mvn quarkus:dev
open http://localhost:8080/creatures with Firefox
hit the "create" button

Current result
Sometimes the transaction is silently rolled back and you'll see nothing. Sometimes you'll see some data. The "delete" button usually does not work for the same reason - connection is closed, transaction rolled back, etc.
Expected result
You should see a table with 20000 records. The "delete" button should work without problems.
Solution
quarkus.web-bundler.browser-live-reload=false or do no use Firefox.

I've tried to debug this issue and what I've noticed is that, when cancellable.cancel() in UniResponseHandler caused MutinySessionImpl.Transaction#rollback to be called as expected.
Two things in particular aren't clear to me:

What's causing  the cancel of subscription in UniResponseHandler?
The rollback returns a uni, but cancellable.cancel() is not reactive. Should we close the request before the rollback has finished?

I suspect reactive doesn't log anything because the rollback is called directly and there are no exception and the return value is ignored. But I'm still debugging.

It seems that closing the event source during a transaction happening on another request is cancelling the unrelated transaction.

Ok, I've managed to create a new live-reload.js that uses the browser SSE re-connection manager and no need to cancel the connection manually anymore. I will publish a fix soon.
Still I think there is something undercover this issue.

I can confirm that https://github.com/quarkiverse/quarkus-web-bundler/pull/268 fixed the problem but I agree with Andy that there's probably another issue underneath. Unfortunately, I have no idea how to reproduce it now.

The rollback returns a uni, but cancellable.cancel() is not reactive. Should we close the request before the rollback has finished?

@DavideD good point. @cescoffier @geoand WDYT?

Web-Bundler is doing this:
 private void closeConnection(Connection connection) {
        connections.remove(connection);
        if (!connection.closed.getAndSet(true)) {
            connection.ctx().vertx().cancelTimer(connection.timerId());
            if (!connection.ctx().response().ended()) {
                connection.ctx().response().end();
            }
        }
    }
    ```

    I've debug it's only closing the SSE connection, not any other connection..


[Question] No module named 'ae' when running train.py

❓ Question
I want to use SAC in CarRacing-V2, and when I typing "python train.py --algo sac --env CarRacing-v2", I got the Error be like:

how can I deal it?
Checklist

[X] I have checked that there is no similar issue in the repo
[X] I have read the SB3 documentation
[X] I have read the RL Zoo documentation
[X] If code there is, it is minimal and working
[X] If code there is, it is formatted using the markdown code blocks for both code and stack traces.

Hello,
if you look at the config:https://github.com/DLR-RM/rl-baselines3-zoo/blob/382dcabbd9815cb9557503a7caa0c54a562fa7fb/hyperparams/sac.yml#L168
It comes from this video serie: https://m.youtube.com/playlist?list=PL42jkf1t1F7dFXE7f0VTeFLhW0ZEQ4XJV


Fix integration test of parallel prehandling

What problem does this PR solve?
Issue Number: close #8246
Problem Summary:
The problem is when a retryable exception is thrown in a thread, it will notify other thread to abort. As a result, other thread will abort. If the abort result is captured before the retryable result by head split, it will abort directly without retry.
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

which unstable test case is caused by this issue?

The fix of https://github.com/pingcap/tiflash/pull/8199#discussion_r1372552881 is not pushed?

/run-all-tests

/run-all-tests


Refactor comments

@mradkov I still can't ask @Liubov-crypto to review a PR :(

My comments are duplicated as well as tipping:

@CedrikNikita that issue is reproducible also on develop branch. This is a wallet bug fixed in https://github.com/aeternity/superhero-wallet/pull/571
@Liubov-crypto I have accidentally dropped the code that was removing the duplicate comments, I have reverted it back

The comment appeared twice with a delay of several seconds (9 sec).

The comment thread looks different here: https://feature-refactor-comments.aggregator.z52da5wt.xyz/tip/1375_v1/comment/756
and on development:
https://develop.aggregator.z52da5wt.xyz/tip/1375_v1/comment/756

As far as I know currently we don't have such logic implemented that would allow us to display "Most Relevant" comments. I guess it's something to do with the reccomendation engine. So our logic is purely related to the moment of posting a comment or reply.
Most recent comments (also a new comment by user) should be displayed on topafter the specific thread. It doesn't make sense to scroll to bottom to see the most recent comments which is the case now. Also the label says "Latest comments"
which clearly indicates that newest should be seen first.
Additionally we can add an icon to reverse the display order but latest first should be the default behaviour.

The comment appeared twice with a delay of several seconds

I can't reproduce this

The comment thread looks different here

I have fixed over filtering that I have added before
Regarding sorting: I have implemented descending sorting of comment list and ascending sorting of comments replies (because the new comment form placed at the end of them).

I checked, comments are not duplicated. When I comment on a comment in a thread, that comment is not displayed (click on the arrow to view it). I don't know how critical it is...just a report:

When I comment on a comment in a thread, that comment is not displayed

This works the same as when commenting on a tip
I think this PR is ready to get merged 🚀

@CedrikNikita please, aprove this one to be able to merge it


[ADD] add docs for Auto-GPT integration

added DOCS  that will fit the modifications for
https://github.com/Significant-Gravitas/Auto-GPT/pull/2594
from
https://github.com/DGdev91/Auto-GPT
works on my side.

does this actually work? i dont see the base_url or embed_dim environment variables

did you mean to say use the fork from this PR https://github.com/Significant-Gravitas/Auto-GPT/pull/2594? if so then please update the readme and i'll go ahead and merge

did you mean to say use the fork from this PR Significant-Gravitas/Auto-GPT#2594? if so then please update the readme and i'll go ahead and merge

Yes you are right its not clear


Add advanced/path-operation-advanced-configuration.md Japanese ver.

This PR translates the advanced/path-operation-advanced-configuration.md to Japanese.
Please review advanced/path-operation-advanced-configuration.md and ja/mkdocs.yml.

@tokusumi Thank you for your nice suggestions ! I've fix everything, so please check again.

I resolved all conflicts (All were mkdocs nav order)
https://github.com/tiangolo/fastapi/pull/2124/commits/e381456178f0a8bf3f18beb8fab6b329ce2b598d

@tiangolo, This PR is Japanese tlanslation. could you add some labels?

@delhi09 Thank you for your nice suggestions ! I've fix everything, so please check again.

@delhi09 Thank you for your nice suggestions ! I've fix everything, so please check again.

Great! Thanks @Attsun1031 ! :bow: :rocket:
And thanks @tokusumi and @delhi09 for the reviews :mag: :cake:

Great! Thanks @Attsun1031 ! :bow: :rocket:
And thanks @tokusumi and @delhi09 for the reviews :mag: :cake:


allow for nesting of commands

doing something like: app list tables and app list counts is difficult currently.
the current workaround is app list --tables etc, but this is not as intuitive.
table and counts are really commands, not flags.
possible code:
command :list do |c|
  c.command :tables do |table_command|
     table_command.action do
        DB.show_tables
     end
  end
end

Yeah, this is planned for GLI2, and is discussed in issue #22.  I  think the syntax you propose is a good one


AddAzureADBearer case might be wrong(?)

AddAzureADBearer appears to be : AddAzureAdBearer as far as I can tell?  When I attempt to use that line in VS2022, it prompts that the method should be AddAzureADBearer :

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

ID: 906f1086-776e-9abe-b6cc-bf93e84dc238
Version Independent ID: 50b27c9e-8f8a-3425-a1bb-b5d30e9d95e4
Content: Configure protected web API apps - Microsoft Entra
Content Source: articles/active-directory/develop/scenario-protected-web-api-app-configuration.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@MikeHingley22
Thanks for your feedback! We will investigate and update as appropriate.

Thanks the feedback.
This is a question of version. The article describes how to move from the old generation AddAzureADBearer (https://docs.microsoft.com/en-us/dotnet/api/microsoft.aspnetcore.authentication.azureadauthenticationbuilderextensions.addazureadbearer?view=aspnetcore-6.0) to Microsoft.Identity.Web.


Docker environment variable for disabling file share

It would be great if you could disable the file share via a Docker variable and let users only send text. Or is there already a suitable Docker variable here? I haven't found anything suitable in the source code. Great project anyways!

There is not. However it's only a frontend thing. There is no server validation, for the server it's just text anyways. Does it make sense then?


Merge 'master' into 'beta'

Description
Merge 'master' into 'beta'
Breaking Changes
NA
JIRA Link
NA
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 20.1.0-alpha.1 :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 20.1.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fielder's Choice event showing runners from 1st and 3rd scoring

Game ID 0078d5d5-dc02-4025-821a-21fb7cc20556
"Kennedy Loser batting for the Crabs.",
"Ball. 1-0",
"Ball. 2-0",
"Kennedy Loser reaches on fielder's choice. Parker Parra out at second base. Valentine Games scores"
Event ends up with this:
"base_runners": [
		{
			"runner_id": "84a2b5f6-4955-4007-9299-3d35ae7135d3",
			"responsible_pitcher_id": "20fd71e7-4fa0-4132-9f47-06a314ed539a",
			"base_before_play": 0,
			"base_after_play": 1,
			"was_base_stolen": false,
			"was_caught_stealing": false,
			"was_picked_off": false
		},
		{
			"runner_id": "c675fcdf-6117-49a6-ac32-99a89a3a88aa",
			"responsible_pitcher_id": "20fd71e7-4fa0-4132-9f47-06a314ed539a",
			"base_before_play": 3,
			"base_after_play": 4,
			"was_base_stolen": false,
			"was_caught_stealing": false,
			"was_picked_off": false
		},
		{
			"runner_id": "7dcf6902-632f-48c5-936a-7cf88802b93a",
			"responsible_pitcher_id": "20fd71e7-4fa0-4132-9f47-06a314ed539a",
			"base_before_play": 1,
			"base_after_play": 4,
			"was_base_stolen": false,
			"was_caught_stealing": false,
			"was_picked_off": false
		}

Fixed in 1.3.0