Temurin JDK/JRE 11 - jdk-<IP_ADDRESS>+1 - Upgrade issue

Please provide a brief summary of the bug
Hello,
Downloaded and installed - https://github.com/adoptium/temurin11-binaries/releases/download/jdk-<IP_ADDRESS>%2B1/OpenJDK11U-jdk_x64_windows_hotspot_<IP_ADDRESS>_1.msi
We are facing issue during upgrade.
Installed the latest version "<IP_ADDRESS>" on top of "<IP_ADDRESS>" - Both versions co-exist as shown below:

Tested the following scenarios:
Installed <IP_ADDRESS> on top of <IP_ADDRESS> - CO-EXIST
Installed <IP_ADDRESS> on top of <IP_ADDRESS> - Upgrades
Installed <IP_ADDRESS> on top of <IP_ADDRESS> - Upgrades
Tried the same with x86 as well - https://github.com/adoptium/temurin11-binaries/releases/download/jdk-<IP_ADDRESS>%2B1/OpenJDK11U-jdk_x86-32_windows_hotspot_<IP_ADDRESS>_1.msi
Same behavior as x64.

Could you please help us with this?
Thank you.
Please provide steps to reproduce where possible
Installed https://github.com/adoptium/temurin11-binaries/releases/download/jdk-11.0.20%2B8/OpenJDK11U-jdk_x64_windows_hotspot_11.0.20_8.msi with default selection.
On the same account, installed https://github.com/adoptium/temurin11-binaries/releases/download/jdk-<IP_ADDRESS>%2B1/OpenJDK11U-jdk_x64_windows_hotspot_<IP_ADDRESS>_1.msi with same default selection.
Both versions co-exist without upgrading to higher version.

Expected Results
It should be upgraded with earlier version like the following:
<IP_ADDRESS> on top of <IP_ADDRESS> - Upgrades.
<IP_ADDRESS> on top of <IP_ADDRESS> - Upgrades.
Actual Results
While installing<IP_ADDRESS> on top of <IP_ADDRESS> it should upgrade, not CO-EXIST.
What Java Version are you using?

What is your operating system and platform?
Windows 10
How did you install Java?
Java is not installed.
Did it work before?
Installed <IP_ADDRESS> on top of <IP_ADDRESS> - Upgrades.
Installed <IP_ADDRESS> on top of <IP_ADDRESS> - Upgrades.

Did you test with the latest update version?
Yes

Did you test with other Java versions?
No

Relevant log output
No response

@gdams & @douph1 - Are you able to take a look?  I suspect the exrta point release logic failed somewhere

Windows installer don't provide update capabilities for same version and Windows Installer is limited to 3 numbers in versioning so for Windows Installer
<IP_ADDRESS>  = 10.0.20
<IP_ADDRESS> = 10.0.20
No upgrade support
This is a know limitation of Windows Installer and our 4 digit versionning :
https://adoptium.net/installation/windows/#_upgrade_limitation


Chore: CSS cleanup + dark mode support

Oof, I did a bad merge somewhere and this PR is looking rough... I think I'll close and create a new branch


ci: Fix aarch64 container build & additional cleanups

Diff is okay, I'll merge and release a version that'll finally build the container version...


Correct example in the documentation

Fixes #586

LGTM; rebased and merged as https://github.com/spring-cloud/spring-cloud-stream/commit/bac8877a524042cac993402fb8d30dd9c264b82e


Failed to execute npm run release

Hi，
I am not sure if this is a problem,
When I execute the npm run release command, an error is thrown: No gulpfile found.
Is it because of gulpfile.js -> gulpfile.cjs. I tried to "release": "gulp release" -> "release": "gulp -f gulpfile.cjs release" in package.json, the command can be executed correctly.

Hey @verybigzhouhai - I'd ask on the forum if you're not sure if what you hit is a bug or not: https://community.cesium.com/
I think you just need to run npm run build to generate that file before running npm run release.


Out of date content: Building the ExpressRoute Gateway and VPN gateway sequence has changed.

Hello,
as per subject the sequence of build the coexist gateways via powershell has changed. Please see the error I got when i follow the steps:
PS C:> $gw = New-AzureRmVirtualNetworkGateway -Name "Gateway-ER1" -ResourceGroupName "RG1" -Location "EastAsia" -IpConfigurations $gwConfig -GatewayType "ExpressRoute" -GatewaySku Standard
WARNING: The output object type of this cmdlet will be modified in a future release.
New-AzureRmVirtualNetworkGateway : To deploy both an Expressroute and VPN gateway in the same vnet, the Expressroute
gateway must be created first. Delete the VPN gateway /subscriptions/d07bc3c7-6bd0-496b-a3bf-497e9dee8476/resourceGroups/RG1/providers/Microsoft.Network/virtualNetworkGateways/VPN-Gateway and retry.
StatusCode: 400
ReasonPhrase: Bad Request
OperationID : '70be57fd-3db1-4872-8de4-aa12320e57d6'
At line:1 char:7

$gw = New-AzureRmVirtualNetworkGateway -Name "Gateway-ER1 ...

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

CategoryInfo          : CloseError: (:) [New-AzureRmVirtualNetworkGateway], NetworkCloudException
FullyQualifiedErrorId : Microsoft.Azure.Commands.Network.NewAzureVirtualNetworkGatewayCommand

Anyway, I've successfully built the coexist connections with this sequence: ExpressRoute -> ExpressRoute Gateway -> ExpressRoute Connection -> VPN Gateway -> VPN Connection.

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

ID: 23d4f079-654e-2053-4654-115a55e9379f
Version Independent ID: 2c032f28-ebd6-513e-4324-622ac80e85d6
Content: Configure ExpressRoute and Site-to-Site VPN connections that can coexist: Resource Manager: Azure
Content Source: articles/expressroute/expressroute-howto-coexist-resource-manager.md
Service: expressroute
Product: unspecified
GitHub Login: @charwen
Microsoft Alias: charwen,cherylmc

@antony-chan Could you please verify that there is not already a gateway attached to the virtual network? Also, please verify that your Azure PowerShell is up to date.

@antony-chan in the past, you must create VPN gateway first. We're rolling out the code to remove this limitation. Our doc is a bit ahead of our code rollout. For now, please create ER gateway first.

@antony-chan We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@TravisCragg-MSFT Has this been fixed, and if not, can you tell us when it will be?
Additionally, is there no way to change the size of the subnet gateway without completely removing the VNG?
Thanks!

@jlon125 It has not been fixed, and I am unable to get an exact timetable. For now go ahead and create the ER gateway first.
You must remove / recreate the VNG in order to make changes to the Gateway Subnet.

@TravisCragg-MSFT Thank you for the reply Travis! I don't need expressroute now - but we plan on adding it later (in a few months) Will I be ok to create the S2S now, and when ready, add the expressroute? (assuming the code has been rolled out, and is retroactive for this? I assume we aren't more than a few months out for this code rollout?
Additionally, is there a way to keep the same public IP when removing and re-adding the gateway? Or somehow preserving vpn connection properties? We had a heck of a time getting our S2S working, and unfortunately only have a /28 GatewaySubnet, so to add expressroute we need to resize.
Thank you!

@jlon125 I cannot give an exact time for the update, but it is very likely that you will be ok to create your S2S now.
If you would like to keep the same Public IP when removing & adding the gateway, I would first set the public IP on your gateway to "Static", and make sure to note the Name & Resource Group of the Public IP.
Next, you can delete the Gateway. When you recreate the new Gateway, be sure to specify your old Public IP in the "First IP Configuration" area.

Has this been corrected? At least in the portal, It will not let me add an ExpressRoute Gateway to my vnet (says I already have a gateway assigned, which I do, for the VPN)

Keen to know if this has been resolved yet also. Have a client who is deploying S2S VPN first, but end-state will be ExpressRoute with VPN failover.


Teach Arduino to detect number of connected drives

This may not be possible.

One thing to try might be to set the drive enable pins on the arduino to inputs temporarily (during startup or something) and also turn on their pull-ups, then read them all and (big maybe here, would need to see floppy circuit diagrams) whichever ones read as low are likely to be connected.

Further experimentation welcome here, but it doesn't seem easily possible. Leaving it as a constant for now.

The floppy's enable and step pins are pulled high (+5) by default, and upon startup of the Arduino we can probably enable pull-downs on the pins used to control the floppies and easily discern which pins are connected to drives.

Merged with 0ad0073.


AltGr Key not working in vnc-console with KVM and non-US keyboard

Bug Report
Version of OpenNebula

Development build

Component

Sunstone

Description
noVNC does not work for AltGr enabled characters
From Steffen Limmer

i have the same problem (using iceweasel 38.6.1). I tried to debug the noVNC code and found the following lines in the keyboard.js from noVNC:
// Firefox sends keypress even when no char is generated.
// so, if keypress keysym is the same as we'd have guessed from keydown,
// the modifier didn't have any effect, and should not be escaped
if (queue[0].escape && (!cur.keysym || cur.keysym.keysym !== queue[0].keysym.keysym)) {
cur.escape = queue[0].escape;
}
When i comment this out and run Sunstone in development mode, AltGr is working for me.
Additionally, there is no problem with AltGr when i connect with vncviewer. So it seems to be an issue with noVNC?
Hope, this information will help to fix the bug.

Patch already applied in 5.4.x


Only initialize MPI if necessary

In the master branch of simsopt presently, MPI is initialized whenever any part of simsopt is imported, even just import simsopt. This is a problem because some HPC systems, such as Cobra and Raven at IPP, only allow MPI to be initialized when programs are submitted through slurm, not on login nodes. This means simsopt is not presently usable on the login nodes even for non-computationally-expensive scripts. This might also be frustrating for new users, if they try import simsopt on a login node and it crashes with an MPI error. This PR fixes this issue by modifying some imports so that MPI is only initialized when it is really necessary, such as for the MpiPartition or Vmec classes. A test was also added to the CI to verify that import simsopt and importing a surface class do not initialize MPI now.

In that case, sys modules should have mpi4py.MPI as one of the modules.
Let me do a few checks regarding the approach I proposed.
On Sun, Nov 21, 2021, 11:31 AM Matt Landreman @.***>
wrote:

@.**** commented on this pull request.
In tests/verify_MPI_not_initialized.py
https://github.com/hiddenSymmetries/simsopt/pull/164#discussion_r753824311
:

+"""

+import simsopt
+from simsopt.geo.surfacerzfourier import SurfaceRZFourier
+
+import mpi4py
+# The next line tells mpi4py to not initialize MPI when mpi4py.MPI is imported:
+mpi4py.rc.initialize = False
+
+from mpi4py import MPI
+
+if mpi4py.MPI.Is_initialized():

print("ERROR! Importing simsopt should not initialize MPI")
exit(1)
+else:
print("Verified that importing simsopt does not initialize MPI")

Here's the reason the new test was written that way. FiniteDifference and
MpiFiniteDifference are in the same module, with the serial
FiniteDifference imported when the top-level simsopt module is imported
via least_squares_serial_solve. So the @simsoptRequires in front of
MpiFiniteDifference requires that simsopt tries to import mpi4py when a
user calls import simsopt or imports the serial FiniteDifference.
Importing mpi4py does not initialize MPI; usually importing mpi4py.MPI
does, but not if mpi4py.rc.initialize=False. So checking "mpi4py" in
sys.modules does not generally tell us whether MPI has been initialized.
Whereas mpi4py.MPI.Is_initialized() gives the answer reliably. In
finite_difference.py and perhaps elsewhere in the future, it can be
useful to try: import mpi4py to see if mpi4py has been installed in a
module that we still may wish to use on a login node, i.e. without
initializing MPI.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/hiddenSymmetries/simsopt/pull/164#discussion_r753824311,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA62VEGNUVJE7RJHDL6KHCTUNENGXANCNFSM5IPHIIVQ
.

In the current master branch, importing simsopt imports mpi4py and
mpi4py.MPI modules. So checking for mpi4py.MPI should be sufficient. After
installing your branch

import simsopt

import sys

"mpi4py" in sys.modules

True

"mpi4py.MPI" in sys.modules

False
Bharat Medasani
Engineer
Princeton Plasma Physics Lab (PPPL)
On Sun, Nov 21, 2021 at 11:51 AM Bharat Medasani @.***> wrote:

In that case, sys modules should have mpi4py.MPI as one of the modules.
Let me do a few checks regarding the approach I proposed.
On Sun, Nov 21, 2021, 11:31 AM Matt Landreman @.***>
wrote:

@.**** commented on this pull request.
In tests/verify_MPI_not_initialized.py
https://github.com/hiddenSymmetries/simsopt/pull/164#discussion_r753824311
:

+"""

+import simsopt
+from simsopt.geo.surfacerzfourier import SurfaceRZFourier
+
+import mpi4py
+# The next line tells mpi4py to not initialize MPI when mpi4py.MPI is imported:
+mpi4py.rc.initialize = False
+
+from mpi4py import MPI
+
+if mpi4py.MPI.Is_initialized():

print("ERROR! Importing simsopt should not initialize MPI")
exit(1)
+else:
print("Verified that importing simsopt does not initialize MPI")

Here's the reason the new test was written that way. FiniteDifference
and MpiFiniteDifference are in the same module, with the serial
FiniteDifference imported when the top-level simsopt module is imported
via least_squares_serial_solve. So the @simsoptRequires in front of
MpiFiniteDifference requires that simsopt tries to import mpi4py when a
user calls import simsopt or imports the serial FiniteDifference.
Importing mpi4py does not initialize MPI; usually importing mpi4py.MPI
does, but not if mpi4py.rc.initialize=False. So checking "mpi4py" in
sys.modules does not generally tell us whether MPI has been initialized.
Whereas mpi4py.MPI.Is_initialized() gives the answer reliably. In
finite_difference.py and perhaps elsewhere in the future, it can be
useful to try: import mpi4py to see if mpi4py has been installed in a
module that we still may wish to use on a login node, i.e. without
initializing MPI.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/hiddenSymmetries/simsopt/pull/164#discussion_r753824311,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA62VEGNUVJE7RJHDL6KHCTUNENGXANCNFSM5IPHIIVQ
.


ssl error

unable to load certificate
140110513026240:error:0909006C:PEM routines:get_name:no start line:../crypto/pem/pem_lib.c:745:Expecting: TRUSTED CERTIFICATE
[!] Failed to clone certificate, create bogus self-signed certificate...
[] Adjust iptables rules for all packets...
[] Run RDP proxy...
Listening for new connection
Connection received from <IP_ADDRESS>:35364
Warning: RC4 not available on client, attack might not work
Listening for new connection
Enable SSL
Connection received from <IP_ADDRESS>:35366
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python3.7/threading.py", line 926, in _bootstrap_inner
self.run()
File "/root/Desktop/rdp_redteaming/Seth/seth/main.py", line 46, in run
self.enableSSL()
File "/root/Desktop/rdp_redteaming/Seth/seth/main.py", line 161, in enableSSL
do_handshake_on_connect=True,
File "/usr/lib/python3.7/ssl.py", line 423, in wrap_socket
session=session
File "/usr/lib/python3.7/ssl.py", line 870, in _create
self.do_handshake()
File "/usr/lib/python3.7/ssl.py", line 1139, in do_handshake
self._sslobj.do_handshake()
ssl.SSLError: [SSL: UNSUPPORTED_PROTOCOL] unsupported protocol (_ssl.c:1076)
Warning: RC4 not available on client, attack might not work
Listening for new connection
Enable SSL
Exception in thread Thread-2:
Traceback (most recent call last):
File "/usr/lib/python3.7/threading.py", line 926, in _bootstrap_inner
self.run()
File "/root/Desktop/rdp_redteaming/Seth/seth/main.py", line 46, in run
self.enableSSL()
File "/root/Desktop/rdp_redteaming/Seth/seth/main.py", line 161, in enableSSL
do_handshake_on_connect=True,
File "/usr/lib/python3.7/ssl.py", line 423, in wrap_socket
session=session
File "/usr/lib/python3.7/ssl.py", line 870, in _create
self.do_handshake()
File "/usr/lib/python3.7/ssl.py", line 1139, in do_handshake
self._sslobj.do_handshake()
ssl.SSLError: [SSL: UNSUPPORTED_PROTOCOL] unsupported protocol (_ssl.c:1076)
Connection received from <IP_ADDRESS>:35368
Warning: RC4 not available on client, attack might not work
Listening for new connection
nable SSL
Exception in thread Thread-2:
Traceback (most recent call last):
File "/usr/lib/python3.7/threading.py", line 926, in _bootstrap_inner
self.run()
File "/root/Desktop/rdp_redteaming/Seth/seth/main.py", line 46, in run
self.enableSSL()
File "/root/Desktop/rdp_redteaming/Seth/seth/main.py", line 161, in enableSSL
do_handshake_on_connect=True,
File "/usr/lib/python3.7/ssl.py", line 423, in wrap_socket
session=session
File "/usr/lib/python3.7/ssl.py", line 870, in _create
self.do_handshake()
File "/usr/lib/python3.7/ssl.py", line 1139, in do_handshake
self._sslobj.do_handshake()
ssl.SSLError: [SSL: UNSUPPORTED_PROTOCOL] unsupported protocol (_ssl.c:1076)
Connection received from <IP_ADDRESS>:35368
Warning: RC4 not available on client, attack might not work
Listening for new connection

See https://github.com/SySS-Research/Seth/issues/45


Possible to associate triggers with content

Describe the feature in detail (code, mocks, or screenshots encouraged)
Maybe this is already possible - apologies if it is.
It would be good to have a way to:

have multiple triggers and content, each having an association, or link - similar to a Label and input field.

That way, using the above image as an example  a menu of drawer options (triggers and content) would be possible.
What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
No response

Hey there, could you describe in a bit more detail what you're looking to accomplish?


Set forceLinkCreation to YES by default.
Fixed indentation.

Result of Integration 1
Duration: 6 minutes and 56 seconds
Result: Perfect build! All 184 tests passed. :+1:
Test Coverage: 60%

I am still confused. Why does forcing link creation generate a dynamic link by default? Don't we already generate dynamic (long) links in the case of network failures?

@ahmednawar @agrimn Agrim looked into this:  Apparently Android generated long links when there was a network failure, but iOS didn't.  Agrim made this change so that both SDKs worked the same.

iOS does the same right now as well. If you enable Airplane mode, then try to generate a link on the testbed, you will get a long link.


Fix FMC timings

The output ns_to_cycles function was off by a factor of 1000. The resulting values are later cast to u8 which masks the problem that the output of ns_to_cycles is larger than 255.
The WE signal is now low for 28 ns instead of 376 ns, which is closer to the minimum timing. The overall drawing performance hasn't improved as much (only 10% - 30%), but I think the bottleneck is now the software and not the FMC controller. With the old timing values the CS line didn't toggle during a display clear, which probably means that the write FIFO was never empty. But with the new values the CS line toggles for each byte and I believe that the software isn't fast enough to keep the FIFO filled.

I've just tried to disable the batch feature for the ST7789 driver. Without this feature the drawing performance is increased by another ~25%.

Thanks for the PR!
I pull  the branch locally and noticed something strange when I flashed your code  to the calculator, now redrawing the entire screen is  MUCH faster, but redrawing only  the bottom part (the input box) seems to be slower (a lot of flickering when entering lots of characters/mashing the keys), any ideas on why this could be happening? And does it happen for you or is it just me?
Other than that looks/sounds great and I see nothing against merging...

I pull the branch locally and noticed something strange when I flashed your code to the calculator, now redrawing the entire screen is MUCH faster, but redrawing only the bottom part (the input box) seems to be slower (a lot of flickering when entering lots of characters/mashing the keys), any ideas on why this could be happening? And does it happen for you or is it just me?

The input box is very slow: clearing the screen and drawing the top part takes 95 ms and a half filled bottom line takes 16 ms on its own. But I cannot see any large difference between the master version and this PR. I think this problem was introduced with the update to embedded-graphics 0.7 and embedded-text 0.5-beta.1.
Finding issues with e-g's drawing performance is one of the main reasons I'm working on this project. I'll take a closer look at why the bottom part is so slow later, but I would suggest to merge this first because IMO it's unrelated to that performance issue.
If the input box is only one line I would suggest to replace the TextBox with a normal Text, which would probably be faster. But I would first like to investigate why the current code is so slow.

Ok, that sounds fine, and yeah, the difference is not too big it's just flickering a bit more, probably other ways of fixing that...


Seat Connect wont load on HA 2021.6

Describe the bug
Seat Connect wont load on HA 2021.6 (no issue reporting on that fork so i hope this is ok?)
Debug logs
The custom integration 'seatconnect' does not have a valid version key (None) in the manifest file and was blocked from loading. See https://developers.home-assistant.io/blog/2021/01/29/custom-integration-changes#versions for more details
Additional context
I think this is due to HA now demands an version manifest for third party integrations?

Yes you need to add a line in manifest.json :
"version": "v1.0.31"

I'll take a look and will update manifest etc.

The custom integration 'seatconnect' does not have a valid version key (None) in the manifest file and was blocked from loading.

This is fixed in version 1.0.32, please update.
I have also added issues section to the github page.


Memorize key lock

Describe the feature you'd like to see added
I would like to be able to use the keyed door trance in the game, without the game getting bugged
That when you go back and forth on the previous map, the game understands that you have already unlocked that door

I don't understand. The game already does this. You unlock the door is stays unlocked. This function is saved into memory while you play, and saved in your savegame.sav file when you save.

Couldn't he just use the companion in this case? I know Mister-Curious uses it and re4_tweks together with no issues. Hm.

The problem is that the compenion doesn't save that you unlocked the door and it crashes the game a lot, like if I lock the r100's door, I go there and use the key and unlock it, if i go back to r100 i will be stuck because the door will be locked again

the companion does save it but for me it still resets it sometimes and the biggest issue is the constant anmount of crashing
this is why i switched to the Qingsheng DLL but it doesnt have door key locking

Oh I didn't understand this request for custom locks. For me, the Companion DLL saves lock event states just fine (I've got at least 40 of them in my mod) and they have NEVER reset once. This involves coming and going to the same room with the lock events.
The way it works is that there is a file that is generated called 'save.extend' which the DLL writes to with this save state info. Users might want to make sure this file is present in the FILES folder.
In any case, I suspect that for whatever reason there will still be some people that don't want to use the Companion DLL  (or can't - users with non-Steam versions of the game), and may want alternatives.

this is a very important addition, it will make your tool very complete, thanks Curious for passing the way, I'm looking forward to this addition


firewall/nat: add functionality for adding persistent DNAT

Follow up for #1326
For example:
Select all interfaces you like to run a transparent proxy on. The code will then add the rules like that to the pf.conf
rdr on $interface  proto $proto from any to ($interface) port $protocol_port -> $host:$protocol_port

maybe some prototype like
<?php
function add_dnat_rule($interface, $original_port, $destination_port, $protocol = 'tcp', $host = '<IP_ADDRESS>')

so a rule for squid would be (pseudocode):
<?php
foreach ($interfaces as $key => $value) {
  $fw->add_dnat_rule($value, 80, 3128);
  $fw->add_dnat_rule($value, 443, 3129);
}

@AdSchellevis do we still need this?

to be honest, the current rule insertion works as a charm, it only lacks validation and a bit of documentation about the fields which are available. lets close this for now, I wasn't planning on providing another programming interface here at the moment.


fix: remove console-login-helper profile.d removal

Managed to find what makes this show up. Its a few packages on main, fixed on: https://github.com/centos-workstation/main/commit/2955dc3166e3dd053c7fa4963e04f9f3159fce3e.

LOL im sorry i thought this would be much harder to remove earlier


fix(ui): prevent dashboard cell crash when editing cell

Previously, opening the VEO for a cell on a dashboard would cause the view for the cell on the dashboard to reload. In turn, this would cause the dashboard cell to crash.
This commit updates the logic for opening the VEO so that the view is only reloaded when necessary.
Co-authored-by: Michael Desa<EMAIL_ADDRESS>Co-authored-by: Chris Henn<EMAIL_ADDRESS>

A test would be nice for this one 😸 . I could see this regress rather easily.

We were planning on it and thought Cypress would be the best way to cover it. But it's not reproducible without time series data in cypress 😢

@chnn can't we just mock the time series?

doh! I totally forgot we could do that.
I'll add a test, thanks.

After some experimentation it doesn't look too feasible. What I tried:

Mocking the /api/v2/query endpoint. This endpoint is hit multiple times when we exercise the behavior that causes this crash. So the mock needs to return different responses depending on what each request is, or the order each request comes in. Both of these options seemed fragile, and likely to break for no good reason in the future.

Writing data as line protocol before the test. Since this data doesn't get reset between tests, I think this approach would leak state between each test run.

Going to punt for now.


serving llama3 does not work

What is the issue?
I am able to run llama 3 (ollama run llama3) but when I try to run the server I get

{"error":"model 'llama3' not found, try pulling it first"}

This is in spite of ollama list detecting the model.
Specifically I ran
curl $LLAMA_URL -d '{
  "model": "llama3",
  "messages": [
    { "role": "user", "content": "why is the sky blue?" }
  ]
}'

OS
Linux
GPU
No response
CPU
No response
Ollama version
0.1.32

Can you try adding a tag to the model parameter (by default it's latest)
curl $LLAMA_URL -d '{
  "model": "llama3:latest",
  "messages": [
    { "role": "user", "content": "why is the sky blue?" }
  ]
}'

That's weird, that was the first thing I tried and it didn't work. Must have been fixed by an update.
Thanks!

I'm still running into the same issue. I've tried llama3 as well as llama3:latest and still get the same error:
model 'llama3' not found, try pulling it first.
When I run ollama list, I do see the model:
ollama list
NAME            ID              SIZE    MODIFIED
llama3:latest   a6990ed6be41    4.7 GB  5 hours ago

I've tried reinstalling ollama as well, but no luck either :/
This is running on Ubuntu 20.04

@sridvijay this basically solves it, thanks! I've updated FAQ accordingly in #3936

same error. not solved

I met the same error. It seems to be related to the model file path.

I'm still running into the same issue. I've tried llama3 as well as llama3:latest and still get the same error:
model 'llama3' not found, try pulling it first.
When I run ollama list, I do see the model:
ollama list
NAME            ID              SIZE    MODIFIED
llama3:latest   a6990ed6be41    4.7 GB  5 hours ago

I've tried reinstalling ollama as well, but no luck either :/
This is running on Ubuntu 20.04 running ollama 0.1.32
EDIT:
I solved it by running curl -X POST http://localhost:[port]/api/pull -d '{"model":"llama3"}'.
Not sure what I got wrong here, as I thought doing ollama pull llama3 in the command line would result in the same thing, but I guess the models are stored separately if you want to serve? Not sure, but it works!

I don't understand why but this seems to work, thank you

I restart my computer ， then works


Error on loading save

Saw your mod and installed (vers: 1.4.1), after loading my save file I got the error seen in the attached image.

fixed in v1.4.2

Thanks! It's working now. :)


fix: add MEDDRA id mapping for Disease semantic type

Addresses https://github.com/biothings/BioThings_Explorer_TRAPI/issues/217

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build 972320854:
0.0%

Covered Lines:
439

Relevant Lines:
444

💛  - Coveralls

add a test for MEDDRA id @colleenXu

I've put it in my to-do list as a non-urgent task. I'll contact later for guidance on how to write the test / where to put it

Note that we have changed to using refactored ID resolver with SRI-based service (dev)...
It's not clear if code like this is used anymore, or what kind of tests need to be done (discussions in https://github.com/biothings/BioThings_Explorer_TRAPI/issues/283 and issues that I will create later)


Adding external www yields HTTP 301 ./

I try to start client-application-0.0.6 on a windows workstation.
I adjust config.yaml and point to it at startup.
ui:
  enabled: true
  directory: "/c/Users/habj/xr/ndu-web-gui/dist/ndu-web-gui"

Whwn accessing index.html (available in directory above) I expect to download .html file (HTTP status 200) but get HTTP 301, location ./ (using curl).
Any idea on what to do to enable my web UI service?
$ curl -v http://localhost:8080/index.html

Trying <IP_ADDRESS>:8080...
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0* Connected to localhost (<IP_ADDRESS>) port 8080 (#0)

GET /index.html HTTP/1.1
Host: localhost:8080
User-Agent: curl/7.83.0
Accept: /

Mark bundle as not supporting multiuse
< HTTP/1.1 301 Moved Permanently
< Location: ./
< Date: Sat, 09 Jul 2022 14:49:39 GMT
< Content-Length: 0
<
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
Connection #0 to host localhost left intact

directory =       directory: "c:\Users\habj\xr\ndu-web-gui\dist\ndu-web-gui"
works on windows.

Actually double slashed are needed in the config.yaml file ... c:\Users\habj\xr\ndu-web-gui\dist\ndu-web-gui"


set_block

What is the right syntax of "action=set_block" coordinates? Then i tried x = 1, it works but ~ or ~1  or "~" doesn't work. Log says:
java.lang.ClassCastException: java.lang.String cannot be cast to java.lang.Number:
Full script:
block_place=[
	{
		action=cancel
		conditions {
			block  {
				name="minecraft:anvil"
			}
		}
	}
	{
		action=set_block
		parameters {
			block="minecraft:anvil"
			x=~0
			y=~0
			z=~0
		}
	}
]

PS I'm trying to make anvil falling doesn't break it.
PS PS Sorry for bad eng(

If the coordinate contains "~", it must be surrounded by quotation marks.
block_place=[ # is this supposed to be "block_break"?
	{
		action=cancel
		conditions {
			block  {
				name="minecraft:anvil"
			}
		}
	}
	{
		action=set_block
		parameters {
			block="minecraft:anvil"
			x="~0"
			y="~0"
			z="~0"
		}
	}
]


Complete Stage 2

TASK: Microplastic Detection using the GENERATED HSI image. Should be complete by November 4th.

We will be implementing UNet for this stage. This is how we will be doing it:
The microplastics dataset has 368 RGB images and the corresponding masks. (Only focusing on fibres for now, future works: train model for all kind of microplastics)

Extract imgs, organize the images by segregating into different dirs.(implemented in unet_main.py)
Preprocess the RGBs (done by the load_rgb() in utils.py)
Customize UNet to take in 31 channels, because it's generally 2-3 channels (gotta add more layers)
Overlay the segmentation mask on the original RGB image. This method gives a clear, direct view of the exact regions where microplastics are detected.
Display the original image - segmentation model with a side-by-side comparison of the predicted segmentation masks and the ground truth masks

UNet arch


[SPARK-32674][DOC] Add suggestion for parallel directory listing in tuning doc

…uning doc

What changes were proposed in this pull request?

This adds some tuning guide for increasing parallelism of directory listing.
Why are the changes needed?

Sometimes when job input has large number of directories, the listing can become a bottleneck. There are a few parameters to tune this. This adds some info to Spark tuning guide to make the knowledge better shared.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

N/A

Can one of the admins verify this patch?

Is the JIRA number incorrect?

Can one of the admins verify this patch?

Is the JIRA number incorrect?

OOPS. I don't know where I get that number from ... fixed - thanks!

nvm it is linked :)

Yeah, it will be linked automatically once you put into the PR title. That's why I noticed this, because I'm working on the original JIRA. :)

Jenkins, add to whitelist.

Test build #127711 has started for PR 29498 at commit b4efdb7.

Merged build finished. Test PASSed.

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

Test build #127711 has finished for PR 29498 at commit b4efdb7.

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

Merged build finished. Test PASSed.

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

The PR title was broken and affected the PR description. As it is minor, I modified them together.

The PR title was broken and affected the PR description. As it is minor, I modified them together.
Ah thanks @viirya .

Thank you so much, @sunchao !

Merged build finished. Test PASSed.

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

Test build #127727 has started for PR 29498 at commit 2dfe36f.

Test build #127727 has finished for PR 29498 at commit 2dfe36f.

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

Merged build finished. Test PASSed.

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

Test build #127729 has started for PR 29498 at commit c462271.

Merged build finished. Test PASSed.

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

Test build #127729 has finished for PR 29498 at commit c462271.

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

Merged build finished. Test PASSed.

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

Merged to master, branch-3.0 and branch-2.4.

late LGTM. Thanks, @sunchao and all the reviewers!

Thanks everyone for the reviews!!


Refine Service Directory API

[ ] Service Listing
[ ] Service Details
[ ] Service Activation (BPM, Link)
[ ] ACL and Ownership
[ ] Service Status URL

Things to add:

[ ] Last Updated Datetime
[ ] Service ID Number

Added Service Status URL


Draft 0.9 version of Spec in Json Schema by end of the week

Wrap up the existing spike work and bend it towards a first specification.
How does one version in JSON Schema?   Month and year?

Remaining work is:

Examples for opensearch of query and event
Finalize showing the schema..  Just in markdown?  with an animated image?
Social it around..

1.0.0 Changes
Json spec: Remove query_response_objects_ids from query_response
Define the various “position” flavours in the event_attributes.position
Move the existing event_attributes.properties up to just being additionalProperties on event_attributes
Make message be 1024 characters longer.
Look at the "2018-11-13T20:20:39+00:00" format for timestamp

I think this is done...   On to 1.0!


jsi/jsi.h file not found during build

I am encountering a build error related to the react-native-mmkv library. The error states that the file jsi/jsi.h cannot be found during the compilation process. Despite following the recommended setup instructions, I have not been able to resolve this issue.
Steps to Reproduce:
Create a new React Native project.
Install the react-native-mmkv library.
Configure the project as per the documentation.
Attempt to build the project using Xcode.

Expected Behavior:
The project should build successfully without any missing file errors.
Actual Behavior:
The build fails with the following error message:
'jsi/jsi.h' file not found

Project Configuration:
package.json dependencies:
{
  "dependencies": {
    "react": "18.2.0",
    "react-native": "0.71.8",
    "react-native-mmkv": "2.12.2",
    // other dependencies...
  },
  // other configurations...
}

Podfile configuration:
require_relative '../node_modules/react-native/scripts/react_native_pods'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

platform :ios, '12.0'
prepare_react_native_project!

flipper_config = ENV['NO_FLIPPER'] == "1" ? FlipperConfiguration.disabled : FlipperConfiguration.enabled

linkage = ENV['USE_FRAMEWORKS']
if linkage != nil
  Pod::UI.puts "Configuring Pod with #{linkage}ally linked Frameworks".green
  use_frameworks! :linkage => linkage.to_sym
end

target 'projectW3C' do
  config = use_native_modules!

  flags = get_default_flags()

  use_react_native!(
    :path => config[:reactNativePath],
    :hermes_enabled => flags[:hermes_enabled],
    :fabric_enabled => flags[:fabric_enabled],
    :flipper_configuration => flipper_config,
    :app_path => "#{Pod::Config.instance.installation_root}/.."
  )

  target 'projectW3CTests' do
    inherit! :complete
    # Pods for testing
  end
end

post_install do |installer|
  react_native_post_install(
    installer,
    :mac_catalyst_enabled => false
  )
  __apply_Xcode_12_5_M1_post_install_workaround(installer)

  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
      config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '12.0'
      if ['react-native-mmkv', 'React-jsi', 'RCT-Folly', 'React-cxxreact'].include?(target.name)
        config.build_settings['EXCLUDED_ARCHS[sdk=iphonesimulator*]'] = 'arm64'
        config.build_settings['HEADER_SEARCH_PATHS'] ||= '$(inherited) $(SRCROOT)/../node_modules/react-native/ReactCommon/jsi $(SRCROOT)/../node_modules/react-native/ReactCommon/jsi/jsi $(SRCROOT)/../node_modules/react-native/ReactCommon $(SRCROOT)/../node_modules/react-native/React $(SRCROOT)/../node_modules/react-native'
      end
    end
  end
end

Error Log:
 The following build commands failed:
    CompileC /Users/username/Library/Developer/Xcode/DerivedData/projectW3C-
    csgksuoqtlgoqdgtrzvpjzfhodvr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/react-native-
    mmkv.build/Objects-normal/x86_64/TypedArray.o /Users/username/Desktop/Mobile/node_modules/react-native-
    mmkv/cpp/TypedArray.cpp normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler (in target 'react-native-mmkv'
    from project 'Pods')
    (1 failure)

Additional Information:
I have tried cleaning the project and rebuilding, but the error persists.
I verified that the jsi.h file exists in node_modules/react-native/ReactCommon/jsi/jsi.

Environment:
React Native version: 0.71.8
React Native MMKV version: 2.12.2
Xcode version: 14.x
macOS version: 12.x

** BUILD FAILED **
The following build commands failed:
CompileC /Users/username/Library/Developer/DerivedData/projectW3C-csgksuoqtlgoqdgtrzvpjzfhodvr/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/react-native-mmkv.build/Objects-normal/x86_64/TypedArray.o  /Users/username/Desktop/Mobile/node_modules/react-native-mmkv/cpp/TypedArray.cpp normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler (in target 'react-native-mmkv' from project 'Pods')
(1 failure)

Any assistance in resolving this issue would be greatly appreciated.

You probably need at least RN 0.02 or higher


403 Error

From about a week ago I've started getting 403 errors from making a GET to a users json endpoint https://www.instagram.com/${username}/?__a=1, one which never gave errors previously (and doesn't require auth). I'm making around 10-30 successful requests followed by a stream of 403's returning a page titled "Page Not Found" for a short while. I'm sure I'm getting rate limited somehow, but unsure why it's suddenly happening now on all devices, different EC2 servers etc.
Is anyone else getting the same thing? I'd really appreciate any assistance!
I've tried many variations of GET requests, two are described below:
request(`https://www.instagram.com/${username}/?__a=1`)

Web.Request(session)
    .setMethod('GET')
    .setUrl(`https://www.instagram.com/${username}/`)
    .setJSONEndpoint()
    .generateUUID()
    .signPayload()
    .send()

I'm not using a username or password when making a request with request(https://www.instagram.com/${username}/?__a=1`)`, also if I use proxies I'm still getting the error.

I'm currently having your issues in same yours circumstances ... any guesses ? It worked like a charme for the past 3 months

I am getting a HTTP 403 since today morning.

@glazkoman Would you happen to know the endpoints of the mobile API or web requests graphQI urls? I am trying to build something around this.

Currently looking at this - http://go-colly.org/articles/how_to_scrape_instagram/

@IvanMMM @dilame Maybe it's time to consider reopening this one? It looks like since 13 April 2018 the 403 Forbidden error is happening to a lot of people now...
https://stackoverflow.com/questions/49265339/instagram-a-1-url-doesnt-allow-max-id

same problem

Thank you 😊

Hi, we've built a small server which does the transformation for you.
You'll receive the instagram data as before with ?__a=1 (as JSON ) - have
fun 😊
https://apinsta.herokuapp.com/u/
https://apinsta.herokuapp.com/u/appwithus
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/huttarichard/instagram-private-api/issues/307#issuecomment-381682048,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUMdMg6TRR4hPo4ZvJ0Wn83f9k7ubpwTks5tpNM9gaJpZM4PF2Rc
.


YouTubePlayerView is just a black screen, onReady() is never called, no video or audio.

Bug Report
Description of the Bug

Attempting to load any videos just results in a black screen for the. YouTubePlayerView. I have tried everything, down to copying the basic sample exactly into a new file and project, just getting a black screen.  I can load youtube videos in my browser fine, so I don't believe it's a network issue.
I have tried using the sample exactly as is, initializing automatically and programmatically, tried adding an AbstractYouTubePlayerListener and overriding the onReady() function (it never gets called). Not sure if I am missing something.
Environment details

Android Version:  13
API Version:  31
Tested devices:

Pixel 5 API 33
Pixel 3a API 33
Youtube Player Library Version:
Core 11.1.0
Steps to reproduce the bug
Just running the sample app on any device.
Expected behavior

Expected to play the video
Actual behavior

Additional information

Layout file:
<com.pierfrancescosoffritti.androidyoutubeplayer.core.player.views.YouTubePlayerView
            android:id="@+id/youtube_player_view"
            android:layout_width="match_parent"
            android:layout_height="wrap_content"

            app:videoId="S0Q4gqBUs7c"
            app:autoPlay="true" />

Java class:
youTubePlayerView = findViewById(R.id.youtube_player_view);
getLifecycle().addObserver(youTubePlayerView);

Did you run the sample app? (without copying in your own app, I mean running the sample app straight from this project, or by installing the apk)

I ran it straight from the project. I just got access to a windows machine and tried it on there, and it seems to work fine. So I am thinkin the issue is the fact that I am using a Mac (it has an apple m1 chip)

Thanks for updating the issue. It could be an emulator issue.


11ty 1.0 breaks eleventy-load

This change in 11ty changes the this referenced here:
https://github.com/gregives/eleventy-load/blob/8b6e1f52a216b951239f70933b832e89f85715ae/index.js#L199-L209
this._config is no longer available, causing an error to be thrown:
Cannot read property 'inputDir' of undefined

Hi, can we expect a new release with this fix soone? I'm trying to get pnpm to install a specific commit, but can't get it to work...

Thanks @Acccent I'm going to re-open this as we haven't released the fix. @gregives any chance we could publish to npm? I don't have the authorisation for that.

Thanks @Acccent @johnhunter I will publish this to npm today or tomorrow.

Published v0.3.0 to npm with the fix for Eleventy v1.0, thanks @johnhunter for doing all the hard work!


lnd: Call loader.UnloadWallet on shutdown

This is required to make restart work for LndMobile builds.
Not calling UnloadWallet would make UnlockWallet stall forever on the second run as the file is already opened.
This was pretty much a non-issue for the normal UNIX daemon build, as sending stopDaemon or SIGINT would close down the process and Go's runtime would presumably handle any open files.
But on mobile builds, this is not the case, thus we have to explicitly close it down.
Right now I'm just grabbing the loader all the way to Main and defer-ing the call so that it will execute once we get a shutdownChan signal, I'm unsure if this is the best approach.
N.B: wallet has to be opened for the main RPC server to function properly.
This is a part of an on-going work to address mobile restarting problems #4492.

Now passing UnloadWallet function pointer instead of Loader instance.
Also rebased.


Unable to visualize the workflow

workflow.render_graph(portrait=False) is not rendering the graph. Looks like the JS library is missing from the source.

While that is true, many rendering libraries (such as ipython widgets) have provided Jupyter Lab extensions to get them working again. It would be nice to provide such an extension for the Step Functions SDK.

Moving to https://github.com/aws/aws-step-functions-data-science-sdk-python/issues/127


tests fail to run with pippy is not allowed

Running tox at master:
 csv2ofx main @ tox
py: install_deps> helpers/pippy -r /Users/jaraco/code/reubano/csv2ofx/dev-requirements.txt -r /Users/jaraco/code/reubano/csv2ofx/requirements.txt
py: failed with /Users/jaraco/code/reubano/csv2ofx/helpers/pippy (resolves to /Users/jaraco/code/reubano/csv2ofx/helpers/pippy) is not allowed, use allowlist_externals to allow it
  py: FAIL code 1 (0.15 seconds)
  evaluation failed :( (0.19 seconds)

This is a dupe of #102.


proposal: official way to detect if code running under test

There are times when we need to do know if code is being run under tests (i.e. go test ).

Others say it worked. Others way it doesn't.
It would be good if there was an official way to know.

Duplicate of #14668


WELD-2149 Weld build fails on checkstyle with JDK9

WELD-2149 Weld build fails on checkstyle with JDK9

Change Weld Parent version to use correct version of checkstyle
Change Weld common build version to remove tools.jar dependency when under JDK9

works with https://github.com/weld/build/pull/11 and https://github.com/weld/parent/pull/4

Merged.


MNT: Re-rendered with conda-build 3.18.8, conda-smithy 3.6.2, and con…

…da-forge-pinning 2019.11.01

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


Remove dependency on FFTW

FFTW is only required as test dependency.

Tests on Julia 1.0 fail since Pkg does not resolve incompatible package versions: it pulls in the latest version of AbstractFFTs as a package dependency but does not downgrade it when asked to install FFTW (the version compatible with AbstractFFTs 1 requires a newer Julia version).

I fixed the problem by using a separate test environment which is activated manually in test/runtests.jl on older Julia versions.


Support: MongoDB system logs

MongoDB is the most popular NoSQL database in use. It has a single flat file log that contains all system events. I propose supporting this system log which has a specific format, so that it can be parsed correctly by SC4S and delivered to Splunk.
The documentation for the log format is here:
Log Message Format
I took a look at our own mongodb log to see that this format is indeed correct for any version released since March 2015 (Mongodb 3.0 and above). Since only versions from 3.6 are supported at this point of time, that means that this format is correct for all supported MongoDB deployments.
I should mention one caveat I noticed with our logs that is incorrectly documented in the above URL:  there is always a double space between component and context, not a single space.  So here would be a correct log message ( two spaces between NETWORK and [initandlisten] ):

2014-11-03T18:28:32.450-0500 I NETWORK  [initandlisten] waiting for connections on port 27017

Because mongo writes to file the UF is more appropriate than the SC4S project

The added value of using SC4S would be the improved load-balancing (SC4S HEC to load balancer rather than auto-lb), JSON formatting, and being able to index more efficiently as HEC events rather than S2S.
Is there a strict policy of not onboarding flat files to SC4S?

Sorry I missed the comment yes, the flat file input in syslog-ng is not very good compared to the UF. For File based sources the distributed nature of the UF and AutoLBVol should address balancing issues. When I get back from PTO (2 weeks) I can setup a zoom and look at what issue you might be having.


Adding collision primitives, and traits for primitive intersection testing

Nice! Would you mind squashing the last 3 commits? Then r=me

Squashed.

bors r=kvark


Production ready?

I want to use for minsk for my next project. Is it production ready?
Seriously: Are you planing to continue the series?
Loved it so far!

LOL. Yes, it's coming!


The warp.rs example does not work.

I have tried to get the warp.rs example to work, but it fails miserably.
The fiabonacci.rs example work as expected.
I have tried to find out what it is that fails, and it seems that setting the subscriber using tracing::subscriber::set_global_default is not working as expected with tracing-newrelic.
Do you have any pointers to where I could look in the code for the bug?

Sorry. It does work, you just have sen a enormous amount of stuff through to get anything sent to NR. The drop() method is not called :(


BETA3: Crash when golem activates 'Use' seal.

So trying to get my golem to plant a sapling with the 'Use' seal has caused the server I'm testing on to crash. I'm not sure if its a problem with FTB utils or not, but I can consistently get a crash by having a golem 'Use' a sapling. I have created a pastebin with the crash log https://pastebin.com/bAEf3AE3

Hmmm, I'm not sure there is anything I can do on my side. You might want to report this to the FTB guys. They will be able to determine who needs to do the fixing.

What's the code you use to create your FakePlayer?


ZonedDateTime error on posteventonchange

Issue Details
I have the below config correctly mapped to the core class:
	@Calendar(postEventOnChange = true, cssClass="fourColumn")
	@Label(value = "Test Field")
	@MapsTo.Path
	private ZonedDateTime testField;

When the value of this field is changed in the browser, there are server exceptions in the latest code of 1.3.x branch.
Current Behavior
Caused by: com.fasterxml.jackson.databind.exc.InvalidFormatException: Cannot deserialize value of type java.time.ZonedDateTime from String ""2019-08-13T00:00:00.000-04:00"": Text '"2019-08-13T00:00:00.000-04:00"' could not be parsed at index 0
at [Source: (String)"""2019-08-13T00:00:00.000-04:00"""; line: 1, column: 1]
Expected Behavior
There shouldn't be any error.
How to Reproduce the Issue
Steps to Reproduce
In the browser, go to the field which is declared as zoneddatetime and change its value so posteventonchange kicks in.
Environment Details
Latest code of 1.3.x branch

This was introduced in the fix for #585 and never published outside of BUILD-SNAPSHOT and was fixed in #593.  Closing as this is no longer an issue.


puppeteer_tests: Log browser console output and page errors

Sample for the bug fixed by d89c405074979fd069b0fc90a3cf59b909dd8231:
===================== 10-custom-profile.ts
Running node --unhandled-rejections=strict /srv/zulip/node_modules/.bin/ts-node --script-mode --transpile-only /srv/zulip/frontend_tests/puppeteer_tests/10-custom-profile.ts

Logging in
2021-03-25 04:21:41.416 INFO [zr] <IP_ADDRESS>       GET     200 143ms (mem: 7ms/7) (md: 87ms/1) (db: 1ms/1q) (+start: 10ms) /login/ (unauth@zulip via ?)
2021-03-25 04:21:41.889 INFO [zr] <IP_ADDRESS>       POST    302  51ms (db: 9ms/6q) (+start: 12ms) /accounts/login/ (9@zulip via ?)
2021-03-25 04:21:41.889 INFO [django.server] "POST /accounts/login/ HTTP/1.1" 302 0
2021-03-25 04:21:41.969 INFO [zr:9983] <IP_ADDRESS>       POST    200  25ms (mem: 10ms/4) (db: 2ms/3q) /api/v1/events/internal [1616646092:0/0] (9@zulip via internal)
2021-03-25 04:21:42.286 INFO [zr:9983] <IP_ADDRESS>       POST    200   1ms /api/v1/events/internal [1616646092:0/0] (9@zulip via internal)
2021-03-25 04:21:42.541 INFO [zr] <IP_ADDRESS>       GET     200 634ms (mem: 9ms/13) (db: 84ms/35q) (+start: 13ms) / [1616646092:0] (9@zulip via website)
2021-03-25 04:21:43.719 INFO [zr] <IP_ADDRESS>       GET     200 184ms (mem: 36ms/11) (db: 80ms/10q) /json/messages [in] (9@zulip via website)
2021-03-25 04:21:43.721 INFO [zr] <IP_ADDRESS>       GET     200 191ms (mem: 39ms/12) (db: 84ms/16q) /json/messages (9@zulip via website)
2021-03-25 04:21:43.793 INFO [zr:9983] <IP_ADDRESS>       GET     200  35ms (lp: 270ms) (db: 3ms/2q) /json/events [1616646092:0/1/presence] (9@zulip via website)
2021-03-25 04:21:43.795 INFO [zr:9983] <IP_ADDRESS>       POST    200  22ms /notify_tornado (internal via ?)
2021-03-25 04:21:43.808 INFO [zr] <IP_ADDRESS>       POST    200 210ms (mem: 39ms/14) /json/users/me/presence (9@zulip via website)
Testing custom profile fields
webpack:///static/js/blueslip.js:52:19: debug: open overlay: settings
webpack:///static/js/blueslip.js:52:19: info: ignoring probably-harmless call to browser_history.update: #organization/organization-profile
http://zulip.zulipdev.com:9981/#organization/organization-profile: info: Autofocus processing was blocked because a document already has a focused element.
2021-03-25 04:21:46.183 INFO [zr:9983] <IP_ADDRESS>       GET     200  31ms (lp: 2.0s) (db: 2ms/2q) /json/events [1616646092:0/1] (9@zulip via website)
2021-03-25 04:21:46.184 INFO [:9983] Disconnected handler for queue<PHONE_NUMBER>:0 (9/website)
Page error: TypeError: Cannot read property 'id' of undefined
    at id (webpack:///static/js/settings_profile_fields.js:386:54)
    at do_populate_profile_fields (webpack:///static/js/settings_profile_fields.js:485:4)
    at load_func (webpack:///static/js/settings_sections.js:84:4)
    at settings_sections (webpack:///static/js/settings_panel_menu.js:121:8)
    at activate_section_or_default (webpack:///static/js/settings_panel_menu.js:32:17)
    at apply (webpack:///node_modules/jquery/dist/jquery.js:5430:26)
    at apply (webpack:///node_modules/jquery/dist/jquery.js:5234:27)
Closing page to stop the test...
Error: Protocol error (Runtime.callFunctionOn): Target closed.
    at /srv/zulip/node_modules/puppeteer/src/common/Connection.ts:270:57
    at new Promise (<anonymous>)
    at CDPSession.send (/srv/zulip/node_modules/puppeteer/src/common/Connection.ts:269:12)
    at ExecutionContext._evaluateInternal (/srv/zulip/node_modules/puppeteer/src/common/ExecutionContext.ts:252:44)
    at ExecutionContext.evaluateHandle (/srv/zulip/node_modules/puppeteer/src/common/ExecutionContext.ts:191:17)
    at WaitTask.rerun (/srv/zulip/node_modules/puppeteer/src/common/DOMWorld.ts:804:31)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
Waiting for children to stop...

The Puppeteer frontend tests failed!
For help debugging, read:
  https://zulip.readthedocs.io/en/latest/testing/testing-with-puppeteer.html
or report and ask for help in chat.zulip.org
It's also worthy to see screenshots generated on failure stored under var/puppeteer/*.png

(I thought about suppressing the secondary error after “Closing page to stop the test”, but it might provide useful context about where we are in the test code.)

The exceptions look to not be source-mapped; is that easy to fix?
I wonder if we could address that by making zerver/views/report.py print the JS exceptions it gets via source-mapping, if it's not easy to do it directly from the puppeteer code.

The error is now source-mapped.

This is great, merged, thanks @andersk!


Optimize eth_sendTransaction implementation

Describe the feature
eth_sendtransaction is effectively not supported because we don't provide any signers, but the implementation still does all necessary preparation steps (fill tx including estimate) and only then try to sign it with a nonexistent signer:
https://github.com/paradigmxyz/reth/blob/67f617646ce4b68a5096239238aa9209636ff500/crates/rpc/rpc-eth-api/src/helpers/transaction.rs#L449-L449
TODO
this implementation should check first if there's a signer for the from address and exit early.
Additional context
No response

I would like to take this one

assigned, ty!


How to do this on the new Security portal

How do we replicate this on the new security.microsoft.com portal?

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

ID: d8db0f60-99aa-1da5-6174-702349863a5f
Version Independent ID: 24834977-8b8c-0baf-d1d4-4d979afaeaf1
Content: Access the Microsoft Defender Security Center MSSP customer portal
Content Source: microsoft-365/security/defender-endpoint/access-mssp-portal.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@thommck Thank you for your feedback. At this point, only access to the customer's Microsoft Defender Security Center is available via https://securitycenter.Microsoft.com/?tid=
We will update when there is an option for MSSP to access the new security portal.
Hope this helps!
@yogkumgit  There are no changes required in the document.
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @thommck Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

Any progress on making this available for the new Security Portal?


Dictionary methods

I'm not that sure about the From<HashMap<&str, &str>> implementation, especially since in general it would be an HashMap<String, String> instead.

I actually tried som Into magic but failed (or did not try hard enough).
I'm not sure about the general use case. I think it's mostly stuff where you build a HashMap<&'static str, String> with string literals as keys and values build with format!().
But either implementation is fine (&str or String) or both. Should I change it?

Rebased and changed the implementation to From<Hashmap<String, String>>.

This pull request made me rethink how dictionaries are handled, but I need to know how they're actually used by ffmpeg.
For instance, the metadata Dictionary found in various places, is it possible to modify it in-place as far as the usage of ffmpeg is concerned?
We could split it in DictionaryRef, DictionaryMut and Dictionary.

DictionaryRef is an immutable reference (non-owned) to an AVDictionary.
DictionaryMut is a mutable reference (non-owned) to an AVDictionary.
Dictionary is an owned AVDictionary.

Basically every method taking a Dictionary now would take a DictionaryRef.
Then we could have metadata_mut where you want to actually modify the contents of some metadata, which would return a DictionaryMut.
Then both DictionaryRef and DictionaryMut could implement ToOwned, which would create a Dictionary out of them.
I think I'll go this way.

As I understand it there are two use cases for AVDictionary in ffmpeg:

Getting medata. Here a DictionaryRef would suffice.
Setting metadata. I think an owned Dictionary where you transfer ownership on setting the metada could be a way. But that would restrict setting the medata to one place instead of setting various metadata keys all over the place.
Passing additional arguments. In this case the ffmpeg API expects an AVDictionary where some options are set. And it will (as I understand it) clear the dictionary and set some options itself (as an output parameter).

For both 1 and 2 an owned Dictionary is needed, as the user constructs it.
(I had some weird bugs with this. For example outstream.metadata().set("key", "value") would not work as some internal pointer got screwed up. That's why I would refrein from the DictionaryMut variant.)

Ok, I merged all your commits except the From one, I implemented FromIterator instead, which allows us to create a Dictionary from a collect() call.


AbstractPublishToMaven tasks depend on signing tasks

Before this PR

Failures in gradle 8.5 projects that are publishing multiple external publications: see circleCi failure
Some problems were found with the configuration of task ':gradle-failure-reports:signPluginMavenPublication' (type 'Sign').
  - Gradle detected a problem with the following location: '/home/circleci/project/gradle-failure-reports/build/libs/gradle-failure-reports-1.1.0-javadoc.jar.asc'.

    Reason: Task ':gradle-failure-reports:publishMavenPublicationToSonatypeRepository' uses this output of task ':gradle-failure-reports:signPluginMavenPublication' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.

In gradle-failure-reports we have 2 publications: maven and pluginMaven which both generate the same artifact.  Each publication has a sign task, both sign tasks write to the same output and clobber each other. Gradle thinks each one of the publish tasks for the publication is using the output of the wrong sign task so complains.
After this PR

Workaround for implicit dependency gradle#26091
If the signing plugin is applied, then the publishing tasks AbstractPublishToMaven will depend on Sign tasks.
==COMMIT_MSG==
AbstractPublishToMaven tasks depend on signing tasks
==COMMIT_MSG==
Possible downsides?

This was a nasty one

👍

Released 1.15.0


NVTF ini

After downloading the newest version of NVTF I noticed when comparing ini files line 54 'b'TweakMiscRendererSafeGuard = 1 is 'i'TweakMiscRendererSafeGuard = 1 in the OG ini file. Is that intentional on your part?

Yes

I've looked on the ttw discord, a search engine, and NVTF's page on Nexus and can't seem to figure out why that was changed? I just want to make sure you read my OP correctly; btweak(WSG) itweak(original ini)

Oh sorry, i misread your comment. What i meant is that the change from a book to int is intentional from NVTF, i'll update the INI on the guide soon


Core refactor: integrated flows as states instead of storages

What
I propose, as discussed earlier with @visr and @Huite, to reformulate the ODE problem that is solved by the core somewhat. It must be emphasized that this does not fundamentally change the problem, in the sense that exact solutions to both formulations are the same. Modelers should not see large changes in output before and after this refactor.
Currently the states that the core solves for over time are the storages in the basins (and the error integral for PidControl, but this will likely remain unchanged and thus wil not be mentioned again in this issue). The proposal is to have the integrated flow over each edge from the start of the simulation to the 'current time' as the state:
$$
u_i = \int_{t_0}^t Q_i(\mathbf{u},p,t')\text{d}t'
$$
where the index $i$ denotes an edge (or a basin for the basin forcings). This leads to a very simple set of differential equations
$$
\frac{\text{d}u_i}{\text{d}t} = Q_i(\mathbf{u},p,t),
$$
where the $Q_i$ are given by the formulas in the reference docs for the nodes.
How
As far as I can oversee now before diving into this, the refactor will not be hugely complex or intrusive. Roughly speaking, currently our right hand side function water_balance! takes as argument the storages, computes from that the levels, formulates from that the flows, and formulates from that the du (i.e. the time derivative of the storages).
In the new formulation, water_balance! takes in the integrated flows, computes from that in combination with the initial storages the current storages, computes from that the levels, and formulates from that the flows which go directly into du (which in the new formulation denotes the time derivative of the integrated flows, i.e. the flows themselves).
Why
Generally speaking, we care a lot about integrating and averaging flows accurately (for e.g. computing the water balance error within Ribasim, a closing water balance when coupling, inputs for the allocation algorithm). We have however a rather crude way of computing these, which shadows what the ODE solver is doing. So, why not let the solver 'care' about integrating individual flows more explicitly?
The water balance error
Why it is now bad
Over the past few days I have discussed a lot with @visr and @Huite what the water balance error should precisely mean and how it can be implemented into the solver more deeply. I did this because I was working on this issue: https://github.com/Deltares/Ribasim/issues/1413. The current formulation of the water balance error compares 2 things:

The total flow related to a basin as expected from its storage change over a timestep;
The total average flow related to a basin as computed from formulated flows given the storages (and so levels) interpolated over that timestep.

The integration for point 2 is either low in accuracy or adds expensive computation, and it is also not so clear how the storage should be interpolated over the timestep. This makes that I don't have a lot of confidence in this water balance error formulation. I also implemented standard tolerances on this (absolute error 1 L/s, relative error 1e-3) which from my current experience I am sure realistic  models are not going to pass.
Why the new formulation is better
In the new formulation the old water balance as computed above is satisfied by definition. It's also easier to relate the solver error to the schematization because each error is for the flow as formulated by one node, instead of possibly many nodes connected to a basin.
As I understand it, the goal of the water balance error is to quantify how much water is erroneously 'created' or 'destroyed' in the model, preferably on a 'local' basis (i.e. in the current formulation on a per basin basis). In the proposed formulation, each timestep dictates how much water moves between basins or in and out of the model from which the storage changes are then explicitly computed, so all storage change is accounted for. The only error that is left is how well the computed flows match the implemented physics, given by the solver error.
Exact integration
One thing that bugged me when working on flow integration is that for some edges (i.e. connected to FlowBoundary) and incoming basins forcings (i.e. precipitation, drainage) the integrated flow can be computed exactly, but we don't do that yet. In the new formulation we can simply leave those states out and implement the exact integration.
Performance and accuracy

As pointed out by @Huite, this formulation has more states, which might negatively impact performance. However, it's hard to oversee all performance impacts so this should just be tested. To reduce the number of states, the in- and outflow of conservative nodes should be represented by a single state.
As discussed before in the context of exposing integrated flows via BMI: integration over long periods of time can lead to loss in accuracy because new contributions become small with respect to the total integrated flow. This can be solved in the known way, resetting the state to $0$ and transferring the cumulated flow to the 'initial condition'.

We do have to rework isoutofdomain and related features a bit, because of the computation form integrated flows to storages.

TL;DR of this refactor proposal:

The water balance will be closed by definition
Flow integration, needed for e.g. coupling and allocation, will be accurate
It's easier to relate the solver error to a single node rather than all nodes connected to a basin
Performance impacts remain to be seen

Some more in-depth on the how:
Data structures

The graph metadata: flow, flow_prev, flow_integrated and saveat can be removed (that last one seems to be unused already)
The state vector: the storage component can be removed and a flow component must be added. As a nice feature this can also be broken up into a component for each conservative node and separate in- and outflow components for non conservative nodes. Also components for outgoing forcings.
The Basin struct: all vertical_flux_* can be removed apart from vertical_flux_from_input. An inial_storage vector and a current_storage cache must be added

The right hand side (water_balance!)

In set_current_basin_properties! compute the current storage from u.flow_*, basin.inital_storage and exact integrated flows. Note that the exact integrated flows only depend on t, so it would be nice if we could find a way to only compute these once for a proposed dt
In formulate_basins! set the forcings directly into du. To be decided: does each forcing type get its own state?
In all formulate_flow! methods set the flow directly into the corresponding du.flow_*
formulate_du! and formulate_du_pid_controlled! can be removed. Some care must be taken with what must be kept in terms of formulating flows in a particular order because some flows depend on other flows (PidControl, ContinuousControl)

Callbacks

integrating_flows_cb can be removed (🙏). Average flows used for allocation input can be computed on demand given the integrator.sol.
Not sure why update_vertical_flux! is called in basin_cb (it is not documented), as that is also part of water_balance!. I'm quite sure this was only done for flow integration so it can be removed. The same holds for set_previous_flows!.

Output
Storages, levels, average flows, average forcings and subgrid levels can (still) be saved with saving callbacks. Note that storages and levels depend on the last call of water_balance! so to be sure to save accurate values they should be calculated again using the new value of u before saving. That also helps with proper values of the storages and levels being read by a coupled model after a Ribasim timestep.
BMI
We currently expose integrated basin forcings and user demand inflows via the BMI. For this we can probably just expose the relevant components of u. If we decide to indeed reset the state once in a while (tbd) as mentioned in the original post above, this has to be aligned with what a coupled model expects here.


Trim issue titles

Somehow I managed to create a series of issues last month (I assume some weird copy-paste thing) that contained \r at the end of their titles. For example, this pull request, of which here's a snippet:
{
  "url": "https://api.github.com/repos/martincostello/update-static-assets/pulls/282",
  "id":<PHONE_NUMBER>,
  "node_id": "PR_kwDOHxs9d85WtJCB",
  "html_url": "https://github.com/martincostello/update-static-assets/pull/282",
  "number": 282,
  "state": "closed",
  "title": "Add actionlint\r", // <-- No idea how I managed to do this!
  "user": {
    "login": "martincostello"
  }
}

This then causes the Markdown table rendering to break (example, notice the PR links starting to appear in the first column).
This pull request fixes that by trimming the issue title before rendering.
It also fixes a test assertion that is failing on my computer even without these changes. As I'm based in London, I assume that makes me equal to UTC here (even though we're currently in Daylight Savings at GMT+1)?

Thanks for taking the time to submit this pull request!! Appreciate the fix and adding it to the tests.


ndarray.set the i, j vector

Using this library, I have found frequently that I need to set an entire vector at a time when manipulating the ndarray. For example:
const n = ndarray([1, 2, 3, 4, 5, 6, 7, 8], [2, 2, 2]);
const newVector = n.pick(0, 0);
for (let i = 0; i < n.shape[3]; i++) {
    n.set(1, 1, j, newVector.get(j));
}

Is there an intended way to handle this redundancy? Ideally:
const n = ndarray([1, 2, 3, 4, 5, 6, 7, 8], [2, 2, 2]);
const newVector = n.pick(0, 0);
n.set(1, 1, newVector);

Context:
Writing an image processing library. I wrote a median filter for the image. I need iterate over all the pixels and set them to the median of their neighbors.

I think there is no way to do it since the underlying data might not be contiguous in your newVector. Except some cases only you know about, there are no more efficient ways than the one you showed.

I have the same requirement. I need to copy a subset of an ndarray into a different bigger ndarray as efficiently as possible. I think this can be implemented faster than the for loop above by using the compilation techniques that ndarray uses.

You can use ndarray ops to do this, or for your median filter it might be faster to use cwise.

Ah, right! I didn't see that, it's under "Special cases". So, it's:

ops.assign(dest, src) copies one array into another

Meaning you slice your source and destination arrays properly and then do assign().
This issue can be closed then.


Can't upgrade to new version

Whenever I try to upgrade or build the new version I get this error: no override and no defaul toolchain set.

Can you tell me more information? OS, Rust version, and what did you do?

Okay, maybe i found what cause this issue.
You are using rustup and not set default toolchain, That is a omission of PKGBUILD, I will push new PKGBUILD to AUR.
Ref: https://aur.archlinux.org/packages/himalaya-git/#comment-804053

And I recommend you to use extra/rust instead of community/rustup.
The rustup will store data more than 1 GiB in your $HOME/.rustup and $HOME/.cargo. And these data will not be automatically cleaned up after you uninstall rustup.
But another one rust doesn't store these data.

Fixed:

mpv-handler: https://aur.archlinux.org/cgit/aur.git/commit/?h=mpv-handler&id=f47724674034997d3c90c3946e9e9efb910c00a8
mpv-handler-git: https://aur.archlinux.org/cgit/aur.git/commit/?h=mpv-handler-git&id=c6572819d9eb9ab34986fd5fbb7a224a87275af8

Fixed, thank you.


Dotfiles Role become unpriviledged user fails

https://docs.ansible.com/ansible/latest/user_guide/become.html

Closing: No longer applies


Highlighting doesn't handle some optional semicolons

This does not highlight correctly:
EXAMPLE <- 0.5
class Example {}

Adding a semicolon fixes the highlighting:
EXAMPLE <- 0.5;
class Example {}

^^^^ It's kind of convenient and amusing that GitHub's highlighting behaves the same way.


I'm sorry to try to contact you by this since you are not provide your email anywhere.

Hi Mr. Yu,
I'm confused that in you paper Don’t Listen To Me: Understanding and Exploring Jailbreak Prompts of Large Language Models. Figure 4(b) seems to have something wrong with it.

You try to proof that the experts have better performance than the novices, but the diagram shows that the experts are more likely to have less accumulative success than the novices due to the left-shift of the probability density center.
Would you mind explaining this to me?
Best regards.

Thanks for your interest in our project! Please see my reply via email.


⚠ Bofrost.at has degraded performance

In 176c283, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1675 ms

Resolved: Bofrost.at performance has improved in df3f1d4 after 6 minutes.


[build-script] Move dotest arguments up to the configuration phase.

Rather than constructing the dotest arguments at test-time, we should
build them at configuration time and have CMake know about them. That
way you can run check-lldb in the LLDB build directory and have the same
behavior compared to running the test suite with build script.

LGTM


Popup/balloon inline link and image editing in WYSIWYG mode?

Summary
Hello. I am working on an editor with TUI Editor and I would like to add popup/balloon inline link and image editing to the WYSIWYG mode of my editor (screenshot below). I looked but didn't see a way to enable this functionality with TUI Editor currently.
I was wondering if it's possible with TUI Editor's API or with my own code while using TUI Editor and how it could be done? For example, I was thinking I'd need a "caret on enter link node" event so that I could trigger the popup to appear, a "caret on leave link node" so that I could hide it once it's no longer necessary, and functions to get and modify the current node programmatically by interacting with the popup. In the case of images, the events would need to be when the image is clicked-on/selected instead plus the other functions. I'd also need additional info about the current node like position so that I could position the popup by it.
Inline editing of links and images via popups/balloons would be really great because then TUI Editor's WYSIWYG editing mode would be fully capable with no need to switch to markdown editing mode for any changes (which isn't the case right now if you want to edit existing links/images as far as I know). I was thinking that this issue could also maybe be a feature request in addition to question.
Screenshots
Here is an example of the sort of thing I'm trying to do re: inline popups/balloons (word processor is Google Docs):

In my case, I think I would just have the link editing show immediately with input fields for link text and link URL though.
Version
TUI Editor Version 3.1.1 (released Wed Oct 27 2021, latest).

@Alyw234237
It seems possible to implement using our popup widget. However, there is currently no event to judge whether the caret is located on a specific node when mouseenter, mouseleave are triggered in our editor.
So, We need to improve our event support and also check whether the event affects editing performance. I'll consider the feature, but please note that the priority may be low.

@js87zz Thanks for the information! I tried doing it with a popup widget like you suggested and was able to get the popup to show successfully. However, I ran into some difficulties after this, like the popup preventing the caret from moving into the link (with arrow keys or mouse click), editor.addWidget adding the widget to the parent frame instead of the editor iframe, positioning of the popup widget being relative to the caret instead of the link element, and the widget disappearing upon being clicked which made the needed subsequent user interaction to update the link information not possible. I think it might not be possible to use editor.addWidget for this right now without a lot of fixes unfortunately. Thanks for your willingness to make changes to improve support for this sort of thing though, and I understand about the priority being low.
So I decided to try doing it with vanilla JavaScript instead and I managed to get it working pretty well this way. Here's a screenshot:

I did have to put the popup element in the parent frame instead of the editor iframe but this works fine in any case. I just use absolute positioning and JavaScript onscroll events to update the positioning and keep the popup in place.
One thing I ran into a lot of trouble with though was programmatically updating the link display text. For the URL, updating that via JavaScript's Element.setAttribute on the element href worked fine and TUI Editor accepted that without an issue. However, trying to update the display text via e.g. Element.innerText or Element.innerHTML wouldn't work correctly (the new link text would be inserted but the link text would be partly link and partly text and other problems). I didn't see any functions for updating links with TUI Editor in the API docs. The closest solution was TUI Editor's editor.getRangeInfoOfNode() plus editor.deleteSelection() followed by editor.exec('addLink'). However, this didn't work satisfactorily for me because editor.getRangeInfoOfNode() would get the next node instead of the link node if the caret was at the very end of the link (in contrast to JavaScript's document.getSelection().anchorNode.parentNode which I'm using to identify when the caret is in a link node to show the popup). In the end, this hackish solution is what I arrived at and is working okay for now:
  // Setting link.innerText doesn't work so need to do this hack instead (to-do: fix when possible)
  // Can't use TUI Editor's getRangeInfoOfNode() + deleteSelection() followed by TUI Editor's exec('addLink')
  // because getRangeInfoOfNode() will get the next node instead of the link if caret is at end of link
  link.remove();
  // setTimeout hack needed here so that remove element can complete first apparently
  setTimeout(function () {
    editors[active_tab_id]['tui_editor'].exec('addLink', {
      linkText: link_popup_text.value,
      linkUrl: link_popup_url.value,
    });
  }, 5);

It causes a brief flash and moves the caret to the end of the link though and could definitely be much better.
It'd be really helpful if there was a way to use editor.getRangeInfoOfNode() to get the node immediately preceding the caret position and if TUI Editor had a function for updating links instead of only adding new ones. Alternatively, if things were changed so that updating Element.innerText or Element.innerHTML worked without any issues, that could work instead.
I haven't tried popup editing of image attributes yet. It seems that that may be tricky because images can't be click-selected in TUI Editor right now as contenteditable=false is present on them. But I think I will give it a try later.


[1.8] Update changelog for 1.8.0

I want to release 1.8 with the latest changes, have I missed anything or is there something that should go into this release still?
https://github.com/getsentry/sentry-php/compare/1.7.1...master

It seems fine to me


Obfuscate sensitive data in deploy task output

This is more like a proof-of-concept, but it does the job even in its current state:
  triggers.%:      "" => "2"
  triggers.chart:  "" => ""
  triggers.values: "" => "replicaCount: 2\n\nimage:\n  repository: gpii/universal\n  checksum: sha256:d0b69df1e900912eab6db7185c5d998c25729f0bffeec0996ec433a750b52b64\n\nissuerRef:\n  name: letsencrypt-staging\n  kind: ClusterIssuer\n\ndnsNames:\n- flowmanager.natarajaya.dev.gcp.gpii.net\n\ningress:\n  disable_ssl_redirect: true\n\nmatchmaker_url: \"http://flowmanager.natarajaya.dev.gcp.gpii.net\"\n\ndatasource_hostname<EMAIL_ADDRESS>

LGTM,
perhaps it's worth to capture the output of the sh method and make the substitutions using ruby code. Avoiding the storing of extra files, and the using of external commands.
https://apidock.com/ruby/FileUtils/sh

@mrtyler Why do you think it is weird?
It is straightforward, flexible and universal (since sed is a default part of almost any unix-like OS distribution) in comparison to other solutions like Terrahelp that relies on terraform.tfvars content to perform substitutions and, in our case, can miss sensitive data in output of various resources (i.e. local_file, etc). If you don't want to make another dependency on sed, we can make this optional and do not filter output if sed is not present (on developer Windows machine maybe?).
I think that our main goal here – not to leave sensitive data in CI logs. The fact that secrets are being written to file on disk does not concern me much, since it is easier to control access on worker instance (use encrypted partition / forcibly wipe it up after build is finished).
@amatas We can not just capture sh output and then make our substitutions, because it is going to delay any output of rake deploy for the duration of xk up execution, which, I think we all agree, is not very convenient.
The only "realtime" output processing solution that I can think of in pure ruby requires something like IO.popen: https://ruby-doc.org/core-2.3.0/IO.html#method-c-popen
And may look like this:
    @secrets.each do |key, val|
      out = out.gsub("#{val}", "<sensitive>")
    end
    puts out
  end```

I am not sure if it is any better...

@amatas @mrtyler Please check out new sed-less solution, I would like to have this one merged before I start working on Google KMS integration for secrets.

@mrtyler I addressed all issues from your comments, please check latest changes.


Port list download code to JS

I wish I could remove all the shell scripts but for the sake of compatibility 🥲

The names whitelist.csv and input.csv shouldn't be completely removed for compatibility with older versions (maybe add a fallback?). And I think it'll be a good idea to just go all-in on Node.js for allow/blocklist downloads and replace the contents of get_recommended_*.sh with node [script].js commands so the lists can be downloaded on any system, including Windows.

Hmm... Let me revert the name changes then.

Fair enough. I didn't notice that. There's still a probability that the script's going to break for people using their own lists though.

Hmm... Let me revert the name changes then.

No need to revert them, just dynamically adjust the name of files being used based on which one exists. You can use fs.existsSync() for that.

So here it is 😁

May want to add that these scripts support hosts, domain and adblock formats but work best with wildcard domain (without asterisk) format now.

Thanks! I’m feeling guilty for burdening you with so many requests, but there’s one more tiny thing. Could you please make it so that the download_lists Node script grabs both allowlists and blocklists when it’s run with no arguments? That’d be good for convenience.

Alright.

Thanks for contributing so much :)


Thanks a lot for making this, I have an issue with block time

Just specifying block interval as 60 seconds doesn't enforce that and I get random times close to 5 min between blocks
Any recommendations how to get it to 1min blocks?

Just specifying block interval as 60 seconds doesn't enforce that and I get random times close to 5 min between blocks Any recommendations how to get it to 1min blocks?

You need to run the mutiny-net fork with adjustments to bitcoind if you need proper control over the block-time production, otherwise Bitcoin Core will end up adjusting difficulty back towards the 10 minute standard.


output is not generating letter by letter, instead all at once

The out put from the models is not generating/showing letter by letter, but instead, its taking time till the end of the API call, and showing all at once. How do i get it to show letter by letter like chat GPT or something?

you're essentially asking for streaming

@zz2115 thanks, I'll look into it


Add syntax for changing text colour

Currently, the only way to do this would be to use HTML, which is against the style guidelines of the project.
For the curriculum guides in particular, text colour will be a very useful way of making the documents more reader friendly, and making certain parts stand out more than others.
I'm happy to work on this if nobody else is able to pick it up in the near future.

Where's an example of this were it'd be better than italics or emphasis? :)

I should have mentioned, this will also help to make the documents more visually appealing.
Just to clarify too, I'm referring to the assessment guides (not the chapters, which are fine as they are).

I think the best way forward for this would be to create a set of markdown extensions or macros that are defined in CSS and which add a class to the element by the parser.
Thoughts

Keep content and presentation separate
Designers can modify what their opinion of ‘visually appealing’ means ;)
Readability for vision impaired users.

On Jun 20, 2016, at 11:13 AM, Heidirose<EMAIL_ADDRESS>wrote:
I should have mentioned, this will also help to make the documents more visually appealing.
Just to clarify too, I'm referring to the assessment guides (not the chapters, which are fine as they are).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/uccser/cs-field-guide/issues/360#issuecomment-227026295, or mute the thread https://github.com/notifications/unsubscribe/AAU8hL-3z63y5Dtpi3H6qpt7vmWO27fmks5qNc0VgaJpZM4I5EcX.

Yeah I agree, I'd also go about it like that.

This can be added into the system, but before we make any major changes, @Heidirose could you please post a screenshot, say of a curriculum guide, with the text coloured to show what the end result might look like?

@timbellnz Can you please confirm that it is okay for me to spend time on this? Just might take me awhile to do as I haven't done much before in terms of modifying CSS with developer tools (which I think is the best way of doing this).

No longer relevant as we're changing the format of curriculum guides.


ros2 build farm error Rolling

@norro
the ros2 build farm does not run for Rolling:
https://build.ros2.org/job/Rbin_uF64__rclc_lifecycle__ubuntu_focal_amd64__binary/44/console
23:06:13 /usr/lib/ccache/cc -DDEFAULT_RMW_IMPLEMENTATION=rmw_cyclonedds_cpp -DRCUTILS_ENABLE_FAULT_INJECTION -DROS_PACKAGE_NAME=\"rclc_lifecycle\" -Drclc_lifecycle_EXPORTS -I/tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/include -I/tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src -isystem /opt/ros/rolling/include  -g -O2 -fdebug-prefix-map=/tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0=. -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC   -Wall -Wextra -Wpedantic -std=gnu99 -o CMakeFiles/rclc_lifecycle.dir/src/rclc_lifecycle/rclc_lifecycle.c.o   -c /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c
23:06:13 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c: In function ‘rclc_make_node_a_lifecycle_node’:
23:06:13 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:50:5: warning: passing argument 9 of ‘rcl_lifecycle_state_machine_init’ makes pointer from integer without a cast [-Wint-conversion]
23:06:13    50 |     true,
23:06:13       |     ^~~~
23:06:13       |     |
23:06:13       |     int
23:06:13 In file included from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/include/rclc_lifecycle/rclc_lifecycle.h:24,
23:06:13                  from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:17:
23:06:13 /opt/ros/rolling/include/rcl_lifecycle/rcl_lifecycle.h:253:49: note: expected ‘const rcl_lifecycle_state_machine_options_t *’ {aka ‘const struct rcl_lifecycle_state_machine_options_t *’} but argument is of type ‘int’
23:06:13   253 |   const rcl_lifecycle_state_machine_options_t * state_machine_options);
23:06:13       |   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
23:06:13 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:41:23: error: too many arguments to function ‘rcl_lifecycle_state_machine_init’
23:06:13    41 |   rcl_ret_t rcl_ret = rcl_lifecycle_state_machine_init(
23:06:13       |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
23:06:13 In file included from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/include/rclc_lifecycle/rclc_lifecycle.h:24,
23:06:13                  from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:17:
23:06:13 /opt/ros/rolling/include/rcl_lifecycle/rcl_lifecycle.h:244:1: note: declared here
23:06:13   244 | rcl_lifecycle_state_machine_init(
23:06:13       | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
23:06:13 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c: In function ‘rcl_lifecycle_node_fini’:
23:06:13 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:233:13: error: too many arguments to function ‘rcl_lifecycle_state_machine_fini’
23:06:13   233 |   rcl_ret = rcl_lifecycle_state_machine_fini(
23:06:13       |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
23:06:13 In file included from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/include/rclc_lifecycle/rclc_lifecycle.h:24,
23:06:13                  from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:17:
23:06:13 /opt/ros/rolling/include/rcl_lifecycle/rcl_lifecycle.h:278:1: note: declared here
23:06:13   278 | rcl_lifecycle_state_machine_fini(

https://build.ros2.org/job/Rbin_ufv8_uFv8__rclc_lifecycle__ubuntu_focal_arm64__binary/39/
23:05:57 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:50:5: warning: passing argument 9 of ‘rcl_lifecycle_state_machine_init’ makes pointer from integer without a cast [-Wint-conversion]
23:05:57    50 |     true,
23:05:57       |     ^~~~
23:05:57       |     |
23:05:57       |     int
23:05:57 In file included from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/include/rclc_lifecycle/rclc_lifecycle.h:24,
23:05:57                  from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:17:
23:05:57 /opt/ros/rolling/include/rcl_lifecycle/rcl_lifecycle.h:253:49: note: expected ‘const rcl_lifecycle_state_machine_options_t *’ {aka ‘const struct rcl_lifecycle_state_machine_options_t *’} but argument is of type ‘int’
23:05:57   253 |   const rcl_lifecycle_state_machine_options_t * state_machine_options);
23:05:57       |   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
23:05:57 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:41:23: error: too many arguments to function ‘rcl_lifecycle_state_machine_init’
23:05:57    41 |   rcl_ret_t rcl_ret = rcl_lifecycle_state_machine_init(
23:05:57       |                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
23:05:57 In file included from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/include/rclc_lifecycle/rclc_lifecycle.h:24,
23:05:57                  from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:17:
23:05:57 /opt/ros/rolling/include/rcl_lifecycle/rcl_lifecycle.h:244:1: note: declared here
23:05:57   244 | rcl_lifecycle_state_machine_init(
23:05:57       | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
23:05:57 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c: In function ‘rcl_lifecycle_node_fini’:
23:05:57 /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:233:13: error: too many arguments to function ‘rcl_lifecycle_state_machine_fini’
23:05:57   233 |   rcl_ret = rcl_lifecycle_state_machine_fini(
23:05:57       |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
23:05:57 In file included from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/include/rclc_lifecycle/rclc_lifecycle.h:24,
23:05:57                  from /tmp/binarydeb/ros-rolling-rclc-lifecycle-1.0.0/src/rclc_lifecycle/rclc_lifecycle.c:17:
23:05:57 /opt/ros/rolling/include/rcl_lifecycle/rcl_lifecycle.h:278:1: note: declared here
23:05:57   278 | rcl_lifecycle_state_machine_fini(

prepared new bloom release: https://github.com/ros2/rclc/pull/72

Created bloom realease https://github.com/ros/rosdistro/pull/28887

Rolling release back normal:
https://build.ros2.org/job/Rbin_ufv8_uFv8__rclc_lifecycle__ubuntu_focal_arm64__binary/42/
https://build.ros2.org/job/Rbin_uF64__rclc_lifecycle__ubuntu_focal_amd64__binary/47/


brave: 1.7.92 -> 1.8.90

Motivation for this change
Package update. Security fixes, bugfixes, etc.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

superseded by #87277 assuming it gets merged

I guess this can be closed, #87277 was merged.


Resize logo to decrease page load time.

Create a smaller version of the logo, but don't remove the current one. Then change the references to the new smaller (and ideally optimized) version.
The largest version of the logo we actually use on the site is just 20px wide, so it can be shrunk to that size. Then compressed using the steps laid out here https://developers.google.com/speed/docs/insights/OptimizeImages

Hi! First time contributor, I'd like to make this change

Go for it @chridgely! We'd love to have you on board!

Please check the updated PR, I've updated the logo size from 22.4 KB to 1.04 KB

Hi @TechiePriyaRanjan, what is the number for that PR? I can't seem to find it.

#16 , I just forget to make PR and merged it with me mine itself 🏷
Now it's reflected in PR tab :)

Thanks @TechiePriyaRanjan! @chridgely, I'm going to close this issue. If you'd like to grab another, we can definitely still use your help! https://github.com/approachable-io/software-interview-prep/issues