Provide get_groups function in apply block

It seems there's no way to find out the parent groups of the current target (as defined in the inventory.yaml). That'd be really helpful information, for example how to lookup data in Hiera:
version: 5

hierarchy:
  - name: "Per-node data"
    path: "nodes/%{trusted.certname}.yaml"

  - name: "Per-groups data"
    mapped_paths: [groups, tmp, "groups/%{tmp}.yaml"]

  - name: "Per-OS defaults"
    path: "os/%{facts.os.family}.yaml"

  - name: "Common data"
    globs: ["common/**", "common.yaml"]

My current approach is to define them manually in my deploy plan, see https://github.com/puppetlabs/bolt/issues/1765#issuecomment-1015116707
Having a $target.get_groups() function available in the apply block would solve this issue.

It would be even better to have all of a target's groups as a top level variable so it can be used in Hiera hierarchy as well.


Wrong ticker setup

In WeatherStationDemoExtendedDST The Ticker object appears to be used incorrectly. Upon inspecting the library, it appears that each function requires a separate Ticker object. You had one Ticker object, and was setting it to setReadyForWeatherUpdate, and then resetting it to setReadyForDHTUpdate. After adding an additional Ticker object, e.g. ticker1 and ticker2 instead of just ticker, it updates both correctly. Hope this helps, you probably already fixed it.
...
Ticker ticker1;
Ticker ticker2;
...
ticker1.attach(SET_INTERVAL_SECS, setReadyForWeatherUpdate);
ticker2.attach(60, setReadyForDHTUpdate);
...
Please send me your email, I don't have it set up on linux (I just use a browser), and it keeps trying to set it up when I try to email you. I won't bug you. BTW using pin D4 for DHT was confusing when I first tried to use my oled soldered directly to the D1 mini (nice trick). It sort of worked for a while before dying (with no DHT attached) was the weird thing. Of course when I changed DHT to D6, and uncommented out the 2 lines to power up D4, it worked correctly. I also have PST/PDT (Los_Angeles) code if you want.
-- Chris H.

Here is more information:
WeatherStationDemoExtendedDST
thingspeak metric conversion in function drawThingspeak(), so it displays F or C properly based on IS_METRIC:
display->drawString(64 + x, 12 + y, (IS_METRIC ? thingspeak.getFieldValue(0) : String((thingspeak.getFieldValue(0).toFloat()*9/5)+32)) + (IS_METRIC ? "°C" : "°F"));
instead of:
display->drawString(64 + x, 12 + y,  thingspeak.getFieldValue(0) + "°C");

Los Angeles Pacific Time Zone for settings :
#define Los_Angleles
...
#ifdef Los_Angeles
//DST rules for US Pacific Time Zone (Los Angeles, San Diego)
#define UTC_OFFSET -8
struct dstRule StartRule = {"PDT", Second, Sun, Mar, 2, 3600};    // Pacific Daylight time = UTC/GMT -7 hours
struct dstRule EndRule = {"PST", First, Sun, Nov, 1, 0};               // Pacific Standard time = UTC/GMT -8 hour
// Uncomment for 24 Hour style clock
//#define STYLE_24HR
#define NTP_SERVERS "us.pool.ntp.org", "time.nist.gov", "pool.ntp.org"
// Wunderground Settings
const boolean IS_METRIC = false;
const String WUNDERGRROUND_API_KEY = "";
const String WUNDERGRROUND_LANGUAGE = "EN";
const String WUNDERGROUND_COUNTRY = "CA";
const String WUNDERGROUND_CITY = "Los_Angeles";        // for city you can use Postal codes also, e.g. "92127" or "92127-1439" to get more local weather
#endif
I really like this particular weather demo, and it works good now!
Thank You!
-- Chris H.


Machine status marked as 'Timeout' after host reboot with running containers

If the host machine is shutdown with any containers running the Docker machine will have a status of 'timeout' when the host is rebooted. This appears to be irreversible. The only solution I've found is to destroy the Docker machine and recreate it from scratch.
Related? Machine status marked as 'Timeout' after host suspend ended with Virtualbox
Example
docker-machine ls
NAME      ACTIVE   DRIVER       STATE     URL   SWARM   DOCKER   ERRORS
default            virtualbox   Timeout

Environment Info
uname -a                 Darwin ColeyMBPR 15.4.0 Darwin Kernel Version 15.4.0: Fri Feb 26 22:08:05 PST 2016; root:xnu-3248.40.184~3/RELEASE_X86_64 x86_64
docker version           1.11.0
docker-machine version   0.7.0
vboxmanage --version     5.0.20r106931

Attempted Solutions
I've tried the following things in no particular order:

Restarting the docker machine.
Running eval "$(docker-machine env default)"
Regenerating the certificates docker-machine regenerate-certs default
Restarting my host box.
Upgrading Docker.
Reinstalling Docker.
Upgrading VirtualBox.
Removing all VirtualBox host-only network devices.

Hack
The only thing that's working for me right now is destroying the docker machine and recreating it. This destroys all my images and containers, and it's incredibly time consuming to set it up again.
docker-machine rm -y default && docker-machine create -d virtualbox default && eval $(docker-machine env)

Steps to Reproduce
I can reliably reproduce this problem with the following steps:

Start with a freshly created Docker machine.
Use docker-compose up to build some containers.
Shutdown the computer with the containers running.
After reboot the Docker CLI doesn't work due to the Docker machine timing out.

Now I've discovered what's causing the problem I'll make sure I kill all containers at the end of the day and see if the problem is resolved.

I am having the same issue with machines created with other drivers than virtualbox :S. There is also a question about it on stackoverflow
I was once able to get it back working but I am not able to reproduce. I tried everything and after a host reboot it worked again.
It is important to add that ssh is possible, even after cert renewal.

Upgrading to Docker Beta solved this problem for me.

On 24 May 2016, at 17:20, jdcaballerov<EMAIL_ADDRESS>wrote:
I am having the same issue with machines created with other drivers than virtualbox :S. There is also a question about it on stackoverflow https://stackoverflow.com/questions/37069718/docker-machine-timeout-how-to-fix-without-destroying-the-machine
I was once able to get it back working but I am not able to reproduce. I tried everything and after a host reboot it worked again.
It is important to add that ssh is possible, even after cert renewal.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/docker/machine/issues/3418#issuecomment-221325177

I got the same error message when connected with my company's VPN,
and resolved it by disconnecting the VPN.
Error message:
[sso@MBP ~]$ docker-machine ls
NAME      ACTIVE   DRIVER       STATE     URL   SWARM   DOCKER   ERRORS
default            virtualbox   Timeout
[sso@MBP ~]$

[sso@MBP ~]$ docker-machine regenerate-certs default
Regenerate TLS machine certs?  Warning: this is irreversible. (y/n): y
Regenerating TLS certificates
Waiting for SSH to be available...
Detecting the provisioner...
Copying certs to the local machine directory...
Copying certs to the remote machine...
Setting Docker configuration on the remote daemon...

This machine has been allocated an IP address, but Docker Machine could not
reach it successfully.

SSH for the machine should still work, but connecting to exposed ports, such as
the Docker daemon port (usually <ip>:2376), may not work properly.

You may need to add the route manually, or use another related workaround.

This could be due to a VPN, proxy, or host file configuration issue.

You also might want to clear any VirtualBox host only interfaces you are not using.
[sso@MBP ~]$

Same issue here.
uname -a                 Linux lapland 3.19.0-65-generic #73~14.04.1-Ubuntu SMP Wed Jun 29 21:05:22 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Docker version           1.11.2, build b9f10c9
docker-machine version   0.7.0, build a650a40
vboxmanage --version     5.1.0r108711

@zz22394 This definitely a VPN problem. I executed both the docker-machine regenerate-certs and create commands without the VPN being on and it worked.

same here
Version 1.13.1 (15353)
Channel: Stable
94675c5a76

Proxifier would cause the same problem. Try to create a new direct rule will solve it.

Not on a VPN and I'm seeing this too. I'm using https://github.com/machine-drivers/docker-machine-driver-xhyve
I had to create a plist separate from the homebrew one to get it to work. I installed docker-machine through brew. Note, I'm using UserName and it doesn't work without it (homebrew doesn't have that).
cat > ~/Library/LaunchAgents/cicd.start-docker-machine.plist <<EOD
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
    <dict>
        <key>EnvironmentVariables</key>
        <dict>
            <key>PATH</key>
            <string>/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin</string>
        </dict>
        <key>Label</key>
        <string>cicd.start-docker-machine</string>
        <key>ProgramArguments</key>
        <array>
            <string>/usr/local/bin/docker-machine</string>
            <string>start</string>
            <string>default</string>
        </array>
        <key>UserName</key>
        <string>cicd</string>
        <key>RunAtLoad</key>
        <true/>
    </dict>
</plist>

then chown cicd:cicd ~/Library/LaunchAgents/cicd.start-docker-machine.plist


react-signals with next.js 14

Just curious if anyone has successfully gotten signals to work with next.js 14? I get this error when I try:
Unhandled Runtime Error
TypeError: Cannot read properties of undefined (reading 'length')
Call Stack
areHookInputsEqual
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (10921:0)
updateEffectImpl
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (12303:0)
updateEffect
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (12323:0)
updateSyncExternalStore
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (11802:0)
Object.useSyncExternalStore
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (13448:0)
useSyncExternalStore
node_modules/next/dist/compiled/react/cjs/react.development.js (1797:0)
P
<EMAIL_ADDRESS>(1:2148)
Object.set [as current]
<EMAIL_ADDRESS>(1:2976)
renderWithHooks
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (10968:0)
updateFunctionComponent
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (16163:0)
beginWork$1
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (18359:0)
beginWork
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (26741:0)
performUnitOfWork
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (25587:0)
workLoopConcurrent
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (25573:0)
renderRootConcurrent
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (25529:0)
performConcurrentWorkOnRoot
node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js (24382:0)
workLoop
node_modules/next/dist/compiled/scheduler/cjs/scheduler.development.js (261:0)
flushWork
node_modules/next/dist/compiled/scheduler/cjs/scheduler.development.js (230:0)
MessagePort.performWorkUntilDeadline
node_modules/next/dist/compiled/scheduler/cjs/scheduler.development.js (534:0)

Automatic tracking for now is not working with next.js. You can use signals with my lib to allow signals tracking for components. I also working on swc transform plugin to allow use signals in next.js https://github.com/XantreGodlike/preact-signals/pull/45

The new 2.0.0 version of @preact/signals-react provides a useSignals() hook that you can place in any component that reads a signal's value. It appears to work with Nextjs 14+ for me.

I've implemented swc plugin how to use
#297
https://codesandbox.io/p/github/XantreGodlike/preact-signals-nextjs/main

The @preact/signals-react documentation mention two ways:

Adding a babel transform - didn't work for me in Next 14.0.1

{
	"plugins": [["module:@preact/signals-react-transform"]]
}

call the useSignals() - work for me
hook to make your components reactive, but a bit tedious calling it in every component.

For next JS you need to use SWC plugin

The @preact/signals-react documentation mention two ways:

Adding a babel transform - didn't work for me in Next 14.0.1

{
	"plugins": [["module:@preact/signals-react-transform"]]
}

call the useSignals() - work for me
hook to make your components reactive, but a bit tedious calling it in every component.

Why not simply use the useSignal() hook to instantiate a signal

Consider a scenario below
<button onClick={() => count.value++}> Value: {count}, value x 2 = {double} </button>
The above has been referred from the NPM documentation but, small difference is they referred signals as count.value and double.value in the JSX. It didn't seem to update the same in UI.

Consider a scenario below
<button onClick={() => count.value++}> Value: {count}, value x 2 = {double} </button>
The above has been referred from the NPM documentation but, small difference is they referred signals as count.value and double.value in the JSX. It didn't seem to update the same in UI.
Not until I referred the signals as signal instead of signal.value in the JSX.

Reading signal .value field subscribes a component to it changes only if you are using babel plugin or used useSignals() on top of a component

So even when I am creating a signal using the useSignal() hook, still I have to manually call the useSignals()?

Consider a scenario below
<button onClick={() => count.value++}> Value: {count}, value x 2 = {double} </button>
The above has been referred from the NPM documentation but, small difference is they referred signals as count.value and double.value in the JSX. It didn't seem to update the same in UI.
Not until I referred the signals as signal instead of signal.value in the JSX.

Reading signal .value field subscribes a component to it changes only if you are using babel plugin or used useSignals() on top of a component

Also then how using only signal instead of signal.value works.

Consider a scenario below
<button onClick={() => count.value++}> Value: {count}, value x 2 = {double} </button>
The above has been referred from the NPM documentation but, small difference is they referred signals as count.value and double.value in the JSX. It didn't seem to update the same in UI.
Not until I referred the signals as signal instead of signal.value in the JSX.

Reading signal .value field subscribes a component to it changes only if you are using babel plugin or used useSignals() on top of a component

Also can you explain me the phrase subscribes a component to it/

So even when I am creating a signal using the useSignal() hook, still I have to manually call the useSignals()?

Conceptually react integration works like this. Every component wrapped with try finally to start tracking and finish it in finally block.
// start tracking reading of signals
const store = _useSignalsImplementation(1);
try {
  return renderYourComponent(props);
} finally {
  // stop tracking reading of signals
  store.f();
}

If you're using useSignals()
// start tracking and will stop tracking on new microtask
useSignals()
// code
return jsx

You can think about it like every component code wrapped in effect. If use are not using any react integration - there is no such effect

The @preact/signals-react documentation mention two ways:

Adding a babel transform - didn't work for me in Next 14.0.1

{
	"plugins": [["module:@preact/signals-react-transform"]]
}

call the useSignals() - work for me
hook to make your components reactive, but a bit tedious calling it in every component.

useSignal doesn't work for me, can't enter anything in the input field

It works, finally!

It works, finally!

Setup is almost correct. But you should import signal, computed, effect and other functions from @preact-signals/safe-react not @preact/signals-react (to avoid undefined behaviour and reduce bundle size)

When I use signal() and return signal.value, it never gets updated on further signal.value updates. The only way to show it and update it is to just return signal, but in that case, if I have a complex object, such as User, I can't just do signal.username, I must do signal.value.username (which would not work as I said above, and as a bonus, if you return signal.value.something on a object, it always shows undefined)

When I use signal() and return signal.value, it never gets updated on further signal.value updates. The only way to show it and update it is to just return signal, but in that case, if I have a complex object, such as User, I can't just do signal.username, I must do signal.value.username (which would not work as I said above, and as a bonus, if you return signal.value.something on a object, it always shows undefined)

Maybe you need valtio.

When I use signal() and return signal.value, it never gets updated on further signal.value updates. The only way to show it and update it is to just return signal, but in that case, if I have a complex object, such as User, I can't just do signal.username, I must do signal.value.username (which would not work as I said above, and as a bonus, if you return signal.value.something on a object, it always shows undefined)

If you need deep reactivity you can use @preact-signals/utils

@OpenSource03 Please provide an example.
That goes for everyone else too, without reproductions, we cannot help.
This thread has gotten pretty off topic so I'm going to close it out as answered.

I've implemented swc plugin how to use #297 https://codesandbox.io/p/github/XantreGodlike/preact-signals-nextjs/main

Work for me huge thanks :)

Dunno if it's compatible with preact but perhaps https://github.com/dai-shi/use-signals might work?
There was a nice writeup here too: https://thenewstack.io/did-signals-just-land-in-react/


Not able to Uninstall the program

I want to remove this program but it is not listed in the Control Panel.

You can't have tried too hard looking for the uninstaller.. Simply searching for Messenger within the start menu will link you to the uninstaller.

This should be resolved now. Please see https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/453


Listen and propagate RD_KAFKA_RESP_ERR__PARTITION_EOF

Sometimes it is nice to know that partition/topic was read to EOF and it is supported by librdkafka.
It should be explicitly enabled with property enable.partition.eof=true and error is handled, e.g.:
        for _ in 0..<maxEvents {
            let event = rd_kafka_queue_poll(self.queue, 0)
            defer { rd_kafka_event_destroy(event) }

            let rdEventType = rd_kafka_event_type(event)
            guard let eventType = RDKafkaEvent(rawValue: rdEventType) else {
                fatalError("Unsupported event type: \(rdEventType)")
            }

            switch eventType {
            case .error:
                let err = rd_kafka_event_error(event)
                if err == RD_KAFKA_RESP_ERR__PARTITION_EOF {
                    let topicPartition = rd_kafka_event_topic_partition(event)
                    if let topicPartition {
                    ... return events

Probably, it could be extended with current api e.g.:
public struct KafkaConsumerMessage {
    /// The topic that the message was received from.
    public var topic: String
    /// The partition that the message was received from.
    public var partition: KafkaPartition
    /// The key of the message.
    public var key: ByteBuffer?
    /// The body of the message.
    public var value: ByteBuffer
    /// The offset of the message in its partition.
    public var offset: KafkaOffset
    var eof: Bool {
        self.value.readableBytesView.isEmpty
    }

    /// Initialize ``KafkaConsumerMessage`` as EOF from `rd_kafka_topic_partition_t` pointer.
    /// - Throws: A ``KafkaError`` if the received message is an error message or malformed.
    internal init(topicPartitionPointer: UnsafePointer<rd_kafka_topic_partition_t>) {
        let topicPartition = topicPartitionPointer.pointee
        guard let topic = String(validatingUTF8: topicPartition.topic) else {
            fatalError("Received topic name that is non-valid UTF-8")
        }
        self.topic = topic
        self.partition = KafkaPartition(rawValue: Int(topicPartition.partition))
        self.offset = KafkaOffset(rawValue: Int(topicPartition.offset))
        self.value = ByteBuffer()
    }

or changed to enum:
enum KafkaConsumerMessage {
    case message(topic: String, partition: KafkaPartition, key: ByteBuffer?, value: ByteBuffer, offset: KafkaOffset)
    case eof(topic: String, partition: KafkaPartition, offset: KafkaOffset)
}

@felixschlegel Can you check this?

IMO we should make the KafkaConsumerMessages AsyncSequence throw when we encounter RD_KAFKA_RESP_ERR__PARTITION_EOF since this is an option that is explicitly set by the user.
Alternatively, we could emit this as an event in the KafkaConsumerEvents AsyncSequence though I am more inclined towards just throwing in KafkaConsumerMessages.
On that note: for this feature, we would also have to expose an option like isPartitionEOFEnabled to the KafkaConsumerConfiguration.

Can we ever recover from a partition EOF error? Like does this happen during rebalance or is this really a terminal state

Ah right, it is just when we hit the end of the partition and then we can still continue once more messages have been produced. So finishing the sequence is not the correct thing. I think the only thing we can really do is have an enum on the sequence. If we produce the EOF event into a separate sequence then there might be reordering problems when consuming both sequences where the EOF happens before the last message.
Overall, I am wondering how the consumption pattern look like and what you do when you hit EOF. @blindspotbounty could you provide some examples?

Hi @FranzBusch, thank you for looking into this case!
That would be great to have such enum in KafkaConsumerMessage to determine that partition is read up to the end.
The major thing for us is to determine that all messages are read after adding new or old service recovered/started successfully. Hitting partition EOF for all partitions for us means that now we are up to date and process real time data for partitions.
Other less important but still nice to have use-case is related to offset commits: usually commit every message is too slow, so we commit them time to time (after some number of entries or time passed). However, when there is no more messages we wait for next one. Partition EOF solves this as we know that no more messages will be received within partition and thus offset can be committed.
Currently we solve main problem by parsing statistics on partitions from librdkafka. That adds extra overhead from librdkafka to construct json and from our side to parse it. Additionally, it is less reliable because statistics is provided time to time, so with large message flow it can be hard to catch statistics with zero lag.
Looking forward to help/provide more details if needed!


Rename object store directories in parallel

In this PR:

Refactored DeleteBuffer to OperationBuffer to use for renaming large directories as well

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2574/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2579/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2580/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2581/
Test PASSed.

Thanks @LuQQiu. Please take another look.

@calvinjia please take a look. thx

Correct, the heavy lifting should be on the s3 side (using the Pit-Copy API https://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectCOPY.html).
Limiting concurrent renames has the downside of lower performance vs. higher resource consumption  with more concurrency

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2590/
Test PASSed.

@madanadit Thanks for the update. The only outstanding concern I have is around this threadpool overdrawing from the s3a threadpool. Could we enforce the values to be reasonable or base one value off the other?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2591/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2605/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2606/
Test PASSed.

@calvinjia please take another look

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2608/
Test PASSed.

@calvinjia PTAL

alluxio-bot, merge this please.


Compilation for Linux

Thanks the start you provided in #17 - I did some investigation on my machine and got the simple mouse example working with some tweaks (I went a little overboard - sorry) to the binding.gyp file.  Hopefully it all still works on MacOS :)

Looks good! I'll confirm it still works on Mac, then merge it!

Everything looks good!
COMMENCING MERGE.

Thanks again for your help, I'll make sure to thank you in the readme!


Allow ENV Variables in docker-compose.* definition

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe
Using env variables in docker-compose.* should be possible, like docker compose does.
Describe the solution you'd like
In my default docker-compose.yml I was able to use imported variables like this:
# docker-compose.yml
version: '3.6'
services:
    web:
        env_file:
            - .env
        volumes:
            - ${MY_VARIABLE}:/var/www/public

Doing this within the ddev context:
# .ddev/docker-compose.env-file.yaml
version: '3.6'
services:
    web:
        env_file:
            - ../.env

# .ddev/docker-compose.mounts.yaml
version: '3.6'
services:
    web:
        volumes:
            - ${MY_VARIABLE}:/var/www/public

will return

The "MY_VARIABLE" variable is not set. Defaulting to a blank string."

I also tried to merge docker-compose.env-file.yaml and docker-compose.mounts.yaml with no luck.
It would be great to have this feature in ddev context!
Describe alternatives you've considered
No response
Additional context
No response

Ah, I'm wrong. Compose also does not respect the env_file definition for my purpose. The solution is to move the .env into the .ddev directory, where docker is able to load them by default.
However, maybe this hint is helpful to someone else.

You saw the options in https://ddev.readthedocs.io/en/latest/users/extend/customization-extendibility/#providing-custom-environment-variables-to-a-container right, with the .env file description there? I do keep expecting to spend some more time in that area because I think it should be a little easier.


Feat(eos_cli_config_gen): extend dot1x with radius av-pair and mac based authentication

Enhancement summary
dot1x global configuration will be extended with missing options for radius av-pair and mac based authentication.
Which component of AVD is impacted
eos_cli_config_gen
Use case example
Global dot1x configuration for campus leafs in NAC setup
dot1x
  mac based authentication delay 0 seconds
  mac based authentication hold period 1 seconds
  radius av-pair service-type

Describe the solution you would like
dot1x:
  mac_based_authentication:
    delay: < 0-300 in seconds >
    hold_period: < 1-300 seconds >
  radius_av_pair
    service_type: < true | false >
    framed_mtu: < 68-9236 >

Describe alternatives you have considered
This can be accomplished today with cvp configlet or eos_cli:.
Additional context
I would like to provide the implementation on this
Contributing Guide

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

Thank you for bringing this enhancement. I agree on the data model after checking the options in CLI. Please note that we require these options to be added to the schema for eos_cli_config_gen. I hope it is reasonably intuitive, but otherwise feel free to reach out for assistance.


build failed under miniconda3

Environment: miniconda3
Python = 3.12.2
torch.version = 2.3.0+cu121
pip install flash-attn
pip install flash-attn --no-build-isolation
both failed to build

Building wheel for flash-attn (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py bdist_wheel did not run successfully.
│ exit code: 1
╰─> [886 lines of output]
fatal: not a git repository (or any of the parent directories): .git
  torch.__version__  = 2.3.0+cu121

...
RuntimeError: Error compiling objects for extension
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for flash-attn
Running setup.py clean for flash-attn
Failed to build flash-attn
ERROR: Could not build wheels for flash-attn, which is required to install pyproject.toml-based projects

Hey bro! I encountered the same issue as you did; may I ask if you have found a solution?

It's compiling right now, so it probably works. Make sure you install cuda 12.1 and tick the library and tools checkboxes in the cuda installer.
My ENV in windows:
CUDA_HOME=C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1
CUDA_PATH=C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1
CUDA_PATH_V12_1=C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1

Make sure the file C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1\bin\nvcc.exe exists
conda create --name stableaudio python=3.8.12
conda activate stableaudio
pip install wheel
pip install ninja
pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cu121
pip install flash_attn

I think the problem is somewhere in a buggy CUDA_HOME detection of pytorch and flash_attn builder, where it tries to detect cuda paths in different ways using different ENVs

It's compiling right now, so it probably works. Make sure you install cuda 12.1 and tick the library and tools checkboxes in the cuda installer. My ENV in windows:
CUDA_HOME=C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1
CUDA_PATH=C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1
CUDA_PATH_V12_1=C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1

Make sure the file C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.1\bin\nvcc.exe exists
conda create --name stableaudio python=3.8.12
conda activate stableaudio
pip install wheel
pip install ninja
pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cu121
pip install flash_attn

I think the problem is somewhere in a buggy CUDA_HOME detection of pytorch and flash_attn builder, where it tries to detect cuda paths in different ways using different ENVs

This doesn' t work for me.
Instead wheel file is successful to install.


Casefold regarding the IRC server's casemapping

By default irc3 doesn't do any casefolding on nicks/channels/etc. This can create issues in things like the userlist plugin where channels are seen as different where they should be the same, accessing user nicks can be problematic, etc.
I've made this a core plugin rather than putting it in irc3_plugins because I think the core should be casefolding things like its data structures, the userlist plugin, etc.
Note: This is an in-progress PR, I'm planning to make the userlist plugin use this by default.

Can be cool if you can add a few unit tests too. But ok, let's merge this one


Show schema in pg_namespace if user has permissions on any contained table

Fixes https://github.com/crate/crate/issues/14907

@BaurzhanSakhariev pushed a fixup


Perform cortical parcellation from plane and sulci definition

Subtask of https://github.com/PerkLab/BwhNeuroImagingCollab/issues/4
Based on the tracing of the sulci as well as the user defined planes, perform the cortical segmentation of the space between the white/grey matter border and the surface of the cortex.

By changing the contents of parcellation.qry, users can define the cortical parcellation.


Adding Sound to the Simulator

Hey Guys out there,
I need to implement sound into the Carla simulator (especally Engine sounds and enviroment). Does anyone of you has done this before with succes?
I am realy thankfull for every hint i can get.
Thanks a lot
Greetings

It would be cool having an additional sensor for audio (microphone sensor) like the camera sensor.
As a workaround I use pygame to play sounds/music via the client.
modified manual_control_steeringwheel.py:
self.crashSound = pygame.mixer.Sound('scream.wav')

def _on_collision(weak_self, event):
......
    if isinstance(event.other_actor,carla.Walker):
        self.crashSound.play()


SSL/TLS-Modules

Added the SSL/TLS modules to scan for ssl certificate vulnerabilities and to scan for old ssl/tls version or bad cipher suite.
Checklist

[x] I have followed the Contributor Guidelines.
[x] The code has been thoroughly tested in my local development environment with flake8 and pylint.
[x] The code is Python 3 compatible.
[ ] The code follows the PEP8 styling guidelines with 4 spaces indentation.
[x] This Pull Request relates to only one issue or only one feature
[ ] I have referenced the corresponding issue number in my commit message
[ ] I have added the relevant documentation.
[ ] My branch is up-to-date with the Upstream master branch.

Changes proposed in this pull request
Your development environment

OS: Kali GNU/Linux
OS Version: Rolling x
Python Version: Python 3.11.9

@securestep9 I'm unable to add you as a reviewer here. Could you review when you get a chance?

@arkid15r, I have made the requested changes please review it again. Also on your last comment, I can't think of how to add cipher value based test as per the current login if the ssl connection is established using the provided unsecure_cipher than the method returns True else it returns False. So how can I write a test based on the cipher value?


Some templates do not work in Dev tools -> Templates, but work in Markdown card

Smth strange is happening in 2023.9.2 (may be happened earlier…)
This works:
Translated en: {{ "sun.sun" | ct_state_translated("en") }}
Translated ru: {{ "sun.sun" | ct_state_translated("ru") }}

but this does not:
Translated en: {{ ct_state_translated("sun.sun", "en") }}
Translated ru: {{ ct_state_translated("sun.sun", "ru") }}

Same for ct_translated:

Same for ct_state_attr_translated:

Also ct_all_translations - does not work:

For ct_eval - none of variants works:

Now the interesting part.
All this things work in Markdown:
**ct_state_translated**
State: {{ states("sun.sun") }}
Translated en: {{ ct_state_translated("sun.sun", "en") }}
Translated en: {{ "sun.sun" | ct_state_translated("en") }}
Translated ru: {{ ct_state_translated("sun.sun", "ru") }}
Translated ru: {{ "sun.sun" | ct_state_translated("ru") }}

**ct_state_attr_translated**
Attribute: {{ state_attr("automation.test", "mode") }}
Translated en: {{ ct_state_attr_translated("automation.test", "mode", "en") }}
Translated en: {{ "automation.test" | ct_state_attr_translated("mode", "en") }}
Translated ru: {{ ct_state_attr_translated("automation.test", "mode", "ru") }}
Translated u: {{ "automation.test" | ct_state_attr_translated("mode", "ru") }}

**ct_translated**
Translated en: {{ ct_translated("component.sun.entity_component._.state.below_horizon", "en") }}
Translated en: {{ "component.sun.entity_component._.state.below_horizon" | ct_translated("en") }}
Translated ru: {{ ct_translated("component.sun.entity_component._.state.below_horizon", "ru") }}
Translated ru: {{ "component.sun.entity_component._.state.below_horizon" | ct_translated("ru") }}

**ct_eval**
{% set template_text = "{{ states('sun.sun') }}" -%}
{{ ct_eval(template_text) }}
{{ template_text | ct_eval }}

I did not put a code with ct_all_translations - output is too large.
So, the plugin partly does not work in Dev tools → Template, but seems to work in Markdown.

config:
custom_templates:
  preload_translations:
    - en
    - ru

It seems to be related to cache bypassing added here

Fixed in v1.2.2

Confirmed! thank you!


VIM keybindings

Hey there, thanks for developing this. I'm loving what I see so far.
Describe the feature
I would love to be able to navigate basilk via VIM keybindings. It's quite common for TUIs to support HJKL as an alternative to the arrow keys, which is the primary thing I'm missing with basilk.
Describe your idea
Add alternative keybindings for the arrow keys, following the VIM keybinds.

Hi @Gitznik, thanks for the idea. It will definitely be released in version 0.2.0


[Error] : set flood failed : Property is not supported! propertyid: 6

I'm using Orbbec Gemini 2 and I wanna set up flood light for my camera. Encounter this issue when trying to call the service ( rosservice call /camera/set_flood "{true}" ) . Please help.

should be "{data: true}"


method_exists don't work with php 7.4

For example, to get syntax(string) always return "null" (attach screenshots)
                    'sentences' => method_exists($syntax, 'sentences') ? $syntax->sentences() : null,
		'tokens' => method_exists($syntax, 'tokens') ? $syntax->tokens() : null,
		'language' => method_exists($syntax, 'language') ? $syntax->language() : null,

Hi, can you PR a failing test or steps to reproduce? It does work for us and everyone else?

closing since no reply.


catch TokenError on parse

closes #1787.
Reproduced by running atheris locally on the coverage.py oss-fuzz runner:
p = PythonParser(text="\r'\\\n'''")
p.parse_source()

This string passes ast.parse but fails tokenize.generate_tokens (on 3.10 but not 3.12):
s = "\r'\\\n'''"
ast.parse(s) # passes
generate_tokens(s) # errors on 3.10

so we can't rely on ast.parse to give us nice syntax/indentation errors.

This is now released as part of coverage 7.5.2.


Update_header.html.erb

added about to the nav bar

@tadake thanks for this contribution. Let me clean up the /about page in the next day or two and will merge this in when ready.

Your welcome!My father's a disabled veteran and  he's a programmer and is also my inspiration for coming into the field. I really think this is an amazing program.
sincerely,
kyle<EMAIL_ADDRESS>Sent from my T-Mobile 4G LTE Device-------- Original message --------From: David Molina<EMAIL_ADDRESS>Date: 5/3/2016  1:19 AM  (GMT-07:00) To: OperationCode/operationcode<EMAIL_ADDRESS>Cc: tadake<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [OperationCode/operationcode] Update_header.html.erb (#218)
@tadake thanks for this contribution. Let me clean up the /about page in the next day or two and will merge this in when ready.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

Also @davidmolina im pretty well versed in HTML5 CSS3 adn frmaeworks like bootstrap and materialize. If you need any thing else let me know.

@tadake Most definitely. The landing and some of the other pages needs some work. Invite to our Slack incoming.

@tadake What's your email?  Feel free to pursue other ones here. Thanks.

@tadake Added an about page. Can you pull again so you're current and I merge this in. Thanks.


beeper: 3.106.2 -> 3.107.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for beeper is: Universal chat app
meta.homepage for beeper is: https://beeper.com
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.107.2 with grep in /nix/store/57yiz4irlgpz0hhpfi124mjwvkk7wfsn-beeper-3.107.2
found 3.107.2 in filename of file in /nix/store/57yiz4irlgpz0hhpfi124mjwvkk7wfsn-beeper-3.107.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

beeper

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/57yiz4irlgpz0hhpfi124mjwvkk7wfsn-beeper-3.107.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A beeper https://github.com/r-ryantm/nixpkgs/archive/be1194b370a0269ccea1ff9f1b02ae10dbf2ce61.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/be1194b370a0269ccea1ff9f1b02ae10dbf2ce61#beeper

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/57yiz4irlgpz0hhpfi124mjwvkk7wfsn-beeper-3.107.2
ls -la /nix/store/57yiz4irlgpz0hhpfi124mjwvkk7wfsn-beeper-3.107.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

beeper

Maintainer pings
cc @jshcmpbll @mjm @edmundmiller for testing.

Closing to regenerate.


alphabetized Game.lua and init.lua

Coverage remained the same at 95.752% when pulling 70cac0b8692c2abb83c92704ef66ae0fcce51769 on aportner:alphabetized-game into dc63d21c97df1528b902421b9447d8b72fde3a8a on LPGhatguy:master.


Fixed a typo in Albanian language Hello World

I fixed a typo in the Albanian language, it was same as what google translate shows but there is a typo so this is the correct one.
account number: dfddf07ec15cbf363ecb52eedd7133b70b3ec896b488460bcecaba63e8e36be5

@odetolataiwo @Rajputusman, 500.


examples: Remove alpine examples backend

Commit Message: examples: Remove alpine examples backend
Additional Description:

remove alpine-dev images
fix confusing/confused pattern of backend services with envoy in same container
general cleanups

Partial fix for #16275
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

this doesnt remove all - some are a bit more complex - eg zipkin, so i will fix those in a follow up pr


UTC time conversion in Columns/Form

Hi, i am storing the time of an event in UTC time (thats how is given to me) in the database table.
Is there any snazzy way of converting the time to local time in the Grid/Form?
I looked in to adding an Expression, but ideally dont want to do conversion in SQL as C# has better date handling stuff and it would be maybe a variable in C# that hold my current UTC offset.
Any suggestions welcomed
Thanks
Paul

Add [DateTimeKind(DateTimeKind.UTC)] to your field

Perfect, thank you again.
Just to confirm for anyone else, apply this to the field in your xxxxxxRow.cs file


Add the Binadables object to the generated play_2_x_json file

Work in progress here:
https://github.com/gilt/apidoc-generator/tree/apidoc-277-play-json-bindables

See https://github.com/gilt/apidoc-generator/pull/1 for PR

fixed in 0.8 release which should be online next week


Developing locally plugin packages throws wrong error when user is not a global owner of npm

- Do you want to request a feature or report a bug?
bug
- What is the current behavior?
When I run netlify build --dry I receive error below. However this error doesn't occur when I run command with sudo.

❯ Installing plugins dependencies

┌─────────────────────────────┐
│    Netlify Build Error!     │
└─────────────────────────────┘

TypeError: 'undefined' plugin not installed or found
Path must be a string.
    at resolve (/home/jozef-piecyk-10clouds/.config/yarn/global/node_modules/resolve/lib/async.js:47:19)
    at Promise (internal/util.js:274:30)
    at new Promise (<anonymous>)
    at resolve (internal/util.js:273:12)
    at resolvePlugin (/home/jozef-piecyk-10clouds/.config/yarn/global/node_modules/@netlify/build/src/plugins/install.js:28:30)
    at Promise.all.pluginsOptions.map.pluginOptions (/home/jozef-piecyk-10clouds/.config/yarn/global/node_modules/@netlify/build/src/plugins/install.js:17:81)
    at Array.map (<anonymous>)
    at installPlugins (/home/jozef-piecyk-10clouds/.config/yarn/global/node_modules/@netlify/build/src/plugins/install.js:17:60)
    at build (/home/jozef-piecyk-10clouds/.config/yarn/global/node_modules/@netlify/build/src/core/main.js:53:35)

- If the current behavior is a bug, please provide the steps to reproduce.

Run netlify build --dry when the current user is not the owner of npm global files

- What is the expected behavior?
User is warned that the error is due to lack of permission.
- Local Environment Information
netlify version --verbose
Linux Mint 19 Cinnamon

Hi @masives,
Could you please print the output of netlify version --verbose. Thanks!

oh, sorry about that
netlify-cli/2.20.0 linux-x64 node-v10.21.0

This problem might be fixed in the latest Netlify CLI (2.58.0 at of now).
Would it be possible to upgrade, and try again? Thanks!

What's interesting is that it shows different version when using sudo

Anyways it looks like my local setup causes it so I'm closing the issue


Add Syntax Highlighting & Fix Install Command

This adds syntax highlighting & standardization and fixes the install command.

@MatthewCallis thank you for taking your time to make this prettier!


Minor optimisations for prepLookups

Changed prepLookups to return a flat list instead of a list of lists. Also, added some newtypes like PageSpan and PageNo, and removed an indirection from the compact index search code.
I compared benchmarks before and after the change, using a fixed seed for the prepLookups micro-benchmark. I only benchmarked the default configuration with only positive lookups, because it exercises the most code (i.e., one bloom query and index search for each key-run combination).I also look at the allocations vs. iterations regression.
 Time goes down a little it for "in-memory lookup", and allocations go down by roughly 0.017 * 10 ^ 8 . Though this may not look like much in the grand scheme of things, it shaves off roughly 1/3 of the allocations for prepLookups that can not be attributed to bloom filter queries or compact index searches.
These results also re-emphasise that bloom filter queries are quite costly

Also nice to introduce the PageNo newtype, although the convenience of the Num instance takes away some of the benefits.

Removed the Num instance


LTO support

Clang ThinLTO now work fine on Linux and MinGW, just add -flto=thin, but there is currently an issue where executing strip with static libraries built with LTO may fail, which makes it impossible to build curl with LTO/CFI (-fsanitize=cfi). Therefore, we should remove strip from the build script and strip only for curl executables or dynamic libraries.

I've made a journey with LTO many years ago, and the experience still hurts. What hurt most was the LTO metadata using a format that wasn't compatible between toolchain releases, requiring to rebuild every object after compiler updates. This isn't an issue in CI, but it is burden for local testing. Another issue was sensitivity to 3rd party issues (e.g. bumped into an OpenSSL function that was declared and defined differently, surfacing only when doing LTO). And slow link times.
Things may have improved and it wasn't ThinLTO yet back then, so we might give it a cautioius retry!
But, we do have to strip static libs to remove the non-reproducible bits from them, because we ship them in the curl-for-win package.
It seems CI bumped into this right away:
+ llvm-strip-17 --enable-deterministic-archives --strip-debug _a64-linux-gnu/usr/lib/libz.a
llvm-strip-17: error: '_a64-linux-gnu/usr/lib/libz.a(adler32.c.o)': The file was not recognized as a valid object file

https://github.com/curl/curl-for-win/actions/runs/7128972278/job/19411993913#step:3:2843
Do you see any way around this? Enabling this for a future LLVM version perhaps?

Here's a test build with ThinLTO:
https://github.com/curl/curl-for-win/actions/runs/7131676251
The results mirror my memories: All binaries grew, by a large margin.
Static libs are not as larger as I remember with fat LTO, but still add
to the already inflated sizes (after --gc-sections).
I agree with you that for curl, the bottleneck is not here. It's rather in
the network code, protocol implementation (which has seen a lot
of streamlining this year, and more is coming up.), DNS resolver and
parallelization.
This and the test binaries are not convincing me that curl-for-win
is benefitting from LTO.
There was also a fallout with macOS + llvm where the linker is missing
some LibreSSL functions. Here's the patch if you'd like to experiment:
--- a/_build.sh
+++ b/_build.sh
@@ -973,6 +973,11 @@ build_single_target() {
     fi
   fi

+  if [[ "${_CC}" = 'llvm' && ( "${_OS}" != 'mac' || "${_TOOLCHAIN}" = 'llvm-apple' ) ]]; then
+    _CFLAGS_GLOBAL+=' -flto=thin'
+    _CXXFLAGS_GLOBAL+=' -flto=thin'
+  fi
+
   # curl recommended options to reduce binary size:
   # https://github.com/curl/curl/blob/master/docs/INSTALL.md#reducing-size

@@ -1309,6 +1314,7 @@ build_single_target() {
     _STRIP_LIB="${_STRIP_BIN}"
     _STRIPFLAGS_LIB='--enable-deterministic-archives --strip-debug'
   fi
+  _STRIP_LIB='echo'
   export _OBJDUMP="${_BINUTILS_PREFIX}objdump${_BINUTILS_SUFFIX}"
   export _READELF="${_BINUTILS_PREFIX}readelf${_BINUTILS_SUFFIX}"  # symlink to llvm-readobj in llvm
   if [ "${_OS}" = 'win' ]; then


cuda11.8 for v0.2.4

Hi, there is no cuda11.8 wheel in releases.

cc @WoosukKwon
This is a known issue. We will address them.

Hi @WoosukKwon, could you please share instructions on how to build from source with 11.8? #2072

I successfully build docker image with cuda11.8 and recent vllm. Here is git patch:
(briefly: add pypi index + fix setup + rename packages in reqs + change cuda in dockers).
diff --git a/Dockerfile b/Dockerfile
index dd48677..d38c4ff 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -2,7 +2,7 @@
 # to run the OpenAI compatible server.

 #################### BASE BUILD IMAGE ####################
-FROM nvidia/cuda:12.1.0-devel-ubuntu22.04 AS dev
+FROM nvidia/cuda:11.8.0-devel-ubuntu22.04 AS dev

 RUN apt-get update -y \
     && apt-get install -y python3-pip git
@@ -11,7 +11,7 @@ RUN apt-get update -y \
 # https://github.com/pytorch/pytorch/issues/107960 -- hopefully
 # this won't be needed for future versions of this docker image
 # or future versions of triton.
-RUN ldconfig /usr/local/cuda-12.1/compat/
+RUN ldconfig /usr/local/cuda-11.8/compat/

 WORKDIR /workspace

@@ -78,7 +78,7 @@ RUN --mount=type=cache,target=/root/.cache/pip VLLM_USE_PRECOMPILED=1 pip instal
 # We used base cuda image because pytorch installs its own cuda libraries.
 # However cupy depends on cuda libraries so we had to switch to the runtime image
 # In the future it would be nice to get a container with pytorch and cuda without duplicating cuda
-FROM nvidia/cuda:12.1.0-runtime-ubuntu22.04 AS vllm-base
+FROM nvidia/cuda:11.8.0-runtime-ubuntu22.04 AS vllm-base

 # libnccl required for ray
 RUN apt-get update -y \
diff --git a/requirements.txt b/requirements.txt
index 05ec2e8..461128c 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -3,6 +3,7 @@ psutil
 ray >= 2.9
 sentencepiece  # Required for LLaMA tokenizer.
 numpy
+--extra-index-url https://download.pytorch.org/whl/cu118
 torch == 2.1.2
 transformers >= 4.38.0  # Required for Gemma.
 xformers == 0.0.23.post1  # Required for CUDA 12.1.
@@ -13,4 +14,4 @@ prometheus_client >= 0.18.0
 pynvml == 11.5.0
 triton >= 2.1.0
 outlines >= 0.0.27
-cupy-cuda12x == 12.1.0  # Required for CUDA graphs. CUDA 11.8 users should install cupy-cuda11x instead.
+cupy-cuda11x == 12.1.0  # Required for CUDA graphs. CUDA 11.8 users should install cupy-cuda11x instead.
diff --git a/setup.py b/setup.py
index 57d7a13..93730f5 100644
--- a/setup.py
+++ b/setup.py
@@ -472,6 +472,7 @@ def get_requirements() -> List[str]:
     else:
         with open(get_path("requirements.txt")) as f:
             requirements = f.read().strip().split("\n")
+    requirements = filter(lambda x: not x.startswith('--'), requirements)
     return requirements

@@ -508,6 +509,9 @@ setuptools.setup(
                                                "examples", "tests")),
     python_requires=">=3.8",
     install_requires=get_requirements(),
+    dependency_links=[
+        'https://download.pytorch.org/whl/cu118'
+    ],
     ext_modules=ext_modules,
     cmdclass={"build_ext": build_ext} if not _is_neuron() else {},
     distclass=BinaryDistribution,


Hard limit of 100 commits in Repository.searchCommits

Hi.
I am currently rewriting last bits of library that use git binary invocation and found that searchCommits will look only for 100 commits. Should I copy that behavior or remove limit?

Yes... you can change it to pass the limit through argument.


DynamicMessage can't handle strings

Description
I prefered to use DynamicMessage instead of FormatCustomText because it's easier to set up.
so I added a DynamicMessage widget and update its value with a string, and I get an error that the g format code doesn't work with an str
ValueError: Unknown format code 'g' for object of type 'str'
Code
import progressbar, time

widgets = [
    progressbar.Percentage(),
    progressbar.Bar(),
    progressbar.DynamicMessage('user'),
]
with progressbar.ProgressBar(max_value=100, widgets=widgets) as bar:
    for i in range(100):
        bar.update(i, user='me')
        time.sleep(0.2)`

Versions

Python version: 3.7.3 (default, Mar 26 2019, 21:43:19)
[GCC 8.2.1 20181127]
Python distribution/environment: CPython
Operating System: Antergos (Arch Linux)
Package version: 3.39.3

Solution
In order to fix it I changed the code of function call of DynamicMessage like this
def __call__(self, progress, data):
    val = data['dynamic_messages'][self.name]
    if val:
            if isinstance(val,str):
                return self.name + ': ' + '{:6s}'.format(val)
            else:
                return self.name + ': ' + '{:6.3g}'.format(val)
    else:
        return self.name + ': ' + 6 * '-'

I've released a new version that supports strings as well :)
https://github.com/WoLpH/python-progressbar/blob/7075248b3320270116028be08fc5866b3847771c/examples.py#L439-L455

That's so cool, congratulations ! :+1:


This library failed if only emojies text

I am using this library in my project. But library is not working if string has only emojis.
"message":"😉😊"
See 1 user send only emojis. and app crashed in production version. Android 10

I never tested it. Please if you know how to fix it, share a PR .


broken link for "Manually Trigger a Deployment"

The "Manually Trigger a Deployment" netlify link on the Release Procedures page is broken and redirects to a "Page Not Found" error.

Nice catch. When we moved to the docs.kargo.io domain about a month or two ago, we actually created a new site and the old one has since been deleted.
The correct link should be: https://app.netlify.com/sites/docs-kargo-io/deploys


Add lua

Basic lua support

probably want to uncomment https://github.com/knausj85/knausj_talon/blob/main/core/modes/language_modes.py#L27

I think this is fine to merge now but I'll wait a day or two just to make sure there are no other comments

@fidgetingbits A couple errors I noticed with this PR:
WARNING actions: skipped because they have no matching declaration: (user.code_state_continue)
WARNING tags: skipped because they have no matching declaration: (user.nvim_lua)

Looks like you are missing a declaration for this action (missing file?) and may want to remove the tag since there is no other mention of nvim_lua in knausj currently.


Optimize user submission flow

This PR has several enhancements:

auth:  the user will only need to maintain a single credential ~/.gen3/gen3_client_config.ini per gen3-client instructions

a '--profile' option now exists on all commands

metadata generation:  the system will default to filtering uploaded files to those files in the current manifest (that have been successfully uploaded), previously the system considered all files uploaded to indexd --source [manifest|indexd]

metadata submission:  the system will now automatically generate and submit metadata. Additionally the system will default to immediately returning the job identifier.  The user can then query job status.

The user can override this behavior on the gen3_util files manifest upload command:

a) not to generate and submit --meta_data
b) choose to wait for the job to complete --wait

Had to go digging to figure out that the gen3_util by default looks for the first profile in the ~/.gen3/gen3_client_config.ini if you don't specify a profile using the flag. This should be somewhere in the docs or the very least in the README of this repo for developers wondering why their the authentication pytests are failing.
1 test still fails:
pytest tests/integration/test_project.py::test_project_bad_ping
seems to rely on an option that may not exist anymore.


Remove spaces from computer name

Windows disallows spaces in computer names. This change should remove the space.

Thanks for this. Can you submit the PR against the 2404.1 branch? I'll merge it to main later this month. https://github.com/rbalsleyMSFT/FFU/tree/15-24041

Sorry, I didn't see your message. I will do this now.


Some samples issues

I've noticed some issues with the samples:

The sample of "sample" requires camera permission , even though it should run just fine without it, as there are other ways to get an image (from storage, for example), as is done on "quick-start" sample.
The "Done" action item icon has circle background, in the "quick-start" sample
The "quick-start" sample marks a circle crop, but when cropping, it's a normal rectangle.
The toolbar icons/text sometimes have different color from others, like on the "sample" sample and the "test" sample.
The navigation drawer on the "sample" sample cannot be opened easily at all via swiping from the left. Clicking at the top is the only safe way to do it.
On the "sample" sample , the multitouch feature seems a big buggy in how it works. Hard to explain. However, I can say that I can't zoom or pan using it. I don't understand what it's used for.
When rotating the result activity (the one that's shown after finished cropping), the image is gone.

There a "feature" in Android that you must explicitly request CAMERA permission on Android M and above when you open intent if you requested it in manifest, see fixes in versios 2.1.4
I tested changing the icon and was lazy to use a different icon, why?
See FAQ: Why is the cropped image rectangular when I used oval crop shape?
So?
It's a sample app to play around...
Multitouch was added in PR without real effort, I probably should just delete it
It's a sample app.. why it's important?

Sorry for that. I didn't know.
It's just that it doesn't look well on the toolbar. You should have used a more native one.
So what's the use of cropping in oval shape, if it doesn't do anything? I don't get it. It does the exact same thing as rectangle cropping.
It looks bad currently. Items on the toolbar should have consistent colors.
Question is how to fix it in case we use it in an app, that needs to have this view, together with a drawer, and why can't it work well on the sample.
I see.
If I rotate the screen while using this library, I should know how to handle it well, including still showing the image and also have the cropping&zooming states restored. The sample doesn't show this important thing. It has bugs regarding this.

None of the points you mention affect the library, samples are used to showcase the library capabilities and help contributors debug.
It's not shipped with the library and not intended to educate android development.
But if you feel so strongly about it, please send a PR

I don't understand. Shouldn't the sample work fine with the library?
How can you demonstrate how to use it properly, without a sample?
Currently I see not just cosmetic issues (incorrect icons and colors), but also crucial ones (incorrect cropping, inability to use drawer with this view,  and incorrect handling of configuration changes). How doesn't it affect the library?
A sample should show the library at its best.
Not being able to handle very common scenarios, and features of the library, inside the sample, shows that the library could have issues.
If the library isn't the one that has issues, the sample should be updated. If the library has issues, the library should be updated.
In any case, there should be fixes.

Well, we don't have to agree.
The beauty of open source is that anyone can contribute. I see you're passionate about the sample app, so please fix and send a PR.

I can't fix something that I don't know the reason for why it's faulty.
best the creator to know how to fix it.
Why doesn't the cropping in oval shape work?
What's the way to save the state of the view, across configuration change?
What's the correct way to use the view, if there is a drawer?
Those questions cannot be answered by me, as I've only recently was even aware of this library.

The only things I can fix, which are very easy, are the cosmetic ones.

That's how software works, you see something that you think is wrong.. Let's say oval bitmaps, you do the research on how stuff work in bitmaps, in alpha chsnnels, in Android image api, Android rendering api, performance considerations, etc. you debug the source to see how the flow goes, what api it's using and where is the code responsible for the issue you're looking into.
By now you may learn that what you thought is incorrect is actually a software compromise that has to be there, or you discover that there is a better way to solve this problem and you have a lead on how to solve it.
That's what I do, every day, that's software development.

I don't see any way that oval bitmap cropping should behave exactly like rectangle one.
About the output bitmap, if it has alpha, it should be used, and if not, the function to crop should require a color to set for around the circle.
But in no possible scenario, an oval crop should return the exact same image as rectangle one (unless the input bitmap is oval too, of course).
There is no compromise here. It's a bug.

Please explain.
Why the correct behavior of oval cropping should be rectangle cropping?
Why can't it always be oval, as this is the name of the function?
Why, if the function doesn't do oval cropping, does it even exist?

The FAQ briefly explains that, I don't want to go into lengthy discussion beyond that.

If you mean this:

The resulting cropped image is always rectangular simply because bitmaps are.
Usually, the oval shape is created during rendering of the image, there are many solutions for it.
But if you do want the resulting image pixels to reflect the oval shape you can use CropImage.toOvalBitmap(Bitmap) helper method that does exactly that.

That's not what I meant. I didn't mean why the bitmap is rectangular. Bitmaps are supposed to be rectangular, as they have width and height to define their rectangle shape size.
I mean why the content is not oval, within the bitmap.
See here:
"sample" project does correct circular cropping:

But "quick-start" project doesn't:

It's the exact same input image.

So again, why should it work differently?

And, how can I make the view retain its state?
And if there is a drawer, how should it work with the view?

Debug the code

Well, the triggers seem to be ok. On both samples, they do pass that they wish it to be oval-shaped cropping.
I tried to dig for a clue for this issue, but I just can't find a usage of the oval-shape cropping on "quick-start" sample.
This is called:

So I don't see a call to using the shape here.
The result bitmap is sent back to the view:

Which does nothing special to it either (and it shouldn't, because the cropping should be done on the bitmap before it's sent to it) :

Even on "CropImageActivity" , I don't see the shape being used:

Again, I'm not familiar with the code, but to me it seems that everything starts fine, but when the time comes to crop the image, the shape is ignored.

Why close it? Have you fixed it?

like I said before, this is not a bug

How can it not be a bug?
It calls to crop in an oval shape, but it doesn't crop in oval shape, even though on another sample, it does work fine.


Fixing links, and some wording changes

I changed some of the wording to make it less chatty. I hope that's ok!

Also updated for the qc_mm --> ingest pipeline


Build LTR plugin for OpenSearch 2.15.0

Build LTR plugin for OpenSearch 2.15.0

Similar to #34, probably best to see if we can get the plugin integrated with the OpenSearch release process.

@jzonthemtn i up open search version to 2.15 but i have same test problems (https://github.com/quorcs/opensearch-learning-to-rank-base). I skipped tests and build the plugin. now my colleagues try to testing it

Including the plugin into the full release is a good idea, but that is a long conversation. I recommend fixing up the repo and releasing 2.15 of this plugin.
If someone can help get this plugin to branch on main with OpenSearch 3.0 and on 2.x with 2.x, and do the work of bumping versions, the infrastructure team can help with setting up 1-click release. You can work with @gaiksaya on the latter.

[Catch All Triage - 1, 2, 3, 4]

^^ @sstults I think the comment above from @dblock lays out what we need to get done.....    I almost want to suggest we rename this Issue to "Build LTR plugin for OpenSearch 2.11 to 2.17"!


Why are embeddings stored as List?

Hi,
I am wondering why the class Embedding internally stored the vector as List<Double>. I think double[] or float[] would be more memory efficient.

List
Firstly, List is a basic collection in Java, capable of storing only objects instead of primitive types like double or float.
Using collections allows for more flexible usage than basic arrays of primitive types, and depending on the use case, you can choose implementations such as ArrayList or LinkedList.
If you need to add a new value exceeding the predefined size of a double[n] array, you must allocate a new double[n+1] and copy the existing values before adding the new one. However, when using a List, you can easily handle adding values to the array through the abstracted add() method.
Double
A primitive type double variable defaults to 0.0. Can you determine if an operation was successful or failed when the result is 0.0?
In such cases, 'null' can serve as a means to express the absence of a value or an impossible calculation, and by using the Double wrapper class, which inherits from Object, you can represent null.

@youngmoneee I believe that's exactly the point @agoerler is trying to make.
While the list may be more flexible, it is significantly less efficient than a simple float[] which is why pretty much LLM and most frameworks that work with them, including Langchain and Langchain4j, use primitive float[] to represent embeddings.
For one, the additional flexibility offered by a list is completely unnecessary, as the operations you need to do with the embeddings, such as calculations necessary for the similarity search, are just as easy, if not easier to do with arrays than with lists, and you can actually leverage SIMD support in the modern CPUs to do them more efficiently. Not to mention that you can actually use classes like FloatBuffer to access float[] allocated in native memory by ONNX, for example, directly, but can't do that with a list and have to box every single value into a Double on the way in (and likely unbox into a float on the way out).
But more importantly, storing 1,536 dimension vector returned by OpenAI, for example, requires 6,144 bytes when stored as a float[], and 36,864 bytes when stored as a List<Double>, for the exact same payload (ignoring the space used by the array or list instance itself, which is approximately the same and irrelevant in this case). That's 6x the memory cost, for no good reason, as it is neither faster nor easier to work with. Quite the opposite, actually.
Ultimately, nobody will ever store only a handful of vectors in a vector store, so these size differences add up in a hurry, and you'll need 6x the memory to store the exact same data using List<Double>. There seems to be a trend towards models that can create quantized vectors that use single byte per dimension (int8/uint8) or even a single bit per dimension (binary), in order to reduce the space required for vector storage 4x or 32x without significant accuracy loss (take a look at Cohere, for example), so going in the other direction and making vectors 6x bigger than necessary seems like a bad idea.
The bottom line is that for some of us implementing vector stores, especially in-memory vector stores, the difference between using primitive arrays/buffers and collections containing boxed wrapper types is so significant that the latter is a non-starter. The only way for us to support Spring AI at the moment would be to convert from List<Double> to a float[] on the way in, and the other way around on the way out. That is certainly doable, but is not free, and most importantly, it shouldn't be necessary.

@youngmoneee As for "the ability to store null values" in a list, but in the array, it makes no sense in this context. Embeddings are typically dense vectors, so there shouldn't be any missing values, and you can't do calculations with null anyway.
For cases where sparse vectors are used, they are are better data structures to use than either primitive array or a list of objects, but embeddings are not such a use case.

@youngmoneee I believe that's exactly the point @agoerler is trying to make.

Yes.
I think float[] would be much more memory efficient as compared to List<Double>. Moreover I doubt that the option to store null would be ever needed. As pointed out by @aseovic, float[] would be better compatible with e.g. Langchain4j and would hence require less costly conversions.
Also, I think vector embeddings are typically not manipulated but rather used in semantic search similarity search. I don't see how the List interface is useful when dealing with vector embeddings.

Hi @aseovic, @agoerler
I apologize for not considering the specific context of embeddings and making an incorrect judgment based solely on general cases. As you mentioned, even though there is overhead in memory and computation, I thought it would not pose a significant issue in modern computing servers, especially compared to the bottlenecks caused by I/O-bound tasks.
However, after considering your comments, I agree that regardless of the impact's magnitude, performing unnecessary wrapping/unwrapping and occupying additional memory and processing time is indeed unwarranted.

Solved in https://github.com/spring-projects/spring-ai/pull/1002


⚠ Bofrost.com has degraded performance

In 236e7ee, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 2634 ms

Resolved: Bofrost.com performance has improved in 3aeb948.


Can't load to SBCL

The first Issue I had was with cl-environments
https://github.com/alex-gutev/cl-environments/issues/15
But the solution was to git clone cl-environments.
After that, however, a new problem occurred while loading:
(SB-INT:%FIND-PACKAGE-OR-LOSE "CLAYLIB/WRAP")
0] ##########################################################################
; Loading "trivial-extensible-sequences"
[package org.shirakumo.trivial-extensible-sequences]
To load "claylib":
  Load 1 ASDF system:
    claylib
; Loading "claylib"
[package babel-encodings].........................
[package babel]...................................
..................................................
[package cffi-sys]................................
[package cffi-features]...........................
[package cffi]....................................
.............................
debugger invoked on a PACKAGE-DOES-NOT-EXIST in thread
#<THREAD tid=259 "main thread" RUNNING {7005B305C3}>:
  The name "CLAYLIB/WRAP" does not designate any package.

Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.

restarts (invokable by number or by possibly-abbreviated name):
  0: [CONTINUE                     ] Use the current package, COMMON-LISP-USER.
  1: [RETRY                        ] Retry finding the package.
  2: [USE-VALUE                    ] Specify a different package
  3: [RETRY                        ] Retry
                                     compiling #<CL-SOURCE-FILE "claylib/wrap" "claylib-wrap">.
  4: [ACCEPT                       ] Continue, treating
                                     compiling #<CL-SOURCE-FILE "claylib/wrap" "claylib-wrap">
                                     as having been successful.
  5:                                 Retry ASDF operation.
  6: [CLEAR-CONFIGURATION-AND-RETRY] Retry ASDF operation after resetting the
                                     configuration.
  7: [ABORT                        ] Give up on "claylib"
  8: [REGISTER-LOCAL-PROJECTS      ] Register local projects and try again.
  9:                                 Exit debugger, returning to top level.

(SB-INT:%FIND-PACKAGE-OR-LOSE "CLAYLIB/WRAP")
\

I am on SBCL 2.4.2 .

Ah I see the installation procedure for claylib/wrap is more complicated and requires manual steps.


[Enhancement] User friendly error message when AWS parseRegion gets error

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[X] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[X] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:disappointed: fail : 0 / 9 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/fs/hdfs/HdfsFsManager.java
0
9
00.00%
[1219, 1220, 1221, 1222, 1224, 1225, 1227, 1228, 1229]

[FE PR Coverage Check]
:disappointed: fail : 0 / 3 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/fs/hdfs/HdfsFsManager.java
0
3
00.00%
[1219, 1220, 1222]

[FE PR Coverage Check]
:disappointed: fail : 0 / 3 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/fs/hdfs/HdfsFsManager.java
0
3
00.00%
[1219, 1220, 1222]

[FE PR Coverage Check]
:disappointed: fail : 0 / 3 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/fs/hdfs/HdfsFsManager.java
0
3
00.00%
[1219, 1220, 1222]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


Error at the end of the installation script execution (+ kcl binary missing?)

Bug Report
1. Minimal reproduce step (Required)
Install KCL with the script
wget -q https://kcl-lang.io/script/install.sh -O - | /bin/bash

2. What did you expect to see? (Required)
Installation running smoothly without errors
3. What did you see instead (Required)
The installation goes well, but at the end it gives the following messages, with an error:
Please add /usr/local/kclvm/bin into your PATH
Remeber run the command source ~/.bash_profile or source ~/.bashrc to ensure your PATH is effective
main: line 331: kcl: command not found
Reopen a terminal and execute  to ensure successful installation

This is because the message at line 331 has backticks like a markdown test, but it gets evaluated and there's no kcl binary found.
Also, the documentation talks about the kcl command, but there's no such binary in the /ust/local/kclvm/bin folder, only kclvm_cli, kcl-language-server and libkclvm_cli_cdylib.so.
Is the documentation outdated or does the build process have some trouble renaming the binary?
4. What is your KCL components version? (Required)
kclvm_cli 0.8.3

Nevermind, I was looking at an outdated documentation (0.4 - don't ask me why, I suppose google chose for me)


warning: kqueue is not supported on this platform

tweetstream-2.6.1/lib/tweetstream/client.rb:401: warning: kqueue is not supported on this platform

Just started getting this warning message in the logs, which I think has been triggered by updating eventmachine from 1.0.7 to 1.0.8. This corresponds to line 418 in the current source.
EventMachine.kqueue

I presume this is okay -- being only a warning, and all seemingly working okay -- but could you confirm.
Also, is there a way to stop the message?
Thanks.

My recollection is that these no-op but i'll check.

Yep, it's just a warning: https://github.com/eventmachine/eventmachine/blob/67fd2af021159fc8a8b72f24bd6eff6f45669bdb/ext/rubymain.cpp#L1040-L1049
I think I can suppress the message by changing tweetstream's code from EM.kqueue to EM.kqueue = true if EM.kqueue? which is actually how they use it in their tests: https://github.com/eventmachine/eventmachine/blob/36003fcf1675e8718a9d45b83e70c2054ddb7c40/tests/test_file_watch.rb#L28-L34
It does appear that this was changed recently: https://github.com/eventmachine/eventmachine/commit/9af94064e1f05b373145ba0c3cac41daa9daf369

heh, it could probably be simplified to EM.kqueue = EM.kqueue? which seems silly but ¯_(ツ)_/¯

Cool. Thanks.


[GraphScope/Docs] Add docs for positioning GraphScope

As titled

https://alibaba-graphscope-build-pr-2390.surge.sh/graphscope_positioning.html


Typo in command mapping?

Hi! I see that you have mapping for <Leader> to :ImportJSWord, but README says it should be <Leader>j. Is this a typo?
Thanks for the plugin btw!

Here you go: #39


I Got It Working, But Found A Potential Freeze With Invalid Lua

I don't know what I did to get the module to work (probably the part where I renamed the gdns file to LuaScript.gdns instead of just Lua.gdns). This is aside from my merging the latest commit of Godot 3.2-dev with my copy of Godot (which fixed a freeze on all GDNativeLibraries).
However, I noticed loading an invalid Lua file freezes the game as a whole.
Trying to load this file causes the freeze.
local class = require 'lua.class' -- Import the system class library
godot.Node = require 'godot.Node' -- Make sure to import the base class

local NewScript = class.extends(godot.Node) -- Create the user subclass

function NewScript:_ready()
	print("Hello Lua")
end

function NewScript:_process(delta)

end

function NewScript:command(variable)
	return "Variable: %s" % variable
end

function NewScript:get_class()
	return "Test Lua"
end

return NewScript

This file works fine.
function sum(...)
    result = 0
    local arg = {...}
    for i,v in ipairs(arg) do
       result = result + v
    end
    return result
end

The original file was from a lua module to be compiled into Godot.
This matters as I plan on sandboxing the Lua programs and letting players write lua for automation in their game. Also, the code would run server-side, so it is not good if the server freezes due to invalid lua.

With a valid lua script, if changes are made rapidly for testing, it can freeze the main thread of the game.
I managed to pull an error with Error: script not loaded (test_sandbox) where I am testing cSum(1, 3) with a sandboxed lua function I am working on. I am going to try to put this on a separate thread and lower its priority so it doesn't make my game run at 90% cpu when the script freezes up. I'll report if this workaround works.


Case insensitive admin email auth

Description
Email Password auth for admins now searches for admin matches
with a case insensitive email

Motivation and Context
While logging into the admin panel, email is considered to be case sensitive which can lead to confusion if the sign up was done with some capitalisation.
More details here: https://www.notion.so/lyrasis/Change-Palace-Collection-Manager-login-screen-email-from-case-sensitive-to-not-case-sensitive-b5bb506842084cc98874c6de554a817e

How Has This Been Tested?
A new unit test has been added and the login page was tested manually

Checklist

[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.

A couple more comments.
Also, do we need to introduce a constraint (and associated test(s)) to ensure that "duplicate" admins (e.g<EMAIL_ADDRESS>and<EMAIL_ADDRESS>cannot be created?

In this case the constraint should be on the database, by either doing an in-code lower() or using the citext extension

Also, do we need to introduce a constraint (and associated test(s)) to ensure that "duplicate" admins (e.g<EMAIL_ADDRESS>and<EMAIL_ADDRESS>cannot be created?

In this case the constraint should be on the database, by either doing an in-code lower() or using the citext extension

I think you could also define a unique index on UPPER(email).

Also, do we need to introduce a constraint (and associated test(s)) to ensure that "duplicate" admins (e.g<EMAIL_ADDRESS>and<EMAIL_ADDRESS>cannot be created?

In this case the constraint should be on the database, by either doing an in-code lower() or using the citext extension

I think you could also define a unique index on UPPER(email).

Ok, I looked this up again, a Unique index can be created for the upper() email. (unique constraint cannot use functions, but an index can)
However, since there is already data in the production DBs this might be an un-enforcable index (in case conflicting data already exists). we can test this out by running the following query on the production DBs
select * from (select UPPER(email), count(id) as c from admins group by UPPER(email)) as t where t.c > 1

I ran that query and found no duplicates in the prod CMs.
That said, the migration should be robust to both (1) an already created index, in which case it should not create it, but note that this condition occurred; and (2) index creation failure caused by, e.g., duplicate entries, in which case an exception should be raised (with a helpful message) to abort the migration.

I think you could also define a unique index on UPPER(email).

Updated


Script size HPE_HEADER_OVERFLOW

Hi! We are facing a problem while interacting with your API, for some reason, we can't execute a webpagetest script that contains more than a certain number of characters. Error in the screenshot below.

Would you be able to share your script so we can see what's causing the error?


Fix for Vanilla bug with adding pin after deletion

If you create 2 or more pins and then delete anyone but the last, and
then add a new one then the last will be changed instead. It is the same if you add a new one but instead of pressing Ok you press Delete, then the last one will be deleted instead.
I don't know why it happens, in the RequestMapPin function the ID is
correct but when you enter any of the On-functions the ID have changed
to the highest. This is present in vanilla but I got feed up by this
issue so I fixed it for them :wink:
To fix this I store the location as well and fetch the Pin using this
when buttons are pressed.

The code looks good, but I'm going to hold off on merging until after the workshop update goes out so that any potential issues have time to percolate out before it hits the general user population


Support containerd

Is your feature request related to a problem? Please describe.
Does not work on Kubernetes with containerd.
Describe the solution you'd like
Adapt to the containerd environment.
Describe alternatives you've considered
No response
Additional context
No response

Hi @yonch
Since I don't have a compilation environment, I'm currently using the v1.0.2 executable file released on GitHub, which does not have k8s-collector. At present, I have implemented a bridge program from containerd to docker in golang to proxy the request to containerd.

Oh I had the (wrong) assumption you were running under Kubernetes.
Can you share the proxy? How different are the two protocols?

Oh I had the (wrong) assumption you were running under Kubernetes.
Can you share the proxy? How different are the two protocols?

I do run on Kubernetes, but I don't use k8s-collector. Compiling opentelemetry-network is a bit difficult, so I use the v0.10.2 executable released on github, which does not have k8s-collector. If you release an executable for k8s-collector, I would consider using k8s-collector.
The protocol of containerd is similar to that of docker, mainly implementing the methods of these docker interfaces and proxying to containerd. Specifically, you can view the data structure of docker by sending curl requests to docker, and view the api and data structure of containerd through the containerd sdk.
	mux.GET("/containers/json", s.handleList)
	mux.GET("/:version/containers/json", s.handleList)
	mux.GET("/containers/:id/json", s.handleInspect)
	mux.GET("/:version/containers/:id/json", s.handleInspect)
	mux.GET("/_ping", func(ctx *gin.Context) {})

Got it. We can keep this issue if someone wants to implement containerd support directly in the k8s collector.
There are container images for the two k8s-collector components:
https://hub.docker.com/r/otel/opentelemetry-ebpf-k8s-watcher
https://hub.docker.com/r/otel/opentelemetry-ebpf-k8s-relay
Might they be of use to you?

@yonch Thanks! I'll try it.

The helm chart can serve as a recipe of how they interconnect
https://github.com/open-telemetry/opentelemetry-helm-charts/tree/main/charts/opentelemetry-ebpf

OK. I have another question about k8s-collector. If I want to collect network request monitoring indicators such as dns, tcp, and http of the container, is it enough to deploy only k8s-collector? Do I also need kernel-collector? I didn't see more detailed instructions in the documentation.

I understand that k8s-collector is only responsible for collecting metadata, and kernel-collector is needed to collect the underlying network monitoring indicators. Given that k8s-collector requires more permissions and has two additional components, I personally think it is easier to get metadata directly from containerd.


WD generation has lots of unnecessary links to EDs

Was regenerating CSS3 Text for WD publication, and had to do a lot of manual cleanup. Things like combinators linking to the ED of values and units L4 for no apparent reason (when there's a perfectly acceptable definition in L3), "inline base direction" also linking to the ED of css-writing-modes-4 instead of css-writing-modes-3, various terms linking to ED of css-cascade-4 (even though css-cascade-4 is CR on /TR).

I thought I'd documented this problem already, but yeah, it's known - the "only pay attention to the latest version" code doesn't care about the current/snapshot distinction, and if you have a level N+1 current spec but only have snapshots for level N, the level N links will get ignored early.  I need to fix this.

various terms linking to ED of css-cascade-4 (even though css-cascade-4 is CR on /TR).

This is something new, tho, can you provide an example of it?

Cascade 4: https://drafts.csswg.org/css-text/#index-defined-elsewhere 'inherit', 'shorthand', 'shorthand property' all link to the ED.

Fixed - it was a Shepherd issue.
Debugging guidance, for future reference: ran bikeshed refs --text="inherit" --type="value", only got a single result, for the ED.  Checked the TR version, saw the dfn is definitely marked up correctly.  Checked Shepherd, saw that Cascade 4 has only a "draft" url.  Fixed. ^_^

Well, fixed the latter, more confusing issue.    The bug is that, somehow, "current" == config.refStatus.snapshot is returning True.  It doesn't always return true; if you compare it with a string that's not in the enum at all, it returns false. But it somehow always compares true if the string is in the enum at all, regardless of which enum value you're comparing it to.

All right, should be fixed now - I confirmed that Text 3 now links only to V&U 3 and Writing Modes 3, since all of its references exist there in snapshot specs.

Yay, thanks!
Wrt Shepherd issue, can you confirm if Ruby has the same problem? I'm getting links to the ED on that one, too. (searching for ''collapse'')

No, Ruby is in there, tho the latest draft is from 2014 https://www.w3.org/TR/css-ruby-1/

Responding here for posterity - the TR Ruby draft has some weird title attributes on its value definitions that is preventing them from getting autolinked properly.


New update reportedly available when manually checked and running from source

Running off master at c4e83a630, Windows 7 x64.
Steps to repo:

Help > Check for Updates

Expected results: dialog saying no update available.
Actual results: Reports the v1.2 update is available, adds update icon. Goes away upon app reload.
This only happens when manually checking for updates, the auto-check does not trigger this. Since I'm running off v1.3 source, I did not expect to be told the v1.2 update was available.
This is happening because the condition at  at line 376 unconditionally checks if allUpdates is true, which is is because force makes it true. A simple edit to make the condition read if (allUpdates  && !force) { fixes it and shows the up-to-date dialog.

It seems this behavior has been present since the update system was introduced in #1386, but that fix is causing three tests to fail. That's fine, because it was the wrong fix. The correct fix is if (allUpdates  && _buildNumber !== 0) {. This triggers the no update dialog only on dev build and keeps the unit tests running. :)

I believe it's by design that the dev build always shows all available updates.  If you're in a dev build, Check for Updates isn't able to give a truly correct result either way.  Is it any more correct for clicking it to do nothing, even when your dev build might be behind the latest stable release?

I vote for leaving it as it is. At the very least, it's great for seeing and reviewing the Brackets update notifications (and translations) in context.

Thinking it over, yea, it should be left as-is. It just feels a bit weird, but if it was by design and is useful for some things, then it shouldn't be changed.
Closing.


Edit terminal commands in Run Recent Commands

Feature to edit recent terminal commands in Terminal: Run Recent Commands instead of just choosing from commands listed.
Because sometimes there is a need to edit the command slightly before executing.

You can now remove entries from previous session:

And edit the files easily when they're sources from a file (coming v1.96):

No plans to add edit


Update SPIRV-Tools and SPIRV-Headers

SPIRV-Tools adds:

optimizer fixes and speed improvements
optimization pass --eliminate-dead-functions
validator does more type checking

Tested on Linux Release and Debug builds
LGTM


GA: Media_player trait not working for other persons, only my voice

**Home Assistant release with the issue:**96.5
Last working Home Assistant release (if known): no idea
Operating environment (Hass.io/Docker/Windows/etc.): HassOS
Component/platform: Google Assistant (cloud edition)
Description of problem:
I have exposed some lights/switches and also my media_player
everything is working fine like it should, but only on my voice (lights/swiches/media_player)
for other persons speaking to the google_mini, also everything is working, except the media_player ...
so if i say : Hey google, set volume on 25 => my media player (denon) goes to volume level 25
if another persons says , Hey google, set volume on 25 (checked with myactivity.google.com) , then he/she gets a response like, this service is not linked yet, so it doesnt reach HA at all at that point
if i look on myactivity.google.com , i also see a difference there, see screenshot below
i activate a light and switch, you can see it activates HA with that homeautomation.nl website
for the volume, there is no such string/link, but the command is actually passed to HA anyway, and also my media_player used the command, so its working
https://www.dropbox.com/s/wsw42oe16ff4dpq/volume.PNG?dl=0
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
in google.yaml
    media_player.denon_avrx4000:
      name: Volume
      room: Living Room

in customize.yaml:
media_player.denon_avrx4000:
  device_class: speaker

no logs, since there is no log because it doesnt reach my HA when another persons uses the phrase,

I noticed a similar issue with my wife and I.  I found that I had to add devices to rooms in order for her to control them, anything without a room and it was not controllable.  Are all of the devices in question assigned to a room in the google home app?

no, they are all in a room

note : seems if  share my "house" with all devices in it ; to the user that is speaking to the google home mini, that the volume switch is working
verry strange that only the volume entity is an issue, maybe because an reserverd keyword for google?

Definitely a permissions type thing on the google side. Nothing HA can do about that.
If you want things to work consistently for members of your home, you need to share your "home" to other google members. Alternatively (but more obnoxious) you can have other members add your app to their own "home"

yeah, i now shared my home, so they can also use the volume trait now


Provide Example using HTTP

Hello, i would like to see some examples of using this library with http

An example has been added:
https://github.com/Foo-x/react-tea/blob/master/example/src/HttpRandomCat.tsx
Additionally, live examples have been deployed:
https://foo-x.github.io/react-tea/


Error in Azure CLI version 1.28

This is autogenerated. Please review and update as needed.
Error when trying to list models using Azure ML CLI version 1.28
Command Name
az ml model list
Errors:
cannot import name 'deploy_config_dict_to_obj' from 'azureml._model_management._util' (/anaconda/envs/azureml_py38/lib/python3.8/site-packages/azureml/_model_management/_util.py)
Traceback (most recent call last):
python3.8/site-packages/knack/cli.py, ln 231, in invoke
    cmd_result = self.invocation.execute(args)
...
<frozen importlib._bootstrap>, ln 219, in _call_with_frames_removed
extensions/azure-cli-ml/azext_ml/model.py, ln 7, in <module>
    from ._util import cli_context, collect_input_models, file_stream_to_object, \
extensions/azure-cli-ml/azext_ml/_util.py, ln 54, in <module>
    from azureml._model_management._util import deploy_config_dict_to_obj
ImportError: cannot import name 'deploy_config_dict_to_obj' from 'azureml._model_management._util' (/anaconda/envs/azureml_py38/lib/python3.8/site-packages/azureml/_model_management/_util.py)

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ml model list

Expected Behavior
Environment Summary
Linux-5.4.0-1046-azure-x86_64-with-glibc2.10
Python 3.8.1
Installer: PIP

azure-cli 2.22.1 *
azure-cli-ml 1.28.0

Extensions:
azure-cli-ml 1.28.0

Additional Context

route to service team

Issue was caused by version mismatch between azureml CLI and azureml sdk libraries.  Upgrading the azureml sdk in my conda environment fixed the issue.


An application that demonstrates throttling input when output is slow

@amberarrow please see

This is a very cool example which illuminates a rather obscure corner of the platform -- thanks for writing it.

@amberarrow incorporated all other comments. Thanks for your comments.

Made the change.
On Wed, Nov 9, 2016 at 11:16 AM, amberarrow<EMAIL_ADDRESS>wrote:

@amberarrow commented on this pull request.
In tutorials/throttle/README.md
https://github.com/DataTorrent/examples/pull/42#pullrequestreview-7877879
:

@@ -0,0 +1,8 @@
+Application shows you how to throttle input operators in the application when the downstream
+operators are slower.

+It uses a combination of stats listener and operator request to achieve this. The throttler is
+a stats listener that is registered with the operators and when it notices that the window gap
+between operators is widening and crosses a configured threshold, it sends a request to the
+input operator to slow down and coversely when the gap falls below the threshold it requests
+the input operator to go back to it's normal speed.
"It's" is not quite right, cause it's possessive and so doesn't take an
apostrophe but it's ok for now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DataTorrent/examples/pull/42#pullrequestreview-7877879,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AENqu0q2kT8ZBYSjql8RZNSNAhWylkv9ks5q8hv1gaJpZM4Kqf7E
.


CC3XX: Add AEAD single-part entry points

AEAD enabled in wrapper
Multipart hash temporarily disabled
Needs filled in paths to local filesystem

Signed-off-by: Torstein Grindvik<EMAIL_ADDRESS>

@Vge0rge please rebase onto feature-cc-psa-crypto-drivers