Access Modifiers issue

ngx-treeview: 6.0.0
ERROR in node_modules/ngx-treeview/src/dropdown-treeview.component.d.ts.DropdownTreeviewComponent.html(3,9): : Directive DropdownToggleDirective, Property 'dropdown' is private and only accessible within class 'DropdownToggleDirective'.
node_modules/ngx-treeview/src/dropdown-treeview.component.d.ts.DropdownTreeviewComponent.html(6,9): : Directive DropdownMenuDirective, Property 'dropdown' is private and only accessible within class 'DropdownMenuDirective'.
node_modules/ngx-treeview/src/dropdown-treeview.component.d.ts.DropdownTreeviewComponent.html(3,9): : Directive DropdownToggleDirective, Property 'dropdown' is private and only accessible within class 'DropdownToggleDirective'.

This would also fix my issue, probably duplicated: #165


IE9 - SCRIPT16385: Not Implemented

I am doing some cross-browser testing of the Candy.js implementation I'm working on and finding that it doesn't work with IE9.
If you open the Developer Tools Console, there is a mysterious "SCRIPT16385: Not Implemented" error shown.
At first I thought it was a modification I had made somewhere, but this happens for me on the demo on the GitHub site.
Any assistance with this issue would be greatly appreciated!

Weird. Which version of candy do you use?

I'm using the latest (1.7.1) and jQuery 1.11.1
However, I see the same behaviour on the Candy Demo page: http://candy-chat.github.io/candy/#demo

Here is the SauceLabs log of the issue: https://saucelabs.com/tests/54e281557cc64cf9a62539f8801ff698

seems it has something to do with the native audio stuff, feature detection probably not done correctly.
Could you send me line number for a candy.bundle.js (including candy.bundle.js instead of candy.min.js)?

Don't you just love IE?
The line number in the unminified file is:
candy.bundle.js, line 3405 character 17
And here's the updated video it came from:
https://saucelabs.com/tests/e90774d1d90f427596730c6f4826e8e5
Looks like you're correct, that section is referring to audio. Perhaps some feature detection code could be borrowed from Modernizr:
https://github.com/Modernizr/Modernizr/blob/214f02af7b6e030de5d6422133c36975f74929bb/feature-detects/audio.js#L27-L46
It looks like the major difference (apart from formatting) is to wrap the test in a try/catch.

#351 has been updated.

Fixed by #351


How to find rvl_cdip_dataset.csv?

Hi,
Thank you for sharing your great work. Can you please also share your rvl_cdip_dataset.csv file or refer me to where I can find it?
Thanks,
Sayna

Hi,
That was a test file, and I don't think, we would be able to share it, since it had some issues. But, I can give you a general idea, about how to use the DocFormer for your particular task at hand.
I have mentioned it here, all you need to do is, download any of the document images dataset, containing/not containing labels (For unsupervised work), and then follow the steps here,
https://github.com/shabie/docformer/issues/23
Hope this helps, and do let us know if you need any more information.

Maybe, if the issue has been resolved, can we close this issue?


Unhelpful StackOverflow exception if the XboxServices.config file is missing

I was trying to build a sample app to demonstrate another issue, and run into this, think it should be fixed as it could get extremely frustrating if someone forgets to assign XboxServices.config to the project with Content and CopyIfNew properties set.
The issue is happening on Windows 10, Debug mode targeted to x64 (Target 16299, MinVersion 10240)
Microsoft.Xbox.Live.SDK.WinRT.UWP nuget package.
Nuget package version is v2018.4.20180411.2 so i think it is version 1804
I have installed XDK 1804 from https://developer.xboxlive.com/
OS is Windows 10 build 16299
IDE is Visual Studio Professional 2017 version 15.6.6

Hmm, it should fire a debug assert if its missing

Seems like it only fires if Mixed Mode (or native ?) Debugging is selected. Shouldn't we see it in Managed mode as well?


How to split listeners into different js files in bolt js?

Description
Describe your issue here.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[ ] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[ ] I've read and agree to the Code of Conduct.
[ ] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version:
node version:
OS version(s):
Steps to reproduce:

Expected result:
What you expected to happen
Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

Hi @AlishaK30, thanks for asking the question! I think there are several ways to organize your modules but one simple way could be like this. I know it's not a fancy way but it works.
// --------------------------------------------
// app.js
const { App } = require("@slack/bolt");
exports.app = new App({
  socketMode: true,
  appToken: process.env.SLACK_APP_TOKEN,
  token: process.env.SLACK_BOT_TOKEN,
});

// --------------------------------------------
// listeners.js
// You can have multiple listener enablers
exports.enableListeners = function (app) {
  app.event("app_mention", async ({ event, logger }) => {
    logger.info(event);
  });
};

// --------------------------------------------
// main.js
const { app } = require("./app");
const { enableListeners } = require('./listeners');
enableListeners(app);
(async () => {
  await app.start();
  console.log("⚡ Bolt app is running!");
})();

We'd love to know how the folks in the community are organizing listeners!

@raycharius Thanks for sharing your approach! The idea looks so insightful and practical 👍

The tasks-app's approach can be also helpful for deciding your way to manage listeners in separate files. https://github.com/slackapi/tasks-app

Let us close this issue now. Thanks again @AlishaK30 for asking the question, @raycharius for sharing your insights here!


Added 'gbd' alias for 'git branch -d'

Simple addition to the set of aliases available for git.
Wondered why it was not available by default (let me know if there is a special reason).
No other git alias has the same string (gbd) defined for it.

Hey there -- I really appreciate the contribution.
Cheers,
Robby


How to dynamically change the colorscheme?

I've got this vim script:
func! s:ChangeBackground()
  if (v:os_appearance)
    let g:lightline.colorscheme = 'nightfox'
    set background=dark
    colorscheme nightfox
  else
    let g:lightline.colorscheme = 'dayfox'
    set background=light
    colorscheme dayfox
  endif
  redraw!
endfunc

augroup AutoDark
  autocmd!
  autocmd OSAppearanceChanged * call s:ChangeBackground()
augroup END

It does not seem to be making the change to the colors for the lightline.
What's the best way to solve this?

Please refer to :h lightline-problem-13.

Ah, wonderful. Sorry I missed it!


XLIFF export uses debug serve function

Motivation

In the XLIFF export, the debugging function serve() is used to deliver the XLIFF files to the user.

Warning
This view will only work if DEBUG is True.
That’s because this view is grossly inefficient and probably insecure. This is only intended for local development, and should never be used in production.

– Django Documentation
Proposed Solution

Use a custom FileSystemStorage and serve XLIFF files like the other static files via the Apache/Nginx webserver.
Alternatives

Serve the file as normal HttpResponse by manually setting the correct headers
Additional Context

I think I'd prefer the HttpResponse solution. While the content is probably not really sensible, it is quite possible that there is unpublished content which should not be available publicly. Therefore it would be nice to serve the content in a protected end point.

I think I'd prefer the HttpResponse solution. While the content is probably not really sensible, it is quite possible that there is unpublished content which should not be available publicly. Therefore it would be nice to serve the content in a protected end point.

Do you think putting each file in a random UUID directory might suffice as "not publicly available" anymore? From a performance standpoint, I'd prefer the static files solution. But sure, requiring the login would be bullet-proof and since the zip only contains text-files, the file size is probably manageable by Django.

Okay. Lets hope that nobody enables directory listing by accident ;-)


Add topic and subscription overridden field into pubsub proto.

r: @chamikaramj

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

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

Run Java PreCommit

Run Java PreCommit

Java Tests / Java Unit Tests does't fail on this change but on test setup. I'm going to merge this PR to unblock https://github.com/apache/beam/pull/14141. Thanks for the quick review!


Added possibility to not purge the libdir directory

Hi. Can you add this to the base mcollective class, and then have config reference that?


Add scroll lock when using trackpad on web

Related PR: https://github.com/flutter/engine/pull/50645 (Depending on where it should be fixed this or the other PR is obsolete)
--
This is an attempt to fix weird scrolling behavior with trackpads on Flutter web.
It's done by looking at the last scroll events and locking the axis to the major axis of the first scroll axis within a series.
The lock resets automatically if no scroll event is received within x amount of time (e.g. 10ms).
Attempts to fix https://github.com/flutter/flutter/issues/143276
Check https://github.com/flutter/flutter/issues/143276#issuecomment-1950566307 for reasons for introducing this change on web and screen recordings before and after.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

My thinking was, scrollable.dart should register a "claim" for this scroll sequence. Then all the scrolls in the current series will be routed to that scrollable, and not go to others. This would let free-axis diagonal scrolling work. While letting individual scrollables lock-out other scrollables. However you do have an issue of who should do the "claim", because you may need a few events to make the judgement clear.

This can't be fixed in the engine. My heuristic already exists in the engine to identify which scroll events are from trackpad vs mouse wheel. Then it's up to the framework to decide how to handle the events differently. Implementing a scroll axis lock in the engine would break diagonal scrolling as it would destroy off-axis deltas before they reach the framework. Instead I still think the best way to go forward here is to enhance PointerSignalResolver with support time-windowed routing locks for discrete scroll events.

Ah thank @moffatman for the context!

(Triage) @IchordeDionysos are you still working on this PR?

@Piinks I did work on this as a side-project.
I do lack some more fundamental knowledge on how to best achieve only locking scrolls for vertical scrolls.
In the scrollable.dart there seems to be no data if it's a free-axis scrolling or not ...
I wasn't able to spend more time since then!
I can take another look today, if someone of you would like to take it over, that would be fine for me as well.

Yeah, I am not sure the information we need here is available on web right now. I don't know that anyone will be able to pick this PR up right now. You are welcome to continue working on it, otherwise we can close it to return to later. I appreciate your work here, thank you!

@Piinks were you talking about whether trackpad events are properly exposed on the web or whether the widget is vertically, horizontally, or all-axis scrollable?
I understand that the first is right now not possible due to browser API limitations.
However, the second should be possible (even on the Web), no?
Generally, when I use e.g. a SingleChildScrollView I can specify an axis and this axis information would be interesting when trying to build a heuristic, e.g. to only lock the scroll axis (and only expose scroll events in a single axis) when the scroll container expects a single axis scroll.

Generally, when I use e.g. a SingleChildScrollView I can specify an axis and this axis information would be interesting when trying to build a heuristic, e.g. to only lock the scroll axis (and only expose scroll events in a single axis) when the scroll container expects a single axis scroll.

The only all-axis scrollable (can scroll in two directions) is TwoDimensionalScrollable, which is used by TwoDimensionalScrollView. SingleChildScrollView can only scroll in one axis, so it does not need to lock one axis or the other, because there is only one.

were you talking about whether trackpad events are properly exposed on the web

Yup! this is what I was referring to.

Oh and I forgot to mention, the TwoDimensionalScrollable has axis locking options through DiagonalDragBehavior. :)

The issue here is, we need to lock it to the same widget destination. I have an idea how to do it, it's very possible to improve this (not make it perfect but a lot better), I just dont have much effort now to write it, especially not writing tests. you can contact me on flutter discord or something @IchordeDionysos I can help you out with it.


pytest: port eip spec version checking to a pytest plugin

This PR ports the reference spec checking by implementing the functionality in a pytest plugin: spec_version_checker.
If the checks fail, they get reported as warnings in the pytest test summary.
See for example the following two runs that report warnings that:

pytest was unable to check the spec version, and,
the spec version is missing/notprovided, respectively, out of date.

There is no way from pytest's exit codes to tell whether warnings were raised. If we want to be able to check whether these warnings are present, via CI, for example, we'd have to run pytest as:
pytest -W error

This won't run fillers though, as it's currently implemented, as the warning gets treated as an error in test setup. This will also treat all warnings in the test session as errors, with a bit more work, we could filter using our custom warnings using, for example,
pytest -W error::NoReferenceSpecDefined

In general, however, the output using -W error:: is very noisy as an error for every test is raised. The following is example output with -x (stop after first fail):

Thanks @marioevz. Perhaps we could try and improve this in the future by enabling the spec checks in a separate test(?). The idea being that it could be ran as a separate check. Currently, you have to execute all the fillers in order to check that the specs are up-to-date. I think this PR adds the existing functionality from tf to pytest, though.


Add example to assign licensing to Azure Security Group

Please update your powershell example to include how to assign a license to a security AD Group

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

ID: 7d1a3b25-8cbd-29f9-d205-31d320f6d315
Version Independent ID: 6711b6ba-154a-d078-d119-603d2c169453
Content: Assign licenses to a group - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/licensing-groups-assign.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

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

Hi @mwolfaardt
Security groups are fully supported for licensing purposes and as far I can confirm they are not treated any differently. The article in question should have PowerShell examples and doesn't. I'm taking a work item to put that together. It will be triaged and prioritized with the rest of my work, but usually the PM writes for this feature so I might just be the driver for this. Closing this bug as I don't know when this gets done.
Work item:
https://identitydivision.visualstudio.com/Technical Content/_boards/board/t/Technical Content Team/Doc PBI/?workitem=942183
Thanks for your contribution!
#please-close

Hi @mwolfaardt
I have the time to do this booked for April and am working on it. Thanks for your patience!

Thank you


Outline bugs

These are all functions/methods added in a chapter and seemingly important but not described in the text. These are critical and we need to resolve them stat!

Browser.clear_pending_tasks in Chapter 12
Browser.new_tab_internal in Chapter 12
Browser.clear_data in Chapter 13
JSContext.dispatch_RAF in Chapter 15

I think dispatch_RAF is probably easy and just needs to be mentioned by name in the text. new_tab_internal possibly can do the same. but the two clear ones fix real bugs and we don't describe it at all—needs a fix fast!

@chrishtr: can you take a look at this?


Add ability to install gateway helm chart with dual-stack service def…

…inition
Please provide a description of this PR:
Gateway routing currently functions for experimental dual-stack, but this PR adds the ability to modify the service definition to support dual-stack configurations.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @adams-shaun. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@adams-shaun: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
b73d46c7d1c63f587fa5fb90ecf6d30b33a6e616
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jacob-delgado: new pull request created: #46681

In response to this:

/cherry-pick release-1.18

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/cherry-pick release-1.19

@jacob-delgado: new pull request created: #46683

In response to this:

/cherry-pick release-1.19

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Add dotty support

This is experimental so we may delay merging it if preferred
This PR adds cross compliation to dotty. So far no source code needs to be changed but there are some notes:

Obviously the macros module doesn't compile
Tests are not being compiled nor executed as scalatest isn't available. Also I move the dotSyntax tests to a scala-2.13 specific dir
I removed kind-projector which doesn't seem to be in use
dotty doesn't work on js

Amazing, thanks a lot @cquiroz
Just to be sure, is it cross build with dotty in GitHub actions?

Yes, it behaves like a cross compilation though tests are only compiled and run on 2.13


add  requirements

I added it now

Great, you added the full freeze output there. I just added the base requirements.


Show hint in "add LOI" tasks that the task will be used to create an LOI

Suggested: Show a "+" in the geometry icon, and update the subtitle to "New data collection site". Subsequent screens will show subtitle as "Unnamed point" or "Unnamed area".

@gino-m Is this still needed?

Yes! @rawbzz for suggestions


Language server related messages persist

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
Zed messages about updating or trying to download language servers persist and never go away on my end.  Closing and opening Zed doesn't fix the problem, as it resurfaces after awhile.  Sometimes its about C++ and other times it's about Go.  If you look at the screenshot below, this is what shows on all of my instances, all of the time.
Expected behavior

Environment
Zed 0.65.2 – /Applications/Zed Preview.app
macOS 13.0.1
architecture arm64
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue

Here's a session where it's happening with the Python language server.  This is on v0.66.0.

I think one way to get into this state is to run a project search that rapidly opens buffers in several languages.

I'm having the same issue

OS: macOS 13.5.0
Memory: 8 GiB
Architecture: aarch64

Happens with TypeScript on several latest zed versions up to Zed 0.100.1, mac M1, OS 13.4.1

I had this issue for a few weeks
I tried to install manually typescript-language-server and put it inside the language folder here ~/Library/Application\ Support/Zed
Now it works


Feature/screenshot capture tool

We are close to a working state, with some last integrations getting done on Feature/screenshot annotator and painter. We are also going to merge that one in the following days.

Resolves #4, at least as a proof of concept. We still need to get everything into the Slicer Testing standard.


key "del" delete char but text does not shift to the left.

key "del" delete char but text does not shift to the left. should i wait fix this? or may be i do something wrong?

sorry) keepCharPositions = false


Augment Data Task

So this is the data I found on RoboFlow, combined, and organized:
https://github.com/ellenguyen/CIS4496_EY/tree/main/given/OtherTeamData
And this is some code that I started for you:
https://github.com/ellenguyen/CIS4496_EY/blob/main/AugmentDataTask.ipynb

I can take over this 💪

Check out my recent commit. I'm looking into semantic segmentation but have not decided to train on Roboflow or Rio since they are both in bounding box format.


CRASH GAME (NOT SERVER)

[] This issue isn't duplicated - you can check if it is by using the search bar located at the top left hand corner and select "Issues" on the left.
[] This issue includes appropriate markdown for sections - e.g. code blocks for crash dumps.
[] This issue is understandable - feel free to use your native language to write issues if you are not comfortable with English.

Issue description
What problem do you have?

Reproducing the issue

when players do /plot auto  their game crash (not server)
...

Client information
PocketMine-MP Version: pocketmine 1.7dev-1001
Plugin Version: 1.2.5
PHP version: 7.2.1 (default)

Optional information
Put a console error here, but please make sure you have your debug level set to 2 in your pocketmine.yml

no errors

After teleport then crash.  world done Preparing

are you teleporting into a loaded chunk?

hmm i guess yes. maybe u can join my server and try it?

Does the crash only happen when this plugin teleports the player in the plot world or can you use the teleport command to go to the same place?

When player going plot world it doesnt crash. But when they do /plot auto it does im not sure about about teleport command because i dont know the location.

ok. once you test the teleport command, then I might be able to help you.

But how do i know the location

I don't know. You will have to figure it out. also try /p warp 0;0

ok

/p warp 0;0 crash too

Can you join? so you will see

No. I don't join servers. The issue you have is not an issue coming from this plugin. It's from PocketMine. I recommend you ask the pmmp forums for help. https://forums.pmmp.io


Add the ability to disable storage of failed delivery events in EN DA

COS is used in the EN DA for the storage of failed delivery events. However there is no way to opt out of this feature. Can we add a boolean to disable it? If disabled, ensure the cos bucket does not get created too.

requirement to be able to use an existing cos bucket in the DA (as opposed to the one created by default)

@vburckhardt its already possible to use an existing bucket using the existing_cos_bucket_name input


[SPARK-14314] [SPARK-14315] [ML] [SparkR] Model persistence in SparkR (glm & kmeans)

What changes were proposed in this pull request?
SparkR glm and kmeans model persistence.
How was this patch tested?
Unit tests.

Test build #57336 has started for PR 12778 at commit d6c7b6c.

Test build #57336 has finished for PR 12778 at commit d6c7b6c.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  class GeneralizedLinearRegressionWrapperWriter(instance: GeneralizedLinearRegressionWrapper)
 class KMeansWrapperWriter(instance: KMeansWrapper) extends MLWriter
 class KMeansWrapperReader extends MLReader[KMeansWrapper]

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/57336/
Test PASSed.

cc @mengxr

LGTM. Merged into master and closed #12680 and #12683. Thanks!


FLACCue package install announces that SPK is requiring DSM7

Hello,
First, thanks @acenko for this terrific effort that looks to solve an old achille's heel of Plex 🥇
On my own Synology I can't find a way to make the kindly provided unsigned SPK installation a success:

Trying to manually install it through DSM GUI, DSM returns me that it requires DSM7 to be installed
I'm unsure if I did install the pip part right

Regarding 1 :

on one hand I'm, by choice, on DSM 6.2.4-25556 Update 6
on the other hand I don't find any prerequisite regarding DSM version in your manual
Therefore I guess this error is not really tied to that, but interpreted like that for whatever reason

Regarding 2 :

During installations, I did get multiple errors about "The script XXX is installed in '/var/packages/py3k/target/usr/local                                     /bin' which is not on PATH." and consequently errors about "pip: command not found" when trying to run the "pip install ffmpeg-python mutagen numpy" command.
I would believe I did workaround that through adding this path to my $PATH, even if a temporary solution. As both all ffmpeg-python mutagen numpy dependancies packages returns "Requirement already satisfied:" when I try to install them. I even double checked that executing the command from within this path ("cd /volume1/@appstore/py3k/usr/local/bin").

But unfortunately it doesn't seem to be sufficient to help me progress in the installation steps.
I also get a WARNING "Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv" but I guess the dependancies packages being installed this should not be a concern in the whole process.
As a side note:

Python 3.8.5, and ffmpeg are installed
sadly I'm not fluent on Linux systems, so there might be something really simple or stupid that I'm missing, sorry in advance
I've saved the Shell full text of my trials & errors (but did not post it here firsthand to not create mess), if it can help

With regards to DSM 6 vs 7, there is no basic reason why the code shouldn't work on either--the SPK was only built for 7.0 because I no longer had access to a 6 system and there were significant changes needed to get it working in 7. If you look through the change history, there is an SPK file for DSM 6 but this used an old version of the code. You might be able to create a new SPK for DSM 6 with the Synology toolkit, just changing "-v 7.0" to "-v 6.0" in the command listed in the readme, but I can't guarantee the changes needed for 7 will still work in 6 (I suspect they will, though, because it mostly became more restrictive). You can look at the "plex_scanner" tag version of this repository to see what worked in DSM 6--I'd recommend trying to get the newer code working if possible but you could always try the SPK from there to start as well.
For the pip step, the code runs using "python3". To test if the necessary packages are installed, type "python3" at the command line and then try "import  ffmpeg, mutagen, numpy". If that succeeds without error, the packages are installed.
If you have the python part working but are still having issues on the Synology side, you could try manually running the code. You'd need a copy of fuse.py, fuse.conf, and either flaccue.py from the base repository or FLACCue from the synology_package (these should be the same file, other than name). Put them all in your current directory and then try "sudo python flaccue.py / /flaccue/". You may first need to create a /flaccue directory ("sudo mkdir /flaccue"). In a separate terminal, you should then be able to "ls /flaccue" and see a mirror of your system (if the directory is empty, something is wrong). I don't recommend this as a long term solution (particularly running as root), but it would give you an idea of whether the core code is working or if something else is still wrong.
Hope something in here helps.

Same problem, my Synology running on DSM6 (too slow for upgrade on DSM7)

I have a question. I am trying to run the spk using manual update on 7.2.1-69057 Update 4, but have the following error note: ffmpeg should be installed first. What should I do then?

As mentioned in the readme, "FFMPEG will need to be installed (and the spk should warn you of this if it's not already there)."
The easiest way I've found to do this is to enable the SynoCommunity packages (instructions at https://synocommunity.com/) and then install it through the package manager Community tab. My system currently has "FFMPEG 4" installed, which works fine for me, although there appear to be newer versions available as well which I have not tested. I have no affiliation with SynoCommunity but I have found no issues running it for years now. You could also go directly to the FFMPEG site (https://ffmpeg.org/), although I suspect you'd need to build it from source there. Again, I have no affiliation with the FFMPEG project but it is a widely used open source project (e.g., used by VLC).


Create drawAuras hook

Creates 3 informational hooks, deletedAuras, createdAuras, and updatedAuras. Hopefully self-explanatory.
The arguments they give are the AuraRenderers object itself, and then an array of slugs of the auras that were somehow changed.
Primarily for use with the WIP PF2e Graphics module, such as animating size-changing auras. Ex. Issue https://github.com/MrVauxs/pf2e-jb2a-macros/issues/157

augury for this says "woe"

I might be amenable if there's no other recourse to a drawAuras hook that passes in the previous slugs as an argument, but doing the computation ourselves in what is already a performance sensitive code just in case a module uses it isn't something I could really agree with.

I might be amenable if there's no other recourse to a drawAuras hook that passes in the previous slugs as an argument, but doing the computation ourselves in what is already performance sensitive code just in case a module uses it isn't something I could really agree with.

So using this PR as an example, it would be a hook that provides the preUpdateSlugs and the token that's called after this.draw()?
So something akin to
 async reset(slugs?: string[]): Promise<void> {
        const preUpdateSlugs = Array.from(this.keys());

        ...

        return this.draw().then(() => Hooks.callAll("drawAuras", this.token, preUpdateSlugs));

I would await instead of using then, but yes, though perhaps in an object, so Hooks.callAll("drawAuras", this.token, { preUpdateSlugs }).
I don't mind if its before or after, whichever follows the pattern of most draw/render hooks.

I would use await as well if it was possible to have code run after returning it lol
And I don't know if anything relies on reset() returning that yet.

its a Promise. The only reason we return is because returning a promise is the same as awaiting.

The only reason we return is because returning a promise is the same as awaiting.

TIL
Making the changes.

As awkward as it may be to work around, remove the preUpdateSlugs. It has more of a chance going through without it. I'll do research on existing hooks to see if we need to change the name as well.

As a note, I have removed the preUpdateSlugs. The changeset is now just adding a Hooks.callAll("drawAuras", this.token); at the end.

I've reconnected after, and without the diffing of before/after or some way to even store what it was before, this PR has no use. It would be better served by a post prepared hook, which would be able to handle this situation. I'd recommend asking core foundry for such a hook though.


Add Board Structure

Add new board structure to FAQ.

o/  - hiya !
While this page now shows up on the site: https://www.resf.org/faq/board-structure - it is missing from the FAQ overview, so it's unreachable right now.


Cranelift AArch64: Fix the atomic memory operations

Previously the implementations of the various atomic memory IR operations ignored the memory operation flags that were passed.

Merged upstream into this PR after merge conflicts with #4830.

(Actually disabling auto-merge here as I won't be around when it goes in and letting two run in parallel is risky; I'll merge this tomorrow.)

We still have the rule to sink an atomic load, it's just that the helper data structure and functions were unnecessary, save for a bit of ISLE code.

We still have the rule to sink an atomic load, it's just that the helper data structure and functions were unnecessary, save for a bit of ISLE code.

Ah, yes, I missed that but see it now, for the (uextend (atomic_load ...)) case. Thanks!


Possibly add compute env info

Based on the Native Data Set Environment info you provided in the metadata file, I would assume that I can run this on my local computer. Is that true? If you ran your code in the cloud or on an HPC, I would request some more info on the environment you used.

You can indeed run this all locally or on HPC!  Environment yamls and Readme instructions in the data prep and modeling code walk through how to set up and run everything.  I've added a note to this extent in the Native Data Set Environment Info.


BUG: Qubino ZMNHTDx Smart meter S4 S5 S6 missing binary switches

https://community.home-assistant.io/t/bug-qubino-zmnhtdx-smart-meter-s4-s5-s6-missing-binary-switches/51096?u=nsoares

Please open a new issue and fill in the issue template properly.


alphabetical list of localities--page does not exist message

Selecting from the list of letters of the alphabet at the top of the page returns an error, page not found message: https://sandstorm.taxonworks.org/tasks/gis/list#C

Looking at what we have here I suspect this task should be redone from scratch, in Vue, with proper endpoints worked up.

Current list of links homogenizes similar, but distinguishable localities, albeit with same coordinates.
Enumeration of empty alphabetic zones seems unnecessary .
Current scoping of result count in lib queries seems to return less relevant results. (in uniquify people)

@jrflood let's open a new issue, if you feel its worth pursuing, for uniquify people, keep this one focused on the task listed.

@jrflood @TuckerJD seems there is still an error with the tweaks you made.
Steps to reproduce:

Draw polygon
Click 'Show' (records return)
Click 'C'
Get error

New / different features implemented under #643.


document to reflect no username hash

docs inline with 2f127512f84d9f9791ddc8ea903b04923778eb92
my first ever PR!  Hope it's not too major.

Coverage remained the same at 81.64% when pulling 86ef6228c561a830e1b135c730cd100c95904efd on derooy:hash-document-change into 2f127512f84d9f9791ddc8ea903b04923778eb92 on apexskier:master.

Coverage remained the same at 81.64% when pulling 86ef6228c561a830e1b135c730cd100c95904efd on derooy:hash-document-change into 2f127512f84d9f9791ddc8ea903b04923778eb92 on apexskier:master.

Great! Thanks for contributing.


Add a network card to a VM

Hi!
Anyone know how to add a NIC to a VM using knife vsphere?  I've searched and couldn't find anything.  The VM template contains one NIC.  After the VM is deployed, would like to add two more for iSCSI connection.
Any help would be appreciated.

Hey, we have knife vsphere vm config which sets properties like CPU and memory, but it doesn't add new devices. knife vsphere vm net get pretty close here, it would just need to create a new backing instead of using an existing one, and change the operation to :add (I think :) )
VirtualDeviceConfigSpec docs
Any interest in giving it a shot?

Hi, @theobromina
did you find a solution to your problem?
Regards,

Thanks for asking, @mlckha
No, not really.  I wound up using knife vsphere vm execute and called a PowerCLI script on a Windows server that, in turn, added the SAN NICs to the target VM.  If there is a better solution, it would be greatly appreciated.

UP, if someone have a solution :+1:
Thank's for helps
Best regards :)

Any way to add network cards yet?

Still no, sadly. I have some code in a script I can probably put into this gem though.

I've had a shot at creating a gem for adding a NIC to a VM.
I haven't used ruby before, so I'm not sure how to go about getting this packaged into something that Chef can install but here is my code.
https://github.com/coda4096/knife-vsphere/blob/master/lib/chef/knife/vsphere_vm_nic_add.rb
@swalberg
Can I please have some help with testing and the pull request on this?
Thanks
~Scott

I'm glad this feature will be added. Let's get your code into a pull request so we can discuss it. After that, I merge the pull request and cut a release of the gem, after which anyone installing the gem can use the command you added.
It looks like you've got your VNC feature and some testing work on your master branch, so let's try to separate this change into its own branch so that we can have a distinct PR to talk about. Here's your git history:

The two outlined commits are the ones related to the nic change.
git checkout c311106   # Point your local copy to the last commit on the project
git checkout -b add_nic # Create a new local branch called `add_nic`
git cherry-pick 9d02cee 83e4430 # pick the two commits you already did and replay them on your current branch as new commits
git push origin add_nic  # push your branch to your github fork of the project

Go back to GitHub and you should see a bar on the page saying you recently pushed changes, along with options to compare and make a pull request. Please have a look at the comparison to make sure it's just the changes you want merged, then make a pull request.
With git and projects with more than one person, it's often easier to keep your work in a branch. Git lets you easily switch between your change sets and pull in changes from other people.
So if you need to make changes to your branch, git checkout add_nic to get onto your branch (if you're not already there), make your change and commit it, then git push origin add_nic again and GitHub will update the pull request. If you want to get back to your master branch with the VNC feature then git checkout master. Just remember that after this your NIC changes will be duplicated on both master and add_nic, and if you want to make changes you should only do them on add_nic otherwise they'll never be seen by the pull request.

Thanks for your help. I have added two pull requests and all Checks have passed.
What do I do next?

Thanks for putting the two PRs up. I'll look them over and try them out (and anyone else, particularly the add nic one as there was lots of interest in that feature). Might be some discussion about the code within the PR. Also, if Hound made some comments about style please try to address them and push the changes up to the same branch. Hound is an automated tool that keeps the code style consistent, which helps a project like this where there are multiple contributors.
I'm in Eastern time so it's late night here (and currently watching this crazy election coverage) so should be able to take a peek tomorrow my time.
After we're good on the code, I merge them and cut a release.

this can probably be closed. anyone looking to add a Network card to a VM can now use the new command

knife vsphere vm network add VMNAME NETWORKNAME


Typescript support

Do you plan to provide Typescript interfaces? (For example by porting the project to Typescript or by adding types to "definitely typed").
I think this is by far the most convenient DICOM package for JS, which is why I would love to see TS support.
Thanks for your work!

Hello David! Thank you for your kind comments! Typescript support was in my list but now that I know that there's at least one person needs it, it will move up in priority!

Hi @PantelisGeorgiadis, thank you for the great work!
It would be great if typescript support is added. We are also waiting for this, thanks for raising the priority.

Hello @andreidubov! Thank you for your kind comment!
It is definitely in my highest priorities! Hope to find the needed time soon!
Any PRs or contributions would be greatly appreciated!

Hello @dmatuschek!
Thank you for your awesome work! I guess my response (https://github.com/PantelisGeorgiadis/dcmjs-dimse/issues/17#issuecomment-1155062520), regarding providing typescript support, was totally misleading… ☹
All I meant was that it is in my list to provide typescript types (i.e. an index.d.ts file) and not move the whole project to typescript, which, I haven’t used in any of my other projects.
Can we somehow keep both worlds (JS and TS) in an easy and maintainable way?

Oh, that's unfortunate. Anyway, I think keeping both versions is not a good idea. I would suggest reverting all changes and creating a type file. I will do this as soon as I can.

Thank you @dmatuschek and sorry for the misunderstanding! We are looking forward for your type file PR!

Hello @dmatuschek and @andreidubov!
Latest release (v0.1.7) contains typescript type declarations! I would be grateful if you could check them out and provide feedback.


Access to the S. maltophilia dataset needs to be unrestricted

https://zenodo.org/record/6504124#.YwiI6uxByEA

Also it should say 49 genomes. There are only 49 genomes in the dataset.

Done


Ensure DataArray retains specified column names

Follows on from #422 and should wait until that PR is merged. This PR simply ensures that the specified x and y column labels are inherited by the xarray DataArray returned by the canvas.points and canvas.points methods.

Ready to merge once #422 has been merged.


Shorten kibana_discover_url links

Description

https://github.com/jertel/elastalert2/discussions/445
Checklist

[x] I have reviewed the contributing guidelines.
[x] I have included unit tests for my changes or additions.
[x] I have successfully run make test-docker with my changes.
[x] I have manually tested all relevant modes of the change in this PR.
[ ] I have updated the documentation.
[x] I have updated the changelog.

Questions or Comments

@jertel
I got an error, but I don't know the cause.

So in the case of opendistro it could actually be AWS signed as well. All the same authentication methods would apply.

I will give up this function because there is no operation check environment.


create event in calendar

Hello,
From server i got some date and i want to show this date as a event and mark it. I want to use java but i can't find a helpful sample to add event into the calendar.
I am stack on this. say i have a array of date i want to show this date into the calendar with a mark
Thanks in advance

Hi there!
To display your custom event on the Calendar you have to create a class that represents your custom event and  implements the CalendarView.DateIndicator interface. Doing that, you'll be able to add instances of the event class to the Calendar:
 List<EventItem> eventItems = generateEventItems()
  calendarView.setDatesIndicators(generateEventItems());

I've added a sample that demonstrates how to display events on the Calendar using Java.
You can find the sample here.

thanks a lot

i got an error during compile your code
error: incompatible types: List cannot be converted to List<? extends DateIndicator>

That's strange.  I have just cloned the repo, build the sample app and there is no error on my machine. Please, provide the full build log output.

you can check this demo.
https://github.com/paveltech/EventCalender

I checked your demo app. Could you give me write access to your repo to push a fix for your demo app?

ya sure


Codecov reports

Enabling coverage reports via codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1f6cf43). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #42   +/-   ##
=========================================
  Coverage          ?   67.04%
=========================================
  Files             ?      119
  Lines             ?    16150
  Branches          ?     2462
=========================================
  Hits              ?    10827
  Misses            ?     4681
  Partials          ?      642

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


 IF YOU DO NOT FOLLOW THE FORMAT OR MAKE A GENERIC TOKEN REQUEST YOUR ISSUE WILL BE DELETED WITHOUT COMMENT
YOUR JUSTLIST MUST FOLLOW THE JSON SPECIFICATION
https://github.com/justswaporg/justlists/blob/main/example.justlists.ts
Checklist

[x] I understand that this is not the place to request a token listing.
[x] I have tested that my JustList is compatible by pasting the URL into the add a list UI at justswap.org.
[x] I understand that filing an issue or adding liquidity does not guarantee addition to the justlists website.

Please provide the following information for your token.
JustList URL must be HTTPS.
JustList URL:
JustList Name:
Link to the official homepage of the JustList manager:

Ok

QQQQQ

QQQQQ

QQQQQ

Please submit the correct information, invalid information will cause the issue to close

QQQQQ


pelelm.incompressible flag segfaults

Hi,
If I run with the incompressible flag switched on, the code segfaults. I tracked it down to the line in the Utils that tries to get the reactions MF without a flag check (e.g. m_do_react or m_incompressible, referenced below). I could fix it and put in a pull request but figured someone could add this quick fix without needing a full pull request.
https://github.com/AMReX-Combustion/PeleLMeX/blob/2b95bc8df51829d5f81a48bc7a9b060fc7fe8545/Source/PeleLMeX_Utils.cpp#L971
Thomas

@ThomasHowarth good catch. Thanks for finding this and tracking down the source of the error. I think a single check on m_do_react should resolve the issue for both incompressible and nonreacting but compressible flows.
Currently all changes, whether a single line or a whole new capability, must be made through a PR. So please do submit a PR with the fix if you are willing! The PR should go through easily for something like this.


Upgrade path from Binary to .deb / .rpm

The installation docs do not reflect the current .deb and .rpm releases, as they mention direct go binaries instead.

How does one install headscale using the new release format?
How does one upgrade from the previous method to the new one, given the manual SystemD install process?

This is both a matter of documentation and upgrade functionality. I assume eventually headscale will be released on default package mangers, at which point the process should be simplified.

Hi, I will work on documentation on how to install it on various platforms today; but in the mean time:

Download the file that is associated with your linux distribution; .deb for Debian/Ubuntu and .rpm for Fedora/Alma/CentOS and use the platform specific installation instructions to install them.
The ideal would be to remove everything from before, except for the config.yaml, and use the method above.

We do not aspire to add it to the package managers, and we will most likely just release it via Github as now.

I see the new documentation is up, so I will close this.
This was a roadbump for my transition, but all is good now.
https://github.com/juanfont/headscale/issues/1342#issuecomment-1516779155

As I mentioned on the other thread still having problems with permissions, /var/run/headscale/ in particular

Please make sure to run this with user being the user that launches Headscale :
sudo chown user /var/run/headscale


fix syntax errors

Signed-off-by: S.Çağlar Onur<EMAIL_ADDRESS>

Manually resolved conflicts and commit fixes.


⚠ BCR Bot has degraded performance

In 1c9fd79, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 855 ms

Resolved: BCR Bot performance has improved in 831e38c.


Rename JDBI 3 JUnit Jupiter extensions without Dropwizard in the name

The JDBI 3 Jupiter extensions have Dropwizard in their names but,
unlike the JDBI 2 extensions, they are independent of Dropwizard
and therefore can be renamed.

DropwizardJdbi3DaoExtension -> Jdbi3DaoExtension
DropwizardJdbi3Extension -> Jdbi3Extension
DropwizardJdbi3Helpers -> Jdbi3Helpers

Fixes #117

Self-merging since trivial...


Support taking an ObjectSerializer as well on pubsub message handling

Describe the proposal
Porting a suggestion from user bencod#6730 in Dapr Discord:

On publishing, Dapr client accepts message data as Object and it serialize the Object using ObjectSerializer - the Dapr one or user provided one.
On message handling user receives a CloudEvent that has data which according to documentation is String or Json, or the binary data of the message.
It requires user to deserialize the message. This behavior is not consistent with publishing serialization which is done by client.
I would expect deserialization to be done by infra as well, to allow user to provide an ObjectSerializer and CloudEvent to be typed. Something as follows:
@Topic(name = "testTopic", pubsubName = "testPubSub", serializer = MyMsgObjectSerializer.class)
public Mono<Void> handleMessage(@RequestBody(required = false) CloudEvent<MyMsg> cloudEvent) {
   ...
}

CloudEvent is a JSOn object in Dapr. So, this feature would only be applicable for the data part of the pubsub event.

I think this issue can be solved if we remove CloudEvent from the equation. For example, if the subscriber receives the object directly - similar to how is done in the .Net SDK. Something like this:
@Topic(name = "testTopic", pubsubName = "testPubSub", serializer = MyMsgObjectSerializer.class)
public Mono<Void> handleMessage(@RequestBody(required = false) MyMsg myMsg) {
   ...
}

I have a few open questions on dapr kafka pubsub component,
1)In pub/sub in what situation Dapr retries sending a message again..
I read in documentation that the delivery is at least once.
what is considered successful delivery and how to test it?
2) How dapr handle Autocommit? can we have the option to set autocommit time?
3) Dapr pub/sub tries to send it to subscribers and subscribers fail to process, what would happen then?
4) Unique ID for each message which gets published through pubsub
5) Ability to "ResetOffset" for Kafka pub/sub?
6)Is it possible to set Kafka producer/consumer properties when using it as pub/sub component in Dapr?
Example :  kafka.NewProducer(&kafka.ConfigMap{
"bootstrap.servers": "host1:9092,host2:9092",
"client.id": socket.gethostname(),
"acks": "all",
"batch.size": "32000",
"compression.type": "gzip",
})

Can we configure PubSub publish retry count?
if the sidecar crashes do all pending messages get lost?  Where is this described in the dapr docs?
how does Dapr guarantee messages are never lost when a sidecar gets the message from the message bus,
but then it crashes before it can deliver it to the subscribing service code?


Update stable version in repo from 1.11.3 to 1.14.0

Think about update stable version to 1.14.0
Version: stable=1.11.3 (123 days old) => latest=1.14.0 (15 days old)
Installs: stable=178 (54.94%), latest=35 (10.8%), total=324
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 1.14.0 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

1.15.1 already exists.


Add className prop to button

Previously, a custom className could be appended to the Button component. I noticed this functionality was removed in 0.0.26 and I implemented it to work with the button changes in 0.0.26

Hey there, this was a design choice, see #92

@dilan-dio4 as @tulup-conner mentioned, it was a design choice. We are not allowing direct access to className props in order to keep the design system consistency. If there are any edge case, it can be discussed here.
I'm closing the PR, since it will not be accepted for now.

Hm. I understand that you want to unify the design process, but I guess I'm confused as what the benefit is of making components less customizable?
Overriding classnames directly makes it very easy to do one-off configurations, especially with tailwind integration.

@dilan-dio4 you still can customize things by using the theme feature. The benefit of using the theme to customize the library components is that you keep the design consistency all over your application.
Even so... can you give me one example where the className is a must have and it cannot be replaced by the theme support?

I appreciate the standard design system and I plan on using it. This package lives in the React + tailwind universe in which className is a standard of. What if I want to add a small margin to one button? Now I have to wrap in a Div? What is lost by keeping className as an option? Mui has a standard themeing system and also className.
It's not a deal breaker to me. This is fantastic library that I'll use either way. Just a small loss of functionality.

I understand your point, @dilan-dio4. But we are not removing any TailwindCSS functionality. You still able to use TailwindCSS, as I said, you can customize it using the theme, and it will affect all the components, keeping your customization solid, and concise around all the places.
If you want to add some space between components, like the example that you gave, where you want to add a margin to the button you need to wrap it with a div. Try to see like this, the margin isn't a change to the button style, but to the layout/content structure of your system. So, this is why it does make sense to wrap it with a div and adding the margin to the div, not to the button. I don't know if I explain it correctly.
If you start to have access directly to the className prop to the design system component, you are going to have serveral miss-behaviors in your system.

Unpredictable styling behaviors (as @tulup-conner mentioned) - some components handle states and more.
You are going to start to have incontinence user UX
You are going to start to have code repetition (if you need to every time overwrite the class manually)
You are going to start to mix layout/content structure concepts to component concepts

Trying to clarify all the things:

If you need that all your component have some property, use the theme support and add it to all;
If you need one component to have a specific style, it is a custom component (outside the design system);
If you have to add anything that doesn't actually change the button style, but it is related to the content structure, it doesn't belong to the component; in some cases can be done by adding something to the children or wrapping the component;

I do understand that other design system can support className, but as we said, this is a design choice, we are worried about UX, a11y, consistency, predictability and testability. This is why we are not allowing access directly to the component className.
And so far, I'm not convinced that given the access to it would bring any benefit that can't be achieved in some more predictable way.


Add-PnPFieldToContentType does not support field types of "Taxonomy Field"

Hi all,
I have created a new Taxonomy Field using the command "Add-PnPTaxonomyField"
When I've tried to add this field to content type using the command  "Add-PnPFieldToContentType" I get this error:
Add-PnPFieldToContentType : Column 'QP_Department' does not exist. It may have been deleted by another user.
any help would be appreciated thank you in advance

Which module are you using and which OS? I see you opened issue #434 with the old module (SharePointPnPPowerShellOnline) so if you're in the same scenario, please install PnP.PowerShell 1.4.0 and feedback if still not working. Thanks


namespace fix; form action path fix

This change fixes NameError (uninitialized constant Rizzo::SailthruController::SailthruService). Calling SailthruSevice from Rizzo::SailthruController was throwing NameError due to improper nesting (see. http://guides.rubyonrails.org/autoloading_and_reloading_constants.html#nesting). I decided to use module nesting convention instead and it fixed the problem.

failing build is not related to this change and is fixed https://github.com/lonelyplanet/rizzo/commit/e99ebb130850e39f3f3f147ee7d2040842939b32

:+1:


add clusterrole for assigning userinfo to templateinstances; add clus…

…terrolebinding to new clusterrole to k8s garbage collector
/hold
until I can bring up a cluster with clusterbot from this PR and run the template e2e suite against it
/assign @adambkaplan
@bparees FYI
this change is needed to allow the k8s garbage collector to complete deletion of templateinstances, otherwise deletes never complete and the e2e's fail
the SAR check that fails without these new clusterrole/clusterrolebindings is https://github.com/openshift/openshift-apiserver/blob/master/pkg/template/apiserver/registry/templateinstance/strategy.go#L148-L158
when I added debug there during manual testing, I discovered the existence of the system:serviceaccount:kube-system:generic-garbage-collector SA

/test e2e-aws

@gabemontero any idea how long TemplateInstance has been broken? This sounds like something we will need to backport.

@gabemontero any idea how long TemplateInstance has been broken?

the e2e's were disabled in 4.2 so potentially since then ... I'll need to do a sanity check against a cluster at that level to see if the various clusterrole / clusterrolebindings are such that the same issue with GC occurs
This sounds like something we will need to backport.
without question

Have this PR up via clusterbot and running template tests now

And we have clean template e2e's against this PRs cluster from clusterbot (template service broker are the ones that are skipped):
started: (0/1/14) "[sig-devex][Feature:Templates] templateinstance readiness test  should report ready soon after all annotated objects are ready [Suite:openshift/conformance/parallel]"

passed: (1m8s) 2020-04-27T15:17:12 "[sig-devex][Feature:Templates] templateinstance readiness test  should report ready soon after all annotated objects are ready [Suite:openshift/conformance/parallel]"

started: (0/2/14) "[sig-devex][Feature:Templates] template-api TestTemplateTransformationFromConfig [Suite:openshift/conformance/parallel]"

passed: (18.2s) 2020-04-27T15:17:30 "[sig-devex][Feature:Templates] template-api TestTemplateTransformationFromConfig [Suite:openshift/conformance/parallel]"

started: (0/3/14) "[sig-devex][Feature:Templates] templateservicebroker security test  should pass security tests [Suite:openshift/conformance/parallel]"

skip [github.com/openshift/origin/test/extended/templates/templateservicebroker_security.go:50]: The template service broker is not installed: services "apiserver" not found

skipped: (18.1s) 2020-04-27T15:17:48 "[sig-devex][Feature:Templates] templateservicebroker security test  should pass security tests [Suite:openshift/conformance/parallel]"

started: (0/4/14) "[sig-devex][Feature:Templates] templateservicebroker end-to-end test  should pass an end-to-end test [Suite:openshift/conformance/parallel]"

skip [github.com/openshift/origin/test/extended/templates/templateservicebroker_e2e.go:57]: The template service broker is not installed: services "apiserver" not found

skipped: (18s) 2020-04-27T15:18:06 "[sig-devex][Feature:Templates] templateservicebroker end-to-end test  should pass an end-to-end test [Suite:openshift/conformance/parallel]"

started: (0/5/14) "[sig-devex][Feature:Templates] templateservicebroker bind test  should pass bind tests [Suite:openshift/conformance/parallel]"

skip [github.com/openshift/origin/test/extended/templates/templateservicebroker_bind.go:45]: The template service broker is not installed: services "apiserver" not found

skipped: (17.9s) 2020-04-27T15:18:24 "[sig-devex][Feature:Templates] templateservicebroker bind test  should pass bind tests [Suite:openshift/conformance/parallel]"

started: (0/6/14) "[sig-devex][Feature:Templates] templateinstance cross-namespace test should create and delete objects across namespaces [Suite:openshift/conformance/parallel]"

passed: (41.8s) 2020-04-27T15:19:06 "[sig-devex][Feature:Templates] templateinstance cross-namespace test should create and delete objects across namespaces [Suite:openshift/conformance/parallel]"

started: (0/7/14) "[sig-devex][Feature:Templates] templateinstance creation with invalid object reports error  should report a failure on creation [Suite:openshift/conformance/parallel]"

passed: (19.2s) 2020-04-27T15:19:25 "[sig-devex][Feature:Templates] templateinstance creation with invalid object reports error  should report a failure on creation [Suite:openshift/conformance/parallel]"

started: (0/8/14) "[sig-devex][Feature:Templates] templateinstance readiness test  should report failed soon after an annotated objects has failed [Suite:openshift/conformance/parallel]"

passed: (42.5s) 2020-04-27T15:20:08 "[sig-devex][Feature:Templates] templateinstance readiness test  should report failed soon after an annotated objects has failed [Suite:openshift/conformance/parallel]"

started: (0/9/14) "[sig-devex][Feature:Templates] templateinstance impersonation tests should pass impersonation update tests [Suite:openshift/conformance/parallel]"

passed: (30.6s) 2020-04-27T15:20:38 "[sig-devex][Feature:Templates] templateinstance impersonation tests should pass impersonation update tests [Suite:openshift/conformance/parallel]"

started: (0/10/14) "[sig-devex][Feature:Templates] template-api TestTemplate [Suite:openshift/conformance/parallel]"

passed: (18s) 2020-04-27T15:20:56 "[sig-devex][Feature:Templates] template-api TestTemplate [Suite:openshift/conformance/parallel]"

started: (0/11/14) "[sig-devex][Feature:Templates] templateinstance object kinds test should create and delete objects from varying API groups [Suite:openshift/conformance/parallel]"

passed: (20.7s) 2020-04-27T15:21:17 "[sig-devex][Feature:Templates] templateinstance object kinds test should create and delete objects from varying API groups [Suite:openshift/conformance/parallel]"

started: (0/12/14) "[sig-devex][Feature:Templates] templateinstance impersonation tests should pass impersonation creation tests [Suite:openshift/conformance/parallel]"

passed: (27.2s) 2020-04-27T15:21:44 "[sig-devex][Feature:Templates] templateinstance impersonation tests should pass impersonation creation tests [Suite:openshift/conformance/parallel]"

started: (0/13/14) "[sig-devex][Feature:Templates] templateinstance impersonation tests should pass impersonation deletion tests [Suite:openshift/conformance/parallel]"

passed: (32.6s) 2020-04-27T15:22:17 "[sig-devex][Feature:Templates] templateinstance impersonation tests should pass impersonation deletion tests [Suite:openshift/conformance/parallel]"

started: (0/14/14) "[sig-devex][Feature:Templates] templateinstance security tests  should pass security tests [Suite:openshift/conformance/parallel]"

passed: (29.9s) 2020-04-27T15:22:47 "[sig-devex][Feature:Templates] templateinstance security tests  should pass security tests [Suite:openshift/conformance/parallel]"

/hold cancel
@adambkaplan PTAL

I've also initiated a thread with @deads2k in forum-apiserver about all this

net with @deads2k ... ACL change alone seems fine, but the question of why https://github.com/openshift/openshift-apiserver/blob/master/pkg/template/apiserver/registry/templateinstance/strategy.go#L136 did not prevent us from doing the SAR check at all when the GC is coming through needs to be understood
/hold

looks like we have an upstream bug around GC and mutating GC only metadata short circuit checks ... see https://coreos.slack.com/archives/CB48XQ4KZ/p1588008887095400 and https://kubernetes.slack.com/archives/C0EG7JC6T/p1588010128112000 for
tl;dr
We'll need to monitor the yet to be crafted upstream fix and do something analogous to the upstream fix

UPDATE: the upstream PR which in theory should NOT require the permission add I've done in this PR is https://github.com/kubernetes/kubernetes/pull/90534
@deads2k informed me I can just copy that change into openshift-apiserver vs. picking / vendoring in the upstream change
I'll work on that officially once that PR merges, and may try intermediate forms of it on a personal cluster to verify it removes the need for the permission update in this PR.

confirmed in a local cluster that use of the current fix in https://github.com/kubernetes/kubernetes/pull/90534 in a 4.5 cluster negates the need for this PR
@deads2k @dmage FYI
also note, confirmed that we only need the openshift-apiserver changes in 4.5 ... earlier versions don't need the change when I run the template suite against those
I'll start prepping a 4.5 openshift-apiserver change for any use of TestIsOnlyMutatingGCFields


Clean up sighash.json to confirm with keccak

Description
To make hash results inside sighash pass unit test. This necessary because keccak is the new hash functiion
closes https://github.com/wu-emma/bitgesell/issues/14
Items done

Change all signature_hash (result) inside sighash.json
Add CBlock block = getBlock6548(); inside bloom_tests to compile code.

Notes
Only work on hash results inside sighash.json
BTC/BGL PR reward address
This is part of Gitcoin bounty, https://gitcoin.co/issue/wu-emma/bitgesell/14/100023919

./src/test/test_BGL -t sighash_tests
Running 2 test cases...

*** No errors detected

@janus great job! please could you fix merge conflict (it's just comment difference) so I could cleanly merge this PR?
@wu-emma this is working, and test file data contains the same number of records! Issue/bounty can be closed.

@van-orton  Thanks so much for you review. I have resolved the 'comment conflict'.


I want to contribute to your project by adding some Cheetsheet And Notes Related Data Science

This will be very helpful for beginner

Yes, you can do that @nihal-soni , I assigned this to you

ok thank -you


Player gets on his bicycle when entering a cycling road tile

Added a CyclingRoad Thing that when stepped on, forces the player to get on the bicycle.
Some comments missing.
Placeholder function for forcing movement.

Removed functionality of forceMovement


pkg/asset/templates/kube-proxy.yaml: fix typo in image's URL

fwiw spotted it while playing with #88 interimn "solution"

lgtm


MSBUILD : error MSB1008: Only one project can be specified.

This doesn't look like an issue with the VSTS build task to me. The task is passing the command line to msbuild and msbuild is returning -1. How do you build from the command line? Please provide details if you find the task is specifying the command line incorrectly, otherwise close.

I have reconfigured the Build Agent with proper Working folder and that fixed the Issue.
Thanks

For me the issue was that i had invalid Arguments. Removed those and it worked.

For me the issue was that i had invalid Arguments. Removed those and it worked.


Add support for using the PDC (Private Data Connection) with this datasource

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Please can you support the use of the PDC (Private Data Connection) for this datasource. Although it is possible to use the Elasticsearch datasource plugin with its warnings of versions it would be better suited to have this built into the OpenSearch especially as AWS OpenSearch gives you the option to run Opensearch within a private VPC
Describe alternatives you've considered
It is possible to use the Elasticsearch Datasource but it's not ideal
Additional context
None required

Thank you for the feature request! I'm going to add it to our backlog to estimate and prioritize.

Link (grafana only) to the implementation guide: doc

Blocked on https://github.com/grafana/opensearch-datasource/issues/442


Modify .gitmodules to point to loopandlearn organization

Summary
This PR modifies the .gitmodules file to point to the loopandlearn github username as the source for Trio submodules.

This enables Trio to have slight differences in submodule code with respect to LoopKit while maintaining connection to the LoopKit primary respositories
This enables Trio developers to provide translations in the submodule repositories under loopandlearn if the ones at LoopKit are not being updated

Rationale
Trio uses submodules based on those used by the Loop app, located at GitHub LoopKit

There is a need to update these submodules for

bug fixes
differences in desired default behavior
translation updates

There are no Trio developers who can merge changes in LoopKit for most repositories

This is currently causing a roadblock for code changes and translations

This PR changes the source for these submodules to be the GitHub loopandlearn organization

By using a fork of the LoopKit repositories with a specific trio branch, updates that are merged into LoopKit can be captured easily in the loopandlearn forks
Trio developers will be added to the loopandlearn organization repositories

State before the PR
This PR is designed to only change the .gitmodules file and not to change any SHA.
These are the SHA for each module prior to this PR

SHA on 13 June 2024
repository
comment

a92e9752994e7b143cdb007d3c7bcba0c0cc9214
CGMBLEKit
(v3.1-166-ga92e975)

b5e992e211d2ac6224acb105dd97fb484767da72
G7SensorKit
(heads/main)

c01eba63e94e9f6f2841a8835680c4e39c61b18d
LibreTransmitter
(heads/main)

2f535b3ca46825e82e0dd1b5ef9daccd53a3f0ca
LoopKit
(v3.0-1039-g2f535b3)

4a6df31192cbbddf62ed36ad78b1ca29649bd851
MinimedKit
(heads/main-6-g4a6df31)

85fc3c6d4805d580acdf6592b220717b6e842558
OmniBLE
(heads/dev)

a80e38b1b7f203014b461f8aff8cead2c067e39d
OmniKit
(heads/main)

a0e419da314d0ad42b41ccb04af73cd1fbf41257
RileyLinkKit
(heads/dev)

a2ccad72a55600c28549ab86ab1964c0d6558868
TidepoolService
(heads/dev-3-ga2ccad7)

21d8657d727f26df76342188e580022bf4884714
dexcom-share-client-swift
(v0.4.1-156-g21d8657)

Other information
Information used while preparing this PR are stored in this sheet: Submodules for Trio
The sheet should be visible to anyone with a link.

Test:
create a manual clone from the source for this PR:
git clone --branch=modify_submodule_source --recurse-submodules https://github.com/loopandlearn/Trio.git

There is one extra file (other than .gitmodules) that appears with  git diff alpha modify_submodule_source --name-status after cloning from loopandlearn:modify_submodule_source. I confirmed the diff is because a PR was merged into alpha after I made my feature branch and the diffs are as expected.
M       .gitmodules
M       FreeAPS/Sources/Modules/Stat/View/ChartsView.swift
Ensure the SHA for all submodules are unchanged.
Ensure that the clone builds.
Ensure no extraneous files get modified by Xcode.

Merging this with two approving reviews and general consensus among admins.


Only take the last commit to diff

Only compare to parent, not grand parent. Not sure why it's previously been ~2.
You can try git diff HEAD~2 --name-only and git diff HEAD~1 --name-only in master to see that the first one actually returns names for two last merged libdefs instead of one.
Noticed this behaviour here: https://github.com/flowtype/flow-typed/pull/1919 See the travis output where unrelated libdefs fails: https://travis-ci.org/flowtype/flow-typed/jobs/350025576

Seems reasonable to me, merging. We can change it back if necessary. @GAntoine

No need, AFAIk this change makes sense. 👍


Subscriptions will not work if query sent to route ending in "/" and server is not configured on "/" ending route

Describe the bug
Subscription performance depends on graphql url definition. If configured url is not ending in / executing subscription from playground query will return an error.
To Reproduce
Steps to reproduce the behavior:

Run  StarWarz example
Run simple subscription query in Playground
subscription{ onReview(episode:EMPIRE){  commentary  stars } }
You will get and error:
{ "error": "Could not connect to websocket endpoint ws://localhost:5000/graphql/playground/... Please check if the endpoint url is correct." }

Expected behavior
Playground should say Listening..
Desktop (please complete the following information):

OS: WIN 10, Chrome Version 81.0.4044.122 (Official Build) (64-bit)
Git commit 2d724adcf2f8b45fe268ca0598bd5798b320291f

Additional context
Problem seems to be in combination of how url Playground in using to execute queries and how path comparation is implemented in SubscriptionMiddleware.IsValidPath method. Playground will always add / at the end of url. If server is configured with .UseGraphQL("/graphql"), SubscriptionMiddleware.IsValidPath will return false as "/graphql" and "/graphql/" are not equal. Interesting is that with same configuration of server url queries will executing if you send the both to /graphql and /graphql/ routes.

I got same issue: https://github.com/ChilliCream/hotchocolate/issues/1712
Track this one: https://github.com/ChilliCream/hotchocolate/issues/1630

Since as @sergeyshaykhullin pointed out this is a duplicate we are gonna close this one and mark it as duplicate.

#1630


Setting up case for a no plea state

I have looked to comment out some case information which the Legal Advisor wouldn't see on a No Plea case. Also made a couple of small things which were confusing users, such as a pre-populated Magistrates name field.

The changes you have been making to the data in the routes.js file are not correct. Removing data entirely will prevent the prototype from working..
It's good to comment out the code you are changing with notes. But it should be done like this:
Original:
req.session.data['defendents-email'] =<EMAIL_ADDRESS>New:
// we don't know this income yet for no plea req.session.data['defendents-email'] =<EMAIL_ADDRESS>req.session.data['defendents-email'] = ""
I've updated the data for this but the logic for the prototype needs to be updated to hide or update things that are not relevant for a 'No plea received' scenario. These include:
Hide 'Dates to avoid'
Hide 'Online plea'
Need to show links for 'Add email' and 'Add mobile'
Updates are needed so the Income tab will display correctly
This sounds easy but it's intense work that need focus and I won't have the time to do this today or tomorrow with the distraction of the house move. I will be able to work on this next week.

Okay thanks,
Lets' leave it for the moment - having not written the code myself it's
hard to see the thread of what's pulling data from what - was hoping
commenting would be less destructive if I'd got it wrong.
On 30 August 2018 at 06:40, Larry Miles<EMAIL_ADDRESS>wrote:

The changes you have been making to the data in the routes.js file are not
correct. Removing data entirely will prevent the prototype from working..
It's good to comment out the code you are changing with notes. But it
should be done like this:
Original:
req.session.data['defendents-email'] =<EMAIL_ADDRESS>New:
// we don't know this income yet for no plea req.session.data['defendents-email']
=<EMAIL_ADDRESS>req.session.data['defendents-email'] = ""
I've updated the data for this but the logic for the prototype needs to be
updated to hide or update things that are not relevant for a 'No plea
received' scenario. These include:
Hide 'Dates to avoid'
Hide 'Online plea'
Need to show links for 'Add email' and 'Add mobile'
Updates are needed so the Income tab will display correctly
This sounds easy but it's intense work that need focus and I won't have
the time to do this today or tomorrow with the distraction of the house
move. I will be able to work on this next week.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/blackmonday/atcm-legaladviser/pull/5#issuecomment-417196235,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgTRlXFqIxRx6YyLq5XkR4GI7NMtMLs3ks5uV3rBgaJpZM4WRutE
.


af.slidemenu.js is no longer exist in 3.0?

No sliding menu, then my project can not be migrated to 3.0

This is due to an underlying change in how the composition of elements were created.  Someone can always port the plugin over.


Fixed link to zlib documentation

Fixes #23

Thanks for fixing the link!

You are welcome, thanks for the lib, it's very helpful ;)


Project needs converting to Kicad v6

The files all need resaving in the new version and then packaged up for here. Probably best doing this in a new branch.

Completed in #3


v4 support client method with only required parameters

Please take a look whether it is OK for you.
If it is OK, I will need to update test cases, since some parameter sequence got updated.

Sample generated code
    /**
     * Lists all containers and does not support a prefix like data plane. Also SRP today does not return continuation token.
     *
     * @param resourceGroupName
     * @param accountName
     * @param maxpagesize
     * @param filter
     * @throws IllegalArgumentException thrown if parameters fail the validation.
     * @throws CloudException thrown if the request is rejected by server.
     * @throws RuntimeException all other wrapped checked exceptions if the request fails to be sent.
     */
    @ServiceMethod(returns = ReturnType.COLLECTION)
    public PagedFlux<ListContainerItemInner> listAsync(String resourceGroupName, String accountName, String maxpagesize, String filter) {
        return new PagedFlux<>(
            () -> listSinglePageAsync(resourceGroupName, accountName, maxpagesize, filter),
            nextLink -> listNextSinglePageAsync(nextLink));
    }

    /**
     * Lists all containers and does not support a prefix like data plane. Also SRP today does not return continuation token.
     *
     * @param resourceGroupName
     * @param accountName
     * @throws IllegalArgumentException thrown if parameters fail the validation.
     * @throws CloudException thrown if the request is rejected by server.
     * @throws RuntimeException all other wrapped checked exceptions if the request fails to be sent.
     */
    @ServiceMethod(returns = ReturnType.COLLECTION)
    public PagedFlux<ListContainerItemInner> listAsync(String resourceGroupName, String accountName) {
        final String maxpagesize = null;
        final String filter = null;
        return new PagedFlux<>(
            () -> listSinglePageAsync(resourceGroupName, accountName, maxpagesize, filter),
            nextLink -> listNextSinglePageAsync(nextLink));
    }

    /**
     * Lists all containers and does not support a prefix like data plane. Also SRP today does not return continuation token.
     *
     * @param resourceGroupName
     * @param accountName
     * @param maxpagesize
     * @param filter
     * @throws IllegalArgumentException thrown if parameters fail the validation.
     * @throws CloudException thrown if the request is rejected by server.
     * @throws RuntimeException all other wrapped checked exceptions if the request fails to be sent.
     */
    @ServiceMethod(returns = ReturnType.COLLECTION)
    public PagedIterable<ListContainerItemInner> list(String resourceGroupName, String accountName, String maxpagesize, String filter) {
        return new PagedIterable<>(listAsync(resourceGroupName, accountName, maxpagesize, filter));
    }

    /**
     * Lists all containers and does not support a prefix like data plane. Also SRP today does not return continuation token.
     *
     * @param resourceGroupName
     * @param accountName
     * @throws IllegalArgumentException thrown if parameters fail the validation.
     * @throws CloudException thrown if the request is rejected by server.
     * @throws RuntimeException all other wrapped checked exceptions if the request fails to be sent.
     */
    @ServiceMethod(returns = ReturnType.COLLECTION)
    public PagedIterable<ListContainerItemInner> list(String resourceGroupName, String accountName) {
        final String maxpagesize = null;
        final String filter = null;
        return new PagedIterable<>(listAsync(resourceGroupName, accountName, maxpagesize, filter));
    }

Preferably I should generate this (directly call the async/sync method with full parameters, instead of calling listSinglePageAsync).
I will find time later to update it (probably need 2 new type of client methods).
    public PagedFlux<ListContainerItemInner> listAsync(String resourceGroupName, String accountName) {
        final String maxpagesize = null;
        final String filter = null;
        return listAsync(resourceGroupName, accountName, maxpagesize, filter);
    }

I choose not to overload XXXWithResponseAsync method. This overload is not useful for fluent. Let me know if you prefer to overload this as well.

Agreed.


Appending text to a subgroup

It does not appear possible to append text directly to a subgroup, without introducing characters in between.
I have a metric that outputs the exact HTTP return codes, and I'd like to instead view these as their groups. (e.g. 2xx, 4xx, 5xx)
I can do this with some very ugly nested label_replace's:
sum(label_replace(label_replace(label_replace(label_replace(rate(nginx_ingress_controller_requests[2m]), "status_code", "5xx", "status", "5.*"), "status_code", "4xx", "status", "4.*"), "status_code", "3xx", "status", "3.*"), "status_code", "2xx", "status", "2.*")) by (status_code)
But I would much prefer to use a regex subgroup to grab the group code, then append xx onto it. For example:
label_replace(rate(nginx_ingress_controller_requests[2m]), "status_code", "$1xx", "status", "(.)..")
However this has the weird behavior of actually doing nothing. If you put a space or any symbol between $1 and the trailing text, it works, but it would be nice for there to be some method to do this cleanly.

Try ${1}.
It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

Thanks, for anyone searching this does resolve the problem I was having.


Roles Button Dropdown

Roles Button Dropdown
Issue Number(s): #160 .
What does this PR change and why?
Checklist

[ ] Requirements have been implemented
[ ] Acceptance criteria are met
[ ] Database schema docs have been updated or are not necessary
[ ] Code follows design and style guidelines
[ ] Commits follow guidelines (concise, squashed, etc)
[x] Relevant reviewers (Senior Dev/EM/Designers) have been assigned to this PR

Critical Changes

Database change/migration to run
Environment config change
Breaking API change

Related PRs

#number_of_pr

Testing
Enumerate steps to test the functionality of your ticket. This should include edge cases and testing of error handling, if applicable.

Resolved by #275


coherence score per topics

OCTIS version:
Python version:
Operating System:

Description
I am working on topic modeling for noisy short texts, trying to get topic significance scores per topic.
What I Did
for t in output: #'output' is the model itself

    significance_uniform_score = topic_signif_uniform.score(t)
    print("Topic Significance Uniform Score: "+str(significance_uniform_score))

I get the following error message:

TypeError                                 Traceback (most recent call last)
 in
1 # Retrieve metrics score
2
----> 3 for t in output[:]:
4
5     #topic_diversity_score = topic_diversity.score(t)
TypeError: unhashable type: 'slice'
Is it possible to get topic significance score per topic?

Hi!
I just added this feature for the kl-uniform (I think this is not possible for the other significance metrics) with the new release of OCTIS. You can update the library using pip install -U octis. Your code should look as something like this:
model =  # define your model
model_output = model.train_model(dataset)

topic_signif_uniform = KL_Uniform()
significance_uniform_score = topic_signif_uniform.score(model_output, per_topic=True)

The per_topic parameter allows the score method to return a list of values, corresponding to the values of the KL for each topic.
Silvia

It worked @silviatti ! Thank you very much.
-Atakan


powershell 6 and command line defaults

Hello Roman,
hope you are doing well. I have a couple of questions:

I don't seem to be able to figure out how to make far use powershell 6, is this possible?
Is it possible to have far execute powershell by default, eg without the ps: prefix?

Many thanks,
Andrew

Hi Andrew,
You are talking about the PowerShellFar module, right? It is not designed for
PowerShell Core (v6). It is built for and works with the current installed
Windows PowerShell (v2 - v5).
The PowerShellFar module commands are invoked using the prefix ps:, there is
no other way. These commands are invoked in the session "inside Far".
As for PowerShell apps (either v2 - v5 powershell.exe or v6 pwsh.exe),
You can run any of them using the Far command line as usual external console
applications, "outside Far".
In theory you can try to play with COMSPEC and set it to powershell.exe or
pwsh.exe. Then (in theory!) you can run PowerShell commands without prefix
and executable name. I never tried this and not going to. Not sure it's a
good idea, I would not like to lose the usual CMD commands.
Hope this helps. What are you trying to do and "have issues"?
Please ask more specific questions.

Thank you, that's fine.
In 2019 I personally feel that CMD does not have much of utility. In the rare cases when it's required, you an drop to CMD from powershell.
I have many powershell utility scripts that I would like to execute directly from the Far Manager command line, that is I have myscript.ps1 somewhere on PATH and I'd like to type myscript -a b -c d and get it executed. I have to have a separate from Far window for that (or I'll have to type ps: every time).
Similarily some of the scripts are written for powershell 6 and won't work with powershell 5, so desire is to have it at fingertips inside Far Manager - working in a single window could be more convinient when switching between two.
From your answer I understand that you have no desire of supporting or reasearching that, and that's fine, I can understand that. I thank you for the prompt answer, and for all the for you've done on PowerShellFar and FarNet.

Ah, I see. Well, for running scripts you do not have to type ps: or exe name.
You have to set your associations properly, that's it.
System association:
cmd /c ASSOC .ps1=Microsoft.PowerShellScript.1
cmd /c @"
FType Microsoft.PowerShellScript.1=$PSHOME\PowerShell.exe -NoProfile "& '%1' %*"
"@

Plus, add .ps1 to PATHEXT, so that you can omit .ps1 in commands.
After that, given a script foo.ps1 in the path, you can run it as
foo arg1 arg2 ...

See, not prefix, no exe name, just script name and parameters.
Alternatively you can do this using Far Associations.
But I prefer system, they work outside Far as well.

By the way, I have my own helper p6.cmd
@echo off
%powershell6% -NoProfile -ExecutionPolicy Bypass %*

where %powershell6% points to my ...\pwsh.exe. Then I just type p6 foo ... when I need to run v6.

Thank you for this, lot's of useful tips!


fix: nil pointer on incomplete config for accounts

https://github.com/ekristen/aws-nuke/issues/385

:tada: This pull request is included in version 0.21.2 :tada:
The release is available on GitHub release :rocket:


Layout options

Add layouts options to :

init filter
init chart
add some help text

Split with 1 ticket per request


Choosing custom/default for Amazon EC2 shows no security group in UI

Rancher Version: master 9/10
Steps to Reproduce:

Go to Add Host & select Amazon EC2
Fill out Account Access, Availability Zone sections
In Security Group section click on Custom and leave it on Default, click Next

Results:   Looks like there is no Security Group selected

Expected:  Should say default (sg-1919567c)

@westlywright It needs to take/set an array now instead of a single group.

Version - v1.2.0-pre3-rc3
Verified fixed


Enclose openstack conf in quotes to handle special chars

Coverage remained the same at 62.401% when pulling f3bb9a8d0321e5fe3aaaaebc2df20560e6bfe2cf on gvlproject:quote_special_chars into bad1317fb2100fd329e4ee6d0b6990245a07e5ca on galaxyproject:master.


[WIP] Use stub to get chain code function and arguments

This PR adds chaincode function/args state to the stub.
Description

Motivation and Context

Fixes #
How Has This Been Tested?

Checklist:

[] I have added a Signed-off-by.
[] I have either added documentation to cover my changes or this change requires no new documentation.
[] I have either added unit tests to cover my changes or this change requires no new tests.
[] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

Signed-off-by:

@gabre API's with []string cannot be replaced with []byte.... should be [][]byte.

@srderson what is your opinion about using [][]byte instead of []byte? @esgott ?

@gabre please edit the description to add the DCO signoff. thanks!

@gabre seems you have migrated this PR to Gerrit, can we close this then?

@gabre I am closing this as it seems to have been migrated.


Log long running queries before they complete

Problem Statement
When diagnosing cluster overload problems, or a crash, one thing to review is what queries were running on the server at the time of the issue. In particular, a possible reason to an overload situation would be a single long running query, perhaps typed by hand by a user, that consumes all resources. However, since CrateDB logs queries only when they have completed (because it logs duration of the query), then if the cluster was restarted or crashed, the query doesn't complete and is never logged.
Note that it is possible to see currently running queries in sys.jobs. However, often the cluster is so overloaded it is not possible to open a connection, nor to query sys.* tables.
Possible Solutions
Log queries when they have started. Note that it is not necessary to log all queries, rather, for this particular problem statement, it is sufficient to only log the slowest queries. So for example, all queries that are shorter than N seconds, would be logged as normal when they end. Queries that run longer than N seconds would be logged at that point, and then again once they complete.
Considered Alternatives
for the stated problem, it would also be sufficient to somehow dump all unfinished queries when a node is restarting or crashing. However, this would be unreliable. Especially in OOM cases there's no time for the process to do anything like that. It is killed from the outside and there isn't much we can do a t that point.

it would also be sufficient to somehow dump all unfinished queries when a node is restarting or crashing. However, this would be unreliable. Especially in OOM cases there's no time for the process to do anything like that
This isn't unre

The default configuration of CrateDB tells the JVM to take a heap dump if a node crashes due to an OOM.
This is realiable and the heap dump will contain all queries/operations still running.
The problem is if the node doesn't crash and I'm not sure if a "slow query log" as described here would help. One problem is that if one query is overloading the node - especially if it's due to high memory allocations/GC pressure - it affects other queries running on the node too, so all queries will become slow.
Then you run into the problem described here: https://github.com/crate/crate/issues/14938

it affects other queries running on the node too, so all queries will become slow.

This can be avoided by setting the threshold time high enough. By definition, what we're trying to do here, is to capture queries that caused a cluster crash or restart. So either the slow / long running query happens often, and there are queries in the log that have completed before. Or, it is one (maybe mistyped / mistought) query that didn't complete ever.  It is sufficient to just capture the latter case. In theory the set of queries that would be logged with this feature enabled is small.
In practice setting the threshold high enough but not too high might be difficult. Of course it could be set dynamically to 0.001% of all queries, or something like that.

This can be avoided by setting the threshold time high enough

I don't see how? If you have a high GC load that affects all queries. Even a select 1 could start taking up seconds if it has to wait in a queue because all other threads are busy and if GC is eating up all CPU time.
The issue reports indicate that the cluster is often not reachable or that the admin-ui no longer works. That is an indication that ^ is happening - otherwise the cluster would still be responsive.
This is one reason we added statement_timeout & memory.operation_limit - to cancel queries before they impact the cluster stability. Or the reason we did work like https://github.com/crate/crate/pull/13290

what we're trying to do here, is to capture queries that caused a cluster crash or restart

Again. Cluster crashing with OOM causes a heap dump to be taken - that has all the information one needs, in a better format than what logs would provide.

Again. Cluster crashing with OOM causes a heap dump to be taken - that has all the information one needs, in a better format than what logs would provide.

Yes, this part is clear and solves half of the problem.

I don't see how? If you have a high GC load that affects all queries. Even a select 1 could start taking up seconds if it has to wait in a queue because all other threads are busy and if GC is eating up all CPU time.

Ok, if it's the case that one  query almost immediately impacts performance of other queries and they all therefor are stuck and accumulating somewhat identical response times, then I agree all those queries would end up in the log. But in this case those queries would nevertheless be delineated in time, so that djring normal operation, there would be little to none log entries, and this would only trigger after system stalled for longer than the threshold.
But focusing on the JFRs then...
In parallel the cloud team is working on automating a stream of JFRs to be stored in Pyroscope and vizualized in Grafana. If we have this, then it seems we don't need additional logging. (This ticket would be redundant, with the exception that logging probably for some user is more convenient/easierl

Ok, if it's the case that one query almost immediately impacts performance of other queries and they all therefor are stuck and accumulating somewhat identical response times, then I agree all those queries would end up in the log

If this weren't the case you could connect to the cluster, query sys.jobs and order by duration and get the info you need.
To get to that point, we should rather invest in https://github.com/crate/crate/issues/14068 where I've some new ideas to explore - but what would really help for that would be to have some use-cases/loads we can run that bring a cluster into an unresponsive state

To summarize, three scenarios:

Cluster has high load, but is responsive: Query sys.jobs to figure out what's running or look at always-on JFR/pyroscope profile
Cluster has high load, is unresponsive but recovers eventually: Look at always-on profiling data or wait until it's recovered and then check sys.jobs_log (maybe configure filters or persistent logging for slow queries to ensure the problematic queries are kept)
Cluster crashes with OOM: You'll have a heap dump on disk which can be analyzed

@mfussenegger For the second bullet point, your position is to wait it out and not restart the node?

For the second bullet point, your position is to wait it out and not restart the node?

Depends of course on the context.
If it's an environment where you can affort waiting - it would allow to query the sys jobs tables afterwards.
If you don't want to / can't wait, it helps to first get all information you can:  JFR + heap-dump