Don't write to closing streams, fix #5363

Catching up on older issues - here's a simpler fix for #5363. :) @EndUser509, does that look somewhat correct to you?

Note that this broke the WireGuard mode, because mitmproxy_wireguard.TcpStream does not implement is_closing method.

Eh, well spotted. Why is CI not catching this?

Looks like the wireguard ServerInstance test was not run during CI?

https://github.com/mitmproxy/mitmproxy/actions/runs/3106426607/jobs/5033294929#step:7:1000 looks right. Likely we don't catch this because it happens during teardown. I would have expected mypy to catch this though.

I assume TcpStream.is_closing is never called because we're monkeypatching ConnectionHandler.handle_client with a method that doesn't call is_closing on the stream?

Yeah, that explains the working test. 👍
Here's why mypy isn't complaining:
diff --git a/mitmproxy/proxy/server.py b/mitmproxy/proxy/server.py
index 1341564ad..4781294a9 100644
--- a/mitmproxy/proxy/server.py
+++ b/mitmproxy/proxy/server.py
@@ -367,6 +367,7 @@ class ConnectionHandler(metaclass=abc.ABCMeta):
                 elif isinstance(command, commands.SendData):
                     writer = self.transports[command.connection].writer
                     assert writer
+                    reveal_type(writer)
                     if not writer.is_closing():
                         writer.write(command.data)
                 elif isinstance(command, commands.CloseConnection):

λ tox -e mypy
mypy run-test: commands[0] | mypy
mitmproxy/proxy/server.py:370: note: Revealed type is "Union[asyncio.streams.StreamWriter, Any, mitmproxy.net.udp.DatagramWriter]"
Success: no issues found in 273 source files
ERROR: InvocationError for command /mnt/c/Users/user/git/mitmproxy/.tox/linux/mypy/bin/mypy (exited with code 1)

wg.TcpStream registers as Any somehow, presumably because mypy can't find the definitions. Do we need to set py.typed somewhere or something like this?

Do we need to set py.typed somewhere or something like this?

No idea :( This is the first time I'm working with a "native" Python module. But maturin already includes the .pyi and py.typed files automatically, so that shouldn't be it.

I went ahead and implemented is_closing on TcpStream as it was pretty easy to do. I'll bundle in the fix for the InvalidStateError race condition and then I'll publish a new release :)

Release 0.1.10 of mitmproxy_wireguard should fix this problem:
https://github.com/decathorpe/mitmproxy_wireguard/releases/tag/0.1.10


TypeError: unsupported operand type(s) for -: 'NoneType' and 'float'

Hi,
I am getting the following error:

When I run ASR_diarization pipeline with the following video:
https://drive.google.com/file/d/1wBwAs5Ubp-70fAf7uwevEu8XFlqksQja/view?usp=sharing
It's happening because the end timestamp of the last segment is None. I found this - https://github.com/huggingface/transformers/issues/23231.
For now, I was able to suppress the error by removing None from end_timestamps array. I understand that I lose some text towards the end of the video but for my use-case, that's not a problem. I wanted to bring this to your attention for a fix in the future. Let me know if you need anything additional. Thanks!

#32

FYI, having same issue

I am seeing this issue with at least 75% of the files I am testing on.


[Feature request] Add network support to OsmAnd protocol

There are few possible usecases:

Test purposes
"Self-made" devices
Forwarding data from unsupported devices to traccar
Possible using in Traccar Client

I guess the biggest question here is how to store. Maybe we should allow serialized position in JSON body? I guess we can even create separate protocol for it.

Don't you like pass "mcc","mnc","lac","cid" as GET parameters? Or I'm missing something?

There can be multiple cell towers. Also there can be WiFi access points. Now Traccar can use all that information for better accuracy.

You are right, too many parameters. May be POST with standardized object will be solution (looks like new protocol).

Sounds good to me.

OsmAnd support seems to be added. Issue can be closed.


Unable to get metrics

Environment

Telemetry Streaming Version: f5-telemetry-1.19.0-3.noarch   and  f5-telemetry-1.15.0-4.noarch
BIG-IP Version: BIG-IP <IP_ADDRESS> Build 0.0.4 Point Release 6

Summary
After running it for a while, I couldn't get metrics, not in either version.  But the other F5 devices are working fine.
{
"code": 400,
"message": "remoteSender:10.0.x.x, method:GET ",
"referer": "10.0.x.x",
"restOperationId":<PHONE_NUMBER>,
"errorStack": [
"java.util.concurrent.TimeoutException: remoteSender:10.0.x.x, method:GET ",
"at com.f5.rest.common.RestWorker.logAndFailExpiredOperation(RestWorker.java:3153)",
"at com.f5.rest.common.RestWorker.checkForExpiredOperations(RestWorker.java:3142)",
"at com.f5.rest.common.RestServer.checkAndExpirePendingWorkerOperations(RestServer.java:1457)",
"at com.f5.rest.common.RestServer.access$400(RestServer.java:48)",
"at com.f5.rest.common.RestServer$4.run(RestServer.java:1430)",
"at com.f5.rest.common.ScheduleTaskManager$2$1.run(ScheduleTaskManager.java:123)",
"at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:473)",
"at java.util.concurrent.FutureTask.run(FutureTask.java:262)",
"at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)",
"at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)",
"at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1152)",
"at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:622)",
"at java.lang.Thread.run(Thread.java:748)\n"
],
"kind": ":resterrorresponse"
}
Steps To Reproduce
After reinstallation, it will resume and run for a period of time

It is possible that you are encountering problems with restjavad memory usage. Have you tried increasing the memory provisioned for restjavad? https://clouddocs.f5.com/products/extensions/f5-telemetry-streaming/latest/troubleshooting.html?highlight=restjavad#why-is-my-big-ip-experiencing-occasional-high-cpu-usage-and-slower-performance

I have the same Problem (timeout exception), but increasing memory to restjavad is not an option in production env.
Telemetry Streaming Version: f5-telemetry-1.20.1
BIG-IP Version: BIG-IP <IP_ADDRESS>
just waiting until we can upgrade to higher BIG-IP Version.


Is project steel alive?

Hi, Steve, it's a very good idea to convert C# type to TypeScript.
But as I can see project in pre-release state - it is usable for now?)

Hi - unfortunately I have moved on in my career and I am no longer a C# programmer. If you want to fork it and try to use it, please do!


Missing argument for parameter dataModelManager in call...

Attempting to incorporate RocketData in my first app, following along through the sample app and ran into an issue that doesn't crop up in the sample app, but does in my app.
The sample app contains the following code in ChatsViewController.swift:
fileprivate let dataProvider = CollectionDataProvider<UserModel>()

When I put the following in my app, I get the error "Missing argument for parameter dataModelManager in call"
fileprivate let dataProvider = CollectionDataProvider<Episode>()

Both view controllers import RocketData and implement the CollectionDataProviderDelegate protocol. What am I missing here?

Take a look at the extensions in https://linkedin.github.io/RocketData/pages/040_setup.html
Basically, the singleton isn't set up for you so you need to add extensions to get it to work with a cleaner API.

Ok great, thanks for the info!

@plivesey I thought that name sounded familiar, I actually just got done watching one of your talks on managing consistency with immutable models. Well done!


Feat/control image width logics

This closes #469, also fixes https://github.com/ExperienceLovelace/ha-floorplan/discussions/320 (Discussion).
I'll be waiting for feedback, before merging this.

Just took a look, it works great and as you described!

@knicholson32 Thank you for the quick follow up. I'll wait for another confirmation, and release it hereafter.


use epoll on linux

select.select() is significantly slower on linux than select.epoll(), so, especially in the case of network services, there's a huge performance gain in using epoll.
I see a few ways forward:

declare it WontFix.
abstract the filehandle-watching so it can be switched based on what OS is running
find/use something like libevent() which does the abstraction for you
make completely different SystemBases - one for select(), one for epoll()

I agree that select() is generally the least-performant approach.  It is the currently implemented form because it is also the most broadly portable (for both operating systems and python versions).
In reference to #3 in your list, Python has the "selectables" in the standard library, but this was only introduced in 3.4.
Right now, Thespian supports all versions of Python from 2.6 through 3.5 by taking the "common path" as much as possible.  This is definitely a performance issue that should be addressed at some point, so I'm not going to take method 1 (WontFix), and I would prefer to provide internal adaptation and avoid a proliferation of SystemBases (i.e. your method 2 as preference to method 4).  If there is too much variation or there is a need for externally-supplied configuration, it may be necessary to introduce a new SystemBase but in general I would prefer to internalize the complexity to keep the Actor usage simple.
Keeping this as an open enhancement request to flag this for future performance work.

By "selectables" do you mean selectors ?  If so, the selectors34 package provides a forward-compatible version for   python 2.6, 2.7 and 3.3... perhaps that's the correct way forward?

another option would be just to use asyncio?


How to make a particular script to run on a particular integrated terminal in vscode?

I have 2 scripts.
Script 1 creates a tmux session or attaches a tmux session, if already exist.
Script 2 creates another tmux session and again attaches it, if already exists.
I want script 1 to run in integrated terminal 1
and script 2 to run in integrated terminal 2
whenever I reload the VSCode window or launch VSCode.
How can I achieve this?
Generalizing this issue, I want to run a particular script to run a particular integrated terminal.

/question

I've asked it in stack overflow -
https://stackoverflow.com/q/64841384/5347487
But usually I don't get a response there

See, I have raised the question on SO 5 days back still I don't see any response to it. I don't see a good response to any questions raised on SO in VSCode category.


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Thu Mar 24 2016 11:03:20 GMT 0100 (W. Europe Standard Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\ngorshkov\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\ngorshkov\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)
    at EventEmitter.<anonymous> (C:\Users\ngorshkov\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\main\features\core\controlBar.js:12:45)
    at emitTwo (events.js:87:13)
    at EventEmitter.emit (events.js:172:7)
    at EventEmitter.<anonymous> (C:\Users\ngorshkov\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\api\lib\web-contents.js:134:25)
    at emitTwo (events.js:87:13)
    at EventEmitter.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470


Initial Commit for Dev

This contains Ollama and OpenAI Connectivity codes

Thanks for the PR, Suraj. Merged with the main repo, but have not tested it yet.
From: Suraj R S @.>
Date: Friday, 13 September 2024 at 6:12 AM
To: balajivis/BrahmaSumm @.>
Cc: Subscribed @.***>
Subject: [balajivis/BrahmaSumm] Initial Commit for Dev (PR #2)
This contains Ollama and OpenAI Connectivity codes

You can view, comment on, or merge this pull request online at:
https://github.com/balajivis/BrahmaSumm/pull/2
Commit Summary

d47aecdhttps://github.com/balajivis/BrahmaSumm/pull/2/commits/d47aecded08ea31c5f208f162d7c7bec0725c168 Initial Commit for Dev

File Changes
(4 fileshttps://github.com/balajivis/BrahmaSumm/pull/2/files)

M config/config.yamlhttps://github.com/balajivis/BrahmaSumm/pull/2/files#diff-38df760413887d0cc4ee5707919e6390456a863d326f6080644cc3d9c72e4bba (5)
M src/models/models.pyhttps://github.com/balajivis/BrahmaSumm/pull/2/files#diff-32c65642b81541b58973dfdc6406ee6eaa3b19467f9f89569ac4949f7d188d11 (49)
M src/summarize.pyhttps://github.com/balajivis/BrahmaSumm/pull/2/files#diff-2d680d87592ba43269b51d1ce0ee8c80d0f892fdf6baf4493aa675762f90f23c (13)
M temp.txthttps://github.com/balajivis/BrahmaSumm/pull/2/files#diff-50ea7be4eeb62777c05b06e21e9e5f6e8ce39442e7efbd17da8d57d5b18f5035 (1)

Patch Links:

https://github.com/balajivis/BrahmaSumm/pull/2.patch
https://github.com/balajivis/BrahmaSumm/pull/2.diff

—
Reply to this email directly, view it on GitHubhttps://github.com/balajivis/BrahmaSumm/pull/2, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A43CPIAFV5XZQC5STJAM6PDZWLQDJAVCNFSM6AAAAABOFLI4UCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUZDIOBSGA3TCNQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Error : fakerootkey not defined

Error details:
libfakeroot: FAKEROOTKEY not defined in environment

Wh

<IP_ADDRESS>

This script is written in Python and employs some interesting techniques, including the use of marshalling. Here's an overview of the script:
Marshalling: The script uses Python's built-in marshal module, which is used for serializing and deserializing Python objects. In this script, marshal.loads() is used to load serialized code (marshalled data) represented as a string.
Executing Marshalled Code: The marshalled code is directly executed using exec(). The exec() function is powerful and potentially dangerous. It runs the code it's passed with the current scope. This means that if the code is provided from an untrusted source, it can cause serious security issues, as it can run any Python command.
Now, discussing the security implications:
Code Obfuscation: The usage of marshalling here is a form of code obfuscation. It's unclear what the marshalled code does without reversing it back to its original form. This can be a potential security issue as it hides the true intentions of the code, which could be harmful or malicious.
Arbitrary Code Execution: The exec() function can execute any Python code, which makes it a serious security risk if the input to exec() is not controlled or is obtained from an untrusted source. In this script, exec() is used to execute the unmarshalled code. If the unmarshalled code is malicious, it will be executed without any checks.
Lack of Input Validation/Sanitization: The script does not have any input validation or sanitization, making it more vulnerable to security issues.
What this could do depends entirely on the code that is unmarshalled. It could do anything that Python is capable of doing, which includes but is not limited to: file manipulation (read/write), network communication, starting other processes, loading other modules, etc. Without knowing the contents of the marshalled code, it's hard to say definitively.
In conclusion, it's crucial to never run such scripts unless you are completely sure about the source of the marshalled data and trust it completely. It's always recommended to review and understand the complete code before executing it, especially when exec() and marshalling are involved


Automatically create catalogs from maven

Cannot the catalog be created by the operator if not present in the cluster?
It seems to me a simple maven build and it saves a lot of work when people want to try a version of camel for which we don't provide the catalog out of the box.
Wdyt @lburgazzoli , @valdar ?

yes that's possible, I did it locally in the past but I've removed it when we moved to semver matching

An error happens when using e.g. runtime 1.0.6 (from staging) and camel 3.0.0-RC1

That part has been reworked as part of the Quarkus support and that issue should be fixed with #1013.

This should now be fixed as #1013 has been merged.


[Addition]: CalChip Connect Remote witness

Hotspot b58 Addresses
112DyXUfFhANdf8r3ZrEq3ZvFWU7TToFPG99suiz8twrUcw3ZUjM
Discord Handle
No response
Reason(s)

Witness with hotspots at 36km while it is physicaly impossible to be in line of sight.

We are all grasshoppers in the same boat!
Please spare the user！
We report blacklists every day！
It will only provoke more people to report to each other！
Current hNT price！
We can't sell the equipment now！
Don't forget！
We are all leeks！
No new users come in！
We can't cash it out. Bye！


Install dateutil which is required by ghp-import

r? @jamesmunns
(rust-highfive has picked a reviewer for you, use r? to override)


parameter is incorrect

Connect when Bluetooth is on.
Start Device Detection

DotSpatial version: Current
If I turn my BlueTooth off there is no error. So its trying to connect to a bluetooth device but I'm getting no error other then System.ComponentModel.Win32Exception:'The parameter is incorrect'
Im assuming this is a device that is cached, but I can't remove it. I need to handle the error and move on. I don't really care if it can't connect to the bluetooth other then to telling me it can't instead of breaking the app.
Any help please.

I finally got a stack trace from getting the error from the appdomain.unhandledexception:
at DotSpatial.Positioning.BluetoothDevice.Refresh()
at DotSpatial.Positioning.BluetoothDevice.get_Name()
at DotSpatial.Positioning.Device.BeginDetection()
at DotSpatial.Positioning.Devices.BeginBluetoothDetection()
at DotSpatial.Positioning.Devices.DetectionThreadProc()
at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ThreadHelper.ThreadStart()

I would love a Devices.UnhandledException

I download the source code and I'm getting -2147024809 error code in the refresh method. Is it ok if I just skip this error code.

I am getting a similar problem. For me, it occurs when the device resumes from sleep mode:
System.ComponentModel.Win32Exception, System, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089The parameter is incorrect   at DotSpatial.Positioning.BluetoothDevice.Refresh()
at DotSpatial.Positioning.BluetoothDevice.get_Name()
at DotSpatial.Positioning.Device.BeginDetection()
at DotSpatial.Positioning.Devices.BeginBluetoothDetection()
at DotSpatial.Positioning.Devices.DetectionThreadProc()
at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ThreadHelper.ThreadStart()System.ComponentModel.Win32Exception (0x80004005): The parameter is incorrect
at DotSpatial.Positioning.BluetoothDevice.Refresh()
at DotSpatial.Positioning.BluetoothDevice.get_Name()
at DotSpatial.Positioning.Device.BeginDetection()
at DotSpatial.Positioning.Devices.BeginBluetoothDetection()
at DotSpatial.Positioning.Devices.DetectionThreadProc()
at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ThreadHelper.ThreadStart()80070057

I have the exactly same problem as TasGrinder.
The application breaks, try/catch helps nothing.
Is there any solution?


Reworked cursor rendering, fixing several bugs

Block cursors were rendered with a fixed width, rather than the width of the current character, when after an inlay hint and using a variable width font.

In visual normal mode, the selection highlight didn't extend past the end of the current line when the cursor was past the end of the line.

In visual normal mode, if there was an inlay hint after the end of the current line and the cursor was past the end of the line, the cursor was rendered after the inlay hint.

In visual normal and block modes, when selecting backwards to the first character after an inlay hint, the selection highlight extended leftward of the cursor to include the inlay hint.

When there were multiple cursors on the same line, only one was rendered.

The selection highlight for visual block mode excluded the rightmost column when selecting right and up or left and down.

When in visual block mode, when the cursor was to the left of the anchor column of the selection and past the end of the line, the cursor wasn't rendered.

Previously, the lines containing the cursors were highlighted in insert and normal modes, but not in any of the visual modes. This PR changes it so that the lines containing the cursors are highlighted in all modes, as this slightly simplifies the code. If highlighting the line containing the cursor in visual mode is undesirable, this could be changed.

Previously, the lines containing the cursors were highlighted in insert and normal modes, but not in any of the visual modes. This PR changes it so that the lines containing the cursors are highlighted in all modes, as this slightly simplifies the code. If highlighting the line containing the cursor in visual mode is undesirable, this could be changed.

current line isn't highlighted in visual mode was to keep the same behaviour with Vim.

current line isn't highlighted in visual mode was to keep the same behaviour with Vim.

I've changed it to not highlight the current line in visual mode, and fixed an issue with the previous version of the PR that would have resulted in the line containing the active breakpoint not being highlighted unless there was a cursor on it.

For removing the whitespace at the end of comments, I think we should keep those in the future. (I feel like I commented on this before saying whatever/its fine to remove them, but might have not; anyway after recently checking what was up with RA not rendering them I'm readjusting my opinion)
The double spaces are the markdown syntax for making a linebreak, which allows seeing them as newlines within rustdoc. Rust analyzer at the moment trims this whitespace when doing hover documentation, which I believe it did not do previously. See https://github.com/rust-lang/rust-analyzer/issues/15877
(So in future prs please don't cut off the double spaces. Rustfmt doesn't do so by default and Rustfmt would catch extra whitespaces in code)
(also, thanks for the pull requests they're great)

@MinusGix thanks for the heads up - I wasn't aware that trailing whitespace had semantic meaning in Markdown.
I personally think that a trailing backslash is a cleaner way to indicate a hard line break. Lapce, VS Code and cargo doc all support this.
Still, I'll make sure not to strip trailing spaces from doc comments in the future.


Remove Duplicate Event Listener Code

Remove duplicate logic.

Deployed to UI instance and issue is still happening for non -swagger reviews.


Feature request: Popover to support placement as function, html content, etc.

I know you guys are aware of the missing features (as documented in source code) but I just want to raise this issue so that it is properly tracked and prioritised.
And yes, I would love to have this feature - you can't really do much with current popover other than just plain text!

It seems like here is a feature you talking about. https://github.com/angular-ui/bootstrap/pull/1848, isn't it?


foundry PDKs

one of the biggest limiting factors for a larger adoption of gdfactory is that we don't provide official PDK foundry support
what is the best way to share PDKs for different foundries?
Most fabs support cadence and synopsis PDKs at the moment, and it would be great to enable an open source layout, simulation and verification alternative (which in my opinion is much nicer to use)
maybe you can also help us gain some traction by asking your partner fabs to work with us on this. Feel free to share and comment this post.
I see 3 options:

In the ideal case the foundry makes their PDK open source. See for example google/skywater130 pdk This enables a large, community built and maintained PDK. No NDAs required for accessing PDK technology needed for building chips. For protecting some particular cells, we can always use some black boxed cells, that can be replaced by the  foundry before fabrication
Create a private Github repository, that the foundry controls access, and only users with valid NDA get granted permission to access.
Foundry builds and maintains a gdsfactory PDK for their technology and they share it with their users. This is the least collaborative approach and does not allow users to share improvements, as the information only flows in one direction. This is option is based on the current model and requires extra effort from the foundry to build and maintain the PDK. Therefore PDKs are hard to maintain, hard to track changes and hard to improve.

what do you think?
@quantamHD
@mithro
@thomasdorch

FYI - @proppy @msaligane
Let's start with the option (1) for SKY130, the second fully open source process we plan to release this year and ASAP7 (non-manufacturable)? We don't even need any of the complexity around black boxed cells here. After that all already works, then we can then look at the more complicated option (2) and/or option (3).

I can't disclose the PDK, but if it of interest to folks and I can put the AIM photonics layer stack in gdsfactory format for folks to use on their tapeouts.

Hi Alex,
thank you for the offer, currently most PDKs are distributed by foundries under NDA.
The only one that does not require NDA at the moment is the skywater PDK, which was open sourced by Google and skywater on 2020, and the ubcpdk that Lukas uses on his EDX course.
The decision about open sourcing should come from the foundries, so if you don't want to be the only one maintaining that PDK please ask your foundry for advice


direct routing license requirements

Is direct routing not applicable with a business voice without calling plan subscription?
Because here https://docs.microsoft.com/en-us/MicrosoftTeams/teams-add-on-licensing/microsoft-teams-add-on-licensing?tabs=small-business#what-voice-features-are-available-with-my-plan you can find that direct routing is possible with business voice.
On this page here the requirements are only mention the phone system license.

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

ID: e26faf8c-529e-b317-5934-cd41cfe8fa24
Version Independent ID: 3ddd5d0a-6dbb-4940-1814-8346e293701b
Content: Plan Direct Routing - Microsoft Teams
Content Source: Teams/direct-routing-plan.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Hi scanum, any updates on this? We need a clear statement on this for a custome project

Hi @SRE93, thank you for your feedback.
Business voice includes Phone System. Please check it here:
https://docs.microsoft.com/microsoftteams/business-voice/whats-business-voice


Added CredentialProviderFactory.stsWebIdentityTokenFile

Used in EKS clusters. Looks for environment variables AWS_WEB_IDENTITY_TOKEN_FILE, AWS_ROLE_ARN, and AWS_ROLE_SESSION_NAME to fill out AssumeRoleWithWebIdentity request

Do you want to give the latest a go


Working nightfall browser ropsten deployment with trusted stakeholders

What is our MVP (that can be on testnet or mainnet)

Polygon web wallet users on Ethereum to deposit ERC20 tokens with practical cost
Polygon web wallet users on Ethereum to transfer ERC20 tokens with privacy and practical cost
Polygon web wallet users on Ethereum to withdraw ERC20 tokens with privacy and practical cost
This will require one honest Block proposer and Challenger. Trusted model (Needs fees and penalties defined for proper incentivisation and penalisation mechanism, no censorship guarantees because there is only one proposer, theft funds relies on trust assumption that toxic waste from MPC is deleted)
Deployment needs to be upgradeable such that contract, nightfall services can be updated for bug fixes or enhancements


Size reports: Fix stale-commit determination

Problem
Some size reports are missing. The reason is that gh_report.py uses
the commit ‘committer’ time to determine which commits are outdated,
and a rebase sets that to the same value for all commits in a PR.
Change overview
Use the ‘author’ timestamp as a secondary sort key.
Testing
Manual run confirming sort order.

Fast track: support script, not affecting main sdk functionality


forward enable

Hi I wanted to know if I have two servers one with the original interface and another with the modified web interface.
The GPS data go to the first server and are stored there.
with "forward.enable" command I can forward data to the other Web servers ui keeping the data on the first server?
in essence, the data arriving to the first server and venvono stored and at the same time are forwarded to the second server.
It's possible?

It's possible to forward essential GPS data to OsmAnd port, but you will lose all or most additional attributes.


SignIn/SignUp page

Follow the workflow diagram given in the README.md file and implement a SignIn/SignUp page. Please refrain from adding a UI Library without discussion with the project managers.
Do not forget to mention the issue id in the Pull Request you are going to merge so that it is easy to keep track of this issue.

interested

Interested


bug: Neo tree stop working because of the path separator

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.4
Operating system/version
Windows 10
Describe the bug
On windows 10, the file list is empty when opening neo-tree window with "<leader>e". And the files show when open neo-tree window with "<leader>E".
Maybe it's because the new version of "Util.root.get()" or "Util.root()" returns the path with forward slash separator. And "<leader>E" works because the path separator is backslash returned by "vim.loop.cwd()"
Steps To Reproduce
On windows 10, open a file in a git repo with neovim, and open an neo-tree window with "<leader>e" shortcut.
Expected Behavior
To see all the files in the git repo when open the neo-tree file window.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

I have the same exact issue, also on windows 10 and nvim 0.9.4

Fixed. Will make a new release later

Thank very much! Your response is incredibly quick.
LazyVim is amazing!!!


Remove eslint action from workflow

Why:
Currently the tinovyatkin/action-eslint@8b32772f075413f9192f43641e03d9bd0abae621
action fails silently and does not fail for the same reasons as yarn lint
This commit:
Removes the action until we can troubleshoot it.

figured out the issue on this one: https://github.com/Path-Check/covid-safe-paths/pull/909, closing as this is no longer necessary.


Support Node.js 10

We observed some weird behaviors with node-ldapjs when upgraded to Node.js 10 LTS in our project.

unbind not work (already reported as https://github.com/joyent/node-ldapjs/issues/483)
ldapjs UT hangs with Node.js 10

This issue is caused by some internal change in Node.js. See https://github.com/nodejs/node/issues/24577.
In summary, in Node.js 10, the close event depends on an internal end listener. so when ldap client use removeAllListeners(end`) to do cleanup work, the close listener stops working.
Node.js guys suggest that we should use removeListener to explicitly remove the listener we want to remove instead of using removeAllListeners. Node.js document also suggests that we should use removeAllListeners carefully.
Note that it is bad practice to remove listeners added elsewhere in the code,
particularly when the EventEmitter instance was created by some other component or
module (e.g. sockets or file streams).

This PR didn't touch all of the removeAllListeners in client.js, just fixed the one who breaks close event.
@jgeurts, @melloc, could you help to review this PR?

@melloc @pfmooney
Could you help to take a look?
@acappella2017 has found the root cause (removeAllListeners even removes TLS socket's internal listeners and breaks socket's event sequence) and discussed with Node team. And this PR contains the safest change we can find, it doesn't change current code logic.
Many projects and npm modules depending on ldapjs need it to work on Node.js V10. Hope to see this PR get reviewed.

Hi. Any progress on this?

I just forked the repo and am using my own version with this fix in place.
"ldapjs"<EMAIL_ADDRESS>

Hi!
Any updates/plans on this?  It seems multiple individuals are left to fork to get this fix into their projects, is it mergeable here or is non-review work outstanding?

@zachlendon This project is pretty much dead. None of the maintainers have commented on here in years.

For what it's worth, we started having major issues with ldap using a verdaccio server with the verdaccio-ldap plugin after upgrading to Node 10. The verdaccio ldap plugin uses this package.
I tried using the fork of this project to resolve the issues without any luck, so we had no choice but to downgrade to Node 8.

Any updates or reasons to not merge this?

@acappella2017 would you be able to add a unit test for this?

@acappella2017 would you be able to add a unit test for this?

@jsumners Without this pr, ldapjs UT hangs with node 10. I believe current UTs are able to cover this change.

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


appsearch-glib

I am trying to build GIMP from source in msys2. This was possible in the previous version of GIMP. The new version requires appsearch-glib v .77 or higher. Is this available on msys2? I can't find it, but perhaps it has a different name. I have also seen libappsearch.

Yes, thank you. appstream-glib
On Sun, Oct 21, 2018 at 2:25 PM Christoph Reiter<EMAIL_ADDRESS>wrote:

Do you mean appstream-glib?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Alexpux/MSYS2-packages/issues/1462#issuecomment-431696613,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjEqmAUqt6gZZJ4wpbIeu1zdsKXTm4w2ks5unMo4gaJpZM4XyAae
.


XR809 support

I have a few door sensors that have XR3 chips in them. I have been struggling to flash these manually. The few documented flashing guides I have found are a little confusing to me. I am pretty sure I'm doing it right but just can't seem to get it to flash.
What are the possibilities for adding XR809 chip support to the cloud cutter app? I will keep trying to flash the manual way for now but it would be a whole lot easier to do with cloud cutter.

The XR Chips are ARM based - https://developer.tuya.com/en/docs/iot/xr-series-modules?id=Kaiuyiuke8fu1
I know for sure that the XR3 chip is ARM based - https://developer.tuya.com/en/docs/iot/xr3-datasheet?id=K98s9168qi49g
Unfortunately, I have been unsuccessful if dumping the base firmware. The method of flashing the device requires something like 7 flashing pins to be connected at once and the use of a program called PhoenixMC to flash the device. which is not in English so it's a matter of guessing.
There have been some successful flashing with devices over on the elektroda forums.
https://www.elektroda.com/rtvforum/topic3806769.html
https://www.elektroda.pl/rtvforum/topic3771510.html
https://www.elektroda.pl/rtvforum/topic3890640.html
Some of these links are not in English but using TWP - Translate Web Pages extension for firefox help fix the problem
If I do manage to get the firmware for you I will upload it here. For now, I'm stuck.

Hi, just checking to see if this moved forward etc.?

No progress, no dumps have been provided to be analyzed.  I have not yet been able to procure a Tuya device with this chip yet either.

Where are you based? I'm in Australia and could maybe ship mine?
I'm ok with electronics and might be able to create a dump, if you can suggest low cost tech and a guide?

I'm based in the US, but there are many things we need to know before direct hardware even becomes useful, getting a dump is the first step.  There are many things we need to work out first, like encryption (it appears XR809 is unencrypted, but I'm not certain so far), Tuya SDKs and debug files, protocols present in SDKs, OTA formats, and ARM instructions available.  There is no guarantee this is even possible.
I have no experience with XR809, it looks like https://www.elektroda.com/rtvforum/topic3806769.html would be the best resource for getting the tools to potentially create a dump.

https://www.elektroda.com/rtvforum/viewtopic.php?p=21146063#21146063


524 Fix missing image

The copy of jqueryui we had embedded was missing some image
files (https://github.com/miurahr/python-markedit/issues/1).
Fixed by getting rid of the embedded version and loading
jqueryui from Google's CDN, like we're already doing for
some other libraries.
Fixes #524

looks good


[Bug]: appsmith server/container won't start if Admin Setting: Instance Name contains spaces

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When the Admin Setting "Instance Name" )is changed and contains space characters between words the server won't startup after restart.
The following error is shown in the docker logs:
appsmith       | /appsmith-stacks/configuration/docker.env: line 94: **<First word of Instance Name>**: command not found
I manually changed the setting in docker.env filed and removed the space character and the server started correctly after that.
Steps To Reproduce

Go to 'Admin Settings > General'
Change field 'Instance Name' to a name that contain multiple words separated by space.
Restart server/container.

Public Sample App
No response
Version
v1.6.16

Unable to repro. Changed the instance name in admin settings page to have space between words. The app seems to run fine after save and restart.

Hey @kavhad, thanks for reporting. This issue should've been fixed by https://github.com/appsmithorg/appsmith/pull/11799, which is part of the latest release published yesterday. Can you pull the latest image again and check with that please?


color looks wrong when CMYK jpg changed to RGB after opened and saved by image-rs

This happens when I simply used image-rs to open and save a CMYK jpg file
Expected
I expect the CMYK jpg file stays the same(file size, colorspace, etc.)
I did not expect a CMYK jpg be changed to RGB jpg automatically. Because this CMYK jpg is used for printing.
Actual behaviour
the original CMYK jpg file is 12.5Mb, in CMYK; after opened and saved by image-rs, the size was changed to 2.2Mb, color spaced was changed to RGB(found out with Photoshop), and the image color looks different than the original file.
Reproduction steps
Download the original CMYK image file from the link below, and simply run:
use image;
/*
Cargo.toml:
[dependencies]
image = "0.23"
*/
fn main() {
    let img = image::open("cmyk_original.jpg").unwrap();
    img.save("cmyk_open_and_saved.jpg").unwrap();
}

Related Resources
original cmyk image file uploaded to mega.io
open and saved by image-rs

It seems like this is caused by not using the ICC profile embedded in the image to do the CMYK to RGB conversion.

With the risk of being rude(I don't mean to), I just wanna ask is there any update/progress on this issue?
Thanks in advance.


Confusion about *-small icons deprecation

I'm using the 20/solid ArrowSmallDownIcon and ArrowSmallUpIcon in an app, and the React components are now marked as deprecated with not much insight.

The v2.1.0 release notes instruct to use the "mini" set instead, but I can't find references to this set in the published package. The docs page for the "mini" set just seems to be an alias to 20/solid, which is already what I am importing from.
https://github.com/tailwindlabs/heroicons/releases/tag/v2.1.0
https://unpkg.com/browse/<EMAIL_ADDRESS>https://heroicons.com/mini
Then I thought that maybe the 20/solid standard icons would simply be equivalent to the deprecated small icons... but they are not.

Related unanswered discussion: #1149
Would greatly appreciate some clarifications. Will these small variants just be removed at some point and we should adapt to the 20/solid standard icons as the path forward? Is it possible to add details with the @deprecated jsdocs in the components?
Thanks for the great work!

Yeah the small icons in the 24px set (solid or outline) should be replaced with the normal versions in the 20px set. Likewise, the small icons in the 20px set (mini) should be replaced with the normal versions in the 16px set (micro).
I think the idea is that we'll remove these in some later release (my guess would be v3.0 whenever that happens)
I'll look at adding a description to the deprecations 👍


configs/wgan-gp/wgangp_GN_celeba-cropped_128_b64x1_160kiter.py

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
A clear and concise description of what the bug is.
Reproduction

What command or script did you run?

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
What dataset did you use?

Environment
python tools/train.py configs/wgan-gp/wgangp_GN_celeba-cropped_128_b64x1_160kiter.py

Please run python mmgen/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

Hi, @bingpo1314 our MMGeneration does NOT support Dataparallel training. If you start the training by directly using python train.py xxx, the PyTorch will automatically run in Dataparallel. Thus, we recommend that all of our users use distributed training by:
bash tools/dist_train.sh {CONFIG} {GPU_NUM} --work-dir {WORK_DIR}


Issue with auth after computer restart

I've had quite a few issues with auth since starting to use the digital twins explorer. The first time I fired it up, it took multiple tries running az login firing up the app, killing the app, trying az login again, etc. At some point, it just started working, even though the response I got back from az login didn't change.
After that I didn't have problems, but after restarting my computer, I am having issues again, and I am not sure how to debug it. I am getting this pop-up as soon as the page loads:

I have tried logging in with az login and logging out, and restarting the app.

hi @alecmanfre ,
Are you working off the master branch? We have made some updates to the auth package on the main branch that might assist with this.
NOTE: main will become the default branch tomorrow after we retire master.

I just tried going to main, but unfortunately, I am getting the same error. My domain is the same one I was using before and I am listed as an owner on that domain.

I found the issue when trying to add a new user...Our initial subscription had expired on azure, but it wasn't obvious within the portal there was an issue. I have it working now.


Remove procfile worker task for panopticon

The process in the Procfile is no longer necessary and has been removed (https://github.com/alphagov/panopticon/pull/409).

:+1:


Disable ability to change your own Role and Tenant in the User Profile

When editing your own profile in Traffic Portal, it is possible for a user to remove higher level privileges/capabilities for Role and Tenant with no way to reset them to their previous setting as you can only set your role and tenant lower on their trees. We should disable the ability to change your own role and tenant to prevent users from losing capabilities and access.

looks somewhat related to #4064

@chadgilloth Are you still able to reproduce #2731 now that Roles and Permissions were reworked in ATC 6.0.0?


Authentification error

I try to run the sample and receive
https://login.live.com/err.srf?lc=1033#error=invalid_request&error_description=The+provided+value+for+the+input+parameter+'redirect_uri'+is+not+valid.+The+expected+value+is+'https://login.live.com/oauth20_desktop.srf'+or+a+URL+which+matches+the+redirect+URI+registered+for+this+client+application.&state=personal

I get the same error. Was this issue ever resolved?

Enter "http://localhost:52026/Account/Redirect" (or whatever your port is instead of :52026) in the RedirectURL box on the API settings tab in the "Application Registration Portal." https://apps.dev.microsoft.com
Plucked this from issue #1 and everything seems to work as of today. However it is using the old 1.x SDK and they've totally changed authentication and haven't provided any web samples. Just some nonsense that's three layers deep and involves writing varbinary to your database.

I would really appreciate a working AspCoreOneDriveExplorer sample, OneDrive for Business that is :)

I would appreciate knowing why OneDrive for Business is this wonky separate thing that ANYONE has to even worry about besides setting an Enum value in one place, but apparently that level of simplification isn't coming anytime soon. In the meantime yes: headless, Windows and Linux samples of both OneDrive and OneDrive for Business that actually work, please.

Aiya I wish this issue was more prominently featured. This fix worked for me as well.

This fix worked for me as well. I entered "http://localhost:5206/Account/Redirect" in the Redirect URLs box on the Web form in the https://apps.dev.microsoft.com portal. However, in the web.config, I'm using ngrok URL:


Release Arrow 27.0.0 (next release after 26.0.0)

Follow on from https://github.com/apache/arrow-rs/issues/2820

Planned Release Candidate: 2022-11-11
Planned Release and Publish to crates.io: 2022-11-14

Items (from dev/release/README.md):

[x] PR to update version and CHANGELOG:  https://github.com/apache/arrow-rs/pull/3089
[x] Release candidate created: https://lists.apache.org/thread/p7841p1hlr11fdtgb6kwr77ywk0qjfql
[x] Release candidate approved: https://lists.apache.org/thread/oh5prd297hgz5pr0sybn07f6yskh6btr
[x] Grant permissions to arrow-cast on crates.io
[x] Grant permissions to arrow-ipc on crates.io
[x] Grant permissions to arrow-csv  on crates.io
[x] Grant permissions to arrow-json  on crates.io
[x] Release to crates.io:

See full list here:
https://github.com/apache/arrow-rs/compare/26.0.0...27.0.0
cc @iajoiner @tustvold @viirya

It would be really cool if https://github.com/apache/arrow-rs/pull/3052 and https://github.com/apache/arrow-rs/pull/3028 could make this release

Looks like https://github.com/apache/arrow-rs/pull/3006 at least is an API change so I will make a new major release 27

It would be really cool if https://github.com/apache/arrow-rs/pull/3052 and https://github.com/apache/arrow-rs/pull/3028 could make this release

These have both been merged 👍

label_issue.py automatically added labels {'arrow'} from #3112

label_issue.py automatically added labels {'parquet'} from #3089

label_issue.py automatically added labels {'arrow-flight'} from #3089

label_issue.py automatically added labels {'parquet-derive'} from #3089


Minor: Increase Annotation Font Size

@thepritam can you increase the font size by 1 point. Close issue when complete.

@cannin this is completed. I am closing this issue.


Stt text to ground truth outlier remover

Some things to point out. when building the docker image the .opus file fails because stt cant transcribe those files. we also need to add add the huge tflite model and huge vocab scorer to the repo. It won't let me push files that big

when building the docker image the .opus file fails because stt cant transcribe those files.

in this case, running stt should be optional

when building the docker image the .opus file fails because stt cant transcribe those files.

in this case, running stt should be optional

I used an if statement to check the length of sys.argv and if its more than 3 it enables it since the last two args would be the stt model and scorer. Is that an okay way to do it?


keep min_seconds and max_seconds of create_base_retry_decorator configurable

Feature request
Hi
Currently, min_seconds and max_seconds of create_base_retry_decorator are hard-coded values. Can you please make these parameters configurable so that we can pass these values from AzureChatOpenAI similar to max_retries
eg:     llm = AzureChatOpenAI(
deployment_name=deployment_name,
model_name=model_name,
max_tokens=max_tokens,
temperature=0,
max_retries=7,
min_seconds=20,
max_seconds=60
)
Motivation
Setting these values will help with RateLimiterror. Currently, these parameters need to be updated in the library files, which is impractical to set up in all deployed environments.
Your contribution
NA

@hinthornw I had also suggested this over slack. Also curious if the docs for with_retry decorator are up? I'd be happy to give that a try

I would also like this feature for embeddings. It's pointless to retry after 4 seconds when my rate-limited provider is telling me to retry after 43 seconds when my quota replenishes...


Add example of dynamic RTM entries with Timing section

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Extend the mechanism for wrapping RTM population logic to include dynamically disabled entries
Add ability for RTMs to contain line separators
Add an example of dynamic RTM contents with an "Add Timing Channels" button
Adds placeholder GUID, page, scripting APIs, etc. for Timing section
Update User Guide documentation with more specific instructions for modifying RTMs

Why should this Pull Request be merged?
I was looking for a way to implement Dynamic menu items like you can specify in the Custom Device XML. Because the RTM Wrapper is already building a custom menu item, I extended the wrapper VIs to allow programmatic disabling of menu items. From there, two things were needed: 1) the ability to call scripting VIs in the Dispatcher class and 2) remove the 'first call' logic to dynamically rebuild the menu items each time the page is loaded.
As an example of these changes, I started implementing support for the "timing channels" recently added to the ARINC 429 Custom Devices. These were implemented with 4x RTM VIs (2 in support, 2 in CD) and a new section in the CD XML. This approach lets the CD Project and CD XML stay relatively uncoupled with the RTM implementation.
What testing has been done?
Manually test the RTM items in System Explorer.

[ ] Not sure how I feel about the highlight parameter on Add Timing Section.vi. Do we have precedent for this elsewhere?

[ ]  Not sure how I feel about the highlight parameter on Add Timing Section.vi. Do we have precedent for this elsewhere?

I was staying consistent with other VIs in the scripting API like Add Ports Section.vi and Add Databases Section. However, I haven't seen them implemented with a highlight input on the actual CDs we branched from the template.


Cannot debug with gdb

I get an error saying 'no line number information' when I try to debug the 'vulkan_samples' executable:
https://gist.github.com/fctorial/e7187ea834a28e5fae6ca3732f43c912
The gdb version is 9.2 and the gcc version is 9.3.

gdb is complaining that it can't find the source code.
https://sourceware.org/gdb/current/onlinedocs/gdb/Source-Path.html
The easiest way from the command line is to launch vulkan_samples executable with the current working directory in the source directory.
The really easy way is to use an IDE.  I personally use CLion, just set the executable's current working directory and it does the rest.

I'm running gdb from the 'Vulkan-Samples' directory. gdb is looking for a file sysdeps/x86_64/start.S. There's no such file on my system:
(gdb) l
1       ../sysdeps/x86_64/start.S: No such file or directory.
(gdb) b create_vulkan_samples()
Breakpoint 1 at 0x47f6c0
(gdb) r hdr
Starting program: /home/fctorial/src/pgs/Vulkan-Samples/build/linux/app/bin/Release/x86_64/vulkan_samples hdr
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/usr/lib/libthread_db.so.1".

Breakpoint 1, 0x000000000047f6c0 in create_vulkan_samples() ()
(gdb) l
1       in ../sysdeps/x86_64/start.S

@fctorial do you have sufficient direction to track this one down?  Unfortunately, this doesn't seem to be anything the samples team can solve as we don't have ability to control gdb requirements, but let us know if we can help further and if we can close this issue.  Thanks!

Yes, I suppose it's not a problem in the vulkan samples project.


Failfast for explicit error http request instead of retrying (#296)

#296

Failfast http status code: 400, 401, 404, 405
Error Code Description: https://www.apolloconfig.com/#/en/usage/other-language-client-user-guide?id=_16-error-code-description

Fantastic work! Could we incorporate some unit tests to ensure coverage of the newly implemented logic?

@nobodyiam Unit tests 已提交（module protocol/http coverage: 61.4% > 69.3%）

Looks like we need to also fix other test cases.

@nobodyiam fixed, and successfully ran all test cases


CustomMap on a TabbedPage does not render

I have tried to embbed the map into a TabbedPage.
It works fine on Android, however it fails on iPhone (the map is white/not displaying). An idea why?
(Setting the BindingContext before displaying also causes a lot of troubles!)

Hey JeanCollas,
I am using a tabbed page on iOS and have no problems.
It seems that your info.plist needs an update:
add the following lines to your info.plist
NSLocationAlwaysUsageDescription
Can we use your location
NSLocationWhenInUseUsageDescription
We are using your location
find a description here:
https://developer.xamarin.com/guides/xamarin-forms/working-with/maps/

Hello,
I think it is not my info.plist as the rendering works if I start directly on the page, but if I put a tabbed page around, the map is white and does not load (or render).
I have the following lines in the info.plist, it seems ok :
NSLocationWhenInUseUsageDescription
You are using location!
NSLocationAlwaysUsageDescription
You are about to use location!

And I have the init in the starting function (anyway without the tabbed page the map works)

Ok, I found a workaround.....
It has to be encapsulated into a NavigationPage, then the map appears.
I don't understand why, but it works using:
NavigationPage > TabbedPage > ContentPage > TKCustomMap
or
ContentPage > TKCustomMap
But not
TabbedPage > ContentPage > TKCustomMap

Doesn't seem to be related to the control. Closed.


chore: moved code into pkg/ folder

Moves cbt source code into dedicated 'pkg' folder.

Thank you, @enocom for the input. I agree with your sentiment and I'll close this PR.


Refactor acquisition functions

I think it will be best to just have a call method. At the moment I'm having to hack to get around the _f interface.

Done. Closing.


INVALIDARGUMENT

1. What version of Caddy are you using (caddy -version)?
Caddy 0.11.1
2. What are you trying to do?
try quic
3. What is your entire Caddyfile?

sub.example.com {
    realip cloudflare
    log /var/log/caddy/cdn.log {
	rotate_size 100 # Rotate a log when it reaches 100 MB
        rotate_age  14  # Keep rotated log files for 14 days
        rotate_keep 10  # Keep at most 10 rotated log files
        rotate_compress # Compress rotated log files in gzip format
    }
    tls Mail {
        key_type p384
        dns cloudflare
    }
    header / {
        Strict-Transport-Security "max-age=31536000;"
        X-XSS-Protection "1; mode=block"
        X-Content-Type-Options "nosniff"
        X-Frame-Options "DENY"
    }
    root /file
}

example.com{
    realip cloudflare
    log /var/log/caddy/main.log {
        rotate_size 100 # Rotate a log when it reaches 100 MB
        rotate_age  14  # Keep rotated log files for 14 days
        rotate_keep 10  # Keep at most 10 rotated log files
        rotate_compress # Compress rotated log files in gzip format
    }
    tls email{
        key_type p384
        dns cloudflare
    }
    header / {
        Strict-Transport-Security "max-age=31536000;"
        X-XSS-Protection "1; mode=block"
        X-Content-Type-Options "nosniff"
        X-Frame-Options "DENY"

    }

    proxy / http://... {
	transparent
	insecure_skip_verify
    }

}

4.  Please paste any relevant HTTP request(s) here.

6. What did you expect to see?
A normal log.
7. What did you see instead (give full error messages and/or log)?
service caddy status:
caddy.service - Caddy HTTP/2 web server
   Loaded: loaded (/etc/systemd/system/caddy.service; disabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Sun 2018-12-02 09:07:53 EST; 22min ago
     Docs: https://caddyserver.com/docs
  Process: 864 ExecStart=/usr/local/bin/caddy -log /var/log/caddy/stdout -quic -agree=true<EMAIL_ADDRESS>-conf=/etc/caddy/Caddyfile -root=/var/tmp (code=exited, status=2)
 Main PID: 864 (code=exited, status=2)

Dec 02 09:07:53  caddy[864]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake.(*cryptoSetupServer).handleMessage(0xc0000d6000, 0xc0003ec000, 0x410, 0x515, 0xc00019c7e0, 0x1, 0xdadc40, 0xc00000c010)
Dec 02 09:07:53  caddy[864]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup_server.go:162 +0x18f
Dec 02 09:07:53  caddy[864]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake.(*cryptoSetupServer).HandleCryptoStream(0xc0000d6000, 0xf, 0xc000436100)
Dec 02 09:07:53  caddy[864]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup_server.go:119 +0x21a
Dec 02 09:07:53  caddy[864]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go.(*session).run.func1(0xc0001b8000)
Dec 02 09:07:53  caddy[864]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/session.go:452 +0x3a
Dec 02 09:07:53  caddy[864]: created by github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go.(*session).run
Dec 02 09:07:53  caddy[864]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/session.go:451 +0x8a
Dec 02 09:07:53  systemd[1]: caddy.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Dec 02 09:07:53  systemd[1]: caddy.service: Failed with result 'exit-code'.

This problem still happened without ‘-quic’:
Caddy status
● caddy.service - Caddy HTTP/2 web server
Loaded: loaded (/etc/systemd/system/caddy.service; disabled; vendor preset: enabled)
Active: failed (Result: exit-code) since Sun 2018-12-02 12:41:31 EST; 19h ago
Docs: https://caddyserver.com/docs
Main PID: 2240 (code=exited, status=2)
Dec 02 12:41:31 caddy[2240]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake.(*cryptoSetupServer).handleMessage(0xc0000d6000, 0xc00047a000, 0x410, 0x51c, 0xc00019a7b0, 0x1, 0xc00004bfa0, 0x74d459)
Dec 02 12:41:31. caddy[2240]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup_server.go:162 +0x18f
Dec 02 12:41:31 caddy[2240]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake.(*cryptoSetupServer).HandleCryptoStream(0xc0000d6000, 0x45bad6, 0xcd5a8c)
Dec 02 12:41:31 caddy[2240]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup_server.go:119 +0x21a
Dec 02 12:41:31 caddy[2240]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go.(*session).run.func1(0xc0001ae000)
Dec 02 12:41:31 caddy[2240]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/session.go:452 +0x3a
Dec 02 12:41:31 caddy[2240]: created by github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go.(*session).run
Dec 02 12:41:31 caddy[2240]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/session.go:451 +0x8a
Dec 02 12:41:31 systemd[1]: caddy.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Dec 02 12:41:31 systemd[1]: caddy.service: Failed with result 'exit-code'.

I'm thinking it has to do with your CLI arguments. Maybe missing = on -log? But it also does seem fishy that it gives you the trace from the quic-go code even with quic turned off.

It happend again
● caddy.service - Caddy HTTP/2 web server
   Loaded: loaded (/etc/systemd/system/caddy.service; disabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Mon 2018-12-03 21:53:23 EST; 35min ago
     Docs: https://caddyserver.com/docs
  Process: 17796 ExecStart=/usr/local/bin/caddy -log=/var/log/caddy/stdout -log=stdout -quic -agree=true -email=email -conf=/etc/caddy/Caddyfile -root=/var/tmp (code=exited, status=2)
 Main PID: 17796 (code=exited, status=2)

Dec 03 21:53:23  caddy[17796]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake.(*cryptoSetupServer).handleMessage(0xc000098280, 0xc0002d0000, 0x410, 0x519, 0xc0009bff50, 0x1, 0xdadc01, 0x1000000010000)
Dec 03 21:53:23 caddy[17796]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup_server.go:162 +0x18f
Dec 03 21:53:23  caddy[17796]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake.(*cryptoSetupServer).HandleCryptoStream(0xc000098280, 0x434299, 0xd04a80)
Dec 03 21:53:23 caddy[17796]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/internal/handshake/crypto_setup_server.go:119 +0x21a
Dec 03 21:53:23  caddy[17796]: github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go.(*session).run.func1(0xc00066e400)
Dec 03 21:53:23  caddy[17796]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/session.go:452 +0x3a
Dec 03 21:53:23 caddy[17796]: created by github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go.(*session).run
Dec 03 21:53:23  caddy[17796]:         /tmp/gopath_12-01-0750.978339518/src/github.com/mholt/caddy/vendor/github.com/lucas-clemente/quic-go/session.go:451 +0x8a
Dec 03 21:53:23  systemd[1]: caddy.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Dec 03 21:53:23  systemd[1]: caddy.service: Failed with result 'exit-code'.

Duplicate of #2346


get_nhdplus throwing an error - crs

Hello -
Thanks for hosting this package and developing new hydrofabric tools. I'm running into a weird, new issue getting NHDPlusV2 data that I'm having trouble fully tracing, explaining, or remedying.
huc02 <- nhdplusTools::get_huc(id = 18, type = "huc02")

fl <- nhdplusTools::get_nhdplus(AOI = huc02, realization = "flowline")

The get_nhdplus call returns an error (Getting tile is printed for tiles 2-29 too but not necessary here):
Getting tile 30 of 30
Error: arguments have different crs
Spherical geometry (s2) switched on

I've had no issue with this call in the past. This error doesn't show up with an AOI-based call to get HUCs - this works fine:
huc04 <- nhdplusTools::get_huc(AOI = huc02, type = "huc04")

The error message seems to come from sf::chk_equal_crs (called in sf::rbind.sf?). I've recently updated relevant packages and my operating system - here's relevant session info:
version  R version 4.3.1 (2023-06-16)
os       macOS Sonoma 14.0
rstudio  2023.09.0+463 Desert Sunflower (desktop)

hydroloom      1.0.0    2023-10-05 [1] Github (doi-usgs/hydroloom@286c61f)
nhdplusTools   1.0.0    2023-10-02 [1] CRAN (R 4.3.1)
sf             1.0-14   2023-07-11 [1] CRAN (R 4.3.0)

Can you reproduce this on your end?

I am able to reproduce and narrowed this down to a couple of 0-row tables in the set of downloaded tables. I should have a fix up soon.


mimxrt1050_evk Fatal fault in thread tests/kernel/mem_protect/stackprot Fatal fault in thread

Describe the bug
kernel mem_protect stack prot
To Reproduce
Steps to reproduce the behavior:

mkdir build; cd build
cmake -DBOARD=mimxrt1050_evk ..
make
See error

Expected behavior
test PASS
Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)
Screenshots or console output
***** Booting Zephyr OS v1.14.0-rc1-1369-gf30a1c03a260 *****

Starts alternate_thread

alternate_thread: Input string is too long and stack overflowed!

***** Stack Check Fail! *****

Current thread ID = 0x80000068

Faulting instruction address = 0x6000ae44

Fatal fault in thread 0x80000068! Aborting.

Running test suite stackprot

===================================================================

starting test - 1?:抋?x?jW焰}屠.

鳃?
糃?栳?b愞w唼?盛畈c诽F嗕籈vN?媮+8D3?物?孯PH!8y"t3?%g榥[S€<D哥\硉s赼.严/r?贿pI$畴f揗谞偤&ps稦?耋?dI虸╟绕€橛尌z劬??夜∪{?忟W博燘撴?畛?崗D硎:?o??燘 侤?

*"$[8F?L>縞嶊<肌庛
[a?娔?!?t鯜駛诋兾阺x瑆饡怙b???惞MG)圩)fⅵ訒€r$kx櫚倒E5爵A孉~氼6K簳瓞廞愯u旑J爥0?
***** MPU FAULT *****

  Instruction Access Violation

***** Hardware exception *****

Current thread ID = 0x800001ec

Faulting instruction address = 0xe9e8

Fatal fault in thread 0x800001ec! Aborting.

Environment (please complete the following information):

OS: (Linux )
Toolchain ( Zephyr SDK)
Commit SHA or Version used: v1.14.0_rc2

@MaureenHelm

@hakehuang i'm not able to reproduce this.
 I've checked out v1.14.0-rc2 tag, but get the following header when running the board:
***** Booting Zephyr OS v1.14.0-rc1-1238-g350f6f715632 *****
The same goes for #14606 but I was able to reproduce that.

sorry @hakehuang i can't reproduce the MPU FAULT in this issue in rc2

I can't reproduce the fault either. Here's what I'm seeing:
***** Booting Zephyr OS v1.14.0-rc1-1238-g350f6f715632 *****
Starts alternate_thread
alternate_thread: Input string is too long and stack overflowed!

***** Stack Check Fail! *****
Current thread ID = 0x80000068
Faulting instruction address = 0x6000ad20
Fatal fault in thread 0x80000068! Aborting.
Running test suite stackprot
===================================================================
starting test - test_stackprot
test_stackprot: Stack ok
test_stackprot: Stack ok
test_stackprot: Stack ok
test_stackprot: Stack ok
test_stackprot: Stack ok
test_stackprot: Stack ok
PASS - test_stackprot
===================================================================
Test suite stackprot succeeded
===================================================================
PROJECT EXECUTION SUCCESSFUL

this is test setting issue. close it


Query results are inconsistent

presto: 0.193
presto-kudu: 0.193.1
select  col0, col1, col2, colN
from my_table
where year=2018 and month=3 and day=7 and hour=2
limit 1000

Sometimes can find data, sometimes can't.
When the results are empty, I try to comment some columns, and there may be some result to find out.

HI there
I have the same problem too.
when the range partition is {"lower":null,"upper":null}, the result of select count(*) is different.
when I set a specific range partition, it works fine.

Please provide more details which allows to reproduce the problem. There are so many thinkable causes, that it is impossible to get an idea.
Please try to isolate the origin:

is it a problem of some inconsistent data of a Kudu tablet server?
is it a problem of Kudu itself?
is it a problem of the Kudu Java client?
is it a problem of Presto?

Or try to provide a reproducible procedure description.

Can anybody reproduce this issue with Kudu 1.7.0? Thanks.

Hi, When I updated, I will test that. Please wait.

Hi I checked in the env with kudu client 1.7.0.
The problem doesn't happen.

@yechangyao Thanks for checking.


External IPMI Tool Test

External IPMI tool test which is related to https://github.com/openbmc/openbmc/issues/429

Code checked in for review.


Add "update-api" yarn target

To download latest API swagger input file and regenerating our JS types.
Depends on:

[ ] #95

Needs rebase after #89

Rebased


Regex that works with the regex crate but not on a logos token

Firstly, here is the output of cargo tree:
regextest v0.1.0 (<$HOME>/Desktop/regextest)
├── logos v0.12.0
│   └── logos-derive v0.12.0 (proc-macro)
│       ├── beef v0.5.1
│       ├── fnv v1.0.7
│       ├── proc-macro2 v1.0.28
│       │   └── unicode-xid v0.2.2
│       ├── quote v1.0.9
│       │   └── proc-macro2 v1.0.28 (*)
│       ├── regex-syntax v0.6.25
│       ├── syn v1.0.74
│       │   ├── proc-macro2 v1.0.28 (*)
│       │   ├── quote v1.0.9 (*)
│       │   └── unicode-xid v0.2.2
│       └── utf8-ranges v1.0.4
└── regex v1.5.4
    ├── aho-corasick v0.7.18
    │   └── memchr v2.4.0
    ├── memchr v2.4.0
    └── regex-syntax v0.6.25

And here is code that demonstrates the issue:
use logos::Logos;
use regex::Regex;

#[derive(Logos, Clone, Debug, PartialEq)]
pub enum LogosToken {
    #[regex(r"(?m)\(\*([^*]|\*+[^*)])*\*+\)")]
    Comment,

    #[error]
    Error,
}

fn main() {
    let test = "(* hello world *)";
    let r = Regex::new(r"(?m)\(\*([^*]|\*+[^*)])*\*+\)").unwrap();
    assert!(r.is_match(test));

    let mut lexer = LogosToken::lexer(test);
    assert!(matches!(lexer.next(), Some(LogosToken::Comment)));
}

As you can see, the same regex is being used and the same test string but, the first assert passes and the second panics. I'm not sure if I'm missing something or this is a bug

Logos isn't using the regex crate, it's using regex-syntax for syntax only, and implements only a subset of regex.

Logos isn't using the regex crate, it's using regex-syntax for syntax only, and implements only a subset of regex.

I see, could you suggest some changes I could make to the regex in order for it to fit within that subset?

@jawadcode see this really useful discussion on regexs for comments: https://github.com/maciejhirsz/logos/issues/133
Also, I think ?m means multiline mode. If that's the case, one way you could solve this instead is by using separate tokens for (* and )*. Then, use extras to keep track of when a comment begins and ends. Everything in between is part of that comment only if a comment has begun.
Here's a simple example of extras:
https://github.com/maciejhirsz/ramhorns/blob/9f2ed4d3d173cc835dc0a281662b0a8e87a348f2/ramhorns/src/template/parse.rs#L21-L26


Update links for Jira to GitHub issue transition in README (backport #2956)

A backport of pull request #2956

Please sign the commit:
https://github.com/hyperledger/fabric/pull/3402/checks?check_run_id=6373133228

@denyeart I think I've fixed it


feat: github release summary

Release Notes 123
New Features

Release Summary: We've introduced a new feature that provides a comprehensive summary of each release, making it easier for you to stay informed about the latest updates and improvements.

Release Action: We've enhanced our release process with a new action feature, streamlining the deployment and management of updates.

Python Code Conversion: We're in the process of converting parts of our system to Python, which will improve performance and maintainability. This is a work in progress and will be rolled out in phases.

Improvements

Testing Enhancements: We've added new tests to ensure the reliability and stability of our system, providing you with a more robust experience.

We hope you enjoy the new features and improvements in this release. As always, we welcome your feedback to help us continue to enhance our product.


Use @angular-devkit for source code manipulation

https://angular.io/guide/schematics-authoring

virtual-fs (name)
strings (source code)


Support building a Linux ARM64/aarch64 binary

In order to support building for Linux on ARM64/aarch64 (64bit) we could easily cross compile Docker Machine. In Go we use "linux/arm64", but on Linux the architecture is called "aarch64" which can be determined with uname -m.
Testing a build locally within Docker and only compile an ARM64 binary is easy with this command:
USE_CONTAINER=true TARGET_OS=linux TARGET_ARCH=arm64 make build-x

It builds the binary ./bin/docker-machine-Linux-aarch64.
Signed-off-by: Dieter Reuter<EMAIL_ADDRESS>

It turns out that at least one of the dependencies doesn't compile on ARM64 right now. So the build gives the following error messages:
GOOS=linux GOARCH=arm64 CGO_ENABLED=0 go build -o /go/src/github.com/docker/machine/bin/docker-machine-Linux-aarch64 -a  -tags "static_build netgo " -installsuffix netgo -ldflags "-X `go list ./version`.GitCommit=`git rev-parse --short HEAD 2>/dev/null` -w -s -extldflags -static"  ./cmd/machine.go;
# github.com/docker/machine/vendor/github.com/bugsnag/panicwrap
vendor/github.com/bugsnag/panicwrap/monitor.go:57: undefined: syscall.Dup2
mk/build.mk:28: recipe for target 'build-x' failed
make: *** [build-x] Error 2

Hopefully someone can jump in and is able to help to resolve this kind of issue.

Ping @nathanleclaire, with #3999 merged this PR should be also ready to go. Could you just re-run the CI to check it, please?

Sure!

It still fail, but Travis just being grumpy about TLS. Re-trying

Great, now it seems to build successful. CI is green! 🎉

Yay Travis is in a good mood again

LGTM! Thanks @DieterReuter hope you enjoy

WOOT, next part will be build for aarch64. Good to see the progress.


[fix][doc] Fix issues in documentation of topic-pattern parameter

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[ ] doc-not-needed
(Please explain why)

[x] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

/pulsarbot rerun-failure-checks


Migrate to swift 3 and Xcode8

ImageViewer is awesome. I use ImageViewer in an app that is not going to be released any time soon. ImageViewer was easy to integrate with. Love your project.
Xcode8.0 beta was released yesterday and I have started migrating my own projects to swift3.
Any plans for this?

@neoneye we are currently at SF, next week we will look into it. (most probably we will create a swift3 branch a have the work there)

@neoneye we might do this, a bit further down the line, as right now we don't have a lot of pressure to move to Xcode8. If you are interested, you can open a PR.


[labs/react] Custom Element class fields cannot be set on a React component

Which package(s) are affected?
React (@lit-labs/react)
Description
When using a Class Field in a Web Component, it's not possible to set this as a prop on the React component.
Reproduction
import React from "react";
import { createComponent } from "@lit-labs/react";

class SomeElement extends HTMLElement {
  x = "Hello World";
  connectedCallback() {
    setInterval(() => {
      this.textContent = this.x;
    }, 1000);
  }
}
customElements.define("some-el", SomeElement);

const SomeEl = createComponent(React, "some-el", SomeElement);

export default function App() {
  return (

      <SomeEl x={"Hello Web Components"}>Not Connected</SomeEl>

  );
}

The demo will eventually render Hello World but never Hello Web Components.
https://codesandbox.io/s/sad-sky-q5tosq?file=/src/App.tsx
Workaround
The workaround is to refactor the web components properties to not be class fields, and instead use getter/setters:
import React from "react";
import { createComponent } from "@lit-labs/react";

class SomeElement extends HTMLElement {
  #x = "Hello World";
  get x() {
    return this.#x
  }
  set x(v: string) {
    this.#x = v
  }
  connectedCallback() {
    setInterval(() => {
      this.textContent = this.x;
    }, 1000);
  }
}
customElements.define("some-el", SomeElement);

const SomeEl = createComponent(React, "some-el", SomeElement);

export default function App() {
  return (

      <SomeEl x={"Hello Web Components"}>Not Connected</SomeEl>

  );
}

Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Affected versions
1.0.9
Browser/OS/Node environment
I don't think these details are relevant to this issue.

Hey @keithamus thanks for submitting!
I don't think the above example would work even without the wrwapper.

@keithamus thanks for submitting!
I don't believe the above would work
Here's a gist on lit.dev showing the vanilla behavior here.
https://lit.dev/playground/#gist=a9139562a4ae08ce32c3b27e31a4fdd6
The main takeaway is Element.textContent on a custom web component is (most likely) not reactive.
However using attributes as a way to change props gives the web component to update what is rendered.
Here's an example showing how to get the functionality you demonstrated with a shadowRoot and using getAttribute('x').
https://codesandbox.io/s/beautiful-shadow-dglv3v?file=/src/App.tsx:0-691

Closing issue, feel free to reopen :)


Silent Hill 2: Enhanced Edition - Tabbing out of the game can cause loss of mouse functionality.

Describe the bug

Tabbing in and out of the game can cause your mouse pointer to simply not function anymore, not in the application nor in windows itself. I originally thought this was an issue isolated to this game but talking to some other people on the remix server this issue seems to appear in many other titles as well.
Mouse control can be regained by CTRL ALT DELETing.
Mouse control is not always immediately lost on first time tabbing out, continue tabbing in and out over a short period and the likelyhood of the issue happening rises.

Issue is present on all versions of remix

To Reproduce
Steps to reproduce the behavior:

Set the game and Enhanced Edition up by following my guide - https://docs.google.com/document/d/1hayGXNlAJZWOp2ivV8otdSMP23pbXk7DL-pfLq5ZJjU
Start the game and skip the opening cinematics
Once standing in the bathroom begin tabbing in and out for a while

Expected behavior

Tabbing in and out of the game functioning normally

REMIX-2569 for tracking.

Hi @traggey! This issue was addressed in a recent release and confirmed internally. As such, we're going to close this ticket. Please open another one if the issue persists for you.

@NV-LL @nv-ajaus
Just tested, doesn't seem fixed as it still occurs.

Hey, @traggey - we recently pushed an update to the bridge binary that we think should address this. Will you please check with the latest bridge build and let us know? Thanks!

Hi @traggey - checking in to see if you were ever able to confirm the fix from July per the comment above. When you have the chance, please re-test on the latest build and let us know if the problem persists. Thanks!

Hi @traggey - checking in to see if you were ever able to confirm the fix from July. When you have the chance, please re-test on the latest build and let us know if the problem persists. We'd like to confirm with you before we close the ticket as stale. Thanks!

@NV-LL
Apologies for the slow reply, I broke my arm a bit back and have been out of commission.
Did some testing today and it has KIND of been fixed? I can now tab in and out of the game without issue, however after having tabbed in and out and then trying to use the developer menu in remix using alt x, the mouse gets frozen in place and can no longer interact with the menu, it works okay in the game still but the menu is a no-go.


Refactor NumberDisplay

Make Animator more customizable.

This control should be removed as user customization is not supported.


User guide

Description
This is the "official" beta-worthy version of the User Guide. This branch has been merged with the latest develop branch prior to submitting this PR.
Proposed changes
I'm assuming we'll want to release this with v2.0. It includes three examples that worked for the developer of said examples but I haven't had time to verify. A fourth example (using ns-3) is underway but won't be ready by the time of v2.0 release. Most of the supporting text for the example has been written but the full co-sim has not been executed.
I'm not assuming what I've written is without flaws and it is incomplete in some places. Its probably 90% of what I hoped it would be and I'm going to call that good enough for v2.0 release. I plan on working on it after the v2.0 release, as well.

Just added several of you as PR reviewers; treat this as a suggestion rather than a hard request. Remove yourself if you can't perform the review or feel free to give a cursory review before approval.

And I've already noticed that I didn't complete the section on "Integrating a New Simulator" as well as not providing enough instruction on using helics_cli. I'll be working in on the former today; I assume it will not end up as a part of this PR (right?). The later I will need some help to understand better before I can write on it.

I've just realized I didn't finish the "Timing" section, either. I'm going to cancel this PR and put another one together by COB Thursday.


Check if last seed button exists

For ttN conflict as per https://github.com/chrisgoringe/cg-use-everywhere/issues/187

Thanks, @TinyTerra


[Feature Request]: Right click to copy link

requested on the forum
This is something I'd like to support, and would accept a PR for. The only issue I can foresee is that I'm not sure if there is a good way to dynamically change the context menu based on what's being clicked.
What would be nice is the ability for a plugin developer to provide a function that gets applied to the text under the cursor which would determine if the menu item appears or not. I think similar functionality is already available and could maybe be extended for this use.

I had actually started making some changes towards this, because I wanted to add support for saving an image from the editor, but didn't go very far.
I think the current approach for context menu is not ideal. Instead of declaring in advance the menu, the app should call some function in the plugin, and the menu could be built at that point. With that method the plugin can display different menus depending on the context.
If I remember well the tricky part was to change some sync calls to async in the app, so that it's possible to call the plugin and wait for the result.

Fixed in #30


Spaces in between words are kept when editing or adding names

This may be undesirable when deleting students as the exact number of spaces must be keyed in between the names again. This may also be undesirable when viewing students as it looks weird.
However, it does not exactly affect normal operations and is therefore not a must to fix.

This is similar to the strict view parsing bug that Ansel has found. We could consider modifying the parser to trim out additional whitespaces between names.


[ Feature Request ] Fine tune the output of status command.

As discussed with @gauravtyagi20 the output in % form would be a better UI .
Will start working on this one soon.

Currently printing the output in the following format
devnagri-cli/_dev git/develop*
❯ ./beta status
The current status of your project from Devnagri

Project Status :  draft

~~~~~~~~~

langName :  hi
pendingWordsCount :  290
reviewedWordsCount :  3
totalWordsCount :  293
translatedWordsCount :  0


rearranged folder structure

Moved js and html files into the main node directory so that they will be found by node red after installing with npm

test comment

Fixed the issue by updating the package.json file, no need to move the files out of the folder now. Thanks for looking into the issue.


[WIP] Fe setup structure

This is a draft for the future base class implemetnations for setting things up.
It allows the development of new protocols, such they fit seemingly into the concept.
This PR was discussed by the protocol devs.

Hello @RiesBen! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file gufe/__init__.py:

Line 34:80: E501 line too long (86 > 79 characters)

In the file gufe/setup/alchemical_network_planner.py:

Line 11:80: E501 line too long (82 > 79 characters)

In the file gufe/setup/chemical_system_generator.py:

Line 11:1: E302 expected 2 blank lines, found 1
Line 20:80: E501 line too long (81 > 79 characters)
Line 25:36: W292 no newline at end of file

In the file gufe/setup/network_planning/atom_mapping_based/atom_mapping_scorer.py:

Line 28:80: E501 line too long (111 > 79 characters)
Line 40:80: E501 line too long (89 > 79 characters)

In the file gufe/setup/network_planning/atom_mapping_based/ligandnetwork.py:

Line 15:80: E501 line too long (96 > 79 characters)

In the file gufe/setup/network_planning/component_mapping_scorer.py:

Line 7:5: E271 multiple spaces after keyword
Line 9:1: E302 expected 2 blank lines, found 1
Line 24:80: E501 line too long (111 > 79 characters)
Line 34:80: E501 line too long (89 > 79 characters)
Line 36:13: W292 no newline at end of file

In the file gufe/setup/network_planning/network_plan.py:

Line 129:80: E501 line too long (81 > 79 characters)
Line 315:80: E501 line too long (80 > 79 characters)

In the file gufe/setup/network_planning/network_planner.py:

Line 10:1: E302 expected 2 blank lines, found 1
Line 17:55: E231 missing whitespace after ':'
Line 17:80: E501 line too long (95 > 79 characters)
Line 18:80: E501 line too long (97 > 79 characters)
Line 27:22: E222 multiple spaces after operator
Line 30:5: E303 too many blank lines (2)
Line 30:66: E225 missing whitespace around operator
Line 30:80: E501 line too long (80 > 79 characters)
Line 34:80: E501 line too long (96 > 79 characters)
Line 34:81: E225 missing whitespace around operator
Line 44:12: W292 no newline at end of file

@RiesBen please let me know when you'd like a review from me! Happy to help you get this into gufe!

@RiesBen can you offer a response to @IAlibay above? Can you give us a checklist of things you aim to do before you declare this PR ready for review?

@RiesBen can you offer a response to @IAlibay above? Can you give us a checklist of things you aim to do before you declare this PR ready for review?

Sure :)
ToDos:

[ ] discuss change suggestions in order to find a clear direction with some changes

[ ] Labels for ChemicalSystems transfer them from OpenFE to here or not?
[ ] NetworkPlan  name improvement
[ ] How to proceed with code/import structure.

[ ] adapt the PR to the discussions.
[ ] Check and adapt unit test

Current Failure reasons are 2:

Gufe key stability: the key for LigandNetwork and LigandAtomMapping is not equal to the expected GufeKey. (affects: test_key_stable, test_repr
test_graph_annotations: The Annotations of the LigandAtomMapping is lost, after testing found this to have some weird behavior when you call ligand_network.edges and `ligand_network.graph..


[4.0] Deleting the old image while replacing is missing in docs.

Bug report
What I did
I added a field with the type of image for the Article model. I used the function setImageAttribute from the docs here: https://backpackforlaravel.com/docs/4.0/crud-fields#image
What I expected to happen
When I replace the image while updating the article, the old image should be removed.
What happened
When I replace the image while updating the article, the old image does not remove.
What I've already tried to fix it
I solved the issue by adding
// delete the old image from disk
\Storage::disk($disk)->delete($this->{$attribute_name});

inside the if statement
if (starts_with($value, 'data:image'))

Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 7.4.0 (cli) (built: Nov 27 2019 10:14:18) ( ZTS Visual C++ 2017 x64 )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
LARAVEL VERSION:
v7.0.8@d71ba61d51aec79a0a0f907991ec215dc24b09d4
BACKPACK VERSION:
4.0.49@986f5e69593eab33e567787e0c28efb6747cd7b5

Hello once again @mamprogr :)
So .. this is a documentation issue ?
I think it's safer check if Storage::has() before you delete it ?

Yes, the problem in deleting from disk. The database part is working well.
As you said, it is a documentation issue.

I think it's safer check if Storage::has() before you delete it?

Yes, here is the final piece of code:
if (\Storage::disk($disk)->exists($this->{$attribute_name}))
{
    \Storage::disk($disk)->delete($this->{$attribute_name});
}

Once again, thank you very much @mamprogr . I'v just submited THIS PR
Going to close this. :+1:


Hardening of resource-metadata lambda

On AWS, we are using AWS Control Tower / Landing Zone to manage a multi AWS account environment. By using Control Tower, multiple Lambda Functions are created in each account that is managed by AWS Control Tower. Those lambdas have very restrict service control policies, prohibiting most roles/users to read/describe the lambdas - actually the lambdas should be managed by the Organization account only.
The resource-metadata lambda stumbles on those lambdas and terminates with an Error (see below). In my opinion, the lambda should be hardened and skip those resources (with some log output with severity WARN), but should not fail completely in such a situation:
ERROR	Invoke Error
{
    "errorType": "AccessDeniedException",
    "errorMessage": "User: arn:aws:sts::12345678900:assumed-role/Coralogix-resource-metadata-IXjzla5XZGnF-Role/Coralogix-resource-metadata-IXjzla5XZGnF is not authorized to perform: lambda:GetFunction on resource: arn:aws:lambda:eu-central-1:12345678900:function:srv-lz-create-123-xyz with an explicit deny in a service control policy",
    "name": "AccessDeniedException",
    "$fault": "client",
    "$metadata": {
        "httpStatusCode": 403,
        "requestId": "2759b649-59a2-4153-bfca-9f101cd24418",
        "attempts": 1,
        "totalRetryDelay": 0
    },
    "message": "User: arn:aws:sts::12345678900:assumed-role/Coralogix-resource-metadata-IXjzla5XZGnF-Role/Coralogix-resource-metadata-IXjzla5XZGnF is not authorized to perform: lambda:GetFunction on resource: arn:aws:lambda:eu-central-1:12345678900:function:srv-lz-create-123-xyz with an explicit deny in a service control policy",
    "stack": [
        "AccessDeniedException: User: arn:aws:sts::12345678900:assumed-role/Coralogix-resource-metadata-IXjzla5XZGnF-Role/Coralogix-resource-metadata-IXjzla5XZGnF is not authorized to perform: lambda:GetFunction on resource: arn:aws:lambda:eu-central-1:12345678900:function:srv-lz-create-123-xyz with an explicit deny in a service control policy",
        "    at throwDefaultError (/var/task/node_modules/@smithy/smithy-client/dist-cjs/default-error-handler.js:8:22)",
        "    at /var/task/node_modules/@smithy/smithy-client/dist-cjs/default-error-handler.js:18:39",
        "    at de_GetFunctionCommandError (/var/task/node_modules/@aws-sdk/client-lambda/dist-cjs/protocols/Aws_restJson1.js:2676:20)",
        "    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)",
        "    at async /var/task/node_modules/@smithy/middleware-serde/dist-cjs/deserializerMiddleware.js:7:24",
        "    at async /var/task/node_modules/@aws-sdk/middleware-signing/dist-cjs/awsAuthMiddleware.js:14:20",
        "    at async /var/task/node_modules/@smithy/middleware-retry/dist-cjs/retryMiddleware.js:27:46",
        "    at async /var/task/node_modules/@aws-sdk/middleware-logger/dist-cjs/loggerMiddleware.js:7:26",
        "    at async file:///var/task/lambda.js:49:32",
        "    at async traverse (file:///var/task/common.js:32:22)"
    ]
}

Hi @dennischristmann
The fix is released in the latest version of the lambda. Let me know in case you run into any further issues.


⚠ Samsung has degraded performance

In 113cabe, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2382 ms

Resolved: Samsung performance has improved in 73775a9.


Menubar Designed

Preview

(The debug button won't show up in the prod version)
New things

Menubar's popup will auto update on height change.
ipcMain listens for 'closeMenubar'

Since the menubar popup is just a react component we can even put it in the components sheet, so I did :)
@keybase/react-hackers & @cecileboucheron

Looking great!!
Any chance we can push down the folder icons a little bit? I think they're a bit too high. I would say 3px down.Apart from this, it's great for me.

@chrisnojima changed all the style stuff, tal please?

:+1:


fix(discovery): remove unused shared discovery service code

This code is only relevant to MTC and should be removed.

What: Closes #

Additional issues:

Awesome, thanks @ibolton336! Looks like this may close #68?


[Map] Add cameras to centcomm

About the PR

Added camera routers and cameras
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

no cl

did you use savemap for this

did you use savemap for this

savegrid this time