Additional training data for the segmentation model

This should fix #984 #1198 #1187 #1155
The commit is mine, i use my credentials but with the wrong email/name

coverage: 40.755%. remained the same
when pulling 7fbbdaf9cabbe3639f48fc65733c0e19935953a4 on new-training-data-das
into f4d6245d443570614359da08bbd27e14a4623b36 on master.


Major repo refactoring

Major refactoring of legacy code

Closing this PR due to conflicts. Recreated proper PR here - #649


Bot won't start up

So I run update_deps.bat and get the following:

Requirement already up-to-date: discord.py[voice]~=0.12.0 in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from -r requirements.txt (line 1))
Requirement already up-to-date: youtube_dl in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from -r requirements.txt (line 2))
Requirement already up-to-date: pip in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from -r requirements.txt (line 3))
Requirement already up-to-date: cffi==1.6.0 in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from -r requirements.txt (line 4))
Requirement already up-to-date: aiohttp<1.1.0,>=1.0.0 in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from discord.py[voice]~=0.12.0->-r requirements.txt (line 1))
Requirement already up-to-date: websockets<4.0,>=3.1 in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from discord.py[voice]~=0.12.0->-r requirements.txt (line 1))
Requirement already up-to-date: PyNaCl==1.0.1 in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from discord.py[voice]~=0.12.0->-r requirements.txt (line 1))
Requirement already up-to-date: pycparser in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from cffi==1.6.0->-r requirements.txt (line 4))
Requirement already up-to-date: chardet in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from aiohttp<1.1.0,>=1.0.0->discord.py[voice]~=0.12.0->-r requirements.txt (line 1))
Requirement already up-to-date: multidict>=2.0 in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from aiohttp<1.1.0,>=1.0.0->discord.py[voice]~=0.12.0->-r requirements.txt (line 1))
Requirement already up-to-date: async_timeout in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from aiohttp<1.1.0,>=1.0.0->discord.py[voice]~=0.12.0->-r requirements.txt (line 1))
Requirement already up-to-date: six in c:\users\amanda\appdata\local\programs\python\python35\lib\site-packages (from PyNaCl==1.0.1->discord.py[voice]~=0.12.0->-r requirements.txt (line 1))
C:\Users\Amanda\MusicBot>

Then I try to run runbot.bat, and it hangs and never fully loads.  The bot account comes online in discord, but the cmd doesn't fully start up.
I've tried reinstalling both the bot, python, and Git Bash, which has not worked.
It gets this far into loading and then hangs:

Microsoft Windows [Version 10.0.14393]
(c) 2016 Microsoft Corporation. All rights reserved.
C:\Users\Amanda\MusicBot>python run.py
Connected!  Musicbot v1.9.5_2
Bot:<PHONE_NUMBER>13321985/Music#8814
Owner:<PHONE_NUMBER>57559808/Azsunyx#9267
Server List:

zsazouli tavern
Holy Crit
Execute

Bound to text channels:

Holy Crit/music-bot
zsazouli tavern/music-bot
Execute/bot

Not autojoining any voice channels
Options:
Command prefix: #
Default volume: 10%
Skip threshold: 2 votes or 50%
Now Playing @mentions: Enabled
Auto-Summon: Disabled
Auto-Playlist: Enabled
Auto-Pause: Disabled
Delete Messages: Enabled
Delete Invoking: Disabled
Debug Mode: Disabled
Downloaded songs will be deleted

Could you please try the following:

Open lazy_cmd.bat in the MusicBot folder
Run the command pip uninstall -r requirements.txt and everytime you're prompted with the question whether you want to proceed, answer with y
Run the run.bat file as administrator and wait

What do you exactly mean with "the cmd doesn't fully start up."?

siku2, I get the following error when I try to uninstall:

Microsoft Windows [Version 10.0.14393]
(c) 2016 Microsoft Corporation. All rights reserved.
C:\Users\Amanda\MusicBot>pip uninstall -r requirements.txt
Cannot uninstall requirement discord.py, not installed
C:\Users\Amanda\MusicBot>

arunesh90, I mean in the cmd window when I run the run.bat file, the bot doesn't fully start up

Well that's interesting...
Try running update_deps.bat once more and make sure to run it as administrator
Once you've done that open any command window (for example by double clicking the lazy_cmd.bat) and run pip freeze. Once you've done that please pass along the results (screenshot or just copy paste).

Here are the results:

Microsoft Windows [Version 10.0.14393]
(c) 2016 Microsoft Corporation. All rights reserved.
C:\Users\Amanda\MusicBot>pip freeze
aiohttp==1.0.5
async-timeout==1.1.0
cffi==1.6.0
chardet==2.3.0
discord.py==0.12.0
multidict==2.1.4
pycparser==2.17
PyNaCl==1.0.1
six==1.10.0
websockets==3.2
youtube-dl==2017.2.22
C:\Users\Amanda\MusicBot>

Just to clarify... You've actually tried to use the bot before?
As in:

giving him the necessary permissions
joining a voice channel to which he has access to
using !summon to get him to join your channel

According to the log the bot starts up just fine. You didn't define any "autojoin" channels so of course he wouldn't join one randomly...

Yes, the bot was working fine last week, these problems started around sunday

So, I finally got it working again, I reinstalled youtube-dl and discord.py[voice] then I ran update_deps.bat again, and it started right up this time.


[MetricsAdvisor] API design and renaming updates

This PR

Updated createdDataFeed method to take one DataFeed object.
Renamed listValuesOfDimensionWithAnomalies method to listDimensionValuesWithAnomalies.
Renamed model ListValuesOfDimensionWithAnomaliesOptions method to ListDimensionValuesWithAnomaliesOptions.
Renamed properties viewers , admins and their accessors to viewerEmails and adminEmails respectively on
DataFeedOptions model.
Renamed model DataSourceMissingDataPointFillType to DataFeedMissingDataPointFillType.
Renamed properties on MetricEnrichedSeriesData and MetricSeriesData model.
Renamed method setSeverity to setSeverityRangeCondition on MetricAnomalyAlertConditions model.

Since this PR was getting too long, updates coming in follow up PR:

Update models to take required constructor args
More renames

Oh I need to fix the test pipeline first, will merge after https://github.com/Azure/azure-sdk-for-java/pull/16213


ci(frontend): add VITE_BTC_TO_CKBTC_EXCHANGE_ENABLED env var

Motivation
We include the feature flag VITE_BTC_TO_CKBTC_EXCHANGE_ENABLED for the staging environment.

not needed, it is a duplicate, the var already exists there


Error In the Terminal

I just try the example command in https://github.com/khrome/ascii-art/blob/master/docs/Terminal.md, it shows me an error below

perhaps related to localization? what happens when you explicitly use an alphabet with -a?

It works...  What could be the cause?

this tool is interesting and I wanna to fix it 😀

I'm going to guess that the default is somehow changed as part of localization... my hint being the undefined entry using a localized string in the list of available alphabets. Thanks for looking at it... let me know if you discover anything!

Theoretically this alphabet should have no effect, as it's initialized in main, but only used if images are rendered. probably just poorly handled init logic.

the 2.0 architecture changed things around, this should not be the case anymore.


Tiktok protobuf's import problem

After I parsed the apk package of tiktok, I found a problem, its UserStructV2 referenced ChallengeStructV2, the same ChallengeStructV2 also referenced UserStructV2, which led me need to import ChallengeStructV2.proto when I rewrite UserStructV2.proto , the same in ChallengeStructV2.proto I need to import UserStructV2.proto, so when I compile these proto files, protoc will prompt File recursively imports itself: ChallengeStructV2.proto -> UserStructV2.proto -> ChallengeStructV2.proto I know this way of import each other is wrong, but I don't know how to write proto file can compile success,can you give me some advice, thanks.

First of all, please, note that my protobuf conversion is from an old source of TikTok. According to files I have on my computer, it seems to be v7.2.4 (newest files I have is v17.8.4, but not decompiled nor analyzed...). Which means many change could have been made here.
As you can see with my source code, I'm mainly using C#. Could you specify me which programming language you're trying to use?
Kind regards.

First of all, please, note that my protobuf conversion is from an old source of TikTok. According to files I have on my computer, it seems to be v7.2.4 (newest files I have is v17.8.4, but not decompiled nor analyzed...). Which means many change could have been made here.
As you can see with my source code, I'm mainly using C#. Could you specify me which programming language you're trying to use?
Kind regards.

First of all, please, note that my protobuf conversion is from an old source of TikTok. According to files I have on my computer, it seems to be v7.2.4 (newest files I have is v17.8.4, but not decompiled nor analyzed...). Which means many change could have been made here.
As you can see with my source code, I'm mainly using C#. Could you specify me which programming language you're trying to use?
Kind regards.
EDIT: Nevermind, 7.x doesn't have protobuf, and I wrote "15.0.1". So my protobuf files shouldn't be so old, and still usable I guess.

Thank you for reply.
I have solved this problem, but I still can't understand this question.
My tiktok version is v17.8.4, as you say your protobuf can be used after add some members, but I want to know how you generate those protobuf files, by hand or by protoc.
Cause I find your file ChallengeStructV2.cs 's [ProtoMember(5)] and UserStructV2.cs 's [ProtoMember(189)] both refer from the declaration type of each other. I try to rewrite it by offical proto language as this:
// ChallengeStructV2.proto
syntax = "proto3";
import "UserStructV2.proto";

message ChallengeStructV2 {
    string cid = 1;
    UserStructV2 author = 5;
}

// UserStructV2.proto
syntax = "proto3";
import "ChallengeStructV2.proto";

message UserStructV2 {
    string uid = 1;
    ChallengeStructV2 cha_list = 189;
}

when I execute the command line protoc --python_out=XXX UserStructV2.proto CMD will return error:
UserStructV2.proto:4:1: File recursively imports itself: UserStructV2.proto -> ChallengeStructV2.proto -> UserStructV2.proto

I must use java or python language, so I must use protoc. Now my solution is delete the file ChallengeStructV2.proto, and delete the other file's code which associated with ChallengeStructV2, then I can generate java/python code successfully. I want to know how you generate those .cs protobuf files, by hand write or protoc command : )

First of all, please, note that my protobuf conversion is from an old source of TikTok. According to files I have on my computer, it seems to be v7.2.4 (newest files I have is v17.8.4, but not decompiled nor analyzed...). Which means many change could have been made here.
As you can see with my source code, I'm mainly using C#. Could you specify me which programming language you're trying to use?
Kind regards.
EDIT: Nevermind, 7.x doesn't have protobuf, and I wrote "15.0.1". So my protobuf files shouldn't be so old, and still usable I guess.

Thank you for reply.
I have solved this problem, but I still can't understand this question.
My tiktok version is v17.8.4, as you say your protobuf can be used after add some members, but I want to know how you generate those protobuf files, by hand or by protoc.
Cause I find your file ChallengeStructV2.cs 's [ProtoMember(5)] and UserStructV2.cs 's [ProtoMember(189)] both refer from the declaration type of each other. I try to rewrite it by offical proto language as this:
// ChallengeStructV2.proto
syntax = "proto3";
import "UserStructV2.proto";

message ChallengeStructV2 {
    string cid = 1;
    UserStructV2 author = 5;
}

// UserStructV2.proto
syntax = "proto3";
import "ChallengeStructV2.proto";

message UserStructV2 {
    string uid = 1;
    ChallengeStructV2 cha_list = 189;
}

when I execute the command line protoc --python_out=XXX UserStructV2.proto CMD will return error:
UserStructV2.proto:4:1: File recursively imports itself: UserStructV2.proto -> ChallengeStructV2.proto -> UserStructV2.proto

I must use java or python language, so I must use protoc. Now my solution is delete the file ChallengeStructV2.proto, and delete the other file's code which associated with ChallengeStructV2, then I can generate java/python code successfully. I want to know how you generate those .cs protobuf files, by hand write or protoc command : )

I made a converter of java to c# converter that you can find here: https://github.com/ShiiroSan/TikTokForWindows/tree/master/ProtobufConverter. It just read java extracted source code and make C# protobuf file. As I'm using regex, the converter might not work properly with latest update... In the coming days I'm planning on updating Protobuf files to use 17.8.4. But I can't really give you an ETA as I have other things to do before that.
Let's be honest with it, it's purely ghetto way. But it seems to work with my actual source code. Even tho, my source code is more a PoC than nothing else atm...

I made a converter of java to c# converter that you can find here: https://github.com/ShiiroSan/TikTokForWindows/tree/master/ProtobufConverter. It just read java extracted source code and make C# protobuf file. As I'm using regex, the converter might not work properly with latest update... In the coming days I'm planning on updating Protobuf files to use 17.8.4. But I can't really give you an ETA as I have other things to do before that.
Let's be honest with it, it's purely ghetto way. But it seems to work with my actual source code. Even tho, my source code is more a PoC than nothing else atm...

I made a converter of java to c# converter that you can find here: https://github.com/ShiiroSan/TikTokForWindows/tree/master/ProtobufConverter. It just read java extracted source code and make C# protobuf file. As I'm using regex, the converter might not work properly with latest update... In the coming days I'm planning on updating Protobuf files to use 17.8.4. But I can't really give you an ETA as I have other things to do before that.
Let's be honest with it, it's purely ghetto way. But it seems to work with my actual source code. Even tho, my source code is more a PoC than nothing else atm...

I know what should I do, thank you 😊


feat: Add decimal target type

Adding getDecimalTargetTypes and getDecimalTargetTypes and modified related unit test cases.
Fixes internal issue b/234571438

Raised b/239372813


[ru] Localize Service Mesh into Russian

Description
This Issue is to localize Service Mesh into Russian.
Checklist before opening your PR (put x in the checkboxes) — note that you have to add this checklist to your PR and check all the boxes for it to be approved.

[] :warning: This PR uses the dev-ru branch as a source/target.
[] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

ETA: TBD
Useful links:
Как внести свой вклад
Руководство по стилю
Slack channel
Telegram channel

Hi, I'd like to work on this one.


refactor: consistent bytes new make

Pull Request Test Coverage Report for Build 2246
Details

10 of 11 (90.91%)  changed or added relevant lines in 8 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 90.907%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

builtin/bytes.mbt
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

builtin/intrinsics.mbt
1
50.0%

Totals

Change from base Build 2241:
-0.02%

Covered Lines:
3659

Relevant Lines:
4025

💛  - Coveralls

LGTM


Qeustion/Feature request: Is it possible to sufix the commit messages with the branch name?

In my company we always format the commit messages including the branch name.
E.g. for a branch AS-123:
AS-123: commit message
Is it possible to configure this tool to do so?

Currently, there is no way of doing it because the tool adheres to the Conventional Commit specifications. However, I could allow the user to edit the system instructions, so you could ask the system to include the branch name in your commit message.

I can do that, no worries. :smile:

Feature implemented. Run autocommit edit to open the system default message. In case you mess up, if the text file is empty or does not exist, the tool will recreate it.

Awesome, thanks! will try it in a bit


OpenAI API hits NotFoundError: Error code: 404 - {'error': {'code': '404', 'message': 'Resource not found'}}

Hello Azure team,
Thanks for this good repo!
I am facing a small issue during the installation.

I am very close to run it.
Any idea how could it be fixed?

Did you try running connections/create-connections.ipynb? Also make sure your .env file has the correct keys and endpoints

I had similar issue. What fixed for me was changing the api version param. The one used in the code does not work locally because it was not deployed

I had similar issue. What fixed for me was changing the api version param. The one used in the code does not work locally because it was not deployed

Same deal for me, except I needed to change the model that was deployed (the code has gpt-4, but I only have gpt-35-turbo)


change TEST_INIT to use echo, not echo -e

"echo -e" misbehaves for sh, which is the default shell for Make, though it works fine for bash

@liquidaty:
This has already been fixed under https://github.com/liquidaty/zsv/pull/299.
Replaced echo -e with printf. Working fine for all the platforms.
The color codes were not working with FreeBSD under sh.
With printf, they are working fine now.


Signin messaging not working when starting clean

When starting clean, sign in messaging does not work.  After signing in, the games lists still say "you must sign in to see your active games" etc.

Fixed, will be merged at next opportunity.


fix pylint problems already on master

test models please

test models please

please test models

bypassing review rules and merging because gpkit_Push is currently failing on master and this fixes that.

++branch name


Ppx deriving constructs currently have [@@@coverage off]

Ppx deriving constructs are currently disabled via [@@@coverage off] due to a generated non visitable coverage point:
     let _ =
       fun (_ : t) ->
       ___bisect_visit___ 0;
       ()
     ;;

This is a known issue with work in progress to fix.
Current work around involves an include T0 pattern, for example:
  module Mode = struct
    module T0 = struct
      [@@@coverage off]

      type t =
        | Quiet
        | Default
        | Verbose
        | Debug
      [@@deriving compare, equal, enumerate, sexp_of]
    end

    include T0
  end

Linking with https://github.com/ocaml-ppx/ppxlib/issues/473

Work in progress to fix in #5

Fixed by #6


fix: do not execute <PrismicToolbar>'s script during Happy DOM tests

Types of changes

[ ] Chore (a non-breaking change which is related to package maintenance)
[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
This PR patches <PrismicToolbar> to prevent the component's <script> element to execute in Happpy DOM environments. This is useful when using Happy DOM in tests, such as in Vitest.
The included patch will be pruned during code minification in non-test environments.
Checklist:

[ ] My change requires an update to the official documentation.
[x] All TSDoc comments are up-to-date and new ones have been added where necessary.
[x] All new and existing tests are passing.

🐍

Codecov Report

Merging #151 (72aaa1e) into master (add20dd) will decrease coverage by 0.26%.
The diff coverage is 50.00%.

:exclamation: Current head 72aaa1e differs from pull request most recent head 9c18562. Consider uploading reports for the commit 9c18562 to get more accurate results

@@            Coverage Diff             @@
##           master     #151      +/-   ##
==========================================
- Coverage   92.06%   91.79%   -0.27%
==========================================
  Files          18       18
  Lines         315      317       +2
  Branches       74       75       +1
==========================================
+ Hits          290      291       +1
  Misses          5        5
- Partials       20       21       +1

Impacted Files
Coverage Δ

src/PrismicToolbar.tsx
88.88% <50.00%> (-4.87%)
:arrow_down:

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 add20dd...9c18562. Read the comment docs.


Doc/cyclic symmetry

Add an example to demonstrate that pyDPF Composites can be used for results of cyclic symmetry analyses as well.
While playing with the cyclic symmetry model and going through the DPF Core examples, I figured out that the RST streams provider is often used. Therefore, I thought it makes sense to expose it through a method. Let me know if you have objections.

Codecov Report
Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 91.62%. Comparing base (a5d3831) to head (07d3688).

Files with missing lines
Patch %
Lines

...sys/dpf/composites/_composite_model_impl_2023r2.py
50.00%
1 Missing :warning:

src/ansys/dpf/composites/composite_model.py
50.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #517      +/-   ##
==========================================
- Coverage   91.66%   91.62%   -0.05%
==========================================
  Files          50       50
  Lines        3397     3401       +4
==========================================
+ Hits         3114     3116       +2
- Misses        283      285       +2

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


Update doc for building on MacOS

Update doc for building on MacOS

Hi @kilasuit,
I'm interested in contributing to this issue and noticed it is still open. Are there specific parts of the macOS build docs that need updates?
I’m happy to test the current steps, identify any gaps, and propose updates. Any guidance or additional context would be helpful!
Thanks!
Rahul Mansharamani


Allow optional breakpoint arguments to be passed on to nested media query mixins

Hello
In cases where a mixin internally uses one of the new media query mixins, the outer mixin should pass the local $breakpoints variable onwards to the nested mixin, otherwise it's defaulting back to the global $grid-breakpoints.
@mixin make-grid-columns optionally accepts a $breakpoints argument already, but attempting to use this doesn't result in anything sensible. Internally it defers to media-breakpoint-up, relying on the default $grid-breakpoints and ignoring what you've passed in.
This is inconsistent with how @mixin media-breakpoint-only and @mixin media-breakpoint-between behave, where the optional $breakpoints argument is passed on internally.
The change to @mixin make-container-max-widths is also included, so that all cases where the media query mixins are nested are consistent.
Doesn't change how anything compiles by default, just gives the option consistently for those using the sass mixins in their own code.

Good catch! Thanks!


Failed to sign payload data

When we try to call some CsobClient's methods (e.g. echo), it fails when signing payload data.
Traceback:
File ".../site-packages/pycsob/client.py", line 242, in echo
    ('dttm', utils.dttm())
  File ".../site-packages/pycsob/utils.py", line 56, in mk_payload
    payload['signature'] = sign(payload, keyfile)
  File ".../site-packages/pycsob/utils.py", line 27, in sign
    key = RSA.importKey(open(keyfile).read())
  File ".../site-packages/Crypto/PublicKey/RSA.py", line 733, in import_key
    return _import_keyDER(der, passphrase)
  File ".../Crypto/PublicKey/RSA.py", line 676, in _import_keyDER

It's probably related to https://github.com/Legrandin/pycryptodome/issues/188 , because update pycryptodome to 3.6.5 solved this problem.

Fixed in #9


Fix undefined everyoneMentioned

Bug that caused Message.everyoneMentioned to be undefined if it was false.
Credits to @GexoXYZ

Use grunt --dev to process lib files

in theory could not defining everyoneMentioned if everyone is not mentioned (99% of the time) save a considerable amount of memory?

@macdja38 It could depending on how many messages are cached


Please update this mod for update 4

Did not work after update 4

If you dont have time please give sources of mod to community, so mod dont disappear. It very useful and cool mod!

I agree i really want this mod

This is an amazing mod and has helped me build giga-factories in no time. Love every aspect of it.
And we really want to save this mod, after Update 4 it could disappear!
Please update it or post an exclusive repo to the community modders on Discord if you're wary of sharing the code.
Please save it!

Unfortunately, it will probably be the same here as with many other mods: there will be no more updates, especially since the modder has not even commented here yet.

i agree ;(
On Mon, Apr 26, 2021 at 5:59 AM Iwan1803 @.***> wrote:

Unfortunately, it will probably be the same here as with many other mods:
there will be no more updates, especially since the modder has not even
commented here yet.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cepheidqq/FactorySkyline/issues/42#issuecomment-826739185,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATVTWSK3P47V7FMQ62IQOEDTKVBQLANCNFSM424CT23A
.

Let's all comments ! Maybe something can happen x)...


Allow setting default_device with platform names.

Thank you both for reviewing my first PR :).  I've made the requested changes and updated the PR. Please take another look.


Error when trying to use device discovery (O3R camera)

Hi. A couple of things I want to raise/ask.
1) I am trying to using the DeviceDiscovery function from the camera base class to find the O3R cameras connected to my VPU. I am getting the following exception when trying to do so. When I check the console it has the line, "An invalid argument was supplied" printed. I have tried running it directly using the ifm3d command line tool using "ifm3d discover" command but I get the same "An invalid argument was supplied"  error message:

I am able to run ifm3d dump and here is the result for that:
ifm3d dump.txt
2) I would also like to confirm whether the following (highlighted in red squares) are LEDs that are supposed to turn ON when a camera is connected in the corresponding port. In my case, they don't turn on when I  have a camera connected:

Hope I can get some help on these issues.

Hey @mehranq , the LEDs are supported in the next firmware release series 0.14.x which we currently do internal testing with.

@mehranq on discovery:  I have tried to reproduce the issue but was not able to reproduce it on windows nor on Linux. can you provide more information on which windows and Visual studio version you are using and would also want to know the build type of ifm3d (Release/Debug)
ifm3d discover uses udp communication for discovery of devices in subnet. Are udp packets allowed on your PC and not blocked by firewall on network. can you try creating a outbound and inbound rule on firewall for udp port 3321 ( this port is used by ifm3d for discovery).

Hey @mehranq , the LEDs are supported in the next firmware release series 0.14.x which we currently do internal testing with.

@graugans Thank you for the info.
@inbangsa Here is the info you requested:
Windows version:

Visual Stuido version:

I checked the firewall settings and the inbound rules were already setup like so:

I added the outbound rule as per your suggestion like so:

I restarted my pc and re-ran the ifm3d discover command. However, I still get the same error message  "An invalid argument was supplied"
FYI, I built the library from source on windows in RELEASE config as instructed on your github page for building on windows. I built the library about a month ago. Could it be the case that this issue existed previously but does not occur in the latest version of the library?

@mehranq there is no change in this feature from last couple of months.
we are not able to reproduce this issue, and all tests are working fine on windows and linux machine. we are in developement of windows installer for ifm3d which will provide binary version and we would like you to test this feature with that.


Making MergeReducedEvents optional

In cases where merging is not required (e.g. if the files are so large that it would be 1-to-1 merging, of if it's simply not desired), it should be possible to skip it.
This is mean that subsequent tasks (UniteColumns, ProduceColumns, WriteHistograms, PrepareMLEvents, MLEvaluation) should have the option to directly retrieve their inputs from ReduceEvents.
This could be solved by having in intermediate wrapper-like task that provides the input with a clean interface, so that switches between different requires()/input() structures only exist once.
Also see https://github.com/columnflow/columnflow/discussions/383.

Closed by #388


Add loggingEnabled option to navigation container

This fixes https://github.com/react-community/react-navigation/issues/851.
Test plan
See that no information is logged to console unless option loggingEnabled is set to true in a navigator.

@satya164 I disagree.
Perhaps the prop isn't the most efficient solution, but when debugging, I find the logs extremely useful. It's only during production that we have a performance problem that we need to address. Therefore, I believe the flag should be present.

When you are debugging it's not a lot of work to put logs where you want them, or put a breakpoint. You are going to do that for other parts of code anyways. Also when you are using Redux, you can use the logger middleware. I don't see the point of the prop.

@skevy, would you mind weighing in on the value of those console statements? Seems like they were introduced by https://github.com/react-community/react-navigation/commit/c37ad8a0a924d13f3897bc72fbda52aac76904b6. Do you think they're useful in general development of the library?
If they are indeed we could perhaps debate on other possible ways to guard them.
It would be good to provide users of the library with an easy option to opt-in to more logging information.

So what's wrong with putting console logs yourself or using redux logger?

@satya164 Nothing wrong, it's just inconvenient. Also we can't expect people to use Redux.
In my humble opinion, until react-navigation becomes the official navigation solution for React Native we should do what we can to encourage people to easily debug the library and improve it. Also, I believe there is a reason why those statement were landed and not removed for 2 months.

@satya164, I'm fine with either really. If @skevy doesn't reply or some other contributor doesn't oppose by mid-week next week, I'll go ahead and apply your suggestion to remove the logs altogether.

it's just inconvenient

How is using <MyNavigator onNavigationStateChange={(...args) => console.log(args)} /> or using redux-logger (which you probably use if you use redux) more inconvenient than <MyNavigator loggingEnabled />?

we should do what we can to encourage people to easily debug the library and improve it.

Sure, but how is adding logging encourages anyone to improve the library? And people are looking into the code to add a simple console log where they need it? There are so many other places which you also want to debug, why have logs only here and not everywhere?

I believe there is a reason why those statement were landed and not removed for 2 months.

Or the reason could be just that nobody bothered to remove them.

I think the logging should be the default behavior of the navigation state event. If you provide an empty handler, the logging will stop.
The logging was put there intentionally, and I think it helps for the average user to understand what is happening with the navigation state.

@ericvicenti can you give a code example of what needs to be done to disable the default logging?

Instead of loggingEnabled, can you add an onNavigationState option to the container options?
This event will fire any time the navigation state changes, or when it is initially known.
The default onNavigationState handler is the logger. This way, logging can be disabled by setting an empty function as the onNavigationState option.

@ericvicenti

The logging was put there intentionally, and I think it helps for the average user to understand what is happening with the navigation state.

I think the same can be said for whole application state. I don't think navigation state is any different. As a user, I've used redux-logger or redux devtools in every project because of this reason.

Instead of loggingEnabled, can you add an onNavigationState option to the container options?

Isn't it same as the onNavigationStateChange prop already exists? That's what I've suggested above to use for logging. Also Redux users have redux-logger.

The default onNavigationState handler is the logger. This way, logging can be disabled by setting an empty function as the onNavigationState option.

I don't think we should enable logging by default. It affects performance negatively https://facebook.github.io/react-native/docs/performance.html#console-log-statements

I think the same can be said for whole application state. I don't think navigation state is any different. As a user, I've used redux-logger or redux devtools in every project because of this reason.

Yep, but if you're using redux for your navigation state, then the container does not log.
As for performance, maybe we should disable it when __DEV__===false
The reason we can't use onNavigationStateChange is because this is a container feature so it needs to be in containerOptions

It looks like this PR solves a real and immediate problem since we are stuck with these expensive console.logs as it stands now.  Long term maybe it makes sense to have the user listen/react to changes if they want to console.log on their own, but for now I think step 1 is to let the user prevent these console.logs.

I am going to close this PR in favour of changes suggested by @ericvicenti and @satya164

Mike, are there any PRs with those suggested changes?
On Tue, Apr 18, 2017, 12:04 Mike Grabowski<EMAIL_ADDRESS>wrote:

Closed #861 https://github.com/react-community/react-navigation/pull/861
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/react-community/react-navigation/pull/861#event-1046347955,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACmsIl-4VYhWGH_XE6X3csLUilkjUwfGks5rxIqsgaJpZM4Mtk3A
.

Yes, please see
https://github.com/react-community/react-navigation/pull/1091 which is
likely going to land later today.
On Tue, 18 Apr 2017 at 15:12 Marcin Hagmajer<EMAIL_ADDRESS>wrote:

Mike, are there any PRs with those suggested changes?
On Tue, Apr 18, 2017, 12:04 Mike Grabowski<EMAIL_ADDRESS>wrote:

Closed #861 <
https://github.com/react-community/react-navigation/pull/861>
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/react-community/react-navigation/pull/861#event-1046347955
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ACmsIl-4VYhWGH_XE6X3csLUilkjUwfGks5rxIqsgaJpZM4Mtk3A
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/react-community/react-navigation/pull/861#issuecomment-294837637,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWcxgpmsd4Vw9tOBIXhoBOG34emQIDfks5rxLbBgaJpZM4Mtk3A
.


jqLite: .val() on  with no options chosen should return an empty array instead of null

Do you want to request a feature or report a bug?
A behavior change, though not a bug, strictly speaking.
What is the current behavior?
For the jqLite element representing a select element in the multiple variant with no options chosen the .val() getter returns null.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (template: http://plnkr.co/edit/tpl:yBpEi4).
See http://plnkr.co/edit/QCWwa2AHNAhLKd2iyZPw
What is the expected behavior?
angular.element('<select multiple>').val() should return [].
What is the motivation / use case for changing the behavior?
The proposed change is in line with what jQuery 3 will be doing. Returning an empty array instead of null is more natural as the data type doesn't change.
Which versions of Angular, and which browser / OS are affected by this issue? Did this work in previous versions of Angular? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
All versions.
Other information (e.g. stacktraces, related issues, suggestions how to fix)

The jQuery issue about this change: https://github.com/jquery/jquery/issues/2562.


Update WebRTC for SCTP reliability

This enables us to pass the entire go-libp2p-transport/test suite.
Relates to #10
cc pions/sctp#21

Decided to merge for the reliability improvements already realized. Keeping #10 open until the changes land in pions/webrtc master (or v2.0.0).


Publish render from SP

 Freshdesk Ticket ID: 259
 Freshdesk Ticket Agent: Jakub Jezek
 Freshdesk Ticket Agent Email<EMAIL_ADDRESS> Ticket Priority: High
 Freshdesk Ticket Description: Until TV Paint integration is finished, we need to be able to publish renders from standalone publisher. Currently there is only option for plates and image files, but render is missing.

Agent Jakub Jezek linked Freshdesk ticket 259 for this issue.


Unable to query extensions after updating to 1.19.3

VSCode Version: 1.19.3
OS Version: windows 10

Steps to Reproduce:
1.Update to 1.19.3
2.Click on show popular extensions
I am not behind a proxy.

Furthermore: I noticed i cant even go to the marketplace online via this url https://marketplace.visualstudio.com/

Seemed to have resolved itself. I just wanted and restarted my computer


WIP

Fixes #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for feature-requests.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #10156 (dd2409b) into main (51fad38) will increase coverage by 0%.
The diff coverage is 44%.

:exclamation: Current head dd2409b differs from pull request most recent head 5049869. Consider uploading reports for the commit 5049869 to get more accurate results

@@          Coverage Diff          @@
##           main   #10156   +/-   ##
=====================================
  Coverage    60%      61%
=====================================
  Files       203      204    +1
  Lines      9239     9284   +45
  Branches   1490     1501   +11
=====================================
+ Hits       5634     5667   +33
- Misses     3128     3138   +10
- Partials    477      479    +2

Impacted Files
Coverage Δ

src/platform/common/refBool.ts
100% <ø> (ø)

src/platform/api/kernelConnectionWrapper.ts
43% <43%> (ø)

src/platform/api/kernelApi.ts
72% <100%> (+1%)
:arrow_up:

src/platform/common/utils/multiStepInput.ts
74% <0%> (-1%)
:arrow_down:

src/platform/export/fileConverter.node.ts
100% <0%> (ø)

src/platform/common/constants.node.ts

src/platform/common/scriptConstants.ts
85% <0%> (ø)

src/platform/common/utils/localize.ts
98% <0%> (+<1%)
:arrow_up:

...form/common/process/pythonExecutionFactory.node.ts
75% <0%> (+1%)
:arrow_up:

... and 2 more


Enhance JWT Token Management

Fixed Issue #99
Changes Made to the codebase
1. Added Middleware for Environment Variable Validation
Introduced a checkKey middleware function to validate the presence of ACCESS_TOKEN_SECRET in environment variables
and assign it to req.Key for use in subsequent functions.
2. Refactored generateAccessToken Function
Improved error handling with specific messages for token generation failure and server errors.
Returned the token as a JSON object with a clear key.
** 3. Improved decodeAccessToken Function **
Streamlined error checking for the authorization header.
Removed redundant checks and ensured the function handles decoding the token appropriately.
Returned error messages with appropriate status codes for missing authorization headers, token decoding failures, and invalid or expired tokens.
** 4. Consistent Error Responses **
Ensured that all error responses are formatted consistently with proper status codes and messages.

@Harshdev098 please review this pr and add necessary tags(hacktoberfest and gssoc)

OR you can update the functions in the files

@Harshdev098 i think refactoring the code will cause too much issue in the future , let me try to simplify and change the code according to its needs

Yah, right

@Harshdev098 check if this is good enough or not and let me know


Compile error when starting server

Hi!
After installing the module and configuring it, I tried to start my server again, but this fails due to a compile error:
[22.02.2022 09:02.35.807] [ERROR] WARNING! Could not validate config file. Starting with default configuration. Please correct syntax errors at or above this line: /home/pi/MagicMirror/modules/MMM-OnThisDay/EventService.js:18
_html = '';
^
SyntaxError: Unexpected token =
at Module._compile (internal/modules/cjs/loader.js:760:23)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:827:10)
at Module.load (internal/modules/cjs/loader.js:685:32)
at Function.Module._load (internal/modules/cjs/loader.js:620:12)
at Module.require (internal/modules/cjs/loader.js:723:19)
at require (internal/modules/cjs/helpers.js:14:16)
at Object. (/home/pi/MagicMirror/modules/MMM-OnThisDay/node_helper.js:10:22)
at Module._compile (internal/modules/cjs/loader.js:816:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:827:10)
at Module.load (internal/modules/cjs/loader.js:685:32)
(Note: the up-arrow in the error message points at the equal-sign)
Any idea what's wrong here?

Don't know what I did, but when I tried again today, it works (probably some update of MagicMirror solved it).
So closing this issue

Glad to hear it works now :)
11.11.2022 10:49:13 Christian Presslmayr @.***>:

Don't know what I did, but when I tried again today, it works (probably some update of MagicMirror solved it).
So closing this issue
—
Reply to this email directly, view it on GitHub[https://github.com/nkl-kst/MMM-OnThisDay/issues/23#issuecomment-1311471102], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AFY47RRONVZESLGPF2XRMITWHYJBPANCNFSM5PAT67GQ].
You are receiving this because you are subscribed to this thread.[Verfolgungsbild][https://github.com/notifications/beacon/AFY47RWXTKGI63HEWHYE4N3WHYJBPA5CNFSM5PAT67G2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOJYVXL7Q.gif]


Describe HTTP signed data gateway signing and signature recovery processes

A user on Discord noted that we do not explain how the HTTP signed data gateway signs the data nor how to verify the signature on-chain.
@Ashar2shahid provided the following links to the Airnode repo:

https://github.com/api3dao/airnode/blob/32afa71f0f20dc2297d66c223d57c01a00c1acd4/packages/airnode-node/src/api/index.ts#LL261C8-L261C8
https://github.com/api3dao/airnode/blob/32afa71f0f20dc2297d66c223d57c01a00c1acd4/packages/airnode-node/src/api/index.ts#L92
https://github.com/api3dao/airnode/blob/32afa71f0f20dc2297d66c223d57c01a00c1acd4/packages/airnode-protocol/contracts/rrp/AirnodeRrpV0.sol#L245-L251

@dcroote Shall we close this one?


Create ephemeral environments (deploy, do something, then destroy)

def validate = new TerraformValidateStage()
def deployQa = new TerraformEnvironmentStage('qa')
def destroyQa = new TerraformDestroyEnvironmentStage('qa')

validate.then(deployQa)
        .then(destroyQa)
        .build()

deployQa would deploy an environment.
I'd have the ability to interact with the environment
destroyQa would tear down the environment

destroyQa would behave much like deployQa, possibly doing a terraform plan -destroy, and then automatically prompting the user with something along the lines of "You're about to terminate the QA environment - are you sure you want to do this?", and the user has to explicitly click "yes" in order to proceed forward.
TerraformEnvironmentStage also behaves differently in master vs a branch. For one-to-one behavior, I think I would make TerraformDestroy behave the same way too. If you're on a branch, then TerraformDestroyEnvironmentStage might run a terraform plan -destroy, but not provide the user the ability to actually destroy the environment without merging into master.

This UseCase was originally described in issue #88, but was subsequently broken out, as it's more difficult to implement.

Ephemeral environments probably make more sense on a branch/PR, but there's currently no good way to distinguish behavior between branch/PR and master.  What if we did the following?
<EMAIL_ADDRESS>_
Jenkinsfile.init(this)
def pipeline = new Pipeline
def validate = new TerraformValidateStage()
def deployQa = new TerraformEnvironmentStage('qa')
def deployUat = new TerraformEnvironmentStage('uat')
def deployProd = new TerraformEnvironmentStage('prod')
pipeline.startsWith(validate)
.then(deployQa)
.then(deployUat)
.then(deployProd)
def deployEphemeral = new TerraformEnvironmentStage()
def destroyEphemeral = new TerraformDestroyEnvironmentStage()
pipeline.onBranchOrPr(validate)
.then(deployEphemeral)
.then(destroyEphemeral)
pipeline.build()

Unaddressed problem:

Most of our apps are built around the idea of separate code/configuration.
Most configuration comes from some external store (like ParameterStore)
We'd need some sensible/easy way to pass configuration for the ephemeral environment.

To summarize the use case, I want each developer who is developing terraform to be able to develop their own terraform in isolation while testing in isolation to ensure that when something is merged to master it has been through a build/testing process. With the potential of more than one developer working on terraform at the same time, they each should be able to make changes and consume underlying modules without reliance on someone else to not only merge into master while also taking down an existing environment to do their testing. They should be able to have their own sandbox to test in until they changes are successful and can be merged.
The use case I am trying to complete has three broad assumptions

The master branch will build all proper environments. By proper I mean environments like qa1, uat, production.
Every other branch or PR should have it's own unique environment.
Unique environments can share reference components in a qa1 environment; however, when terraform destroy is run must not delete those resources. Those resources must only be deleted when run from the master branch.

When digging deeper into the idea behind non-proper environments, there are a couple of assumptions that I wanted to try to complete for that use case

There should be a shared environment that these environments use. This environment will have a default set of terraform variables so that developers didn't have to create their own variables for their specific terraform run.
A route 53 record would need to be created for each environment so that developers can access their unique resources. The example for this could be something like cardNumber-testEnviornment.mywebsite.com
When submitting a PR to master, the PR should be blocked from being approved unless the terraform destroy command is run to clean up the test environment.
Each environment should create it's own state file so that each branch/each PR will effectively start with it's own terraform statefile.

Ephemeral environments probably make more sense on a branch/PR, but there's currently no good way to distinguish behavior between branch/PR and master. What if we did the following?
@Library(['terraform-pipeline']) _

Jenkinsfile.init(this)

def pipeline = new Pipeline()
def validate = new TerraformValidateStage()
def deployQa = new TerraformEnvironmentStage('qa')
def deployUat = new TerraformEnvironmentStage('uat')
def deployProd = new TerraformEnvironmentStage('prod')

def deployEphemeral = new TerraformEnvironmentStage()
def destroyEphemeral = new TerraformDestroyEnvironmentStage()

pipeline.startsWith(validate)
        .then(deployQa)
        .then(deployUat)
        .then(deployProd)

pipeline.onBranchOrPr(validate)
        .then(deployEphemeral)
        .then(destroyEphemeral)

pipeline.build()

It's possible to expose the underlying github variables? When thinking about a declarative pipeline, this would be an example where env.Branch_Name is the name of the github branch that is being executed.
checkout poll: false, scm: [$class: 'GitSCM', branches: [[name: "${env.BRANCH_NAME}"]], doGenerateSubmoduleConfigurations: false, extensions: [[$class: 'CloneOption', noTags: false, reference: '', shallow: true]], submoduleCfg: [], userRemoteConfigs: [[credentialsId: "someCredential", url: "someRepo"]]]
One thought I had is what about using something like this?
`@Library(['terraform-pipeline']) _
Jenkinsfile.init(this)
def pipeline = new Pipeline()
def validate = new TerraformValidateStage()
if (env.Branch = master) {
def deployQa = new TerraformEnvironmentStage('qa')
def deployUat = new TerraformEnvironmentStage('uat')
def deployProd = new TerraformEnvironmentStage('prod')
pipeline.startsWith(validate)
.then(deployQa)
.then(deployUat)
.then(deployProd)
}
else {
def deployTest = new TerraformEnvironment('test)'
pipeline.startsWith(validate)
.then(deployTest)
}
pipeline.build()`

It's possible to expose the underlying github variables?

I'm not quite sure what you're asking?
I think your example is the intent of the code below?
@Library(['terraform-pipeline']) _

Jenkinsfile.init(this)

def pipeline = new Pipeline()
def validate = new TerraformValidateStage()
def deployQa = new TerraformEnvironmentStage('qa')
def deployUat = new TerraformEnvironmentStage('uat')
def deployProd = new TerraformEnvironmentStage('prod')

def deployEphemeral = new TerraformEnvironmentStage()
def destroyEphemeral = new TerraformDestroyEnvironmentStage()

pipeline.startsWith(validate)
        .then(deployQa)
        .then(deployUat)
        .then(deployProd)

pipeline.onBranchOrPr(validate)
        .then(deployEphemeral)
        .then(destroyEphemeral)

pipeline.build()

The way I imagine the code above working:

If branch is master, then run validate, deployQa, deployUat, deployProd
If the branch is anything other than master, or if the branch is a PR, then run validate, then the "ephemeral" environment.  <-- if there were multiple PR's, then each PR would get its own unique environment name.

I just realized that I left out a critical piece of information.  When I refer to "ephemeral environment", and def deployEphemeral = new TerraformEnvironmentStage() <-- I meant that it would automatically pull the branch name or PR name from git, and use that as it's environment name.  This would allow each ephemeral environment to be unique identified.

Ephemeral environments probably make more sense on a branch/PR, but there's currently no good way to distinguish behavior between branch/PR and master. What if we did the following?
@Library(['terraform-pipeline']) _

Jenkinsfile.init(this)

def pipeline = new Pipeline()
def validate = new TerraformValidateStage()
def deployQa = new TerraformEnvironmentStage('qa')
def deployUat = new TerraformEnvironmentStage('uat')
def deployProd = new TerraformEnvironmentStage('prod')

def deployEphemeral = new TerraformEnvironmentStage()
def destroyEphemeral = new TerraformDestroyEnvironmentStage()

pipeline.startsWith(validate)
        .then(deployQa)
        .then(deployUat)
        .then(deployProd)

pipeline.onBranchOrPr(validate)
        .then(deployEphemeral)
        .then(destroyEphemeral)

pipeline.build()

What do you mean by there's currently no good way to distinguish behavior between branch/PR and master?
A couple things I did want to point out for discussion, while working on this I found that i'm not able to use env.Branch_Name directly. In order to use it I needed to use Jenkinsfile.instance.getEnv().BRANCH_NAME} which isn't a big deal but worth mentioning.


Complete implementation of the OpenStack block api interface in ciao-controller

Modify ciao-controller to implement the interface for the openstack ShowVolumeDetail and ListVolumeDetails endpoints.

Coverage increased (+0.02%) to 66.44% when pulling 89fa222ed33ae03b16324b5bb62d92828434cb88 on kaccardi:topic/storage-fork into 2286c30d3c6afb6b97a02328fa44617c51806fbe on 01org:master.

Coverage decreased (-0.05%) to 66.366% when pulling f226c14487885c7d783860f9871bce36de0792ff on kaccardi:topic/storage-fork into 2286c30d3c6afb6b97a02328fa44617c51806fbe on 01org:master.

Coverage increased (+0.07%) to 66.491% when pulling db6e91cdd19725c5f20b6c73f81d6e3e1de4eb99 on kaccardi:topic/storage-fork into 2286c30d3c6afb6b97a02328fa44617c51806fbe on 01org:master.

Coverage increased (+0.07%) to 66.491% when pulling db6e91cdd19725c5f20b6c73f81d6e3e1de4eb99 on kaccardi:topic/storage-fork into 2286c30d3c6afb6b97a02328fa44617c51806fbe on 01org:master.

Other than the couple questions on consistency and the hypothetical race (which looks to be "ok" in practice), looks good to me.

Coverage increased (+0.02%) to 66.445% when pulling a141d49a904bc67898badbb2c96cd3886f309871 on kaccardi:topic/storage-fork into 2286c30d3c6afb6b97a02328fa44617c51806fbe on 01org:master.


Trigger the FhirTaskPlanWorker every time tasks are generated

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #1909
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

lgtm! will merge on passing

Resolve merge conflicts and failing tests @owais-vd  so we can merge this.


[ADD]: Event Countdown Timer

The name of the WEB 3 project you want to add:
Event Countdown Timer
Describe the WEB 3project you want to add:
The Event Countdown Timer is a web application designed to help users keep track of upcoming events by providing a visual countdown. Users can input the name and date of an event, and the timer will display the time remaining until the event occurs.
Features:
Event Input: Users can enter the event name and select the date and time.
Real-Time Countdown: The timer updates every second to show the remaining days, hours, minutes, and seconds until the event starts.
User-Friendly Interface: A simple and intuitive design makes it easy for anyone to use.
Alerts: Users receive a notification when the countdown reaches zero, indicating that the event has started.
Tech Stack of the project:
Technologies Used:
HTML: For the structure of the webpage.
CSS: For styling and layout.
JavaScript: For countdown functionality and dynamic updates.
Screenshots /Videos(Mandatory)

Additional context(Mandatory)

[✅] I have read the Contributing Guidelines *
[ ] I'm a WOB'24(Winter Of Blockchain) contributor
[✅] I'm a GSSOC'24 Extended contributor
[ ] I'm a Hacktoberfest'24 Extended contributor
[✅] I have starred the repository *

@adwityac basically as I have already mentioned in the contributing.md file also that this repo is just accepting the WEB3 projects only(maybe based on blockchain , selenium , polygon ,eth , solana) but this project is based on web2 or web1 ...
But sorry to close this issue as this will not be a valid contribution for our repo
But u can still make new issues on which u want to work on

okay got it


Issue #180

closes #180
This PR depends on #204 and the multi_table branch of SpatialData and addresses all the tasks in #180 as far as they were still applicable. One difference is that it both removes spot_diameter and point_diameter as they didn't seem to have an effect. Ultimately spot_diameter was used in case someone clicked on an item of AListWidget which for some reason would add a new napari layer if the ndim of the vector was equal to 2. This has been removed.

Looks great to me thanks! 🔥


Add copying of application code back into deploy dockerfile

Is it that this was inadvertently / incorrectly removed in PR #957?
(It'd be great to have that context — and what went wrong, if anything — in the commit message or as a comment here.)
Also they were previously in alphabetical order, so it'd help diffing a tiny bit if this put them back identically.

Thanks @jmarshall good points, I've added a reference to the PR in the commit, and fixed the ordering to match what it was previously for better diff/blame.


can't add server in Android app

Summary
The android app can't connect to a Mattermost server with a .audio domain name. Works on the web, the Linux desktop app and on iOS.
Environment Information

Device Name: Nokia 6.2
OS Version: 11
Mattermost App Version: 2.3.0
Mattermost Server Version: 7.10.0

Steps to reproduce
try add a server that has a .audio domain . (I suspect that's the culprit of the issue)
Expected behavior
I can't login on the device. It works on the web, the Linux desktop app and on iOS
Observed behavior (that appears unintentional)

Are you sure that the Domain is the problem? We have such Problems too with .de Domain. For me it looks more like a general Problem with mattermost and the App. See the last zwo issues posted here.

Have you check that the SSL cert and its chain is fully valid? You can try ssllabs.com to validate

OK, will investigate. If that's the culprit, it means that the desktop client and the iOS app don't bother about a fully valid SSL cert?

even with Grade A SSL security it's not working.

If you can, please share you server url with me over DM in our community server.. my username there is @elias so I can investigate the cause

It's fixed. Admin fixed it with a docker-update, something related to IPv6.

This is really very strange. I was able to solve the problem for us, it was also a problem with IPv6 for us. With us, the server generally no longer responded to IPv6. This only happened shortly after the Mattermost update.

Same problem here, since last week. Our server is running in a lxc container on Proxmox. I disabled IPv6 on both just to be sure, but nothing work son the mobile Android app. Has someone figured out what was the source issue ?

A restart of the phone possibly helped resolve the issue.

Could someone be a little more precise what had to be fixed with IPv6 to have it running again?

As I wrote before, IPv6 just didn't work correctly overall. A reboot of the VM finally solved the problem.
So make sure that the server actually responds to IPv6 and delivers content. Many of the online tools only check whether an AAAA record exists and only output the IPv6 address or ping it, but do not return any concrete results, e.g. B. the web server actually delivers the page via IPv6.


Increment version for core releases

Increment package version after release of azure-core

/check-enforcer override


Removal of architectures other than amd64

The latest automatic updates have removed all architectures except amd64, see https://github.com/adoptium/containers/commit/1c2bc74272de764359d9840d73a6caa2d423b438
I guess it's not intentional.

It's just the way the GitHub bot updates the files. We create a new one when a new release is available and then automatically add each architecture back when it's been made available. Do you rely in some way on the dockerfiles themselves?

Thanks for the feedback. Not directly, it's just that I couldn't find the 18.0.1 image on docker hub.


Dynamic plugin not showing up in VizTypeControl

I am trying to load a plugin via url.
Expected results
Expect plugin to appear in the list of viz types.
Actual results
Plugin is not showing up in viz type list. I can see that a request is made to https://mysupersethost/dynamic-plugins/api/read and response contains:
{
  "count": 1,
  "label_columns": {
    "bundle_url": "Bundle URL",
    "changed_by": "Changed By",
    "changed_by_fk": "Changed By Fk",
    "changed_on": "Changed On",
    "created_by": "Created By",
    "created_by_fk": "Created By Fk",
    "created_on": "Created On",
    "id": "Id",
    "key": "Key",
    "name": "Name"
  },
  "list_columns": [
    "name",
    "key",
    "bundle_url",
    "id"
  ],
  "modelview_name": "DynamicPluginsView",
  "order_columns": [
    "name",
    "key",
    "bundle_url",
    "id"
  ],
  "page": null,
  "page_size": null,
  "pks": [
    15
  ],
  "result": [
    {
      "bundle_url": "url_from_which_plugin_is_served",
      "id": 15,
      "key": "superset-indicator-chart",
      "name": "Indicator Chart"
    }
  ]
}

However, the plugin chart does not appear in the modal. I believe this could be a recent regression because I remember being able to use this feature.
How to reproduce the bug

Write your own plugin using https://github.com/apache-superset/dynamic-import-demo-plugin as template
Run npm install and npm run build, then npm run serve to serve dist dir
Expose port via HTTP server
Enable feature flag on Superset host, run superset init
Go to 'Settings -> Plugins' and create a plugin entry containing plugin url
Go to 'Chart -> Create New' and verify that your plugin does not appear in the list of viz types

Environment
(please complete the following information):

superset version: latest master as of now, latest commit 57035c1b93331b925e9bf91f757f365807ca70ab
python version: 3.8.7
node.js version: v14.16.0

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
I have verified that the plugin is working when imported directly in source and linked as here: https://superset.apache.org/docs/installation/building-custom-viz-plugins.
I also debugged and stepped over the piece of code importing the plugin (https://github.com/apache/superset/blob/9773aba522e957ed9423045ca153219638a85d2f/superset-frontend/src/components/DynamicPlugins/index.tsx#L150) and made sure there is no error upon importing.
I have also made sure that the name of the plugin in the package.json is the same as the one in Superset.

I have debugged this further and the piece of code selecting charts that will be displayed in the modal:
https://github.com/apache/superset/blob/e466066229ec8262583bc118af1f39e2dbf77bcf/superset-frontend/src/explore/components/controls/VizTypeControl/index.jsx#L167-L190
is executed before the
 MyPlugin().configure({ key: 'my_plugin' }).register()
So the plugin gets registered into ChartMetadataRegistry after the registry is used to list viz types in VizTypeControl.
I would really appreciate if someone could take a look into this.

Thank you so much for your debugging efforts @krsnik93! I think I understand the problem before even needing to repro 😁  I am actively working on this file so I should be able to address the problem quite soon.

Hi @suddjian I tried testing the changes and the plugin is still not showing up in the modal. I think the listener in:
https://github.com/apache/superset/blob/b489cffb570c4d078ec982fd11c3e46dd0310f24/superset-frontend/src/components/DynamicPlugins/index.tsx#L215-L220
might not be working as the action is not dispatched after the plugin is registered into the metadataRegistry.

Just to make sure, did you run npm install?

yup...

@suddjian hello, any updates on this issue?

@suddjian I have the same issue on latest superset version - dynamic plugins are successfully loaded (without any errors in log), its but not listed.


Fix for variable name

Fix image resolution for ghcr

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist:

[ ] squashed commits into logical changes
[ ] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests

Merging as on a team call


MC19 connector: Scale intervention levels to match the model's historical data

The model obtains intervention/distancing levels from Google mobility data.
This may vary slightly from the intervention levels provided by the unified UI.
To maintain the model's fit, scale the distancing levels as follows before running the model:

compute the ratios between the two distancing levels for all time points up to the present day
fit a function to these ratios by interpolation
use that function to scale distancing levels provided after the present day.
The last intervention period is scaled by a constant factor since it is assumed to be constant.

This allows us to create runs that more closely match MC19's own Current Indefinite scenario.

Updated to:

use latest model version
fix scaled distancing levels to remain within [0..1]
improve comments and structure of pure functions after paired review with @aeisenberg
fix a bunch of warnings that shouldn't stop execution

Got :+1: on the approach from the MC19 team, so will merge now to test this out.


Improve ROI creation

Fixes #29
Mostly refactoring of long methods and cleaning up. Also fix edge cases for the top and right edge when making rectangle ROIs. Still some weirdness when zooming in a lot with the right and top edges but I think that's a ginga problem and I should figure out a simple way to expose the problem in a ginga issue. Nonetheless, I don't see it as being problematic for the average user so I'm not concerned.

@godber I can't see any other way to improve it besides refactoring and cleaning contains_arr but I don't understand the algorithm enough to do so. I plan on submitting a PR to ginga to clean up their version of the method and ask for help understanding how it works. If you can merge, go ahead if you think it is ready. Otherwise let me know it's good to go and I'll merge

Coverage decreased (-0.2%) to 94.744% when pulling 58fe55565b423431d5d69176538955693ce1ca20 on pbvarga1:issue29_improve_roi into d3df580b2997d50db4fb27a28d18f4b1d7c7daa1 on planetarypy:master.

Coverage increased (+0.4%) to 95.399% when pulling 1f5fa8d8dedf95ba5a198c1be61ff32350d0f325 on pbvarga1:issue29_improve_roi into d3df580b2997d50db4fb27a28d18f4b1d7c7daa1 on planetarypy:master.

Coverage increased (+0.3%) to 95.309% when pulling 1f5fa8d8dedf95ba5a198c1be61ff32350d0f325 on pbvarga1:issue29_improve_roi into d3df580b2997d50db4fb27a28d18f4b1d7c7daa1 on planetarypy:master.


Implement counts

Implements counts in two places:

As a specifier for nodes, as in o3t inserting 3 trues as children of the cursor
Before a command, as in 3u to jump back 3 undo levels

@Kixunil, if you have the time to review this much code then that would be awesome - but there are plenty of unit tests for the changes (and I want to simplify them in future), so if you don't then that's fine.
Edit: This PR is wayyyy bigger than I thought it would be :laughing:.  Perhaps moving all the unit tests around wasn't the best idea... but it's done now.

Unfortunately I don't have a lot of time lately. I may be able to look at it during weekend, not sure.

Unfortunately I don't have a lot of time lately. I may be able to look at it during weekend, not sure.

It's fine - don't stress yourself over it.  Most of it is of the janky-but-works type of code, but it'll be refactored into cleaner code along with other things...

It's fine - don't stress yourself over it.  Most of it is of the janky-but-works type of code, but it'll be refactored into cleaner code along with other things...

I think in the interest of avoiding mega-big merge conflicts, I'm gonna merge this this evening.  If you happen to have a sudden desire to review ~1k lines of code then now is the time :grin:.

I think in the interest of avoiding mega-big merge conflicts, I'm gonna merge this this evening.  If you happen to have a sudden desire to review ~1k lines of code then now is the time :grin:.


Provide a way to use plugin definition from version catalog without version

If you apply a plugin to root project, it is part of the class path.
If you then also apply it to a subproject, it must not contain a version definition or Gradle errors out.
Same if you have a plugin as dependency in buildSrc and then want to use that plugin in the main build.
Currently (Gradle 7.2-rc-2) this is not really well supported with version catalogs.

you cannot define a plugin without version in the version catalog
you cannot set the version to null manually

Things that would be nice if supported and would help here:

Being able to define a plugin without version in the version catalog like[plugins]
kotlin-withoutVersion = { id = "org.jetbrains.kotlin.jvm" }

so that you can use it like plugins { alias(libs.plugins.kotlin.withoutVersion) } if the plugin somehow already landed in the class path, like for example having it as dependency in buildSrc.
Being able to unset a version for a plugin accessor for example like
plugins { alias(libs.plugins.kotlin) version null } or
plugins { alias(libs.plugins.kotlin) withoutVersion() } or
plugins { alias(libs.plugins.kotlin.withoutVersion()) } or

Dirty hack-around for Kotlin DSL:
in the root build script
alias(libs.plugins.kotlin)

and in the subproject build script
alias(libs.plugins.kotlin).apply {
    javaClass.getDeclaredField("version").apply { isAccessible = true }.set(this, null)
}

In Groovy DSL I didn't find a hack-around, as the AST transformation validates that only id or alias is done on top-level and that only apply or version are called on the PluginDependencySpec.

for kotlin DSL this is what (hack) i did
plugins {
    id(libs.plugins.kotlin.get().pluginId)
}

Ah, that's much better of course, and should also work with Groovy DSL, thanks.
But I'm keeping this open for the consideration to be able to declare without version in the catalog right away.

The workaround does not seem to work with the Groovy DSL. For example, with this libs.versions.toml file:
[versions]
kotlin = "1.5.30"

[plugins]
kotlin = { id = "org.jetbrains.kotlin.jvm", version.ref = "kotlin" }

and the following plugins block:
plugins {
    id(libs.plugins.kotlin.get().getPluginId())
}

I get the following error:
* Exception is:
org.gradle.groovy.scripts.ScriptCompilationException: Could not compile build file '[snip]\build.gradle'.
    [huge stack trace removed]
Caused by: org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
build file '[snip]\build.gradle': 2: argument list must be exactly 1 literal String or String with property replacement

See https://docs.gradle.org/7.2/userguide/plugins.html#sec:plugins_block for information on the plugins {} block

 @ line 2, column 5.
       id(libs.plugins.kotlin.get().getPluginId())
       ^

It seems that anything other than a literal string or a literal string with interpolation from project properties is not accepted here... This makes plugins in version catalogs useless for multi-module projects, which is a shame :(

Will be partially fixed by #19015, another PR will be made for specifying without versions in the catalog, and this issue shall be left open until then.

Just as an FYI, currently the error message suggests that withoutVersion already exists on PluginAliasBuilder:
Possible solutions:
  1. Call `.version()` to give the alias a version.
  2. Call `.withoutVersion()` to explicitly declare that the alias should not have a version.

Yes, but IIRC there's no way to do it in TOML, that's what my comment / linked issue was about.

Is it planned to still allow declaring plugins without versions? I am still getting issues with precompiled plugins in a build-src included build.

Just leave out the version in the toml, or use withoutVersion() in the DSL

@Vampire I am getting plugin version '' is invalid: cannot be null or empty if I remove the version and I only have the id.
withoutVersion() doesn't exist.
I am on Gradle 7.5

I can't do the second, it fails because it says there is an unknown version on the classpath.
I am doing the first one, but it is annoying, to be honest, I am going to try the third one, thank you!

Ah, no, I mixed up things again.
Even in the settings DSL it only works for libs, not for plugins, sorry.

If the second fails, then because you somehow get the plugin version in using an unsupported way - in terms of that feature.
You could potentially change that.

How? I am creating some precompiled plugins in an included build that is not buildSrc. But I am not adding them to the classpath, I guess this is being done automatically, but not sure how to avoid that.
The third doesn't work as you said, it only works for libraries.

How do y'all use withoutVersion()? I'm getting this error

Could not find method withoutVersion() for arguments [] on object of type org.gradle.api.internal.catalog.DefaultVersionCatalogBuilder$DefaultPluginAliasBuilder.

Looking at the source the method exists for DefaultLibraryAliasBuilder but not for DefaultPluginAliasBuilder

We don't

I thought that's what you said you were doing here? https://github.com/gradle/gradle/issues/17968#issuecomment-1189078037

Read on, two comments below that one for example. ;-)

version catalogs is a disaster

A proposed fix: #27200

until the MR from @unshare got merged, to make it clear, the work around right know that is working from me based on @Vampire`s explanation
libs.versions.toml
[versions]
jacoco = "_"

[plugins]
jacoco = { id = "jacoco", version.ref = "jacoco" }

build.gradle
plugins {
    libs.plugins.jacoco.get().pluginId
}

Has this functionality regressed? I'm getting
> Invalid catalog definition:
    - Problem: In version catalog libs, dependency alias builder 'foo.bar' was not finished.

      Reason: A version was not set or explicitly declared as not wanted.

      Possible solutions:
        1. Call `.version()` to give the alias a version.
        2. Call `.withoutVersion()` to explicitly declare that the alias should not have a version.

      For more information, please refer to https://docs.gradle.org/8.11/userguide/version_catalog_problems.html#alias_not_finished in the Gradle documentation.

but .withoutVersion doesn't exist for plugins.

I'd say that might be two new issues that you should report.

plugin("foo.baz", "foo.baz") should not suggest adding withoutVersion() if it does not exist and should imho also not talk about "dependency alias builder" but "plugin alias builder"
A withoutVersion() should probably be added to the settings script API. Currently you can work-around it using plugin("foo.baz", "foo.baz").version {}.


fix: resolve TODO comments

Description
Resolve TODO comments, finish undone tasks.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Appropriate labels applied
[x] Targeted PR against correct branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards.
[x] Wrote unit and integration
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[x] Re-reviewed Files changed in the Github PR explorer
[x] Review Codecov Report in the comment section below once CI passes

@hallazzang Could you resolve a conflict?

@dongsam Yes it's done now


Request: provide sample and some information

Can't find how to use this library.
Is it still being developed?

This library is a dependency for Soot and maintained as such.

Can it be used to parse APK files in some way?
If so, please show how.
May I use it for this purpose?
Is it still being developed?


 Specifically, visual identifiers in RPROMPT were still cut off because of the color reset. The fix is even simpler than the previous PR: just remove the %f after the visual identifier.
Here is a screenshot showing the cut-off before (the first line) and after (the second line) applying the patch:

Thanks for yet another fix! This is important to me, as adding a space after the icon does no longer seem to be a workaround in konsole as suggested by the readme. It seems like the space now has the same background color as the segment, and still overdraws the right part of the glyph.
Am I understanding correctly that this PR is scheduled to land in 0.7.0?

@ytang - Thanks so much for this PR! Sorry it's taken us a few weeks to circle around to it.
There are now conflicts with the merge due to the change in the color naming that was merged in a different PR. They look pretty straight-forward to resolve, though.
@dritter - I see you tagged this one for v0.7.0, and we for sure need to port it over to the new codebase. I think this one is probably also worth merging into master, here, as soon as we can since it's a useful bugfix. Thoughts?

I agree, this is also worth merging into master.
And as @LordTyrius pointed out, this fix is only for older Konsole versions. For newer Konsole versions this won't work, and this needs to be fixed on Konsole's side AFAIR.

FYI, test/segments/context.spec also fails in master and should be unrelated to my patch.

@ytang I created #1020 to fix the issues with context.spec.


planning: when rerouting, add unpassed waypoints.

@startcode The variable names are a bit confusing through.


Compromised user passwords.

From turbocat2001:

With "strace sudo" all user passwords can be compromised. To do this, you need to patch strace by increasing the output for the "read" syscall.

This misses the actual issue which is that a ptrace'd process that execs a SUID binary should not actually SUID.

There are many situations the SUID handling doesn't catch that it should - ToaruOS is not secure and the entire infrastructure behind sudo is a toy, a facsimile of the real deal.
Anyway, fixed in 5f643d703478a5f36a1c00ef1171d1b0429537c5 and sudo will report when it didn't get an effective uid of 0 as of<PHONE_NUMBER>255fe2b289a72508ee532927ea0ca8

@turbocat2001 You're unblocked, file your own bug reports.


style: integrate commitlint

Codecov Report

Merging #96 (8436f83) into master (c8e295b) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #96   +/-   ##
======================================
  Coverage    9.64%   9.64%
======================================
  Files          42      42
  Lines         197     197
  Branches       23      23
======================================
  Hits           19      19
  Misses        178     178

Flag
Coverage Δ

nx-remotecache-s3
9.64% <ø> (ø)

pellegrims-dev
9.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 62a6811...8436f83. Read the comment docs.


add IPFS hash of publication

to ensure availability of these files even if the site hosting them goes down.

The licensing conditions of most academic publications does not allow the papers to be distributed freely. They do however allow a self-published version by the author, which we already link to if possible. I'm afraid that linking to (possibly) pirated versions of the papers could be illegal.


Bump ray from v2.1.0 to v2.2.0

Motivation
We need to bump ray to v2.2.0 to support the relative path.
Please refer to this issue and this PR for details.
TODO

[ ] Test

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@12b09c8). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #121   +/-   ##
=======================================
  Coverage        ?   72.79%
=======================================
  Files           ?       59
  Lines           ?     1632
  Branches        ?      235
=======================================
  Hits            ?     1188
  Misses          ?      345
  Partials        ?       99

Flag
Coverage Δ

unittests
72.79% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
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.

@nijkah
Unfortunately, chdir_to_trial_dir is not working in Trainer class.
Refer to this

This line seems to be the problem.


High Gas Use - August Drop Rate

Hi!
I have been watching the performance for this bot for the month of August and it looks like it has a high drop rate in ETH and BSC. I would appreciate if you can check it out, and consider sharding as a potential solution to mitigate this.

solved in https://github.com/NethermindEth/Forta-Agents/pull/453


fix error strings be capitalized

What this PR does / why we need it:
According to Golang coding convention, error strings should not be capitalized.

it looks like the tests are failing because they expected the capitalized form. Would you mind fixing that?


Added Ephemeral Execute with Tag option

cli part for https://github.com/carbynestack/ephemeral/pull/17

@sbckr have rebased to current workflow files

I have just realised, that the newly added option is not yet described in the readme`. (see README.md#usage).
So far we have inserted the respective help output at the corresponding place

I have just realised, that the newly added option is not yet described in the readme`. (see README.md#usage).
So far we have inserted the respective help output at the corresponding place

Needs to be addressed in a holistic way (see carbynestack/amphora#34).


Extract  from  redundant components

This makes sense, and passing a class name as a prop should allow logic flow internally so Build can render individually. Also makes maintainability/etc. easier.

From previous issue brainstorming:
[Use a] function that checks for elemental prop, slices filtered skills from hash.
Consider coming up with a better idea than the array skill solution in general...it feels... "good enough"... not "awesome!!"

Fixed with #20


Build Issue arising from fmt library

Problem description
Issue observed when building latest main, looks to stem from the fmt library - could it be related to the recent fmt version upgrade?
System information
Velox System Info v0.0.2
Commit: 02a4f6c3ddd37f5656488d81517bb9042ee857f2
CMake Version: 3.27.7
System: Darwin-23.1.0
Arch: arm64
C++ Compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
C++ Compiler Version: <IP_ADDRESS>00040
C Compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
C Compiler Version: <IP_ADDRESS>00040
CMake Prefix Path: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.0.sdk/usr;/opt/homebrew;/usr/local;/usr;/;/opt/homebrew/Cellar/cmake/3.27.7;/usr/local;/usr/X11R6;/usr/pkg;/opt;/sw;/opt/local
CMake log
In file included from /Users/svm/presto/presto-native-execution/velox/velox/common/memory/tests/MockSharedArbitratorTest.cpp:19:
In file included from /usr/local/include/fmt/format.h:44:
/usr/local/include/fmt/core.h:1168:20: error: assigning to 'const void *' from incompatible type '__bit_iterator<vector<bool, allocator<bool>>, true>'
    custom.value = &val;

Hmm, @svm1 Looks like you are running on an M1 , The m1 build for that sha did succeed : https://app.circleci.com/pipelines/github/facebookincubator/velox/44031/workflows/4521fd66-c537-4aa9-99a5-b1b8880aa243/jobs/307056 . I am building locally on my machine to validate this.
I presume you did a make clean and rebuilt ? Also presuming you made sure you have the latest dependencies, and if not run setup-macos.sh.

@svm1 I was able to build fine. Can you try out some of the suggestions above ? Thanks.

@kgpai Thanks - I ran make clean and rebuilt, but forgot to update dependencies with setup-macos.sh, my bad. Build succeeded, apologies for the false alarm.


Add wasm web test and common tasks

This configures the counter, counter_without_macros, and counters examples to use a set of reusable tasks to  support testing and verification.
Verification:
From the root directory:

Run cargo make test-examples
Run cargo make verify-examples (Currently fails checks for some examples but the task works)

From the example directory:

Run cargo make test-local
Run cargo make web-test
Run cargo make check-style (counter and counters currently fail checks but the task works)
Run cargo make verify-local (counter and counters currently fail checks but the task works)

This PR will make it easier to work through the existing style issues.

Thanks!  Run the tests locally when I have time. Our CI issues have resolved with the most recent commit I made, I think, but just to double check.

That sounds good. Thanks for the update @gbj.


Merge Upstream to support latest API Changes

Discord API has changed a lot since the beginning of the year, 0.15 of Eris has support for the new Voice Regions in Voice channels as well as Stage channels. Creating this PR to see where any custom implementations will cause conflicts with the upstream.

Closed to redo properly


This does not work - What is this? how does it work?

I downloaded the .zip, I see CentOS.exe, now what?
If I double click CentOS.exe, I get the following:
c:\os\wsl\centos_wsl>.\CentOS.exe
Installation directory not found: .
Make sure it exists or reinstall.
Press any key to continue...

same when I run from the command line.
Is there a .tar anywhere for CentOS7 that will work if I do:   cmd>wsl --import ?
I just want CentOS to work under WSL.

Please tell me
wslconfig /l
output?

This is the output:  I have installed several Linux distributions from the Windows Store, I am trying to get CentOS 7.1905 onto my Windows machine.  Thank You.
c:\>wslconfig /l
Windows Subsystem for Linux Distributions:
Ubuntu (Default)
kali-linux
SLES-12
Alpine
WLE
Debian

Please excute
CentOS.exe clean command and
double click CentOS.exe ?


Working Java drivetrain

Working java drivetrain code.   Tested on Apollo and Swervebot (we need a better name for this robot).
Now this PR is going to be UGLY.  These are 87 files marked as changed.  Try to go through as many as possible to make sure I haven't made any problems.

A significant portion of these changes are due to cleaning up the git repository where is was storing a number of the build artifacts (.gradle/, build/).
We also added the pathplanner path source data (in the deploy directory).
The original version did not have any drivetrain code.  This code main purpose is to all this
The Constants.java file was copied from the NerdHerd repository (where we were developing the drivetrain) and therefore the names changed for most of the subsystems.


please add Before function to timestamp

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
I need to sort a slice of timestamps, which needs a comparison function
Describe the solution you'd like
A clear and concise description of what you want to happen.
A 'Before' function like there is for time.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Writing the function myself, but this seems common enough that I shouldn't have to.
Additional context
Add any other context or screenshots about the feature request here.
Thank you for a very useful module.

Can you justify why Before needs to be added when you can do something like:
ts1.AsTime().Before(ts2.AsTime())

The moment we add a Before helper method, it opens the door to adding all other helper methods defined on time.Time to timestamppb.Timestamp. We don't want to duplicate all the methods of time.Time onto timestamppb.Timestamp.

I cannot justify it. What you proposed is a better solution. Thank you for a quick reply.


Not able to disable Ripple Effect.

if I click on button to perform any action, it will first finish the whole ripple effect than perform the action.
I m using Android 6.0.1 On moto g3.

Did you try to set "animate" attribute to false? It's used here https://github.com/navasmdc/MaterialDesignLibrary/blob/master/MaterialDesignLibrary/MaterialDesign/src/main/java/com/gc/materialdesign/views/Button.java#L38


Aqualab: Detector for case a player used two hints within THRESHOLD seconds

THRESHOLD should be an input parameter configured in AQUALAB.json.

 Aqualab Detector: Used two hints within [5, 10, 20] seconds (arbitrary time)

Added in commit 038b88c60357e9654400cdb57de90622571a7086


[WIP] Verify DataFrame API on pandas 1.3.0

pandas just published 1.3.0rc1. I started this branch to verify the DataFrame API against this release candidate and address any bugs that arise.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@rohdesamuel @robertwb - Do we need to do anything related to the errors here? It looks like pandas 1.3.0 is released and it is the latest version.

If someone has fine to take a look at the remaining failures that would be great, but IMO it's also fine to just support 1.2.x in Beam 2.32.0

If someone has time to take a look at the remaining failures that would be great, but IMO it's also fine to just support 1.2.x in Beam 2.32.0

@rohdesamuel - if you have time please take a look at the failures. Otherwise, I agree, it is fine to just support 1.2.x in Beam 2.32.0.

We can close this. https://github.com/apache/beam/pull/15165 is a clone of this.


Run tasks before and after infrequent channels

There are four channels:

hourly
daily
weekly
monthly

Of those, I am categorising weekly and monthly as 'infrequent'.
After the channel immediately preceding an infrequent channel (i.e. the hourly channel that runs an hour before it), any built-up notifications for users on that infrequent channel should be checked to see if those threads still exist (maybe even check posts individually, as a bonus).
(It occurs to me that it's probably worthwhile to only run this on the weekly channel rather than both the weekly and monthly channel.)
After the channel immediately following an infrequent channel, infrequent users should be checked to ensure that on the prior channel, they were actually notified. If not, they should be re-notified.
Base

[ ] Find a reliable way of determining the current channel's relationship to another channel (and, depending on the implementation, maybe use this method to determine which channels should be activated for a given channel)
[ ] Check that a) apscheduler starts tasks within one minute of the declared time guaranteed, b) that pycron always accurately assesses that time

Before

[ ] On the frequent channel before an infrequent channel, check for deleted threads

After

[ ] On the frequent channel after an infrequent channel, check that subscribers of the infrequent channel were notified

Although that being said... if the original notification procedure is done correct, this would never be called. Is this a good idea? Is it worth the effort?

Quick test to see if apscheduler and pycron are both reliable and play nicely with each other. This will emit a timestamp every minute and a True or False. The cardinality must alternate between each emission. If a True ever follows a True or vice versa, it is not reliable.
>>> sch = BlockingScheduler()
>>> sch.add_job(lambda: print(time.time(), pycron.is_now("*/2 * * * *")), CronTrigger.from_crontab("* * * * *"))
<Job (id=3467d7a45a554c1aa74d9570787097f9 name=<lambda>)>
>>> sch.start()


Incorrect exercise: [courseId=grammar-intro_eng][lessonNumber=3][exerciseId=4318]

Description
English prompt implies a person (as opposed to an undefined entity) has to be the one teaching us. Maybe a genericization for the English prompt referring to x1 is warranted?
Diagnostics information (do not remove)

Exercise identifier: [courseId=grammar-intro_eng][lessonNumber=3][exerciseId=4318]
Exercise payload: {"sentences":[{"text":"Somebody taught us.","lojbanic":false}],"title":"Translate this sentence","type":"typing"}
Attempted solution: {"text":"lo prenu ku ctuca mi"}
Attempted solution was classified as correct: no
Validation payload: {"canonicalAnswer":"ctuca mi","correct":false}

Thank you! You are correct, the English "somebody" has a possible implicit implication of the entity being a person, which could reasonably lead students to answer with "lo prenu ku".
I made some changes to accept "lo prenu ku" whenever "somebody" is expected. So both "ctuca mi" and "lo prenu ku ctuca mi" will be considered correct now.


How to train sam from a COCO json custom dataset?

I have my custom dataset as a json in COCO format.
Is there a simple way to use a dataset that uses a coco.json file for training? Please teach me.

same question，maybe you need to change the dataloader


23 - Overview of total money owed & contacts count

Codecov Report

Merging #30 (bf90159) into master (494eafc) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #30      +/-   ##
============================================
- Coverage     72.37%   72.31%   -0.06%
  Complexity      409      409
============================================
  Files            71       71
  Lines          1267     1268       +1
  Branches        131      131
============================================
  Hits            917      917
- Misses          314      315       +1
  Partials         36       36

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/model/person/Money.java
66.66% <0.00%> (-8.34%)
:arrow_down:

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 494eafc...bf90159. Read the comment docs.

Looks good, will go ahead and merge.