RoaringBitmap 1.2.0 version is missing from Maven central

Describe the bug
1.2.0 version has been released according to GitHub releases.
The artifacts are missing from Maven Central, https://repo1.maven.org/maven2/org/roaringbitmap/RoaringBitmap/ .
To Reproduce
https://repo1.maven.org/maven2/org/roaringbitmap/RoaringBitmap/1.2.0/ returns 404

We push our releases to maven central. It can take hours, days or weeks for the releases to appear, and they are sometimes dropped. It is an antiquated system. Please be patient or report the issue to Sonatype, the vendor responsible for Maven central. There is nothing we can do as we only provide the software packages.

This is a recurring problem, see https://github.com/search?q=repo%3ARoaringBitmap%2FRoaringBitmap+"Maven+central"&type=issues

To clarify the process, the GitHub release is something that we do manually, after having pushed the artefacts.

@lhotari Note that I do so 1.2.0 right now. Are you sure it is missing?

https://repo1.maven.org/maven2/org/roaringbitmap/RoaringBitmap/1.2.0/

Thank you @lemire for addressing the issue.


Returning error when library isn't declared.

Stack:

#4014 ⬅
#4013
#3993

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

tests/bug_samples/issue4007.cairo line 7 at r1 (raw file):

#[starknet::contract]
mod lama_test {

Suggestion:
TestContract


Out dated help screen and Current Visual Interactions Edit Interactions Useless

This is outdated. The PBI screens do not look like this anymore. The Visual Interactions is a drop down and is completely useless. The slicer impacts all the visuals. Toggle on or off the Edit interactions or Drilling filters of other Visuals is impotent. This is a fundamental feature of PBI. To be able to have slicers work on specific visuals.

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

ID: 9b7c372d-539c-5857-01c5-8a8d835380a8
Version Independent ID: 08d1564e-d1dc-ae37-8147-ef21c80feccc
Content: Change how visuals interact in a report - Power BI
Content Source: powerbi-docs/create-reports/service-reports-visual-interactions.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @v-keknec. This issue has been assigned to the doc author for review.


Update aws-java-sdk-bundle to 1.12.343

Updates com.amazonaws:aws-java-sdk-bundle from 1.12.330 to 1.12.343.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-bundle" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #2401.


bootstrap export media issue

Prompted by a question on the community forums I just re-tested Bootstrap export and not sure if this is just an issue when viewed locally rather than uploading and viewing online but when exporting and viewing locally the xml contains links to media like the following:
header="FileLocation + '/media/barrier.png'"
url="FileLocation + '/media/page_types_screenshot.png'"
The first forward slash means the media folder and therefore images aren't found.
Changing to:
header="FileLocation + 'media/barrier.png'"
url="FileLocation + 'media/page_types_screenshot.png'"
seems to work fine.
Like I say I haven't checked if that changes if the zip is uploaded, extracted and viewed via a VLE etc but I suspect it might be the same.
Haven't looked at how easy this would be to fix yet - I suspect quite easy but got to go offline for a while now.

The problem is affexting both bootstrap AND xerte toolkits projects. FileLocation is an empty string (in bootstrap projects) and undefined (in toolkits projects), so paths cannot be properly eval'd.
In the case of the bootstrap example, a simple check an solve the issue:
If (FileLocation == ''){
var FileLocation = String(window.location).substr(0, String(window.location).lastIndexOf('/')  + 1);
}
In the case of a toolkit project, FileLocation is undefined. Pesumably something similar will work in Xenith.js.
I'm trying to work out how FileLocation is set in toolkits, and how we can reliably test for an exported project and calculate the path. Tom m,ay have some other approach?
At least we know what the issue is: it's not an additional slash being added; the paths do actually work fine if Filelocation is what it should be.
So we need to test and set it.

Be careful:
The FileLocation is supposed to be set in the main htm file for the exported object. The source is rloObject.htm (modules/site/player_html5 and modules/xerte/player_html5). On export the %XMLPATH%  is supoosed to be replaced by a proper value (like '.')
If this doesn't work anymore,we need to check the export code!

Just noticed a perhaps related issue with the stand alone decision tree template...
Create a project based on the project template, using the editor add an image. The image displays in the editor itself but doesn't show when playing the project. Looking at the xml it has FileLocation + '/media... so wondering if this is a related issue?

It seems all paths from the new editor have /media (i.e. with the leading slash): in the old editor FileLocation always held the trailing slash. The new format works as expected on the server: I expect the decision tree template is expecting the old format?

Hmmm.... I changed the code back so that only media is used in stead of /media like the old editor did.
Otherwise offline export won't work, because it cannot get the current location in all cases.
It would be good to test this.
This can be problematic for some of the new objects. If you export before publishing again, you might end up in the situation that the export doesn't work. Publish and re-export should solve that.

attempted to export from bootstrap on v3.0 and it says it works with VLE but i want just the offline version like one can do in XOT so i can put on a website i am building with laravel
what do u suggest?


Setting MQ_DISABLE_WEB_CONSOLE exits the container

If you set the MQ_DISABLE_WEB_CONSOLE variable, then the container exits when it tries to set up the web console.  This looks like an issue with the newly-added "source" for the script.

Fixed


you've found a bug in the Crystal compiler.

I'm created a new Dokku app in my Digitalocean droplet. Show me this error when I push Amber app with Buildpack. What's the solution?
Counting objects: 58, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (47/47), done.
Writing objects: 100% (58/58), 35.59 KiB | 3.56 MiB/s, done.
Total 58 (delta 1), reused 0 (delta 0)
-----> Cleaning up...
-----> Building amber from herokuish...
-----> Adding BUILD_ENV to build environment...
-----> Fetching custom buildpack
-----> Crystal app detected
-----> Installing Crystal (0.25.0 due to latest release at https://github.com/crystal-lang/crystal) from https://github.com/crystal-lang/crystal/releases/download/0.25.0/crystal-0.25.0-1-linux-x86_64.tar.gz... done
-----> Installing Dependencies... Fetching https://github.com/amberframework/amber.git
       Fetching https://github.com/amberframework/amber-router.git
       Fetching https://github.com/mosop/cli.git
       Fetching https://github.com/mosop/optarg.git
       Fetching https://github.com/mosop/callback.git
       Fetching https://github.com/mosop/string_inflection.git
       Fetching https://github.com/elorest/compiled_license.git
       Fetching https://github.com/jeromegn/kilt.git
       Fetching https://github.com/TechMagister/liquid.cr.git
       Fetching https://github.com/phoffer/inflector.cr.git
       Fetching https://github.com/amberframework/micrate.git
       Fetching https://github.com/crystal-lang/crystal-db.git
       Fetching https://github.com/crystal-lang/crystal-mysql.git
       Fetching https://github.com/will/crystal-pg.git
       Fetching https://github.com/stefanwille/crystal-redis.git
       Fetching https://github.com/ysbaddaden/pool.git
       Fetching https://github.com/jwaldrip/shell-table.cr.git
       Fetching https://github.com/jeromegn/slang.git
       Fetching https://github.com/crystal-lang/crystal-sqlite3.git
       Fetching https://github.com/mosop/teeplate.git
       Fetching https://github.com/amberframework/granite.git
       Fetching https://github.com/amberframework/quartz-mailer.git
       Fetching https://github.com/arcage/crystal-email.git
       Fetching https://github.com/amberframework/jasper-helpers.git
       Fetching https://github.com/amberframework/citrine-i18n.git
       Fetching https://github.com/TechMagister/i18n.cr.git
       Installing amber (0.8.0)
       Installing amber_router (0.1.0)
       Installing cli (0.7.0)
       Installing optarg (0.5.8)
       Installing callback (0.6.3)
       Installing string_inflection (0.2.1)
       Installing compiled_license (0.1.3)
       Installing kilt (0.4.0)
       Installing liquid (0.3.0)
       Installing inflector (0.1.8)
       Installing micrate (0.3.3)
       Installing db (0.5.0)
       Installing mysql (0.5.0)
       Installing pg (0.15.0)
       Installing redis (2.0.0)
       Installing pool (0.2.3)
       Installing shell-table (0.9.2)
       Installing slang (1.7.1)
       Installing sqlite3 (0.10.0)
       Installing teeplate (0.6.1)
       Installing granite (0.12.1)
       Installing quartz_mailer (0.5.1)
       Installing email (0.2.5)
       Installing jasper_helpers (0.2.0)
       Installing citrine-i18n (0.3.2)
       Installing i18n (0.2.0)
       done
remote: Error writing file: Broken pipe
remote: ???
remote: ???
remote: ???
remote: ???
remote: ???
remote: ???
remote: ???
remote: ???
remote: ???
remote: ???
remote: ???
remote: ???
remote:
remote: Error: you've found a bug in the Crystal compiler. Please open an issue, including source code that will allow us to reproduce the bug: https://github.com/crystal-lang/crystal/issues
remote: Unhandled exception: Error writing file: Broken pipe (Errno)
remote:   from ???
remote:   from ???
remote:   from ???
remote:   from ???
remote:   from ???
remote:   from ???
remote:   from ???
remote:   from ???
remote:   from ???
remote: Failed to raise an exception: END_OF_STACK
remote: [0x7fdd84057446] ???
remote: [0x7fdd83719369] ???
remote: [0x7fdd83720699] ???
remote: [0x7fdd8371d8cb] ???
remote: [0x7fdd85496afe] ???
remote: /usr/bin/ld: cannot find -lyaml (this usually means you need to install the development package for libyaml)
remote: collect2: error: ld returned 1 exit status
remote: Error: execution of command failed with code: 1: `cc "${@}" -o '/tmp/build/app'  -rdynamic  -lz `command -v pkg-config > /dev/null && pkg-config --libs --silence-errors libssl || printf %s '-lssl -lcrypto'` `command -v pkg-config > /dev/null && pkg-config --libs --silence-errors libcrypto || printf %s '-lcrypto'` -lyaml -lpcre -lm -lgc -lpthread /tmp/crystal/share/crystal/src/ext/libcrystal.a -levent -lrt -ldl -L/usr/lib -L/usr/local/lib`
To coreajans.com:amber
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@dokku.me:amber'

Hello @set,
From the output:
remote: /usr/bin/ld: cannot find -lyaml (this usually means you need to install the development package for libyaml)
remote: collect2: error: ld returned 1 exit status

It appears that is failing to link with libyaml, which might be missing from the base image that is used by the buildpack and that normally contains all the build dependencies.
Can you confirm these images are in their latest/stable version and try again?
Hope that help.

Hi @set this looks like an amber issue :sweat_smile:
Are you using amber buildpack?
I think @eliaspr already fixed this issue on amber buildpack

Closing. There's nothing actionable here left.


MobileBERT support for eager execution

Referring to this module - https://tfhub.dev/google/mobilebert/uncased_L-24_H-128_B-512_A-4_F-4_OPT/squadv1/1.
I was trying to use it inside in a Keras-like model but it seems like the MobileBERT model is not yet supported in eager mode. Any way to remedy this other than using import tensorflow.compat.v1 as tf?

@sayakpaul,
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!


Wyoming Satellite Service: INFO:root:Disconnected from server

Hi all,
The satellite keeps disconnecting and reconnecting to the server after almost every voice command.  It's linked to using two Python scripts that control a servo (for R2-D2) and an LED (for C3-PO), which are used in the --detect-command and --tts-stop-command
--detect-command '/home/satellite1/wyoming-satellite/awake.sh'
--tts-stop-command '/home/satellite1/wyoming-satellite/done.sh'
Here's the log from: journalctl -u wyoming-satellite.service -f
These are the two lines below to look at:
Mar 13 16:08:34 satellite1 run[2876]: INFO:root:Disconnected from server
Mar 13 16:08:34 satellite1 run[2876]: INFO:root:Connected to server
Mar 13 16:08:26 satellite1 run[2910]: Playing raw data 'stdin' : Signed 16 bit Little Endian, Rate 16000 Hz, Mono
Mar 13 16:08:31 satellite1 run[2908]: LED on
...
Mar 13 16:08:31 satellite1 run[2908]: LED off
Mar 13 16:08:31 satellite1 run[2908]: End of done.sh script
Mar 13 16:08:31 satellite1 run[2908]: All done!... Goodbye
Mar 13 16:08:34 satellite1 run[2876]: INFO:root:Disconnected from server
Mar 13 16:08:34 satellite1 run[2876]: INFO:root:Connected to server
Mar 13 16:08:36 satellite1 run[2912]: Start PWM running, but with value of 0 (pulse off)
Mar 13 16:08:36 satellite1 run[2912]: Turn R2-D2's head
Mar 13 16:08:36 satellite1 run[2912]: End of awake.sh script
Mar 13 16:08:36 satellite1 run[2876]: INFO:root:Waiting for wake word

awake.sh
#!/usr/bin/env python3

# Import libraries
import RPi.GPIO as GPIO
import time

# Cowards way out, but I think there's a "awake.sh:11 RuntimeWarning: This channel is already in use" because there's a Mic HAT on the 40 pins.
# GPIO.setwarnings(False)

# Set GPIO numbering mode
GPIO.setmode(GPIO.BOARD)

# Set pin 16 as an output, and set servo1 as pin 16 as PWM
GPIO.setup(11,GPIO.OUT)
#GPIO.setup(37,GPIO.OUT)

servo1 = GPIO.PWM(11,50) # Note 11 is pin, 50 = 50Hz pulse

#start PWM running, but with value of 0 (pulse off)
servo1.start(0)
print ("Start PWM running, but with value of 0 (pulse off)")
time.sleep(0)

# Move R2-D2's head
print ("Turn R2-D2's head")
servo1.ChangeDutyCycle(7)
time.sleep(0.5)
servo1.ChangeDutyCycle(11)
time.sleep(0.5)
servo1.ChangeDutyCycle(10)
time.sleep(0.2)
servo1.ChangeDutyCycle(12)
time.sleep(0.5)

#Clean things up at the end
servo1.stop()

GPIO.cleanup()

print ("End of awake.sh script")

done.sh
#!/usr/bin/env python3

import RPi.GPIO as GPIO         # Import Raspberry Pi GPIO library
from time import sleep          # Import the sleep function

pinLED = 14                     # LED GPIO Pin

GPIO.setmode(GPIO.BCM)          # Use GPIO pin number
#GPIO.setwarnings(False)         # Ignore warnings in our case
GPIO.setup(pinLED, GPIO.OUT)    # GPIO pin as output pin

sleep(0.5)

# Flash LED for C-3PO talking
GPIO.output(pinLED, GPIO.HIGH)
print("LED on")
sleep(0.1)
GPIO.output(pinLED, GPIO.LOW)
print("LED off")
sleep(0.1)
GPIO.output(pinLED, GPIO.HIGH)
print("LED on")
sleep(0.3)
GPIO.output(pinLED, GPIO.LOW)
print("LED off")
sleep(0.1)
GPIO.output(pinLED, GPIO.HIGH)
print("LED on")
sleep(0.2)
GPIO.output(pinLED, GPIO.LOW)
print("LED off")
sleep(0.1)
GPIO.output(pinLED, GPIO.HIGH)
print("LED on")
sleep(0.2)
GPIO.output(pinLED, GPIO.LOW)
print("LED off")

GPIO.cleanup()
print ("End of done.sh script")
print ("All done!... Goodbye")

Here's the wyoming-satellite.service config:
[Unit]
Description=Wyoming Satellite
Wants=network-online.target
After=network-online.target
Requires=wyoming-openwakeword.service
Requires=2mic_leds.service

[Service]
Type=simple
ExecStart=/home/satellite1/wyoming-satellite/script/run \
  --name 'R2D2 Satellite' \
  --uri 'tcp://<IP_ADDRESS>:10700' \
  --mic-command 'arecord -D plughw:CARD=seeed2micvoicec,DEV=0 -r 16000 -c 1 -f S16_LE -t raw' \
  --snd-command 'aplay -D plughw:CARD=seeed2micvoicec,DEV=0 -r 16000 -c 1 -f S16_LE -t raw' \
  --snd-command-rate 16000 \
  --snd-volume-multiplier 0.5 \
  --mic-auto-gain 30 \
  --detect-command '/home/satellite1/wyoming-satellite/awake.sh' \
  --tts-stop-command '/home/satellite1/wyoming-satellite/done.sh' \
  --mic-noise-suppression 2 \
  --wake-uri 'tcp://<IP_ADDRESS>:10400' \
  --wake-word-name 'hey_are_two' \
  --event-uri 'tcp://<IP_ADDRESS>:10500' \
  --awake-wav sounds/r2d2-awake.wav \
  --done-wav sounds/r2d2-done.wav
WorkingDirectory=/home/satellite1/wyoming-satellite
Restart=always
RestartSec=1

[Install]
WantedBy=default.target

Removing these two lines below from the config fixes the issue, so I assume there's something wrong with the scripts?!?!
--detect-command '/home/satellite1/wyoming-satellite/awake.sh'
--tts-stop-command '/home/satellite1/wyoming-satellite/done.sh'
Thanks in advance for any help.

Not sure if this helps, but I have tried changing the _PONG_TIMEOUT and _PING_SEND_DELAY in the satellite.py file /home/satellite1/wyoming-satellite/wyoming_satellite/satellite.py

Didn't resolve?

I was able to replicate the behaviour if I block the detect script.
However, I noticed what you really want is maybe the detection script. And this one seem to not cause disconnection issue when blocked.
What I had in mind didn't work to solve it.
You would need to change your script to something like a non blocking fire and forget.

Thank you so much for testing it and getting back to me. I kind of understand what you mean, but as you can see, I'm not really much of a coder.  Can you explain what you mean by "You would need to change your script to something like a non blocking fire and forget.", or point me in the direction of info on how to make it a non-blocking script?
As a side note, what I'd really like is for the LED blinking to loop forever while the audio playback is happening (C-3PO is talking), then have a script to stop the LED blinking when the audio playback stops.  Is this possible?
I have tried using a third script using the --tts-played-command, but I can't get it to interact with the done.sh LED blinking script to stop the loop.


New logo design for the project

Hello! I want to contribute on your project with making logo design. What do you think about it?
Best Regards,
Arslan Şahin
Graphics Designer

Hey @Arslanshn , I'm glad to hear that you want to contribute to the project. Feel free to work on the logo design. :+1:

https://drive.google.com/drive/folders/1uIpnKO8dhyTQ653ZJWUrulXlHUg4FknZ?usp=sharing
hello I did the logo work I hope you like:)

Hi, divyan;-)

Hi I wonder what you think about the logo?

Hi

Hi @Arslanshn !! I really liked the design. But I think colors are bit dull. Can it be improved. What do you say?

Of course, what is the suggestion

Hello Divyanshu Logo I used different colors.

@Arslanshn Thanks a lot!! Looks good. I will apply the logo soon. :+1:

I'm glad you like it. Divyanshub:-)

Hi, Divyanshu;-)

Thanks @Arslanshn for the amazing logo. I have updated the Readme.


Tagging and document structure in community.sensenet.com

How to display different categories in the documentation? How do we tag each markdown file?
We might need an automation for tagging.
First steps

[ ] The generated docs for client-js are in the community site with the same skin we use
[ ] It is beautiful

Closed as obsolete.


Fix issue #67: Implemented scrollbar and toggle functionality for theme list

Changes Made:

Added scrollbar to the theme list for easier navigation!
Implemented toggle functionality for the theme button to show/hide the theme list.

Details:

Added CSS styles to create a scrollbar for the theme list container
Modified the ThemeButton component to toggle the visibility of the theme list on clicking the theme button by utilizing the useState hook in React.
Tested the changes locally to ensure smooth functionality

Related Issue:
This pull request addresses issue baoliay2008#67

@imsachin49 Thank you so much for your prompt pull request, excellent work.

Regarding the toggling functionality, I noticed three issues with its behavior:

Click the theme button, click outside, then click the theme button again.

#67-bug-1.webm

Click the theme button, select an item from the theme list, then click the theme button again.

#67-bug-2.webm

Click the theme button, drag the scrollbar, then click the theme button again.

#67-bug-3.webm

To address these issues:

For Problem 1: We should add an event listener to detect when a click occurs outside of the dropdown component.

#67-bug-1-fixed.webm

For Problems 2 and 3: Simply remove the onBlur event.

#67-bug-2-fixed.webm
#67-bug-3-fixed.webm

Any other feedback is welcome. If you agree with these suggestions, I'll proceed with merging the pull request.

Yeah those changes makes sense and i believe they effectively resolves the identified issues.
You can proceed with merging the pull request.

Yeah those changes makes sense and i believe they effectively resolves the identified issues.
You can proceed with merging the pull request.


GTP filter Policy

Hello,
Is there somewhere an example, if we want to forward packets sent from a UE behind the eNB to the LBP server?
I have experimented with the GTP filter field but I cannot get hold of how it works.
Thank you!

Regarding this question, has anyone forwarded GTP packets to the LBP server? and what was his setup?
Is it supported or should I close this issue?
Best

No, this feature is not supported


Stringify output

Hello! Can you consider exporting the stringify function, or have some other wayto obtain the stringified output without having to shell out?  e.g. accepting a buffer or stream to write into.
I am using this as a library (i.e. require-ing it) and am mainly interested in getting the string output, not the data structure itself.

If your concern is the result would be too long for a string to handle, then there's no way to handle this situation properly, since this tool is based on the concatenation of strings. There's no way to output a stream without storing a long string of scanned report first.
Otherwise, the processing of file node should be changed to a stream way.
For now, I can only put the output string in the resolved result with the  data structure as well, and the stream or buffer can be generated from the ouput string.

Hi! I don’t think I was worried about performance, the streaming nature was not important to the feature request.
I think the problem I ran into was that the library write directly to the console, so if I want to capture the formatted string for use inside javascript code, I need to run this as a child process and capture its standard output.
That seemed unnecessary, since the code is fully capable of just doing the work of collecting the files/directories and doing the formatting in memory and returning a string, instead of printing things directly to the console.

Hi! I don’t think I was worried about performance...

I see, I've fixed this and exported a new data structure in the resolved result with a report string, which might be what your want. Just get it with:
require('tree-cli')('your/root/path', ({ report, data }) => {
 getTheReport(report);
});

Pls check out the latest version 6.0.2.

Thank you! I’ll give it a try!

@chancancode Sorry, my bad for the callback style example, actually it only support the promise way. I've changed the example above, pls check it out.


Fix Xaml syntax error

@MikeHillberg : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jwmsft Could you review the proposed changes? Thanks.

#sign-off


ETag is mangled with a compression method

Using curl
curl -I https://lexfridman.com/feed/podcast

I get the following headers, as I'd expect:
HTTP/1.1 200 OK
Date: Sat, 19 Aug 2023 12:14:24 GMT
Server: Apache/2.4.55 (Ubuntu)
Last-Modified: Thu, 17 Aug 2023 20:37:40 GMT
ETag: "36d4b983b1bf2e278de73dd388cbb77c"
Link: <https://lexfridman.com/wp-json/>; rel="https://api.w.org/"
Content-Type: text/xml; charset=UTF-8

I'm aiming for the ETag, however when using OkHttp I'm getting "36d4b983b1bf2e278de73dd388cbb77c-gzip". It seems like okhttp has added the compression method?
Here's a failing test case:
@Test fun foo() {
  val response = OkHttpClient().newCall(Request.Builder().get().url("https://lexfridman.com/feed/podcast").build())
    .execute()

  assertEquals("36d4b983b1bf2e278de73dd388cbb77c", response.headers[HttpHeaders.ETag])
}

Initially, I reported this at Ktor but it seems like this is a bug in okhttp.

Now if you use the etag returned from OkHttp, there's no match:
curl -I -H 'If-None-Match: "36d4b983b1bf2e278de73dd388cbb77c-gzip"' https://lexfridman.com/feed/podcast
HTTP/1.1 200 OK
Date: Sat, 19 Aug 2023 12:18:42 GMT
Server: Apache/2.4.55 (Ubuntu)
Last-Modified: Thu, 17 Aug 2023 20:37:40 GMT
ETag: "36d4b983b1bf2e278de73dd388cbb77c"
Link: <https://lexfridman.com/wp-json/>; rel="https://api.w.org/"
Content-Type: text/xml; charset=UTF-8

When I remove the -gzip prefix though it works as expected and I get a 304 Not modified:
curl -I -H 'If-None-Match: "36d4b983b1bf2e278de73dd388cbb77c"' https://lexfridman.com/feed/podcast
HTTP/1.1 304 Not Modified
Date: Sat, 19 Aug 2023 12:18:52 GMT
Server: Apache/2.4.55 (Ubuntu)
Last-Modified: Thu, 17 Aug 2023 20:37:40 GMT
ETag: "36d4b983b1bf2e278de73dd388cbb77c"

Note that the actual value of the ETag might change by the time you've had a look.
Something else also happens for the talking kotlin RSS Feed: https://talkingkotlin.com/feed
OkHttp prepends an W/ which also breaks the ETag usage.

It seems like the server can decide what is relevant to the etag. Are you sure this is a bug?
OkHttp automatically advertises gzip support, unless you set "accept-encoding: identity", so you can disable it.
But can you change your repro to make a second request and see whether it's returning a 304?
I'll take a look tonight, don't have laptop with me this weekend.

For the second case, it looks like the server is marking these as weak validated.
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/ETag
Isn't the etag all server side?

Seems like the etag on the server is broken, definitely no bug in okhttp. The server responds differently for HEAD and GET requests. When I do curl https://lexfridman.com/feed/podcast (without -I) I get
Date: Sun, 20 Aug 2023 15:29:24 GMT
Server: Apache/2.4.55 (Ubuntu)
Last-Modified: Thu, 17 Aug 2023 20:37:40 GMT
ETag: "36d4b983b1bf2e278de73dd388cbb77c-gzip"
Link: <https://lexfridman.com/wp-json/>; rel="https://api.w.org/"
Vary: Accept-Encoding
Content-Encoding: gzip
Transfer-Encoding: chunked
Content-Type: application/rss+xml; charset=UTF-8

And the test passes when using HEAD

@Test fun foo() {
    val response = OkHttpClient().newCall(Request.Builder().head().url("https://lexfridman.com/feed/podcast").build())
        .execute()
    assertEquals("\"36d4b983b1bf2e278de73dd388cbb77c\"", response.headers["ETag"])
}

Isn't the etag all server side?

From my understanding not. I want to basically do the same as here: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/ETag#caching_of_unchanged_resources
But thanks for enlightening me that this is due a head/get difference, thanks!

An ETag is an opaque identifier assigned by a Web server to a specific version of a resource found at a URL.

I just think the server is taking advantage of caching compressed and uncompressed content separately.

Indeed.

curl --compressed -I -X GET https://lexfridman.com/feed/podcast

HTTP/1.1 200 OK
Date: Sun, 20 Aug 2023 16:32:24 GMT
Server: Apache/2.4.55 (Ubuntu)
Last-Modified: Thu, 17 Aug 2023 20:37:40 GMT
ETag: "36d4b983b1bf2e278de73dd388cbb77c-gzip"
Link: <https://lexfridman.com/wp-json/>; rel="https://api.w.org/"
Vary: Accept-Encoding
Content-Encoding: gzip
Transfer-Encoding: chunked
Content-Type: application/rss+xml; charset=UTF-8

I also found a SO question on this topic: https://stackoverflow.com/questions/29127144/cant-cache-resource-when-having-both-gzip-and-etag - it seems like they suggest to remove the content encoding prefix. Which is something I could do.

Just for the record. I wrote that RSS Feed author and asked him to fix this server side. The jetbrains feed does a nice job by supporting the -gzip etag correctly:

curl --compressed -I -X GET https://blog.jetbrains.com/idea/feed/

HTTP/2 200
etag: "f699d4a8ed7477c34339b9970db52f6e-gzip"

curl --compressed -I -X GET https://blog.jetbrains.com/idea/feed/ -H 'If-None-Match: "f699d4a8ed7477c34339b9970db52f6e-gzip"'


Xdebug fails install

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 2.1.1
VirtualBox 5.2.12r122591
ansible 2.5.2
  config file = None
  configured module search path = [u'/Users/upchuk/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /Library/Python/2.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.10 (default, Oct  6 2017, 22:29:07) [GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.31)]

Your OS

macOS (latest)

Full console output
TASK [geerlingguy.php-xdebug : Build Xdebug.] **********************************
changed: [drupalvm] => (item=phpize)
failed: [drupalvm] (item=./configure) => {"changed": true, "cmd": "./configure", "delta": "0:00:00.557990", "end": "2018-05-17 16:11:38.571881", "item": "./configure", "msg": "non-zero return code", "rc": 1, "start": "2018-05-17 16:11:38.013891", "stderr": "configure: WARNING: You will need re2c 0.13.4 or later if you want to regenerate PHP parsers.\nconfigure: error: not supported. Need a PHP version >= 7.0.0 and < 7.3.0 (found 5.6.36-1+ubuntu14.04.1+deb.sury.org+1)", "stderr_lines": ["configure: WARNING: You will need re2c 0.13.4 or later if you want to regenerate PHP parsers.", "configure: error: not supported. Need a PHP version >= 7.0.0 and < 7.3.0 (found 5.6.36-1+ubuntu14.04.1+deb.sury.org+1)"], "stdout": "checking for grep that handles long lines and -e... /bin/grep\nchecking for egrep... /bin/grep -E\nchecking for a sed that does not truncate output... /bin/sed\nchecking for cc... cc\nchecking whether the C compiler works... yes\nchecking for C compiler default output file name... a.out\nchecking for suffix of executables... \nchecking whether we are cross compiling... no\nchecking for suffix of object files... o\nchecking whether we are using the GNU C compiler... yes\nchecking whether cc accepts -g... yes\nchecking for cc option to accept ISO C89... none needed\nchecking how to run the C preprocessor... cc -E\nchecking for icc... no\nchecking for suncc... no\nchecking whether cc understands -c and -o together... yes\nchecking for system library directory... lib\nchecking if compiler supports -R... no\nchecking if compiler supports -Wl,-rpath,... yes\nchecking build system type... x86_64-unknown-linux-gnu\nchecking host system type... x86_64-unknown-linux-gnu\nchecking target system type... x86_64-unknown-linux-gnu\nchecking for PHP prefix... /usr\nchecking for PHP includes... -I/usr/include/php/20131226 -I/usr/include/php/20131226/main -I/usr/include/php/20131226/TSRM -I/usr/include/php/20131226/Zend -I/usr/include/php/20131226/ext -I/usr/include/php/20131226/ext/date/lib\nchecking for PHP extension directory... /usr/lib/php/20131226\nchecking for PHP installed headers prefix... /usr/include/php/20131226\nchecking if debug is enabled... no\nchecking if zts is enabled... no\nchecking for re2c... no\nchecking for gawk... no\nchecking for nawk... nawk\nchecking if nawk is broken... no\nchecking whether to enable Xdebug support... yes, shared\nchecking whether to enable Xdebug developer build flags... no\nchecking Check for supported PHP versions... ", "stdout_lines": ["checking for grep that handles long lines and -e... /bin/grep", "checking for egrep... /bin/grep -E", "checking for a sed that does not truncate output... /bin/sed", "checking for cc... cc", "checking whether the C compiler works... yes", "checking for C compiler default output file name... a.out", "checking for suffix of executables... ", "checking whether we are cross compiling... no", "checking for suffix of object files... o", "checking whether we are using the GNU C compiler... yes", "checking whether cc accepts -g... yes", "checking for cc option to accept ISO C89... none needed", "checking how to run the C preprocessor... cc -E", "checking for icc... no", "checking for suncc... no", "checking whether cc understands -c and -o together... yes", "checking for system library directory... lib", "checking if compiler supports -R... no", "checking if compiler supports -Wl,-rpath,... yes", "checking build system type... x86_64-unknown-linux-gnu", "checking host system type... x86_64-unknown-linux-gnu", "checking target system type... x86_64-unknown-linux-gnu", "checking for PHP prefix... /usr", "checking for PHP includes... -I/usr/include/php/20131226 -I/usr/include/php/20131226/main -I/usr/include/php/20131226/TSRM -I/usr/include/php/20131226/Zend -I/usr/include/php/20131226/ext -I/usr/include/php/20131226/ext/date/lib", "checking for PHP extension directory... /usr/lib/php/20131226", "checking for PHP installed headers prefix... /usr/include/php/20131226", "checking if debug is enabled... no", "checking if zts is enabled... no", "checking for re2c... no", "checking for gawk... no", "checking for nawk... nawk", "checking if nawk is broken... no", "checking whether to enable Xdebug support... yes, shared", "checking whether to enable Xdebug developer build flags... no", "checking Check for supported PHP versions... "]}
failed: [drupalvm] (item=make) => {"changed": true, "cmd": "make", "delta": "0:00:00.003167", "end": "2018-05-17 16:11:38.816101", "item": "make", "msg": "non-zero return code", "rc": 2, "start": "2018-05-17 16:11:38.812934", "stderr": "make: *** No targets specified and no makefile found.  Stop.", "stderr_lines": ["make: *** No targets specified and no makefile found.  Stop."], "stdout": "", "stdout_lines": []}

Summary
Trying to set up a box with Ubuntu 1404, PHP 5.6 and xdebug. But it fails the install...Any clue @geerlingguy ?

It looks like the problem is the version of XDebug you have configured is not compatible with PHP < 7.0:
error: not supported. Need a PHP version >= 7.0.0 and < 7.3.0 (found 5.6.361+ubuntu14.04.1+deb.sury.org+1)

See the note for PHP 5.6 for php_xdebug_version in the role documentation: https://github.com/geerlingguy/ansible-role-php-xdebug#role-variables

@geerlingguy Great, that worked. Thanks for the tip. Now it went further and still being tripped by 1771 :(

Awesome, closing out this issue then.


fix figsizes in  Nonconvex_scalar by loading matplotlib in separate cell

Also I added a comment that the last example doesn't work in the html version:  I think because of multiple sliders not converting to jsAnimation, this only works for a single time-slider.  Maybe we should figure out a way to handle other cases?
Hmm, I take it back.  For some reason the last time I created the new html file it did work, though it didn't previously. See http://www.clawpack.org/riemann_book/html/Nonconvex_scalar.html

Looks like the last example works in the linked version, but only has the slider for t (I guess that's what you said).  I'll go ahead an merge this.


NHAIS data received does not comply with validation requirements

According to the NHAIS specification (here: https://digital.nhs.uk/binaries/content/assets/website-assets/services/nhais/gp-spec-chapter3.doc), the Address - County field must "Have alphanumeric, SPACE, "'", "-", "." and "," characters only."  This means  "TYNE & WEAR" should not be allowed as ampersands are not included in the set of allowed characters, however we noticed that this appears in the test data.  When we sent registrations with "TYNE AND WEAR" as the county, we got back amendment messages to change the county to "TYNE & WEAR" which seems to contradict the specification.
Please can we have this clarified, as we have implemented our validation to disallow the & character as per the spec.

@davesmith9088 as discussed previously in the slack channel please see the reply below from the Hawthorne team. Based on this i will close this.

Hawthorne Team:

Sorry. This was a mistake on our part.
& is NOT a valid ‘GP Links EDIFACT Level A character set’ character. This was accidentally set up on our test machine and is not correct. We shall alter the data on Hawthorn now so that no further &s should be sent out.
Apologies for this.


Deprecate create static methods

The create static methods, like Observable.create and Subject.create are rife with problems.
While it's GREAT to have a functional way to create these things without the new keyword... it should be noted that:

TypeScript hates what we're doing here.
Subject.create doesn't actually create a Subject, but rather an AnonymousSubject, which I would REALLY like to rename as FrankenSubject because that describes what it is, you basically glue an Observer to an Observable and call it a "Subject".
None of the Subjects have a create method that makes sense, they all get the one from Subject (see 2)
Neither does ConnectableObservable... it just gets the one from Observable.

Proposal
Deprecate all create methods.
Then either:

Do nothing - no other library deals with wrapping constructors in functions for people, afaik.
Supply lowercase functional versions of each:  subject, behaviorSubject, connectableObservable
Make each constructor callable without new (the experimental version can do this, but the current version here can't really because we're using ES2015 classes). So like Observable(fn) is the same as new Observable(fn)

Thoughts?

I'm for removing create methods, then doing nothing for now. Is there an argument for "functional" versions that don't require new, other than disliking the keyword?

Is there an argument for "functional" versions that don't require new, other than disliking the keyword?

You can't call ES2015 class constructors properly with call or apply.
You can pass a constructor to a function that accepts a function and have it realize it needs to be called with new.  e.g. function execute(fn) {  fn(); }... (UNLESS You try/catch and handle the TypeError: Class constructor Test cannot be invoked without 'new' and then call it appropriately, I guess)

Those are really the only solid arguments.

no other library deals with wrapping constructors in functions for people, afaik

FWIW, Leaflet does this. It provides classes and lower-case factory functions.

Make each constructor callable without new

Changing Observable, etc. to not be a class would be a breaking change, so I don't see how, in v6, the create functions could be deprecated and replaced with ctors that are callable without new.
My vote would be to deprecate and then do nothing.

Notes from core team meeting: In favor of deprecating create methods.

I wasn't even aware that there are create methods, so +1 for deprecation.
As for what to do next, i think new-less ctors align nicely with web platform, e.g. Array() but are not a necessity. This is in contrast with factory functions like createImageBitmap - notice the create prefix.

What would be the deprecate message? or just mark it as deprecate?

@dkosasih The message should say something about using new instead. E.g. for Observable.create:
/**
 * Creates a new cold Observable by calling the Observable constructor
 * @static true
 * @owner Observable
 * @method create
 * @param {Function} subscribe? the subscriber function to be passed to the Observable constructor
 * @return {Observable} a new cold observable
 * @nocollapse
 * @deprecated use new Observable instead
 */
static create: Function = <T>(subscribe?: (subscriber: Subscriber<T>) => TeardownLogic) => {
  return new Observable<T>(subscribe);
}

I found this github issue by trying to find why wasn't create being exported as other creation methods such as from, fromEvent, interval, etc.
Is it possible to deprecate this static method, but export a function with the same name? such as this:
export const create = <T>(subscribe?: (subscriber: Subscriber<T>) => TeardownLogic) =>
     new Observable<T>(subscribe);


Adding non emdat data

Closes #21
Closes #22
@jessegrabowski - I have converted them into .py files based off of Camilo's work
Andrew

Superseded by #33


Possible to chain eager to query more than once and/or appendRelationMappings for an eager?

Hi there, welcome back, and thanks again for your continued development on Objection! Sorry for the multiple issues/questions, but I've found another scenario where I'm looking for a solution, and perhaps there's an enhancement that can be made if there's not already something I'm missing.
I have a scenario where I have a "base" relationExpression that I'd like to use in the eager for a query. Depending on some logic, I may want to add more relationships to the eager. It seems like when chaining multiple eagers to a query, they overwrite one another. My solution has been to "build" my relationExpression, but it doesn't feel right and could actually get quite nasty in some scenarios. It would be nice if there was a way to append more eagers to a query. Example problem code:
let q = User.query().select().eager('account');
...
if (req.user.shouldSeeSecrets === true) {
  q.eager('secretStuff');
  // This eager seems to replace the earlier eager, so 'account' will not be loaded
}
...

Existing solution/workaround:
let eagerArray = ['account'];
let q = User.query().select();
...
if (req.user.shouldSeeSecrets === true) {
  eagerArray.push('secretStuff');
}
let eagerString = `[${eagerArray.join(', ')}]`;
q.eager(eagerString);
// This works, but I imagine this could get tricky when trying to generate more complex eagers like:
// '[account.[notSecretStuff, secretStuff], secretStuff]'
...

Possible solution:
let q = User.query().select().eager('account');
...
if (req.user.shouldSeeSecrets === true) {
  q.addEager('secretStuff');
  // This eager would hopefully have the same effect as if the query was:
  // User.query.select().eager('[account, secretStuff]');
}
...

Is there already a way to accomplish this add/append eager behavior?
Thanks!

Thanks!
There is no way to currently accomplish this, but this sounds like a good idea! We could add a mergeEager method that merges the two eager trees. It would work even in these cases:
q.eager('a.b')
q.mergeEager('a.c')
// --> a.[b, c]

q.eager('[a, b.c]')
q.mergeEager('[a.d, b.[e, f], g]')
// --> [a.d, b.[c, e, f], g]

I'll implement this for 0.7. Please add any additional ideas and comments here.

That behavior looks great to me and those examples cover simple and more complex use cases very nicely! Would love to see it in future versions! Thanks!

My previous work on #235 (default eager loading) would benefit from this enhancement too!

@kos

@koskimas lots of great changes scheduled for 0.7.0! Any idea when it's coming out?
Thank you very much for all your work!

Thanks 🙏 I'll release it today


fix(curriculum): add test for link to stylesheet

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

This should help reduce confusion to those who do not read.
Although, I am wondering if this is the best solution - up for discussion.

I think, not including it, this is actually good in the long run.
The first notable mistake I made is linking stylesheet, and people who mess up in controlled environments are more likely to ask for help.

I vote for this change. Yes, in a perfect world people will read all of the instructions. But having people post the same issue over and over again in the forums has its own problems. Much better to prevent it in the first place.
My suggestion would be to make the hint a little clearer.  Either give them the actual link tag or direct them to the note at the end of the instructions.

@moT01 That is a good point. That is probably why we did not include it in the first place.

Is there any way would could check to see if the user has added CSS to styles.css and only do the check if they have?

I think we can see if the "file" has content, yes.

The more I think about this, the more I am of the opinion we should not add such a test.
Having a test that sometimes does nothing could lead to worse confusion than just keeping the Note we have.


TypeScript errors in 1.0.0 and 1.0.1

Hey guys!
It's a great idea to rewrite Sindre's pQueue for Deno. But there are 2 TypeScript errors in both releases.
$ deno info<EMAIL_ADDRESS>Compile<EMAIL_ADDRESS>error TS2322: Type '() => undefined' is not assignable to type '() => Promise<TaskResultType>'.
  Type 'undefined' is not assignable to type 'Promise<TaskResultType>'.

► https://deno.land/x/p_queue@1.0.1/mod.ts:287:37

287                         fallbackFn: () => {
                                        ~~~~~~~

  Did you mean to call this expression?

    ► https://deno.land/x/p_queue@1.0.1/mod.ts:287:37

    287                         fallbackFn: () => {
                                            ~~~~~~~

error TS1196: Catch clause variable cannot have a type annotation.

► https://deno.land/x/p_queue@1.0.1/mod.ts:297:33

297                 } catch (error: unknown) {
                                    ~~~~~~~

Found 2 errors.

I can't find a way to run my script which depends on this lib, even if I try to tell Deno to ignore errors (which I can't and I would avoid)...
How can I manage to deal with it? Or maybe have you planed to fix this?

Hey @ilgazil, I'm not able to reproduce your issue.
~/Code/pQueue tags/1.0.1
❯ deno --version
deno 1.10.2 (release, x86_64-apple-darwin)
v8 9.1.269.27
typescript 4.2.2

~/Code/pQueue tags/1.0.1
❯ deno info<EMAIL_ADDRESS>local: /Users/khushrajrathod/Library/Caches/deno/deps/https/deno.land/b6ff25c2831f2674940cc21ed69bd54d05fb7e93277b51c0ba8efc2a15c89bdc
type: TypeScript
emit: /Users/khushrajrathod/Library/Caches/deno/gen/https/deno.land/b6ff25c2831f2674940cc21ed69bd54d05fb7e93277b51c0ba8efc2a15c89bdc.js
dependencies: 6 unique (total 21.62KB)

<EMAIL_ADDRESS>(14.42KB)
├─┬<EMAIL_ADDRESS>(95B)
│ └──<EMAIL_ADDRESS>(3.15KB)
├─┬<EMAIL_ADDRESS>(1.61KB)
│ └──<EMAIL_ADDRESS>(268B)
├─┬<EMAIL_ADDRESS>(1.5KB)
│ ├──<EMAIL_ADDRESS>(599B)
│ ├──<EMAIL_ADDRESS>*
│ └──<EMAIL_ADDRESS>*
└──<EMAIL_ADDRESS>*

Can you make sure that you're on the latest version of deno (and therefore TypeScript) by running deno upgrade?

Oh my... Sorry @KhushrajRathod my Deno version was stuck in 0.9 and even if I tried to upgrade with the curl command, it stayed stuck. There was no error while trying to upgrade so I was sure the have the proper version...
After uninstalled and reinstalled, it's ok now and the TS error vanished...
Sorry for inconvenience...

No problem! lmk if you have any other issues


Command to list app memory & disk usage (as was removed from cf apps)

I just discovered that the information about instances, memory, and disk was removed from the cf apps command in CF CLI v7.
Unfortunately, this makes it a lot harder to create an overview of the memory & disk usage of our organisation's application stack, because I now need to cf app on every application separately to compile those data.
This adds up to a rather tedious job for the nearly 100 apps we have running instead of the few seconds that it used to take.
Therefore I was trying to figure out how to add the missing columns and was a bit puzzled to read in Upgrading to CLI v7 doc that this functionality was removed altogether; and I couldn't find any other way to achieve the same summary overview.
In this particular case I use that as part of our carbon footprint reporting, but it would be useful for anyone who is interested in an overview of their application's memory / disk usage.
I can imagine that those columns were removed to improve readability, but if it was available as a switch - for instance, cf apps -m (show memory column), cf apps -d (show disk column) it wouldn't get in the way, either :)
Thanks!
Lúthien

Hello @Luthien-in-edhil, thanks for submitting this request!
This removal happened because of a switch to using the /v3/apps endpoint as opposed to the v2 apps summary endpoint.  There isn't an equivlant apps summary endpoint in v3, although this has been previously requested on a cloud controller issue
This might be something worth considering though, @a-b @piyalibanerjee @sethboyles, either in the CLI or in the Cloud Controller as this isn't the first time we've had requests for this.

Thank you for your submission. If you still run into this issue with the latest CLI v8, please open a new ticket with the necessary information mentioned in our issue template.


Variables Became Null Or Empty After Taking Picture Using Camera Only On Android One Device

So, because of flow request put by a client, I'm now using a controller class to take and show pictures in my Activity. The pictures is going to be put in ArrayList and RecyclerView. Here is how I took the picture:
METHOD 1
My controller class:
companion object{
    var runnable:Runnable? = null
    var imagePath:String? = null
    var index = 0 //To mark which ArrayList index I would like to put the taken picture
}

fun init(){
    runnable = Runnable{
        //get image and put it to the Imageview
        arrayList[index].image = imagePath
        recyclerView.adapter!!.notifyDataSetChanged()
    }
}

Then I would run that runnable in my Activity like:
override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
    super.onActivityResult(requestCode, resultCode, data)
    Controller.imagePath = ImagePicker.getFilePath(data)
    Controller.runnable.run()
}

METHOD 2
This time I don't use static variables, except an Int variable to mark the ArrayList's index where I would add image to its item.
Controller class:
companion object{
    var index = 0
}

fun updateImage(data:Intent,position:Int){
    val imagePath = ImagePicker.getFilePath(data)
    arrayList[position].image = imagePath
    recyclerView.adapter!!.notifyDataSetChanged()
}

Then I would use that function from my activity like:
override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
    super.onActivityResult(requestCode, resultCode, data)
    controller.updateImage(data!!,controller.index)
}

RESULT:
Both of those are working fine on any devices I've tried, except an Android One device, which is Mi A1. For some reason after taking picture, using METHOD 1, the runnable would be suddenly null when camera closed, and then crash. And for METHOD 2, the ArrayList would suddenly become empty, the size is 0 after taking picture using camera, therefore also crash.
Actually the crash is kinda random, sometimes it's fine, sometime it's crashing. But mostly crashing.

Facing Same Issue.


xy_points.csv directly to stereo_xyz_points.csv

In the issue title here I'm fudging from my current naming convention of these things (point_data.csv and stereotriangulate_points.csv or something very close to that).  But the title text represents the direction I think things should go.
Currently I have an overly complicated workflow to go from 2d points to xyz points. Not only is it complicated, it is relatively slow (re-reading and re-processing each frame). I should be able to do this in a much more time efficient way, and doing it in bulk on the entire data set will just lay the stage for quickly doing it frame by frame in real time when needed.
Keep in mind that it may be helpful to maintain the stereopair projections in the PointEstimateData class as a way of better understanding what is going on with the camera array estimates....

This got fixed with the big refactor last week...


Added sea mode button

Fixes Issue
Closes #1262

Changes proposed
Added sea mode toggle button

Screenshots

Checklist

[x] You've linked this PR to the correct issue.
[x] You have checked that the code is working correctly.
[ ] You ⭐ the repository!

Note to reviewers

Looking forward to more contributions from you. 🧑💻

Thank you so much for your contribution.🌟
Make sure to ⭐ and follow me.


Typo "Azure Open AI"→"Azure OpenAI"

https://learn.microsoft.com/en-us/azure/ai-services/language-service/question-answering/how-to/azure-openai-integration
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/ai-services/language-service/question-answering/how-to/azure-openai-integration.md
#PingMSFTDocs

@hyoshioka0128 - thank you for your contribution!

merging


Add section on classes

Hi there, I wanted to add a section about classes, static variables, static methods, inheritance, super, constructors, and more. I think it's an important part of javascript.

Hello, thanks a lot for the contributtion!
I think the class scrutruce solve problems that object/modules also solve, but I've only written about objects yet and still haven't written about modules (which will come on the upcoming section)
After modules i was gonna add a part about "What you haven't learned in this guide" and classes was gonna be one of those concepts. Would you mind if I put this PR on hold until I reach that part of the guide, and than I can see how to incorporate the content you've written into mine?
Anyways thanks a lot for the PR gain, I have some styling suggestions to make to standardize what you've written with the current file but I wouldn't suggest you to do it now

ok, that seems fine :). I'll see what else I can contribute

So, I decided not to write abou classes and just reference them on the end of the guide. I've added an intro about how to help with revision and contribution in order to avoid other people writing a lot of content that doesn't get into the final material, I thank you a lot for the time you spent to do this but as of now I'd need more help with revision than with changing the direction of the guide as a whole.
Thanks again for the efforts!


Duplicate trips get added on refreshing

Steps to reproduce

Open app.
Go to My Trips and add some trips.
Refresh page multiple times(continuously).

Screenshot

Expected behaviour
It should not allow user to refresh continuously
Actual behaviour
Single trip is seen multiple times
Would you like to work on the issue?
Yes !

Great Finding @Balram02 👍

Can I try to fix this?

How do I onboard on the app, as I have been trying for a while now to create an account to no avail @Swati4star

Hi @AbdulDroid What problem are you facing while creating a new account?

I keep getting invalid emails or password every time @Swati4star please help

Are you sure the password is valid?
Password must contain minimum 8 characters, at least 1 Alphabet, 1 Number and 1 Special Character.

I followed all and I have to wait out the loading process to get the toast that says my email or password is invalid, and I believe this matches your criteria T3ster&1 for a password

You are doing this on signup screen. Right?

Yes right

I just tried creating a new account with some email id, I am not facing any issue. There is no issue with server.

I have tried several times again still the same result
Device info
Emulator: Pixel 2XL running Android P
Please try with same device or OS and let me know if you still get results @Swati4star

@AbdulDroid I created a new sample user.
Try signing in using these credentials:
email<EMAIL_ADDRESS>password: Test@123


add php ^5.5|^7.0 requirement in composer.json and remove php7 from allow_failures in travis config

styleCI error is not related with this


Need clarity on --docker-bridge-address option of aks create command

I'm really not clear on what this flag does. I guess what I'm trying to understand is;

how will this impact the network layout/configuration?
what part of the system will care about that value (Kubernetes, AKS, thing outside of the cluster)?

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

ID: d75c3a8d-1187-bea1-a7d4-7ebba71c1790
Version Independent ID: 75b0c352-762e-16ae-5cc6-d807ffc1dc3f
Content: az aks
Content Source: src/command_modules/azure-cli-acs/azure/cli/command_modules/acs/_help.py
Service: container-service
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

It's not even clear if it has to be a /16 range, and if you can reuse the same range for two AKS clusters in two Vnet that are peered

https://docs.microsoft.com/en-us/azure/aks/configure-azure-cni#deployment-parameters has the docker bridge explanation.


[BUG] Modular Armor hud symbol not displaying properly.

I also tested this using no mods other than the ones required to run Indrev using the most recent mod loader for both fabric and quilt which did not fix the issue. Issues remains on both fabric loader and quilt loader.

I have the same situation with 1.20.1/Fabric.
I feel uncomfortable with the transparency/lightness of the GUI.

Yup can confirm, also the chat gets transparent, is there a way to disable that feature or something?

Everyday I hope for a fix for this. My only issue with this mod.

Everyday I hope for a fix for this. My only issue with this mod.

Hi! I think I managed to remove the hud from the screen!
Try this: find the hud file in the game folder.json on this path: config/indrev/hud.json
And set these values there, as in the screenshot. Well, or any other astronomically huge numbers so that the HUD is rendered outside the field of view of your monitor

I don't see the need for this, there is Powah! in my modpack which charges armor in all dimensions

I don't see the need for this, there is Powah! in my modpack which charges armor in all dimensions

No not the energy, the protective shields from the protection module that protects you from damage. In the middle of combat you wouldn’t know how much damage you can take without the hud.


Flask scripts

This should be reviewed after flask_python gets merged, this branch only has the script changes and it should be a lot easier to browse through them (and not the other 130 flask library files)


Carry#13921 : Expand /info: Expose OSType (GOOS), Architecture (GOARCH)

This carry #13921 from 🌟 @olleolleolle to add OSType and Architecture to /info endpoint.

This change amends the diagnostic output of /info, to add two more keys. This supports Swarm's effort of helping users target multi-architecture Swarm clusters. See #13634.
These keys were introduced:

OSType (runtime.GOOS)
Architecture (runtime.GOARCH)

Trying to take into account the comment from @tiborvass in here : https://github.com/docker/docker/pull/13921#issuecomment-140270124

We can't use GOARCH, because https://github.com/docker/docker/pull/13921#issuecomment-130106474 , but GOOS does not have that problem so we should be able to rely on runtime.GOOS.
Let's find a way to detect runtime architecture for each OS:
Windows: https://msdn.microsoft.com/en-us/library/windows/desktop/ms724381(v=vs.85).aspx
Linux: syscall.Uname()
Darwin/FreeBSD: uname -m (shell out)
After seeing "ostype" in sysctl -a on both linux and darwin, I think we could use OSType (as suggested by others in earlier comments). What's more, the word "type" does not exist in the /etc/os-release spec: http://www.freedesktop.org/software/systemd/man/os-release.html
So for OSType, we could use runtime.GOOS
For a prettier or more structured representation of OS information, I suggest let's not decide now on what scheme to adopt for a crossplatform solution. Instead let's introduce OSType and keep OperatingSystem for the pretty version, because that's what it already is today.

/me hopes it builds on other platform
🌟 @tiborvass @runcom

/ping @tiborvass updated with a pkg/platform init() :wink:

LGTM
ping @docker/core-maintainers

LGTM

@tiborvass done 😇

LGTM


Use SilentError when no config is found

Throwing a regular error gave an IMO confusing error message, with the full stack trace etc.

Thank you, @simonihmig!

Released in v0.7.5


Failed to chat in the playground

Describe the bug

Steps to reproduce

Deploy a model.
Go to playground and chat.

Result

Returned an error:  Service unavailable. Please retry your requests after a brief wait.

Failed to curl after setting 'stream=true'

Expected behavior

Environment

GPUStack version: main c9bb9f3
OS: Ubuntu 22.04
GPU: NVIDIA 4090

Additional context
Add any other context about the problem here.

Should be resolved with box 0.0.81

GPUStack version: a14d498
llama-box: v0.0.81
Test Result: Pass


New guidance

Update welcome message based on codebuddies/codebuddies#911
Also add a /help command to ping various bits to users - such as /help hangouts will give the info about hangouts only

Is this still necessary, since we added more information to the welcome message and also have a slack etiquette at http://codebuddies.org/slack

We have done both of those, but might be nice to quick-link to http://codebuddies.org/slack via a /help

That's something that could also be set up quickly on the Slack admin
autoresponder, though I'm not against someone submitting a PR on greetbot
for it too.
On Wed, Nov 14, 2018, 1:29 AM Marc Baghdadi<EMAIL_ADDRESS>wrote:

We have done both of those, but might be nice to quick-link to
http://codebuddies.org/slack via a /help
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/codebuddies/greetbot/issues/86#issuecomment-438596028,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AETbu7Sj6-XAoDZnW_u9bz14KiRXjXitks5uu-KJgaJpZM4UoU3V
.


Prevent FORMS_URLFIELD_ASSUME_HTTPS warning on Django 5.0

Prevent this warning:
$ python -Wall manage.py help
/.../django/conf/__init__.py:214: RemovedInDjango60Warning: The FORMS_URLFIELD_ASSUME_HTTPS transitional setting is deprecated.
  warnings.warn(
...

The approach of listing the setting in deprecated_settings is the same as for STORAGES in #349, but it does not need to be conditional.

Tests are passing but Codecov is failing to upload:
[2024-05-09T10:11:52.172Z] ['error'] There was an error running the uploader: Error uploading to https://codecov.io: Error: There was an error fetching the storage URL during POST: 429 - {'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 925s.', code='throttled')}

I think this needs an admin to reinstall the app or grab a token from codecov.io. Or we could ditch codecov per https://hynek.me/articles/ditch-codecov-python/ - that’s what I did for my open source projects.

Tests are passing but Codecov is failing to upload:
[2024-05-09T10:11:52.172Z] ['error'] There was an error running the uploader: Error uploading to https://codecov.io: Error: There was an error fetching the storage URL during POST: 429 - {'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 925s.', code='throttled')}

I think this needs an admin to reinstall the app or grab a token from codecov.io. Or we could ditch codecov per https://hynek.me/articles/ditch-codecov-python/ - that’s what I did for my open source projects.

Thanks @adamchainz I opened an issue about that https://github.com/jazzband/django-configurations/issues/390

@pauloxnet - do you have permissions to rebase this PR so it reruns the tests now that the codecov issue is resolved?

@pauloxnet  - can you also confirm if you have been given full maintainer permissions on this project yet please

@pauloxnet - can you also confirm if you have been given full maintainer permissions on this project yet please

@uhurusurfa please avoid tagging me 2 times in 1 minute in 2 separate comments, as I already wrote I don't have full maintainer permission and I can't access the settings or force a rebase.
In future when you tag me in an issue please wait for a response, I'll try to reply when I'll find the free time to do that, it can be days, I understand your enthusiasm, but this is a total volunteer and unpaid job for me, and I can't assure any time of response or unlimited amount of time in it.
In the alternative, tag other maintainers, or apply to be part of the project.

@pauloxnet - can you also confirm if you have been given full maintainer permissions on this project yet please

@uhurusurfa please avoid tagging me 2 times in 1 minute in 2 separate comments, as I already wrote I don't have full maintainer permission and I can't access the settings or force a rebase.
In future when you tag me in an issue please wait for a response, I'll try to reply when I'll find the free time to do that, it can be days, I understand your enthusiasm, but this is a total volunteer and unpaid job for me, and I can't assure any time of response or unlimited amount of time in it.
In the alternative, tag other maintainers, or apply to be part of the project.

Fair enough. I will try alternative roputes


Add user notifications for suspending & unsuspending

Add alert & email notifications (not sure if they work, my local install may have some email errors) to alert user that got suspended/unsuspended.
I'm sure I did something wrong. Please correct me, this took me about 1 1/2 hours to figure out 🙄 and get somewhat working. The notifications don't seem to work all the time for some reason. I've been debugging and the sync method always gets called (if the field suspendUntil is updated), and I haven't been able to find the cause of it.
Please throw your knowledge at me.
Closes flarum/core#322

Ok, merge conflicts should be fine now. However, I see that none of the code works anymore because of master changes related to notifications, the classes are no longer found.
Should I implement the new changes?
I think it's something like this, based on looking at https://github.com/flarum/flarum-ext-mentions/blob/a07077230ca5624624a74fa38a709b16e3377957/src/Listener/UpdatePostMentionsMetadata.php#L55
/**
 * @param ConfigureNotificationTypes $event
 */
public function addNotificationType(ConfigureNotificationTypes $event)
{
  $event->add(UserSuspendedBlueprint::class, UserSerializer::class, ['alert']);
}

@datitisev yes the namespace changes are mostly moving them to their relative type (user related events to User etc).

Done! I would definitely test this locally first, though. I haven't had a change to test it, with all the changes in flarum/core & flarum/flarum, it just breaks my installation.

@datitisev could you please update this to use the events I just merged in #15 :)

Done, and after setting up flarum/core master again, seems to work just fine...
Except for locale being weird and me not having the correct fonts, because I cloned flarum/flarum and then composer install'd with all versions set to @dev

Thanks heaps @datitisev!

@tobscure No problem! 😄

Woop woop woop 1 year to merge a PR!
Still thrilled to see this one in, we need far more Event Posts.

@datitisev As far as I can see there was never a PR for flarum-ext-english to add the translations for this. Could you please open one?

@tobscure Hm, yeah, I don't think I ever made a PR
And I actually designed those notifications wrong, regarding the translator.
I'll make PR for that.

See #20 & flarum/flarum-ext-english#131.

Thanks!


super-slicer-beta: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 269969 run on x86_64-linux 1

2 packages built:

super-slicer-beta
super-slicer-beta.debug

@ofborg eval


Feature request: disable opening browser to PR with av stack submit

If I have several stacked branches, I'd like to be able to pass a flag to av stack submit so that it doesn't open a tab to each PR for every single branch.

We might want to revisit whether the functionality of opening the browser by default. We can just print a link and most terminals these days will linkify it.
In the mean time, you can disable this functionality by setting pullRequest.openBrowser to false in the config file: https://docs.aviator.co/reference/aviator-cli/configuration.


Req: Method that returns local hostname

There's some nice DNS wrappers for gethostname and getaddrinfo, but I'm not seeing a good way of getting the hostname of the local machine.  If that were made available through libuv, I'd use it and get rid of another dependency in my projects.

You mean as a wrapper for gethostname(3)? I think I'd like that.

You could lift the implementation from node.js, it's the GetHostname() function in src/node_os.cc.

@jdonaldson are you planning to work on this? If not, I'd like to take a shot.

Sorry, on vacation here last few days.  By all means give it a shot, I won't have a good chance to work on this for some time.


Velocity calculation doesn't take parallax layers into account

Placing a rope on a parallax layer and moving the camera causes a movement in the rope when motion_scale on the ParallaxLayer is set to anything other than Vector2.ONE.
This feels like incorrect behaviour.
Had a quick look at the code but couldn't see a fix.

Technically it is correct behavior, because the node is moved and the rope inherently reacts to movement.
It's beyond the scope of this plugin to check if the rope is placed in a parallax layer.
You can tick the pause property on the rope object to disable simulation.

Thanks for the information.
I'm using the rope decoratively for an object on a parallax layer.
Hence, I don't want the rope to react simply while the camera is moving.
However, I do still need the rope to simulate when the object is moved. Also, I can't just pause it while the camera moves, as the object can be dragged at the same time the camera is moving.
I'll have a look to see if it's as simple as checking if the rope is on a parallax layer, and then applying the basis of the motion scale.

I see. Luckily I just remembered there is a simple workaround.
You can use a RopeRendererLine2D for rendering, if you are not already using it.
Place it in the parallax layer and the rope node outside the parallax layer.
This way the rope is rendering is affected by the parallax layer but not the rope itself. And you can still interact with the rope.
An alternative solution would be to use a CanvasLayer with follow_viewport_scale instead of a ParallaxLayer. This works because it does not modify the node's position.
However, it performs perspective scaling which might be undesired.

I'll have a look to see if it's as simple as checking if the rope is on a parallax layer, and then applying the basis of the motion scale.

It's not as simple unfortunately. In order to cancel the movement, the position difference of the parallax layer needs to be tracked. Then this difference needs to be added to all rope points, not just the root point, so the rope is moved uniformly. Additionally, the old points array needs to be updated accordingly as well, so the position shift is not interpreted as velocity.
Easy so far.
Problems arise with RopeHandles. These are also affected by the parallax layer and update the corresponding rope points to their new position. Now if we apply the shift from earlier, these points will get moved double the distance.
That means, handles need to be aware of parallax layers as well.
As you see there are quite some interactions going on that need to be handled properly.
Also a generic solution would be nice, since this problem can also be transferred to e.g. an elevator-like situation, where the rope is part of a moving entity but should not be affected by the outer movement.
While not impossible, this will require quite some time and effort to implement in a clean and reliable manner.
So, ideally we can make the workarounds work for you.

I assume this is fixed, otherwise let me know.


Issue with SwitchLBRule and DNAT for connectivity from external to service within VPC

Kube-OVN Version
v1.12.18
Kubernetes Version
Client Version: v1.28.10
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.10
Operation-system/Kernel Version
Ubuntu 22.04.4 LTS
6.5.0-1023-azure
kubespray cluster deployed to an Azure VM.
Description
Dear Community,
I am trying to create a DNAT rule to allow external connections to reach Pods created by a deployment within a VPC. According to the docs (and other older issues, e.g. https://github.com/kubeovn/kube-ovn/issues/3130 or https://github.com/kubeovn/kube-ovn/issues/2636), I created a Vip, a SwitchLBRule, an EIP as well as a DnatRule, but still did not manage to connect to the service from outside.
(My base setup is the one described under https://github.com/kubeovn/kube-ovn/issues/4259#issue-2392733349)
Some questions to help me troubleshooting this issue:

Is the Vip resource necessary or not? There is no mention of it under https://kubeovn.github.io/docs/stable/en/advance/vpc-internal-lb/, but only under https://kubeovn.github.io/docs/stable/en/advance/vip/
Am I supposed to use an IP from the service-cluster-ip-range as Vip, or can it also be an IP from the internal VPC range? I initially tried with the internal VPC range, but I was even not able to access the VIP from within the VPC, so I switched to an IP from the service-cluster-ip-range
Based off https://github.com/kubeovn/kube-ovn/issues/2636, I understand that I may be missing some routes, but I don't really know which ones
I am currently using Iptables-*-based resources, but I have also seen documentation about Ovn-*-based resources (https://kubeovn.github.io/docs/stable/en/advance/ovn-eip-fip-snat/#4-ovn-dnat), so not sure which ones to use

I would be happy to provide further details if needed.
Thanks in advance for your help!
Steps To Reproduce
Please note:

I'm running kubespray with service-cluster-ip-range: "<IP_ADDRESS>/18"
My base setup is the one described under https://github.com/kubeovn/kube-ovn/issues/4259#issue-2392733349

Steps:

Create the following resources:

---
apiVersion: kubeovn.io/v1
kind: Vip
metadata:
  name: test-ovn-arp-proxy-vip-lb-http
spec:
  namespace: test-ovn
  subnet: test-ovn-internal-subnet
  v4ip: <IP_ADDRESS>
  type: switch_lb_vip
---
apiVersion: kubeovn.io/v1
kind: SwitchLBRule
metadata:
  name: test-ovn-incoming-lb-http
spec:
  vip: <IP_ADDRESS>
  sessionAffinity: ClientIP
  namespace: test-ovn
  selector:
    - app:nginx
  ports:
  - name: http
    port: 80
    targetPort: 80
    protocol: TCP
---
kind: IptablesEIP
apiVersion: kubeovn.io/v1
metadata:
  name: test-ovn-incoming-ip-http
spec:
  natGwDp: test-ovn-gw
  v4ip: <IP_ADDRESS>
  externalSubnet: ovn-vpc-external-network
---
kind: IptablesDnatRule
apiVersion: kubeovn.io/v1
metadata:
  name: test-ovn-incoming-dnat-http
spec:
  eip: test-ovn-incoming-ip-http
  externalPort: '80'
  internalIp: <IP_ADDRESS>
  internalPort: '80'
  protocol: tcp

Try to access the service from outside:

$ curl -v http://<IP_ADDRESS>
*   Trying <IP_ADDRESS>:80...
* connect to <IP_ADDRESS> port 80 failed: Connection timed out
* Failed to connect to <IP_ADDRESS> port 80 after 134683 ms: Connection timed out
* Closing connection 0
curl: (28) Failed to connect to <IP_ADDRESS> port 80 after 134683 ms: Connection timed out

Access from a Pod within the VPC is working, through Service IP:

$ curl http://<IP_ADDRESS>
Welcome to nginx!

Current Behavior
Access to VIP within a VPC through VPC Gateway / DNAT not working
Expected Behavior
Access working :-)

about this case:
how about using loadbalancer instead of SwitchLBRule and DNAT?
@hongzhen-ma @tgdfool2

about this case: how about using loadbalancer instead of SwitchLBRule and DNAT? @hongzhen-ma @tgdfool2

Any example of such a config?

@bobz965: you mean this I assume? https://kube-ovn.readthedocs.io/zh-cn/latest/en/guide/loadbalancer-service/
The issue is that it only supports services in the default VPC. In our case, we would have several VPCs on our cluster, and all of them would have services that need to reachable.
So maybe you have an idea on how to make DNAT rules to work? You mentioned some missing routes, but how can I find which ones?

@bobz965: you mean this I assume? https://kube-ovn.readthedocs.io/zh-cn/latest/en/guide/loadbalancer-service/

yes.
@hongzhen-ma can this lb support  custom vpc?

In the end, this would be the setup that we would like to achieve:

Would be great if you could give us some hints on how to achieve this :-)
Thanks in advance for your support!

Other question: is this expected that the NAT Gateway is not able to ping the Pods in the same VPC?
In my picture above, from the NAT Gateway:

Ping works to <IP_ADDRESS> (its own IP)
Ping doesn't work to <IP_ADDRESS> and <IP_ADDRESS> (IPs of other Pods in the VPC/Namespace)

Just wondering if this is by design or if something is wrong...

Regarding the missing routes with the SwitchLBRule setup, I did some further investigation, but sill with no luck. Here is my current setup:

With this setup:

Access from all Pods, including VPC Gateway to Nginx Pod (<IP_ADDRESS>:80) is working
Access from Test Pod Alpine (<IP_ADDRESS>) to SwitchLBRule VIP (<IP_ADDRESS>) is working
Access from VPC Gateway to SwitchLBRule VIP (<IP_ADDRESS>) is NOT working

Remarks:

Route table of the VPC Gateway:Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
<IP_ADDRESS>         <IP_ADDRESS>      <IP_ADDRESS>         UG    0      0        0 net1
<IP_ADDRESS>       <IP_ADDRESS>     <IP_ADDRESS>     UG    0      0        0 eth0
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS>     U     0      0        0 eth0
<IP_ADDRESS>      <IP_ADDRESS>         <IP_ADDRESS>     U     0      0        0 net1

Currently no static route configured on the VPC

My assumption is, prior to configure the DNAT rule to allow external connections to the SwitchLBRule VIP, we need to ensure that the VPC Gateway can access the VIP. Is this correct? If so, then what I am missing to get this connection to work?
Thanks in advance for your support!

you can try use lb svc， if have any bug. please post .
I think it could be used in custom vpc.

Using switch lb rule with nat gw is much more complex than LB service.

Thanks. Will give a try and let you know the outcome.

@bobz965
The following Service has been deployed:
---
apiVersion: v1
kind: Service
metadata:
  annotations:
    lb-svc-attachment.kube-system.kubernetes.io/logical_switch: ovn-vpc-external-network
    ovn.kubernetes.io/attachmentprovider: ovn-vpc-external-network.kube-system
  name: test-clearml-ttr-svc
  namespace: test-clearml
spec:
  loadBalancerIP: <IP_ADDRESS>
  ports:
  - name: http
    protocol: TCP
    port: 80
    targetPort: 80
  selector:
    app: nginx
  sessionAffinity: None
  type: LoadBalancer

Which resulted in the following error in the controller Pod:
W0712 12:56:28.091566       7 service.go:495] pod for lb svc test-ovn/test-ovn-ttr-svc is not running: pod of deployment test-ovn/lb-svc-test-ovn-ttr-svc not found
I0712 12:56:29.190111       7 service_lb.go:311] add eip rules for lb svc pod, [<IP_ADDRESS>/24,<IP_ADDRESS>]
E0712 12:56:32.295601       7 service_lb.go:271] failed to ExecuteCommandInContainer, errOutput: failed to exec command "arping -c 3 -s <IP_ADDRESS> <IP_ADDRESS>"
E0712 12:56:32.295834       7 service_lb.go:276] command terminated with exit code 1
E0712 12:56:32.295854       7 service_lb.go:313] failed to add eip for pod, err: command terminated with exit code 1
E0712 12:56:32.295868       7 service.go:529] failed to update pod attachment network for service test-ovn/test-ovn-ttr-svc: command terminated with exit code 1
E0712 12:56:32.295942       7 service.go:166] error syncing 'test-ovn/test-ovn-ttr-svc': command terminated with exit code 1, requeuing

Could this be related to the fact that this infrastructure is running in Azure, with ipvlan and not macvlan? Do you see a way to overcome this?

run arping -c 3 -s <IP_ADDRESS> <IP_ADDRESS> manually in vpc nat gw pod, and see the err details.

Output:
# arping -c 3 -s <IP_ADDRESS> <IP_ADDRESS>
Interface "net1" is not ARPable

do you have iptables-eip?

do you have iptables-eip?

if you do not have eip, the net1 arp is off.

We do have an EIP, but we had to disable ARP on net1 due to the following limitation: https://github.com/kubeovn/kube-ovn/issues/4259#issuecomment-2211161528

And after re-enabling ARP on net1, we get the following:
# arping -c 3 -s <IP_ADDRESS> <IP_ADDRESS>
ARPING <IP_ADDRESS> from <IP_ADDRESS> net1
Sent 3 probes (3 broadcast(s))
Received 0 response(s)

(<IP_ADDRESS> is the default gw of our Azure subnet)

if you has a iptables-eip, and net1 <IP_ADDRESS> default routes is already exist, you should turn arp on

Yes, that's what we tried to do but unfortunately, as soon as ARP gets turned on, the NAT GW loses connectivity with the other Azure VMs in the same subnet. Probably something to do with Azure networking.

sorry, I'm not familiar with Azure.

In the end, this would be the setup that we would like to achieve:
Would be great if you could give us some hints on how to achieve this :-)
Thanks in advance for your support!

Hi kube-ovn experts,
it looks like the lb svc in the default VPC  only support external user to access those SVC in the default VPC.
how external user to access the SVC which created in their private VPC?
can we support it?
many thanks

the lb svc is switch lb rule, and the switch is isolated by vpc.
so the lb svc is isolated by vpc.

the lb svc is switch lb rule, and the switch is isolated by vpc.
so the lb svc is isolated by vpc.

yes, can we have any mechanism to expose these types of SVC (defined in user private VPC) to public network? we do have such a requirement.

there is a loadbalancer svc mechanism in kube-ovn, you can try.

there is a loadbalancer svc mechanism in kube-ovn, you can try.

really appreciate your quick reply. actually, I have tried it and it works. but it need manually updated the VPC GW and it not convenient.
what I mean is that can we add the support on kube-ovn side?

do you mean in ovn way instead of in the way of vpc nat gw?

each way should be OK  to support exposing SVC to public network in my opinion, :)

ovn dnat is based on ovn router lb, you can try to make a lb feature like ovn dnat based on ovn lb.

ovn dnat is based on ovn router lb, you can try to make a lb feature like ovn dnat based on ovn router lb.

thanks, as kube-ovn's code is very complicate, could you kubeovn experts do us a favor?

ovn dnat is based on ovn router lb, you can try to make a lb feature like ovn dnat based on ovn router lb.

thanks, as kube-ovn's code is very complicate, could you kubeovn experts do us a favor?

next month, maybe I will have time to establish a ovn svc


ignore .dSYM along with .ipa files

Reasons for making this change:
Often .dSYM and .dSYM files stored with .ipa files, so they should also be ignored if .ipa ignored.
Links to documentation supporting these rule changes:
Not clear what should be here.
You can get more info what is .dSYM file here

@evfemist that SO link helps, thanks!


Ability to edit SSH address in "Clone this repository"

It's already possible to change the URL of the website, but it would be nice to be able to change the SSH address too
For example I access my server through a VPN so my repo is not reachable with ssh://git@localhost:22/admin/repo1.git

Doesn't this work for you?
[server]
DOMAIN       = example.com
HTTP_PORT    = 3000
ROOT_URL     = http://example.com

in app.ini file

Hi, maybe helpful: https://discuss.gogs.io/t/how-to-config-ssh-settings/34


[RTM] Pin new nipype - updates ReconAll response to existing subjects

(Minor) unexpected behavior: stats and labels directories were getting updated by repeat runs of ReconAll.
Once nipy/nipype#1824 is merged, will update pin and mark [RTM].

Rebased, but should wait on Circle. If it errors, we'll know that there's an upstream bug that needs dealing with.

Should be good to go.


Update SNOW_SHOVEL.json

fine jani ill fix it

That’s the cost to place the snow and not the cost of the item, it should be there.

ty <3

wait 4real?

wait 4real?

Yes, check it at the dude on top of the mountain

Clown Moment from me


HHH-8529 - Test @ElementCollection Map<K, V> properties with convertible keys and @Enumerated and @Temporal values.

This is only a test. I'll describe what need to be fixed on the JIRA issue.

Issue fixed (?). All tests in hibernate-core and hibernate-entitymanager are passing.
I just signed CLA.

Linked this PR at https://hibernate.atlassian.net/browse/HHH-8529

Sadly, this branch is too old. I will create new PR.


TrailUrl Search terms should be persisted

Forensic currently calculates the searchTerms for a url each time it runs.  Instead we need to determine and persist the search terms when the url is added during trailing.
I've added a searchTerms property to the dw-trail-url.json model but nothing currently populates it.

Completed and submitted pull request

Tested.  Looks good with the exception of Google and Wikipedia


AudioStreamPlayer2D starts playback with a noticeable delay

Tested versions
Reproducible in: v4.2.2.stable.mono.official [15073afe3]
System information
Godot v4.2.2.stable.mono - macOS 14.3.0 - Vulkan (Forward+) - integrated Apple M3 Max - Apple M3 Max (14 Threads)
Issue description
When I use AudioStreamPlayer2D to play a sound, the sound can be heard around 110ms after the keypress. This is a noticeable delay, and a player action such as firing a gun or pressing a button doesn't quite feel "immediate".
The sound sample I'm using does not have any silence at the beginning.
To measure the delay, I recorded an audio track with Audacity. I can clearly hear both the click of my keyboard and the sound being played, and also see the waveforms of both events in the audio track, so the delay is easy to calculate in Audacity's UI.
I measured this delay on three systems:

MacBook Pro / M3 Max, macOS 14, Studio Display speakers
Mac mini / M2 Pro, macOS 14, built-in speakers
A recent gaming PC / Windows 11, wired headphones, onboard sound card

I measured a delay of around 110ms on both macs, and a delay of 160ms on the PC.
Is this something that can be improved, either in the engine, or in my game? Is the delay caused by my systems? Or is such a delay expected in Godot?
To get a few reference points, I did some research and an "Input lag" commonly refers to the latency between the physical input and the display reacting to that input. 60ms is generally considered pretty good, but this includes the response time from the display hardware itself.
In my small Godot test project, I not only play a sound but also toggle the visibility of a sprite on screen. I recorded a video at 60 fps showing how I press the button on a keyboard, and the sprite appears on the screen after 4 frames on each of the three systems, that is within 67ms.
I also wrote a small command line tool that uses Apple's AVFoundation framework to play the same sound file when I press a key. The delay in that tool is around 70ms. This feels  more "immediate" than a delay of 110ms or even 160ms.
I don't know much about the signal chain and at which points in the chain various delays occur and their typical magnitudes, so I'm wondering if it's reasonable or not to expect faster response times. My naive expectation would be:

Around 30ms latency from the keyboard
Up to 16.6ms latency for the next physics frame update in Godot
Up to 15ms latency for Godot's audio output latency (default value)

So around 60ms in total.
As per danluu's keyboard latency test (https://danluu.com/keyboard-latency/), Apple's keyboards are pretty fast. I'm using an Apple USB keyboard on my gaming PC and the builtin keyboard on my MacBook, so that should probably not be an issue.
Steps to reproduce
This is how I play the sound in GDScript:
extends Node2D

var sound = preload("res://sound.wav")
var player
@export var sprite: Sprite2D

func _ready():
	player = AudioStreamPlayer2D.new()
	player.stream = sound
	add_child(player)

func _physics_process(delta):
	# Assigned to 'M' key
	if Input.is_action_just_pressed("tap"):
		player.play()
		sprite.visible = !sprite.visible

See also the attached example project.
Minimal reproduction project (MRP)
InputLatencyTest.zip

Try adjusting the Audio > Driver > Output Latency project setting. It defaults to 15 so there's probably not much to gain here, but it's still worth trying lower values just in case.
Note that Godot currently doesn't expose the audio buffer length, but the current hardcoded value of 512 audio frames is already quite low.
Also try adjusting Audio > Driver > Mix Rate and see if using 48000 instead of 44100 reduces latency.

 Over the past days I did several tests and a long series of measurements to get a better understanding of this issue.
The most important result for me is that with the Godot test script I managed to get an input-to-sound latency as low as 70 ms on Windows, which is very good.
Other takeaways:

I found a setting on Windows named "3D Spatial Sound" that I had tried out a while ago. Once I disabled that, the latency improved in general by around 50 ms on that system.
Changing the "Output Latency" from 15 ms to 5 ms in Godot's project settings reduces the measured latency by an additional 10 ms.
As a rough benchmark, the latencies I measured in various games vary between 100 ms and 200 ms.
Different input devices have different latencies. In the test script, for example, I get the mentioned latency of 70 ms for a mouse click input and 110 ms latency for a key press (both on Windows; original test script without the suggested changes).

For the sake of completeness, here are the specs of my Gaming PC:

CPU: AMD Ryzen 7 7800X3D 4.2 GHz 8-Core
GPU: GeForce RTX 4090, 24 GB
Memory: 64 GB (2 x 32 GB) DDR5-6000 CL40
SSD: Samsung 980 Pro 2 TB
OS: Windows 11
Mouse: Razer Basilisk, RZ01-02330, USB
Keyboard: Apple Keyboard, A1243, USB
Headphones: Wired / 3.5 mm headphone jack

I personally find that while 110ms is an acceptable delay for action games, 70 ms feels very different. With such a short delay, I have this visceral feeling as if I'm physically causing the sound with my finger press. This is not the case for the 110 ms delay, where the effect feels more "detached" from the action. It would be great to find a way to get the delays always that short for any input in Godot, on both macOS and Windows.
As the delay is already at 70 ms for mouse input on Windows, it looks to me like larger latencies (for keyboard input or on macOS) aren't related to Godot's audio engine. They're probably caused by something earlier in the signal chain.
Here's the small command line tool that I wrote for macOS and which plays back sounds after a key press with a 60-70 ms delay. When running the tool, the sound file sound.wav is expected to be in the same directory as the tool. To build the tool, create a new Command Line Tool in Xcode and paste the code into main.swift.
TestToolMac.zip


Add Flo in front of Runner

Hey, I just made a Pull Request!
Description
Add Flo in front of Runner when logging the flo startup message
Motivation and Context
Make it easier to grep for and detect flo in logs.
Have you tested this? If so, how?

Checklist for PR author(s)

[ ] Changes are covered by unit test
[ ] All tests pass
[ ] Code coverage check passes
[ ] Error handling is tested
[ ] Errors are handled at the appropriate layer
[ ] Errors that cannot be handled where they occur are propagated
[ ] Relevant documentation updated
[ ] This PR has NO breaking change to public API
[ ] This PR has breaking change to public API and it is documented

Checklist for PR reviewer(s)

[ ] This PR has been incorporated in release note for the coming version
[ ] Risky changes introduced by this PR have been all considered

👍


Add a note in the documentation warning not to use data-cart-submit on checkout form

I was implementing CartJS and stumbled significantly when trying to get my cart / checkout form to work properly because I had data-cart-submit="data-cart-submit" which was causing the form request to fail.
Would be nice to include a disclaimer for folks that had the same issue as me : )

@pjatx thanks - have added a note in the documentation on this.


Fixed a few mistakes in the readme

Fixed a few minor spelling errors within the readme. Also reworked a few sentences to hopefully make them flow a bit better.

Thank you @lbogerd!


Incorrect parse on static property followed by method

// @target: esnext
// @noImplicitAny: false

class C {
    static static
    foo() {}
}

Here, ASI needs to take effect and the parse should be a class with two members: a static property named "static", and an instance method named "foo".
Expected
No errors. The file is parsed as
class C {
    static static;
    foo() {}
}

Actual
'static' modifier already seen.

class C {     static static;     foo() {} }
?

@DanielRosenwasser There is a similar test for the index signature
https://github.com/microsoft/TypeScript/blob/f68a8393ef51d1c8b797cec440ba82201d0c83e9/tests/cases/compiler/staticAsIdentifier.ts#L1-L4
Should it be parsed similarly to your example?
class C {
    static static;
    [x: string]: string;
}


⚠ Bofrost.at has degraded performance

In 392ed4f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 997 ms

Resolved: Bofrost.at performance has improved in 92ace3a.


No podman in compilers download

Podman/Docker don't work well in docker-on-macos for whatever reason, leading to being unable to download the compilers.
Downloading the images manually is easy enough to do, the OCI Distribution specification is easy enough to follow (see this.

This is great! I think you can also remove the docker/podman requirements from the poetry file :+1:

This is great! I think you can also remove the docker/podman requirements from the poetry file 👍

Done. The first commit now bumps poetry to 1.6.1 (without any other changes), while the second commit removes the docker/podman dependencies (along with implementing the raw download)

Yeh I think that's ok, these particular images have linux/darwin flavours, but by default we lookup the "non-arch name" image

I'll try to silence the error if we have an os-specific flavor for the image, as I feel it may be a bit misleading.
So left on my TODO:

Silence the warning
Figure out what the fudge is going on in CI (see https://github.com/decompme/decomp.me/actions/runs/6372041695/job/17294505786?pr=874 - I'll probably need to ssh into the runner to figure this out)

I'll try to do that tomorrow.

I'll suggest a change when I'm at my computer, or if you look at what I did for the podman pr, just undo the change:)


WebDriver IO and Draft JS

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
It does not apply the right value on html, when I use this code
const draft = browser.element('.public-DraftEditor-content') }
draft.setValue('Test')

What is the expected behavior?
the value 'Test' appears on html.
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in
"draft-js": "^0.10.0", Version 65.0.3325.181 (Official Version) 64 bits, MAC OS Version 10.13.3
previous versions of Draft.js?
No

Hey @HelderSprinklr, I doubt setValue is the right command when dealing with contenteditable HTML. In any case, you should probably have a look at #1460.

@thibaudcolas, thank you! 👍


Extend bf utility to support multiple payloads

The bf C++ example is a handy utility to work with files containing
arbitrary Bond payloads. This change extends it to support files with
more than one Bond payload.
The most obvious use case is files with a sequence of records (e.g.
logs). The less obvious but very powerful scenario is extracting some
kind of header that precedes the actual Bond payload of interest. This
is possible because a Bond schema in Simple Binary protocol can be used
to model many kinds of arbitrary headers (e.g. any fixed size header
aligned to octet boundary).
The change is backward compatible and existing command line arguments
retain their old semantics. In order to process multiple payloads user
can specified multiple --schema and/or multiple --from arguments,
e.g.:
bf --from=simple --schema=header.json,payload.json file

In the above example bf will first use Simple Binary to decode header
in schema specified in header.json and then will try to guess the
protocol of the next payload (since only one --from argument was
specified) and decode it using schema specified by the payload.json
file.
Multiple values for the --schema and --from arguments can be
specified either as comma delimited values (like in the example above)
or by passing the argument multiple times, e.g.:
bf --from=fast --from=fast file

@chwarr, make the changes you requested.