C2ME Fabric crashes on dedicated server because Connector tries to load some client-only JiJ mod.

Mod Name
C2ME Fabric
Mod Homepage
https://modrinth.com/mod/c2me-fabric
Minecraft version
1.20.1
Describe the bug
It crashes.
Steps to reproduce

Install Minecraft Server, Connector and C2ME fabric mod.
Launch it.

Logs
https://gist.github.com/heipiao233/f337fad5c024525399f335c08111a9af
Additional context
This is different from prev issues.

duplicate of https://github.com/Sinytra/Connector/issues/88

@Su5eD This is different from #88.

Yeah but the first problem is it doesn't work without disable some config so he will fix also this problem at the same time he makes the compat for c2me


MATPOWER 8 runpf prints warnings

It is related to https://github.com/yasirroni/matpower-pip/issues/17, where warning is printed lots of times due to oct2py conversions.

Solved in https://github.com/blink1073/oct2py/pull/327 and https://github.com/yasirroni/matpower-pip/commit/c3cde7a2de6b72c9cddf11169ba3d3563a87d155


gles2 surface should destroy previous texture resources when replacing them

So i guess this is fixed now? The previous texture is not destroyed but reused instead.

No, gles2_texture_upload_{pixels,shm} still calls gles2_texture_gen_texture to gen new textures without releasing or reusing the old ones.

the function simply returns if the texture was already created... gles2_texture_assure_texture or something like this may be a better name


Fix: returning invalidParams instead of unknownOption error code in ledger_entry (#4303)

High Level Overview of Change

Fix for #4303; Replacing unknownOption error with invalidParams in ledger_entry
Type of Change

[ x] Bug fix (non-breaking change which fixes an issue)
[ x] Tests (Updated the existing tests)

Before / After
Test Input Payload:
{    "method":"ledger_entry",    "params":[       {          "ledger_index":"validated"       }    ] }
Before:
{    "result":{       "error":"unknownOption",       "ledger_hash":"CA38067ED8FCDCFDE8CAB00A7F83BD90FE0B75D4906D42CCA13EE02B15EC1D10",       "ledger_index":29063797,       "request":{          "command":"ledger_entry",          "ledger_index":"validated"       },       "status":"error",       "validated":true    } }
After:
{    "result":{       "error":"invalidParams",       "ledger_hash":"FC5DCCCEACDEEB69F3E0E812A3BE737A8546C8B3BB42BFB1BE8B51CE628E8354",       "ledger_index":29059824,       "request":{          "command":"ledger_entry",          "ledger_index":"validated"       },       "status":"error",       "validated":true    } }

wouldn't this be a small breaking change?

How is it that so ? I have updated the existing unit test as well. I was told that we do not want unknownOption at all but the API currently returns it. I might be missing something, do you think I should be updating something else also to make it a non-breaking change if it is a breaking change ?

Take it with a grain of salt, but it seems unknownOption have existed for quite some while for this API. I think this change would break the error handling for existing applications if the error message is suddenly changed from unknownOption to invalidParams

But I think it would be okay if invalidParams is the intended behavior. I would check with @ckniffen if it's affecting anything.

Do you think we should be deprecating the unknownOption error code ?

Thanks @intelliot ! Here are the results of the changes made:
Input:
{ "method":"ledger_entry", "params":[ { "api_version": 2, "ledger_index":"validated" } ] }
Output:
{"result":{"error":"invalidParams","ledger_hash":"2532DD58AF4FDC24EAD3F713280C91781C9A0C9CA1DB51EF9BA8946B1D3C1B4E","ledger_index":29343470,"request":{"api_version":2,"command":"ledger_entry","ledger_index":"validated"},"status":"error","validated":true}}
Input:
{ "method":"ledger_entry", "params":[ { "ledger_index":"validated" } ] }
Output:
{"result":{"error":"unknownOption","ledger_hash":"BB17A4E96BB0D8411B36AA003AF6E25DC71B3B5E29E16C2FC2F2F88A8C17251B","ledger_index":29343475,"request":{"command":"ledger_entry","ledger_index":"validated"},"status":"error","validated":true}}

Currently, a bunch of account_info unit tests fail (rippled/src/test/rpc/AccountInfo_test.cpp) because of setting ApiMaximumSupportedVersion to 2. It seems there exists version specific tests in that and by default all tests are run on the max supported version. I am wondering if I should start a new issue regarding this or do the changes here ? I think starting a new issue might make more sense.

Yes, a new issue (or better - a PR) makes sense.

Suggested commit message:
APIv2(ledger_entry): return "invalidParams" when fields missing (#4552)

Improve error handling for ledger_entry by returning an "invalidParams"
error when one or more request fields are specified incorrectly, or one
or more required fields are missing.

For example, if none of of the following fields is provided, then the
API should return an invalidParams error:
* index, account_root, directory, offer, ripple_state, check, escrow,
  payment_channel, deposit_preauth, ticket

Prior to this commit, the API returned an "unknownOption" error instead.
Since the error was actually due to invalid parameters, rather than
unknown options, this error was misleading.

Since this is an API breaking change, the "invalidParams" error is only
returned for requests using api_version: 2 and above. To maintain
backward compatibility, the "unknownOption" error is still returned for
api_version: 1.

Related: #4573

Fix #4303

@arihantkothari can you confirm the commit message above looks correct to you?

@arihantkothari can you confirm the commit message above looks correct to you?

Hi @intelliot , Sorry I was offline last few days. Yes, this looks good.  Thanks,


removed OpenMP ugly fix for Mojave

@henry0312 I think 3.16.2 is quite stable and we can require 3.16 for macOS with AppleClang as we discussed earlier.

PR to Homebrew has been proposed: https://github.com/Homebrew/homebrew-core/pull/48822.

Hmmm, everything works fine on our CI services, but fails during building Homebrew formula (even for macOS 10.13, which should be unaffected).
@henry0312 Would you mind take a look: https://jenkins.brew.sh/job/Homebrew Core Pull Requests/54922 ?

Sorry, I don't know about Homebrew.

@henry0312 OK, no problem! I think it's OK to leave the formula as is because it doesn't affect users' experience.


Heredoc formatting

Expected Behavior
I would expect this code to be properly formatted:
val foo =
    """
        ${
            if (1 == 1)
                1
            else
                2
        }
    """.trimIndent()

Observed Behavior
ktlint prefers following:
val foo =
    """
        ${
    if (1 == 1)
        1
    else
        2
    }
    """.trimIndent()

Steps to Reproduce
Just let ktlint format the code above. Also observed in Idea with ktlint plugin:

Your Environment
gradle:
id("org.jlleitschuh.gradle.ktlint") version "10.2.1"

ktlint plugin:
0.8.2

* Version of ktlint used: 0.43.2
* Version of Gradle used (if applicable): 3.7.1
* Operating System and version: Any

The code is formatted in same way as the default IntellI IDEA Formatter:
val foo =
    """
        ${
        if (1 == 1)
            1
        else
            2
    }
    """.trimIndent()

If we would use another formatting style like the one you expected (which undoubtly is better) than we would produce code that would be rejected/reformatted by the IntelliJ IDEA formatter. For now, we still follow the rule that code produced by ktlint should not conflict with the IntelliJ IDEA formatter even in case it results in ugly code.
The problem in this case is cause by the indent before opening of the Heredoc. And there is a good reason for that. Suppose that we would exagarrate that indent to:
val foo =
    """
        some looooooooooooooooooooon text ${
        if (1 == 1)
            1
        else
            2
    }
    """.trimIndent()

There is no way that we can format this properly in a consistent way with what you expect.


web-bluetooth: specify blacklisted UUIDs

This CL simply specificies which characterististics/services are now blacklisted from read/write actions.
https://codereview.chromium.org/1649253002
R=@jeffposnick

Hey @beaufortfrancois—It's hard to tell what changed as part of the blacklisting, since it looks like there's a diff for every line in all those files due to adding content to the <option> elements.
So what's been removed? Is it the sort of thing that needs some explanatory comment in the sample or code so that someone doesn't get confused why a UUID that would otherwise work doesn't?

I didn't add any characteristics/services there.
I "simply" added a label to each <option> and add "(blacklisted)" for 3 of them.
That's all there ;)

Sorry, I missed that from scanning through the diff :smile:

Thx!


Update messages.go

Hey! Thanks for the PR, you'll need to register the command in the app.go file so it get's registered on boot-up.


overlay suggestion

it would be really cool if there was an overlay showing the result of the rolled dice
and also an overlay of mana points for systems that use mana

I'll check the feasibility once I'm done making modules compatible with 0.8.
though mana points should already be possible with the custom settings?

Closing because of inactivity


Consider removing brush transitions from Control.

The color change of clicking Button cannot be observed because of brush transition.

https://github.com/carina-studio/AppSuiteBase/commit/49c758ac66aefe56d69b5b59de4b2f0cde637b2f


Scala Hadoop Shred: apply truncation logic to atomic-events TSV

The logic currently is in EnrichmentManager.

Logic will be moved out of Common Enrich as it is storage-target specific (actually Postgres-specific - Redshift COPY supports TRUNCATECOLUMNS

For posterity, this is how I generated the full list of maximum field lengths:
cat atomic-def.sql | grep  '^\s*"' | awk '{print $2}' | awk -F' |\(|\)' '{if ($0 ~ /char/) print $2","; else print "null,"}'

Having access to an AST for the table would make this easier to do programmatically...


Kong mixing the traffic for service when IP and Port reused.

Hi Team,
There seem to be an issue with kong, how to work on this kind of scenario, is it a bug?
Problem: We have 2 services, ServiceA and ServiceB both are deployed in AWS ECS Container. And both of them have /status as there healthcheck point.
So Lets assume
ServiceA Deployed on IP-1:PortA
ServiceB Deployed on IP-1:PortB
both of them are successfully registered to kong gateway.
Now suppose ServiceA and ServiceB got restarted multiple times being in same ECS cluster Now serviceB got respaned With IP-1:PortA as that was free port and got reused.

Now when we access
kong.apigwatey.internal.com/serviceA → This traffic is served by both A and B service.
Reason being that Targets are not cleaned by kong and they are actively checked constantly if they turn healthy.
How to deal in this kind of scenario.

@puneet739, how would you solve the issue?

We don't have a solution right now.. this is something common, thought there will be a way available with kong team regarding this.

@puneet739, what do you think about having SRV records with very short TTLs for those services? For example:
servicea.apigwatey.internal.com -> IP-1:PortA
serviceb.apigwatey.internal.com -> IP-1:PortB

Kong will have to resolve those records frequently, and if the service is out, you should get a name resolve error, thus the target will be marked as unhealthy.

@locao
The problem still remains the same.
Inital config
servicea.apigwatey.internal.com -> IP-1:PortA
serviceb.apigwatey.internal.com -> IP-1:PortB
Now there were multiple restarts. Now new config is
servicea.apigwatey.internal.com -> IP-1:PortX, IP-1:PortA(previously added SRV record , which was marked unhealhty now healhty again because of serviceB)
serviceb.apigwatey.internal.com -> IP-1:PortA(port got reissued to service A), IP-1:PortB(Previous record, which is not unhealthy)
Thus in this scenario.
servicea.apigwatey.internal.com --> This will roundrobin to both ServiceA and ServiceB.
Am i missing something here.

What i think can be a solution is to purge the unhealthy target on some TTL, that way once the instance got unhealthy and comes back after some time, a new entry is created and it wont mix with other previous records.
is there anything like this available with kong?

Also one pointer, there isnt any specific service registry, the services are in different pods, some in AWS, some in AZURE. But they have and SDK which is responsible for sending requests. Via that we can do whatever we want, but once it will close there wont be a way to send a delete.
One approach will be to run a cron for cleanup based on tags of last updated time(Custom created) but that wont be an ideal solution.

Hi @locao @bungle  any update on this

My suggestion was to update the SRV record every time the service changes its address, so Kong always resolve it and get the current IP/port pair. Other than that, currently Kong doesn't support anything like you need.

Dear contributor,
We're closing this issue as there hasn't been any update to it for a long time. If the issue is still relevant in the latest version, please feel free to reopen it. We're more than happy to revisit it again.
Please have a look at our pledge to the community for more information.
Sincerely,
Kong Gateway Team


Changing Color for certain text in Console.WriteLine?

So basically I want to do something like this:

But in Console.WriteLine since it can mix up with other output when I use this code:
`
Console.Write(@"[", Color.Cyan);
Console.Write(0.5, Color.Yellow);
Console.Write(@"] ", Color.Cyan);
Console.Write(@"Code: ", Color.Orange);
Console.Write("test");
Console.Write(" [", Color.Cyan);
Console.Write(status, Color.Lime);
Console.WriteLine("]", Color.Cyan);
`

Hi @zix3r,
Thanks for the question.  You should be able to do this using a StyleSheet  and WriteLineStyled:
StyleSheet styleSheet = new StyleSheet(Color.White);
styleSheet.AddStyle("[\\[\\]test]", Color.Cyan);
styleSheet.AddStyle("0\\.5", Color.Yellow);
styleSheet.AddStyle("Code:", Color.Orange);

Console.WriteLineStyled("[0.5] Code: test []", styleSheet);

Note that the string argument to StyleSheet.AddStyle is a regular expression.
There's also another overload for AddStyle that allows you to pass a function that determines how the coloring happens -- see Examples.cs for an example: https://github.com/tomakita/Colorful.Console/blob/master/src/ExampleConsoleApplication/Examples.cs#L184
I hope that helps!

Thanks a lot! Your help is appreciated!


Added sfdx authURL authentication to smoke test pipeline

Smoke tests have been repeatedly failing when pull requests are raised.

I looked into azure devOps to inspect the error and found this.

Updated the authentication to AuthURL

authfile JSON has been uploaded to library.

Tested Pipeline and here are screenshots


Temp Key

Hi,
I want to review this tool for my channel https://www.youtube.com/@fahdmirza , but I don't have the infermatic key, is there a way to get the temp key to do a demo? Thanks.

Sorry I can't, but I might look into how to implement some sort of support to kobold API, so that any user can use the app with a local model running on koboldcpp, but seem like something weird to do, if the user has access to koboldcpp, it had most of my features and more, my app was made with focus on fixing the fact that no other notebook app was working with infermatic API.
In theory any other API services that uses an API address + api_key should work with my app if you change those things on my code, (change the URL and the key inside the config.json), but I can't guarantee, also my app is made for text completion, so if an API works only with chat completion, even if worked, you would get subpar results.
What other API services do you have? Gemini? I believe Gemini offers a free API key, I could look onto how to add gemini support if you wanted that.

The only reason my app won't support kobold as of right now is because Kobold API has no key field, only address, I would have to change that in my code to work differently if Kobold was being used.

Sure Gemini or openai should be good. thanks.

In theory if you have access to openai API key, it should work out of the box if you change the url address to the OpenAI one and put all address to 0.0 except the ones that OpenAI actually support, but not sure if it will work actually, maybe openai will give error to min_p even if sent at 0.0, I should look at what parameters there services support and make a new setting for my app that disables settings temporarily to avoid errors, like allowing to disable the ones that Gemini or OpenAI don't support, if not because of min_p, I think my app would already work with OpenAI

OpenAI might return an error as it does not support Min p and my app sends min_p always.

If I were to focus on Gemini or OpenAI, which one do you think would be better for you? I just looked into Gemini website and seems easier to implement it, their website is more friendly, and Gemini offers API for free.

Sure Gemini is fine. Thanks.

I gonna test it out with a free key I got from gemini, so this way I can be sure it should work on yours once its done.

Sure Gemini is fine. Thanks.

Are you willingly to wait? This is my first big project I made, I want to make sure this integration is well done, that is not something made poorly, I will probaly add a USE_GEMINI to my config json the same way I added a toggle for NovelAI voice, it will come disabled by default, anyone that wants to use Gemini, should only change that from false to true, like this:
{
    "INFERMATIC_API_KEY": "your_api_key_here",
    "USE_TTS": false,
    "NOVELAI_API_KEY": "your_novelai_key_here",
    "USE_GEMINI": true
}

I will make so if USE_GEMINI is true, the infermatic key becomes the Gemini Key, and it will automaticly use the url from gemini instead of infermatic

Sure mate, take your time. No rush. Thanks.

sorry, I could not find a way to use Gemini without using their library, also even if I had managed to, while reading their docs I discovered their API does not support text completion, only chat completion, so it would be incompatible with my app anyway, my best bet now would be to focus on adding support to using the API for a local model through koboldcpp, this type of support would also open doors to running a model on Google collab for free and use that api and address on my app, can I close this issue, or will you want me to try the local model option?
PS: this is already going out of the original scope of this project, that is to be an app mainly focused on infermatic users.

All good mate. Up to you if you want to close the issue. Great effort though, thanks.


Not splitting gcode at new line characters

When trying to run Gcode exported from Fusion 360 (and possibly other CAM software) the program seems to send Gcode to the machine based on when a new G command is issued, rather than by newline characters
The splitting not working correctly results in either GRBL error 25 or 11 mostly since it is being sent many lines at once
Here is a sample of the Gcode that is not working for me, however there doesn't seem to be anything wrong with it so i'm unsure of how to fix the issue.
(1001)
G90 G94
G17
G21
G28 G91 Z0
G90
(2D Pocket1)
G54
G0 X104.324 Y81.735
Z15.2
Z5.2
G1 Z3.3 F500
X104.32 Y81.763 Z3.118 F333.3
X104.307 Y81.844 Z2.954
X104.283 Y81.969 Z2.822
X104.243 Y82.125 Z2.734
X104.189 Y82.297 Z2.7
G3 X98.811 Y80.407 Z2.387 I-2.689 J-0.945
X104.189 Y82.297 Z2.075 I2.689 J0.945
X98.811 Y80.407 Z1.762 I-2.689 J-0.945
X104.189 Y82.297 Z1.449 I2.689 J0.945
X98.811 Y80.407 Z1.137 I-2.689 J-0.945
X104.189 Y82.297 Z0.824 I2.689 J0.945
X98.811 Y80.407 Z0.511 I-2.689 J-0.945
X104.189 Y82.297 Z0.199 I2.689 J0.945
X98.811 Y80.407 Z-0.114 I-2.689 J-0.945
X104.189 Y82.297 Z-0.427 I2.689 J0.945
X98.811 Y80.407 Z-0.739 I-2.689 J-0.945
X104.189 Y82.297 Z-1.052 I2.689 J0.945
X98.811 Y80.407 Z-1.365 I-2.689 J-0.945
X104.189 Y82.297 Z-1.677 I2.689 J0.945
X101.5 Y84.202 Z-1.8 I-2.689 J-0.945

So I've looked through the code and you are right that many of your commands are being sent as one large message. In general there should be a G command on each line, however I have seen before that some gcode generators will omit the command if they intend to only update the parameters of the previous command.
This bug smells very similar tohttps://github.com/sir-buckyball/chrome-gcode-sender/issues/38, which used Fusion360 to generate files like yours (with the actual command being omitted). In that case, only a single carriage-return character was used and the newline was omitted. There is handling for that case, but not if you have a carriage-return+newline sequence.
Could you attach the exact file you are trying to process? In particular I am looking at the newline sequences used (which don't copy/paste very well).
I've also done some searching through the closed issues to see if there was a reason why newlines are being ignored, however nothing popped up. I think this may have been purposeful to make sure that the rate limiting worked properly (eg, knowing when to wait for an OK versus not), however I cannot find proof of that in the code.

Hey there, a couple of hours after posting that issue this morning I have discovered that fusion 360 can output gcode with a G command on every line and so I have infact used that to cut several things today and I can confirm that it is working.
I will attach the original file as a .nc anyway since it would be better if the default fusion post processor worked.
Link to gcode file: File
The issue can be circumvented (for anyone else who comes across this issue in the future) by editing the post processor config.
Search for gMotionModal and add {force:true} so that it matches as below:
var gMotionModal = createModal({force:true}, gFormat); // modal group 1 // G0-G3, ...
I have to say that once I got the program working, the gcode sender works brilliantly and even seems to continue when the screen turns to standby which is great. Good Job!


syncer: support TLS for region syncer

What problem does this PR solve?
Currently, the region syncer establish the connection by using WithInsecure() which doesn't use TLS.
What is changed and how it works?
This PR adds TLS support for the region syncer.
Check List
Tests

Manual test

Related changes

Need to cherry-pick to the release branch
Need to be included in the release notes

Codecov Report

Merging #1728 into master will decrease coverage by 0.23%.
The diff coverage is 30.76%.

@@            Coverage Diff             @@
##           master    #1728      +/-   ##
==========================================
- Coverage   76.75%   76.51%   -0.24%
==========================================
  Files         158      158
  Lines       15580    15600      +20
==========================================
- Hits        11959    11937      -22
- Misses       2602     2638      +36
- Partials     1019     1025       +6

Impacted Files
Coverage Δ

server/region_syncer/client.go
66.32% <10%> (-14.69%)
:arrow_down:

server/region_syncer/server.go
82.05% <100%> (+0.15%)
:arrow_up:

pkg/testutil/operator_check.go
83.33% <0%> (-16.67%)
:arrow_down:

server/kv/etcd_kv.go
70.12% <0%> (-9.1%)
:arrow_down:

server/tso/tso.go
77.06% <0%> (-1.84%)
:arrow_down:

client/client.go
65.77% <0%> (-1.24%)
:arrow_down:

server/grpc_service.go
57.48% <0%> (-0.44%)
:arrow_down:

server/cluster.go
82.84% <0%> (-0.26%)
:arrow_down:

server/schedule/operator/operator.go
86% <0%> (+0.36%)
:arrow_up:

server/handler.go
50.13% <0%> (+0.52%)
:arrow_up:

Continue to review full report at Codecov.

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


lora with more than one key-value pair

Currently, when finetuning with the lora example, only the key 'text' is recognized an the rest is ignored. I am finetuning a model for which I would like to also provide some context for the model to also take as input. Is there any plans to implement this by modifying the code a bit?

We could add this but it isn't a huge priority at the moment for a couple reasons:

It's not clear it matters that much to not take loss from the context. (But I'd be interested to hear if you notice a big difference there).
There is a balance between keeping the example simple (and useful for instruction) and fully featured and I think it's at a good place there right now.

I encourage you to extend it to support for that feature in your own fork. And if you think it makes sense to contribute back to here then we're happy to take a look at it!


Icon Request: Stackshare

Icon Request

Icon name: stackshare
Tags of this icon: stackshare
Use case: Stackshare has risen in popularity as a service to document your app's stack
Screenshots of similar icons:

Large companies all the way from Google to Uber use this service and it would be nice to see an icon for it, if the requested icons are too many I am able to help either by contributing via Pull Requests or monetarily whichever you prefer.

I will make a design guide for RemixIcon so that designers can build this product together with us.

Here are your icons.
https://github.com/Remix-Design/RemixIcon/blob/master/icons/Development/share-stack-line.svg
https://github.com/Remix-Design/RemixIcon/blob/master/icons/Development/share-stack-fill.svg
You can also find them from our website v2.1.0

This is great, however one small issue, the name is labelled in reverse it should be named stackshare or stack-share if you would like

(⊙o⊙)…
I'll fix it in the next version.


BUG: GH12824 fixed apply() returns different result depending on whet…

[x] closes #12824
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

In _wrap_applied_output for NDFrameGroupBy, if the first result of apply is None, find the first non-None result to determine behavior

can you rebase / update

Rebased and made the changes you suggested but there was a new error with the test. It is introduced by #12743 which now throws an error in _concat_objects when any of the values are None (see below). The new PR makes a small change to deal with this, can you check if ok?
======================================================================
ERROR: test_groupby_apply_none_first (pandas.tests.test_groupby.TestGroupBy)
----------------------------------------------------------------------
Traceback (most recent call last):
  File ".../pandas/tests/test_groupby.py", line 6292, in test_groupby_apply_none_first
    result = test_df.groupby('groups').apply(test_func)
  File ".../pandas/core/groupby.py", line 651, in apply
    return self._python_apply_general(f)
  File ".../pandas/core/groupby.py", line 660, in _python_apply_general
    not_indexed_same=mutated or self.mutated)
  File ".../pandas/core/groupby.py", line 3241, in _wrap_applied_output
    not_indexed_same=not_indexed_same)
  File ".../pandas/core/groupby.py", line 824, in _concat_objects
    values = reset_identity(values)
  File ".../pandas/core/groupby.py", line 809, in reset_identity
    ax = v._get_axis(self.axis)
AttributeError: 'NoneType' object has no attribute '_get_axis'

----------------------------------------------------------------------

[Current coverage][cc-pull] is 84.14%

Merging [#12977][cc-pull] into [master][cc-base-branch] will increase coverage by +<.01%

@@             master     #12977   diff @@
==========================================
  Files           137        137
  Lines         50287      50294     +7
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42311      42319     +8
+ Misses         7976       7975     -1
  Partials          0          0

File ...das/tseries/index.py (not in diff) was modified. more

Misses -1
Partials 0
Hits +1

Powered by Codecov. Last updated by [af7bdd3...e4fa8d2][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/af7bdd3883c8d61e9d9388d3aa699930eee7fff8...e4fa8d2a42df10908aed735b201ad40bed3acad9
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/12977?src=pr

can you squash

thanks @adneu


Import statements not generated

Hi,
While experimenting with capnproto-java we faced the fact that it actually does not generate the import statements in java classes. This is not a problem when dealing with a trivial project having a single package, but in a multi-package java project, the generated classes won't compile.
Example:
Suppose those schema files:

'com/example/foo/foo.capnp' file:

    @0xad8d23def0a030d1;
    using Java = import "/java.capnp";

    $Java.package("com.example.foo");
    $Java.outerClassname("FooCapnp");

    struct FooStruct
    {
        code @0 :Text;
    }

and 'com/example/bar/bar.capnp' file:

    @0x92a6d746aea85bb2;
    using Java = import "/java.capnp";

    using import "/com/example/foo/foo.capnp".FooStruct;

    $Java.package("com.example.bar");
    $Java.outerClassname("BarCapnp");

    struct BarStruct
    {
        foo @0 :FooStruct;
    }

Then BarCapnp.java do not contain the necessary import statement:

    import com.example.foo.FooCapnp;

For your information, we are implementing a capnp-maven-plugin which provide compilation at build phase. It will be available shortly in github.
Thanks!

Thanks for the report!

For your information, we are implementing a capnp-maven-plugin which provide compilation at build phase. It will be available shortly in github.

I'll be excited to see this.

I've addressed the problem with https://github.com/dwrensha/capnproto-java/commit/037bebfb9ca9eafa3383cbccf1e5ccd6aacc1ab4. Please let me know whether this fixes things for you.

Yes, it worked! Thanks a lot!
And thank you also for your quick response!

Because our plugin needs to contain the capnp compiler and capnpc-java (statically compiled for linux, mac and windows), is there any particularities related to licensing that we should be aware?
We are currently using the MIT License, as you do.

I don't know of any licensing particularities with that.

Hi David,
A word to let you know that our capnp-maven-plugin is now available as a SNAPSHOT on nexus.
Marc


Peekable is missing inner Stream accessors

Most concrete stream types (Fuse for example) provide get_ref, get_mut, and into_inner functions to let you access the inner Stream. Peekable does not, which means that there's no good way to use something that's Peekable<Fuse<_>>, as you cannot access the Fuse<...> methods from the Peekable outer layer.
The implementation will be slightly different to other concrete stream types, as Peekable adds its own Fuse layer in the middle, which you'll probably want to skip as an implementation detail.

I think when these were originally added I just wasn't 100% sure what to do wrt the buffered item, but I think it's fine to add the standard accessors to the stream regardless.


[Feature] Enable a default sort order for Tables

Summary
provide ability to automatically sort data from tables, dropdown, ...
Motivation
tedious process to use js function to do so every single time
Additional Context
usually it has to look like this:
{{
    (function(){

return  (
source_data.data.map((member)=>{ return {label: member.name,value: member.gid}})
).sort((a, b) => {
    const aName = a.label;
    const bName = b.label;
    return aName.localeCompare(bName);
  });
  })()
}}

@jmcouffin clicking on the column of the table will automatically sort it. Is this not working for you?

not exactly as expected:
I want the data to be sorted beforehand, while fetched by the ressource / at the source level maybe.
Right now, fetching data from a rest api, it comes out by creation date basically / id. so if I want to create a dropdown with this data, I have to file in the code quoted above to make it do the thing.
is a 'sort' option possible in the dropdown options (and all other widgets)?

@jmcouffin Ah I see, you're looking for a default sort order property in the table. Thanks for clarifying that!
If you have a large dataset, I still recommend you configure your API to do this as it can be inefficient in memory :)

we are small 😃
usually 4 pages tops.
I understand that in much bigger datasets I would better go through a middle man database;

but my usecase is mostly for dropdown. Right now my workaround is to assemble 3 queries results in one in a table and then use the table data in the dropdown.

@jmcouffin I'm not sure if dropdowns are inherently meant to do sorting by default. Would a reusable JS sort function that could be run on top of the API be a better fit here?

on top of the API? how/where? in the ressource or on the dropdown widget?

@jmcouffin in the API itself, we're planning to introduce an option to add a JS function so that users can transform the response of their API in a single place and then just use that in the widget.

that would make sense.
a bit like in retool in the transform section where you can add your code to transform the data (but not the query itself)

Yes! This is currently being worked on as part of #1751

Closing this in favour of #1751


Changed Navbar Hamburger design for smaller devices

Here I have changed the Navbar hamburger design from

to

Resolves #620

Deploy preview is failing. Please make sure that the code changes are properly indented

@smaranjitghose  I have made the necessary changes. Kindly Check

Can't see the changes create a new pull request from the latest code and try doing it.


0.27 Cover/command_line doesn't run commands

Home Assistant release (hass --version):
0.27.0
Python release (python3 --version):
3.4.2
Component/platform:
cover/command_line
Description of problem:
Commands don't run when pressing the button on the frontend. Toast messages appear when pressing the up and down button, but nothing happens.
Expected:
Commands run 😄
Problem-relevant configuration.yaml entries and steps to reproduce:
cover:
- platform: command_line
  covers:
    living:
      open_cmd: /home/pi/bin/kaku B 1 off
      close_cmd: /home/pi/bin/kaku B 1 on

Add cover/command_line
Configure "open_cmd" and "close_cmd"
Press up/down on frontend

Traceback (if applicable):

Additional info:

I migrated from the rollershutter component after it disappeared in the frontend upon updating to 0.27
I tried the old "upcmd" and "downcmd" instead of the new properties outlined in the documentation to no avail.
As a workaround I configured the cover as a command_line switch, the commands run perfectly.

You are using wrong config for commands.
You should use opencmd and closecmd

Any info from the logs would be great :)

Thanks! Very sneaky fix there :P.
https://github.com/home-assistant/home-assistant.github.io/commit/7ac8c005b282c98e528a17cea07f09481631b82d
On a different note, seems like "stopcmd" isn't actually required for the cover to function. It would be great if the stop button could be hidden from the frontend if the command isn't specified. In my case,


Add builder patter methods for changing default nullability

Add builder pattern style methods to avoid defaults having to be defined twice for Changing Default Nullability. This helps with ergonomics and readability of these options.

withDefaultFieldNullability
withDefaultInputFieldRequiredness

-export const builder = new SchemaBuilder<{
-  DefaultFieldNullability: true;
-}>({
-  defaultFieldNullability: true,
-});
+const builder = new SchemaBuilder({}).withDefaultFieldNullability();

-const builder = new SchemaBuilder<{
-  DefaultInputFieldRequiredness: true;
-}>({
-  defaultInputFieldRequiredness: true,
-});
+const builder = new SchemaBuilder({}).withDefaultInputFieldRequiredness()

Putting it the builder pattern together would end up as:
const builder = new SchemaBuilder({})
  .withScalars({ Date: DateScalar })
  .withDefaultFieldNullability()
  .withDefaultInputFieldRequiredness();

Related to https://github.com/hayes/pothos/issues/910 & https://github.com/hayes/pothos/pull/914

Looking into changing how nullability works in v4, so not sure this would be worth doing right now.  Typescript 5.2 iteration plan mentions investigating some stuff related to partial inference, so that may provide a better way to handle this as well


[AppBar] Fix onLeftIconButtonTouchTap event

[ ] PR has tests / docs demo, and is linted.
[ ] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[ ] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Bound the onClick event of the leftIconButton in AppBar to the onLeftIconButtonTouchTap event.
Fixes issue #2482

@pratimary Thanks for this PR. It's should be solved now on the master branch.

Can we get this fix published?

Isn't it in the v0.16.0 release?

Yes, confirmed that it's fixed in v0.16.0.  Thanks!


/opt/cache/mixed-full.html: commitAndReleaseBuffer: invalid argument (invalid character)

Tried to upgrade the site and now seeing:
[25/Feb/2018:11:37:45 +0000] During processing of request from <IP_ADDRESS>:40864
request:
"Request <\n    server-name: <IP_ADDRESS>\n    remote: <IP_ADDRESS>:40864\n    local: <IP_ADDRESS>:10010\n    Headers:\n      ========================================\n          User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:58.0) Gecko/20100101 Firefox/58.0\n Upgrade-Insecure-Requests: 1\n Connection: close\n DNT: 1\n Host: <IP_ADDRESS>:10010\n Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8\n X-Real-IP: <IP_ADDRESS>\n Accept-Encoding: gzip, deflate\n Cache-Control: max-age=0\n Accept-Language: en-US,en;q=0.5\n      ========================================\n    content-length: Nothing\n    method: GET\n    version: (1,0)\n    cookies:\n      ========================================\n      \n      ========================================\n    pathinfo: \n    contextpath: /\n    URI: /\n    params:\n      ========================================\n      \n      ========================================\n>"
A web handler threw an exception. Details:
/opt/cache/mixed-full.html: commitAndReleaseBuffer: invalid argument (invalid character)

Any ideas @chrisdone? Not sure what changed since last time I looked (long time now ...).
I suck at screen so fired up a tmux session there.

Nice one!


[Cosmos] Added HttpLoggingPolicy to pipeline

Resolves issue #8309
Also cleans up some old files in the repo and fixed some deprecation warnings in the tests.
Some test files have been renamed to be in keeping with pytest convention.

Can one of the admins verify this patch?

/azp run python - cosmos - tests

/azp run python - cosmos - tests

/azp run python - cosmos - tests

Need to investigate if this is needed. If so, will rebase and reopen.

Looks like this is still needed - not sure why it wasn't merged before.
Will fix the conflicts.

Reopened as #10295


Delete quotes from cooperation page #1669

Deleted quotes from cooperation page

Added tests for 100% coverage

Also don't forget to remove the same quotes in ua JSON translations

Also don't forget to remove the same quotes in ua JSON translations

Thank you. Really forgot about it. I fixed it


Use unique name for exported targets and avoid exporting binary targets

Bug fix
Fixed bug
The libraries in rmf_ros2 cannot be linked to by using ament_target_dependencies, forcing downstream users to use target_link_libraries instead.
Fix applied
Based on the guidelines in the ament_cmake docs, specifically the sample library:
install(
  TARGETS my_library
  EXPORT export_${PROJECT_NAME}
  LIBRARY DESTINATION lib
  ARCHIVE DESTINATION lib
  RUNTIME DESTINATION bin
)

ament_export_targets(export_${PROJECT_NAME} HAS_LIBRARY_TARGET)
ament_export_dependencies(some_dependency)

The exported target has a unique name rather than the library name, this is also explicitly suggested in a later paragraph that says the following:
The EXPORT notation of the install call requires additional attention: It installs the CMake files for the my_library target. It must be named exactly the same as the argument in ament_export_targets. To ensure that it can be used via ament_target_dependencies, it should not be named exactly the same as the library name, but instead should have a prefix like export_ (as shown above).
Furthermore, we were exporting binaries and targets and downstream users using ament_target_dependencies would get cryptic CMake failures such as:
CMake Error at /opt/ros/jazzy/share/ament_cmake_target_dependencies/cmake/ament_target_dependencies.cmake:146 (target_link_libraries):
  Target "[...]" links to:

    rmf_fleet_adapter::read_only

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

Call Stack (most recent call first):
  CMakeLists.txt:28 (ament_target_dependencies)

CMake Generate step failed.  Build files cannot be regenerated correctly.

So I split the installation step to make sure we only export targets for the library and not for the executable, again following the documentation.
These changes made it possible to compile downstream applications that link to rmf_fleet_adapter (I haven't tested the other packages but applied the same fix for consistency)

One thing to note is that ament_target_dependencies is actually being deprecated to encourage the use of target_link_libraries.

One thing to note is that ament_target_dependencies is actually being deprecated to encourage the use of target_link_libraries.

I'd propose to fully transition out of it once there is an official notice on the ROS 2 side, I noticed that it's still not 100% official yet


How exactly KAN can better interpreted than MLP?

It is not clear why KAN can be interpreted better than MLP ?

Did you at least read the paper and the documentation of this package?
One of the fundamental things pykan is able to do is symbolic transformation, i.e. you are able of expliciting the model formula itself. Every activation is a (smooth) function itself, easily plottable and applicable. I could continue, but I think I already gave too much explanations.

Yes you did I guess .
I read the paper twice and read the documentation too .
I could not connect the idea . Thanks for explaining it to me.
On Mon, May 13, 2024 at 10:27 AM Alessandro Flati @.***>
wrote:

Did you at least read the paper and the documentation of this package?
One of the fundamental things pykan is able to do is symbolic
transformation, i.e. you are able of expliciting the model formula itself.
Every activation is a (smooth) function itself, easily plottable and
applicable. I could continue, but I think I already gave too much
explanations.
—
Reply to this email directly, view it on GitHub
https://github.com/KindXiaoming/pykan/issues/175#issuecomment-2106655556,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ56RSWGD5QJVZGCDAANSUTZCBB2DAVCNFSM6AAAAABHTIXK5GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWGY2TKNJVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Also , can I some how help in making it super easy to understand . Let me
know  what you think?
On Mon, May 13, 2024 at 10:31 AM Mandar Gite @.***> wrote:

Yes you did I guess .
I read the paper twice and read the documentation too .
I could not connect the idea . Thanks for explaining it to me.
On Mon, May 13, 2024 at 10:27 AM Alessandro Flati <
@.***> wrote:

Did you at least read the paper and the documentation of this package?
One of the fundamental things pykan is able to do is symbolic
transformation, i.e. you are able of expliciting the model formula itself.
Every activation is a (smooth) function itself, easily plottable and
applicable. I could continue, but I think I already gave too much
explanations.
—
Reply to this email directly, view it on GitHub
https://github.com/KindXiaoming/pykan/issues/175#issuecomment-2106655556,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ56RSWGD5QJVZGCDAANSUTZCBB2DAVCNFSM6AAAAABHTIXK5GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWGY2TKNJVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Cannot sign in .. and user experience is TERRIBLE

Storage Explorer Version: 1.9.0
Build Number: 20190701.8
Platform/OS: macOS
Architecture: x64
Regression From:  I assume it works ...
Bug Description
{
"message": ""An error occurred while sending the request.""
}
Steps to Reproduce
Try to log into Azure subscription.  Taken to my orgs sign in page.  Go through normal sign in process.  Then "Unable to sign in: An error occurred while sending the request."
Expected Experience
Login success
Actual Experience
Really bad error message (no details).  Then, even worse was the sign up to GitHub.  But I'm here now.  Why would I want to sign up to GitHub to report a bug in the software?  The software should provide a reasonable error so I can see if the problem is because the signin was invalid, or the message couldn't be passed into the application.
Additional Context

Also, I have tried looking for self signed certs (none), starting the app with the "--ignore-certificate-errors" flag, and locking my login keychain.  None of these troubleshooting steps made any difference to the resulting error message (really bad message).

Are you behind a proxy?  If so, you need to config your proxy settings in Storage Explorer.
Also, have you tried to use device code flow sign-in?  It's under the Preview menu.

Hi, sorry that you're having sign in issues. There's some fairly involved steps I'd like you to take to help us diagnose what's going wrong. I know it's a lot, but this is what will help us figure out how to help you. If you have any problems with the below steps let us know and we'll do our best to help you. If you don't have time to do all of these steps, then you can probably ignore the proxy and Fiddler stuff for now, but it's usually easiest to get all of this collected in one go.
Note: these steps are for Windows, let me know if you need them for another OS.
Part 1: Reset Auth/Sign Out of Everything

Open Storage Explorer
Go to "Help" -> "Reset", make sure "Authentication" is selected (you can deselect "Quick Access" if you'd like), click "Reset"
After the reset dialog closes, go to "Edit" -> "Configure Proxy". Enable proxy, set the URL to 'http://localhost' and the port to '8888' (if you already have a proxy, skip this step and ignore all steps related to Fiddler), click "OK"
Close Storage Explorer

Part 2: Setup Fiddler

Install Fiddler from https://www.telerik.com/fiddler (Note: you can put in a fake email when they ask you for one 😋)
Start Fiddler
Go to Tools -> Options
Click on the HTTPS tab
Make sure Capture HTTPS CONNECTs and Decrypt HTTPS traffic are checked
Click on the Actions button
Choose "Trust Root Certificate" and then "Yes" in the next dialog
Click on the Actions button again
Choose “Export Root Certificate to Desktop”
Go to your desktop
Find the FiddlerRootCer file
Double click to open
Go to the "Details" tab
Click "Copy to File…"
In the export wizard choose the following options

Base-64 encoded X.509
For file name, Browse… to C:\Users\<your username\AppData\Roaming\StorageExplorer\certs, and then you can save it as any file name

Close the certificate window

Part 3: Clear Auth Logs

Go to C:\Users<your username>\AppData\Local\Temp\servicehub\logs
Delete Everything

Part 4: Try Signing In!

Open PowerShell
Goto where Storage Explorer is installed, if you installed at the default location you can just do cd 'C:\Program Files (x86)\Microsoft Azure Storage Explorer\'
Execute $env:SERVICEHUBTRACELEVEL="all"
Close as many other running programs as possible (excluding Fiddler and PowerShell)
Start Storage Explorer from PowerShell: .\StorageExplorer.exe
Try signing in, after your issue happens, close Storage Explorer
In Fiddle, in the bottom left where it says "Capturing", click that, and then "File" -> "Save All Sessions", and save the file wherever you'd like

Part 5: Share results with us

Go back to Go to C:\Users\<your username>\AppData\Local\Temp\servicehub\logs
Zip up all of the log files and attach them to a comment on this issue
Do not attach us the file you saved in Part 4!!! If we need that, we'll give you an email to send it to. That type of file can contain some super sensitive information so we do not want you putting it in a public place.

PS: sounds like you work at Microsoft. Feel free to look me up on teams for add'l help as well.

As stated in the issue metadata, this is for MacOS.  Are there similar steps for a normal OS?
I can login using a connection string, but not to the Azure account.


Feedback section (e.g. a simple form) taking audiences thoughts about PasteMe

A simple form implementation gathering audience's thoughts about PasteMe!

A simple form implementation gathering audiences' thoughts about PasteMe!

We need to discuss related (required) features in discussions section.


Bluetooth: CAP: Add support for doing just disable for unicast stop

The unicast_stop function is changed to primarily do a
BAP disable instead of a release, with optional
support for releasing the streams once they have been disabled.
fixes https://github.com/zephyrproject-rtos/zephyr/issues/70818

Investigating a possible issue

Investigating a possible issue

There is indeed an issue, but it's not caused by this PR nor easily fixable by this PR, and will be fixed in https://github.com/zephyrproject-rtos/zephyr/pull/79014


Remove findDOMNode before 1.0?

Looking at the docs, it seems that there are no upsides and is going to be removed so why keep it for the initial release? Once it's released, it will be there for some time

@Keats Removing it from core will break inferno-compat unless we spend time majorly re-writing certain parts of how inferno-compat can regain this feature. So it's definitely on the roadmap, although unlike to happen for 1.0 given the aim is to get 1.0 out in the next week.

We can look into this post 1.0. So let's add to roadmap @Havunen @nightwolfz @lukeed @LukeSheard


voidvault crashes asking for root priviledges

voidvault version: 1.5.2
voidlinux iso: void-live-x86_64-20181111-mate.iso (official)
I booted the live system adding rd.live.overlay.overlayfs=1 to the kernel command line to have more space available, installed the necessary dependencies and started the installation with voidvault new. The program crashed immediately after the timezone selection with the following error:
root privileges required   in method bootstrap at  /home/anon/voidvault-1.5.2/lib/Voidvault/Bootstrap.pm6  (Voidvault::Bootstrap) line 25   in method new at /home/anon/voidvault-1.5.2/lib/Voidvault.pm6  (Voidvault) line 46   in sub MAIN at /home/anon/voidvault-1.5.2/bin/voidvault line 128   in block <unit> at /home/anon/voidvault-1.5.2/bin/voidvault line 259
It is obvious that an installer needs to run as root, but running as a user up to a point and then crashing is... confusing :)

Will likely solve by clarifying the instructions.

You can also check and abort with an error message if the UID is not 0.


Observatory: "Tag order: None" too slow to display

With 40k samples from dart2js, the UI is locked, adding rows, for several minutes.

Is this still true after the last changes to the page?


An issue about Accordion -- The title and content are not shown

Detailed description
When I copy the code for Accordion to my vue project, the title and content can't be shown as below:

Is this issue related to a specific component?
Accordion

What did you expect to happen? What happened instead? What would you like to see changed?
I need to show the title and contents.

What browser are you working in?
Firefox

What version of the Carbon Design System are you using?
"carbon-components": "^8.18.13",

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

Steps to reproduce the issue

Step one
Step two
Step three
etc.

Additional information

The following code is for main.js

Screenshots or code
Notes

Add labels
Please choose the appropriate label(s) from our existing label list to ensure that your issue is properly categorized. This will help us to better understand and address your issue.

Hi @whwangp thank you for trying to report an issue. Did you want to enter this issue against https://github.com/nicholaslee119/carbon-components-vue first?

@asudoh Thanks for your replying. That's right. This issue is about carvue. I just changed it ca-accordion. And it is normally running now.


fcc authors page project skips too many concepts that need to be explained before step 2 is presented

Describe the Issue
In step 2 of the fcc authors page project (js beta), the following intro paragraph is provided:

The Fetch API is a built-in JavaScript interface to make network requests to a server. It has a fetch() method you can use to make GET, POST, PUT, or PATCH requests. In this project, you'll make a GET request to a URL for a JSON file with information about authors on freeCodeCamp News.

We have quite a few concepts here that a beginner would have no clue about and they're being presented here with zero introduction.
1- API - does anyone who has never programmed before know what an API is?
2- built-in interface - again, what should people understand  from these words having never been exposed to interfaces or whatever is meant by 'built-in' here?
3- network requests - maybe people know that 'network' is the internet, but do they have an idea about what a request is?
4- JSON file - we haven't talked about JSON files at all. Or even what JSON format is except if you count talking about JSON.stringify back in the todo list app.
It seems like these concepts may represent a whole project that is missing? Or maybe this 29 step project really needs at least 4 more steps to give even the smallest intro for these concepts? Or at the very least, can we simplify the text of this step -drastically- to just say something more easily digestible by a newbie (perhaps this is the hardest of all options)?
Note that it isn't until we get to the "Back End Development and APIs" curriculum that we are introduced to the concepts above. Here's the intro from Basic Node and Express section:
https://www.freecodecamp.org/learn/back-end-development-and-apis/basic-node-and-express/serve-json-on-a-specific-route
To be clear about where I stand, I wholeheartedly think that an earlier introduction to these concepts belongs in a JS course, absolutely yes. I just want it to be a much more complete one than the above sentence.
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-fetch-and-promises-by-building-an-fcc-authors-page/step-2
Your code
N/A
Expected behavior
define the new concepts over a number of steps or rewrite step 2 to use words that the camper has already learned about
Screenshots
No response
System
N/A
Additional context
No response

We could certainly provide some definition and brief descriptions for parts of this. But we can't get people programming quickly and also teach them computer science at the same time.

As an aside. It makes it sound like the fetch API is part of JavaScript, when in fact it is part of the Web APIs. The runtime must support it (e.g. it wasn't even in Node until more recently). Part of the text is taken almost verbatim from MDN, but the changes to it, makes it sound like it is a core JavaScript feature and not part of the Web APIs.

It sounds like we agree that something is missing in this step in terms of providing some definitions or explanation.
But just to throw it out there, if I want to not use any of these new terms to explain something like this I may say (please don't take this as a suggestion to change the text but just as an illustration of how the text can be changed to make it more digestible)
At some point in your coding journey you'll need to get some data from the internet. For example, you may need to work on creating a weather app and need to get weather predictions from an online source to do that. This data can be retrieved by asking for it online from the server (that is, a computer) that has this data. To do this we need to make use of an API (Application Programming Interface) which is a layer of code created to allow people outside an organization to access its internal data. It is similar to an app you would use to order products online but the code in this case allows you to request data instead.
Here is how you can make a GET request with the fetch() method (which is just a type of request that gets data):
fetch("url-goes-here")
Make a GET request to this URL: "https://cdn.freecodecamp.org/curriculum/news-author-page/authors.json". Don't terminate your code with a semicolon yet.

Opening this one up if anyone is interested in poking at it.

I almost prefer just simplifying all the terms for this step. Though I really think at some point there should be some kind of computer science intro that explains what XML and JSON files are before the start of the curriculum.


Explanation of 'changes on disk' dialog added, focusing on 'file comparison'

@blis-mendix can you confirm whether I correctly described the functionality, specifically wrt the 'Modified' status?

Yeah seems good

@MariaShaposhnikova This can be merged after review/approval


Add minimum TVL filter

Video here
https://github.com/beefyfinance/beefy-v2/assets/100975625/843bd446-ac34-4ca0-bf30-0b11bc6eaed9

Anything else needed here or are we good to merge? 👀

Anything else needed here or are we good to merge? 👀

Can you resolve the merge conflicts please

Anything else needed here or are we good to merge? 👀

Can you resolve the merge conflicts please

Done!


Add indication of old approvals

Approvals that have been open for a long time are prime candidates for revoking.

@rkalis I like the project a lot, I can take a look at this. Would adding an icon/tooltip that appears to the right of the last updated column text be sufficient? Also what do you think the cutoff should be for showing the indication (over 6 months, over 3 months, etc.)?

Hey @natebeauregard, that sounds great! Yeah I think having a tooltip that says something like "This approval is over X months/years old" sounds great.
We have a similar thing for spenders that were involved in hacks on the "Authorized Spender" column (https://github.com/RevokeCash/revoke.cash/blob/master/components/allowances/dashboard/cells/SpenderCell.tsx#L48)

I think if we do a similar thing on the Last Updated column, but a bit less "dangerous-looking" that would be great (maybe a yellow sign instead of a red one - not sure what the best representation would be). Let's start with checking >12 months old.
Let me know if you run into any issues!

@rkalis Seems like @natebeauregard is on this. If it is still open, I'd like to hop in.

Hey @bobeu, thanks for your interest! I just merged #177 yesterday, but I forgot to close this issue. I see you also commented on another issue, I'll add some more information there!

Hey @bobeu, thanks for your interest! I just merged #177 yesterday, but I forgot to close this issue. I see you also commented on another issue, I'll add some more information there!

That's great. Thank you.


The program crashes in half an hour or hour after starting a series of downloads images and video from search pages on Tumblr

Hello Johannes, I hope you can solve this problem.
Since version <IP_ADDRESS> the program crashes in about half an hour or an hour usually if I add all the page searches to the queue and start the download. Version <IP_ADDRESS> is stable and crash does not happen, so I use this version mostly. I use the program only for downloading images and videos from the searches pages on Tumblr, I do not download regular blogs already, so I do not know if the crash is observed if download them. Also it seems to me that the note to release <IP_ADDRESS> can be related to this behavior of the program.

Somewhat "fixes" the timeout. Thus, if you have a wonky connection that frequently gets interrupted, TumblThree shouldn't stall anymore. The timeout value now counts for the whole connection time regardless of it's state. E.g. if you won't finish downloading a large file (video) within 120 seconds (default) increase the value or the file is truncated. If the release has any side effects since I've had to modify the core webrequest/downloader/crawler code for this, please try the v<IP_ADDRESS> (#116).

Below here is the archive of my Index folder, so you can check if there is the crash on other machines (do not worry, the program should not download a lot of files, the entire date has already been downloaded and all file names are written in the index files).
Index.zip
Steps to reproduce the crash

run the program with my index folder
sort the "Url" column in alphabetical order, select all pages and add them to the queue, perhaps need to repeat several times
start the download
minimize the program, wait from half an hour to an hour approximately
crash occurs.

If this is not the case, and the crash is not observed, then this is likely my technical problems, then sorry for this. I just wrote this, because I want to update the program to version <IP_ADDRESS>, but I can not because of the crash.

I noticed that the speed of downloading from the GitHub is very slow. I uploaded to Mega for better speed https://mega.nz/#!bksC3IIQ!Fd47Tb6vhixKBqnKNJ1QIqJq3_NToR4w6GLfVYMk2Zk

Might be related to #164.

I've changed the default behavior of unobserved task to not terminate the application. In that commit you've quoted, I've wrapped all the connections into a Task in order to be able to terminate them if there is no response after the defined timeout. Since you might remember that the default timeout mechanism in the C# webrequest API doesn't work for async methods as I've found out in  #116.
That means the application shouldn't crash anymore but all the timeouts get silently dropped. It requires some more code to notify the user in what method what exactly dropped (e.g. if during a download of a file, or crawling the blog/search or checking if a raw file exists the connection was terminated because it's over the timeout).

With the new release, the crash disappeared. Everything works fine. Thanks for the fix!


Frontend - Signup page

https://www.figma.com/file/x778CXJcDSH3prY4woyxV4/Mosaic-(Community)?type=design&node-id=0%3A26&mode=design&t=z8OmBDqwnHVgcymp-1
Something like this is what we are going for. Ideally use tailwindcss, which needs to be set up. Roles should not be an option. Please check the swagger schema for what needs to be sent to the backend.
Work on the design first, I'm still building the backend logic.


Filtering a list item breaks rendering

Versions
expo: 33.3
react-native: https://github.com/expo/react-native/archive/sdk-33.0.0.tar.gz
Steps To Reproduce

Render a list of components
Filter one of them out and re-render
The height on the array of rendered components then collapses

This is the code which renders the list:
{articles.filter(filterRead).map(article => (
  <GuideItem
    article={article}
    key={article.pageProperties.id}
    onPress={() => {
      navigation.navigate('Guide', { article })
    }}
  />
))}

https://www.loom.com/share/ecbaed9a273f4fcc8decf0b682ce5322
## Describe what you expected to happen:
The item is simply removed and all items retain their prior height

https://github.com/facebook/react-native/issues/16183 seems to also reference this issue, although I am not using a FlatList; the problem seems more general than that. I managed to follow the workaround suggested there and manually add a height to fix the issue. However, I wanted to post this issue for any other users who needed items of differing height.
Apologies for not providing a reproduction; my setup is fairly involved atm.

Please use FlatList https://facebook.github.io/react-native/docs/flatlist


djlint reformatting broke relation edit form

When clicking on the edit button of a relation, the javascript console says:
Uncaught SyntaxError: "" string literal contains an unescaped line break

This is due to dba667f56394f7bf471770d97eb15498026abed8 reformatting all the templates and adding line breaks before the if statement in:
https://github.com/acdh-oeaw/apis-core-rdf/blob/7569984bb3a1b27607265e328b51d9fb88b9c452/apis_core/apis_relations/templates/apis_relations/edit_button_generic_ajax_form.html#L3
This seems to be due to the blank_line_before_tag and blank_line_after_tag setting

The same is true for the delete button and: https://github.com/acdh-oeaw/apis-core-rdf/blob/7569984bb3a1b27607265e328b51d9fb88b9c452/apis_core/apis_relations/templates/apis_relations/delete_button_generic_ajax_form.html#L5


Sub-directory per players in same port.

Give more information

Like this: localhost/bot/KaffinPX for prismarine-viewer not main directory.

Yes
Its my request Kaffin

I have a code but it crashes mostly, we need to integrate it to viewer

youre right its that
localhost:8080/bot/Fox3000
for have fp view of Fox3000

Please give more information
What is the use case ?
What do you expect this to do

I’ll clarify (they explained to me in Discord). Basically, since Heroku doesn’t allow for them to bind to multiple ports and they want to have multiple bots (each with its own viewer), the obvious answer is to use subdirectories (like https://example.com/bot-01 https://example.com/bot-02 and https://example.com/bot-03)

yeah its that

I think it would make sense for you to directly use the core lib to set that up
You can use https://github.com/PrismarineJS/prismarine-viewer/blob/master/lib/mineflayer.js as an example


[cli] How to ignore known conflict rules like arrow-body-style and prefer-arrow-callback?

It is said You can still use these rules together with this plugin if you want, because the bug does not occur all the time. But if you do, you need to keep in mind that you might end up with invalid code, where you manually have to insert a missing closing parenthesis to get going again., but I found no way to ignore it with command eslint-config-prettier .eslintrc.js.

Hi!
So it seems like if you enable both the prettier/prettier rule and arrow-body-style, the CLI helper prints this and exits with status 2:
❯ npx eslint-config-prettier index.js
The following rules are enabled with config that might conflict with Prettier. See:
https://github.com/prettier/eslint-config-prettier#special-rules

- arrow-body-style

Just to see if I understood correctly, that’s what you mean?
Can you tell me more about how you use the CLI helper tool? Do you run it in CI or something?

Yes, I'm running it on CI
https://github.com/1stG/configs/runs/1933571863?check_suite_focus=true#step:8:16

Thanks. I think we should change so that those rules always print warnings, but never cause non-zero exit.

I think error is fine to enforce users notice about the known issue and an option to disable the check if the user decide to risk after reading the document.

Fixed in v8.0.0. You should be able to just upgrade and not have to do anything special in your CI setup.


Avoid or mitigate progress hiccups

As a user I want the slider to stay where I lifted by finger, not to jump around, especially annoying when playing short content.
Acceptance criteria

Slider jumps on finger release do not occur anymore.

Idea
Keep efficient seeks but have two progress values:

The real progress, as we have today, whose value jumps once the player resumes playback.
An interpolated progress which smooths out jumps.

The interpolated progress should be calculated as follows:

When the user releases their finger the interpolated progress is set to the interactive progress location. Once the player resumes playback the real progress is calculated from the current player position.
If the interactive progress is lagging behind, with each progress tracker interval t we add a fixed increment dt to the interactive progress, i.e. the real progress advances from t, while the interactive one advances from t+dt. dt should be a reasonable fraction of the progress difference. If we want both progresses to again match after N ticks we should make dt equal to the initial progress difference divided by N.
If the interactive progress is lagging behind, with each progress tracker interval t we remove a fixed increment dt from the interactive progress. The rest is similar.
If a new interaction takes place this mechanism is reset.

I guess this idea could be implemented directly in smoothCurrentTimePublisher and would work for DVR streams as well but this requires a bit of thought.
Not sure how interpolation should work when seeking at the end, though.
Tasks

[ ] Implement the above algorithm.

Not sure this is a good idea, what we currently have is likely good enough for now. Closed.


Fix pip failure on Debian Bookworm

🗣 Description
This pull request:

Removes a vars file specific to Ubuntu Xenial
Modifies the role to install system packages vice pip packages where possible
Adds a CLI argument to pip to fix broken builds on Debian Bookworm

💭 Motivation and context

We no longer support Ubuntu Xenial as it is EOL.
On Debian-based systems, newer versions of pip balk at installing pip packages under the system Python, as pip now (correctly) regards the system Python as "externally managed".  In general, it is better to install system packages against the system Python wherever possible.
This Ansible role requires the installation of one pip package, since there is no system package available.  In order to do this on Debian Bookworm we must insist via a CLI argument that pip install the package against the system Python, contrary to pip's better judgement.

🧪 Testing
All automated tests pass.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated to reflect the changes in this PR.
[x] All new and existing tests pass.

What does @cisagov/team-ois think about creating an issue to get rid of the srvlookup dependency, so that we are no longer installing pip packages under the system python? I think it is a good idea.

I'm all for it. If we think of reasons why this is a bad idea, they can be discussed in that issue.

See #71 and commit d43d19c.

What does @cisagov/team-ois think about creating an issue to get rid of the srvlookup dependency, so that we are no longer installing pip packages under the system python? I think it is a good idea.

I'm all for it. If we think of reasons why this is a bad idea, they can be discussed in that issue.

See #71 and commit d43d19c.

Commenting here to tie it together (commented on the issue already) but I think we should explore using pipx as this problem will be a recurrent thing when Debian Bookworm officially releases later this year (2023). Playing whack-a-mole with specific dependencies is not a long term solution in my opinion. I think Python virtual environments or installing with pipx are better solutions to pursue.

What does @cisagov/team-ois think about creating an issue to get rid of the srvlookup dependency, so that we are no longer installing pip packages under the system python? I think it is a good idea.

I'm all for it. If we think of reasons why this is a bad idea, they can be discussed in that issue.

See #71 and commit d43d19c.

Commenting here to tie it together (commented on the issue already) but I think we should explore using pipx as this problem will be a recurrent thing when Debian Bookworm officially releases later this year (2023). Playing whack-a-mole with specific dependencies is not a long term solution in my opinion. I think using Python virtual environments directly or installing with pipx are better solutions to pursue.

Noted.  At the same time, this Ansible role is broken today and this PR remedies that.  In addition, getting rid of pip installs on the system Python is a good thing to do no matter what you end up doing down the line (pipx, Rid-X, etc.).

What does @cisagov/team-ois think about creating an issue to get rid of the srvlookup dependency, so that we are no longer installing pip packages under the system python? I think it is a good idea.

I'm all for it. If we think of reasons why this is a bad idea, they can be discussed in that issue.

See #71 and commit d43d19c.

Commenting here to tie it together (commented on the issue already) but I think we should explore using pipx as this problem will be a recurrent thing when Debian Bookworm officially releases later this year (2023). Playing whack-a-mole with specific dependencies is not a long term solution in my opinion. I think using Python virtual environments directly or installing with pipx are better solutions to pursue.

Noted. At the same time, this Ansible role is broken today and this PR remedies that. In addition, getting rid of pip installs on the system Python is a good thing to do no matter what you end up doing down the line (pipx, Rid-X, etc.).

Oh yeah I'm totally fine with this PR to get things going now. I'm simply talking about ways to long-term resolve this problem.


Specify proc-macro2 version more precisely

Since we can't compile 0.4.8 in current nightly, I think it's more convenient to specify 0.4.9.
See https://github.com/rust-lang/rust/pull/52552/ and https://github.com/alexcrichton/proc-macro2/commit/9f0a28a9c95d59c6969dd05121115884138dede4 for detail.

Thanks!


signing 2/2 from mobile with paytaca for mistswap swap fails on paytaca app

same is in most recent issue: can not reproduce. had one signer in brave with metamask, another in safari over wallet connect to paytaca web app. swapping bch to mist in mistswap app worked just fine.

i was trying with mobile paytaca

I am on ios and have no access to mobile paytaca. did you try to retest?

can we assume it is paytaca-specific issue?


Allday events not working properly

Hi,
We have implemented your adapter, but are currently having issues with allDay events. They will not get displayed as events that are for the whole day (or multiple days) but they will show as events that block the whole day.
I tried to pass a moment object for Start and end date, as well as a date object, but the result is the same.
I am pretty sure the problem is that our allDay events are from gmt+2 time zone, but can’t find a solution.
In what format should i put the start and end date for allDay events to work?

Hi there, thanks for reaching out. Currently this library doesn't support all-day events, as it wasn't a use case when I initially developed the library. However, I'm open to receiving pull requests to add the feature.
It would involve checking whether the ical parser's CalendarComponent dates have the dateOnly flag set to true, creating a new field on this library's CalDavEvent to indicate it's an all-day event, and then setting the allDay field when creating the EventData object to create the ICS string.

Closing this issue for now. Feel free to request that it be reopened if you'd like to implement the necessary changes.


Suggestions for Tracking Players in Hockey

Attached is the use case video with this issue
The situation we are working on
We are working on a player-tracking solution for Ice Hockey. We are trying to record the statistics and analytics of a game match with Deep Learning. We will be recording metrics like, on a given frame who are all inside the rink (playing area), player position,  and many others like who made a goal and how many goals the goalie (defender) defended. To track these, it is necessary to know which player stood where. Since the game is so fast-moving, it is impossible to recognize all players' jersey numbers in each frame. This is where tracking of each player helps to get the analytics better.
Need help with
A video input will be given with a duration ranging between (10 - 40 seconds). We have a custom-trained Yolo model which will detect three classes, player, goalie, and referee. We need a tracking solution that should take care of moving cameras, and reidentification of the same players if they are out of frame for say 1 second. We would like to get suggestions on which model from norfair to go with (or which combination) and some inputs on the hyperparameters for the case of ice hockey.
Solutions Tried
We have already tried the following from norfair,
- Tracking
- Tracking with ReID
- Tracking with Camera Motion
- Tracking with OpenPose
Issues Faced
1. The same player was given multiple track ids within a few seconds.
2. In some tuning cases, the tracking id of one player was given to another player if they collide with each other which will mess up the entire solution we have developed
We tuned some of the parameters but were unable to get significant improvement. With ReID and Camera Motion, the player was not given the same id after getting into the frame after say 15 frames (we did try tuning the hit_counter_max initialization_delay parameters)
We would like to get some suggestions on model selection and tuning parameters
https://github.com/tryolabs/norfair/assets/102944639/59b814c6-c994-4087-87fd-ce7e61ecd334

Hi @AbinashSankaran! Great to hear you are considering using Norfair for this exciting project!
As you know, multiple object tracking in sports is challenging and requires significant effort to make it work. But the scenario you describe (10 to 40 seconds videos) is one where you could get good results.
To get a sense of the current state of your tracking output, can you share a video where we can see the problems? With that, I can suggest some specific ideas to help you solve your problems. Or at least try.

Yes, I fine-tuned a YOLOv8 model using a small dataset that I constructed from your video, which can be found here. The initial model used for this process was the pre-trained version provided by Ultralytics, specifically, I believe it was the nano version.

Hi @AbinashSankaran, I have conducted several experiments and would like to share the results with you, which you may find useful for your own trials.
For these experiments, I fine-tuned a YOLOv8 model using a limited set of examples to detect the classes that you specified.
Below are the tests I performed:

Norfair stock (excluding ReID or camera motion): I employed iou as the distance_function and set 0.8 as the distance_threshold across all experiments. The results can be viewed here. Upon comparison with the video you uploaded, I noticed some enhancements. It may be beneficial to reassess the detector.
Norfair with camera motion: There was no noticeable improvement in comparison to the stock experiment. The camera movements were quite subtle, suggesting that this feature may not be necessary in this context. The results can be found here. The only modification made was the min_distance, which was set to 7. The code was derived from the camera motion demo. Please remember that it's crucial to mask the detections as demonstrated in our demo. In your videos, it's equally important to mask the channel logo and the scoreboard, as these elements remain static throughout the video. We aim to exclude points from these areas to improve the accuracy of the camera motion estimations.
Norfair with ReID: Although we do not currently have a user guide, I can detail the methods we employed and provide a video to demonstrate a potential outcome of using the ReID feature with embeddings. We utilized this repository, where a model for ReID can be fine-tuned. The repository includes a model zoo section with pre-trained models across various domains. In this instance, we used the ResNet50 trained with the Market1501 dataset. We fine-tuned this model for several players, the referee, and the goalie. The results indicated improvements and maintained consistent IDs for the referee and goalie. However, there is significant room for enhancement. The solution is not robust enough to retain the same ID for players, and we believe that if you do not require an online tracker (such as Norfair, which necessitates an ID for each person in every frame), better results can be achieved by analyzing the entire video prior to returning the results. The primary reason is that the player's number is not always visible and players often resemble each other. Additionally, it is likely that using the position of each player could further improve the results and some other rules that you know about this domain.

Here is the detector that I've fine-tuned, should you wish to replicate the results obtained from the first two alternatives.
We can delve deeper into each alternative based on your feedback on the performance of the videos I sent you, and explore ways to enhance your results. Please let me know your thoughts.

Hi @AbinashSankaran, I have conducted several experiments and would like to share the results with you, which you may find useful for your own trials.
For these experiments, I fine-tuned a YOLOv8 model using a limited set of examples to detect the classes that you specified.
Below are the tests I performed:

Norfair stock (excluding ReID or camera motion): I employed iou as the distance_function and set 0.8 as the distance_threshold across all experiments. The results can be viewed here. Upon comparison with the video you uploaded, I noticed some enhancements. It may be beneficial to reassess the detector.
Norfair with camera motion: There was no noticeable improvement in comparison to the stock experiment. The camera movements were quite subtle, suggesting that this feature may not be necessary in this context. The results can be found here. The only modification made was the min_distance, which was set to 7. The code was derived from the camera motion demo. Please remember that it's crucial to mask the detections as demonstrated in our demo. In your videos, it's equally important to mask the channel logo and the scoreboard, as these elements remain static throughout the video. We aim to exclude points from these areas to improve the accuracy of the camera motion estimations.
Norfair with ReID: Although we do not currently have a user guide, I can detail the methods we employed and provide a video to demonstrate a potential outcome of using the ReID feature with embeddings. We utilized this repository, where a model for ReID can be fine-tuned. The repository includes a model zoo section with pre-trained models across various domains. In this instance, we used the ResNet50 trained with the Market1501 dataset. We fine-tuned this model for several players, the referee, and the goalie. The results indicated improvements and maintained consistent IDs for the referee and goalie. However, there is significant room for enhancement. The solution is not robust enough to retain the same ID for players, and we believe that if you do not require an online tracker (such as Norfair, which necessitates an ID for each person in every frame), better results can be achieved by analyzing the entire video prior to returning the results. The primary reason is that the player's number is not always visible and players often resemble each other. Additionally, it is likely that using the position of each player could further improve the results and some other rules that you know about this domain.

Here is the detector that I've fine-tuned, should you wish to replicate the results obtained from the first two alternatives.
We can delve deeper into each alternative based on your feedback on the performance of the videos I sent you, and explore ways to enhance your results. Please let me know your thoughts.

Do you example code where you have used ReID with Norfair? Thanks

Hi @AbinashSankaran, I have conducted several experiments and would like to share the results with you, which you may find useful for your own trials.
For these experiments, I fine-tuned a YOLOv8 model using a limited set of examples to detect the classes that you specified.
Below are the tests I performed:

Norfair stock (excluding ReID or camera motion): I employed iou as the distance_function and set 0.8 as the distance_threshold across all experiments. The results can be viewed here. Upon comparison with the video you uploaded, I noticed some enhancements. It may be beneficial to reassess the detector.
Norfair with camera motion: There was no noticeable improvement in comparison to the stock experiment. The camera movements were quite subtle, suggesting that this feature may not be necessary in this context. The results can be found here. The only modification made was the min_distance, which was set to 7. The code was derived from the camera motion demo. Please remember that it's crucial to mask the detections as demonstrated in our demo. In your videos, it's equally important to mask the channel logo and the scoreboard, as these elements remain static throughout the video. We aim to exclude points from these areas to improve the accuracy of the camera motion estimations.
Norfair with ReID: Although we do not currently have a user guide, I can detail the methods we employed and provide a video to demonstrate a potential outcome of using the ReID feature with embeddings. We utilized this repository, where a model for ReID can be fine-tuned. The repository includes a model zoo section with pre-trained models across various domains. In this instance, we used the ResNet50 trained with the Market1501 dataset. We fine-tuned this model for several players, the referee, and the goalie. The results indicated improvements and maintained consistent IDs for the referee and goalie. However, there is significant room for enhancement. The solution is not robust enough to retain the same ID for players, and we believe that if you do not require an online tracker (such as Norfair, which necessitates an ID for each person in every frame), better results can be achieved by analyzing the entire video prior to returning the results. The primary reason is that the player's number is not always visible and players often resemble each other. Additionally, it is likely that using the position of each player could further improve the results and some other rules that you know about this domain.

Here is the detector that I've fine-tuned, should you wish to replicate the results obtained from the first two alternatives.
We can delve deeper into each alternative based on your feedback on the performance of the videos I sent you, and explore ways to enhance your results. Please let me know your thoughts.

In your 3rd bulletpoint, you mentioned that using player positions can improve the result. How can we integrate the player information, specially if they move out of the frame. I am doing player reid for basketball. Often times the players go out of the frame and come back later (~30 seconds later). I am using clip_reid for reid purposes, but it still tags different players with same  id. I was interested in incorporating the player positions as well. I would really appreciate any help.


bug(sidenav): sidenav width "jumps" without any interaction

Reproduction
Use StackBlitz to reproduce your issue:

https://stackblitz.com/github/ecodev/natural/tree/blinking2

Steps to reproduce:

With Google Chrome only
Click alternatively on Home and Other page menu entry
See that the pink area extend for about ~1 seconds and then shrink again

Expected Behavior
I expect the <mat-sidenav> width (in pink) to remain constant at all times.
Actual Behavior
The width of <mat-sidenav> is changing for about ~1 second before returning to its correct width.
Since my code is doing pretty much nothing at all, I don't expect "jumping" behavior like that. Especially not after what seems to be a setTimeout().
In the video below, I click exactly one time per button, yet we clearly see what seems to be a 2-step "animation" of the  <mat-sidenav> width for each click.
https://user-images.githubusercontent.com/72603/120990600-ac645380-c7bb-11eb-9f1e-972500bb3d69.mp4
Environment

Angular: 12.0.3
CDK/Material: 12.0.3
Browser(s): Chrome 91.0.4472.77  (Firefox is OK)
Operating System (e.g. Windows, macOS, Ubuntu): Ubuntu

I think the problem is that you're using mat-nav-list, but you haven't included a Material theme. This means that when you click the link, Material tries to create a ripple which increases the width. There's some information about theming in the getting started guide: https://material.angular.io/guide/getting-started

Unfortunately this is not the source of the problem. I pushed the Material theme to the StackBlitz and the bug still appears:
https://user-images.githubusercontent.com/72603/121005654-fe60a580-c7ca-11eb-9d55-688095119e4c.mp4

It seems like it's related to the ripple, because it goes away if I set disableRipple on the mat-nav-list.

Having a very similar issue on mat-drawer

I can confirm that adding disableRipple will fix the issue. However it is not an applicable workaround for me, as I need those ripples.
I updated the StackBlitz to entirely remove <mat-nav-list> and use a native <ul> instead. And worse, I moved the problematic button from the menu to the main page content, so outside of the jumping layout, and the bug still occurs:
https://user-images.githubusercontent.com/72603/121106803-b5980380-c841-11eb-911c-61db3bb6aa47.mp4
I also noticed that removing fxFill would "fix" the issue, but since it breaks the whole layout it is also not a viable workaround.
So far I think the bug appear when <mat-sidenav-container>, fxFill and ripple effects are used together, though they might not be the only combinaison to trigger the bug.
@dan5082 could you please confirm whether you use @angular/flex-layout in your project ?

The StackBlitz is now using Angular 12.1 and all latest deps, but the bug still occurs.
And since @angular/flex-layout pretty much did not change between 11.0 and 12.0, it seems unlikely that it would have been introduced by flex-layout. But just for the sake of it, I tested with 11.0.0-beta.33 and it also is reproducible. That means that the bug has be introduced by a change in either @angular/material or @angular/core, but not @angular/flex-layout.

After an update to Angular 13 I cannot reproduce the issue anymore. Going back to Angular 12, I also cannot reproduce the issue anymore. My conclusion is that the bug was actually in Chrome and now with Chrome 98.0.4758.102, I cannot reproduce at all.
I consider this issue solved somewhere between Chrome 91.0.4472.77 and 98.0.4758.102.


Implement MediaElement on every platforms

MediaElement (or MediaPlayer+MediaPlayerElement) is a difficult control to implement (much more than other, because of SMTC) but without it, Uno seems unusable to me because all my works use MediaElement.
The solution has to work on everyplatform (even WASM with ).
Obviously some advanced features can't work (like DLNA casting) on other platforms but basic functionnality could be a good start.

Same for me, my primary app relies heavily on MediaElement and integrating the host platform's player on each target platform would be big effort (each developer would have to do then over and over again).
Especially supporting custom transport controls would be a big plus.

This issue has been superseded by https://github.com/nventive/Uno/issues/154, you can follow the progress there, which has been significant!


List edit don't update, when iwad_path\pwad_path button click

I think I fix that in 1.2.2. Today. Maybe.


Bringing the repository into the modern age

At the moment these bindings are not tested to ensure pull requests build. Probably would be a good idea to setup github actions for this.
There's also some other things such as migration to 2018 edition, running cargo fmt and resolving the clippy lints (we probably want to suppress keysyms.rs).

what is the problem with keysyms.rs?

Hmm looking at it further it might be the overzealous clippy::all lint that is enabled.

I agree, I've just removed this lint in my local repo

closed by #23


content collections type only v2.5.0+

Someone was getting an error setting up their content collections with type: content because they hadn't yet upgraded to 2.5.0.
This quick change explicitly says that the type property was added in 2.5.0. We shouldn't need this for very long, and we can figure out a better way to handle changes like this. But there will still be some people on other pre 2.5 versions of Astro who may run into this issue if they just try setting up content collections from scratch with our existing docs, not realizing this isn't just how it always was.

Yep this is perfect and very clear, thank you @sarah11918 !


[Refactoring] Noise protocols

Solves #583.
The idea would be to introduce three types of Noise: PulseNoise, PostProcessingNoise and BlockNoise.

BlockNoise are used to modify a block using the transpiler function set_noise introduced in #563.

PostProcessingNoise would be only used to modify the output of a program. At the moment it only includes readout, which modifies the output samples. At the moment, it is backend independent.

PulseNoise used to create a SimConfig instance for the Pulser backend.

There are two approaches to the interface I can think of.

The first easier option would be to keep the Noise dataclass, just adding a type to it, so this does not modify much the current codebase. We can introduce the three types as children. Noise would only serve the purpose  of holding options and protocol name besides for PostProcessingNoise  where we extract a method add_noise from a module to be used in apply_noise which is called in a few backends.

The second one would require more changes as it means making consistent which noises can be used at the backend level and the method called. For instance, the pulser backend can only accept a PulseNoise in the run method, and all backends can accept a PostProcessingNoise in the sample method. A problem we can foresee is consistent typing with base/abstract classes.  Serialization may be also challenging.

Thanks @chMoussa :)
IMO, my feeling is that this separation could create some confusion. For example:

You suggest BlockNoise to represent the noise models that go to PyQ, but the Qadence design principle is that all programs are composed of "blocks", even ones that compile to the Pulser backend (or will compile in the future).
The PulseNoise is defined as specifically creating a SimConfig, but we also have time-dependent blocks that go to PyQ, and in principle this PulseNoise could also have an implementation there.
PostProcessingNoise -- There's not really post processing going on yet right? I feel like post-processing is what you would do then for noise mitigation.

Maybe something like:

DigitalNoise: for noise protocols described as discrete events acting on a block (that also represents a discrete operation). In PyQ it's essentially what is already implemented. In the future when we have digital operations in other backends (including pulser or a pulser-like backend), these can have an implementation there as well.
AnalogNoise: for noise protocols described as a continuous effect that affects some other continuous operation. For Pulser it can plug into the SimConfig like you said, while for PyQ we still need to implement it, but from my understanding this would be about including some extra terms in the Hamiltonian evolution right?
ReadoutNoise: what you called the "post processing" one

In principle, I think the set_noise function could be available for both, but it would fail if we try to do set_noise of a AnalogNoise on a digital gate, and possibly fail if we try to set a digital noise on a time-dependent hamiltonian evolution.
Wdyt?

Hi @jpmoutinho,
Thanks for these clarifications. It helps. On analog noise, indeed it would be just setting an option in the solvers with the list of operators. For Postprocessing (only happening in the sample methods of the backends), I thought that one could add modules if there was a need for it later. But right now we can restrict to ReadoutNoise. I'll try to adapt your suggestions rn

I agree with @jpmoutinho on this. It makes more sense to talk about digital and analog noises in this context. As for the readout noise, I tend to think this should go in the backends too but it is currently not supported in Pulser unless I'm wrong. This is why it still remains in Qadence and induces confusion and difficulties in having a clean interface. Maybe this is something we can ask the Pulser people to implement ? It'll make our life much easier though. Wdyt ?

I agree with @jpmoutinho on this. It makes more sense to talk about digital and analog noises in this context. As for the readout noise, I tend to think this should go in the backends too but it is currently not supported in Pulser unless I'm wrong. This is why it still remains in Qadence and induces confusion and difficulties in having a clean interface. Maybe this is something we can ask the Pulser people to implement ? It'll make our life much easier though. Wdyt ?

If it is possible for them to work on this feature on their side, then I am fine by it and to put it in Pyqtorch. It would be a matter of instantiating the right object.

Closes as #591 supersedes this.


[SPARK-14382][SQL]QueryProgress should be post after committedOffsets is updated

What changes were proposed in this pull request?
Make sure QueryProgress is post after committedOffsets is updated. If QueryProgress is post before committedOffsets is updated, the listener may see a wrong sinkStatus (created from committedOffsets).
See https://amplab.cs.berkeley.edu/jenkins/job/spark-master-test-maven-hadoop-2.2/644/testReport/junit/org.apache.spark.sql.util/ContinuousQueryListenerSuite/single_listener/ for an example of the failure.
How was this patch tested?
Existing unit tests.

Test build #54894 has started for PR 12155 at commit 9717c55.

Test build #54894 has finished for PR 12155 at commit 9717c55.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54894/
Test PASSed.

Test build #54925 has started for PR 12155 at commit 41b4996.

Test build #54925 has finished for PR 12155 at commit 41b4996.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54925/
Test PASSed.

LGTM. Merging to master.