update ip

Locks and plans deleted for the projects and workspaces modified in this pull request:

dir: infrastructure/cloud/cloudflare/access workspace: terragrunt
dir: infrastructure/cloud/uptime-robot/otaru workspace: terragrunt

Ran Plan for 2 projects:

project: infrastructure/cloud/uptime-robot/otaru dir: infrastructure/cloud/uptime-robot/otaru workspace: terragrunt
project: infrastructure/cloud/cloudflare/access dir: infrastructure/cloud/cloudflare/access workspace: terragrunt

1. project: infrastructure/cloud/uptime-robot/otaru dir: infrastructure/cloud/uptime-robot/otaru workspace: terragrunt
Plan Error
Show Output
running "terragrunt init -reconfigure" in "/atlantis-data/repos/siutsin/otaru/853/terragrunt/infrastructure/cloud/uptime-robot/otaru": exit status 1: running "terragrunt init -reconfigure" in "/atlantis-data/repos/siutsin/otaru/853/terragrunt/infrastructure/cloud/uptime-robot/otaru":
04:35:33.639 INFO   Downloading Terraform configurations from ../../.. into ./.terragrunt-cache/yLawb58AuOnbQYtAkCFisCkl_OA/oUu0VOkpC7olajR8pAEH0tyb_Ec
04:35:34.208 STDOUT tofu: Initializing the backend...
04:35:34.588 STDOUT tofu:
04:35:34.588 STDOUT tofu: Successfully configured the backend "s3"! OpenTofu will automatically
04:35:34.588 STDOUT tofu: use this backend unless the backend configuration changes.
04:35:34.678 STDOUT tofu: Initializing provider plugins...
04:35:34.678 STDOUT tofu: - Reusing previous version of integrations/github from the dependency lock file
04:35:34.833 STDOUT tofu: - Reusing previous version of vexxhost/uptimerobot from the dependency lock file
04:35:34.949 STDOUT tofu: - Reusing previous version of paultyng/unifi from the dependency lock file
04:35:35.001 STDOUT tofu: - Reusing previous version of hashicorp/aws from the dependency lock file
04:35:35.072 STDOUT tofu: - Reusing previous version of backblaze/b2 from the dependency lock file
04:35:35.198 STDOUT tofu: - Reusing previous version of cloudflare/cloudflare from the dependency lock file
04:35:36.636 STDOUT tofu: - Installing backblaze/b2 v0.8.12...
04:36:03.515 STDOUT tofu: - Installing cloudflare/cloudflare v4.42.0...
04:36:05.394 STDOUT tofu: - Installing integrations/github v6.2.2...
04:36:14.532 STDOUT tofu: - Installing vexxhost/uptimerobot v0.8.2...
04:36:21.892 STDOUT tofu: - Installing paultyng/unifi v0.41.0...
04:36:29.624 STDOUT tofu: - Installing hashicorp/aws v5.67.0...
04:37:43.378 STDERR tofu: ╷
04:37:43.381 STDERR tofu: │ Error: Failed to install provider
04:37:43.382 STDERR tofu: │
04:37:43.382 STDERR tofu: │ Error while installing backblaze/b2 v0.8.12: mkdir .terraform: no such file
04:37:43.383 STDERR tofu: │ or directory
04:37:43.383 STDERR tofu: ╵
04:37:43.383 STDERR tofu:
04:37:43.383 STDERR tofu: ╷
04:37:43.383 STDERR tofu: │ Error: Failed to install provider
04:37:43.383 STDERR tofu: │
04:37:43.383 STDERR tofu: │ Error while installing cloudflare/cloudflare v4.42.0: mkdir .terraform: no
04:37:43.383 STDERR tofu: │ such file or directory
04:37:43.383 STDERR tofu: ╵
04:37:43.383 STDERR tofu:
04:37:43.384 STDERR tofu: ╷
04:37:43.384 STDERR tofu: │ Error: Failed to install provider
04:37:43.384 STDERR tofu: │
04:37:43.384 STDERR tofu: │ Error while installing integrations/github v6.2.2: mkdir .terraform: no
04:37:43.385 STDERR tofu: │ such file or directory
04:37:43.385 STDERR tofu: ╵
04:37:43.385 STDERR tofu:
04:37:43.385 STDERR tofu: ╷
04:37:43.386 STDERR tofu: │ Error: Failed to install provider
04:37:43.386 STDERR tofu: │
04:37:43.387 STDERR tofu: │ Error while installing vexxhost/uptimerobot v0.8.2: mkdir .terraform: no
04:37:43.387 STDERR tofu: │ such file or directory
04:37:43.387 STDERR tofu: ╵
04:37:43.388 STDERR tofu:
04:37:43.388 STDERR tofu: ╷
04:37:43.388 STDERR tofu: │ Error: Failed to install provider
04:37:43.388 STDERR tofu: │
04:37:43.388 STDERR tofu: │ Error while installing paultyng/unifi v0.41.0: mkdir .terraform: no such
04:37:43.388 STDERR tofu: │ file or directory
04:37:43.388 STDERR tofu: ╵
04:37:43.388 STDERR tofu:
04:37:43.388 STDERR tofu: ╷
04:37:43.388 STDERR tofu: │ Error: Failed to install provider
04:37:43.388 STDERR tofu: │
04:37:43.388 STDERR tofu: │ Error while installing hashicorp/aws v5.67.0: mkdir .terraform: no such
04:37:43.388 STDERR tofu: │ file or directory
04:37:43.388 STDERR tofu: ╵
04:37:43.388 STDERR tofu:
04:37:43.391 ERROR  tofu invocation failed in ./.terragrunt-cache/yLawb58AuOnbQYtAkCFisCkl_OA/oUu0VOkpC7olajR8pAEH0tyb_Ec/modules/uptime-robot
04:37:43.392 ERROR  error occurred:

* Failed to execute "/usr/local/bin/tofu init -reconfigure" in ./.terragrunt-cache/yLawb58AuOnbQYtAkCFisCkl_OA/oUu0VOkpC7olajR8pAEH0tyb_Ec/modules/uptime-robot
  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing backblaze/b2 v0.8.12: mkdir .terraform: no such file
  │ or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing cloudflare/cloudflare v4.42.0: mkdir .terraform: no
  │ such file or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing integrations/github v6.2.2: mkdir .terraform: no
  │ such file or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing vexxhost/uptimerobot v0.8.2: mkdir .terraform: no
  │ such file or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing paultyng/unifi v0.41.0: mkdir .terraform: no such
  │ file or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing hashicorp/aws v5.67.0: mkdir .terraform: no such
  │ file or directory
  ╵

  exit status 1

2. project: infrastructure/cloud/cloudflare/access dir: infrastructure/cloud/cloudflare/access workspace: terragrunt
Plan Error
Show Output
running "terragrunt init -reconfigure" in "/atlantis-data/repos/siutsin/otaru/853/terragrunt/infrastructure/cloud/cloudflare/access": exit status 1: running "terragrunt init -reconfigure" in "/atlantis-data/repos/siutsin/otaru/853/terragrunt/infrastructure/cloud/cloudflare/access":
04:35:33.646 INFO   Downloading Terraform configurations from ../../.. into ./.terragrunt-cache/HOQjTfu7Twkm1SmYWZYwkgpRcHE/oUu0VOkpC7olajR8pAEH0tyb_Ec
04:35:34.227 STDOUT tofu: Initializing the backend...
04:35:34.587 STDOUT tofu:
04:35:34.587 STDOUT tofu: Successfully configured the backend "s3"! OpenTofu will automatically
04:35:34.587 STDOUT tofu: use this backend unless the backend configuration changes.
04:35:34.755 STDOUT tofu: Initializing provider plugins...
04:35:34.755 STDOUT tofu: - Reusing previous version of integrations/github from the dependency lock file
04:35:34.849 STDOUT tofu: - Reusing previous version of vexxhost/uptimerobot from the dependency lock file
04:35:34.956 STDOUT tofu: - Reusing previous version of paultyng/unifi from the dependency lock file
04:35:35.002 STDOUT tofu: - Reusing previous version of hashicorp/aws from the dependency lock file
04:35:35.066 STDOUT tofu: - Reusing previous version of backblaze/b2 from the dependency lock file
04:35:35.259 STDOUT tofu: - Reusing previous version of cloudflare/cloudflare from the dependency lock file
04:35:36.594 STDOUT tofu: - Installing hashicorp/aws v5.67.0...
04:37:44.135 STDOUT tofu: - Installing backblaze/b2 v0.8.12...
04:37:46.146 STDOUT tofu: - Installing cloudflare/cloudflare v4.42.0...
04:37:47.726 STDOUT tofu: - Installing integrations/github v6.2.2...
04:37:49.157 STDOUT tofu: - Installing vexxhost/uptimerobot v0.8.2...
04:37:50.496 STDOUT tofu: - Installing paultyng/unifi v0.41.0...
04:37:51.119 STDERR tofu: ╷
04:37:51.119 STDERR tofu: │ Error: Failed to install provider
04:37:51.119 STDERR tofu: │
04:37:51.119 STDERR tofu: │ Error while installing hashicorp/aws v5.67.0: mkdir .terraform: no such
04:37:51.119 STDERR tofu: │ file or directory
04:37:51.120 STDERR tofu: ╵
04:37:51.120 STDERR tofu:
04:37:51.120 STDERR tofu: ╷
04:37:51.120 STDERR tofu: │ Error: Failed to install provider
04:37:51.120 STDERR tofu: │
04:37:51.120 STDERR tofu: │ Error while installing backblaze/b2 v0.8.12: mkdir .terraform: no such file
04:37:51.120 STDERR tofu: │ or directory
04:37:51.120 STDERR tofu: ╵
04:37:51.120 STDERR tofu:
04:37:51.120 STDERR tofu: ╷
04:37:51.120 STDERR tofu: │ Error: Failed to install provider
04:37:51.120 STDERR tofu: │
04:37:51.120 STDERR tofu: │ Error while installing cloudflare/cloudflare v4.42.0: mkdir .terraform: no
04:37:51.120 STDERR tofu: │ such file or directory
04:37:51.120 STDERR tofu: ╵
04:37:51.120 STDERR tofu:
04:37:51.120 STDERR tofu: ╷
04:37:51.120 STDERR tofu: │ Error: Failed to install provider
04:37:51.120 STDERR tofu: │
04:37:51.120 STDERR tofu: │ Error while installing integrations/github v6.2.2: mkdir .terraform: no
04:37:51.120 STDERR tofu: │ such file or directory
04:37:51.121 STDERR tofu: ╵
04:37:51.121 STDERR tofu:
04:37:51.121 STDERR tofu: ╷
04:37:51.121 STDERR tofu: │ Error: Failed to install provider
04:37:51.121 STDERR tofu: │
04:37:51.121 STDERR tofu: │ Error while installing vexxhost/uptimerobot v0.8.2: mkdir .terraform: no
04:37:51.121 STDERR tofu: │ such file or directory
04:37:51.121 STDERR tofu: ╵
04:37:51.121 STDERR tofu:
04:37:51.121 STDERR tofu: ╷
04:37:51.121 STDERR tofu: │ Error: Failed to install provider
04:37:51.121 STDERR tofu: │
04:37:51.121 STDERR tofu: │ Error while installing paultyng/unifi v0.41.0: mkdir .terraform: no such
04:37:51.121 STDERR tofu: │ file or directory
04:37:51.121 STDERR tofu: ╵
04:37:51.121 STDERR tofu:
04:37:51.128 ERROR  tofu invocation failed in ./.terragrunt-cache/HOQjTfu7Twkm1SmYWZYwkgpRcHE/oUu0VOkpC7olajR8pAEH0tyb_Ec/modules/cloudflare-access
04:37:51.128 ERROR  error occurred:

* Failed to execute "/usr/local/bin/tofu init -reconfigure" in ./.terragrunt-cache/HOQjTfu7Twkm1SmYWZYwkgpRcHE/oUu0VOkpC7olajR8pAEH0tyb_Ec/modules/cloudflare-access
  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing hashicorp/aws v5.67.0: mkdir .terraform: no such
  │ file or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing backblaze/b2 v0.8.12: mkdir .terraform: no such file
  │ or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing cloudflare/cloudflare v4.42.0: mkdir .terraform: no
  │ such file or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing integrations/github v6.2.2: mkdir .terraform: no
  │ such file or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing vexxhost/uptimerobot v0.8.2: mkdir .terraform: no
  │ such file or directory
  ╵

  ╷
  │ Error: Failed to install provider
  │
  │ Error while installing paultyng/unifi v0.41.0: mkdir .terraform: no such
  │ file or directory
  ╵

  exit status 1

Plan Summary
2 projects, 0 with changes, 0 with no changes, 2 failed


populate HTMLEditorExtender from codebehind

My understadning is that the HTMLEditorExtender causes two div elements to but placed in the page. When the client puts some content in the content div, and sends it back to the server, the server receives a html version of that content, i.e the version from the source div. I want to store that content and send it back to another textbox (with HTMLEditorExtender) in a different place/page. The problem is that what comes back is the html, but it gets placed in the content div rather than the source div.
If i do something in the codebehind to store the textbox.text and then send it to a diferent a textbox then my content gets converted to html. I'm happy that this isn't a bug, but rather the main feature of the extender.
So, is there a way to populate the source div from the codebehind?
My reason for doing this is that i want to allow text and pictures to be typed/pasted into a single container and have the formatting preserved.

Yes, you are right, HtmlEditorExtender sends encoded HTML to and from the server to populate the content div.
It has no built-in capabilities for populating source div, because source tab is optional and only content tab is always present.

Would you please describe your scenario in greater detail?
The goal can be achieved with the HtmlEditorExtender, but it will be a complex solution.
We'll try to find an easier approach.

The goal is to give a user a box where they can write a message and paste an image if they want to. It for feedback on a website. I am storing the data in sql and then listing it in a page. The sql part is ok and text with fancy formatting is fine. I just can't get he image to display. I've tried sending the stored html to a new div in the page as htmlgenericcontrol.innerhtml rather than back to a textbox and it almost works, but the image doesn't get drawn.
If i return the html to the textbox with entender i can copy it, paste it to the source tab and when i return to the content tab the pic gets drawn.

some additional info. I have created a small example to demonstrate the current behaviour.
The are two stages:

User input - here everything looks as intended
Server response - this is what comes back from the server

Here is the html that actually appears in the page, grabbed using the browser developer tools.
html for content tab

text
link

html for source tab

text
link

text in source tab
text
link
response
html for content tab

text
link
<img src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAUAAAAFCAIAAAACDbGyAAAAEElEQVQImWNQ0o9BRgwU8gGejBDmY/IxUwAAAABJRU5ErkJggg==" alt="">

html for source tab

text<br><a href="link.com">link</a><br>&lt;img src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAUAAAAFCAIAAAACDbGyAAAAEElEQVQImWNQ0o9BRgwU8gGejBDmY/IxUwAAAABJRU5ErkJggg==" alt=""&gt;

text in source tab
text
link
<img src="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAUAAAAFCAIAAAACDbGyAAAAEElEQVQImWNQ0o9BRgwU8gGejBDmY/IxUwAAAABJRU5ErkJggg==" alt="">
So the only difference is the
 tag is converted to < img >

Thank you for this clarification.
Unfortunately, I cannot reproduce this markup change.
Would you please attach a sample project that illustrates this issue?

Based on your previous posts I would suggest using simple div with separate file upload controls for the feedback form, provided this feedback is not visible to public.

I have just updated my version to a much newer version and am still getting some odd results.
if i go for an image pasted in with no text or text after the image it works fine.
if i put some text before the image then the image simply disapears when the content is sent to and return from the server.
i'll test with the newest version of the toolkit as soon as i can get a copy installed
I wasn't able to get the latest version because the download link is dead on this page: http://cdn.devexpress.com/Release/ACT/16.1/20160811/AjaxControlToolkit.Installer.16.1.1.exe
giving this response:
This XML file does not appear to have any style information associated with it. The document tree is shown below.
MissingKeyMissing Key-Pair-Id query parameter or cookie value

@dominicafonso Thank you for pointing out this link to us! We are fixing it.
You can download the installer from the GitHub release as well: https://github.com/DevExpress/AjaxControlToolkit/releases/download/16.1.1/AjaxControlToolkit.Installer.16.1.1.exe
As for the HTML change, the extender performs a set of replacements to remove malicious markup. This is the reason why the part of the initial markup may disappear.

Feel free to reopen this issue if you have any questions regarding it.


[feature request] add regions defaults

Add a region defaults in settings or on this tab

Where you can choose between NA/EU/JP regions that got recently introduced, from what I saw from normal strings it's ~region(jp) // ~region(na) // ~region(eu) in between master user ID and nonce

Already done just isn't a release for it yet https://github.com/pypy-vrc/VRCX/commit/afe7fa98aaeef7196138449abbdff696860cee61


Failing tests due to missing OpenAI API key

Description
Currently, there are approximately 60+ tests failing when the OpenAI API key is not set, leading to the following error:
FAILED path/to/test/testfile.py::test_name - openai.OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable

Root Cause
The tests depend on the OpenAI API for their functionality. If the OPENAI_API_KEY environment variable is not set or the key is not passed to the client, the tests are unable to proceed, resulting in an OpenAIError.
Proposed Solution
To prevent these tests from failing, I propose implementing a custom decorator @requires_openai_api_key in tests/store/conftest.py. This decorator will:

Check if the OPENAI_API_KEY is set: Before each test runs, the decorator will verify the presence of the OpenAI API key.
Skip tests with a descriptive message: If the API key is not set, the decorator will skip the test with a clear and descriptive message indicating that the test requires an API key to run.

Example message:
Skipping test: OPENAI_API_KEY environment variable is not set.
This test requires an OPENAI_API_KEY to run and is not included in the results.

Benefits

Prevents unnecessary test failures: By skipping tests that require an API key, we avoid misleading test failures that are unrelated to the actual code functionality.
Improves CI/CD Workflow: This solution will help prevent the GitHub Actions workflow from failing due to the absence of an API key, leading to a more stable and reliable CI/CD pipeline.

Additional Notes
This decorator could be extended in the future to check for other required environment variables or configurations.

I like this idea, though I'm not completely sure a custom decorator is necessary if we can use the existing pytest skip decorator on a whole test class:
@pytest.mark.skipif(openai_api_key == None, reason="Skipping test: OPENAI_API_KEY environment variable is not set. ")

Oh maybe I did not write it out properly, I actually meant kind of exactly what you proposed.
I was not aware you can use it on a whole test class, I have some code where I used it on just every function that needs to have it, and the good thing is some test classes like test_chroma and test_duckdb_adapter have (will have) parameterised tests and you could also just add it in there.
in conftest.py:
requires_openai_api_key = pytest.mark.skipif(
    os.getenv('OPENAI_API_KEY') is None,
    reason="Skipping test: OPENAI_API_KEY environment variable is not set. \
            This test requires an OPENAI_API_KEY to run and is not included in the results.",
)

parameterised example:
@pytest.mark.parametrize("collection, model, requires_key", [
    (None, None, False),
    ("one_collection", None, False),
    pytest.param("test_openai_collection", "openai:", True, marks=requires_openai_api_key),
    pytest.param("test_openai_full_collection", "openai:text-embedding-ada-002", True, marks=requires_openai_api_key)
])
def test_the_embedding_function_variations(simple_schema_manager, example_texts, collection, model, requires_key):

and for any other function just put that on top
@requires_openai_api_key

However right now from what I saw we do not necessarily have full test classes. Its test files with a lot of functions mostly. Thats why a decorator works well. However if we restructure this, definitly true, we could maybe put all openai tests In one class or at least limit the amount of times we have to use a custom decorator.
As I think the implementation I suggested works pretty fast without a lot of changes to the test suite I kind of feel we should do this for a quick fix and can adapt later when we have more structured test classes?


Selected State does not annonce when Button Component selected

Description
Does not announce "selected" on the focus of the Button Component.
This issue is covered in-depth by parent issue #30842, please check there for more details.
React Native version:
0.63

Thank you for listing the tasks and detailed explanations! @amarlette 😄

@amarlette PR #31001 should resolve this aswell. Can you please test ?

@huzaifaaak I will follow up with @kacieb on the status of testing for this.

@amarlette PR #31001 should resolve this aswell. Can you please test ?

I just confirmed by testing on my device that the linked PR resolved this issue! This issue can be closed.

Great thank you for testing!


Implement optimize command

Databricks delta lake provides an optimize command that is incredibly useful for compacting small files (especially for files written by streaming jobs which are inevitably small). delta-rs should provide a similar optimize command.
I think a first pass could ignore the bin-pack and zorder features provided by Databricks and simply combine small files into large files while also setting dataChange: false in the delta log entry. An MVP might look like:

Query the delta log for file stats relevant to the current optimize run (based on predicate)
Group files that may be compacted based on common partition membership and less than optimal size (1 GB)
Combine files by looping through each input file and writing its record batches into the new output file.
Commit a new delta log entry with a single add action and a separate remove action for each compacted file. All committed add and remove actions must set the dataChange flag as false to prevent re-processing by stream consumers.

Files no longer relavant to the log may be cleaned up later by vacuum (see https://github.com/delta-io/delta-rs/issues/97)

While vacuum is a big step forward, it does not obviate need for optimize. These two commands have different purpose, and ( looking further out ) optimize will eventually become part of either batch or background worker, continuously operating with lower priority, as is common to all relational engines.

@wjones127 - looks like #607 was merged!!  Will it be relatively easy to expose this functionality via the Python bindings?

@MrPowers Yeah shouldn't be too difficult. I created #622 to track that.

Resolved by #607.


prevent trigger in dragging

done

Why close this?

Sorry, Because you reply to me too slow, I don't have time to spend with wait for your reply, now, I am already copy your code to my project.

For your kind information, I have a full-time job, and a personal life and work on this project in my free time. So Sorry, for late replies (and having a full time job).
I guess you are new to open source. Anyways thanks for your contribution

Okay,Thanks


V2.13.2 d3x trace viewer support in mlflow

closing in favour of - https://github.com/abikki/mlflow/pull/6


On Android, this library is not optimized

Hello,
On iOS it is working perfectly. On Android, the map is so laggy, just to recalculate the cluster....
Someone has a tip ?

same issue here, any updates?

same issue here, any updates?

Hi,
Still the same issue...Did you find another library ? I found supercluster but it looks more complicated to implement...

I think my problem is not from this lib, but from react-native-maps or just bad coding from my side. I have an array for Markers after some research i found this prop "tracksViewChanges - Sets whether this marker should track view changes. It's recommended to turn it off whenever it's possible to improve custom marker performance." the default value is true so i changed to false i saw a huge difference but using it brokes my UI i have  children components arround the Marker, an image component and some Views. Unfortunately some markers render perfectly and other don't.

@0xcD3v Did you manage to solve not rendering images on android? It works perfectly for me on ios, but I also have difficulties with android...


Comment in the README the primary difference between this and goldmark-meta

Searching for "markdown frontmatter" brought up this package first.  However, it looks like goldmark itself already has a "metadata" extension which can do something similar.  Skimming through the documentation, it I see two main differences:

Your extension can use TOML (with +++ as delimiters) in addition to YAML
Your extension can marshal data into a struct, whereas goldmark-meta can only give you a map[string]any.

Anything else in particular why you decided to develop your own extension rather than using goldmark-meta?
Thanks!

Thanks for opening this, @gwd.
Yeah, those are both the differences and the reason I developed the extension:
I wanted the option for a type-safe API instead of map[string]any.
I needed the ability to decode other popular frontmatter formats. (Hugo supports TOML, so I wanted support.)
The extension can also be configured to recognize custom frontmatter formats.
One other difference: this extension does not provide the WithTable option to render the metadata as a table. YAML frontmatter can be arbitrarily complex structs, and those don't translate to a two-dimensional table well.
For context, I developed this extension to use in stitchmd. stitchmd transforms Markdown documents so it needed to support both popular frontmatter formats.
I've tried to list the features of this extension without referencing the goldmark-metadata extension, but maybe you're right and that isn't enough. I'll add some more to the README.


Clean up some relative-path handling in lister using Git magic

This lets us cut out the line which hard-codes how deeply nested in
the tree the run-mypy script is, making it simpler to borrow these
scripts in other projects.

Automated message from Dropbox CLA bot
@gnprice, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

(The build failure is in the py3k checker, and it appears to be a genuine specific issue in the code in master.)

Nice CLA auto-message. :-)  I'm a Dropbox employee, but I also just went and signed the CLA so that we're covered even if a given contribution is for any reason outside the scope of my employment.

Automated message from Dropbox CLA bot
@gnprice, thanks for signing the CLA!

Yep, merged the fix for master and just reran the py3k check

I guess it'll still fail until you rebase those, so rerunning was likely pointless

Merged, thanks @gnprice!  @sharmaeklavya2 as FYI in case I missed anything in my review.


Add a validator for Active Storage attached files

Alright, this is ready to go. I will create draft PRs from now on so I can make sure everything is alright before I request a review.


Proposal: Interpolated Parameterization

One of my personal favorite ways to deal with SQL is using interpolated strings to deal with parameters. The risk of SQL injection can be nullified by defining a special InterpolatedStringHandler. I've implemented this before in internal projects, and in a toy project of mine for postgres.
The tradeoff is that methods that accept parameters (query and live) would need to commit to this interpolated string handler only, not accepting regular strings with parameter objects anymore (so as to prevent accidental injection by previously interpolated strings that have since been interpolated into an actual string).
Ill put up some examples in a bit.

An example of querying with interpolation:
var published = true;
var tags = new string[] { "surrealdb", "dotnet" };
var posts = await client.Query<Post>($"""
    SELECT *
    FROM post
    WHERE published = {published}
    AND tags CONTAINSALL {tags}
    """);

The driver will then use a Surreal-specific InterpolatedStringHandler to handle escaping for safety, and potentially even circumvent the allocation of a query string altogether and directly prepare the byte buffer to be sent over HTTP/WS for performance. The generated json would look similar to:
{
  "method": "query",
  "params": ["SELECT *\nFROM post\nWHERE published = $p_published\nAND tags CONTAINSALL $p_tags",
  {
    "p_published": true,
    "p_tags": ["surrealdb", "dotnet"]
  }
}

To make this work safely, we would preferably drop the Query(string) overload to prevent the following situation:
var injection = "; REMOVE USER root -- or whatever you want";
var oops = $"""
    SELECT * FROM post WHERE tags CONTAINSALL {injection}
    """;
var posts = await client.Query(oops);

In this example, the interpolation occurs outside the scope of the method call, at which point the compiler will use the DefaultInterpolatedStringHandler (the one used normally), which will not parameterize the query safely and create a regular string.
If we allow the Query(string) overload, the user might shoot themselves in the foot. But if we only allow the interpolated overload e.g. Query(SurrealInterpolatedStringHandler) the compiler would disallow regular strings from being passed in, protecting the user from making the mistake.
There are ways to circumvent this by allocating the interpolated string handler manually and passing that in, but at that it's safe to assume the user knows what they are doing.
In terms of DX, even if we don't allow the Query(string) overload, users would not necessarily need to keep the interpolation in mind; as the Query(SurrealInterpolatedStringHandler could be called (even without parameters) by putting the familiar $ in front of the string, which is how the compiler is instructed to use the interpolation overload.
var posts = await client.Query($"SELECT * FROM post");
                              ^^^
                               | The $ becomes mandatory

The only public example I have is one of my toy projects, but I'm doing more than just interpolation here so the idea is a lot harder to follow (it's a source generator that also validates queries at compile time, and the interpolation logic is source generated as well).

This is interesting stuff indeed. Reminded me of Dapper in a way.
First, I can understand the will to use interpolated string to prevent developer bad practices when using standard string (non interpolated). However, I think we should let developers have the ability to use a simple string as just SELECT * FROM myTable; or creating a string in a previous context. I think the best option here would be to create a Roslyn Analyzer that will detect the input of the Query method and trigger a warning (or error) to offer guidance on how to do proper parameterization.
GIven your example, I would prefer the use of indexed parameters in order to not interfere with the existing params that can be consumed either from the Set method or directly set inside the Query method. And it will reduce the string length as a secondary benefit. So, it should give something like SELECT *\nFROM post\nWHERE published = $0\nAND tags CONTAINSALL $1. Other than that, I agree that using parameters this way gives a better DX than having to create a Dictionary by hand.
Finally, your idea of validating queries at compile time is interesting. I suppose simple queries can be easily ran and checked against a SurrealDB in-memory instance running in a background process. I am curious about string parameterization, I suppose you would have to take random values according to the parameter types?

The toy project I referenced earlier is inspired by rust's sqlx crate, so that may be the best way to look for proper/robust ideas. I'm checking parameters currently by simply generating data based on the type, but there are circumstances where this won't work (constraints etc).
Note that since it's a toy project, I haven't explored the ins and outs of what is possible (and more importantly what isn't).
A roslyn analyzer as opposed to a forcedly correct method signature can also work. I prefer the signature approach though as a roslyn analyzer would require additional maintenance.
About the parameter naming; there isn't really an ideal solution here, people can still SET $0 = 1 if they want to (not likely but still). I don't believe string length will have any noticeable performance hit; and if that is a concern then we should probably just wait for the binary rpc protocol 😄.


Stage1 tool's --write-parameters flag is superflous

The stage1 tool has a flag -write-paramters that's not useful since it can only set DataWriter.write_parameters to True, which already is the default.
Flags are only useful if they set values other than the default.
There is also currently no nice way to deactivate writing parameters, so the flag should probably be --no-parameters instead.
We could also follow a common pattern and provide pairs of flags for everything, so I would propose to change
--write-images, --write-paramters to --images, --no-images, --parameters, --no-parameters.

I'm actually surprised that traitlets.config's "boolean flag" functionality doesn't automatically add negative flags... it really should (e.g. --no-write-images).  Or we make the defaults all False maybe?

In any case, we may have to think about what these really mean in the future: I called them write-X for a reason: they do not tell the tool if it should compute them (that depends on  the input), only if they should be in the output.

I'm actually surprised that traitlets.config's "boolean flag" functionality doesn't automatically add negative flags... it really should (e.g. --no-write-images). Or we make the defaults all False maybe?

Actually it does, when using traitlets.config.boolean_flag (or ctapipe.core.traits.flag as we call it):
flag("write-parameters", "DataWriter.write_parameters", "store parameters in output", "dont store"))

yields the following --help output:
--write-parameters
    store parameters in output
    Equivalent to: [--DataWriter.write_parameters=True]
--no-write-parameters
    dont store
    Equivalent to: [--DataWriter.write_parameters=False]

Ha, nice. Can you make a PR fixing that for our tools?

Will do


Is it possible to make terraform ignore desired_size

First of all, thank you for the work in all these terraform modules, it's been a great help to my work.
Use case: On a eks with cluster-autoscaler
Behavior: Updated desired size get overridden by terraform on apply
Desired behavior: desired_size is ignore by terraform
Is it possible (or a good idea) to make desired_size a ignore attribute.
Best option would be to create a flag to conditionally ignore it, but since https://github.com/hashicorp/terraform/issues/3116 lifecycle block does not allow variables.
Example:
## Add ti resource.aws_eks_node_group.default
  lifecycle {
    ignore_changes = [
      scaling_config["desired_size"],
    ]
  }

Hi @dreamrace adding following code to node_group config works for me:
lifecycle {
    ignore_changes = [scaling_config.0.desired_size]
  }

Hi all, I just opened a PR for this.

@maxgio92 @mbebjak having managed nodegroup:
desired: 1
max: 1
min: 1

if i add below:
lifecycle {
  ignore_changes = [scaling_config.0.desired_size]
}

and then update nodegroup to :
desired: 2
max: 2
min: 2

i get error:
Error: error updating EKS Node Group (XXXXX) config: InvalidParameterException: Minimum capacity 2 can't be greater than desired size 1
│ {
│   RespMetadata: {
│     StatusCode: 400,
│     RequestID: "b80be2de-b265-4388-bdff-6e3582706117"
│   },
│   ClusterName: "onegha",
│   Message_: "Minimum capacity 2 can't be greater than desired size 1",

idea is to have cluster autoscaler manage desired and terraform plan to not show changes to desired size and apply works if we want to change min/max .


Add cube example on readme

To be honest it was intended to not put cube sample here, because it is not very interesting sample

To be honest it was intended to not put cube sample here, because it is not very interesting sample

Well i should close then


fix(core): fixing order-by enum handling by MySQL and MariaDB driver

Fixes #5004

Added getOrderByExpression to AbstractSqlPlatform.ts as an overridable method to format orderBy queries. This uses syntax that is recognized by both SQLite and PostGreSQL
Added override of getOrderByExpression to MariaDbPlatform.ts and MySqlPlatform.ts that adds functionality to handle (ASC|DESC) NULLS (FIRST|LAST) syntax

can you check how sqlite behaves? we might need the same there

This uses syntax that is recognized by both SQLite and PostGreSQL

edit: i should read the whole thing first :]

It probably would make sense to create test suites for each language. Currently the QueryBuilder.test.ts file is only testing against MySQL from what I could see.


Update security-checker to new URL to fix build.

Proposed Changes
Currently the Travis build is breaking because the URL used by the security-checker no longer exists.
This MR updates that URL to the new home in order to fix the build.

@mjrider, can you review (and optionally merge) this, if @frenck is unavailable?

Looks like this fixes part of the issue... PHP5.4 and PHP5.5 need Trusty to build.
However, changing the matrix to this:
matrix:
  include:
    - php: 7.2
      env: SNIFF=1
    - php: 5.5
      # As the latest Debian not supporting PHP 5.5 anymore, we need to force using 'trusty'.
      dist: trusty
    - php: 5.4
      # As the latest Debian not supporting PHP 5.4 anymore, we need to force using 'trusty'.
      dist: trusty
    - php: 5.3
      # As the latest Debian not supporting PHP 5.3 anymore, we need to force using 'precise'.
      dist: precise

did not yield the expected result (i.e. Trusty being used...)

Oh and as you're making changes to the Travis script now anyway, why not add a build against "7.4snapshot", possibly with an allow_failures for it too ?

@mjrider I'm all done. Can you please review/approve so these changes can be merged to master?
They are needed before I can update the other open MRs.

The command "composer install-codestandards" exited with 0.

Should this exit code be 1 ?


Custom base class not working

When you have a partial class and you want to give that partial class it's own base class it's currently not supported.
For example:
Generated class:
public partial class ProductDetails : Base, IHeader
Custom partial class:
public partial class ProductDetails : HeaderCompostion
HeaderCompostion class (only here for clarification)
public class HeaderCompostion : Base, IHeader
So in this case it would be nice if the generated class wouldn't have the base class because that's already defined in the partial class.

Skype chat to clarify why I would like to do this:
[02-03-15 16:28:07] Jeroen Breuer - Developer: Hi
[02-03-15 16:28:33] Jeroen Breuer - Developer: Can I ask a question about the Models Builder and Compositions?
[02-03-15 16:29:22] Stéphane Gay: hey
[02-03-15 16:29:23] Stéphane Gay: sure
[02-03-15 16:30:19] Jeroen Breuer - Developer: With a doctype that has compositions is generated as a class with interfaces for each composition
[02-03-15 16:30:44] Jeroen Breuer - Developer: What if there is a property on a composition that I want to override?
[02-03-15 16:31:22] Jeroen Breuer - Developer: I know you can do that with the ImplementPropertyType attribute on a partial class
[02-03-15 16:31:36] Stéphane Gay: properties that come from compositions are defined on the interface BUT are actually implemented in each content type
[02-03-15 16:31:57] Jeroen Breuer - Developer: But would that mean I have to do it on each generated models that inherits that interface?
[02-03-15 16:32:05] Stéphane Gay: if you want to override one of these properties... you have to either do it on each generated models...
[02-03-15 16:32:21] Stéphane Gay: or to do it on a base class that all of these models would inherit
[02-03-15 16:32:58] Jeroen Breuer - Developer: the base class sounds interesting
[02-03-15 16:33:15] Jeroen Breuer - Developer: the models all have a base class, but that's the base doctype
[02-03-15 16:33:16] Stéphane Gay: that's the only way I see (and I think I've used it)
[02-03-15 16:33:38] Jeroen Breuer - Developer: I don't know if the base doctype has those properties
[02-03-15 16:33:44] Stéphane Gay: well... you need to insert an intermediate class between that base doctype and your models
[02-03-15 16:33:57] Jeroen Breuer - Developer: is that possible?
[02-03-15 16:34:02] Stéphane Gay: eg ContentBase -> Composition -> Model
[02-03-15 16:34:07] Stéphane Gay: should be possible
[02-03-15 16:34:30] Stéphane Gay: create your Composition class that inherits from ContentBase (and implements the IComposition interface)
[02-03-15 16:34:43] Stéphane Gay: then for each model, create a partial & specify that the model interits from Composition
[02-03-15 16:34:50] Stéphane Gay: should work
[02-03-15 16:35:07] Stéphane Gay: (thinking it might be good to put that in the doc once you get it to work)
[02-03-15 16:35:19] Jeroen Breuer - Developer: Won't that give errors with the generated classes
[02-03-15 16:35:24] Jeroen Breuer - Developer: those also have a base class
[02-03-15 16:35:38] Jeroen Breuer - Developer: can you tell your partial class to have a different base class?
[02-03-15 16:36:13] Stéphane Gay: I guess that at the moment they have base class ContentBase... if you specify another inheritance in your partial, I think we detect that and when you rebuild, we do not set a base class on the models
[02-03-15 16:36:21] Stéphane Gay: at least that's the plan ;)
[02-03-15 16:36:30] Stéphane Gay: I'll feel better when you tell me it works
[02-03-15 16:36:56] Jeroen Breuer - Developer: Haha thanks I'll do some testing this week
[02-03-15 16:37:21] Stéphane Gay: plz let me know how it goes (email me code if you can) => must put it in the docs, it's good material
[02-03-15 16:37:38] Jeroen Breuer - Developer: Sure I will

So the plan was to have it work OOTB. And I'm pretty sure that if your content type is a root content type and would inherit from PublishedContentModel, we do detect that it already has a base class. But I may be wrong. In your case, your content type is not a root content, it has a parent content type... and I probably overlooked that situation.
Will fix.

Reproduced. If a type does not have a parent, its generated model should inherit from PublishedContentModel - unless we have a partial that defines another inheritance. That works. However it fails if the type does have a parent. Fixing.

Fixed - but we cannot ensure that the inheritance chain is not broken. If type TypeA has parent TypeB and you create a partial class for TypeA indicating that it inherits from Whatever... then it's up to you to make sure that Whatever somehow inherits from TypeB. You're on your own ;-)

No problem :-). I just want an extra base class that has the correct parent in the chain. That way I can create my composition interfaces on that.


Add amor con hielo by morat

Song Name
amor con hielo
Artist Name
morat
Source
Youtube
Youtube Link
https://youtu.be/4-cbwT3OHkc?si=5Ks1bhjWEgfSh2Gr
Direct File Link
No response
Song ID
739991
Start Offset [ms]
12
Extra
Created from Auto Nong.
Created for level: 53858796

reject
popular songs like "sphere" are too inconvenient to be replaced by smth unrelated and are therefore usually rejected.
Note that you can still use the song yourself, but it won't be public


Example demo hanging

I have installed all dependencies, and
cd examples/
./demo.sh

The script has downloaded GloVe glove.840B.300d.txt, but it seems to hangs. No files written in the log folder and no python running on top. Any hint?

-- Are you using Python 2 or 3? The code only runs under 2.
-- make the folder log

@YingyuLiang By defaults in 2.7.10. I have realized and created the log/ folder within the example folder. Is that right?

Yes I think so.

@YingyuLiang uhm maybe this is the problem:
(.env) [loretoparisi@:mbploreto log]$ ls -l
total 8
-rw-r--r--  1 loretoparisi  staff   0 19 Lug 17:56 output_sim_sif.txt
-rw-r--r--  1 loretoparisi  staff   0 19 Lug 17:56 output_sim_tfidf.txt
-rw-r--r--  1 loretoparisi  staff  64 19 Lug 17:56 output_supervised_sif_proj.txt
(.env) [loretoparisi@:mbploreto log]$ cat output_supervised_sif_proj.txt
./demo.sh: line 19: ./supervised_sif_proj.sh: Permission denied

@YingyuLiang It seems it still hangs. I'm running on macOS. I bid that this could be the issue.

Yes, loading the word vectors takes time, especially because the full glove vector file is quite large.  You can try to use only a subset of that file, using only the words with large enough frequency.

@YingyuLiang You were right...it's working
word vectors loaded from ../data/glove.840B.300d.txt
word weights computed from ../auxiliary_data/enwiki_vocab_min200.txt using parameter a=-1.000000
remove the first 0 principal components
                    MSRpar2012   0.454328

word vectors loaded from ../data/glove.840B.300d.txt
word weights computed from ../auxiliary_data/enwiki_vocab_min200.txt using parameter a=-1.000000
remove the first 1 principal components
                    MSRpar2012   0.364071

word vectors loaded from ../data/glove.840B.300d.txt
word weights computed from ../auxiliary_data/enwiki_vocab_min200.txt using parameter a=0.001000
remove the first 0 principal components
                    MSRpar2012   0.436383

word vectors loaded from ../data/glove.840B.300d.txt
word weights computed from ../auxiliary_data/enwiki_vocab_min200.txt using parameter a=0.001000
remove the first 1 principal components
                    MSRpar2012   0.356372
...

Closing this then.

NOTE Just for the records, I had an issue with lasagne and theano version
ImportError: cannot import name downsample
that I have solved in this way:
(.env) [loretoparisi@:mbploreto SIF]$ pip install --upgrade https://github.com/Lasagne/Lasagne/archive/master.zip
Collecting https://github.com/Lasagne/Lasagne/archive/master.zip
  Downloading https://github.com/Lasagne/Lasagne/archive/master.zip
     | 235kB 905kB/s
Requirement already up-to-date: numpy in ./.env/lib/python2.7/site-packages (from Lasagne==0.2.dev1)
Installing collected packages: Lasagne
  Found existing installation: Lasagne 0.1
    Uninstalling Lasagne-0.1:
      Successfully uninstalled Lasagne-0.1
  Running setup.py install for Lasagne ... done
Successfully installed Lasagne-0.2.dev1
(.env) [loretoparisi@:mbploreto SIF]$ python -c "import lasagne; print lasagne .__version__"
0.2.dev1

I'm using
(.env) [loretoparisi@:mbploreto SIF]$ python -c "import theano; print theano .__version__"
0.9.0


Fix failing tests

Fix missing package.json git repo info
Assign missing baseUrl in server/server.js
Add smoke test
Allow mocha globals in linter
Add mocha to dev deps

@slnode test please

@slnode test please

@slnode test please

@slnode test please

@slnode test please

@slnode test please

@bajtos Merging without review as this is a simple fix for CI.

@bajtos Merging without review as this is a simple fix for CI.


php: 7.2.22 -> 7.2.23, 7.3.9 -> 7.3.10

Motivation for this change
Changelog: https://www.php.net/ChangeLog-7.php#7.3.10
Changelog: https://www.php.net/ChangeLog-7.php#7.2.23
So far I've only made sure the hashes works, but I will build all the things on a more powerful system and check boxes below accordingly.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Diff seems fine, currently running nix-review locally.

Btw nix-review runs into a timeout due to libcouchbase. This appears to be an issue with openssl 1.1.1, I'll check if upstream has some patches for this :)

On my home server php 7.2.23 (with libressl) is working fine.

Yeah, the problem isn't related to php, but to libcouchbase (which is used by phpPackages.couchbase). I'd try to fix the libcouchbase issue tomorrow, but it's perfectly fine if someone merges this before :)

Libcouchbase is fixed in #70455. I'll rerun nix-review with my patch, but I don't expect any further issues :)

Thanks! Backported as 63b18e757620223b4f9ac1cd2f65fb9709d2aa2a, 6ae771e8d5319f45401e3afba90b5eafc2c48635.

@Ma27 Thanks yourself for looking into libcouchbase :)


Sync updates from NCIA

As we are finally getting into more regular releases, it may be a good time for a PR

Beware it contains stuff specific to our repo - mostly in .travis.yml


Fix usage of finalizers

Fix usage of finalizers

The "kubernetes" finalizer is a special finalizer token not meant for use by other projects.
The finalizer token values should also be qualified names.
Updated the code  to use sets for handling finalizers (ordering is not required)

@MHBauer -- the finalizer token "kubernetes" is used in namespace deletion which predates objectmeta.finalizers, but may converge in the future, so i do not want catalog to use it.  the only other two finalizer tokens that are delivered in the core are related to garbage collection and are "orphan" and "foregroundDeletion".  any finalizer token not in the core should be a qualified name with a prefix.  for example, federation uses a finalizer token "federation.kubernetes.io/delete-from-underlying-clusters" with proper qualified prefixing.

@arschles -- updated.  for reference, ordering does not matter in finalizers list.

@derekwaynecarr Is there a finalizers document somewhere that has
 any finalizer token not in the core should be a qualified name with a prefix. for example, federation uses a finalizer token "federation.kubernetes.io/delete-from-underlying-clusters" with proper qualified prefixing.

in it? Seems like a great example that would help anybody coming along to write or understand finalizers in the future.


Improve UI/UX

Improve UI/UX with CSS Animations, Hover Effects, subtle animations for buttons, etc. The source code can be converted to Javascript before implementation of CSS but this is not necessary

can you assign this to me

!assign @Salma-101

can u pls assign this to me @sujith27pes

@sujith27pes only one person can be assigned one issue at a time

!deassign

assign me this pls

pls assign

!assign @Lalith2907

@nikhila-kuchimanchi how much time do i have??

the timer is automatically set to 45 min, if you require more time I will assign it to you once this timer runs out

ohh alrightt thank youu
will let you know if required

@nikhila-kuchimanchi could i pls get a little more time??

!deassign

!assign @Lalith2907 15

can you assign me

@Salma-101 I'll assign it to you in 15 min so you can commit your code

can you assign me.. im done with it

can i be assigned to it

!assign @Salma-101

!deassign

!assign @Salma-101

can i be assigned to it

can i be assigned?

!deassign


Implementation of double exponential filtering for synaptic traces

Dear authors,
In the implementation for $\alpha * \big(\epsilon * S_j(t)f^{\prime}U_i(t)\big)$ (eq. 18):
https://github.com/fmi-basel/latent-predictive-learning/blob/dd486d092b8cea9bfa1f27f42209f9174b82e61c/spiking_simulations/LPLConnection.cpp#L303-L316
It seems the two exponential filterings of $\alpha$ are done differently:

The outer filtering which produces el_val_flt using time constant tau_el_decay through AurynVectorFloat::follow:
https://github.com/fzenke/auryn/blob/6174a67a56074e8b72a94a763277131f62778713/src/auryn/AurynVector.h#L372-L377
The inner filtering which produces el_val is updated through add_to_syntrace( didx, syn_trace_input ) and el_val->scale(scale_const).

My questions are:

Why the two synaptic traces are computed differently? Why not use AurynVectorFloat::follow for both of them?
Related to the first question, add_to_syntrace only updates the trace by adding the $\epsilon * S_j(t)f^{\prime}U_i(t)$ before scaling it. This results in the following discrete update I think:
$\bar{c}[t+1]=\exp(-dt/\tau)(\bar{c}[t]+c[t])\approx (1-dt/\tau)(\bar{c}[t]+c[t])=(1-dt/\tau)\bar{c}[t]+(1-dt/\tau)c[t]$
According to eq. 21, the discrete update should be:
$\bar{c}[t+1]=(1-dt/\tau)\bar{c}[t]+(dt/\tau)c[t]$, which is implemented correctly in AurynVectorFloat::follow, which renders the inner and outer update scheme different. I am wondering if I am missing something here.
Why is the outer filtering updated first (L313) before the inner filtering (L316)? Is this intentional?

Thanks again for your question. Conventionally, I do the double filtering for synaptic traces differently when the inputs are spikes so that the jump for spike arrival in the first filter is always one regardless of the time step. However, the alpha trace does not receive spikes as inputs, so this is unnecessary here. I can only assume that I did it out of habit, but frankly, I do not remember off the top of my head.
In any case, the only thing it changes is the amplitude of the filter kernel, which can be absorbed in the learning rate. Concerning your second question, why is one filtering done first? This choice was probably unintentional, and the two are approximately the same for small time steps (or long enough time constants). Strictly speaking, all updates should be done simultaneously anyway. However, in Auryn, we deliberately do in-place operations for performance reasons.
I will keep your two tickets open until I have time to look into the code again in more detail. But as I already mentioned, it will take me some time. If we confirm discrepancies between the code and our methods, we will issue an erratum.
Thanks heaps again for your careful reading of the code, and let me know if you have any further questions.

Understood. Thank you!


feature/wallet build failure

Here's where I'm up to with this branch:
:app:packageAllBetaDebugClassesForMultiDex FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:packageAllBetaDebugClassesForMultiDex'.

java.util.zip.ZipException: duplicate entry: android/support/v4/content/Loader$OnLoadCompleteListener.class

I had to add support-v4 jar to the project to get it to compile, which might be where this is coming from.


Cannot retrieve the Error property

Version of the custom_component
v1.0.0
Describe the bug
The integration cannot successfully retrieve the Error property/attribute. After including "Error" for "Additional properties to poll" in the configuration UI panel, it shows property_error_error on Submit and will not proceed/save. See screenshot & debug log.
Note: This is with the Epson TW9400 (= 5050UB). The other properties retrieve just fine, and overall the integration seems to work wonderfully. It is however the Error property I'm particularly interested in, specifically "No signal", and for which I installed this integration :-)

Debug log
2024-04-16 09:38:11.839 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _send_request: command="PWR?"
2024-04-16 09:38:11.839 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] connect
2024-04-16 09:38:11.849 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] connect: response="ESC/VP.net " response_bytes_hex=4553432f56502e6e6574100300002000 len=16
2024-04-16 09:38:11.886 INFO (MainThread) [custom_components.epson_projector_link.projector.projector] connect: Connection opened
2024-04-16 09:38:11.887 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _listen: Listening to connection
2024-04-16 09:38:11.889 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _listen: response="PWR=01
:" response_bytes_hex=5057523d30310d3a len=8
2024-04-16 09:38:11.889 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _handle_property: prop=PWR value="01"
2024-04-16 09:38:11.889 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _update_property: prop=PWR, value=on
2024-04-16 09:38:11.889 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _send_request: command="LAMP?"
2024-04-16 09:38:11.893 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _listen: response="LAMP=478
:" response_bytes_hex=4c414d503d3437380d3a len=10
2024-04-16 09:38:11.893 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _handle_property: prop=LAMP value="478"
2024-04-16 09:38:11.893 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _update_property: prop=LAMP, value=478
2024-04-16 09:38:11.893 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _send_request: command="IRIS?"
2024-04-16 09:38:11.896 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _listen: response="IRIS=00
:" response_bytes_hex=495249533d30300d3a len=9
2024-04-16 09:38:11.897 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _handle_property: prop=IRIS value="00"
2024-04-16 09:38:11.897 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _update_property: prop=IRIS, value=Off
2024-04-16 09:38:11.897 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _send_request: command="ERR?"
2024-04-16 09:38:11.900 DEBUG (MainThread) [custom_components.epson_projector_link.projector.projector] _listen: response="ERR
:" response_bytes_hex=4552520d3a len=5
2024-04-16 09:38:11.900 WARNING (MainThread) [custom_components.epson_projector_link.projector.projector] _handle_err: Received error response for command="ERR?"
2024-04-16 09:38:11.900 ERROR (MainThread) [custom_components.epson_projector_link.projector.projector] _send_request: command="ERR?" error=Received error response for command="ERR?"
Traceback (most recent call last):
  File "/config/custom_components/epson_projector_link/projector/projector.py", line 228, in _send_request
    return await request.future
           ^^^^^^^^^^^^^^^^^^^^
custom_components.epson_projector_link.projector.exceptions.ProjectorErrorResponse: Received error response for command="ERR?"

I have the same challenge on my Epson EH-TW9400W.

By the way, there are a few more error codes nowdays, one might add the last few lines into /homeassistant/custom_components/epson_projector_link/projector/const.py:
PROPERTY_ERR_CODE_MAP = {
"00": ERROR_NONE,
"01": "Fan error",
"03": "Lamp failure at power on",
"04": "High internal temperature error",
"06": "Lamp error",
"07": "Open Lamp cover door error",
"08": "Cinema filter error",
"09": "Electric dual-layered capacitor is disconnected",
"0A": "Auto iris error",
"0B": "Subsystem Error",
"0C": "Low air flow error",
"0D": "Air filter air flow sensor error",
"0E": "Power supply unit error (Ballast)",
"0F": "Shutter error",
"10": "Cooling system error (peltiert element)",
"11": "Cooling system error (Pump)",
"12": "Static iris error",
"13": "Power supply unit error (Disagreement of Ballast)",
"14": "Exhaust shutter error",
"15": "Obstacle detection error",
"16": "IF board discernment error",
"17": "Communication error of Stack projection function",
"18": "I2C erro",
"1A": "Lens shift error",
"1B": "Quarts N Polarizer error",
"1C": "No lens error",
"1D": "Subsystem error 2",
"1E": "Power supply voltage error",
"1F": "Other error",
}
Btw. I can not find the "ERR?" command - or I can not read python code maybe is more correct...


fix: do not exclude child methods on alias classes

Addresses #317 by ensuring we include methods defined in a class, even we are documenting an alias of that class.

@has2k1 do you mind taking a look, to see if this fixes the issue for you? (Thanks for raising, it seems like a pretty serious bug).

Yes this fixes the issue. It took a while to confirm as there were confounding issues that are in griffe and have been filed there.

👍 This PR was exactly was I was looking for.
Have been scratching my head for some hours on trying to figure out why methods on my classes were not being rendered.
I can verify that it works for my examples as well. :shipit:

Hi @machow any plans to release a new version soon of quartodoc, I use this feature and would prefer to install a numbered version instead of a git hash? 🙏

Hey, sorry for the wait--this is now released as v0.7.3. I'm planning a round of improvements, so if you've run into any issues in the meantime, definitely let me know :)


Make minor changes to OasGenerator

Added log message
Remove Reset function from OasGenerator interface, use Stop+Start instead
SetEntriesQuery returns a bool stating whether the query changed

I don't see the change from reset to stop in the controller

I don't see the change from reset to stop in the controller

It's in Enterprise.
https://github.com/up9inc/mizu-enterprise/pull/137/commits/29a819e512052c3d982da7fb82f5fdd5b48b51a3


on Windows computers, the upload file service using blob is unstable

Package Name: @azure/storage-blob
Package Version: 12.0.0
Operating system:Windows 10
[ ] nodejs

version: 8.12.0

[ ] browser

name/version: electron

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
We have encountered a problem when we upload file to Azure Blob Storage on Windows  using type of nodejs sdk , we find the nodejs sdk cannot fully occupy  bandwidth with highest speed of 200KB/s ，but on the same network environment，ios sdk can reach 2M/s ，can you give us some advice
To Reproduce
Steps to reproduce the behavior:

const result = await blockBlobClient.upload(data, data.length);
Can't wait for the result to return

Expected behavior
File uploaded successfully

/cc @jeremymeng

@zhangjiayu1993 Thank you for bringing this to our attention. I wonder if it's an issue with Electron specifically. Are you able to test your scenario out using Node on the same machine? I also notice your node version is quite old, The latest Electron release is on Node 16.14.2

/cc @mikeharder who has done a lot of storage perf investigation.

@zhangjiayu1993: Would you be able to provide a small application to reproduce this behavior, plus a description of your test environment?  Here is the configuration and results of our internal perf test.  We are seeing throughput over 500x faster than you reported (128 MBps vs 0.2 MBps), but there are a lot of differences between the two environments.

Client

Operating System: Windows Server 2022
Azure VM Size: D4ds_v5
Accelerated Networking: Enabled
Node Version: 14.19.3

Service

Storage Account Type: Premium BlockBlobStorage
Storage Account Location: Same region as client VM

Package

Package Name: @azure/storage-blob
Package Version: 12.10.0

Test

Blob Size: 20 MB
Throughput: 128 MBps (1028 Mbps)


CI/CD for Google Play Store using Circle CI and Fastlane

Continuous Integration and Deployment of Mobile App should happen using Fastlane with the help of Circle CI.
Changes pushed to master branch should automatically trigger Circle CI pipeline and Incremental build should automatically be deployed to Google Play Store.

@tachyons @Nitish145 Need your opinions on which platform we should deploy, GitHub Actions or Circle CI ?

Can use github actions as we are already using it for CI

@Nitish145 @satu0king any updates about the Fastlane deployment variables being set in GitHub Actions ?

Will push the deployment action soon. Thanks for your PR :) @manjotsidhu


set provenance: false for image builds

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 60.72%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
810

Relevant Lines:
1334

💛  - Coveralls


Mal rating not working for overlay on anime

Version Number
1.18.0
What branch are you on?
master
Describe the Bug
The mal rating is not showing up for overlays. I'm not sure why as the connection to mal says it's successful at startup.
Relevant Collection/Overlay/Playlist Definition
No response
Logs
https://gist.github.com/ShanaryS/8467f18f2bf1e6ff358ee95679fcb7c4

Just did more testing and mal is working, all rating providers do. However they all ONLY work as rating2_image. rating1_image and rating3_image does not show up, only for anime.

youre not actually telling pmm to pull in any ratings. Youre just telling it to overlay with a specific logo from a specific plex attribute. You need to run the corresponding library operations to actually update those ratings to have values Wiki Page


Update rate limiting landing page

this is great @lmilan thank you! Can you add an empty page for kic and mesh and link to those?

this is great @lmilan thank you! Can you add an empty page for kic and mesh and link to those?

Done!


How to upgrade database version or data migration

Like in Android, we just increment the database version and it will be updated. Is there any similar method?

That's generally handled by PRAGMA user_version. Here's an extension:
extension Database {
    public var userVersion: Int {
        get { return Int(scalar("PRAGMA user_version") as! Int64) }
        set { try! run("PRAGMA user_version = \(newValue)") }
    }
}

In use:
if db.userVersion == 0 {
  // run migrations for version 1
  db.userVersion = 1
}
if db.userVersion == 1 {
  // run migrations for version 2
  db.userVersion = 2
}
// etc.

Thanks man! Appreciate it!


Generate per-route redirects in Vercel build output instead of per-page

Describe the problem
In order to get the correct trailing slash redirect behaviour for prerendered pages on Vercel, it's necessary to add an override plus two routes for each page:
{
  "version": 3,
  "routes": [
    // ...
    {
      "src": "/docs/introduction",
      "dest": "/docs/introduction/"
    },
    {
      "src": "/docs/introduction/",
      "status": 308,
      "headers": {
        "Location": "/docs/introduction"
      }
    },
    {
      "src": "/docs/configuration",
      "dest": "/docs/configuration/"
    },
    {
      "src": "/docs/configuration/",
      "status": 308,
      "headers": {
        "Location": "/docs/configuration"
      }
    },
    // ...
  ],
  "overrides": {
    // ...
    "docs/introduction.html": {
      "path": "docs/introduction"
    },
    "docs/configuration.html": {
      "path": "docs/configuration"
    },
    // ...
  }
}

Since there's a limit to how many routes you can have in a project, this means that there's also a (quite low) limit on the number of prerendered pages.
Describe the proposed solution
We should be able to create a pair of Vercel redirect routes for each SvelteKit route, instead of for each prerendered page:
{
  "version": 3,
  "routes": [
    // ...
    {
      "src": "/docs/([^/]+)",
      "dest": "/docs/$1/"
    },
    {
      "src": "/docs/([^/]+)/",
      "status": 308,
      "headers": {
        "Location": "/docs/$1"
      }
    },
    // ...
  ],
  "overrides": {
    // ...
    "docs/introduction.html": {
      "path": "docs/introduction"
    },
    "docs/configuration.html": {
      "path": "docs/configuration"
    },
    // ...
  }
}

(I actually can't recall why the first of those is necessary in the trailingSlash: 'never' case, need to investigate.)
The challenge will be to do this while respecting edge cases like this...
src/routes/[a=b]/+page.svelte
src/routes/[c]/+page.svelte

...where the two routes have different trailingSlash config and /[a=b] is a prerendered route that shouldn't prevent /[c] from matching.
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

How long can a src regex become? We could do /path/(possibility1|possibility2|..)
{
  "version": 3,
  "routes": [
    // ...
    {
      "src": "/docs/(introduction|configuration)",
      "dest": "/docs/$1/"
    },
    {
      "src": "/docs/(introduction|configuration)/",
      "status": 308,
      "headers": {
        "Location": "/docs/$1"
      }
    },
    // ...
  ],
  "overrides": {
    // ...
    "docs/introduction.html": {
      "path": "docs/introduction"
    },
    "docs/configuration.html": {
      "path": "docs/configuration"
    },
    // ...
  }
}

Not sure if this has any implications on performance when the regex becomes too big or if there are nested possibilities.

Facing the same issue.
Does this mean that we basically can't deploy projects with a large number of pre-rendered pages to Vercel?
Like the previous commenter, it looks like setting 'trailingSlash' doesn't really do anything and you always end up with a very large config.json file with all the redirects..

I ran into this issue as well. Is there a known workaround?


Replace java.sql.Timestamp with java.time.LocalDateTime

In case the host machine has a timezone other than UTC, parquet/FieldValueSpec tests fail.
GitHub Actions using UTC by default shadows this.
This happens due to java.sql.Timestamp depending on host machine timezone setting implicitly.
Not only toString but also other conversion methods (e.g. toLocalDateTime) on java.sql.Timestamp carry this issue.
java.time.LocalDateTime is a newer (java 8+) alternative which doesn't depend on the host timezone for any conversion incl. toString
User facing change will be the replacement of empty char   with T between date and time
java.sql.Timestamp: 2022-02-02 04:02:03.123
java.time.LocalDateTime: 2022-02-02T01:02:03.123

I agree that java.sql.Timestamp was not a good choice!
Now, though, the only class that uses it is this one and I deprecated it as of the 0.21.0 release.
Going forwards, we will use this caster for casting to parquet, and it uses java.time.Instant for timestamps.

thanks @istreeter ! closing this one


Getting the existing symbols could be easier

Right now we iterate through all the pages and then all the layers to get the symbol masters.
There is a convenient method on MSDocumentData called allSymbols and we should just use it.

It's actually not that simple: injectSymbols reset the Symbols page and render all the symbols that we found. So if we use allSymbols, it will bring the symbols from other pages as well.

It would be great if we could retrieve all of the available symbols into an array. Currently in v2 I couldn't find a way to achieve this.


Pi Boot Up Time

45 seconds on a Pi Zero is much longer than the near instant bootup of the ESP32 version. While complete match in startup performance should not be expected, minimizing this disparity via scripts or a custom light image would be a great improvement.

Ahoy, what's your boot time on ESP32 ?
Nice work, actually modding a footswitch to integrate a Pi0W with tinderBox !
Cheers from France 🇫🇷
Julien


Need Date Picker

https://www.google.com/design/spec/components/pickers.html#pickers-date-pickers
Fitness app wants to be able to enter dates.

Fitness app has a date picker now. It seems to be having issues on iOS; I'll file a separate issue for that.


Support create Dir and listBucket add Dir info

Fix bugs with WinSCP as S3 Client:

Can not create Dir.
Can not list files with empty Dir.

Tested with WinSCP + Alist S3 + AliyunOpen Backend.
Please review / merge it, and make a new release v0.0.6

I will create PR of alist-org/alist when it got merged, depending on the new alist-org/gofakes3:v0.0.6 release.

@akang943578 You'd better add your git commit e-mail to Github or your commit won't be properly authored.

Please don't submit the PR in alist/alist and hold it for a while. 😊


Add documentation for use in IDEs.

Please tell us about your environment:

winston version?

[ ] winston@2
[x] winston@3

node -v outputs: v10.5.0
Operating System? Windows
Language? ES6/7

What's the feature?
Include documentation on common issues when using winston in an IDE (not from the command line).
Why this should be added.
I use Visual Studio Code. While trying to work with winston on one of my projects, I was having a hard time getting anything to print to the terminal. I had no idea why this was not working, even when using the example code directly.
After some time, I realized that the issue may be due to VS Code rather than the library. I finally found what I was looking for, I had to add an additional option to the standard launch configuration to get winston to work. Namely, "outputCapture": "std" (https://github.com/Microsoft/vscode/issues/34702).
It would save first time users a lot of time if there was a section in the documentation which mentions setup in popular IDEs. Thanks.

This seems like a good idea to me!  Would you like to open a little PR that has a few sentences on how to do this for VS Code? (What you mentioned)


Download PDF option on the left side does not work

Empty doc (0 bytes) is generated. Works fine for other services within Azure portfolio, kindly fix it

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

ID: 33a7931f-82f5-dde3-3d8d-63e6d88dfd4a
Version Independent ID: 38bc1524-942b-5a49-d680-81368f3f8ef7
Content: Secure your bot - Bot Service
Content Source: articles/dotnet/bot-builder-dotnet-security.md
Product: bot-framework
GitHub Login: @RobStand
Microsoft Alias: kamrani

@mikekinsman - could you take a look at this issue.

@ignaciofls - this issue has been resolved. Thanks for reporting.


Add BitMidi to the readme

Add my new website BitMidi to the readme. It uses Tachyons!

This is awesome, thanks!


mini.convert plugin to handle conversion of colors between hsv, rgb and hex, conversion of sizes, etc

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.convert
Description
Hi again! Is something like https://github.com/cjodo/convert.nvim a valid feature request which can be incorporated as part of the mini.nvim suite.

Thanks for the suggestion!
Although I can see how this can be useful for some day-to-day work, this doesn't look like a necessary feature for a good all-round editing experience.
On the other hand, there might be a good place for it in the planned 'mini.cycle' (or similar name). I'll try to think about it during that module development (which does not yet have high priority, I am afraid).
Closing as "maybe planned as part of another module".


Make search globally accessible?

Would it be possible to have the search button, or a shortcut key to search, always accessible in the interface?
It is not very obvious for perusing casual users to first have to go to Home, and then find the looking glass button in the music sources header bar on the right - i'd say Search deserves an entry of its own in the top level menu maybe? Just some thoughts on enhancing an already brilliant interface to LMS, thank you!

This has been requested before, but I will not be adding. If I the albums list open I would expect to only search in albums, if I was listing an album of many tracks then search should be limited to them, etc. LMS does not, AFAIK, support that - hence its placement in the home screen.


Notifications should be disable-able via package config

Beta tester mentioned that these are a bit annoying

closed in https://github.com/learn-co/learn-ide/commit/0b64df0f4ae7bb2dce650f58ff560c08bebf3cf7 and https://github.com/learn-co/learn-ide/commit/236ac226014bc765f0362ff23aa2ca393ca825f6


Plan for low resource devices

Looks like a tentative plan is still to use jsdom on low resource devices. Getting Node.js to run on those devices looks hard, but if we can compile jsdom to C, we should hopefully be able to run it straight on those devices. This project looks promising for compiling JS to C: https://github.com/andrei-markeev/ts2c/blob/master/README.md
The project is planning on having 95% of ES3 capabilities by the end of the year. We could go from jsdom, which is probably written in ES5, to ES3, and then to C. I'm not sure if jsdom uses any node APIs, we would have to somehow port those over. That looks like a good path. This means we still need to implement custom elements in jsdom. That seems like a solid solution to eventually getting web components to work on all devices.
Now, another possibility is to wait for a good web assembly interpreter that can run on low resource devices. Then we could compile jsdom to wasm, and run it right on the device like that. We'll have to watch the landscape to see how low resource wasm code can run.
So, the conclusion is, implementing custom elements in jsdom looks like the next best step to getting web components onto devices that cannot run Electron or Node.js. In order of resources needed: Electron -> Node.js -> wasm -> C. JSDOM should work for all devices in this spectrum.

Instead of compiling jsdom to C, what if we compiled the HTML and DOM implementations from Chromium to wasm? For more info: https://github.com/tmpvar/jsdom/issues/1772

Looks like we can move forward with jsdom. Take a look at the end of the comments here: https://github.com/tmpvar/jsdom/issues/1030#issuecomment-297089279
We should be able to use this library to get jsdom to work. We'll have to hack around again so that Node is available to the code
https://github.com/WebReflection/document-register-element


Add a Strict mode to Style/MutableConstants

This has been something that I have wanted to do for a while. Right now, we only freeze literals that are assigned to constants. This leaves many constants unfrozen and therefore potentially unsafe.
I consider this to be experimental functionality. I'm not sure how many other people would be interested in this feature. I think I have all of the main use cases covered. I am trying to account for any edge cases now.
Something interesting that I learned from this is that concatenating frozen strings produces an unfrozen string.

@bbatsov this is ready for review.

Thanks for the feedback. I'll try to setup some rules for the comments that you made.

@bbatsov this is ready for review again. I added some rules to Style/RedundantFreeze in addition to the other changes.

I removed strict mode as the default setting for RuboCop, and reverted the changes made by that. This is ready for review again.

Several reviews by @Drenmi are left, but dc3631d in other points looks good to me 🌟
I have a question. Is 6febf1a different commit from dc3631d? If so, if you open 6febf1a as a different PR, dc3631d would like to merge first if the following has been updated.

https://github.com/rubocop-hq/rubocop/pull/6126#discussion_r255288373
https://github.com/rubocop-hq/rubocop/pull/6126#discussion_r255288419

6febf1a is different from dc3631d. I can break these out into separate PRs.

This looks good! Thanks @rrosenblum.


Skip response validation if BaseModel provided

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
Or, I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Problem
FastAPI force validation in serialize_response even when validated model provided, by calling
https://github.com/tiangolo/fastapi/blob/dc5a966548df973d0fd2d3a28bad65d0436f6fb9/fastapi/routing.py#L110
and then we convert BaseModel again to dict
https://github.com/tiangolo/fastapi/blob/dc5a966548df973d0fd2d3a28bad65d0436f6fb9/fastapi/routing.py#L66
which will be validated in next steps insideserialize_response
Twice model validation looks like wasting resources.
Also Pydantic allow creating model without validation: https://pydantic-docs.helpmanual.io/usage/models/#creating-models-without-validation and invalid models can't be returned as response (may be this ok, idk)
Expected behaviour
When i return list/dict it will be validated trough response_model
When i return response_model instance no validation applied
Example
from typing import Dict
from typing import Literal

import uvicorn
from pydantic import BaseModel
from fastapi import FastAPI

app = FastAPI()

class MyModel(BaseModel):
    msg: Dict[Literal['Hello'], Literal['World']]

@app.get("/revalidate", response_model=MyModel)
def revalidate():
    return {'msg': {"Hello": "World"}}

@app.get("/no_revalidate", response_model=MyModel)
def no_revalidate():
    return MyModel(msg={"Hello": "World"})

if __name__ == '__main__':
    uvicorn.run(app)

There's this PR that I think would resolve your issue: #2347

Actually not, cause it's just use existing mechanic with custom Response
I can do that manually. But i need wrap my result model with ModelResponse each time, for all methods as suggested in #2347
I think we can do that more transparently
We also use fastapi-jsonrpc, and fix in basic serialization mechanic will automatically improve performance in json-rpc serialization

To be clarify we explicit wraps result to Response Schema Model. This allow us use type-hinting to ensure all things are right ;)

This also correlates with https://github.com/tiangolo/fastapi/issues/2296
If we will use return type annotation instead response_model we should return prepared model instead ModelResponse (as suggested in #2347)


Incorrect spelling of video ("vedio")

Incorrect spellings of video ("vedio") found in your .mk. Thanks.

Incorrect spellings of video ("vedio") found in your .mk. Thanks.

thanks


[zh] sync /workloads/pods/pod-lifecycle.md

Sync with en #41297
content/zh-cn/docs/concepts/workloads/pods/pod-lifecycle.md

See preview please.

/lgtm

/approve


grabValueFrom - everytime string

Why grabValueFrom everytime return value as string?
For example, input can contains numbers, etc.
Testers who use typescript, will be like checked types in inputs...

It's all about specifications.
For instance in Webdriver, grabValueFrom method uses the getAttribute method which returns the null or string.
In Playwright grabValueFrom method uses the getProperty and the jsonValue and the last one returns a JSON representation of the object. If the object has a toJSON function, it will not be called.

It's all about specifications.
For instance in Webdriver, grabValueFrom method uses the getAttribute method which returns the null or string.
In Playwright grabValueFrom method uses the getProperty and the jsonValue and the last one returns a JSON representation of the object. If the object has a toJSON function, it will not be called.


Quoted "<" breaks diff oO

Using version 1.18.0: http://jsfiddle.net/yptmch4k/1/
When I compare the string c=< against c="<" using strictEqual the diff breaks, is it intended?

For the record, no, that's not intended. Looking for help to get this fixed.


GH932: Added MD5 checking for packages.config to bootstrapper

Added MD5 checking for packages.config to bootstrapper to ensure correct version of Cake is used.
@patriksvensson This is what we talked about in #922
Not tested on OS X, md5sum is not standard on OS X, but md5 is and by adding option -r it should print sum same way as Linux variant.
I made the checking/content of the hash file to be identical between Windows/Linux in order to use same file on both (tested with Bash for Windows)...

Relates to #932

@devlead @patriksvensson this looks good to me.  Is one of you able to verify the running of this on something other?
Also, assuming that we pull this into the Cake repo, should we also update the resources repository version of the bootstrapper's to also include this?

@mholo65 we are bumping this from the 0.13.0 release, as there are some other discussions happening that will effect this.
Will keep you posted on what the outcome is.
Thanks!

@mholo65 I have taken the liberty to rebase and merge this PR here: https://github.com/cake-build/cake/commit/de08d3144b7906335f0d65b1ab5975f8c9805ffc As a result, I will have to manually close this PR, however, your work has been credited.  Thanks for your contribution to Cake!

@gep13 glad to hear this got merged :smile: just curious, did anyone have time to test it on Mac?

@mholo65 said...
just curious, did anyone have time to test it on Mac?

"Seat of the pants" sort of thing here.  This is now running on the main Cake build, on both Bitrise and Travis, so we will know pretty quickly if something isn't working :-)


add osx_arm64 build

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

This is ready for review @conda-forge/urwid.

Another gentle ping for review  @conda-forge/urwid.


Hmusum/order zookeeper server list so localhost comes first

@hakonhall FYI

@bratseth FYI

the order in the connectstring doesn't matter. The list is shuffled by the ZooKeeper client.

Not in the code we looked at. At cfg3 was also always on top.


tool - resample ITCH to BBO file

one of my most important applications to decent ITCH parser would be resampling data files into BBO (only best bid price, total size, # of orders, best ask, total size, # of orders) format. plus trades.
the output format would be simple fixed offsets with timestamp, current bid & ask, order book id in case of new quote. and timestamp, price, aggressor, size of a trade.

helix-trace works directly on files so you can use that for this. There's even a Python script to plot the sample data.

thanks

but helix-trace reads the data from network, would be great to be able to read ITCH from files instead of multicast groups. there is no need to do tcpreplay or alike

@divaykin Aah, right, I only implemented it for helix-svm in commit d3c06055b35fd730fb15e88fc160126b6c627b49. The capability is there, we just need to make helix-trace use it.

@divaykin Take a look at NASDAQ Market Data Tools as well.

thanks @jvirtanen will do


fix: principal install missing argocd-agent-principal-userpass secret

I am following the quickstart and after the step kubectl apply -n argocd -k install/kubernetes/principal, the principal agent pod is not coming up.
The error was:
MountVolume.SetUp failed for volume "userpass-passwd" : secret "argocd-agent-principal-userpass" not found

This fix is to ensure the kustomization.yaml properly contains the required secret for install.

Thanks! However, I believe this is a documentation issue.
IMHO, the secret should not be installed automatically, but users should create the secret manually - so the manifests can be used out of the box even if you use a secret manager to create/manage that secret.
So from my PoV, the real fix would be to add the manual step to the docs :)


Missing closing XML tag

Configuration file template cookbooks/bcpc-hadoop/templates/default/hdp_ssl-server.xml.erb is missing closing </description> tag.

Fixed in #117


Failed to compile spin on Mac (Intel chip)

Today I found it suddenly compiling the main branch gives errors:
   Compiling socket2 v0.4.4
   Compiling matches v0.1.9
   Compiling scoped-tls v1.0.0
   Compiling version_check v0.1.5
   Compiling half v1.8.2
   Compiling iovec v0.1.4
   Compiling percent-encoding v1.0.1
   Compiling unicode-bidi v0.3.7
   Compiling percent-encoding v2.1.0
   Compiling itoa v1.0.1
   Compiling sha1_smol v1.0.0
   Compiling rustls v0.20.4
error[E0583]: file not found for module `sys`
   --> /Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/lib.rs:124:1
    |
124 | mod sys;
    | ^^^^^^^^
    |
    = help: to create the module `sys`, create file "/Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/sys.rs" or "/Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/sys/mod.rs"

error: Socket2 doesn't support the compile target
   --> /Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/lib.rs:127:1
    |
127 | compile_error!("Socket2 doesn't support the compile target");
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

1780 | from!(Socket, net::TcpListener);
     |               ^^^^^^^^^^^^^^^^ expected struct `TcpListener`, found `()`
     |
    ::: /Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/lib.rs:104:16
     |
104  |             fn from(socket: $from) -> $for {
     |                ---- implicitly returns `()` as its body has no tail or `return` expression

error[E0308]: mismatched types
    --> /Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/socket.rs:1781:15
     |
1781 | from!(Socket, net::UdpSocket);
     |               ^^^^^^^^^^^^^^ expected struct `UdpSocket`, found `()`
     |
    ::: /Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/lib.rs:104:16
     |
104  |             fn from(socket: $from) -> $for {
     |                ---- implicitly returns `()` as its body has no tail or `return` expression

Some errors have detailed explanations: E0061, E0308, E0412, E0422, E0425, E0432, E0433, E0583, E0599.
For more information about an error, try `rustc --explain E0061`.
error: could not compile `socket2` due to 185 previous errors
warning: build failed, waiting for other jobs to finish...
error: build failed
make: *** [build] Error 101

There are 185 errors so I am not going to show all of them, but it seems like it's related to socket2.
My environment:
Default host: x86_64-apple-darwin
rustup home:  /Users/mossaka/.rustup

installed targets for active toolchain
--------------------------------------

wasm32-unknown-unknown
wasm32-wasi
x86_64-apple-darwin

active toolchain
----------------

stable-x86_64-apple-darwin (default)
rustc 1.60.0 (7737e0b5c 2022-04-04)

I am not sure if this is an OS thing, but since I am using a Intel-based Mac to compile it, I suspect that's one of the potential issues so I put it in the title.

Sounds like this is a recent thing - do you know offhand what the previous commit you had (that did compile) was?
(If you have the enthusiasm to do a full bisect that would be super helpful - I am not sure any of the maintainers have Intel Macs to test on.)

I was building spin in from previous commits but they all failed.
Out of hundres errors, this one is getting my attention but I have no clue why
error: Socket2 doesn't support the compile target
   --> /Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/lib.rs:127:1
    |
127 | compile_error!("Socket2 doesn't support the compile target");
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Could you share the full command you used to build, please?
That looks like perhaps you are trying to compile the Spin CLI using a Wasm toolchain?

I was just doing make build
(base) ➜  spin git:(main) ✗ make build
cargo build --release
   Compiling socket2 v0.4.4
   Compiling rustc-demangle v0.1.21
   Compiling wasmtime-cache v0.34.1
   Compiling dirs-sys-next v0.1.2
   Compiling leb128 v0.2.5
   Compiling mime v0.3.16
   Compiling untrusted v0.7.1
   Compiling wasm-bindgen v0.2.79
   Compiling wasmtime v0.34.1
   Compiling paste v1.0.6
   Compiling io-extras v0.13.2
   Compiling pulldown-cmark v0.8.0
error[E0583]: file not found for module `sys`
   --> /Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/lib.rs:124:1
    |
124 | mod sys;
    | ^^^^^^^^
    |
    = help: to create the module `sys`, create file "/Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/sys.rs" or "/Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/sys/mod.rs"

error: Socket2 doesn't support the compile target
   --> /Users/mossaka/.cargo/registry/src/github.com-1ecc6299db9ec823/socket2-0.4.4/src/lib.rs:127:1
    |
127 | compile_error!("Socket2 doesn't support the compile target");
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The attribute on that line is #[cfg(not(any(windows, unix)))] which does feel like it could be a Wasm target somehow...

Btw, weirdly enough... spin does not build on my Linux devbox too, with a different error (socket2 seems to compile fine)
➜  spin git:(main) make build
cargo build --release
   Compiling spin-templates v0.1.0 (/home/mossaka/developer/spin/crates/templates)
   Compiling wasmtime v0.34.1
   Compiling spin-cli v0.1.0 (/home/mossaka/developer/spin)
error: failed to run custom build command for `spin-cli v0.1.0 (/home/mossaka/developer/spin)`

Caused by:
  process didn't exit successfully: `/home/mossaka/developer/spin/target/release/build/spin-cli-122fa54863fdf2d4/build-script-build` (exit status: 127)
  --- stderr
  /home/mossaka/developer/spin/target/release/build/spin-cli-122fa54863fdf2d4/build-script-build: error while loading shared libraries: libgit2.so.1.4: cannot open shared object file: No such file or directory
warning: build failed, waiting for other jobs to finish...
error: build failed
make: *** [Makefile:6: build] Error 101

Btw, weirdly enough... spin does not build on my Linux devbox too, with a different error (socket2 seems to compile fine)

Okay I was able to resolve Linux build issue by

brew install libgit2
running ldconfig to dynamic link runtime bindings for libgit2.o

The attribute on that line is #[cfg(not(any(windows, unix)))] which does feel like it could be a Wasm target somehow...

Found out the issue!!

I had this issue because I tried spin new with the default rust-hello template in my developer directory, which is a parent directory of all my rust projects... and that template creates a .cargo folder :(

Closing this since issue got resolved.


Multiple definitions in HLIL SSA

Version and Platform (required):

Binary Ninja Version: 2.5.3140-dev
OS: macOS
OS Version: 11.3

Bug Description:
HLIL SSA form contains variables with multiple definitions for the same variable version.

I found this bug while walking def-use chains... and unexpectedly found a cycle. I went to look at the HLIL and saw this wacky output. I do not have much experience with SSA form so if this is intended behavior I'm sorry and feel free to close.
Steps To Reproduce:

Open the attached libc - note it's just from some random ctf challenge and was probably compiled from source (ie. it doesn't match any from common distros)
Switch to HLIL SSA form
Go to 0x73a2e

Expected Behavior:
SSA form would have a single definition for each variable version
Screenshots:
above
Additional Information:
In addition to the multiple definitions, I’m not sure the handling of writes to subsets of registers is really right:
rcx_2#0.b = /* something */

being treated as just a use is really correct? Seems misleading at best… then again, I don’t really have much experience with SSA form.

I think this is blocked on #2809 being fixed.

Pretty sure @rssor meant #2823 instead.


[5pt] Replace Fiona with pyogrio

Placeholder for a future issue: "GeoPandas 1.0 will switch to use pyogrio as the default engine, since pyogrio can provide a significant speedup compared to Fiona." We may want to evaluate pyogrio sooner for potential performance gains.
From the geopandas docs (e.g., https://geopandas.org/en/stable/docs/user_guide/io.html#reading-spatial-data):

GeoPandas currently defaults to use Fiona as the engine in read_file. However, GeoPandas 1.0 will switch to use pyogrio as the default engine, since pyogrio can provide a significant speedup compared to Fiona. We recommend to already install pyogrio and specify the engine by using the engine keyword (geopandas.read_file(..., engine="pyogrio")), or by setting the default for the engine keyword globally with:
geopandas.options.io_engine = "pyogrio"

pyogrio claims "We have seen >5-10x speedups reading files and >5-20x speedups writing files compared to using non-vectorized approaches (Fiona and current I/O support in GeoPandas)." https://github.com/geopandas/pyogrio?tab=readme-ov-file

Additionally, installing pyarrow in combination with GDAL 3.6+ enables a further speed-up when specifying use_arrow=True. https://pyogrio.readthedocs.io/en/latest/install.html


Allow specifying GET request parameters using a dictionary

Added "sugar" function that allows specifying GET request parameters in a dictionary.

Not sure how I feel about this but I will look into this or something that allows for the same feature in a different way
Keep track at https://github.com/hallas/agent/pull/26

Not particularly happy with this, will look at doing this another way, however I feel that this can be added before using Agent.


added new responsive footer

@devjainofficial I checked everything, and the footer works fine.

I found it a bit confusing when I compared my new pull request to your latest commit, as there were no changes mentioned (like there was change in my branch code and your main branch but not showing in compare). I spent about an hour trying to resolve the pull request issue. However, during this process, I learned many things, such as understanding the concepts of "upstream" and "origin."
@devjainofficial Please verify it and merge the changes, and kindly add the tag of hacktoberfest.


Add Windows build to CI

This PR adds a basic Windows build to CI (GitHub actions). Currently it's failing due to a missing library for floats:
java.lang.UnsatisfiedLinkError at Z3SolverFPTest.java:45

But if we fix that and rebase this PR, I think it should work. But let's wait for that fix first.

See #131

Update: we skip float tests on Windows for now, until we can somehow port the required library to Windows.