What happens when "charge on below" and "charge off above" are equal?

Just want to clarify. We have TWO options to modify:

I want to ask whether something 'bad' happens when "turn battery charging on below" and "turn battery charging off above" options are equal, say 80% (see screenshot). In this case, will the program's behaviour be "turning the battery off when it reaches exactly 80.00%, and then, just a few seconds after turning it on because the battery went down to 79.99%"? Or is it "charging up to 80% and then just hold it there providing just enough charge for that"?
Cause if it's the first option (the constant charge-on/charge-off loop), then I'm a bit concerned about it's "safety" for my battery. What I want is the second one (like on the new iPhones or some of Android phones), when I am able to just "freeze" the charging at 80%.
Sorry if I sound stupid, but that's what I really need to clarify.

Charge levels are reported strictly in integers, so it would start charging again once it drops to 79%. Once charging is disabled, MacBooks keep running off AC, so the battery will not be utilized at all unless the AC provides too little energy to keep up. From there, the battery will still slowly discharge, but it will typically take several days (I think sometimes even weeks? Never thought about it much) to drop from 80% to 79%.

Okay, so is it technically and physically safe for my battery to use your program that way?

Yes. It might not be considered best practice (as Apple themselves implemented a hysteresis for “trickle discharging”), but it should be fine.

thank you man!


Errno::ENOENT: No such file or directory - C:/dev/null when executing jar command on windows

Torquebox 4 beta 3, JRuby <IP_ADDRESS>
 C:\scripts\app>bundle exec torquebox jar
 19:09:10.670 INFO  Bundling gem dependencies
 Errno::ENOENT: No such file or directory - C:/dev/null
  initialize at org/jruby/RubyFile.java:360
        open at org/jruby/RubyIO.java:1129
       <top> at <script>:1
             at :1
             at :1
        load at org/jruby/RubyKernel.java:962
       <top> at C:\NotBackedUp\brownr12\downloads\jruby-<IP_ADDRESS>\bin\torquebox:22

I can see at https://github.com/torquebox/torquebox/blob/master/core/lib/torquebox/cli/jar.rb#L327 we check the PLATFORM constant for mswin however when using jruby this constant will always return java
I have hacked my gem to change this line to:
dev_null = 'NUL'

... now I don't get any error but I don't get any jar.

I was able to move past the error by commenting the following lines out of cli.rb
   # unless %W(DEBUG TRACE).include?(TorqueBox::Logger.log_level)
      # dev_null = PLATFORM =~ /mswin/ ? 'NUL' : '/dev/null'
      # ruby.evalScriptlet("$stdout = File.open('#{dev_null}', 'w')")
    # end

Same problem here.
Have you made a fork?
The strange thing is that my Rails conf is config.log_level = :debug so it should not go inside that block.
Why do I have TorqueBox::Logger.log_level = INFO even if Rails log level is DEBUG?

No fork from me as yet.  I only know how to get around it for myself, not sure how not break it for others,.  Also not exactly sure what this is trying to do in the first place.
Note: your rails log level is different to you torquebox log level - they are different logs, by default.


Support Laravel 5.2

Laravel 5.2 changed the way global scopes are handled. The VersioningScope class needs updating so that it correctly adds/removed the scopes.

I merged some old pending requests, and I'm going to look at the new 5.2 scopes to address this.

I believe I've resolved this, I updated the trait to handle the new scoping, and it passes all tests with updated Laravel 5.2 dev requirements.


Refetch projects when opening selector

Minor QoL improvement to refetch the list of projects when you open the selector. No longer require a page refresh to pick up a new project that you have placed in your data directory.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.80%. Comparing base (bf669f5) to head (2789508).

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk      #80   +/-   ##
=======================================
  Coverage   96.80%   96.80%
=======================================
  Files          33       33
  Lines        1410     1410
=======================================
  Hits         1365     1365
  Misses         45       45

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Running test with a comment at the end makes the test fail.

Describe your problem and how to reproduce it:
Running the test will fail if you have anything commented after the code.
Deleting the comment line allows us to pass the test.
The presence of a comment should not affect the output of the test.
(function() {
  console.log("A cozy nest is ready");
})();
// makeNest();

Add a Link to the page with the problem:
Object Oriented Programming: Understand the Immediately Invoked Function Expression (IIFE)
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures/object-oriented-programming/understand-the-immediately-invoked-function-expression-iife
Tell us about your browser and operating system:

Browser Name: Chrome
Browser Version: Version 84.0.4147.105 (Official Build) (64-bit)
Operating System: Catalina 10.15.6

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

It looks like this is caused because the test is removing every whitespace and make the code one line, then test it with a regex, that test if the string ends with a ) parentheses

This can be easily fixed by changing the regex string from instead of testing if it ends with a () it will test if it's an IIFE function. I can do this.

@LazaroFilm thanks for reporting this, it's definitely not working as intended.
@JoshuaPelealu, that's my assessment, too.  A PR would be very welcome.


add gestureHandling

gestureHandling
Type:  string
This setting controls how gestures on the map are handled. Allowed values:
"cooperative": Two-finger touch gestures pan and zoom the map, as do all mouse gestures. One-finger touch gestures are not handled by the map. In this mode, the map cooperates with the page, so that one-finger touch gestures can pan the page.
"greedy": All touch gestures pan or zoom the map.
"none": The map cannot be panned or zoomed by user gestures.
"auto": (default) Gesture handling is either cooperative or greedy, depending on whether the page is scrollable or not.

This project is no longer being maintained here. It has moved to https://github.com/parttio/Google-Maps-for-Vaadin
Thanks.


two input node give error

Pulling out two-input node give window alert dialog box.
solves issue #112 .

@orionpax00 @keshan @ivantha @sumedhe Do have a look and review this pr :)

Closing as stale.


[4.0 Beta 4] - Windows 11 64Bit - HTML5 - "max_fps" limit 30 FPS / "physics_ticks_per_second" no limit ?

Godot version
v4.0.beta4.official [e6751549c]
System information
Windows 11 64Bit - Edge/Firefox/Chrome/Opera
Issue description
Hi,
Running v4.0 Beta 4 on Windows 11 Pro 64Bit.
HTML5 builds are locked to 30 Frames Per Second when using "max_fps" ?
Using "physics_ticks_per_second" instead creates no limit on FPS ?
Is above intended behavior of Godot v4.0 ?
Thanks!
Jesse
Steps to reproduce
Download and open below project:
https://github.com/BetaMaxHero/GDScript_Godot_4_T-Story
It uses by default "physics_ticks_per_second" and there is no limit to FPS.
Switch all "physics_ticks_per_second" to "max_fps" and then build and run in browser.
HTML5 build is limited to 30 FPS on Windows 11 64Bit ?
Minimal reproduction project
https://github.com/BetaMaxHero/GDScript_Godot_4_T-Story

related to #68439

Yes, sounds like a continuation of #68439 which was closed as the 30 FPS limit appears to be a Windows 11 regression.

Duplicate of https://github.com/godotengine/godot/issues/68439. Wait for a fix to land on Windows 11 :slightly_smiling_face:


Expose augmentColor/allow for custom palette filed to

Given a user would want to follow the Color Intentions pattern for properties other than default (primary, secondary, error).  The documentation forwards a user to the material.io Color Tool.  The functionality exists in the createPalette function.

[X] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡
Expose method and dependencies of augmentTool or add flag to color definitions of setIntention: true.
Examples 🌈
import { createMuiTheme, augmentColor } from "@material-ui/core/styles"; // augmentColor isn't currently exported
import { amber, green } from "@material-ui/core/colors";
export default createMuiTheme({
  palette: {
    success: { main: green[600], augment: true },
    info: augmentColor(amber)
  }
});

Motivation 🔦
Allow user to generate a custom color intention pattern.

This is in the theme object, which is after the theme is generated.

Uh, whoops, yes. I'm not sure why it's exposed in the theme in that case. 🤔
You could probably reduce line 3 to:
const { augmentColor } = createPalette({});

But that's only a little less silly. I have no objection to it being exported directly.

Would the other proposal of providing some way to notate custom colors to augment in createTheme be something the MUI team is interested in?

@minervabot If I understand you correctly, this is now shown in https://mui.com/material-ui/customization/palette/#generate-tokens-using-augmentcolor-utility.

Upon reviewing the discussion, I agree with @minervabot. We cannot directly export the augmentColor utility because we require specific values (tonalOffset and/or contrastThreshold); otherwise, default values will be applied. This is detailed in the documentation: https://mui.com/material-ui/customization/palette/#generate-tokens-using-augmentcolor-utility. Therefore, I am closing this issue.


Some cover not detected

Hi,
I have a Delta Dore Hub with 2 covers. I use this plugin with good results since some mounth. Recently, I'm no longer able to control one of my cover, the second one is operating as expected. I have tried to uninstall plugins and MQTT, MQTT discovery is on and found my functional cover. I'm able to command both cover from Delta Dore app. Here is my log :

""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""""
TYDOM WEBSOCKET CONNECTION INITIALISING....
Building headers, getting 1st handshake and authentication....
Upgrading http connection to websocket....
Attempting websocket connection with tydom hub.......................
Host Target :
<IP_ADDRESS>
Tydom Client is connected to websocket and ready !
Requesting 1st data...

Sending to tydom client..... GET /info
##################################
##################################
Sending to tydom client..... POST /refresh/all
Sending to tydom client..... GET /configs/file
Sending to tydom client..... GET /devices/data
<<<<<<<<<< Receiving from tydom_client...

I think that lines :
1554930574 not in dic device_name
1554930574 not in dic device_type
indicate some problems ? Any idea ?

If you could run this script and run
npx tydom-client request /configs/file /devices/data /devices/meta /devices/cmeta --file tydom_output.json --username 001A25XXXXX --password YouPassword
and send us the file generated we may have a better chance to help you resolve your bug.
By any means have you renamed your cover ? It may be safer not to use accent.

Hi,
Thank you for your answer !
Here is the generated JSON (I renamed it from .JSON to .LOG in order to be able to upload it to GitHub). I confirm that I have no accents on my devices names (screen shot of the Tydom app where you can see my devices names). I effectively rename the problematic cover when he became not available, without any reappearance.
tydom_output.log
Screenshot_20201216-214329

As you can see here
"endpoints": [       {         "id_endpoint":<PHONE_NUMBER>,         "first_usage": "shutter",         "id_device":<PHONE_NUMBER>,         "name": "Porte Fenetre",         "anticipation_start": false,         "picto": "picto_shutter",         "last_usage": "shutter"       },       {         "id_endpoint":<PHONE_NUMBER>,         "first_usage": "shutter",         "id_device":<PHONE_NUMBER>,         "name": "Porte Fenetre",         "anticipation_start": false,         "picto": "picto_shutter",         "last_usage": "shutter"       },       {         "id_endpoint":<PHONE_NUMBER>,         "first_usage": "shutter",         "id_device":<PHONE_NUMBER>,         "name": "Volet Jardin",         "anticipation_start": false,         "picto": "picto_shutter",         "last_usage": "shutter"       }     ],
Your tydom returned 3 covers :<PHONE_NUMBER>,<PHONE_NUMBER> and<PHONE_NUMBER>. The first one is duplicated. And I guess it may be the cause of your trouble.
Maybe you could try to remove the "Porte Fenetre" cover from your tydom and see if the tydom remove both of them in the API response and then add it again.
Otherwise, you may need te reset you tydom to remove this ghost cover.

Hi,
It finally rework alone... don't know why...


comment configs and correct HTTP watch default

Closes #466
Added copious comments to conf/scope.yml and adjusted
conf/scope_protocol.yml to match. The idea is to document the config
schema there in the stock configs.
During the documentation effort above, found the enable setting for the
HTTP watch was incorrect in the code. The corresponding entries in the
stock config were commented out. This change puts things back to the way
they're supposed to default with the HTTP watch enabled by default and
automatically disabled when connected to LogStream.

Looks great!  Merging.


Can't display formatted markdown in vue component

I have a simple FAQ.vue component. The content is displayed but not formatted as markdown. What am I doing wrong?
I tested the simple index.html example and understand there is a dependency on vue-markdown.js. Can you explain why the vue-markdown.js file is downloaded as a script file in the webpack-simple index.html file and not included as a dependency in the npm package? I did not see instructions in the readme for how to use vue-markdown.js?
I am using the ES6 example to import vue-markdown. I get no compilation errors and no inspector warning in Chrome. I have a large dashboard application with many vue components that is supported by a laravel backend. I plan to load vue-markdown with a large markdown file via AXIOS ajax call.
Thanks for your help,
faq.vue
<template>
<div>
    <vue-markdown>##Testing 1,2,3</vue-markdown>
    <vue-markdown :source="source"></vue-markdown>
</div>
</template>

<script>
import VueMarkdown from 'vue-markdown'

export default {

components:{ 'vue-markdown' : VueMarkdown },

  data () {
    return {
        source:"#BLAH BLAH BLAH"    }
  },
...

result on screen
##Testing 1,2,3
#BLAH BLAH BLAH

Please ingore. My markdown was incorrectly formatted.


Duplicate component names in style imports file

Curious moment. If the style files of each component are renamed to index.scss, then it becomes possible to remove the duplication of names in the global index.scss file.
For example:

Rename styles with index.scss.
Update component generator to account for changes.

Hello @anjali29singh! I've assigned this issue to yourself 😄 Let us know if you have any questions/difficulty working on this issue.

PR: https://github.com/openedx/paragon/pull/2408


EventKit + Sidebar Fix

Description
Creates an EventKit like the BrandKit.

use markdown files for content
refactor several files from BrandKit to be reused in EventKit

Other changes
switched the sidebar to use css sticky instead of interaction observer
Tested
manually Confirmed that changes to existing page tests did not result in visual regressions.
new

Related issues

Fixes https://github.com/celo-org/celo-labs/issues/457
fixes https://github.com/celo-org/celo-labs/issues/456
fixes https://github.com/celo-org/celo-labs/issues/455
fixes https://github.com/celo-org/celo-labs/issues/460
fixes https://github.com/celo-org/celo-labs/issues/458
fixes https://github.com/celo-org/celo-labs/issues/459
fies https://github.com/celo-org/celo-monorepo/issues/3690

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

my apologies for the size of this

my apologies for the size of this

Yeah... I've had my share of large PRs but over 100 files is intense. Any chance we can break this up?

closing for now to breakup

starting here https://github.com/celo-org/celo-monorepo/pull/3971


fix: added rel_noopener attribute to footer external links

Fixes #2380
Describe the changes you have made in this PR -

Added rel="nooperner" attribute to footer external links

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.751% when pulling f1170805e04dcec3e5f89c34a23dbd10990bad0e on vedant-jain03:issue_2380 into 1e0321ca4d3a69966b71a315dc7efb274c81b9eb on CircuitVerse:master.

@tachyons


S3 - Fix exceptions for VersionID (#3884)

Add inspection to ensure VersionID is a valid uuid v4
Add inspection to raise exception NoSuchVersion if key and version are missing

Having a difficult time getting test suite to run correctly... attempted to follow contributing guidelines but it appears that some unrelated tests are failing locally. Could I get some help from a maintainer?

@bblommers I'm a little confused about the statement

As far as I know, the version is not guaranteed to be in UUID format

I assumed because there is limited detail about the versionID from the s3 api, Moto fell back to use uuid4s to to denote unique versions of an object. Is this assumption incorrect?
If moto is using uuid4s for versionIDs, I would think that is would be safe to inspect  if a uuid4 is being used when the user requests for an object.
I suppose an alternative PR would be changing the versionID that moto uses from a uuid4 to an encoded string that has a time stamp value. Upon requests to moto the encoded string could be decoded and inspected for validity.
Yeah in the interim we should fix the error message, but we should also take a look at getting closer to support the s3 api.
Is the a committee of maintainers that large changes have to get approved by? Or do you just have to know one or two of the bouncers to get into the club? 😛

Moto fell back to use uuid4s to to denote unique versions of an object

That assumption is correct, but if we use this as the basis for validation, the behaviour would be different between AWS and Moto.
I.e., the following would pass in Moto, but fail in AWS:
v_id = "71f5ebb0-697e-490f-871f-38cb834a66ec"
s3 = boto3.client("s3", region_name="us-east-1")
s3.get_object(Bucket=name, Key=key, VersionId=v_id)

The ideal situation/fix would be for Moto to use exactly the same format as AWS. Without knowing exactly what that is, however, I prefer Moto to be feature incomplete, rather than inconsistent.

Is the a committee of maintainers that large changes have to get approved by?

Yeah, big or far-reaching changes would have to be approved by spulec as well. I hope this PR will not become that big though :)

https://docs.aws.amazon.com/AmazonS3/latest/userguide/versioning-workflows.html
Version IDs are Unicode, UTF-8 encoded, URL-ready, opaque strings that are no more than 1,024 bytes long.
Doesn't tell much about the specific format. Doing some simple testing, it almost looks like it verifies the input against a list of all known version ID's.
If the version ID is not known: Invalid version.
If the version ID is known, but does not belong to this key: Unknown key.

@bblommers I'm going to close this PR and open up a new one to address the exception message. We might want to keep #3884 open so we can track the migrating from a uuid4 to another format for the VersionID.

Raised https://github.com/spulec/moto/pull/3887


about checkpoint_name: indicating the location for the pre-trained visual model, mainly for IU Xray dataset

Is checkpoint_name in Step 1 resnet101-5d3b4d8f.pth? If not, Could you please provide the pre-trained model which mentioned in yout paper("For IU X-RAY, we further fine-tune ResNet-101 on CheXpert")? Or should i train from scratch?

excuse me,do you solve your problems?

Sorry for not replying in time and I’ve been busy these day.
You can either tune the model yourself or you can remove the loading module and use the original resnet directly. Both solutions should work.
As for the checkpoint use in the paper, I will upload it later.
Best,
Ethan

Thank you very much for your answer and look forward to the release of the checkpoint file.

Hi,
The pretrained checkpoint has been uploaded at https://drive.google.com/file/d/1BnnnwFf3TRba-frOMY8xQEz6YIFmCI-s/view?usp=sharing
Feel free to ask other questions.
Best,
Ethan

非常感谢您
------------------ 原始邮件 ------------------
发件人: "Ethan, Wenjun @.>;
发送时间: 2023年10月31日(星期二) 下午2:03
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [wjhou/ORGan] about checkpoint_name: indicating the location for the pre-trained visual model, mainly for IU Xray dataset (Issue #3)
Hi,
The pretrained checkpoint has been uploaded at https://drive.google.com/file/d/1BnnnwFf3TRba-frOMY8xQEz6YIFmCI-s/view?usp=sharing
Feel free to ask other questions.
Best,
Ethan
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


How to transfrom pixel coordinates in a color stream to the pixel coordinates in the depth stream avoiding rs2::align?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I have a rectangle in a color image and I wanna get this rectangle for the depth image. Now I use rs2::align object with process method. But it is very costly (probably because this method align all image while I need only 4 pixels). Is it possible to find coordinates of a rectangle got for a color image in the corresponding depth image? (maybe using intrinsic/extrinsic). I had an idea to transform color pixel corrdinates to 3D color coordinates then to 3D depth coordinate and finnaly this 3D depth coordinates to 2D depth coordinates. But it seems impossible to get 3D color points from 2D color points because the rs2_deproject_pixel_to_point(float point[3], const struct rs2_intrinsics * intrin, const float pixel[2], float depth) function needs the depth parameter.

[Realsense Customer Engineering Team Comment]
@AkhmedAnam
There is no closed formula/transformation defined to align 2D images directly.
There are ways to tackle this issue. The canonical method , imho, is to apply CV algorithms based on SIFT, like OpenCV, still other ways.
Since inside the LibRS, we focus on delivering depth data, it requires depth frame to be one side of the alignment(from <-> to).

[Realsense Customer Engineering Team Comment]
@AkhmedAnam
Answered your question?

@RealSense-Customer-Engineering
I know a pixel (i,j) in the unaligned color frame and want to know the pixel's 3d coordinates (x,y,z).
"unaligned color frame" I mean the original color frame that hasn't been aligned.
I think the process should be a pixel (i,j) in the unaligned color frame -->a pixel in aligned color frame->a pixel in depth frame, then I can know the pixel's 3d coordinates (x,y,z).
As you said , "There is no closed formula/transformation defined to align 2D images directly.", you mean a pixel in the unaligned color frame can't directly transform to a pixel in the aligned color frame.
Then you said "it requires depth frame to be one side of the alignment(from <-> to).",  Can it be unaligned color frame  ->depth frame? is it possible?
Thank you very much.

Hi @yuemasy
We have added a pull-request to enable color pixel to depth pixel mapping, it will be available in next release.

One more question, if depth frame is aligned to color frame, then can I calculate 3d point based on the aligned depth frame? According to my experiment, the answer is No. I want to confirm with you.

At the moment no, but we are going to make a fix later today to allow it. Also planned for upcoming release.
Thank you for your feedback, it is important to us.

hi  @dorodnic ,
Thanks for reply. Looking forward to the new release.

I forgot to update - rsutil.h:116 lets you map color pixel to depth.

I believe can be closed


SSL handshake error

Neo4j version: Community 4.0.4
Neo4j Mode: Single instance
Driver version: Python neo4j v.1.7.6
Operating system: MacOS Catelina (10.15.4)

Steps to reproduce

Expected behavior

Following the guidelines trying to set up a connection from a python script to my neo4j instance running on localhost:

Running neo4j - no issues.

Trying to connect with python:
pip install neo4j

from neo4j import GraphDatabase

driver = GraphDatabase.driver("bolt://localhost:7687", auth=("neo4j", "password"))

Actual behavior

Trying this resolves in a ssl handshake error (see output.txt). I've tried multiple versions of openssl that are still compatible with neo4j but all with the same error result. Also tried downgrading to python3.7, didn't work either.
Please see output.txt and env.txt for the set up. This was all run inside a miniconda env.
env.txt
output.txt
neo4j.conf.txt (sorry had to convert to .txt to be able to upload here)
Please let me know if you need any additional info! Hope you can help!

See issue #368. This is due to Neo4j V4 not having a certificate. So set encrypted to false or generate a certificate and put it in the neo4j.conf


chatrooms is offline message doesnt show the channel name, but shows the chatroom name

Fixed in #1063


Add a Gitter chat badge to README.md

casdoor/casdoor now has a Chat Room on Gitter
@hsluoyz has just created a chat room. You can visit it here: https://gitter.im/casbin/casdoor.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Invalid rule violation when import is used only for type hints (flake8-bandit S404)

Hello, thanks for Ruff again! I noticed that enabling that rule can cause (probably) false-positive violations which I can't say with guarantee they're false-positive, but probably they are
Of course I can just disable that rule in the configuration file, but may be it should be discussed here about ignoring such cases
from __future__ import annotations

from subprocess import Popen

import ffmpeg

# ffmpeg here is library https://github.com/kkroening/ffmpeg-python
# Popen here is just type hint
process: Popen = ffmpeg.run_async(...)

Ruff 0.1.15
ruff --isolated --preview --select S404 --target-version py312

> S404 `subprocess` module is possibly insecure

Also it violates on such code
from subprocess import TimeoutExpired

import ffmpeg

try:
     ffmpeg.run(...)
except TimeoutExpired:
     ...

Hm this rule just seems way overzealous but there are a bunch of rules in flake8-bandit that just flag imports like this. I think my best recommendation is to turn it off. I know if we should change it unless the upstream does. I think there are other rules that actually cover use of subprocess.

Thanks for helping, actually I've turned it off, because I didn't find good reason to keep it

Yeah I'm torn on these. They're really strict, but they're also security-oriented rules, so it feels like it's against the intent to relax them. I'm gonna close for now, though I do empathize with the annoyance.


[DO NOT MERGE] Spike 3DS

Summary of changes

Spike to initiate a 3DS Verification Flow
This requires an application_context when creating an Order via ppcp_merchant_server
For now, we're forwarding these parameters to the merchant server
In the future, we should find a way to set return_url and cancel_url when calling confirm-payment-source to avoid requiring a merchant to provide a server-side integration in order to perform 3DS Verification

Checklist
~- [ ] Added a changelog entry~

Would love to see what the API looks like for the merchant. Do we have any mock up docs for what a merchant would do to integrate with 3ds?

Would love to see what the API looks like for the merchant. Do we have any mock up docs for what a merchant would do to integrate with 3ds?

Not at the moment. Currently, 3DS requires return_url and cancel_url to be forwarded to the create order POST, which would require us to provide a server side example. The assumption is that we'll be able to provide these values to confirm-payment-source in the near future and it'll just work.

Closing in favor of #62.


nice screenshots, what's with the tracking?

why?
https://github.com/KELiON/cerebro/pull/345/files

Does this just track npm usage? seems like a lot of the electron apps are invasive. Track personal stuff. Seems like I saw in a comment somewhere that this[cerebro] has that problem. Is it true? if so, can you stop? I stopped using atom for that reason, mostly. Also atom, though has cool plug-ins, mostly remote support, still kind of sucks. plenty of important things just don't work. auto indentation. Making it not better than a lot of things.

@kylepwarren it doesn't collect any personal information, just information about some application events. How often users use mouse and clicks and how often they do it with keyboard. What plugins are used more. With this information we can improve UX of the application. And it is not to collect any personal stuff. Also, since it is open-source you can always check in source code, what is tracked and what is not.
If the way of your usage of this app is so personal for you, I don't mind if you create pull-request to add checkbox in settings that switches tracking. Let me know if I can help you with code base

BTW, there is already an issue for that: #392

Is this implemented? I do not see a setting for disabling tracking?

How often users use mouse and clicks and how often they do it with keyboard. What plugins are used more.

This is as personal as gets, not sure why the developers do not think that this is a bit creepy, you are literally watching someone doing stuff in realtime.

Is this implemented? I do not see a setting for disabling tracking?

How often users use mouse and clicks and how often they do it with keyboard. What plugins are used more.

This is as personal as gets, not sure why the developers do not think that this is a bit creepy, you are literally watching someone doing stuff in realtime.

https://github.com/cerebroapp/cerebro/commit/8d2369f1c05e56bbd4f620677af12f91c31af53d
https://github.com/cerebroapp/cerebro/issues/512


Update fr.json

Update for last keys. Please consider to use the weblate foundry hub to help translators maintaining the lang files. Improve also the fame of your module with other languages as well ! Thanks for all you are doing for the foundry community.

Thanks! I'll have a look at weblate.

FYI I've added Drag Ruler to weblate


myjenkins: Update to jenkins 2.32.2 LTS

Fix https://github.com/gmacario/easy-jenkins/issues/142
Signed-off-by: Gianpaolo Macario<EMAIL_ADDRESS>

Tested locally.
LGTM, merged


Question: Deriving public key child

In implementing stealth transfers in JavaScript, I ran into tweak_add api call used to derive a key.  I have not been able to locate any code or documentation.  I'll make this ticket incase someone can direct me to this.
https://github.com/cryptonomex/secp256k1-zkp/blob/bd067945ead3b514fba884abd0de95fc4b5db9ae/src/secp256k1.c#L354-L377
secp256k1_ec_pubkey_tweak_add
This is the code that calls this API.  Also, this could be a work-around (add the call to the crypto api):
fc::ecc::public_key crypto_api::child(fc::ecc::public_key public_key, fc::sha256 child)
{
  return public_key_type( public_key.child( child ) )
}

https://github.com/cryptonomex/secp256k1-zkp/blob/bd067945ead3b514fba884abd0de95fc4b5db9ae/src/eckey_impl.h#L165
IIRC the pubkey P is tweaked with scalar b by computing P' = P + b*G (where G is the generator).

Thank You!

Why did you revert this commit?  Has the function been implemented in GUI making the API call unnecessary?

In other implementations, usually see modulus applied using the N point of the secp256k1 curve pIL.add(this.keyPair.d).mod(curve.n):
https://github.com/bitcoinjs/bitcoinjs-lib/blob/master/src/hdnode.js#L245-L246
I get the same result without the mod, but this will not always be the case.
This is where the tweak add is called but there is no modulus:
https://github.com/cryptonomex/secp256k1-zkp/blob/bd067945ead3b514fba884abd0de95fc4b5db9ae/src/secp256k1.c#L365-L375
https://github.com/cryptonomex/secp256k1-zkp/blob/a9b6595ef812e8e52b0cb56d22109de5636b82aa/src/eckey_impl.h#L170
It looks like secp256k1_ecmult is used only for adding (not modulus):
/** Double multiply: R = na*A + ng*G */
static void secp256k1_ecmult(const secp256k1_ecmult_context_t *ctx, secp256k1_gej_t *r, const secp256k1_gej_t *a, const secp256k1_scalar_t *na, const secp256k1_scalar_t *ng);

You're comparing the tweaking of a privkey in javascript (with modulus) with the tweaking of a pubkey in secp256k1.
If you look at
https://github.com/cryptonomex/secp256k1-zkp/blob/bd067945ead3b514fba884abd0de95fc4b5db9ae/src/secp256k1.c#L331
instead you will find that it uses for example
https://github.com/cryptonomex/secp256k1-zkp/blob/bd067945ead3b514fba884abd0de95fc4b5db9ae/src/scalar_4x64_impl.h#L83
which reduces the result if it overflows.

What still needs done on this ticket?

Nothing ..


ABD-35: Update Event Emitter to encrypt event messages

Update Event Emitter to send encrypted event messages to SQS.
Add Event Emitter Configuration to enable client applications to use it
for configuring Event Emitter.
Authors: @adityapahuja @michaelwalker

Need to do further investigation.


Investigate a way of using vscode-tmgrammar-test for tests

It would be better to investigate a way of using vscode-tmgrammar-test for tests.
See the official document

For grammar authors
See vscode-tmgrammar-test that can help you write unit tests against your grammar.

Thanks for the reference.
The current approach comes from https://github.com/microsoft/vscode/tree/main/extensions/vscode-colorize-tests/test and seems close to the snapshot test approach of vscode-tmgrammar-test. I will have a closer look to understand the difference.


combinedAnswers.attributesToDisplay?.filter is not a function

🐛 Current behavior
In a custom template, we used to be able to send in a dictionary as the attributesToDisplay in order to provide some additional attributes for the template engine to display. The referenced line breaks that possibility.
https://github.com/algolia/instantsearch/blob/c8411331353d700fab508849c7e93f782f37e4e6/packages/create-instantsearch-app/src/cli/postProcessAnswers.js#L68
Here's en example of how our config looks like:
    "attributesToDisplay": {
        "title": {
            "href": "url",
            "text": "hierarchy.lvl2",
            "highlight": true
        },
        "subtitle": {
            "text": "hierarchy.lvl1"
        },
    },

🔍 Steps to reproduce

Use a dictionary as the value of attributesToDisplay parameter.
Run create-instantsearch-app.

Live reproduction
https://codesandbox.io/p/devbox/lh2cmc
💭 Expected behavior
Run ./create in a terminal and witness the error.
Package version
4.64.2
Operating system
macOS 14.2.1
Browser
Firefox 122.0
Code of Conduct

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

attributesToDisplay is meant to be an array of strings, I wonder if what you're looking for is a separate data-bag to store these more "static" information. Would that help your use case?
Something along the lines of:
{
  // ... other configuration
  "attributesToDisplay": [],
  "data": {
    "hit": {
      "title": {
        "href": "url",
        "text": "hierarchy.lvl2",
        "highlight": true
      },
      "subtitle": {
        "text": "hierarchy.lvl1"
      }
    }
  }
}

If you're interested in that, we'd be happy to see a PR adding a new data field from config that's copied over to the template variables.

Thanks for the response @Haroenv. Well the benefit of being able to use a dictionary rather than an array is extending its usability which also eliminates the need for a loop when it comes to using them in the template. Unless there is an easier way to know whether attributesToDisplay contains title for example. Consider this usage:
{{#if attributesToDisplay.title}}
{{#if attributesToDisplay.title.href}}
{{attributesToDisplay.title}}
{{/if}}
{{/if}}

The attributesForFaceting seems to tolerate non-array structures:
{
    attributesForFaceting:
        Array.isArray(combinedAnswers.attributesForFaceting) &&
        combinedAnswers.attributesForFaceting.filter(
            (attribute) => attribute !== 'ais.dynamicWidgets'
        ),
}

I see your use case, but attributesForFaceting also doesn't tolerate non-array shapes, it in fact completely ignores the value and will put false if you put anything but an array of strings. I'm still happy to approve a purpose-built feature though. You're right that attributesForFaceting and attributesToDisplay are inconsistent, so I'll make a fix to reject non-arrays in both in the same way.

@Haroenv Perfect! Since I'm using it in a custom template, I can live with that for now. Thank you :)


Multiple of the same buttons - accessibility issue

Some pages like https://thecodingtrain.com/CodingChallenges/ have multiple buttons with the same text. "Watch on YT" for example. While this works fine it's problematic for accessibility.
One off the most used features of a screen reader is getting a list with all the links on a page.
When users are presented with a bunch of the same text, they are forced to go through the whole page to get a clearer picture.
What I suggest is to add a more detailed description of the link in a span that puts it offscreen.
.sr-only {   position: absolute;   width: 1px;   height: 1px;   padding: 0;   margin: -1px;   overflow: hidden;   clip: rect(0,0,0,0);   border: 0; }
    Watch on YT     : Coding challenge #121 - Logo Interpreter

Maybe instead of integrating it into the button title, we could maybe make use of the title attribute on the anchor tag?

While that should work, not all screen readers use the alt or title tag if there is a link text. If we use a span in a link text this issue is avoided


[Category] Cache Coveo call for Category Info

We're getting a lot of queries to Coveo for the Individual Category Info Pages.
This is good from an analytics PoV however, from a Coveo Search PoV, not so much.
Could we cache the call to fetch the Category Information when the page loads? Maybe cache internally for 24 hours or so.
Also, if possible, please include an option to clear this cache for the "I need to see this on Production NOW" scenarios?

Cancelling Issue because we will be fetching the Category Info details from the Crosswalk Index instead of Coveo.


start the commands "xtensa-esp32-elf-gdb -x gdbinit build/blink.elf" and the openocd reports the error below

Error: 989 14792 esp108_common.c:177 esp108_do_checkdsr(): cpu0: esp32_fetch_all_regs (line 234): DSR (8011CC13) indicates DIR instruction generated an exception!
Error: 1007 14804 FreeRTOS.c:310 FreeRTOS_update_threads(): FreeRTOS maximum used priority is unreasonably big, not proceeding:<PHONE_NUMBER>
I use the Jlink, and I can not debug the program.
Hope to get your help.
Include two log files
openocd.txt
gdb_log.txt

There is something worth sharing is that I have just succeeded in debugging.
The solution is that I am modifying "adapter_khz" parameter within "esp-wroom-32.cfg":
old:adapter_khz 20000
new:adapter_khz 2000
JTAG Wiring Table(JLINK same applies)

Just an FYI, I have succeeded to use debugger with a wroom-32 based custom
hardware and the tiao usb jtag debugger at 20Mhz (and 26Mhz).
Thanks for the openocd support!
It may be that your wires are too long for 20mhz.  I even got it at 26mhz
but didn't want to push the limits too much and didn't see much performance
difference.  The jlink may be loading down the wires with too much cap or
series r.
On Aug 11, 2017 4:15 AM, "俞敏涛"<EMAIL_ADDRESS>wrote:
Closed #901 https://github.com/espressif/esp-idf/issues/901.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/espressif/esp-idf/issues/901#event-1202278922, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHaxCPsM5y_9jBfnCUxWdA-TXiZTmB5Iks5sXBucgaJpZM4O0Zbq
.


Enable tests on CI

As mentioned in https://github.com/async-rs/async-std/pull/286#discussion_r331923531, if cargo test contains --doc arguments, only doc test is executed.

Some tests seem broken (because they were merged without being tested on CI).

could you rename test_buffered_writer_inner_into_inner_does_not_flush to test_buffered_writer_inner_into_inner_flushes? :)

Done.

Feel free to merge the PR after the change.

I don't have merge right...


Celery health check timeout exceptions and TaskRevokedError

When I use the celery check, I will get some errors every 2-3 minutes. there are two kinds of errors like below. However my celery runs well without a problem.
2023-10-07T00:02:53.476107594Z unavailable: TaskRevokedError: The task was revoked, likely because it spent too long in the queue
2023-10-07T00:02:53.476143094Z Traceback (most recent call last):
2023-10-07T00:02:53.476149594Z   File "/usr/local/lib/python3.9/site-packages/health_check/contrib/celery/backends.py", line 20, in check_status
2023-10-07T00:02:53.476164894Z     result.get(timeout=result_timeout)
2023-10-07T00:02:53.476168894Z   File "/usr/local/lib/python3.9/site-packages/celery/result.py", line 250, in get
2023-10-07T00:02:53.476172794Z     return self.backend.wait_for_pending(
2023-10-07T00:02:53.476176494Z   File "/usr/local/lib/python3.9/site-packages/celery/backends/base.py", line 763, in wait_for_pending
2023-10-07T00:02:53.476180394Z     return result.maybe_throw(propagate=propagate, callback=callback)
2023-10-07T00:02:53.476184194Z   File "/usr/local/lib/python3.9/site-packages/celery/result.py", line 364, in maybe_throw
2023-10-07T00:02:53.476188094Z     self.throw(value, self._to_remote_traceback(tb))
2023-10-07T00:02:53.476191695Z   File "/usr/local/lib/python3.9/site-packages/celery/result.py", line 357, in throw
2023-10-07T00:02:53.476195595Z     self.on_ready.throw(*args, **kwargs)
2023-10-07T00:02:53.476199495Z   File "/usr/local/lib/python3.9/site-packages/vine/promises.py", line 234, in throw
2023-10-07T00:02:53.476203395Z     reraise(type(exc), exc, tb)
2023-10-07T00:02:53.476206995Z   File "/usr/local/lib/python3.9/site-packages/vine/utils.py", line 30, in reraise
2023-10-07T00:02:53.476210795Z     raise value
2023-10-07T00:02:53.476214395Z celery.exceptions.TaskRevokedError: expired

and
2023-10-07T00:43:51.514487089Z unavailable: TimeoutError: The task took too long to return a result
2023-10-07T00:43:51.514537089Z Traceback (most recent call last):
2023-10-07T00:43:51.514543189Z   File "/usr/local/lib/python3.9/site-packages/health_check/contrib/celery/backends.py", line 20, in check_status
2023-10-07T00:43:51.514547189Z     result.get(timeout=result_timeout)
2023-10-07T00:43:51.514558590Z   File "/usr/local/lib/python3.9/site-packages/celery/result.py", line 250, in get
2023-10-07T00:43:51.514563590Z     return self.backend.wait_for_pending(
2023-10-07T00:43:51.514567390Z   File "/usr/local/lib/python3.9/site-packages/celery/backends/base.py", line 755, in wait_for_pending
2023-10-07T00:43:51.514571290Z     meta = self.wait_for(
2023-10-07T00:43:51.514574890Z   File "/usr/local/lib/python3.9/site-packages/celery/backends/base.py", line 791, in wait_for
2023-10-07T00:43:51.514588490Z     raise TimeoutError('The operation timed out.')
2023-10-07T00:43:51.514592190Z celery.exceptions.TimeoutError: The operation timed out.

Is it expected or I am missing something?
Thanks!

We have the same problem:
unavailable: TaskRevokedError: The task was revoked, likely because it spent too long in the queue
Although the issue is happening randomly, the service and the connection still work properly.
Do we know why that is happening suddenly? Is there any workaround we can use?

@miaoz2001 @raulgzm did you guys ever figure this out?


[[FIX]] Remove tautological condition

The parsing logic for the ( prefix token returns early if the token is
determined to mark the beginning of an arrow function parameter list.
Logic that follows can safely assume that the token is not part of an
arrow function parameter lists, and explicit conditions to verify this
are unnecessary.

Coverage remained the same at 94.82% when pulling f0bff585a5e3cdbacf6d9ec5f4b22c05bc3a0c67 on jugglinmike:remove-tautology into 4e59553f7e2e09ba6c15643fdcb5b4416301148f on jshint:master.

I mentioned the need for this change while reviewing that patch that introduced the code, but the change was dependent on another patch that was under review, and I forgot to follow up once it landed.


Add subscription selector and vrg selectors to avr

In this PR, review this commit only: https://github.com/RamenDR/ramen/pull/42/commits/f3accc74525b68093f0a668acc19448be4abf75e
at some point, I'll merge the two PRs.

Closing this Pull Request.  I have cherry-picked the commit in this PR into https://github.com/RamenDR/ramen/pull/41


Refactor: Decouple section separator from sections

Requested feature

Allow transitional separators at component label instead of section label . While keeping current functionality intact
Motivation

I want to separate the section separators we have now from sections and handle them at component label . The statusline function in init.lua is too complicated it is in dire need of a refractor IMO . Also allowing highlighted transitional separators will help with custom statusline a lot. And we can create cooler looks with color option. But I'm out of idea on how to do it without overcomplicating things . If anyone has an idea please share here .

I agree the statusline function is over the top for what it does. Please provide more information about your seprator idea. I don't quite get what you want to achieve here.

Please provide more information about your seprator idea.

I want to give components a global option to set left & right transitional separator . This'll be particularly useful for component that use different color in a section then the theme (Like with color option) . The evilline didn't have transitional separators if it did it would have been pretty unintuitional and complicated to handle.


getScrollAccessor on window seems wrong

https://github.com/react-bootstrap/dom-helpers/blob/536ae60e6cb13df301575ffb497f9ba02bace32c/src/getScrollAccessor.ts#L19
When offset is pageYOffset, scrollTo will be invoked with scrollTo(currentPageYOffset, newPageYOffset). When offset is pageXOffset, scrollTo will be invoked with scrollTo(currentPageXOffset, newPageXOffset).
This doesn't seem to be right. What we want to do is to preserve the value of X when we are changing Y and vice-versa.

ya seems like a bug!


lint: Rerun black with latest version

Reviews in this chain:
└https://github.com/Skydio/revup/pull/135 lint: Rerun black with latest version

#
head
base
diff
date
summary

0
679b3855
d982a75e
diff
Dec 22 15:31 PM
2 files changed, 47 insertions(+), 65 deletions(-)


Updates version number to 2.2.4

Fixes #1377

Ping @huguesv @DinoV @int19h got time to check this?

👍


pull request for e2e test at 01/21/2017 08:45:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


hook managed method

How can I add hook managed method non static?

You even don't want to check the source code


JavaScript error on document write in Sync Gateway 1.x

Bug Report
Steps to reproduce

Generate a sync function from document definitions (e.g. ./make-sync-function -j test/resources/date-doc-definitions.js build/example-sync-function.txt)
Copy and paste the sync function into a Sync Gateway configuration file
Launch Sync Gateway 1.x with the Sync Gateway configuration file (e.g. bin/sync_gateway /path/to/sync-gateway-config.json)
Add, replace or delete a document. For example:

PUT /test/dateDoc HTTP/1.1
Host: localhost:4985
Content-Type: application/json

{
    "formatValidationProp": "2016-07-17"
}

Actual behaviour
The server responds with a status of 500 Internal Server Error and the following body:
{
    "error": "Internal Server Error",
    "reason": "Exception in JS sync function"
}

Expected behaviour
The server should respond with a status of 201 Created if the document does not already exist.
Environment

Synctos: 2.2.0
Couchbase Server: any
Sync Gateway: 1.5.1
Operating system: any

Addressed by #271


Explain config_options and correct syntax in stunnel.conf.erb.

The config_options array is a powerful way to pass-in configuration options to the stunnel config file, but the current syntax is broken.

Some way to pass arbitrary options is needed, for example on Centos6 I get a FIPS warning and need to add 'fips = no' to the configuration.

Can this be merged?

Hi @qixtand
Thanks for sending in this pull request. DNSimple has adopted this cookbook from Heavy Water and I've cleaned up the master branch to get the test suites working correctly again. If you would like to get these changes into master, please rebase against master and we can look at merging this in once again.

Closing due to inactivity.
If this is still an issue please reopen or open another issue. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help!
Thanks,
Sous-Chefs


Add helm chart

As mentioned in https://github.com/kiali/kiali/issues/195 this is the chart which I've been working on.
I've done a little bit more work so that the chart can be created and packaged from the Makefile. I'm not sure if that's necessarily where you want this logic but I was expecting that this would be best to publish on Helm hub or just directly as a release artifact.
Ultimately it's not a lot different from the chart @jmazzitelli wrote originally, I did add in support for creating ingress resources from an Istio point of view (gateway/virtualservice) for now but I could see that being something the operator generally supports going forward.

Quick run of make build-helm-chart results in:
....
Building helm chart for Kiali operator
/bin/bash: helm: command not found
make: *** [build-helm-chart] Error 127

I am contemplating switching over our current "non-OperatorHub" install mechanism (bash) to this helm chart, versus what we have today which is this bash script : https://kiali.io/getLatestKialiOperator.
This will involve a lot of coordination and work (it will involve updating docs, changing the molecule tests, updating the Makefile, and notifying users). I'll need to discuss this with the Kiali team first before committing to this. But I'm starting to think about it. I'd rather have a helm chart and OperatorHub as the two main installation mechanisms rather than have a bash script and OperatorHub plus a third (helm) sitting off to the side.

FYI: I'm taking this PR and building off it to get it to work and with additional stuff. I will most likely close this PR and create a new one.

Closing this in favor of https://github.com/kiali/kiali-operator/pull/93
I took the commit in this PR and built on it to get a working helm chart. Will continue to work on it.


chore: remove engines

close #2531

https://github.com/vercel/swr/blob/8f8501ce8bfad4e8a079ad4dbe00c0b287987900/package.json#L13
we do have packageManager"<EMAIL_ADDRESS>in our package.json. So i think it is safe to delete the engines field ?


Added a custom event system

Added support for custom events.
Code is super wacky but hm... it works?
Changes :

Layer class now has a OnCustomEvent function
Application class now has a OnCustomEvent method where it iterates all the layers and calls Layer::OnCustomEvent
Added a new EventType as well as a new EventCategory
CustomEvent class and CustomEventDispatcher class

How to use :
I added a TestEvent example in Editor Layer, it works great. Right now it triggers the event on key press just for testing purposes, but it can works with any kind of condition as long as you call the CustomEvent properly.

This is actually really awesome! Though I feel like having 2 different dispatchers is weird, maybe instead of splitting them, have a function (or struct variable) to set that will define a normal dispatcher as a custom one; which will in turn alter it's functionality a small bit?
Doing the above would remove lots of redundant code and make more sense in my opinion.

About the IsCustomEvent inheritance stuff, because GetEventType needs to be override and the type conflicts, it's impossible. It's also why i can't have a OnEvent function that groups hazel events and custom events.

I share the same opinions like CleverSource, 2 dispatchers is weird, it should share the same dispatcher
About the CustomEvent, that looks good, but I wonder what the difference is between CustomEvent and the ApplicationEvent class? Isn't that why the ApplicationEvent class is there? At least CustomEvent should be an ApplicationEvent since it's raised by the application, in that case it should be named as AppCustomEvent and be put in the ApplicationEvent.h file?
I feel like that would be the primary 'separation' and could be used by the dispatcher. After that, there should be another mechanism to seperate different derived classes of CustomEvent, what about the CustomEvent class providing a unique ID for each type of derived class? Maybe CustomEvent itself could then have the unique ID 0?

I also believe doing what LovelySanta is stating would allow for a function such as IsCustomEvent which by default returns false but for AppCustomEvent will return true by overriding. This would allow you to remove OnCustomEvent from the Application class and just use OnEvent and alter one like of code for custom events.
I'll test this theory out later, but it sounds to me like it would work, if someone else can test this before I can please comment on your results!

We'll be moving to C# soon for scripting, so a "gameplay" style event system isn't really needed in C++ (you would just build one of these in C#). The core engine event system is meant more for system/application events, such as the ones already present, and isn't meant to be extended.


[TCK]:  Inconsistent Enum Persistence Between Jakarta Persistence and Jakarta NoSQL Affects Query Behavior in TCK Tests

Specification Version
1.0.0
Bug report
There is an inconsistency in how enums are persisted by default in Jakarta Persistence (JPA) and Jakarta NoSQL, which significantly impacts queries and the related TCK (Technology Compatibility Kit) tests that involve fields of enum type.
Issue Overview:

Jakarta Persistence (JPA):

By default, JPA persists enums as ordinal values (integers) in SQL databases. This means the position of the enum in its declaration (e.g., 0 for the first constant, 1 for the second, etc.) is stored in the database.

Jakarta NoSQL:

In contrast, Jakarta NoSQL persists enums as their string names by default in NoSQL databases like MongoDB. This means that the actual name of the enum constant (e.g., "ONE", "PRIME", "COMPOSITE") is stored.

Impact on Query Behavior:
This difference in storage strategies leads to inconsistencies in query results between SQL and NoSQL databases when dealing with enum fields. Specifically:

Sorting Behavior:

In SQL, sorting by an enum field will follow the ordinal (numerical) order. For example, ONE (0), PRIME (1), COMPOSITE (2).
In NoSQL, sorting by the same enum field will follow lexicographical (alphabetical) order. For example, "COMPOSITE", "ONE", "PRIME".

Filtering and Equality Checks:

Comparisons and equality checks against enum fields will behave differently depending on whether the field is stored as an ordinal or a string, potentially leading to unexpected results in TCK tests.

Solution
That is a classic case for test specialization; thus, I could create tests to verify it on the NoSQL side; we already have a package for NoSQL. The goal could be to use the NoSQLEntityTests for this.

I mean, that's why we say explicitly:

This specification does not define an order for the sorting of Java enum values, which is provider-dependent. A programming model for entity classes might allow control over the order of enum values.

Thus, I only need to change the TCK;
@njr-11
please let me know if it is ok to create those tests at NoSQLEntityTests.

It looks like TCK tests were never updated accordingly after the decision was made to have order of enums be vendor-specific. I see at least 4 tests that depending on a NumberType attribute to be ordered in a particular way.  Creating NoSQL-specific tests isn't the right answer.  The TCK would still be invalid for anything else that isn't JPA.  JPA by the way can configure how the enum value is persisted and could be made to use a String instead, but that isn't the right answer either due to what the spec says.  I'm thinking the least intrusive way to correct this will be to duplicate the NumberType attribute on the entity into two attributes where one is the enum (preserving most tests as is because they don't order by it) and another is a numeric constant value that can be used to consistently order in the way that tests already expect.  I can look into this on Monday.


Review NPC multiattack output and tab index order

As per requests here - https://app.roll20.net/forum/post/1108598/#post-1255815

Can you change the display order so that the multiattack output appears below the effect?  This way the first attack triggered is listed first in the chat output instead of last.

Macro output will be significantly changing due across the sheet with the upcoming roll templates feature due to be released soon on Roll20.  As such i'm closing this issue and the changes for all sections of the sheet will be tracked through issue #79


Page title

current page title, ie the name in the browser tab, displays "System Object Example". I think this should be the name of the world.

taken care of in #153


Attributes requiring single quotes don't work

I use the HTMX inline partial pattern from Luke Plant (https://github.com/spookylukey/django-htmx-patterns/blob/master/inline_partials.rst#block-selection-in-the-template) regularly. The pattern relies on using hx-vals='{"use_block": "some_block_in_my_template"}'. Importantly, the first quotes need to be single quotes, not double quotes. However, Cotton seems to convert all single quotes to double quotes. It would be awesome if Cotton would keep the quotes as given.

@killianarts which version are you running, there was a PR that added this functionality a while back..

@wrabit I am on the latest version (1.1.1).

form.html
<c-vars method="post" hx-swap="outerHTML" class />
<form action="." method="{{ method }}" hx-{{ method }}="." class="{{ class }}"
      hx-swap={{ hx_swap }} {{ attrs }}>
    {% csrf_token %}
    {{ slot }}
</form>

<c-form hx-target="#dashboard-content"  hx-vals='{"use_block": "content"}'>
        {{ form }}
        <button name="submit" type="submit">
            {% trans 'Create Ticket' %}
        </button>
    </c-form>

I originally wanted to do this:
<c-vars method="post" hx-swap="outerHTML" block class hx-vals />
<form action="." method="{{ method }}" hx-{{ method }}="." class="{{ class }}"
      hx-swap={{ hx_swap }} hx-vals='{"use_block": "{{ block }}" {% if hx_vals %}, {{ hx_vals }}{% endif %}}' {{ attrs }}>
    {% csrf_token %}

    {{ slot }}
</form>

<c-form hx-target="#dashboard-content" block="content" class="space-y-5" hx-vals='"test": "test value"'>

But this is the rendered HTML:
<form action="." method="post" hx-post="." class="space-y-5" hx-swap="outerHTML" hx-vals="{"use_block": "content" , ""test": "test}" hx-target="#dashboard-content" value"">

Thanks! I ran pip freeze | grep cotton and it confirmed my version is 1.1.1. After some messing around with some other stuff, this problem seems to have resolved itself...
Thanks for the advice on constructing the component! action and method are there for progressive enhancement (plus my experience has been that less things mysteriously break if I do things the browser expects).

@killianarts seems to be resolved but let us know if the issue comes back


Special For Python3

prepare_data.py
"from urlparse import urlparse" should be replaced by
"from urllib.parse import urlparse"

"python -m SimpleHTTPServer"    should be replaced by
"python -m http.server"

Hey - fixed the import issue with Python3.
The http server snippet was just a suggestion, so leaving as-is ATM.
Cheers


terraform provider

Expected Behavior
Mergify terraform provider
Actual Behavior
Cannot setup mergify with infrastructure as code
Specifications
All setting in admin dashboard of mergify available to setup via terraform

Duplicate of https://github.com/Mergifyio/mergify/issues/5080


Refactor existing go API to AWS lambda API

Lots of stuff in this PR:

Move to AWS lambda API
Using terraform to apply changes
Added crontab example
Updated reporting-tool to work with new API

Closes #5


valgrind.supp: suppress helgrind warnings for external libraries

Fixes: #14163
Signed-off-by: Jason Dillaman<EMAIL_ADDRESS>

verified correct results: http://qa-proxy.ceph.com/teuthology/jdillaman-2016-01-12_16:46:19-rbd-wip-14163-jewel---basic-multi/26127/teuthology.log


NoneType' object has no attribute 'tzinfo'

2020-03-17 10:33:23,427/1584441203.427092:DEBUG:Connected to ClickHouse server version 20.3.2, revision: 54433
2020-03-17 10:33:23,527/1584441203.527656:DEBUG:Query: INSERT INTO salaryboard.experiences_2 (id, internal_id, source, source_id, source_last_updated, employee_id, location, job_title_user_input, sbji_id, sbji_wi, sbji_wv, l1_code, l2_code, l3_code, l4_code, l5_code, onet_l5, onet_l6, company_user_input, company_id, company, industry_user_input, industry_id, industry, sb_updated, created_at, updated_at, start_date, start_month, start_year, end_month, end_year, end_date, duration, grade_prefix, current_position, employment_type, performance_appraisal_rating, has_bonus, has_aws, has_sales_commission, has_tips, has_stock, bonus_amount, bonus_currency, aws_amount, aws_currency, sales_commission_amount, sales_commission_currency, tips_amount, tips_currency, owner_type, owner, owner_id, status, auto_match_sbci_done, auto_match_sbji_done, instance, sbgi_l5, sbgi_l4, sbgi_l3, sbgi_l2, sbgi_l1, latitude, longitude, auto_match_sbgi_done) VALUES
2020-03-17 10:33:23,528/1584441203.528208:DEBUG:Block send time: 0.000047
2020-03-17 10:33:24,864/1584441204.864394:CRITICAL:QUERY FAILED
2020-03-17 10:33:24,864/1584441204.864563:CRITICAL:ex='NoneType' object has no attribute 'tzinfo'

Same issue. Did you check your datetime columns?

Got same issue here. What could be the root cause?
This often happens.
I test with the following mysql table:
CREATE TABLE `users` (
  `id` int(10) unsigned NOT NULL AUTO_INCREMENT,
  `role_id` int(10) unsigned NOT NULL,
  `name` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL,
  `email` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL,
  `password` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL,
  `password_timestamp` timestamp NULL DEFAULT NULL,
  `last_login` timestamp NULL DEFAULT NULL,
  `active` smallint(6) NOT NULL DEFAULT 1,
  `remember_token` varchar(100) COLLATE utf8mb4_unicode_ci DEFAULT NULL,
  `created_at` timestamp NOT NULL DEFAULT '0000-00-00 00:00:00',
  `updated_at` timestamp NOT NULL DEFAULT '0000-00-00 00:00:00',
  PRIMARY KEY (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=333 DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;

Encountered same problem, just wondering why was this issue closed?

My original mysql table structure is
t_users (
`created_at` datetime(3) DEFAULT NULL,
 `updated_at` datetime(3) DEFAULT NULL,
 `deleted_at` datetime(3) DEFAULT NULL
)ENGINE=InnoDB CHARSET=utf8mb4 COLLATE=utf8mb4_general_ci

But migrated clickhouse table structure is
created_at DateTime
updated_at Nullable(DateTime)
deleted_at DateTime


Fix tests for Sphinx 7.2

Sphinx changed “Permalink” to “Link” in https://github.com/sphinx-doc/sphinx/issues/10049.
I will try to make it backwards-compatible in the next commit.

The updated regression outputs are working, but the test fixture change isn't working for Python 3.7 and 3.8 which use older versions of sphinx

Decided that we should stop testing earlier versions of sphinx (and therefore Python), so we don't have to keep updating tests for non-functional changes to sphinx and its dependencies.


Show json invalid hint if json is invalid, not only if it's invalid according to schema

Invalid Json Hint above Questionnaire Execution only appears, if JSON is valid, but not according to schema. Also show it if JSON is invalid.
I removed the validation by jsonschema library. Therefore each editor (form, JSON editor) has its own validation by its own features. Before, validation was a combination of own and common functions. If the two validations lead to different results in any case, we should have a closer look. But as both base on JSON schema, this should actually never happen.
If the editor is in JSON Mode: Should we block changing back to form mode?
If the user added invalid properties or even the JSON is completely invalid, this would not be visible in the forms, but JSON is still invalid.

I like the change. And I would not block the switching between Json and Form mode, if the JSON is invalid.

The hint above the questionnaire execution still stays and points out that something is wrong (even though you cannot see the error in the Form Editor)
Someone who messes up the Json in the JsonEditor should also find the way back to the JsonEditor, if something is not working and the message reads "Json invalid" above the execution.

I just found a bug on that pull request. Seems like onChange and onValidationError do not always trigger in the same order. Therefore sometimes JSON is a valid object, but invalid according to the schema, but the invalid message disappears. I'll have another look at that. 🙂


fix(docs): Extract jmespath code examples

Issue number:

#1064

Summary
Extract code examples from jmespath functions documentation and fix syntax errors
Changes

Please provide a summary of what's being changed

Changes:

Extract code examples
Run isort, black
Fix python syntax errors
Fix line highlights
Add make task

Fixes:

[x] Fix Deserializing JSON before using as idempotency key example in JMESPath - Missing import
[x] Fix custom_jmespath_function.py example in JMESPath - Invalid python syntax

User experience

Please share what the user experience looks like before and after this change

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of my this change
[x] Changes are tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

View rendered docs/utilities/jmespath_functions.md

Codecov Report

Merging #1120 (82f8de2) into develop (b577366) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1120   +/-   ##
========================================
  Coverage    99.96%   99.96%
========================================
  Files          119      119
  Lines         5378     5378
  Branches       613      613
========================================
  Hits          5376     5376
  Partials         2        2

Continue to review full report at Codecov.

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


Add wasm_runtime_get_wasi_exit_code

https://github.com/bytecodealliance/wasm-micro-runtime/issues/1738

i plan to use this for https://github.com/WebAssembly/wasi-testsuite/pull/16


[Node.js] Update packaging script to fix npm install error

We need to add the wrappers/CMakeLists.txt file to make it compile when npm install.

@halton PTAL thanks.


New test creation feature

Fixed Issues
#20859, #20858, #28867, #20869 and #20870
Files Changed
TestsController.cs - created server side api
Test.cs- created model for test
Response.cs- created model to store response
ITestsRepository.cs-declared methods for adding new test, checking a name to be unique and generating random links.
TestsRepository.cs- methods for adding new test, checking a name to be unique and generating random links by implementing ITestsRepository.cs
tests-dashboard.component.ts- modified open dialog method
test-create-dialog.component.ts- method for adding new test
test-create-dialog.html- created two way data binding with test name and added validations for test name
test.model.ts- added a model Response for storing response.
tests.service.ts- added a method for get and post service
test.module.ts- solved lint issue
IStringConstants.cs-  added a property which stores characters for random link creation
StringConstants.cs-  returning characters for random link generation by implementing IStringConstants.cs
Checks

[x] Naming Conventions
[x] Server side code comments (proper English, grammar and no spelling mistakes)
[x] Client side code comments (proper English, grammar and no spelling mistakes)
[x] Unused variables, methods, blank spaces and name spaces.
[x] Optimal Code and code formatting
[x] Commit History is proper, no merge is done.
[x] Commit/pull request messages should be proper to justify your feature
[x] All test cases are executed provided by tester.

Known Issues
-There was an issue with component interaction and was resolved successfully

There was problem in rebasing this new-test-creation-feature branch with the development branch and was resolved later.
-Many a times faced issues with the Visual Studio application and the system which took a lot of time

Comments
Test cases provided by the tester applicable in the feature upto now,  are verified.

SonarQube analysis reported 4 issues

 2 critical
 2 major

Watch the comments in this conversation to review them.
2 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 TestsRepositoryTest.cs#L31: Return "Task" instead.
 TestsRepositoryTest.cs#L42: Return "Task" instead.

Required changes done, kindly review

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 TestsRepositoryTest.cs#L13: Remove the "_bootstrap" field and declare it as a local variable in the relevant methods.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 TestsRepositoryTest.cs#L16: Remove the "_bootstrap" field and declare it as a local variable in the relevant methods.

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 TestsRepositoryTest.cs#L16: Remove the "_bootstrap" field and declare it as a local variable in the relevant methods.


Rename Porter to PorterLB

A couple of reasons from my consideration:

There is a CNCF sandbox project Porter already
The domain name is porterlb.io
LB means Load Balancer, it might be easier to understand and memorize the project feature.

If you agree to change Porter to PorterLB, please comment on this issue and make the relevant changes accordingly.

/assign @kubesphere/toc @duanjiong @Patrick-LuoYu

Agree. PorterLB sounds better to me.

So far, most of the project maintainers have agreed with this proposal. Can we start to update the project name including the Github repository and website?

Of course you can, you may need someone with permission to rename the repository, and then I'll change the name of the imported package in the code later

Of course you can, you may need someone with permission to rename the repository, and then I'll change the name of the imported package in the code later

Ok, I got it.

The repository and project name has been renamed. Resolved and closed.


Improve the checkOverrides function

The checkOverrides function in sort.go is part of the "hot path" for redirection.
However, it does significant string parsing for each invocation.  Switch it to pre-process all the overrides to a CIDR at startup so the "hot path" does no string manipulation, just byte manipulation.

@bbockelm Please assign this issue to me


Confused with binning output file

Hello, thanks for the exciting tool.
I would like to try out the tool but I am not sure about the requested binning file.
I tried using the prepResult.py script but I suspect the output is wrong. As input, I used the folder of the SPAdes output (metasample1/metaSpades). I ran it as following
python prepResult.py --binned 'metasample1/metaSpades' --assembler SPAdes --output 'metasample1/metaSpades/z_graphbin'
The following message was sent to stdout:
Formatting initial binning results
Writing initial binning results to output file
Formatted initial binning results can be found at /metasample1/metaSpades/z_graphbin/initial_contig_bins.csv
Bin IDs and corresponding names of fasta files can be found at metasample1/metaSpades/z_graphbin/bin_ids.csv
Thank you for using prepResult for GraphBin!
The file bin_ids.csv has this:
before_rr.fasta,1
contigs.fasta,2
first_pe_contigs.fasta,3
scaffolds.fasta,4
While the file initial_contig_bins.csv has this:
NODE_1,1
NODE_2,1
NODE_3,1
...
NODE_452809,4
NODE_452810,4
If I understood correctly, does this mean that all contigs belong to 4 bins?
Also, if this is correct which .gfa file should I use as input? SPAdes produces assembly_graph_after_simplification.gfa, assembly_graph_with_scaffolds.gfa, and strain_graph.gfa. I tried using all of them with contigs.paths and initial_contig_bins.csv
and obtained the same error:
ERROR - Please make sure that you have provided the correct assembler type and the correct path to the binning result file in the correct format.
Sorry for the long post I am new to whole metagenomics, trying to catch up.

Hello @FilipeMatteoli,
Thank you for posting this issue. The prepResult.py script is used to format the binning result obtained from an existing binning tool, not the original assembly.
You should firstly bin the contigs found in the contigs.fasta file using some binning tool (e.g. MaxBin2, MetaWatt, CONCOCT, etc.). Normally these binning tools provide the contigs in fast files corresponding to each bin. You have to provide the folder with this binning result to the prepResult.py as the --binned parameter. Hope this clears out your issue.
Let me know if you have any further questions.
Thank you!

Hello @FilipeMatteoli,
Thank you for posting this issue. The prepResult.py script is used to format the binning result obtained from an existing binning tool, not the original assembly.
You should firstly bin the contigs found in the contigs.fasta file using some binning tool (e.g. MaxBin2, MetaWatt, CONCOCT, etc.). Normally these binning tools provide the contigs in fast files corresponding to each bin. You have to provide the folder with this binning result to the prepResult.py as the --binned parameter. Hope this clears out your issue.
Let me know if you have any further questions.
Thank you!


[Refactor]: footer component of privacy policy page ⚒✨

File Name
policy/policy.html
Reason for Refactoring the Code

The footer component of Privacy Policy doesn't have necessary links like other pages of OS-CODE Website.
So the user could not able to navigate to other pages once they arrived Privacy Policy Page.
We need to refactor the Footer Component, So that all links should be available to the User for Navigation.

Screenshots (Current State)

HOMEPAGE - FOOTER
PRIVACY POLICY PAGE - FOOTER

Checklist

[X] I have checked the existing issues.
[X] I have read the Contributing Guidelines.
[X] The changes don't break the code and don't introduce new functionality.
[X] I am willing to work on this issue (optional).
[X] I am a GSSoC23 contributor.

Hey there @Divesh2003 👋🏻
Please make sure to assign this issue to me to make the work done, Thank You!


no way to start the server

please check it on https://repl.it!
or google cloud shell.
I really need this to work.
and I can help improving it after I have a working copy.

please use search bar on issues
https://github.com/localtunnel/server/issues?q=start+server


Install script may need updates for focal++

[X] I have checked the wiki
[X] I have searched for similar issues

Describe the bug
The script references PPAs etc. that don't work in focal+ (jammy, impish), though if you ignore those, you can proceed with manual installs. Those PPAs are stebbins/ and mc3mann/
It seems like everything is there whether or not you use those particular PPAs (handbrake and abcde still get installed) when following through with manual installation procedures. Are there more reliable sources for those particular pieces of software? It looks like both of those PPAs last got updated with Focal and haven't seen many updates (or logins etc.) since.
Could be a non-issue if the maintainers update, just not sure on that based on their history.
I know jellyfish is very new but impish isn't. Could be they were waiting for the next major release (though impish was, though interim).
To Reproduce
Steps to reproduce the behavior:

Attempt to install with script on jammy jellyfish
Observe some PPAs error out
Try to install manually, after having ran the script once (despite apt errors), skipping the first line referencing mc3mann's PPA.
Everything installs with no issues.

Environment
Ubuntu Jammy Jellyfish
ARM Latest stable

Sorry it took so long for me to get to this, I do want to provide context for closing this.
ARM is fundamentally built on 20.04. Over the past few years, after the upstream went silent and people made piecemeal contributions, the userbase spread to multiple different versions of Ubuntu, Debian, Gentoo, etc. The docker version was built on Ubuntu 20.04 as well.
At the moment, I am solely focused on recombining the stale docker branch and main thick-client branch into one branch that works for both. I don't have time or resources to test the thick client on multiple versions of multiple different OSs, especially when the docker container is pegged to a single version of a single OS.
Once this work is completed and we don't have to worry about maintaining two versions (or more accurately, letting one die and focusing on the other) of the same codebase, then I'm happy to reconsider supporting multiple OSs and all the OS-specific glitches that come with it. For the moment, however, I don't think it's a tough ask for users to use one OS and version.
Tl;dr: The codebase for ARM is built on pillars of sand that we're finally making progress on shoring up, I can't focus on that when I have to test every fix on a dozen different OS/Version combos.
Hope that helps!

@shitwolfymakes That makes total sense and I suspected something similar. I've since downgraded back to Focal (Jellyfish still has some other growing pains it seems) so everything is fine for now anyhow. 😊

Lovely! I don't wanna come off as some arch btw-esque distro zealot. I just don't have time for it!


Feature request: Allow Manual Reminder(s) Settings

I'm in search of a way to add All-Day Calander Events to my Google Calendar with appropriate Reminders.  With very specific G.Cal and Iphone settings I can get desired results (with an additional manual step). Within HA, All-Day Events alarm at Event Notifications time and not All-Day Event Notifications.
Searching around the WWW I found that the API supports it and ventured into HA to make the change VIA custom_components. Only to find the underlying dependency.
According to some forums, it seems this is a known behavior that the allow-day event notifications doesn't work outside of the Google environment. Which leaves defining non-default reminders.
Is there anyway the reminders API portion could be implemented?
event = {
  'summary': 'Google I/O 2015',
  'location': '800 Howard St., San Francisco, CA 94103',
  'description': 'A chance to hear more about Google\'s developer products.',
  'start': {
    'dateTime': '2015-05-28T09:00:00-07:00',
    'timeZone': 'America/Los_Angeles',
  },
  'end': {
    'dateTime': '2015-05-28T17:00:00-07:00',
    'timeZone': 'America/Los_Angeles',
  },
  'recurrence': [
    'RRULE:FREQ=DAILY;COUNT=2'
  ],
  'attendees': [
    {'email'<EMAIL_ADDRESS>    {'email'<EMAIL_ADDRESS>  ],
  'reminders': {
    'useDefault': False,
    'overrides': [
      {'method': 'email', 'minutes': 24 * 60},
      {'method': 'popup', 'minutes': 10},
    ],
  },
}

Can you explain a bit more about the use case of what you are trying to accomplish? (e.g. the actual real world scenario). Thanks, its helpful to have additional context to understand the implementation goals.

This has been released with 4.2.0 https://github.com/allenporter/gcal_sync/releases/tag/4.2.0

Thank you for your prompt response.
The use case is adding an All-Day Event from HomeAssistant. The shortfall is that, doing so currently renders the reminders as 1 hour before (23:00) and 1 day before Midnight (00:00).
I'm wanting to add a Reminder Override via Service Call in HA.
This morning I bumped gcal_sync to 4.2.0 and tested. Just one little problem.
Get an error: Error executing script. Error for call_service at pos 1: Error from API: 400: Cannot specify both default reminders and overrides at the same time.: Bad Request
HA Object Call:
  id=None
  ical_uuid=None
  summary='Bowling'
  start=DateOrDatetime(date=datetime.date(2023, 3, 11),
  date_time=None, timezone=None)
  end=DateOrDatetime(date=datetime.date(2023, 3, 12),
  date_time=None, timezone=None)
  description='Birthday bowling'
  location=None
  transparency='opaque'
  status=<EventStatusEnum.CONFIRMED: 'confirmed'>
  event_type=<EventTypeEnum.DEFAULT: 'default'>
  visibility=<VisibilityEnum.DEFAULT: 'default'>
  attendees=[]
  attendees_omitted=False
  recurrence=[]
  recur=None
  recurring_event_id=None
  original_start_time=None
  reminders=Reminders(
    use_default=False,
    overrides=[
      ReminderOverride(
        method=<ReminderMethod.POPUP: 'popup'>,
        minutes=60)])

json.loads(event.json(exclude_unset=True, by_alias=True)) parse:
{
   "summary":"Bowling",
   "start":{
      "date":"2023-03-11"
   },
   "end":{
      "date":"2023-03-12"
   },
   "description":"Birthday bowling",
   "reminders":{
      "overrides":[
         {
            "method":"popup",
            "minutes":60
         }
      ]
   }
}

Notably, use_default disappears. Which I believe is the result of exclude_unset=True.
model.py update:
use_default: bool = Field(alias="useDefault", default=**True**)

Thanks, do you want to send a PR to fix? Sounds like the issue is the default...

Thanks, do you want to send a PR to fix? Sounds like the issue is the default...


add cropZoomImgs option for fullscreen images

Adding an option to crop and zoom pictures so that no borders are shown if image ratio does not match the screen ratio.

Nice idea. Is it ready to merge?

but please create pull request to develop...see CONTRIBUTING.md

Nice idea. Is it ready to merge?

I would feel better if someone could take a second look.

but please create pull request to develop...see CONTRIBUTING.md

Ah, I was not aware of that. Installed git-flow and tried to init, but it says there is no "next release" branch. Maybe you can help me to get everything in the needed format?

to be continued here: https://github.com/LukeSkywalker92/TeleFrame/pull/130


adjusted dwb_local_planner_params

use split_path option to enforce following complex paths (see here)
more trajectory samples over a smaller simulated time. This fixes a
problem where the robot would stop too far away from the goal, as all
possible trajectories either overshot the goal, or were too short to
reach into the next gridcell of the critics.
removed Oscillation and PathAlign critics during the debugging process. Did not put them back in as they don't seem to be necessary.

Thanks! I've put the PathAlign critic back in (without it, the robot veers more from side to side instead of keeping the nose on the path), changed some comments and merged via bc6c4ff4 .


add autocomplete for adb -s option

send commands to a specific device

Most desired feature.

Awesome!

@neworld hi~ I create a new PR show device info #6472


syntax higlighter processor add text is none condition

:facepalm: need the condition on one more place.
Now tested again the whole code where we use structlog_pretty and working.
Sorry I have not tested thoroughly enough the first time.

Cool, no worries! Could you rebase onto master though?

Should be done now.

But the CircleCI tests failed :ocean:

Would you know what the circleci: Build Error means?

the one which failed to run through (curcleci error
I did not change anything in the build pipeline... @underyx

#!/bin/sh -eo pipefail
# ERROR IN CONFIG FILE:
# [#] required key [jobs] not found
#
# -------
# Warning: This configuration was auto-generated to show you the message above.
# Don't rerun this job. Rerunning will have no effect.
false

Exited with code exit status 1

CircleCI received exit code 1

@underyx please if you have time

I think it's just faulty CI config on master.


Get old value and new value when suscribing to a value changed event

Hello!
I would like to compare the old value with the new value generated when subscribing to some variable changes. Is this supported or do I have to find a workaround somehow?
Thanks!

Is this supported
I don't think it is supported.

do I have to find a workaround somehow?
Yeah. Maybe you need to cache the old value and compare with the new value.


Skipped tasks show up multiple times in report

A little bug snuck into the 0.28.0 release. Will submit PR shortly.

Fixed by #2177


Problem with Any Bootstrap Theme

Buttons in Any bootstrap theme become all white and uncoordinated, all in their colors and classes. Result: all become white or invisible.

It was after installing version 0.9.6-beta.18

Can you send a screenshot of your any bootstrap config menu.

only happens with bootswatch themes

Can you also check the plugin version? Is it 0.5.13

Can you also check the plugin version? Is it 0.5.13

Yes sr!, It is the version

Because this was actually a problem in 0.5.12
Can you try to move one of the color selectors a little bit, click the finish button, and perhaps try ctrl + f5 to reload the css files
I hope this does it already

Ok, Thank you! @christianhugoch, is solved..


Add Bunyan stream for OpenTelemetry

Is your feature request related to a problem? Please describe
This is a feature request to build a package to export Bunyan logs to OpenTelemetry backends. This would likely be done as a custom Bunyan stream utilizing the OpenTelemetry Logs API.
Describe the solution you'd like to see
Users can configure Bunyan to export logs to OpenTelemetry backends.
Related to https://github.com/open-telemetry/opentelemetry-js-contrib/issues/1558

Here is a proof-of-concept for this, as a start. I have a number of open questions and TODOs, but I'd appreciate feedback. If/when this seems a reasonable direction, then I can work on a PR to this repo for this.  @hectorhdzg You also mentioned you were interested in this.
https://github.com/trentm/otellogsplay
Usage looks like this:
// ...
var log = bunyan.createLogger({name: 'myapp'});
log.addStream({
  type: 'raw',
  stream: new OTelBunyanAppender()
})

And that "appender" implementation is here: https://github.com/trentm/otellogsplay/blob/main/lib/OTelBunyanAppender.js

Here is an example run of that script:
% node bunyan-otel.js
{"name":"myapp","hostname":"purple.local","pid":46450,"level":30,"msg":"hi","time":"2023-07-10T23:16:17.930Z","v":0}
{
  timestamp:<PHONE_NUMBER>930000,
  traceId: undefined,
  spanId: undefined,
  traceFlags: undefined,
  severityText: 'info',
  severityNumber: 9,
  body: 'hi',
  attributes: { name: 'myapp', hostname: 'purple.local', pid: 46450 }
}
{"name":"myapp","hostname":"purple.local","pid":46450,"level":30,"msg":"in a span","time":"2023-07-10T23:16:17.941Z","v":0}
{
  timestamp:<PHONE_NUMBER>941000,
  traceId: '2891fcee99788edfabfa3c344edb70f8',
  spanId: '81b5a5a8de16a34d',
  traceFlags: 1,
  severityText: 'info',
  severityNumber: 9,
  body: 'in a span',
  attributes: { name: 'myapp', hostname: 'purple.local', pid: 46450 }
}
{
  traceId: '2891fcee99788edfabfa3c344edb70f8',
  parentId: undefined,
  traceState: undefined,
  name: 'manual-span',
  id: '81b5a5a8de16a34d',
  kind: 0,
  timestamp:<PHONE_NUMBER>940000,
  duration: 1946,
  attributes: {},
  status: { code: 0 },
  events: [],
  links: []
}

Does this look like a reasonable start?
If so, I have a number of open questions and TODOs in the README and in https://github.com/trentm/otellogsplay/blob/main/lib/OTelBunyanAppender.js  I'm happy to discuss them here, in a PR, or on Slack.

@trentm I think this is a good start, it would be easier to provide feedback directly in a PR, I think it would be good to align as much as possible with what Instrumentations are doing, maybe create similar abstract class, use global LoggerProvider by default, expose setLoggerProviderMethod, enabled/disable funcitonality, getLogger receiving LoggerOptions, etc.

Thanks for taking a look.

it would be easier to provide feedback directly in a PR

Understood. One thing I'm confused/struggling with is where this should live. Would this be a package, say at opentelemetry-js-contrib/packages/[opentelemetry-]bunyan-log-appender?   A comment on this weeks SIG call and your mention of Instrumentations makes me wonder if some think this should be an instrumentation -- i.e. if it would fit in with the existing opentelemetry-js-contrib/plugins/node/opentelemetry-instrumentation-bunyan.
My initial thought was the former -- that usage of this would be as follows, so a user could explicitly setup their logger to send to whatever OTel SDK has been setup:
const bunyan = require('bunyan');
const { BunyanAppender } = require('@opentelemetry/bunyan-log-appender'); // or whatever package name

const log = bunyan.createLogger({name: 'myapp'});
log.addStream({
  type: 'raw',
  stream: new BunyanAppender(/* ... */)
})

And then possibly the opentelemetry-instrumentation-bunyan instrumentation would change to support automatically calling .addStream(...) on created Bunyan loggers.
Thoughts?

Yeah I'm more inclined into having a single bunyan package that provides trace injection functionality and auto generation of Log telemetry through the appender in same place, I can see having two different packages causing a lot of confusion to users, @pichlermarc what do you think?

I think it makes sense to include trace context injection in this module, so users can install just one package and get all the benefits of OTel. With that said, I think the existing bunyan instrumentation will need to continue to exist for use cases where users don't want to send the logs to OTLP backends.
As a side note, I think the name of the package/class should be stream, not appender, to be consistent with Bunyan's terminology and other existing streams.

There are 2 concerns with logs:

Enrichment with otel context
Capturing for OTLP export

I think it would be helpful if the logs SIG provided guidance around which of these concerns (or both) instrumentations should be handling. Should it be configurable?

(FYI: I haven't made progress on this because I was on vacation last week and am away this week.)

I have a PR now for this at https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1713
It adds to the existing @opentelemetry/instrumentation-bunyan.


Plotting samples to example.pdf...Error in x[j] : invalid subscript type 'list'

Have an input file with coverage from 30 samples, across 1 gene.
Each row in the bed file covers 100bp, like shown in your example.
Here is my command:
/data2/bsi/staff_analysis/m088378/CNView.R 17 41205689 41277589 PDXsamples.names.txt PDXsamples.chrless.bed example.pdf --probs

I am following the format exactly specified. However I get this error:
+-------------------+
| CNView Visualizer |
|     (c) 2016      |
+-------------------+
Reading sample IDs from PDXsamples.names.txt
Filtering & loading coverage matrix... Complete
Compressing coverage matrix [900 bp bins]...  Complete
Performing intra-sample normalization... Complete
Performing inter-sample normalization... Complete
Plotting samples to example.pdf...Error in x[j] : invalid subscript type 'list'
Calls: CNView -> pt -> [ -> [.data.frame
Execution halted

I'm going to start going through the code step by step, but I thought I'd create an issue to make you aware of a potential situation that the code is not handling.

As I was digging into the code, I did discover a different issue. Semi-unrelated.
Line 206:
##Generate Colors##
      if(pt(plotSet[1,sampIdx[k]],df=(ncol(plotSet)-8)) >= 1-(0.05/(ncol(plotSet)-7))){
        colval <- "blue"
      }else if(pt(plotSet[1,sampIdx[k]],df=(ncol(plotSet)-8)) <= 0.05/(ncol(plotSet)-7)){
        colval <- "red"
      }else{
        colval <- "gray40"
      }

The 1st sample name I provided was "sample3-1", which caused
sampIdx[k] = integer(0)

I'm still working on the original issue, but error handling names with hyphens would be a nice addition. I'll just avoid it for this version.

Thanks for raising these issues. The "invalid subscript type" error usually occurs when either there are no lines in the input file or the sample IDs specified aren't in the header of the input matrix.
Could you post a head of PDXsamples.chrless.bed as well as the contents of PDXsamples.names.txt?
Also, are the bins in your input matrix uniform sized and continuous from the start of your plot window to the end?
Currently, CNView doesn't support nonuniform bin sizes like exome data; I've been playing with some modifications to adapt to these data types and may include them as features in the future.
Thanks for the note about hyphens in sample IDs -- this should be an easy fix, and I can include it in the next future update.
Ryan

I've figure out the culprit of the error...unsure how to fix it.
The variable "sampIdx" is parsing the columns and names, and overlapping the column indexes.
Here's the print out of that variable, with my inputs.
$s_Panc3_1
[1] 4
$s_Pax101_F2AR
[1] 5
$s_Pax102_F2
[1] 6
$s_Pax110_F2AR
[1] 7
$s_Pax113_F2AR
[1] 8
$s_Pax124_F2AL
[1] 9
$s_PAX12
[1]  9 10
$s_Pax79_F3AL
[1] 32
$s_Pax82_F2
[1] 33
$s_Pax89_F3AR
[1] 34
$s_Pax97_F2DR
[1] 35
$s_PAX9
[1] 35 36

Note how samples s_PAX12 & s_PAX9 double up with their neighbors.
Here are the head of my input files.
PDXsamples.chrless.head.txt
PDXsamples.names.head.txt

Great, this makes the error very clear. I have implemented a fix and pushed it to the master; try pulling the updated version and rerunning now.
The issue was the implementation of R grep() to subselect the appropriate sample ID from the column indexes. However, grep() was only searching for partial substring matches instead of exact matches. I wrapped the sample IDs in word boundaries (\b) to get around it.
This should fix the issue, but please let me know if it does not.
Ryan

Great it works! Thank you for the very quick response.


enhance performance by ignoring rules that are never used #12592

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
not likely

Tests pass?
yes

Fixed issues
#12592

Could you add a CHANGELOG line?

I repackaged it into 1 commit. Thx!

As this could break BC for people overriding init() method and it is a critical part for website security I'd prefer to move this PR to 2.1.

agreed

@dynasource can you make the same changes based on the 2.1 branch and send the PR to that branch?

@dynasource Could you update UPGRADE.md and mention this change as a potential BC break?

:+1: