How to update expired bot secret key in communicator app?

Dear team,
I have successfully installed the company communicator app from beginning to the entire end process. In regards to step 5 of https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/wiki/Deployment-guide#1-register-azure-ad-application, I have created the bot secret that will expired in 1 year time.
My question is prior to when the bot secret has  or will expired on the due date next year, can you please tell me the process steps on how to update an expired bot secret in Azure vault and where else in the app/config settings that uses the bot secret value will need to be manually updated?

Perhaps if you can update the instruction on how to update an upcoming expired bot secret in the deployment guide (https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/wiki/Deployment-guide) would be much appreciated for end-users who are currently using company communicator app.
Thanks.
Kind regards
David

Hi @aosolis
Would you be able to help in giving updated instructions to the community on how to update the company communicator app if the bot secret key value has expired?
Thanks.

You can directly update new secret in App service's configuration. New Author's app registration secrets needs to updated at following places AuthorAppPassword, AzureAd:ClientSecret and other User app registration secrets need to update at UserAppPassword.


circular-json not working when minified and uglified

Opening new issue as previous one was closed.
I have used ../bower_components/circular-json/build/circular-json.js in my project. It is working fine in my local where it is used as is(without minification and uglification)
When i am creating war of my frontend and deploying. I dont get any window.CircularJSON object. Please help.
I have used in the specified way it self.  I could not find CircularJSON on window object

This was unnecessary. Please follow instructions. AFAIK there's no bower version of this project. Use modern bundlers or use the script tag to include the file globally.
If you'll open another identical report I will block you from this repository.

Hi Andrea,
Thanks for your reply. I was using it as bower component as i found it too and the file seemed same. Currently installed through npm. Its working fine now. One suggestion for you, be a little bit polite while replying

As you can see, you can write again in the same "bug" even if I've closed it. I take your advice but you should never re-open the same bug twice in a row if it's been closed already, that makes maintainers nervous, they might think they are dealing with a troll.


[media library] use provided configuration for upload max file size

Hate to do this to you, but I just merged #243
It reorganizes the repo into a better monorepo setup using lerna. The main thing that concerns this PR is all the core files have moved to packages/core.

Closing as stale. Feel free to reopen this PR once it has been updated and the comments addressed.


weights size mismatch

Hi,
Appreciate such awesome work!
I just bump into the size difference of weights issue between what i got and what it should be.
From the README, it should be 1.6G but I only see a shy 800M.
I used the same path to download and i saw that wget reached to 100%.
Whenever you have a chance, can you please take a quick look at it?
Thank you.
best,
heejong

Sorry!  That's a mistake in the README!

gotcha.
thank you for the clarification and quick fix.
best,
heejong


fix of removing the Iops limits on StorPool volumes

Description
Remove the Iops limits when resizing a volume with disk offering without iops limits or with a custom Iops limits in offerings
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[X] Minor
[ ] Trivial

How Has This Been Tested?
Test resizing volume with different disk offerings by providing/removing iops limits on StorPool volumes

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4752

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-5396)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 39563 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6902-t5396-kvm-centos7.zip
Smoke tests completed. 104 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File


add threads-plugin to gp function webpack build

This allows threads-js to be used in geoprocessing functions, and webpack will convert the worker import as appropriate, as discussed here - https://threads.js.org/getting-started#build-with-webpack

Once you're done testing tomorrow @avmey this can be merged

Working without issues in production, merging this in


Problem with Hero on dev version 1.15.3 on iOS and android

Steps to Reproduce

Run flutter create bug.

Update the files as follows: ...

...

Expected results:
Hero show between pages smooth and without glitches like beta version 1.14.6
Video included version 1.14.6:
https://drive.google.com/file/d/1szsNg1H4-r0rHfEZo6-C7324-oqIhFP8/view?usp=sharing
Actual results:
Hero glitch
Video included version 1.15.3:
https://drive.google.com/file/d/1id7DeWDFIYv2FgnqLpMQv8PPPPQJsfTV/view?usp=sharing

Logs

No issues found! (ran in 2.6s)

[✓] Flutter: is fully installed. (Channel dev, v1.15.3, on Mac OS X 10.15.4 19E224g, locale en-IL)
[✓] Android toolchain - develop for Android devices: is fully installed. (Android SDK version 29.0.0-rc2)
[✓] Xcode - develop for iOS and macOS: is fully installed. (Xcode 11.3.1)
[✓] Android Studio: is fully installed. (version 3.5)
[✓] Connected device: is fully installed. (3 available)

Hi @tamirrab
Could you post a minimal code sample to reproduce it?

It's the simple code of hero,
It's happen also with this example:
import 'package:flutter/material.dart';
class MainScreen extends StatelessWidget {
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(
title: Text('Main Screen'),
),
body: GestureDetector(
onTap: () {
Navigator.push(context, MaterialPageRoute(builder: (_) {
return DetailScreen();
}));
},
child: Hero(
tag: 'imageHero',
child: Image.network(
'https://picsum.photos/250?image=9',
),
)),
);
}
}
class DetailScreen extends StatelessWidget {
@override
Widget build(BuildContext context) {
return Scaffold(
body: GestureDetector(
onTap: () {
Navigator.pop(context);
},
child: Center(
child: Hero(
tag: 'imageHero',
child: Image.network(
'https://picsum.photos/250?image=9',
),
)),
),
);
}
}

Hi @tamirrab, I checked out the exact tag but with the sample code above I couldn't reproduce the broken pop animation shown in your 1.15.3 video.
[✓] Flutter (Channel unknown, v1.15.3, on Linux, locale en_US.UTF-8)

It's hard to reproduce with a simple code maybe because it happen with a full app,
It's took me 2 hours to find that the problem with the version of flutter.
The same code exactly with different version of flutter gave me a different results of Hero widget

If you want me to do something that can direct you to the bug please ask me, I will help as much as I can, but for now I can't upgrade my flutter version because this bug

It would be very difficult to investigate if the bug is not reproducible. Any chance you could go to the flutter directory and do a git bisect between v1.14.6 and v1.15.3, to find out the culprit commit (make sure to kill the app between each test run)? Hopefully this will give us some insights.

I can try but only next week
Hope I’ll find something
If not I’ll try to create new app and will try to reproduce the bug

Hi @tamirrab
Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Modify index to year 1 index to correctly utilize tou demand function

The TOU demand function changed in https://github.com/NREL/ssc/pull/1135/files expects the year 1 index. Correct this, and add a test that reproduces the issue by setting up a multi-year rate forecast object.

Issue reported in SAM forum: https://sam.nrel.gov/forum/forum-general/4659


Add Chinese simplified Translation

Thank you @expload233, could you just fix the merge conflicts?

Thank you @expload233, could you just fix the merge conflicts?

Ok,I fixed now.


Terraform crash creating Server Profile Template

I've been working on creating a new Server Profile Template. It's been working well until today when I started including "boot" items in the resource. Terraform 0.13. OneView 6.10. REST API 2600.
I was trying to isolate the issue by commenting in/out sections of the resource. I think the problem is within the boot section. Problem/crash only happens in the "apply" command. "Plan" completes successfully.
`  boot {
manage_boot        = var.boot_options.manage_boot_order
boot_order         = var.boot_options.boot_order
}
boot_mode {
manage_mode        = var.boot_options.manage_boot_settings
mode               = var.boot_options.boot_mode_selected
pxe_boot_policy    = var.boot_options.pxe_boot_policy
}`
I'll attach the crash log, my main.tf, variables.tf, and variables.tfvars.
I use the command "terraform apply -var-file="variables.tfvars"
terraform.tfstate.txt
variables.tfvars.txt
crash.log
main.tf.txt
variables.tf.txt

Hmmm. Each of those items about that are larger and bold were commented out with a "#" sign. Don't consider them as active code. Thanks.

@wayno123 Hi, Thank you for raising this issue. We have identified the bug. We are working on it and will keep you posted on this.

Thank you for the quick resolution. Is the fix available in any release yet? Can I get it?

It will be available with OV 6.20 . The release is scheduled for mid July.

Hi Wayne, We are closing this as it will be available with OV 6.2 release. Please reopen if it is still not working.


inference without speech

hi!
may i know how to do the inference without speech?
I've set the --no_speech but so that the output is [].
And when i do inference in activitynet and charades dataset, the output looks like it only considers the speech feature
Thank you!

Using --no_speech does not throw any token in the T5 encoder.
If you use a model that has been trained using speech, you instead need to provide empty subtitles which will give the T5 encoder an EOS token as input.
As you can see from the results in the VidChapters paper, for the current video chapter generation models, speech is the most important modality, hence it may be possible that on the downstream tasks, similar behavior can be found.

Using --no_speech does not throw any token in the T5 encoder. If you use a model that has been trained using speech, you instead need to provide empty subtitles which will give the T5 encoder an EOS token as input. As you can see from the results in the VidChapters paper, for the current video chapter generation models, speech is the most important modality, hence it may be possible that on the downstream tasks, similar behavior can be found.

Understand. While for the inference of activitynet and charades, the quality of asr feature is too bad.
is it possible to import the checkpoints of the vid2seq model in your last vid2seq paper with little modification into this pytorch inference script to do dense video captioning task, and should also change the t5 to t5 1.1?

It should be possible -- you would just have to make sure that the config matches the one of t5v1.1; I am also not sure if the change from Google ASR to Whisper without retraining would work well.


Add UML diagram for the project

Hi i am interested in contributing to this project , can you please assign it to me
@suyashgautam


"make teardown-secrets" doesn't seem to run to conclusion

↳ make teardown-secrets
Deleting secrets is not reversible, are you sure you want to delete the secrets? [y/N]:
y
{
    "ARN": "arn:aws:secretsmanager:us-west-2:514226198018:secret:ci-user-aws-keysd7db59ec-kS3uv6",
    "Name": "ci-user-aws-keysd7db59ec",
    "DeletionDate": "2020-10-16T16:22:44.857000+00:00"
}
{
    "ARN": "arn:aws:secretsmanager:us-west-2:514226198018:secret:terminus-k-stage-rds-d7db59ec-vfASuh",
    "Name": "terminus-k-stage-rds-d7db59ec",
    "DeletionDate": "2020-10-16T16:22:48.114000+00:00"
}

An error occurred (ResourceNotFoundException) when calling the DeleteSecret operation: Secrets Manager can't find the specified secret.
make: *** [Makefile:53: teardown-secrets] Error 123

There's an attempt to remove a secret that apparently doesn't exist.

This is happening because the sendgrid secret is only conditionally added but there's not a condition when it's removed. For each of these secret removal lines we can just add a || echo "Secret already removed" or something. To do that we'll need to change the fact that they are all chained together with && \ which I don't like anyway.
And to remove that I think we'll need to define the region and pager at the top, I think the only reason they are chained is for those vars:
export AWS_DEFAULT_REGION := <% index .Params `region` %>
export AWS_PAGER := ''

This would be nice too because then we can just use this region var in any place in the makefile where we were previously using the templated value, which is in quite a few places.

Things not destroyed properly with me:

aws secret - piggycloud2-me_cf_keypair

aws secret - piggycloud2-me-stage-vpn-wg-privatekey-E4979216

aws role - piggycloud2-me-eks-cluster-creator

Solution:
#1 - need a separate script together with scripts/import* one
#2 - need put a fix into Makefile
#3 - error as below
   $ aws iam delete-role --role-name $PROJECT-eks-cluster-creator       An error occurred (DeleteConflict) when calling the DeleteRole operation: Cannot delete entity, must detach all policies first.

AWS_DEFAULT_REGION doesnt actually work on some resources
some commands requires explicit --region , they somehow just doesnt honor the default region


Inconsistent stacktraces are printed for runtime errors

Consider the following main program:
package samples.main;

import ballerina.lang.system;
import ballerina.lang.jsons;

function main(string[] args){
    testJson();
}

function testJson() {
    json j = `{"name":"supun"}`;
    string adddress  = jsons:getString(j, "$.address");
    system:println(adddress);
}

Stacktrace when running the program by giving the package:
error in ballerina program: failed to get string from json. Jsonpath $.address does not match any element: No results for path: $['address']
	 at ballerina.lang.jsons:getString(samples/main/mainFunction.bal:12)
	 at testJson(samples/main/mainFunction.bal:7)
	 at main(samples/main/mainFunction.bal:6)

Stacktrace when running the program by giving the file name:
error in ballerina program: failed to get string from json. Jsonpath $.address does not match any element: No results for path: $['address']
	 at ballerina.lang.jsons:getString(mainFunction.bal:12)
	 at testJson(mainFunction.bal:7)
	 at main(mainFunction.bal:6)

Issue:

When a program/service is run by giving the package, the file paths are printed along with directory structure.
Native constructs are printed along with their package name, while other ballerina custom constructs are printed without the package name.
In the stacktrace, at testJson(mainFunction.bal:7) points to the function-invocation location, where as at main(mainFunction.bal:6) points to the function-declaration location.

Fixed in v0.89 release


Feature/svg cursors

Following up on @dannyrb's #612 PR (thanks for getting the ball rolling!):
Implements the following properties to BaseTool:
svgCursor
If svgCursor is set, the mouse cursor will be changed to the set value when the tool is active and mouseButtonMask === 1. I.e. the cursor for the primary mouse click is shown. Touch tools are unaffected.
SVG cursors are stored in assets/svgCursors.js, you can alter their mousePoint property to change the mouse position to either the top left (e.g. magnify) or center (e.g. pan) of the icon.
hasCanvasCursor
If hasCanvasCursor is true, the canvas will re-render on MOUSE_MOVE whilst the tool is active. This is intended for tools that need to render a cursor on the canvas, if the cursors could potentially be greater than 128x128 pixels in size. Canvas cursors should be used sparingly, and SVG cursors are preferable wherever possible.
hideDefaultCursor
If hideDefaultCursor is true and svgCursor is unset, the default cursor will be hidden when hovering over the cornerstone enabled element. You may want to do this if using canvas cursors, but as canvas cursors can be complex and dynamic, this is optional. See FreehandMouseSculpterTool to see an example of this being switched on/off during tool usage.
As an example I've implemented svg cursors for MagnifyTool, RotateTool and PanTool; as well as canvas cursors for BrushTool and FreehandMouseSculpterTool. Tools without custom cursors use the default mouse cursor.

As this builds on mine, I'll close #612. Tagged myself to review this. I'll try to give it a look before Monday and draft some good follow-up questions.

@dannyrb , sounds good, note this is probably quite outdated by now. Happy to discuss/scrap/rewrite anything, this was very much a playground at the time 👍 .

@JamesAPetts what's left for us to discuss on this one? It's been a while, haha.

Hey @dannyrb , not much. Iirc we didn't implement it purely because we don't have actual cursors for all tools, and it doesn't make sense to release it until its functional for all core tools. So more of an art issue.


More robust tests

Our test suite is lacking. We should get some full stack tests going along with more unit tests.
Missing tests:

[ ] dk.form
[ ] dk.selectOne

Closed with #346


can't set undefined value for control input or textfield

✖ CSS ISSUES → Post on https://github.com/Semantic-Org/Semantic-UI
✖ USAGE QUESTIONS → Use these dedicated resources:
Docs - http://react.semantic-ui.com
Chat - https://gitter.im/Semantic-Org/Semantic-UI-React
SO - https://stackoverflow.com/questions/tagged/semantic-ui-react?sort=votes
✔ BUGS → This form is required:
Steps
try set undefined value for Input or TextArea via state
Expected Result
input value is undefined
Actual Result
state.value === undefined but input value is old value
Version
0.71.2
Testcase
https://codesandbox.io/s/3q51385485
[Fork, update, and replace this pen to show the bug]:
https://codesandbox.io/s/3q51385485

In React, undefined is equivalent to not setting any value.  Set an empty string '' to clear an input or textarea's value.  Working fork:
https://codesandbox.io/s/3qymy21px5

Released in<EMAIL_ADDRESS>


Support for React 17

Updated package.json to support React 17

You're creating a PR on the wrong fork.


Removed of_rna transcription logic and tests for rna_transcription exercise

Fixes #231
Tested locally:
ruby rna-transcription/rna_transcription_test.rb
Run options: --seed 27636

# Running:

.........

Finished in 0.001411s, 6378.4550 runs/s, 6378.4550 assertions/s.

9 runs, 9 assertions, 0 failures, 0 errors, 0 skips

Also cleaned up example.tt to look more like the others. Everything was nested in some phantom block string...

This is missing all of the skips in the test suite, I think.

Oh snap... Good catch. Will update.

Updated

Would you mind squashing the commits?
http://blog.steveklabnik.com/posts/2012-11-08-how-to-squash-commits-in-a-github-pull-request

Never squashed before. Pretty cool.

:tada:


Fix usage of RAILS_ENV

Summary
This PR attempts to properly match NODE_ENV to RAILS_ENV in case of NODE_ENV not being set.
Pull Request checklist
Remove this line after checking all the items here. If the item does not apply to the PR, both check it out and wrap it by ~.

[ ] Add/update test to cover these changes
[ ] Update documentation
[ ] Update CHANGELOG file

Other Information
Fixes: #365

Waiting for @alecslupu.

@alecslupu Please let me know. If I don't hear from you, I'll close, and we can consider re-opening.

We can close this. As previously said, the issue was the fact that prod and test environment had different compiling strategies, that impacted the libraries i was used.
Now that i fully understand the issue, and i have a workaround, there is no reason to keep this PR open.


Plot state-dependent text annotations not refreshing when animating data

Describe the bug
Subject line says it all
Helpful additional information

Did VisIt crash: no
Did you get wrong results: yes

To Reproduce
Plot state-dependent text annotations not refreshing when animating database with the "play" button.  When viewing a single state statically, pressing "Reopen" does refresh the annotation to the value associated with the current state.
Attachments
I can give a (nice and small) database exhibiting the issue on the rz.
Desktop

OS and version: toss_3_x86_64_ib
VisIt Version: 3.3.1
Server info (if applicable): rzgenie

@nDimensionalSpace what if you go to VisIt's Preferences->Database tab and check Treat all databases as time varying?

Yup, that fixed it.  If reasonably possible, it probably makes sense to decouple the annotations from the re-reading of the mesh.

Unforunately, I don't think that it is reasonably possible...at least not with current way VisIt's notion of metadata is designed. Anything with annotations is part of the metadata.
Way back when, there was a decision made to optimize the way we handle metadata and read it only when necessary...not all the time. This was probably overkill for most situations but did avoid potentialy high cost of re-acquiring basically the same information as time step is changed for example. It has lead to no end of issues for users.
Eventually, we'll change VisIt so that it always keeps metadata up to date without incurring high cost to do so and many problems like this will go away.

Way back when ...  It has lead to no end of issues for users.

Pretty sure that's how it always is . . .


Structure and publish the walkthrough for each platform use case

Go through each sdk (especially higher demand sdk's like web, ios, and android) and go through the walkthrough's (or make them). With each walkthrough add steps to overcome issues that someone could run into (or that I run into).

[x] Web
[ ] iOS
[ ] Android
[x] Node
[x] Dotnet
[x] Go
[ ] Java
[x] Python

Aiming to finish this task by Feb 4th

@MichaelEdwardBlack  any updates on this?

Do issuers need a wallet? some don't and they provide issuance only.

Some don't but as of now when you create an account you're also creating a wallet so it just comes by default.

[ ] Add ecosystem steps to quickstarts (either through seed data or ad hoc ecosystem)
[ ] Prepare replit envs
[ ] ship ios and android quickstarts


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 6a895d2:
:clock10: Full build: the average full build time is 5 min(s) 6 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 6a895d2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:warning:Warning
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)
Details

docs-ref-conceptual/testing.md

[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.

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


hangouts: add warning for matrix.org homeserver

add a warning about matrix.org homeserver is blocking monero.social

after long time later the issue seems to be fixed


Add support for therminator combi boiler (log wood + pellet)

Hi there,
i just found your great repo!
Is it possible to get this module working for my therminator II combi boiler?
Can I use the module already now?
Based on the operation instructions there is a offset to use for the heating-circute readings of 200:
https://www.solarfocus.com/produkte/Waermepumpe/EcoManager-t_Modbus-TCP-Registerdaten_Anleitung.pdf
:)
Will look forward for this feature! i can test with my thermi II

Hi, great you like it!
The offset it currently not supported. In addition, the "Kessel" data is not yet supported. Everything else should be fine.
Am I correct, that you system is comprised of

Heating circuit
Buffer
Kessel
Right?

Let me try to add the Kessel information. However, as I have no means for testing, I'd need your feedback then ;)

Please try this beta release: https://github.com/LavermanJJ/home-assistant-solarfocus/releases/tag/v1.2.0-beta-1
Please make sure, that you enable "Kessel" or "Pelletsboiler" when adding the integration to your home-assistant instance.
If you use HACS, please check the "pre-release" button, to see this beta version.
Looking forward to your feedback!

BTW, the states / modes are only displayed by their number. No text yet ;)

Oha... That was fast.
Until now i'm missing the ModbusTCP setting. Solarfocus Support must activate it first... Than i can test ;) Thx!

I have installed the beta-4.
I'm getting this error in log:
 Unexpected error fetching solarfocus data: unsupported operand type(s) for &=: 'bool' and 'NoneType' Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 205, in _async_refresh     self.data = await self._async_update_data()   File "/config/custom_components/solarfocus/coordinator.py", line 54, in _async_update_data     success &= True and await self.hass.async_add_executor_job( TypeError: unsupported operand type(s) for &=: 'bool' and 'NoneType' Config entry 'Solarfocus' for solarfocus integration not ready yet: None; Retrying in background
I cant add the "Senec" intigration as described in the docu, cant find it. is it nessesary for this component?

oh - that "Senec" one is a copy and past error :innocent: ... I rectified that.
The error looks like there is no communication between home-assistant and the heating possible?

The 502 is the port that is usually used for Modbus TCP, and the 10 seconds are the interval home-assistant polls the data from your heating.
Could you please verify in your eco-manager touch that Modbus is ready?  Should look something like the following (just that you probably have different "buttons" at the bottom.

... if I remember correctly, I had a similar issue when I initially set up Modbus with the heating. I could ping it, but Modbus wouldn't work. What solved the issue for me, was restarting the heating (power off/on). Afterwards, it was working fine.

I deleted and recreated the intigration. Now i only selected the "Kessel" option. Now i get readings.
On the solarfocus itself, Modbus shows "Status 50003" and Function-Code 4

Readings wont get any values (beside there are "Pelletsboiler" instead of Log"

Unfurtunatly i cant restart right now. its currently working ^^ so i have to wait...

Unfortunately, I have no clue what that status code is. However, it seems that home-assistant is able to query it. The Function Code 4 is "read multiple input registers" which sounds good. Also the time since last query is < 10s which is the default query interval of this integration.
If the reboot helps to remove the red status code of the interface, I'd expect/hope that you receive valid readings. If on solarfocus side everything is green, but you still get invalid readings, it's due to this integration.
If the error persists, you should check with the solarfocus support what this status code means.

Got feedback from support.
The error code stands for "Invalid modbus mapping". I got a list of all status messages if u need it. (wont post here, maybe via some private message?)
:/

Would be great if you could send it to me via mail<EMAIL_ADDRESS>thanks!
Did the error persist after the restart? And did Solarfocus mention if it is caused by the integration?

I was wondering, what your eco manager touch version is? Currently the pysolarfocus lib queries Register 2400 + 11 registers which is valid for the latest version (202202). It could be, that due to an older version there are no 11 registers for the "Kessel" defined, hence, the error?
If you are familiar with python, you could clone the https://github.com/LavermanJJ/pysolarfocus repository, and adapt the respective constants?

My Kessel got Version 22.021

Maybe i found the issue!
In your mappings, u have on line 244 "11" counts, but in the "PB_REGMAP_INPUT" object you only map 10 registers.
And the address for "MESSAGE_NUMBER" is (based on found docu) "2404", u put there "2". All other registers adresses have to follow. there seems to be a gap between 2401 and 2404 :)
(see this document: https://www.solarfocus.com/de/partnerportal/pdf/open/UGFydG5lcmJlcmVpY2gtREUvUmVnZWx1bmdfZWNvbWFuYWdlci10b3VjaC9BbmxlaXR1bmdlbi9lY29tYW5hZ2VyLXRvdWNoX01vZGJ1cy1UQ1AtUmVnaXN0ZXJkYXRlbl9BbmxlaXR1bmcucGRm/117920/0/Lng_YSxpM245S30zMTc4W2Y8cVRRXWlJVWRQJDsv?serialNumber=68905 )
maybe this is the problem?
Greetings!

Correct - for the other items these gaps in the register numbers are not there in reality, therefore, I assumed they would not be there for "Kessel" as well. However, I tried to read out the registers with my system as well (even though I don't have a "Kessel"), and can confirm, that the gaps are there. In addition, the "2409 Kesselbetriebsart Therminator" seems not to be present
What did I do?
I read used plain Modbus to read out the registers:
from pymodbus.client.sync import ModbusTcpClient as ModbusClient

client =  ModbusClient(host="<...add IP-address...>", port=502)
client.connect()
result_input = client.read_input_registers(
            address=2400, count=12
        ).registers

print(f"Result: {result_input}")

This gives me (even without a Kessel) the following output:
Result: [0, 6, 0, 0, 0, 0, 65535, 65535, 133, 1300, 1300, 0]

65535 equals -1 which seems to be the default for non-existent
133 is the outdoor temperature -> 13.3
1300 is according to the spec the value for missing sensor.

It would be great if you could try to run this python script and post the output here.

copy the snippet in a file e.g. test.py
install python3
install the pymodbus lib: pip3 install pymodbus
open a command line and execute the file python3 test.py

I'll upload a fixed version of the pysolarfocus lib as well as the hess-integration tonight. But I'm not sure if it'll fix the issue. As still with the current version, I don't get such error :upside_down_face:

with beta-5 this is what it looks like for me:

btw. I renamed it from "Pelletsboiler" to Biomass boiler, as this seems to be the generic term used by Solarfocus, to cover pellets, wood chips, and log wood. Do you agree?

I Tried it, but still no result and getting error 50003 on Solarfocus.
"Biomass" seems to be correct so far ;)
Here is the readings with the help of WPF Modbus Examiner tool:

i can get the values.
But not with the custom component. do i need to install something more? i only add the solarfocus add-on from hacs.
And this is the components folder content:

Great that you were able to verify, that it works with the examiner tool 👍
The list of files within the custom component look good. Therefore, I guess the issue is in the way the pymodbus lib queries the data. As all the home-assistant mapping to names and converting values is transparent for the biomass boiler.
To localize the issue, it would be great if you could run the python script above and see if this works (please find the pymodbus client docs here and here). It seems like the biomass boiler is upset about the way we retrieve the data ... however, I have no clue why

if i run this script i get this error:
/home #   File "/home/test.py", line 7, in <module> /bin/sh: syntax error: unexpected newline /home #     ).registers /bin/sh: syntax error: unexpected ")" /home # AttributeError: 'ExceptionResponse' object has no attribute 'registers'
:/

A more explanatory description of the exception can be found here: https://www.simplymodbus.ca/exceptions.htm
However, I'm confused whether the error number (4) or the name (illegal value) are correct ... I assume the latter.
Did play around with the count attribute? For example, to only read 1 register?

I allready tried this. no success at all...
i also tried with other addresses, with and without "address=",...
from pymodbus.client.sync import ModbusTcpClient as ModbusClient

client =  ModbusClient(host="<my_solarfocus_ip>", port=502, log="info")
client.connect()

print(f"Connected: {client.is_socket_open()}")
# tested different settings...
#address=2400, count=1
#2400,1
#2400,count=1
req = client.read_input_registers(address=2400,count=1)
client.close()
print(f"Error: {req.isError()}")
print(req)
result_input = req.registers

print(f"Result: {result_input}")

@ragesoft
Hello,
I also tried to get this working with my Terminator II and encountered the same issue as above.
After some testing with pymodbus the problem seems to  be the unitID passed to the client.read_input_registers() function.
In the pymodbus source the UnitID or SlaveID defaults to 0x00. For some reason the Therminator II can't work with IDs smaller than 1.
By forcing the unit to 1 i got a valide response from the server.
from pymodbus.client.sync import ModbusTcpClient as ModbusClient

HOST = "*"

client =  ModbusClient(HOST, port=502)
client.connect()

print(f"Connected: {client.is_socket_open()}")

response = client.read_input_registers(address=1901, count=1, unit=1)
if response.isError():
    print(f"Error!")
else:
    print(f"Success!")
    print(f"Value: {response.registers}")

client.close()

If this also works for you i could implement this into the pysolarfocus package.

Kudos that you found the issue! I can confirm that it also works with the unit=1 for the eco manager-touch. PR to the pysolarfocus would be great 👍 You could re-use the unit=SLAVE_ID also for the read_input and read_holding.

So great man!
/home # python3 test.py
Connected: True
Error: False
ReadInputRegistersResponse (12)
Result: [726, 301, 0, 0, 0, 0, 43, 64537, 100, 0, 0, 0]

It works like a charm!
:) @LLukas22 thx for this finding! @LavermanJJ im looking forward to see an update on the homeassistant plugin :) great work!!!
Also works for Heizkreis (start at 1100 count 7) and puffer (start at 1900, count 5) :)
👍

Alright i will probably create a PR tomorrow.
I will also add in some error handling and better logging via the default python logging library.

I've built a pre-release containing @LLukas22 changes: https://github.com/LavermanJJ/home-assistant-solarfocus/releases/tag/v1.2.0-beta-6
Feedback welcome!

😞 somehow doesn't work for me (no updates happening) ... will see tonight, if I can spend some time to debut.

Im getting the same Error in my log
2022-10-02 14:38:10.004 ERROR (SyncWorker_2) [root] Connection to modbus is not established!
2022-10-02 14:38:10.004 ERROR (SyncWorker_2) [root] Connection to modbus is not established!
2022-10-02 14:38:10.005 ERROR (SyncWorker_2) [root] Connection to modbus is not established!
2022-10-02 14:38:10.005 ERROR (SyncWorker_2) [root] Connection to modbus is not established!
2022-10-02 14:38:10.005 ERROR (SyncWorker_2) [root] Connection to modbus is not established!
2022-10-02 14:38:10.006 ERROR (SyncWorker_2) [root] Connection to modbus is not established!
2022-10-02 14:38:10.006 ERROR (SyncWorker_2) [root] Connection to modbus is not established!
2022-10-02 14:38:10.006 ERROR (SyncWorker_6) [root] Connection to modbus is not established!
2022-10-02 14:38:10.006 ERROR (SyncWorker_8) [root] Connection to modbus is not established!
2022-10-02 14:38:10.006 ERROR (SyncWorker_8) [root] Connection to modbus is not established!
2022-10-02 14:38:10.006 ERROR (SyncWorker_8) [root] Connection to modbus is not established!
2022-10-02 14:38:10.006 ERROR (SyncWorker_8) [root] Connection to modbus is not established!
2022-10-02 14:38:10.007 ERROR (SyncWorker_8) [root] Connection to modbus is not established!
2022-10-02 14:38:10.007 ERROR (SyncWorker_3) [root] Connection to modbus is not established!
2022-10-02 14:38:10.007 ERROR (SyncWorker_3) [root] Connection to modbus is not established!
2022-10-02 14:38:10.007 ERROR (SyncWorker_3) [root] Connection to modbus is not established!
2022-10-02 14:38:10.007 ERROR (SyncWorker_3) [root] Connection to modbus is not established!
2022-10-02 14:38:10.007 ERROR (SyncWorker_0) [root] Connection to modbus is not established!

The problem seems to be that the modbus connect methode is never called.
Looking into the init.py the modbus_client is created but never connected to the server. I also couldn't find any other function that would connect it to the server.
I guess the error message is correct because we actually aren't connected to the server.

Good catch! 👍  I've added it to the SolarfocusDataUpdateCoordinator as the coordinator handles the retrieval of the data via the lib. and provided a new release

Just updated, the connection is working now 👍

The values also seem to be valid, exept for the boiler  ash container (maybe the sensor is blocked or dirty).
Im gonna take another look tomorrow.
@ragesoft Would be great if you could also validate that the data retrieval is working for you.

@LLukas22  Same here. Looks a little strange. But I get data now!

@LavermanJJ  Is it possible to subsequently change the extensions for heating circuit and buffer on the integration?

btw:
https://github.com/LavermanJJ/home-assistant-solarfocus/blob/5a9c0f4878531223f48a85ecce520497ea79d398/custom_components/solarfocus/strings.sensor.json#L203
There is a typo. Status 301 missing because of none closing ' " '  and new line :)

Typo is fixed.
What changes for heating circuit and buffer are you referring to? Are the register numbers different? Or are you referring to the state messages (offset +200)? The latter should be present and the former #worksOnMyMachine 🤣 ... no seriously, if there is something with the registers, that would mean eco manager-touch and terminator are incompatible, i guess 🤔

I mean, if i initial add the solarfocus intigration, i got this options, but i first only selected boiler. Now i want to extend my integration with the other options. only change right now is to delete current integration and add it back with new settings (i guess) :)

That's right 🙈 ... that would need the option flow. However, so far I couldn't get it to work. Therefore, the remove and re-add is the only option for the moment. But as the entities are named the same, the history won't be lost 👍

nice. i see. :)
One last issue (i hope) with this integration for the therminator:
it seems some readings are not correct? What does the "heating state" shows? in my setting it toggles quite often... but this numbers are no known status numbers... (maybe there is some miss-match in mapping to the "heating mixer value"??)
"buffer mode" is a number instead of text
values for "Solarfocus Heating supply temperature" seems to be correct.
:)

Buffer mode should be buffer state, therefore the number to text mapping isn't working.
Seems that there are a few things twisted 😟
Could you upload a screenshot of your modbus analyzer of buffer and heating registers to ease debugging?

heating circuit 1 and buffer 1:

Thanks for providing the screenshots. I've attached the same from my system. As you can see the issue is with the "missing" /empty registers on my system. As the pysolarfocus lib is currently working with offsets, that doesn't work. Therefore, the two implementations (Therminator, Vampir) aren't compatible. We'd need to

take a deeper look into the pymodbus lib, or
Adapt the implementation of pysolarfocus to either

Use two different register definitions to continue the support of range requests
Switch from range requests to query distinct addresses

Check with Solarfocus, why they have different implementations

In my oppinion the best option we have is to change the pysolarfocus library to work with distinct addresses.
I dont think pymodbus has a feature that would allow us to dynamically parse/request addresses.
Solarfocus probably wont change their modbus implementation, there are already system build depending on the existing address  layout.
Changes i would recomend for the pysolarfocus library:

We could implement an enum that defines if we connect to a Terminator or a Vampir system and pass it to the constructor (In homeassistant that could be a dropdown while adding the integration)
All address ranges shared between the systems dont need to be changed. Depending on the passed enum specific addresses definitions are loaded
Every address definition should be changed to have the following values:

Address
Range (Read 1 or multiple registers)
Type (Int/UInt etc.)
Name/Key (A unique Key to assign a mapper for this value)

Sounds like a plan 👍 I did not do any performance testing, but I assume the range requests being less overhead and hence faster and "cheaper". Therefore, if we anyway have to differentiate between the two systems, why not provide two definitions than to support ranges for both?
While I agree, that Solarfocus will most likely not adapt it, I guess it will nevertheless make sense to make them aware.
Regarding implementation:

I will provide Modbus Examiner screenshots of the VampAir registers, to identify which registers are affected.
@ragesoft can you confirm (via Modbus Examiner), that for Therminator the register values fit 100% to the spec?
I could adapt the integration config-flow to provide the selection ... maybe I have a run and fix the option-flow as well 😄
@LLukas22 would you want to give the pysolarfocus changes a try and provide a PR for that?

What about the multiplier information? Would that also be locate in the address definition? (or maybe I did not yet fully understand your idea)

Regarding the performance of range reads compared to multiple single address reads you are probably right.
I will have another look at the implementation after work, i wanted to abstract the single components of the system into objects.
These objects will define the address range and a parse function. To implement the different Solarfocus devices we only have to verwrite these component objects, define the range and overwrite the parse function.
I will porbably try to implement this for the existing VampAir implementation first, then i can evaluate if we actually need it or if we should stick with the dictionary definitions.

Yesterday my Integration stoped updating the values. A Quick check on termi Shows error 32601. There where more then 260.000 query until this point. The "Master" counter Shows "5" => maybe there is some problem with the latets update? It reconnects to often? It dont close connection correct?
I Updates to Main 1 or 2 days before this error.
Only reboot has solved the error (until now)
Also i want to change the pull intervall, how can i achiv this? Only through delete and recreate?

I'm still struggling with the "options" flow, so for now delete and re-create is the only way unfortunately.
Regarding the reconnects: I haven't experienced this myself so far. Might be what you assume, not sure.

@ragesoft
I can't test this right now, but it's a possibility that the querry counter overflows or some stuff like that. (I've read throught some of their documentation and it's not really helpfull)
The connection should probably be "self-healing", meaning that we automatically diconnect and reconnect when we don't get valid data from the server. This is probably a seperate issue with no connection to the Therminator compatability. You should probably create a new issue for this.

I've merged #14 - please let me know, if (after re-adding the integration) things are working with the Therminator.

Danke für die Super integration in Homeassistant.
Bei der Target Temperatur Boiler wird ein Falscher wert angezeigt.

Thanks for reporting, I've opened a new issue for that #15

@LavermanJJ This seams to work now, the only thing throwing me off are the numeric values for the states e.g. the Solarfocus Buffer state. These should probably be translated or mapped to meaningful strings.

@LLukas22  I've fixed all, but the biomass boiler status (https://github.com/LavermanJJ/home-assistant-solarfocus/releases/tag/v1.2.0-beta-12). For the moment that is still in German.

@LavermanJJ
🤞 Everything seams to work now.
@ragesoft
Would be great if you could also test the v1.2.0-beta-12 release.

Hi @LLukas22 and @LavermanJJ
I've just installed latest Version. Seems to work finde. All stats and text are there. While creating New Integration i can choose now the Model. Nice work.
Greetings


Sharing

Should have more options for collaboration!!

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

ID: 21f47742-00cf-eac0-7880-18eb5e988003
Version Independent ID: 0010dce8-b1e0-b1b4-9840-ff2c3e32b53e
Content: Share Power BI dashboards and reports with coworkers and others - Power BI
Content Source: powerbi-docs/service-share-dashboards.md
Service: powerbi
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @alokgupta1986 -- Thanks for your comment! It seems more like feedback about the product than about the documentation. Here's the Power BI product feedback site: https://ideas.powerbi.com/forums/265200-power-bi. Search to see if anyone else has raised a similar issue. If they have, you can vote for it. If not, create a new issue. Good luck!

Hi,
Some users, with permissions keep on having pop up messages requesting access when they open reports whereas they already have access. "x Let's get you permission to view this report".
The accesses have been removed and setted back nothing changed
Any advice?


Use build options to determine which commands to include

Not all demo systems support the same command interfaces/implementations.
Right now, builds for each demo system involve manually changing which command/*.zig files are used. This process can be streamlined to use build options, such that a CI/CD pipeline can generate build artifacts for each of our demo systems.

Resolved with a configuration file rather than build options.


Filter parsing - can't have multiple != conditions on same field in filter

Retina's filter validation marks multiple conditions on the same protocol.field as "duplicate pattern". This mistakenly marks patterns like "tcp.port != 80 and tcp.port != 100" as invalid filters.
Checking for duplicate or incompatible patterns may still be valuable, so this would ideally change/correct how fq_pattern.has_duplicate_fields() is calculated.

Fixed by multi-subscription


E2E test for inventory update on client firstboot

There's still an issue where you have to SIGTERM (Ctrl-C) the script after it's done, but the test needs to be ran manually anyways so it's a non blocking issue for now.


non-compliant form encoding

Just an FYI, the code here https://github.com/ralscha/blog2019/blob/master/java11httpclient/client/src/main/java/ch/rasc/httpclient/File.java#L69-L91
is not compliant with the RFC. From the RFC:

The boundary delimiter MUST occur at the beginning of a line, i.e., following a CRLF, and the initial CRLF is considered to be attached to the boundary delimiter line rather than part of the preceding part. The boundary may be followed by zero or more characters of linear whitespace.

The code adds a CRLF at the end of the file, so the 'part end' becomes compliant.
It should be adding a CRLF as part of the first boundary, otherwise the data stream looks like:
headerCRLF
header...CRLF
--boundary...

when it should be
headerCRLF
header...CRLF
CRLF--boundary...

I think most parsers will deal with this regardless, but I felt it should be noted.

Is this correct?
https://github.com/ralscha/blog2019/commit/512f648a55237d2c0aff70d33d2879d63ce4aa1d

That will fix it afaict.
I think you could reorganize the code to add it to the separator and stop adding it after the file bytes for additional clarity on what is happening but it isn’t strictly necessary.


Self opening select input

Hi,
Firstly thanks for x-editable, it's really great!
The inplace select input is very nice, would it be possible to have an option to automatically show the list of options when you click on it?
Currently, you have to click three times:
(1) to toggle editing mode
(2) to show the list of options
(3) to select from the list
I like to be able to change the value in just two clicks (1) and (3)

Hi alltonp, did you solve your issue?? I have the same problem now. Thank you very much.

+1

(function ($) {
    "use strict";

    var autoSelect = function (options) {
        this.init('autoSelect', options, autoSelect.defaults);
    };

    $.fn.editableutils.inherit(autoSelect, $.fn.editabletypes.select);

    $.extend(autoSelect.prototype, {
    	/**
        Activates input: sets focus on the first field.

        @method activate()
       **/
       activate: function() {
    	   this.$input.focus().attr("size", this.sourceData.length);
       }
    });
    autoSelect.defaults = $.extend({}, $.fn.editabletypes.select.defaults, {});

    $.fn.editabletypes.autoselect = autoSelect;

}(window.jQuery));

Hi michail, where should I put this snippet??  In the "ace-editable.min.js"?? or in "bootstrap-editable.min.js". I've tried both with no success.
$sel.editable({
type: 'select',
...
autoselect: true
});
Is it also necessary this??

@xabedamadev , no. this code define new editable type named 'autoselect'
put code after x-editable core files
$sel.editable({
type: 'autoselect'
});

Ok, now it works, thank you, but when I click an item in the select, nothing happens. Do you know why?? Thank you!!

@xabedamadev  what did you expect to been happends?

When click the item, the select should close and the item is selected.

Item is selected already. Other functionality you can do by youreself ;)

So I think I have a problem, bacause when I click the item, the select do not close and the item is not selected... Does anybody have the same problem??
Thank you very much michail.


context error on iOS 4.2

Hi! I'm debugging my application on iOS 4.2 GM seed, and I obtain the following error on the simulator console:
: CGContextClipToRect: invalid context 0x0
: CGContextGetClipBoundingBox: invalid context 0x0
: CGContextConcatCTM: invalid context 0x0
: CGContextDrawImage: invalid context 0x0
: CGBitmapContextCreateImage: invalid context 0x0
: CGContextClipToRect: invalid context 0x0
: CGContextGetClipBoundingBox: invalid context 0x0
: CGContextConcatCTM: invalid context 0x0
: CGContextDrawImage: invalid context 0x0
: CGBitmapContextCreateImage: invalid context 0x0
Error seems issued by LeavesCache.m on the following member function

(CGImageRef) imageForPageIndex:(NSUInteger)pageIndex

I just start to debug the application, so these are just my first impressions.
Hope to a fix soon, or maybe an hint to fix the so problem so I'll be able to commit the fix.
Thx,
Diego

Almost a year later ... thanks for debugging this issue.


Fix template generation

Type of Change

Build Scripts: Template generation

What issue does this relate to?
N/A
What should this PR do?
Updates do-vue & do-bulma, removes document.write calls causing issues in dev.
What are the acceptance criteria?
npm run dev can build and run the tool without issue.

Leaving this as a draft until the community PR gets merged and makes it to production, when this will resolve itself.


Add shadows utilities

Added shadow utilities for elements like divs, images, etc.
Issue
https://github.com/twbs/bootstrap/issues/23437

This branch seems a little whatever, no offense. Please make sure tests pass before pushing.

i don't see the need for .box-shadow for every declaration
just the .box-shadow-[position] would be enough to me
plus imo the variables should be in variables.scss

@gijsbotje Nice suggestions. I did both changes.

This is too much for Bootstrap—we don't need every direction for shadows. We need uniform and consistent shadows across our components, as well as a handful of utility classes. Thanks, though.


Update viva-connections-overview.md

https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/2946
Fixed link

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LoreenLa PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Add Node status to Arc PG overview

New feature - display node status on the overview page of the Arc PG dashboard

Fix is in the release branch


Interaction Responding Error

Summary
When using interaction.response.send_message() and a ctx.send or blahblah_channel.send() if yo put the channel.send before it acts like an interaction and throws a already responded error
Reproduction Steps
I have a ticket_channel.send() then an interaction.response.send_message()
then when i press the button i get an error: interaction already responded to.
Minimal Reproducible Code
@discord.ui.button(label='\U0001f4e9 Create Ticket', custom_id="ticket", style=discord.ButtonStyle.secondary)
    async def ticket(self, button: discord.ui.Button, interaction: discord.Interaction):
        with open("data.json") as f:
            data = json.load(f)

            ticket_number = int(data["ticket-counter"])
            ticket_number += 1

            ticket_channel = await interaction.guild.create_text_channel("ticket-{}".format(ticket_number))
            await ticket_channel.set_permissions(interaction.guild.get_role(interaction.guild.id), send_messages=False,
                                                     read_messages=False)

            for role_id in data["valid-roles"]:
                    role = interaction.guild.get_role(role_id)

                    await ticket_channel.set_permissions(role, send_messages=True, read_messages=True,
                                                         add_reactions=True,
                                                         embed_links=True, attach_files=True,
                                                         read_message_history=True,
                                                         external_emojis=True)

                    await ticket_channel.set_permissions(interaction.user, send_messages=True, read_messages=True,
                                                         add_reactions=True,
                                                         embed_links=True, attach_files=True,
                                                         read_message_history=True,
                                                         external_emojis=True)

                    pinged_msg_content = ""
                    non_mentionable_roles = []

                    if data["pinged-roles"] != []:

                        for role_id in data["pinged-roles"]:
                            role = interaction.guild.get_role(role_id)

                            pinged_msg_content += role.mention
                            pinged_msg_content += " "

                            if role.mentionable:
                                pass
                            else:
                                await role.edit(mentionable=True)
                                non_mentionable_roles.append(role)

                    message_content = "Please wait and a moderator will assist you! To close this ticket press the `Close` button below"
                    em = discord.Embed(
                        title="New ticket from {}#{}".format(interaction.user.name, interaction.user.discriminator),
                        description=f"{message_content} {pinged_msg_content}", color=0x00a8ff)
                    view = Close()
                    await ticket_channel.send(embed=em, view=view)

                    data["ticket-channel-ids"].append(ticket_channel.id)
                    data["ticket-counter"] = int(ticket_number)
                    with open("data.json", 'w') as f:
                        json.dump(data, f)

                        em3 = discord.Embed(title="TMS Tickets",
                                            description="Your ticket has been created at {}".format(
                                                ticket_channel.mention),
                                            color=0x00a8ff)
                        await interaction.response.send_message(embed=em3, ephemeral=True)

Expected Results
It to work, and it to send the message to the ticket channel then respond to the interaction
Actual Results
It responds then sends the ticket message 2 times, and throws an error saying its already been responded to. (this is a persistent view btw)
Intents
discord.Intents.all()
System Information
Python v3.9.6-final

discord.py v2.0.0-alpha

discord.py pkg_resources: v2.0.0a3637+g0637a628

aiohttp v3.7.4.post0

Checklist

[x] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
No response

Can you shorten your MRE to be an actual minimal example, it would help you debug this as well as us (we don't have half of these variables)

Fixed with return await respond statement


invalid arithmetic operator

i'm getting
iosnoop-perf -s | ./dsk-peek.sh -v /dev/sda >out &
[1] 1640645
./dsk-peek.sh: line 46: ((: skp = end.*512: syntax error: invalid arithmetic operator (error token is ".*512")
./dsk-peek.sh: line 54: [: : integer expression expected
./dsk-peek.sh: line 54: [: BYTES: integer expression expected
on proxmox pve 8.1.4 / debian 12 server

it appears, it's being caused by the first two lines of ioperf-snoop  output, as they contain no data but header information
iosnoop-perf -s | ./dsk-peek.sh -v /dev/sda >/dev/null
blk=end.
./dsk-peek.sh: line 51: ((: skp = end.*512: syntax error: invalid arithmetic operator (error token is ".*512")
./dsk-peek.sh: line 59: [: : integer expression expected
blk=BLOCK
./dsk-peek.sh: line 59: [: BYTES: integer expression expected
this can be fixed by 2 read commands before reading the data
skip first 2 lines of iosnoop output, as it does not contain data
read;read
while read -r tm app pid op dsk blk sz l

Thanks for your input.
I've added fix, as you suggested.


Update README.md with ask-and-tell example

Hi,
I think it would be good to add a quick ask-and-tell example to the README.md. This is something quite unique about scikit-optimize that we should highlight more.
Also, not sure why the README now includes twice the CircleCI badge.

Also, not sure why the README now includes twice the CircleCI badge.

I think we made a mistake during the conversion to restructured text.


adding the word "in" to "...lets you deliver a set of Tweets that you’re in...

adding the word "in" to "lets you deliver a set of Tweets that you’re interested in to a Dropbox folder."

Hi @lucaslin, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Improve README to include examples or a better description of the project

The README will now be what users see first when visiting the PyPi package page, so we should make sure it is a bit more informative to end users (so far it only has information for contributors, not users).

Is this issue resolved now?

I guess before releasing a 1.0 it might be good to go through the docs and probably make some improvements, like adding some example, more like the channels user guide (although if we consider only the API spec part it might make less sense). That said, we can also create more specific issues. I leave it to you to close or leave as you prefer.

Good point. Right now, I feel it will be easier to manage if the requirements are listed in a single issue (here). So, here is a short list of requirements copy-pasted from what you wrote above (please feel free to extend it):

[ ] Add usage examples (like the channel user guide)


Check for None

What does this PR do?
The CV models test showed a failure when args is defined but kwargs are none (why only them, unsure as gpt2 didn't face this, but I digress). This check makes sure that the padding is only checked for if the value is not None (otherwise we get an error about how we can't run it w/o a tensor-like item)
Fixes # (issue)
Failing test on main
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[x] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@SunMarc

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


DOC: add nlargest/nsmallest to API.rst

At the moment there's no easy way to partially sort the DataFrame to get top N rows, could be useful feature.
There's partial sorting available for numpy within the bottleneck library.
Here's link to SO question about numpy partial sort - http://stackoverflow.com/questions/10337533/a-fast-way-to-find-the-largest-n-elements-in-an-numpy-array

http://pandas.pydata.org/pandas-docs/stable/basics.html?highlight=nlargest#smallest-largest-values
(this is even faster than partsort as its grabbing the top values)
though these are not mentioned in API.rst. want to do a PR to add them?

Ok, but this one is for Series, my issue was about getting top N rows from DataFrame ordered by columns I want, similar to SQL select top (@N) * from <Table> order by <col1> asc, <col2> desc without actually sorting the whole dataset

df.apply(Series.nlargest)

Thanks again, but it still will not work if I want to sort by several columns. As an example - suppose I have this dataset:
	A	B
0	1	5
1	2	5
2	2	4
3	2	3
4	3	2
5	3	1

And I want to take top 4 records ordered by A desc, B desc (rows 5, 4, 1, 2)

In [42]: df.sort(['A','B'],ascending=[0,0])
Out[42]:
   A  B
4  3  2
5  3  1
1  2  5
2  2  4
3  2  3
0  1  5

I don't think there is an easy way to do this w/o a full-sort. partsort (and most other algos operate on a 1-d array).


Add some export snippets

I always tend to modulate my projects (reference), so I usually have a index.js file with something like this:
export { default as Component } from './component';
export { default as actions } from './actions';
export { default } from './reducers';
export { default as sagas } from './sagas';

I have this snippets in my VSCode, but I decided to add in this project because maybe they'll be useful to someone else.

@arthurdenner Thanks for contributing :) I will update it immediately 💃


add join with enter key feature to join modal

created event listener and new callback function so user can join with "enter" key press after entering a valid video session code
commented a console.log code found
Close #80


[SPARK-33107][SQL] Remove hive-2.3 workaround code

What changes were proposed in this pull request?
This pr remove hive-2.3 workaround code.
Why are the changes needed?
Make code more clear and readable.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing unit tests.

Test build #129609 has started for PR 29996 at commit c3460a3.

Thank you for the clean-up, @wangyum !

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34212/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34212/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/34212/
Test PASSed.

Thanks, @wangyum.

cc @juliuszsompolski FYI

Test build #129609 has finished for PR 29996 at commit c3460a3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/129609/
Test PASSed.


Feature/scorejs code

Fixed some bugs Logic.js
Added score.js to allow adding/clearing  scores to localStorage
added Li data answer to CSS file
Added WAV files into the html
updated Read me file

Fixed some bugs Logic.js
Added score.js to allow adding/clearing  scores to localStorage
added Li data answer to CSS file
Added WAV files into the html
updated Read me file


[Bug] Start as Exe not working as expected

It should be a combo box to enable/disable the setting or use the server default.
But is seems the server default is not used when the flag is not set for a table.

When other backglass settings changed, the startExe flag is saved with "0".

Fixed with 3.4.0


if with default selected when init, the is a bugs while select others

As below image show:

@denghuiquan I'd suggest trying to replicate the bug using the latest code. Unfortunately the demo doesn't appear to be running the latest code so will contain fixed issues.

We'll need more details on how to reproduce, or we'll have to close this one.

this is how to reproduce, and relative to one selected but title tell none selected.

I can only reproduce this on the old demo, not on mine which is newer (but still outdated)
https://chris--jones.github.io/angular-2-dropdown-multiselect/

ok, i don't not your newer demo link before, thanks, i can't reproduce  that as well, sorry for bother you.
BTW, your new one looks nice jobs, thanks.


column() must be specified after wrapper()

@vinkla Love the new column() method. I noticed that if you specify column() followed by wrapper(), wrapper() will override column(); example:
Image::make('Logo')
  ->instructions('For use on light backgrounds')
  ->column(50),
Image::make('Logo on dark')
  ->instructions('For use on dark backgrounds')
  ->column(50)
  ->wrapper(['class' => 'dark-bg']),

In this case ^, the second Image field won't be 50%. The simple solution is to move column() after wrapper():
Image::make('Logo')
  ->instructions('For use on light backgrounds')
  ->column(50),
Image::make('Logo on dark')
  ->instructions('For use on dark backgrounds')
  ->wrapper(['class' => 'dark-bg'])
  ->column(50),

and I can confirm that in this case ^, the class of 'dark-bg' still gets applied. So this isn't really a bug, but perhaps you could add a note to the README?

I'm glad you like it, Kaelan! 🙌
If you're willing, please send a pull request to update the column section in the documentation.

Another idea is to support merge the existing settings in the wrapper method. This is the way.

Added in 13.7.1


Support for downloading song in Lossless format?

Duplicate of https://github.com/glomatico/gamdl/issues/10.


Ad-hoc modules or DCG and modules do not work

Was trying this Prolog text. It did consult:
foo:bar --> jack:jill.

But then I am not sure whether the result is correct, I am expecting a different head:
listing.
% :/4
:(foo,bar,_1,_2) :- jack:jill(_1,_2).

Obviously it doesn't run:
?- foo:bar(1,2).
uncaught exception: error(existence_error(procedure,bar/2),top_level/0)

Fixed. Thanks for your report.


[UI][Machine] Sending Hosts and HostUpdated JSON attributes when creating machine

v0.13.1
Steps:

Create an new Machine/Host

Results:
Sending a hosts and hostsUpdated JSON attributes that the API ignores
Expected:
Don't send those

v0.15.1
Verified fixed


Add index page

Coverage decreased (-16.6%) to 59.146% when pulling 7cb6f419fb21cbf9f9d3a8548abbbc82ad451206 on human-friendly-index into e06c0d3f5de509dd82b2fb0bf9715e3228b27b56 on main.


Detect missing section Security considerations

This section is mandatory and if missing the bot can report it.

Fixed in https://github.com/ethereum/eipw/commit/b610d17d3730d1932019b2684337d299794a7c85


Import-Module : Could not load file or assembly

Have you seen this error?

@jaredneedell I'm getting the same error on Windows Server 2012 R2.

Your screenshots are showing Install-Module not Import-Module.
Assuming that you have this module installed in Powershell modules path you should be able to run:
Import-Module gmail.ps

An example for using the script is, I've bolded the part that includes the referenced assembly
Import-Module ".\gmail.ps\Gmail.ps.psm1"
Add-Type -Path '.\gmail.ps\AE.Net.Mail.dll'
$userName = "[YOUREMAILHERE]"
$password = "[YOURPASSWORDHERE]"
$nc = New-Object System.Net.NetworkCredential($userName, $password);
$creds = new-object PSCredential($nc.UserName, (ConvertTo-SecureString $nc.Password -AsPlainText -Force))
Invoke-GmailSession -credential $creds -ScriptBlock {
param($gmail)
$msgs = $gmail | Get-Mailbox | Get-Message -unread
foreach($msg in $msgs)
{
$msgs.From
$msgs.Body
}
}


[Do not merge yet] Adds 'Mizuki', an SPH/Nbody code based on FDPS

Very simple C++ SPH+Nbody code, based on the nbody+sph c++ example of FDPS.
My intention is for this code to be used as a 'kick' code / field code, not as much as an integrator (though it can and does work as one).
Not finished yet so don't merge yet.

Renamed the code

(closed, reopened as #942)


Configurable bcrypt rounds with a min value

also bumped default from 10 to 12 runs as I have heard criticism of using 10 but honestly this is a complicated matter as per the link in the comments.

sorry about all the edits, was a hasty PR, should be good now

resubmitted with a cleaner pr


feat(KONFLUX-926): Track git commit status

Description
Adding additional metadata to our JSON result file - adding info about e2e-tests and infra-deployments repo.
Issue ticket number and link
https://issues.redhat.com/browse/KONFLUX-926
Type of change
New feature (non-breaking change which adds functionality)
How Has This Been Tested?
Will run CI in this PR

/test ?

/test load-test-ci-10u-10t

/lgtm

/retest

/lgtm

/approve


dihydropterin oxidase activity (GO:0004154) parentage

From the EC xref on this:
[Term]
id: GO:0004154
name: dihydropterin oxidase activity
namespace: molecular_function
def: "Catalysis of the reaction: a 7,8-dihydropteridine + O2 = a pterin + hydrogen peroxide." [GOC:mah, PMID:1745247, PMID:6815189]
xref: EC:1.5.3.-
is_a: GO:0016491 ! oxidoreductase activity
it's parent should be changed to this:
[Term]
id: GO:0016645
name: oxidoreductase activity, acting on the CH-NH group of donors
namespace: molecular_function
def: "Catalysis of an oxidation-reduction (redox) reaction in which a CH-NH group acts as a hydrogen or electron donor and reduces a hydrogen or electron acceptor." [GOC:ai]
synonym: "oxidoreductase activity, acting on the CH-NH group of donors, other acceptors" NARROW []
xref: EC:1.5.-.-
is_a: GO:0016491 ! oxidoreductase activity

will also remove 3 digit ec for 4154, as it is a grouping term.

EC:1.5.3.- maps to GO:0016647  oxidoreductase activity, acting on the CH-NH group of donors, oxygen as acceptor, so I'll  use THAT as parent rather than : GO:0016645 name: oxidoreductase activity, acting on the CH-NH group of donors

Thanks Harold.


Empty namespace xmlns="" added to several dependentAssembly nodes in web.config during DNN upgrade

Description of bug
Module installed on upgraded DNN 9.4.0 RC1 can stop working with exception like this:
Could not load file or assembly 'System.Web.Http.WebHost, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
Steps to reproduce
List the steps to reproduce the behavior:
Clean DNN install scenario

Clean install DNN 9.4.0 RC1 using DNN_Platform_9.4.0-rc1.20742_Install.zip.
Install module that depend on eg. System.Web.Http.WebHost.

Here is 2sxc install package that works on DNN 7.4.2 - 9.4.0 RC1 2sic_2SexyContent_09.43.02_dnn940comp_Install.zip

And module to page. Module is working.

Upgrade DNN scenario

Clean install DNN 7.4.2.
Upgrade to DNN 9.4.0 RC1 using DNN_Platform_9.4.0-rc1.20742_Upgrade.zip.
Install module that depend on eg. System.Web.Http.WebHost.

Here is 2sxc install package that works on DNN 7.4.2 - 9.4.0 RC1 2sic_2SexyContent_09.43.02_dnn940comp_Install.zip

And module to page. Module is not working. We get exception:

Could not load file or assembly 'System.Web.Http.WebHost, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
Current result
In upgraded web.config, several nodes for dependent assemblies have namespace attribute empty xmlns="".
      <dependentAssembly xmlns="">
        <assemblyIdentity name="System.Web.Http" publicKeyToken="31BF3856AD364E35" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>
      <dependentAssembly xmlns="">
        <assemblyIdentity name="System.Net.Http.Formatting" publicKeyToken="31bf3856ad364e35" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>
      <dependentAssembly xmlns="">
        <assemblyIdentity name="System.Web.Http.WebHost" publicKeyToken="31bf3856ad364e35" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>
      <dependentAssembly xmlns="">
        <assemblyIdentity name="System.Web.WebPages.Razor" publicKeyToken="31bf3856ad364e35" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>

Expected result
In clean install web.config, nodes for dependent assemblies do not have namespace attribute that is empty.
      <dependentAssembly>
        <assemblyIdentity name="System.Web.Http" publicKeyToken="31BF3856AD364E35" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>
      <dependentAssembly>
        <assemblyIdentity name="System.Net.Http.Formatting" publicKeyToken="31bf3856ad364e35" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>
      <dependentAssembly>
        <assemblyIdentity name="System.Web.Http.WebHost" publicKeyToken="31bf3856ad364e35" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>
      <dependentAssembly>
        <assemblyIdentity name="System.Web.WebPages.Razor" publicKeyToken="31bf3856ad364e35" />
        <bindingRedirect oldVersion="<IP_ADDRESS>-<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
      </dependentAssembly>

Screenshots
If applicable, provide screenshots to help explain your problem.
Error log
Paste the error log that is related to this issue.
Additional context
Add any other context about the bug that may be helpful for resolution.
Affected version

[x] 9.4.0 release candidate

There is a currently known limitation with 2sxc and the 9.4.0 RC.  I know that Daniel is working on a fix.

@mitchelsellers actually @tvatavuk is working on it, and ran into this separate problem. The issue seems to be, that the DNN installer adds more unnecessary entries, which then cause this. So this is a DNN-update issue, unrelated to the Dependency Injection issue of .net core.

@iJungleboy Thanks for this, it looked a bit different, I'll have a look at this to see what we can find.
We don't have any changes in this area that are part of the 9.4.0 release, so this one is a bit odd.

@mitchelsellers just fyi: as far as I understood @tvatavuk a clean 9.4 doesn't have this problem, but an upgraded one does.

I can replicate this @tvatavuk, @iJungleboy. It appears that you don't need to install 2sxc to get these xmlns="" in the web.config. They happen regardless. But I've also seen this in other 9.x installations. And those just work fine. Is there anything you can think of that causes the 2sxc to fail because of this?

Also: I've tried the same with 9.3.2 and this is also the same result. So it looks like this wasn't introduced in 9.4.0 but earlier.

@donker thx for verifying. @tvatavuk will give some more infos, but 2sxc uses Razor and I believe the webhost, probably also system.http.
My guess is that the versions in the web.config are off. For example, the DLL versions for Razor are 3.0.2 (not 3.0.0) and our DLLs probably bind to 2.x or similar, and are now redirected to a 3.0, which doesn't exist. But that's just a guess - basically these entries don't seem to contribute anything, so I can't guess why they were added in DNN, especially only in upgrade routines.

Alright, I think I've tracked this down.  It appears this all came from the 8.0.0 install.    These added or updated dependentAssembly elements for the following assemblies:

Newtonsoft.Json
System.Web.Http
System.Net.Http.Formatting
System.Web.Http.WebHost
System.Web.WebPages.Razor

However, the xmlns attribute is missing on these, so when adding those nodes, they explicitly clear the parent namespace using xmlns="".  As a result, these dependentAssembly do nothing and no binding redirect is created.  But not always, because when the matching dependentAssembly element already existed, it was overwritten using text, not using the API (https://github.com/dnnsoftware/Dnn.Platform/blob/release/8.0.0/DNN Platform/Library/Services/Installer/XmlMerge.cs#L493), and therefore the xmlns="" was not inserted.  This is why this issue is inconsistent, it depends on the state of the web.config at the time of the upgrade.
So, I believe, your issue stems from those binding redirects not being in place as they should, and therefore not effectively helping you when you're compiling against different versions than what is in the bin.
As far as I can see, there are two action items from this:

DNN should find and rewrite invalid binding redirects with xmlns="" (carefully, so that we don't activate a binding redirect that is no longer valid).  This should happen upon upgrade to some future version.
The XmlMerge logic for handling collision="overwrite" has already been fixed, but collision="save" still sets InnerXml instead of using the APIs correctly (which, in this case, masked the fact that the merge files were wrong). See https://github.com/dnnsoftware/Dnn.Platform/blob/development/DNN Platform/Library/Services/Installer/XmlMerge.cs#L562

@bdukes that sounds impressive 👍. I must admit I didn't understand all of it, but that's because I've never looked at the DNN installer/upgrade process. Just to understand the situation:

This seems to affect all systems which started as a pre 8.0.0 and were upgraded afterwards
It doesn't effect systems that started their life as a post <IP_ADDRESS>
...meaning that the web.config systems "in the wild" have 2 inconsistent states as of now (correct?)

And to fix it, we should:

correct how the xml-merging happens (because it's faulty)
... - this would also modify how future upgrades actually run the 8.0 scripts, but this also risks us creating a third possible state for the web.config
find a way to re-sync the inconsistencies of installations out there, and ideally remove the concerned bits which should never have been there in the first place

Is this correct? thx!

Merded in #2979


Run tests as part of CI

Run tests as part of CI.
This requires the following:

Go to repository Settings -> Environment
Create new environment with the name build_test
Add an environment secret with the name TEST_API_KEY and set its value to your finnhub api key
Do not specify any deployment branches. That way the environment can be available to all branches.

Also, when merging this, lets try a squash commit instead of a merge commit to see if the history looks any cleaner.

Yeah the squash and merge looks cleaner in my history

I agree - much better. Can you rerun the action to see if it succeeds?

Hm. The environment seems to work since the create_client test succeeded, but the actual tests failed. I wonder if the max requests for the api key was exceeded. Maybe the sandbox api key would work?

Yes I agree, I'm going to test it

Hmm, looks like it's not related to that

Weird. I just ran locally after syncing with your main branch it worked. Also, the test worked in my env. Maybe something is up with the github runners.


Update poet2 branch from master

Most of the files in sawtooth-poet master branch in this PR is not present in poet2 branch and poet2 branch has a different directory structure. So it’s not straightforward to merge these changes to poet2 branch.
We may have to manually pick the changes an update the Dockerfiles in poet2 branch


Support multiplication on ANSI interval types

Contributes https://github.com/NVIDIA/spark-rapids/issues/4151
Supports

Year-month interval * (byte, short, int, long, float, double)
Day-time interval * (byte, short, int, long, float, double)

Note: not support decimal type because of CUDF Decimal128 does not handle overflow, like:
CUDF positive decimal * positive decimal can generate negative value.
val d1 = ColumnVector.fromDecimals(new java.math.BigDecimal("123456789012345678901234567890123456.11"))
val d2 = Scalar.fromDecimal(new java.math.BigDecimal("12345678901234567890123456789012345611"))
val r = d1.mul(d2)
println(r)

123456789012345678901234567890123456.11 *<PHONE_NUMBER>1234567890123456789012345611 = -1053265448675579329726886670761974900.55

Signed-off-by: Chong Gao<EMAIL_ADDRESS>

build

build

Waiting #5150 to merge, has some conflict on Spark33XShims.

build

build

build

build

build

build

build


Implement isomorphic "fetch" function

A HTTP "fetch" function should be implemented, that provides a single and easy to use interface regardless if you are calling it from client (wasm) or server side (native).
Manipulation of headers (content type, authentication...) and request body should be possible without too much hassle.

You should take a look at the reqwest crate, which implements the same HTTP request API for both platforms.

Nice!
There should be an example using this method.


frontend: Add global ^K and / shortcuts

Closes https://github.com/NixOS/nixos-search/issues/762
cc @garbas

I would also love to see a "press / to search" hint

I would also love to see a "press / to search" hint in the textbox

I think "/" already works.


Full configuration is available here.
Expected behavior
There are no errors on startup.
Metadata
[ivan770@wolfram:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.19.1-xanmod1, NixOS, 22.11 (Raccoon), 22.11.20220819.495b19d`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.10.3`
 - channels(ivan770): `"nixpkgs"`
 - channels(root): `""`
 - nixpkgs: `/home/ivan770/.nix-defexpr/channels/nixpkgs`

https://discourse.nixos.org/t/removing-persistent-boot-messages-for-a-silent-boot/14835 seems related, no solution though. This may be related to systemd-boot somehow, as all configurations provided inside of this thread have it enabled.

I'm not sure if this issue is still relevant for anyone, as I can't reproduce it myself. Closing per https://discourse.nixos.org/t/its-issue-spring-cleaning-time-first-edition-2024/43793, feel free to reopen if it's still relevant for you.


Dealing with NSE

Hi
Is there a way to parse and deal with NSE scripts? I mean, if we can deal with scripts as objects and set its arguments, particularly --script-args=

You can use #script_params, example:
# list databases on MySQL servers. Must have login credential
Nmap::Program.scan do |task|
  task.ports = 3306
  task.script = 'mysql-databases'
  task.script_params = 'mysqluser=<username>,'\
                       'mysqlpass=<password>'
  task.targets = '<server>'
end

For parsing the NSE output, see the Scripts module, which provides .scripts raw output and .script_data parsed structured output, respectively. These methods are available on xml.prescript, xml.postscript, host.host_script, and any port object.

@kcning @postmodern Thank you guys, both answers combined are all that I needed


GitHub Advanced Security custom plugins for Copilot for Security

The solution contains 2 plugins enabling Copilot for Security to read the content of the security alerts generated by  GitHub Advanced Security (GhAS) in GitHub environments and one additional plugin enabling Copilot for Security to read the content of the security alerts generated by  GitHub Advanced Security for Azure DevOps (GAzDO) in Azure DevOps environments.
Please refer to the readme files for details.

Acknowledged , Review in process


⚠ Bofrost.be has degraded performance

In 4322bda, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1096 ms

Resolved: Bofrost.be performance has improved in fee9b9b after 2 minutes.


Enable support for 'output' ipywidgets and 'interact' ipywidgets

For #9524
Rework our loading of the WidgetManager to match the jupyter lab manager. This allows us to use a different OutputView and OutputModel

Codecov Report

Merging #10792 into ds/ipyWidgets will decrease coverage by 0.06%.
The diff coverage is 11.76%.

@@                Coverage Diff                @@
##           ds/ipyWidgets   #10792      +/-   ##
=================================================
- Coverage          60.74%   60.68%   -0.07%
=================================================
  Files                580      580
  Lines              31454    31501      +47
  Branches            4473     4480       +7
=================================================
+ Hits               19107    19115       +8
- Misses             11375    11414      +39
  Partials             972      972

Impacted Files
Coverage Δ

src/client/datascience/jupyter/jupyterNotebook.ts
4.26% <0.00%> (-0.11%)
:arrow_down:

src/client/datascience/jupyter/jupyterSession.ts
56.80% <0.00%> (-2.79%)
:arrow_down:

...ascience/jupyter/liveshare/guestJupyterNotebook.ts
9.62% <0.00%> (+0.25%)
:arrow_up:

src/client/datascience/types.ts
100.00% <ø> (ø)

src/client/datascience/webViewHost.ts
9.09% <0.00%> (-0.25%)
:arrow_down:

.../datascience/interactive-common/interactiveBase.ts
5.61% <33.33%> (-0.04%)
:arrow_down:

...ience/interactive-common/interactiveWindowTypes.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Login template

Hi,
Im trying to use this example.
In my configuration, I use Python 3.7.3 and Django 2.2.3
I changed in the settings MIDDLEWARE_CLASSES to MIDDLEWARE
TEMPLATES = [
    {
        'BACKEND': 'django.template.backends.django.DjangoTemplates',
        'DIRS': [
            PROJECT_PATH + '/templates/',
            ...

But when I try to install on the demo shopify, the Login ask me for the store url and after click on Install I get this error message on Django

[20/Jul/2019 09:51:30] "GET /accounts/login/?next=/ HTTP/1.1" 200 347
Internal Server Error: /accounts/login/
Traceback (most recent call last):
File "/home/sebastian/PycharmProjects/auth_demo/venv/lib/python3.7/site-packages/django/db/models/query.py", line 538, in get_or_create
return self.get(**kwargs), False
File "/home/sebastian/PycharmProjects/auth_demo/venv/lib/python3.7/site-packages/django/db/models/query.py", line 408, in get
self.model._meta.object_name
auth_app.models.AuthAppShopUser.DoesNotExist: AuthAppShopUser matching query does not exist.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/sebastian/PycharmProjects/auth_demo/venv/lib/python3.7/site-packages/django/db/backends/utils.py", line 84, in _execute
return self.cursor.execute(sql, params)
File "/home/sebastian/PycharmProjects/auth_demo/venv/lib/python3.7/site-packages/django/db/backends/sqlite3/base.py", line 383, in execute
return Database.Cursor.execute(self, query, params)
sqlite3.IntegrityError: NOT NULL constraint failed: auth_app_authappshopuser.last_login

I changed manually in the SQLite3 last_login NOT NULL false
But the next problem is, after installation, I get a Error 404

Page not found (404)
Request Method:	GET
Request URL:	https://5cdf423d.ngrok.io/login/finalize/auth_app.views.home
Using the URLconf defined in mysite.urls, Django tried these URL patterns, in this order:

login/ ^finalize/$
login/ ^authenticate/$
login/ ^$
^$
The current path, login/finalize/auth_app.views.home, didn't match any of these.

What do I have to do to solve the problem?
I didnt chnaged anything from auth_app

For anyone that finds this, this error comes from the default value of Django LOGIN_URL being equal to /accounts/login'. I just had to set LOGIN_URL=/login` in my settings.


Take first from providers and reuse it

Take first from providers and reuse it, instead of generating misleading paths when having multiple namespaced vendors
fixing https://github.com/nWidart/laravel-modules/issues/202

Thank you for the contribution! 👍
Could you add test(s) for this feature to make sure it keeps working in the future please ?
Thanks

Tomorrow I will try to update this PR :)

Seems now not failing

Hello,
I'm trying to understand how this fixes your issue of multiple namespaces for multiple modules.
How does it fix it ?
$concreteModule = $this->laravel['modules']->find($module);
        if ($concreteModule && count($concreteModule->providers) == 1) {
            return $concreteModule->providers[0];
        }

From what I see is you search for the module, if that module exists and it has a service provider, take the first service provider.
This seems pretty risky & a breaking change for people that have a different order.

It fixes config publishing, because having multiple namespaces, there is no other option to retrieve namespace associated with module, except via module.json itself..
It doesn't break nothing (or at least I don't know such case). That method is called only on publishing configs via artisan, and scans module.json in same manner like your service provider acts while registering modules, same order, and additional guard (like you said) prevents from loading multiple service providers. So it must exist only one that publish config, in other case with multiple declarations in module.json - would work as previous by default.

You can take a look at https://github.com/nWidart/laravel-modules/blob/master/src/Commands/stubs/json.stub#L8
this line count($concreteModule->providers) == 1 works only just then 1 registered

I see, but since it takes the first provider listed, what if the first provider listed, is one of a random package ?

Then project shouldn't work at all I guess, not even config publishing. Because modules loading works according that json.

Hm don't think so as it uses the configuration value.

Let's get this clear from very beginning :) This functionality was mentioned in previous discussion
I modified private method getServiceProviderForModule, and this is used only in publishConfiguration, when running from artisan module:publish-config, so it doesn't break anything further.
This PR extends with ability to publish not only under module directory, ex.: having 'namespace' => 'ModuleProjectNamespace' original method would make ModuleProjectNamespace\ModuleName\Providers\ModuleNameServiceProvider.
Once enabled
scan, it didn't build proper namespace. Let's say having secondary path. As workaround we used default laravel command:
vendor:publish --provider="Vendor\Package\Providers\PackageServiceProvider"
in compare with current module functionality:
vendor:publish --provider="ModuleProjectNamespace\Package\Providers\PackageServiceProvider")
With this functionality we bring possibility again to publish module's config, no matter under what namespace it exists .
BTW, I saw PR that works pretty similar like this one, but I left default namespace building if there exists more providers in module.json file as uncertain behaviour, like you told order or smth other.  Again, no breaks.
There wasn't fully exploit the potential for providers key previously, because only module registering was using it registerProviders, so I still don't see any problems reusing it.
Actually tests pretty well reflects all cases:
With 1 module named Blog:

having 1 provider (Vendor1/PackageNamespace/ServiceProvider) in json - publishing Blog, it will publish Vendor1/PackageNamespace/ServiceProvider
having more than 1 (Vendor1/PackageNamespace/ServiceProvider, Vendor2/PackageNamespace/ServiceProvider) in json - publishing Blog, it will publish Modules\\Blog\\Providers\\BlogServiceProvider (default)

With 2 modules Name1Module, Name2Module, having respectively Vendor1/PackageNamespace/ServiceProvider, Vendor2/PackageNamespace/ServiceProvider - and  publishing all modules without module parameter

it will publish  Vendor1/PackageNamespace/ServiceProvider and Vendor2/PackageNamespace/ServiceProvider accordingly

Thanks for taking the time to explain.
I did understand it's only for the publishing of configuration. I also understand it's to allow multiple module namespaces.
And I still stand by the point that it will fail, if you set an external service provider first, in the providers key.
Take this providers array:
"providers": [
    "FloatingPoint\\Stylist\\StylistServiceProvider",
    "Modules\\Core\\Providers\\CoreServiceProvider",
    "Modules\\Core\\Providers\\SidebarServiceProvider",
    "Modules\\Core\\Providers\\AssetServiceProvider",
];

This will use the FloatingPoint namespace correct ?

It won't execute at all, because, count is 4

Ok, and
"providers": [
    "FloatingPoint\\Stylist\\StylistServiceProvider",
];

Yes, testing with integrational testing here would be enormous preparation, to create separate folders for each test cases, and modules with in it :smile:  so I mock everything what is out of module scope
If I get you right, you mean that declaration for non existing.
So generated command from module:publish-config would be vendor:publish --tag=config --provider="FloatingPoint\\Stylist\\StylistServiceProvider", despite that module name itself would be different than Stylist.
Running any not existing provider (simplified to asdfg)
$ php artisan vendor:publish --tag=config --provider=asdfg
Publishing complete.

nothing happens, no errors on non existing provider.
Also there should be such case, because project wouldn't work at all, it gives an error:
Class 'FloatingPoint\Stylist\StylistServiceProvider' not found
at Module->registerProviders() in Module.php line 231
at Module->register() in Repository.php line 296

So it happens on registering.

If we switch context, and module A have
"providers": [
    "Common\\B\\BServiceProvider",
];

and module B have
"providers": [
    "Common\\A\\AServiceProvider",
];

then project works, but publishing module:publish-config A configs it generates BServiceProvider. This is expected workflow, because registering providers, with module A is associated BServiceProvider. Or do you want to publish according module A
vendor:publish --tag=config --provider="Common\\A\\AServiceProvider", which is belonging for B?

I'm not saying it doesn't work. I'm just pointing out that it assumes the first provider is one from the module, which doesn't have to be the case.

Hey. I'll try to step in and clear things up a bit :)
I don't know about the validity of the PR, but I do know the problem itself. So I'll try to explain it and maybe you will know the solution.
We are currently developing our modules as packages and import them to our projects via composer. Every package has their own module.json file and acts as nWidart module. To make our packages to be loaded, we publish module.php config and enable scanning like so:
'scan' => [
    'enabled' => true,
    'paths' => [
        base_path('vendor/our-company/*/src'),
    ],
],

Every module is being loaded and everything works fine. However, php artisan module:publish-config SomeModule can not be used. It finds the module, just no files are published. Keep in mind that our packages are under different namespace than the one defined here:
/*
|--------------------------------------------------------------------------
| Module Namespace
|--------------------------------------------------------------------------
|
| Default module namespace.
|
*/

'namespace' => 'Modules',

Their namespace is something like this: OurCompany\SomeModule.
Now, if we use php artisan vendor:publish --tag=config --provider=OurCompany\SomeModule\Providers\SomeModuleServiceProvider it works and files are being published.
Maybe you could identify where the problem lies and fix it or just guide us on how to fix it and we will update the PR :)
Thank you in advance.

@nWidart ping :)

Hello & sorry for the late reply.
I'm not sure why you felt the need to re-explain the issue.
I don't think this is the best solution to the issue for the reasons I stated before.

So how you suggest to use vendor scan if that functionality doesn't work?

People are using the vendor scan so it does work.
Also take a look at https://github.com/joshbrw/laravel-module-installer which might interest you.