chore: update ui components

Closes #909

@TC-MO can we configure lychee to not fail on timeout errors, or at least retry them a few times? this is quite tedious, lucky me that I can merge without the checks passing :D


bump jira client to v3.4.1

Newer jira client versions should only retry on 429 instead of 502, 503, 504, and 401 (including POST requests). This should be safer to do than current behavior.

/cc @adriengentil @rccrdpccl

/lgtm


Pipeline - multiple clusters on same git repo

question/enhancement
Is it possible to limit pipeline execution by cluster name (or branch name, but store it in project rather that in ".rancher-pipeline.yml")?
For example, we have separate production and development clusters.

there is an open feature request: https://github.com/rancher/rancher/issues/17474

I would love to have the same features, we use seperate cluster for dev, staging and production running the same pipeline. The issue is that we cannot limit the pipeline execution on specific branches ( for example only run the dev pipeline on dev branch ect )


Merging on master

LGTM. Thanks for the sumbission!

Thanks Pierre! I can't see the project in the list yet though?
Le sam. 13 juin 2020, à 01 h 41, Pierre Bellec<EMAIL_ADDRESS>a
écrit :

LGTM. Thanks for the sumbission!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BrainhackMTL/school/pull/64#issuecomment-643535117,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH7Q24YFW5HQIZ7JPLIHQCTRWLDK5ANCNFSM4N4XDDTQ
.


Update tankix to 638

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 840eeed5cc8d46a5b2d61505b328870b580fb672. Your contribution is still credited to you.


Dark Mode - hard to see active row in menus

In Dark Mode, when you are in Command Palette and Quick Switcher menus, it is very hard to see which row is active when you are navigating options.

Good catch! I updated the theme so selected items are more visible (I actually love how it turned out more than the original, hint: it uses the Things blue). I'll push the update and include it in the next release - expected to publish today or tomorrow. Thanks!

Thanks, it's a great theme and I'm really enjoying using it!
On Tue, 9 Nov 2021 at 18:28, Colin Eckert @.***> wrote:

Good catch! I updated the theme so selected items are more visible (I
actually love how it turned out more than the original, hint: it uses the
Things blue). I'll push the update and include it in the next release -
expected to publish today or tomorrow. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/colineckert/obsidian-things/issues/13#issuecomment-964419622,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHRIJTSTYD7ADGRAK7OWCTULFR4HANCNFSM5HU7SOSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[daw-header-libraries,daw-json-link]: head-lib update to 2.93.1, json-link 3.17.2

…-link update to 3.17.2

[*] Changes comply with the maintainer guide
[*] SHA512s are updated for each updated download
[*] The "supports" clause reflects platforms that may be fixed by this new version
[*] Any fixed CI baseline entries are removed from that file.
[*] Any patches that are no longer applied are deleted from the port's directory.
[*] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[*] Only one version is added to each modified port's versions file.

#31782 #31783


[2.0.0 Beta] Block Grid class needs width

[class*=block-grid-] needs a width of 100%.

On develop branch.


[EAK-299] Website first impl

Closes: #299

Should be merged with squash option

I have read the CLA Document and I hereby sign the CLA

I have read the CLA Document and I hereby sign the CLA

I have read the CLA Document and I hereby sign the CLA

recheck

I have read the CLA Document and I hereby sign the CLA


Finish implementation of geom_text and geom_label

complete with https://github.com/TidierOrg/TidierPlots.jl/commit/6cc55c4183852ceb90574c04021f6406873d9933


Snapshot: fix share agent convo + remove agent data from response

Thank you for contributing to the Cohere Toolkit!

[ ] PR title: "area: description"

Where "area" is whichever of interface, frontend, model, tools, backend, etc. is being modified. Use "docs: ..."  for CI changes.
Example: "deployment: add Azure model option"

[ ] PR message: Delete this entire checklist and replace with

Description: a description of the change
Issue: the issue # it fixes, if applicable
Dependencies: any dependencies required for this change

[ ] Add tests and docs: Please include testing and documentation for your changes

[ ] Lint and test: Run make lint and make run-tests

AI Description

This pull request updates the src/backend/schemas/snapshot.py and src/backend/services/snapshot.py files to include the Field class from the pydantic module and modify the SnapshotAgent class.
Summary
The changes in this PR are:

The Field class from the pydantic module is now imported in src/backend/schemas/snapshot.py.
The agent field in the SnapshotData class in src/backend/schemas/snapshot.py is now annotated with Field(exclude=True).
In src/backend/services/snapshot.py, the agent_crud module is now imported from backend.crud.
The get_agent function from snapshot_crud is replaced with get_agent_by_id from agent_crud in the wrap_create_snapshot function.
The tools_metadata list is now created by converting each metadata object in agent.tools_metadata to a dictionary using the to_dict function.

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

Project coverage is 81.44%. Comparing base (540ce78) to head (9e1933b).
Report is 1 commits behind head on main.

Files
Patch %
Lines

src/backend/services/snapshot.py
50.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #448      +/-   ##
==========================================
- Coverage   81.45%   81.44%   -0.01%
==========================================
  Files         198      198
  Lines        8239     8242       +3
==========================================
+ Hits         6711     6713       +2
- Misses       1528     1529       +1

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


correct the misspell of Quantize

Can one of the admins verify this patch?

@103yiran could you sign the CLA?


Laravel 10.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 10.x.
Before merging, you need to:

Checkout the l10-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: Shift detected a dependency for the laravel/framework. Unless you have a special requirement for depending on the entire Laravel framework, it's recommended to add dependencies for the specific illuminate packages instead.

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 10. However, you should review these changes to ensure the desired combination of versions are built for your package.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/voyager.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "tcg/voyager": "dev-l10-compatibility",
    }
}

Finally, run: composer update

This was not enough for me, I encountered bugs when going to Databases and BREAD
Error message :
Call to undefined method Doctrine\DBAL\Connection::createSchemaManager()
To solve the bug, I removed what concerns Larapack from /vendor/tcg/voyager/src/VoyagerServiceProvider.php (line 19 and 62)

To solve the bug, I removed what concerns Larapack from /vendor/tcg/voyager/src/VoyagerServiceProvider.php (line 19 and 62)

Tried and didn't work :(

To note, PHP 8.1+ deprecation warnings are been logged by users (#5603, #5737) which might want to be addressed first before pursuing Laravel 10 support.

There are a couple more problems we hit, mostly around orchestra/testbench.
Please check the 1.6-l10 branch

I've created a PR (#5753) that should make it work with Laravel 10.
@u01jmg3 's comment about the deprecation warnings have also been included in it. Hope it can be merged !


fix allow deny for object store subscription

Signed-off-by: Roke Jung<EMAIL_ADDRESS>https://github.com/open-cluster-management/backlog/issues/16662

/test test-unit

/test test-unit


CHANGELOG: add v3.1.11, bug fixes

In preparation for 3.1.11 patch release

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9717a12). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #8930   +/-   ##
=========================================
  Coverage          ?   76.05%
=========================================
  Files             ?      359
  Lines             ?    29791
  Branches          ?        0
=========================================
  Hits              ?    22658
  Misses            ?     5549
  Partials          ?     1584

Impacted Files
Coverage Δ

embed/config.go
62.24% <100%> (ø)

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 9717a12...8cc9063. Read the comment docs.


is it possible to use Region and Country from DNN Lists

Can I use the Region and Country fields in my Feedback instance and have a dropdownlist listing the Regions (States) and Countries instead a Text field?

these profile properties and controls have been rewritten in DNN 7.4.1, any supporting module version will not work with previous DNN versions (however, it sounds like a nice improvement)


🐛 Random Crash after Battle

Exception in thread Thread-7:
Traceback (most recent call last):
  File "C:\Users\Lukem\AppData\Local\Programs\Python\Python39\lib\threading.py", line 980, in _bootstrap_inner
    self.run()
  File "C:\Users\Lukem\AppData\Local\Programs\Python\Python39\lib\threading.py", line 917, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Repositories\Rush-Royale-Bot\Src\bot_handler.py", line 127, in bot_loop
    grid_df,bot.unit_series,bot.merge_series,bot.df_groups,bot.info = combat_loop(bot,grid_df, user_level, user_target)
  File "C:\Repositories\Rush-Royale-Bot\Src\bot_handler.py", line 82, in combat_loop
    grid_df,unit_series,merge_series,df_groups,info = bot.try_merge(prev_grid=grid_df,merge_target=user_target)
  File "C:\Repositories\Rush-Royale-Bot\Src\bot_core.py", line 235, in try_merge
    grid_df=bot_perception.grid_status(names,prev_grid=prev_grid)
  File "C:\Repositories\Rush-Royale-Bot\Src\bot_perception.py", line 60, in grid_status
    unit_guess= match_unit(filename,ref_colors,ref_units) if rank !=0 else ['empty.png',0]
  File "C:\Repositories\Rush-Royale-Bot\Src\bot_perception.py", line 43, in match_unit
    unit_colors = get_color(filename,crop=True)
  File "C:\Repositories\Rush-Royale-Bot\Src\bot_perception.py", line 36, in get_color
    index = np.where(counts == sorted_count[i])[0][0]
IndexError: index 2 is out of bounds for axis 0 with size 2

I get this same error as well.

Thanks for the bug report!
I saw this bug yesterday as well but only added the fix to branch I was working on.
It's merged into main now https://github.com/AxelBjork/Rush-Royale-Bot/pull/30
The issue was if an image only had a single pixel color.

Awesome work
Closed via #30


FileNotFoundError: [Errno 2] No such file or directory: '/home/zch/anaconda3/envs/u-sleep/lib/python3.7/site-packages/utime/bin/defaults'

Hi, when I init a project, there is a problem.
FileNotFoundError: [Errno 2] No such file or directory: '/home/zch/anaconda3/envs/u-sleep/lib/python3.7/site-packages/utime/bin/defaults'
How can I solve it ,thank you!

Did you have the ERROR that utime and mpunet require the different version of packages ?
Just like:"ERROR: mpunet 0.2.12 has requirement matplotlib>=3.3.0, but you'll have matplotlib 3.1.3 which is incompatible.
"
and "ERROR: utime 0.0.2 has requirement matplotlib==3.1.3, but you'll have matplotlib 3.4.3 which is incompatible.“

Hi！I have the same question. But i have another question begore this:
Did you have the ERROR that utime and mpunet require the different version of packages ?
Just like:"ERROR: mpunet 0.2.12 has requirement matplotlib>=3.3.0, but you'll have matplotlib 3.1.3 which is incompatible.
"
and "ERROR: utime 0.0.2 has requirement matplotlib==3.1.3, but you'll have matplotlib 3.4.3 which is incompatible.“

Hi！I have the same question. But i have another question begore this:
Did you have the ERROR that utime and mpunet require the different version of packages ?
Just like:"ERROR: mpunet 0.2.12 has requirement matplotlib>=3.3.0, but you'll have matplotlib 3.1.3 which is incompatible.
"
and "ERROR: utime 0.0.2 has requirement matplotlib==3.1.3, but you'll have matplotlib 3.4.3 which is incompatible.“
Did you meet this problem?Thank you!

Hi！I have the same question. But i have another question begore this:
Did you have the ERROR that utime and mpunet require the different version of packages ?
Just like:"ERROR: mpunet 0.2.12 has requirement matplotlib>=3.3.0, but you'll have matplotlib 3.1.3 which is incompatible.
"  Then I updated this package.
But "ERROR: utime 0.0.2 has requirement matplotlib==3.1.3, but you'll have matplotlib 3.4.3 which is incompatible.“
Did you meet this problem?Thank you!

Hello!
I think your problem is the version of the package.
The original project requires a lower version of the installation package, and you install a higher version.
You can try to reduce the software version to the project requirements.
Good luck!
------------------ 原始邮件 ------------------
发件人:                                                                                "perslev/U-Time"                                                                                @.>;
发送时间: 2021年11月11日(星期四) 上午9:19
@.>;
@.>;"State @.>;
主题: Re: [perslev/U-Time] FileNotFoundError: [Errno 2] No such file or directory: '/home/zch/anaconda3/envs/u-sleep/lib/python3.7/site-packages/utime/bin/defaults' (#40)
Did you have the ERROR that utime and mpunet require the different version of packages ?
Just like:"ERROR: mpunet 0.2.12 has requirement matplotlib>=3.3.0, but you'll have matplotlib 3.1.3 which is incompatible.
"
and "ERROR: utime 0.0.2 has requirement matplotlib==3.1.3, but you'll have matplotlib 3.4.3 which is incompatible.“
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Hello!
I think your problem is the version of the package.
The original project requires a lower version of the installation package, and you install a higher version.
You can try to reduce the software version to the project requirements.
In fact, I didn't run u-time successfully.
It always has one mistake or another.
Good luck!
------------------ 原始邮件 ------------------
发件人:                                                                                "perslev/U-Time"                                                                                @.>;
发送时间: 2021年11月11日(星期四) 上午9:21
@.>;
@.>;"State @.>;
主题: Re: [perslev/U-Time] FileNotFoundError: [Errno 2] No such file or directory: '/home/zch/anaconda3/envs/u-sleep/lib/python3.7/site-packages/utime/bin/defaults' (#40)
Hi！I have the same question. But i have another question begore this:
Did you have the ERROR that utime and mpunet require the different version of packages ?
Just like:"ERROR: mpunet 0.2.12 has requirement matplotlib>=3.3.0, but you'll have matplotlib 3.1.3 which is incompatible.
"
and "ERROR: utime 0.0.2 has requirement matplotlib==3.1.3, but you'll have matplotlib 3.4.3 which is incompatible.“
Did you meet this problem?Thank you!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Hi!it seems that I run usleep successfully. I just ran the first command in the demo again:git clone http://github.com/perslev/U-time.gitI'm new in DL and CS. I may have mistakes. I hope it's helpful to u发自我的华为手机-------- 原始邮件 --------发件人： CandyZhao123 @.>日期： 2021年11月11日周四 11:00收件人： perslev/U-Time @.>抄送： emleeee @.>, Comment @.>主    题： Re: [perslev/U-Time] FileNotFoundError: [Errno 2] No such file or directory: '/home/zch/anaconda3/envs/u-sleep/lib/python3.7/site-packages/utime/bin/defaults' (#40)
Hello!
I think your problem is the version of the package.
The original project requires a lower version of the installation package, and you install a higher version.
You can try to reduce the software version to the project requirements.
Good luck!
------------------ 原始邮件 ------------------
发件人:                                                                                "perslev/U-Time"                                                                                @.***>;
发送时间: 2021年11月11日(星期四) 上午9:19
@.***>;
@.>;"State @.>;
主题: Re: [perslev/U-Time] FileNotFoundError: [Errno 2] No such file or directory: '/home/zch/anaconda3/envs/u-sleep/lib/python3.7/site-packages/utime/bin/defaults' (#40)
Did you have the ERROR that utime and mpunet require the different version of packages ?
Just like:"ERROR: mpunet 0.2.12 has requirement matplotlib>=3.3.0, but you'll have matplotlib 3.1.3 which is incompatible.
"
and "ERROR: utime 0.0.2 has requirement matplotlib==3.1.3, but you'll have matplotlib 3.4.3 which is incompatible.“
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.

Hi!
it seems that I run usleep successfully. I just ran the first command in the demo again:"git clone http://github.com/perslev/U-time.git"
I'm new in DL and CS. I may have mistakes. I hope it's helpful to u!


Infinite money duplication glitch [1.19.4]

Game version: 1.19.4
Fabric version: 0.14.19
How to recreate:
I'm not too sure how to recreate it but maybe it's like this

Install Open parties and claim
set claim
other player who don't have the permissions to claim tries to put the shop in the claim
the shop disappear from hand
go out from the claim region
swap the shop with coin (gold/bronze/silver)
right click the items and keep swaping

Video of the bug/glitch
https://www.youtube.com/live/iv0-rCBN5OE?feature=share&t=2520
at time 42:00

Ping

There are two problems with this issue

I haven't been able to reproduce it in my testing, although that may be because I didn't quite understand the instructions (what does swapping mean exactly?)
There are only two possible outcomes - either this issue is purely client-sided (and thereby purely cosmetic) or it's a fundamental problem of how OPAC handles item usage in claims, which is not something that Numismatic should be responsible for or even can fix

If you could try giving us a better description, ideally with a video and a minimal modlist, for reproducing this, we can try taking another look at it - but for now we'll treat this is as an external issue that you should probably report on OPAC separately
Cheers

We are not sure whether this still is an issue. If you can reproduce this in a controlled context feel free to re-open this. Otherwise, it will stay closed for now. If you do find another bug or glitch please open a new issue


More Robust Application Examples

I think the time has come to include complete working project examples in the GitHub source. Suggestions are welcome. Contributions are encouraged.

Some more examples... actually, all of the examples in the Readme were quietly added in v0.8.


Minor: avoid trailing spaces

Is it an actual bug?
Yes, minor
Did you check if the bug has already been reported?
Yes
Describe the bug
When adding context or projects using the button, sleek adds a trailing space after the context/project label. When saving the task line in the todo file, if this results in a trailing space at the end of the line, the trailing space should be removed.
The trailing space is not a problem wrt the todo.txt specification. However, one of the advantages of the todo.txt format is that it is purely textual and line based, so diffs can be taken (directly or via a revision control system such as git). Trailing spaces are invisible and if present and then removed end up in spurious diffs, so they'd better be avoided beforehand.
To Reproduce
Steps to reproduce the behavior:

add a task, completing it with a context entry obtained by pressing a context button.
check the saved todo file
note a trailing space at the end of the task line

Do you see any error entries in sleeks developer tools?
N/A
Expected behavior
Would be great if sleek always saved task lines with no trailing spaces
Screenshots
N/A
Desktop (please complete the following information):

OS: Manjaro
Version of sleek: 1.3.1
Source: Flathub

@thecnoNSMB could you please check if this still occurs in this pre release? https://github.com/ransome1/sleek/releases/tag/v1.3.2-rc.1

Seems fine, thanks!


DebouncedTextFieldFormatChangeRerenderTest: fix culture issue on non-english locales

Description

Fixes format mismatch during TextFieldTests on non english systems.
On my system (set to german) the DebouncedTextFieldFormatChangeRerenderTest fails with the following message:
Expected textField.Text to be "2023/07/12", but "2023.07.12" differs near ".07" (index 4). since commit 47a1372
How Has This Been Tested?

Ran the unit tests and DebouncedTextFieldFormatChangeRerenderTest worked afterwards
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Probably something for @henon  to check out. I just skipped the bug report because I came across this while working on another pr and already had the code ready

Oh, so it doesn't work on English locales now?

Oh, so it doesn't work on English locales now?

No, my issue is that before your commit, the tests worked on my pc with german locale and after the commit they broke due to two different formats being used.
Also the Culture Property is from MudBlazors MudBaseInput

Got it. I have a freak setup. My windows is in US English because I can't stand the German translations but I use German keyboard and date time settings. I still wonder why the test did not fail for you before my commit.
Anyway, thanks for the fix!


Update set-up-windows-containers-with-powershell.md

Summary
The example Dockerfile did not work. Updated to get tag ltsc2019 which will be installed with powershell 7, so "RUN powershell.." switched to "RUN pwsh..".

Hi @stackoverjoe, thank you for submitting this PR.
If you use FROM mcr.microsoft.com/windows/servercore:ltsc2019 then by default it uses PowerShell version 5.

And that's the reason while running the step RUN pwsh -Command Add-WindowsFeature Web-Server fails with following error.

So it's better to keep the RUN powershell  step as is.
FROM mcr.microsoft.com/windows/servercore:ltsc2019
LABEL Description="IIS" Vendor="Microsoft" Version="10"
RUN powershell -Command Add-WindowsFeature Web-Server
CMD [ "ping", "localhost", "-t" ]

And that can be run successfully.

Reference
https://devblogs.microsoft.com/dotnet/we-made-windows-server-core-container-images-40-smaller/

Hi @sughosneo thank you for the reply.
I was mistaken by thinking that mcr.microsoft.com/powershell installed the same image on windows! Thank you for the correction!

Hi @sughosneo thank you for the reply.
I was mistaken by thinking that mcr.microsoft.com/powershell installed the same image on windows! Thank you for the correction!

Looks good ! Thank you


--force not working

Hi Ben!
It seems like there's a bug in using npx lets-version get-bumps --force. When running it on a repo that has no new changes, I'm consistently getting:
No bumps can be applied. This likely means you have not yet created your first version with the lets-version library, or no changes have occurred since the last version bump.

It seems like there might be a bug here: https://github.com/benduran/lets-version/blob/main/src/lets-version.ts#L375-L404 -- the result of updating bumpTypebyPackageName never seems to be used.
I found that I had to downgrade to version 0.8.3 to get --forceAll to work as expected, which means I also had to downgrade turbo-tools to 0.5.5 :'(

Hi @amybingzhao thanks for the report! My apologies for not seeing this sooner.
I'm in the process of moving, so I may not be able to get to this until later in the week. Happy to review any PRs you may have, though.
Thanks for your patience!

@amybingzhao would it be possible for you to setup a repro dummy repository? 🤞

This issue has become stale and may likely be fixed with the latest release (per the comment above). If it persists, please feel free to open another issue. Thanks!

Oops sorry I missed your replies Ben! Thanks for addressing! Hope the move went smoothly!


currently failing on 3 tests as part of `Breadcrumb directive with mu…

…ltiple-interpolation conf` test

renders the correct state chain and views content,
renders the correct state chain and views content, and
renders the correct state chain and views content

change all occurences of $viewContentLoaded to $stateChangeSuccess

Merge  with Master

Coverage increased (+2.4%) to 97.059% when pulling bdb89f3b529c2a987a0fdeb0690efb9120be6bdc on haxxxton:master into 301df6351f9272347ad3bd4b75f51a4d245dd2ef on ncuillery:master.


Allow user to configure defaults for clinvar submission

Is your feature request related to a problem? Please describe.
We need for an easy way to let the user configure several defaults when creating a clinvar submission set. This reduces the number of clicks and key strokes.
Describe the solution you'd like
Add a new wizard page where users can configure the following values:

[ ] individual sample source
[ ] individual variant origin
[ ] invidiual tissue
[ ] individual pubmed citation

Describe alternatives you've considered
N/A
Additional context
N/A

See #286 for implementation proposal.

Resolve with future clinvar upload feature.


Deployment Error

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [x] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Used GitHub codespaces
Used existing resources and set their environment variables
Error when running 'azd up'

Any log messages given by the failure
Problem: Error when running 'azd up'
Received the following error, but the client has the role 'owner' for the used resource group.
Response: 403 Forbidden
Error code: Authorization Failed
Message:
The client does not have authorization to perform action 'Microsoft.Resources/deployments/write'.

Expected/desired behavior
Succesful deployment

OS and Version?

Windows 11

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

I have the same issue. I think this is because it makes a deployment under the subscription level, where you (we) don't have the necessary rights. I don't have any solution for it yet, I would appreciate a solution which only requires to be owner of a resource group.

I am working on my local environment and got the same error message when I run
azd init -t azure-search-openai-demo

A detailed permission list can be provided to run the demo.

refer https://github.com/Azure-Samples/azure-search-openai-demo/issues/4#issue-1619906532

+1 this, I have the perms but still the same issue

https://learn.microsoft.com/en-us/azure/ai-services/openai/concepts/use-your-data


Add reservation reducer to the store

Added reservation reducer since someone already worked on the destination reducer that was my task to do.

The reservation reducer is to be added by Hubert


Multiple attribute example forces Binder instead of IBinder implemenation

Because you can only use Binder there is no way to mock the IBinder interface for unit testing. It's very confusing behavior and is there any reason for this?

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

ID: 7a4532b8-8c27-ca15-edc6-85fc38cab341
Version Independent ID: e06de78b-9271-25a4-439d-2a024f31693e
Content: Azure Functions C# developer reference
Content Source: articles/azure-functions/functions-dotnet-class-library.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@jvanderbiest Thank you for your feedback! We will review and update as appropriate.

@jvanderbiest Apologize for late response here! We typically try to keep the Azure-Docs channel scoped to feedback directly about the docs. Since your question goes beyond what is discussed in the doc, we kindly invite you to share your question in Q&A or perhaps in the Functions repo Thank you for your understanding.


Feature/rpcbatch3

Apologies for the previous closed pushes. This one is the right/last one.
Adds RPC batching over the 'develop' branch.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jeffrey Rainy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jeffrey Rainy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


import fventuri's changes

@fventuri
https://github.com/fventuri/machine/tree/504-kvm-qemu-driver
https://github.com/fventuri/machine/commit/872782272fa16017b58569a12f6662f17e3c997e

Everything in the old-style driver should already be included in the new one ?
Networking is slightly different, in that it now allows for using "user" (SLIRP)


Lazy assembly download

Current behavior - All the .NET assemblies are downloaded asynchronously during the first page load
New feature requirement - It would be nice if only required assemblies get downloaded during the first-page load. Remaining assemblies should get download as a user navigates to other pages.

I think there may be a mix up on that. It seems to me he means, If Page a.html needs system.x.dll he it will download and use it, If it also needs system.yy it will download and use it. When you hit page b.html it wont need to download system.yy as its in your cache.
Excluding some websites most websites just get what they need page to page and reuse what they already have. Pretty sure this would be automatic so the programer wouldnt even have to worry the chain of dependents, as it sounds like that may be your concern here.
There may be value in some use cases, downloading everything up front but in general i dont think there is.

We're tracking lazy loading as #25, so will close this as a duplicate


Release 0.6.0 checklist

[x] tag and build 0.6.0 (https://github.com/sigstore/sigstore-java/releases/tag/v0.6.0)
[x] release to maven central (https://repo1.maven.org/maven2/dev/sigstore/sigstore-java/0.6.0/)
[ ]  release gradle plugin to gradle plugin portal
[ ] update project version and README

FYI: @patflynn @vlsi

WDYT of releasing 0.6.0 as 1.0.0?

I think we've made a few changes since?

Id like to do a 0.7.0 with the new release mechanism first? And maybe we have. .sigstore.json as the default on the bundle output?

Id like to do a 0.7.0 with the new release mechanism first?

That is not a user-visible change, so it can go within any version.

And maybe we have. .sigstore.json as the default on the bundle output?

I missed the change.
For reference, here are the traces:

https://github.com/sigstore/sig-clients/issues/8#issuecomment-1663046593
https://docs.google.com/document/d/1kbhK2qyPPk8SLavHzYSDM8-Ueul9_oxIMVFuWMWKz0E/edit#heading=h.frw4lfrx4pv0

If clients serialize the bundle to a file, the file SHOULD have the extension .sigstore.json. To write multiple bundles in one file, clients SHOULD use the JSON Lines format (format each bundle without newlines, and write one bundle per line) and the extension .sigstore.jsonl

@hboutemy , do you know if Sonatype OSSRH is prepared to handle .sigstore.json (and .sigstore.jsonl) without .sigstore.json.asc?
Frankly speaking, I am not sure we need .sigstore.jsonl for Java-like use cases yet, however, the spec suggests two formats.

Yeah for .sigstore.jdon I think Joel is working on it

this is superseded by 0.7.0 release


Update to Swift 5.2 / Vapor 4.0.0

Unfortunately the checks are failing because they're using Swift 5.1 - @fabianfett will need to update the Github Action, I think.

@alfavata thanks so much for this. Would you mind changing the GitHub action to swift 5.2?
The file is located in .github/workflows/ci.yaml

Great, I will test this later today to give my final go and merge. Thanks, so much!

Dropped in favor of #4


Tree height and width are f32 but deserialised as Option

Tree::height and Tree::width are f32 but are trying to be deserialised as Option<f32> resulting in an error
https://github.com/rerun-io/egui_tiles/blob/3e7f32435543660f6c763747bb73462d5d4aa579/src/tree.rs#L57-L62

relevant: https://github.com/emilk/egui/issues/5176

I think we need to revert

https://github.com/rerun-io/egui_tiles/pull/85

Either revert it or also handle the serialization the same way:
serde(serialize_with = "serialize_f32_infinity_as_null") for the fields and then:
#[cfg(feature = "serde")]
fn serialize_f32_infinity_as_null<S: serde::Serializer>(
    t: &f32,
    serializer: S,
) -> Result<S::Ok, S::Error> {
    if t.is_infinite() {
        serializer.serialize_none()
    } else {
        serializer.serialize_f32(*t)
    }
}

handle the serialization the same way

The deserialiser is expecting an Option<f32>, so you would need to serialise it as an Option<f32>. That is, you would need to use something like serializer.serialize_option(t) instead of serializer.serialize_f32(*t). However, this would result in a surprising incoherence between the struct fields as f32 and their serialisation as Option<f32>.

Can JSON just add support for +/-inf already 🙄

Wow, TIL JSON doesn't have inf and NaN support.
I'm using RON in my projects, so I was blissfully ignorant of the JSON problem.


Small edits to JOSS manuscript

Ref: https://github.com/openjournals/joss-reviews/issues/6667

Looks good to me as well.  Merging now.


Missing/Wrong Command Duo To Invalid Commands On The File.

While Updating The Tool I Forgot To Add The New Commands Including The Old Ones Without Spaces. That Disallow The User From Using The Shell. If You're Using This Version Of The Tool Please Update.

Fixed.


Unable to run yarn lint for this repo on Node 10

Hi!
Before opening PR #2522, I was hoping to check my changes locally using yarn lint.
However trying to install the dependencies under Node 10 failed:
webpack.js.org $ yarn
yarn install v1.10.0
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
info<EMAIL_ADDRESS>The platform "win32" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=4 <=9". Got "10.10.0"
error Found incompatible module

The latest version of upath supports Node 10, and is in the semver range of the dependency on it in this repo, so I guess yarn.lock just needs refreshing?
Fwiw, we've had great results using https://renovatebot.com to automate lockfile refreshing/dependency updates (it's also much less noisy than Greenkeeper and has superior lockfile support).

Closed as it's no longer valid.


Steam Recording Bug

Describe the bug
Hi, I noticed a big with the recording on steam on a LCD steam Deck.
When strat using de steam + A button it will record but when you press it again to stop the controller stop working and bazzite it self restart.
What did you expect to happen?
I expected that the on and off recording function as intended.
Output of rpm-ostree status

Hardware
No response
Extra information or context
No response

Seems similar to #1862, perhaps you can try switch to Steam client beta and see if the issue goes away.


server setup + first route

Issue Type
☐ Bug
☑ Feature
☐ Tech Debt
Description

Set up node express server in server.ts
Created first route and first middleware (realized get all users)

Steps to Validate Feature

Run npm run dev to start back end
postman GET localhost:3001/api/getusers, see response with 1 dummy data row.

Previous behavior
N/A
Expected behavior
Follow steps
Screenshots & Videos
N/A
Additional context
Will communicate with front end developers and add more routes

Need to npm install
Need to use .env file with POSTGRES_URI
Confirmed that it works.

GET http://localhost:3001/api/getusers HTTP/1.1
HTTP/1.1 200 OK
X-Powered-By: Express
Content-Type: application/json; charset=utf-8
Content-Length: 125
ETag: W/"7d-ssz70QPWWM7RTPgQRS5QldX96qE"
Date: Fri, 06 Oct 2023 02:41:16 GMT
Connection: close
[
{
"userid"
"username"
"fn"
"ln"
"email":
"password"
}
]


Changing message frequency

The players on my server are complaining that the bot posts update message (the ones that say how much money was gained or lose, and the playtime of the savegame) too frequently. I took a brief look through the source but couldn't determine where this value was set. Could you please point me in the correct direction?

Hi @sigvaldr,
The messages related to the savegame (money and play time) are posted whenever a change is detected. Therefore, the frequency of those messages are determined by the 'Save Interval' setting of your dedicated server. You could try increasing this value in your server's settings.
The FS22_BOT_POLL_INTERVAL_MINUTES setting for this bot determines how often it checks for changes. However, this would typically be a low value.


Provide es6 modules

This is a request for an es6 module version of the project.
Generally I'd suggest simple running rollup with the rollup-plugin-commonjs which converts many legacy repos. Alas this does not work here. There is another issue request for CJS format, and if that is done, then rollup-plugin-commonjs would work.
Just to be clear, and I've guided other repos thru the process, the simplest approach is:

Convert to modern es6+ or at least es6 import/export
Use Rollup to convert that to UMD or IIFE formats

In other words, the root design is es6 modules, converting to any other format with Rollup. It Just Works.

Yup, this is a huge goal of mine going forwards, especially as I want to use Bliss in ESM projects of mine!


Cow: Watch whimsihub.com

Cow requests the watch of the watch_keyword whimsihub\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

whimsihub\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Nick in Charcoal HQ


Add petting interaction to bots

About the PR
Added a petting interaction to honkbots, cleanbots, and medibots. Audio for honkbots is a honk, for cleanbots and medibots a beep.
Closes #10093. Thanks for the walkthrough.
Changelog
:cl: Anthemic

add: You can now pet bots.

good PR, lgtm

@jproads Your branch got deleted, not sure why.


VS Code 1.75 cannot run interactive Python

With the new version:
Version: 1.75.0
Commit: e2816fe719a4026ffa1ee0189dc89bdfdbafb164
Date: 2023-02-01T15:29:17.766Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.1.8-200.fc37.x86_64
Sandboxed: No

VS Code cannot run Python interactively. Please, see the attached video.
https://user-images.githubusercontent.com/75945439/216589875-9dd7f7da-44ee-4c8d-a83d-f9e611857cbf.mp4

By using a newly created .ipynb file, everything seems to work fine.


Some requests that should generate 404s generate 500s ("template not found")

500 error at /3/favicon.ico

File 'page.mustache' not found in dirs: ['github_tools/templates', 'shared_web/templates']

Reported on github_tools by logged_out

            --------------------------------------------------------------------------------
            Request Method: GET
            Path: /3/favicon.ico?
            Cookies: {}
            Endpoint: None
            View Args: None
            Person: logged_out
            Referrer: None
            Request Data: {}
Host: localhost:5002
Accept-Encoding: gzip
Cf-Ipcountry: CN
X-Forwarded-For: <IP_ADDRESS>, <IP_ADDRESS>
Cf-Ray: 487644131fa49312-SJC
X-Forwarded-Proto: https
Cf-Visitor: {"scheme":"https"}
Accept: */*
User-Agent: Mozilla/5.0 (Windows NT 6.1; Trident/7.0; rv:11.0) like Gecko
Cf-Connecting-Ip: <IP_ADDRESS>
X-Forwarded-Host: gh.pennydreadfulmagic.com
X-Forwarded-Server: gh.pennydreadfulmagic.com
Connection: Keep-Alive
Labels: github_tools; TemplateNotFoundError

Hm, this was on someone's localhost. Closing.

500 error at /adminer.php

File 'page.mustache' not found in dirs: ['github_tools/templates', 'shared_web/templates']

Reported on github_tools by logged_out

            --------------------------------------------------------------------------------
            Request Method: GET
            Path: /adminer.php?
            Cookies: {}
            Endpoint: None
            View Args: None
            Person: logged_out
            Referrer: https://gh.pennydreadfulmagic.com/adminer.php
            Request Data: {}
Host: localhost:5002
Accept-Encoding: gzip
Cf-Ipcountry: US
X-Forwarded-For: <IP_ADDRESS>, <IP_ADDRESS>
Cf-Ray: 48ace61eb804bb1e-SEA
X-Forwarded-Proto: https
Cf-Visitor: {"scheme":"https"}
Referer: https://gh.pennydreadfulmagic.com/adminer.php
User-Agent: Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1)
Cf-Connecting-Ip: <IP_ADDRESS>
X-Forwarded-Host: gh.pennydreadfulmagic.com
X-Forwarded-Server: gh.pennydreadfulmagic.com
Connection: Keep-Alive
Labels: github_tools; TemplateNotFoundError

On gh also.

This is gh.pennydreadfulmagic.com specific. The main pdm site does the right thing.

These still trickling in all the time.

Still a thing on github_tools and (I think) modo_bugs instances.

Ah, so that's what it was.

Still seeing this (or some variation of this) on gh.pennydreadfulmagic.com via /stiemap.xml, /wp-login.php

I guess this is long enough ago it's not the same issue.


Question about timeouts and disabling buttons

Hi, so I have a leaderboard command that shows a leaderboard of all the users in my database.
Is it OK to not use a timeout or to not disable the buttons at all so that people can check the leaderboard whenever they want?
Does it cause any issues? I have commented out these options and it works just fine, I just wanted to know if that's OK.
    await pagination({
      embeds: leaderboardEmbeds /** Array of embeds objects */,
      interaction: interaction,
      //ephemeral: true,
      //time: 40000 /** 40 seconds */,
      //disableButtons: true /** Remove buttons after timeout */,
      fastSkip: false,
      pageTravel: false,
      buttons: [
        {
          type: ButtonTypes.previous,
          label: "Previous Page",
          style: ButtonStyles.Primary,
        },
        {
          type: ButtonTypes.next,
          label: "Next Page",
          style: ButtonStyles.Success,
        },
      ],
    });

Hi, yes, you can disable time etc. But note this:
If the bot stop / restart for X or Y reason, the message will not work anymore, and you will have to retype the command to create your leaderboard.


Allow dynamic configuration of security list management via annotations

Users of the CCM can choose to disable or modify the behaviour of security list management in configuration. For customers running in a management environment, they will not be able to modify the configuration of the CCM.
It may be the case that some teams will want to manage security lists manually and some dynamically within the same cluster. It would therefore be useful to allow this level of customisation via service annotations.

Also referenced in #227


Digital miner won't silk touch ores!

Whenever I set my digital miner to silk touch with redstone and diamonds as a filter, it has 8 speed upgrades, and when I try to start it, all the screens remain black, it tells me the remaining amount to mine, and then doesn't do anything.
Steps to reproduce:

Got digital miner
Added filters of diamond and redstone.
Put silk touch on, and then nothing happened.

I am on all the newest versions of Mekanism on 1.7.10 (I just downloaded it from the website) so it shouldn't be a version problem.

with that many speed upgrades you cant feed it energy fast enough. start with 1 upgrade and increase till it  stops. this will tell you how many you can safely add.

Just removed ALL of my speed upgrades, and it is working now. I thought an Ultimate would work for this kind of immense energy, but I guess not. My issue is solved, thank you.

You could also try to add some energy upgrades to it, it does help a lot with large energy demands


Hidden test case for level 2/3

Summary
Issue with hidden test case using current implementation
Does anybody have an idea what the hidden test case is at the end of level 2? Seem to have used up all the resources on one of the test cases before proceeding to level 3.
Screenshots (if any):

hi u might want to check #281, testing locally helped me understand what caused the error

Hi @Noel-Lee, think somebody experienced something similar to your case in a previous thread as mentioned by @ryuniqlo.
Hope this helps!


setOnTouchListener + Scrolling not work

Hi Friend
I need to return to another Activity in the application when there is no user action.
PDFView pdfView = (PDFView) findViewById(R.id.info);

       pdfView.setOnTouchListener(new View.OnTouchListener() {
           @Override public boolean onTouch(View view, MotionEvent motionEvent) {
               timer.cancel();
               timer = null;

               timer = new Timer();
               timer.schedule(new UpdateTimeTask(), 8000);
               return true;
           }
       });

       pdfView.fromAsset("info" + file + ".pdf").onRender(new OnRenderListener() {
           @Override
           public void onInitiallyRendered(int pages, float pageWidth, float pageHeight) {
               pdfView.fitToWidth();
           }
       }).load();

The problem is that the scrolling stops working.
Is there any way to determine the touch of the screen and not break the scrolling?

Try using OnClickListener

This works for a single touch, but does not work for scrolling.

So there is .onPageScroll() shown in readme.

Thanks, it works!


More info on error when custom function not found

When running custom function, if the function is not found, the error is is KeyError.

Provide a more meaningful error:
custom function {name_here} not found.

More meaningful error:
.
.
 elif wrangle.split('.')[0] == 'custom':
                # Execute a user's custom function
                try:
                    custom_function = functions[wrangle[7:]]
                except:
                    raise ValueError(f'Custom Wrangle function: "{wrangle}" not found')


Archive (Rsync) on non-standard SSH Port

Hello,
I am new to plotman (are'nt we all?) and I'm looking a way to make the archive to work on a non-standard port for SSH.
It's very common security precaution to change the SSH port so I would love not to expose my farmer's 22 port to the internet.
Rsync work with -e "ssh -p 1111" parameter where 1111 is the SSH port.
I tried to find any answer here, but I onlt found about rsync port 12000 on #169
Thank you,
and happy ploting :)
Alex

That's a good idea, but unfortunately not currently configurable, but we do have issue #6 to make this better.  For now your only option is probably to go into the code and hack in the -p parameter where the code calls ssh.


Several prominent Drupal8 modules require php-bcmath

Commerce suite (including address field) requires php bcmath.  We should consider adding it to our php modules installed.

I don't see a reason why we could not add this. I think one of our own country sites already requires php-bcmath.

I will try to make a PR today.


New events to allow extension.

Replaces #6
Fixes #5
It adds two new events:

winter.sitemap.processMenuItems
Allow processing the existing results, allowing to modify/replace/remove existing items

winter.sitemap.makeUrlSet
Allow injecting / replacing properties to the URLSet

It also provides the full itemInfo data to the winter.sitemap.makeUrlElement event, allowing more flexibility (It's going to be used by Winter.Translate to add the rel=alernate links to each Url elements)
Used by https://github.com/wintercms/wn-translate-plugin/pull/56
Credits to @LukeTowers and @AIC-BV for their work on this.

Should solve https://github.com/wintercms/wn-sitemap-plugin/issues/5

Note that when adding your own custom plugin to the sitemap, if you are not using winter.sitemap.processMenuItems, but pages.menuitem.resolveItem instead, you will have to adjust your function to support ML sitemap like this for example
public static function resolveMenuItem($item, $url, $theme)
    {

        $result = null;
        $locales = Locale::listEnabled();

        if ($item->type == 'blog-x-detail') {
            if (!$item->reference || !$item->cmsPage) {
                return;
            }

            $record = self::find($item->reference);
            if (!$record) {
                return;
            }

            $alternateLinks = [];
            foreach ($locales as $locale => $name) {
                if ($pageUrl = static::getMLPageUrl($item->cmsPage, $record, $theme, $locale)) {
                    $alternateLinks[$locale] = Url::to($pageUrl);
                }
            }

            foreach ($alternateLinks as $locale => $link) {
                $result[] = [
                    'url' => $link,
                    'mtime' => $record->updated_at,
                    'alternateLinks' => $alternateLinks,
                ];
            }

        } elseif ($item->type == 'blog-x') {

            $records = self::orderBy('title')->get();
            foreach ($records as $record) {
                $alternateLinks = [];
                foreach ($locales as $locale => $name) {
                    if ($pageUrl = static::getMLPageUrl($item->cmsPage, $record, $theme, $locale)) {
                        $alternateLinks[$locale] = Url::to($pageUrl);
                    }
                }

                foreach ($alternateLinks as $locale => $link) {
                    $result['items'][] = [
                        'url' => $link,
                        'mtime' => $record->updated_at,
                        'alternateLinks' => $alternateLinks,
                    ];
                }
            }

        }

It's not hurting, and it's only triggered once per page request.

@mjauvin what's left to do on this and the related PRs? @LukeTowers had a comment here.

@bennothommo As we discussed with you regarding the halting event, this won't be possible since we'll diverge from the way the StaticMenu does it (without halting).
So I would go forward with the current PR, adding a new event to handle these.


Flag for enforcing explicit 'return' keyword

Is there a flag that would cause rustfmt to appropriately add an explicit return keyword at the end of functions?
Before:
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        return false;
    }

    lhs % rhs == 0
}

After:
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        return false;
    }

    return lhs % rhs == 0;
}

Diff:
diff --git a/example.rs b/example.rs
index 96b10a6..5af00c8 100644
--- a/example.rs
+++ b/example.rs
@@ -3,5 +3,5 @@ fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
         return false;
     }

-    lhs % rhs == 0
+    return lhs % rhs == 0;
 }
\ No newline at end of file

Thank you for the question @UebelAndre. No, there is no flag nor config option that will have rustfmt convert the last item to a return expression. Needless return expressions are generally considered to be bad style and non-idiomatic in Rust (there's even a Clippy lint that will warn on them).
If you prefer to use utilize explicit return expressions within your code then you can certainly do so, and rustfmt will still happily format them. However, IMHO rustfmt should not be in the business of trying to change these expression types as part of style formatting, particularly to one that will cause clippy to emit warnings.

@calebcartwright Thank you so much for your reply! 😃
I would agree that rustfmt should not start adding ways to fight clippy, however, I don't at all understand the known issue or the justification for why this is bad in the link you sent. Could you elaborate on those to help me understand this better?

My understanding is that elision of the return keyword is preferred because rust is intended to be more of an expression-based than statement-based language. For example, fn foo() { 1 } evaluates to the expression 1, and while fn foo() { return 1; } does the same thing in practice, to a reader, the latter example may read as the last statement of foo instructing the return operator to pass 1 to the caller, which feels a bit different than an expression evaluating to something.
This convention comes from more functional languages where return doesn't make much sense to begin with because every function evaluates to something (even if it is the unit type, or () in rust).
The rust book has some more insight on statements vs expressions: https://doc.rust-lang.org/reference/statements-and-expressions.html.

I guess there's a question of whether rustfmt should do the reverse, ie convert terminating explicit returns into expressions.

@UebelAndre

I don't at all understand the known issue or the justification for why this is bad in the link you sent

Yeah the lint description there isn't exactly detailed is it :laughing: As @ayazhafiz noted, it really is more about convention, and like with any convention, there's not always explicit origin story or obvious explanation, but conventions arise nevertheless. The explanation @ayazhafiz shared is similar to what I've heard before elsewhere, and I've also found the avoidance of return expressions to be helpful in other places, like closures.

@ayazhafiz

I guess there's a question of whether rustfmt should do the reverse, ie convert terminating explicit returns into expressions.

TBH I wouldn't be too keen on trying to take that on for the same reason as the requested inverse in the OP, and I'd rather defer this to the developer. Obviously different story if our hand gets forced by the style guide or it becomes a popular ask from the community, but for now it just feels like it'd be another vector for bugs.

The explanation @ayazhafiz shared is similar to what I've heard before elsewhere, and I've also found the avoidance of return expressions to be helpful in other places, like closures.

I feel like the intention of the implicit return is great but the execution could be improved. There should be a way to write code that doesn't require the use of the return keyword for control flow as well. The example in the rust documentation
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        return false;
    }

    lhs % rhs == 0;
}

The mix of a return keyword and the implicit return from the expression make it unclear as to how to read this. I'm most interested in making sure my code is consistent and readable so if there was a smarter way to handle control flow that avoids this situation then I'd love to learn about it. I otherwise feel adding return at the end is the nicer thing to do.

I feel like the intention of the implicit return is great but the execution could be improved. There should be a way to write code that doesn't require the use of the return keyword for control flow as well.

Perhaps I am misunderstanding you here, but I feel like this is quickly getting off topic and really outside the scope of rustfmt anyway. Requests/suggestions for changes to the language itself need to be made elsewhere. rustfmt is just the code formatting tool.

The example in the rust documentation

fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        return false;
    }

    lhs % rhs == 0;
}

Please take note that your snippet here is not the same as the book reference (you've added a semicolon to the last line making it a statement), and it actually will not compile because that path doesn't return a boolean value.
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=05b76e95debd0932b5bdfa558b28bddb

The mix of a return keyword and the implicit return from the expression make it unclear as to how to read this. I'm most interested in making sure my code is consistent and readable so if there was a smarter way to handle control flow that avoids this situation then I'd love to learn about it. I otherwise feel adding return at the end is the nicer thing to do.

Again, not really sure what to make of this considering your snippet is invalid :smile: That trivial example from the book is simply illustrative, showing both usage of an early return as well as last-line expressions for implicit returns.
There's a lot of alternative ways you could choose to write this example from the book, for example:
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        false
    } else {
        lhs % rhs == 0
    }
}

or
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        return false;
    }

    return lhs % rhs == 0;
}

You're welcome to use any approach you prefer, but as discussed above, rustfmt isn't going to convert your code from one of those approaches to another; rustfmt will ensure that, whichever approach you prefer and use, it follows the formatting/style established by the Style Guide.

Again, not really sure what to make of this considering your snippet is invalid

My apologies, this is a typo, I copied the wrong snippet above and forgot to remove that but it looks like you understood what I was getting at with that.

Perhaps I am misunderstanding you here, but I feel like this is quickly getting off topic and really outside the scope of rustfmt anyway. Requests/suggestions for changes to the language itself need to be made elsewhere. rustfmt is just the code formatting tool.

This has shifted toward a discussion about the language and I can happily move the conversation. Before opening this ticket, it was unclear to me that the implicit return was "enforced" by clippy, which to me means the conversation should be at least be moved over there and I think this is reason enough to close this issue. However, if it weren't the case that this is "enforced" in clippy, I do feel it would be appropriate for rustfmt to provide this functionality.

Before opening this ticket, it was unclear to me that the implicit return was "enforced" by clippy

A clippy warning does not qualify as enforcement. As noted above, it's convention and considered to be idiomatic to use an expression over an explicit return expression statement.

However, if it weren't the case that this is "enforced" in clippy, I do feel it would be appropriate for rustfmt to provide this functionality

I understand your ask, but I just disagree. rustfmt's job is to ensure that any two programs parsed to the same AST are formatted the same way, in accordance with the style guide. It's not a design goal of rustfmt to support changing the AST structure while maintaining program logic.
IMHO, rustfmt should not support allowing the conversion of last-line-in-block expressions to an explicit return expression statement because it is out of scope, the cons of effort/code/etc. outweigh any user preference, and the output from rustfmt would be flagged by clippy.
I'm inclined to close this as wontfix but cc @topecongiro for your thoughts in case you disagree

A clippy warning does not qualify as enforcement

This is for lack of a better word. It does appear that clippy seems to have more authority over this decision so any decision making would be deferred to the decisions made there. If that makes more sense.

I understand your ask, but I just disagree. rustfmt's job is to ensure that any two programs parsed to the same AST are formatted the same way, in accordance with the style guide.

It does seem you understand my ask but I think I fail to communicate that I also understand your responses. I completely agree with your stance on the responsibility of rustfmt. This simply isn't the place to discuss the language or style guide. I just want it to be clear that if this gets flagged with wontfix that it's because of the decisions being made in clippy, something specified in the style guide, or the engineering overhead by supporting this feature.

This is for lack of a better word. It does appear that clippy seems to have more authority over this decision so any decision making would be deferred to the decisions made there. If that makes more sense.

I really want to stress that there's not anywhere to go if you're looking for something that will rewrite your code from:
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        return false;
    }

    lhs % rhs == 0
}

To a functionally equivalent alternative like:
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        false
    } else {
        lhs % rhs == 0
    }
}

// Or
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    if rhs == 0 {
        return false;
    }

    return lhs % rhs == 0;
}

// Or
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    match rhs {
        0 => false,
        _ => lhs % rhs == 0,
    }
}

// Or...

because which features of the language to use to implement logic is really left to the discretion of the programmer.
Conventions/best practices/official approaches/etc. often stem from collective experience and wisdom around what the experts and community at large have found to be optimal, and how to best take advantage of the features provided by the language. It's stylistic in nature, but explicit return expression statements are similar here because they're redundant and don't take advantage of the expression oriented nature of the language (you may find https://users.rust-lang.org/t/explicit-and-implicit-return-in-functions/17089/5 helpful as well).
There's other common/preferred stylistic conventions such as if let. For a highly contrived example, while you could write something like this:
fn greet(user: Option<&str>) {
    if user.is_some() {
        let name = user.unwrap();
        println!("Greetings {}", name);
    } else {
        println!("Hello world!");
    }
}

It's more conventional to leverage if let (or perhaps a match), like:
fn greet(user: Option<&str>) {
    if let Some(name) = user {
        println!("Greetings {}", name);
    } else {
        println!("Hello world!");
    }
}

Similarly to how you seem to prefer the explicit return expression statements, there's probably someone out there who stylistically would prefer the is_some()/unwrap() combo from the above example, and that's fine!
Conventions are just that, and folks can (and do) go against the conventions sometimes. However, you probably shouldn't be surprised that the official tools won't help you enforce anti-convention nor convert convention-following code to non-convention-following code.
All that being said, if you'd like more info (or to challenge) clippy's needless return lint then the clippy repo (https://github.com/rust-lang/rust-clippy) would indeed be the place to do so.
Lastly, if you'd like to suggest changes to Rust keywords/syntax/constructs/etc., then you'd need to follow the RFC process https://github.com/rust-lang/rfcs.

I just want it to be clear that if this gets flagged with wontfix that it's because of the decisions being made in clippy, something specified in the style guide, or the engineering overhead by supporting this feature.

No, that's not what I said. What I said is that you're welcome to write in whichever style you prefer and rustfmt will format either, but that I view this type of conversion of the input code as out of scope.
Additionally, as noted above, I don't think official tools should invest in support for converting from convention to anti-conventions, but your ask here is for rustfmt to do just that. The fact that clippy (the linting tool) warns about this is just another point of evidence that your ask is for something that goes against convention; clippy doesn't have any decision making authority over rustfmt and vice versa.
Lastly, even in the absence of all the above issues, although rustfmt could probably implement the ask from a technical  perspective, IMO it would add undo overhead to the rustfmt source and I suspect would likely be a source of bugs and idempotency issues in "real" code scenarios.

I really want to stress that there's not anywhere to go if you're looking for something that will rewrite your code from:

Yeah, this was not what I was request.

To a functionally equivalent alternative like:

But the alternatives you give here are awesome and satisfy my desire for the return keyword. In especially like:
fn is_divisible_by(lhs: u32, rhs: u32) -> bool {
    match rhs {
        0 => false,
        _ => lhs % rhs == 0,
    }
}

So thank you so much for all the info and taking the time to explain everything to me 😄

So thank you so much for all the info and taking the time to explain everything to me smile

Sure! Sounds like we've come to a logical resolution on this, so going to go ahead and close this issue.


Added profile

Added my profile

Thanks for your contribution!

Please add hacktoberfest-accepted label otherwise it won't be considered.
On Wed, 12 Oct, 2022, 11:33 Ankita Malik, @.***> wrote:

Thanks for your contribution!
—
Reply to this email directly, view it on GitHub
https://github.com/AnkitaMalik22/devmate/pull/6#issuecomment-1275635517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXGFWQ2MQPPBXJF4MTYDCRTWCZIEXANCNFSM6AAAAAARCQUJEY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Please add hacktoberfest-accepted label otherwise it won't be considered.
…
On Wed, 12 Oct, 2022, 11:33 Ankita Malik, @.> wrote: Thanks for your contribution! — Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXGFWQ2MQPPBXJF4MTYDCRTWCZIEXANCNFSM6AAAAAARCQUJEY . You are receiving this because you authored the thread.Message ID: @.>

Please add hacktoberfest-accepted label otherwise it won't be considered.
…
On Wed, 12 Oct, 2022, 11:33 Ankita Malik, @.> wrote: Thanks for your contribution! — Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXGFWQ2MQPPBXJF4MTYDCRTWCZIEXANCNFSM6AAAAAARCQUJEY . You are receiving this because you authored the thread.Message ID: @.>

already did!

Please add hacktoberfest-accepted label otherwise it won't be considered.
…
On Wed, 12 Oct, 2022, 11:33 Ankita Malik, @.> wrote: Thanks for your contribution! — Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXGFWQ2MQPPBXJF4MTYDCRTWCZIEXANCNFSM6AAAAAARCQUJEY . You are receiving this because you authored the thread.Message ID: @.>

already did! please check

Yess, thankyou! :)
On Wed, 12 Oct, 2022, 11:56 Ankita Malik, @.***> wrote:

Please add hacktoberfest-accepted label otherwise it won't be considered.
… <#m_-9136392864202813294_>
On Wed, 12 Oct, 2022, 11:33 Ankita Malik, @.> wrote: Thanks for your
contribution! — Reply to this email directly, view it on GitHub <#6
(comment)
https://github.com/AnkitaMalik22/devmate/pull/6#issuecomment-1275635517>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXGFWQ2MQPPBXJF4MTYDCRTWCZIEXANCNFSM6AAAAAARCQUJEY
https://github.com/notifications/unsubscribe-auth/AXGFWQ2MQPPBXJF4MTYDCRTWCZIEXANCNFSM6AAAAAARCQUJEY
. You are receiving this because you authored the thread.Message ID: @.>
already did! please check
—
Reply to this email directly, view it on GitHub
https://github.com/AnkitaMalik22/devmate/pull/6#issuecomment-1275654155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXGFWQ2ENXXJ7ICXYE5XYRTWCZKXXANCNFSM6AAAAAARCQUJEY
.
You are receiving this because you authored the thread.Message ID:
@.***>


[EventRequest] OnBeforeLookupGLAccountFilter in Table 841 "Cash Flow Account"

Hello, please consider adding the following event to Table 841 "Cash Flow Account":

[IntegrationEvent(false, false)]
procedure OnBeforeLookUpGLAccountFilter(var Rec: record "Cash Flow Account"; var IsHandled: Boolean)
begin
end;

and raise it accordingly on field 37 "G/L Account Filter" :

        trigger OnLookup()
        var
            GLAccList: Page "G/L Account List";
            IsHandled: Boolean;

        begin
            IsHandled := false;
            OnBeforeLookUpGLAccountFilter(Rec, IsHandled);
            if IsHandled then
                exit;

            GLAccList.LookupMode(true);
            if GLAccList.RunModal = ACTION::LookupOK then
                "G/L Account Filter" := CopyStr(GLAccList.GetSelectionFilter, 1, MaxStrLen("G/L Account Filter"));
        end;

Thank you!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Serialization -> deserialization: total count ends up wrong

Not sure what the cause is yet, but further testing with random values and random counts is failing to deserialize into exactly the same total count as what was serialized.

It looks like this issue is correct behavior, in some cases at least: if an individual value's count has saturated, the total count might not have (especially with smaller value count types than u64). When deserialized, the total count is calculated by summing the value counts, so it would end up smaller than the original total count.
I'll close this once (if) I'm convinced it also addresses the u64 cases.

u64 tests had issues with accumulated counts exceeding i64::max_value() and therefore not being serializable, so this is resolved.


Added tests for orders redux action creators.

Some notes:
Seems to that creating order is not an atomic operation on API side. Do we have some scenario what happens when some stage of this process fails?

Codecov Report

Merging #95 into master will increase coverage by 2.38%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
+ Coverage   12.54%   14.93%   +2.38%
==========================================
  Files          66       66
  Lines         869      864       -5
  Branches       37       36       -1
==========================================
+ Hits          109      129      +20
+ Misses        729      705      -24
+ Partials       31       30       -1

Impacted Files
Coverage Δ

src/redux/Actions/OrderActions.js
100% <ø> (+100%)
:arrow_up:

src/Helpers/Order/OrderHelper.js
100% <100%> (+100%)
:arrow_up:

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 316cba4...a246e36. Read the comment docs.


Tracker certificate expired

The tracker wss://tracker.btorrent.xyz SSL certificate has expired!
Time to renew it ;)

Done. thanks.


clean up whitespace in Makefile

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "clean-up-whitespace" git@github.com:friism/infrakit.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Current coverage is 70.35% (diff: 100%)

Merging #286 into master will not change coverage

@@             master       #286   diff @@
==========================================
  Files            25         25
  Lines          1366       1366
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            961        961
  Misses          326        326
  Partials         79         79

Powered by Codecov. Last update 4ddefd3...18d50c7


module can't work with mobile-app metadata

For adding metadata to the report we can add the following for browsers:
metadata:{
    browser: {
        name: 'chrome',
        version: '60'
    },
    device: 'Local test machine',
    platform: {
        name: 'ubuntu',
        version: '16.04'
    }
}

Fore native / hybrid apps you don't have a browser. If you don't provide the browser-objectthe module will fail in rendering the templates.
Solution:
OR the browser-object or the app-object needs to be optional. The app-object can be something like this
app:{
    name: 'app-name',
    version: 'xx.xx'
}

Also:

add an icon
specs
gh-pages

Fixed this in version 1.2.0


When name is provided it should be used to make the images accessible

When a name is provided to the ngx-avatar-component, I would like it to use the name as an alt-text on the img to make it accessible.

It makes sense. I will add this into the component

Is this already in latest version or it's yet to be updated


Fix errors and warning when compiling with iojs

Heh, I also created this pr =)
Are you think that use v8::Number is more reasonable than strict converting to int32_t?


including instructions for Capistrano deployment in README.md

See issue #319

merged in https://github.com/thoughtbot/ember-cli-rails/commit/c8c1fd17a77aa2314a7d744d6252ffce7f123c27.
Thanks @dchersey!


spdycat does not send a Host header

spdycat is not sending a Host header. This causes some SPDY servers to send back a 400 (Bad Request).

OK, added host header field in 45376c6b

I tested it, and it works. Thank you Tatsuhiro.


Should we change camelCase variables to snake_case to fit the Godot style guide?

As much as I love camelCase (guilty), snake_case is Godot's standard so maybe it's worth it to change it.

Yes

+1


Backdrop?

Is there a way to create a backdrop?

The documentation site uses a more site specific approach;
http://github.hubspot.com/shepherd/docs/welcome/
.hero-outer .hero-inner > * {
    transition: opacity 0.4s;
    opacity: 0.3;
    pointer-events: none;
}
.hero-outer .hero-inner .shepherd-target.shepherd-enabled {
    opacity: 1;
    pointer-events: auto;
}

Not the most reproducible approach on a large scale, but it does give a means to reproduce if you required it. I'd like to have this feature soon myself.

+1

Unfortunately, this is hard to include as a core feature. It has been requested a few times, but there are a lot of edge cases, and it is more reliable to do the modal implementation in your app yourself.


The existence of SetupAdminUser user

Should SetupAdminUser user account be created on the VMs of service fabric hosted on Azure?

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

ID: 3ac102a5-e9ad-0046-f217-446f1ad263c5
Version Independent ID: a47523e2-0c30-dee4-0037-5f58836fa37f
Content: Add an HTTPS endpoint using Kestrel to a Service Fabric app in Azure
Content Source: articles/service-fabric/service-fabric-tutorial-dotnet-app-enable-https-endpoint.md
Service: service-fabric
GitHub Login: @rwike77
Microsoft Alias: ryanwi

Thanks for the feedback! We are currently investigating and will update you shortly.

Let me rephrase my question. As I understood, the mentioned admin account should be added to the administrator's group on each node of the service fabric. How can this be done in a batch or per node? We cannot RDP to all machines one by one and add this user to the group. Please elaborate the method in more automated way.

@Arash-Sabet  I think we can use extensions  to do that job.
In this case, if we add an extension to create an admin user, Then extension has to be executed first and the the service fabric application should be started. I will check about this order and let you know.
cc: @MicahMcKittrick-MSFT

@rwike77 and @aljo-microsoft to comment further.

@Arash-Sabet
That referenced tutorial document articulates this:
 <Principals>
    <Users>
      <User Name="SetupAdminUser">
        <MemberOf>
          <SystemGroup Name="Administrators" />
        </MemberOf>
      </User>
    </Users>
  </Principals>

E.G. Your Application Model (Manifests) declare the principals your SF runtime will provision,
and further the How To content is located:
https://docs.microsoft.com/azure/service-fabric/service-fabric-run-script-at-service-startup
https://docs.microsoft.com/azure/service-fabric/service-fabric-application-runas-security
@MicahMcKittrick-MSFT
Please close this.


Document gaps in SCIM docs

Is your suggestion related to a missing or misleading document? Please describe.
In addition to the changes described in [1], document the following:

Bulk role creation, update and remove

[1] https://github.com/wso2/product-is/issues/12127

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


build issue aws amplify

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v16.19.0
Amplify CLI Version
10.8.1
What operating system are you using?
Windows
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
None made
Describe the bug
I run into a build script issue. I understand that it is not in my scripts under my package. However, I do not know what to put into the script. I just want it to run my api.
Expected behavior
run amplify publish:
publish to frontend (yes) and have it run
Reproduction steps
amplify init

Enter <
Yes
AWS Access Keys (keys added)
US-east-2
Yes

amplify add api

REST
Enter <
Enter <
Enter <
NodeJS
Serverless ExpressJS function (Integration with API Gateway)
No
No
No
No

amplify push

Yes

amplify hosting add

Amazon CloudFront and S3
DEV (S3 only with HTTP)
Enter <
Enter <
Enter <

amplify publish

Yes

ERROR appears

package:

I check status with
amplify status

the REST API Endpoint has this error

and the Hosting Endpoint:

Project Identifier
Project Identifier: 6af7648d47226c99e6855efb189d959c
Log output

# Put your logs below this line
0 verbose cli /usr/local/share/nvm/versions/node/v16.19.0/bin/node /home/codespace/nvm/current/bin/npm
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing npm:load:whichnode Completed in 0ms
4 timing config:load:defaults Completed in 1ms
5 timing config:load:file:/usr/local/share/nvm/versions/node/v16.19.0/lib/node_modules/npm/npmrc Completed in 1ms
6 timing config:load:builtin Completed in 1ms
7 timing config:load:cli Completed in 1ms
8 timing config:load:env Completed in 1ms
9 timing config:load:file:/workspaces/COP4934-SD-Project/.npmrc Completed in 1ms
10 timing config:load:project Completed in 1ms
11 timing config:load:file:/home/codespace/.npmrc Completed in 0ms
12 timing config:load:user Completed in 0ms
13 timing config:load:file:/usr/local/share/nvm/versions/node/v16.19.0/etc/npmrc Completed in 0ms
14 timing config:load:global Completed in 0ms
15 timing config:load:validate Completed in 0ms
16 timing config:load:credentials Completed in 12ms
17 timing config:load:setEnvs Completed in 1ms
18 timing config:load Completed in 18ms
19 timing npm:load:configload Completed in 19ms
20 timing npm:load:mkdirpcache Completed in 4ms
21 timing npm:load:mkdirplogs Completed in 1ms
22 verbose title npm run-script build
23 verbose argv "run-script" "build"
24 timing npm:load:setTitle Completed in 2ms
25 timing config:load:flatten Completed in 3ms
26 timing npm:load:display Completed in 6ms
27 verbose logfile logs-max:10 dir:/home/codespace/.npm/_logs
28 verbose logfile /home/codespace/.npm/_logs/2023-03-13T04_54_19_087Z-debug-0.log
29 timing npm:load:logFile Completed in 8ms
30 timing npm:load:timers Completed in 0ms
31 timing npm:load:configScope Completed in 0ms
32 timing npm:load Completed in 40ms
33 silly logfile start cleaning logs, removing 1 files
34 silly logfile done cleaning log files
35 timing command:run-script Completed in 5ms
36 verbose stack Error: Missing script: "build"
36 verbose stack
36 verbose stack To see a list of scripts, run:
36 verbose stack   npm run
36 verbose stack     at RunScript.run (/usr/local/share/nvm/versions/node/v16.19.0/lib/node_modules/npm/lib/commands/run-script.js:98:13)
36 verbose stack     at async module.exports (/usr/local/share/nvm/versions/node/v16.19.0/lib/node_modules/npm/lib/cli.js:78:5)
37 verbose cwd /workspaces/COP4934-SD-Project
38 verbose Linux 5.4.0-1104-azure
39 verbose node v16.19.0
40 verbose npm  v8.19.3
41 error Missing script: "build"
41 error
41 error To see a list of scripts, run:
41 error   npm run
42 verbose exit 1
43 timing npm Completed in 77ms
44 verbose code 1
45 error A complete log of this run can be found in:
45 error     /home/codespace/.npm/_logs/2023-03-13T04_54_19_087Z-debug-0.log

Additional information
I looked up several sources and was left with no solution. I have been at this for hours and I'm turning for assistance because I am stuck. I would immensely appreciate any help at all. I will follow suggests and provide information requested to help.
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @MatthewGladkowski, thank you for reaching out. wanted to get some information. were there any changes to the API gateway?
additionally, how is the API url being utilized. are you calling the api with the path? for example: https://<id>.execute-api.us-east-1.amazonaws.com/<env stage>/items.
on the API gateway console, are you able to run a test on the client in the resources under the path provided?

Hey @MatthewGladkowski, thank you for reaching out. wanted to get some information. were there any changes to the API gateway? additionally, how is the API url being utilized. are you calling the api with the path? for example: https://<id>.execute-api.us- ![image_2023-03-13_143227833](https://user-images.githubusercontent.com/115107906/224796858-6e1cd72e-958f-4594-805b-80a5e86aa2ac.png) east-1.amazonaws.com/<env stage>/items. on the API gateway console, are you able to run a test on the client in the resources under the path provided?

I changed the resources to match my api. using router names and get/put/etc. functions. i used the "hello world" preset for the home route (/items -> /routes).
and using the url you suggested (my REST API endpoint) did fix that issue

This is the layout for my urls

I execute (url)/routes/users/all and get:

The purpose of that function is to output my array of objects (user data)
this works fine with localhost but I am having issues with it here.
Thank you for the help with the url addition. It did help so that's progress and I really appreciate it :)

hey @MatthewGladkowski, from the error message it appears the Lambda function may be erroring out.
could you try using the test button as shown below to test the endpoint.

if the test fails with the same error, this would point that the Lambda function code may be erroring out. I would suggest checking the Lambda function logs or the log group in the monitoring tab for any errors provided by the function.
Could you provide us some additional information on your use case? are you calling the API from your frontend?
if yes, are you utilizing amplify libraries to call the API? Please refer to https://docs.amplify.aws/lib/restapi/fetch/q/platform/js/ providing information on this.

hey @MatthewGladkowski, unfortunately lambda function has the limits as follows. https://docs.aws.amazon.com/lambda/latest/dg/gettingstarted-limits.html

You may need to chunk the json payload and pass it the lambda function or may need to utilize a custom solution such as an S3 bucket to fetch the portion of data from storage bucket and process the data in the Lambda function.
I was able to find this article but as this is a 3rd part resource I would suggest testing your use case thoroughly before utilizing this in your prod if the solution meets your use case.
additionally, I would also suggest reaching out to the Lambda function support team for any additional information on the use cases.

hey @MatthewGladkowski, unfortunately lambda function has the limits as follows. https://docs.aws.amazon.com/lambda/latest/dg/gettingstarted-limits.html

You may need to chunk the json payload and pass it the lambda function or may need to utilize a custom solution such as an S3 bucket to fetch the portion of data from storage bucket and process the data in the Lambda function.
I was able to find this article but as this is a 3rd part resource I would suggest testing your use case thoroughly before utilizing this in your prod if the solution meets your use case.
additionally, I would also suggest reaching out to the Lambda function support team for any additional information on the use cases.

Thank you very much I will look into this. I believe I was able to successfully satisfy my original issue and I greatly appreciate your assistance as I was able to get my API working  :)


Remove Carbon Kernel dependencies from TestGrid

Description
Currently, we are using org.wso2.carbon.config.provider.ConfigProvider to load YAML configuration.
Shall we add YAML loading logic itself into TestGrid and remove the Carbon dependency?

Not an immediate requirement. :)


Add synonyms for combined studies

Context
Not all degrees have subjects, and some degrees have several subjects. Talking with Shaun, the correct option to pick in this case is Combined Studies.
This came up via support with a trainee that had a degree from the Open University where the subject is listed as open degree.

 Add synonyms for combined studies degree subject


Adding styles to Accordion

Hi,
I couldn't find any options to style Accordion. ngClass and ngStyle don't work. Am I missing something?

@nithyanv @OliPelz You just need to wrap your accordion heading and content in another element (div) and use the ng-class on that.  Something like so:
<accordion>
    <accordion-group ng-repeat="phenoGroup in exp.phenotypeGroups">
        <accordion-heading>

                {{phenoGroup.phenoName}} ({{phenoGroup.countPhenos}})

        </accordion-heading>

            Hello,hidden world

    </accordion-group>
</accordion>

Closing this for now.


[Event Request]Codeunit 5760, functions Code & InitSourceDocumentLines

Hi,
We need 2 new events in Codeunit 5760 Whse.-Post Receipt.

Function InitSourceDocumentLines

With parameters:

Function Code

With parameters:

Thanks in Advance

These events already exists


Support for plurals

What problem are you facing?
I never get the right command at first. Because I use projects, or devices and not project and device
How could the Equinix Metal CLI help solve this problem?
should we add support for plurals?

Probably I am using an old version! Thanks @displague !https://github.com/packethost/packet-cli/pull/60

I'm also interested in aliases for get and list, I can never remember which tool uses which verb.

And wouldn't it be nice if packet get devices worked too? If param[1] is one of the common actions (get,list,delete,update,create), would it be crazy to just swap the two parameters?
What if someone said packet get should it know that there are 12 resources that support get and offer those as help? Maybe.

packet ssh-keys should be added :-(


IPS alerting disabled rules

OPNsense 21.1.1 (amd64, OpenSSL).
I have several policies which disable several rules.
After fixing https://github.com/opnsense/core/issues/4695#issuecomment-777745841 they are properly disabled.
Wierd is that SOME of disabled rules (not all) ale still reported on alerts page, why?
Picture is self explanatory.
Both selected rules are NOT enabled, still there are reported.
Lot of other disabled rules are triggered and not alerted properly.

Thanks

accidentally patched the already working version? screenshot doesn't look like it belongs to 21.1.1

Followed only official upgrade path 20.1-20.7-21.1 etc + applied patch from your post https://github.com/opnsense/core/issues/4695#issuecomment-777345538

patch is already included in 21.1.1, adding it again will revert...

no its not :/
Right now I have reinstalled opnsense package and bug in https://github.com/opnsense/core/issues/4695#issuecomment-777345538 re-appeared.

checked again, patch is included in 21.1.1, which doesn't mean you don't have another issue, but applying the patch again certainly won't help.

I don't get it.
Im on 21.1.1.
I did audit health check, all files are ok.
Matching by classtype is not working as described in https://github.com/opnsense/core/issues/4695
So I ran opnsense-patch b465a41
| src/opnsense/scripts/suricata/lib/rulecache.py | 9 ++++++++-
| 1 file changed, 8 insertions(+), 1 deletion(-)
|
|diff --git a/src/opnsense/scripts/suricata/lib/rulecache.py b/src/opnsense/scripts/suricata/lib/rulecache.py
|index 85e02739e7..7bd3754aca 100755
|--- a/src/opnsense/scripts/suricata/lib/rulecache.py
|+++ b/src/opnsense/scripts/suricata/lib/rulecache.py
--------------------------
Patching file opnsense/scripts/suricata/lib/rulecache.py using Plan A...
Hunk #1 succeeded at 82.

Reappling policies. All are matching ok, even classtypes.
Audit healthcheck
Checking all packages: ....
opnsense-21.1.1: checksum mismatch for /usr/local/opnsense/scripts/suricata/lib/rulecache.py

I have downloaded these source codes and rulecache.py is without patch https://github.com/opnsense/core/releases/tag/21.1.1

@mnaiman ok, now I'm mixing two different patches, https://github.com/opnsense/core/commit/b465a418a0f9e17b7f73cc9dafb1cb4571cd6fb7 is indeed not included (should be in 21.1.2) the other patches are in. with the details provided I don't think we can track this further, sid 29456 seems to be some rule from snort, details about the environment might matter.... we offer issue templates to streamline the proces, it's probably better to close this issue and add a new one using our templates and proving more details about how one could reproduce your issue.


Remove 'else' before 'this.topojson = topojson'

This matches the current d3.js (as of 3.5.5) method of exporting the library.  I was running into trouble on GitHub's Atom-Shell (now called electron) when I tried to use topojson.  It would fail to import correctly by not creating a topojson object.  The error message was:
[22095:0503/213544:INFO:CONSOLE(370)] "Uncaught ReferenceError: topojson is not defined", source: file:///homes/rmason/code/menubar/example/index.html (370)

and was included in index.html as follows:
<script src="topojson.js"></script>

All of the tests still pass:
><EMAIL_ADDRESS>test /homes/rmason/code/topojson
> vows; echo

·· ·· ···· ·· ·· ····················· ·············· ···· ··················· ········· ······· ·· ········ ·········· ····· ········ ········ ··· ············ ······································ ······································· ······· ·········· ······ ························· ········································ ···················································· ··
  ✓ OK » 361 honored (5.680s)

But it may be helpful to add a test for this case.  I'm not exactly sure what that would entail though as it is hard to search google for this. =  and if it is always useful to do this and what the failure modes for it are.

:+1: Sounds good. Will merge into the next release.

Since 1.6.22, TopoJSON has been using Rollup to generate the UMD bundle.


ImportError: cannot import name '_RoIAlignFunction' from 'torchvision.ops.roi_align'

Newer versions of torchvision has rewritten the parts of the logic where RoIAlign is imported. As a result, the following error was reported in script masked_roi_align.py and box_pair_poolers.py
from torchvision.ops.roi_align import _RoIAlignFunction
ImportError: cannot import name '_RoIAlignFunction' from 'torchvision.ops.roi_align' (/home/fred/miniconda3/envs/pocket/lib/python3.8/site-packages/torchvision/ops/roi_align.py)

Since both scripts that used _RoIAlignFunction are no longer actively used, they have been removed.

Since both scripts that used _RoIAlignFunction are no longer actively used, they have been removed.


[SMALLFIX] remove @Ignore in metrics command integration test

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20432/
Test PASSed.

Merged build finished. Test PASSed.


Fixes #3380 - "Nearby Place found" despite already Nearby upload

Description (required)
Fixes #3380
What changes did you make and why?
Added a condition -> if the place is not null, then don't search for nearbyPlaces
Also if the user confirms a nearby found place then update the place property in uploadItem.
Tests performed (required)
Tested betaDebug on Motorola One Fusion + with API level 29.

Codecov Report

Merging #4408 (16b770b) into master (2b62d84) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #4408      +/-   ##
============================================
- Coverage     10.28%   10.28%   -0.01%
  Complexity      476      476
============================================
  Files           342      342
  Lines         13149    13154       +5
  Branches       1076     1076
============================================
  Hits           1353     1353
- Misses        11728    11733       +5
  Partials         68       68

Impacted Files
Coverage Δ
Complexity Δ

...in/java/fr/free/nrw/commons/upload/UploadItem.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...mons/upload/mediaDetails/UploadMediaPresenter.java
54.28% <0.00%> (-1.60%)
15.00 <0.00> (ø)

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 2b62d84...16b770b. Read the comment docs.


update workflow

Electron bindings will still fail and also not sure if we need anything else from https://github.com/iotaledger/wallet.rs/tree/c88070e3e85fbf71c14d26897f1393a5d9520657/bindings/nodejs/scripts again


Update the SPI entries for SwiftUI for direct tracking access

The SwiftUI entry point for direct tracking access had a potential flaw with the lifetime of the AccessList storage - this change alters the signatures to provide the type for storage for direct access to SwiftUI as SPI. That SPI is just a deconstructed version of the public API.

@swift-ci please smoke test


Add school

My school doesn't have an official website but the domain is:
hkikiczenica.edu.ba

@polarrbtw I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host hkikiczenica.edu.ba not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!