Question: How do I determine what deprecated features I am using?

Title: How do I determine what deprecated features I am using?
Envoy version: Varies as we rollout between 1.10.0, 1.11.0, 1.11.1
Description:
We have envoy instances emitting the envoy.runtime.deprecated_feature_use metric but I am unable to figure out what features we are using that are deprecated.
The breaking change policy mentions the following:

during the first release cycle after a feature has been deprecated, use of that feature will cause a logged warning, and incrementing the runtime runtime.deprecated_feature_use stat.

However, I am unable to see these logs. These envoys are using the INFO log level.
How I tried to figure this out:

Compare the deprecation log with our config.
Run envoy locally with debug log level and attempt to use a deprecated feature.
View the logs of all envoys deployed for any mention of deprecated.

Questions:

At what log level can I expect to see the logged warning on use of a deprecated feature?
How do you expect people to go from runtime.deprecated_feature_use to the specific feature that is deprecated?

Relevant Links:

https://www.envoyproxy.io/docs/envoy/latest/intro/deprecated.html
https://github.com/envoyproxy/envoy/blob/master/docs/root/intro/deprecated.rst

You should see WARN logs for every deprecated feature you are using: https://github.com/envoyproxy/envoy/blob/cb15cc3d2010aff77d6e022ddf6d723fa8becbc0/source/common/protobuf/utility.cc#L199
It's possible there is some path that is not warning correctly, but we might need more info to debug. cc @alyssawilk who worked on this awesome feature!

Would it be good to have a stat around this so that we do not have to grep through fleet of Envoys?

@ramaraochavali there is a stat already?

@mattklein123 Ok. Cool. Did not check the stat.

Yeah, it should be in warning logs.   I'll add that to the docs for runtime.rst as it's been asked once or twice.
It would be lovely if there were a stat per deprecated feature but I'm not sure if that'd be too expensive.  Internally our equivalent used power of two backoff on both logs and stats and I don't think Envoy has that equivalent yet

So I think now that the stat points to the log messages (and gives you something to grep for).
The logs both point to the deprecation docs (https://www.envoyproxy.io/docs/envoy/latest/intro/deprecated) which say how proto field X is replaced by proto field Y, and also to the runtime overrides ("using runtime overrides for deprecated features") for temporary workarounds.  Is there anything else you think we can do to make this easier / more informative?

Thanks for clarifying.
We are not seeing any log messages from envoy containing the word "deprecated". I determined this by looking for envoy instances that are consistently outputting the envoy.runtime.deprecated_feature_use metric and searching the logs.
Is it possible that there are code paths that send the metric but do not log?
Some more specifics:

all envoys are statically configured via json file.
the envoys that are sending the most of these metrics seem to be starting envoy more than others. i.e. It seems that this metric is emitted on startup of envoy.

If it helps, I can send you an example of the static config we use. I would prefer to send this via a private channel if possible.

I did some further digging and found that the instances on envoy 1.11 and higher were not sending this stat. I have confirmed this locally using our config with the three versions I mentioned earlier. Only version 1.10.0 emitted this deprecated metric.
It seems that this was caused by this bug you mention.
Thank you for taking the time.


1.3.0 Release

TODO:

[ ] Merge #1267
[ ] Merge #1538
[ ] Upload to CDN
[ ] Update SRI hashes

Coverage remained the same at 79.49% when pulling ba9638f5d623d7313ab1845863d3f81a2743f701 on 1.3.0-release into 9274fac6d100a6300eb2faa4c94aa2431b1ca4b0 on dev.


InteractiveImage not updating plot

It is my impression that InteractiveImage(bokeh_plot, callback) is supposed to update the plot using the callback function when zooming but I'm not able to make that happen even by running the example notebooks.
There's no errors, the zoomed image is just becoming more pixelated.
Bokeh version: 0.12.1
Datashader version: 0.3.0

You'll need datashader version 0.3.2 (conda install -c bokeh datashader) to use Bokeh 0.12.x.

Thanks.

It seems to work if callback is a function that calculates new values, but if we have data stored in e.g. a dataframe such as
def update_image(x_range, y_range, w, h, df=df):
    cvs = ds.Canvas(x_range=x_range, y_range=y_range, plot_width=w, plot_height=h)
    agg = cvs.line(df, 'f_timestamp', 'x', agg=ds.any())
    return tf.interpolate(agg)

p = base_plot()
InteractiveImage(p, update_image, df=df)

the plot is not updated with higher resolution details when zooming. Not sure if this is intended?

The various example notebooks should demonstrate updating with more resolution on zooming for dataframes, so no, what you are describing is not intended.  You may find a useful message on the JavaScript console.  I'm assuming the function you wrote works fine on its own, without InteractiveImage?

Using the JS console I discovered the ipywidgets  didn't even load but upgrading to version 5.0.0 seems to fix that. In the JS console I get this error when zooming:
Uncaught TypeError: Cannot read property 'get' of undefined
  update_plot @ VM323:18
  c @ main.min.js:95

update_image works fine on its own.

I found the error. I was importing from datashader.callbacks import InteractiveImage which should be from datashader.bokeh_ext instead.

Ah, ok.  That used to to be the right way to do it, and maybe you had a stray .pyc file around that was making that fail to raise the appropriate error.  Glad it works now!


log entry replicated to the quorum may not be committed

Sorry For my poor english.
Steps to Reproduce：

Assume S5 become leader initially
New Request on S5, and S5 replicate the new log(index 1) to followers
After S5 finish replication and before S5 send next heartbeat(with commit index=1), stop S5 immediately
Assume S2 Become leader later
S2 probe match indexs of followers, and find that the log with index=1 has been already replicated to quorum, BUT it's term is not the term of S2, so this log will not be commited unless we create a new request.

Hi @jonahgao. The condition you've removed is necessary for the correctness of the protocol. See Section 3.6.2 "Committing entries from previous terms" in my dissertation for the explanation, including a diagram on the previous page.
I guess you're probably bothered by how the entries from the earlier term are not committed until new requests have arrived. In the core Raft algorithm in isolation, this doesn't really pose a problem. It's only a problem if you're trying to serve read-only requests without using the log, which this visualization does not include. Many implementations do care, and they tend to have any new leader append a blank "no-op" entry to its log. You can simulate this behavior in the visualization by clicking on the new leader and submitting a client request.

Thank you very much for such a detailed reply. The condition is necessary indeed.


Create a UI or any other visual method for quickly displaying useful information

Information like type of fix, number of satellites and uncertainty values need to be viewed frequently. Having some sort of visual output for quickly glancing this information will be useful. It  should also include information from the IMU like the current attitude. Ideally it would be best to have this as a RViz or Mapviz Plugin.
Msg types that can be used are GPSFix and Imu.

Resolved https://github.com/spraybot/mapviz/commit/05657075069ebdfbac94ab3e225fb84376394c65 https://github.com/spraybot/spraybot_utils/commit/95e392b75c0f151a1e851a88be9dbf2ef5ddc32b


as6 : a bug for oj

Guess that OJ uses gcc 4.7 which has problem with std::tie.Trying to fix it.

Note : this bug was caused by a GCC compiler bug.


LCFS - Optimize SQLAlchemy Session Management for Enhanced Database Efficiency in FastAPI Integration

Describe the Task
Review and refactor the backend Python FastAPI integration with the PostgreSQL database focusing on the SQLAlchemy session management. The current implementation is leading to redundant database commits within a single request cycle, which unnecessarily increases the number of database calls. The aim is to streamline session management to ensure optimal performance and database interaction efficiency.
Purpose
The main goal of this task is to eliminate unnecessary database commits that occur multiple times within a single request cycle. By refining the session management strategy and adhering to a more efficient pattern, we aim to reduce the load on the database, enhance the application's performance, and ensure the scalability of our backend services. This task will contribute to improving the overall reliability and efficiency of our application.
Acceptance Criteria

[ ] Review the current SQLAlchemy session management setup and identify instances of redundant database commits.
[ ] Refactor the session management to utilize get_async_db_session for creating and managing database sessions, ensuring a single commit per request cycle.
[ ] Remove the commit_to_db() method from the base repository pattern, or refactor it to align with the new session management strategy.
[ ] Ensure that the refactoring does not introduce any regressions or functionality loss in the application.
[ ] Update relevant documentation and comments in the codebase to reflect the new session management approach.
[ ] Conduct thorough testing to verify that the changes lead to the expected reduction in database calls and overall performance improvement.

Additional Context

The current setup utilizes a custom commit_to_db() method in our base repository for committing transactions, which has contributed to the issue of multiple commits within a single request cycle.
The proposed solution involves refactoring the session management to leverage the AsyncSession from SQLAlchemy and ensure that commits are handled efficiently at the end of each request cycle.
Example code snippets provided indicate how the get_async_db_session should be used to manage sessions and commits.
This task requires an understanding of asynchronous database operations and the FastAPI framework's dependency injection system.

This refactor has resulted in re-writing our service methods to allow for un-committed session objects. Estimate bumped from 3 to 5.


Add function to check if stateroot is booted

Complete the isRollbackAllowed logic
Add interface for RPMOSTreeClient in order to be able to mock it in tests
Add uber-go/mock package
Add genmock to Makefile to generate mock files

/cc @jc-rh @donpenney @pixelsoccupied

/hold
wait till ostree query is fixed

rebased
@tsorya @donpenney @jc-rh

/unhold

/lgtm


bower registry

Any reason this is still marked as private in the bower registry?

@arnold-almeida this is likely because I haven't registered this in the bower registry.


OSO about page refresh

What is it?
Some of this content will be redundant with the new pages we're creating. We should trim this and make it prettier

Made a couple minor edits to Plasmic v 74.0.3
Will close for now until we have clearer ideas on what we want to do here


Please remove column logs.

File: sqlx-core/src/mysql/cursor.rs
println!("[def] {:?}", column);
Please remove the column logs.

Oops. That's embarrassing. Fixed and out as v0.3.2.


Markdown Rendering Issue

Environment
python3.10, gardio 3.40.0
Issue
When browsing some questions (I forget which), I encounter this backend gardio error:
ValueError:
\ket{\psi} = \frac{\ket{00} + \ket{01} + \ket{10}}{\sqrt{3}}
^
ParseFatalException: Unknown symbol: \ket, found '\'  (at char 0), (line:1, col:1)

try installing httpx==0.25.2.


Update and fix GitHub Actions release workflow

The current workflow passes a build argument PUPPET_VERSION whilst the Dockerfile does not use it, instead it expects PUPPETSERVER_VERSION.
Reworked the workflow so that the same job can be used to release different versions of Puppetserver depending the tag that triggered the workflow run.

@ldaneliukas may you please sign all you commits?

i will close this, the build has changed at all and this is not needed anymore. if you think otherwise let us know and we can repoen this.


uv no longer requires editable installs

Reacting to https://github.com/Lightning-AI/litgpt/pull/952#issuecomment-2016972782

Blocked by https://github.com/Lightning-AI/litgpt/pull/952#issuecomment-2017075104

UV was recently updated, so the issue should be resolved (after --prerelease=allow is removed).
I reran CI workflows and despite the issue arising at the "Install minimal dependencies" all the tests passed 🤷♂.
Additionally, since now UV seems to work a bit more stable, maybe we should also use it in the GPU workflow?

Additionally, since now UV seems to work a bit more stable, maybe we should also use it in the GPU workflow?

That would be very welcome Andrei


Redmi note 10s does not support

Redmi note 10s does not support this

Hi, thanks for the report.
Which model exactly do you have, there are multiple Redmi Note 10S models.
If you're not exactly sure, you can copy your device information inside the app and paste them here for me to make it easier.

I've excluded all of the available models as most likely none of these phones support dimming the flashlight.


Channel discord with errors

The option to add a channel where bot errors, for example from the logs.txt file, will be sent.
Example:
The bot receives an error and sends it to the error channel along with various additional information.

Hey, thank you for your suggestion. I understand some setups might not have permissions for the bot to write to a logs.txt file so I'll look into adding that new option to sort this.

Hey, this is now added as a new config option in this commit: 0a4f785


Making Todo List App with Javascript

I am a GSSOC'21 Participant. I would like to add the Todo List. I want to work on this Issue.
Please assign me this issue..

I am a GSSOC 2021 participant , please assign this project to me.

Since this project already exist, so i am closing this issue.


product_code and unique_id return " None "

Pitupongs-Air:~ Best3140$ python3 testNin1.py
The Legend of Zelda™: Breath of the Wild
None None
2017-03-03 00:00:00
1
ESRB Everyone 10+
https://www.nintendo.com/fr_CA/games/detail/the-legend-of-zelda-breath-of-the-wild-switch

Demo Available : False
DLC Available : True
Nintendo Switch Online Required : True
Save Data Cloud Supported : True
Pitupongs-Air:~ Best3140$

Hi @best3140, sorry for the late response.
Sadly, nintendo has removed the product code from the NA eshop sites so there is no way for me to fetch it. Also, since the unique_id is constructed based on that code, that one is also unavailable.

many thanks, hope you can find the way soon ^^
On Mon, 26 Apr 2021 at 16:56, Fede Calendino @.***>
wrote:

Hi @best3140 https://github.com/best3140, sorry for the late response.
Sadly, nintendo has removed the product code from the NA eshop sites so
there is no way for me to fetch it. Also, since the unique_id is
constructed based on that code, that one is also unavailable.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fedecalendino/nintendeals/issues/9#issuecomment-826694215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMKQTVLC7DBXCQ6X5NQAB3TKU2FLANCNFSM43D7MVFQ
.


ApplicationModuleTest context not reused if inherited config from parent

Discussed in https://github.com/spring-projects/spring-modulith/discussions/343

Originally posted by JiriFrankCZ October 23, 2023
Hi all, i´m just wondering whether is correct to bootstrap Spring again and again if we use abstract parent for multiple test with @ApplicationModuleTest. Better explain code:
Base parent class
@Transactional
@AutoConfigureMockMvc
@ActiveProfiles("test")
@RecordApplicationEvents
@TestInstance(Lifecycle.PER_CLASS)
@TestClassOrder(ClassOrderer.OrderAnnotation.class)
@TestMethodOrder(OrderAnnotation.class)
@Import({IntegrationTestConfig.class})
@NestedTestConfiguration(EnclosingConfiguration.OVERRIDE)
public abstract class BaseIntegrationTest {

Base module class
@ApplicationModuleTest(extraIncludes = "config")
public abstract class BaseLightingModuleTest extends BaseIntegrationTest {
    @MockBean
    protected MessageSender messageSender;
    @MockBean
    protected PositionService positionService;
}

And real tested class in given module:
class PositionEventListenerTest extends BaseShelfModuleTest {

I suppose I´m doing something wrong, or this is bad approach, but I found it useful to use it this way. And from my point of view context should be reused, eg. to bootstrap spring context once per module via this abstract parent class. But because of my bad configuration or lack of support in Spring/Modulith, it is currently not easily possible, because conext is bootstraping again and again with every single class like PositionEventListenerTest.

Can you elaborate in which context you'd like to see reuse? @AMT will definitely need to create different contexts based on the module the actual test class is located in. It would be helpful to learn more about for which exact scenarios you see the context recreated (classes? if so, in which packages? methods?).

Closing this due to the lack of feedback. Feel free to reopen with additional information provided.


Create a gallery page for applications built using AutoGen

What feature would you like to be added?
The link should be /gallery to show case the applications.
Why is this needed?
For community

Travel chatbot example - https://github.com/lordlinus/travel-chatbot

@lordlinus Can you please add the GitHub topic autogen-sample so it can be discovered here -> https://github.com/topics/autogen-sample


[Report] Use unambiguous digit group separator

No-break thin space instead of locale-dependant symbol

Refer to this link for build results (access rights to CI server needed):
http://jenkins.tribler.org//job/Graphalytics_pull_request_tester/36/
Test PASSed.

On my machine it seems that this Unicode symbol is converted to an ASCII space during HTML generation, but I like it better than a comma/full stop/whatever else locales dictate. If I ever feel inclined to look at Unicode support in Thymeleaf, I might fix this, but for now I am merging.

I checked it in Firefox. When I copy the number from the report, the delimiter is a narrow no-break space (U+2027). It also works in practice: Firefox will not break the number when zooming in.

You are correct, it is working fine. I am somehow still expecting Unicode characters in my terminal to show up as garbage, even though Unicode-compatible terminals have been around for ages... Never mind me.


[Enhancement] Preview for KCL code run

Enhancement

The feature can only be triggered when a valid KCL file is in the active editor:

If the KCL file is under a compile stack (kcl.mod or kcl.yaml), the preview button and command will be shown. And the YAML representation of the stack configuration will be previewed.
If the KCL file is not under a stack, the preview button and command will be shown. And the YAML representation of that single file will be previewed.
If the configuration is not valid (contains some syntax error or could not pass the check validation):

the preview command in the palette, the button in the toolbar, and the command in the editor tab will still be shown but should be disabled(with gray font and can not be triggered). And there should be a warning on that command or button to intimate that the code needs to be fixed first.
If the user triggers the preview with the shortcut, then the preview panel should be displayed with empty data content, but a warning to intimate the code needs to be fixed first.

In other situations, that preview button and command will not be displayed.

support viewing the corresponding YAML data of a KCL configuration

single Yaml view: could be triggered by:

shortcut: ⇧⌘V
right-click on the editor Tab and select Open Preview (⇧⌘V)
Command Palette: KCL: Data Preview (⇧⌘V)

side-by-side live view. could be triggered by:

shortcut: ⌘K V
clicking the preview button in the editor toolbar
command Palette: KCL: Open Preview to the Side⌘K V

Reference
Similar feature for helm chart template(from vscode kubernetes extension maintained by microsoft), could be triggered by command: Helm: Preview Template

This feature is implemented using vscode webview, and the yaml format preview doesn't support highlight or folding.

Working on it @Peefy

Thank you! @ibishal I will assign it to you.


Email for requesting preview only available to Microsoft employees.

I got this error when trying to request an upgrade to the azure cache for redis service (from 4.x to 6.0)

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

ID: 49ca97a8-1b7e-e5b4-3c0e-df5abf71de9a
Version Independent ID: 37a81e16-457a-5944-6e30-857e81b44eac
Content: Set Redis version for Azure Cache for Redis (Preview)
Content Source: articles/azure-cache-for-redis/cache-how-to-version.md
Service: cache
GitHub Login: @yegu-ms
Microsoft Alias: yegu

@tstahnke2 We are checking this and we will get back to you.

@tstahnke2 The AzureCacheFT is just member of the main<EMAIL_ADDRESS>Your email has still gone through to relevant people related to Azure Cache.
There is no action needed for this. We will be going ahead and closing this issue for now.


Profile Lookup

Challenge Name
Profile Lookup Challenge( JavaScript)
Issue Description
I have doubt in your testing..as it seems my answer is correct and is not allowing to pass
Browser Information

Browser Name, Version: Chrome latest version
Operating System: Windows 8.1

Your Code
//Setup
var contacts = [
{
"firstName": "Akira",
"lastName": "Laine",
"number": "0543236543",
"likes": ["Pizza", "Coding", "Brownie Points"]
},
{
"firstName": "Harry",
"lastName": "Potter",
"number": "0994372684",
"likes": ["Hogwarts", "Magic", "Hagrid"]
},
{
"firstName": "Sherlock",
"lastName": "Holmes",
"number": "0487345643",
"likes": ["Intriguing Cases", "Violin"]
},
{
"firstName": "Kristian",
"lastName": "Vos",
"number": "unknown",
"likes": ["Javascript", "Gaming", "Foxes"]
}
];
function lookUpProfile(firstName,prop ){
for( var i = 0; i < contacts.length; i++ ){
if( contacts[i].firstName == firstName ) {
if( contacts[i].hasOwnProperty( prop ) ) {
return contacts[i][prop];
} else {
return "No such property";
}
}
return "No such contact";
}
}
// Change these values to test your function
lookUpProfile("Kristian","lastName");
Screenshot

@sawanmakwana305 Thank you for reporting this issue. Your code is incorrect. Please use the Gitter chat rooms linked below for asking for help. GitHub issues are not for coding help.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.

Hi,
Thanks for reporting this issue.
Here are some next steps that you should follow before we can help you with your query:

Check these troubleshooting steps first.
Please use the Chat Rooms or try looking through our forum for help with a specific challenge and any coding assistance.

IMPORTANT:
Please NOTE that this tracker is for reporting bugs and enhancements to freeCodeCamp's code base only.
Happy Coding!


How to extend the user registration UI?

I would love to see a small section regarding how to extend the user registration UI once one has scaffolded a ASP.NET Core 3.0 SPA Identity-backed project.

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

ID: 21cb7455-3305-9316-5440-bdc22947a82d
Version Independent ID: 137d4b94-7b26-3911-b22d-42c754a95fc1
Content: Introduction to authentication for Single Page Apps on ASP.NET Core
Content Source: aspnetcore/security/authentication/identity-api-authorization.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @javiercn
Microsoft Alias: scaddie

I'm assuming the scaffolding explained here does not work https://docs.microsoft.com/en-us/aspnet/core/security/authentication/scaffold-identity?view=aspnetcore-3.1&tabs=netcore-cli

I haven't tried the IdentityServer's UI scaffolding. I'm not sure if it would work.
https://github.com/IdentityServer/IdentityServer4.Quickstart.UI

Can I extend the UI with anything other than Razor pages (Angular, for example)?

Moved to master issue: Authentication and authorization for SPAs #16406


Added support for sessionStorage.

This PR has been created on behalf of @jsmunroe, Added support for sessionStorage

Hi @sushinpv thanks for the useful react-secure-storage package.
Do you have any plans on merging this PR? That would be great 🙏

@davidher28 Hi
We will be merging this PR by 24th March and along with that, we will be also exposing all the internal functions too.

@davidher28 We are testing out the entire use cases and we want to make sure that, the patch will scalable and it won’t break any of our existing use cases, due to this reason! It will take few more days to merge this PR

i really need this, cant wait

Is there an estimate on when this PR will be merged? Would love to transition into session storage.
Thanks!

We will merge this PR by this week

I would like to inquire if there are any follow-up actions regarding this discussion? I have also encountered a situation involving the use of session storage. Thank you, everyone.

Hi All, Sorry for the deploy happened on implementing Session Storage support, We will be closing this PR ASAP


Testnet chainspec for Light Client Demo

Is your feature request related to a problem? Please describe.
We at Parity are preparing a light client demo for Bitcoin Miami 2022. We want to use a substrate connect light client to move kBTC. It would be nice to do that on the testnet that's live at https://testnet.interlay.io/bridge
With substrate connect, light clients connect to a relay or parachain without syncing with it. All they need is the correct chainspecs. Greg forwarded some chainspecs from Interlay devops:

https://interlay-public.ams3.digitaloceanspaces.com/staging-latest-parachain-1.7.4.json
https://interlay-public.ams3.digitaloceanspaces.com/rococo-local-0.9.16.json

However these chaispecs don't give us the necessary information on bootNodes. They are:

empty on relay chainspec
<IP_ADDRESS> on para chainspec

Without the correct information on bootNodes, we can't get the light client to connect to Interlay testnet, because they don't know how to find the chain.
Describe the solution you'd like
We need chainspecs with bootNodes that are prepared to launch nodes outside of Interlay infrastructure.
Describe alternatives you've considered
Worst case, we will launch our own testnet. But using Interlay infrastructure would be very helpful to us.
Additional context
Here's a screenshot of the burnr wallet in action:

The plan is to have kBTC instead of WND, and interact with the testnet chain over the light client that we will launch from the provided chainspecs.

The chainspec files have been updated. Please download them again and test it out.

We added an additional boot nodes on both the parachain and the relay chain; The chainspec files have been updated.

Thank you @ns212 .
Will test new chainSpecs and let you know of the outcome.

@ns212 Can you please also add the following settings in collator (and relay if possible) so that they are listening for WebSocket connections? In order to do that - please add --listen-addr /ip4/<IP_ADDRESS>/tcp/30333/ws --listen-addr /ip6/[::]/tcp/30333/ws. This must be done on before the -- that separates the collator CLI options from the relay chain options, if there is any.
This is necessary, because we would like to connect to the collator from substrate-connect, which runs in the browser. Browsers only allow webpages to open WebSocket connections, meaning that the node we want to connect to needs to listen for WebSocket connections.

@wirednkod Thanks for the clarification. The chainspec files are updated and include 2 nodes (both ipv4 & ipv6) with websocket p2p ports.
Please check on your end if this setup works.

Hey,

https://interlay-public.ams3.digitaloceanspaces.com/staging-latest-parachain-1.7.4.json

When connecting to the bootnodes in this chain spec, I'm getting a genesis block hash mismatch between the bootnodes and the chain spec.
In other words, what is under raw doesn't match what the chain spec of the bootnodes have under raw.

From my investigation, it seems that everything is working more or less fine, you just need to wait a very long time (several minutes) for smoldot to initialize.
The problem is that the relay chain has a new session every 10 blocks. Compare this for example to Polkadot, where there's a session every 2400 blocks. The block at every session needs to be verified by smoldot, so in total this takes a lot of time.
One way to mitigate this would be to generate a checkpoint of the relay chain by calling the sync_state_genSyncSpec RPC call. See this script for example. This will generate a chain spec that contains the current finalized block, meaning that you will start syncing from that block and not from genesis.

@ns212 any updates?

From my investigation, it seems that everything is working more or less fine, you just need to wait a very long time (several minutes) for smoldot to finish syncing the relay chain.
The problem is that the relay chain has a new session every 10 blocks. Compare this for example to Polkadot, where there's a session every 2400 blocks. The block at every session change needs to be verified by smoldot, so in total this takes a lot of time.
One way to mitigate this would be to generate a checkpoint of the relay chain by calling the sync_state_genSyncSpec RPC call. See this script for example. This will generate a chain spec that contains the current finalized block, meaning that you will start syncing from that block and not from genesis.

Does smoldot have to verify both the relay and the parachain state on the session change or is that only required for the relaychain?

@tomaka @bernardoaraujor this is the chain spec generated by the script https://interlay-public.ams3.digitaloceanspaces.com/rococo-local-0.9.16-syncspec.json

Does smoldot have to verify both the relay and the parachain state on the session change or is that only required for the relaychain?

The parachain state is simply downloaded from the relay chain.


DO NOT MERGE Try reproduce the problem for different UBIs

chown: changing ownership of '/etc/pki/entitlement': Operation not permitted

Served the purpose. Closing.


OPNsense 23.7 - OpenVPN Instances - Reneg. Time & Auth Token Lifetime set to 1 Day results in error

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
As already mentioned above: When I try to set Renegotiate time & Auth Token Lifetime to 86'400 sec (= 1 Day), OPNsense displays the error invalid integer value on the right side of the input fields:
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:

Go to 'VPN' -> 'OpenVPN' 'Instances [new]'
Create a new Server Instance
Try to set 'Renegotiate time' and 'Auth Token Lifetime' to a high value as e.g. 86'400 sec. and click on 'Save'
The Error is displayd on the right side of the input fields.

Expected behavior
If there is a limit how high those values can be, I would expect an understandable error message, like You must define a value between 0 and XXXX
If those values can be set that high, I would expect no error message displayed at all ;)
Describe alternatives you considered
I have set those values now to 65'000 which doesn't results in those errors
Screenshots

Relevant log files

Additional context

Environment
Hardware:
OPNsense DEC2750
Software:
OPNsense 23.7-amd64
FreeBSD 13.2-RELEASE-p1
OpenSSL 1.1.1u 30 May 2023

https://github.com/opnsense/core/commit/9dc2f01f274061be4cbe3a01b77e32e95ffee7bd should remove the upper bounds check, integer max should be ok for these as far as I could find.


Improvement suggestion

When I try to handle errors with ? I recieve this:
error[E0277]: the trait bound `bybit::http::Error: std::error::Error` is not satisfied
   --> src/bybit_exchange/bybit_inter.rs:19:61
    |
19  |             let tickers = api.fetch_tickers("BTCUSDT").await?;
    |                                                             ^ the trait `std::error::Error` is not implemented for `bybit::http::Error`

Is there some way to impliment std:error:Error for bybit:http:Error ?

Hi, thank you for creating this issue and the corresponding PR!
I have left some feedback on your PR, it would be great if you could implement the requested changes.
If not, just let me know and I will implement the changes myself, no biggie!

Hi, thank you for creating this issue and the corresponding PR!
I have left some feedback on your PR, it would be great if you could implement the requested changes.
If not, just let me know and I will implement the changes myself, no biggie!

Hi, trying to implement your changes without removing the desired functionality was difficult because I am basically guessing and trying to get it to work. I have now added macros for ws and rest errors due to a stack overflow (unironically), but now it seems alright. However, I am guessing you may have implemented this module in a program of yours, so I appreciate testing of this pr. You can just replace the git repo in your Cargo.toml to my one instead.


Test-DbaDatabaseOwner and unknown login

Bug Report
General Troubleshooting steps

[x] Verified running latest release of dbatools?
[x] Verified errors are not related to permissions?
[x] Can duplicate in new/fresh PowerShell.exe session?

Version Information

Operating System (Name|Version): Windows 10
PowerShell Version: PS 5.1
SQL Server (Edition|Version): SQL2016/DEV

Steps to Reproduce
Run this command:
Test-DbaDatabaseOwner -SqlInstance localhost\SQL2016 -TargetLogin UnknownUser
Result:
Invalid login: UnknownUser
At line:91 char:6

                throw "Invalid login: $TargetLogin"

                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (Invalid login: UnknownUser:String) [], RuntimeException
FullyQualifiedErrorId : Invalid login: UnknownUser

Feature Request
Catch error with a better error message

This item seems to be resolved in the latest version of dbatools


Add some scripts for help verifying the RC

download-rc-directory.sh downloads all files in directory from RC URL to current directory
verify-release-file.sh verifies signature, MD5, SHA of the archive file

Requires gpg and wget.
Might be better to add guide to verify RC in DEVELOPER.md or so. Welcome any ideas!

+1


Update PPP to include PRs

Codecov Report
Base: 60.66% // Head: 60.66% // No change to project coverage :thumbsup:

Coverage data is based on head (d44e774) compared to base (c7501a9).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #251   +/-   ##
=========================================
  Coverage     60.66%   60.66%
  Complexity      553      553
=========================================
  Files           103      103
  Lines          2044     2044
  Branches        206      206
=========================================
  Hits           1240     1240
  Misses          736      736
  Partials         68       68

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

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


Buggy collisions with CharacterBody3D (CylinderShape3D) against StaticBody3D (ConcavePolygonShape3D)

Tested versions
Reproducible in v4.2.1.stable.mono.official [b09f793f5]
System information
Godot v4.2.1.stable.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated AMD Radeon (TM) R9 390 Series (Advanced Micro Devices, Inc.; 27.20.20913.2000) - Intel(R) Core(TM) i5-4430 CPU @ 3.00GHz (4 Threads)
Issue description
When a CharacterBody3D with CylinderShape3D collides against particular triangles on a StaticBody3D with ConcavePolygonShape3D, the collision results after invoking MoveAndSlide() and GetSlideCollision() are sometimes extremely incorrect.
The reproduction project includes an example of a CharacterBody3D on the edge of a StaticBody3D, moving only very slightly into it, which causes an extreme shift (The collision has a depth of .244, despite the cylinder only having a radius of .25. This becomes significantly more problematic if collision response is used to control character velocity)

Steps to reproduce
Run the example project. The CharacterBody3D is moving into the StaticBody3D at a very low speed, but the collision depth+normal pushes it extremely far to the left, as displayed through the console logs.
Minimal reproduction project (MRP)
CollisionDepthIssue.zip

Possibly related to #87521 or #77051


Update dependabot.yml

Dependabot was complaining about the client and server packages not being in the workspace root. I believe that since / is a workspace it will automatically monitor all packages dependencies (we'll see if that's true).

Indeed it looks good after merge.


Provide the optimization plugin for Next.js

Is your feature request related to a problem? Please describe.
The Current SWC plugin doesn't work well with recent Next.js.
In order to reduce bundle size in Next.js application, we have to use its babel plugin so we have to put custom babel config file, which disables SWC compiler.
Describe the solution you'd like
My suggestion is providing the optimization plugin for Next.js, which wraps babelOptimizerPlugin.
We don't have to care about SWC version mismatch.
Also, we don't have to prepare custom babel config file, so we can still get benefits from Next.js's SWC compiler.
(I'm not familier with Next.js plugin system, so this explanation might be wrong.)
Describe alternatives you've considered
No response
Is your feature request related to a problem? Please describe.

https://github.com/dotansimha/graphql-code-generator/issues/9450

Hey @kaorun343! The SWC issue is indeed quite a complex topic, read https://github.com/swc-project/swc/issues/8315 for more context.

My suggestion is providing the optimization plugin for Next.js, which wraps babelOptimizerPlugin.
We don't have to care about SWC version mismatch.

Unfortuntely, that's not possible, babelOptimizerPlugin comes from our client-preset package which we maintain in Javascript, Next.js compiler plugins are only written in Rust, because SWC, the underlying compiler is written in Rust, that's why we need to have a completely separate package with separate challenges for the SWC plugin.
We can't simply port it, or wrap it for Next.js usage.

Hi, @YassinEldeeb!
Thanks for your reply! 👍
I checked the other packages that provides plugins for Next.js, and I found they uses babel internally. So, it might be possible to do so, I think.

vanilla-extract provides @vanilla-extract/next-plugin , which is based on @vanilla-extract/integration and this uses babel internally

https://github.com/vanilla-extract-css/vanilla-extract/tree/master/packages/next-plugin

Kuma UI also provides @kuma-ui/next-plugin , which uses @kuma-ui/webpack-plugin and this uses babel internally, too.

https://github.com/kuma-ui/kuma-ui/tree/main/packages/next-plugin

They seems setting their original webpack loader in webpackFinal function.


Error to install bcrypt via npm

Hello, e have a error to install bcrypt via npm:
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No README data
npm http GET https://registry.npmjs.org/bcrypt
npm http 304 https://registry.npmjs.org/bcrypt
npm http GET https://registry.npmjs.org/bindings/1.2.1
npm http GET https://registry.npmjs.org/nan/1.8.4
npm http 304 https://registry.npmjs.org/nan/1.8.4
npm http 304 https://registry.npmjs.org/bindings/1.2.1

><EMAIL_ADDRESS>install /home/rocketcode/trimtrim/node_modules/bcrypt
> node-gyp rebuild

gyp ERR! build error
gyp ERR! stack Error: not found: make
gyp ERR! stack     at F (/usr/local/lib/node_modules/npm/node_modules/which/which.js:43:28)
gyp ERR! stack     at E (/usr/local/lib/node_modules/npm/node_modules/which/which.js:46:29)
gyp ERR! stack     at /usr/local/lib/node_modules/npm/node_modules/which/which.js:57:16
gyp ERR! stack     at Object.oncomplete (fs.js:107:15)
gyp ERR! System Linux 3.13.0-29-generic
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/rocketcode/trimtrim/node_modules/bcrypt
gyp ERR! node -v v0.10.26
gyp ERR! node-gyp -v v0.12.2
gyp ERR! not ok
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is most likely a problem with the bcrypt package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls bcrypt
npm ERR! There is likely additional logging output above.

npm ERR! System Linux 3.13.0-29-generic
npm ERR! command "/usr/local/bin/node" "/usr/local/bin/npm" "install" "bcrypt"
npm ERR! cwd /home/rocketcode/trimtrim
npm ERR! node -v v0.10.26
npm ERR! npm -v 1.4.3
npm ERR! code ELIFECYCLE
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/rocketcode/trimtrim/npm-debug.log
npm ERR! not ok code 0

What is this?

gyp ERR! build error
gyp ERR! stack Error: not found: make

You need to install your distribution's make package.
something like sudo apt-get install make


Fallback Method in @Fallback not being called when combined with @Retry

Environment Details

Helidon Version: 2.2.0
Helidon MP
JDK version: JDK 11.0.7
OS: Windows 10
Docker version (if applicable):

Problem Description
The fallback method defined in @Fallback is not getting called after reaching max retries when the target method is also annotated with @Retry. When the target method is only annotated with @Fallback, the fallback method gets called when the execution of the target method failed.
Expected behavior:
The fallback method get called after the max number of retries has been reached.
Steps to reproduce
With Resource as:
`
private final BusinessService businessService;
@Inject
public GreetResource(BusinessService businessService) {
        this.businessService = businessService;
}

@POST
@Path("/process")
@Consumes(MediaType.APPLICATION_JSON)
@Produces(MediaType.APPLICATION_JSON)
public Response processReversal(JsonObject jsonObject) {
    String requestId = jsonObject.getString("requestId");
    String result = businessService.callUnreliableApi(requestId);
    return Response.status(Response.Status.NO_CONTENT).build();
}`

And Business service as:
`@ApplicationScoped
public class BusinessService {
@Fallback(fallbackMethod = "testFallback", applyOn = RuntimeException.class)
@Retry(maxRetries = 3, retryOn = RuntimeException.class )
public String callUnreliableApi(String requestId)  {

	HttpURLConnection conn = null;
	int responseCode = -1;

	try {
		URL obj = new URL("http://localhost:8080/api");
		conn = (HttpURLConnection) obj.openConnection();
		conn.setRequestMethod("GET");
		responseCode = conn.getResponseCode();
		String responseMsg = conn.getResponseMessage();
	}
	catch (IOException e) {
		throw new RuntimeException("IOException ocurred");
	}finally {
    	if (conn != null) {
    		conn.disconnect();
    	}
    }

	if (responseCode != 200 ) {
		throw new RuntimeException("Unsuccessful call");
	}

    return "API called successfully!";
}

public String testFallback(String requestId) {
	return "API call failed!";
}

}`

@aivarhe Yeap, there's definitely a problem. It relates to threads and losing track of the request scope. As a possible workaround try setting delay=0 and 'jitter=0in@Retry`. Probably not ideal for you, but a good test. That should force a different threading logic.

@spericas I set the settings as you suggested and I can confirm that the fallback method does get call. In our case, this workaround wouldn't work as we want to have some delay between retries

Trying to get https://github.com/oracle/helidon/pull/2748 into 2.2.1 to be released this week. Should fix this issue.

PR #2748 merged.


Custom neat grid

Probably want to add custom gutters for the lols

oops…

With markup

And the first example markup

looks great! 👡


Save into a folder does not handle duplicate file names in different relative folders correctly

Expected Behaviour
When saving a file model into a folder, models with duplicate file names but stored in different relative folders are exported correctly, and data is not lost.
The definition of correctly is open to interpretation, but I would expect that when saving a file with duplicate names, would not result in losing n-1 of those files. This could be solved by keeping the original relative folder structure intact, or by adjusting the names when this edge case occurs.
Actual Behaviour
When saving a model to a folder, the file structure is flattened. This is problematic when a model contains duplicate file names in different relative paths. An example of this behaviour is provided in bug/save-folder-does-not-handle-relative-paths-with-duplicate-file-names-correctly.
Although this is a bit of a contrived example, it does illustrate the problematic behaviour. To my understanding this is a perfectly valid model (in the sense of file structure, the actual contents are most likely humbug).
Discussion
I would personally argue for a separation of concerns, and properly identify the following three concepts:

Save: stores a model as is, given its current file path.
Save as: stores a model in the given path, adjusting it's file path (i.e. it modifies the state of the model)
Export: stores a model in the given path, without adjusting it's file path (i.e. it does not modify its state).

Technically, we already support these concepts in HYDROLIB, but they are not explicitly defined. Save is done when calling the save method without file path. Save as corresponds with changing the file path and then calling save without a file path. And lastly export is done when calling saving with a file path.
I would argue the export function should be separated from the current save function, and the save with a file/folder path should act as a save-as. This would mean we shouldn't directly manipulate the filepath any more (i.e. it should be a readonly property).
However I understand this is more of a philosophical discussion that might be out of scope for this issue.

Based on the online discussion we've had.
Features

[MUST/SHOULD] User should be able to (edit and) save 1 file in an hierarchy in it's current location. So load a MDU and only edit and save the linked structures.
[NICE] On export, saving all files to a new folder, only the files underneath/relative to the root (xml/mdu?) are moved with it, while paths above it are linked/save as is.

Interface
Personally, I expected the following methods:

Save: stores a model as is, given its current file path.
Save(recurse=true): stores a model and all its children, given their current filepaths (could also be named save_tree)
Export(folder=x): stores and moves a model, including its children to a new location (modifying state)

Note this is different from the current functionality, where the export was mixed with the save method.
You can always edit filepaths manually by manipulating the filepath attribute.
Issues
The issue is mostly with the export methods, as multiple questions arise:

Which files are saved to a new location (see nice to have)
Where to save when duplicate names arise
Where to save, based on original relative structure (when read) or on a default one (from scratch)

Advice

KISS
Keep the user responsible for changing state and paths
Make the export work without state syncs or backlinks


Exceeding Azure Maps S0 usage - API behaviour?

For Azure Maps S0 consumers, have you documented how the API behaves when usage limits are exceeded? Is throttling applied internally or does the client receive an appropriate error response?

Hello @jimborogers  Thank you for your feedback! This channel is for driving improvements towards MS Docs. In order to best address your question with the right team, could you provide us with the URL of the MS Doc that you were following, if any?

Hi there, this is the page I am referring to: https://docs.microsoft.com/en-us/azure/azure-maps/choose-pricing-tier

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

ID: a40139e7-7b9e-e3c5-4cf6-860fefdbcf79
Version Independent ID: 768e160e-b9c5-c8ad-9d4a-56eff82ebd38
Content: Choose the right pricing tier for Azure Maps
Content Source: articles/azure-maps/choose-pricing-tier.md
Service: azure-maps
GitHub Login: @walsehgal
Microsoft Alias: v-musehg

Hi @jimborogers, the S0 clients receive a 403 response when usage limit is exceeded. You can have a look here (https://docs.microsoft.com/rest/api/maps/search/getsearchfuzzy#response) to see all the response types.


MacOS support?

Hi there -
Will this be updated soon to include MacOS support? (or is there a way to get it to work?)
Thanks!

Closing as a duplicate of a newer issue with more info

I can't find the newer issue - do you have the issue#?


wmii.nix: fix module args

Without this custom module args will not work.

Thanks! My fault for undoing the ellipses in my previous commit but I live and learn

@KoviRobi, maybe merge?..


Support for attaching to a parent window

This is a tracking issue for supporting passing in a parent handle when creating a window. This functionality will enable users to use Druid as a child window for platform-native windows not created by Druid.
Progress
See rest of issue for context.
General idea for implementation, might not be 100% accurate. Will update this section if needed.

Platform-independent

[ ] Figure out what type to use for the window handle in the user-facing API (not c_void *)
[ ] User-facing API: Add an Option<T> field in WindowDesc, and create a public function for the user to pass the window handle to. WindowDesc::build_native should essentially end up passing the field through to the platform-specific WindowBuilder.

Windows

[ ] Add an Option<T> field in WindowBuilder, and create a function for WindowDesc::build_native to call to set it.
[ ] When creating the window (create_window in WindowBuilder::build), pass the value if the new field is Some, otherwise pass 0 if the new field is None.

Mac OS

[ ] Add an Option<T> field in WindowBuilder, and create a function for WindowDesc::build_native to call to set it.
[ ] When creating the window (WindowBuilder::build), attach to the given parent window if the new field is Some, otherwise create a normal root window if the new field is None. See Raph's vst branch for ideas.

X11

[ ] Create an entire X11 backend (see #475, at least the "bare minimum" bullet point)
[ ] Add an Option<T> field in WindowBuilder, and create a function for WindowDesc::build_native to call to set it.
[ ] When creating the window, attach to the given window if the new field is Some, otherwise create a normal root window if the new field is None. See this code for ideas.

GTK

[ ] (Is there an equivalent for GTK? Implement if so.)

Background / Motivation
This functionality has been desired by many in the RustAudio group for a while now: it is a requirement for creating GUIs in the VST plugin format, one of the most popular plugin formats in modern Digital Audio Workstations (DAWs). Currently, no GUI crate in the Rust ecosystem fully meets all of the requirements to create GUIs for VST plugins, so this will be a step towards making Druid the first one.
(A "plugin", here, is basically a third-party musical "device" that the user can load into the DAW to synthesize or alter sounds. Plugins typically have a GUI that enables the user to change parameters for this device so that they can tweak the sounds/effects as they see fit, among other features.)
In DAWs following the VST specification, the DAW is responsible for handling the lifecycle of all plugin GUI windows: creation, opening, closing, etc. When the user loads a plugin into the DAW, the DAW probes the plugin to figure out if it supports having a GUI window. If so, the DAW creates the window for the plugin, then passes a handle to that window (the "parent window handle") to the plugin so that the plugin can attach to that window.
This parent handle is always a handle to a platform-native window (HWND on Windows, NSView on Mac OS, or an X11 window ID on X11 systems), which can be used in the platform-native window creation function to "attach" to the parent window. (examples: Windows, Mac OS, Linux/X11)
Therefore, the ideal workflow from a VST developer's perspective would be:

DAW loads the plugin, opens a window, and passes a handle to that window to the plugin.
The plugin code takes that parent handle and passes it to Druid via Druid's app creation API; Druid then handles the rest of the window creation/attaching process.
The plugin code now has access to a window that it can draw on, using standard Druid API calls. The GUI portion of the plugin now functions as a typical Druid application.

Discussion on impact to Druid's current project roadmap scope
Before I get too much farther into the weeds here, I should mention: while I would love to see some crate within the Rust ecosystem handle our use case, I still consider this to be pretty low priority as far as Druid is concerned. I know there's a lot of other stuff on your plate for the near-future, and I would like to disrupt that roadmap/timeline as little as possible.
I'm not specifically requesting the Druid core developers to implement this feature for us; some of us in the RustAudio group are willing to do the grunt work. That said, we might have some specific design questions as they relate to the architecture/vision of the crate, PRs take time to review, and all code in a codebase has some sort of maintenance cost.
This feature might be the first of a few for implementing VST support. I'm not sure what else will be needed afterwards, but I feel like parent window handle support is a self-contained enough feature to not be too disruptive, while allowing us to prove that Druid is a viable option for us, and also allowing us to dig in and identify future requirements.
If you feel like any aspect of us implementing VST support is too disruptive, please let us know and we can hold off. :) Similarly, if you feel like some aspect is completely out-of-scope for Druid as a whole, please let us know that as well.
Implementation details / proposal
Essentially, we have a handle to a parent window, and we want to pass that handle all the way down to druid-shell's platform-specific window creation functions (Windows, Mac OS).
In vst-rs, the API currently gives you a parent: *mut c_void. I don't find this to be a particularly good public API for use in Druid, so I propose using something like rust-windowing's raw-window-handle instead to pass the parent handle around. Alternatively, we could make our own thing if raw-window-handle doesn't suit our needs, or if we want to reduce our dependencies. Either way, it would be up to the developer to transform this *mut c_void (or whatever they have, for other use cases) into the correct handle type.
I think the best place to expose this API to users would probably be in WindowDesc. We could make the function something like:
pub fn parent(self, parent: RawWindowHandle) -> Self

If the user does not call this function, Druid should operate as normal (creating its own window instead of attaching to a parent window).
I think the rest of the implementation is basically just plumbing some data around, but let me know if there's some roadblocks I didn't foresee.
NOTE: X11 support needed to get VSTs working, see #475 for the tracking issue + context.
Unresolved issues and questions

DAWs expect to be the main controller of its threads; for example, they expect to be able to call something like window.open() and have the control given back to the DAW when that function is done. Therefore, handling events inside a runloop, which never gives back control to the DAW, is problematic. It looks like we might be able to get around this by making an Application manually instead of launching an AppLauncher, and just not creating that RunLoop. Would there be any issue doing it this way? Would events still be handled? (here's where my knowledge of how GUIs work drops off.)
I'm not sure that it's possible to do any of this with GTK. More research needed (anyone familiar with it to know if this would work?).

Update:

In GTK + X11 (i.e. not all gtk platforms) there is: GtkSocket, and GtkPlug which implement the XEmbed spec (caveat, not exactly familiar with any of these APIs/specifications).

Not sure if there's an equivalent for GTK on other platforms.
Miscellaneous resources and links

The Rust implementation of VST 2.4 that we are specifically targeting is RustAudio's vst-rs crate, but this feature should work with anything that has a similar use case.
Raph prototyped VST support in druid for the Mac OS platform about a year ago in his vst branch. While a lot has changed in the crate since then, it still might be a good reference (especially for Mac OS implementation specifically).
I have created a few prototypes in the past as a proof-of-concept that platform-native windowing is viable in Rust for VST use cases. This repo might be a useful reference for implementing this feature in Druid. rtb-rs might also be a good reference.
I (along with some others in RustAudio) have written up a few pages of background research on the VST GUI problem space in vst2-gui-research.
@wrl's rutabaga GUI toolkit (in C) is being used in a production VST (which he also made), so it might prove to be a valuable resource as well.

Note: raw_window_handle::RawWindowHandle is what wgpu uses, which allows binding to wgpu. In that respect, choosing that handle type and exposing it would make things progress too for #891, although for that issue the workflow is reversed (embedding child window into druid, instead of embedding druid into parent window).

Raw window handle support for Windows and Mac has been merged, see #1586.


[Event Request] table 5409 "Prod. Order Routing Line" - OnAfterCopyFromPlanningRoutingLine

There is an event for OnAfterCopyFromRoutingLine(Rec, RoutingLine),
but not for OnAfterCopyFromPlanningRoutingLine.
Could you please add that event, we have an urgent need for it. Thank you

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Vio output latency at higher resolutions

Hi spectacular AI team, thanks for your work.
I have a question. while using tracking api on an OAK-D pro W camera, I noticed that when running the vio pipeline in a ros node with an input resolution of 800p, there is a consequent delay in the output, compared to the default input resolution of 400p (please see the graph below) .

red curve is the speed output of a camera running at 400p, blue one is the speed output of a second cam running at 800p doing the same movement at the same time.
Do you know where this delay comes from and if there is a way to reduce it ?

Hello,  it works well, thanks.


DNM bump ansible testing version to 2.2.1 for the 2.2 environment

2.2.1 is not released yet, but this will be what we should rely on once it gets out.

Everything failing is expected since 2.2.1 is not out yet.

@alfredodeza so we should just wait for the release right?

Depending on the gdeploy situation (https://github.com/gluster/gdeploy/issues/253), it might be nice to test with both <IP_ADDRESS> and <IP_ADDRESS>, so we can evaluate the feasibility of keeping parity with what they're doing.

@leseb yes, we should wait until the release is out.
@ktdreyer we can certainly add variants (we do have them available in tox.ini right now). But maybe we could set those to run daily (vs. for every pull request)

jenkins test this please

Marking for backport so we'll test stable-2.1 with this version of Ansible as well.


Does it even work?

Hi @lucas23456 what is going wrong in your case?
Is it location-based, marker-based or NFT?
Thanks.

I'm also running on my fedora "new location based" example. I got different results on samsung galaxy S7 and galaxy S22.
In Galaxy S7 (older) some examples works but in Samsung S22 nothing appear in camera.
Why this  different behavior?
I think also that examples dirs need a review...I can help you. We can keep update on Telegram or give me your favourite channel. I really want this library goes good. I'm ready to collaborate. (Full stack web-developer here)

I'm also running on my fedora "new location based" example. I got different results on samsung galaxy S7 and galaxy S22. In Galaxy S7 (older) some examples works but in Samsung S22 nothing appear in camera. Why this different behavior? I think also that examples dirs need a review...I can help you. We can keep update on Telegram or give me your favourite channel. I really want this library goes good. I'm ready to collaborate. (Full stack web-developer here)

Yes, we can collabrate together. I can send you my telegram username:) but  not here.

Hi @lucas23456 what is going wrong in your case?
Is it location-based, marker-based or NFT?
Thanks.

The examples that in  Aframe website  doesn't work

Hi, it's possible these need to be updated, @nicolocarpignoli would know more.
For up-to-date documentation I would recommend you look at the docs here: https://ar-js-org.github.io/AR.js-Docs/
I am maintaining the location-based side of things, while @kalwalt is looking after the NFT and marker-based parts.

@vokai thanks for your offer of collaboration. It's difficult to say why the location-based examples work on some devices and not others, on the S22 I think you'd need to do a bit of debugging to try and see if any errors come up on the console.
I'm not really on any channels besides GitHub and GitLab, I can give you an email if that helps.

I think the problem of different behaviors may be due to the fact that modern smartphones have 2/3 cameras.
In fact, with my S22 the wide-angle camera is opened by default.
Can this be due to this?

@lucas23456
I'm sorry to be harsh, but there is a issue template that has to be used. I understand it might be a bit of work from your side, but it's a great amount of time saved from the developer who are willing to help you.
Opening an issue with an empty description and a title saying "does it work?" is simply not a collaborative way of expose your problems.
Anyway, apart from this, what @vokai says is probably the problem here: on AR.js we never really managed to properly handled multicamera phones. That part needs probably a big update/fix that is crucial for Android phones - no problem instead for multicamera iPhones.
I have not time to prepare a PR for this but I'm okay with providing a review if someone can do the task. :)

@vokai feel free to submit a PR to fix this if you are able.
I could probably do it myself as long as the media devices API supports this (haven't checked), but I don't have a multi-camera phone to use for testing.
(If I did, I'd probably need to give it to you for testing if you are happy with this). Thanks.

I have little knowledge of the repository, I am not confident in doing PR but I have done some research.
In this repository there is the choice of device to use with the API.
(https://github.com/philnash/mediadevices-camera-selection --> livetest: https://philnash.github.io/mediadevices-camera-selection/)
I think we need to parameterize the camera stream then enumerating the available devices.
In my case running the test I have 4 cameras available (front: wide and normal, back: wide and normal).
So the last one in the list is the one that should go by default.
Or if you want to provide for device switching.
I am currently developing my own project using the "new location based," so I can do all available tests from that example.

@vokai maybe for this issue our project webarkit/ARnft may help. Look at CameraRenderer class, It has a routine to use the right camera.

@vokai @kalwalt thanks for the info.
@vokai if you want to do this then give it a go, if not I should be able to implement it. It is likely to be in the release after next, as I have a few fixes to new-location-based that I'd like to merge and release this week.

I think I was wrong. I can't see nothing in S22 because it has by default the wi-fi support to get better position.
So I was geolocated based on wifi and not my actual location, even using mobile network data same result.
By disabling the network to determine location, I was back to being properly positioned.
Nothing about choosing camera. I'm so sorry for my wrong debug.

@vokai no worries. We may implement the multi-camera feature anyway, because it will allow a user to choose their camera.


Port polylabel algorithm

I'd love to see the algorithm of https://github.com/mapbox/polylabel in Shapely.

Ostensibly, given the name, the purpose of porting over polylabel would be for finding an interior point to place labels.  I am not intimately familiar with all of the GEOS/JTS algorithms, but doesn't the existing representative_point() suffice?  Using the first test case from polylabel , the representative point from GEOS is fairly close to the label point from polylabel.

There is quite a nice description of how the algorithm used by representative_point works here:
http://gis.stackexchange.com/a/76563
The only guarantee of the "point on surface" function is that the point it produces will fall inside the polygon. This is in contrast to the centroid, which for certain polygons (e.g. a "C" shape) will fall outside. However, it is by no means the most visually appealing point for labelling.
The polylabel algorithm approximates the pole of inaccessibility, which is a useful function in general as I don't think GEOS provides this natively.

Thanks for the link.
I agree that JTS/GEOS doesn't have a single function for calculating the pole of inaccessibility, but I have seen it done through a series of steps using existing functionality.  That said, I can't recall the specifics at the moment, but it involved densify and Voronoi diagrams.
Having a pole of inaccessibility function in Shapely would be handy, I can think of a couple uses, but I don't know if using such a function would provide significantly better label locations than representative_point.

Is anyone working on this? If not, I would be interested in giving it a try.

@allanadair, I have/had the interest, I just can't quite seem to find the time in the next few weeks.  I say go for it.

@allanadair before the 1.6 release we'll need to figure out how to use the tolerance value to get the best label points. See in the image below the difference between using the default (1.0) and a value of 20.0 where the polygon is roughly 20x20 units in size.

This is great, and so are Shapely's SVG representations in Jupyter notebooks.


rename everything to gocommerce

@biilmann @calavera this should be the code half of what is needed. I'll rename the repo after this.
Also - I went with gocommerce do you actually want GoCommerce in the code?

messed up on rebase


Add Notification message to Open Folder after New Evidence Project is created

in an open workspace with other files and folders. See https://github.com/evidence-dev/evidence-vscode/issues/48#issuecomment-1575046988

The final new project -> open folder workflow:


Class tymon.jwt does not exits

I'm submitting a bug report

Provide information & explanation
I wanted to upgrade your project from 3.2 to 3.4 (cause i want use the 5.3 Laravel Version), so I started with installing the blank project.
But when I want to use protected Routes (as I could use in 3.2) The server returns an internal Server Error with the error below.

ReflectionException in Container.php line 749:
Class tymon.jwt does not exist
in Container.php line 749
at ReflectionClass->__construct('tymon.jwt') in Container.php line 749
at Container->build('tymon.jwt', array()) in Container.php line 644
at Container->make('tymon.jwt', array()) in Application.php line 709
at Application->make('tymon.jwt') in Container.php line 1203
at Container->offsetGet('tymon.jwt') in JwtAuthGuardServiceProvider.php line 29
at JwtAuthGuardServiceProvider->Irazasyed\JwtAuthGuard\{closure}(object(Application), 'api', array('driver' => 'jwt-auth', 'provider' => 'users')) in AuthManager.php line 111
at AuthManager->callCustomCreator('api', array('driver' => 'jwt-auth', 'provider' => 'users')) in AuthManager.php line 90
at AuthManager->resolve('api') in AuthManager.php line 70
at AuthManager->guard('api') in Authenticate.php line 61
at Authenticate->authenticate(array('api')) in Authenticate.php line 41
at Authenticate->handle(object(Request), object(Closure), 'api') in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 104
at Pipeline->then(object(Closure)) in Router.php line 644
at Router->runRouteWithinStack(object(Route), object(Request)) in Router.php line 618
at Router->dispatchToRoute(object(Request)) in Router.php line 596
at Router->dispatch(object(Request)) in Kernel.php line 267
at Kernel->Illuminate\Foundation\Http\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in CheckForMaintenanceMode.php line 46
at CheckForMaintenanceMode->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 104
at Pipeline->then(object(Closure)) in Kernel.php line 149
at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 116
at Kernel->handle(object(Request)) in index.php line 53

You can reproduce this by following steps:

Follow the installation guide for the 3.4 version descriped at https://laravel-angular.readme.io/docs/install
Go to routes/api.php and protect the sendResetLinkEmail-fkt with the following code:

Route::group(['middleware' => ['api', 'auth:api']], function ($api) {
   $api->post('auth/password/email', 'Auth\PasswordResetController@sendResetLinkEmail');
});

Register a new frontend user and do a successful login with its credentials
Try to insert any valid email address at the "email-forgotten-form"

I tried following solution, mentioned by bayssmekanique https://github.com/irazasyed/jwt-auth-guard/issues/5#issuecomment-175152252 with no success. It throws no error but returns unauthorized
Besides that - Thank you very much for your awsome work :)

Hi @aceart
Thanks for reporting this
There is an easy temporary workaround in this issue #425
I would like to solve it properly and I also tried the fix you mentioned but it didn't work
I'm hoping to try again as soon as I have a bit of free time
I'll close this as a duplicate, feel free to share more info if you've got any
Cheers!

Issue fixed


Site vehicles not loading - JS Error

Not getting this error at the moment? Is this still happening for you?

OK, this was fixed by @xssfox while I was sleeping :-)
Someone uploaded data with bad formatting, and it broke things.


installing tensorflow 2.11.0 on windows using poetry fails to install tensorflow-intel dependency

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.11.0
Custom Code
No
OS Platform and Distribution
Windows
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
After installing tensorflow 2.11.0 on windows using poetry, importing tensorflow fails with ModuleNotFoundError: No module named 'tensorflow'. You can see on the pypi page of 2.11 that all the windows wheels are empty (only 1.9kb) and so are the arm packages as reported here
Installing with pip works as it seems to pick up on a tensorflow-intel dependency from somewhere (which then also contains tensorflow). We saw something like that in the Metadata file of the tensorflow wheel ~ however that seems not to be enough to give poetry that info.
Standalone code to reproduce the issue
mkdir poetry-test
cd poetry-test
poetry init
poetry add tensorflow
python -c "import tensorflow"

-> ModuleNotFoundError: No module named 'tensorflow'
Relevant log output
No response

Hi @twerkmeister !
Thanks for sharing your observation on installation on Windows through poetry.
@SuryanarayanaY !
Could you look at this issue.
Thank you!

CC @nitins17 @learning-to-play

@twerkmeister ,
Could you please check now and close the issue if resolved.
Thankyou!

Hey @SuryanarayanaY, any idea how I can test this right now? Since this is about the wheel thats published on pypi, I would need to new wheel with the change to be published, right? So I'll probably have to wait until the next micro release ~

You cannot test right now with the 2.11. You could try nightly or wait until 2.12

not stale - will test with nightly -

Can confirm that, for example, in the nightly version 2.12.0.dev20221212 tensorflow-nightly-intel is picked up as an additional requirement. However, anything after 20121212 failed bc it resolved to requiring tf-nightly-cpu-aws:
$ poetry add tf-nightly==2.12.0.dev20221213
Because tf-nightly (2.12.0.dev20221213) depends on tf-nightly-cpu-aws (2.12.0-dev20221213) which doesn't match any versions, tf-nightly is forbidden.
So, because poetry-install-tf depends on tf-nightly (2.12.0.dev20221213), version solving failed.

This behavior could be due to the recent collaborator build process for windows by Intel from Tensorflow 2.11 release.
tensorflow-intel/tf-nightly-intel dependency is built when user is on windows machine which installs the TensorFlow package built by Intel.
Similarly, 'tensorflow-cpu-aws'/'tf-nightly-cpu-aws' is built when TensorFlow package built by AWS if the user is running Linux on an Aarch64 machine.
we will look into the dependency issues. Thanks!

I'm seeing the same issue. Same environment as OP.
I worked around the issue by just installing tensorflow using pip, and the only thing it did actually install was tensorflow_intel-2.11.0-cp39-cp39-win_amd64.whl

That is the case from TF 2.11 onwards: Windows builds are being done by Intel, so the actual code comes from tensorflow_intel wheels.
The above applies for CPU builds. GPU builds should be the Linux ones, via WSL-2.
See https://blog.tensorflow.org/2022/09/announcing-tensorflow-official-build-collaborators.html

Another issue that arises from metadata mismatch between different wheels based on platform is described in the issue referenced below.
A TL;DR version is that poetry also relies on wheels having the same metadata even for wheels that don’t match the current system/arch when using a legacy repository (e.g. Nexus, artifactory, devpi, pypiserver…). It simply downloads the first wheel and expects it to have the same metadata.
Would very much appreciate it if the fix was applied the other way around, too, if that makes sense for this project. (i.e. full dependencies listed for win and other variations).
As it’s not part of the standard, it may not make sense to make such a change. I can’t judge either way.
https://github.com/python-poetry/poetry/issues/4000#issuecomment-1456170476

As it’s not part of the standard

just dropping by to say that I think this is a misunderstanding of something I wrote in that other issue.  PEP508 environment markers are very much part of the standard and are encouraged as the way to specify platform-dependent requirements
ie requirements like
foo>=1 ; sys_platform == "win32"
bar>=1 ; sys_platform == "darwin"

are nowadays preferred over putting differing requirements in the windows / macos distributions.

hi, by the #58770 pull request, this should be fixed, but doesn't the if condition only work for linux environments ?

hi, by the #58770 pull request, this should be fixed, but doesn't the if condition only work for linux environments ?

Since poetry doesn't used the inferred environment to download the wheel for metadata collection, it could download the linux wheel even in Windows, if the linux wheel is the default.
There is a small poetry plugin that patches poetry to download the wheel for the inferred environment, and it solved our tensorflow issue. It is an extremely hacky plugin that is prone to break ... "It is what it is".

@twerkmeister , Could you please consider the above comment and let us know if that helps you. Thank you.

Still had this issue, also with tensorflow-cpu.

@mraunak , Could you please take a look into this. Thanks!

Hi @RoelantStegmann, could you please share the error you are facing while installing tensorflow-cpu using poetry? Is it Linux wheels getting installed along with Window's wheel

I get the same error with:
mkdir poetry-test
cd poetry-test

conda remove --name poetry-test --all -y
conda create --name poetry-test python=3.10 -y
conda activate poetry-test
pip install poetry

poetry init  # this requires manual input

poetry config virtualenvs.create false
poetry add tensorflow-io-gcs-filesystem==0.31.0  # this one doesn't install, so I have to add it manually
poetry add tensorflow-cpu==2.12.0
python -c "import tensorflow as tf"  # fails

How to verify that it's because of the wheel confusion? :) It seems related to https://github.com/tensorflow/io/issues/1617

Hi @RoelantStegmann, please follow the steps given below and let me know if you face any issue further
Steps to install TensorFlow through poetry.docx

@dimbleby try:
poetry add tensorflow-intel


[produce] non-ASCII app name in produce 2/2

PR 2/2
The Problem with non ASCII characters support in Produce is infact a portal problem.
tunes supports all utf8 characters (as they get converted with .to_json)
This one trys to fix this, allowing non-ASCII characters in app_name.

the app name is in fact still be latinzed but, to prevent an empty string as app_name
i added a md5 of the string

lane :produce_demo do
  produce(app_name: "名倉堂áásaa1")
end

Branch with all PRS:

https://github.com/hjanuschka/fastlane/tree/produce_non_latin_full

PRS:

1/2 -> https://github.com/fastlane/fastlane/pull/7210
2/2 -> https://github.com/fastlane/fastlane/pull/7211

Issue:

https://github.com/fastlane/fastlane/issues/7206
https://github.com/fastlane/fastlane/issues/5236

(When it was introduced)

History:

https://github.com/fastlane-old/produce/pull/17/files

@milch  feedback addressed - UI.important on name translation

@hjanuschka Great, thanks! 🚀  The spaceship PR is merged, which means we need to release a new version of produce once we release the next spaceship version.


⚠ SceneTime has degraded performance

In 04e2bd1, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1588 ms

Resolved: SceneTime performance has improved in 37fcac6 after 9 minutes.


Remove packages resolutions

I wonder if we (still) need these resolutions.
https://github.com/QuantStack/jupytergis/blob/0762f88ae18f9cd18edc35454342f493a3a235dc/package.json#L53-L58
Removing it doesn't seem to break installation (locally), as soon as the yarn.lock is also reset.

Probably the same issue as in https://github.com/jupytercad/JupyterCAD/issues/101


[Feature request] Silence option for "Are you sure" nag on delete.

VSCode Version: Code 1.8.1 (ee428b0eead68bf0fb99ab5fdc4439be227b6281, 2016-12-19T14:41:20.664Z)
OS Version: Linux x64 4.4.0-59-generic
Extensions:

Extension
Author
Version

vscode-markdownlint
DavidAnson
0.6.1

EditorConfig
EditorConfig
0.3.4

jslint
ajhyndman
1.2.1

jshint
dbaeumer
0.10.15

vscode-eslint
dbaeumer
1.2.2

tslint
eg2
0.8.1

cpptools
ms-vscode
0.10.0

Spell
seanmcbreen
0.9.1

vim
vscodevim
0.5.2

I would like an option to silent the message, "Are you sure?" when I delete a file. Maybe add a checkbox for "Don't ask again."

Steps to Reproduce:

Select a file
Delete it

+1

+1 useless click

need this, since we can get files back from trash..
just add a setting like promptOnDeleteFile: false

Come on guys Please implement this...we need it!

Agreed, this dialog is incredibly annoying. This is just flat out poor UX. Either prompt the user to confirm whether they really want to do something or (preferably) assume they know what they are doing and give them a way to undo it later if needed. But don't do both! That's just silly. The target audience here is developers who know how the trash/recycling bin works in their OS and should be using source control anyway as cmodien points out.

https://github.com/Microsoft/vscode/issues/1538


[Bug]: Day shift is not applied?

Maybe I am missing something (if so, please consider this issue more as a question), but it seems that day shift is not applied in the conversion.
Example:
from cron_converter import Cron
from local_crontab import Converter

tz = "Pacific/Auckland"
local_schedule = "30 5 * * FRI"

Cron(
    cron_string=Converter(local_schedule, str(DC["nz"].value.tz_info)).to_utc_cron(),
    options={"output_weekday_names": True}
).to_string()

30 16 * * FRI

I would expect this to become 30 16 * * THU

Hi @FBruzzesi, I’ll test your snippet as soon as possible to identify any issues. It wouldn’t be surprising if there’s a bug in the local-contrab or cron-converter code.


CAN send/receive ids are null

Hi team - We were using the commit 74163b455f070f8fd925955bb33911bfcd83a092 which was a few weeks old. Recently upgraded to the latest version at 3b33829178dc854e8a578ff66be7459137017391
With the new version of the library, the can send/receive ids appear as None. This was working with the previous version.
A sample test program.
import odxtools

database = odxtools.load_file("./Sibros_TCU_V1.0.1.pdx", enable_candela_workarounds=True)
for dlc in database.diag_layer_containers:
    for ecu_variant in dlc.base_variants:
        print("physical_request_id = ", hex(ecu_variant.get_receive_id()))
        print("physical_response_id = ", hex(ecu_variant.get_send_id()))

Error:
Traceback (most recent call last):
  File "test.py", line 6, in <module>
    print("physical_request_id = ", hex(ecu_variant.get_receive_id()))
TypeError: 'NoneType' object cannot be interpreted as an integer

Gitlab does not allow me to upload the PDX files. Probably you can test with the PDX files that you have.

Question: Are these APIs deprecated?. Should I be looking at a different function?. Please advise.

@smileprem you can put the PDX in a ZIP and upload it

That's brilliant. Why didn't I think of that?. Here is the PDX file that you can test with.
Sibros_TCU_V1.0.1.pdx.zip
Also, I put up an quick MR for this. https://github.com/mercedes-benz/odxtools/pull/84

Hi Team - The latest commit from main branch (cdb078019e41c84f4763d15d44166635b86259e4) fixed the issues we observed with the earlier version. Thanks for fixing the errors. I'll close this ticket.


supersid.py python

I downloaded the supersid-master files and found that with my RPi 3B I had to use #!/usr/bin/python in the supersid.py file, as opposed to just #!python.

According to Wikipedia, Shebangs must specify absolute paths (or paths relative to current working directory) to system executables.
https://en.wikipedia.org/wiki/Shebang_(Unix)#Program_location
For this reason #!python appears to be wrong.
A thread on stackoverflow discusses shebang in Python scripts.
https://stackoverflow.com/questions/6908143/should-i-put-shebang-in-python-scripts-and-what-form-should-it-take?rq=1
The highest ratet answer suggests the use of #!/usr/bin/env python3 or #!/usr/bin/env python2 while #!/usr/bin/env python should not be used, unless the script is compatible with Python 2.x and 3.x.
This statement seems to be inline with https://www.python.org/dev/peps/pep-0394/.
Distributors are encouraged to change python shebangs to python3 when python 3.x is supported.
Script publishers can write shebangs as #!/usr/bin/env python.
python.org states that Python 2 is no longer supported as of January 1st, 2020. https://www.python.org/doc/sunset-python-2/.
This raises the question, whether one should not drop the Python 2 support and use the shebang #!/usr/bin/env python3? This would also limit testing efforts.

I'm in favor of dropping python 2.x support. I will simplify the code and the testing.
My vote would be for
#!/usr/bin/env python3

Fixed. All runnable modules have #!/usr/bin/env python3 at the top.


Error: CHANNEL_INVALID

Hi! I'm trying to join channel, but have no success.
I tried different variations of the channel_id, for example:<PHONE_NUMBER>, -1001188085, 1188085, -1188085.
Maybe i using wrong access_hash? Here is hash, returned from auth.signIn method.
const channel = await client('channels.joinChannel', {
        channel: {
            _: "inputChannel",
            channel_id: -1001001188085,
            access_hash: 76**************92
        }
    });
    console.log(channel);

returns me:
Error: CHANNEL_INVALID

Maybe you connecting to testnet ( {dev: true } )?

@OwnageIsMagic No, here is my config:
const api = {
	invokeWithLayer: 0xda9b0d0d,
	layer          : 57,
	initConnection : 0x69796de9,
	api_id         : config.TELEGRAM.ID,
	app_version    : '1.0.1',
	lang_code      : 'en'
}

const server = {
	webogram: true,
	dev: false
};

I think something wrong with your channel id
Its bigger than 4kkk  (32bit int max value)

what about contacts.resolveUsername? you can try this method to obtain correct access_hash

@dew01ke exactly, my access_hash was wrong.


Fallback Autocomplete

Hi, thanks for making this plugin!
I have found that if anything goes wrong with this plugin (rls doesn't load for some reason, rls crashes, etc.), no error message is shown and furthermore autocomplete just completely stops working. I would prefer the default autocomplete over no autocomplete. Would it be possible to fallback to that if no completions are available from RLS?
Also, if there could be an atom command to restart RLS without restarting the entire editor, that would really help as well. :smile:

there could be an atom command to restart RLS without restarting the entire editor

+100

Default atom auto-complete should still work. However, Rls completion does need to improve, I think that's the main problem right now (and in the hands of https://github.com/rust-lang-nursery/rls).
For example right now I get the following completion, not from Rls. So it doesn't seem like any completion is removed by ide-rust.

For restarting the servers we're waiting on https://github.com/atom/atom-languageclient/issues/71, with that a restart-rls-servers command will be easy to add. Until then ctrl-shift-F5 with the rest of us.

 This usually happens to me a few minutes or a few hours after I start my editor. My editor can stay open for days at a time as I usually never restart my computer.
It might be that ide-rust or rls are crashing. When that happens (usually silently), the autocomplete just stops working entirely. No autocomplete menu shows up at all. I reported this bug because I usually have to restart my editor to get things working again.
Good tip on the reload command! With that we can at least get it working again quickly without restarting everything.

That's a pain that it can take a while to reproduce. I haven't seen Rls stop working in that was since I added the non-blocking request logic to it, before long running requests could block the server completely for long periods of time.
You could try piping stderr from rls to a log file & atom.config.set('core.debugLSP', true), when Rls stops working again you can look at the atom console & the log file and maybe get a clue to the source.

Though it's unfortunate that it takes a while, I have noticed that I'm able to make it happen quite reliably. I'll try re-enabling ide-rust and turning on debugging.

Cool that would be helpful. I've just been investigating our logging situation, really atom-languageclient should be handling stderr logging options so I'll try to improve this so everything is in the console.

Okay here's some more information: the autocomplete seems to stop working when RLS is still building. The loading spinner at the bottom that says "RLS building " is spinning. I can't access the default autocomplete, snippets, or anything until that finishes (which doesn't seem to ever happen).
$ rustc -V --verbose
rustc 1.22.1 (05e2e1c41 2017-11-22)
binary: rustc
commit-hash: 05e2e1c41414e8fc73d0f267ea8dab1a3eeeaa99
commit-date: 2017-11-22
host: x86_64-unknown-linux-gnu
release: 1.22.1
LLVM version: 4.0

$ rustup run nightly rustc -V --verbose
rustc 1.24.0-nightly (0a2e9ade8 2017-11-29)
binary: rustc
commit-hash: 0a2e9ade83ff253bb489c63a95b1f499c5e0916f
commit-date: 2017-11-29
host: x86_64-unknown-linux-gnu
release: 1.24.0-nightly
LLVM version: 4.0

I think we may as well close this. I think this package handles this ok, so I'm not sure how to improve this client-side. Further improvements to completion need to happen in Rls & racer.
Please re-open if you have any ideas to improve this client-side.


Show all accounts

Once a user clicks on "Confirm" they should be brought to the Account Management Screen.

Verified this issue on dev app v1.9.5(339)


Add LogAdd command to add a Log between volunteer and befriendee

As a manager, I want to log the sessions between volunteer and befriendees so that I can monitor the participation of volunteers, and the satisfaction of befriendees

Closed by PR #90


[Ameba] Restructure PersistentStorage

Problem

Ameba's current implementation of persistent storage is storing key-values in wrong places, causing commissioning issues
Data stored outside of chip-factory, chip-config and chip-counters are not stored in proper domains
Wrong error code in AmebaWiFiDriver::Init
Wrong integer type used in AmebaConfig::ReadConfigValue, causing read value to be wrong
InitOTARequestor is called after Server::Init, causing runtime hardfault

Change overview

Add more kConfigNamespace to organize key-value data into proper domains
Fix wrong error code in AmebaWiFiDriver::Init
Fix integer type in AmebaConfig::ReadConfigValue
Fix initialization order in chipinterface

Testing

Tested ble-wifi commissioning with chip-tool

#17051 is the prerequisite for this PR

#17320  is the prerequisite for this PR.

/rebase

/rebase


Topic suggestions

Including the mention in upcoming newsletter 187, it looks like we have at least a half-dozen mentions of proof of reserves, so it seems to warrant a topic page.  See also, BIP127 and earlier versions of BIP322

v3 transaction relay

x-only public keys

v3 transaction relay

v2 I think you mean? https://bip324.com/

I think @harding may have been referring to @glozow's version-3 transaction proposal, not the v2 P2P transport protocol

LN pathfinding (or some other topic where we can put pickhardt payments, LND mission control stuff, etc...)

LNURL

Something like "efficient invoice encoding" to capture all the times we've talked about encoding for QR codes and other stuff, including maybe LN invoices.

Taproot annex (when we have there mentions; I expect we'll have two mentions as of the 2023-06-14 newsletter).

Cross-input signature aggregation

I’ve gotten some requests to write “a BIP” about coin selection. Since that’s not something that different implementations need to coordinate on, I think it might be better to backfill the topic page. I’d like to elaborate that page ~sooonish. What do you think?

I’ve gotten some requests to write “a BIP” about coin selection. Since that’s not something that different implementations need to coordinate on, I think it might be better to backfill the corresponding Optech topic page. I’d like to elaborate that page ~sooonish. What do you think?

Sounds great!  Our longest page currently is soft fork activations, which is pretty darn long, so you should feel comfortable writing long and detailed if you want.  I don't think we currently have a way to add footnotes to topic entries; if that's something you're gonna want, let me know and I can add that to the layout.
I'm currently just starting the process of doing a general refresh of our topic pages, so both for @murchandamus and everyone else, here are some of the things I'm thinking about in case they would affect your work or you just have opinions about them:

Adding an RSS feed to every topic page.  So, for example, if you don't want to read every optech newsletter but you do want to learn about news related to a certain set of topics (e.g. coin selection), you can just subscribe to those topic's feeds.  Before doing this, I have several topics I want to rename.
Adding an RSS feed for each of our categories.  Same thing as above, but a single feed will include all updates to any entry within that category (e.g., you can subscribe to the Privacy Enhancements feed to get info about both coinjoin and payjoin).  Before doing this, I want to significantly rework our categories
Adding category overview pages.  Sort of like topic pages but covering a whole category, it will provide a summary of the category and links to important information.  Again depends on a recategorization of existing topics
For longer topic pages, I think we need a way to separate highlights from deep details.  I don't have an exact endpoint in mind for this, but I'm thinking about starting out by allowing topic pages to have sub-pages, e.g. /en/topics/soft-fork-activation/history.  A summary of the sub-page will be listed on the main page along with a link to the details.
Integration with our podcasts, such as listing each episode where that topic was discussed in detail with a subject-matter expert

We seem to have mentioned "headers-first sync" a fair number of times.  It's not something I would have thought to create a topic about (the work on it was done years ago), but if we're going to keep mentioning it, it might be worth a topic.  Also, I wrote a bunch about it for Bitcoin.org and that content is currently hard to find, so I'd kinda like to resurface it.

One or more topic pages for the John Law family of LN protocols.  We've mentioned his stuff more than three times, with more mentions expected, but I'm not currently sure how to either group all of that together or how to split it into separate topics that each have three mentions.  Anyway, putting it here so I think about it as part of my ongoing topics update initiative.

It was pointed out to me by Xavier Fiechter that we don’t have BIP329, and it looks like we have mentioned it in a few different newsletters already

It was pointed out to me by Xavier Fiechter that we don’t have BIP329, and it looks like we have mentioned it in a few different newsletters already

https://bitcoinops.org/en/topics/wallet-labels/
I went with a more generic topic (wallet labels versus BIP329 wallet label export), but that topic was created primarily to capture the discussion about BIP329.  A PR is welcome to add BIP329 as an alias.

CLTV expiry delta.  See newsletter 274 for a list of many (but not all) previous places we've mentioned it.

Separate OP_CAT into into own topic (its currently combined with OP_CSFS

LN stuck funds problem, see:
https://bitcoinops.org/en/newsletters/2023/05/31/#eclair-2666
https://bitcoinops.org/en/newsletters/2020/02/19/#c-lightning-3500
Newsletter 276

BIP129

Weak blocks

"data carrier transactions" via https://github.com/bitcoinops/bitcoinops.github.io/issues/1462

"Ordinals" via https://github.com/bitcoinops/bitcoinops.github.io/issues/1364

topic page for something like "Specification management" to track BIP and BINANA meta discussion

DNS payment instructions as specified in BIP353, with relation to some other documentation (e.g. for LN).

txid malleability (second party and third party), may be able to simply copy text from MB3E when its license switches to CC-BY-SA in November.

DNS seeding and (hardcoded) seed nodes

libbitcoinkernel


PFS2 Error performing REST-API call: open_channel

Thanks for filing a bug report :-)
Steps to Reproduce

The logs will be updated shortly
Scenario pfs2_simple_no_path Failed
Expected Result

Actual Result

Might be related to the retry async solution
@palango to attach the logs

We had to adjust some server settings and during re-deployment the logs were lost. Nothing we can do here now.


Add stopp-loss to LOB vyper contract

Background
Volume is using the limit order bot vyper contract for palomabot.ai, but the vyper contract does not yet support having a stop loss price at which the token is swapped out again
Done when

[x] https://github.com/VolumeFi/limit-order-bot-vyper-uniswap-v2 supports having a stop loss price for a deposit
[ ] Readme is updated with the new functions

@webelf101 please update the readme with the new functions as well, so we can close out this ticket

https://github.com/VolumeFi/limit-order-bot-vyper-uniswap-v2/pull/2