run cargo update

Codecov Report
Patch coverage: 82.85% and project coverage change: +0.81% :tada:

Comparison is base (92667ac) 59.81% compared to head (ecdb319) 60.62%.
Report is 25 commits behind head on dev.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #487      +/-   ##
==========================================
+ Coverage   59.81%   60.62%   +0.81%
==========================================
  Files          33       33
  Lines        5412     5422      +10
==========================================
+ Hits         3237     3287      +50
+ Misses       2175     2135      -40

Files Changed
Coverage Δ

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

zingolib/src/wallet.rs
66.56% <0.00%> (+2.00%)
:arrow_up:

zingolib/src/lightclient.rs
64.53% <66.66%> (+1.60%)
:arrow_up:

zingo-testutils/src/lib.rs
88.94% <92.00%> (+4.86%)
:arrow_up:

... and 6 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@fluidvanadium this can merge once you review it.


Updated README File

Changes
Added the feature of json file being generated along with printing search results along with the new search command.
Desired Reviewers
@AdiChat

Looks great 😍
Hope you are enjoying your journey 🚆  with OpenGenus


Issue fixes

Fixes #178, #195, #196, #197, #198, #199, #200
Director shot collision now updates appropriately
Camera shots are not linked anymore when created together
On application start, the changed boolean is now correctly set to false
Collisions are updated appropriately when creating camera shots via DirectorDetailView
Director shots are not deleted anymore when deleting a camera shot via detail view
You can now create director shots with 0 camera shots
Text in a text field is auto selected on focus.

Looks fine to me!

Looks good. Second member can merge.


types: new seccomp format

engine-api changes related to https://github.com/docker/docker/pull/24510
/cc @justincormack
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

/cc @justincormak

Quick question, otherwise LGTM 🐸

LGTM


DBAPI cursor receiving 2 arguments instead of 1

What happened?
SQLAlchemy==1.4.45
Called connection.execute with two arguments: statement=TextClause, parameters=[{...}]. This resulted in the StatementError described in the Relevant log output. The relevant code where the error occurs is below:
# base.py/_ConnectionFairy/cursor line 1111
def cursor(self, *args, **kwargs):
        """Return a new DBAPI cursor for the underlying connection.

        This method is a proxy for the ``connection.cursor()`` DBAPI
        method.

        """
        return self.dbapi_connection.cursor(*args, **kwargs)

When I inspect args, I see what looks like a memory address: something like ('c_7fbb330ce780_1',)
DuckDB Engine Version
duckdb-engine==0.6.5
DuckDB Version
duckdb==0.6.1
Relevant log output
Exception has occurred: StatementError
(builtins.TypeError) cursor() takes 1 positional argument but 2 were given
[SQL: INSERT INTO schema.table
(id, created, updated, ...)
VALUES (?, ?, ?, ...)]
[parameters: [{'id': '1', 'created': '2006-07-24T12:12:46Z', 'updated': '2022-09-23T15:54:17Z', ...}]]
File "/workspaces/idw/target-duckdb/.venv/lib/python3.8/site-packages/sqlalchemy/engine/base.py", line 1800, in _execute_context
    context = constructor(
  File "/workspaces/idw/target-duckdb/.venv/lib/python3.8/site-packages/sqlalchemy/engine/default.py", line 1020, in _init_compiled
    self.cursor = self.create_cursor()
  File "/workspaces/idw/target-duckdb/.venv/lib/python3.8/site-packages/sqlalchemy/engine/default.py", line 1388, in create_cursor
    return self.create_server_side_cursor()
  File "/workspaces/idw/target-duckdb/.venv/lib/python3.8/site-packages/sqlalchemy/dialects/postgresql/psycopg2.py", line 636, in create_server_side_cursor
    return self._dbapi_connection.cursor(ident)
  File "/workspaces/idw/target-duckdb/.venv/lib/python3.8/site-packages/sqlalchemy/pool/base.py", line 1111, in cursor
    return self.dbapi_connection.cursor(*args, **kwargs)

Code of Conduct

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

I'm not quite sure how you've managed to trigger it, but it looks like it's breaking on trying to use server side cursors (which DuckDB doesn't support). I've put up a PR to disable them for now, but some sample code that triggers your issue would be appreciated 😁

@Mause Thank you for the speedy response. It looks like this one is user error (as expected!). I'm using Meltano's sql connector from their singer-sdk. In the create_sqlalchemy_connection function they specifically tell users to override that method if the target database doesn't support server side cursors. Thanks again for the help!

@Jamiel-Carter this has now been released in v0.6.6 - can you confirm it works as expected? :grinning:


Introduce useImpulse<number>() and Impulse.of<number>() alias

The idea is to extend the signature of useImpulse by optional values, so useImpulse<number>() is equal to useImpulse<undefined | number>(undefined), the same way as React.useState does

Done in #426


Studio does not allow to move a component to a created folder

Rekit studio allows to create a new folder or its subfolders, but it does not allow to move a component to a newly created folder.

Thanks @legiaquangvn ! Currently Rekit doesn't support sub-folders for components or actions. This duplicates https://github.com/supnate/rekit/issues/43


Add "Additional Options" link to Add Service page

Add an "Additional Option Link" that will show the tabs when clicked.

Date - 11/13
Verified added


props function repeatedly firing under specific circumstances

Version
3.0.1
Reproduction link
https://codepen.io/anon/pen/ePrqVe?editors=1010
Steps to reproduce
Open up the codepen.
Click Foo Link 1 to navigate to that route.
Scroll. Notice the console.log fires repeatedly during scroll.
What is expected?
vue-router props function would only fire once on route change
What is actually happening?
vue-router props function fires repeatedly

The problem goes away if you remove the line from the template

it only appears once

I've made a simpler repro:
https://codepen.io/anon/pen/mzKbMr?editors=1010
The problem is definitely there. The callback is being fired basically whenever any of the parent reactive data values change. It doesn't seem to me that it should?

@posva Can you test again?

I did test but forgot to leave a message hehe. It renders the router-view again because it's a functional component, therefore casting the props function again. You can always wrap it in a stageful component but the props, when defined as a function should be relatively simple not to be a perf problem


Multicast listener choses wrong interface on multi-homed system

Hi,
there is currently an issue with a software called the "Nanoleaf adaptor for ioBroker" that uses this library for handling UDP multicasts. Unfortunately, here on my system it didn't work as expected, and I started debugging. I discovered that there is a bug in node-upnp-ssdp that I'm going to describe here in the following.
At first, you can read the whole story with a detailed analysis of this issue in this bug report: https://github.com/daniel-2k/ioBroker.nanoleaf-lightpanels/issues/16.
Problem:
Your code binds a multicast listener to the first local IP address it finds, obtained via a call to ip.address(). On my system, I have two local interfaces: enp1s0 in direction to a PPPoE-driven ADSL modem, and br0 being my bridge towards my local LAN. In my setup, a call to ip.address() reveals the local IP address of enp1s0 instead of that of br0. This causes multicast communication to fail.
Furthermore, if i consider having multiple internal LAN segments (one for PCs, one for WLAN, one for gadgets...) this approach would also fail completely.
Solution:
The "addMembership" call should omit any IP address. Then the listener may bind to every suitable interface instead of just to a "random first local". By doing that, the interface selection ("ip mroute show") is performed on the basis of the current routing table. This is the expected behavior of an application dealing with UDP multicasts.
Fix:
The fix is trivial: please just remove the call to ip.address() while adding the multicast membership:
powerstation node-upnp-ssdp # diff index.js index.js~
86c86
<     udpServer.addMembership(BROADCAST_ADDR);
---
>     udpServer.addMembership(BROADCAST_ADDR, ip.address());
powerstation node-upnp-ssdp #

This fixes any of the observed misbehaviors of the referred ioBroker adaptor software and would also enable such software to run in setups having multiple internal LANs.
Thanks in advance for fixing this :-)
Florian

Hi,
I saw that you added that behavior explicitly with commit https://github.com/bazwilliams/node-ssdp/commit/569e35d4ab3b343baa535b127fcfa771a2258f63. You mentioned that you had this issue:

Seems to cause issues with receiving messages when multiple interfaces are available (VM).

Can you please give some information what this problem was about? Maybe the correct fix is NOT to bind to an (arbitrary) interface. Maybe just a dedicated route for multicasts was missing... or something like that.
I want to help to get this issue fixed. Currently I see no reason to keep the current behavior as it definitly causes problems especially on systems with multiple interfaces. How can we iron that out?
Regards,
Florian


[FEATURE] Add scroll to top button for better user experience

Hi @vijayyprajapati,
I visited the website, and the geeky UI is really really impressive, but I saw that a "scroll to top button" is missing here. I think if we add a scroll to top button it will enhance the user experience. I can add a sticky scroll to top button, matching to the current design. I am very curious to do that!
May you please assign it to me?

 I will create a PR by today itself.

hey @vijayyprajapati can you assign this issue to me i can work on it

Hey @Ishikasaraf20  this issue already has been assigned
You can add your profile and bookmark links and raise a PR.

hey @vijayyprajapati can you assign this issue to me i can work on it

@Ishikasaraf20 This issue was generated by me and also assigned to me. I have started working on it and will create a PR very soon. I think you are new to open source, analyze the project carefully, create relevant issues by yourself and work on that. I hope you understand.

Go ahead @heyaadarsh

Hi @vijayyprajapati
I have added "Scroll to Top Button" and created a PR #214, please check it out and merge it into this wonderful repo and if you have any suggestions then I'm ready to work on that.
Thank You!


ci(mergify): do not create draft PRs if not needed

Motivation
We included Merge Freeze (https://www.mergefreeze.com/) as a tool to halt PRs when a critical path has to be accomplished, a release for example, and we don't want other PRs to get merged as this would invalidate part of the work being done for this critical path.
But we also have Mergify, which creates drafts PRs before merging (as we did not want all PRs being co-authored by Mergify), but those PRs also get freezed as those are not part of the exception rule, and are dynamically created.
Fixes #4137
Fixes Partially #3360
Closes #4110 (caused by a temporal mergify error)
Partially reverts: https://github.com/ZcashFoundation/zebra/commit/6fafd1af57ea806c67b03d1b56161c6ead4693ed
Solution

Fallback to use allow_inplace_checks

Review
Anyone from @ZcashFoundation/devops-reviewers can review this
Follow Up Work

Confirm everyone has logged in in the Mergify dashboards

Mergify configurations changes are not automatically merged.


test issue

Description:
Summarize your issue or feedback
If it's a feature request: what do you want added? why?
If it's a bug report: what did happen? what should have happened? can you list the steps to reproduce the bug?
Browsers or devices observed on:
Version numbers are helpful whenever possible!

test comment


updating package lock

self explanatory - should fix vulnerability issue

I know it's for the other repo - https://github.com/Azure/platform-chaos-cli/issues/13 the ajv-keywords package wants 6.0.0+ - Is there the possibility of a similar need here?
Also agreed on Ally's point.


Example with correct TS lib

How to configure TS? When creating <div css={{}} /> I get an error:
Type '{ children: Element[]; css: { padding: string; }; }' is not assignable to type 'DetailedHTMLProps<HTMLAttributes<HTMLDivElement>, HTMLDivElement>'.   Property 'css' does not exist on type 'DetailedHTMLProps<HTMLAttributes<HTMLDivElement>, HTMLDivElement>'.
I tried for example importing this with no effect:
/// <reference<EMAIL_ADDRESS>/>

Maybe I'm missing something. Thx in advance 😅

hey @Tymek - can you paste your full code file example? it looks like those are the react typings (or similar), so if the imports anything from react those may override the mitosis ones

https://codesandbox.io/s/blissful-paper-4busx?file=/src/testfile.tsx
(wait couple of seconds until TS loads,

)

Super helpful reproduction, thanks for sharing!
If you remove anything react related from that project, it works as expected: https://codesandbox.io/s/young-rgb-ql03s?file=/src/testfile.tsx

oh, ok! I get it. "compilerOptions" > "jsx": "preserve" 👍🏻


Include open-vm-tools

Any chance we can get https://wiki.alpinelinux.org/wiki/Open-vm-tools included in the images?
https://wiki.alpinelinux.org/wiki/Open-vm-tools#Installation

With https://github.com/ArchiveTeam/Ubuntu-Warrior/blob/1a7271e7c15c5078e3380e67234e81dd9327c610/stage.sh#L67
It looks like we just need to add something like (probably after the docker section, and/or move apk update earlier):
apk add open-vm-tools
apk add open-vm-tools-guestinfo
apk add open-vm-tools-deploypkg
rc-update add open-vm-tools boot


Updated dockertools to container-scripts to be consistent with other products

Hi,
As suggested by Monica, to be consistent across WLS, FMW Infra and FMW, standardized container-scripts inside SOA image as 'container-scripts' instead of 'dockertools'.
Thanks
Samba

Please remember to update your master branch from the upstream master before you submit another commit.


Added some css styling and edited assembly page to pull images, but i…

…t didn't work
Signed-off-by:Vithusan<EMAIL_ADDRESS>

Bad merge


Hijack the build for msys2 :P

This makes it so that you can build with MINGW/GCC in windows rather than with MSVC.
fixes #2887

Hi there. Thanks for the PR! The default windows build should now support clang/mingw/gcc over msvc. Let me know if you hit any troubles compiling it with CGo.


Correct Usage for DataStream API

Is your feature request related to a problem? Please describe.
When users use the DataStream API, they often encounter the following problems:

Incorrect use of no lock mode.
e.g. debeziumProperties.put("debezium.snapshot.locking.mode", "none")
The captured table could not be set up correctly..
e.g. tableList("tableName")
Unable to get json output.
e.g. deserializer(new StringDebeziumDeserializationSchema())

Describe the solution you'd like
Correct usage for DataStream API:
Properties debeziumProperties = new Properties();
      debeziumProperties.put("snapshot.locking.mode", "none");// do not use lock
      SourceFunction<String> sourceFunction = MySQLSource.<String>builder()
      .hostname("yourHostname")
      .port(yourPort)
      .databaseList("yourDatabaseName") // set captured database
      .tableList("yourDatabaseName.yourTableName") // set captured table
      .username("yourUsername")
      .password("yourPassword")
      .deserializer(new JsonDebeziumDeserializationSchema()) // converts SourceRecord to JSON String
      .debeziumProperties(debeziumProperties)
      .build();

Closing this issue because it was created before version 2.3.0 (2022-11-10). Please try the latest version of Flink CDC to see if the issue has been resolved. If the issue is still valid, kindly report it on Apache Jira under project Flink with component tag Flink CDC. Thank you!


Allow custom icons in SVG format

On Linux you are used to getting your icons from one central place, your icon theme. This provides all the icons necessary and more often it even provides icons for webapps already. The Numix Icon Theme is an example for this.
Also those icons are provided as SVG, making them a lot sharper and easier to scale on HIDPI screens.
My suggestion is to provide a way to specifiy a custom svg icon. The easiest way would be to take the icon specified in the .desktop-File (The "launcher" for the application on Linux).
Another way would be to just look for icon.* instead of icon.png.

I was just testing this using the dockerized nativefier: setting --icon /app/icon.svg almost works.
Not sure who's the fault here: the SVG file is copied to resources/app/ dir but Unity (Ubuntu 17.04) doesn't recognize the icon when the binary is executed, showing the default icon instead.
When using --icon with a PNG file it works as expected.
I don't know a lot about Electron but I can help diagnose further.

@vegbrasil thanks for the feedback 👍.

I don't know a lot about Electron but I can help diagnose further.

Sure! More info and PR always welcome 🙂.


Profile page

If we can view a profile page as an HTML page, it will make invitation easier.

Mastodon's user endpoint supports both application/activity+json and text/html MIME types, though, it is difficult for API Gateway to efficiently implement it. I do not think we have to provide a profile page from the same endpoint as the API. I am thinking to serve profile pages under something like /view/users/{user-id}.

Maybe we should to associate the profile page with the WebFinger output.

Maybe we should to associate the profile page with the WebFinger output.

We can use http://webfinger.net/rel/#profile-page for rel.

Thanks to cdk-qwik-bundle, we can bundle a Qwik app in a CDK stack.


Git Clone Issue on Windows

Hello
I think I found an issue related to the submodules present in this repository.
Issue.

git clone with --recursive option fails
Possible fix.
I think the issue is cause by setting the repository using SSH. The issue solves when I change the repository to HTTPS.
By editing .gitmodules and then replace line:
url = git@github.com:JonathonLuiten/diff-gaussian-rasterization-w-depth.git
to:
url = https://github.com/JonathonLuiten/diff-gaussian-rasterization-w-depth.git

Then I reload the submodules using : git submodule update --init --recursive
But even so, I still issues with the submodules, during conda env install. 😑

Hi @java311 just making sure are you ssh'd to your github account from the terminal you're using in windows. I think in my case it asked me the ssh password when I used the git submodule update --init --recursive and it was able to clone it. This looks like more of a git issue rather then a repo issue

Closing this issue due to inactivity, please feel free to re-open. I would recommend following: https://github.com/spla-tam/SplaTAM/issues/9


Modify console app example to do flush

The console app example used as such will not send any data to Azure backend, as the app dies before telemetry is flushed from channel. Need to modify the example to avoid issues like this SO issue.
https://stackoverflow.com/questions/55289534/why-is-ilogger-not-logging-to-azure-app-insights/55290922#55290922

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

ID: b09a09da-72ae-d5c0-fe25-69274054d302
Version Independent ID: 6911ee60-7c91-f024-8788-94939567d477
Content: Explore .NET trace logs in Azure Application Insights with ILogger
Content Source: articles/azure-monitor/app/ilogger.md
Service: application-insights
GitHub Login: @cijothomas
Microsoft Alias: cithomas

@cijothomas Thank you for the feedback!  We will assign this to the document author so the can update the documentation accordingly.

Please update this.. I spent hours pulling my hair hour before finding an answer on stack overflow.

@anish714 surely. I opened this from the SO post only.

Please close this.

@cijothomas I see that the said PR was merged about 6 hours ago. Hence closing the issue.


segfault on read_parquet

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
ubuntu 16.04.2
Modin installed from (source or binary):
with pip install modin[ray]
Modin version:
0.6.3
Python version:
3.6.5
Exact command to reproduce:
read_parquet()

Describe the problem
I tried reading a parquet file with modin.pandas inside a Jupyter Notebook and failed due to segmentation fault
Source code / logs
pd.DataFrame(np.random.randint(1,10,(3,3)), columns=["a","b","c"]).to_parquet("example.pqt")
pd.read_parquet("example.pqt")
error stack:
(pid=4065) Fatal Python error: Segmentation fault
(pid=4065)
RayWorkerError                            Traceback (most recent call last)
 in
1 pd.DataFrame(np.random.randint(1,10,(3,3)), columns=["a","b","c"]).to_parquet("example.pqt")
----> 2 pd.read_parquet("example.pqt")
~/shahaf_env/lib/python3.6/site-packages/modin/pandas/io.py in read_parquet(path, engine, columns, **kwargs)
25     return DataFrame(
26         query_compiler=BaseFactory.read_parquet(
---> 27             path=path, columns=columns, engine=engine, **kwargs
28         )
29     )
~/shahaf_env/lib/python3.6/site-packages/modin/data_management/factories.py in read_parquet(cls, **kwargs)
42     @classmethod
43     def read_parquet(cls, **kwargs):
---> 44         return cls._determine_engine()._read_parquet(**kwargs)
45
46     @classmethod
~/shahaf_env/lib/python3.6/site-packages/modin/data_management/factories.py in _read_parquet(cls, **kwargs)
46     @classmethod
47     def _read_parquet(cls, **kwargs):
---> 48         return cls.io_cls.read_parquet(**kwargs)
49
50     @classmethod
~/shahaf_env/lib/python3.6/site-packages/modin/engines/ray/generic/io.py in read_parquet(cls, path, engine, columns, **kwargs)
212         # column.
213         dtypes_ids = list(blk_partitions[-1])
--> 214         dtypes = pandas.concat(ray.get(dtypes_ids), axis=0)
215         if directory:
216             if len(partitioned_columns) > 0:
~/shahaf_env/lib/python3.6/site-packages/ray/worker.py in get(object_ids)
2245             if isinstance(value, RayError):
2246                 last_task_error_raise_time = time.time()
-> 2247                 raise value
2248
2249         # Run post processors.
RayWorkerError: The worker died unexpectedly while executing this task.

Hi @shgoren thanks for posting!
Does this happen every time you run it? Sometimes Ray crashes and the fix is just to restart the kernel and try again.

Please, recheck and close.

Reproducer above started to work with Ray version >= 0.8.1
Current Modin master uses 0.8.7 Ray Version, so that bug is fixed


Won't cache facebook profile photo

What did you do?
I need cache facebook profile photo. Here (in code example scope) is a link to my profile photo, Alamofire load photo but does not cache it. If you exit the application and run it again the download starts from the beginning and when phone offline photo also not available. But when I use the link that has .jpeg or .png in ending that works fine.
What did you expect to happen?
Alamofire should cache facebook profile photo.
What happened instead?
Alamofire doesn't cache facebook profile photo.
Code example:
let url = "https://graph.facebook.com/1067979000032009/picture?type=large&return_ssl_resources=1"
userImageView.af_setImage(withURL: url)

Alamofire Environment
Alamofire version: 4.7.3
Xcode version: 9.4.1
Swift version: 4.2
Platform(s) running Alamofire: IOS 11.4.1
macOS version running Xcode: 10.13.6
Does someone got a solution for this issue?
Thank you!

I believe you meant to ask this for AlamofireImage. AFI does not have a persistent cache. If you want to store the profile images for later, you'll need to do so yourself or find a way for the URLCache to handle it for you.


chore: Release v16.1 with v16 unstable metadata

[16.1.0] - 2024-09-16
Added

v16: Add unstable metadata v16 #82

cc @paritytech/subxt-team

The top level RuntimeMetadata enum grew a V16 variant, so we'd need to do a major bump here to avoid breaking things :)


Include Abot.xml in NuGet package

Right now we don't get all the nice comments in the source code as intellisense documentation when using the NuGet package.
You should output the "XML documentation file" in the build of Abot and include the generated Abot.xml in the NuGet package

Thank you for the suggestion. Also please consider submitting a pull request. The nuspec file is in the Abot project csproject directory. Its invisible to visual studio.


AnimationPlayer not showing the full Path

Godot 3.1 stable, Windows 10.
AnimationPlayer used to give me the full path for a node in the scene, but now it only shows some of the path. Is it possible to see the full path somehow? It really gave me a good overview of where the find the nodes. Funny enough, if you delete a node from the scene it will give you the full path.

Sorry, didn't see that you can click a node and see the full path, this might do the trick.


chore: implement a test watcher that checks which tests are "dirty"

Waiting 2.5 minutes for all tests after each change isn't really a good turnaround time, so hopefully detecting and running only changed tests cross-repo can help with that.

@zwave-js-bot approve


Change release build so that a panic results in an abort

Because we are being called from C, we should abort instead of panic. Panicking unwinds the stack, which has undefined behavior on a stack made by C.
Fix should be bundled with #98

With Rust 1.24, it will abort by default when hitting a C FFI boundary.
This also will not be an issue in wlroots-rs since we have panic catcher set up.


Allow using external Postgresql database

I use Cockroachdb to host several Postgres databases and would like to use it with this project. Plus, if Firezone will be horizontally scalable one day, it will need to be able to connect to an external db.
For this to work, I primarily need to be able to set these options for the Postgres connection:
https://github.com/firezone/firezone/blob/master/config/releases.exs#L43
connect_opts = [
  ...
  ssl: true,
  ssl_opts: [
    cacertfile: ...,
  ],
  parameters: [options: "--cluster=..."]
]

When I was hacking around Firezone, I discovered that :ssl is not currently compiled into the app; at least that's what i think this error is telling me (I'm not an Elixir dev):
SSL connection can not be established because `:ssl` application is not started,
    you can add it to `extra_applications` in your `mix.exs`

Finally, the chef recipe that creates the user, database and extensions would need to be updated to accept those params.
firezone/omnibus/cookbooks/firezone/recipes/database.rb
although an option to skip that would be great as well.
(I've been manually editing the recipes to just skip those parts as I manually created the user, database, and extensions)

Thanks for the suggestion! I'll add this to our backlog.
You're correct -- we'll have to change how Postgres gets initialized via Chef. There's already a config option to disable Postgres, and Gitlab (also uses Omnibus) achieves this, so it should be entirely possible.

@efossas This was added in #421 and will roll out with the next release 0.2.10 in the next day or two.

Release 0.2.10 is live: https://github.com/firezone/firezone/releases/tag/0.2.10
See the new configuration file options default['firezone']['database']['ssl_opts'] and default['firezone']['database']['parameters'] as referenced here: https://docs.firez.one/docs/reference/configuration-file/


Describe the GOV.UK development/deployment pipeline

We've been talking about replatforming GOV.UK and what that would look like for the development experience. This is an attempt to document what  our current pipeline looks like, so we're all on the same page.

A couple of comments, only one which probably requires any change (and only minor at that). Looks like a great overview :+1:

I like how simple and clear this is.
I see that points 4, 5 and 6 reference https://docs.publishing.service.gov.uk/manual/deploying.html.
Would it be worth adding points 1, 2 and 3 there as well, basically merging these two docs?
It looks like the info in these two docs could live well together in one page.

I've replaced the text in the deploy steps with a link to the deployment page, because it otherwise risks duplicating that. Thanks @cbaines and @AlanGabbianelli !


Workspace locked on test-ibmcloud-win2012r2-x64-2

Please put the system name in the title of this issue. https://ci.adoptopenjdk.net/computer/test-ibmcloud-win2012r2-x64-2/

Link to any log file showing the problem: https://ci.adoptopenjdk.net/job/Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1/37/console

Please describe the issue: Test run failed as it could not delete previous workspace:

Waiting for next available executor on ‘[ci.role.test&&hw.arch.x86&&sw.os.windows](https://ci.adoptopenjdk.net/label/ci.role.test&&hw.arch.x86&&sw.os.windows/)’
Running on [test-ibmcloud-win2012r2-x64-2](https://ci.adoptopenjdk.net/computer/test-ibmcloud-win2012r2-x64-2/)[](https://ci.adoptopenjdk.net/job/Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1/37/console#)[](https://ci.adoptopenjdk.net/job/Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1/37/console#)[](https://ci.adoptopenjdk.net/job/Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1/37/console#)[](https://ci.adoptopenjdk.net/job/Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1/37/console#)[](https://ci.adoptopenjdk.net/job/Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1/37/console#) in E:\workspace\Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1
[Pipeline] {
[Pipeline] timeout
Timeout set to expire in 1 hr 0 min
[Pipeline] {
[Pipeline] cleanWs
[WS-CLEANUP] Deleting project workspace...
[WS-CLEANUP] Deferred wipeout is used...
ERROR: Cannot delete workspace :Unable to delete 'E:\workspace\Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1'. Tried 3 times (of a maximum of 3) waiting 0.1 sec between attempts. (Discarded 26 additional exceptions)
[Pipeline] }
[Pipeline] // timeout
[Pipeline] echo
Exception: hudson.AbortException: Cannot delete workspace: Unable to delete 'E:\workspace\Test_openjdk17_hs_extended.openjdk_x86-32_windows_testList_1'. Tried 3 times (of a maximum of 3) waiting 0.1 sec between attempts. (Discarded 26 additional exceptions)

This was rebooted in April which would have cleared any locks: https://github.com/adoptium/infrastructure/issues/2493#issuecomment-1097132715


Fix PICA result format

PICA format is likely not final. Find out what is actually expected and fix it.

Done with https://github.com/gbv/coli-ana/commit/f6d2cdd912a8069515de979017ef5f0cab26791b


Allow users to not set local stratum

Pull Request (PR) description
The template uses a conditional statement but it always evaluates to true
and a result users can't disable the usage of local stratum.
We can fix that by adjusting the data type of the local_stratum
parameter to accept the Boolean false value. So, users can set it to
false and the template doesn't render the specific block as the
conditional statement evaluates to false.
This Pull Request (PR) fixes the following issues
No issue was opened for this.

Closing and reopening in the hope to have the test suite running against this PR…

Can you add a unit test to ensure we do not break this in the future?

Added https://github.com/voxpupuli/puppet-chrony/pull/113/commits/a352594ba938ebb0f8827690cf73940b52c95c3f but I can't run the specs locally I am getting some error related to the helper.  I will wait for the CI to run.

The CentOS failures can be ignored for now, I am trying to fix them in #114. When this get merged, rebasing on top of it should be all green grinning

Nice to know that.

Looks good.  Maybe also update the docs here https://github.com/voxpupuli/puppet-chrony/blob/adbbd7f13f538c2938b042d01b56cc85bd12d55e/manifests/init.pp#L92 ?

Looks good. Maybe also update the docs here
https://github.com/voxpupuli/puppet-chrony/blob/adbbd7f13f538c2938b042d01b56cc85bd12d55e/manifests/init.pp#L92
?

Is this https://github.com/voxpupuli/puppet-chrony/pull/113/commits/b7f453b9c30d06f822d34dee2f35245c8a6f8387 good enough?

@unixsurfer CI should be fixed in master. Can you please rebase on top of it and force push to update this PR?

@unixsurfer CI should be fixed in master. Can you please rebase on top of it and force push to update this PR?

Done.


Proposal: use standard imports for operators/hooks

Currently, Airflow uses a relatively complex import mechanism to import hooks and operators without polluting the namespace with submodules. I would like to propose that Airflow abandon that system and use standard Python importing.
Here are a few major reasons why I think the current system has run its course.
Polluting namespace
The biggest advantage of the current system, as I understand it, is that only Operators appear in the airflow.operators namespace.  The submodules that actually contain the operators do not.
So for example while airflow.operators.python_operator.PythonOperator is a thing, PythonOperator is in the airflow.operators namespace but python_operator is not.
I think this sort of namespace pollution was helpful when Airflow was a smaller project, but as the number of hooks/operators grows -- and especially as the contrib hooks/operators grow -- I'd argue that namespacing is a good thing. It provides structure and organization, and opportunities for documentation (through module docstrings).
In fact, I'd argue that the current namespace is itself getting quite polluted -- the only way to know what's available is to use something like Ipython tab-completion to browse an alphabetical list of Operator names, or to load the source file and grok the import definition (which no one installing from pypi is likely to do).
Conditional imports
There's a second advantage to the current system that any module that fails to import is silently ignored. It makes it easy to have optional dependencies. For example, if someone doesn't have boto installed, then they don't have an S3Hook either. Same for a HiveOperator
Again, as Airflow grows and matures, I think this is a little too magic. If my environment is missing a dependency, I want to hear about it.
On the other hand, the contrib namespace sort of depends on this -- we don't want users to have to install every single dependency. So I propose that contrib modules all live in their submodules: from airflow.contrib.operators.my_operator import MyOperator. As mentioned previously, having structure and namespacing is a good thing as the project gets more complex.
Other ways to handle this include putting "non-standard" dependencies inside the operator/hook rather than the module (see HiveOperator/HiveHook), so it can be imported but not used. Another is judicious use of try/except ImportError. The simplest is to make people import things explicitly from submodules.
Operator dependencies
Right now, operators can't depend on each other if they aren't in the same file. This is for the simple reason that there is no guarantee on what order the operators will be loaded. It all comes down to which dictionary key gets loaded first. One day Operator B could be loaded after Operator A; the next day it might be loaded before. Consequently, A and B can't depend on each other. Worse, if a user makes two operators that do depend on each other, they won't get an error message when one fails to import.
For contrib modules in particular, this is sort of killer.
Ease of use
It's hard to set up imports for a new operator. The dictionary-based import instructions aren't obvious for new users, and errors are silently dismissed which makes debugging difficult.
Identity
Surprisingly, airflow.operators.SubDagOperator != airflow.operators.subdag_operator.SubDagOperator. See #1168.
Proposal
Use standard python importing for hooks/operators/etc.

__init__.py files use straightforward, standard Python imports
major operators are available at airflow.operators.OperatorName or airflow.operators.operator_module.OperatorName.
contrib operators are only available at airflow.contrib.operators.operator_module.OperatorName in order to manage dependencies
operator authors are encouraged to use __all__ to define their module's exports

Possibly delete namespace afterward

in operators/__init__.py, run a function at the end of the file which deletes all modules from the namespace, leaving only Operators. This keeps the namespace clear but lets people use familiar import mechanisms.

Possibly use an import function to handle ImportError gracefully

rewrite import_module_attrs to take one module name at a time instead of a dictionary.

+1

+1

+1
also:

currently we do lots of unused imports, a typical airflow run will import all operators, hooks and related libs, where in reality it needs only one operator and related deps
I haven't looked at the latest utils refactor either, but we should do that there too
we need to document the full pythonpath location of operators and maybe cleanup the module names where/if needed

but:

that breaks the API, lots of pipelines will need to be altered. Breaking the API mean going to 2.0, is there an option there to have some sort of environment hook for monkey patching operators into airflow.operators. Maybe the currently hard coded data structure that does this (airflow.operators.__init__) goes away but can be passed in from airflow_settings.py, then we can get away with just an aggressive warning in the next release notes
individual operators depending on hooks only (as opposed to other operator) is sane, though deriving an operator is ok

@mistercrunch excellent point. I think we can take advantage of the fact that the objects imported into airflow.operators are not the same as the ones in airflow.operators.op_module. We could mark all of the top-level objects as deprecated using something like http://docs.zope.org/zope.deprecation/api.html#deprecating-objects-inside-a-module. That way people could continue to use them, but would receive a deprecation warning saying for example that they will be removed in 2.0

Sounds perfect!

It works! see: https://github.com/jlowin/airflow/commit/973295c4aa7df1134849f945066622b7f8547cc2
In [1]: import airflow
[2016-03-29 11:49:06,956] {__init__.py:36} INFO - Using executor SequentialExecutor

In [2]: from airflow.operators import PigOperator
/Users/jlowin/anaconda3/bin/ipython:1: DeprecationWarning: PigOperator: Importing
PigOperator directly from 'airflow.operators' has been deprecated. Please import
from 'airflow.operators.[operator_module]' instead. Support will be dropped entirely
in Airflow 2.0.
  #!/bin/bash /Users/jlowin/anaconda3/bin/python.app

In [3]: PigOperator
Out[3]: pig_operator.PigOperator

In [4]: from airflow.operators.pig_operator import PigOperator

In [5]: PigOperator
Out[5]: airflow.operators.pig_operator.PigOperator

Yep I currently do this for apply_defaults in  airflow/utils/init.py


Problem to forward requests when container has more than one network

Hello,
I'm having problems to forward requests when container has more than one network.
In the docker-compose file, I create 2 networks: frontend for apps and backend for database. The container that is causing me problems is the dspace that has the 2 networks. When I leave only the frontend network the routing of the traefik works.
I've tried using the options "providers.docker.network=frontend" and "traefik.docker.network=frontend" but still don't work.
docker-compose.txt

Hello,
it's not the right place to ask a question.
You can join the community forum https://community.containo.us/


[FLINK-14468][docs] Update Kubernetes docs

use Deployment from apps/v1 instead of extensions/v1beta1
run as unprivileged user

What is the purpose of the change
This pull request updates the Kubernetes documentation to use the current Deployment resource and run unprivileged
Brief change log

Docs update

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency):  no
The public API, i.e., is any changed class annotated with @Public(Evolving):  no
The serializers:  no
The runtime per-record code paths (performance sensitive):  no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector:  no

Documentation

Does this pull request introduce a new feature?  no

Thanks for your work!


Feature Request: Shop Price Adjustment

I've been feeling like the game can turn a bit grindy when trying to get enough money to buy things in the shop.
Could we get some options to adjust the price of things?
As an example, the Ocarina of Time AP allows shop prices of "affordable", random within various limits, etc..

Boy do I have good news for you! I actually already implemented it, and it is playable in a preview release.
apworld
Client
After some more testing and client side stuff I will put out a full release of 0.7.0 so stay tuned for that.


Allow to custom the ellipsis on overflow.

Allow to custom the text-overflow property, please.

Sure!

Implemented in version 1.2.4


English as a Foreign Language UI improvements

Context

A set of small improvements identified during the build of https://trello.com/c/9fk7yluU .
Changes proposed in this pull request

Add length limits to the EFL forms.
Change IELTS band score from input field to drop down
Make EFL controller navigation more robust

(see commits for more detail)

IELTS band score change

Guidance to review

Review app:

Switch on efl_section feature flag
As a candidate, select any nationality other than British and Irish.
Click through to the EFL section via the main application form screen.

Link to Trello card

https://trello.com/c/9fk7yluU
Things to check

[ ] This code doesn't rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

Looks like the band score is prefilled with 0.0 the first time you get there:


Update local-windows-event-log-source.md

This is not formatted correctly, I don't think this should be showing the ":::" in plaintext. It should probably be some sort of bubble or module?
Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

Thank you, @bnartiff ! The formatting is pretty wonky on this page. I'll go update the rest as well.


related work

hi kugelrund,
can you give a reference paper or something which you used as inspiration for this work?
i already googled and found some papers for mesh clustering by spectral clustering.
Which would you suggest?
Good work by the way and kind regards!
ps. maybe that one?
@inproceedings{liu2004segmentation,
title={Segmentation of 3D meshes through spectral clustering},
author={Liu, Rong and Zhang, Hao},
booktitle={Computer Graphics and Applications, 2004. PG 2004. Proceedings. 12th Pacific Conference on},
pages={298--305},
year={2004},
organization={IEEE}
}

Hey Thomas,
yes, that is the paper we were using, as stated in the readme. However it was given to us by the professor, so i can't help you with where to get it unfortunately. For spectral clustering in general, "U. Luxburg, A Tutorial on Spectral Clustering, Statistics and Computing, 2007." was the main reference for the course.
Have fun with looking into this topic!

Oh cool :+1:
thanks for the advise, didn’t see that in the readme.
its free to read. ftp://www1.idc.ac.il/ICGF/GraphicsSeminar2006/spectralSegmentation04.pdf


Fix typo in ifndef OPENSSL_NO_ENGINES.

All other instances are OPENSSL_NO_ENGINE without the trailing "S".
Fixes build when configured with no-engine.
CLA: trivial

Merged to dc7e354275fccc3f13d30a0024f18b5ead560532, thanks!


Testing Framework

So I'll help write tests. Anyone have an opinion on what they'd like to use? I thought the simplest thing might be Mocha/BDD/JSDOM.
Thoughts. Votes.

I would be willing to write some tests as well.
If it were up to me I would choose Karma with Jasmine. Purely from the standpoint of thats what I have the most experience with.
Karma will let you test in many different browsers easily which I'm expecting to be important on a project like this.
http://karma-runner.github.io/0.13/index.html

I don't mind Karma, I just hate configuring it. You want to pick up that part?

sure thing, karma works great with mocha too, wanna use that instead of Jasmine?

Yeah. I like Mocha more than Jasmine. But either way works. I've used them all at some point. The syntax will basically be the same.

there ya go...

Would be nice to setup TravisCI as well..

You guys rock, thanks!
And yes, I will set up TravisCl once we have some tests :)

:+1: for sure, would be nice to see that tests are passing on pull requests.

Hmm, anyone has any idea why Travis doesn't seem to be working? I activated this repo on travis-ci.org, I added a .travis.yml file, nothing… What did I forget?

@LeaVerou I'm wondering if it's because there's nothing in the master branch. Could be that we need to tell Travis that we're not using master somehow?

Try something like?
# whitelist
branches:
  only:
    - gh-pages

Ah. Here we go. Seems like this is the official answer. http://stackoverflow.com/questions/12878568/travis-ci-triggers-build-only-for-master-branch-in-github-repo

Ok. We have builds and a testing framework. Closing this out. We should probably make individual issues for needed tests.


....dirtyForms is not a function

After #20 I see this error in the console:
Uncaught TypeError: jquery__WEBPACK_IMPORTED_MODULE_1___default(...)(...).dirtyForms is not a function
    at HTMLDocument.<anonymous> (admin-entry.js:153)
    at mightThrow (admin-entry.js:17717)
    at process (admin-entry.js:17785)

Here's an image of it:

It was a cache issue or whatever of some sort. Just removed the node_modules and yarn.lock files. I love JS ._.


How manual pairing code is generated

Reproduction steps
Hi everyone, i have a task about understanding matter protocol.
During pair process, i don't understand how manual pairing code is generated base on vendorID, productID,...
i read the document but still don't get it
Can someone give me an explanation about this? thank a lot!!!
Bug prevalence
everytime
GitHub hash of the SDK that was being used
all-cluster-app and lighting-app for esp32
Platform
esp32
Platform Version(s)
No response
Type
Test Improvement
Anything else?
No response

Hi @jarvissd , I think you might check the
Matter-1.3-Core-Specification, Chapter 5. Commissioning
Available here.
I think this should answer your question, in
5.1.4 Manual Pairing Code on page 236.

i tried to follow this instruction from document to gen manual code
but when i tried to decode manual code: 3497 011 2332 ( manual code default) i didn't get right information
like: vendor ID, discrimiator, product ID,...

Which value you want to find vor Vendor ID, Discriminsator, Passcode, and VID_PID_PRESENT ? Also, is your decoding a script or a manual application. Maybe check the src/setup_payload/Setup_Payload.cpp will help you

Version:             0
Vendor ID:           65521    (0xFFF1)
ProductID:           32768    (0x8000)
Custom flow:         0        (STANDARD)
Discovery Bitmask:   0x02     (BLE)
Long discriminator:  3840     (0xf00)
Passcode:            20202021
these default informations i tried on examples lighting-app
and manual code  is 3497 011 2332
also with QR code MT:Y.K9042C00KA0648G00
i tried to decode it thought base 38 but it still wasn't right
@QuentinCaldeira-eaton
can u explain for me pls :<

0001 0011 0100 0100 0010 0010 0101 //PASSCODE 20202021

		    1111 0000 0000 // DISCRIMINATOR 3840

	       1111 1111 1111 0001 // VENDOR ID 0xFFF1

### First digit

VID_IS_PRESENT Seems to be 0, so it will be DISCRIMINATOR >> 10,
so 1111 0000 0000 >> 10 = 0b11, or 3 in decimal

### Second to sixth digit

((DISCRIMINATOR & 0x300) << 6) | (PASSCODE & 0x3FFF)

DISCRIMINATOR & 0x300 = 0b0011 0000 0000 << 6 = 0b0000 1100 0000 0000 0000

PASSCODE & 0x3FFF = 0b0000 0000 0000 0000 0010 0010 0101

so...

  0000 0000 0000 0000 0010 0010 0101 |
  0000 0000 0000 1100 0010 0010 0101 =

  0000 0000 0000 1100 0010 0010 0101 or 49701 in decimal

### Seventh to thenth digit

PASSCODE >> 14 = 0b0000 0100 1101 0001 so 1233 in decimal

## Last digit

Follow an algorithm, don't have the time to help you on this one

So, the encoding is working, and so on for the decoding. If you want to decode, follow the same path, but invert. Please be aware of logical expression, and use a proper converter to pass from binary to decimal. Is this helpful enough ?

ok thanks for all


Update rebind and rebindAll to return target

The README states that rebind and rebindAll should return the target.

Well spotted, thank you :-)


Firewall link

Link under Security Policies & Recommendations section on the bullet point "Use a firewall from a Microsoft partner to increase your security protections." goes to an unrelated Azure Sentinel page.

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

ID: 5524b067-f3c4-2ef2-7a82-00214aa146c0
Version Independent ID: 0ad17af0-c850-16cb-6b1d-2350573ba426
Content: Azure operational security checklist
Content Source: articles/security/fundamentals/operational-checklist.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @unifycloud
Microsoft Alias: tomsh

@cerbusjpeg Thanks for the question. We are investigating the issue and will update you shortly.

@cerbusjpeg - Thank you for your interest in our documentation. A fix to the link is being published.
#please-close


Adding recursive search and full path filename

When I tried to use the tool, I missed recursive search and also the option of being able to enter the file name with the full path.

Made changes for recursive. Call with the -r flag.

also added -f flag to specify individual files


[Breeding] Slimeon Breeding Ability Visual Glitch

Describe the bug
Slimeon had its ability changed and Hawky's slimeon was
To Reproduce
Steps to reproduce the behavior:

Get a female Slimeon that had its ability "Gooey" that changed to HA
Breed it (female or not I think)
Hatch the egg from it
Put it on gts to see visual bug as Run Away (HA) or adaptability (HA)

Expected behavior
A visual bug as Run Away (HA) or adaptability (HA) will appear on GTS or in /wt list
Screenshots

Username: Temperestre
Server: Atlas (All servers possibly)
Logs:
https://pastebin.com/LwFENh1X
Hatched an[06Dec2023 07:46:43.709] [main/INFO] [net.minecraft.client.gui.NewChatGui/]: [CHAT] Your Egg hatched into a baby Eevee!
Putting the hatched eevees on gts:
Additional context
This might also happen rarely with a male slimeon as parent. I haven't hatched a real HA eevee yet. Am I supposed to?

After 7 of these eggs reproduced, I haven't received one real HA eevee.

Confirmed in game, parents were both another OT so it never will pass down HA.
However the visual glitch for it passing down (HA) next to any ability is still there.

This is likely the old bug with modifiers, it's been now since fixed for all NEW modifier actions, but old Pokemon that were affected by this bug (roughly December) need to be modified manually with the wand, @CptTCovenant

Closing, OP inactive.


Cannot install with pip 22.0.4

Currently running python 3.9.11 on Debian. Not sure if this has ever been reported, but pasting my output here. Anchor, solana, and pip are installed. setuptools is also up to date. Any help would be appreciated.
pip install anchorpy
Collecting anchorpy
  Using cached anchorpy-0.8.1-py3-none-any.whl (43 kB)
Collecting jsonrpcclient<5.0.0,>=4.0.1
  Using cached jsonrpcclient-4.0.2.tar.gz (6.5 kB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [1 lines of output]
      ERROR: Can not execute `setup.py` since setuptools is not available in the build environment.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
(pyvenv39) #( 03/17/22@ 4:48PM )( lemons@basecamp ):~
   anchor --version
anchor-cli 0.22.1
(pyvenv39) #( 03/17/22@ 4:48PM )( lemons@basecamp ):~
   python --version
Python 3.9.11
(pyvenv39) #( 03/17/22@ 4:48PM )( lemons@basecamp ):~
   pip --version
pip 22.0.4 from /home/lemons/.pyenv/versions/3.9.11/lib/python3.9/site-packages/pip (python 3.9)

Can you try pip install jsonrpcclient? That seems like the part that pip is complaining about?

Yep, runs into same issue. I'll file a request with their team. Can close this out


[Feature] Pre-build a list of OpenDevin supported sandbox images

What problem or use case are you trying to solve?
Relavant discussion: https://github.com/OpenDevin/OpenDevin/pull/2658#issuecomment-2212544525
User may have diverse environment needs (e.g., some need python 3.10, 3.11, some need nodejs, etc). We already supported custom sandbox images that allow the user to specify a sandbox image they want to use in OpenDevin (e.g., nodejs), and OpenDevin can locally build it to be compatible with OpenDevin.
Describe the UX of the solution you'd like

[ ] Instead of user need to build their custom image locally, maybe we can support a list of commonly used docker images (e.g., prevalent python/nodejs version, etc) as "officially build and supported" images. This will get re-build when a new version is released to save CI time.
[ ] Select from a drop-down list on the frontend interface on which image you want to start the session with

Do you have thoughts on the technical implementation?
Maybe we can create a .txt file, where each line is an upstream docker image, e.g., https://hub.docker.com/r/ubuntu/python.
And we add a Github workflow to read this file and build the corresponding images at time of each release.
Describe alternatives you've considered
Additional context

Maybe we can start by supporting these common images:

python3.9 - 3.12
node18/20.

Also, what is the naming convention for these images? How about ghcr.io/opendevin/sandbox:${release_version}-${image_repo}__${image_tag} ?
e.g. ghcr.io/opendevin/sandbox:0.7.1-python__3.12


✨ Add implicit selection of CRDs, APIBindings

Summary
This PR modifies the meaning of the downsync part of the "what predicate" in an EdgePlacement to implicitly include CustomResourceDefinitions and APIBinding objects that define the kinds of other selected objects.
This PR also makes the meaning downsync part of the "what predicate" exclude the things that match upsync part.
Related issue(s)
I think this addresses the concerns behind #1064 and #1080.

/cc @clubanderson
/cc @waltforme
/cc @yana1205
/cc @dumb0002
/cc @KPRoche
/cc @francostellari

This PR makes it so that you do not have to explicitly talk about the CRD objects in the "what predicate" regardless of whether they define a namespaced resource or a cluster-scoped resource. Implicit matching of CRD objects is demonstrated in the update to example1.

/lgtm

This is a backward-compatible change.

/approve


Support SET key=value syntax

Background
context: #8906
Description
This PR introduces a new way to set environmental options for executing a SQL query on Pinot.
Syntax examples for single and multiple environment options:
SET skipUpsert = 'false';
SELECT count(*) FROM tbl

SET foo = 'bar';
SET some_key = 'some_value';
SELECT * FROM tbl

Details

; at the end of the SET statement indicate it is an end of a statement; However, Pinot doesn't support multiple statements, thus all SET statement(s) are only valid within the SQL query string context.

this means first executing ALTER SESSION SET key = 'value'; and then executing another query won't affect the query.
this also means, effectively one pinot query string is one individual, isolated user session.

we allow multiple SET statements within a single query string

Codecov Report

Merging #9017 (c260a3e) into master (de16a0a) will decrease coverage by 3.23%.
The diff coverage is 74.32%.

:exclamation: Current head c260a3e differs from pull request most recent head 5b7c51c. Consider uploading reports for the commit 5b7c51c to get more accurate results

@@             Coverage Diff              @@
##             master    #9017      +/-   ##
============================================
- Coverage     70.08%   66.85%   -3.24%
+ Complexity     4957     4766     -191
============================================
  Files          1827     1369     -458
  Lines         96064    69842   -26222
  Branches      14356    10912    -3444
============================================
- Hits          67327    46693   -20634
+ Misses        24092    19864    -4228
+ Partials       4645     3285    -1360

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
66.85% <74.32%> (-0.02%)
:arrow_down:

unittests2
?

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

Impacted Files
Coverage Δ

...g/apache/pinot/common/metrics/ControllerMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../pinot/core/query/scheduler/PriorityScheduler.java
77.77% <ø> (-0.31%)
:arrow_down:

...local/indexsegment/mutable/MutableSegmentImpl.java
54.34% <42.85%> (-0.56%)
:arrow_down:

...org/apache/pinot/sql/parsers/CalciteSqlParser.java
86.41% <77.77%> (-1.56%)
:arrow_down:

.../pinot/common/restlet/resources/TableTierInfo.java
100.00% <100.00%> (ø)

...rg/apache/pinot/sql/parsers/SqlNodeAndOptions.java
100.00% <100.00%> (ø)

...g/apache/pinot/sql/parsers/dml/InsertIntoFile.java
85.18% <100.00%> (ø)

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 700 more

Continue to review full report at Codecov.

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


smaller area with multipolygons accounted for

Should you wish to do a smaller area, say a suburb or campus: overpass and osm2geojson might be useful.
query = """
[out:json][timeout:30];
// area-of-interest here
area[name='Cape Peninsula University of Technology (Bellville Campus)']->.a;

// --when areas have duplicate names given the world has a limited amount of uniquely named places
// --main area
//area[name="Western Cape"]->.b;
// -- target area ~ can be way or relation
//relation(area.b)[name="Woodstock"];
//map_to_area -> .a;

(
  (
    // I want all buildings
    way[building](area.a);

    // plus every building:part
    way["building:part"](area.a);
    // and relation type=multipolygon ~ to removed courtyards from buildings
    relation["building"]["type"="multipolygon"](area.a);
  );
-
  // excluding buildings with relation type=building role=outline
  // to remove outlines that surround building:part
  (
    // for every way in the input set select the relations of which it is an "outline" member
    rel(bw:"outline")["type"="building"];
    // back to the ways with role "outline"
    way(r:"outline");
  );
);
out body;
>;
out skel qt;
"""

url = "http://overpass-api.de/api/interpreter"
r = requests.get(url, params={'data': query})
#rr = r.read()
gj = osm2geojson.json2geojson(r.json())

will you give you what you need. Hereafter you can continue with a slightly modified hdb2d.py. The image below as an example of the result.

Courtyards are correct with features in courtyards retained (bottom left) and buildings with correct height when building:part have a different height to the outline (above half-way up on the right). notebook here.
Thank you for this wonderful resource.

Thanks Adrian! Great idea


some bug

7:51:17 PM] Cracken: hey
[7:51:28 PM] Cracken: not sure if you know about this bug
[7:51:42 PM] Cracken: in slide ninja slide
[7:52:17 PM] Cracken: when you place the walking thing [thaw] right at the edge of every "safe zone", basically between the safe zone and the ice, and then you walk thru it, you can walk in ice

Duplicate issue #8


Allow AppRole auth in Hashi Vault secrets helper

AppRole auth is supported in the vault gem so we should allow this method for use in the secrets helper

Done


[Question] Flash attention only applies to prefilling stage

I have a question arising from reading the code. I notice that in ~/lightllm/models/llama2/layer_infer/transformer_layer_infer.py, the flash attention is only applied to the prefilling stage, i.e. the context_attention_fwd, but not to the decoding stage, i.e. token_att_fwd. Am I correct in this understanding?
In principle, token attention doesn't conflict with flash attention. Do you plan to combine them both in the decoding stage?
Also, what is the obstacle of directly using flash attention repo with the token-level memory management?

@KexinFeng We try to implement this triton kernel that use flash attention。but currently it is not fast enough。


add dashboard creator as owner when dashboard is created

a report of this bug:
https://airbnb.slack.com/archives/caravel/p1472094489000732

there's a work around, but it's not very clear.


Extend support for Input::Message type fields in std::inputs

Also adds tests for untested functions currently in the stdlib.

Can we add support for getting the hash of the script bytecode? I've been using this in the current predicate for the bridge:
use std::constants::ZERO_B256;

const GTF_SCRIPT_SCRIPT_LENGTH = 0x005;
const GTF_SCRIPT_SCRIPT = 0x00B;

/// Get the hash of the script bytecode
pub fn tx_script_bytecode_hash() -> b256 {
    // Get the script memory details
    let mut result_buffer: b256 = ZERO_B256;
    let script_length = __gtf::<u64>(0, GTF_SCRIPT_SCRIPT_LENGTH);
    let script_ptr = __gtf::<u64>(0, GTF_SCRIPT_SCRIPT);

    // Run the hash opcode for the script in memory
    asm(hash: result_buffer, ptr: script_ptr, len: script_length) {
        s256 hash ptr len;
        hash: b256
    }
}

I feel like it belongs in tx.sw since it requires the constant values for certain GTF fields.

Can we add support for getting the hash of the script bytecode?

@pixelcircuits I'm not opposed to this. Would you mind opening a separate issue to track it?

Can we add support for getting the hash of the script bytecode?

@pixelcircuits I'm not opposed to this. Would you mind opening a separate issue to track it?

Sure! #2721

This PR should wait until #2810 is finished since there is a lot of overlap

This PR is looking good. I just wanted to point to the tests on my now defunct branch for testing things like predicate bytecode and data when you get to that.

Can we go ahead and use this issue to make all GTF_ const definitions public (pub) so that they can be referenced for more lower level solutions like the current bridge message-contract-predicate?

Can we go ahead and use this issue to make all GTF_ const definitions public (pub) so that they can be referenced for more lower level solutions like the current bridge message-contract-predicate?

I think so. So far I've only been exposing them as needed, but no reason to keep any of them private.

hmm. @mohammadfawaz any ideas on why CI would fail for this unrelated test?


Feature/vbeck labelupdates

Critical Changes
Changes

updated labels throughout EDA Settings

Issues Closed
New Metadata
Deleted Metadata
Testing Notes

Spot Checked the EDA Settings & Custom Settings. Looks good.


Obsolete M2ModeShape enum

Remove gmt_dos_clients_io::gmt_m2::M2ModeShape enum.

It has been removed

It has been removed


Initialize workspace asynchronously

In cases where we load a significant number of packages and/or compile a significant amount of Q# code at kernel initialization time, it's possible to exceed Jupyter's default 60-second timeout for kernel_info_reply, which results in an error message being displayed to the user (even though the initialization does eventually succeed).
In addition, client-side features such as Q# syntax highlighting don't light up until the kernel_info_reply is received.
This change moves the workspace initialization to be asynchronous, such that the kernel_info_reply can be sent prior to loading the packages or compiling the Q# code in the workspace. Cell execution will now wait for the workspace initialization to complete before attempting to execute the cell.
This PR is intended to address issues discovered in CI runs and local testing for https://github.com/microsoft/QuantumKatas/pull/468 and https://github.com/microsoft/QuantumKatas/pull/494.

            }

Should we not wait here then, if we want to load this async?

Refers to: src/Core/References/References.cs:135 in ff9efa1. [](commit_id = ff9efa10d186957d34d319a3d68c1edd8f5c54c2, deletion_comment = False)

            }

Should we not wait here then, if we want to load this async?
Refers to: src/Core/References/References.cs:135 in ff9efa1. [](commit_id = ff9efa1, deletion_comment = False)

So AutoLoadPackages here is typically just Microsoft.Quantum.Standard. That package has always been loaded synchronously at startup, and I wanted to avoid changing that behavior here. Anything in DeferredLoadPackages (which typically just contains Microsoft.Quantum.Standard.Visualization) will be loaded asynchronously.
Although I certainly agree with your point, and we could consider moving Microsoft.Quantum.Standard into DeferredLoadPackages. But, for example, in some places we rely on python -c "import qsharp" to help populate NuGet caches, etc., and changing the load strategy for Microsoft.Quantum.Standard may "break" that.


Changing font-weight for input and changing padding size for links

Hello, I have two questions

the font-weight for input, select, textarea tags seems to be too less that it is too difficult to use/recognize. So I want to change the font weight to bold. That is
from this to:

this:

Also, I think we need to increase the padding-bottom for links so that it looks good. That is
from this to:

this:

if this looks fine, I will make a PR.

Also I am having a small doubt. in the search page: http://api.cltk.org/search
there is only text called "Search" in it. Where as in home page: http://api.cltk.org/
there is an input for search. So can't we just use the same input for search in search page.

Thank you.

Thanks, @achaitanyasai--#1 above with the visuals for input values sounds good. As far as /search versus the search on the homepage, let's make a new issue.

Thanks, I will submit PR with in 10 hours.

Looks good--thanks!

Thank you sir.


Fix invalid iterator resulting in 100% cpu load

@simonschmeisser 's commit 218a71845b375b7038f50cd1b61027d88c06aee5 unfortunately resulted in an invalid iterator. Each detach lead to one 100% cpu load thread, which caused the driver to die after some operations.
This pull request fixes this issue and fixes some copy-paste errors and spellings.

I haven't observed that but your PR totally makes sense.
I don't think my half-way fixed version actually produces invalid iterators, but it skips one after each deletion which also doesn't make sense. The state before my PR however produced invalid iterators ...
Also thanks a lot for spotting those small but painful other errors. I'll test this on Friday or early next week and approve then

That's really weird. Maybe it depends on the gcc version or similar.. We had this issues on ubuntu 18 with gcc 7.5.0. Here, it really ends up in kind of an infinite loop (without looping anymore). It never ends the Detach function, but each Detach call spawns a new 100% cpu load thread (observed with htop).
Thank you very much for testing! Would be interesting to know which ubuntu/gcc version you are using.

I also asked on stackoverflow how this can happen

I also see this warning after each grip:
Received command error code: E_CMD_PENDING

but I guess that is another issue?

Thanks for testing. We've also seen this warning during the gripping, but also before. Seems like a kind of "device busy" message while executing commands.

@ipa-lth friendly ping :-)


Change AWS credentials priority, to accept AWS_SESSION_TOKEN

What is the purpose of this change? What does it change?
Change AWS credentials priority, to accept AWS_SESSION_TOKEN.
This  functionality was added in https://github.com/restic/restic/pull/1647 and broken in https://github.com/restic/restic/commit/b358dd369b2cd01e354211d32455446e3e070c8b
Was the change discussed in an issue or in the forum before?
https://github.com/restic/restic/pull/1647
Checklist

[x] I have read the Contribution Guidelines
[x] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Codecov Report

Merging #1882 into master will decrease coverage by 4.54%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1882      +/-   ##
=========================================
- Coverage   51.04%   46.5%   -4.55%
=========================================
  Files         169     169
  Lines       13099   13099
=========================================
- Hits         6687    6092     -595
- Misses       5406    6054     +648
+ Partials     1006     953      -53

Impacted Files
Coverage Δ

internal/backend/s3/s3.go
59.39% <100%> (-0.76%)
:arrow_down:

internal/backend/b2/b2.go
0% <0%> (-78.54%)
:arrow_down:

internal/backend/swift/swift.go
0% <0%> (-78.45%)
:arrow_down:

internal/backend/gs/gs.go
0% <0%> (-73.61%)
:arrow_down:

internal/backend/azure/azure.go
0% <0%> (-69.46%)
:arrow_down:

internal/backend/swift/config.go
36.95% <0%> (-54.35%)
:arrow_down:

internal/archiver/file_saver.go
85.96% <0%> (-2.64%)
:arrow_down:

internal/archiver/blob_saver.go
95.06% <0%> (+2.46%)
:arrow_up:

Continue to review full report at Codecov.

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

done

Thanks!


Review the ReadMe

Update the info on the Read-me, do some re-up to make it current with the API.

This is actually done and the PR has been accepted...


fix(mempool): panic when app's CheckTx returns an error while rechecking

Hotfix for #2225

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[ ] Title follows the Conventional Commits spec

What's the reason to add errors to the checkTx() application call? I don't understand the point for the current behavior: "if one of the transactions from the mempool returns an error, no more transactions are checked".
This behavior is dangerous, since it's not what usually a developer expects from a callback. To stop iterations, usually there is a boolean that indicates the caller should stop, not an error.
@hvanz @mzabaluev

I agree that CheckTx should not return an error but we would need to change the interface for that. For the moment I'd recommend to return always nil, and encode any error in the CheckTx response.

Here's a test that fails with and without the return line: https://github.com/cometbft/cometbft/compare/hvanz/mempool-test-recheck-return-error

It sounds like a more complex solution is needed. Let's continue the conversation in the issue.


Deploy Flows via Terraform

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
Please do not leave +1 or me too comments, they generate extra noise for issue followers and do not help prioritize the request.
If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Terraform Version

Affected Resource(s)

prefect_2.3.0

Terraform Configuration Files

resource "prefect_flow" "flow" {
  name         = "cowsay-flow"
  workspace_id = var.prefect_cloud_workspace_id
  tags         = ["tf-test"]
}
#
resource "prefect_deployment" "deployment" {
  name                     = "Cowsay Example Deployment"
  description              = "string"
  workspace_id             = var.prefect_cloud_workspace_id
  flow_id                  = prefect_flow.flow.id
  entrypoint               = "cowsay_example.py:cowsay_flow"
  tags                     = ["staging"]
  enforce_parameter_schema = false
  parameters = jsonencode({
    "creature" : "cow",
    "message"  :"Hello World!"
  })
  parameter_openapi_schema = jsonencode({
    "type" : "object",
    "properties" : {
      "creature" :  {"type": "string"}
      "message" : {"type": "string"}
    }
  })
  paused              = false
  storage_document_id = "eb5bbb9b-b499-4895-8932-8e366aa4b627"
  version             = "v1.1.1"
  work_pool_name      = "main"
  work_queue_name     = "default"
}

Debug Output

Panic Output
╷
│ Error: Normalized JSON Unmarshal Error
│
│   with module.flows.prefect_deployment.deployment,
│   on ../../modules/flows/main.tf line 7, in resource "prefect_deployment" "deployment":
│    7: resource "prefect_deployment" "deployment" {
│
│ json string value is null
╵
2024-10-28T21:47:39.083Z [DEBUG] provider.stdio: received EOF, stopping recv loop: err="rpc error: code = Unavailable desc = error reading from server: EOF"
2024-10-28T21:47:39.085Z [DEBUG] provider: plugin process exited: path=.terraform/providers/registry.opentofu.org/prefecthq/prefect/2.3.0/linux_amd64/terraform-provider-prefect_v2.3.0 pid=92
2024-10-28T21:47:39.086Z [DEBUG] provider: plugin exited
[01JBAFEXR70ME9JN0GVB6ES058] Unexpected exit code when applying changes: 1

Expected Behavior
Deployment should succeed
Actual Behavior
Failure as reported in Panic Output
Steps to Reproduce

terraform plan
terraform apply

References
https://github.com/PrefectHQ/terraform-provider-prefect/issues/238

Thanks for the report here, I'm working on replicating. What I've found so far is that you apparently need to specify all of the following:

job_variables
parameters
parameter_openapi_schema

They're optional, but it looks like we need to improve the way we unmarshal them in case they're not configured. Will look into it a bit more and put up a PR.

Thanks again @vnagaraj, we've just released https://github.com/PrefectHQ/terraform-provider-prefect/releases/tag/v2.5.1 which should address this. Please reach out if you run into any other issues.


Extention not in Root/Extentions

Hi,
first of all thanks for that add-on, it will allow our users to easily create different types of materials and later choose them in our online editor.
The first try was successful, simple cube and sphere:

But when I use a different model there is no "extensions" object on the root level, the meshes have it though:

Material_Variants.zip

As you can see, I also tried the newest gltf exporter but with no luck

So I figured its working with the gltf exporter 1.7.33 (https://github.com/KhronosGroup/glTF-Blender-IO/tree/blender-v3.0-release) but not with older nor newer versions ;)

A colleague and me tried to debug the problem.
The gather_gltf_hook was renamed to gather_gltf_extensions_hook in the exporter: https://github.com/KhronosGroup/glTF-Blender-IO/commit/28ac0983cf65144116817ff027e0a77a517e5e9a
Changing gather_gltf_hook to gather_gltf_extensions_hook in __init__.py makes the exporter functional again.


Updating NAT gateway EIPs during VPC update

This is PR is to add support for updating NAT gateways in an existing VPC according to changes in disco_vpc.ini. NAT gateways in a live VPC can now be switched between dynamically generated EIPs and pre-allocated EIPs, or between two different sets of pre-allocated EIPs.

Overall looks good

👍


Hyprland does not like monitor descriptions with commas

Hyprland Version
Hyprland, built from branch  at commit v0.29.1  ().
Bug or Regression?
Bug
Description
I have a monitor that, for some reason, provides a description that includes a comma:
HYC CO., LTD. DUAL-DVI (DP-3 via HDMI)
Hyprland's configuration parser does not like that as it's a comma separated value :laughing:
Anyway, I can get by using the connector ID instead, but any way to fix that so I can either escape the comma manually or put the entire description in quotes to parse it properly?
How to reproduce
Use a monitor that provides a comma in the description and try to add it to your monitor configs.
Crash reports, logs, images, videos
No response

Just a note, this monitor is one of those old korean models that lets you overclock it by providing the kernel with a fake EDID, so this issue is definitely not something that most people would likely run into.
However, I think it might be a good idea to allow escaping commas out regardless for any other part of the config to prevent any similar issues. That's mostly why I'm opening this ticket.

#2457


Create azure-webapps-node.yml

Exit
Em seg., 22 de ago. de 2022 20:01, thomas000274 @.***>
escreveu:

You can view, comment on, or merge this pull request online at:
https://github.com/Hack-with-Github/Awesome-Hacking/pull/142
Commit Summary

343f9be
https://github.com/Hack-with-Github/Awesome-Hacking/pull/142/commits/343f9bef71bfaa60a81cc9fae9ef9ae5701b8433
Create azure-webapps-node.yml

File Changes
(1 file
https://github.com/Hack-with-Github/Awesome-Hacking/pull/142/files)

A .github/workflows/azure-webapps-node.yml
https://github.com/Hack-with-Github/Awesome-Hacking/pull/142/files#diff-1fb87ddee3523b7c03d0b53301850aa22f8290ad676b3746f18229649ea7f6f5
(79)

Patch Links:

https://github.com/Hack-with-Github/Awesome-Hacking/pull/142.patch
https://github.com/Hack-with-Github/Awesome-Hacking/pull/142.diff

—
Reply to this email directly, view it on GitHub
https://github.com/Hack-with-Github/Awesome-Hacking/pull/142, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A2ME6AXZII5R6CEOS4GPMBLV2QBDTANCNFSM57JLMY7Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Sets CI's ANSIBLE_VERBOSITY to 4

With the default verbosity it is very difficult to triage issues hit
during CI runs. The additional outputted information should help.

I had hijacked this PR and set the WIP label while working on a CI build issue. I have resolved the CI issue and removed the WIP label so this PR is ready for review.

We will start with 2. If we go to 4, we could provide a log trimming tool.

Lets give this a try, and we can tweak it if we need later.

Maybe I am late on the game, but the verbosity also depends on callback plugin. I haven't checked the yaml callback display code for a while.


[BUG] Unable to specify schema for specific media type if same as response_class

Describe the bug
When the (inferred) response_class matches the media type key in responses, I can't override the schema for that media type—it just uses the response_model value. The media type schema override does work for a media type that doesn't match response_class.
To Reproduce
Steps to reproduce the behavior with a minimum self-contained file.
Replace each part with your own scenario:

Create a file with:

from typing import Union, List
from enum import Enum

from fastapi import FastAPI, Header
from starlette import status
from pydantic import BaseModel, Field

app = FastAPI()

class DogsJson(BaseModel):
    dogs: list

class DogFeature(BaseModel):
    type: str = Field("Feature", const=True)
    geometry: dict
    properties: dict

class DogsGeoJson(BaseModel):
    features: List[DogFeature]

class DogsRequestAcceptHeaders(Enum):
    json = "application/json"
    geojson = "application/geo+json"

@app.post(
    "/dogs",
    response_model=Union[DogsJson, DogsGeoJson],
    responses={
        status.HTTP_200_OK: {
            "content": {
                DogsRequestAcceptHeaders.json.value: {
                    "schema": {
                        "$ref": f"#/components/schemas/{DogsJson.__name__}"
                    }
                },
                DogsRequestAcceptHeaders.geojson.value: {
                    "schema": {
                        "$ref": f"#/components/schemas/{DogsGeoJson.__name__}"
                    }
                },
            }
        },
    },
)
async def get_dogs(
        accept: DogsRequestAcceptHeaders = Header(DogsRequestAcceptHeaders.json),
) -> Union[DogsJson, DogsGeoJson]:
    dogs = ["dog_1", "dog_2"]

    if accept == DogsRequestAcceptHeaders.json:
        return DogsJson(dogs=dogs)
    else:
        dog_features = [DogFeature(geometry={}, properties={"name": dog}) for dog in dogs]
        return DogsGeoJson(features=dog_features)

from uvicorn import run
run(app, host="localhost", port=8000)

Open the browser at localhost:8000/docs.
Inspect the 200 response for /dogs and note that media type application/json's schema is still derived from response_model and not overridden by what's in responses. Also note that application/geo+json is overridden as expected.

Expected behavior
OpenAPI schema declarations in responses should override any other conflicting definitions.
Screenshots
application/json schema matches response_model when it should match what's in responses.

application/geo+json schema matches what's in responses, as expected.

Environment

OS: macOS
FastAPI Version: 0.48.0
Python version: 3.8.1

@maflaven I’m facing the same issue, did you find a workaround?
@tiangolo I believe one way of fixing such issues could be to make responses the only place to put the responses declaration.
Would you accept a feature request (and possibly a PR) aiming at making response_* overridable in the responses dict (and even deprecate them).

@johanfleury nothing yet

I could not reproduce this in newest version.
@maflaven could you confirmed this?

@eth3lbert yes this is fixed!

Thanks for the help here everyone! :clap: :bow:
Thanks for reporting back and closing the issue @maflaven  :+1:


Advanced Tab - Valid Proxy IP

If you are using a load balancer or a proxy, you will need to whitelist its ip in order to enable ip checks for logins or migration webservices.

Suggested: If you are using a load balancer or a proxy, you will need to whitelist its IP in order to enable IP checks for logins or migration webservices.

done


Add SonarCloud integration

Please describe.
SonarCloud integration will help patch code smell and other related bugs.

https://github.com/SonarSource/sonarcloud-github-action


Disposed logger still reacts

Testing #172057
Given this code:
// The command has been defined in the package.json file
// Now provide the implementation of the command with registerCommand
// The commandId parameter must match the command field in package.json
let disposable = vscode.commands.registerCommand('test-telemetry-api-ts.helloWorld', () => {
	// The code you place here will be executed every time your command is executed
	// Display a message box to the user
	vscode.window.showInformationMessage('Hello World from test-telemetry-api-ts!');

	const logger = vscode.env.createTelemetryLogger({
		sendEventData: function (eventName: string, data?: Record<string, any> | undefined): void {
			console.log("sendEventData: ", eventName, data);
		},
		sendErrorData: function (error: Error, data?: Record<string, any> | undefined): void {
			console.log("sendErrorData: ", error, data);
		}
	}, {});
	logger.dispose();

	console.log("logger.isErrorsEnabled", logger.isErrorsEnabled);
	console.log("logger.isUsageEnabled", logger.isUsageEnabled);

	logger.onDidChangeEnableStates(e => console.log("onDidChangeEnableStates", e));

	logger.logUsage("myEventName token=1234 /Users/bpasero/Development/Microsoft/vscode/src/vs/platform/extensionManagement/common/extensionsProfileScannerService.ts", { "key": "/Users/bpasero/Development/Microsoft/vscode/src/vs/platform/extensionManagement/common/extensionsProfileScannerService.ts", "common.extversion": "roflcopter" });
	logger.logError(new Error("foo token=1234 /Users/bpasero/Development/Microsoft/vscode/src/vs/platform/extensionManagement/common/extensionsProfileScannerService.ts"));
	logger.logError("hello error token=1234 /Users/bpasero/Development/Microsoft/vscode/src/vs/platform/extensionManagement/common/extensionsProfileScannerService.ts", { "key": "token=1234" });
});

I was expecting my event handlers not to fire since the logger is disposed, but they still fire.

This still repros for me 🤔


Minigui ponder broken in study mode

expected: ponder reads <N> works
actual: all moves get 8 reads, ponder does not continue

my bad, was using outdated command.


Uncaught Error: Module did not self-register.

[Enter steps to reproduce:]

Reveal a file in the tree
Duplicate the file from the tree view
Open the newly duplicated file

Atom: 1.50.0 x64
Electron: 5.0.13
OS: Mac OS X 10.14.6
Thrown From: Atom Core
Stack Trace
Uncaught Error: Module did not self-register.
At electron/js2c/asar.js:138

Error: Module did not self-register.
    at process.func [as dlopen] (electron/js2c/asar.js:138:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:881:18)
    at Object.func [as .node] (electron/js2c/asar.js:147:18)
    at Module.load (internal/modules/cjs/loader.js:701:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:633:12)
    at Function.Module._load (internal/modules/cjs/loader.js:625:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:14:16)
    at customRequire (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:1:695414)
    at get_KeyboardLayoutManager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2715520)
    at get_manager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2715622)
    at Object.getCurrentKeyboardLayout (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2715748)
    at e.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1047867)
    at KeymapManager.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1228085)
    at KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1226070)
    at WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:284829)

Commands
     -1:32 tree-view:reveal-active-file (div.title)
     -1:06.7.0 core:copy (input.hidden-input)

Non-Core Packages
go-plus 6.1.0

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/20222. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Create baseline unit tests

After #10 is complete, I'll be able to build some baseline unit tests, and possibly integration tests.

Unit tests for the data validation functions have now been added.

More specific tests that involve mocked data (to test json parsing etc) will be managed in separate issues.


Fixed some typos in UX Live Components code examples

Hey!
I fixed some code examples typos in the official UX Live components documentation.
Thanks in advance.

Q
A

Bug fix?
yes

New feature?
no

Issues

License
MIT

Thanks Ricardo!


Add contributors section to intro.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Kill Juttle.adapters

Just use the adapters module directly where it is needed.
This is a little step on the path leading to dismantling the Juttle global object.
This change will require adapting the adapters.
Part of work on #97.

@demmer

I'm ok with this change -- note that juttle-service will also need to be updated.


Slider detects clicks in incorrect position when it was scrolled

Here is an example:
https://jsfiddle.net/ux93ar4u/
Just scroll it to the right and try to click at "80", "90". It selects "50" and "60" instead.

You can try using real-time, like https://jsfiddle.net/ux93ar4u/1/.

Great, thank you, @NightCatSama . Any possible performance issues?

Each time you drag or click, the component will calculate and update the width and height. I think it should not have serious performance problems.

Ok, nice to hear. It works now.


⚠ wiki.minetest.net has degraded performance

In e9d5278, wiki.minetest.net (https://wiki.minetest.net/Main_Page) experienced degraded performance:

HTTP code: 200
Response time: 2682 ms

Resolved: wiki.minetest.net performance has improved in c344cf6 after 13 minutes.


Cannot download long videos from Youtube because of DNS flood

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.03.14. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.03.14

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

Didn't check because this is irrelevant to the issue.
What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
15:05 tmp: ./youtube-dl -v --hls-prefer-native --format 300 --ffmpeg-location /d/port/libav-11.3/usr/bin/  -o v6.avi --no-mtime 'https://www.youtube.com/watch?v=QL0nLqDdKdI'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'--hls-prefer-native', u'--format', u'300', u'--ffmpeg-location', u'/d/port/libav-11.3/usr/bin/', u'-o', u'v6.avi', u'--no-mtime', u'https://www.youtube.com/watch?v=QL0nLqDdKdI']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.03.14
[debug] Python version 2.7.10 (CPython) - CYGWIN_NT-5.1-2.5.2-0.297-5-3-i686-32bit
[debug] exe versions: avconv 11.3, avprobe 11.3
[debug] Proxy map: {}
[youtube] QL0nLqDdKdI: Downloading webpage
[youtube] QL0nLqDdKdI: Downloading video info webpage
[youtube] QL0nLqDdKdI: Extracting video information
[youtube] QL0nLqDdKdI: Downloading m3u8 information
WARNING: [youtube] QL0nLqDdKdI: Skipping DASH manifest: ExtractorError(u'Cannot decrypt si
gnature without player_url; please report this issue on https://yt-dl.org/bug . Make sure
you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-
dl with the --verbose flag and include its complete output.',)
[debug] Invoking downloader on u'https://manifest.googlevideo.com/api/manifest/hls_playlis
t/id/QL0nLqDdKdI.1/itag/300/source/yt_live_broadcast/requiressl/yes/ratebypass/yes/live/1/
cmbypass/yes/goi/160/sgoap/gir%3Dyes%3Bitag%3D140/sgovp/gir%3Dyes%3Bitag%3D298/hls_chunk_h
ost/r4---sn-1uxaxjvh-axql.googlevideo.com/ei/V6mvWon2Oofp7QSF2ragBA/force_finished/1/playl
ist_type/CLEAN/gcr/ru/initcwndbps/10830/mm/32/mn/sn-1uxaxjvh-axql/ms/lv/mv/m/pl/24/dover/1
0/keepalive/yes/mt/1521461539/ip/<IP_ADDRESS>/ipbits/0/expire/1521483192/sparams/ip,ipbits
,expire,id,itag,source,requiressl,ratebypass,live,cmbypass,goi,sgoap,sgovp,hls_chunk_host,
ei,force_finished,playlist_type,gcr,initcwndbps,mm,mn,ms,mv,pl/signature/1AFC483209EEA66BC
FB93D8146BB164EBFF61665.70AB5D835FFCDDA56EDF29F1AC0FE56E5DCB6B9D/key/dg_yt0/playlist/index
.m3u8'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 2880
[download] Destination: v6.avi
[download]   2.5% of ~1.34GiB at 10.66MiB/s ETA 01:09
ERROR: Interrupted by user

Description of your issue, suggested solution and other information
Some videos are stored as collection of small segments at Youtube. When downloading a large video from Youtube that contains many segments, Youtube-dl sends new DNS request for every segment even if the host name is the same. Example video URL: https://www.youtube.com/watch?v=QL0nLqDdKdI . At the moment of writing this video contains 2880 segments, all located on the same host. On fast Internet channel youtube-dl is downloading 4 segments per second, making 4 DNS requests per second with the same host name. I am using DNS server <IP_ADDRESS> thats stops responding after sending approximately 1000 requests with the same name in short interval of time. After trying several more times, youtube-dl exits with error:
[download]  26.2% of ~1.35GiB at  9.74MiB/s ETA 06:30ERROR: unable to download video data:
 <urlopen error [Errno 8] Name or service not known>

Please note that the error message is confusing, because it doesn't say anything about the real problem.
The DNS response TTL field value for r4---sn-1uxaxjvh-axql.googlevideo.com equals 12 minutes and 3 seconds.
I have searched issues for "dns cache", "long videos", "getaddrinfo failed".
In an issue #13734 @dstftw wrote that this is an issue with a DNS server. I cannot agree with that. Sending thousands of similar queries looks more like flood or DoS attack, no wonder that DNS server stops responding. I think this is an issue with youtube-dl. I looked for an options to make pauses between downloading segments, but couldn't find any. I have also searched for "dns cache" in README but didn't find anything.
Please note that the issue is with sending too much DNS request. Using or not using persistent HTTP connections have no relation to it. So the issue #14523 is irrelevant.
I found a similar issue with downloading long video #15641 . DNS fllod might the the problem of errors there. Also, an issue #15746 - looks like the same problem.
Using --hls-prefer-native or --hls-prefer-ffmpeg doesn't make any difference. ffmpeg does the same thing.
My suggestion is to implement a simple DNS cache that caches the IP address of a host or DNS response for the lifetime of a process. It is highly unlikely that IP address would change during download so there is no need to handle expire time. It is much easier than make user configure local DNS cache and somehow point youtube-dl to it.
To repoduce:

make sure you have fast enough Internet connection
set your DNS server as <IP_ADDRESS> and disable local DNS cache
run command  ./youtube-dl -v --hls-prefer-native --format 300 --ffmpeg-location <location>  -o v1.avi --no-mtime 'https://www.youtube.com/watch?v=QL0nLqDdKdI'
after several minutes server <IP_ADDRESS> will stop responding

Also I have found an issue #4966 where there might be the same problem, with ffmpeg sending too many requests.

I have tried to use --sleep-interval options but it doesn't work for fragments. The option --limit-rate set to some small value like 100K works, but the download becomes very slow.

Encountering a similar issue with TBS/Adult Swim.


Extract text segmentations from search plugins and merge into codebase

Tasks

[ ] Research integration of PaddleHub & Postgres tsvector

[ ] https://github.com/PaddlePaddle/PaddleHub#the-simplest-cases-of-chinese-word-segmentation
[ ] https://github.com/PaddlePaddle/Paddle.js

[ ] Extract segmentations from search plugins

References
https://github.com/thematters/lambda-handlers/blob/main/sql/create-table-search-index-article.sql#LL30C1-L33C1
currently all tsvector are automatically generated by Postgres with column of GENERATED ALWAYS AS (to_tsvector('jiebacfg', title)) STORED
ALTER TABLE search_index.article ADD COLUMN title_jieba_ts tsvector
  GENERATED ALWAYS AS (to_tsvector('jiebacfg', title)) STORED ;
ALTER TABLE search_index.article ADD COLUMN summary_jieba_ts tsvector
  GENERATED ALWAYS AS (to_tsvector('jiebacfg', summary)) STORED ;
ALTER TABLE search_index.article ADD COLUMN text_jieba_ts tsvector
  GENERATED ALWAYS AS (to_tsvector('jiebacfg', text_content_converted)) STORED ;

need to research if these can be done in program with
INSERT search_index.article (title_jieba_ts, ...)

to_tsvector( ${PaddleHub lac.cut (title)} )

...

and the SQL at search time is:
FROM search_index.article, plainto_tsquery('jiebacfg', searchKey) query
WHERE
  (query @@ title_jieba_ts OR query @@ summary_jieba_ts OR query @@ text_jieba_ts)`

because of no jiebacfg, this has to update to something like:
SELECT ...
FROM search_index.article, to_tsquery(preparedSearchKey) query
WHERE
  (query @@ title_tsv OR query @@ summary_tsv OR query @@ text_tsv)`

will not be able to use plainto_tsquery but have to parse and prepare the query key:
any search phrase like 南京市长江大桥 will have to be parsed in app to something like 南京市 & 长江 & 大桥

https://www.notion.so/matterslab/Search-with-in-application-word-segmentation-83c6a9fe3e7742cb9e7238c57f7c4815?pvs=4