Cannot query with updatedAt as range key

Context: I wish to query a global secondary index using updatedAt as the range key.
Example:
import dynamoose from 'dynamoose';

// dynamoose config here

const Schema = dynamoose.Schema;

// Create the model
const Record = dynamoose.model('Record', new Schema({
  recordID: {
    type: String,
    hashKey: true
  },
  tableID: {
    type: String,
    required: true,
    index: {
      global: true,
      rangeKey: 'updatedAt'
    }
  }
}, {
  timestamps: true
}));

// Add some data to the table (in seperate requests so that the updated time is different)
new Record({
  recordID: 'ca4cba8c-d7e7-4845-b554-cb8ee6859642',
  tableID: '78d650c5-9b7f-404e-9873-03068f06b51d'
}).save();

new Record({
  recordID: '534d980b-3cf5-455d-9a79-b31e68db4bca',
  tableID: '78d650c5-9b7f-404e-9873-03068f06b51d'
}).save();

new Record({
  recordID: 'c00a9bfb-3450-4356-b94e-94c02887e649',
  tableID: '78d650c5-9b7f-404e-9873-03068f06b51d'
}).save();

new Record({
  recordID: '1a2415c6-d3cc-412d-a945-bb1ef398c7f7',
  tableID: '78d650c5-9b7f-404e-9873-03068f06b51d'
}).save();

new Record({
  recordID: 'd2d1512f-627e-4199-9eb2-2ad52ebe07fd',
  tableID: '78d650c5-9b7f-404e-9873-03068f06b51d'
}).save();

// Query the GSI
const startTime =<PHONE_NUMBER>544; // Pick a time somewhere in the middle to see if it works.

Record.query('tableID').eq('78d650c5-9b7f-404e-9873-03068f06b51d')
  .where('updatedAt').ge(startTime)
  .exec((err, res) => {
    console.log(res);
  });

In this example ge and gt return no results, while lt and le just return all of the records.
There is no issue doing this with createdAt (to test, replace updatedAt with createdAt in the schema and query) or any other range keys, just with updatedAt.

So I did some digging and think I've found the source of the problem. In Schema.js line 59, updatedAt is defined with a setter:
obj[updatedAt] = { type: Date, default: Date.now, set: function() { return Date.now(); } };

This setter overwrites any user input and returns the current date.
When the query is executes, the toDynamo function in Attribute.js is called, which subsequently calls the setter on all attributes, if they exist. This changes the query values for updatedAt to the current time.
This can be fixed by changing the setter to only return the current time if there is no value passed in:
function(val) {
  if (val === undefined) {
    return Date.now();
  } else {
    return val;
  }
}

This fixes the problem for updatedAt in particular, but I'm not sure if there are other considerations that need to made for querying rangeKeys that have setters in general

Uh, after some further testing that change doesn't seem to work. I almost feel like the setter shouldn't be called during a query, but there are probably cases where you would want it called. Seems like an edge case for updatedAt. Anyone have any thoughts for how to approach this?

Looks like this issue is basically the same as #382.


Add YOLOv5x coco pretrained checkpoint

Is it possible to add a YOLOv5x coco pretrained checkpoint?

Hi,
Currently, we are working on adding YOLO-X architecture into the repo (pre-trained checkpoints and recipes)
As a part of this effort we are training YOLO-X X as well

Im closing this issue down as we removed YoloV5 models from our repository.
As discussed above, YoloX L model, pretrained weights and training recipe are now available.

Im closing this issue down as we removed YoloV5 models from our repository.
As discussed above, YoloX L model, pretrained weights and training recipe are now available.


shadowing outer local variable

When I try to load hjson I get the following warning:

/var/lib/gems/2.3.0/gems/hjson-0.1.0/lib/hjson/ast/nodes/string.rb:48: warning: shadowing outer local variable - uffff

Here's the entire ruby script:

#!/usr/bin/ruby -w
require 'hjson'

Yes, my script turns on warnings. Still, it seems like a warning worth heeding.
Here are some specs:
Operating system	Ubuntu Linux 16.04.1
Kernel and CPU	Linux 4.4.0-78-generic on x86_64
ruby 2.3.1p112 (2016-04-26) [x86_64-linux-gnu]
Please let me know if there is other information that would help.

thanks, should be fixed.


added callouts

Signed-off-by: Debopriya Bhattacharjee<EMAIL_ADDRESS>Description
Added callouts to various resources in place of the list present previously.
This PR fixes #524
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

A small nitpick: is it possible have all of the cards of equal height (size)?

@leecalcote  They seem to have the same height for me, is it different for you?

safari doing it's thing again.


Flashing lights toggle + old code updated with new asset system

cool.

uhh I think the lose text isn't very good. I think it was removed for a reason lol. Maybe take that out.

uhh I think the lose text isn't very good. I think it was removed for a reason lol. Maybe take that out.

it isnt used in the actual gameover, i just updated the old gameover state code incase someone wants to reuse it

oh mk


Playground: option to view/copy tsconfig.json

it would be cool if you could export the config set here to a tsconfig.json format

And what about importing one as well?


docs(README): Fix links

Hey !
Thanks for working on this. I spotted a few typos in the links on the readme, here is a fix !

Thanks, they were indeed out of date with the new folder name.


close time validation

(from https://www.metaculus.com/question-writing/)
The stated requirement for the close time to be at least 1 hour before resolution time seems to not be enforced. I am able to create and approve a question with identical closing date and resolving date:

In fact, I'm able to create and approve a question where the closing time is after the resolution time.

I am able to create and approve a question with identical closing date and resolving date:

That's fine, we should updated the FAQ.

I'm able to create and approve a question where the closing time is after the resolution time.

That's not fine.


Two too huge images

Where to find the issue
https://www.coronawarn.app/en/
Describe the issue
Two of the images used on the website aren't optimized regarding their (file) size.
Suggested change
The images used on the website should get optimized.

Maintainers, your feedback is requested :) @SebastianWolf-SAP @tkowark @jakobmoellersap


Add icons when saving web app onto home screen

index.html

Update index.html to show favicon
Update index.html to show boba icon when bookmarking on iOS
Update index.html to show boba icon when saving to iOS home screen

webmanifest

Add webmanifest
Add boba icon to appear on android home screen when save to home screen

Nice icon


Sankey: names of nodes with zero demand should be hidden

Minor issue: it would be nice when the nodes with zero demand would be hidden in the sankey

Just out of curiosity; how much time would it cost to fix this @antw ? We were thinking about giving more attention to the sankey diagrams; then it would be nice to have this fixed 🙂

Just out of curiosity; how much time would it cost to fix this @antw ?

The list of nodes are drawn when the chart is first rendered. It's easy enough to hide those which have zero demand, but when the scenario is changes so that they have zero demand... they stay hidden. I can fix that, but then there are other oddities such as links being drawn with the wrong height:

I'd need to investigate further to come up with an accurate estimate.

Some Sankey-inspiration 😉


Can not running postinstall

When I follow install traffic OPs
[root@localhost ~]# /opt/traffic_ops/install/bin/postinstall
postinstall requires the postgres user to have superuser access to postgresql. Aborting.
but when i bypass check
root@localhost ~]# su - postgres psql -w -c 'show is_superuser'
su: invalid option -- 'w'
Try 'su --help' for more information.
and i test
[postgres@localhost ~]$  psql -w -c 'show is_superuser'
psql: error: fe_sendauth: no password supplied
What problem
OS Rocky Linux

You are using su improperly, consult man su.
Your PostgreSQL setup does not allow for the postgres user to authenticate without a password, consult the PostgreSQL documentation.
For troubleshooting help, please join our Slack channel(s) on the ASF Slack workspace instead of opening issues.

This command inculde postintall script
make sure installed with proper permissions
umask 022
if [[ ! $(su - postgres psql -w -c 'show is_superuser' </dev/null 2>/dev/null) =~ 'on' ]]; then
echo >&2 "postinstall requires the postgres user to have superuser access to postgresql. Aborting."
exit 1
fi
and document ats for traffic ops have command :
su - postgres -c '/usr/pgsql-13/bin/initdb -A md5 -W' #-W forces the user to provide a superuser (postgres) password
This conflict command and script included /opt/traffic_ops/install/bin/postinstall

For troubleshooting help, please join our Slack channel(s) on the ASF Slack workspace.


Unhandled interrupt 6 on cpu 0! (IDFGH-7642)

Environment

Development Kit:   none
Module or chip used: ESP32-WROOM-32D
IDF version (run git describe --tags to find it):  v4.3.2
Build System:  idf.py
Compiler version (run xtensa-esp32-elf-gcc --version to find it):  xtensa-esp32-elf-gcc.exe (crosstool-NG esp-2020r3) 8.4.0
Operating System:  Windows
Using an IDE?: Yes, visual studio code with espressif extension
Power Supply:  external 3.3V

Problem Description
The ESP32 is used in a custom module that collect data from other devices connected to it through rs485 and push the data to Azure using mqtts over a wifi connection, it can also be interacted using bluetooth (as it is a ble peripheral using nimble).
The issue appears when the esp32 is running after some time (days) and it reconnects, for any reason, to the cloud, then when it tries to reconnect using the DPS service, Azure occasionally returns a message like:

{"errorCode":404203,"trackingId":"7180aa22-6cd0-4879-9b10-942143e51461","message":"Operation not found.","timestampUtc":"2022-06-18T13:51:58.7132797Z"}

Whatever that this means. Once Azure starts to return this message the ESP32 is unable to connect successfully to the cloud (since it needs the hub destination). I already opened a ticket to Microsoft about that and I am waiting their answer as the only way to solve this issue is rebooting the ESP32.
But the real issue comes when after X retries of reconnection attempts (from one to thousands) it always ends with this error, that freezes the system:

Unhandled interrupt 6 on cpu 0!

As far I can guess, for some reason freertos stopped to handling its tick interruption (interrupt 6), which freeze all tasks and cause that the watchdog triggers.
I mention the Azure issue as is the only way to reproduce the issue.
Expected Behavior
The ESP32 should keep trying to reconnect to Azure indefinitely without stop handling interrupt 6.
Actual Behavior
After the connection to the cloud fails and ESP32 tries to reconnect it, it eventually stops handling interrupt 6
Steps to reproduce

Retry connections to Azure until it start to fail
Wait until the interrupt 6 start to be unhandled

Code to reproduce this issue
I can not show the chunk of code, as it is large and confidential. On other hand I did not find it useful to solve this issue. Tasks does not block the cpus as they have more than 10ms of task delay on each iteration.
Debug Logs
This data is from a system working fine a couple of days:

mem
Free mem:     54024
Minimum free: 37664
Heap summary for capabilities 0x00001000:
At 0x3ffb2730 len 15448 free 92 allocated 13732 min_free 92
largest_free_block 0 alloc_blocks 22 free_blocks 0 total_blocks 22
At 0x3ffb6388 len 7288 free 188 allocated 5476 min_free 172
largest_free_block 0 alloc_blocks 46 free_blocks 0 total_blocks 46
At 0x3ffb9a20 len 16648 free 192 allocated 14832 min_free 192
largest_free_block 0 alloc_blocks 47 free_blocks 0 total_blocks 47
At 0x3ffd5958 len 42664 free 284 allocated 40756 min_free 284
largest_free_block 0 alloc_blocks 70 free_blocks 0 total_blocks 70
At 0x3ffe0440 len 15072 free 228 allocated 13220 min_free 224
largest_free_block 0 alloc_blocks 56 free_blocks 0 total_blocks 56
At 0x3ffe4350 len 113840 free 53040 allocated 59176 min_free 36700
largest_free_block 38912 alloc_blocks 208 free_blocks 7 total_blocks 215
Totals:
free 54024 allocated 147192 min_free 37664 largest_free_block 38912

cpustats

Task          Run time (ms)    Use Time (%)

Console        	76296		<1
IDLE<PHONE_NUMBER>		70
IDLE<PHONE_NUMBER>		206
C_Core<PHONE_NUMBER>		127
BLE_Peripheral 	257925092		14
mqtt_task      	196712958		10
Wifi_manager   	45905259		2
Led task       	39689349		2
tiT            	170561898		9
main           	88300239		4
C_Manager<PHONE_NUMBER>		91
sys_evt        	3813		<1
ipc1           	61467		<1
Tmr Svc        	13631		<1
btController   	283811946		15
wifi<PHONE_NUMBER>		104
esp_timer      	332860200		18
ipc0           	27		<1
ble            	5669		<1

tasks

Task           State  Priority  Stack  Num CPU

Console           X	2	1932	12	-1
IDLE           	    R	0	240	          7	  1
IDLE           	    R	0	252	          6	  0
C_Manager      B	9	2136	17	-1
BLE_Peripheral B	6	2564	16	-1
tiT            	    B	18	1572	19	-1
Led task           B	0	268	        13	-1
main           	    B	1	600	          5	 0
Wifi_manager  B	5	616	        22	-1
mqtt_task        B	5	688	        25	-1
C_Core            B	10	1636	18	-1
ipc1           	   B	9	996	          2	 1
Tmr Svc           B	24	1588	  8	 0
ipc0           	   B	24	1076	  1	 0
btController    B	23	2080	14	 1
wifi           	   B	23	4744	21	 0
esp_timer        S	22	592	          3	 0
ble            	   B	21	1168	15	 1
sys_evt        	   B	20	720	        20   	 0

This is the log of the issue:

E (172816594) CORE: Disconnected from wifi
I (172816765) CORE: Connecting wifi...
I (172817504) esp_netif_handlers: sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (172817517) CORE: Connected wifi
I (172837531) CORE: Connecting to cloud...
W (172837534) AZURE: DPS initialized: Connecting to mqtts://global.azure-devices-provisioning.net/0ne000FE197/registrations/AC67B2FCDD1A/register?api-version=2019-03-31:8883 as 0ne000FE197/registrations/AC67B2FCDD1A/api-version=2019-03-31, wait until receive a destination...
W (172839792) AZURE: Sent request to dps {"registrationId":"AC67B2FCDD1A"}
W (172839889) AZURE: Received from Azure 151:
{"errorCode":404203,"trackingId":"cd3846b9-e171-42c7-8137-c714c7d7d3d5","message":"Operation not found.","timestampUtc":"2022-06-19T11:19:31.8640956Z"}
E (172840131) CORE: Failed cloud connection
I (172861530) CORE: Connecting to cloud...
W (172861532) AZURE: DPS initialized: Connecting to mqtts://global.azure-devices-provisioning.net/0ne000FE197/registrations/AC67B2FCDD1A/register?api-version=2019-03-31:8883 as 0ne000FE197/registrations/AC67B2FCDD1A/api-version=2019-03-31, wait until receive a destination...
W (172863862) AZURE: Sent request to dps {"registrationId":"AC67B2FCDD1A"}
W (172865882) AZURE: Received from Azure 151:
{"errorCode":404203,"trackingId":"b76d47be-ac75-497c-9ce6-79396aa5b7db","message":"Operation not found.","timestampUtc":"2022-06-19T11:19:57.8649053Z"}
E (172866124) CORE: Failed cloud connection
.
.
.
Unhandled interrupt 6 on cpu 0!
.
.
.
Unhandled interrupt 6 on cpu 0!
E (207015044) task_wdt: Task watchdog got triggered. The following tasks did not reset the watchdog in time:
E (207015044) task_wdt:  - IDLE (CPU 0)
E (207015044) task_wdt: Tasks currently running:
E (207015044) task_wdt: CPU 0: IDLE
E (207015044) task_wdt: CPU 1: IDLE
E (207015044) task_wdt: Aborting.
abort() was called at PC 0x4014052c on core 0
Backtrace:0x400818c3:0x3ffbea60 0x400933bd:0x3ffbea80 0x40099e92:0x3ffbeaa0 0x4014052c:0x3ffbeb10 0x40083739:0x3ffbeb30 0x4019c2e7:0x3ffbc110 0x400da102:0x3ffbc130 0x40094bad:0x3ffbc150 0x40096ddd:0x3ffbc170
0x400818c3: panic_abort at C:/Proyectos/Firmware/esp-idf/components/esp_system/panic.c:393
0x400933bd: esp_system_abort at C:/Proyectos/Firmware/esp-idf/components/esp_system/system_api.c:112
0x40099e92: abort at C:/Proyectos/Firmware/esp-idf/components/newlib/abort.c:46
0x4014052c: task_wdt_isr at C:/Proyectos/Firmware/esp-idf/components/esp_common/src/task_wdt.c:192
0x40083739: _xt_lowint1 at C:/Proyectos/Firmware/esp-idf/components/freertos/port/xtensa/xtensa_vectors.S:1105
0x4019c2e7: get_mask at C:/Proyectos/Firmware/esp-idf/components/efuse/src/esp_efuse_utility.c:261
0x400da102: esp_register_freertos_idle_hook_for_cpu at C:/Proyectos/Firmware/esp-idf/components/esp_common/src/freertos_hooks.c:71
0x40094bad: prvIdleTask at C:/Proyectos/Firmware/esp-idf/components/freertos/tasks.c:3846 (discriminator 1)
0x40096ddd: vPortTaskWrapper at C:/Proyectos/Firmware/esp-idf/components/freertos/port/xtensa/port.c:168

It seems that the interrupt handler pointer got overwritten (which is the reason why the interrupt isn't handled), this can be either a stack corruption or some other kind of an out-of-bounds write.
Just an idea, is it possible that the task handling the Azure connection overflows the stack and overwrites some of the static data? Perhaps you can try to introduce this or a similar error artificially (e.g. prohibiting the connection from Azure side) and then check that your reconnection process doesn't result in a stack overflow.
You can also try enabling CONFIG_FREERTOS_WATCHPOINT_END_OF_STACK=y option to detect stack overflows.

I set the CONFIG_FREERTOS_WATCHPOINT_END_OF_STACK=y, now I have to wait until crash.

Perhaps you can try to introduce this or a similar error artificially (e.g. prohibiting the connection from Azure side) and then check that your reconnection process doesn't result in a stack overflow.
I already tried a lot of approximations trying to reproduce the issue at will, but no one seems to accelerate the process.

Also, following your suggestion that maybe can be a pointer out of control, I increased the stack use, declaring and using a dummy array in the first function of the task related with the issue. The point behind that is to see if the overwrite stop happening over the interrupt handler 6.

I increased the stack use, declaring and using a dummy array in the first function of the task related with the issue.

I'm not sure this change will have any effect, as this array will be located near the top of the stack (i.e. higher absolute value of memory address) while stack overflow happens near the bottom of the stack (i.e. lower absolute address).

@AngelVersa Could you reproduce the problem again? Are there any updates?

The problem did not appear again. But probably is because now I reboot the system once every day.

@AngelVersa We will close this issue then for the time being. If you have any new information, please let us know and feel free to re-open this issue.


when ever i run script/bootstrap i receive this

when ever i run script/bootstrap or npm install i receive this.
npm ERR! git clone git@github.com:github:maddox/harmonyhubjs-client Cloning into bare repository '/root/.npm/_git-remotes/git-github-com-github-maddox-harmonyhubjs-client-def8ff3d'...
npm ERR! git clone git@github.com:github:maddox/harmonyhubjs-client Permission denied (publickey).
npm ERR! git clone git@github.com:github:maddox/harmonyhubjs-client fatal: Could not read from remote repository.
npm ERR! git clone git@github.com:github:maddox/harmonyhubjs-client
npm ERR! git clone git@github.com:github:maddox/harmonyhubjs-client Please make sure you have the correct access rights
npm ERR! git clone git@github.com:github:maddox/harmonyhubjs-client and the repository exists.
npm ERR! notarget No compatible version found: harmonyhubjs-client@'github:maddox/harmonyhubjs-client#for-harmony-api'
npm ERR! notarget Valid install targets:
npm ERR! notarget ["1.1.2","1.1.3","1.1.4","1.1.5","1.1.6","1.1.7","1.1.8","1.1.9","1.1.10"]
npm ERR! notarget
npm ERR! notarget This is most likely not a problem with npm itself.
npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.

npm ERR! System Linux 4.9.80-v7+
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! cwd /home/pi/Downloads/Harmony/harmony-api
npm ERR! node -v v4.8.2
npm ERR! npm -v 1.4.21
npm ERR! code ETARGET
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/pi/Downloads/Harmony/harmony-api/npm-debug.log
npm ERR! not ok code 0

same error here, any suggestions?

What is this path?

/root/.npm/_git-remotes/git-github-com-github-maddox-harmonyhubjs-client-def8ff3d

What environment are you all running this on? This is super weird.

Oh i suppose it's doing a git clone when npm installing directly from github. It's using SSH to clone down the package. It's a public repo, so I'm not sure why it's complaining.

If you set up git on your device with github support (add your ssh key to your github account) it should work.
If you ssh into your device with ssh -A host, it will forward and use the key on your local device and probably work.
I should probably somehow ensure this uses https and not ssh for the package
I'm pretty sure what ever changes I had in my fork, were merged into the original package, I should just move back to it.

I just opened #94.
Could someone switch to this branch: back-to-regular-harmonyhubjs-client and try script/bootstrap
Try to rm -Rf node_modules first.

That branch did the trick for me!
I'm getting a few warnings but it works again!
Thanks! Very happy with it!
The warnings I'm getting:
pi@raspberrypi:~/harmony-api $ script/bootstrap
npm WARN engine<EMAIL_ADDRESS>wanted: {"npm":">=3.0.0"} (current: {"node":"4.8.2","npm":"1.4.21"})
npm WARN deprecated<EMAIL_ADDRESS>use @xmpp/xml package https://www.npmjs.com/package/@xmpp/xml
npm WARN deprecated<EMAIL_ADDRESS>use @xmpp/jid package https://www.npmjs.com/package/@xmpp/jid
<EMAIL_ADDRESS>node_modules/parameterize

Great! I’ll merge the pr. Those other messages are benign.


Update self-publish Tutorial

Acceptance

[x] all steps in our self-publish tutorial are up to date and easy to understand to a first time user

Tasks

[x] review our self-publish tutorial and make necessary fixes while following along the steps in it 2023/01/30 See commit

@PhilippeduPreez i'm moving this out of the sprint if that is ok - is there any reason to think it is out of date?

@rufuspollock I think mostly just replacing flowershow commands with flowershow@latest If omitting @latest causes problems sometimes

FIXED, see commit here


Add maximum allowed monitor number.

Description
Adds the maximum allowed number of monitors to the existing Monitors doc.
Issues Resolved
Closes #1798
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@natebower This is ready for review. Thanks!


Minecraft exitcode-1073740791 Crash

Describe the bug
I have recently been having Minecraft crashing when I have been performing certain actions. It was crashing when I would use the bed. When I would reload the world after the crash, I would try again, and it would crash again. I fixed this issue after uninstalling java and reinstalling it. I have recently been having an issue with breeding cows 5-10 seconds after breeding them and getting the achievement for it, it crashes. It repeats every time I do it. The file I attached is the most recent crash log from the cows.
OS & version [e.g. Windows 10, 1822]:
Windows 10 , 64 bit build 19041
MSFT Build of OpenJDK Version:
OpenJDK Runtime Environment Microsoft-32931 (17.0.3+7) (build 17.0.3+7-LTS)
To Reproduce:
Steps to reproduce the behavior:

Feed to cows' wheat until they bread and create a calf
The achievement is produced and for breeding animals for the first time
5-10 second later Minecraft crashes
producing exitcode-1073740791

hs_err_pid16696.log

Ok thank you


xMySqlProvision - changed members in mySql 2.0

In the updated 2.0 version of xMySql, 'ServiceName' is no longer needed and there are a couple new required members that need to be added in addition. (SingleNodeEndToEndWordPress.ps1, line 109)

PR #10 submitted to resolve this issue.


False positive in Rails/FindEach with Capybara

Rails/FindEach picks up on Capybara node traversal as though it were ActiveRecord statements and suggests find_each as a nonexisting alternative to find.
Details below, but I'm not sure what I'd recommend as a fix - perhaps
Expected behavior
Rails/FindEach should only detect ActiveRecord specific violations.
Actual behavior
Rails/FindEach suggests violation that would break Capybara code.
Steps to reproduce the problem
Any Capybara code using all followed by each, eg
page.all('.row').each do |row|
  do_stuff(row)
end

RuboCop version
➜  rubocop -V
0.49.1 (using Parser <IP_ADDRESS>, running on ruby 2.3.3 x86_64-linux)

I'm not sure what I'd recommend as a fix

Me either. How can Rubocop differentiate ActiveRecord from Capybara? It's just doing static analysis.
Maybe exclude your test / spec for Rails/FindEach?

Excluding spec/features/**/* from inspection seems like the right call. It should possibly even be excluded by default.
Rails/FindEach:
  Exclude:
    - "spec/features/**/*"

@drenmi that would seem to be a sensible approach, I'd advocate for that to be a default as well. If there's any way to also highlight the Cop's limitations in the documentation, that would also be great for people who end up using Capybara elsewhere in their projects, and as such need to add additional exclude paths or go without the cop entirely.

I'm closing this ticket due to no recent activity. Feel free to re-open it if you ever come back to it.

@bbatsov Fair enough. The mitigation of excluding specs from Rails/FindEach works well enough for us, I would still wonder if that should be a default. That said I can't attest to Rubocop's current behaviour here since we have just gone with our existing config since reporting this.


Specify what "Preview" status mean for AKS version

For my current cluster 1.14.0 I see 2 available updates, but both marked as "preview".
What does it mean for AKS version to be in a "preview" status? (I didn't find an answer here and here  https://docs.microsoft.com/en-us/azure/aks/support-policies)
Do Microsoft provide support for such "preview" versions or it's my(in general - customer's) own risk?
Do Microsoft provide guarantees that cluster upgraded to "preview" version, in a future can be upgraded to the same number but "current" (stable) version? (without delete and re-creating)
Will I need to upgrade cluster from 1.14.3 (preview) version to 1.14.3 (current) or Microsoft will change the status of a release from preview to current without a requirement to upgrade version?

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

ID: ec484163-5d73-af3a-3d7b-9e0d421d2824
Version Independent ID: 42da7aed-a33d-e331-fb7f-4fb4b40e5813
Content: Supported Kubernetes versions in Azure Kubernetes Service
Content Source: articles/aks/supported-kubernetes-versions.md
Service: container-service
GitHub Login: @sauryadas
Microsoft Alias: saudas

@SychevIgor Thanks for your questions.

What does it mean for AKS version to be in a "preview" status?

Preview versions doesnt come under general support policy and should not be used for production.
Below note is from the documentation for the preview features
AKS preview features are self-service, opt-in. They are provided to gather feedback and bugs from our community. In preview, these features aren't meant for production use. Features in public preview fall under 'best effort' support. Assistance from the AKS technical support teams is available during business hours Pacific timezone (PST) only. For additional information, please see the following support articles:

Do Microsoft provide support for such "preview" versions or it's my(in general - customer's) own risk?

Above note answered this question also. Support is in PST only and its not recommended for production use.

Do Microsoft provide guarantees that cluster upgraded to "preview" version, in a future can be upgraded to the same number but "current" (stable) version? (without delete and re-creating)

Since preview versions have limited support and its meant for collecting feedback and finding bugs, Its not recommended to upgrade to a preview version. In your case the version 1.14.0 is not supported list either. I am sure when you have created the cluster that version was in preview.  AKS supports 2 latest patch releases for a minor release.  Thats why 1.14.0 is removed from supported version when 1.14.3 is added. you can list the supported version by using this command az aks get-versions --location eastus --output table.
For your case, you can wait till the version 1.14.1 is moved out of the preview version if your env is  production.  Else you can create a new user and migrate the workloads.

Will I need to upgrade cluster from 1.14.3 (preview) version to 1.14.3 (current) or Microsoft will change the status of a release from preview to current without a requirement to upgrade version?

Microsoft wont automatically upgrade the cluster. But it will send notifications to you when a new version is released or you existing version is old.
All the above points are discussed in this document.
please let me know if you have more questions

@jakaruna-MSFT awesome answer. Shortly, for me was not clear that AKS version falls to "AKS preview features". I didn't thought that version can be classified as feature.
After first answer - everything was already clear


how can set password for my users

hello, thanks for supporting and thanks for your good script. i'm beginner in ubuntu. i install your script, but script don't ask password for my client. how can set password and how can set expire date for users for example expire after 1 month and how can use tcp? thanks again.

Hi @mahmoodhojabri - Please look through the closed issues for answers to your questions before you post. This is now the fourth time passwords have been requested, they wont be added because they don't fit the intended use of this project.

Passwords were been addressed in #36, #40, #44, #97 I doubt this is any different.
Currently the script can revoke certificates manually, but if you're interested in automating check out  these docs

@mahmoodhojabri looking through the public fork of this project I found this fork which has expiring password certificates. I've not tested, but it lists the features you're looking for. Best of luck.

Thanks @stefancrain.
To use TCP, just change proto udp to proto tcp for both the client and server config files (or just edit the script before installing).


Restore tweens function fix

_restorePlayingTweens was throwing an error because this._savedTweens was sometimes undefined (I'm not sure what caused that, but this check should have been done either way!)
Also small things:

added gulp-babel to dependencies
fixed test line length error in spec/tweener.coffee

Hi Chen!
Thanks a lot, I will review it shortly.
// @chenasraf

I've found exactly the case when this._savedTweens is undefined - when you open a page with animation in background tab, and then go to that tab. So pageVisibility changes and there is no _savedTweens yet. Can be solved also by declaring _savedTweens as empty array in _vars method.

I opened a new PR at #85, from dev to dev - apologize for the multiple PRs, but hopefully I'm still useful :)

I'm closing this in favor of the new PR.


fix: filename ast-grep

Sorry, overlooked the missing hyphen, until I noticed it not working here on my end. By having the filename without, adding ast-grep as formatter does not work, even though you'd expect it to be due to the it being spelled that way since it's the official spelling.

LGTM


Consider renaming max_outcomes (and min_outcomes)

outcomes vs. truncate
In favor of outcomes:

Intuitive in that one could imagine that the dice in the pool each have a max_outcome.
Easy to remember.

In favor of truncate:

More specific.
Less likely to be confused with the singular max_outcome.

The other word

max: The original.
top: Short, but I'd rather not introduce another synonym for "max".
right: I don't like this as another synonym for max, and it's already used for median and ppf with a different meaning. The one upside is that it seems to be the most common word used with truncation.
hi: "High" is already used to refer to dice rather than outcomes.
upper:
above:

Went with truncate_max.


Docs.MS Release Updates for azure-core-serializer-json-jackson

Update docs metadata and targeting for release of azure-core-serializer-json-jackson

Docs Build status updates of commit 42eebc3:
:clock10: Full build: the average full build time is 97 min(s) 54 sec(s), this is based on the last 825 build(s) for this repository.

Docs Build status updates of commit 42eebc3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

legacy/docs-ref-autogen/overview/EventHubs/Client.yml
:warning:Warning
View (azure-java-legacy)
Details

docs-ref-services/core-serializer-json-jackson-readme.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

package.json
:white_check_mark:Succeeded

legacy/docs-ref-autogen/overview/EventHubs/Client.yml

[Warning-DuplicateUids] Uid(azure.java.sdk.landingpage.services.eventhub.Client) has already been defined in docs-ref-services/messaging-eventhubs-readme.md.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Refactoring: replace snprintf and sscanf with sto* and to_string() where practical

Will be cleaner and more concise since it can be used directly in m_transport->SendCommand() without needing multiple lines of code.
This is not intended to be a ban on the C-style functions, only a cleanup where cleaner code results.

Maybe we should wait for C++20 which introduces std::format for printf-like string formatting in C++?


add dm_conversation_id to TDMEventV2Field

https://developer.x.com/en/docs/x-api/direct-messages/lookup/api-reference/get-dm_events

Thank you! Actually the rest of the fields seem wrong, maybe you could try to fix that too?

Okay!

Hello,
I believed you added "texts" to BaseDMEventV2, but I don't think "ParticipantsJoin" events have text
What I meant is that the other TDMEventV2Field accepted strings seemed wrong to me, maybe I'm wrong

Ye

Hello,
I believed you added "texts" to BaseDMEventV2, but I don't think "ParticipantsJoin" events have text
What I meant is that the other TDMEventV2Field accepted strings seemed wrong to me, maybe I'm wrong

My bad. I fixed it

thank you! (id, name, event type are missing too, I'll add them myself)

Published in 1.18.2


fix async chain in registering routes

Short description and why it's useful

Should fix a problem with urlDispatcher in some edge cases during SSR load.
@SebSzcze please confirm that it helps.
After confirmation i'll back-port it also to 1.10
Which environment this relates to
Check your case. In case of any doubts please read about Release Cycle

[x] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[ ] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change

@patzick I couldn't reproduce the error, so I think this solves the problem


Cookie path should be set explicitly (was: Sessions not working over 302 redirect)

Hacking on something at work; uses Github OAuth, and it stopped working. As far as I can tell, it's because session cookies don't work reliably over a 302 redirect (some SameSite shenanigans).
So: look carefully at how sessions are done over redirects. This probably means that #13 needs to be WONTFIXED, and that #6 needs looking at -- if you're doing it and then redirecting, stuff might break.

It's like this:

I had some older code that was doing set-cookie from /auth/github, which Firefox remembered as path=/auth/github
Because of the better match, it would send that cookie first, and we'd successfully update that session.
Back on the home page, it would send the original cookie (implicitly path=/), which didn't match the session.

"But Roger", you say, "you've been restarting the app frequently, surely you'd create a new session, to avoid session fixation attacks with the permissive mode? See https://cheatsheetseries.owasp.org/cheatsheets/Session_Management_Cheat_Sheet.html#session-id-generation-and-verification-permissive-and-strict-session-management if you don't know what I'm referring to."
"You're correct", I say, "I am doing that, but each time we issue a new session, it implicitly gets reissued with the broken path, and we see the same behaviour again".
tl;dr: the browser defaults the cookie path to the current location; set path=/ if you want it to work for the entire site.

Further information:

You can't see this in the server; the client sends both session cookies to /auth/github, but it doesn't include the path, so you can't easily see which is which. It sends them ordered by best-match (i.e. longest path prefix match).
You have to look in the browser; in Firefox, this is in dev tools / storage / cookies. That shows the path, and makes it more obvious.

I don't know whether this is Firefox-specific behaviour.


HIVE-27365: Fix test acid_bloom_filter_orc_file_dump

What changes were proposed in this pull request?
Sort directories the in PostExecOrcFileDump
Why are the changes needed?
Tests dumps orc files of a table may get flaky due to order of directories contain the orc files is not deterministic.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
mvn test -Dtest.output.overwrite -Dtest=TestMiniLlapLocalCliDriver -Dqfile=acid_bloom_filter_orc_file_dump.q,orc_file_dump.q -pl itests/qtest -Pitests

@aturoczy
Could you please point which cast do you refer to?
Creating a new array list instance was necessary to copy the original list. Otherwise the sort would have an affect on the instance stored in FetchWork.partDir which I try to avoid.

@zabetak
Yes, PostExecOrcFileDump is used only in tests.
The files/subdirectories are also sorted in the In the printFileStatus method:
https://github.com/apache/hive/blob/4f1dba31b25fb2c3b071492ff06b54f76a55c99a/ql/src/java/org/apache/hadoop/hive/ql/hooks/PostExecOrcFileDump.java#L104


:memo: Complete relational modeling docs

Covering the remaining relational specific configuration APIs.
Also splitting some relational docs that were inline in general configuration into the new relational sub-section

:shipit:


Lock file can be non-exhaustive

Currently no CI exists to ensure that the lock file (requirements.txt) exhaustively pins all project dependencies. As a result of this, deployment checks are non-deterministic

@garryod I've seen your PR on this but it fails because we don't normally commit a requirements.txt file.
The checks are deterministic with respect to the container that is released along side the build. For a container centric approach do you think this is adequate?

Hi @gilesknap, after giving it some further consideration, the method used should suffice for the vast majority of cases and requires significantly less developer interaction, it seems like a good trade-off to me


Enable Clippy and fix badges

I've noticed that the badges in the readme still point to those of the kms (license, build).
Also, I think it would be a good idea to enable clippy in circle-ci. The kms repos has an example on howto do so:
https://github.com/tendermint/kms/blob/3b3aabf8eab64959dad29ee2d5b747dc5fe813ac/.circleci/config.yml#L16-L20

On that note I'd suggest to enable pedantic clippy checks as it helps esp. in with overflows and precision loss for numbers. For someone interested in hardmode:
#![deny(missing_docs)]
#![deny(warnings)]
#![forbid(
    clippy::all,
    clippy::cargo,
    clippy::nursery,
    clippy::pedantic,
    clippy::restriction
)]


Update for Vapor 3.0

Vapor 3 has a completely new model for serializable content and request routing, along with changes in Swift 4 that require some significant changes/updates to this code in order to make it work.

Hey @paulyhedral
You are right, there is certainly some amount of work we have to put into this to update it for Vapor 3. But unfortunately I don't have enough time right now to do this. I'm accepting pull requests though...


[Currency Bot] Bumped patch/minor dependencies

Elasticsearch v8.14.0 introduced @elastic/transport v8.6.0, which now requires a minimum Node.js version of 18.
In Elasticsearch version 8.13.1, @elastic/transport was pinned to ~8.4.1 to ensure compatibility for users of the Elasticsearch 8.13 client, avoiding the need to update to Node.js v18+.
Additionally, the subdependency undici was updated to v6 in @elastic/transport, dropping support for Node.js v16 and below. This update has impacted test failures, highlighting the need for further attention.
For more details, refer to the release notes.

Elasticsearch v8.14.0 introduced @elastic/transport v8.6.0, which now requires a minimum Node.js version of 18.

Elastic bumped straight from 8.5.0 to 8.6.0, but the breaking change was introduced in 8.5.1.
Further refs:

https://github.com/elastic/elastic-transport-js/issues/90


Some enhancements (scrambler/descrambler, mpe)

Hello. I have some questions for enhancements.

I would like to ask, it there could be scrambler/descrambler with XOR algorithm. I mean... I would like to make some stream, specify PID for audio/video and encrypt that PIDs with static 184 bytes long XOR key and on another device decrypt it with the same key. When Adaptation Field is present, leave all 184 bytes unencrypted. I know that XOR can be easily hacked, but it's experimental (if it's not hard to add this command).

Can MPE plugin has output for filtered packets throught IP? For now, we can filter MPE packets and save it only to the file with -o argument. It would be awesome, if that output can be pushed to IP output. I mean, something like -O ip (address), but when I do that in this case, it will be streaming all MPE packets without any filtering. And last question...

--min-udp-size and --max-udp-size works perfectly, but I discovered, that sometimes they are sending some garbage text informations in that range.  After long analysing, they are not sending packets bigger than 1468 bytes. If there can be some argument for dropping packets with specific size, it would be awesome, too!! :) For this example something like this: --drop-udp-sizes 12,102,212,336,465,466,467,477,478.
Probably, they are using variable sizes for MPEG-TS packets, but fixed sizes for text messages.
Thank you so so much for your powerful TSDuck !!! You're incredible @legard !!!
dukey914
PS: When will be released new binaries?

Hi,

Your request for an XOR plugin is very specific and experimental for you only. Too specific for a general case. Moreover, and you mentioned it, using XOR for encryption is a joke. I mean literally a joke. In the crypto community, "use an XOR" is the usual joke to say "this is crap". I cannot spend time developing code for one person only if it does not benefit to the community or increase the value of the project. But, you could develop it yourself, it is quite easy.

Did you try -P mpe --udp-forward --redirect address:port?

The trailing text should be an hexa + ascii dump of the packet. If you think it is incorrect, please post an example stream to reproduce the problem. Only the MPE PID, only a few packets would be sufficient.

New binaries for version 3.15 will be published when the new tsswitch utility is sufficiently tested by users, as discussed in issue #73. If you need the latest development, you can build it yourself, see the instructions here.

OK, I understand. Thanks for reply anyway! :) I will try -P mpe --udp-forward --redirect address:port.

Looks like -P mpe --udp-forward --redirect address:port works, but (I didn't mentioned that.. sorry) I have to use --skip 20 argument for droping MPE header, which works only for -o output.
[80 21 FA C5 45 DC 0C F4 28 28 6C 00 01 03 40 40 00 00 00 00] 47 40 00 39 A6 00 FF FF FF FF FF FF FF...

...so if I'm not lost, mpe successfully decapsulates UDP from MPE and that header 80 21 ... 03 40 40 00 00 00 00 has nothing in common with MPE - It's UDP header? And... Exists some way how to drop it (live)?

Look at the MPE spec in chapter 7 of the DVB data broadcast spec.

TS packets of the MPE PID contain DSM-CC sections
DSM-CC sections contain a network datagram, including (in the case of UDP datagram):
-- MAC address info (part of the Ethernet frame header)
-- IP header
-- UDP header
-- UDP payload

You should make a step backward, ie. stop thinking in terms of bytes but in terms of features. With --udp-forward, the mpe plugin forwards the UDP datagrams on the local network using the original destination IP address (usually a multicast one) from the IP header in the MPE section. Thus, the broadcast network acts as a VPN tunnel for IP multicast. With --redirect, it also modifies the destination IP address and UDP port, in case the original multicast address is no longer suitable on the local network or to transform multicast in unicast or vice-versa.
In all cases, the net result of the UDP forwarding is that you receive the original UDP messages when you listen to the UDP port on the local network.

OK, Thanks for explanation @lelegard !!
For now, I'm using this way:
-P mpe --pid 2009 -d "<IP_ADDRESS>:6000" --min-udp-size 480 --max-udp-size 1468 --skip 20 -o "tmp01.bin" and using ffmpeg just after I run this command for live streaming tmp01.bin throught udp (or hls), but I have to restart tsp after some hours, because tmp01.bin should mess my HDD.
I just looked for some way, how to not mess my HDD (without restarting tsp). How to stream that -o output without saving it to HDD. Maybe it's not possible. I understand. :)

How to stream that -o output without saving it to HDD.

That is exactly what I was trying to explain. Simply replace -o with --udp-forward.

--skip 20 do the trick, but it works only with -o. When I do it with --udp-forward, looks like streaming works perfectly, but VLC (or another player) doesn't play it correctly, because they're broadcasting it with that header, which --skip skips, so the picture is clear (only 188 bytes long MPEG-TS packets), but when I do it with --udp-forward (even with --skip 20), --udp-forward will not skip that extra bytes, so it will not stream clear 188 bytes packets.
See example:

-o do the trick, but --udp-forward not.

Again, you should think in terms of formats instead of bytes. What is the formal format of the UDP datagrams from your MPE stream? TS encapsulation inside UDP can be raw (no header), RTP, etc. So, having a header inside the UDP payload before the first TS packet is not unusual.

OK. I'm sorry, if I was saying some nonsense. I really enjoy your app, because it's really powerful and can do a lot of things. Looks like TSDuck do everything correctly, but they're just broadcasting it a little bit modified. Later I will post some short TS record of one PID with MPE. Thanks for your efforts. I hope it wasn't waste of time with me.
See you for now @lelegard !!

Your request for an XOR plugin is very specific and experimental for you only. Too specific for a general case. Moreover, and you mentioned it, using XOR for encryption is a joke. I mean literally a joke. In the crypto community, "use an XOR" is the usual joke to say "this is crap". I cannot spend time developing code for one person only if it does not benefit to the community or increase the value of the project. But, you could develop it yourself, it is quite easy.

Sorry to bring this old issue up again but I'd just like to point out that XOR is actually used, it seems that Compel and Rosscrypt CA systems use XOR so implementing it in TSDuck would actually allow hackers to gain access to those few channels using this CA system so it's better for TSDuck not to implement this to stay out of trouble.

OK. Yes, I noticed that info about Rosscrypt and Compel, too. Now I
understand. Thanks for reply anyway.
See you.
PS: I'm still using MPE plugin. :)
ne 9. 12. 2018 v 9:55 odesílatel Jan Reznicek<EMAIL_ADDRESS>napsal:

Your request for an XOR plugin is very specific and experimental for
you only. Too specific for a general case. Moreover, and you mentioned it,
using XOR for encryption is a joke. I mean literally a joke. In the crypto
community, "use an XOR" is the usual joke to say "this is crap". I cannot
spend time developing code for one person only if it does not benefit to
the community or increase the value of the project. But, you could develop
it yourself, it is quite easy.

Sorry to bring this old issue up again but I'd just like to point out that
XOR is actually used, it seems that Compel and Rosscrypt CA systems use XOR
so implementing it in TSDuck would actually allow hackers to gain access to
those few channels using this CA system so it's better for TSDuck not to
implement this to stay out of trouble.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tsduck/tsduck/issues/137#issuecomment-445521064, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ap5DYuMZuYkhR9ohDbLuI_hI7hLiMp8Pks5u3NAUgaJpZM4Xes90
.

it seems that Compel and Rosscrypt CA systems use XOR

I do not know these CA systems but I seriously doubt about XOR being used as content or key "encryption". This would be less than weak or vulnerable. I would simply call it "over stupid". They would be well-known in the crypto-community, precisely because of using XOR as pseudo-encryption.
These two CA systems are not well referenced on the Web.

Compel: a PDF document from 2004 mentions "RSA Data Security has audited COMPEL CA.". Audited XOR ? No way...
Rosscrypt: the Wikipedia page on CA systems mentions that Roscrypt (with one 's') has been introduced in 2006. A Web archive page from a Russian federal institute mentions "Modern encryption methods, openness of the system for appropriate Russian specialists". This, in itself, does not mean anything valuable. But Russia is probably in the top 5 countries for cryptography. So, using XOR is not credible at all.

Now, "encrypting" a 184-byte message with a 184-byte key looks like stream cipher or OTP. But, in that case, the security is in the generation of this key stream. And being a "key stream", not a single reusable key, it changes every message.

I don't know if the system really is Compel, it's just what Lyngsat says for those 2 horse betting channels that use XOR as encryption with the XOR key broadcast in clear in the video PID. There is no CAT, no ECM and no EMM, so who knows, might be something else.


Please Check to see if the "pytest tests" pass successfully - I think the "USD_PER_TPU" variable may need importing

Please Check to see if the "pytest tests" pass successfully - I think the "USD_PER_TPU" variable may need importing

============================= test session starts ==============================
platform darwin -- Python 3.8.16, pytest-7.4.3, pluggy-1.3.0
cachedir: .tox/py3/.pytest_cache
rootdir: /Users/reibs/Projects/tokencost
collected 79 items

tests/test_costs.py .................................................... [ 65%]
...........................                                              [100%]

============================== 79 passed in 0.98s ==============================
py3: commands[1]> coverage report -m
Name                     Stmts   Miss  Cover   Missing
------------------------------------------------------
tokencost/__init__.py        2      0   100%
tokencost/constants.py       3      0   100%
tokencost/costs.py          55      0   100%
------------------------------------------------------
TOTAL                       60      0   100%
py3: OK ✔ in 11.11 seconds

I'm showing full test coverage with tax. Can you show which tests are failing?

tokencosts_test_results.txt

Can you verify you're on the latest version of the main branch? Something's up with your unit tests. For example:
__________________ test_count_string_tokens[gpt-3.5-turbo-4] __________________

model = 'gpt-3.5-turbo', expected_output = 4

    @pytest.mark.parametrize(
        "model,expected_output",
        [
            ("gpt-3.5-turbo", 4),
            ("gpt-3.5-turbo-0301", 4),
            ("gpt-3.5-turbo-0613", 4),
            ("gpt-3.5-turbo-16k", 4),
            ("gpt-3.5-turbo-16k-0613", 4),
            ("gpt-3.5-turbo-1106", 4),
            ("gpt-3.5-turbo-instruct", 4),
            ("gpt-4-0314", 4),
            ("gpt-4", 4),
            ("gpt-4-32k", 4),
            ("gpt-4-32k-0314", 4),
            ("gpt-4-0613", 4),
            ("gpt-4-1106-preview", 4),
            ("gpt-4-vision-preview", 4),
            ("text-embedding-ada-002", 4),
        ],
    )
    def test_count_string_tokens(model, expected_output):
        """Test that the string tokens are counted correctly."""

        # 4 tokens
>       assert count_string_tokens(STRING, model=model) == expected_output
E       AssertionError: assert 2 == 4
E        +  where 2 = count_string_tokens('Hello world', model='gpt-3.5-turbo')

There are no tests for the string "Hello world". The string shared across test cases is "Hello, world!" (notice the punctuation marks). Did you modify the tests?

You are absolutely correct! As part of my troubleshooting for issue#14 I had modified the 'tests_cost.py' file.
I changed the string back to "Hello, world!" and all tests complete successfully:
I feel that I may have been more of a hinderance than a help 😔
============================================================================================== test session starts ==============================================================================================
platform win32 -- Python 3.11.3, pytest-7.4.3, pluggy-1.3.0
rootdir: C:\Users\xxx\OneDrive\Projects\Projects\chatgpt\tokencost-main\tokencost-main
plugins: anyio-3.7.1
collected 79 items

tests\test_costs.py ...............................................................................                                                                                [100%]

============================================================================================== 79 passed in 0.60s ===============================================================================================

You are absolutely correct! As part of my troubleshooting for issue#14 I had modified the 'tests_cost.py' file.
I changed the string back to "Hello, world!" and all tests complete successfully:
I feel that I may have been more of a hinderance than a help 😔
============================================================================================== test session starts ==============================================================================================
platform win32 -- Python 3.11.3, pytest-7.4.3, pluggy-1.3.0
rootdir: C:\Users\xxx\OneDrive\Projects\Projects\chatgpt\tokencost-main\tokencost-main
plugins: anyio-3.7.1
collected 79 items

tests\test_costs.py ...............................................................................                                                                                [100%]

============================================================================================== 79 passed in 0.60s ===============================================================================================

No worries! You pointed out some issues with the readme, arguably the most important part of the library :) I appreciate the help


updating iter_content docstring to match functionality

Sorry to be bombarding you guys with PRs. This one is pretty minor but explains what I would say was non-obvious behaviour of the function before PR #3368 and PR #3365.

So this isn't quite right. In particular, iter_content with stream=True and a chunk encoded response will iterate in chunk size. So it would be better for this to say that it iterates in chunks of undefined size.

Looks like I failed to set stream=True in my test case. I was misinterpreting the call to r.content as the stream read. Thanks @Lukasa, string updated.

Cool, I'm happy with this! Thanks @nateprewitt! :sparkles: :cake: :sparkles:


Repeating sentence

The following sentence is repeated at the end of the first paragraph then at the beginning of the second:
TCA is a method of calculating the cost between the main formula item for a batch order and the co-products that are defined for the formula.

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

ID: ac539690-eba4-bb20-caaa-60d11bb9e0ed
Version Independent ID: 40d3c424-03f1-db9c-05f3-2367999ade4e
Content: Total cost allocation method - Supply Chain Management | Dynamics 365
Content Source: articles/supply-chain/cost-management/methodology-total-cost-allocation.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @AndersGirke
Microsoft Alias: mguada

@Kelkane, thanks for making us aware of this!
@kamaybac, please update the content.

@sericks007 , I fixed this in the private repo and merged it to master already, so I'm closing it here.


Crate 0.1 SMBus functionality not exposed

Just creating a note as I discovered this while adding some additional example code.  A new version should be released soon that fixes this.

Thanks!  I hadn't come across it yet, if you'd like in a couple of days I should have another example if you'd like to link it or add it or whoever can just follow this comment to my repository when it's up!
I learned a lot from your code, thanks!

I'm also in the process of adding some additional examples to the library to continue to see where the pain points are, starting with the MPL115A2 pressure/tempeature sensor.  Based on this work, I may end up adding an abstraction on top of registers (maybe a macro)  that should allow users to use a higher level abstraction when supporting devices.  The low-level interface will always also be available as devices are all over the map as far as what is required to talk to them.

Fixed in master, so closing.  I am in the process of changing around quite a few things that should help the lib to support things beyond linux (e.g. microcontrollers, unit tests mocks).  Eventually, I would like to add support for zinc.  This may involve splitting the library into separate crates with the core being no_std.


Usage of custom icon slot causes unexpected event bubbling

vue-datepicker version:  4.2.1
Describe the bug
Whenever I try to use the "calendar-icon" or "clock-icon" slot to overwrite the icons, something in the component seems to break silently (or at least behaves weirdly). I'm opening the date picker inside another dialog which has a directive, that is listening for an outside click (events: click or mousedown) to close it. I'm opening the date picker with the teleport prop and everything works fine:
<vue-date-picker v-model="modelValueProxy" :flow="['month', 'day', 'time']" :teleport="true" />

Now I use one of the mentioned icon-slots (e.g. like this):
<vue-date-picker v-model="modelValueProxy" :flow="['month', 'day', 'time']" :teleport="true">
    <template #calendar-icon>
      <v-icon class="ml-2" icon="mdi-calendar-blank" />
    </template>
  </vue-date-picker>

The picker-dialog opens (with the month-picker, as it should), but immediatly closes after you select (click) on any month.
I can see that the clickoutside of my outter dialog triggers (it doesnt when the slot is not set).
I've attached  a screenshot of the event-timelines (with and without the slot), which might describe my problem better.

(Left with the default icon / Right with the "calendar-icon" slot)

It is known that when using teleport  with a custom click outside directive doesn't play well. What you can do is to try converting an icon into a functional component like here and provide the component instead of an icon or if you have the event available, try to ignore the one coming from the icon element.

I might be able to fix this


feat: add drawer component using react-modal

Is the 7xl wide drawer off?

We don't have a 7xl token, was a typo. Fixed


Is it true that 'Dispatch' method name is not allowed when defining a IClientProxy?

Hi,
I have a Hub<IServerToClient> where IServerToClient:
    public interface IServerToClient
    {
    Task Dispatch<T>(T message) where T : IMessage;
    Task Dispatch(string message);
    }

If I try to call this Dispatch method anywhere in the hub, eg this.Clients.All.Dispatch("message"), it does not get fired on clients side (my client is C# client). It works when I rename the method to DispatchX for example.
So, is it true that Dispatch method name is not allowed when defining a IClientProxy?  Is this some kind of keyword?
Well, I realized that S2C_Dispatch or Send behaves the same way.. hopefully I find one available synonym that I could use for this method name..
Is there a list of names to avoid?

multiple handlers on same method name

Yeah, SignalR doesn't support overloading because it doesn't know what kind of types were used by the caller (since we support .NET, JavaScript and other clients), so when you have two handlers for the same method name, they should both be called.

no handler could be invoked either.

This is surprising to me. If you call .On twice with the same name on the client it should register two handlers for the same method. Would you be able to provide an example project that reproduces the issue you're seeing? I want to make sure the behavior is clear. If you can provide an example project and describe what you'd expect to happen compared to what actually happens that would help identify if there's a problem in documentation or a problem in the code :)

I realized I had this behavior due to a (probably) invalid handler (note the IMessage, instead of Message):
    _hubConnection.On<IMessage>("Dispatch", (msg) =>
    {
    Console.WriteLine($"Received Message: msg.Description}");
    });
    _hubConnection.On<MyMessage>("Dispatch", (msg) =>
    {
    Console.WriteLine($"Received MyMessage: msg.Description}");
    });

Btw, do you have some samples of how to use such inheritance on Messages with SignalR

Well, I find it very strange that after changing IMessage to Message in the first handler... the second handler is not getting called for some reason.
To have it clear, MyMessage derives from Message

Btw, do you have some samples of how to use such inheritance on Messages with SignalR

I wouldn't recommend using SignalR directly to handle the inheritance. Since we serialize the JSON and back, the only way to accurately represent the specific type of the instance is to encode it in the JSON, which a) gets ugly fast and b) opens a security risk if not done extremely carefully.
Instead, I would suggest that you consider using an enum or some other value to indicate the kind of the message. If you need to send different payloads for different message types, I'd suggest defining a different method on the client for each.

Well, I find it very strange that after changing IMessage to Message in the first handler... the second handler is not getting called for some reason.

That is odd and shouldn't be the case. Would you be able to put an example project up on GitHub? That will make it really easy for us to identify if there's an issue there.

@anurse
Ok, I will attach my test client code here.. the server is just echoing back the original message
Program.zip
I just noticed that I get different behavior based on the order of the handlers. Is that possible?
So, when I have On<Message>, On<MyMessage> order .. only one of them gets called.. but when I have On<MyMessage>, On<Message> order, then both get called

Found the reason you are getting inconsistent behavior.
https://github.com/aspnet/SignalR/blob/e41764c161574983c49362cfb29ce23eb5941b74/src/Microsoft.AspNetCore.SignalR.Client.Core/HubConnection.cs#L1241
I'm not sure we'll make any changes here, we'll discuss it

Yeah, On is not designed to be used with handlers that take different types. Attaching multiple handlers to a method is an extremely uncommon scenario. In general, SignalR does not support overloading (having multiple methods with the same name but different parameter types).
As @BrennanConroy noticed, this behavior is consistent with our design, so I'm going to re-close this issue. Happy to continue to help you find the right pattern for your application though, so feel free to keep commenting!

Good day Sir!
Im having a trouble implementing the SignalR on my Ionic 3 Application
the behavior is something like this

I Implemented it on my Web app , and its working fine and good

here is the issue

I try implementing the SignalR on my Ionic 3 Project , and same way with the WEB, later on when the

.on(instanceName, data => {   // some codes here });

inside the functions, it triggers 2x or 3x on my client part

is there any way i can see the error?

How are you invoking the method on the client? Are you using Clients.Group or Clients.Groups?


Editor crashes when saving scene with tool script

On Godot_v3.1.1-stable_x11.64 on Arch Linux, the editor crashes when saving a scene with the following script:
tool
extends Spatial

export(NodePath) var path

# Called when the node enters the scene tree for the first time.
func _ready():
	path = get_node(path)

The editor will crash upon saving as long as path is unset.
Taking out the line with tool will prevent the editor from crashing while saving. Strangely, adding tool back in to the script and then saving will not crash the editor, but next time the project is loaded the editor will crash upon saving again.
I do not know if this script crashes the game upon running because the editor will crash when saving whenever I try to run the project.
The workaround for this is to not do the above action in a script. Simply doing var node = get_node(path) instead of path = get_node(path) will prevent the crash from occurring.
Steps for reproducing:

Create a new project
Make a scene with a Spatial node (I have yet to try other nodes) that has the above script
Close the editor and re-open it
Save the project

This crash, because there is infinite recurrence inside Godot

Still valid in 94e5f66


[Feature] Bold for keyword/func_name etc?

Would be great to add bold for keyword etc.

Already supported 🙂
Check :help highlight-args for all options. These options are also supported with other styles like commentStyle, functionStyle, variableStyle.
Refrence README.md#configuration for default value.
require("github-theme").setup({
  keywordStyle = "bold",
})


No way to change tags on an issue

It appears that there is no way to alter the tags on am issue ifbthe issue does not already have tags associated with it?

I take that back,  it's easier to alter tags if some already exist..  But the it's also a way to add new visa the edit menu option.
Feel free to close this,  though I still think it's not immediately obvious


Fix: MyEventType has an additional curly brace at the end

Remove the additional curly brace at the end in MyEventType.

@ggailey777
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Need help to get out of the below error.

Hi All, while i am executing 228-semantic segmentation of arieal images using U_net i encountered the below error. Please help me in resolving this.
WARNING:matplotlib.image:Clipping input data to the valid range for imshow with RGB data ([0..1] for floats or [0..255] for integers).
IndexError                                Traceback (most recent call last)
 in
7 plt.imshow(np.reshape(image_dataset[image_number], (patch_size, patch_size, 3)))
8 plt.subplot(122)
----> 9 plt.imshow(np.reshape(mask_dataset[image_number], (patch_size, patch_size, 3)))
10 plt.show()
IndexError: list index out of range

can you help me with specified version or requirement.txt


Potential improvement of encoding/decoding times

Thank you for this interesting project!
As mentioned already in the readme, both the encoding and decoding process can be a bit slow. While this might not necessarily be that relevant for the encoding side, decoding speed of files can be problematic for longer files. On my system, encoding speed is around 0.25x  while decoding is at 0.5x. The culprit is the MDCT conversion that is calculated quite a lot.
One simple approach I did use to improve this is by simply precalculating the MDCT over all possible coefficients when calling the encode/decode function. This needs to be done for short and long window sizes separately, with each lookup table having the size of (winsize * winsize / 2) floats.
At the expense of some temporary memory allocation, this yielded quite some improvements in my tests.
Using a 28 second test file, without the lookup table I had the following times:

encoding: 122s
decoding: 51s

With the lookup table:

encoding: 45s
decoding: 2s (!)

Encoded and decoded files are of course identical between the two variants, just the processing is around 3 times fast for encoding and around 25 times faster for decoding.
Not sure if this is of interest to any of you, but I thought I would just mention it here in case anyone is playing around with the library like I did. :)

Hi, thanks for the interest in the project, glad you had fun with it!
Bear in mind that the intended use case for the codec (particularly the decoder) is decoding short one-shot samples for 64k intros.
That said, even for tooling (or for just playing around with other use cases like you seem to have done) a faster MDCT would be desirable. I think a satisfying solution to this is to somehow decouple the MDCT impl from the rest of the code, assuming that can be done with zero (and I mean zero) additional bytes in the decoder when used in intros (the decoder is already a couple hundred bytes larger than I had hoped!).
I recall @going-digital doing some work to that effect, on this branch (which he mentioned to me in a twitter dm but I unfortunately forgot to reply until now, sorry about that!), which uses the classic resonator approach instead of precalcing a whole table, which is nice (that said, I haven't really looked in detail admittedly as my time is fairly limited for these kinds of projects).
Anyways, for something like this to be merged I'd need to see that it doesn't increase code size and isn't too complicated. I like @going-digital's approach I think but haven't done the due-dilligence, so there's an opportunity there perhaps :)

Thanks for your response. Just to clarify, I never wanted this test of mine to be an actual replacement for the intended use case of short samples in 64k demos - I know that there every byte matters ;)
I am currently just experimenting with taking this codec as a base/idea for my own projects in a different scenario where actual ecoding and decoding times matter, so the naive pre-calcs actually worked for me. Funnily, last week also experimented with simple oscillators and had similar results to the repo/branch you posted.
For example, here is one of my recent tests:
Encoding of a 13s file:

vanilla: 85s
pre-calc: 18s
osc(toby): 24s
osc(going-digital): 25s

Decoding of a 13s file:

vanilla: 63s
pre-calc: 2s
osc(toby): 7s
osc(going-digital): 6s

I will test the increase in code size tomorrow, i.e. how many bytes each of the variations add to the base code.


PR MS2: Consul to have all the services

Service discovery to be extended to all the services

Hello Harish,
We have added all the services to Consul
You could check the same on http://<IP_ADDRESS>:8500/ui/us-central/services

Oh nice...I have validated it. Please do close the issue if there isn't anything remaining for this task.
Also, Please do add the root cause of the issue and the fix you provided. Helps us understand more and learn about the issue and other teams can avoid such issues.
Thanks,
Harish.

We have just installed consul on each server for each server and configured them together like how we had done it for DB Service

Verified. 4 of the services are discovered and registered on to Consul.


Update README.md

The language option when using the -jar requires language_in, but a different key is required when using this library.

Thanks, merged. This also applies to gavinhungry/closure-compiler-service, I'll update that README.md as well.


Tests added for retry-limit and skip-limit - Issue #19 and #23

Closing and opening pull request to trigger new Travis build.


TextField.textCapitalization updates are not respected

When changing the value passed to TextField.textCapitalization at runtime, the software keyboard on iOS continues to use the previous text capitalization value.
https://user-images.githubusercontent.com/19204050/156460765-681879a8-8aa3-4d03-ad69-068b9f739ad7.MP4
Steps to Reproduce
Run the following code:
import 'package:flutter/material.dart';

void main() {
  var noCap = false;
  runApp(MaterialApp(
    home: Scaffold(
      body: StatefulBuilder(
        builder: (context, setState) {
          final capitalization =
              noCap ? TextCapitalization.none : TextCapitalization.words;
          return Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: [
              TextField(
                textCapitalization: capitalization,
                decoration: InputDecoration(
                  labelText: '$capitalization',
                ),
              ),
              ElevatedButton(
                child: const Text('Toggle capitalization'),
                onPressed: () {
                  setState(() {
                    noCap = !noCap;
                  });
                },
              ),
            ],
          );
        },
      ),
    ),
  ));
}

Expected results: after pressing the button, the keyboard should no longer capitalize the first letter of the word but it still does.
Actual results: you can see that when the capitalization is TextCapitalization.none, the words are still capitalized.
Platforms
Special note: I only tested this on iOS. This not working on iOS is enough for me to open the issue. If someone wants to confirm that this is also broken on Android (or is not), that would be super helpful and we could elaborate on that in the issue description.

cc @jonahwilliams

The text capitalization property is used to create a TextField on the native side,which flutter bridges using the message channel APIs. I am not sure if it is actually possible to update the type of a text field after it has already been created, or if the engine would need to tear it down and reset it.
Needs investigation.
References:
Android: https://github.com/flutter/engine/blob/main/shell/platform/android/io/flutter/plugin/editing/TextInputPlugin.java#L255
iOS: https://github.com/flutter/engine/blob/615a60512c64fc4ce3230b315eb502e194cc3ab0/shell/platform/darwin/ios/framework/Source/FlutterTextInputPlugin.mm#L786


pulsar_out_bytes_total is not pegged for non-persistent topic

Describe the bug
pulsar_out_bytes_total is not pegged for non-persistent topic
To Reproduce
Steps to reproduce the behavior:

Send messages to a non-persistent topic
Wait for a few minutes
pulsar_out_bytes_total is not pegged for the non-persistent topic

Expected behavior
There should be the number of bytes for this metrics
This was tested on 2.8.

#13827  can fix this issue ?

#13827 can fix this issue ?

I think so.

@zzzming  #13827  has merged, please help close this issue. THx.


xfail finance test and applied black

This PR resolves #147.
PR Checklist
Please ensure that you have done the following:

[x] PR in from a fork off your branch. Do not PR from <your_username>:master, but rather from <your_username>:<branch_name>.
[x] If you're not on the contributors list, add yourself to AUTHORS.rst.

Quick Check
To do a very quick check that everything is correct, follow these steps below:

[x] Run the command make check from pyjanitor's top-level directory. This will automatically run:

black formatting
pycodestyle checkin
running the test suite
docs build

Code Changes
If you are adding code changes, please ensure the following:

[x] Ensure that you have added tests.
[x] Run all tests ($ pytest .) locally on your machine.

[x] Check to ensure that test coverage covers the lines of code that you have added.
[x] Ensure that all tests pass.

Documentation Changes
If you are adding documentation changes, please ensure the following:

[x] Build the docs locally.
[x] View the docs to check that it renders correctly.

PR Description
Please describe the changes proposed in the pull request:

Marks the finance test as xfail, so that this does not cause the test suite to fail even if the API call fails.

Relevant Reviewers
Please tag maintainers to review.

@ericmjl

@szuckerman if everything looks good to you, please go ahead and push the big green button! :smile:

I actually think that the currency API 200 check should be removed. The test doesn't have anything to do with our code, just uses requests to test the site.
A better pattern would be to mock the response from the API in the other finance tests and remove this one; I'll look at adding that soon.
I'm going to approve this cause it's small and will get the build going, but I will try to remove it in the near future.


UNAVAILABLE: Keepalive failed

Always report errors when installing tgz apps: io.grpc.StatusRuntimeException: UNAVAILABLE: Keepalive failed. The connection is likely gone

Hello; thanks for using microBean Helm.  It is difficult to understand what you're reporting, but you may wish to build the ManagedChannel that your Tiller instance uses with appropriate timeouts set.  Please see https://grpc.io/grpc-java/javadoc/io/grpc/ManagedChannelBuilder.html#keepAliveTime-long-java.util.concurrent.TimeUnit- for possible ways forward.


Setup build process with local properties file based on username

From @GoogleCodeExporter on May 11, 2015 15:4
Setup build process with local properties file based on username

* Include a sample properties file with sensitive data not included
* Build project into a single jar
* Build source jar
* Build javadocs
* Build api (javadoc) zip
* Build distribution folder with binary, source and javadocs
* Build distribution zip

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 13 Mar 2011 at 6:02
Copied from original issue: twogee/missing-link#3

From @GoogleCodeExporter on May 11, 2015 15:4
Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 12:01

Added labels: Milestone-Release1.1

From @GoogleCodeExporter on May 11, 2015 15:4
Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 12:01

Added labels: Milestone-1.0.1
Removed labels: Milestone-Release1.1

From @GoogleCodeExporter on May 11, 2015 15:4
Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 2:27

Added labels: Project-ant-http

This issue was moved to twogee/ant-http#3

From @GoogleCodeExporter on May 11, 2015 15:4
Done, no need for local props

Original comment by<EMAIL_ADDRESS>on 19 Mar 2011 at 2:18

Changed state: Verified


Calculator Example

a) The calculator example is broken...
Will successfully parse '5+3' giving the answer '8'...
But will also successfully parse '5+' giving the answer '5'.
The parser returns a 'MatchResult' with an error condition 'not enough arguments;expected DecimalDigit or WS' which is correct but the 'Success' value is set to 'true' and not 'false'.
The match is not a success so why is it being reported as a success???
b) Also for '5+3' there is also the same error condition... just a different errorindex...
c) As an aside: Why is there no parse tree being constructed and returned as part of the 'MatchResult'? Does this have to be constructed manually using actions?

a) The matcher will match as much as possible, and no more.  Observe that the NextIndex property of the result for "5+" is 1, not 2, which indicates that the matcher only matched 1 character, not 2.
If you want to make sure to match to the end of the input, add a rule EOF = ~.; and use it at the end of your topmost rule.  You can see an example of this in Source/Library/Generator/Parser.ironmeta.
The error condition is simply the latest error encountered in any of the choices of the grammar.  The presence of an error does not mean that the match as a whole failed.
c) No parse tree is explicitly constructed; you will indeed need to construct one using actions.


Add new resource cloudflare_magic_firewall_ruleset

Opening a draft PR for visibility.
This PR implements https://github.com/cloudflare/terraform-provider-cloudflare/issues/880 and introduces a new resource cloudflare_magic_firewall_ruleset for managing Magic Transit Firewall Rules (Magic Firewall). Magic Firewall API is piggybacking on the new Rulesets API by using

ruleset kind: root
ruleset phase: magic_transit

Docs:

API Docs for the Rulesets API: https://api.cloudflare.com/#rulesets-properties
Magic Firewall using Rulesets: https://developers.cloudflare.com/magic-transit/magic-firewall

The ruleset kind and ruleset phase for Magic Firewall Rules are abstracted by cloudflare-go and this PR https://github.com/cloudflare/cloudflare-go/pull/558
Resource Layout
resource "cloudflare_magic_firewall_ruleset" "example" {
  account_id = "b66603270ac98750f6598cba38ec720b"
  name = "Sample Ruleset"
  description = "Magic Firewall Rules"

  rules = [
    {
      action = "allow"
      expression = "tcp.dstport in { 32768..65535 }"
      description = "Allow TCP Ephemeral Ports"
      enabled = "true"
    },
    {
      action = "allow"
      expression = "udp.dstport in { 32768..65535 }"
      description = "Allow UDP Ephemeral Port range"
      enabled = "true"
    },
    {
      action = "block"
      expression = "ip.len >= 0"
      description = "Block all"
      enabled = "true"
    }
  ]
}

The resource uses a list of maps with a custom validator because the order of rules is important here. @jacobbednarz, I'd be happy to discuss if there's a smarter approach once the PR is ready to be reviewed.
Acceptance Tests:

Create Magic Firewall Ruleset and test if exists
Update Ruleset name and force new ID
Update Ruleset description and check if ID persists
Create Ruleset with single rule and test if exists
Update Ruleset with a higher priority rule

ToDo:

Finish and merge cloudflare-go PR to vendor (https://github.com/cloudflare/cloudflare-go/pull/558)
Write acceptance tests
Write documentation

I think you're on the right track with this 👍 The ordering you're intending to do here seems like a good fit for TypeList. As you've probably found out, the TypeMap can be a little finicky to build/expand but it can be done. It's a bit of a shame we can't use something like the following (which you added for Rulesets) but I can understand the type challenges.
rule {
  # ...
}

rule {
  # ...
}

rule {
  # ...
}

I'll hold off any deep review until the cloudflare-go changes have landed to avoid any knock on changes but this is an awesome start 👏

@patryk @jacobbednarz I'm opening this PR for review now that we have the relevant changes in upstream. Looking forward to your thoughts!

@patryk @jacobbednarz I'm opening this PR for review now that we have the relevant changes in upstream. Looking forward to your thoughts!

linking #671 as here as well as it's a similar case whereby we can't really test it without real disruption.

linking #671 as here as well as it's a similar case whereby we can't really test it without real disruption.

integration is green
=== RUN   TestAccCloudflareMagicFirewallRulesetExists
--- PASS: TestAccCloudflareMagicFirewallRulesetExists (8.18s)
=== RUN   TestAccCloudflareMagicFirewallRulesetUpdateName
--- PASS: TestAccCloudflareMagicFirewallRulesetUpdateName (15.39s)
=== RUN   TestAccCloudflareMagicFirewallRulesetUpdateDescription
--- PASS: TestAccCloudflareMagicFirewallRulesetUpdateDescription (14.22s)
=== RUN   TestAccCloudflareMagicFirewallRulesetSingleRule
--- PASS: TestAccCloudflareMagicFirewallRulesetSingleRule (7.56s)
=== RUN   TestAccCloudflareMagicFirewallRulesetUpdateWithHigherPriority
--- PASS: TestAccCloudflareMagicFirewallRulesetUpdateWithHigherPriority (14.47s)
PASS
ok      github.com/cloudflare/terraform-provider-cloudflare/cloudflare  59.839s
?       github.com/cloudflare/terraform-provider-cloudflare/version     [no test files]

integration is green
=== RUN   TestAccCloudflareMagicFirewallRulesetExists
--- PASS: TestAccCloudflareMagicFirewallRulesetExists (8.18s)
=== RUN   TestAccCloudflareMagicFirewallRulesetUpdateName
--- PASS: TestAccCloudflareMagicFirewallRulesetUpdateName (15.39s)
=== RUN   TestAccCloudflareMagicFirewallRulesetUpdateDescription
--- PASS: TestAccCloudflareMagicFirewallRulesetUpdateDescription (14.22s)
=== RUN   TestAccCloudflareMagicFirewallRulesetSingleRule
--- PASS: TestAccCloudflareMagicFirewallRulesetSingleRule (7.56s)
=== RUN   TestAccCloudflareMagicFirewallRulesetUpdateWithHigherPriority
--- PASS: TestAccCloudflareMagicFirewallRulesetUpdateWithHigherPriority (14.47s)
PASS
ok      github.com/cloudflare/terraform-provider-cloudflare/cloudflare  59.839s
?       github.com/cloudflare/terraform-provider-cloudflare/version     [no test files]


Questions about num_rels

parser.add_argument("--num_rels", type=int, default=4,
choices=[3, 4, 11, 12, 14],
help="how many relations are computed")
The choices of the hyper-parameter num_rels is [3, 4, 11, 12, 14], but in the conll dataset, the class is 15. So how to choose num_rels for the conll dataset?
By the way, could you please offer the whole hyper-parameters for the conll dataset? And, could you please describe the meaning of [3, 4, 11, 12, 14], respectively?

Hi, thanks for your questions. In CoNLL share task, there are 14 discourse relations + EntRel. However, EntRel is not included in the implicit part. Hence, you can directly set the num_rels as 14 for the CoNLL data.
As for another settings, 4 corresponds to the Comparison, Contingency, Expansion, and Temporal, which are the four top topics; 11 corresponds to Comparison.Concession, Comparison.Contrast, Contingency.Cause, Contingency.Pragmatic cause. Expansion.Alternative, Expansion.Conjunction. Expansion.Instantiation. Expansion.List, Expansion.Restatement, Temporal.Asynchronous, and Temporal.Synchrony, which are the eleven second-level senses.
It seems 3 and 12 are not belonging to the implicit classification task, so you can ignore them.

Thanks for your reply!


[Demo] Incorporate collapsible nested accordion view in demo app

I've implemented a collapsible accordion-style view for displaying hierarchical data. This was implemented as a proof-of-concept to validate the feasibility of implementing such UI for two of our iOS projects that incorporates such design. I thought it'd be a good idea to get this into the demo app.

Checklist
Please do the following before merging:

[ ] Ensure any public-facing changes are reflected in the changelog. Include them in the Main section.

Closing due to staleness. Please feel free to reopen if you feel strongly this should land.


Maximum sleep time due to LDL_MAC_ticksUntilNextEvent() values

Hello,
I updated successfully to the latest version 0.4.5 (https://github.com/cjhdev/lora_device_lib/commit/597630c030b035fc6f82aaf9916cefd466268693).
I see a different behaviour, compared to previous versions. The function LDL_MAC_ticksUntilNextEvent() returns always values less than 60 seconds, due to a armed LDL_TIMER_BAND timer. I use the value to calculate possible power down times for my application and that value is rather short.
Is it really necessary to have a timer for band events? The processing function is in any case called from LDL_MAC_process and at first glance I see no problems if more ticks elapse before the next call. I would like to get UINT32_MAX from LDL_MAC_ticksUntilNextEvent() when the last receive window is closed and no (MAC) messages are queued.
Kind regards,
Frank

Hello
Previous versions also set the band timer to 60s. It's a legacy thing. I kept it because the low power MCUs I use wake up regularly when hardware timers overflow and so it made no difference.
The interval could be increased. I would choose a maximum of INT32_MAX ticks as a way to ensure the time tracking in processBands() never overflows. I notice processBands() ignores differences larger than INT32_MAX but that can be changed.
LDL_TIMER_BAND exists because it seemed easier to use a single purpose timer rather than overload one of the others and then have to work out which part of the code was responsible for clearing it.
I think it is useful to have an event for when a ldl_mac.band counts down to zero. Perhaps LDL_MAC_ticksUntilNextEvent() should be refactored to check ldl_mac.band as well as inputs and timers.

I've increased LDL_TIMER_BAND interval in master to a large number that won't break time tracking

I've increased LDL_TIMER_BAND interval in master to a large number that won't break time tracking

Thanks!

Meanwhile I found a working solution for my use case, so I reopen the issue to document it. The comitted changes for LDL_TIMER_BAND interval were not sufficient.
Summary of my requirements:
I'm able to call LDL_MAC_process() regulary, with a maximum interval of several minutes, but want to deep sleep when no transmission or receive event is pending. So I want to distinguish between "important" events (retry of transmission, when channel is not available; wait for reiceive window; ..) and "unimportant" events (band events). In the latter case the LDL_MAC_ticksUntilNextEvent() wait time should not inhibit the sleep.
My solution:
I call LDL_MAC_process() in a loop only, when the mac.state is not LDL_STATE_IDLE. To compensate the missed band events after a deep sleep I ensure that LDL_MAC_process() is called before any other LDL function call.
As a result the process runs in the LDL_MAC_process() loop only after a transmit request - until the transmit was successful and the receive windows elapsed. Afterwards the device goes to deep sleep for several minutes.

Thanks for sharing your workaround.
I don't think there is a quick fix to avoid spurious wakeups with the current implementation, the whole thing needs to be reworked.

I've solved this problem in my current working branch (sx1262). I expect to merge this month.

I'm still running my mentioned solution with the new LDL version (https://github.com/cjhdev/lora_device_lib/commit/450ae80c2ca5b2651df0536d1c6157c6dfeb9807). It works well. I have also implemented a check for a maximum sleep time, so even between longer retransmission attempts (channel not available, OTAA pending) I can go into deep sleep. I see only the requirements, that after deep sleep the timer is roughly adjusted and LDL_MAC_process() is called again.
I see that LDL_MAC_priority() may also help in many situations, but did not touch my existing solution.
I close the issue, because solutions exists for the described requirement.


conductor: rework filtering and logging of data submitted to data availability

for later fixing: I wonder if a warn here is too heavy handed?  Maybe we have a warn but only at a "received X number of non parseable blobs from DA at height X"? And then have a debug level for each individual one?
Similar below, although we likely want to differentiate between non-parsable and incorrectly signed. Both could be an issue, but also could be just normal noise.

Originally posted by @joroshiba in https://github.com/astriaorg/astria/pull/150#discussion_r1262721108

closing as this doesn't seem relevant as data reading was moved


docs: add manylesson to users

Motivation
add manylesson to users
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

ok,  I enabled HTTPS for my website


Design system / badges

Updated badges to have a corner radius of 14px to appear more different than the button UI.
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=1155%3A10199


Alter isPinOrFingerprintSet typing so it is a function

Description
A simple typing fix. I'm not the best at Typescript definitions, but this small change results in Typescript transpiling code that looks like the readme with no issues, and I have verified the callback is being called on a real device
Fixes #486
Checklist

[ ] I have tested this on a device/simulator for each compatible OS
[ ] I added the documentation in README.md.
[ ] I mentioned this change in CHANGELOG.md.

@machour this should also be good to go. worst case the typing for this function was already broken but people can use the function regardless (just with the type error)


Set preferred size for a pane

Purpose
As a developer I would like to set a preferred size for when the user double clicks a sash adjacent to a pane.
Proposal
Let's play with the idea of using defaultSizes purely for setting the sizes on initial mount. The main use case here is for applications to persist the layout.
We can then introduce a concept of preferred size purely for use when resetting the panes' layout.
A good starting point would be to look at the VS Code Grid component.

Acceptance criteria

[ ] As a developer I can set a preferred size for a pane
[ ] As a developer when I double clock a sash adjacent to a pane with a preferred size that pane should be set to that size (unless minimum or maximum size conflict)

A proof of concept. The sidebar has a preferred size of 300px.

We might need to introduce the internal concept of high/low priority views. What's currently happening is that the views are given the correct sizes. However, when we relayout, we distribute space from right to left. So if the last view has been shrunk, it will immediately be expanded to fill in the missing space!

It would be helpful to allow the preferred size to be dynamic, e.g.

Percentage based size would also be nice.

Make sure to consider the behaviour when adding a new frame. Should it take on its preferred size?
Also, what happens when we reset the whole split view? How should space be allotted amongst multiple panes with preferred sizezs?


[Event Request] Report 99001025 Refresh Production Order

Could you please expand the Publisher "OnBeforeCalcProdOrder" with the new Parameter "Direction".
The Parameter is set by OnInitReport or can be choosen by User on the Request Page.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Add hidden divs for frame_id and frame_styling

Resolves #1124

In trying this out, I realize that in slicer_cli_web's code, we had imagined that we would be getting the value of an input control rather than the contained text of a div.  That could be changed and then this will work as is.


Create Distrib Test for C++ Examples

Our CI tests should catch breakages in the example code.

please CC me on the PR once you have something.

@ncteisen
Actually, such distribtest already exists https://github.com/grpc/grpc/blob/master/test/distrib/cpp/run_distrib_test.sh, but no one is checking if it passes and it seems to be broken for other reasons.
https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_distribtest/architecture=x64,language=cpp,platform=linux/3205/console