example wrongly returns 'Address INVALID' when it should return 'valid' on @'Linux Mint 18.3 Sylvia'<EMAIL_ADDRESS>

Hello friends,
sorry, kind od newbie here, I don't have any idea how it can be debugged... But I'm pretty sure that the module doesn't work on my system.
I've tried altcoin-address both in myApp.js (i.e. node myApp.js in terminal) and also in plain Node's REPL - no matter what BTC address (I tried many valid and to be sure also many invalid addresses) I pass to validate function it always evaluates to false.
Here's an output from my console with evaluation of your "valid" example in Node's REPL:
➜  btc-sender-bot node
> var altcoin = require('altcoin-address');
undefined
> var valid = altcoin.validate('1KFzzGtDdnq5hrwxXGjwVnKzRbvf8WVxck', 'bitcoin');
undefined
> if (valid) { console.log('This is a valid address');} else { console.log('Address INVALID'); }
Address INVALID
undefined
>
(To exit, press ^C again or type .exit)

Thanks in advance for any information ;)
Kind regards
onslm

Hi Ondrej,
This sounds to me like a problem with your local configuration. I'm not
really in a position to provide a great deal of support with this library.
Firstly can I just check that you've installed the module correctly via
"npm install altcoin-address"?
I have attached a dump of the steps I used to test that everything still
works on my own machine:
~$ npm install altcoin-address
/home/ryanralph
└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>npm WARN enoent ENOENT: no such file or directory, open
'/home/ryanralph/package.json'
npm WARN ryanralph No description
npm WARN ryanralph No repository field.
npm WARN ryanralph No README data
npm WARN ryanralph No license field.
~$ nodejs

var altcoin = require('altcoin-address')
undefined
var valid = altcoin.validate('1KFzzGtDdnq5hrwxXGjwVnKzRbvf8WVxck',
'bitcoin');
undefined
valid
true

I hope this is of use in your debugging efforts. Please reply back if your
issue is resolved.
On Sat, Apr 28, 2018 at 8:18 PM Ondrej Salamon<EMAIL_ADDRESS>wrote:

Reopened #8 https://github.com/ryanralph/altcoin-address/issues/8.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ryanralph/altcoin-address/issues/8#event-1600052879,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABr4d9w8yS3K5diPzmxxfn_sSJZ4GV_nks5ttEIKgaJpZM4TrVtW
.

--
Ryan Ralph
m: 0408 362 026
e<EMAIL_ADDRESS>w: http://ryanral.ph

~$ npm install altcoin-address
/home/ryanralph
└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>npm WARN enoent ENOENT: no such file or directory, open
'/home/ryanralph/package.json'
npm WARN ryanralph No description
npm WARN ryanralph No repository field.
npm WARN ryanralph No README data
npm WARN ryanralph No license field.
~$ nodejs
> var altcoin = require('altcoin-address')
undefined
> var valid = altcoin.validate('1KFzzGtDdnq5hrwxXGjwVnKzRbvf8WVxck',
'bitcoin');
undefined
> valid
true


Add geography limits

This adds the ability to limit the geography for specific tables, as mentioned in #15 and #13. It uses a tuple, GEO_LIST, to the base class, which subclasses can override to specify a more narrow range of tables (similar to the YEAR_LIST already in use). If a user calls a method that isn't within the allowed geographies, it raises a NotImplementedError.
I also added pretty substantial tests to cover most of this implementation.
One possible improvement would be to move away from representing geographies as strings, because it can get a little confusing to switch between the GEO_LIST items, csv geography suffixes, and actual census API geography filters. Ideally, we would use some sort of canonical enum to store all the different representations for each geography, but I didn't want to make such a large structural change to the library without discussing it first.
This was just a first pass, so I'm happy to make any adjustments as necessary!

Thanks again for all this great work. I've merged it into master.
I've also made a significant refactor of the base.py file to break out the downloading logic into a separate file and set of classes.  If you have a minute, give it a look and let me know what you think.

Thanks for this! I think the new base.py looks really good - a massive improvement vs handling the download logic in each method.
One tiny semantic note -- does it make sense to move base.py into core/? It seems like it fits better with the core functionality than with all the child downloaders.
Part of me still wishes there was a way to avoid the long list of decorated functions that just return their downloader class. My functional programming instinct leans towards creating a map of geography string -> geography class downloader (or just an enum of geographies) then iterate over that to create all the callable methods.
On the other hand, that's much less Pythonic than this class-based approach, so it's probably best to stick with this.
This is probably a separate ticket, but it would also be nice to refactor the tests to mock the Census API instead of hitting it directly -- that would make refactors like this much easier by speeding up the test runtime, and making it so the tests don't rely on a Census API Key (which is killing the Travis builds right now).

Thanks for the feedback. I've moved the base.py file into the new core folder, and made another round of changes to try to streamline things.
I'm potentially up for either of the changes you propose, though I haven't made my mind up.


enable password logins

By default, logins are not allowed via passwords.  They need to be enabled using:
cat <> /etc/ssh/sshd_config
Match user lab*
PasswordAuthentication yes
EOF
service sshd restart

@Miouge1 do you want to put this into the playbooks?

Added to the README

Yes, I'll add that as a check in playbook.

I've added that in the playbook with dfdde8ef57a1695809e1163d1db58d8d68f883da do we still want to keep a mention of it in the README?


streams: Show alert when stream name already exists.

Show Alert when when we try to create a stream with an existing name.
Fixes: #5199

Thanks for review @chrisbobbe!, I've made changes to the PR. Please have a look. Thanks 😊

Also, did you see #5331 (comment) ?

Yes @chrisbobbe!, I read that comment.
So, now we want to handle the edge case where, the stream already exists and user isn't aware of the stream because the stream itself is a private stream.
So as per the suggestion from https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/.23M5199/near/1363029, the server will always throw an error and we have handle it from the server.
So, I've one more doubt @chrisbobbe!, I'm still not able to understand

we have handle it from the server.

It would be a great help, if you could explain this line a little. Thanks 😊

I've responded in the chat thread: https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/.23M5199/near/1366320

Hey @chrisbobbe!, I made some changes please have a look.
Here is the preview
https://user-images.githubusercontent.com/48220369/166110185-da9979d9-d117-4233-ad0d-060b87057fcb.mp4
I hope this will be fine 🤞🏻
Thanks 😊


unix: replaced scale functions with macro and added missing invocations

Hi,
just run into following limitation when using minifb, hope this enhancement can help others.
Best
Johannes

Thanks for the fix!


Progress bar/control bar hiding bugs

On video pages, the "Default" setting now hides both the progress bar and control bar, when default Youtube behaviour is to only hide the progress bar.
Also, on embedded and channel videos, both of the bars will hide no matter what the setting is.
This started happening after updating to build 406.
EDIT: Seems like the first bug only happens on Flash player, but the second happens on both.

Fresh install of Windows 8.1, FireFox 35, and latest developer build. Also have this bug.


Microreactor power cell research

About the PR

made microreactor good (medium powercell that recharges itself in 1 minute)
added t3 research to make it

Why / Balance

microreactor was previously unused
science equivalent of salv finding an antique powercell, its worse overall but can be mass produced which is good especially for med since if salv gets one they are just gonna put it in a floodlight but sci can put microreactors in every defib and scanner :trollface:

Technical details
no
Media
the tech

the recipe

the battery

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
no
Changelog
:cl:

add: Microreactor powercells can be researched under the Tier 3 Portable Fission technology, they slowly recharge on their own.

also thinking of adding hyper cells to it / another t3 as a counterpart with much higher capacity but not self charging

aren't antique powercells small-capacity cells with self reload tho?

no they have best capacity in the game and charge in 30s

needs to be blacklisted as a potential power source for any guns with interchangeable batteries (if any still exist cuz i forgor), causes a major balancing issue otherwise
we used to have something similar called a hardsuit power cell that could be slotted into virtually anything that adopted small battery cells, which pretty much only omitted the standard laser rifle and x-ray cannon back when it was mapped. this is one of the reasons why self-charging batteries have been absent for as long as they have been.


Couldn't find AirNet/requirements.txt file

At the readme file it says to run the following for installation -
pip install -r AirNet/requirements.txt
but the file is not there. Can I find it elsewhere along with the installation instructions?

Thank you for pointing this out! The file wasn't uploaded for some reason. I've added the file now but I had to create it from scratch so there might be some requirements missing.


Another layout throws "out of range" error

Category
[*] Bug
[ ] Enhancement
Problem Area
[ ] Page Transformation: Error during the setup/use of the Page Transformation UI solution (did you check our troubleshooting guide?)
[ ] Page Transformation: Error during the use of page transformation from PnP PowerShell
[*] Page Transformation: Error during the use of page transformation from .Net
[ ] Page Transformation: Page is not looking correct after transformation
[ ] Modernization Scanner: something went wrong...
Expected or Desired Behavior
A classic WebPart page with the following layout : Right Column, Header, Footer, Top Row, 3 Columns is correctly transformed into a modern client-side page. The target classic page has:

1 WebPart in the Header
1 WebPart in the Center Column
2 WebParts in the Center-Right Column

or

2 WebParts in the Header
1 WebPart in the Right Column
2 WebParts in the Center-Left Column
2 WebParts in the Center Column

Observed Behavior
The page transformation engine throws the following error :

Page ClassicPage.aspx could not be transformed: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index

Steps to Reproduce

On a modern team site (no Office 365 group), create a WebPart page inside the Site Pages library with the following page layout : Right Column, Header, Footer, Top Row, 3 Column.
Insert a WebPart in the following zones:

1 WebPart in the Header
1 WebPart in the Center Column
2 WebParts in the Center-Right Column

or

2 WebParts in the Header
1 WebPart in the Right Column
2 WebParts in the Center-Left Column
2 WebParts in the Center Column

Run the page transformation engine (page transformation from .Net using SharePointPnP.Modernization.Framework.dll version 1.0.2001).

Additional informations
This issue might be linked with this one : #371
We can reproduce the same behaviour with the following page layout : LeftColumn, Header, Footer, Top Row, 3 Column
Thanks for your help

Thanks for logging this @a1mery , I'm able to repro and will fix this for the upcoming February release

@a1mery : problem has been fixed. The issue was that we normalized row numbers which resulted in a row number shift (web parts in 3 column row ended up in the above row which just had 1 column, causing the error). The solution is to not do row number normalizing for web part pages as it makes no sense there, this only is needed for wiki pages.
The fix will be part of the February release which will come out either tomorrow or Monday next week.

Great ! Thanks a lot for your work !

Closing this one, please let me know if you still see issues with the February release


ISPN-5154 Generalize JCache constructor so that it can be reused by a custom CachingProvider.

https://issues.jboss.org/browse/ISPN-5154

@pferraro Can you create a JIRA for this?
Also, I can see why you'd want to use an interface as parameter instead of a class. I'm not 100% sure about the need to remove final though. Can you post a link to a branch where you're integrating JCache into Wildfly? I'd like to have a look to see if it could be done any different.

In Infinispan's implementation, there is a 1:1 relationship between a JCacheManager and an EmbeddedCacheManager.  In WildFly's implementation, I expect this to be a many:1 relationship.  That is, rather than a JCacheManager being uniquely identified by both URI and ClassLoader, WildFly's CachingProvider will use a CacheManager per URI, and identify a Cache by both module and cache name.  In this way Cache.getName() might be "foo", but the underlying Infinispan Cache.getName() is actually "foo+module-id".  To do this, I would need to be able to override the JCache.getName() method.

@pferraro Looks ok to me, but I'd squash the commits and add the issue number in the commit message.

Done.

@pferraro Wouldn't delegation work here as well? I agree that's more cumbersome and extra cache object, but a new JCache implementation for WF delegating to the infinispan-cache one might have worked... I'm happy to integrate as is, but just wondering...

Delegation won't work if I want to override the behavior of JCache.getName(), since internal (i.e. within JCache itself) requests for the name of the cache will use the JCache implementation, instead of the decorator implementation.

@pferraro Makes sense, I'll get this integrated right away.


JSON files served with application/octet-stream mime type

This prevents web apps from parsing the contents as a JSON and lead to hard to debug errors.

Merged PR, this will be in the next version of the buildpack.
Thanks for using Cloud Foundry!

Just released v0.5.2.


README: mailpit is in normal Homebrew now

I haven't used Mailpit yet, but it was in the New Formulae list when I updated Homebrew just now.

That's strange because it's not listed/available on https://formulae.brew.sh/formula/ . Are you 100% sure?

Actually I just found https://formulae.brew.sh/formula/mailpit - my guess is it's so new (13 hours) that it's not showing in the search yet. I'll continue to monitor this.

That's strange because it's not listed/available on https://formulae.brew.sh/formula/ . Are you 100% sure?

> which mailpit
/opt/homebrew/bin/mailpit

> brew info mailpit
==> mailpit: stable 1.7.1 (bottled)
Web and API based SMTP testing
https://github.com/axllent/mailpit
/opt/homebrew/Cellar/mailpit/1.7.1 (6 files, 18.1MB) *
  Poured from bottle using the formulae.brew.sh API on 2023-07-14 at 16:14:06
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/mailpit.rb
License: MIT
==> Dependencies
Build: go ✘, node ✔
==> Analytics
install: 0 (30 days), 0 (90 days), 0 (365 days)
install-on-request: 0 (30 days), 0 (90 days), 0 (365 days)
build-error: 0 (30 days)

Yes. Either that, or I'm having the weirdest dream ever.

It's now appearing in the search, so it must be some cache rebuild thing on the brew website. Looks legit, so now I need to figure out how to deprecate my tap without breaking systems.


fix order of arguments in maybeCreateInternetGateway

:)

Oh gees. And they say Stringy typed APIs are the worst 😝
Thanks!


Update policy-csp-admx-windowsstore.md

Why
Current document is incorrect.
Changes
RemoveWindowsStore registry key which is set by this CSP (and by the equivalent GPO) does not work on Win 10 Pro SKU (1809 and later) or Win 11 Pro SKU (any release). Tested multiple times, Pro device allows the user to download and install apps but as soon as you uplift the device to Enterprise SKU and restart, the store becomes blocked.
Old article referring to 1809 and 1903 here: https://docs.microsoft.com/en-us/troubleshoot/windows-client/group-policy/cannot-disable-microsoft-store

Closing this, as the changes are now tracked in https://github.com/MicrosoftDocs/windows-docs-pr/pull/7081

@CoasterKaty We're working on revamping the CSP documentation, so I'm closing this PR at this time. But I've opened an issue for this so we keep a track of this problem.

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/10853


Add timeout and error information.

Add timeout and error information to portscanner. Fixing the enhancement.

The results not worth it.


'package:flutter/src/cupertino/slider.dart': Failed assertion: line 330 pos 15: 'value != null && value >= 0.0 && value <= 1.0': is not true.

Hi all,
in Flutter I'm using Slider.adaptive in this way:
Slider.adaptive(
   min: 0,
   max: _songDurationInMilliseconds.toDouble(),
   value: _songDurationInMilliseconds > _currentDurationInMilliseconds ?
      _currentDurationInMilliseconds.toDouble()
      : _songDurationInMilliseconds.toDouble(),
   onChangeStart: (double value) => audioProvider.invertSeekingState(),
   onChanged: (double value) {
      final Duration _duration = Duration(milliseconds: value.toInt(),);
      audioProvider.updatePosition(position: _duration);
   },
   onChangeEnd: (double value) {
      audioProvider.invertSeekingState();
      audioProvider.audioSeek(seconds: value / 1000);
   },
   activeColor: primaryColor,
   inactiveColor: Colors.white.withOpacity(.2),
)

Slider.adaptive is a built-in slider that render a material slider on Android and a Cupertino slider on iOS.
On Android I have no problems, everything just works fine.
On iOS, where it is used Cupertino slider, I got this error message:

Failed assertion: line 330 pos 15: 'value != null &&
value >= 0.0 && value <= 1.0': is not true.

(This assertion is not present in Material Slider)
Basically the assertion is raised by class "_RenderCupertinoSlider".
As I've shown in the code above, I've set "min" and "max" parameter to define the range of values, which in this case are 0 and 80000, the duration of a song in milliseconds.
What I cannot understand is why having set "min" and "max" parameter, and in particular this second one greater than 1.0, is there a need to check if value is less than 1.0? Obviously is not, I've indeed set "min" and "max" value for this purpose.
Anyone could help me and explain me that?
Thanks

Hi @LeLoMAu
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample? I need you to provide a code that I can just copy/paste to reproduce the issue.
Thank you

Here below the output of my flutter doctor -v:

matteozantedeschi@MacBook-Pro ~ % /Users/matteozantedeschi/flutter/bin/flutter doctor -v
[✓] Flutter (Channel stable, 1.20.4, on Mac OS X 10.15.6 19G2021, locale it-IT)
• Flutter version 1.20.4 at /Users/matteozantedeschi/flutter
• Framework revision fba99f6cf9 (3 settimane fa), 2020-09-14 15:32:52 -0700
• Engine revision d1bc06f032
• Dart version 2.9.2
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
• Android SDK at /Users/matteozantedeschi/Library/Android/sdk
• Platform android-30, build-tools 30.0.0
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.0.1, Build version 12A7300
• CocoaPods version 1.10.0.rc.1
[✓] Android Studio (version 4.0)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 50.0.1
• Dart plugin version 193.7547
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
[!] VS Code (version 1.47.3)
• VS Code at /Applications/Visual Studio Code.app/Contents
✗ Flutter extension not installed; install from
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[✓] Connected device (1 available)
• iPhone 11 Pro (mobile) • 00E1E4BE-D794-4DF3-8046-4736E25BC2BE • ios • com.apple.CoreSimulator.SimRuntime.iOS-14-0
(simulator)
! Doctor found issues in 1 category.

I will try to provide you a minimal complete reproducible code sample, because the one used by me is quite complex.
But basically I think that to reproduce the issue just need something which invoke _RenderCupertinoSlider with a value greater than 1.0.
I noticed that this class (_RenderCupertinoSlider) is invoked only when I pop the current screen with slider and while the music is still playing (and conceptually the slider value still in progress) I go back to it.
When I just remain on the screen _RenderCupertinoSlider is not invoked and therefore values greater than 1.0 are not a problem because the assertion is not evaluated.


Manifest support with mantaray

Currently Bee only has support for uploading collections but not adding files to it or deleting files from them. It would be possible to add support for collections if we implemented mantaray support in bee-js. Mantaray is a binary serialization format used in the implementation of collections and (file) manifests.
Then we could download the manifest, manipulate it directly and then reupload it, effectively implementing what add or delete would do on the Bee Go client.
See also #172.

IMO this is solved with the manifest-js library, so closing this.


add popup warning

Linked Issue
Closes #2148
Description
I was stuck for a while on this and finally realized that it was just the display override because of using flex on the [data-popup] div.
Changsets
Instructions: Changesets automate our changelog. If you modify files in /packages/skeleton, run pnpm changeset in the root of the monorepo, follow the prompts, then commit the markdown file. Changes that add features should be minor while chores and bugfixes should be patch. Please prefix the changeset message with feat:, bugfix: or chore:.
Checklist
Please read and apply all contribution requirements.

[x] This PR targets the dev branch (NEVER master)
[x] Documentation reflects all relevant changes
[x] Branch is prefixed with: docs/, feat/, chore/, bugfix/
[x] Ensure Svelte and Typescript linting is current - run pnpm check
[x] Ensure Prettier linting is current - run pnpm format
[x] All test cases are passing - run pnpm test
[x] Includes a changeset (if relevant; see above)

@codercatdev seems like a good warning to add. I might adjust the formatting a bit, but otherwise I'd expect we'll move forward with this. Thanks!
My PR reviews are typically on Fridays, so look for updates later this week.

@codercatdev This solves a very real issue that had I had completely missed!
Please note I've pushed a small change that helps bring this in line with our standard doc paradigms and tone:

Doc info is provided in a "story" format. We install, show basic usage, then provide warnings. So this made most sense in the latter category.
I've rewritten the text to bring it in line with our standard writing style
Removed the .alert class
Made minor adjustments to the code block snippets to help explain the issue and solution.

Overall this is a great addition. I appreciate your contribution!

@endigo9740 glad it helped! If you ever want to come on the pod with Britt and I lmk.
https://codingcat.dev/podcasts

@codercatdev Alex that's super kind offer. Unfortunately I'm juggling like a million things in my personal life right now and my schedule is an absolute disaster. Once things chill a bit I'd love to take you up on that offer though! I think that would be a blast!


Update home link and license type

Closing, since commits skipping CI need to be pushed to master directly.


can't add custom configure args when using presets

Brief Issue Summary
I'd like to add a --trace argument to CMake configure but the code at https://github.com/microsoft/vscode-cmake-tools/blob/e7ed1b7dcd0f371cb84855d18417978b54500b9e/src/drivers/cmakeDriver.ts#L1295-L1305 only reads the cmake.configureArgs setting when not using presets.
I don't see a reason why not let the user add command line arguments even when using presets.
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

We are considering re-enabling this feature, alongside this feature request: https://github.com/microsoft/vscode-cmake-tools/issues/1836

I would like to second this feature, for a similar reason, with the addition of --debug-output, another related issue is that Cmake: Enable Trace Logging is also ignored.

This should be resolved by #3537. Marking this as fixed-pending-release, could you confirm that this fixes your issue?
Additionally, we plan to add support for CMake Presets v7 and v8 in 1.18, which has trace support.

Thanks for the fix. I'm not working with CMake presets currently, so I won't be able to verify.


feat: Add evictionSoft support to kubeletConfiguration

Fixes #
Description

Adds evictionSoft support into kubeletConfiguration
This adds the fields evictionSoft, evictionSoftGracePeriod and evictionMaxPodGracePeriod

apiVersion: karpenter.sh/v1alpha5
kind: Provisioner
metadata:
  name: default
spec:
  ...
  kubeletConfiguration:
    evictionSoft:
      memory.available: "50%"
    evictionSoftGracePeriod:
      memory.available: 180s
    evictionMaxPodGracePeriod: 120

How was this change tested?

make test

Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

Add `evictionSoft` support into `kubeletConfiguration`

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Good comments! @bwagner5


Update trace4cats-exporter-common, ... to 0.12.0-RC1+188-05746b3c

Updates

io.janstenpickle:trace4cats-exporter-common
io.janstenpickle:trace4cats-kernel
io.janstenpickle:trace4cats-model

from 0.12.0-RC1+146-d193db1e to 0.12.0-RC1+188-05746b3c.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.janstenpickle" } ]

labels: library-update, semver-build-metadata

Superseded by #5.


Do not log "Enabling skip mode" in every failed test

Originally reported by @josefsmrz in a comment in #186:

I am currently testing it in our project and I've noticed these messages:
[fail-fast] Enabling skip mode
appear after every failed test. Perhaps this could be replaced by a message counting the number of failed tests so far, something like:
[fail-fast] Failed tests: x/y

To Reproduce
When BAIL option is used, the skip mode might not be enabled in first failing tests, but the message is displayed each time a test fails.
Expected behavior
Do not log "Enabling skip mode" until the plugin actually enables the skip mode. If a test fails but the bail limit is not reached, then log the number of failed tests and the bail limit: [fail-fast] Failed tests: x/y

Released in v3.4.1


Make a live demo server that is reset hourly

Couldn't even add a banner to it that has a countdown

oh it could exist with nothing installed at all..

https://github.com/haxtheweb/issues/issues/2152


Multi-entry indexes unsupportedOnWeb

used:
@Index(type: IndexType.value, caseSensitive: false)
List get descriptionWords => Isar.splitWords(description);  //failed on this line
got:
packages/isar/src/web/isar_web.dart 18:3
unsupportedOnWeb
Version

Platform:  Web
Flutter version: (Channel stable, 3.3.4, on Microsoft Windows [Version 10.0.19044.2006], locale
cs-CZ)
Isar version: ^3.0.0-dev.5

At 3.0.0 version web is not supported, you will need to wait a bit longer before the web is supported.
yoy can use 2.5.0 which is support web

This issue is obsolete now


Chromium Based Browsers do not close the log-out popup automatically. This means the app doesn't logout until manually closed.

Chromium Based Browsers do not close the log-out popup automatically after completing the log-out step. This means the app doesn't logout until the popup is manually closed. This cannot be fixed with JavaScript on the front-end.

Thanks for letting me know! I will try to have a look at this asap!
Sincerely,
@mstaal
tors. 26. okt. 2023 kl. 21.44 skrev edmcquinn @.***>:

Chromium Based Browsers do not close the log-out popup automatically after
completing the log-out step. This means the app doesn't logout until the
popup is manually closed. This cannot be fixed with JavaScript on the
front-end.
—
Reply to this email directly, view it on GitHub
https://github.com/mstaal/msal_streamlit_authentication/issues/23, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4LJ36KFFXGVTDW62EQKF3YBK4SPAVCNFSM6AAAAAA6RXVP4WVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE3DIMRWGA4TMMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


ReferenceError: Cannot access 'Root' before initialization

Doesn't even work.

Please provide a reproduction the preview site is absolute proof that it does work

Reproduction: literally when importing it in ANY component.
`
<PhoneInput value={'+91<PHONE_NUMBER>'} />
`
in routes/+page.svelte

Yeah import from the directory not the .svelte file thats why you are having this issue


[Bug]: If the expression of the schema is incorrect, there should be a corresponding prompt

ODC version
4.3.2
OB version
no related
What happened?

If the expression of the schema is incorrect, there should be a corresponding prompt
What did you expect to happen?
If the expression of the schema is incorrect, there should be a corresponding prompt
How can we reproduce it (as minimally and precisely as possible)?
Create a logical table creation ticket and enter an incorrect table creation statement
Anything else we need to know?
No response

PASS


IDL Beef bindings

IDL bindings for Beef Programming Language (https://www.beeflang.org/).

Note that I only added bindings for static link since Beef can interop with C/C++ normally.


PublishSingleFile, including wwwroot?

Hi! Would it be feasible to make the web assets an embedded resource and extract on first load? Or even mimic a virtual file system to the browser? Interested in .Net Core single-file publishing for copy-paste portable desktop apps.

Check out my suggestion in #14

@sondreb your suggestion work for the sample with vue.js but the problem seems that with WebWindow.Blazor when you publish singlefile you get the wwwroot on the side
On mac this is the result:

So okey i made it work with the sample
in the .csproj i added ExcludeFromSingleFile="false" for the wwwroot
<ItemGroup>
   <Content Update="wwwroot\**" ExcludeFromSingleFile="false">
     <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
   </Content>
 </ItemGroup>

and of course in the Program.cs
var executingDir = Path.GetDirectoryName(System.Reflection.Assembly.GetEntryAssembly().Location);
ComponentsDesktop.Run<Startup>("My Blazor App", Path.Combine(executingDir,"wwwroot/index.html"));

so now when i publish dotnet publish -r osx-x64 -c release /p:PublishSingleFile=true
i get a single executable file and when the content is extracted in the tmp directory i see the wwwroot

Heads up, you need to be on .Net Core 3.1.200-preview-014977 for ExcludeFromSingleFile="false" to work.


Querying users or patients results with error

Description
Invitations, users are queried and getUsersInformation function is called when viewing /users or /patients pages.
Unfortunately, on staging environment it results with "Missing or insufficient permissions" error. I think this is Firebase Rules error. It happens only when particular permissions are granted. If I am not wrong - owner of organization is faulty (@PSchmiedmayer please correct me if I am wrong).
I wasn't able to replicate that locally nor on dev env.
Reproduction

Sign in as owner to staging env
Click Users or Patients

Expected behavior
Users and patients should be queryable
Additional context
No response
Code of Conduct

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

@pauljohanneskraft @PSchmiedmayer I tested it on both dev and staging env - works ✅


Unable to log in to Microsoft account

I built it on a public network server, dotnet and yarn run successfully, but the front end cannot log in, and the click does not respond

My redirection ip in Azure is public IP: https://x.x.x.x:3000

Azure does not support https://x.x.x.x:3000
I set it to http://localhost:3000/https://localhost:3000/https:x.x.x.x:3000 and I can’t log in
Originally posted by @cooptest in https://github.com/microsoft/semantic-kernel/discussions/943

me tooI'm the same, I hope to get an official answer as well.

I copied the .env file, and edited it like this:
REACT_APP_BACKEND_URI=https//<IP_ADDRESS>:40443/
REACT_APP_AAD_AUTHORITY=https://login.microsoftonline.com/common
REACT_APP_AAD_CLIENT_ID=7284df12-1d8f-4b7c-xxxx-xxxxxxxxxxx

我也有同样的疑惑：
1.我的.env与必须带前缀，不然会报错

2.我右击登陆会发现是一张图片。。。

me too

Is there a solution?

Is there a solution for logging in?

May I ask how you have setup the AAD app? The PKCE error makes me wonder if maybe you did not configure as an SPA?
https://learn.microsoft.com/en-us/azure/active-directory/develop/scenario-spa-app-registration#redirect-uri-msaljs-20-with-auth-code-flow

我也有同样的疑惑： 1.我的.env与必须带前缀，不然会报错   2.我右击登陆会发现是一张图片。。。

@wyz9616 your file should be named just .env not env.env

Hi @cooptest, @PolarPeak , @Suevivian, apologies for a delayed reply.
You'll have to enable the webapp to use HTTPS for local development. I've outlined steps on how to do this here: https://github.com/microsoft/semantic-kernel/blob/1f753fb648c29baeeaa79c492c7a1c774bf06082/samples/apps/copilot-chat-app/webapp/README.md#how-to-use-https-for-local-development
Since you're using your own public IP address, I would recommend using mkcert to generate the necessary certificates, and then you should be able to add your https://x.x.x.x:3000 as a single page application (SPA) redirect URL in your AAD app registration.

Is there a solution for logging in? Now,can only be used on localhost http://localhost:3000
if use ip:3000，will not be able to log in normally
for example： http://<IP_ADDRESS>:3000
error： bundle.js:24299 Uncaught (in promise) BrowserAuthError: pkce_not_created: The PKCE code challenge and verifier could not be generated. Detail:TypeError: Cannot read properties of undefined (reading 'digest')

@Suevivian, AAD has a restriction such that redirect URIs must begin with the scheme https. There are some exceptions for localhost redirect URIs, so if you want to use a specific IP address, you have to use https and follow the steps outlined in this README to configure your local environment accordingly.
See more about redirect URL restrictions here: https://learn.microsoft.com/en-us/azure/active-directory/develop/reply-url


Add the ability to use the browser auth flow

I use 2-factor auth for my account and while I know I can just turn it off, authorize the app, and turn it back on, I don't want to do that all the time. This change lets you pass -b/--browser to fediplay.py to make it use the browser to display a grant code, which the user then pastes into the terminal.

+1 on this change, will test tonight.
Especially since getpass() still doesn't work on my machine (might be an issue with Windows/cygwin), I would even be in favor of making this the default behavior.

My change still uses getpass() for input, when the user needs to paste the grant code back into the terminal.
I assume that kind of a token would be sensitive, but if I'm wrong I could change it to a regular input()

@pwoolcoc Thanks for this! I'm wondering if maybe it might be better to just make this flow the only auth flow?

@pwoolcoc Oh, also, I don't think we need to worry about the sensitivity of the token if getpass isn't sufficiently cross-plat. Keep in mind the token is completely visible in the browser. I'm happy to just use input in this case.

@zigg sounds good to me, I'll make this flow the only login flow, and switch to input


Fix/x axis first label limit

[中文版模板 / Chinese template]
🤔 This is a ...

[ ] New feature
[x] Bug fix
[ ] TypeScript definition update
[ ] Bundle size optimization
[ ] Performance optimization
[ ] Enhancement feature
[ ] Refactoring
[ ] Update dependency
[ ] Code style optimization
[ ] Test Case
[ ] Branch merge
[ ] Site / documentation update
[ ] Demo update
[ ] Workflow
[ ] Release
[ ] Other (about what?)

🔗 Related issue link
related https://github.com/VisActor/VChart/issues/3161

🐞 Bugserver case id
66dad41360183000f72cf0bf

💡 Background and solution

📝 Changelog

Language
Changelog

🇺🇸 English

🇨🇳 Chinese

☑ Self-Check before Merge
⚠ Please check all items below before requesting a reviewing. ⚠

[ ] Doc is updated/provided or not needed
[ ] Demo is updated/provided or not needed
[ ] TypeScript definition is updated/provided or not needed
[ ] Changelog is provided or not needed

🚀 Summary
copilot:summary
🔍 Walkthrough
copilot:walkthrough

before:

after:


All Works - Grid View Chip widths

Active column has different widths.
Make Active chip the same width as the Inactive chip so they line up nicely.
This is needed on:

All works page
Templates page
Staff page
Indigenous page
Proponents page
Projects page

As a (type of user)
I want (goal)
so that (reason/ why/ value)
Acceptance Criteria
Scenario 1:
GIVEN ... (you or your condition)
WHEN ... (what you do)
THEN ... (what you see)
Scenario 2:
xxxxxx
Validation Rules?
(If yes, list here)
Screenshot
Private Zenhub Image
(Link above will always be the most up-to-date version.)

@TomChapmanGov is this still an issue?

@Connor1892 No this is done


Failed when sending attachments

SQLSTATE[HY000]: General error: 2006 MySQL server has gone away (SQL: insert into filament_email_log (from, to, cc, bcc, subject, html_body, text_body, raw_body, sent_debug_info, updated_at, created_at) values ()
getting this error.
and this only happens when we send attachments with the email.

Hi there... Usually a " MySQL server has gone away" error has to do with a server timeout. I did some testing with attachments and it seams to work fine (I will include unit testing on the tests folder for this scenarios in a couple of days).
I tried testing with small attachments (100k), and bigger ones (20M) and works fine. If I try a really big file, like 1G, then I get 'out of memory' errors (this could be solved by saving the attachments to the file systems instead of the database and mede it an option in the config file).

so we will going to get new option in config in next updates?


fix(APIGuildMember): make user required and omit in messages

Currently user is optional on APIGuildMember, but the only case it isn't present is in the message gateway events, because those have message.author so message.member.user would be redundant. This leads to a bad experience when using other member objects, since user will be there but the optionality needs to be handled.
This PR sets user required in APIGuildMember, and omits it where it is not present (GatewayMessageEventExtraFields).

oops i thought i did that. fixed


Force shell option

I'm looking for a way to safely run arbitrary Justfiles. I've considered several approaches and they generally require being able to specify a custom "safe" shell that cannot be subsequently changed (i.e. set shell in a Justfile would be disregarded, and the custom shell must be propagated for any nested just calls).
For example, the shell might be specified in the invocation e.g. just --shell-force X --shell-force-arg Y, or in an environment variable such as JUSTFILE_SHELL_FORCE (which perhaps requires less work to ensure propagation to nested just calls).
Thoughts? Is there a better way to do this? If we submit a PR for this or otherwise assist in the work to implement, would you consider accepting?

Regardless of which shell is used, doesn't a justfile potentially contain arbitrary code which might be dangerous?

I was assuming no but perhaps that is not correct? If we assume that the shell is safe (because this would be a custom shell that is designed to be safe... or perhaps the shell is a wrapper to making a call inside a container), then are there native justfile commands that could be unsafe? import might read something it shouldn't, so that might need to be disabled. Are there others that could read or write files (or make network calls, though that's less of a concern for me) without going through the shell?
Hm, maybe an  easier alternative would be to have a standalone justfile parser and have the caller parse the justfile before running, and raise an error if it contains import or shell-set commands...

an untrusted justfile would be able to find a way around any hypothetical "force shell option"

Are you sure? The shell could be a wrapper to a sandbox, in which case it could try to run python but best case, it would still run in the sandbox.

shell for a justfile can be anything: what if you end up forcing a bash-like shell on a justfile that sets its shell to a Python interpreter

Yea, but then your bash-like shell isn't safe and is not the one I would choose to force.
There are many other ways to achieve a "safe" shell without permitting circumvention. The "safe" shell could simply be a wrapper to which runs bash inside a safe container-- e.g. docker or wasmtime-- in which case even if the shell invokes Python, it is still contained. Or it could be a custom shell such as is the subject of this discussion.

could you please explain what is the end goal you would like to achieve

A simple example use would be an online Justfile playground. This isn't exactly my use case but faces similar requirements (and could probably be pretty useful to add to this repo!). The user can enter whatever justfile they want, and hit "Run", and then they see the output of that.
Typically, "playground" means it runs on the client via e.g. JS and/or wasm. However, there are a few problems with this when it comes to just:

the current just source code does not compile to wasm (afaik)
even if it could compile to wasm, it would probably not be able to invoke a shell command since that is equivalent to fork() which in-browser wasm can't (yet) do

A force-shell option would be a simple solution by making it possible to ensure that the shell is always invoked inside a container even if the just running is not in the container.

it would definitely be best to sandbox/containerize the entire just process

Yea, but that can't always work. For example, just won't work in e.g. wasmtime

So I'm honestly pretty skeptical of this 😅 just's whole raison d'etreis to run arbitrary commands, and so trying to lock it down so it can't seems like a can of worms, and I do agree that sandboxing the whole thing is the best idea. Maybe trying to get it to work in wasmtime?
But for the sake of the discussion:
Just has three sources of commands which it invokes:

Linewise recipe lines, backticks, and shell(…) function invocations, run by the command set by --shell / --shell-args / set windows-shell, and some others.
Shebang recipes, run by the #! line
Script recipes, set by the [script] attribute or set script-interpreter

If you control or can audit, with a custom shell, the arguments and environment variables passed to nested just invocations, then you can only guarantee that the latter two have a particular value:

Can be forced with --shell / --shell-args
Cannot be forced, since it's set with #! lines
Cannot be forced, since there is no --script-interpreter CLI option

You could add new flags which could force the second two somehow. But probably the best thing would be to add a single flag or environment variable which forced the command in all cases to match a given path, like:
just --force-command /path/to/my/command

And then invocation of any command which wasn't /path/to/my/command would fail.
This seems pretty dangerous though, since just wasn't designed with this in mind, and I have no idea if there are other things which could be unsafe. Also, the custom shell itself would be extremely limited, since it would have to not invoke any arbitrary commands, and could only, like you mention, be a wrapper around guaranteed sandboxed operations like operating on a docker container.

Maybe trying to get it to work in wasmtime?

Could probably get it to run but it would not be able to invoke any shell at all because of wasm limitations (fork, threading etc). But, maybe this is a compatible goal with the experimental no-shell version and could be a key part of the solution described below

And then invocation of any command which wasn't /path/to/my/command would fail.

That would definitely be sufficient

I have no idea if there are other things which could be unsafe

If all occurrences of read/write/invoke in the source are centralized and any safety issues handled there, is anything else left? But I understand the concern. To work with a wasmtime version, the problem is that I need some for looping, and so without a shell (a wasm limitation we can't get around) I need a built-in for looping (e.g. https://github.com/casey/just/issues/1570) (as an aside, I think there are lots of benefits to this anyway-- see e.g. analogous Makefile example and comments here)
If those three things could be done, then I would be set:

forced /path/to/my/command shell prefix
can run (shell-less) in wasmtime and simply print (rather than try to invoke) the commands
can handle for looping without shell

So maybe I should first be trying to prod those other issues along...


Redesign the README.md file

If you are a first-time contributor you can contribute to this project by re-designing the README.md file. Use your imagination and make it more beautiful. However, please do not remove important information from the README.md file. Feel free to add more information as you see fit.

Can I work on this?

Hi @Arihant25,
Sure you can work on this. I'll assign you. Thanks for contributing to CineSimile. 😇

Hii @Niweera  if this is pending I wanna work on this

@utkarsh4321 yes, you can work on this. I'll assign you.

I'd like to contribute, can I work on it?

Hi @Niweera can you tell me what is the requirements for README.md so I will start working on it asap

Hi @Niweera can you tell me what is the requirements for README.md so I will start working on it asap

Use your imagination. 😁 Sky is the limit.

Hi @Niweera, I am working on it.

@Niweera Hi, is this issue taken care of or are you looking for multiple submissions to the README.md file.  If still available, I would like to work on the issue.  Thank you for your response.

Hi @Maxgit3 you can work on this. I'll assign you.

Hi @Niweera Can I also work on this and add my ideas...?

@ImagineZero0 sure you can work on this. I'll assign you.


php 5 support

Is there any legacy support for PHP 5 with easydb? I noticed right away there were some PHP 7 features in the source code. I need a good PDO wrapper for an older project.

https://github.com/paragonie/easydb/tree/v1.x-legacy

Thanks so much, you're awesome!


Help with installation

Hello there,
first of all thanks for writing this. I got into the whole matrix game recently and your project looks like a nice and good solution for simple webhooks
Sadly, I cannot get it to work. Maybe I just don't understand the documentation, or maybe there is a slight chance it helps you, or others who try it too.
The steps I took were:

Clone your repo
create a user hook1 with a password
Store both in the env file
Create a new room with my "normal" user and add the user hook1 to it. I named the room "webhook"
Configure env file to use MATRIX_ADMIN_ROOM=!webhook:matrix.mydomain.org
Run everything with ./build-sh
And now I don't understand the logs anymore

matrix-webhook  | Starting the Python app...
matrix-webhook  | 2024-06-25 21:25:01,612 | INFO | module:root | Logging in using stored credentials.
matrix-webhook  | 2024-06-25 21:25:01,657 | INFO | module:root | The web server is waiting for events.
matrix-webhook  | 2024-06-25 21:25:02,353 | INFO | module:root | The Matrix client is waiting for events.
matrix-webhook  | 2024-06-25 21:25:02,457 | INFO | module:nio.client.base_client | New joined room !SnckBOBwICNSxFZvff:matrix.mydomain.org

why is it joining a random room and not the one I told it to? There is no welcome message in the "webhook" room
finally, after a few messages, the container exits with
matrix-webhook  | 2024-06-25 21:25:02,462 | INFO | module:root | We synced, token: XXX

debug logs
I have checked the python debug logs and it does seem to connect to the server
matrix-webhook  | 2024-06-25 21:49:39,533 | INFO | module:root | The Matrix client is waiting for events.
matrix-webhook  | 2024-06-25 21:49:39,614 | DEBUG | module:nio.responses | Validating response schema {...

in the reponse schema I can see my "normal" user, so it must be able to find something. I am guessing that it does connect and the SnckBOBwICNSxFZvff room is just some kind of hash?
env file
My full env file is
MESSAGE_FORMAT=yaml
USE_MARKDOWN=False
ALLOW_UNICODE=True
DISPLAY_APP_NAME=True

MATRIX_SERVER=https://matrix.mydomain.org
MATRIX_SSLVERIFY=True
MATRIX_USERID=@hook1:matrix.mydomain.org
MATRIX_PASSWORD='secret'
MATRIX_DEVICE=docker
MATRIX_ADMIN_ROOM=!webhook:matrix.mydomain.org

KNOWN_TOKENS='YOURSECRETTOKEN,!webhook:matrix.mydomain.org,Curl'

Could you help me to try to figure out the problems? Sorry in advance if its some easy fix that I either didnt think about or did not read properly in your documentation

Hi Jonny,
Based on your steps taken everything looks fine, but the only mistake I see
it is related to your environment file setup, on the
MATRIX_ADMIN_ROOM=!webhook:matrix.mydomain.org and
KNOWN_TOKENS='YOURSECRETTOKEN,!webhook:matrix.mydomain.org,Curl'.
Maybe I should specify that is should be the ROOM ID, not the room name.
Indeed would look something similar like you found in your logs
!SnckBOBwICNSxFZvff:matrix.mydomain.org which is the room ID, and because
the user hook1 is already there (added by you) it joins the room when the
hook is running.
Using Element client in browser and entering the room, you can see the room
id in the URL, or alternatively, in the Element app, you can press Invite to this room and when the window opens, on the top side click on share this room which will give you the ROOM ID link.
Your env file should look like this, assuming !SnckBOBwICNSxFZvff: matrix.mydomain.org is your ROOM ID
`MESSAGE_FORMAT=yaml
USE_MARKDOWN=False
ALLOW_UNICODE=True
DISPLAY_APP_NAME=True
MATRIX_SERVER=https://matrix.mydomain.org
MATRIX_SSLVERIFY=True
@.***:matrix.mydomain.org
MATRIX_PASSWORD='secret'
MATRIX_DEVICE=docker
MATRIX_ADMIN_ROOM=!SnckBOBwICNSxFZvff:matrix.mydomain.org
KNOWN_TOKENS='YOURSECRETTOKEN,!SnckBOBwICNSxFZvff:matrix.mydomain.org,Curl'`
Let me know this helps!
În mie., 26 iun. 2024 la 00:03, Jonny Ebinger @.***> a
scris:

Hello there,
first of all thanks for writing this. I got into the whole matrix game
recently and your project looks like a nice and good solution for simple
webhooks
Sadly, I cannot get it to work. Maybe I just don't understand the
documentation, or maybe there is a slight chance it helps you, or others
who try it too.
The steps I took were:

Clone your repo
create a user hook1 with a password
Store both in the env file
Create a new room with my "normal" user and add the user hook1 to
it. I named the room "webhook"
Configure env file to use MATRIX_ADMIN_ROOM=!webhook:
matrix.mydomain.org
Run everything with ./build-sh
And now I don't understand the logs anymore

matrix-webhook  | Starting the Python app...
matrix-webhook  | 2024-06-25 21:25:01,612 | INFO | module:root | Logging in using stored credentials.
matrix-webhook  | 2024-06-25 21:25:01,657 | INFO | module:root | The web server is waiting for events.
matrix-webhook  | 2024-06-25 21:25:02,353 | INFO | module:root | The Matrix client is waiting for events.
matrix-webhook  | 2024-06-25 21:25:02,457 | INFO | module:nio.client.base_client | New joined room !SnckBOBwICNSxFZvff:matrix.mydomain.org
why is it joining a random room and not the one I told it to? There is no
welcome message in the "webhook" room
finally, after a few messages, the container exits with
matrix-webhook  | 2024-06-25 21:25:02,462 | INFO | module:root | We synced, token: XXX
debug logs
I have checked the python debug logs and it does seem to connect to the
server
matrix-webhook  | 2024-06-25 21:49:39,533 | INFO | module:root | The Matrix client is waiting for events.
matrix-webhook  | 2024-06-25 21:49:39,614 | DEBUG | module:nio.responses | Validating response schema {...
in the reponse schema I can see my "normal" user, so it must be able to
find something. I am guessing that it does connect and the
SnckBOBwICNSxFZvff room is just some kind of hash?
env file
My full env file is
MESSAGE_FORMAT=yaml
USE_MARKDOWN=False
ALLOW_UNICODE=True
DISPLAY_APP_NAME=True
MATRIX_SERVER=https://matrix.mydomain.org
MATRIX_SSLVERIFY=True
@.***:matrix.mydomain.org
MATRIX_PASSWORD='secret'
MATRIX_DEVICE=docker
MATRIX_ADMIN_ROOM=!webhook:matrix.mydomain.org
KNOWN_TOKENS='YOURSECRETTOKEN,!webhook:matrix.mydomain.org,Curl'
Could you help me to try to figure out the problems? Sorry in advance if
its some easy fix that I either didnt think about or did not read properly
in your documentation
—
Reply to this email directly, view it on GitHub
https://github.com/gabbysHub/matrix-webhook/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWFFBPBWRY3QIOXEQDOBA7TZJHSKZAVCNFSM6AAAAABJ4VTAUOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM3TGNZUGE4DMNY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Jonny,
Based on your steps taken everything looks fine, but the only mistake I see
it is related to your environment file setup, on the
MATRIX_ADMIN_ROOM=!webhook:matrix.mydomain.org and
KNOWN_TOKENS='YOURSECRETTOKEN,!webhook:matrix.mydomain.org,Curl'.
Maybe I should specify that is should be the ROOM ID, not the room name.
Indeed would look something similar like you found in your logs
!SnckBOBwICNSxFZvff:matrix.mydomain.org which is the room ID, and because
the user hook1 is already there (added by you) it joins the room when the
hook is running.
Using Element client in browser and entering the room, you can see the room
id in the URL, or alternatively, in the Element app, you can press Invite to this room and when the window opens, on the top side click on share this room which will give you the ROOM ID link.
Your env file should look like this, assuming !SnckBOBwICNSxFZvff:matrix.mydomain.org is your ROOM ID
MESSAGE_FORMAT=yaml

USE_MARKDOWN=False
ALLOW_UNICODE=True
DISPLAY_APP_NAME=True

MATRIX_SERVER=https://matrix.mydomain.org
MATRIX_SSLVERIFY=True
MATRIX_USERID=@hook1:matrix.mydomain.org
MATRIX_PASSWORD='secret'
MATRIX_DEVICE=docker
MATRIX_ADMIN_ROOM=!SnckBOBwICNSxFZvff:matrix.mydomain.org

KNOWN_TOKENS='YOURSECRETTOKEN,!SnckBOBwICNSxFZvff:matrix.mydomain.org,Curl'

Let me know this helps!

Hey,
yes this fixed the issue! Thanks a lot now I can play around with it!


Kestrel/async

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: pcoccoli:white_check_mark: subbyte:x: princesaroj111You have signed the CLA already but the status is still pending? Let us recheck it.


Add upgrade authorization

I'm running wstunnel on an public web server, and as the third line of security (after TLS and upgradePathPrefix) I'd like to use Basic HTTP authorization because a path prefix is basically a password stored in plaintext. Should I implement reading from file? The credentials can be read by other processes by looking at the command line of a wstunnel process.

Sorry for the delay, for unknown reason, I haven't seen your PR.
Thanks for taking the time to add the feature :+1:

Hello @fopinappb ,
If you want to do that you can use upgradePathPrefix, just define a random string that you share between server and client, and you will be able to authorize connections to your server.

I just checked the code and indeed my bad, the server does not check the path prefix, I though it was :x


updated link to dashboard

hi @jkremser, can we please also update some of the copy for the app ui view? see suggestions in-line here:
https://capture.dropbox.com/aVNCESkJgWpJ6O3q

closing on behalf of https://github.com/kedify/examples/pull/19


[MM-12155] Create new SendCtrlEnterSection component from AdvancedSettingsDisplay

Summary
Create new SendCtrlEnterSection component from AdvancedSettingsDisplay
Also, use constant for 'code_block_ctrl_enter' from mattermost-redux
Ticket Link
Jira ticket: MM-12155
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)

@saturninoabril what else needs to be done here? maybe if you add more stuff to this PR ask for my review again

I'm closing this PR temporarily and will reopen once I get back into this.


[BUG]Hitting Tab or shift+tab opening all the foldings

Hitting Tab or shift+tab opening all the foldings
recently if hitting tab or shift+tab, opening all the earlier foldings automatically.
Expected behavior
if the foldings remain static then it would not make the interface busy and it would be delightful to work.
Desktop (please complete the following information):

OS: Windows 10
Obsidian Version: 0.12.3

@semits20 Sorry, seems like I can't understand your problem. Could you please add more details?


Create timestamp column on users indicating their obs may need to be aggregated again

During project aggregation, to look up what observations might need to be aggregated into a project, we look at all observations updated since the last aggregation, but also need to look at all the observations of users who may have changed their preferences that might affect aggregation (the Which projects can add your observations? setting or per-project access to private coords). The way this is implemented now is just looking at users that have been updated since the last aggregation (https://github.com/inaturalist/inaturalist/blob/master/app/models/project.rb#L745) but that's including too many users and slowing down aggregation (we update a user record every time they are active on the site).
So it would be great to have a column on user indicating when the last time a user changed their global project settings, and also make sure the ProjectUser updated_at column is updated when users change their project preferences. We can then compare those two timestamps to the project last_aggregated_at time to get a much smaller list of users whose obs need assessing.

Fixed in c0f3d81


feat: handle composition

Why?
When the user inputs with IME, the middle process will be committed, that's unnecessary. This PR wants to optimize this behaviour. When the editor is in the composing state, Yjs don't need to commit the changes. When the composition was finished, the plugin commits the changes.
Before

After

Hi @vincentdchan, thank you for this!
I'm wondering how this workaround behaves in a collaborative environment. I expect that remote changes will delete the composing state which is probably not what you want.
Would it be possible for you to test if this works reliably when other people are editing the same document / paragraph?

I have create a project with y-prosemirror. It use vincentdchan:feat/ime branch.
It has some problem. While delete a some content. The user is under composing, cursor comes wrong.
I have check the shimo's behavior. During composing, It much like a offline editor.

type 123456

insert between 34 with composing aa

cursor data received by another user

😀I have implement a custom provider based on socket.io.
While composing, I make sync suspense. Just like disconnect.
After composing, I send a sync message. It works fine.
😔I tried modify the demo in y-prosemirror, but the disconnect action in y-webrtc provider not works fine.


issue loading traffic profiles

i have installed the Gui version, but i am unable to load the existing yaml profiles from cap2.
Every time  I get "Error Loading file ***.yaml"
I am using Ububtu 20.04
and Trex-stateless-gui -4.5.6 with trex version 2.87

My mistake I was trying to import State full profiles, this is not possible as GUI is stateless, Can import stateless profiles.

My mistake I was trying to import State full profiles, this is not possible as GUI is stateless, Can import stateless profiles.


Transaction enabled migrations always complete successfully even if migration failed.

Description
When mongock is configured to use transactions, and the migration fails for any reason; the migration gets rolled back (as excepted), but the error is not rethrown. This results in the migration to appear as successful even thou no changes where performed.
PRIORITY
CRITICAL - it is not feasable to use mongock transactions as you would not know whether the transaction was completed successfully and thus the rest of the application cannot depend on the migration.
Steps to Reproduce

Enable mongock transactions.
Write migration that fails.
Start application.

Specific scenario
We use Mongock to trigger migrations during application startup. Should the migration fail we expect the application to fail startup, but this is not possible due to this issue.
In our scenario it is required to use transactional migrations since we have a high availability environment and other instances of our application might access the database in parallel.
Expected behavior: On failure of transactional migration we expect to either receive the SpringMigrationFailureEvent or ideally have the migration process fail, in addition to the existing database rollback.
Actual behavior: Transactional migration completes successfully event if it actually failed and rolled back.
Reproduces how often: on failure 100%
Version
      <dependency>
        <groupId>com.github.cloudyrock.mongock</groupId>
        <artifactId>mongock-spring-v5</artifactId>
        <version>4.3.5</version>
      </dependency>
      <dependency>
        <groupId>com.github.cloudyrock.mongock</groupId>
        <artifactId>mongodb-springdata-v3-driver</artifactId>
        <version>4.3.5</version>
      </dependency>

Additional Information
Having reviewed the open source code we believe we have identified the source of this issue.
In SpringDataMongoV3Driver#executeInTransaction ln 112 an exception that happens within the transaction is being caught in order to be able to rollback. In this catch (after the rollback) I would expect that the exception would be rethrown such as throw new MongockException(ex) . This would be similar behaviour to the parent implementation in MongoSync4DriverBase.
The same issue seems to be present (not tested) in SpringDataMongoV2Driver
Sample fix  SpringDataMongoV3Driver#executeInTransaction
  @Override
  public void executeInTransaction(Runnable operation) {
    TransactionStatus txStatus = getTxStatus(txManager);
    try {
      mongoTemplate.setSessionSynchronization(SessionSynchronization.ALWAYS);
      operation.run();
      txManager.commit(txStatus);
    } catch (Exception ex) {
      logger.warn("Error in Mongock's transaction", ex);
      txManager.rollback(txStatus);
      throw new MongockException(ex);
    }

  }

Thank you @stikku , it's already identified and will be released in the version 4.3.6.
Thanks

@dieppa I have tried running my application with using mongock master (built locally) 4.3.6-SNAPSHOT and I am encountering the same issue. I assume the fix is yet to be merged?
Since this issue is a blocking us from deploying some changes to our own application, would you have a rough estimate of when 4.3.6 will be released? or when it will be available on master so we can test it?
Thank you a lot for your assistance and we would be happy to provide any help in solving this issue.

Ah sorry, it's not provided yet in that version.
Unless something unexpected happen, tomorrow will be released ;)

Hello @stikku , can you clone check the branch issue/303? You just need to build it locally and update the mongock-bom version to 4.3.6.SNAPSHOT

Hello @stikku ,
can you checkout the branch issue/303, build locally and confirm the issues is fixed. We have added it to our integration tests, but I prefer you to check first.(next time we'll provide an snapshot version)
Thanks

Hello @dieppa
I have built locally as follows:
mongock -> branch issue/303
mongock-core -> master
And I received the following error during spring startup:
2021-03-04 13:28:08.311  WARN 59182 --- [           main] ConfigServletWebServerApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanDefinitionStoreException: Failed to process import candidates for configuration class [com.github.cloudyrock.spring.v5.config.MongockContext]; nested exception is java.io.FileNotFoundException: class path resource [com/github/cloudyrock/mongock/driver/mongodb/springdata/v3/SpringDataMongoV3Context.class] cannot be opened because it does not exist
2021-03-04 13:28:08.329  INFO 59182 --- [           main] ConditionEvaluationReportLoggingListener :

Error starting ApplicationContext. To display the conditions report re-run your application with 'debug' enabled.
2021-03-04 13:28:08.370 ERROR 59182 --- [           main] o.s.boot.SpringApplication               : Application run failed

org.springframework.beans.factory.BeanDefinitionStoreException: Failed to process import candidates for configuration class [com.github.cloudyrock.spring.v5.config.MongockContext]; nested exception is java.io.FileNotFoundException: class path resource [com/github/cloudyrock/mongock/driver/mongodb/springdata/v3/SpringDataMongoV3Context.class] cannot be opened because it does not exist
	at org.springframework.context.annotation.ConfigurationClassParser.processImports(ConfigurationClassParser.java:610)
	at org.springframework.context.annotation.ConfigurationClassParser.doProcessConfigurationClass(ConfigurationClassParser.java:311)
	at org.springframework.context.annotation.ConfigurationClassParser.processConfigurationClass(ConfigurationClassParser.java:250)
	at org.springframework.context.annotation.ConfigurationClassParser.processImports(ConfigurationClassParser.java:600)
	at org.springframework.context.annotation.ConfigurationClassParser.doProcessConfigurationClass(ConfigurationClassParser.java:311)
	at org.springframework.context.annotation.ConfigurationClassParser.processConfigurationClass(ConfigurationClassParser.java:250)
	at org.springframework.context.annotation.ConfigurationClassParser.parse(ConfigurationClassParser.java:199)
	at org.springframework.context.annotation.ConfigurationClassParser.doProcessConfigurationClass(ConfigurationClassParser.java:304)
	at org.springframework.context.annotation.ConfigurationClassParser.processConfigurationClass(ConfigurationClassParser.java:250)
	at org.springframework.context.annotation.ConfigurationClassParser.parse(ConfigurationClassParser.java:207)
	at org.springframework.context.annotation.ConfigurationClassParser.parse(ConfigurationClassParser.java:175)
	at org.springframework.context.annotation.ConfigurationClassPostProcessor.processConfigBeanDefinitions(ConfigurationClassPostProcessor.java:336)
	at org.springframework.context.annotation.ConfigurationClassPostProcessor.postProcessBeanDefinitionRegistry(ConfigurationClassPostProcessor.java:252)
	at org.springframework.context.support.PostProcessorRegistrationDelegate.invokeBeanDefinitionRegistryPostProcessors(PostProcessorRegistrationDelegate.java:285)
	at org.springframework.context.support.PostProcessorRegistrationDelegate.invokeBeanFactoryPostProcessors(PostProcessorRegistrationDelegate.java:99)
	at org.springframework.context.support.AbstractApplicationContext.invokeBeanFactoryPostProcessors(AbstractApplicationContext.java:751)
	at org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:569)
	at org.springframework.boot.web.servlet.context.ServletWebServerApplicationContext.refresh(ServletWebServerApplicationContext.java:144)
	at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:767)
	at org.springframework.boot.SpringApplication.refresh(SpringApplication.java:759)
	at org.springframework.boot.SpringApplication.refreshContext(SpringApplication.java:426)
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:326)
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1309)
	at org.springframework.boot.SpringApplication.run(SpringApplication.java:1298)
	at com.main(Main.java:10)
Caused by: java.io.FileNotFoundException: class path resource [com/github/cloudyrock/mongock/driver/mongodb/springdata/v3/SpringDataMongoV3Context.class] cannot be opened because it does not exist
	at org.springframework.core.io.ClassPathResource.getInputStream(ClassPathResource.java:180)
	at org.springframework.core.type.classreading.SimpleMetadataReader.getClassReader(SimpleMetadataReader.java:55)
	at org.springframework.core.type.classreading.SimpleMetadataReader.<init>(SimpleMetadataReader.java:49)
	at org.springframework.core.type.classreading.SimpleMetadataReaderFactory.getMetadataReader(SimpleMetadataReaderFactory.java:103)
	at org.springframework.boot.type.classreading.ConcurrentReferenceCachingMetadataReaderFactory.createMetadataReader(ConcurrentReferenceCachingMetadataReaderFactory.java:86)
	at org.springframework.boot.type.classreading.ConcurrentReferenceCachingMetadataReaderFactory.getMetadataReader(ConcurrentReferenceCachingMetadataReaderFactory.java:73)
	at org.springframework.core.type.classreading.SimpleMetadataReaderFactory.getMetadataReader(SimpleMetadataReaderFactory.java:81)
	at org.springframework.context.annotation.ConfigurationClassParser.asSourceClass(ConfigurationClassParser.java:696)
	at org.springframework.context.annotation.ConfigurationClassParser.asSourceClasses(ConfigurationClassParser.java:675)
	at org.springframework.context.annotation.ConfigurationClassParser.processImports(ConfigurationClassParser.java:582)
	... 24 common frames omitted

Following the error and seeing recent changes I moved SpringDataMongoV3Context and MongoDBConfiguration back outside the config directory. Rebuilt and application started successfully.
With regards to this issue (issue 303), mongock looks to be behaving as expected and the issue appear to be resolved on issue/303 branch.

sorry, my mistake. Can you  clone the mongock-core project,, branch issue/303, build it and pull and build the mongock project again. You should use the version 4.3.7.SNAPSHOT

@dieppa
Updated local branches to:
mongock -> branch issue/303
mongock-core -> branch issue/303
tested it and all looks good.
Thanks a lot for your support

Thanks.
The version 4.3.7 will be released soon. I keep you updated

Version 4.3.7 is released.
I close this ticket.
Thanks @stikku


Too much focus on army after drone losses prevents rebuilding a good eco

Description
After losing a lot of drones and bases, Sajuuk tends to focus army too much and not prioritizing rebuilding a strong eco.
Analysis
BuildRequests are collected from all the Managers by SajuukBot, then sorted by priority an supply requirement. However, the priority of the drones build request from the EconomyManager is always the same (BuildRequestPriority.Normal).
However, the WarManager requests can have a higher priority when the enemy has an army superiority. The problem is that with a low eco, the enemy will maintain this superiority, thus we'll never build more drones.
Ideas

We could increase the priority of the drones build request when some time has passed without fulfilling the request
We could increase the priority of the drones build request based on income vs spendings

Depends on https://github.com/Guillaume-Docquier/Sajuuk-SC2/issues/60


material-datetime-picker for targetting an input element and invoking the picker using jquery or javascript?

I want materialize datetime picker for an input form field like we have datepicker and timepicker separately available in jquery and javascript.
For example: timepicker -  $('.timepicker').pickatime()
datepicker -  $('.datepicker').pickadate()
likewise,  I need for datetime picker, is this thing available ?

The library doesn't use jQuery and you really don't need it, but you can create your own wrapper for jQuery using the example code for inputs. :)
i.e. this may work:
jQuery.fn.datetimepicker = function (opts) {
  $(this).each(
    (i, input) => {
      const picker = new MaterialDatePicker(opts);

      input.addEventListener('focus', () => picker.open());
    }
  );
};

After that, you can do this:
$('.c-datepicker-input').datetimepicker()

You'll need also to handle submit event, like updating the current value of the input.


Rename mjs to es.js

.mjs extension are not supported properly by the Webpack tools.

perhaps the right thing to do is to make the cjs files cjs? https://webpack.js.org/guides/ecma-script-modules/#flagging-modules-as-esm

I tried this first but it wasn't enough.
In the end, Next still loads the mjs file for the client part. And the tool I use to transpile nats.ws breaks when using mjs.
I also tried using the "type": "module" property on the package.json of nats.ws but it was worse.
Only renaming mjs to js did the job.
If you don't like the name is used .es.js I'm fine with naming it to .js and rename the common JS version.
However, when using the .cjs extension I had other errors saying that it was trying to import esm modules 🤔

I tried again and have it working in Next when using nats.cjs for the CommonJS version and nats.js for the ESM version. I like it better as it acts like the default/standard one is the ESM version.


Add hex_u32 parser

A parser to read hex-encoded integers.
I'm open to suggestions on how to improve the implementation

Thanks!


Adobe AIR code signing password

I have set up a VS Code Adobe AIR (desktop) project with this excellent extension, and have been following along with the awesome help in the Wiki.
I read "Package an Adobe AIR application in Visual Studio Code" and want to do the task:
Adobe AIR package release Windows application (captive runtime)
Everything seems to be configured correctly in my asconfig.json. In the appropriate place in my signingOptions, I reference the valid p12 signing certificate that we use all the time to build AIR native installers in Adobe Animate. (I didn't think that a captive runtime app even needed code signing since it doesn't make installers??)

It looks like it is going to work...
The task executes in the integrated terminal, writes my swf, and then prompts
Adobe AIR code signing password:
but doesn't actually let me type the password!
Am I missing something obvious?

I saw in another issue that you didn't want the p12 password to be stored in a plaintext file, which I understand. Even if I have to type it manually every time, I don't see a place to put the password.
 Hopefully this package task is supported and there is just something simple I am missing. Thanks for your attention!

but doesn't actually let me type the password!
Am I missing something obvious?

Just click inside the Terminal view where it asks you for the password (to be sure that it has focus) and start typing! I think that you may be confused because you will not see any new characters appear in the terminal while you type your password. That's how it works for many command line applications that accept a password.

I love you Josh! That worked!
Feeling 100% silly for typing so much!

I didn't think that a captive runtime app even needed code signing since it doesn't make installers??

Windows and macOS expect all executables to be signed. Both apps and installers.

I love you Josh! That worked!
Feeling 100% silly for typing so much!

Happy to help! 😄

@ddorris I added a little tip to this page in the wiki about the password being hidden so that no one else has the same issue in the future!


Add meeting notes from hackedu/hackedu

Can you add the date labs_followup.md was first created to its filename? Something like 15-08-16_labs_followup_email.md.

@zachlatta Just pushed.

@zachlatta Hold off on merging this


Fixed formatting issues added example image.

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           56        56
  Branches         4         4
=========================================
  Hits            56        56

Impacted Files
Coverage Δ

src/pytest_locker/fixtures.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Overloaded begin() and end() to return const_iterators, fixed issue 32

Proposed changes
The problem outlined in issue #32 required that the begin and end functions be overloaded to return a const_iterator. I added these overloads and unit tests.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] Unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Codecov Report

Merging #54 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #54   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         706    714    +8
=====================================
+ Hits          706    714    +8

Impacted Files
Coverage Δ

tests/linkedListTest.cpp
100% <ø> (ø)
:arrow_up:

src/linkedList.cpp
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


twister: Implement GNU make jobserver support

When twister is started from make, use the information in MAKEFLAGS to participate in the job server started by make.
When twister is running standalone, start a jobserver to share with child processes. If using the kitware version of ninja, then this will prevent ninja+twister from starting more processes than XX when twister is run with -jXX.
There is no implementation for windows yet.
Signed-off-by: Jeremy Bettis<EMAIL_ADDRESS>

I tested with this:
twister -v -i --clobber -j1 &> /dev/null &
while jobs %% ; do uptime; jobs -x pstree -T -c %% | cat -n ; sleep 1 ; done

And verified that twister+cmake+ninja only ran 10 processes. There were 11 for most of the time, but I think that the extra one is twister's main thread that is just waiting at line 1029, so that still seems fine.
I'm using cmake 3.20.5 and ninja 1.10.2.git.kitware.jobserver-1
If there are additional scenarios you would like to see please reply to this comment.

Also, and this isn't really anything to do with this patch but it's part of the same problem: having a jobserver for build jobs is great, but it doesn't capture everything.  In fact twisters own threads (there are a ton) consume significant CPU while parsing the emulator output, and the SMP emulators are "one job" as far as make/ninja is concerned, but actually spawn one thread per CPU internally.  There's no attempt made to account for this in twister, so the load routinely runs just a tiny bit higher than we want it (e.g. ~136 on my device with 128 host CPUs).  This doesn't sound like a big problem, but the moment you go beyond the hardware limits threads start getting scheduled out by the host kernel, which means we start seeing spurious SMP failures due to races between the two CPU threads.
It would be nice to have some kind of accounting for this; we could maybe move from threads to asyncio (which is a bunch of boilerplate to add "async/await" keywords and to swap the read() calls for their async equivalents, but in practice doesn't change the structure of the code), which would put all of twister in "1 job" which we could account for.  And then we could just have some logic that knows "make run on qemu SMP needs extra jobs" and do the accounting internally).

Also: ninja is very small; it would strike me as very feasible to add this version to the SDK. @stephanosio?

pip install ninja will give you  1.10.2.git.kitware.jobserver-1
Following the Zephyr getting started guide will give you the stock ninja.  Perhaps we could just update the instructions to use pip install or to download directly from kitware's releases at https://github.com/Kitware/ninja/releases instead?

The python code looks ok to me, though I'm not a python person by any means either... its a bummer that stock ninja doesn't respect the job server.

I fixed a bug when running twister from a Makefile without the + sign, now it checks if the passed file descriptors are readable and writable as expected, rather than just checking if the file descriptors exist.
Then I rebased my branch to head to deal with the merge conflict, and somehow the review bot added 90 reviewers, sorry.

Gave this a try on twister with system ninja (arch linux, which has support for -j argument) and ran some tests limiting it to 3 jobs, twister confirmed it was running 3 jobs, however system load quickly went to 20 (this is an 8 core system with hyper threading, would expect to see a system load of 6 or 12 if the job limiting was working correctly) so it does not look like this works for me.

Gave this a try on twister with system ninja (arch linux, which has support for -j argument) and ran some tests limiting it to 3 jobs, twister confirmed it was running 3 jobs, however system load quickly went to 20 (this is an 8 core system with hyper threading, would expect to see a system load of 6 or 12 if the job limiting was working correctly) so it does not look like this works for me.

It sounds like it was working?  make with -j limits the concurrency to -j not the system load.

It sounds like it was working? make with -j limits the concurrency to -j not the system load.

Concurrency of 3 should spawn 3 threads, each thread 1x CPU core, so system load should not be 20, which indicates equivalent of 20 processes want 100% of a single CPU core.

It sounds like it was working? make with -j limits the concurrency to -j not the system load.

Concurrency of 3 should spawn 3 threads, each thread 1x CPU core, so system load should not be 20, which indicates equivalent of 20 processes want 100% of a single CPU core.

Did you use the patched ninja that support the job server protocol? Kitware has a fork for this as noted in the pr. Arch Linux ninja package isn’t the forked version (I use arch as well and had to install the fork)

after many measurements done on a large system (>100 cores), here is a brief summary:

Using make instead of ninja is the way to go if you want to keep the load under control (make was the default for years)
But, make is at least 50s slower in the subset I tested (qemu_x86)
Using the patched version of ninja helps, without this PR
this PR with make has no effect almost, but it works well with default ninja

The last few bumps in the graph below are with make, everythng else is with ninja, with and without this PR and with and without the patched ninja

The first spike is what I tried to first with a large job and with all default, cold ccache

This PR also seems to have broken macOS support:
https://github.com/zephyrproject-rtos/sdk-ng/actions/runs/4016970387/jobs/6915967155#step:14:130


Chose ID Screen needs some space at bottom for add button

you can't scroll further than this and the add button hides information on the last inactive identity

This design in outdated


Remove backports dependency

Since the gem requires Ruby 2.3+ there's no need for the backports gem.
It's also quite large and takes up 1191 files on disk, which is a
wonderfuly nasty performance hit on windows where requiring files is
slow.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

I'll close this one and modify the PR I've made for you to be primary contributor. Thanks for your contribution.


Add host tag to changesets

On OSMCha we flag iD instances that are not known by the OSM community. It would be good if RapiD add the host tag to changesets in order to have it as a verified instance. We would need only to uncomment that line to make it work: https://github.com/facebookincubator/RapiD/blob/master/modules/ui/commit.js#L72

Closed with #6


Is there a product count limitation or request interval to get all product list ?

I cannot get product list over 1,650 even though change "pageSize" parameter in "next_link" URL.
This url is the error link to get next page.
"https://manage.devcenter.microsoft.com/v1.0/my/hardware/products?pageSize=50&continuationToken=PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTE2Ij8%2BPENvbnRpbnVhdGlvblRva2VuPjxWZXJzaW9uPjIuMDwvVmVyc2lvbj48VHlwZT5UYWJsZTwvVHlwZT48TmV4dFBhcnRpdGlvbktleT4xITQ4IWNIVmliR2x6YUdWeWN5MHdNREF3TURBd01EQXdNREF3TURBd01ESTVPVGc1T0RZdzwvTmV4dFBhcnRpdGlvbktleT48TmV4dFJvd0tleT4xITk2IWRYTmxjaTFrWld4bGRHVmtMVEF0SUNBZ0lDQWdTR0Z5WkhkaGNtVkVjbWwyWlhJdGNISnZaSFZqZEhNdE1EQXdNREF3TURBeE16WTVPVE00TkRneU9ESXpOVEE0TWctLTwvTmV4dFJvd0tleT48VGFyZ2V0TG9jYXRpb24%2BU2Vjb25kYXJ5PC9UYXJnZXRMb2NhdGlvbj48L0NvbnRpbnVhdGlvblRva2VuPg%3D%3D"

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

ID: 40f09988-3ba3-0ce2-9952-b11ddd66aff1
Version Independent ID: dd0d280c-f6a0-4a07-7f86-42eafaf6087b
Content: Get all products - Windows drivers
Content Source: windows-driver-docs-pr/dashboard/get-all-products.md
Product: windows-hardware
GitHub Login: @balapv
Microsoft Alias: balapv

@balapv, can you comment on this?

Hi @roys93
I asked one of the engineers on the dashboard team to look into this, and he said:
"I tried to reproduce the below issue with an account which has 2301 products in the PROD environment.

With a page size of 50 results, I was able to fetch all the pages (48 pages of 50 entries).
I preserved the links for all 48 pages and used them in a random order (48th page, 25th page, 37th page, 2nd page). This also worked.

NOTE: as per this StackOverflow entry there is no expiration for the Azure table storage token, and if the table storage data is updated then he may get a different set of results.
Can you please provide us the complete use case  that you have tried and your publisher id to cross check with partner specific data?"
You can reach out directly to me at eliotgra at microsoft dot com.
Thanks,
@EliotSeattle

I have the same question.
https://github.com/Microsoft/SDCM/issues/9
My company are over 10000 products.
I use Get all products API, only can get 8910 products.
At last times, the service shows the error : "500 - Internal Server Error".


Add Warrants to Stakeholder Snapshot sheet

⚠ Note any Warrants implementation in OCX is dependent on the resolution of https://github.com/Open-Cap-Table-Coalition/Open-Cap-Format-OCF/pull/410

:warning: Answers to #27 and #28 must be known before this can be correctly implemented.

Scope
From https://github.com/captable/ocx/files/11265982/OCX.v.0.4.xlsx

Desired outcome

There should be one column for each group of warrants, grouped by the common stock class they convert to per the same algorithm as #22.
The left-to-right column ordering places Warrant groups to the right of Stock Options. If there are multiple Warrant groups, they should be ordered from oldest common stock class board_approval_date to most recent, with nulls last, and name as tie-breaker.
When a Warrant converts to preferred stock, we should seek out the "winning" common stock by following the same algorithm for handling multiple stock class conversion rights for preferred stock classes. Therefore, the function used in the Warrant cells needs to show the converted value (actual holdings * ratio). Nothing should change in the header.

To-Do

[ ] Determine necessary in-tool calculations and document for review and approval
[ ] ocf-package: Are the necessary OCF Files being processed?
[ ] model: Are the necessary OCF Objects being consumed?
[ ] Develop workbook against Model interfaces.
[ ] Implement actual Model
[ ] Integrate and test end-to-end

https://github.com/Open-Cap-Table-Coalition/Open-Cap-Format-OCF/pull/426 will probably replace https://github.com/Open-Cap-Table-Coalition/Open-Cap-Format-OCF/pull/410 for our purposes on this issue.


PDO (mysql) return raw BIT value instead of strings '0' and '1'

Hello,
I have strange behavior (with cloudlinux php): for a BIT mysql field, PDO return '\0' for 0 and '' for 1. With native PHP (there is php-selector) I got normal expected result. I can't find info about that.
I mean when Yii try in (bool) $value for phpType boolean, it give a wrong result:
(bool) "\0" // true
(bool) "" // false, but actual value in the model field is `true`.

I think in that case PDO returned a Raw BIT value:
$pdo = \Yii::$app->db->pdo;

$rows = $pdo->prepare( 'SELECT * FROM `ss_profiles` LIMIT 1' );
$rows->execute();

while ($a = $rows->fetch( \PDO::FETCH_ASSOC )) {
      return ord($a[ 'deleted' ]);
}
// 1 for BIT 1
// 0 for BIT 0
// var_dump( (bool) chr(0) ); // true
// var_dump( (bool) chr(1) ); // true

I have a similar problem


Support for private repos

Private repositories may need private SSH key to access them. For example:

A private GitHub repository foo-repo. A GitHub user foo-user is created and a private SSH key is generated for him. The foo-user then needs to be invited to the private repository, in order to gain read-only access. Jenkins will have knowledge of the private SSH key. That way, Jenkins can impersonate the foo-user and can access foo-repo

Jenkins stores credentials at http://localhost:8080/manage/credentials/ , and we only need the credential ID (it looks like UUID) to configure the project.xml as follows:
      <hudson.plugins.git.UserRemoteConfig>
        <url>git@github.com:foo-user/FooRepo</url>
        <credentialsId>af1bae5a-3a7b-48a3-ae88-a5eb42309b56</credentialsId>
      </hudson.plugins.git.UserRemoteConfig>

Therefore, the Java client simply needs to define an additional field GitRepo.credentialsId: String? which defaults to null. If specified, the project config.xml file is generated accordingly.
The list of credentials is highly specific to the shepherd deployment (each deployment will have different set of credentials). It's not a problem to document this and have shepherd-cli use the credentialsId directly since it's used locally. However, for an UI this may create a huge problem: if the users are allowed to register their own projects, we need to prevent the misuse of other users' credentials.
A solution would be to have a per-user credential store: the users could upload their own credentials and only use those.
That can be achieved in multiple ways: have a Jenkins credential domain per user (identified by an e-mail), or have a set of json files at /etc/shepherd/credentials/email.json which would list the credential IDs; the user could create/modify/edit/delete their own credentials only.

We can treat the Jenkins credentials ID as the unguessable secret and therefore we can store it in GitRepo.credentialsId: String?. That will take care of a proper generation of Jenkins job config.xml.
Additionally, the java library will introduce a set of APIs that allow the users to create their own credentials, list them, modify them and delete them.
The API should accept all kinds of credentials that Jenkins accept: https://www.jenkins.io/doc/book/using/using-credentials/ . For now we can start with the "SSH Username with private key" only.


Include missing example script to PyPI tarball

The script is needed for testing purposes on the packaging side.

Would you consider releasing 0.4.3 with the above change? Without this script, the tests cannot be run during packaging which is quite a regression on our end.
Thanks,
Ghis

Ok, working on it :-)


api: implement smart cache for new endpoints

[ ] subscriptions:

[ ] should be set when the user changes their subscriptions.
[ ] should be set for new users.

[ ] revisionIds:

[ ] should be set when a revision is created or deleted.
[ ] should be set when a new repository is created.

[ ] restore from bin
[ ] /user/profile/:username: should be set for new users.
[ ] events: similarly, depending on actual implementation.
[ ] /api/threads/:id and /api/uuid/:commentid (wenn commentare hinzugefügt / aktualisiert werden)

https://github.com/serlo/frontend/issues/645 related.


Ready for production use?

Firstly thank you for very useful package. Will this project will be continued? can I use it for production?

Yes, absolutely. If you have any problems, feel free to open an issue. I recently switched jobs so I've been late to address stuff on GitHub but I fully intend to maintain and support this project.


Feature/psr2 refactoring

Actually this shouldn't get merged into the main master.
If you want you could merge this else just decline it :)

Please move php unit to require-dev as well. Also if you have written any tests feel free to add those as well.
As for the quality tools,  pheal is using scrutinizer-ci (there should be a link here for your pull request), I don't object to having those tools in require-dev,  but you might just want to connect your fork to scrutinizer-ci, it's a great tool.

I will open a pull request later on with some other changes as well.
Scrutinizer is actually a paid tool isn't it ?

Free for opensource projects

When you make your pull request later, please make it against the development branch


Command redirect did not work

There's no error, but the below format does not do anything. I also found that reversing the placement of "say" and "s" also did nothing, so it doesn't seem to just be me putting stuff in the wrong place. Was the format updated since the latest wiki page was made?
{
    "commandMode": "COMMAND_REDIRECT",
    "redirectCommand": {
        "command": "say",
        "redirectTo": "s"
    }
}

I assume you are using the version 0.6.0, it's a bug with my implementation to work with LuckPerms. You can downgrade to 0.5.0 or install LuckPerms and grant yourself permission of commandaliases.s.
The command should look like this
{
    "commandMode": "COMMAND_REDIRECT",
    "redirectCommand": {
        "command": "s",
        "redirectTo": "say"
    }
}


[Network] Fix output format for list-usages.

Fix #2684.
This is the same fix I applied for VM list-usage.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[N/A] Each command and parameter has a meaningful description.
[N/A] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #3455 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #3455      +/-   ##
=========================================
- Coverage    70.9%   70.9%   -0.01%
=========================================
  Files         394     394
  Lines       25550   25562      +12
  Branches     3892    3894       +2
=========================================
+ Hits        18117   18125       +8
- Misses       6297    6304       +7
+ Partials     1136    1133       -3

Impacted Files
Coverage Δ

...twork/azure/cli/command_modules/network/_format.py
34.3% <ø> (+3.1%)
:arrow_up:

...work/azure/cli/command_modules/network/commands.py
95.51% <100%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
34.69% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
69.62% <0%> (ø)
:arrow_up:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
: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 2e4010d...974d24c. Read the comment docs.


Move TimeSavers context menu item off the top

Please move TimeSavers somewhere lower in the context menu so I can access QuickActions as the top item on the editor context menu.
I suggest moving it to the bottom. Bottom is the next easiest place to access.

We've already had a lengthy discussion about this offline. In addition, just as with the discussion about the Time Savers menu, other extensions can, and do, take the top spot.
I can't agree with you about the bottom being the easiest to access. If it was then the Quick Actions and Refactoring item would be on the bottom :-P
Besides, as the context menu gets longer and longer, it gets increasingly difficult to find anything. That's exactly why the menu was added, instead of adding all of the commands directly in the context menu, even though it added an extra keystroke which I would have liked to avoid.
I thought that adding a menu was the responsible thing to do and was the best compromise.

AS we discussed offline, there's no justification for this move.