Error when printing 'Windows not supported' log

When running the quorum-wizard the above error shows up
Node v14.2.0
Windows 10

Quorum Wizard doesn't work on Windows yet, but we are looking into making it work on windows when using only docker-compose mode. It's supposed to show a message saying that Windows isn't supported, but it looks like it's failing when it tries to print that log. We'll fix that.
In the meantime I would recommend using virtual box or vagrant to run a linux vm. Then you will be able to run the wizard/quorum from inside that vm.

I made an issue to tracker windows support progress here: #63


NLog is Looking for Developers

This year the NLog project has grown a lot:

We have released more often (and  NLog 4.1 is on its way)
We have redesigned and improved the website (nlog-project.org)
The process of giving support and handling issues has been improved
A lot of old issues are fixed/closed.
The website is fully up-to-date, the wiki has been expanded and old forums have been closed.
Twitter and Gitter have been set up.

However, we want to grow even more. We’d like to:

Release even more often
Fix more bugs
Add more features
Improve the support for other platforms like Mono, PCL, Xamarin
Improve the automatic release management
Measure, and improve, the code coverage of the unit tests.

Currently the NLog team consists of two people. If we want to develop NLog further, we need more horsepower!
And so we are looking for developers!
If you are interested in contributing to one of the most popular .Net libraries --we are ranked in the top 100 of NuGet, and when excluding the non-.Net and Microsoft packages, we rank in the top 10.-- then get in touch with us on Gitter or Github :)
Get in touch with us on Gitter or Github :)

I am a .net developer of 10 years experience and would like to be involved. Where to I start.
Kind Regards

Great! I will add you to https://gitter.im/NLog/devs

Hi @tekoscott Do you see the Gitter messages?

@304NotModified I just tried to join by your gitter link. But I see that message:
"404. This is not the chat you're looking for. Sorry. :("

I will invite you :)

OK thanks for that
From: Julian Verdurmen<EMAIL_ADDRESS>Sent: Friday, 28 August 2015 5:21 AM
To: NLog/NLog<EMAIL_ADDRESS>Cc: Scott Brady<EMAIL_ADDRESS>Subject: Re: [NLog] NLog is Looking for Developers (#828)
I will invite you :)

Reply to this email directly or view it on GitHub https://github.com/NLog/NLog/issues/828#issuecomment-135528772 .  https://github.com/notifications/beacon/ADAnWJpriF8CxSdbtXPVx0l4Dqq6xdYdks5or1qvgaJpZM4FmX6h.gif

@tekoscott  I did already added you to the room. Can you follow https://gitter.im/NLog/devs?

Yes thanks
Sent by Outlook for Android
On Thu, Aug 27, 2015 at 12:54 PM -0700, "Julian Verdurmen"<EMAIL_ADDRESS>wrote:
@tekoscott  I did already added you to the room. Can you follow https://gitter.im/NLog/devs?

Reply to this email directly or view it on GitHub:
https://github.com/NLog/NLog/issues/828#issuecomment-135535591

@tekoscott I don't see any chat messages of you in Gitter? Can you see mine?

Yeah I saw them. Sorry I couldn't stay. Have a few clients to see this morning.
Sent by Outlook for Android
On Thu, Aug 27, 2015 at 1:40 PM -0700, "Julian Verdurmen"<EMAIL_ADDRESS>wrote:
@tekoscott I don't see any chat messages of you in Gitter? Can you see mine?

Reply to this email directly or view it on GitHub:
https://github.com/NLog/NLog/issues/828#issuecomment-135549578

@tekoscott I'm confused. After all you weren't interested?

Hi @304NotModified,
I'm a happy nLog user for quite some time, and I will love to contribute.
Have a good day, and let me know what is the next step.

@dnlgmzddr great to hear!  I'm now in holiday, but I will be back this weekend and will give you some more info then :)

No worries, have anice holiday.

Hey @dnlgmzddr please join on on Gitter. I will invite you. Your CV looks really nice :)

@304NotModified
I have been using NLog for about an year or so. I have about 7 years of experience in .Net, how can I help you?

Great to hear! I will invite you to the Gitter chat room this weekend :)

Hi! I would like to make more contributions to the project and I'm hoping you'd have some suggestions for issues to look at next. May I join on Gitter?

Hi,
Did you send me the invite or were you able to add me to the chat room?
On Oct 8, 2015 18:21, "Julian Verdurmen"<EMAIL_ADDRESS>wrote:

Great to hear! I will invite you to the Gitter chat room this weekend :)
—
Reply to this email directly or view it on GitHub
https://github.com/NLog/NLog/issues/828#issuecomment-146702705.

@bhaeussermann Great! You can join us on the public chat: https://gitter.im/NLog/NLog

@SreenathSanthanam. You have been added to the dev channel :) I was a bit (unexpected) busy this weekend. Just chat to use when you have time.

@epignosisx
Can you login on Gitter? Then we can add you to the dev chat channel :)

I'm in!

Hello there! I want to be a contributor. Could you send me an invitation?

Hey! That would be great!

If have send a Gitter invitation.


Installing uni2ts from PyPI gives a ModuleNotFoundError

Describe the bug
I am experiencing a ModuleNotFoundError when attempting to import the package after installation from PyPI. The package appears to install (without error), but it cannot be imported. Other packages in the same environment are working fine.
To Reproduce
Install with PyPI, then do
import uni2ts

Expected behavior
The package should import without errors after installation.
Error message or code output
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'uni2ts'

Environment

Operating system: macOS 14.6.1
Python version: 3.10.13
PyTorch version: 2.4.0
uni2ts version (have you pulled the latest version?): 1.1.0

Additional comments
Installing uni2ts from source works fine.
I am installing uni2ts using poetry by specifying it as a dependency in the pyproject.toml file. I am able to import dependencies required by uni2ts but not uni2ts itself.
Thank you in advance for your help.

Hi @ConstantinaNicolaou,
Could you try again? We found an issue in our previous package on pypi. I just test it and now it should be fine.

Thank you @liu-jc for fixing this so quickly. I can confirm it now works as expected.


Error-prone amount comparison

To compare amounts asset comparison performed prior (assert):
    def __eq__(self, other):
        if isinstance(other, Amount):
            assert other["asset"] == self["asset"]
            return self["amount"] == other["amount"]

This means full dictionary comparison is happening, as self["asset"] is dict with asset data. If any asset data changed between Amounts instantiation, this will lead to error:
a = Amount('10 CNY')
b = Amount('10 CNY')
a['asset']['options']['core_exchange_rate']['base']['amount'] += 1
a == b

The solution is to compare only asset ids:
assert other["asset"]["id"] == self["asset"]["id"]

good idea, can you provide a simple pullrequest to conpare self["asset"]["id"] instead?

Ok, I'll prepare a PR.


Occasional crash when viewing properties of secondary storage device

Crashes while opening properties tab
The app crashes once opening of a properties tab occurs by right clicking.
To Reproduce
Steps to reproduce the behavior:

Go to 'Secondary Storage Device' (Not C:\ Directory)
Right Click on 'Folder'
Scroll down to 'Properties'
Sometimes see error, no idea what's causing it

Expected behavior
To show the properties tab without crashing.
Screenshots
Cannot add, how to add crashing screenshots? All I had was a option saying submit a bug report in the notification panel.
Desktop (please complete the following information):

OS Version: [e.g. Windows 10 2004 19041.508]
App version: [v<IP_ADDRESS>]

Additional context
Does this problem occur again after restarting the app? Yes
Log file
Please post the log file here so that we can understand your problem better. You can access it from Settings->About->Open log location.
debug.txt

This happens when you remove a secondary storage item while the app is still open. After removing the drive, you can still view the files in it. If you rightclick on any item, the full trust process will crash and if you then try to open properties, the application will crash at FolderProperties.cs, line 86.
Instead of letting the user view items of a removed drive, we should display an error message or redirect to home.

@arasthudev Is this still an issue with v0.21.1?

Thanks again for your feedback.
Since we didn't hear from you on this topic for 3+ days, we'll assume everything is worked out and close this feedback item. If you had more feedback to provide, please re-open this and help the owners by describing the current status and any steps taken to troubleshoot this.


nixos/restic: add pre and post start options

Motivation for this change
When creating a backup with restic it is recommended to stop a minecraft server from creating saves. This can be trivially done with these options and I assume that other use cases could profit from this as well.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This would be very nice to have, I would like some way to enable/disable maintenance mode for nextcloud before and after backing up.

Why am I not just writing something like the following instead?
systemd.services restic-backups-foo.serviceConfig.preStart = ''
  # write commands to run before here
'';

systemd.services restic-backups-foo.serviceConfig.postStart = ''
  # write commands to run after here
'';

I don't think that would work because the service definition in the module already populates preStart and postStart ... ?

It works wonderfully! The type is lib.types.lines so everything automatically merges. Additionally you can even use lib.mkBefore, lib.mkAfter, or other related functions to control exactly how the merge occurs. The module system is very powerful.

wow I did not know it would merge, that does make this superfluous then imo


Document the flynn-host fix command

While attempting to recover from every node in my flynn cluster collapsing, I needed some way to try to automatically recover the cluster. I had read a few issues on here before starting and happened upon flynn-host fix which did indeed recover the cluster for me, but it's conspicuously absent from the docs. I only found it referenced on a blog post and in issues on here -- never on the actual documentation.
Whether or not this is the right spot for it is up for debate, but at least right now, it seems like this should be noted somewhere. If this command isn't intended to be used, that's also a fair note, but it did solve most of my cluster problems, which I think makes its value worthy of being documented.

@lmars moved to the debugging section per your request and added a note about collecting logs & backing up. I'm not 100% on relative anchor links in markdown with how this is formatted, though -- so can you let me know if those need to change to be something else (or removed entirely?)?

@lmars okay, I've added the minimum hosts arg and I've changed the anchor link to point to the cluster-backup section instead of backup/restore.

@lmars for the record, I sent this PR because I felt the documentation was needed on the website. If you plan to implement it or have already added it, then feel free to close this PR. If not, do you need additional changes before this can be merged?


Log list: Include log URL in generated code.

When generating a header file, include the URL of the log.

Drive-by LGTM. :wink:

LGTM


App crashes when opening folder

Description
The app crashed when searching for a folder in the "hidden files". But I dont think it necessarily has to do with the "hidden files", because I had the error once before, when just opening a folder.
Steps To Reproduce

Go to folder
Double click to open

Steps to fix
Fix the crash when opening folders
Files Version
<IP_ADDRESS>
Windows Version
Windows 11 22H2 22621.755
Relevant Assets/Logs
Log.txt

Merging with #9921


Faith: colnames replaced with vector of numbers that represent samples

Hi!
I found a bug from estimateDiversity.
I have data that does not contain colnames in assay. Currently Faith is using colnames and because of that samples are NULL --> Faith indices are not calculated.
However, if samples are replaced with just a vector that contains index numbers for samples, samples is not NULL --> Faith indices are calculated.
samples doesn't have any bigger meaning here, so this replacement can be done without errors. However, that error that I got was quite weird and might be hard to resolve for user
-Tuomas

samples doesn't have any bigger meaning here,

By this I mean that colnames are not used as "names", indices works fine

Though I'm not sure if that's more data-related issue than mia-related

Can you bump the version?


fix(rename_doc.py): replace frappe.db.set_value to update sql

Asana: https://app.asana.com/0/1202487840949165/1204163982618361/f
Request:
Update user's email.
Issue:
the Request Time Out pops up when renaming the user's account because it has a lot of related links that are connected
Probably a Solution:
Replace the frappe.db.set_value to Update SQL so that it will directly update the related links and not to call a select query first then update.

Closing PR because I did not change the branch


Appstudio update test-component-pac-syaa

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-syaa-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
10 seconds
init

✅ Succeeded
17 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
18 seconds
deprecated-base-image-check

✅ Succeeded
20 seconds
inspect-image

✅ Succeeded
11 seconds
clair-scan

✅ Succeeded
50 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
13 seconds
label-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
8 seconds
show-summary

Pipelines as Code CI/test-component-pac-syaa-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
22 seconds
clone-repository

✅ Succeeded
34 seconds
build-container

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
17 seconds
inspect-image

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
36 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
6 seconds
label-check

✅ Succeeded
5 seconds
show-sbom

✅ Succeeded
5 seconds
show-summary


follow symlinks for path_relative_to_include

Closes #543
My use case involves wanting to prefix terragrunt dirs with numbers to make the stage order obvious, generate state keys based on path to ensure uniqueness, and use a fair number of terraform_remote_state objects.
Reordering or inserting stages is a time-consuming process involving moving many different state files, adjusting terraform_remote_state objects, and renaming folders.
Also, cding isn't as frictionless as it could be; with the extra often 0-padded prefixes.
Creating numbered symlinks to the terragrunt dirs (but having terragrunt ignore those symlinks) would be an ideal solution.
Things to consider:

I think path_relative_to_include should take a boolean argument if possible; defaulting to the old behavior but allowing the use of this behavior.
Should this be done for path_relative_from_include too?
Haven't yet tested. Especially if not implementing the bool toggle; need to see how this behaves when a relative include path is valid from one but not both of the real and symlinked directories.
Is it alright that this runs filepath.Clean on the path too?

@brikis98 @yorinasub17
Happy to work on this more if I get confirmation that the 'help wanted' tag on #543 is still accurate. (Maybe after >2 years the change is no longer desired?)

@Patricol Sorry for the delay. We are super, super buried right now. The PR is very much appreciated and we'll review it as soon as we can!

Thanks for the PR and your patience as we got around to reviewing it!

Things to consider:

I think path_relative_to_include should take a boolean argument if possible; defaulting to the old behavior but allowing the use of this behavior.

Unless you know of a specific use case where the current approach would break as a result, I'd rather avoid extra flags / branches.

Should this be done for path_relative_from_include too?

Yes, probably.

Is it alright that this runs filepath.Clean on the path too?

Sure.
Note that for symlinks to work, we may also need to solve https://github.com/gruntwork-io/terragrunt/issues/1611.


Add Minimal stylesheet

This adds Minimal stylesheet to the switcher. It also adds a missing closing bracket :)
Note that Minimal stylesheet doesn't have a license. If you don't want to include it because of that, feel free to close this.

@TheLastZombie Thanks for the suggestion! It looks great.
There are only two minor issues with including this, one being the lack of a license as you pointed out (which means that I don't know if it's okay to include the source code here, and others won't know if it's okay to use it in their own projects). The other issue is that the name conflicts with a different Minimal CSS which is already included and has an MIT license.
The Minimal CSS project we have now doesn't seem to be actively developed though (it was last updated 5 years ago), so I would be happy to switch it with this newer one if the license issue could be resolved. Perhaps it might be possible to open an issue at the minimal-stylesheet repo or reach out to @chr15m to request including a license for the stylesheet?

:point_up: :+1:

Awesome, thanks @chr15m! :+1:
@TheLastZombie if you'd still like to submit this PR, could you pull the latest changes to your branch and update the scripts/frameworks.yml file with the relevant info for Minimal CSS? There's no need to manually edit the readme or js script anymore, since they are now auto generated based on the data in the YAML file.
If it would be easier to submit this as a separate PR, that would be fine too. If I can help with anything just let me know!

Sounds good! Two questions though:

Should I really remove Minimal CSS instead of just calling this one Minimal stylesheet CSS?
Minimal stylesheet is actually split into two files, one for basic styles and another for inputs. Before, I simply merged the two files into one, but now? Maybe @chr15m could provide a single "merged" stylesheet.

Hm, good points. I didn't notice that it was separated into two parts. Perhaps we could include the basic HTML version for now and if it's possible for @chr15m to release a combined version at some point we could switch to that instead? It would just be a quick adjustment to the stylesheet URL in the YAML file.
Also, good idea to name this minimal-stylesheet and keep the original minimal css in the repo! I think this will make a nice addition to the list.

This should work hopefully.


fix: remove crd-generator-api dependencies

to be merged after: https://github.com/java-operator-sdk/java-operator-sdk/pull/1436


Conflict detection for PR is broken

Gitea Version
1.16.0, 1.17.0+dev-74-g80048c091
Git Version
2.35.1, gitea demo git version
Operating System
No response
How are you running Gitea?
Built by myself and https://try.gitea.io
Database
No response
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Description

Create new repository
Create commit with simple text file with simple content: 1
Create branch feature1 with modified file content: 11
Create PR feature1 -> master
Create one more commit in master with modified conflicting content: 12
Create branch feature2 with modified file content: 13
Create PR feature2 -> master

Both feature1 and feature2 PRs will show:

This pull request can be merged automatically.
This branch is out-of-date with the base branch

Pressing 'Update branch by merge' or 'Update branch by rebase' will show no any visual changes (just page refresh) on demo site and will show 'Merge conflict checking is in progress. Try again  in few moments' on my own install (refreshing page will show no changes).
Pressing 'Create merge commit' will show red panel reporting merge conflict with bad translation: 'editor.merge_conflict'.
Pressing 'Rebase then fast-forward' or 'Rebase then create merge commit' will show red panel reporting error message.
Pressing 'Create squash commit' will show server internal error 500.
Screenshots

I found at least one problem: https://github.com/go-gitea/gitea/blame/main/services/pull/patch.go#L347
This code applies --3way additional flag for git patch apply. Here goes output of this command (git version 2.35.1, also tested with git version 2.32.0):
git apply --check --cached --3way /tmp/patch1881667419 -vvv
Checking patch README.md...
Applied patch to 'README.md' with conflicts.

and
git apply --check --cached /tmp/patch1881667419
error: patch failed: README.md:1
error: README.md: patch does not apply

And following code is unable to parse output with --3way argument: https://github.com/go-gitea/gitea/blame/main/services/pull/patch.go#L384


feat(utilities): add confirm utility

:tada: This PR is included in version 5.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Event Request] Page 26 Vendor Card - Add new param in publisher OnBeforeCreateVendorFromTemplate

Hi Team
In the Page 26 - Vendor Card, we need to customize some default values. Standard publisher OnBeforeCreateVendorFromTemplate would suit us you could add Vendor record param with following exit functionality. Please refer the red mark in the below image.
Thank Team.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Test Case added CreatePolicy

test cases added for duplicate IDs

Can one of the admins verify this patch?

ok to test


Adding a heading to Infinite Carousel

I have added an appropriate heading and stylised it. I have also added a proper gap between the hero section, carousel and footer

*Sponsors
and  a greeting message like this

improve the responsiveness


Stdio 267 ci for next.js apps

CI for next js  apps
removes conditional check on main ci.yml for node
removes synk job from ci.yml as it was not working. We need to find alternatives
creates ci.typescript.yml for Nextjs reusable worklflow
it is tested on front-office app

Quality work San, thank you.
I have a small suggestion for you:
You made seperate jobs for setup, lint... That's perfectly fine, but in this case, we checkout multiple times and we install dependencies twice which can be time consuming in the future;
My suggestion is: keep one single jobs called "ci" or else, and do 1 checkout and 1 install dependency, we will have substantial gains in execution time of the workflow.
If GH actions had a way to share runners, I would suggest keeping separate job for maintainability and clarity. But Actions does not offer that (there are workarounds that I don't like).
Please tell me what you think or if you need me to implement the suggestion.


New version: gopass.gopass version 1.15.9

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Executable-Error

This manifest adds Apps and Features entries that weren't present in the previous version.


add a group to security associations on clients

Hi,
We have a use-case where we need to add group associations to a client. Currently it's only supported to add a user with the method add_user_associations(). The method is hardcoded to set user=True meaning that we are unable to set groups instead.
I want to be able to do something like this:
associations_list: [
        {
          "group_name": "customer\\Tenant Admin",
          "role_name": "Tenant Admin"
        }

client.add_group_association(associations_list).

Hi @tonyhogsten,
Can you please open Commvault support ticket? We will help you with the enhancements.
Thanks


Fix RedisConnection destruction.

Is there any new information about this fix, is there still a RedisConnection issue?


add "req.authInfo" to the password exchange

This could be used to set other auth params like req.ip, clientAuthenticated, etc.

Coverage remained the same at 100.0% when pulling dc8318a4b3ebf71d8a2f45c449c621759d3fcca9 on jfromaniello:add_auth_info_to_passwd_exchange into 3b23ceb972b3099fe7912a7b6cc6b980e10afa41 on jaredhanson:master.


chore: update to latest kms-go, did-go.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (07098fd) 87.72% compared to head (8759c11) 87.72%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   87.72%   87.72%
=======================================
  Files          53       53
  Lines        7992     7992
=======================================
  Hits         7011     7011
  Misses        714      714
  Partials      267      267

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


Fix some Error Prone and Infer analysis findings

A run of errorprone/infer/fsb shows some basic hygienic things - I fixed a few in this PR:

UnnecessaryParentheses
AutovalueImmutableFields
ClassCanBeStatic
Inefficient keyset iterator

Analysis of this branch shows the above issues are fixed.  I'm open to any preferences or guidelines on tech debt PRs (ex. bundled or individual PRs).
More generally google/error-prone pull requests don't have much in the way of a CI pipeline (just CLA, build) and at least no public facing run of tools including of Error Prone itself.  I've cofounded a group to make running these tools is easier and with zero configuration required. Is there interest in running analysis tools on PRs?  It seems public results from Google's internal processes is unlikely, we could install Muse on this repository to benefit the non-Google contributors.  New bugs are fed back as PR comments but otherwise nothing changes, so in the normal case there is no distraction or mental overhead.
Thoughts?  Is there desire to beef up the CI pipeline and encourage more contributions from the public?

@googlebot I signed it!

@googlebot I signed it!

@googlebot I signed it!


TODO - add Patreon link to homepage

https://www.patreon.com/vuevixens
May as well add it back in. I leave it to you how and where :)

Work in progress


Where can I download the compiler?

Apologies if I am missing something obvious but I could not find a link. Another question I have is, is there a mailing list / google group?
Thanks

Hi @krlvi
As the website says, the language will be released in mid 2019.
You will be able to play with it using an online playground in March.
I've just created a mailing list:
https://groups.google.com/forum/#!forum/vlang


Fixing our use of gherkin-go

… which recently moved the AST into a separate package (cucumber-messages-go)

cc @alexandrinaw  @lgonzalez-silen merged since this is a build fix


can i use with uses-sdk:minSdkVersion 15???

Not for now.
It uses Reveal Animation that only works starting API 21.

I just modified FloatingNavigationView.java to support lower versions of android down to API 15 and Androidx.
Of course without any fancy reveal animations for them, but it works!
FloatingNavigationView.zip
Ps: In order to use this you should create a drawable directory with anydpi-v21 qualifier and move everything from drawable folder to that direction. Then you should use static icons (Or Compat Animated Vector Drawables) for API below 21.

Hello guys! Thanks for your interest in this library.
I just released https://github.com/andremion/Floating-Navigation-View/releases/tag/v1.2.0 using minimum SDK version as 19.
I prefer having a solution that doesn't break the animation since that's the propose of the library.


ElastAlert crashes if value of query_key is not a string

ElastAlert attempts to create the 'key' for silenced alerts by concatenated the value of match[query_key] with the rule name. This causes an error if the value is non-str, for example, a list.
key = '.' + match[rule['query_key']]

TypeError: cannot concatenate 'str' and 'list' objects

Fixed with https://github.com/Yelp/elastalert/pull/57


Is there any API to purge stores?

Is there any API similar to purgeStoredState from redux-persist? Thanks.

I've not found anything made availiable through mobex-persit itself but when purging the stores I have been using methods availiable on local forage. https://localforage.github.io/localForage/#data-api-clear.
eg
if(needToPurge) {
localForage.clear().then(() => rehydrate(myStore))
}

Similarly in react native: AsyncStorage.clear()


Bowflex TC200 Treadclimber support

David Lurie is helping with that. Since step and stair climbers cannot have an interpolated GPS onto a flat running track, first we'll need the ability to upload/export without GPS #243. Maybe besides that we'd want to add vertical elevation climbed to FIT export/upload and other custom formats.
Log 2022-04-08 10_01_14.txt

So unfortunately it sounds like that the TC200 does not implement a standard FTMS service. This will mean we'll need to reverse engineer. The last log here could be better for that.
Log 2022-04-10 09_53_43.txt
According to David "the machine's display console ... has several data displays.  It displays time and distance as the workout progresses (I think I did about 2.4-2.5 miles today), heart rate (although the sensor is kind of flaky - it will display anything from 40-something to over 160 at almost any stage of the workout, but it's usually 80-90 something, which I think is more accurate in the midst of a workout), calories burned (which of course goes up over the course of the workout.  My final number will be 720-740 or 1040+ depending on my workout - it was 720+ today), and speed, which is usually 2.8 or 3.2 mph depending on how I've adjusted the shocks/pistons (2.8 today, with a cool-down at 2.3)."
Log 2022-04-11 09_24_50.txt

I could not identify a clear characteristic which would provide me a time series data. I'll need a better recording.

0xaf20 seems to be constant 88-3F-4A-F9-A6-D6.
0xe520 4 bytes and the first is incrementing every 1-4 minutes (?): B7-CD-78-56
0x9c00 4 zeros, constant
The most promising: 0xb880, 20 bytes long 01-59-AD-00-FC-10-01-00-AF-48-00-00-02-00-00-00-00-00-01-00, always starts with 01, always ends with 00-00-02-00-00-00-00-00-01-00. Looks like we have 9 valuable bytes.

New log from David Laurie "I did over about 5 min., taking a reading every 15-30 sec.  I also took a reading when the machine was in pause, and after I finished the workout."
Log 2022-06-04 14_40_29.txt

I still was not able to clearly identify enough bytes yet

David provided me a new log file.
His notes about the values: "Here's a relatively large log file I did over the last 10-15 min. of my latest workout.  The main numbers on the console are the time and distance, which of course go up as the workout progresses.  Total time was 48:30 and distance was 2.44 miles (I think).  It gives a calories burned total (721) and a calorie burn rate (which was 13.8 for most of the workout and I think 12+ for the last 5.5 minutes).  It also shows speed, which was at 2.8 mph for most of the workout, and 2.3 for the last 5.5 minutes.  It gives a fitness rating at the end, which was 60 today.  The total workout was 48 minutes, and I made recordings from about 35 min.  It gives a heartrate, but the sensor is wonky, and shows anything from 0 to mid-50's to over 150."
Log 2022-08-08 10_06_02.txt

I also realized that Kurt provided me a brief log of the Bowflex BXT216 treadmill, which has the same characteristic 0xb880. I'm trying a new approach now and trying to connect with Bowflex employee via LinkedIn.

One employee marked me back from bowflex / nautilus, but wasn't able to help. Twitter support gave me an email address which bounced but I'll try again


Fix dictionary editor steals focus when reloading script

Fix https://github.com/godotengine/godot/issues/96913
The default value on the form of a typed dictionary (e.g. 0 for an int) would count as an "unsaved change", thus triggering a update_property call, and would trigger the focus.
I am not aware of any case in which focus trigger to the "New Key" or "New Pair" inspectors is desired. Assuming this correct, the proposed change is the simplest possible fix I could find.

But can confirm that this fixes the bug

This is most likely not the right fix as it will break the functionality added here : #88322 for the new key/Value. The bug cited might be a regression from the PR mentioned.

This is most likely not the right fix as it will break the functionality added here : #88322 for the new key/Value. The bug cited might be a regression from the PR mentioned.

Definitely a regression from that. All the functions I debugged through were added by this PR. Will take a look on how we can fix this without breaking the necessary focus change.
I'm thinking of rehauling the focus mechanism instead of removing it. Make it only focus when the type is being changed.

@ajreckof @AThousandShips Can you take a look at the new proposal? It's just a mock. But from my test, the logic prevents needless focus change (i.e. the editor), but causes focus change when the type is changed from the dropdown menu.

So, I’ve reviewed the changes, and unfortunately, the issue persists. While the new proposal does mitigate some problems, changing the type of the new key will refocus the new value, which breaks the functionality in another way.
The section of the code you’ve been modifying is actually correct. The issue stems from the fact that changing_type_index is being assigned a value it shouldn’t have (e.g., -1, which corresponds to new_value when it should be -3 for no_change). I recommend reviewing the parts of the code that assign this value to track down the source of the problem.

I converted the PR to "draft" to make it clear that it still needs more work.

@Dowsley, if you are still interested in addressing this, please let us know. If there’s no response, I will follow up at the end of the month to ensure it is included before version 4.4.

@Dowsley, if you are still interested in addressing this, please let us know. If there’s no response, I will follow up at the end of the month to ensure it is included before version 4.4.

Hey, feel free taking this up. I'm busier than usual unfortunately and won't be able to commit.

superseded by #100932


Add the ability to output a plan to a file and show the plan in json format by adding the -o and -s options to the infrastructure script.

During a pentest the pentester wanted the output of the plan in json format so they could review it. This change adds the ability to output the plan to a file and show the plan in json format by adding the -o and -s options to the infrastructure script.

I still cant quite decide if this is a thing we actually need but sicne I had to write to give stuff to the pentesters we may as well have it in the scripts. it has an issue when you pass in a workspace in that files get called --.json instead of something useful but for the use case its not that much of an issue.


du-mocks

Could you please update the website?
 That said, I've got the base Element class updated (on the ptr branch) so now I have a foundation to build on as I have time.
Also of note, github pages is intended to be the new home for the web/mock codex:
https://1337joe.github.io/du-mocks/web-codex-pts/
That link is to the pts version, since it's what I've started to update. Once it's at least in a consistent state I'll merge it to main and the non-pts codex pages will update.


Output files are not captured yet in AWS Batch Script Runner

Feature description
they are captured but empty
is:

should be:

repro:
id: aws_batch_output_files
namespace: dev

tasks:
  - id: send_data
    type: io.kestra.plugin.scripts.python.Commands
    containerImage: ghcr.io/kestra-io/pydata:latest
    scriptRunner:
      type: io.kestra.plugin.aws.runner.AwsBatchScriptRunner
      region: "eu-central-1"
      accessKeyId: aaa
      secretKeyId: aaa
      computeEnvironmentArn: "arn:aws:batch:eu-central-1:707969873520:compute-environment/kestraFargateEnvironment"
      jobQueueArn: "arn:aws:batch:eu-central-1:707969873520:job-queue/kestraJobQueue"
      executionRoleArn: "arn:aws:iam::707969873520:role/kestraEcsTaskExecutionRole"
      taskRoleArn: arn:aws:iam::707969873520:role/ecsTaskRole
      s3Bucket: kestra-product-de
    commands:
      - python {{workingDir}}/main.py
      - ls
      - cat {{workingDir}}/environment_info.json > {{outputDir}}/environment_info.json
    outputFiles:
      - "environment_info.json"
    inputFiles:
      main.py: |
        import platform
        import socket
        import sys
        import json
        from kestra import Kestra

        print("Hello from AWS Batch and kestra!")

        def print_environment_info():
            print(f"Host's network name: {platform.node()}")
            print(f"Python version: {platform.python_version()}")
            print(f"Platform information (instance type): {platform.platform()}")
            print(f"OS/Arch: {sys.platform}/{platform.machine()}")

            env_info = {
                "host": platform.node(),
                "platform": platform.platform(),
                "OS": sys.platform,
                "python_version": platform.python_version(),
            }
            Kestra.outputs(env_info)

            filename = 'environment_info.json'
            with open(filename, 'w') as json_file:
                json.dump(env_info, json_file, indent=4)

        print_environment_info()

changing cat {{workingDir}}/environment_info.json > {{outputDir}}/environment_info.json
to: cat environment_info.json > {{outputDir}}/environment_info.json
fixes the issue


[Question] any chance we can get a arm64 build for linux and macos?

The prebuilt binaries provided on release page only support amd64 architecture, how about building this tool for arm64 platforms as well on Linux and MacOS? Is it possible with CircleCI?

+1 to this issue.
I am using Apple M1 and when I run make I get the following error:
go build -o ht ./cmd/ht
go: downloading code.cloudfoundry.org/bytefmt v0.0.0-20200131002437-cf55d5288a48
go: downloading github.com/vbauerster/mpb/v5 v5.0.2
go: downloading github.com/logrusorgru/aurora v0.0.0-20200102142835-e9ef32dff381
go: downloading github.com/pborman/getopt v0.0.0-20190409184431-ee0cd42419d3
go: downloading golang.org/x/crypto v0.0.0-20200302210943-78000ba7a073
go: downloading golang.org/x/sys v0.0.0-20200302150141-5c8b2ff67527
go: downloading github.com/VividCortex/ewma v1.1.1
# golang.org/x/sys/unix
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/syscall_darwin.1_13.go:25:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.1_13.go:27:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.1_13.go:40:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.go:28:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.go:43:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.go:59:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.go:75:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.go:90:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.go:105:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.go:121:3: //go:linkname must refer to declared function or variable
../../../../pkg/mod/golang.org/x/sys@v0.0.0-20200302150141-5c8b2ff67527/unix/zsyscall_darwin_arm64.go:121:3: too many errors
make: *** [build] Error 2

Thank you!

@bartmika this looks more like a go tooling problem to me. It makes sense to write which version you were using.

prebuilt binaries would be preferred from the Release page, for some cases need to download this tool to some end user's machine without compiler tools at all


ci: upgrade to node 14

Circle CI is failing with:
[03:18:46] E/direct - Error code: 135
[03:18:46] E/direct - Error message: Could not find geckodriver at null. Run 'webdriver-manager update' to download binaries.
[03:18:46] E/direct - Error: Could not find geckodriver at null. Run 'webdriver-manager update' to download binaries.
[03:19:00] E/driverProvider - Error code: 135
[03:19:00] E/driverProvider - Error message: Unable to create new service: GeckoDriverService
[03:19:00] E/driverProvider - Error: Unable to create new service: GeckoDriverService
Build info: version: '4.0.0-alpha-1', revision: 'd1d3728cae', time: '2019-04-24T16:15:24'
System info: host: '59788622abe5', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'amd64', os.version: '4.15.0-1102-aws', java.version: '11.0.11'
Driver info: driver.version: unknown

Travis CI is reporting some issues with Sauce Connect 4.5.4 failing to connect.
Related note from Sauce Labs support:
Sauce Connect Reminder: End of Support Dates
As part of our continued effort to deliver the best and most secure Sauce Connect experience,
we are focusing on the most recent version, 4.6.3. As announced in March,
we’ve ended support for versions below 4.6.3, and effective July 26,
versions 4.5.4 and below have stopped working.

Action Required: If you are running version 4.5.4 or below,
please upgrade as soon as possible to continue using Sauce Connect.

Thank you @Splaktar for looking into this!!
In relation to travis we shall look into updating https://github.com/angular/protractor/blob/4bc80d1a459542d883ea9200e4e1f48d265d0fda/scripts/sauce_connect_setup.sh


Add cli commands

In robonomics binary we have robonomics io module. But it's not persistent in releases. However it would be good to have access to the library functionality without writing an actual code but rather using an executable script
Please consider adding riio (or call whatever you like) to the repository with at least write datalog, read datalog, write launch, read launch functions

cbf1de51883076ec97bb7a3a36e62ef7017956e0
4cb18e4378fb445cb2f34a31c4895b523b0076b1
1ac66509949830de0f3213f44d55775b17a05c98
2217547c4ed847421d3497b3dc332249ae42fc06
cd9b9c91bda70461ab19319453761a4120469003


Application short "Start In" folder is invalid.

The application shortcut 'Start In' value is invalid. It should be the folder where the executable is located.

Fixed by 6f81c2ca


TestClosestPeer test is sometimes failing

https://github.com/ethersphere/bee/runs/1041108001#step:10:1144
=== RUN   TestClosestPeer
##[error]    kademlia_test.go:538: peers not equal. got<PHONE_NUMBER>000000000000000000000000000000000000000000000000000000 expected<PHONE_NUMBER>000000000000000000000000000000000000000000000000000000
--- FAIL: TestClosestPeer (0.59s)

Closing as duplicate of https://github.com/ethersphere/bee/issues/512. Even this one has up to date failure line number.


generalize ElasticsearchDocumentTransformer

Summary of Changes
Tests
_What tests did you add or modify and why?

Added test to ensure that if no model_class conf is provided for ElasticsearchDocumentTransformer, Exception() will be thrown.
Added test to ensure if model_class conf is provided for ElasticsearchDocumentTransformer is invalid, Exception() will be thrown.

Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[X] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[X] PR includes a summary of changes.
[X] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[X] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[X] PR passes make test
[X] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

Codecov Report

Merging #79 into master will increase coverage by 0.21%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #79      +/-   ##
==========================================
+ Coverage   83.13%   83.35%   +0.21%
==========================================
  Files          53       53
  Lines        2538     2535       -3
  Branches      259      257       -2
==========================================
+ Hits         2110     2113       +3
+ Misses        346      342       -4
+ Partials       82       80       -2

Impacted Files
Coverage Δ

.../transformer/elasticsearch_document_transformer.py
100% <100%> (+19.35%)
:arrow_up:

Continue to review full report at Codecov.

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


Install not working correctly with Faderport 2

Describe the bug
A clear and concise description of what the bug is.
When I install Reasonus everything works as described until I choose CSI in the control surface area. When i choose CSI Reasonus does not apear in the pages section and I do see Faderport 2 in the surface section.
It may be because of this reason that when i choose another automation lane besides volume nothing works. Instead if i choose Pan automation and the volume gets reselected and i can no longer do anything except volume automation.
To Reproduce
Steps to reproduce the behavior:

Go to '...'  Unfortunatley it seems the installer does not work correctly on my device so it would be difficult to reproduce it.
Click on '....'
Scroll down to '....'
See error

Operating System

[] Windows 10 I am on this OS
[] Max OS

Versions
Versions can be found on the About page of the installer

Reasonus Installer: ... 1.01
Reasonus Resources: ...
CSI (ReaSonus): ...

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Sorry for the late response. Can you share the csi.ini file?

Yeah sure here it is,
Cheers
Sean

From: Navelpluisje @.>
Sent: Friday 5 May 2023 17:42
To: navelpluisje/reasonus-install @.>
Cc: jadedsean @.>; Author @.>
Subject: Re: [navelpluisje/reasonus-install] Install not working correctly with Faderport 2 (Issue #40)
Sorry for the late response. Can you share the csi.ini file?
—
Reply to this email directly, view it on GitHubhttps://github.com/navelpluisje/reasonus-install/issues/40#issuecomment-1536511087, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALTPFIY5ZHH6M3BS2OVZRY3XEUUXFANCNFSM6AAAAAAXDFJ6ZA.
You are receiving this because you authored the thread.Message ID: @.***>

I think the attachment didn't make it to github

I cannot up;oad it, it states theses files are not supported here.

Can you change the extension to txt, so it will be named csi.txt?

It works. File looks fine at first sight. Am now not able to dig deeper into it (not at home right now). Will check further next week

Perfect thanks.

Hi any news on this bug fix? I would love to get this up and running.


--format rules SpaceArroundOperator and SpaceBeforeClosingBrace are changing == operator to assignment

Hi,
--format option is changing equal operator == to assignment = in following code:
def something= something.findAll { it.buildgroup == i}

results in:
def something = something.findAll { it.buildgroup = i }

  73    fixed    The closing brace for the closure in class None is not preceded by a space or whitespace  SpaceBeforeClosingBrace
  81    fixed    The opening brace for the closure in class None is not followed by a space or whitespace  SpaceAfterOpeningBrace

tested with:
 npx<EMAIL_ADDRESS>--verbose --noserver --format

That's indeed bad, i'll have a look !
Meanwhile, if you indent your closure you should avoid the problem

Sure, Yeah I'm not complaining, just reporting :)

And i thank you for that ;)

btw there's also a rule called SpaceAfterComma , and it inserts spaces inside strings:

Ooooops not nice at all :/
I'll probably  have to check  there that the char after the comma is not ' or ""

I don't know how it internally works, but I think it would have to semantically know if it is inside string or now, and there are a lot of string notations in groovy. For example It shouldn't change:
"""
test,test
"""

'''
test,test
'''

'an escaped single ,quote: \' needs ,a backslash'


Unable to Build DataImporter

I just fetched the latest source and I am getting build errors. I did the following to try to fix:

Ran Clean on GraphEngine.DataImporter

Changed Framework mode to .net461

Changed Processor Configuration to x64

Executed build.ps1 in the tools directory via PowerShell

Attempted to build GraphEngine.DataImporter project but still getting the compilation error captured in the image below

@yatli Yes- renaming did the trick but a singular error still exist.
This code does not compile:
if (cellid == null) cellid = Global.LocalStorage.NewRandomCellID();
I can't see via intellisense what method should be used instead of NewRandomCellID.

again, NewRandomCellID->NewRandomCellId.

@yatli no - there is not such NewRandomCellId method on Global.LocalStorage  - still getting cannot resolve symbol.

@TaviTruman I think the NewRandomCellId() is removed from LocalMemoryStorage and is replaced by NewCellId() on the "CellIdFactory"
Commit: https://github.com/Microsoft/GraphEngine/commit/19e0e9f018d3b7df01926ab1cc302f1696d9f445

thanks @nvankaam for helping out!
I'd like to update a little bit on this area -- we don't encourage users to use the existing CellIdFactory implementation because it is not collision-free nor distributed-aware. I've been attempting multiple times to bring up something else to provide better guarantees, and get even other benefits like distributed indices (yes they're highly related!). My bandwidth is currently spread across other aspects but once I've got free slots this feature is definitely high on my list.

that being said, we may have inconsistency issues with GraphEngine.DataImporter because it relies solely on CellIdFactory and hash functions to generate cell ids.

related: https://github.com/Microsoft/GraphEngine/issues/118

Yikes 😓 we have a large volume of rdf triples to load; I will see if the fix works after some testing. It looks like newcellid just generates a random number and will not be good enough. So @yati should I even continue to try to use dataimporter given it's current state?

@TaviTruman if you really look into the code or investigate the samples, you'll see that you can already specify a key hasher with TSL attributes.
Generating random ids definitely will not work out for your scenario, because you'll at least have to relate the cell id to a unique identifier (a guid or something).
We've successfully imported FreeBase (billions of triples) with DataImporter.
Please take a look at the sample TSL.

@yati thanks for the feedback Yatao; yes, I see that one is able to adjust the code so that control of the hashkey value is possible. As we all have lots of things in motion it helps to know what one can expect from the code when it comes to built-in capabilities. I know it's possible load large volumes of triples via the DataImporter as your documentation says so; I assumed that specific capabilities is native. The freebase example made available is post processing of the Dataimporter effect. I'm clear on how to proceed - thanks for the clear and poient advice. I'm closing this issue as I have what I need.


"Reach us out through..." Hover Effect Improvement

Current Behavior
The current hover effect of this section is nice but it can be improved by doing some color adjustments. Through some improvements, it can be made even better.
Check the difference of effect in current and first 2 options of expected updated design. In current design, hover effect is bright and uneven. In expected updated effect, you will feel hover effect more real. I have added a deep pressed button effect in hover effect.
Screenshots / Mockups
Current Hover Effect

Expected Updated Hover Effect

I want to work on this improvement as a part of GSSOC-23.

Yeah, looks good @iamRabia-N . Pull the repo properly before updating it and then Open a PR with required screenshots . We will soon review and merge it:)

please assign me this issue

@akshathar-navada Kindly mention the level in the issue that is mentioned in the merged pull request. The merged pull request is mentioned below:
Updated Hover Effect of "Reach us out through..." Section #398


How to pass a custom kubernetes client to helm 3 go client

Given a custom k8s *rest.Config (e.g one created by controller-runtime), how can I can make helm3 clients that uses that?
We using helm to programmatically install charts, and we need this capability to write tests.

@Segflow May be https://github.com/helm/helm/blob/657850e44b880cca43d0606ebf5a54eb75362c3f/pkg/action/action.go#L366

@Segflow did you make it work ? I too have similar situation, need to use controller-runtime rest.Config with helm3 client.
Tried the below:
	kubeConfig := genericclioptions.NewConfigFlags(false)
	kubeConfig.APIServer = &cfg.Host
	kubeConfig.BearerToken = &cfg.BearerToken
	kubeConfig.CAFile = &cfg.CAFile
	kubeConfig.Namespace = &namespace
	if err := actionConfig.Init(kubeConfig, r.Settings.Namespace(), os.Getenv("HELM_DRIVER"), debugLog); err != nil {
		return err
	}

But this doesn't seem to solve the issue, seeing x509: certificate signed by unknown authority error while helm client connecting into api server.

this seems to be passing after having kubeconfig Insecure set with true.

That's too dangerous I believe

That's too dangerous I believe

I think so.

Even with
	configFlags := genericclioptions.NewConfigFlags(false)
	configFlags.APIServer = &config.Host
	configFlags.BearerToken = &config.BearerToken
	configFlags.CAFile = &config.CAFile
	configFlags.CertFile = &config.CertFile
	configFlags.KeyFile = &config.KeyFile
	configFlags.BearerToken = &config.BearerToken
	configFlags.Insecure = &config.Insecure
	configFlags.Namespace = &opts.Namespace

It does not work because when you create an Envtest cluster you get a config where the CERT, CA and keys are not stored inside a file but rather in memory, eg cfg.CertFile is empty, while cfg.CertData is filled. Unfortunately genericclioptions.ConfigFlags does not have those fields.

@yxxhero @Segflow Thanks for coming back on this! yes, I'm using Envtest cluster in the unit tests and i hope it's ok to use insecure option to workaround this issue. do you agree ? But i'm still seeing unable to build kubernetes objects from release manifest: unknown error while installing the chart.

Envtest cluster you get a config where the CERT, CA and keys are not stored inside a file but rather in memory, eg cfg.CertFile is empty, while cfg.CertData is filled

Let me try writing into temp files and use that for genericclioptions.ConfigFlags.

Let me try writing into temp files and use that for genericclioptions.ConfigFlags.

it worked. thanks.


Weird race condition

When I build and run the project the blank output sometimes happens before, sometimes after. I troubleshooted it down to a race condition on line 0. Please take a look.

I looked. A patch has not been generated and a pull request has not been created on a non-existent branch.


refactor(scanner)!: Align provenance storages on write instead of put

Say write instead of put to match read functions and to align with the ScanStorage interfaces.

As discussed, @mnonnenmacher.


Enable  "Dart", "Embedder", and "GC" Timeline streams by default when running a Flutter app

An important profiling use case is tracing app start. In order to use Dart DevTools to visualize Timeline trace from the start of an application, the proper Timeline recording streams need to be set before Dart DevTools connect. DevTools manually sets the "Dart", "Embedder", and "GC" timeline flags when connecting to an app, but this only guarantees that events on these streams will be recorded after DevTools has set those flags.
Even outside of the startup case, having these streams enabled before connecting to tools like DevTools is important so that a user can see a performance problem in their app and immediately open DevTools to analyze the issue. Without the flags set by default, the user would open DevTools, expecting recent app activity to be loaded and ready to analyze, but see nothing. This makes it seem like the tooling is broken, when in reality, it is an issue with the availability of data.
cc @zanderso @rmacnak-google @jonahwilliams @liyuqian

Copying some info from offline thread.
@jonahwilliams:

It looks like those flags are only set when --trace-startup is set?
https://github.com/flutter/engine/blame/e78e405e6a2fcb79956280be019e916a12a8de05/runtime/dart_vm.cc#L376

@jason-simmons:

"flutter run --trace-startup" will enable various timeline streams, collect performance metrics during startup, and then exit the tool.  You can see the accumulated timeline data by running "flutter attach" after it exits and using the observatory.
I vaguely recall that some of the timeline streams used to be recorded by default in old versions of the engine.  I tried to verify this but it looks like old builds of the observatory are not compatible with current versions of Chrome.
If users need timeline recording from the start of the app, then we could expose the Dart VM --timeline_streams flag in the engine and tools.

@jonahwilliams:

I don't really understand why they aren't enabled in the first place. Turning these streams on does not have the memory implications that something like --endless-trace-buffer does right?

@zanderso:

IIRC, the VM buffers for the non-endless timeline are allocated lazily, and are in the 10s of MBs. I'd be reluctant to hardcode in the engine the up-font allocation of these buffers, but if this is something that an engine embedder (think small Fuchsia/CastOS devices, etc.) can configure, then I think it would be okay for the default on desktop, iOS, and Android to be to start with tracing enabled.

@rmacnak-google:

All of the timeline recorders have negligible memory usage until events are recorded. (The fixed-size recorders are allocated up front, but this is direct virtual memory that isn't explicitly initialized, so it doesn't get backed by memory until events are recorded or the timeline is queried.) Which categories/streams are recorded at startup is a policy question for the embedder.
I find it confusing that flutter run --trace-startup automatically kills the application instead of merely enabling streams and choosing the startup recorder.

@liyuqian:

Agree that --trace-startup is very confusing. See https://github.com/flutter/flutter/issues/64781#issuecomment-686089179 and https://github.com/flutter/flutter/issues/58869.
For the missing tracing events during startup, I wonder if it's related to "Fix timeline events lost before dart init https://github.com/flutter/engine/pull/18027"

From the discussion it sounds like enabling these streams by default should be low overhead. @chinmaygarde do you have any concerns with this? @liyuqian Do we have any memory footprint benchmarks that would be able to surface any issues with this change?

Any update on this issue? The "missing" timeline events when a user opens DevTools came up again in a user study. Can we get this fixed soon?

We're unlikely to be able to look at this until next month.

drive by comment, we could get 80% of this by making all of the device class just pass --trace-startup by default and ignoring the command line flag, which has additional complicated behavior

drive by comment, we could get 80% of this by making all of the device class just pass --trace-startup by default and ignoring the command line flag, which has additional complicated behavior

I tried the workflow of running "flutter run --trace-startup", followed by "flutter attach", and I was able to open both Observatory and DevTools with the connected app. The timeline streams are still not enabled though.

Interesting! as far as I could tell that flag would enable them, so there might be something more going on

We observed this as one of the big pain points of UI perf debugging process as well Link

Many noted that it was hard to correlate actions, UI jank, and Timeline events because there is a delay between the UI jank and refreshing the Timeline page. Thus it was easy to miss UI jank issues on the Timeline event chart.
The timeline chart on the Timeline page, performance overlay, and IDE frame rendering chart were also inconsistent.
Participants hoped Timeline could be more automatic and continuous running, just like the IDE plugin performance tools.

We observed this as one of the big pain points of UI perf debugging process as well Link

Many noted that it was hard to correlate actions, UI jank, and Timeline events because there is a delay between the UI jank and refreshing the Timeline page. Thus it was easy to miss UI jank issues on the Timeline event chart.
The timeline chart on the Timeline page, performance overlay, and IDE frame rendering chart were also inconsistent.
Participants hoped Timeline could be more automatic and continuous running, just like the IDE plugin performance tools.

This should be addressed after https://github.com/flutter/flutter/commit/2c23c6c217daf003e09e3dcae8b78cbbd65d4a1d. The engine commit for this https://github.com/flutter/engine/pull/23729 was flagged as causing a perf regression on Golem. I'll keep this open while I investigate that.

This should be addressed after https://github.com/flutter/flutter/commit/2c23c6c217daf003e09e3dcae8b78cbbd65d4a1d. The engine commit for this https://github.com/flutter/engine/pull/23729 was flagged as causing a perf regression on Golem. I'll keep this open while I investigate that.

The above engine change has rolled into the framework.

The above engine change has rolled into the framework.


1ignore

Microsofthttps://github.com/admindev826
<EMAIL_ADDRESS>
From: Bradllez<EMAIL_ADDRESS>Sent: Monday, October 23, 2017 11:22:05 PM
To: Microsoft/vscode
Cc: Subscribed
Subject: [Microsoft/vscode] 1ignore (#36791)

You can view, comment on, or merge this pull request online at:
https://github.com/Microsoft/vscode/pull/36791
Commit Summary

Update browser.js

File Changes

M test/browser.jshttps://github.com/Microsoft/vscode/pull/36791/files#diff-0 (4)

Patch Links:

https://github.com/Microsoft/vscode/pull/36791.patch
https://github.com/Microsoft/vscode/pull/36791.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode/pull/36791, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AcOEgVGvygT1VzNzoSMhp2awAi3yEXozks5svVfdgaJpZM4QDyba.


Being able to run docker srtool without root

Currently, running the docker (in root mode) command:
docker run -i --rm -e PACKAGE=my-runtime -e RUNTIME_DIR=runtime/my -v $PWD:/build $SRTOOL_IMAGE build --app --json -cM

will generate files under root  ownership which is problematic in some scenarios (in our case, github actions).
Forcing the docker user with:
docker run -u $(id -u ${USER}):$(id -g ${USER}) -i --rm -e PACKAGE=my-runtime -e RUNTIME_DIR=runtime/my -v $PWD:/build $SRTOOL_IMAGE build --app --json -cM

fails with error: could not write settings file: '/rustup-home/settings.toml': Permission denied (os error 13):
1.56.1: Pulling from paritytech/srtool
Digest: sha256:ccb1a2407bb779291008d84527c7bd84b93d25350742f873552373e87ee1992d
Status: Image is up to date for paritytech/srtool:1.56.1
docker.io/paritytech/srtool:1.56.1
Build moonbase-local-substitute-tracing…
║ 🧰 Substrate Runtime Toolbox - srtool v0.9.18 🧰
║ - by Chevdor -
info: using existing install for '1.56.1-x86_64-unknown-linux-gnu'
info: override toolchain for '/build' set to '1.56.1-x86_64-unknown-linux-gnu'
error: could not write settings file: '/rustup-home/settings.toml': Permission denied (os error 13)
info: syncing channel updates for 'nightly-2021-11-12-x86_64-unknown-linux-gnu'
error: could not create temp file /rustup-home/tmp/cb3b5166rn_vojo2_file: Permission denied (os error 13)

Ref #35


Missing IterableAssert.inOnlyPane and SingletonAssert.inWindow methods

It seems InterableAssert and SingletonAssert should provide identical pane assertions. But
there is no inOnlyPane for InterableAssert and inWindow for SingletonAssert.
Imported from Jira BEAM-13765. Original Jira may contain additional context.
Reported by: marcin.kuthan.

It has been already resolved #16668


Documentation of winrm options

From the README, it looks like the winrm options should be set under customization_spec.
But from driver.rb in the source code, it seems to be read from [:bootstrap_options][:ssh].
Maybe the bootstrap ssh options could be documented together in their own section?

While not very intuitive, the README does say that the ssh options also apply to WinRM:

[:ssh][:user] user to use for ssh/winrm (defaults to root on linux/administrator on windows)
[:ssh][:password] - password to use for ssh/winrm
[:ssh][:paranoid] - specifies the strictness of the host key verification checking
[:ssh][:port] port to use for ssh/winrm (defaults to 22 for ssh or 5985 for winrm)

Whereas the machine options in customization_spec are for options not covered in the above - things like SSL, timeout, etc.

I may be wrong, but if we look at the code here:
https://github.com/CenturyLinkCloud/chef-provisioning-vsphere/blob/master/lib/chef/provisioning/vsphere_driver/driver.rb

Then, in line 660, create_winrm_transport, options includes winrm_transport and winrm_opts.
create_winrm_transport is called from transport_for, above.
transport_for is called with ssh options, not customization_spec, as in:

transport_for(
        machine_spec,
        machine_options[:bootstrap_options][:ssh]

I did some brief testing and it seemed I had to put the winrm stuff under ssh, not customization_spec.


Love this library but typo in ReadMe

I LOVE this library. THANK YOU very much - it streamlined the heck out of my build/development process (31+ apps!) and I really appreciate you taking the time to make it.
So 99% of this issue is to say thank you.
BUT I think there's a typo on line 165 of the README: link
  `DJANGO_VITE_ASSETS_PATH`. But if you are in production (`DEBUG` is true)

shouldn't that be DEBUG is false?

Many thanks !
Yep you are right, there is a typo, I will fix it as soon as I find time to do it 😄


Add utils.escape tests and fix unicode escaping

I started walking down the path of replacing those myself but once I hit surrogate pairs it seemed to become silly. That's why I went with just adding he.
Fixes https://github.com/mochajs/mocha/issues/2955

Coverage increased (+0.05%) to 89.067% when pulling 61221e2c0eb2bac0076aba19f6ef828af43bf0c5 on jkrems:jk-xunit-valid-xml into 075bd51906b828812b320f33cd1c7fa60d1702f1 on mochajs:master.

@jkrems Thanks!!

The { useNamedReferences: true } part of this PR breaks the XUnit reporter, which now outputs things like ` (which is only valid in HTML, not XML).

Oops, could've anticipated that. @JamesMGreene can you confirm that setting that option to false fixes everything? I'll assume that we could live with slightly uglier " escaping as long as the files are all valid.

Assuming that we're fine with "ugly" escapes for now, there's a potential fix in https://github.com/mochajs/mocha/pull/3003 now. @JamesMGreene Can you give that branch a try to verify that it fixes your issue?


Can't connect to ROS master

Hello,
I've just tried using he library to ocnnect to ROS and I can't for some reason.
Here's my Arduino sketch tha creates a simple publisher:
`#include <ESP8266WiFi.h>
#include <ros.h>
#include <std_msgs/String.h>
#include <std_msgs/Int16.h>
#include <std_msgs/Float64.h>
//WIFI
const char* ssid = "Redmi 4X";
const char* password = "homologacja";
IPAddress server(192,168,172,129);
IPAddress ip_address;
WiFiClient client;
int status = WL_IDLE_STATUS;
class WiFiHardware {
public:
WiFiHardware() {};
void init() {
// do your initialization here. this probably includes TCP server/client setup
client.connect(server, 11411);
}
// read a byte from the serial port. -1 = failure
int read() {
// implement this method so that it reads a byte from the TCP connection and returns it
//  you may return -1 is there is an error; for example if the TCP connection is not open
return client.read();         //will return -1 when it will works
}
// write data to the connection to ROS
void write(uint8_t* data, int length) {
// implement this so that it takes the arguments and writes or prints them to the TCP connection
for(int i=0; i<length; i++)
client.write(data[i]);
}
// returns milliseconds since start of program
unsigned long time() {
return millis(); // easy; did this one for you
}
};
void setupWiFi()
{
WiFi.begin(ssid, password);
Serial.print("\nConnecting to "); Serial.println(ssid);
uint8_t i = 0;
while (WiFi.status() != WL_CONNECTED && i++ < 1000) delay(500);
if(i == 1001){
Serial.print("Could not connect to"); Serial.println(ssid);
//  while(1) delay(500);
}
if(WiFi.status() == WL_CONNECTED)
{
Serial.print("Ready! Use ");
Serial.print(WiFi.localIP());
Serial.println(" to access client");
}
}
ros::NodeHandle_ nh;
std_msgs::String str_msg;
ros::Publisher chatter("chatter", &str_msg);
char hello[13] = "hello world!";
void setup()
{
Serial.begin(115200);
setupWiFi();
delay(2000);
nh.initNode();
nh.advertise(chatter);
}
void loop()
{
str_msg.data = hello;
chatter.publish( &str_msg );
nh.spinOnce();
delay(1000);
}`
On Ubuntu, I first run roscore. Then I do rosrun rosserial_python serial_node.py tcp.
All I see in the Terminal is this:
[INFO] [1512416583.799159]: ROS Serial Python Node Fork_server is:  False [INFO] [1512416583.816673]: Waiting for socket connections on port 11411 waiting for socket connection
After that I turn on my ESP8266. From the Serial Monitor I see that it successfully connects to WiFi. Unfortunately Ubuntu Terminal does not inform about connection. I also cannot see my topic in ROS.
Do you have any idea what might be the issue?
Both Ubuntu and ESP8266 are connected to the same WiFi network of course.

Hi, we've met the same problem. Did you solve it?

I downgraded ROS on the server to kinetic I think. To be exact, I downgraded rosserial only and it worked.

Thanks!

No problem, update this thread later with your result

Hi, I downgraded it and tried your code.  It showed this.
waiting for socket connection [INFO] [1529351133.333148]: Established a socket connection from <IP_ADDRESS> on port 49157 [INFO] [1529351133.334003]: calling startSerialClient [INFO] [1529351135.436264]: Requesting topics... [INFO] [1529351135.438529]: RuntimeError exception caught [INFO] [1529351135.439295]: startSerialClient() exited

@Loreno10 How have you solved it? I have the same problem, I am using ros noetic, Ubuntu 20 and the latest version of rosserial


Corrected typo in "Checkbox list" section.

Changed "commasepareted" to "comma-separated" in the "Checkbox list" section.

thx, we're working on newdocs branch which will hopefully be deployed in a week or two but i'll ensure this is merged in.


bump changelog after 1.8.1

Hello @Seldaek ,
the https://github.com/Seldaek/jsonlint/blob/main/CHANGELOG.md
is missing releases after 1.8.1.
the list of releases does not help, either.

https://github.com/Seldaek/jsonlint/releases contains all changelogs.

thanks to https://github.com/Seldaek/jsonlint/commit/a094f1aaa129f86147d10f41033504a9a286e567
my questions are clear, now.


Just a suggestion

For latest release, the current download URL for binary is:
https://github.com/s0rg/crawley/releases/latest/download/crawley_v1.7.7_$(uname)_$(uname -m).tar.gz
if possible please rename all the release binary files from "crawley_v1.7.7_$(uname)_$(uname%20-m).tar.gz" to "crawley_$(uname)_$(uname -m).tar.gz" (without the version), it will be easy to download from a single URL for example:
eg: wget https://github.com/s0rg/crawley/releases/latest/download/crawley_$(uname)_$(uname -m).tar.gz

Also add "go install -v https://github.com/s0rg/crawley/cmd/crawley@latest" for clean package path download/setup.

Use https://github.com/s0rg/crawley/releases/latest to obtain latest tag and download links
Use: go install -v github.com/s0rg/crawley/cmd/crawley@latest - without http://


Improve story container

The text editors sends its text to the story-container however when there is more text than space it overflows and does not scroll nicely.
Potential fix may be to change the formating of the div to make the story more readable

The respective css for the container can be found here! 😼

Hey @Mera-Gangapersaud and @SeanPrashad I would like to work on this issue. It great that this issue has a mentor assigned, as I am not too familiar with React.

Thanks for tackling this one @deepanjali19 Feel free to ask us questions here or reach out to us on Slack

@deepanjali19 let me know if you run into issues with React, I can help as well.

Thanks @klymenkoo ! I would definitely ask :)

Love that you want to tackle this, despite not being familiar with React @deepanjali19 🙂We'll help you navigate a possible solution for this bug!
Figuring out a solution
To get things started, this is the <div> that we want to have a vertical scrollbar for:
https://github.com/Mera-Gangapersaud/Creative-Collab/blob/3ad0ca36708ee1e0fb9767c9c055c24979a3da39/src/CreativeCollab.jsx#L38
You'll notice it has a className (equivalent to class from regular HTML) of story-container - lets check out the CSS applied to it (found in src/CreativeCollab.css):
https://github.com/Mera-Gangapersaud/Creative-Collab/blob/3ad0ca36708ee1e0fb9767c9c055c24979a3da39/src/CreativeCollab.css#L8-L16
There's a few things that we can do here:

We should add a css property for overflow. This will cause a scrollbar to appear when the contents exceed the height of the <div>. (Extra hint: Look at the documentation for overflow-y 😉)

We can remove these attributes properties of the story-container class:

-  position: absolute;
-  padding-left: 1%;
-  bottom: 5%;
-  left: 20%;

and update the height from 380px; to 340px;.

We can also move it in between the two editors where users author their story. I'll leave this up to you to figure out, but here's what the final product it might look like:

Thank you so much everyone! :)

I think the CreativeCollab.jsx is just a test file. The real game interface shouldn't look like that because it has many probs, for ex: number of users allowed is up to 10, why 2 editors? Is a player supposed to see the other player's editor, or just the output is enough?
After digging into the project for a while, I personally think we are going too fast, making it's hard for further development. What we are lacking of:

The workflow: current description is too general.

When users first get into the app, do they see the game board? No, they need to join a room. So what do they see first? Room list? Button to join? Create room button?
If they create rooms, what's next? Game board now? No, it should be a form with some kind of info, player name? Max number of player? Room name?
Now in game board, at first the host has to wait for other players. How do they know when the game starts? How do they know if it's their turns? Should we have a kind of notification bar or pop up? Now that will affect the interface a lot
So on...

The design: this I believe we should have the wireframes of our app first, then we can start to implement them. For ex:

Now with the two drafts above, we can have a frame for the app, which is very important for collaborating works to be consistent. Our master branch is not well organized. The entry CreativeCollab.jsx is a test file? Should we use App component like other projects? The current App component is TicTacToe game, which is just for research purpose, why it is in master branch? Should we put all components in a folder? For testing, should we include the test to repository or put it in .gitignore? We can do something similar to vuetify, don't include the playground file in the source code, but put them in description of every PR. And so on...

In short words, I believe the app need to be well organized first. Issues like #5 or #9 are good example of making the app bones. This is the very first thing we should do before going for further developments. I'm currently working on #10, and found it quite hard to code because it might conflict with the works of other contributors.
Just some personal opinions 😋

As #19 landed, we can iterate on this initial layout in future PR's

@pynnl Love the level of detail that you've gone into!
If it's ok, I'd like to open a new issue with your post to get some visible discussion going there - stay tuned.

I agree with @pynnl. A few points:

I feel like the current code base has been created without fully realizing how boardgame.io framework works, which is the reason for all the changes introduced in #20.
When I was working on #10 I considered the same issues mentioned in @pynnl's point in The workflow. We need to flush out the details and expectations.
As for the design, PR #20 is a start but it is still only client side. We need to explore how boardgame.io is meant to work with a master server.

I hear ya @dnguneratne - I've made #21 fresh so it's easier for everyone to see and discuss. I'm going to put your comment over there!


Error 419 with Fortify

Bug description
The tenant domains cannot make POST request to Fortify defined routes, getting error 419.
Steps to reproduce

Configure a laravel breeze app with react and fortify.
Make post request to any Fortify route for example /login

Expected behavior
User should be logged in of /login as on central domain but fails with 419 error code.
Laravel version
11.33
stancl/tenancy version
3.8

Please use the right support channels.

Which is? It is not on your website.


Fix navigation state management

State management was broken in Chrome.
Changes the app to store the state as a string instead of a JavaScript object, because it looks like Chrome drops some of the annotations or structure required by Scala.js. Had to fix a bug in the state parsing to ensure that we can restore to the same selection.

Looking at it

Please take another look when you have time @olafurpg

I've been testing this a lot while working on upgrading the Monaco Editor to a newer version. I will it later to unblock that work.


bug/#1040_2 - calls to listeners for ScrollEvents in MapView.setExpectedCenter

Impacted class: MapView

@InI4

Looks reasonable. I will try to dig for an old branch of my app to test this later today.

Don't bother, if the test is more or less "I want to have several ZoomEvents and ScrollEvents during animateTo" I can do that.

Its probably cheap and usually only one listener anyway, but I still feel like the <new ScrollEvent()> could be outside the loop?!

Yeah, I thought the same, and then imagined that you would create a ScrollEvent object for all the cases, even when you don't use at least one listener.
Now that you mention it again I found the solution:
ZoomEvent event = null;
for (MapListener mapListener: mListners) {
	mapListener.onZoom(event != null ? event : (event = new ZoomEvent(this, newZoomLevel)));
}

There are some side-effects, because in MapView.setZoomLevel you call MapView.setExpectedCenter.
Therefore if you animate on both zooms and centers, you may end up calling twice as often ScrollEvents (that happen during zoom change and center change) than ZoomEvents (that happen only during zoom change).

I can confirm your change would have saved the old version of my app which relied on a (Delayed)MapListener which probably also means it had no problems with doubled amounts of events.
Doing no ZoomEvents in case of no listeners? This is memory optimal!

Doing no ZoomEvents in case of no listeners? This is memory optimal!

It was already the case before, as we never entered the listener loop.


CAS Integration

Hello,
I work at a local university in New Jersey, which we have setup Munki. We use Munki extensively on all of our macs, and we are looking to setup MunkiReport-php. Right now, I have setup a PROD and PPRD system for testing, but we need to find a way to integrate CAS so we can use our auth servers for login. Is this something you are planning on integrating or might be able to help me with?
I was looking at different ways that CAS could be integrated, but I hit a few issues when configuring it.

I forgot to mention we are using NGINX as the web server proxy, which is standard in our environment.

I rewrote the whole authentication section quite heavily. It should be easier to integrate CAS authentication, but I don't know much about the innards of CAS. I tried spinning up a docker CAS server but got stuck on configuring it. If you could provide a working CAS setup for testing, I could be of more assistance.

I'm not sure how you setup the CAS server, but if you could give me access to the server or a VM I can install/configure CAS for you to work with. I dont have any spare instances that aren't already being used for production at the moment.

I set up this server: https://github.com/leo108/php_cas_server
I can get the following response:

Current script
index.php
session_name():
session_for:index_php
session_id():
ST-ChL2dxXTcymf4J1pWk2mlldZYMpYE
the user's login is admin.

phpCAS version is 1.3.5.

phpCAS attributes are Array ( [email] =><EMAIL_ADDRESS>[real_name] => admin [oauth_profile] => null ) .

Not sure how to convert that to groups. Could you share your attributes.

What about still having local groups and just match usernames, so for example:
$conf['roles']['admin'] = array('administrator', );
Where administrator is a CAS user account.
I havent had any spare time to get you responses, but if you still need it I will try to ASAP.

Yes, that would be fine..

If you have an idea that you think is better than that, let me know so we can brainstorm. Do you have a branch for the CAS installation by any chance?

No branch yet, I just made some test instances for the CAS server and client. I used this library for the client: https://github.com/apereo/phpCAS

Did you ever figure anything out for CAS? If you have any code that integrates the module already I can mess around with it if you dont have time.

Have we made any headway on this? We would really love this integration.
Also, we use a custom attribute for our SSO authentication called something like ERP_ID, so as long as there is integration for mapping custom attributes we will be set.

Do you have a branch for this somewhere? If so, I can start working on it when I have time since I know you are quite busy.


Merging Accounts Asynchronously Results in SELF_REFERENCE_FROM_TRIGGER Error

This issue has been reported a few times by customers/partners (i.e. #4134 and #3016. But I don't see any further discussions in the Hub.
Summary
If customers are using non-npsp tools to merge records (from an MDM or even just other merge tools), they receive the following error when merging Household Accounts:
There was a problem Merging Account records: npsp.TDTM_Account: execution of AfterDelete

caused by: System.DmlException:
Update failed.
First exception on row 0 with id 001xxxxxxxxxxxx;
first error: SELF_REFERENCE_FROM_TRIGGER, Object 001xxxxxxxxxxxx is currently in a merge operation, therefore a trigger can not update it.: []

Stack Trace
Class.npsp.UTIL_DMLService.updateRecords: line 168, column 1
Class.npsp.HH_HouseholdNaming.UpdateNames: line 128, column 1
Class.npsp.HH_HouseholdNaming.renameHHAccounts: line 452, column 1
Class.npsp.ACCT_AccountMerge_TDTM.handleAccountMergeFixups: line 110, column 1
Class.npsp.ACCT_AccountMerge_TDTM.run: line 66, column 1
Class.npsp.TDTM_TriggerHandler.runClass: line 193, column 1
Class.npsp.TDTM_TriggerHandler.run: line 113, column 1
Class.npsp.TDTM_Config_API.run: line 92, column 1
Trigger.npsp.TDTM_Account: line 33, column 1 | DELETE_FAILED |

Root Cause
This is occurring when a merge operation is called on Accounts asynchronously (I was not able to replicate synchronously via Execute Anonymous). Customers may opt for this method of merging since they are unable to programmatically call the NPSP Contact Merge code. The issue occurs when we attempt to rename the Account of the deleted Account's MasterRecordId.
We did modify the ACCT_AccountMerge_TDTM class in NPSP version 3.166, but I ran a test on 3.165 and still replicated the error.
Steps to Repeat

Create an Apex Class called "BatchContactMerger". Replace the default text with the following:

public class BatchContactMerger implements Database.Batchable<SObject> {
	/* Note that this class only merges 2 Contacts and 2 Accounts at a time.
	 * The customer's system sends a pair of Contacts to merge (no more).
	 *
         * Setup: none required. This will create 2 Contacts to merge.
	 * To run this batch class, enter the following in an Anonymous Apex window:
	 *
         * Id contactBatchId = Database.executeBatch(new BatchContactMerger());
	*/
    public Database.QueryLocator start(Database.BatchableContext bc) {
        List<Contact> contactList = new List<Contact>();
        contactList.add(new Contact(FirstName = 'Test', LastName='Test1'));
        contactList.add(new Contact(FirstName = 'Test', LastName='Test2'));
        insert contactList;

        String query = 'SELECT Id, AccountId FROM Contact WHERE LastName LIKE \'test%\'';
        return Database.getQueryLocator(query);
    }

    public void execute(Database.BatchableContext BC, list<Contact> scope){
        Set<Id> accountSet = new Set<Id>();

        for(Contact con:scope){
            accountSet.add(con.AccountId);
        }
        List<Account> accountList = [SELECT Id FROM Account WHERE ID IN: accountSet];

        // Merge Contacts
        merge scope[0] scope[1];

        //Merge their Accounts
        merge accountList[0] accountList[1];

    }

    public void finish(Database.BatchableContext BC){}

}

Run the following code in an Execute Anonymous Apex window:
Id contactBatchId = Database.executeBatch(new BatchContactMerger());

Confirm that the SELF_REFERENCE_FROM_TRIGGER is surfaced.

To confirm that the merge works synchronously:

Replace all text in the Execute Anonymous Apex window with the following:

List<Contact> conList = new List<Contact>();
conList.add(new Contact(FirstName ='Jess', LastName = 'Test1'));
conList.add(new Contact(FirstName ='Jess', LastName = 'Test2'));
insert conList;

Set<Id> accountSet = new Set<Id>();

List<Contact> conList2 = [SELECT Id,AccountId FROM Contact WHERE LastName LIKE 'test%'];
for(Contact con:conList2){
    accountSet.add(con.AccountId);
}

List<Account> accountList = [SELECT Id FROM Account WHERE Id IN :accountSet];

merge conList2[0] conList2[1];
merge accountList[0] accountList[1];

Confirm that there are no errors.

W-8092617


create2GDAL warning

I downloaded LandsatLink 0.5 in R using
download.file("https://github.com/jdbcode/LandsatLinkr/releases/download/0.5.0-beta/LandsatLinkr_0.5.0.zip", "LandsatLinkr")
I got this Warning message after running LLR Step 6 - Composite imagery:
In create2GDAL(dataset = dataset, drivername = drivername, type = type,  :
mvFlag=NA unsuitable for type Byte

Iam also running landsatlinkr version 0.5 and am encountering this problem
Probably broken install of gdal at 'C:/PROGRA~1/GDAL/'
Error in as.Date.default(unlist(versions)) :
do not know how to convert 'unlist(versions)' to class “Date”
please help

Did you ever solve this problem? I have the same problem.

yeah i actually found a way around it
Happens that landsatlinkr is so choosy when it comes to gdal releases you have to get the very exact release for the version you are using
it could  also be a memory problem
you can try clearing the R memory using this  rm(list=ls(all=TRUE))
just see if that works then we can try other  things

⚠ This repository is no longer in development and is not supported ⚠
You'll likely find that unmaintained dependencies and versioning conflicts make
this package unusable.
Progress is (slowly) being made on ee-LandsatLinkr,
which offers similar MSS-to-successor harmonization procedures using Google Earth Engine.
Unfortunately, I can no longer offer help on this package.


[User Interface: Redesign]: Search result pagination on bottom

Issue Name
Search result pagination on bottom
Issue Description
Currently, the pagination for the search results is at the top.

But, a user might not know if they need to go to the next page, until after they've reviewed what's on the current page. This means that the user will scroll down to browse through the records, and then in order to see the next set of records, they need to scroll all the way back up.
Should pagination elements for the search results be moved (or included) at the bottom?
Issue Discussion
No response
Mockup URL
No response
WCAG Compliance Check

[ ] The ui changes are not expected to significantly affect WCAG compliance (ex- wording changes)
[ ] The ui changes have visual elements which have yet to be tested for WCAG compliance
[ ] The ui changes have visual elements which have been tested for WCAG compliance
[ ] The ui changes have passed all checks for WCAG compliance

Related WBS task

[X] Check here if the ui change is centered on the look and feel (https://github.com/NIAID-Data-Ecosystem/nde-roadmap/issues/52)
[ ] Check here if the ui change includes new features (https://github.com/NIAID-Data-Ecosystem/nde-roadmap/issues/53)
[ ] Check here if the ui change was suggested by the NIAID team (https://github.com/NIAID-Data-Ecosystem/nde-roadmap/issues/2)

For internal use only. Assignee, please select the status of this issue

[ ] Not yet started
[ ] In progress
[ ] Blocked
[ ] Will not address

Status Description
No response
UI change status check list

[X] This ui change has yet to be discussed between NIAID, Scripps, Leidos
[ ] This ui change has been discussed/reported between NIAID, Scripps, Leidos
[ ] This ui change has been mocked up
[ ] This ui change has been implemented on Dev
[ ] This ui change has been approved for Staging
[ ] This ui change has been implemented on Staging
[ ] This ui change has been approved for Production
[ ] This ui change has been implemented on Production

This has been fixed.


Prometheus is not able to talk

I tried it but from the prometheus (kubernetes POD)I am getting below error
time="2018-05-03T17:38:31Z" level=warning msg="Error sending 100 samples to remote storage: server returned HTTP status 400 Bad Request: {"error":"proto: wrong wireType = 2 for field StartTimestampMs"}" source="queue_manager.go:500" time="2018-05-03T17:38:31Z" level=warning msg="Error sending 100 samples to remote storage: server returned HTTP status 400 Bad Request: {"error":"proto: wrong wireType = 2 for field StartTimestampMs"}" source="queue_manager.go:500" time="2018-05-03T17:38:31Z" level=warning msg="Error sending 100 samples to remote storage: server returned HTTP status 400 Bad Request: {"error":"proto: wrong wireType = 2 for field StartTimestampMs"}" source="queue_manager.go:500"

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.


Failure to migrate to Porter v1

I can see an error in the resource processor that indicates there is a failure in the migration process for porter.
We haven't run an upgrade to our TRE in a while and have only just discovered this error. We have managed to upgrade the TRE in every other aspect to v0.14.1, our DEV pipeline is all green.
Describe the bug
In our DEV TRE, any new resources create successfully. Updates to existing resources fails. For example the upgrade to the newer version of the firewall fails. We're unable to see any operations in the UI, in cosmos the error indicates it is unable to find the existing resource.

"resourceVersion": 22,
"status": "updating_failed",
"action": "upgrade",
"message": "db4cced8-7840-4a24-a381-bcd7971d7044: Error message: could not find installation /db4cced8-7840-4a24-a381-bcd7971d7044: Installation not found could not find installation /db4cced8-7840-4a24-a381-bcd7971d7044: Installation not found ; Command executed: az cloud set --name AzureCloud && az login --identity -u 99b4123b-5955-4ecf-b6dc-1198e4b83b41 && az acr login --name acrdevci && porter upgrade \"db4cced8-7840-4a24-a381-bcd7971d7044\" --reference acrdevci.azurecr.io/tre-shared-service-firewall:v1.1.5

When checking the resource processor I can see that this is because the run.sh script failed at the porter migrate step
porter storage migrate --old-home "${PORTER_HOME_V0}" --old-account "azurestorage"
Further investigation shows an error stating:
error reading the schema from the old PORTER_HOME: no file found for  schema: File does not exist

Steps to reproduce

Use Azure Pipelines
Have v0.7.0 of the TRE
Upgrade to v0.14.1, editing the .devcontainer files appropriately
Try to update the firewall or any other resource and it fails.

Porter section of our dockerfile:

Notes:

Porter v1 is successfully set up in MongoDB, files can be found.
The error is exactly the same as mentioned in this issue :https://github.com/microsoft/AzureTRE/issues/3498

But the work around mentioned here doesn't help as it seems to skip the migration.

We haven't run our prod pipeline yet

Please could you advise on what our next steps should be

Hi @JaimieWi , v0.7.0 is going back in time a bit, didn't think anyone would be left on porter v0.
Have you gone straight from  v0.7.0 to v0.14.1? Have you looked at all the breaking changes listed https://github.com/microsoft/AzureTRE/releases on intermediate releases and dealt with them as required?
Afraid you might be in unchartered territory.

Hi @marrobi, Thank you for the quick reply!

I did go through each release, one at a time in our dev pipeline and managed to by pass other errors that occurred. This was all done within the temp TRE in our pipeline. It seems everything else is successful.
I then merged into our DEVCI TRE and needed to update the firewall, which is where I noticed this issue..

Have many people had to do the porter migration?
Just wondering if there are any other troubleshooting steps I can run through?

At the time I believe there were only a small number of production deployments, so maybe a handful, most were in PoC/MVP hence did fresh installs. We, as a dev team did a number on our deployments.
It might be worth raising an issue her e- https://github.com/getporter/porter/issues with the porter specific error.

@marrobi Thank you, I will try raising the issue over there as well!

@JaimieWi have you had any luck?

@marrobi We discussed it internally and have made the decision to redeploy the TRE based on the newest version (and to keep on top of upgrades!)
Thank you for your help

Closing, issue abandoned


Fix missing sessions on voting view

Pull Request Test Coverage Report for Build 323

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0007%) to 99.794%

Totals

Change from base Build 321:
0.0007%

Covered Lines:
2856

Relevant Lines:
2862

💛  - Coveralls


trim #s with pure js

rather than muck with ghost trying to force a helper in. We can achieve basically the same thing by just adding in a class and a bit of pure js.
note - the surrounding brackets define a local scope so that we don't clobber any  var headers which may exist in the global scope.

the only thing I'm not sure about is if the page will initially load with the #s and then disappear. I don't think so ( I think the script is executed before the DOM is visible but without testing it in ghost I'm not positive)

how about now

The fixed typo seems to have fixed the problem in the dev environment. Tag names showing up without "#" symbol.