door-knocker: 0.4.4 -> 0.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/door-knocker/versions.
meta.description for door-knocker is: Tool to check the availability of portals
meta.homepage for door-knocker is: https://codeberg.org/tytan652/door-knocker
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.0 with grep in /nix/store/hc6g8shl3nyjx4spr03c3rm09hshfz19-door-knocker-0.5.0
found 0.5.0 in filename of file in /nix/store/hc6g8shl3nyjx4spr03c3rm09hshfz19-door-knocker-0.5.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

door-knocker

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hc6g8shl3nyjx4spr03c3rm09hshfz19-door-knocker-0.5.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A door-knocker https://github.com/r-ryantm/nixpkgs/archive/442dff4cf654372663950af59ef8e4eaa360750f.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/442dff4cf654372663950af59ef8e4eaa360750f#door-knocker

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hc6g8shl3nyjx4spr03c3rm09hshfz19-door-knocker-0.5.0
ls -la /nix/store/hc6g8shl3nyjx4spr03c3rm09hshfz19-door-knocker-0.5.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

door-knocker

Maintainer pings
cc @symphorien for testing.

Result of nixpkgs-review pr 310506 run on x86_64-linux 1

1 package built:

door-knocker


Splash timeout on all requests.

I have set up docker to work with splash. Docker starts up fine but even the simple pages like google.com time out. Follows the splash terminal output for splash startup.
docker run -p 8050:8050 scrapinghub/splash

2016-02-26 03:39:07+0000 [-] Log opened.
2016-02-26 03:39:07.903064 [-] Splash version: 2.0.1
2016-02-26 03:39:07.903849 [-] Qt 5.5.1, PyQt 5.5.1, WebKit 538.1, sip 4.17, Twisted 15.5.0, Lua 5.2
2016-02-26 03:39:07.903997 [-] Python 3.4.3 (default, Oct 14 2015, 20:28:29) [GCC 4.8.4]
2016-02-26 03:39:07.904119 [-] Open files limit: 524288
2016-02-26 03:39:07.904248 [-] Open files limit increased from 524288 to 1048576
2016-02-26 03:39:08.112195 [-] Xvfb is started: ['Xvfb', ':1047', '-screen', '0', '1024x768x24']
2016-02-26 03:39:08.216409 [-] proxy profiles support is enabled, proxy profiles path: /etc/splash/proxy-profiles
2016-02-26 03:39:08.364839 [-] verbosity=1
2016-02-26 03:39:08.365024 [-] slots=50
2016-02-26 03:39:08.365640 [-] Web UI: enabled, Lua: enabled (sandbox: enabled)
2016-02-26 03:39:08.368755 [-] Site starting on 8050
2016-02-26 03:39:08.369102 [-] Starting factory <twisted.web.server.Site object at 0x7fb0ee1c4860>

Can you please help me with this.

Docker starts up fine but even the simple pages like google.com time out

how are you requesting those simple pages? Logs you posted dont contain any trace of requests being made to Splash, they just show Splash is started. If you posted your logs in full it means that requests you make dont reach Splash. Can you ping localhost:8050 and get demo page? When requests are timing out what error are you getting? Are they timing out with 504 status code or you dont get any status code?

hey Pawl,
Thanks.
I started docker with this command:
docker run -p 5023:5023 -p 8050:8050 -p 8051:8051 scrapinghub/splash
telnet "ping" output is:
telnet localhost 5023
Trying <IP_ADDRESS>...
Connected to localhost.
Escape character is '^]'.
Connection closed by foreign host.

nmap command trace:
nmap -p 8050 localhost

Starting Nmap 7.00 ( https://nmap.org ) at 2016-02-26 21:50 PKT
Nmap scan report for localhost (<IP_ADDRESS>)
Host is up (0.00013s latency).
PORT     STATE SERVICE
8050/tcp open  unknown

Nmap done: 1 IP address (1 host up) scanned in 0.04 seconds

Now I write in address line Firefox (44.0.2): localhost:8050
I write in text-field provided next to "Render" Button. "http://google.com"(its already there btw) and press the button.
I am attaching compete logs of what the splash process outputs. If you look at the logs there is a problem that /etc/machine-id is not found. Well it is a problem in Ubuntu. I "patched" it, Can this be the problem?
Anyways after processing it gives Error 504

HTTP Error 504 (Gateway Time-out)
Type: GlobalTimeoutError

Timeout exceeded rendering page

{
    "info": {
        "timeout": 60
    },
    "description": "Timeout exceeded rendering page",
    "error": 504,
    "type": "GlobalTimeoutError"
}

Thanks again. :)
splash.txt

thanks for the detailed report. Is this happening for all splash endpoints or just execute? (when you press render on startpage it makes request to localhost:8050/execute).
can you try following url?
http://localhost:8050/render.json?url=http://httpbin.org
this makes request to render.json. is it responding with 504 too?

Yes it responds with 504 too
{"description": "Timeout exceeded rendering page", "error": 504, "type": "GlobalTimeoutError", "info": {"timeout": 30}}
attached are the logs
splash-json.txt
Thanks a lot.

Ok now this works fine. The only thing I have changed is my internet connection. Now I have a faster internet and it does not timeout.
The machine-id error is still there so it was not the case.

docker run -p 8050:8050 scrapinghub/splash --max-timeout 3600

Hello Ahmed.
I am retrieving the same mistake all the time
Some scripts is working fine because it runs in less than 30 seconds
Other script is always failing because it is slower. For this, each time that I execute it retrieves "GlobalTimeoutError"
The server is deploy inside "docker container" with the following parameters:
docker run -p 8050: 8050 --proxy-profiles-path / etc / splash / proxy-profiles --max-timeout 240 --filters-path / etc / splash / filters --js-profiles-path / etc / splash / Js-profiles --lua-sandbox-allowed-modules "populare; utils; popularetarj" -lua-package-path /etc/splash/lua_modules/?.lua --slots 500 -v 2
I feel that the configuration of "-max-timeout" is not working but I do not understand why. Whatever help is welcome.
Thank you in advance
Details:
{
"timestamp":<PHONE_NUMBER>,
"user-agent": null,
"client_ip": "<IP_ADDRESS>",
"method": "POST",
"_id":<PHONE_NUMBER>24656,
"path": "/execute",
"load": [
0.09,
0.04,
0.05
],
"status_code": 504,
"error": {
"error": 504,
"info": {
"timeout": 240
},
"type": "GlobalTimeoutError",
"description": "Timeout exceeded rendering page"
},
"active": 0,
"maxrss": 1953936,
"qsize": 0,
"args": {
"wait": "0.5",
"uid":<PHONE_NUMBER>24656,
"expand": "1",
"lua_source": "local mymodule = require('popularetarj') function main(splash) splash.images_enabled = false splash.response_body_enabled = true args = {} args['user']='XXXX'  args['pass']='XXXX'  args['contract']='XXXX'  args['fromDate']='05/03/2017'  args['toDate']='05/04/2017' return mymodule.execute(splash, args) end",
"images": "1",
"timeout": "240.0",
"url": "http://google.com"
},
"fds": 28,
"rendertime": 30.02491569519043
}

@mgcuesta --max-timeout is only an upper bound of the timeout you can set per request. I didn't find a global config to set timeout for all requests though. If you want longer timeout you'll have to set them in each SplashRequest.

@FreeTymeKiyan How to set for each SplashRequest?


Support for File-Specific Code Formatting Settings

I would like to request support for file-specific code formatting settings in VS Code. Specifically, the ability to configure editor.rulers, editor.formatOnSave, and other formatting options based on file patterns or specific file extensions, such as **/*_model.dart.
This feature would be especially useful for Dart and other languages where generated code, like constructors from packages like Freezed, becomes less readable with a standard line length of 80 characters. Being able to specify a longer line length (e.g., 200 characters) for these specific files would improve code readability without affecting the rest of the project.
Example of Desired Configuration:
"[dart]": {
  "editor.rulers": [80]
},
"**/*_model.dart": {
  "editor.rulers": [200]
}

Code Example Highlighting the Issue:
@immutable
@Freezed(makeCollectionsUnmodifiable: false)
class TransactionsRequestModel with _$TransactionsRequestModel {
  const TransactionsRequestModel._();

  @HiveType(typeId: HiveManager.idTransactionsRequestModel)
  const factory TransactionsRequestModel({
    @HiveField(0) @JsonKey(name: 'type') @Default(TransactionType.all) @TransactionTypeConverter() TransactionType type,
    @HiveField(1) @JsonKey(name: 'sort_by') @Default(SortFactorType.displayMostRecent) @SortFactorTypeConverter() SortFactorType sortFactor,
    @HiveField(2) @JsonKey(name: 'date_from') @DateConverter() DateTime? dateFrom,
    @HiveField(3) @JsonKey(name: 'date_to') @DateConverter() DateTime? dateTo,
    @HiveField(4) @JsonKey(name: 'search') @Default('') String search,
    @HiveField(5) @JsonKey(name: 'page') @Default(1) int page,
    @HiveField(6) @JsonKey(name: 'tags') @Default([]) @TagConverter() List<TagModel> tags,
    @HiveField(7) @JsonKey(name: 'category') int? categoryId,
  }) = _TransactionsRequestModel;

  factory TransactionsRequestModel.fromJson(Map<String, dynamic> json) =>
      _$TransactionsRequestModelFromJson(json);

  StatisticRequestModel get toStatistic {
    return StatisticRequestModel(
      type: type,
      dateTo: dateTo,
      dateFrom: dateFrom,
      search: search,
      tags: tags,
    );
  }
}

As shown above, setting the line length to 80 characters makes these constructors difficult to read and manage. A more flexible approach for file-specific formatting settings would greatly improve developer productivity and code clarity.

@sandy081 @jrieken
Please wait... how is this a "duplicate"?
I searched for discussions on a similar topic but couldn’t find anything. Could you please provide a link to the earlier, active issue that this has been marked as a duplicate of? I believe this is a very important feature that will make the IT world more productive, and I would like to track the progress in this direction.
Best regards,
Dmitriy

https://github.com/microsoft/vscode/issues/35350

Thank you for the link!

#35350

It should be required that you provide a link to the duplicate issue when closing someone's issue that they clearly spent far more time writing than you did writing your reply. No one deserves a 1-word rejection.


Remove reporter config from action-shfmt

Changes

Related commit
The shfmt action does not accept the reporter config (either as part of with or passed in as a reviewdog_flags) so this PR removes it from the action-shfmt configuration.

/assign @kvmware

/approve
/lgtm


Update java dependencies in the Ballerina.toml with artifactId and groupId

Description:
In modules like grpc, http, only Ballerina native jar dependencies are added with above ids. Need to update all of them.
For more info refere this - https://github.com/ballerina-platform/ballerina-standard-library/issues/3789

Closing as all the modules with dependencies that has a chance to be used in other modules are fixed now


Fix the owner label

The sample ingress has owner: some-owner but it'll be rejected by the tests in the tutorial unexpectedly. e.g.: https://docs.kubewarden.io/writing-policies/go/04-validation.html
func TestAcceptRequestWithConstraintLabel(t *testing.T) {
    constrainedLabels := make(map[string]*RegularExpression)
    re, err := CompileRegularExpression(`^team-`)
    (snip)
    constrainedLabels["owner"] = re

Thanks!


policy-controller webhook deployment volumes

Add optional additional volume configuration to policy-controller webhook deployment
Description of the change
Add additional vomues and volumeMounts options to allow for things such as CA Certs to be added to the policy-controller webhook deployment
Existing or Associated Issue(s)
Resolves #396
Additional Information
Validated in an environment with a mirrored sigstore-tuf-root that is available over https with a certificate signed by a custom Root CA. By mounting the custom root CA certificate under /etc/ssl/certs the mirror x509 cert can be trusted
Checklist

[x] Chart version bumped in Chart.yaml according to semver. Where applicable, update and bump the versions in any associated umbrella chart
[x] Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
[x] JSON Schema generated.
[x] List tests pass for Chart using the Chart Testing tool and the ct lint command.

Sorry, was missing the values schema for the volumes


Fixed quantized add op for rknn

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


feat: added error section for failed issued document and refactor publish page

In this PR:

Added the error section in the published page where we can be able to view the failed issued document.

Added a button where user can download the error log when there are failed issued document.

Refactor the publish page.

Verified to be working:


Struggling to setup VPD

Hi, I'm having trouble to run train.py - I want to run it without the train.sh file - , i have the following error :
ModuleNotFoundError: No module named 'vpd.models'
Do you have any idea of what could be vausing this error ?
Also, just to be sure sure, is this supposed to be the correct structure ?
-/VPD
-----/checkpoints
---------v1-5-pruned.ckpt
-----/depth
-----/segmentation
-----etc...

Hi,
In train.sh, we have already set the PYTHONPATH such that vpd.models can be found. We highly recommend using this script instead of directly running train.py.

Okay, thanks, and keep up the great work !
Before i close the issue, did you try to train the model with a limited number of GPUs ?

We did not try to train VPD with limited resources since our main contribution is to reveal the power of generative pre-trained diffusion models. However, it would be an interesting future direction to investigate how to train a VPD more efficiently.


TypeError: Cannot read property 'componentOptions' of undefined

Vue.js version
2.1.3
Reproduction Link
Steps to reproduce
Deploy to server but failure. Show the errors:
[ERROR]     at renderElement (/home/leanengine/app/node_modules/vue-server-renderer/build.js:6102:5)
[ERROR]     at renderNode (/home/leanengine/app/node_modules/vue-server-renderer/build.js:6055:7)
[ERROR]     at next (/home/leanengine/app/node_modules/vue-server-renderer/build.js:6187:9)
[ERROR]     at cachedWrite (/home/leanengine/app/node_modules/vue-server-renderer/build.js:36:9)
[ERROR] error during render : /
[ERROR] TypeError: Cannot read property 'componentOptions' of undefined

What is Expected?
What is actually happening?
I'm not sure the errors happen with code or server.

Please follow the guidelines on how to report an issue. In particular, provide an example on www.jsfiddle.net (or a similar service) that reproduces the problem. If necessary, create a repository for us to clone with a minimal reproduction. repositories of actual projects will generally not be accepted.
Thank you.


Update widget_language.md

I worked through the whole code. I left a few comments (with bold to make the referred text stand out) where I wasn't able to figure out what the text could mean. If you can provide some feedback I might be able to fix it.

@macyhan thanks for checking. All comments should be addressed.

@zoechi  Thanks, it's a great improvement to our documentation ❤.   Seems missing some text , and please resolve the conflict.


PowerRun wont open or just occasionally outright crashes

Microsoft PowerToys version
0.45.0
Running as admin

[X] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Press keybind
✔ Expected Behavior
PowerLauncher opens
❌ Actual Behavior
Nothing
or if it does open, is unresponsive until clicked in and then immediately crashes
Other Software
No response

PowerToysReport_2021-09-26-13-23-53.zip

Couldn't find anything in particular from the logs, expect that Event Viewer is reporting the crashes, but I'm unable to find the corresponding information on watson.
PowerToys Run seemed to have been running fine until recently for you, correct?
Does restarting your computer help?
We changed the logic to call PowerToys Run for the next release (0.47.0)
It should be out in the next few days.
Would you mind giving it another try then?

After updating to 0.47.0 the problem seems to have mostly gone away, it still happens occasionally and restarting only seems to fix it 50/50. If It stops working again in the future ill try adding another report but untill then the update seems to have mostly fixed any issue I was having.
The crashes when they did happen though only seemed to happen either after I clicked in the box if it didn't focus correctly or after I typed the first character.
Ill re-open if I encounter the issues again, thanks.


[RFC] drivers/flash: Introduce flash absolute offset processing

Potential fix to: https://github.com/zephyrproject-rtos/zephyr/issues/20425 & https://github.com/zephyrproject-rtos/zephyr/issues/20423
It was not possible to get know base address of the flash memory form the
flash API. It becomes problem on memories with non-zero base address as
flash_map description is absolute which makes usage of flash_map erroneous.
Patch introduces flash shim API for fetch flash memory base address
and uses it in order to make flash_page_layout API absolute offset
oriented.
Signed-off-by: Andrzej Puzdrowski<EMAIL_ADDRESS>

I belive i'm wrong here - flash addresses are relative.


[RemoteMirror] Add missing test.swift file for interop tests.

@swift-ci please smoke test and merge

@swift-ci please smoke test

You didn’t start this but why are all these magic numbers in tests? Almost every time I have written tests like this I have introduced flaky tests.

Sorry, which magic numbers?


Hi !

Hi !
I am looking for an application capable of creating a stream for a "Radio-Cisba" web-radio, we are an association and we broadcast a local web-radio.
We are looking for a solution that could be self-hosted, we are on a shared server, your script seems to be the solution.
Is it functional? How can I set it up? I went through the code, I can see roughly what it does but I can't see how to set the entry and exit points of the stream.
Cordially,
Raf

Hi @Raf-sns!
First of all, thank you for the interest to this project 😅 Unfortunately, it's rather not functional at all. At some point, I created it just to play with Shoutcast protocol, but later lost an interest to it. At this moment, I'm not even conding in PHP anymore. I'm sorry that I didn't help you, hopefully you'll find other tool that fits you needs


Don't require a strict revision match in LLDB

For release-management purposes during development, LLDB's embedded Swift compiler's version number can sometimes be off-by-one in the last digit compared to the Swift compiler.
This patch restores the old behavior from before 17183629e4bd60efe3db79f8f1ff6fb927fe5d00.
rdar://101299168

@swift-ci test

Note that LLDB implements its own check for the compiler version and it will also warn unless all but the last digit match.

@swift-ci test and merge

I forgot to update lldb-moduleimport-test.

@swift-ci test and merge

@swift-ci test and merge


[Feature][Agent] Provides hierarchical guarantee capacity

Description
When use agent to collect data, such as File Agent. There are multi collection tasks in an Agent process, but the importance of each task is different. when resources are insufficient, the most important data should be collected first. so, may be it is necessary to provide the capacity of hierarchical guarantee.
Use case
No response
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

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

In theory, it can be achieved, but in reality, it will not be used in this way. Usually, this requires the use of a separate cluster to ensure full chain support, and it will be added no later when encountered in the future


Not getting all Albums

Hi, i have checked your previous issue's related to this and changed the fetchResults in Setting.swift to fetch all the albums which are available in device but not getting the exact output, please check and let me know.

but not getting the exact output

What ’output’ are you expecting and what are you getting?

Some weird thing is there, if i am adding this with Pod then its not reflecting the albums but if i add the code manually so all things works fine, and showing the albums as well.


Add Rainbow FUN Theme

I have just created another Colour Scheme, Inspired by the colours of the Rainbow. It's called Rainbow FUN. I did the exact same ordering with my "Invert Standard" scheme, but modified the colors like the Rainbow.

Looks good to me, it's in!

Btw, are you in our Discord Server? You'll gain special roles if you are part of it.


Segmentation fault when install pycoral

Hi,
I met a segmentation fault when I try to install pycoral.
I first do this
https://coral.ai/docs/accelerator/get-started/#runtime-on-linux
and then do this
https://coral.ai/docs/accelerator/get-started/#pycoral-on-linux
then I got
Segmentation fault

I was running these in Bullseye system of Raspberry Pi, here is my system information
uname -a
Linux raspberrypi 5.15.32-v8+ #1538 SMP PREEMPT Thu Mar 31 19:40:39 BST 2022 aarch64 GNU/Linux

Could anyone tell me how to solve this problem?
Tyou

got the same error with a raspberry pi:
Linux raspberrypi 5.15.32-v7l+ #1538 SMP Thu Mar 31 19:39:41 BST 2022 armv7l GNU/Linux
I was following the coral guide


Add NoisyTyperSwift 0.1

Add NoisyTyperSwift.app

Add NoisyTyperSwift.rb

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

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

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked that the cask was not already refused in closed issues.

Is this the same thing as your other PR?

@miccal, they are not the same. This one is an improved clone of the previous one. The previous one is classical and still working, but not maintained.

@vitorgalvao this is not distributed via Github releases, therefore the appcast is useless, no?

@miccal Yes, you’re correct. Misread the url.


Examples of how to use mlir dialects to implement gemm.

Please have a look at this and make appropriate changes.

@hxdtest Thanks for providing the examples!
(1) Please reformat your code as @meshtag said above.
(2) How do you measure the execution time? Can we add a GFLOPS report to the example? (see here for an example)
(3) Have you considered adding a vectorization step to the example?

@hxdtest The recommended way to format the MLIR file is to use mlir-opt, for example:
$ mlir-opt gemm.mlir -o formatted-gemm.mlir


Remove version: NelsonNumericalSoftware.Nelson version <IP_ADDRESS>76

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Nelson-numerical-software/nelson/releases/download/v0.7.10/Nelson-<IP_ADDRESS>76-x86-64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)


Issues with User AD Object in Sccm since it was added as default object

Before in SCCM you had to add whenCreated as a custom object to be scanned from AD for user's and that works fine but ever since this object was moved to default it does not pull the date from AD. I have 495 who where added to sccm by AD before and they have whenCreated dates but since the SCCM update that made this object default in SCCM i have 544 users without whenCreated dates in SCCM. I need this to work correctly as I have Audit reports my Help desk uses for Audit new user's that where created in the last 90 day. Please let me know what I can do to fix this issue and get the whenCreated object in AD to sync with SCCM 1710

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

ID: 181af3ed-e579-507a-9885-97284578fc3c
Version Independent ID: 53e85e7f-febf-b8ce-5306-75b84c84cae6
Content: Find help - Configuration Manager
Content Source: sccm/core/understand/find-help.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks for taking the time to share this with the Configuration Manager documentation team. I’m sorry that you’re having problems, and I want to make sure it gets to the right people that can help. There are a couple of options available to provide the best level of support for this:

Post on Configuration Manager forums. These are moderated by our community MVPs who are very experienced, knowledgeable, and helpful.
Use Microsoft Support to search for the issue, or open a case with professional support.
Contact your Microsoft Premier team, such as a Premier Field Engineer or Technical Account Manager.
I'm closing this issue as there's nothing actionable for docs at this time.


Fix autostart test

Fixes #798.
Once the QUnit.load function was overriden, the old QUnit.load had already been suscribed to the window.onload event. This PR removes the old QUnit.load from those event listeners and adds the new one.

I feel like there has to be a better change.

I'd love if you could elaborate a bit more on your response. What is it that you don't like:

Having addEvent and removeEvent added into autostart.html?
Removing the load event suscription?
The general approach used to test this feature?
Any other thing?

Thank you very much for your feedback

Mostly the addEvent and removeEvent logic. See https://github.com/jquery/qunit/pull/791#discussion_r26958221 for related discussion.


Add postgresql analyzer

Postgres analyzer that analyzes postgresql installation and provides a TLDR and details findings report.

Postgresql configuration - postgresql.conf
Client authentication - pg_hba.conf
Linux postgres account
Postgres account bash history and psql history
Postgresql logs

Hi @roshanmaskey , Just wanted to check in on this one to see if you had any updates.


Fix calculation revenue

Notion card

@mayankagnihotri7 is this ready for review?

@vipulnsward Yes it is ready for review. I'll assign Shalaka for the review.

@mayankagnihotri7 Please rename unpaid amount to outstanding in FE as well as backend.  Unpaid word makes confusion now as we have separated overdue from it.

@shalapatil I've made the requested changes. Please review.

@shalapatil I've made the requested changes. Please review.


Update README.md

updated the readme file form the 2022 year to 2023 year

adding changes to master branch


[Starter Set] Von Holzenauer's Products Missing Descriptions

The stuff you can buy from Von Holzenauer's Potion Shop (A Guide to Ubersreik page 39) lack textual descriptions of what they do.From: Andres#7311foundry: 10.291wfrp4e: 6.4.0wfrp4e-core: 3.1.0wfrp4e-starter-set: 3.1.0wfrp4e-up-in-arms: 3.0.2wfrp4e-wom: 3.0.0Active Modules: CautiousGamemastersPack, WhisperBox, about-time, chat-portrait, dice-so-nice, lib-wrapper, nations-of-mankind-wfrp4e, ogre-kingdom-wfrp4e, pings, settings-extender, sync-token-name, t-is-for-target, tabbed-chatlog, the-dwarf-empire-wfrp4e, wfrp4e-gm-toolkit, wfrp4e-laws-pet-shoppe, forge-vtt, forge-vtt-optional

Located in the GM section


JSON source not being processed in converter

https://github.com/limbo-works/Limbo.Umbraco.Tables/blob/9f1c2cfc6431b251dc4eda8e8458f553da4bd3c6/src/Limbo.Umbraco.Tables/PropertyEditors/TablesDataValueConverter.cs#L30
I'm working on a setup where this editor is used inside the new v11 Block Grid combined with the custom Razor previews.
In this situation the argument 'object? source' for method ConvertSourceToIntermediate contains a JObject iso string and is being ignored completely.
I would propose to make a change similar to what I've seen for the MultipleTextString converter to support multiple formats.
https://github.com/umbraco/Umbraco-CMS/pull/11456

Hi @kows and sorry for the lack of reply
Can you provide some information on how to reproduce this issue? I haven't really used the block grid yet, but am I correct to say that Razor previews are not something that is supported in Umbraco by default? Are you using a third-party package for this?

Although I still think this may be due some custom code that handles the parsing different from Umbraco's, the property value converter was updated for v1.0.3, which was released the same day of my last comment.
As I haven't heard back from you, and the issue should be fixed, I'm closing the issue.


Add to maven or jcenter

It would be nice to have this library published on maven or jcenter.

+1


Auto Update!

MultiOgar-Edited has got so many updates so i cant update it and changing gameserver.ini everytime @Megabyte918 We Want Auto Updater buy command /update!
VOTE FOR AUTO UPDATE

What you can do is whatever line is in whatever file, you can copy and paste the edits on that line(s)
Also it's nearly impossible to do so as it's not possible to go through everyone's gameserver.ini
On OgarUL this was a thing, but your entire gameserver.ini will get refreshed (as in it looks like what it was when you got MultiOgar-Edited. Soo... yeah

Just copy and paste the changes he's made.

OGARUL :// ...

@PrinceDark9243 Please refer to #36

OK @Megabyte918


fix: fix slice init length

Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

The intention here should be to initialize a slice with a capacity of  len(podsList.Items)   rather than initializing the length of this slice.
<!-- Enter the release note text here if needed or remove this section! -->

@rusttech thanks for the PR! Can you elaborate a bit in the commit description what exact issue is this change fixing, how did you test that performance is better before and after the change (I assume there is a  performance issue caused by unnecessary allocations made by append) ?

@rusttech thanks for the PR! Can you elaborate a bit in the commit description what exact issue is this change fixing, how did you test that performance is better before and after the change (I assume there is a performance issue caused by unnecessary allocations made by append) ?
Also please follow instructions from the comment above to sign off the commit.

Thanks. I have add sign off info.
As for the change, the current code, because the length of the initialization slice is len (podsList. Items), will ultimately result in the first len (podsList. Items) elements of pods not being the target value, but podsList The null value corresponding to Items
The online demo: https://go.dev/play/p/q1BcVCmvidW

/test

(I just realized that this PR might end up failing CI due to the breakage in main yesterday. We might need to click the "Update with rebase" button under the "This branch is out-of-date ..." section at the bottom of the page)


streams: Do not show archived streams in dropdown.

Dropdown widget in compose box and in message move modal should not show archived streams
in dropdown as we do not allow sending messages or moving messages to archived streams.
We need a more focused audit of code for fixing other things where archived streams are shown.

Fixes:  https://chat.zulip.org/#narrow/channel/9-issues/topic/Deleted.20channels.20still.20show.20up.20when.20composing.20message

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged to get this fix in, thanks @sahil839!


'crc status' should show OpenShift's status as "Starting" instead of "Stopped"

When starting CRC, the OpenShift web console is available pretty soon but doing crc status shows that OpenShift is "Stopped". After a few minutes, it turns to "Running". The "Stopped" status is apparently because few Operators are still starting up while rest of the bits have started.
It would be helpful if the status of OpenShift shows as "Starting" or something other than "Stopped" so that it doesn't get misinterpreted by the end-user.
❯ crc status
CRC VM:          Running
OpenShift:       Stopped    <------   why is this "Stopped"?
Disk Usage:      10.11GB of 32.2GB (Inside the CRC VM)
Cache Usage:     22.99GB
Cache Directory: /home/dshah/.crc/cache

❯ crc version
crc version: 1.5.0+e1c8fb8
OpenShift version: 4.2.14 (embedded in binary)

This should be fixed by https://github.com/code-ready/crc/commit/960a6a2909ac9b187e6d8ac533cc27f5dd0a0a02

What does it mean its fixed? do we need to download new crc version? Is there no alternate fix ?

Yes you'd need a newer crc version to benefit from that fix.


remove duplicate log message being logged under “method”.

As it is, I have the same log message being logged twice; I tried to make it log the actual original method but we can't access the original method from where we are so I removed it entirely.

Hey thanks for your help :) I didn't merge this because I preferred the fix introduced by https://github.com/mokagio/DDAntennaLogger/commit/457dad62176fb66afdddbd8322548c7d7e6cb859 rather than removing the method key entirely.
Cheers :beers:


Failed No org and space targeted

The error message
FAILED
No org and space targeted, use 'cf target -o ORG -s SPACE' to target an org and space

was caused because my account region was in EU not US in step 4.
If the docs noted that for the EU end point the command should be cf api https://api.eu-gb.bluemix.net that would help.

Can you create a pull request with this ?

We should probably link to more thorough instructions hosted elsewhere (since this applies to all of our sample apps)

Ok. I made a pull request.

thanks!

that's a good idea @nfriedly, but the instructions aren't always the same so we need to see how to make the generic part in a central place and the specific part here


Support icons in selected item also, not only in list of options

added config option selectedLabelRenderer, same as labelRenderer to allow showing icons or other custom content in selected item when the select is closed

@AlexeySemenko can you please add more context to the PR, including running tests and maybe including tests runs (example, something more aligned with this PR)

@AlexeySemenko once you resolve the conflicts the PR should be good to go and get merged 👍🏻

@AlexeySemenko once you resolve the conflicts the PR should be good to go and get merged 👍🏻

conflict resolved


how to set global parameters?

the global config is following:
  config.swaggerdoc = {
    parameters: [
      {
        description: 'user token',
        name: 'accessToken',
        in: 'header',
      },
    ],
    securityDefinitions: {
      apiKey: {
        type: 'apiKey',
        description: 'user token',
        name: 'accessToken',
        in: 'header',
      },
    },
  };

the two config both are not effective, i want to set a global header params
i think it will be  a useful functions.

I have the same questions.


Merge 0.4.1 changes

Merge changes from the 0.4.x branch to master after the 0.4.1 release.

@bkutil

+1


chore(master): release 3.2.0

:robot: I have created a release beep boop
3.2.0 (2024-05-16)
Bug Fixes

move-file-mutation: fix invalidation of cache when moving a file (5b974d0)

Miscellaneous Chores

release 3.2.0 (077cede)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/InformaticsMatters/squonk2-data-manager-ui/releases/tag/3.2.0 :sunflower:


chore: update observability instance references

Description
terraform-ibm-observability-agents is stuck back level on 2.5.1 and requires these changes in the basic example
Release required?

[x] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

When merging, use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

/run pipeline

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


Resetting Sphero Orientation

Hi, is it possible to reset the sphero's internal orientation once it's powered? It senses a Quaternion who's default 0 is whichever way the sphero is facing when its powered on. Is it possible to turn the sphero to a heading, and then reset its internal orientation representation to 0?
Thanks for the great work on this library.

Hello!
Do you know how to connect sphero bolt on ubuntu phsical machine(not vm) ?
My bluetooth can detect my bolt ( called SB-X0C6),but I can't connect it by bluetooth gui.
I try to run the code directly by change the mac address,but there still some errors:
For example, I run the wake.py in vscode, and I get:
Traceback (most recent call last):
File "/home/zzrs123/Pysphero/pysphero/examples/wake.py", line 15, in
main()
File "/home/zzrs123/Pysphero/pysphero/examples/wake.py", line 9, in main
sphero.power.wake()
File "/home/zzrs123/.local/lib/python3.8/site-packages/pysphero/device_api/power.py", line 76, in wake
self.request(PowerCommand.wake)
File "/home/zzrs123/.local/lib/python3.8/site-packages/pysphero/device_api/device_api.py", line 35, in request
return self.ble_adapter.write(
File "/home/zzrs123/.local/lib/python3.8/site-packages/pysphero/bluetooth/gatt_adapter.py", line 76, in write
return self.packet_collector.get_response(packet, raise_api_error, timeout)
File "/home/zzrs123/.local/lib/python3.8/site-packages/pysphero/bluetooth/packet_collector.py", line 53, in get_response
raise PySpheroTimeoutError(f"Timeout error for response of {packet}")
pysphero.exceptions.PySpheroTimeoutError: Timeout error for response of Packet(flg: 0x0a did: 0x13 cid: 0x0d seq: 0x01)


spin-framework: init at 0.2.0

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Building currently fails, but I've opened a PR upstream (https://github.com/fermyon/spin/pull/509) to fix that. Once merged, I'll mark this PR ready for review.

Seems like the build.rs script wants rustup at build time, but I'm not sure how to do that. I tried exporting path in preBuild but that didn't work. I'll dig into docs later to see how I can do this.


osc player needs a local cache

reduces loading times for data already accessed

Check out this angular library:
https://github.com/tymondesigns/angular-locker#usage-adding-to-your-project

i actually had a local db cache in place. it was on the stolen laptop and not yet pushed... the local storage options are not so appealing in this scenario due to their limited space. i guess just use a file based cache now.


cors issue

Bug Report
Version
├── tonic v0.11.0
├── tonic-web v0.11.0
│   ├── tonic v0.11.0 (*)
└── tonic-build v0.11.0

Platform
Darwin mbp-1.home 23.3.0 Darwin Kernel Version 23.3.0: Wed Dec 20 21:31:10 PST 2023; root:xnu-10002.81.5~7/RELEASE_ARM64_T6031 arm64
Description
I tried this code:
  let server = HealthServer { addr };

  let svc = pb::health_server::HealthServer::with_interceptor(server, check_auth);

  let cors = CorsLayer::new()
            .allow_headers(Any)
            .allow_methods([Method::GET, Method::POST])
            .allow_origin(Any);

        let serve = Server::builder()
            .trace_fn(|_| tracing::info_span!("health_server"))
            .accept_http1(true)
            .layer(cors)
            .layer(GrpcWebLayer::new())
            .add_service(svc)
            .serve(addr);

I expected to see this happen: should build
Instead, this happened:

Same issue. Mine Grpc setup with cors was working fine before upgrading to 11.0 from 10.02.
    Server::builder()
        .accept_http1(true)
        .layer(CorsLayer::new())
        .layer(GrpcWebLayer::new())
        .add_service(reflection_service)
        .serve(address)
        .await

My error:
error[E0277]: the trait bound `Cors<GrpcWebService<Routes>>: tonic::codegen::Service<request::Request<toni
c::transport::Body>>` is not satisfied
   --> src\launch\launch_server.rs:164:10
    |
164 |         .serve(address)
    |          ^^^^^ the trait `tonic::codegen::Service<request::Request<tonic::transport::Body>>` is not
implemented for `Cors<GrpcWebService<Routes>>`
    |
    = help: the trait `tonic::codegen::Service<http::request::Request<ReqBody>>` is implemented for `Cors<
S>`

I look at #1320 several times, but the examples simply don't work with 11.0.

I found the issue: tonic 11.0 doesn't work with tower-http 0.5.0 above.
Downgrading tower-http to 0.4.4 allow cors works.

I think is same as this issue: https://github.com/hyperium/tonic/issues/1678


default_bot_config.json not written on fresh install since nedry==2.2.0

Really as simple as it is described. When installing Nedry from pypi, I can only get the default_bot_config.json to appear when using the first version available, 2.1.0. On subsequent versions Created default config file 'default_bot_config.json', please add required parameters is output but there is no file to be found. Debian/Ubuntu as platform with deadsnakes python3.9 and default python3-pip package.

Thank you, I just released 2.4.1 to fix this

Working since nedry==2.4.1


 Problem is, eyelashes don't work with the 2x2 lower mid variant, only the 2x2 bottom variant. I was wondering if this was intentional. If so, I'd implore you to change it, and if not, all the same. (Supposing adding this isn't TOO difficult. Would it be?

Second this! Was about to make a new issue regarding this exact situation but I just hope they'll add compatibility!


#7337 round price

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #7337
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Copilot Summary

works fine for integers but not decimals:

@prury it is the same here as well it seems, since we round to 4 places everywhere
should this be changed to 2 then?

pay 10 usd

😍 Perfect, I’ve sent the payout
💵 $10 @ 4.06 USD/DOT ~ 2.463 $DOT
🧗 13Qx65nLd6SwdtjrRyuoEtp9CKXhF651xdHBPaXcvhwKm4N1
🔗 0xd5c08e9c877daced389759c5e2563383bc70b4fe7ba4a03980481d7f09daddd8
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Decimal places

Good Morning.
How I can define 0 decimals for values in rows? Default I are leaving 2 decimals. In the JSON file that receives comes formated without decimals.
$("#output").pivotUI(
vDatos,
{
rows: ["PROVINCIA","MUNICIPIO"],
cols: ["MERCADO"],
digitsAfterDecimal: 0,
aggregatorName: "Suma",
vals: ["OBJETIVOS_MUNICIPIOS"]
//aggregator: sum(intFormat)(["OBJETIVOS_MUNICIPIOS"])
},
false,
"es"
);
Thank you very much.

The formatting is a function of the aggregators, which you either provide yourself directly or via a locale file. If you want control over the number formatting, you will have to essentially create your own locale file.


部署后server error

正常docker部署后，能成功的访问到前端页面，但是无法启用任何后端服务，一直都是server error，查看日志，一直提示连不上数据库，但是数据库的容器却一切正常
urllib.error.URLError:  b'Sentry responded with an error:  (url: https://sentry.io/api/263057/store/)' [2021-01-27 09:46:02] - [ERROR] - [sentry.errors.uncaught:712]  - ['Reconnecting to the database didn\'t help could not connect to server: Host is unreachable\n\tIs the server running on host "oj-postgres" (<IP_ADDRESS>) and accepting\n\tTCP/IP connections on port 5432?\n'] b'[\'Reconnecting to the database didn\\\'t help could not connect to server: Host is unreachable\\n\\tIs the server running on host "oj-postgres" (<IP_ADDRESS>) and accepting\\n\\tTCP/IP connections on port 5432?\\n\']' Sentry is attempting to send 2 pending error messages Waiting up to 10 seconds

数据库连不上


Add Hercules CI support

This adds a mkEffect function to the shell's passthru, so to run an impure go build on CI, you can add
ci.nix part
  shell.mkEffect {
    src = ./.;
    effectScript = ''
      go build
    '';
  }

Details are in the ci.md doc.
I've introduced a sources module argument to take care of the dependency wiring. It gets the job done, but I can remove the indirection if you prefer.
For testing, I upgraded Nixpkgs, which looked like this. If that looks ok, I can add it to the docs in a separate PR.

I don't think I can fix the goreleaser action. Did I target the right branch?


Getting This Error: You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file

ERROR in ./node_modules/pdfjs-dist/build/pdf.js 2418:53
Module parse failed: Unexpected token (2418:53)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
intent: renderingIntent,
renderInteractiveForms: renderInteractiveForms === true,
annotationStorage: annotationStorage?.getAll() || null
});
}
ERROR in ./node_modules/pdfjs-dist/web/pdf_viewer.js 895:20
Module parse failed: Unexpected token (895:20)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
this._on(eventName, listener, {
external: true,
once: options?.once
});
}
node version => v12.16.1
Angular CLI: 9.1.15
Node: 12.16.1
OS: win32 x64
Angular: 9.1.13
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes
Package                           Version
@angular-devkit/architect         0.901.15
@angular-devkit/build-angular     0.901.15
@angular-devkit/build-optimizer   0.901.15
@angular-devkit/build-webpack     0.901.15
@angular-devkit/core              9.1.15
@angular-devkit/schematics        9.1.15
@angular/cli                      9.1.15
@ngtools/webpack                  9.1.15
@schematics/angular               9.1.15
@schematics/update                0.901.15
rxjs                              6.5.5
typescript                        3.8.3
webpack                           4.42.0

Thank you for opening an issue.  However, Angular v9 is no longer supported.  Please consider updating to a supported version of Angular (currently v12 or v13).  For additional information, please see https://angular.io/guide/releases#support-policy-and-schedule.  If the issue is still present after updating, please open a new issue with the project's information.

Dear @clydin I have checked this with angular v12 its working fine there but the problem is that I can not update the angular version. Isn't it any way that I can run it on angular 9. This error is showing when I am add dependency of ng2-pdf-viewer in app.module.ts. Please help I can not find any solution of this.


[NIFI-13303] - Remove text indicating property verification is disabled.

NIFI-13303

Reviewing...


Software Library Updates

Updated all dependencies in the pom.xml file to the latest versions of the software libraries.

Build finished.

Build finished.

Build finished.

Build finished.

retest this please

Build finished.

retest this please

Build finished.


Added face-detection-0107 and 0108

Created two new models that run well on VPU while maintaining decent accuracy compared to other config files.

@Ilya-Krylov, @snosov1, please take a look.

@dqawami please fix pylint issues and add tests

@dqawami please add tests to

https://github.com/opencv/openvino_training_extensions/blob/develop/pytorch_toolkit/object_detection/tests/export_tests_face_detection.py
https://github.com/opencv/openvino_training_extensions/blob/develop/pytorch_toolkit/object_detection/tests/train_tests_face_detection.py

run tests


Js issues

Print 3rd array.

Please sync the latest changes at the time of comiting to avoid any conflicts @VishalD51

Hi @VishalD51 , Thanks for requesting the PR as instructed. But there is a problem you did, and you have to face inconvenience for that, and I am sorry for that in advanced.
You have changes double inverted commas to single. And I don't like that fact. Please revert the changes and use "" commas through out the code.
I come from a Java Background, I don't want to use Single Inverted commas. I understand that different devs has different preferences, but please use as instructed because I have used double invited commas thought out the project.
Thanks.

@lalitkumarthakur Sorry for inconsistency, I will improve it and raise PR.

@lalitkumarthakur , can you please check this PR?

@VishalD51 Hi, I am busy and running out of deadlines, so I could not review as Of now. Please be patient. December 1st and 2nd week will be hectic for me because I have to wrap up JavaScript and Finish 80% of React.js, and we know it's near to impossible to do so.
Till then,
You can see the code, go through the chapters,
Flag any issues and solve current issues present where help is required on urgent basis.
Thanks,
Lalit.

Hi @VishalD51 , I do not see any activity from your side regarding this issue. Please clear me that "are you willing to contribute further? Or leave it from here itself?"

@lalitkumarthakur, Yes  I want to contribute further and also try solve other issues soon. I am busy with other priority projects. Sorry for that.

Hi @VishalD51 , There is no activity from your side. Please be a responsible contributor and complete the task which you have taken. Once the task is completed, then you are free to discontinue or stop contributing to this project.

Hi @VishalD51, there is no activity from your side from a long time. Do you want to contribute to this project and solve the issue?


Support large integer values for events in MariaDB

Fixes #9303
Status
not-tested
Description
Provided a very low filter efficiency, it might be that plain integer (32 bits) is not enough to represent a file mask and its event information (total, first, last event). It only happens in MariaDB though, Oracle maps an INTEGER data type to NUMBER(38), which gives us a very high limit of<PHONE_NUMBER>854775807.
This patch replaces MariaDB integer by BIGINT, which should give us the same upper limit is given by oracle.
Is it backward compatible (if not, which system it affects?)
no, it has database schema changes
Related PRs
External dependencies / deployment changes
no

Patch applied to all production MariaDB agents, as follows:
ALTER TABLE dbsbuffer_file MODIFY events BIGINT UNSIGNED;
ALTER TABLE dbsbuffer_file_runlumi_map MODIFY num_events BIGINT UNSIGNED;
ALTER TABLE wmbs_file_details MODIFY events BIGINT UNSIGNED, MODIFY first_event BIGINT UNSIGNED;
ALTER TABLE wmbs_file_runlumi_map MODIFY num_events BIGINT UNSIGNED;


Adding value to an enum(Fixed) in a new version(with @added) should be flagged as breaking change

enum Abc {
  a,
  @added(v2)
  b
}

Do you mean there should just be a warning for this scenario, because I don't think we have a breaking change tool? We don't call out that using @removed is a breaking change, for example.

Yeah we don't really have any "breaking" change ruleset yet so feel like we should probably decide what goes in there first


Configuration outside project

Use case:
I'm the only Atom user in a team, I'd love to be able to have each project's config file outside project folder, something like ~/.config/remote-sync-pro/{project}.json

I'll look into doing something like that for sure.  In the short term, and you are probably already doing this, but assuming you are using GIT you could still have it locally in your copy if you add the config file to the project's .gitignore and it wouldn't travel across to other devs.

I know, my use case is quite extreme, and this is not applicable. I'm also the only one using git (--separate-git-dir)

I'm also looking for this feature.


week-8-challenge

Use the template, and review one of the other weekly challenges for an example.
Challenges
All challenges should be in English and use JavaScript.
There aren't any restrictions on what libraries or APIs that can be used.
Challenges should be fun, entertaining, and educational.

#67


Console takeover by the main process

I'm running breeze-ui.exe (v0.1.0 alpha) and if I run it from say Windows Explorer, all is fine. However, if I run it from console (most notably FAR Manager, but cmd.exe has the same problem), the wallet process takes over the console, which is not what you expect as a user - suddenly you can work in the console (in cmd.exe try "dir" + ENTER) and at the same time you get output from the wallet process to this very same console. This is very inconvenient (esp. for FAR Manager users, because what you use FAR for is basically running all the stuff in your system, so you don't expect anyone to break your console like this).

@Aprogiena I canno reproduce it. Can you confirm I'm doing it right?

I use Windows 7, x64, I downloaded from https://github.com/stratisproject/Breeze#ci-build
Then I fire up cmd in the unzipped folder and write breeze-ui (I login, click on a couple of things)

From here I can work in the same console and no output from Breeze is being forwarded there.

Just tried it with the latest build from the link you provided (somehow it is also v0.1.0 alpha) and it behaves differently, but the problem is still there. Somehow more silent now.
Easiest way to reproduce it now is:

delete appdata folder
spawn a new cmd.exe and type breeze-ui
wait 10 sec
then you close the cmd.exe's window (this should kill your breeze which is a related bug)
then spawn a new shell and run breeze-ui again (this should kill your daemon, another bug)
click close in the crash dialog -> your new console should be owned

Here is the exception thrown in the console:

 [Breeze-UI] Console takeover by the main process


Values from Typescript StringEnums and NumberEnums are not carried over to the generated code

Typescript has String Enums and Numeric enums which associates a value to the Enum member
Details
Example
export enum DynamoDBAttributeType {
    NUMBER = 'N',
    STRING = 'S',
    BINARY = 'B',
}

console.log(DynamoDBAttributeType.STRING)
>> S

The generated code doesn't provide any way to access those values.
Generated Java
@javax.annotation.Generated(value = "jsii-pacmak/0.7.6 (build 04cb6bb)", date = "2018-09-30T14:18:02.519Z")
@software.amazon.jsii.Jsii(module = com.acme.hello.$Module.class, fqn = "hello-jsii.DynamoDBAttributeType")
public enum DynamoDBAttributeType {
    NUMBER,
    STRING,
    BINARY,
}

That is true. Could you tell us what you need the enum values for?

I encountered a small problem caused by this issue while using the aws-cdk
I'm using the java-cdk and i wanted to define a DynamoDB table with global secondary indexes, this functionality was not available in the Table construct (introduced by this commit but not included in the cdk version that i am currently using) so i had to use the Table resource.
The table resource api is generated from the specs and expects the Attribute type of the key as a string
        TableResource table = new TableResource(this, "table", TableResourceProps.builder()
              ...
                .withAttributeDefinitions(asList(
                        TableResource.AttributeDefinitionProperty.builder()
                                .withAttributeName("id")
                                .withAttributeType(AttributeType.String.name().substring(0, 1))
                                .build(),
                        TableResource.AttributeDefinitionProperty.builder()
                                .withAttributeName("secondary_index")
                                .withAttributeType(AttributeType.String.name().substring(0, 1))
                                .build()))
                .build());
    }

The problem is .withAttributeType() expects a string value, and since there is now way to get the expect value from the AttributeType enum i had to do this .withAttributeType(AttributeType.String.name().substring(0, 1))
The problem is mainly with the generated cdk resources expecting a string instead of an enum in the first place (i'm going to add an issue for that there).

The problem with JSII is inconsistency of the generated code between different languages.
Values are available in typescript and javascript but not in java and dotnet. And i'm also curious how will it look in ruby/python/ho where there is no enum support in the language.
I think at least this limitation should be stated somewhere in the docs.

I think the bug here is that withAttributeType is supposed to except an AttributeType and not a string. Looking into this.

This is fixed in the latest version of the CDK 0.15.1


[1.8.x] clean history_record when endpoint unmatches [7584]

Build status:

Linux
Mac
Windows

Failures on Mac are due to #954 not being backported to 1.8.x


[enhancement] Add missing bower.json.

Hey, maintainer(s) of botmonster/jquery-bootpag!
We at VersionEye are working hard to keep up the quality of the bower's registry.
We just finished our initial analysis of the quality of the Bower.io registry:
7530 - registered packages, 224 of them doesnt exists anymore;
We analysed 7306  existing packages and  1070 of them don't have bower.json on the master branch ( that's where a Bower client pulls a data ).
Sadly, your library botmonster/jquery-bootpag is one of them.
Can you spare 15 minutes to help us to make Bower better?
Just add a new file bower.json and  change attributes.
{
  "name": "botmonster/jquery-bootpag",
  "version": "1.0.0",
  "main": "path/to/main.css",
  "description": "please add it",
  "license": "Eclipse",
  "ignore": [
    ".jshintrc",
    "**/*.txt"
  ],
  "dependencies": {
    "<dependency_name>": "<semantic_version>",
    "<dependency_name>": "<Local_folder>",
    "<dependency_name>": "<package>"
  },
  "devDependencies": {
    "<test-framework-name>": "<version>"
  }
}

Read more about bower.json on the official spefication and nodejs semver library has great examples of proper versioning.
NB! Please validate your bower.json with jsonlint before commiting your updates.
Thank you!
Timo,
twitter: @versioneye
email<EMAIL_ADDRESS>VersionEye - no more legacy software!

bower.json added


Certain tiles are missing from 'tiles as prop' in prop editor

'tiles as prop' 26 and 27 are missing from the prop menu category list (these are present in the official editor pre-downpour) as well as the Vertical Pipe prop from tiles as prop 13 (other individual props may be missing but this is the only one i've found). This means anything in the categories below Temple Stone in the tile editor are unable to be used as props.

Commit was created and in 2.1 will be solved
thank you for support!


Link listed item ID with Reporting page

ID of each post or user should be provided in the report form as an indentifer.


Temporary hack to support bionic

Currently there aren't any official packages for bionic from docker, but
luckily the xenial packages work fine with bionic. So this temporary
hack is to facilitate installing the xenial package on bionic.

Is this the best way to do this currently? According to the following, zabbix is available for ubuntu 18.04:
https://www.zabbix.com/documentation/3.4/manual/installation/install_from_packages/debian_ubuntu
https://support.zabbix.com/browse/ZBXNEXT-4509 (log in to see the comments).
So I tried to patch my /etc/ansible/roles/dj-wasabi.zabbix-agent/vars/zabbix.yml by adding these lines in the "34"-block:
    bionic:
      sign_key: E709712C

Then a was able to install zabbix 3.4. on my test ubuntu 18.04 server.
I hope that helps in some way...

Seems so for the time being, until such time that Docker publishes official packages for Bionic. Feel free to not take the changes, but it seemed a shame to not have a working solution for Bionic in the meantime because of a simple string mismatch.

Oh.  We are talking about docker+ubuntu18 - and not zabbix in any way. My mistake. Sorry for the confusion.

Let's not use hacks and wait for official packages to make it into the pipeline, and make whatever changes are necessary to install them.   In the meantime, those that need 18.04 support can use your workaround.

Closing in favor of #211


pal-platform.py fails to run

pal-platform.py fails to run with the following errors:
Traceback (most recent call last):
  File "pal-platform/pal-platform.py", line 1029, in <module>
    cli(sys.argv[1:])
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 829, in _call_
    return self.main(*args, **kwargs)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 1276, in invoke
    sub_ctx = cmd.make_context(
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 700, in make_context
    self.parse_args(ctx, args)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 1048, in parse_args
    value, args = param.handle_parse_result(ctx, opts, args)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 1623, in handle_parse_result
    value = self.full_process_value(ctx, value)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 1965, in full_process_value
    return Parameter.full_process_value(self, ctx, value)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 1589, in full_process_value
    value = self.process_value(ctx, value)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 1579, in process_value
    return self.type_cast_value(ctx, value)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 1568, in type_cast_value
    return _convert(value, (self.nargs != 1) + bool(self.multiple))
  File "/tmp/python-deps/lib/python3.8/site-packages/click/core.py", line 1565, in _convert
    return self.type(value, self, ctx)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/types.py", line 46, in _call_
    return self.convert(value, param, ctx)
  File "pal-platform/pal-platform.py", line 90, in convert
    return super(DynamicChoice, self).convert(value, param, ctx)
  File "/tmp/python-deps/lib/python3.8/site-packages/click/types.py", line 181, in convert
    if not self.case_sensitive:
AttributeError: 'DynamicChoice' object has no attribute 'case_sensitive'

root cause:
requirements for click are >=6.5, and on version 7.0 the case_sensitive member for click was introduced. Since DynamicChoice doesn't call its parent __init__ function, I believe that it doesn't recognize case_sensitive as its member.
suggested solutions:

call super.__init__ in the child's __init__ function , which will use the inherited attributes (see here)
update the requirements.txt to click>=6.5,<7.0

i would favor of doing both

I think workaround for now is to manually install click 7.0 before any requirements from pal-platform/requirements.txt and mbed-os/requirements.txt.
This can be done with following command:
pip install click==7.0

This will be fixed in 4.4.0 release version.

Fixed in 4.4.0 release.


Xcode Cloud & Prefire

I've got Prefire up & running in a package on my own machine. I'm trying to get the tests running on Xcode Cloud but it seems like the snapshots tests aren't run. The plain XCTest included in the package tests is being run though.
I have added a ci_post_clone.sh script which resolved the "PrefireTestsPlugin" is disabled error I initially had. So again, it all seems set up correctly.
Before I dig deeper & post more info, can anyone confirm it has actually worked on Xcode Cloud?

Hi, I haven't tried using Prefire with Xcode Cloud. Will try it soon and see how it works.

Any news yet?

I can't use Xcode Cloud yet, as I don't have my own developer account 🥲

Any guesses/hunches why it wouldn't work that might help our own investigation?

Any guesses/hunches why it wouldn't work that might help our own investigation?

Found similar problems with other plugins. It seems that Xcode Cloud does not have permission to run the plugin.
https://github.com/realm/SwiftLint/issues/3996

Maybe this might help: https://github.com/realm/SwiftLint/issues/3996#issuecomment-1465866150

I have a similar issue on Github Actions. I was able to resolve plugin permission issue, but snapshot tests aren't run. Just the one I have.

Now I'm trying to pay for a developer account and figure out the problem


Add GitHub coverage summaries

Add a code coverage summary to the Actions logs.
Use the MSBuild version of ReportGenerator.
Fix some markdown lint warnings.

Codecov Report

Merging #532 (02490b3) into main (cd778ea) will not change coverage.
The diff coverage is n/a.

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

@@           Coverage Diff           @@
##             main     #532   +/-   ##
=======================================
  Coverage   97.39%   97.39%
=======================================
  Files          15       15
  Lines         846      846
=======================================
  Hits          824      824
  Misses         22       22

Flag
Coverage Δ

linux
97.39% <ø> (ø)

macos
97.39% <ø> (ø)

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed

Make sure to change the h2 element asap


Allow creating CommandContext with a target entity version

We need to:

Extend Commands.createContext() to accept parameter targetVersion.
Have new method CommandFactory.createContext(int version).

@alexander-aleksandrov please resolve conflicts.

@armiol PTAL

@alexander-aleksandrov something is wrong with the PR description.

@armiol PTAL

@armiol PTAL

Codecov Report

Merging #327 into master will increase coverage by 0.1%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master     #327     +/-   ##
===========================================
+ Coverage     93.11%   93.21%   +0.1%
+ Complexity     2281     2258     -23
===========================================
  Files           220      214      -6
  Lines          7446     7387     -59
  Branches        575      570      -5
===========================================
- Hits           6933     6886     -47
+ Misses          383      374      -9
+ Partials        130      127      -3

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 3024979...f5c9ecf. Read the comment docs.

@alexander-aleksandrov, please resolve conflicts. Let's finish with this PR.


Seek overload with limit now considers end cursor

Previously the seek overload that takes a limit did not take the end cursor into account, only the limit.
cc @pakrym

Hi @aL3891, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Well I guess this is obsolete now given #1092....

Yah...


Supported image formats not specified

I followed the instructions on this page closely but had a lot of difficulty getting it to recognize the images that I uploaded to my blob storage container for training. It kept returning an error message of: "No valid blobs found in the specified Azure blob container."
It took me a long time to figure out that what this error meant was that the format that I was uploading the images in (tif) was not a valid format. Please add clarification to this page regarding the types of image formats that are supported. Even better, re-write the error message that is returned from Form Recognizer to include information about what was found and why they were not valid blobs.

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

ID: 18818774-5076-cb76-ee9e-4ecb632b6caa
Version Independent ID: 22d60153-50fc-024e-6249-edc796dcdf19
Content: Quickstart: Train a model and extract form data using the REST API with Python - Form Recognizer - Azure Cognitive Services
Content Source: articles/cognitive-services/form-recognizer/quickstarts/python-train-extract.md
Service: cognitive-services
Sub-service: forms-recognizer
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@TysonLinger Thanks for the feedback. We are investigating into the issue and will update you shortly.

@TysonLinger The supported formats for custom model are JPG, PNG, or PDF (text or scanned).
The supported formats for receipt model are JPEG, PNG, BMP, PDF (text or scanned) or TIFF.
The supported formats for both models are documented in our overview page of the service.
We will pass the request to display appropriate error message to our product group to review and update the same in future releases.

@TysonLinger We have requested our product team to review the error messages so the same can be corrected in future releases.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply. We will gladly continue the discussion and we will reopen the issue.

Hi @RohitMungi-MSFT. I tried with having scanned PDF invoices kept in our blob storage and got the error "No valid blobs found in the specified Azure blob container".  I'm using custom model training without labels.
What could be wrong? What further details would you require?


custom interaction doesn't work in advanced build

I have created custom interaction as shown in the example:
http://openlayers.org/en/latest/examples/custom-interactions.html
the interaction works perfectly in the SIMPLE_OPTIMITIMIZATIONS option,
but for some reason when I build the project in ADVANCED _OPTIMITIMIZATIONS I have noticed that my interaction stop the event and the other interactions.
also I have noticed that if I return false in the down event handler the event continue to the other interactions but of course stop the event propagation in my interaction.

Here is an example project that uses advanced compilation for this custom interaction example:
https://github.com/tsauerwein/ol3-closure-samples/tree/master/ol3-custom-interaction
Maybe that helps you finding the issue.

Hi,
I am using the other build option of creating a custom build version of ol as shown in this example:
http://openlayers.org/en/latest/doc/tutorials/custom-builds.html
with this json:
{
"exports": [
"ol.Map*",
"ol.View*",
"ol.animation*",
"ol.Overlay*",
"ol.Feature*",
"ol.inherits",
"ol.control.defaults*",
"ol.proj.get",
"ol.geom.Circle*",
"ol.geom.Point*",
"ol.style.Circle*",
"ol.style.Stroke*",
"ol.style.Text*",
"ol.style.Fill*",
"ol.style.Style*",
"ol.tilegrid.TileGrid*",
"ol.layer.Vector*",
"ol.layer.Tile*",
"ol.source.Vector*",
"ol.source.XYZ*",
"ol.format.GeoJSON*",
"ol.source.Vector*",
"ol.interaction.Pointer*",
"ol.interaction.Pointer#getTargetPointers",
"ol.interaction.DragPan*",
"ol.interaction.PinchRotate*",
"ol.interaction.PinchZoom*"
],
"compile": {
"externs": [
"externs/bingmaps.js",
"externs/closure-compiler.js",
"externs/geojson.js",
"externs/oli.js",
"externs/olx.js",
"externs/proj4js.js",
"externs/tilejson.js",
"externs/topojson.js"
],
"define": [
"goog.array.ASSUME_NATIVE_FUNCTIONS=true",
"goog.dom.ASSUME_STANDARDS_MODE=true",
"goog.json.USE_NATIVE_JSON=true",
"goog.DEBUG=false"
],
"jscomp_off": [
"unknownDefines"
],
"extra_annotation_name": [
"api", "observable"
],
"compilation_level": "ADVANCED_OPTIMIZATIONS",
"output_wrapper": "(function(){%output%}).call(window);",
"use_types_for_optimization": true,
"manage_closure_dependencies": true
}
}
by the way, another thing that bother me with this build tactic is that the targetPointers member of the ol.interaction.Pointer is being hashed in the ADVANCED build.

I have found the reason to the problem,
I was using the function: ol.interaction.Pointer.prototype.shouldStopEvent
and when I compile in advanced optimization the function name is changing,
so to summer up:
I think it would be nice if the function ol.interaction.Pointer.prototype.shouldStopEvent will be part of the api.
also I found the member targetPointers pretty helpful and it would be nice if you can add him too to the interactions api.
thanks.


Run terraform command via batect

I want to be able to run my terraform commands through the batect build tool

[ ] ./batect plan runs terraform plan
[ ] ./batect apply runs terraform apply
[ ] ./batect validate runs terraform validate

Not something that we will need as of now.


Custom home page Doc

Updated Custom Home page Doc

@v-sambati : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Docs Build status updates of commit 1356e7f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-community-training/infrastructure-management/configure-your-platform-infrastructure/set-up-custom-homepage-for-your-mct-instance.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

PRMerger Results

Issue
Description

File Change Percent
This PR contains file(s) with more than 20% file change.


Return error when creating public keys from invalid PEM

If the pemBytes passed in is not valid, pem.Decode will return nil. Passing that to x509.IsEncryptedPEMBlock will panic when it tries to read Headers. This fix returns an error early rather than panicking.

Please, can you Signed-off your commit? @mdelillo

Sure thing


script fatal error during game restart

hi! ty for the support, i don't know what to do to fix it

nvm, seen the fix rn, srry


Update dep file for pilot

Sync the .dep locks with the current WORKSPACE.
Which issue this PR fixes *(optional, in `fixes #<i
none

Well, root is not yet working, and the current file doesn't work either.
Once we get a mixer lock file - we can move it, but waiting for the generated files to get in.

I mean: the lock file in istio/pilot needs to be updated ( this PR ).
Separate PR is to get a lock file that works for pilot and mixer, and deletes the istio/pilot lock.


Account quick switch modal

Supercedes #1555
Closes https://github.com/bluesky-social/social-app/issues/1514

Yeah agreed - I left it out because Twitter doesn't do it, but it would definitely make it much more satisfying

I added a small haptic feedback but not as cool as instagram (which does an additional one to indicate the long press has triggered, then another on release)


Add life cycle state extraction.

Add a way to extract the actual PSA/CCA life cycle state from the LifeCycle claim field, representing the state as an enum. Use this internally for validation.

@setrofim : Seems coverage is failing?

Coverage fixed now.


python3Packages.pypck: 0.7.12 -> 0.7.13

Motivation for this change
https://github.com/alengwenus/pypck/releases/tag/0.7.13
Has to go to staging.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 156537 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.lcn
python310Packages.pypck
python39Packages.pypck


internal: Prepare Code extension for bundling

CC #10483
This is slightly ugly, but we'll be able to clean it up after ripping the download parts (unless we decide to temporarily drop support for the nightlies).

bors r+
It's relatively low-risk (the server directory doesn't exist yet) and I hope we can catch Monday's release.


Wrap PyInit_##name in macro to allow macros as module names

This is similar to what pybind11 does:
https://github.com/pybind/pybind11/blob/4ce05175d51a4685232452bdc1e9cbb13a240a65/include/pybind11/detail/common.h#LL372-L374
Pasting the macro argument directly does not work, as demonstrated by: https://godbolt.org/z/Kv9EEfEsY

LGTM, thanks!


centerMenuItems and menuItemWidthBasedOnTitleTextWidth causes wrong placement

If both centerMenuItems and menuItemWidthBasedOnTitleTextWidth are enabled, the labels will be placed incorrectly. If not initally, then on rotation.

They are not supposed to be working together as centerMenuItems is only supposed to center menu items if they're all the same width. Sorry for not making that clear anywhere. I just believe that centering different widths menu items will look funky.

Try this? https://github.com/uacaps/PageMenu/pull/68


Roll engine 12315736f9ca..49ef4adad327 (5 commits)

https://github.com/flutter/engine/compare/12315736f9ca...49ef4adad327
git log 12315736f9ca..49ef4adad327 --no-merges --oneline
49ef4adad Fix a crash when UILaunchStoryboardName is not specified in Info.plist (flutter/engine#6234)
2a433bc68 Roll src/third_party/skia 34e5e1b387f5..76bba35cbaa4 (2 commits) (flutter/engine#6243)
0c28a093a Roll src/third_party/skia 858f648bdae3..34e5e1b387f5 (8 commits) (flutter/engine#6241)
513f8fbae Roll src/third_party/skia 16322637c477..858f648bdae3 (7 commits) (flutter/engine#6240)
9a7d7215a Roll src/third_party/skia<PHONE_NUMBER>3b..16322637c477 (10 commits) (flutter/engine#6238)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@rmistry @liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Trybots failed. These were the failed builds: https://flutter-dashboard.appspot.com/build.html

Commit queue failed; closing this roll.


Feeature request: Tag the source to synchronize with pypi release

It would be appreciated if you could tag the source. Why do I request this feature?

A lot of popular software use tags to release, including Linux, black and a lot of others.
This would allow to use a GitHub release as source to create packages for distributions and run the tests as they would be included, for example Nixpkgs.
It will make using jittor a lot easier if jittor is included in the official repositories of some Linux distros. As far I know, it will make packagers' life much easier if they can browser code in Github instead of downloading from pypi and browser code locally.

Thanks in advance.

Thanks for your advice, now tag is added in git ^_^, see https://github.com/Jittor/jittor/tags

Thanks

<IP_ADDRESS> and <IP_ADDRESS> have been released in Pypi. However, I cannot find corresponding tag in Github.


Ready for review.

Ready for review @smn @saeedmb8 @miltontony @KaitCrawford

:+1:

Perhaps worthwhile to make this go to tech-support@ perhaps and have that to to solutions team?


Some digest names are not processable by EVP_get_digestbyname

In OpenSSL 3.1.0 EVP_get_digestbyname cannot process values returned by EVP_MD_get0_name. One such example is the SHA256 vs SHA2-256.
My scenario was when during TLS connection my providers OSSL_FUNC_signature_digest_sign_init() received the mdname and I tried to process it with EVP_get_digestbyname().

EVP_get_digestbyname() handles only hashes that have a legacy implementation. Why do you need to pass the value returned by EVP_MD_get0_name() to this function? It also (for non-engine-based) implementations returns just the placeholder const EVP_MD object that will trigger implicit fetches. I do not see a scenario where this would be particularly useful.
Especially I do not see why would you call this function inside provider.

I need to do translation from OpenSSL algorithm names to Windows CNG algorithm names. To be at least partially resilient to changes of naming conventions in both projets I need to ideally use some predefined constants.
Since I could not find any documentation about what are the actual values of mdname in OSSL_FUNC_signature_digest_sign_init() could be I resorted to a function listed in the documentation of (EVP_MD_fetch)[https://www.openssl.org/docs/manmaster/man3/EVP_get_digestbyname.html] that is able to parse string names of algorithms and return EVP_MD objects from which I could extract by EVP_MD_get_type NIDs that are all nicely written out in obj_mac.h
And since NIDs are paired to algorithms I was able to have a nice switch statement based on the md_type returned by EVP_MD_get_type(EVP_get_digestbyname(mdname)).

NIDs are legacy thing - they aren't there for any new algorithms that do not have legacy implementation. In general it is not a good idea to use NIDs in providers.
You could use EVP_MD_fetch() to fetch implementation of an algorithm from the mdname. And then use EVP_MD_is_a to match a known good name. Alternatively you can use EVP_MD_get_type if you want to support only the existing hashes that have also a non-provided implementation.

Thank you! For this is a much cleaner solution to my problem and for the explanation about NIDs and EVP_get_digestbyname().


Added soliscloud adapter to latest

RE-CHECK!

RE-CHECK!

First of all - THANK YOU for the time and effort you spend to maintain this adapter.
I would like to give some (preliminary) feedback based on my personal oppinion. This is NOT an offical review and @Apollon77 might have several additional suggestions or even a different oppinion to one or the other statement. Please feel free to contact him (or wait for a response from him) if you cannot follow my suggestsions or before you spend major effort.

[ ] Is 'tab:materialize' needed?
Please check whether
          "tab": "materialize"

is really needed at io-package,json (https://github.com/Trixx34/ioBroker.soliscloud/blob/0d0ffb0f9eeddf93e68188638e0c7be16f9664e0/io-package.json#L84C1-L85C1)

[ ] invalid characters should be filtered from object ids
Some characters are not allowed to be used as part of an object id. If an object id is not hardcoded or even depending on user input you should ensure that such an object id does not contain an invalid character.  Iobroker provides the constant adapter.FORBIDDEN_CHARS, i.e. by using the following snippet:

function name2id(pName) {
    return (pName || '').replace(adapter.FORBIDDEN_CHARS, '_');
}

The best approach for new adapters is to allow a-z,A-Z,0-9,- and _ only. Please filter the user configurable platId and convert alle non a-zA-Z0-9-_ chars into i.e. an _

[ ] reevaluate state roles
Only the values specified here (https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/stateroles.md) may be used as state roles. Do not invent new names as this might disturb the functionalyity of other adapters or vis.
In addition the roles MUST match the read/write functionality. As an example a role value.* requires that the state is a readable state. Input states (write only) should have roles like level.*. Please read the description carefully. States with role 'button' should (must) have attribute 'common.read=false' set. A button ( "Taster" in german only triggers when you press but else has no state), so it should also be not readable. This is also like HomeMatic does it. A switch has clear states true or false and so can be read.
Please avoid using general roles (state, value) whnever a dedicated role exists.
I.e.
value.energy.consumed - energy consumed (unit=Ws, Wh, kWh)
value.energy.produced - power produced (unit=Ws, Wh or kWh)

will fit for some states. Please check all states for matching roles.

[ ] check and limit configurable timeouts / intervals
Node setTimeout/setInterval routines have a maximum allowed value of 2,147,483,647 ms. Using delays larger than 2,147,483,647 ms (about 24.8 days) result in the timeout being executed immediately. So all (user configurable) values passed to setTimeout / setInterval should be checked and limited.

[ ] check and limit intervals used to access cloud services
Limit all intervals used to access cloud services to a reasonable value. If users configure an inapprobiate access rate this could lead to suspending the service by the provider for all ioBroker users.

[ ] standard github testing is missing
standard github testing must be active and pass. You hab an incorrect head branch name at test-and.release workflow. I've generated a PR trying to fix this. Please merge and check that test is running and succeds (green checkmark) for all node releases (16 / 18 / 20)

[ ] adapte engines requirements
As ioBroker reuires node 16 minimum, please adapt engines clause at package.json to require node 16 minimu
https://github.com/Trixx34/ioBroker.soliscloud/blob/0d0ffb0f9eeddf93e68188638e0c7be16f9664e0/package.json#L22

Thanks for reading and evaluating this suggestions.
McM1957
Please add a comment when you have reviewed and fixed the suggestionsor at least commented the suggestions and you think the adapter is ready for a re-review!

Did you have time to review commenst?
Please add a comment when you have reviewed and fixed the suggestionsor at least commented the suggestions and you think the adapter is ready for a re-review!

Hi,
I did check them and was able to already fix a few things.
I hope I get to the other issues in the next few days.
Thanks again for your help in checking my code.
Regards.
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.www.avg.com
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
Op zo 29 okt 2023 om 20:52 schreef Martin M. @.***>:

Did you have time to review commenst?
Please add a comment when you have reviewed and fixed the suggestionsor
at least commented the suggestions and you think the adapter is ready for a
re-review!
—
Reply to this email directly, view it on GitHub
https://github.com/ioBroker/ioBroker.repositories/pull/2761#issuecomment-1784209147,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDBELPXFW32MKILHC4AHIFTYB2QXNAVCNFSM6AAAAAA6BUI7ACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBUGIYDSMJUG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

[ ]  Is 'tab:materialize' needed?
Please check whether          "tab": "materialize"

-> Removed.

[ ]  invalid characters should be filtered from object ids
-> Fixed by only allowing numerical values for the plantID.

[ ]  reevaluate state roles
Only the values specified here (https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/stateroles.md) may be used as state roles. Do not invent new names as this might disturb the functionalyity of other adapters or vis.
In addition the roles MUST match the read/write functionality. As an example a role value.* requires that the state is a readable state. Input states (write only) should have roles like level.*. Please read the description carefully. States with role 'button' should (must) have attribute 'common.read=false' set. A button ( "Taster" in german only triggers when you press but else has no state), so it should also be not readable. This is also like HomeMatic does it. A switch has clear states true or false and so can be read.
-> I think these should be correct now.

[ ]  check and limit configurable timeouts / intervals
[ ]  check and limit intervals used to access cloud services
-> pollIntervall will only allow values between 45 & 1800 seconds.

[ ]  standard github testing is missing
standard github testing must be active and pass. You hab an incorrect head branch name at test-and.release workflow. I've generated a PR trying to fix this. Please merge and check that test is running and succeds (green checkmark) for all node releases (16 / 18 / 20)
-> PR fixed this.

[ ]  adapte engines requirements
As ioBroker reuires node 16 minimum, please adapt engines clause at package.json to require node 16 minimu
https://github.com/Trixx34/ioBroker.soliscloud/blob/0d0ffb0f9eeddf93e68188638e0c7be16f9664e0/package.json#L22
-> updated to node 16 minimum.

[ ]  some labels seem to have no translation into all languages
It seems that some labels are not translated into all supported languages, i.e. ...
Please consider adding missing translationsThanks for reading and evaluating this suggestions.
-> Fixed translations.

RE-CHECK!

Some (more) feedback

[ ] -> Fixed by only allowing numerical values for the plantID. (not blocking)
You should consider to add this check into the code. Config data could be written directly by changing the object tto bypassing any checks at jsonConfig.json. This is not blocking. Buts a giid praxis to check all parameters at adapter start (value and type) and not to trust any confgi limits.

[ ] write attribute must be false for "value*" (must be fixed)
see https://github.com/Trixx34/ioBroker.soliscloud/blob/796fa27180e168d05932bc892976e0157aca04c7/src/main.ts#L28
Roles look good. But most likely states are no input states and hence must have write:false attribute. IF they are really input states (states which ar used to control the inverter) the roles muste be LEVEL.*

[ ] limiting intervalls (not blocking)
as written above, such limits should be coded into source to avoid malfunction due to incorrect config. (i.e. a short time ago, admin had a bug so that it stored numeric values as strings. This caused really unexpected bahavior at adapters. If the adapter would have checkd the parameters a much more clear error message would have been logged)

[ ] consider using integrated deployment (not blocking)
Consider adding standard deploy workflow into test-and-release. This will work flowlessly with alcalzone/release-script.
Of course this is a suggestion only and this will not block addition to repositories.

@mcm1957
Thanks for checking (again :) )

[ ]  -> Fixed by only allowing numerical values for the plantID. (not blocking)
You should consider to add this check into the code. Config data could be written directly by changing the object tto bypassing any checks at jsonConfig.json. This is not blocking. Buts a giid praxis to check all parameters at adapter start (value and type) and not to trust any confgi limits.

->You are correct, I didn't look at it that way. I've now implemented a replace using FORBIDDEN_CHARS before the plantid is created.

[ ]  write attribute must be false for "value*" (must be fixed)
see https://github.com/Trixx34/ioBroker.soliscloud/blob/796fa27180e168d05932bc892976e0157aca04c7/src/main.ts#L28
Roles look good. But most likely states are no input states and hence must have write:false attribute. IF they are really input states (states which ar used to control the inverter) the roles muste be LEVEL.*

-> switched the attributes to false.

[ ]  limiting intervalls (not blocking)
as written above, such limits should be coded into source to avoid malfunction due to incorrect config. (i.e. a short time ago, admin had a bug so that it stored numeric values as strings. This caused really unexpected bahavior at adapters. If the adapter would have checkd the parameters a much more clear error message would have been logged)

-> implemented an additional function to check if the config is correct and contains allowed values before setting the interval.

[ ]  consider using integrated deployment (not blocking)
Consider adding standard deploy workflow into test-and-release. This will work flowlessly with alcalzone/release-script.
Of course this is a suggestion only and this will not block addition to repositories.

-> Added and just used this for the first time, thanks for the suggestion :)

RE-CHECK!

[ ]   consider using integrated deployment (not blocking)
Consider adding standard deploy workflow into test-and-release. This will work flowlessly with alcalzone/release-script.
Of course this is a suggestion only and this will not block addition to repositories.

-> Added and just used this for the first time, thanks for the suggestion :)

Just one remark: (NOT BLOCKING)
I do not see that you added anything to test-and.release workflow and I do not see a deploy job at actions. I'm quite sure that you pushe the latesr release suing npm publish manually.
I referred to the workflow parts you can find i.e. at this link
https://github.com/iobroker-community-adapters/ioBroker.pid/blob/b59fc9089f52147248408753c1d38eb0621ef84d/.github/workflows/test-and-release.yml#L54
If you add this part, the wirkflow will publish a new release without any extra action if checks pass.
But as already mentioned, ist up to you hwo you would like to process you releases. If you need any help, let me know. If you prefer I cann provide a PR with the "missing" parts.

This adapter has been released to latest repository and should be visible within 24h maximum.
Please create a thread at https://forum.iobroker.net/category/91/tester titled like "Test Adapter " to collect some user feedback and provide a link to this topic when requesting addition to stable repository later.
Note: If an other testing topic already exists, ist OK to continue using this topic too.