text:
Redis instrumentation reports unknow_service

The Redis instrumentation is reporting unknown service PIPELINED operations, it comes from a Rpush service, the Rpush gem is bundled into the application but Rpush does not run as embeeded process into the Rails application, it runs as a separate process (systemd).

I think these operations should not be logged since Rpush instrumentation would need a dedicated plugin, plus it is out of process (the Rails process).
What is a PIPELINED operation? Does anyone know how to prevent this logging?
Runtime
Operating system details: CentOS Linux release 7.9.2009 (Core)
Rails: 6.1.5.1
RUBY_ENGINE: "ruby"
RUBY_VERSION: "3.2.2"
RUBY_DESCRIPTION: "ruby 3.2.2 (2023-03-30 revision e51014f9c0) [x86_64-linux]"
initializers/opentelemetry.rb
require 'opentelemetry/sdk'
require 'opentelemetry/instrumentation/all'

OpenTelemetry::SDK.configure(&:use_all) unless Rails.env.development

Trace details
process.runtime.description: ruby 3.2.2 (2023-03-30 revision e51014f9c0) [x86_64-linux]
telemetry.sdk.version: 1.4.0
net.peer.port: 6378
process.command: /var/www/xxx/bin/rpush
telemetry.sdk.language: ruby
db.statement: MULTI ZRANGEBYSCORE ? ? ? ZREMRANGEBYSCORE ? ? ? EXEC
process.runtime.name: ruby
process.runtime.version: 3.2.2
process.pid: 6111
net.peer.name: 127.0.0.1
service.name: unknown_service
telemetry.sdk.name: opentelemetry
db.system: redis

I am not sure which third-party or vendor that you use for visualization of the trace, but the operation more like span name to me (as you can see from redis instrumentation v4 and v5). I don't think redis instrumentation has operation attributes like db.operation.
I am not super familiar with Redis, but from the db.statement: db.statement: MULTI ZRANGEBYSCORE ? ? ? ZREMRANGEBYSCORE ? ? ? EXEC, it seems like the rpush is executing mutliple command ZRANGEBYSCORE, and current redis instrumentation has to treat it as a pipeline operation.
For the unknown_service, it's likely the application didn't setup the service name for otel during configuration or through environmental variable. The default service name is unknown_service. To setup the service name:
export OTEL_SERVICE_NAME=the_service_name_you_want

or see https://opentelemetry.io/docs/languages/ruby/getting-started/#instrumentation
OpenTelemetry::SDK.configure do |c|
  c.service_name = 'dice-ruby'
  c.use_all() # enables all instrumentation!
end

Hey!
Thank you for helping me with this!!.
(I'm using Signoz)
yes, you are right but the thing is, Rpush runs as a separate process (out of the main Rails app process), the Rpush gem is bundled with the Rails app because it has the dependency in the Gemfile (for comfort reasons) but it is started using systemd with an specific unit script, this way
/etc/systemd/system/rpush.service
...
START_COMMAND="$RVM_PATH $GEMSET do bundle exec rpush start -p $RPUSH_PID -c $RPUSH_CONFIG -e $R_ENV"
STOP_COMMAND="$RVM_PATH $GEMSET do bundle exec rpush stop -p $RPUSH_PID -c $RPUSH_CONFIG -e $R_ENV"

RPUSH_USAGE="Usage: $0 {start|stop|kill|status|restart}"

rpush_pid() {
  if test -f "$RPUSH_PID"; then
    echo `ps -ef | grep $(<"$RPUSH_PID") | grep -v grep | tr -s " " | cut -d" " -f2`
  fi
}

start() {
  pid=$(rpush_pid)
  if [ -n "$pid" ]; then
    echo -e "Rpush is already running (pid: $pid)"
  else
    echo -e "Starting Rpush..."
    /bin/sh -c "cd $WORKING_DIR && exec $START_COMMAND"
    status
  fi
  return 0
}
...

that's why the redis instrumentation reports an unknown service, because Rpush is not configured.
The main Rails app (running over a Phusion Passenger server) is properly initialized and reports all the instrumented data as expected.
passenger_env_var OTEL_EXPORTER otlp;
passenger_env_var OTEL_SERVICE_NAME staging;
passenger_env_var OTEL_EXPORTER_OTLP_ENDPOINT http://localhost:xxxx;

I understand (i think so xD) how the redis instrumentation works but what I am trying to understand is why it is logging data from an outside process, I did the setup for the Rails application (the initializer above in my previous note) not for Rpush (Rpush is not an embedded process of the Rails application).

After briefly looked at the rpush gem, I think rpush will load the rails again at load_rails_environment. In my simple rails' setup, environment.rb calls Rails.application.initialize!. If that's also your rails' setup, I think the rpush reinitialize the opentelemetry in its process. However, it's just a guessing, not sure if that's the case.

Hey!
oh, makes sense, thank you very much for your help, I will careful take a look to confirm.

Confirmed, thank you for the help!!!

authors: vzamanillo xuan-cao-swi


text:
Move attributes definitions to registry

here is the list of existing attributes to move , please write a comment if you are planning to work on some components so we don't have duplicates

[x] HTTP by @AlexanderWert https://github.com/open-telemetry/semantic-conventions/pull/208
[ ] Network by @AlexanderWert  https://github.com/open-telemetry/semantic-conventions/pull/390
[ ] Client by @AlexanderWert
[ ] Server by @AlexanderWert
[ ] Source by @AlexanderWert
[ ] Destination by @AlexanderWert
[ ] RPC by @AlexanderWert
[x] URL by @trisch-me https://github.com/open-telemetry/semantic-conventions/pull/400
[ ] User agent by @trisch-me https://github.com/open-telemetry/semantic-conventions/pull/418
[ ] Messaging by @trisch-me
[ ] Error by @mlunadia
[ ] Faas
[ ] Event in logs
[ ] DB in metrics
[ ] Container @ChrsMark https://github.com/open-telemetry/semantic-conventions/pull/417
[ ] Cloud by @kaiyan-sheng https://github.com/open-telemetry/semantic-conventions/pull/425

@trisch-me Please add Code and Thread to the above list with the following PR linked:
https://github.com/open-telemetry/semantic-conventions/pull/434

thanks @ChrsMark I will work on Event in logs

Hey folks,
what about service.name, service.namespace etc.? Should they also be included in the registry?
These attributes cannot currently be found within registry and they are not listed within the issue description.

@bripkens Good point!
Actually there are many more namespaces still missing in the registry. I extended the description of this issue above to list all the missing namespaces I found.

Opened a PR for service (moved service_experimental as well): https://github.com/open-telemetry/semantic-conventions/pull/758

Opened one PR for system: https://github.com/open-telemetry/semantic-conventions/pull/867

source: gharchive/issue

authors: AlexanderWert bripkens gregkalapos mlunadia trisch-me

repo: open-telemetry/semantic-conventions

license_type: permissive

license_source: github-api


text:
i18n: Update Chinese translation

Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[x] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[x] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[x] Testing: Have you written and run sufficient tests for validating the changes?
[x] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description

i18n: Update Chinese translation

Thanks!

source: gharchive/pull-request

repo: open-webui/open-webui


text:
Include subscriber-protocol support for the service /v1/notify-link-updates

Note : This issue can be taken after completing the issue https://github.com/openBackhaul/ApplicationPattern/issues/381
As a part of the request body , there is a new attribute “subscriber-protocol” added.
Include support to read and store the new-application-protocol attribute in the tcp-client instance.
Note : Incase of an existing instance update the incoming values if not create a new instance.

This issue is closed by pull request https://github.com/openBackhaul/ApplicationLayerTopology/pull/251

repo: openBackhaul/ApplicationLayerTopology


text:
Clarification regarding the response of the API /v1/inform-about-preceding-release

This API “/v1/inform-about-preceding-release”  is to get the application-name and release-number of the old release of an application.
We know that , in the initial configuration file of all the application , the application-name of the oldRelease is “OldRelease”.
If the registered application is the very first version and don’t have a previous version ,
then,
the application-name of the old-release is OldRelease.
So, the response of the /v1/inform-about-preceding-release will be something like this ,
{
  "application-name": "OldRelease",
  "release-number": "1.0.1"
}

Kindly please let us know shall we produce a result like this ?
(or)
we shall not send a response code!=200 meaning that there is no previous version of this application is available.
Also we shall look into a related issue in RegistryOffice  https://github.com/openBackhaul/RegistryOffice/issues/527

Yes, please produce a result like this:
{
  "application-name": "OldRelease",
  "release-number": "1.0.1"
}

The response code shall relate to the communication and not the content of the message.

Thank you for the clarification. Closing the issue.

authors: PrathibaJee openBackhaul

repo: openBackhaul/ApplicationPattern


text:
Add indication of currently active firmware

Feature request addressed by Tim Gradel.
Problem:
Currently the ALAP is providing information about the firmware on the device.
It regards all banks, but it does not indicate the currently active bank.
Solution:
Information about the currently active firmware shall be added to the responseBody of /v1/provide-acceptance-data-of-link-endpoint.

ALAP is already providing information about the active/standBy status of the Firmware Components. Example:
"installed-firmware": [
{
"firmware-component-name": "Web Server LCT",
"firmware-component-version": "N96129  01.10.02",
"firmware-component-status": "firmware-1-0:FIRMWARE_COMPONENT_STATUS_TYPE_ACTIVE"
},
{
"firmware-component-name": "Boot",
"firmware-component-version": "E82118  01.00.00",
"firmware-component-status": "firmware-1-0:FIRMWARE_COMPONENT_STATUS_TYPE_ACTIVE"
},
{
"firmware-component-name": "IDU Board Bench 2",
"firmware-component-version": "N60052 01.10.02",
"firmware-component-status": "firmware-1-0:FIRMWARE_COMPONENT_STATUS_TYPE_ACTIVE"
},
{
"firmware-component-name": "IDU Board Bench 1",
"firmware-component-version": "N60052 01.10.02",
"firmware-component-status": "firmware-1-0:FIRMWARE_COMPONENT_STATUS_TYPE_STAND_BY"
}
]
Ticket has to be opened in APT, in order for LinkVis to make use of these available but unused Status Informations.

authors: anacabbar openBackhaul


text:
meta node status is always 'none' in the output of 'show cluster'

Describe the bug（Bug 描述）
in the output of show cluster, meta node status is always none, no matter the node is up or dow.
To Reproduce（Bug 复现步骤）
> show cluster
+---------------------+--------+---------------------+--------+----------+
|        time         | status |      hostname       | nodeID | nodeType |
+---------------------+--------+---------------------+--------+----------+
| 1710300701177100770 | none   | 192.168.1.14:8091   |      1 | meta     |
| 1710300701177100770 | none   | 192.168.1.44:8091   |      3 | meta     |
| 1710300701177100770 | none   | 192.168.1.62:8091   |      4 | meta     |
| 1710300701177100770 | failed | 192.168.1.62:8400   |      2 | data     |
| 1710300701177100770 | alive  | 192.168.1.44:8400   |      5 | data     |
| 1710300701177100770 | alive  | 192.168.1.14:8400   |      6 | data     |
+---------------------+--------+---------------------+--------+----------+
5 columns, 6 rows in set
``

The meta status is not supported. It is always none.

The meta status is not supported. It is always none.

Do you mean it is an expected behavior?

The meta status is not supported. It is always none.

Do you mean it is an expected behavior?

是的，获取meta的状态信息还没实现，所以用none来表示。如果show cluster不显示meta，又感觉缺了meta的状态，这样也不太合适。

In this case, I think change it to something like WIP, TBD or Not Implemented would be better because the meaning of these phrases are clear while none is not.

The meta status is not supported. It is always none.

Do you mean it is an expected behavior?

是的，获取meta的状态信息还没实现，所以用none来表示。如果show cluster不显示meta，又感觉缺了meta的状态，这样也不太合适。

Good idea

authors: localvar shilinlee xiangyu5632

repo: openGemini/openGemini


text:
Soccer as Alternative Label for Football

Hi Guys,
i've seen the activity list feed in json.
Just a question about "Soccer": is it valid to consider it as alternative of "Football" activity?
Really thanks a lot.
Rgds.
Paolo.

Yes, that's valid. The synonyms and description fields aren't really complete yet, so if you have suggestions then please file them here.

authors: SportsOpenData ldodds

repo: openactive/activity-list


text:
Manisa Turkey Addresses

http://imarotomasyon.manisa.bel.tr/arcgis/rest/services/MOBIL_NUBIS_MANISA/MapServer/1

I can't figure out where the street names are.  Anyone know Turkish?

The source has a field named YOLORTAHATYONID which I'm pretty sure is the street id.  There's a table named YOLORTAHATYON but there don't seem to be any textual street names that I could find.

authors: justinelliotmeyers trescube

repo: openaddresses/openaddresses


text:
Connecticut statewide data has gaps

The green points represent data coming from the statewide data: http://data.openaddresses.io/runs/244799/cache.csv
As you can see there are some missing areas. Not entire counties, but instead what looks to be towns/cities in CT.

Looks like our convention would be to call this a statewide-partial file instead, and include a coverage geometry that just has the areas in dark green include (so the areas in purple would be "holes" in a multipart polygon).
See Texas for example:

https://github.com/openaddresses/openaddresses/blob/master/sources/us/tx/statewide-partial.json#L3

Here's the Connecticut source:

https://github.com/openaddresses/openaddresses/blob/master/sources/us/ct/statewide.json

When I did a review of Connecticut, I noticed those, but saw the state-wide source, so I thought I was wrong in my initial assessment. I can pull it all down today and see what is exactly missing.

 This is for all sources for Connecticut (*Including the statewide dataset)

ct_numbers.txt

authors: aaaandrea justinelliotmeyers nvkelso


text:
Fix PyTorch version check

So the check works fine with PyTorch 1.10.0.

I would like to suggest changing
if [int(n) for n in torch.__version__.split(".")] < [1, 7, 1]:
to
if [int(n.split("+")[0]) for n in torch.__version__.split(".")] < [1, 7, 1]:
so that it would work for versions with extra flags such as 1.10.0+cu102 or 1.10.0+cpu

as mentioned by @jiahuei , I got the error when using the latest clip
ValueError: invalid literal for int() with base 10: 'dev20210803'`

as my torch version is 1.10.0.dev20210803+cu111

@elliottzheng the version check should work now

@jiahuei great

Got a problem when torch version is 1.9.0a0+2ecb2c7

repo: openai/CLIP

license: mit

license_source: bigquery


text:
Process finished with exit code 137 (interrupted by signal 9: SIGKILL)

When training on DQN + pybullet , after about 40M timesteps, process will be killed by code 137.
The observation of env is a image input with the shape of 128x128x4 , I think there is plenty of memory on  PC , about 30G.

Can you track the memory usage and make sure that's not the issue? That would be my first guess, especially when observations are that large (that is roughly 2x the number of pixels as for Atari).

repo: openai/baselines


text:
Training Issue

I tried to train a DDPM to produce cat images. But it seems like the training process does not work properly. Can anyone tell me what happened? No error is reported.
I am using Wsl2.
Here is the code:
train_ddpm.sh
#!/bin/bash
MODEL_FLAGS="--image_size 64 --num_channels 128 --num_res_blocks 3"
DIFFUSION_FLAGS="--diffusion_steps 4000 --noise_schedule cosine"
TRAIN_FLAGS="--lr 1e-4 --batch_size 128"
python scripts/image_train.py --data_dir cats/ $MODEL_FLAGS $DIFFUSION_FLAGS $TRAIN_FLAGS
console outputs:
(Generator) root@LAPTOP-I6AJJ63E:/mnt/d/CatPicGenerateDDPM/improved-diffusion# ./train_ddpm.sh
Logging to /tmp/openai-2022-07-20-21-25-58-111937
creating model and diffusion...
creating data loader...
training...
./train_ddpm.sh: line 5:   455 Killed                  python scripts/image_train.py --data_dir cats/ $MODEL_FLAGS $DIFFUSION_FLAGS $TRAIN_FLAGS
Thank you very much!

Well, I figure out the problem. I did not set mircobatch while only having 1 GPU. It works after setting mircobatch.

repo: openai/improved-diffusion

license: MIT


text:
Realtime API example for local development environment

Summary
This example will guide developers through the process of integrating the OpenAI Realtime platform into a Node.js application that runs locally on mac (or other device) for development. It serves as a gateway for developers looking to interact with OpenAI's (supported) GPT models in real-time, using audio input and output to create a conversational assistant.
In this tutorial, developers will learn how to use WebSocket communication to connect to OpenAI's realtime API, stream audio input, and receive responses in real time. This integration provides a way to implement dynamic interactions that utilize GPT's natural language processing capabilities in audio form, making it well suited for conversational AI, live assistant applications, and other real-time use cases.
I have provided examples with a raw websocket as well as the latest openai node module which makes the integration more straightforward (if you're operating in node).
This code sample is also a great way to create a development environment that allows developers to test their own use cases locally, facilitating rapid prototyping and iteration.
Motivation
This example is an easy way for developers to build out functions, and other integrations into the realtime platform using websockets (a non-trivial mechanism) and the latest openai node module! Documentation of the realtime service is scarce both on the OpenAI website and in the community, so this example will help developers get started with the platform! 🚀

For new content
When contributing new content, read through our contribution guidelines, and mark the following action items as completed:

[x] I have added a new entry in registry.yaml (and, optionally, in authors.yaml) so that my content renders on the cookbook website.
[x] I have conducted a self-review of my content based on the contribution guidelines:

[x] Relevance: This content is related to building with OpenAI technologies and is useful to others.
[x] Uniqueness: I have searched for related examples in the OpenAI Cookbook, and verified that my content offers new insights or unique information compared to existing documentation.
[x] Spelling and Grammar: I have checked for spelling or grammatical mistakes.
[x] Clarity: I have done a final read-through and verified that my submission is well-organized and easy to understand.
[x] Correctness: The information I include is correct and all of my code executes successfully.
[x] Completeness: I have explained everything fully, including all necessary references and citations.

cc @khorwood-openai for 👀 as the primary developer on the npm package.

repo: openai/openai-cookbook


text:
Added token calculation using tiktoken for gpt 4 vision and code update for latest version 1.3.5

Summary
Briefly describe the changes and the goal of this PR. Make sure the PR title summarizes the changes effectively.

Added token calculation using tiktoken for gpt 4 vision for both image urls and base64 images
Modified code to support latest version of openai 1.3.5

Motivation
Why are these changes necessary? How do they improve the cookbook?

Previously there was no support for input image prompt token calculation
The code was using older version of openai which was incorrect when executed using latest version

For new content
When contributing new content, read through our contribution guidelines, and mark the following action items as completed:

[x] I have added a new entry in registry.yaml (and, optionally, in authors.yaml) so that my content renders on the cookbook website.
[x] I have conducted a self-review of my content based on the contribution guidelines:

[x] Relevance: This content is related to building with OpenAI technologies and is useful to others.
[x] Uniqueness: I have searched for related examples in the OpenAI Cookbook, and verified that my content offers new insights or unique information compared to existing documentation.
[x] Spelling and Grammar: I have checked for spelling or grammatical mistakes.
[x] Clarity: I have done a final read-through and verified that my submission is well-organized and easy to understand.
[x] Correctness: The information I include is correct and all of my code executes successfully.
[x] Completeness: I have explained everything fully, including all necessary references and citations.

We will rate each of these areas on a scale from 1 to 4, and will only accept contributions that score 3 or higher on all areas. Refer to our contribution guidelines for more details.

Several new models came out in the last month, including gpt-4-1106-preview. It would be nice to update the token counting example to include these new models.


text:
Double-counted tokens?

OpenAI team,
First off, hats off to you for this awesome API and model.
Second, I'd like to bring up some observations because I'm not sure if they are intended or not.
Assuming the default modalities (text, audio) are selected, as the conversation goes on, for each response, all previous conversation items are counted as both input audio tokens and input text tokens.
Now, we can turn off the user input transcription (off by default), which would take care of the input text tokens coming from the user speech transcription, but we have no way of disabling assistant response transcription. So, we end up being charged for both text and audio tokens.
See below an example of a 30-turn interaction (with input_audio_transcription enabled) (and $10 down the drain 😅)
We paid $0.05 for the just the first question, $0.22 for just the 10th, $0.42 for the 20th, $0.58 for the 30th, etc.
If I so much as sneezed after that, it'd have cost me over half a dollar (~2 min of audio, according to the advertised rate.)
My main question is:
Why do we have to pay for both assistant text and audio tokens by default? Does the model need both? (doesn't seem so)
Now, we technically have the ability to delete conversation items and create new ones, so there are definitely ways to deal with the cost. But, they're cumbersome and full of edge cases due to the realtime nature of it.
If you can't "just make it super cheap so we don't have to care", maybe there is some low hanging fruit?
Can we have some easier control over how much of the conversation we want to keep around?
Can we have a conversation.item.update event so that we don't have to recreate the entire conversation tree when we just want to remove the transcription?
Something else better I'm sure you've already cooked up?
Thank you!
Turn 1
User: (audio) What's the weather in New York?
Assistant: (audio) I don't have access to real-time data, including current weather updates. However, you can check the latest weather in New York by looking it up on a weather website or using a weather app on your phone.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
16
236
8
62
$0.05

Total
16
236
8
62
$0.05

Turn 2
User: (audio) What's the weather in San Francisco?
Assistant: (audio) I don't have access to real-time weather updates. For the current weather in San Francisco, please check a weather website or use a weather app on your phone.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
275
173
80
48
$0.06

Total
291
409
88
110
$0.11

Turn 3
User: (audio) What's the weather in London?
Assistant: (audio) I don't have access to real-time data, including current weather updates. To get the latest weather in London, please check a weather website or use a weather app on your phone.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
465
207
138
54
$0.09

Total
756
616
226
164
$0.20

Turn 4
User: (audio) What's the weather in Toronto?
Assistant: (audio) I can't provide current weather updates. For the latest weather in Toronto, please check a weather website or use a weather app on your phone.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
690
169
202
44
$0.10

Total
1446
785
428
208
$0.31

Turn 5
User: (audio) What's the weather in Istanbul?
Assistant: (audio) I don't have access to real-time information, so I can't provide the current weather in Istanbul. You can check the latest weather there by using a weather website or a weather app.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
877
213
256
54
$0.13

Total
2323
998
684
262
$0.44

Turn 6
User: (audio) What's the weather in New York?
Assistant: (audio) I can't provide real-time weather updates. To find out the current weather in New York, please check a weather website or use a weather app.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
1106
163
320
45
$0.15

Total
3429
1161
1004
307
$0.59

Turn 7
User: (audio) What's the weather in San Francisco?
Assistant: (audio) I can't provide real-time weather updates. To find out the current weather in San Francisco, please check a weather website or use a weather app.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
1292
172
375
45
$0.17

Total
4721
1333
1379
352
$0.75

Turn 8
User: (audio) What's the weather in London?
Assistant: (audio) I can't provide real-time weather updates. To find out the current weather in London, please check a weather website or use a weather app.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
1481
163
430
44
$0.18

Total
6202
1496
1809
396
$0.94

Turn 9
User: (audio) What's the weather in Toronto?
Assistant: (audio) I can't provide real-time weather updates. To find out the current weather in Toronto, please check a weather website or use a weather app.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
1662
166
484
44
$0.20

Total
7864
1662
2293
440
$1.14

Turn 10
User: (audio) What's the weather in Istanbul?
Assistant: (audio) I can't provide real-time weather updates. To find out the current weather in Istanbul, please check a weather website or use a weather app.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
1846
172
538
44
$0.22

Total
9710
1834
2831
484
$1.36

Turn 11
User: (audio) What's the weather in New York?
Assistant: (audio) I can't provide real-time weather updates. To find out the current weather in New York, please check a weather website or use a weather app.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
2034
164
592
45
$0.24

Total
11744
1998
3423
529
$1.60

Turn 12
User: (audio) What's the weather in San Francisco?
Assistant: (audio) For the most current weather in San Francisco, please check a reliable weather website or a weather app on your device. They will have the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
2221
189
647
48
$0.26

Total
13965
2187
4070
577
$1.87

Turn 13
User: (audio) What's the weather in London?
Assistant: (audio) To get the current weather in London, please check a reliable weather website or a weather app on your device for the most up-to-date information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
2427
177
705
45
$0.28

Total
16392
2364
4775
622
$2.15

Turn 14
User: (audio) What's the weather in Toronto?
Assistant: (audio) For the latest weather in Toronto, please check a weather website or a weather app on your phone for real-time updates. They will have the information you need.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
2622
195
760
50
$0.31

Total
19014
2559
5535
672
$2.45

Turn 15
User: (audio) What's the weather in Istanbul?
Assistant: (audio) For the current weather in Istanbul, please refer to a weather website or a weather app on your device. They will have the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
2835
168
820
44
$0.32

Total
21849
2727
6355
716
$2.78

Turn 16
User: (audio) What's the weather in New York?
Assistant: (audio) To get the current weather in New York, please check a reliable weather website or a weather app on your device. They will have up-to-date information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
3019
180
874
47
$0.34

Total
24868
2907
7229
763
$3.12

Turn 17
User: (audio) What's the weather in San Francisco?
Assistant: (audio) To get the current weather in San Francisco, please check a reliable weather website or a weather app on your device for the most up-to-date information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
3222
190
931
46
$0.37

Total
28090
3097
8160
809
$3.49

Turn 18
User: (audio) What's the weather in London?
Assistant: (audio) To find out the current weather in London, please check a reliable weather website or use a weather app on your device. They will have the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
3429
175
987
47
$0.38

Total
31519
3272
9147
856
$3.87

Turn 19
User: (audio) What's the weather in Toronto?
Assistant: (audio) For the latest weather in Toronto, please check a weather website or a weather app. They will have up-to-date information on the current conditions.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
3622
167
1044
45
$0.40

Total
35141
3439
10191
901
$4.27

Turn 20
User: (audio) What's the weather in Istanbul?
Assistant: (audio) To get the current weather in Istanbul, please check a reliable weather website or a weather app on your device for the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
3807
155
1099
42
$0.42

Total
38948
3594
11290
943
$4.69

Turn 21
User: (audio) What's the weather in New York?
Assistant: (audio) To get the current weather in New York, please check a reliable weather website or use a weather app on your device. They will have the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
3978
179
1151
47
$0.44

Total
42926
3773
12441
990
$5.13

Turn 22
User: (audio) What's the weather in San Francisco?
Assistant: (audio) To find out the current weather in San Francisco, please check a reliable weather website or a weather app on your device. They will have the most up-to-date information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
4180
188
1208
52
$0.46

Total
47106
3961
13649
1042
$5.59

Turn 23
User: (audio) What's the weather in London?
Assistant: (audio) To get the current weather in London, please check a reliable weather website or use a weather app on your device. They will have the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
4385
176
1270
48
$0.48

Total
51491
4137
14919
1090
$6.07

Turn 24
User: (audio) What's the weather in Toronto?
Assistant: (audio) To find out the current weather in Toronto, please check a reliable weather website or use a weather app on your device for the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
4579
164
1328
44
$0.50

Total
56070
4301
16247
1134
$6.57

Turn 25
User: (audio) What's the weather in Istanbul?
Assistant: (audio) To get the current weather in Istanbul, please check a reliable weather website or a weather app on your device for the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
4761
145
1382
42
$0.51

Total
60831
4446
17629
1176
$7.08

Turn 26
User: (audio) What's the weather in New York?
Assistant: (audio) To find out the current weather in New York, please check a reliable weather website or use a weather app on your device. They will provide the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
4922
178
1434
48
$0.54

Total
65753
4624
19063
1224
$7.62

Turn 27
User: (audio) What's the weather in San Francisco?
Assistant: (audio) For the latest weather in San Francisco, please check a weather website or a weather app on your device. They will have the most current information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
5123
180
1492
45
$0.56

Total
70876
4804
20555
1269
$8.18

Turn 28
User: (audio) What's the weather in London?
Assistant: (audio) To get the current weather in London, please check a reliable weather website or use a weather app on your device for the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
5320
142
1547
43
$0.57

Total
76196
4946
22102
1312
$8.75

Turn 29
User: (audio) What's the weather in Toronto?
Assistant: (audio) To find out the current weather in Toronto, please check a reliable weather website or use a weather app on your device. They will have the latest information.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
5480
176
1600
47
$0.59

Total
81676
5122
23702
1359
$9.34

Turn 30
User: (text) What was the second city I asked about?
Assistant: (audio) The second city you asked about was San Francisco.

Tokens
In Audio
Out Audio
In Text
Out Text
Cost

Current
5656
53
1662
20
$0.58

Total
87332
5175
25364
1379
$9.92

Total Cost: $9.92

I also noted this and conversed with @dnakov over Twitter about it - realtime API costs are much higher than expected and his code saves on it significantly. His repo is here with the "optimizations" to reduce cost:
https://github.com/dnakov/realtime-api-cost

Pushing this upward. I did not do a record on the findings.
However, also realize that if I go to have a chat on Playground for 5 minutes. It cost only around $1. But doing it with the realtime console, it costs $2-4. I am using the same prompt on both versions.
For more information, I am also using the relay.

It's the same, playground or not.

can I ask that how did you get the cost tokens from this api? I wanted to do this but I failed to find a server event that contains the cost.

@spectaclecs take a look at PR #37

#37 needs to be added, it cost me ~$10 for 5 minute conversation lol.  Gotta stick to concurrent processing for now...

Bumping this up!

The increasing costs is just the conversation history that's building up in the thread. OpenAI doesn't know what your scenario is so they don't know how much conversation history to track.  You need at least 2 - 3 turns for features like co-referencing to work. Questions like "how much is that in euros?" wont work because the model won't know what "that" is referring to... Depending on your scenario though you might need 20 turns of conversation history. Lets say your playing a game of 20 questions... The point is OpenAI doesn't know your scenario so they track as many turns as they can fit into the context window which can be expensive.
A feature that lets you configure the number of turns to keep could be handy but you've already shared the code above for how to trim the conversation history yourself.

I don't think that is correct @Stevenic - look at the video demo @dnakov has posted to Twitter showing that it can remember the history even with the optimized/cutdown version:

https://x.com/dnak0v/status/1842685544423182631

authors: FuyukiSakura Stevenic Xoni30 dnakov kyleboddy mhamdan91 spectaclecs

repo: openai/openai-realtime-console


text:
Output File Issue

I want output in .gif or .tiff format.
Anyone can help me to get the output in the format.
I am using Jupiter in anaconda software.

Hi! You can output data in GeoTIFF format by setting the grid output format configuration key to geotiff. Like this in the YAML file:
output_data:
  grids:
    format: geotiff

repo: openamundsen/openamundsen


text:
Sys.getenv("SHINYPROXY_USERGROUPS")) returns empty string

Hello,
I was trying to condition some part of my app on certain user groups, however if try to retrieve the user groups of the logged in user I only get an empty object:
Sys.getenv("SHINYPROXY_USERGROUPS"))
If I try to access the user via:
print(Sys.getenv("SHINYPROXY_USERNAME"))
it works perfectly.
To give a reproducible example, my application.yml looks as follows:
proxy:
  port: 8081
  authentication: simple
  admin-groups: admins
  container-log-path: home/felix/container-logs
  users:
  - name: jack
    password: password
    groups: admins
  - name: jeff
    password: password
    groups: pv_qs
  - name: jane
    password: password
    groups: [management, test]
  docker:
      internal-networking: true
  specs:
  - id: 01_hello
    display-name: Hello Application
    description: Application which demonstrates the basics of a Shiny app
    container-cmd: ["R", "-e", "shinyproxy::run_01_hello()"]
    container-image: openanalytics/shinyproxy-demo
    container-network: sp-example-net
  - id: 06_tabsets
    container-cmd: ["R", "-e", "shinyproxy::run_06_tabsets()"]
    container-image: openanalytics/shinyproxy-demo
    container-network: sp-example-net
  - id: 02_lifecycle
    display-name: Lifecycle
    container-cmd:  ["R", "-e", "shiny::runApp('/root/lifecycle')"]
    container-image: lifecycle
    container-network: sp-example-net

logging:
  file:
    shinyproxy.log

Is this a known bug? Any hints for a solution?
Much appreciated.

I have to correct myself.
It only returns nothing, if I assign at least two groups to the user like I did for Jane.
Another "bug" I have discovered:
When I write "management", everything in small letters, I get in shiny "MANAGEMENT" everything in capital letters. This is very misleading, if condition on this value and if this is not anywhere mentioned in your doc.

Ok, I see that in contrast to the specification of access-groups, I have to separate groups only with a comma. Well, an example in your docs would be good for this...
But the uppercase error persists and is not happening for the user name.

Thanks for the feedback, @panoptikum. We will look into it.

Thanks.
I find it confusing that I have to specify groups for a user as follows:
groups = engineers, scientists
while I specify access-groups for an app as:
access-groups = [engineer, scientists]
Maybe this way to specify groups for a user is incorrect, but [engineers, scientists] does not work.
For clarity: I use shinyproxy 2.3.3

Hi, I just released ShinyProxy 3.0.1 which fixes this.
You can now use array notation for the groups of a user.

authors: LEDfan panoptikum tverbeke

repo: openanalytics/shinyproxy


text:
Fix photos that incorrectly showed yocto

Fix photos in new mac setup guide that showed yocto instead of jubilinux

heh - I was hoping someone would catch and swap those out :) Thank you!

repo: openaps/docs


text:
Annotations are not placed on the exact position of the playhead

Annotations are inserted on the nearest second and not on the exact playback time. This means that the desired playback time is not displayed when selecting the annotation. See tube.switch.ch/videos/fef0e56b

Looks fine on SWITCHcast-Staging. Thx!

repo: opencast/annotation-tool


text:
Enhancement: Enable cast_spaces fixer

This PR

[x] enables the cast_spaces fixer
[x] runs make cs

Replaces #584.
Follows #587.
💁♂ For reference, see https://github.com/FriendsOfPHP/PHP-CS-Fixer/tree/v2.8.0#usage:

cast_spaces [@Symfony]
A single space or none should be between cast and variable.
Configuration options:

space ('none', 'single'): spacing to apply between cast and variable; defaults to 'single'

Thank you, @chartjes!

repo: opencfp/opencfp


text:
NextJS for Server, great! But PHP! WHAT?!?

Is this some sort of "obfuscation" by "mature tech usage" scheme?!?! 🤪
Would love if this was all in NextJS.
Just curious on the devs choice of PHP!

php for web service is ok, i think it is  convenient and efficient. And, the core is llm-server in this project.

Hey, PHP only used on the backend-server side, not llm-server, it should be straight forward to get onboard (standard Laravel MVC), yet the project is still new, and I am open to re-write the backend server in a different stack, do you have any thing on mind?

Laravel is great! Don't change it :+1:

Maybe I should learn Laravel! I have heard great things. Just confused me initially seeing both NextJS and PHP is all. No negativity towards devs at all!

authors: gharbat nhenderson rendao strangelearning

repo: openchatai/OpenChat


text:
[Paper] Thunderstorm nowcasting with deep learning:a multi-hazard data fusion model

https://arxiv.org/pdf/2211.01001.pdf
Detailed Description
This paper describes ML models for predicting thunderstorms, taking in very similar data to MetNet, and dealing with certain modalities dropping out while still working, and gives probabilistic forecasts. Its forecasting 60min ahead at 5 minute intervals.
They did find the satellite data and radar were the most important inputs to the model:

Context
This seems like its trying to do a similar thing to MetNet/MetNet-2, with similar inputs, so might be helpful with expanding MetNet or adding it as a new option.
Possible Implementation
Code: https://github.com/MeteoSwiss/c4dl-multi
Data: https://zenodo.org/record/6802292
Pretrained models: https://zenodo.org/record/7157986

@JackKelly @peterdudfield Some more support for satellite being quite important for short term forecasting at least. Not solar, but they did use EUMETSAT data

https://arxiv.org/pdf/2203.10114.pdf has more details, and is the lightning-only original work

repo: openclimatefix/metnet


text:
Add Sun filter

Would be good to make sure the forecasts are zero at night
TODO check:
Sun Data Source in nowcasting_dataset only produces one elevation for each t0, independent of the forecast horizon.
Tasks:

Add sun to config mvp 2.

Then in model code, perhaps here. Then it would work for all models

Filter the forecasts to be zeros is elevation < X.

Small Problem
At 04:00 the elevation < 0, but at 05:00 there could be solar generation. Maybe there is a way to filter < -5, which might stop the problem at dawn.
Other ideas:

Calucate sun elevation on the fly, and filter forecasts for this. Here writing a function like this would be good

def filter_forecast_on_sun(forecast:ForecastSQL):

     # get location

    # get elevation at all target times

     # zero out if below a certina value

return forecast

adjust SunDataSource, so that values are caluclated for all timesteps, not just t0

Thanks @JackKelly, ok, the we should give pvlib a try.
@jacobbieker would you be up for this?

Yeah, that sounds good, I can tackle it.

FWIW, here's how power_perceiver computes the Sun's azimuth and elevation for each t0 timestep of each example in a batch on-the-fly: https://github.com/openclimatefix/power_perceiver/blob/main/power_perceiver/np_batch_processor/sun_position.py#L50

Yeah, that sounds good, I can tackle it.

Thanks!
Although, as I propose in this comment, I'd suggest that you first give the model the Sun's azimuth and elevation. If you're lucky, that will be sufficient, and you won't have to "manually" filter the output of the model by the Sun's elevation :slightly_smiling_face:

authors: JackKelly jacobbieker peterdudfield


text:
Solve issue number 3

There is a little refactor in documentation needed as per requested in the issue.
Solve issue #3

Thank you for jumping on this. What's the point of linking to a search page in that case?
https://opencollective.com/search?q=open collective
Can you double check the issue you're trying to solve?

Okay, I am getting your point thanks for this, can you suggest where should I link it.

The link could be https://opencollective.com/opencollective
But actually, the string should be saying that we run the company trough multiple Open Collectives, and link to https://opencollective.com/opencollectiveinc

Okay, so this https://opencollective.com/opencollectiveinc should replace https://opencollective.com/search?q=open collective
Am I right?

Yes. But the string should also be updated.

Okay, now I got it making a PR for it soon.

@TG1999 would be nice to have a more descriptive title for the PR, explaining the change itself.

Thanks for pointing out this I will be doing it
On Thu 2 May, 2019, 18:09 François Hodierne, notifications@github.com
wrote:

@TG1999 https://github.com/TG1999 would be nice to have a more
descriptive title for the PR, explaining the change itself.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opencollective/documentation/pull/8#issuecomment-488656514,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIEUAIH7OHPDSDYOP3GGKG3PTLOJTANCNFSM4HJ5XY7Q
.

Hi, @znarf I have changed the PR description please merge it if it looks good.

repo: opencollective/documentation


text:
every time you edit an event it automatically apends '-442ev' to the URL

to test it create an event, publish it and then edit it and save again.
It automatically creates a new url for the event, and the original one renders a grapghql error.

closing, this is a feature not a bug 🤷♀

repo: opencollective/opencollective


text:
Order success: bad tags alignment with long tags

We should text-align: center the tags to avoid the following issue:

In the end, what's the value of tags here?

To easily find related collectives. With the example of the C# community, still small on OC but growing, we can make the assumption that users will be tempted to click on a "csharp" tag to find other collectives they know and would be likely to support.

authors: Betree znarf


text:
cannot accept financial contributions via Stripe for a new event in new org

Describe the bug
When I make a new event and try to set the way I accept financial contributions, clicking on "Connect Stripe" yields no visible response, and the console logs several errors (which happen for each button click):
instrument.js:109 [GraphQL error]: Message: Account must be a collective or a fund, Location: [object Object], Path: applyToHost
...
AcceptContributionsOurselvesOrOrg.js:120 Uncaught (in promise) Error: Account must be a collective or a fund
    at AcceptContributionsOurselvesOrOrg.js:120:13
...

My (uninformed) guess is that this is happening because the code doesn't handle cases where the account is an "event." So perhaps the fix is either to not allow financial contributions to be set per-event, or to handle the case.
P.S. With a little guidance about the expected behavior, I'd be happy to try and fix this bug 🙂
To Reproduce
Steps to reproduce the behavior:

Make a new collective with no finances set
Make a new event with no finances set
Go to the event page and try to set a method to accept financial contributions for it (example URL: https://opencollective.com/testing-event-855924df/accept-financial-contributions/ourselves)
Click "Connect Stripe"
Click does not work

Expected behavior
Should be able to accept financial contributions via Stripe for the event
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: macOS Version 12.0.1 (Build 21A559)
Browser: chrome
Version: 100.0.4896.75 (Official Build) (arm64)

Smartphone (please complete the following information):
N/A
Additional context
Add any other context about the problem here.

Triage Template (core team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

I also get a similar error when I try to add a bank account for the event instead: ("Account must be a collective or fund" error message + console error appears when I try to submit bank details, although they appear to be saved in the org now)

@k-qy Thanks for documenting this issue.
Stripe/Bank account must be connected from the event's parent. We must make sure that "Accept financial contributions" links to /:parent/accept-financial-contributions, not /:event/accept-financial-contributions in this case.
@k-qy As a workaround, I recommend going on the parent's profile to set things up.

thanks for the fix/workaround @Betree!

authors: Betree k-qy


text:
Not able to build

Installed the prerequisites and tried building using command 'bash build.sh'. It was doing nothing. It exited immediately without printing any messages.

TX akashgj, Fixed the Release notes and added the missing Cerberus zephyr porting files.

authors: akashgj presannar

repo: opencomputeproject/Tektagon-OpenEdition


text:
fixed isis_change_lsp_lifetime test by increasing timer

Added a wait after pushing the config to the OTG
Increases wait time for route check

Pull Request Functional Test Report for #3568 / 6a000abe58e734e23d74e2e8fb2e1945d381f45f
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-2.10: IS-IS change LSP lifetime
25502760
Log

Cisco 8000E

RT-2.10: IS-IS change LSP lifetime
af0ef6ea
Log

Cisco XRd

RT-2.10: IS-IS change LSP lifetime
6f347801
Log

Juniper ncPTX

RT-2.10: IS-IS change LSP lifetime
f24e4ca3
Log

Nokia SR Linux

RT-2.10: IS-IS change LSP lifetime
47ce6abe
Log

Openconfig Lemming

RT-2.10: IS-IS change LSP lifetime
35c0b3ee
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-2.10: IS-IS change LSP lifetime

Cisco 8808

RT-2.10: IS-IS change LSP lifetime

Juniper PTX10008

RT-2.10: IS-IS change LSP lifetime

Nokia 7250 IXR-10e

RT-2.10: IS-IS change LSP lifetime

Help

Pull Request Test Coverage Report for Build 11736398123
Details

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

Totals

Change from base Build 11733511880:
0.0%

Covered Lines:
1983

Relevant Lines:
3588

💛  - Coveralls

authors: OpenConfigBot coveralls ram-mac

repo: openconfig/featureprofiles


text:
Bump version v0.2.0

#142 from Mashimiao/tfix-format
#144 from q384566678/up-validate
#140 from Mashimiao/fix-help-info
#148 from q384566678/up-runtime-spec
#132 from q384566678/fix-glide
#139 from Mashimiao/add-bash-completion
#135 from erikh/fix-tar-files
#145 from q384566678/zhou-hao
#134 from erikh/fix-logger
#137 from q384566678/value-test
#138 from jonboulle/master
#133 from Mashimiao/makefile-add-missing-evars
#130 from q384566678/zz-test
#136 from squeed/tar-streams
#128 from q384566678/fix-man
#127 from q384566678/reandme-fix
#126 from ddysher/update-vendor
#124 from xiekeyang/logrus
#103 from q384566678/main-test
#123 from q384566678/fix-list
#121 from q384566678/bump-version
#119 from Mashimiao/fix-ignore

LGTM

@q384566678 You need to respond with your LGTM on the mailing list thread for this PR before it will get merged.

@opencontainers/image-tools-maintainers according to vote result https://groups.google.com/a/opencontainers.org/forum/#!topic/dev/C2kFyvZQ_6A, so far 8/11 maintainers agree this release. PTAL. Thanks.

Bump.

LGTM

Does someone want to push a v0.2.0 tag for 7f277bf?

@wking I'm doing.

push tag to https://github.com/opencontainers/image-tools/tree/v0.2.0

On Mon, Jul 10, 2017 at 11:38:44PM -0700, xiekeyang wrote:

push tag to https://github.com/opencontainers/image-tools/tree/v0.2.0

I see a v0.2.0 branch 1, but not a v0.2.0 tag 2.  I think you
want to delete the branch and create a tag.  Something like:
$ git branch -D v0.2.0
$ git push origin :v0.2.0
$ git tag -s v0.2.0 7f277bf
$ git push origin v0.2.0
and then write some release notes in there (e.g. see 3).

ping @xiekeyang

@xiekeyang Can you fix this ?

Sorry, fixing...

Sorry to late, released on https://github.com/opencontainers/image-tools/releases/tag/v0.2.0

repo: opencontainers/image-tools


text:
Segmentation Fault Using cv2.connectedComponentsWithStats

Expected behaviour
Returns valid outputs corresponding to the following line of code given a numpy.ndarray binary image (black = 0, white = 255) of uint8 pixels:
self.count, self.mask, stats, centroids = cv2.connectedComponentsWithStats(
    cc_img.astype(np.uint8)), connectivity=connectivity
)

Actual behaviour
After raising my minimum allowed opencv-python version from 4.5.1 to 4.5.4, this error emerged.
Steps to reproduce

example code

import numpy as np

X = np.random.randint(0, 1, (100, 100))
X_img = (X * 255).astype(np.uint8)

import cv2
count, mask, stats, centroids = cv2.connectedComponentsWithStats(X_img, connectivity=4)

Result: Segmentation fault: 11

operating system

MacOSX "Darwin", Kernel Version 20.6.0

architecture (e.g. x86)

x86_64

opencv-python version

opencv-python                     4.5.5.62
Issue submission checklist

[x] This is not a generic OpenCV usage question (looking for help for coding, other usage questions, homework etc.)

[ ] I have read the README of this repository and understand that this repository provides only an automated build toolchain for OpenCV Python packages (there is no actual OpenCV code here)

[ ] The issue is related to the build scripts in this repository, to the pre-built binaries or is a feature request (such as "please enable this additional dependency")

[ ] I'm using the latest version of opencv-python

I am not sure this is the right repo to submit this issue, let me know and I will re-submit on the opencv project.
Unfortunately, in order to submit in the main opencv project, I have to refactor this reproducible example into C++ code (likely on this work laptop to trigger the bug).  With that in mind, I am submitting here first -- apologies if it is the wrong project.

Already submitted everywhere. Duplicate.

faced this same issue

authors: alalek chrisdonlan k-for-code

repo: opencv/opencv-python


text:
SPIKE: Investigate building images genericly using RHTAP

OpenShift CI will be replaced by Red Hat Trusted Application Pipelines (RHTAP) to build images. We need to familiarize ourselves with that new build system to collect some ideas on how to effectively build Data Science Pipelines images, as well as handle exceptions like https://github.com/opendatahub-io/data-science-pipelines-operator/issues/142.

Document with some findings about RHTAP: https://docs.google.com/document/d/1b6aoP_rWkt8UuZgBDVwYR8bd9m_EzqQWNRjQafEF04Y/edit?usp=sharing

lgtm

authors: HumairAK gmfrasca rimolive


text:
[Bug]: Imported Custom Notebook image is not listed in the Image selection dropdown

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
After successfully importing a notebook image on the BYON page, the notebook does not appear in the Image Selection dropdown box of the Create workbench page. However, when launching JupyterHub using the Launch application of the Jupyter tile under Application->Enabled, the imported image appears as a new radio button.
The image stream is created successfully after import. The backend ImageStream API also returns all the ImageStreams, including the newly imported ones.
After a quick investigation, I learned that the FrontEnd dashboard app does not include ImageStreams if it does not have the following annotation:
 opendatahub.io/notebook-image-order: <any-number>
If I manually add this annotation to the ImageStream resource, it then appears in dropdown list.
When importing Notebook images, this annotation is not added to the ImageStream object.
Expected Behavior
After importing a Notebook image, it should be available in the Image selection dropdown list of the Create Workbench page.
Steps To Reproduce

Under Settings>Notebook Images, import any notebook images
Go to any Data Science projects and click Create Workbench
Expand the Image selection dropdown list
You will not find the Notebook images that you just imported

Workaround (if any)
Add the following annotation to the ImageStream object:
opendatahub.io/notebook-image-order: 1
What browsers are you seeing the problem on?
Chrome
Open Data Hub Version
Red Hat OpenShift Data Science 1.27.0

Anything else

I just realized that the new image is actually in the list. But it's not visible until you scroll down with the mouse wheel while your mouse pointer is on the dropdown list.
It adds the new notebook image to the dropdown list but the list height does not grow taller, and there is no scroll bar displayed.

@lucferbux @kywalker-rh A scrollbar added to the images dropdown list in frontend code UI of odh dashboard workbench sounds like a good idea to avoid these kinds of confusions.

cc @kywalker-rh -- I believe an intern did the original mocks, but Kyle was part of the effort for the design behind the single dropdown.
I think when we implement the desired changes behind the UX work for custom notebooks to include OOTB images, the admin will be able to sort the order of the images and disable OOTB ones from showing up. This should help significantly with the effort behind "finding the one I want" -- be it custom or OOTB. The work will be behind https://github.com/opendatahub-io/odh-dashboard/issues/1266

The scrollbar on Mac OS is actually a user setting, see the essay here https://support.vagaro.com/hc/en-us/articles/204347160-Disable-Disappearing-Scroll-Bars-Mac.
When I turn it on, I can see it...

I think there is nothing we can do to always show the scroll bar on the Mac OS if the user is not setting that up.

@kywalker-rh We might need some UX support here. When we are in the dropdown and the height of the dropdown menu is limited (because the list could be very long), is there a good way to indicate that we still have some content if you scroll down? (the scrollbar could be hidden by the system settings)

@DaoDaoNoCode I think ultimately we need to see if the PatternFly team has dealt with this in the past, but first, are we restricting the dropdown to a specific size? I.e. Do we let it get larger if there are more images?
And a follow-on to that, if we are restricting it to a specific size, is that based on PF guidance, or another reason?

@kywalker-rh There could be a lot of images in the list. If we let the dropdown grow along with the image number, it could break the page because there could be an extremely long dropdown menu.

@kywalker-rh do we know if patternfly has some indicator to deal with scrolling behaviour? macOS hides the scrollbar by default to mimic what happens in touch based inputs (iOSification of the desktop), it's a pain in the ass sometimes but we shouldn't override that behaviour.

How about setting the height of the dropdown box to be a little taller or shorter so it always shows a little bit of the next item? Say if you have 4 items, set the height to 3.5 items tall. This way the user is aware that there are more items below.

@rossbrigoli the content is somewhat dynamic in height -- you can have two lines or one.

Created an issue in PatternFly https://github.com/patternfly/patternfly-react/issues/9697 to track this issue down.

https://github.com/patternfly/patternfly-react/issues/9697#issuecomment-1766614774 This comment in the PF issue is very helpful, we could either create some custom toolbar styles by ourselves referring to the code sandbox demo in the comment or wait for the PF solution (If they adopt the feature request)

https://issues.redhat.com/browse/RHOAIENG-581

authors: DaoDaoNoCode adnankhan666 andrewballantyne kywalker-rh lucferbux rossbrigoli shalberd

repo: opendatahub-io/odh-dashboard


text:
Update archived models and archived versions components

Closes: RHOAIENG-11509
Description
This PR aims to update

Archived models to read-only by removing the toolbar actions, toolbar kebab and kebab menu from the table.

Archived model details page to read-only by removing all the actions.

Archived version details page to read-only by removing all the actions.

implementation for versions of archived model.

Empty state for version with archived model

How Has This Been Tested?

Archived models details page:

Archive a model
Go to the view archived models
see the archived model versions tab and details tab

Archived version details page:

Archive a version
Go to view archived version in model details
Go to the details page of the archived version

Version details page of archived model:

Archive a model with versions
Go to a version inside it

Test Impact
Added cypress test.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@ppadti there's one fix missing here - it is possible to see the wrong view for a model or version by going to the URL as if it was archived when it's not or vice versa. We should have the pages check whether the model and/or version is actually archived or not archived, and redirect to the correct place if you're at the wrong URL. This is a corner case, but it can come up if you have an old tab open on a model or version before its archived state changes or if you bookmark one.
Example:
If I have a model that is not archived and go its details page, that URL is: /modelRegistry/modelregistry-sample/registeredModels/13/versions and if I edit that to: /modelRegistry/modelregistry-sample/registeredModels/archive/13/versions I incorrectly see the archived view even though the model is not archived. The page should redirect me back to that first URL. If I then archive that model and navigate to it, then change its URL to remove /archive/, we get the opposite problem. Either way it should redirect me to the URL corresponding to the real model state.
The same is true of the archived / non-archived version pages. They should detect which page you should be on based on whether model or version are archived and redirect you there if you aren't already there.

ahh... Some how I missed this,  I thought only the archived models logic is wrong for versions under that. I will update this. Thanks!!


text:
Add intro to user impersonation

Issue #, if available: N/A
Description of changes: I found references to user impersonation in the default config file, but had to dive into the source code to figure out what request header to set to make it work.
Afterwards, I wrote up some a quick introduction that will hopefully save other people from having to go spelunking.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hey @turettn, this looks great. I have some minor style tweaks in mind, but I'm just going to merge and then make them. Thanks!

authors: aetter turettn

repo: opendistro/for-elasticsearch-docs


text:
Record linkage attack notebook

Similar in nature to the reconstruction and membership attack notebooks, it would be useful to have a notebook demonstrating a record linkage attack. As in, show how individuals in a dataset where "public identifiers" are stripped can still be reidentified via linking with other, public datasets.

current attack examples: https://docs.opendp.org/en/stable/examples/attacks/index.html?highlight=attacks

See where/if https://docs.opendp.org/en/stable/user/first-look-at-DP.html is in the new doc structure, and add it to the theory/attacks/ area.

authors: Shoeboxam mccalluc raprasad

repo: opendp/opendp


text:
consider finite types in sum

This resolves issues with finite data types specifically for the sum.
Adds relaxation term from #348 to the sum. See also #5.
Also adds a modular sum aggregator #458. A modular noise addition is subject to a later PR.

Here's a bird's eye view of the algorithms.
First off, we need a solution for overflow. Here are the options:

checked
monotonic saturating sum
ordered saturating sum
split saturating sum
modular sum (not implemented in this PR, integer only)

Just using an overflow solution is sufficient to make integer sums DP. The ideal approach is to attempt a checked sum, then attempt a monotonic saturating sum, and then fall back to one of the other sums.
Notice that only the ordered saturating sum requires an expensive shuffle or sort (can only sort if metric is unbounded).
In addition, for floats we need to compensate for float rounding. Here are the options for floats:

truncated sum, add relaxation term to the map, and use one of the overflow solutions
shuffle, split and rtz with saturation

The second solution is better asymptotically, but for 64 bit floats, it is only better when the datasets are unreasonably large. So I only implemented the first rounding solution, paired with the checked overflow solution and paired with the ordered saturating sum solution.
For floats I opted for the checked solution because the sum is unlikely to be rejected due to the high dynamic range of floats. Should the check fail, I also implemented the ordered sum, because there's a fair chance we'll need to shuffle anyways, when truncating. Truncation only requires ordering if it actually truncates.
Finally, the float relaxation term is dependent on the summation algorithm:

sequential with O(n^2) sensitivity increase
pairwise with O(log_2(n) * n) sensitivity increase

Pairwise is not quite as efficient as sequential. There's also utility in having a sequential map around, to calibrate sensitivity for systems that use sequential summation (like MySql, Sqlite).

Added tests for all functions and ffi functions, and reformatted all sum/ files.


text:
Cherry pick PR #166 to release/2.1 branch

This PR cherry-picks the following commit(s) to release/2.1:

54f66ee56f5c

bors merge

authors: niladrih sinhaashish

repo: openebs/mayastor-extensions


text:
fix(supportability/loki): add missing newlines

feat(supportability/loki): add loki dump subcommand

Signed-off-by: Tiago Castro <tiagolobocastro@gmail.com>

fix(supportability/loki): add missing newlines

It's not entirely clear how this was even working before, but this should work going forward.

Signed-off-by: Tiago Castro <tiagolobocastro@gmail.com>

bors merge


text:
Fix broken links and typos

What this PR does / why we need it:
Fixed some broken hyperlinks and typos in docs

Closing this PR as the DCO is missing. Feel free to raise another PR after fixing the issue.

authors: princerachit rkaranam

repo: openebs/openebs


text:
Add support for configurable filterconfigs.state.

This PR add support for configurable filterconfigs.state at helm chart.
Signed-off-by: fukuta-tatsuya-intec fukuta_tatsuya@intec.co.jp

Thank you @fukuta-tatsuya-intec for this contribution.
Would you also like to push these changes into the following charts:

https://github.com/helm/charts/tree/master/stable/openebs
https://github.com/openebs/charts/tree/master/stable/openebs

@kmova
Thank you for adopting the PR.
I would like this change to be reflected in those repositories as well.
Do I need to submit a new PR from me?

Yes @fukuta-tatsuya-intec - a new PR needs to be raised on those repositories.

authors: fukuta-tatsuya-intec kmova


text:
feat(helm chart): prepare helm chart for release 4.1.0

Changes

Updates the dependent charts with their respective latest releases.
Updates the relevant README

@w3aman Please add the monitoring in this PR.

You'd have to add monitoring here Instead of 'engines', you can put it under 'utils'? WDYT @tiagolobocastro @avishnu @dsharma-dc @w3aman @Abhinandan-Purkait @abhilashshetty04 ?

How about just monitoring?

@Abhinandan-Purkait please also incorporate my commit from https://github.com/openebs/openebs/pull/3747.

You'd have to add monitoring here Instead of 'engines', you can put it under 'utils'? WDYT @tiagolobocastro @avishnu @dsharma-dc @w3aman @Abhinandan-Purkait @abhilashshetty04 ?

How about just monitoring? In the values root.

Yep, works for me. monitoring.enabled.

@Abhinandan-Purkait please also incorporate my commit from #3747.

Done. Please check.

Please squash all of the commits, if possible.

Isn't squash and merge enough here?

authors: Abhinandan-Purkait niladrih


text:
Program Enrollment Page

As a user I want to see program details so that I can decide if I'm interested in the program.

This is a webview based on the discover screen. Would need more input from the design to see if we can convert in to a native page.

repo: openedx/openedx-app-ios


text:
User can view enrollment breakdown for course

User Story
As a course author, I want to view details of the enrollment breakdown for my course, so that I know who is in my course, what enrollment track they're on (when applicable), and I am able to compare the enrollment for this course against my other courses.
Acceptance Criteria
Scenario: A course author can view enrollment information for their course from the course dashboard
Given I am a course author with X permissions
When I navigate to the course dashboard
Then I can see the number of total learners enrolled in my course
And I can see the number and percent of total enrollees for current enrollees and enrollees per enrollment track (of enrollment tracks available on my instance)
And when an enrollment track is not available on my instance
Then I don't see it listed in the enrollment data for my course dashboard

I think this can be charted using the fact_enrollments dataset in Superset.

repo: openedx/openedx-aspects


text:
fix: only include aspects changes in translation workflow

Description
This PR updates the translation workflow to include only changes in Aspects and not in CI. See https://github.com/openedx/tutor-contrib-aspects/pull/450#discussion_r1350391139 for more information

Thanks for the pull request, @Ian2012!  Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

repo: openedx/tutor-contrib-aspects


text:
Add react router; stub settings page

This adds a basic implementation of react-router to allow for displaying different pages within the dashboard.
I've added a stub /settings route which you can visit by clicking the user icon in the top right; clicking on the 'OpenEEW Network Monitoring' link in the header will return to the dashboard.

Opps, looks like I missed commenting on this one -- I'm good to merge this, and it has some packages that I was going to add as part of getting the next version ready for development.

Looks good to me.

authors: gdpelican krook rdkelley

repo: openeew/openeew-dashboard


text:
Signed Integer Overflow in calculateNumTiles (73181093)

Hello OpenEXR team,
As part of our fuzzing efforts at Google, we have identified an issue affecting
OpenEXR (tested with revision * develop 165dceaeee86e0f8ce1ed1db3e3030c609a49f17).
To reproduce, we are attaching a Dockerfile which compiles the project with
LLVM, taking advantage of the sanitizers that it offers. More information about
how to use the attached Dockerfile can be found here:
https://docs.docker.com/engine/reference/builder/
TL;DR instructions:

mkdir project
cp Dockerfile.OpenEXR /path/to/project/Dockerfile
docker build --no-cache /path/to/project
docker run -it image_id_from_docker_build

From another terminal, outside the container:
docker cp /path/to/attached/reproducer running_container_hostname:/fuzzing/reproducer
(reference: https://docs.docker.com/engine/reference/commandline/cp/)
And, back inside the container:
/fuzzing/repro.sh /fuzzing/reproducer
Alternatively, and depending on the bug, you could use gcc, valgrind or other
instrumentation tools to aid in the investigation. The sanitizer errors that we
encountered were:
ImfTiledMisc.cpp:304:48: runtime error: signed integer overflow: 64 + 2147483628 cannot be represented in type 'int'
    #0 0x5cb7c1 in Imf_2_2::(anonymous namespace)::calculateNumTiles(int*, int, int, int, int, Imf_2_2::LevelRoundingMode) /fuzzing/openexr/OpenEXR/IlmImf/ImfTiledMisc.cpp:304:48
    #1 0x5caebb in Imf_2_2::precalculateTileInfo(Imf_2_2::TileDescription const&, int, int, int, int, int*&, int*&, int&, int&) /fuzzing/openexr/OpenEXR/IlmImf/ImfTiledMisc.cpp:324:5
    #2 0x66b134 in Imf_2_2::TiledInputFile::initialize() /fuzzing/openexr/OpenEXR/IlmImf/ImfTiledInputFile.cpp:934:5
    #3 0x66c387 in Imf_2_2::TiledInputFile::TiledInputFile(Imf_2_2::Header const&, Imf_2_2::IStream*, int, int) /fuzzing/openexr/OpenEXR/IlmImf/ImfTiledInputFile.cpp:837:5
    #4 0x630f64 in Imf_2_2::InputFile::initialize() /fuzzing/openexr/OpenEXR/IlmImf/ImfInputFile.cpp:545:32
    #5 0x62feb4 in Imf_2_2::InputFile::InputFile(char const*, int) /fuzzing/openexr/OpenEXR/IlmImf/ImfInputFile.cpp:382:13
    #6 0x55cb1a in Imf_2_2::RgbaInputFile::RgbaInputFile(char const*, int) /fuzzing/openexr/OpenEXR/IlmImf/ImfRgbaFile.cpp:1166:21
    #7 0x524984 in (anonymous namespace)::readSingle(char const*) /fuzzing/security-research-pocs/autofuzz/openexr_fuzz_scanlines.cc:20:14
    #8 0x524805 in LLVMFuzzerTestOneInput /fuzzing/security-research-pocs/autofuzz/openexr_fuzz_scanlines.cc:113:3
    #9 0x516d9c in fuzzer::Fuzzer::ExecuteCallback(unsigned char const*, unsigned long) (/fuzzing/openexr_fuzz_scanlines+0x516d9c)
    #10 0x51655e in fuzzer::Fuzzer::RunOne(unsigned char const*, unsigned long) (/fuzzing/openexr_fuzz_scanlines+0x51655e)
    #11 0x5103bd in fuzzer::RunOneTest(fuzzer::Fuzzer*, char const*) (/fuzzing/openexr_fuzz_scanlines+0x5103bd)
    #12 0x51188f in fuzzer::FuzzerDriver(int*, char***, int (*)(unsigned char const*, unsigned long)) (/fuzzing/openexr_fuzz_scanlines+0x51188f)
    #13 0x51026c in main (/fuzzing/openexr_fuzz_scanlines+0x51026c)
    #14 0x7f5f59cd62b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)
    #15 0x425979 in _start (/fuzzing/openexr_fuzz_scanlines+0x425979)

SUMMARY: AddressSanitizer: undefined-behavior ImfTiledMisc.cpp:304:48

We will gladly work with you so you can successfully confirm and reproduce this
issue. Do let us know if you have any feedback surrounding the documentation.
Once you have reproduced the issue, we'd appreciate to learn your expected
timeline for an update to be released. With any fix, please attribute the report
to "Google Autofuzz project".
We are also pleased to inform you that your project is eligible for inclusion to
the OSS-Fuzz project, which can provide additional continuous fuzzing, and
encourage you to investigate integration options.
Don't hesitate to let us know if you have any questions!
Google AutoFuzz Team
artifacts_73181093.zip

Additional stack trace that triggers the bug from a different code path:
ImfTiledMisc.cpp:304:48: runtime error: signed integer overflow: 1626642713 + 1073741824 cannot be represented in type 'int'
    #0 0x7f91d8ffa497 in Imf_2_2::(anonymous namespace)::calculateNumTiles(int*, int, int, int, int, Imf_2_2::LevelRoundingMode) /fuzzing/openexr/OpenEXR/IlmImf/ImfTiledMisc.cpp:304:48
    #1 0x7f91d8ffa61d in Imf_2_2::getTiledChunkOffsetTableSize(Imf_2_2::Header const&) /fuzzing/openexr/OpenEXR/IlmImf/ImfTiledMisc.cpp:355:5
    #2 0x7f91d9043c28 in Imf_2_2::MultiPartInputFile::Data::readChunkOffsetTables(bool) /fuzzing/openexr/OpenEXR/IlmImf/ImfMultiPartInputFile.cpp:738:36
    #3 0x7f91d9042a0f in Imf_2_2::MultiPartInputFile::initialize() /fuzzing/openexr/OpenEXR/IlmImf/ImfMultiPartInputFile.cpp:429:12
    #4 0x7f91d9041e0b in Imf_2_2::MultiPartInputFile::MultiPartInputFile(char const*, int, bool) /fuzzing/openexr/OpenEXR/IlmImf/ImfMultiPartInputFile.cpp:136:9
    #5 0x50f897 in printInfo(char const*) /fuzzing/openexr/OpenEXR/exrheader/main.cpp:290:24
    #6 0x5113f8 in main /fuzzing/openexr/OpenEXR/exrheader/main.cpp:562:13
    #7 0x7f91d79092b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)
    #8 0x424719 in _start (/fuzzing/openexr/OpenEXR/exrheader/.libs/exrheader+0x424719)

SUMMARY: AddressSanitizer: undefined-behavior ImfTiledMisc.cpp:304:48 in

Attached you will find another an additional sample as well as a patch that fixes the issue.
artifacts_73583817.zip

Attached the patch.
73583817.txt

Fixed by enforcing a limit to maximum number of tiles

authors: Google-Autofuzz peterhillman

repo: openexr/openexr


text:
README: break up WriteAuthorizationModelRequest into smaller understandable pieces

Currently, the WriteAuthorizationModelRequest object used for (write authorization model)[https://github.com/openfga/python-sdk/blob/main/README.md#write-authorization-model] is large and difficult to understand.  Ideally, we want to break it apart into smaller pieces
For example,
body = WriteAuthorizationModelRequest(
    schema_version = "1.1",
    type_definitions=[
        TypeDefinition(
            type="user",
        ),
        TypeDefinition(
            type="document",
            relations=dict(
                writer=Userset(
                    this=dict(),
                ),
                viewer=Userset(
                    union=Usersets(
                        child=[
                            Userset(this=dict()),
                            Userset(computed_userset=ObjectRelation(
                                object="",
                                relation="writer",
                            )),
                        ],
                    ),
                ),
            )
        ),
    ],
)

can be broken up as
user_type =  TypeDefinition(
    type="user",
)
document_type = TypeDefinition(
    type="document",
    relations=dict(
        writer=Userset(
            this=dict(),
        ),
        viewer=Userset(
            union=Usersets(
                child=[
                    Userset(this=dict()),
                    Userset(computed_userset=ObjectRelation(
                        object="",
                        relation="writer",
                     )),
                 ],
            )
        ),
    ),
)

body = WriteAuthorizationModelRequest(
    schema_version = "1.1",
    type_definitions=[
        user_type,
        document_type,
    ],
)

(and we break the document type relations into smaller pieces etc.)
The code in question is in https://github.com/openfga/python-sdk/blob/1f4c73467f13ed3bcd1989a6051695c4abac3c41/README.md?plain=1#L299
Ideally, we will also fix the SDK generator so that newly generated python SDKs will have the corresponding changes. This is tracked as https://github.com/openfga/sdk-generator/issues/126. However, if that is difficult to do, simply focusing on the Python SDK side https://github.com/openfga/python-sdk/blob/main/README.md will help us as well.

Please assign to me

Hi @rhamzeh 👋
I have a PR open in the SDK generator for this issue 😄
Please let me know if this is sufficient 👍
Cheers!

repo: openfga/python-sdk


text:
UTBFVRNS_CRT/BFVrns_FastBaseConvqToBskMontgomery sometimes seg faults

I have not been able to replicate the issue on my machine, but here is an example run where it seg faults: https://github.com/openfheorg/openfhe-development/runs/7136560551?check_suite_focus=true

The unit test is currently commented out. Needs to be turned on before testing.

a struct UnitTestCCParams was not initialized upon creation in BFVrns_FastBaseConvqToBskMontgomery() (UnitTestBFVrnsCRTOperations.cpp) as UnitTestCCParams was meant to be created using a list of values.
Added UnitTestCCParams' initialization

authors: dsuponitskiy sarojaduality yspolyakov

repo: openfheorg/openfhe-development


text:
Onboarding: misplaced overlay

Hi everyone,
The onboarding has a problem on small devices, including the iPod Touch.
All overlays are misplaced. What's the right way to fix it? Hide the overlays completely?

All overlays are misplaced

You mean only the "tooltip", right?
Not a big fan of that tooltip anyway: if you click on "Next" the first time it does not work as it closes the tooltip instead.
A dialog instead of the tooltip? That would solve the issue.

Yes, it's about the black tooltip

I guess we could position the tooltip's bottom to the page bottom minus half the bottom bar height: the tooltip would be visible, the rest of the screen would be visible, and the user would not be tempted to tap unsuccessfully on the "next" button.

De : Edouard Marquez @.>
Envoyé : mercredi 31 août 2022 14:41
À : openfoodfacts/smooth-app @.>
Cc : monsieurtanuki @.>; Comment @.>
Objet : Re: [openfoodfacts/smooth-app] Onboarding: misplaced overlay (Issue #2863)
Yes, it's about the black tooltip
—
Reply to this email directly, view it on GitHubhttps://github.com/openfoodfacts/smooth-app/issues/2863#issuecomment-1232885947, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACYKI3Z6VHVNWTRGZOXE4A3V35HF7ANCNFSM6AAAAAAQADMO3I.
You are receiving this because you commented.Message ID: @.***>

Actually the text doesn't match the UX: "you can tap on any item" - no, you can't, first you have to click anywhere in order to dismiss the tooltip, and then you can tap on any item.
But that's another story.

repo: openfoodfacts/smooth-app


text:
UI bug? SmoothProductImage with different sizes

In https://github.com/openfoodfacts/smooth-app/commit/8d6f045cc0d8a6c8b491af43dca6f438c33af863 (about "responsive design") some code changed: width and height were replaced with maxWidth and maxHeight in SmoothProductImage.
I don't know if it was the original goal, but now the products don't share the same dimensions.
Given that we have no control on the product pictures, it means we can have a very inconsistent display.
The previous display contained the picture into the same rectangle for all products, and that seemed more appropriate.
I'm not a big fan of the current display:

Ohh did not noticed that, I dont like it so either, but providing a fixed size will probably cause issues on different screen sizes. Ill try to fix it

I'm not talking about a fixed size. I'm talking about the same size for all displayed items.

Just width or also height

The human brain puts a premium on regularity.
We should

[ ]  take the whole width of the screen for the suggestions
[ ] create a grid (vertical line for all the text, which start at the same point) to create regularity
[ ] Center the images on the left column

it should be the same size (width and height) for all the cells containing the images ?

@teolemon/ @monsieurtanuki, this is my first attempt. I am not cinvinced by the blur effect behind the images but I cant think of a better alternative,

I like the fact that the suggestions take almost the full width. But why not take all the full width? Instead of having the suggestions hovering over the rest of the screen, they could just push the rest of the screen down.

For the record, what @M123-dev implemented is very similar to the behavior of google search (which can be considered as a reference, can't it?): not full screen width, and hovering over the rest of the screen.
I also assume that the width of the suggestions depends on (and cannot be greater than) the width of the text field.
The initial expressed requirement ("text field with automatic suggestions") is (and was already) fulfilled. It's currently improved, with the larger width. Obviously there's still some fine tuning needed about the ratio and the padding of the pictures, which was precisely the purpose of the current issue.
The topic of text field with suggestions is interesting enough to be worth several discussions and issues, about

the UI
the data sources (local database and/or API?)
the product fields to perform the search on (product name? in which language?)
the order of the results (newer products first?)
the possible action(s) when a suggested product is clicked (add to a list? alway go to the product?)

My personal preference: I'd prefer the pictures to be centered in a square instead of a rectangle. I would also remove the blur and the rounded corners. And add a bit of vertical spacing.

The UI can of course still be fine-tuned, what I wonder, @stephanegigandet , are the suggestions only for the search suggestions or also for the photos in the cards. There the images still have different heights.

I like the fact that the suggestions take almost the full width. But why not take all the full width? Instead of having the suggestions hovering over the rest of the screen, they could just push the rest of the screen down.

As @monsieurtanuki correctly assummed, the size of the suggestion popup depends on the size of the text field. To get the size to full width we would have to remove all padding and a small border does not hurt in my opinion.

the data sources (local database and/or API?)

Of course the API and before its response we can display local products. We could probably add lists pantrys to it, but the app is not that big as Microsoft Teams for example it is rather a nice to have feature, but it is not really needed.

the product fields to perform the search on (product name? in which language?)

Of course product name, other things rather not. For this there is the advanced search on the website. (We can add this later, but we don't really need it).

the order of the results (newer products first?)

What do you mean by "newer"?

the possible action(s) when a suggested product is clicked (add to a list? alway go to the product?)

For the main search bar just opening it, but for the add action block we can add it directly

I finetuned a bit in #311
I left the corners and the blur, the rounded corners because everything in the app has rounded corners, and without the blur the images would have different sizes again (when we dont want to crop them)

@M123-dev Looks good to me!
It fixes the issue (product image size and ratio) and answers an additional requirement by @teolemon about width.

I think we are ready to close this. The main problem of this issue the difference of the picture size in the searchbar and most the other mentioned things are now updated in #311

authors: M123-dev monsieurtanuki teolemon


text:
fix: #1698 - layout page analytics

What

The analytics screen still has a bad layout
The analytics screen has a different font
The analytics screen has an illustration that looks out of place, compared with the Knowledge panels the users has just seen before (filled, vs outline)

Screenshot

Fixes bug(s)

Fixes: #1698

Part of

https://github.com/openfoodfacts/smooth-app/issues/561

looks good to me @cli1005 👍

Codecov Report

Merging #1802 (131ad95) into develop (2ea0da3) will decrease coverage by 2.22%.
The diff coverage is 2.59%.

@@            Coverage Diff             @@
##           develop   #1802      +/-   ##
==========================================
- Coverage     8.86%   6.63%   -2.23%
==========================================
  Files          161     172      +11
  Lines         6623    7792    +1169
==========================================
- Hits           587     517      -70
- Misses        6036    7275    +1239

Impacted Files
Coverage Δ

...h_app/lib/cards/category_cards/abstract_cache.dart
0.00% <0.00%> (ø)

...p/lib/cards/category_cards/asset_cache_helper.dart
0.00% <0.00%> (ø)

...p/lib/cards/category_cards/raster_async_asset.dart
0.00% <0.00%> (ø)

...oth_app/lib/cards/category_cards/raster_cache.dart
0.00% <0.00%> (ø)

..._app/lib/cards/category_cards/svg_async_asset.dart
0.00% <0.00%> (ø)

...smooth_app/lib/cards/category_cards/svg_cache.dart
0.00% <0.00%> (ø)

...t_cards/knowledge_panels/knowledge_panel_card.dart
0.00% <0.00%> (ø)

...t_cards/knowledge_panels/knowledge_panel_page.dart
0.00% <0.00%> (ø)

...knowledge_panels/knowledge_panel_summary_card.dart
0.00% <ø> (ø)

...s/knowledge_panels/knowledge_panel_table_card.dart
0.00% <0.00%> (ø)

... and 105 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5c02c3...131ad95. Read the comment docs.

authors: cli1005 codecov-commenter teolemon


text:
feature/#218 - 3 possible colors for the app theme

Deleted file:

theme_manager.dart: refactored to UserPreferences, for consistency

Impacted files:

main.dart: refactored the theme provider that now uses UserPreferences and provides several colors
profile_page.dart: added the theme palette choice; refactored
smooth_listTile.dart: added flexibility with a possible title Widget instead of a Text
smooth_theme.dart: added the color to the theme; minor bug fix
theme_provider.dart: refactored using UserPreferences; added the color
user_preferences.dart: moved there the "darkness", and added the theme color

Thank you @M123-dev!


text:
feat: #2397 simplify dev mode access

What

simplify dev mode access without having to be logged in (Turn it into a toggle)
hide the toggle in the Setting contribute

Screenshot

https://user-images.githubusercontent.com/87010739/176649295-4cdb6c9b-64f9-4f84-bd9b-2dd92b0e4338.mp4
Fixes bug(s)

Fixes: #2397

Codecov Report

Merging #2437 (9458e6c) into develop (2ea0da3) will decrease coverage by 1.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2437      +/-   ##
==========================================
- Coverage     8.86%   7.55%   -1.32%
==========================================
  Files          161     203      +42
  Lines         6623    9747    +3124
==========================================
+ Hits           587     736     +149
- Misses        6036    9011    +2975

Impacted Files
Coverage Δ

...kages/smooth_app/lib/widgets/attribute_button.dart
0.00% <0.00%> (-92.00%)
:arrow_down:

...s/smooth_app/lib/data_models/user_preferences.dart
8.73% <0.00%> (-23.57%)
:arrow_down:

packages/smooth_app/lib/themes/smooth_theme.dart
60.00% <0.00%> (-22.98%)
:arrow_down:

...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart
15.29% <0.00%> (-18.92%)
:arrow_down:

...mooth_app/lib/data_models/product_preferences.dart
24.65% <0.00%> (-6.78%)
:arrow_down:

packages/smooth_app/lib/main.dart
14.40% <0.00%> (-3.49%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00% <0.00%> (-3.13%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/scan_example.dart
0.00% <0.00%> (-2.28%)
:arrow_down:

...p/lib/pages/onboarding/consent_analytics_page.dart
0.00% <0.00%> (-1.57%)
:arrow_down:

...smooth_app/lib/pages/product/new_product_page.dart
0.00% <0.00%> (-0.88%)
:arrow_down:

... and 203 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 04a37f6...9458e6c. Read the comment docs.

thanks you @cli1005 👍


text:
Update init file to not raise exception

I apologize if this is a duplicate of something you already have documented @j-wags. The main init file in openforcefield looks like it still raises an exception if OpenEye toolkits aren't installed. It also has a few commented out lines at the bottom that seem like they should be uncommented or removed.

Thanks for catching this! It's weird that our tests don't catch this. I'll take a look at this today, and schedule it for the upcoming milestone as I think this is critical.

Sorry to be late to this party. I just checked out @andrrizzi's PR #280 . I agree with him that this exception was getting caught by try/except logic before, but it did result in us double printing the "you didn't install the OpenEye Toolkit" message.
Looking into this also revealed a potential error with our build system (which is I think what @bannanc wanted to chat about tomorrow in Irvine).
Thanks for posting this issue. I didn't even realize that the top-level init had any logic in it before.
Agree that merging #280 should close this.

authors: andrrizzi bannanc j-wags

repo: openforcefield/openforcefield


text:
🐛(sandbox) add middleware to force SameSite=None on Django cookies

Purpose
Django < 3.1 does not allow to set the SameSite="None" attribute for the session and CSRF cookies. As a result, it causes issues with modern browsers that consider SameSite="Lax" by default.
Proposal

[x] Add a middleware to fix this before the public release of Django 3.1.
[x] Test it.

@sampaccoud  Thanks for your feedback, I fixed the typo and added a test.
If you want to review it, i'm waiting before merging this PR.

Issue #41 added :heavy_check_mark:

repo: openfun/ashley


text:
Fix average video views

Purpose
Average video views in the course videos overview dashboard are not properly calculated.
Proposal

[ ] fix aggregation query in the average video views panel from the course videos overview dashboard

During this work I also had to:

[x] upgrade grafana to 8.5.3
[x] fix interval aggregation parameter in ES8 queries

@quitterie-lcs ready for review and extension to other dashboards.

@jmaupetit looks good! I have added a fixup for more consistency in our requests, especially for when panels data are downloaded

authors: jmaupetit quitterie-lcs

repo: openfun/potsie


text:
🐛(api) fix sentry integration

Purpose
When the SENTRY_DSN setting is active, the LRS server cannot start as the sentry_sdk module is not imported.
Proposal

[x] import the sentry_sdk in the LRS server module

Could we remove the # noqa: F821 comment on line 12 of src/ralph/api/init.py?

Well spotted!

repo: openfun/ralph


text:
➕(layout) add 'dj-pagination' dependency and use it on blogposts and persons

Purpose
Previously page listings objects lacked of any pagination. This prooved to be a flaw with some lists which have largely too many objects. However since these pages are CMS pages, we don't have possibility to implement pagination in their views.
Proposal
This commit uses dj-pagination which provides template tags to make pagination in place directly in templates without manipulating anything from any view.
This adds a new dependency, a new template, a new setting, a new sass object source and involves blogpost and person list templates changes.

[ ] Reproduce search pagination CSS from ".paginate-course-search" into the new ".pagination" object which will be shared through every lists;
[ ] Update dj-pagination templates to fit to this CSS;
[ ] Update search components to use the ".pagination" object;

@sampaccoud ok so now, there is only a single '.pagination' CSS object shared through every pages and components with the same behavior and display.

@sampaccoud also note this add new translatable string for pagination (last page, currently reading page, etc..) to collect in PO files.

Also this might just be my opinion but I think this version of the pagination shows way too many pages by default

The one I did in react uses 1, n-2, n-1, n, n+1, n+2, last which seems more reasonable to me

@mbenadda i have added some settings to control the window and margin between showed pages and here is what i've ended to:

Nice job on the pagination size 👍

authors: mbenadda sveetch

repo: openfun/richie


text:
Decapitalize Pringle et al title

In your paper, you reference Pringle with the following entry
Pringle, C. M. (2001). HYDROLOGIC CONNECTIVITY AND THE MANAGEMENT OF BIOLOGICAL RESERVES: A GLOBAL PERSPECTIVE. Ecological Applications: A Publi- cation of the Ecological Society of America, 11(4), 981–998. https://doi.org/10.1890/ 1051-0761(2001)011%5B0981:HCATMO%5D2.0.CO;2

I guess it's nicer to remove the all-caps title and convert to regular capitalised text.

Linked to https://github.com/openjournals/joss-reviews/issues/7025

repo: opengeos/HyperCoast


text:
Running behind a reverse proxy

I am getting the Authentication Error, not a valid openid url when running index.php in examples/consumer. I have tried multiple openid providers and get the same error. When I test the code out on my local machine it runs fine. Is there anything I need to change in configuration to have it run properly on a machine that is behind a reverse proxy?

This repo is being archived. Closing issue.

repo: openid/php-openid


text:
Regenerate PDF of paper 118 - Multilocus sequence typing by blast from de novo assemblies against PubMLST

Hi,
I was wondering if it would be possible to regenerate the PDF of paper 00118 (Multilocus sequence typing by blast from de novo assemblies against PubMLST) as I had a typo in a surname of one of the authors I referenced?
The paper:
http://joss.theoj.org/papers/0b801d23613c9b626c2b6028f8c14056
https://www.theoj.org/joss-papers/joss.00118/10.21105.joss.00118.pdf
The typo fix to paper.bib:
https://github.com/sanger-pathogens/mlst_check/commit/2f8457e7c84f585b8c8a835978f9ae461bab113e
And the paper source itself:
https://github.com/sanger-pathogens/mlst_check/blob/master/paper.md
https://github.com/sanger-pathogens/mlst_check/blob/master/paper.bib
Thanks a million, its a brilliant journal and I really appreciate all the hard work thats gone into it.
Andrew

OK great. That should be updated on the site in the next few hours.

authors: andrewjpage arfon

repo: openjournals/joss-papers