Change PackSummary to use enums instead of strings

The old system just took in strings and would just crash if the given tag string didn't exist, making this system a little stricter via the use of enums should be good at guiding people to tag correctly and gives an easy overview of the current tags.

Makes sense. Now that the pack summary is defined in code instead of the text file, this is a better way to do things.
Would be good to let ocean know since this will require adjustment to the Packmaster integration code.


Add workflow for controlled snapshot deploys

Current dependencies on/for this PR:

main

PR #20

PR #21

PR #22

PR #23   👈

This comment was auto-generated by Graphite.


[Security/Extension] Role encryption/decryption

Description
This idea is coming from @cwperks 's comment
Including encrypted roles in JIT token as payload for privilege validation.
Issues Resolved

Relate: #2573
Relate: https://github.com/opensearch-project/security/issues/2619

Is this a backport? If so, please add backport PR # and/or commits #
Check List

[x] New functionality includes testing
[ ] ~New functionality has been documented~
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2620 (ae89338) into feature/extensions (1681823) will increase coverage by 0.02%.
The diff coverage is 76.19%.

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

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                   Coverage Diff                    @@
##             feature/extensions    #2620      +/-   ##
========================================================
+ Coverage                 61.25%   61.28%   +0.02%
- Complexity                 3337     3340       +3
========================================================
  Files                       261      262       +1
  Lines                     18577    18598      +21
  Branches                   3275     3275
========================================================
+ Hits                      11380    11397      +17
- Misses                     5598     5603       +5
+ Partials                   1599     1598       -1

Impacted Files
Coverage Δ

...curity/authtoken/jwt/EncryptionDecryptionUtil.java
70.58% <70.58%> (ø)

...g/opensearch/security/authtoken/jwt/JwtVendor.java
65.27% <100.00%> (+2.04%)
:arrow_up:

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@opensearch-project/security Hi folks, I may need your opinions on this feature branch testings. For merging this PR, should I cherry pick all the changes from main to fix these issues such as, CVE from whitesource security check? Just let me know, if that is the case, I do have a cherry pick PR ready in my fork. :)

@RyanL1997 For the feature branch you will need to sync it periodically to keep it from drifting from main over time. Similar to what was done on feature/identity in core. See those PRs here: https://github.com/opensearch-project/OpenSearch/pulls?q=is%3Apr+is%3Aclosed+merge+remote+into+feature%2Fidentity+

@RyanL1997 For the feature branch you will need to sync it periodically to keep it from drifting from main over time. Similar to what was done on feature/identity in core. See those PRs here: https://github.com/opensearch-project/OpenSearch/pulls?q=is%3Apr+is%3Aclosed+merge+remote+into+feature%2Fidentity+

Hi @cwperks, thanks for the info. I will cherry pick the changes right away.


draft blog post on Chile law

https://docs.google.com/document/d/1MB1POFj9M_ULu8_hw68QGUYORiUxJcyYuCTTQIuv1qM/edit
https://www.eff.org/deeplinks/2016/05/chilean-bid-help-authors-will-actually-kill-audiovisual-content-online

https://docs.google.com/document/d/1esFp-WfC0A2Tyj1Nc1nVRYZP9Dr_d8IGBjvaMJPW_-Q/edit?usp=sharing

https://blog.creativecommons.org/?p=48446&preview=true

on hold until next action for Senate plenary vote

done https://blog.creativecommons.org/2016/06/16/new-chilean-law-make-harder-authors-freely-share-audiovisual-works/


Remove version: Jackett.Jackett version 0.20.4032

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.4032/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


Launch RMI and RMID only for tests that need them + increase timeout

Resolves backlog/issues/494
Related to backlog/issues/487
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Test : Grinder_JCK/32


Custom animations in tooltips.

Problem
Tooltips only support fade animation. here
Animations should be capable of being outsourced to css.
Benefit: You can use libraries like animate.css in tooltips.
Solution
Integration of custom css animation classes.
To enable tooltip animation you pass { animation: true },
after this you can pass { animation: 'animated bouceInDown'}
and tooltips will automatically use this string as animation class.
Take animate.css for example, it's an awesome collection of CSS3 based animations.
Initialization is kept simplistic.
Dynamic Initialization
$(".music-menu").tooltip({
  animation: 'animated bounceInDown'
});

Auto Initialization

Hi guys, I am actually looking for this. Is this implemented in BS, could anyone give me a link where I can find info over the implementation. Thanks a bunch !

@darma-avalos
This isn't implemented yet.
However, an experimental version for BSv4 is hosted here

Update for v4
Current Tooltip Markup

  ....

animation classes and tether.js positioning, both are done on .tooltip root element.
css animations rely heavily on transforms.
tether.js uses css transforms for positioning, and is overriding transforms of css animations.

I did it for v4_alpha.
I'll update it soon to v4_beta. Didn't realise it was released. 😄


Implement NCT/USD-6 price feed

As requested by Toucan, it should be based on these pools:

mcUSD / NCT https://explorer.celo.org/mainnet/address/0xEc5c9c99aD30aDF396aC4B48FBA09dC34819C65a/transactions
CUSD / NCT https://explorer.celo.org/mainnet/address/0x7dA99753FF017F1B7AfB2c8C0542718dc9f15F21/transactions
CELO / NCT https://explorer.celo.org/mainnet/address/0xdb24905B1b080F65deDb0Ad978aad5C76363D3c6/transactions

I'd rather suggest to solve NCT/USD by the simplest path possible, NCT/CELO/USD. Besides CELO/USD is already deployed and been working for more than 1 year in both Celo testnet and mainnet.

Related PRs:

https://github.com/witnet/witnet-price-feeds/pull/140
https://github.com/witnet/data-feeds-explorer/pull/261

Added change request on https://docs.witnet.io/

I think we can deem this as complete now! :muscle:
https://feeds.witnet.io/celo/celo-alfajores_nct-usd_6


With DB down, infinite retries

When the database is down, an attempt to connect seems to result in an infinite retry loop without ability to cancel.
Example:
var rdbdash = require('rethinkdbdash');
var r = rdbdash({
    discovery: true,
    pool: true,
    buffer: 1,
    max: 10,
    servers: [
        { host: 'localhost', port: 28015, db: 'test' }
    ]
});

It looks like Pool.createConnection calls itself: https://github.com/neumino/rethinkdbdash/blob/master/lib/pool.js#L292
It should stop if the pool is set to drain, but calling .getPoolMaster().drain(); does not work, throwing:
Unhandled rejection TypeError: Cannot call method 'close' of null
    at PoolMaster.drain (/var/local/purge-log-db/node_modules/rethinkdbdash/lib/pool_master.js:498:16)
    at /var/local/purge-log-db/src/index.js:48:23
    at tryCatcher (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/util.js:26:23)
    at CatchFilter.doFilter (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/catch_filter.js:52:40)
    at CatchFilter.tryCatcher (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/promise.js:507:31)
    at Promise._settlePromiseAt (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/promise.js:581:18)
    at Promise._settlePromises (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/promise.js:697:14)
    at Async._drainQueue (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/async.js:133:10)
    at Async.drainQueues (/var/local/purge-log-db/node_modules/rethinkdbdash/node_modules/bluebird/js/main/async.js:15:14)
    at process._tickCallback (node.js:415:13)

Link to near error: https://github.com/neumino/rethinkdbdash/blob/master/lib/pool_master.js#L494
Any known workaround? Thanks!

Infinite retries are expected, the crash was fixed in 2.1.15


App crash after using camera picker

Make a new picture.
App crashes after confirmation.

Correct image is sent to JS code even after app crash. This is not happening when using a picture from gallery. In the onActivityResult the data parameter is null.
I'm getting this on crash:
java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=1, result=-1, data=null} to activity {com.app/com.app.MainActivity}: java.lang.NullPointerException: Attempt to invoke virtual method 'android.os.Parcelable android.content.Intent.getParcelableExtra(java.lang.String)' on a null object reference
RN 0.32.0-rc, Android 6.0.1.
Picker options:
const options = {
  title: 'Select Image',
  storageOptions: {
    skipBackup: true,
    path: 'images',
  },
};

Hi,
Seems like a concurrent request crash the app before the actual picker request.
result=-1 is an Activity RESULT_OK looks strange to me.
I will add a check on the data=null then.
Thanks for the debug!
Sorry for the late response.

Can you confirm that this is still happening on v0.22.2?

Will go back to this issue if I find some more time, sorry!

Hi, the problem is that the module uses
static final int REQUEST_LAUNCH_IMAGE_CAPTURE = 1;   static final int REQUEST_LAUNCH_IMAGE_LIBRARY = 2;   static final int REQUEST_LAUNCH_VIDEO_LIBRARY = 3;   static final int REQUEST_LAUNCH_VIDEO_CAPTURE = 4;
in ImagePickerModule.
You should use more random ints so other activityListeners will not crash the app. If another module uses the same requestcodes, the app will most likely crash.

We probably just need to add mCallback = null;
after line 446

This should be fixed now. I made the activity request codes more unique.


Implement client-side load reporting for grpclb.

This does not yet include any tests.  I'm hoping that we can get your #10360 merged so that I can use that framework to add the tests.
There are a few parts to this change:

The stats are tracked in a grpc_grpclb_client_stats object, which uses an atomic for each counter.  The grpclb policy creates a new instance of this object whenever it establishes a new call to the balancer.  This ensures that we throw away old data whenever the stream breaks, as per the client load reporting spec.

The LB policy API has been extended to allow the policy to modify the call context, which is then passed down into the subchannel call stack.  The grpclb policy uses this mechanism to make the grpc_grpclb_client_stats object available to the client load reporting filter.  Note that if the grpclb policy loses contact with the balancer after the call is created, the call will continue to use the old grpc_grpclb_client_stats object, and the values will be ignored.

The grpclb code now looks for an initial LB response message that indicates the interval for client load reports.  When we see that, we set a timer for the requested interval and send a load report message at that time, then rescheduling the timer for the same interval.

Because we haven't yet implemented drop support, we never report any drops.  When we do add drop support, the grpclb code will need to directly increment both the started and the finished counters in addition to the relevant drop counter, because we will never wind up creating a subchannel call (and hence no client load reporting filter) in that case.

Please let me know if you have any questions about any of this.  Thanks!

No significant performance differences

This now includes a test, so I think it's ready for review.

reviewing...

No significant performance differences

No significant performance differences

No significant performance differences

Known errors: #10839 #10962 #9542


Cyanogen 13 Android 6.0.1 kernel 3.4.110

Can you please help me with the patch? I really need to make it work badly.
Thanks in advance Pelya.

We can't help you if you don't give us enough information.
What is the error message exactly?

Find out the device codename of your LG v10. Example: For "Motorola Moto G4 Play", it is "harpia"!
Find out the SoC codename of your LG v10. Example: For "Motorola Moto G4 Play", it is "MSM8916" by Motorola, so "motorola_msm8916"!

Search in your android settings > about phone , lineageos wiki, lineageos download section, wikipedia.

A detailed step-by-step guide on how to compile lineageos (successor for cyanogenmod) can be found at
https://wiki.lineageos.org/devices/i9300/build . Replace "i9300" in this link with the codename of your device!
You can find out the code name of your device by visiting https://wiki.lineageos.org/devices .
Closing this issue because there has been no answer since 8 months.


Windows clipboard inbound and outbound control without sesman？

I have a requirement that control clipboard inbound and outbound without sesman. In the new version， in sesman can control one-way clipboard data by a setting in the sesman.ini. But now I connect to windows server without sesman. How can I modify the code to realize the same effect.

Yeah, I'm using NeutrinoRDP to connect to a Windows Sever. Can I control local client side  upload or download data through the clipboard alone？ In other words，I want to control both clipboard from server push to client and clipboard from client push to the server.

You can't do this from xrdp, as far as I'm aware.
The neutrinordp module is just a proxy. The clipboard data is carried on the cliprdr RDP virtual channel channel. The PDUs from this channel are just passed through without any inspection.
You can probably block the channel completely, but finer-grained configuration will need to be done at the client or the server.


Verdaccio is not storing the packages in local repo

Here are some of my logs. I think vardaccio is not storing packages but always download it and transfer it. It always shows bytes: 0/3789. I have installed same package on many systems.
http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /on-finished/-/on-finished-2.3.0.tgz', bytes: 0/4552
 http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /fresh/-/fresh-0.3.0.tgz', bytes: 0/2472
 http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /parseurl/-/parseurl-1.3.2.tgz', bytes: 0/3789
 http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /methods/-/methods-1.1.2.tgz', bytes: 0/2475
 http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /proxy-addr/-/proxy-addr-1.0.10.tgz', bytes: 0/4935
 http <-- 200, user: null(<IP_ADDRESS>), req: 'GET /range-parser/-/range-parser-1.0.3.tgz', bytes: 0/2462

Please provide more info about your evrionment, config file and all is helpful for us to help you.

I have installed it on Ubuntu 16.04 Lts using npm. Default configuraion file. I have only add two lines at the end of the configuration file to listen it on <IP_ADDRESS>:4873.

This is the default config file. It allows all users to do anything,
so don't use it on production systems.

Look here for more config file examples:
https://github.com/verdaccio/verdaccio/tree/master/conf

path to a directory with all packages
storage: /home/cache/.local/share/verdaccio/storage
path to a directory with plugins to include
plugins: ./plugins
web:
WebUI is enabled as default, if you want disable it, just uncomment this line
#enable: false
title: Verdaccio
auth:
htpasswd:
file: ./htpasswd
# Maximum amount of users allowed to register, defaults to "+inf".
# You can set this to -1 to disable registration.
#max_users: 1000
a list of other known repositories we can talk to
uplinks:
npmjs:
url: https://registry.npmjs.org/
packages:
'@/':
# scoped packages
access: $all
publish: $authenticated
proxy: npmjs
'**':
# allow all users (including non-authenticated users) to read and
# publish all packages
#
# you can specify usernames/groupnames (depending on your auth plugin)
# and three keywords: "$all", "$anonymous", "$authenticated"
access: $all
# allow all known users to publish packages
# (anyone can register by default, remember?)
publish: $authenticated

# if package is not available locally, proxy requests to 'npmjs' registry
proxy: npmjs

To use npm audit uncomment the following section
middlewares:
audit:
enabled: true
log settings
logs:

{type: stdout, format: pretty, level: http}
#- {type: file, path: verdaccio.log, level: info}
listen:
<IP_ADDRESS>:4873

I don't see anything wrong with the info you have provided, I'm closing this due I think you have some sort of permissions issue. I'll reopen if you provide more info that proves the issue is in our side. Thanks for reporting,


ngrok: 3.1.0 -> 3.1.1

Description of changes
Version bump.
I ran updateScript and tested the new binaries locally.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

ngrok


Fix the next button not showing correctly in wizard form

[ ] Changes have been thoroughly tested to not break existing functionality.
[ ] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

What is this fixing? You haven't provided any description or screenshots.

The PR title is very clear, and the code change is not big, I just added a missing color for a button.
Before:

After:

"not showing correctly" is not clear in my opinion. The PR description even asks you for screenshots.

As for the change itself, I don't think it's correct - the button should be primary.
I have tested 3.x myself and it works fine without this change:

Please create a reproduction repository to I can investigate why it doesn't currently work for you. Otherwise, I will need to close this PR.

I'm not using it in the admin panel, I use it in a Livewire component

Something in your Tailwind setup is probably incorrect then

hmm, it's a clean project i just installed breeze and Filamment, I will check it out.

Make sure you are using the Filament Tailwind preset, and make sure you are using @filamentStyles in the layout.

Make sure you are using the Filament Tailwind preset, and make sure you are using @filamentStyles in the layout.

I'm very sure about these, but i need to investigate more.

If you upload the repository I can take a look

As i said it's very new project, I didn't upload it yet. If I couldn't find what is wrong i will create a new project installing Filament first then Breeze.

I couldn't find the issue, so I uploaded it, you may check the repo.
https://github.com/Xoshbin/ftest

You are missing the Tailwind Config preset https://filamentphp.com/docs/3.x/forms/installation#installing-tailwind-css


Outdoor 1080 YHS.3020

Hi All,
I am tying to flush my cam Outdoor 1080 YHS.3020 Firmware <IP_ADDRESS>_202104231123
I tried 2 different SD card and format it with camera and with PC.
I also tried the latest and the 0.1.8 release h30ga.
The directory factory does not renamed and the yellow light keep on (no blinking and no blue).
What am i missing?
Thanks

What operating system are you using to create the sd card.
Please note that factory is upper case: Factory.
If you try to make a backup, does the procedure work?

@roleoroleo
Good catch, I use Win10
Thanks a lot, I re-format it using my F35 and its worked like a charm...
Thanks!!!


doc: enhance search bar experience by using pymeilisearch

This pull-request enables PyMeilisearch. This tool enhances the user experience when using the search bar.
In https://github.com/ansys/pyfluent/pull/2024/commits/fdf78e4590f9fedaddd32e3bfe165b45243f5d5c the option is enabled in the HTML theme configuration.
Later, in https://github.com/ansys/pyfluent/pull/2024/commits/67de8b71425d1174ace7624d5cbf6be56a2c9e80 and https://github.com/ansys/pyfluent/pull/2024/commits/f9e3ac6a06e185f55e7f661c73ebdc54a9ba9126, the documentation gets indexed.

Thanks for reviewing and approving, @hpohekar. Merging this 🚀


Interview failing at ProtocolInfo for nodes that already have been successfully included

Hi everyone,
I am struggling with an issue I can just not solve. I used to have a Fibaro double switch 2 in my garden that was routed via a roller shutter node. It used to work flawlessly but then stopped due to a hardware defect. I wanted to replace it with another one but did not manage to include it. No matter what I tried. Even tested 2 other switches. With no luck. I then tried to include the switches right next (in the same room) to the controller (Z-Wave.Me USB Stick). Worked like a charm. I included all 3 to have them ready whenever I need them. The next day I installed the switch in the garden again but it appears dead on ZWave2MQTT. Tried another of the ones I included before. Same result…
I have no idea how to make the switches connect to the closest node. Before the first switch stopped working, it has been 100% reliable and nothing else changed.
Also tried healing the network, re-interviewing the nodes…
Two switches are now powered and are just waiting for the controller to say “Hi”… but for some reason they get stuck at the "ProtocolInfo" when trying to re-interview. I attached the log in silly mode.
Would really appreciate any help!
z-ui_2023-05-07.log
It is noded 47 and 48.

Node 47 and 48 after powering them on. Node 49 is a node that is not powered. I just triggered the re-interview to compare between "real" nodes and offline ones. This is so confusing. :-(

Hi @MaKin211 ! Please provide driver log, loglevel debug and attach it here as a file (drag & drop into the text field).
The one you provided is application log file.
Also if possible try using an extension USB cable for your stick, that could fix the issue as it improves connectivity a lot


Action Button > open Safari App crash

Hello,
i went in the following issue.
I want to open the safari with an specified url, after i pressed an action button.
This works in Android perfectly.
In iOS only when the app is in background. When the app is not in background, the app starts and after a while safari starts with the correct url.
When i go back to my app, the app is freezed.
push = PushNotification.init({
        "android": {
            "senderID":     "********",
            "sound":        true,
            "vibrate":      true
        },
        "ios": {
            "alert":        true,
            "badge":        true,
            "clearBadge":   true,
            "sound":        true,
            "categories": {
                "delete": {
                    "yes": {
                        "callback": "DeleteMessage", "title": langtxt_Delete, "foreground": false, "destructive": true
                    }
                },
                "deletelink": {
                    "yes": {
                        "callback": "openLink", "title": langtxt_openLink, "foreground": true, "destructive": false
                    },
                    "no": {
                        "callback": "DeleteMessage", "title": langtxt_Delete, "foreground": false, "destructive": true
                    }
                },
                "deleteimage": {
                    "yes": {
                        "callback": "openImage", "title": langtxt_openImage, "foreground": true, "destructive": false
                    },
                    "no": {
                        "callback": "DeleteMessage", "title": langtxt_Delete, "foreground": false, "destructive": true
                    }
                },
                "deletelinkimage": {
                    "yes": {
                        "callback": "openLink", "title": langtxt_openLink, "foreground": true, "destructive": false
                    },
                    "no": {
                        "callback": "openImage", "title": langtxt_openImage, "foreground": true, "destructive": false
                    },
                    "maybe": {
                        "callback": "DeleteMessage", "title": langtxt_Delete, "foreground": false, "destructive": true
                    }
                }
            }
        },
        "windows": {}
    });

function openLink(data){
    var IABout = cordova.InAppBrowser.open(encodeURI(data.additionalData.url), '_system');
}

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
iOS 10.0.1
Cordova CLI version and cordova platform version
cordova --version 6.1.0
cordova platform version iOS 4.1.0

Plugin version
cordova plugin version 1.6.1

Who can help me?
Regards
Kevin

@appzer Can you try upgrading to 1.8.2? I want to see if the bug is reproducible with the latest plugin which includes a fix for iOS 10.

I will test shortly and report back
Kevin

@macdonst updating the plugin will fix the issue. Thanks kevin


Prevent double posting digital-register-frontend

https://www.pivotaltracker.com/story/show/94306582

Looks good. Left a suggestion to always redirect on post.


Manage database role memberships

name: Feature request/idea :bomb:
about: Suggest a new feature or enhancement

Summary of new feature

One should be able to manage database role mamberships thru dbatools

Proposed technical details (if applicable)
*SSMS screenshot:

Latest version of dbatools as of writing

0.9.751.

Please be specific on what you want to manage. We already have some commands around database and server-level roles in the module. Your screenshot does not appear to have uploaded properly as well.

I want to add a windows user as 'db_datareader' to a specific database only.
Frank confirmed dbatools didn't have this feature yet.
https://twitter.com/osiris687/status/1091536514096480256

The below adds a new user to a database and adds a specific role.
$Database = "<DbName>"
$Username = "<GroupName>"
$db=(New-Object ('Microsoft.SqlServer.Management.Smo.Server') "localhost").Databases[$Database]
$usr = New-Object ('Microsoft.SqlServer.Management.Smo.User') ($db,$Username)
$usr.Create()
$db.Roles['db_datawriter'].AddMember($Username)

Hi @Ayanmullick ,
Currently, you can achieve this by running two different commands.
New-DbaDbUser -SqlInstance localhost -Database DB1 -Login user1
Add-DbaDbRoleMember -SqlInstance localhost -Database DB1 -Role "db_datawriter" -User user1

Can you confirm this is what you are looking for?

Thanks for the update, @ClaudioESSilva. The Add-DbaDbRoleMember cmdlet wasn't there 2 years back when I was trying it.


Container Registry Naming

The naming of the Container Registery is not consistend with other azure resources.
Is there a specific reason why hypens are not allowed.
because currently it is not following the recommended naming convention mentioned here:
https://learn.microsoft.com/en-us/azure/cloud-adoption-framework/ready/azure-best-practices/resource-naming

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

ID: d3a84ce2-7a0b-4558-1e28-aeefc649b567
Version Independent ID: 3dffb383-4087-9aca-3b33-c5f777334062
Content: Define your naming convention - Cloud Adoption Framework
Content Source: docs/ready/azure-best-practices/resource-naming.md
Service: caf
Sub-service: caf-methodology-ready
GitHub Login: @martinekuan
Microsoft Alias: ssumner

@pmdevers
Thank you for bringing this to our attention.
I've delegated this to content author @martinekuan, who will review it and offer their insightful opinions.

@martinekuan
Could you please review add comments on this, update as appropriate.


Update sub test case to retain TTL=1 for incoming packets with TTL = 1

Update sub test case to retain TTL=1 for incoming packets with TTL = 1

Pull Request Functional Test Report for #3656 / 05001bdb8910447c54e7743115af1d80f5f5a3e5
No tests identified for validation.
Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 55.268%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
1983

Relevant Lines:
3588

💛  - Coveralls

@robshakir for 2nd review+approval since other gribi feature owners are unavailable or not connected to this use case.


Uncaught TypeError: $(...).datetimepicker is not a function

Do you want to request a feature or report a bug?
A bug

What is the current behavior?
Uncaught TypeError: $(...).datetimepicker is not a function
If the current behavior is a bug, please provide the steps to reproduce.
Clone the repo: https://github.com/Sparkenstein/webpack_issue_repo.git
Then run: npm install
and then run webpack  to produce webpack bundle. make sure you have webpack installed globally.

What is the expected behavior?
Datetimepicker should appear after clicking the input box
Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.
webpack version: 4.1.1
OS version: windows 10
webpack log:
PS C:\opt\tmp\angularjs\npmtest> webpack
Hash: 662dfb7575cadde332c2
Version: webpack 4.1.1
Time: 1925ms
Built at: 2018-3-26 09:52:15
                               Asset      Size  Chunks                    Chunk Names
776d58f453c8fe5d6a89e9c31ee223ff.svg   248 KiB          [emitted]  [big]
4f0022f25672c7f501c339cbf98d9117.ttf   138 KiB          [emitted]
89889688147bd7575d6327160d64e760.svg   106 KiB          [emitted]
                           custom.js   289 KiB  custom  [emitted]  [big]  custom
                             main.js  2.11 MiB    main  [emitted]  [big]  main
                           styles.js  4.37 KiB  styles  [emitted]         styles
                      styles.min.css   602 KiB  styles  [emitted]  [big]  styles
Entrypoint main [big] = main.js
Entrypoint custom [big] = custom.js
Entrypoint styles [big] = styles.js styles.min.css
   [0] multi jquery bootstrap angular moment eonasdan-bootstrap-datetimepicker 76 bytes {main} [built]
   [1] multi ./node_modules/font-awesome/css/font-awesome.min.css ./node_modules/bootstrap/dist/css/bootstrap.min.css 40 bytes {styles} [built]
[./b.js] 571 bytes {custom} [built]
[./node_modules/moment/locale sync recursive ^\.\/.*$] ./node_modules/moment/locale sync ^\.\/.*$ 2.88 KiB {main} [optional] [built]
[./node_modules/webpack/buildin/module.js] (webpack)/buildin/module.js 519 bytes {main} [built]
    + 158 hidden modules                                                                                font-awesome.min.css:
Child extract-text-webpack-plugin node_modules/extract-text-webpack-plugin/dist node_modules/css-loader/index.js!node_modules/font-awesome/css/font-awesome.min.css:
     2 assets
    Entrypoint undefined = extract-text-webpack-plugin-output-filename                                                                         s/bootstrap.min.css:
       8 modules
Child extract-text-webpack-plugin node_modules/extract-text-webpack-plugin/dist node_modules/css-loader/index.js!node_modules/bootstrap/dist/css/bootstrap.min.css:
     1 asset
    Entrypoint undefined = extract-text-webpack-plugin-output-filename
       8 modules

All the code copied as it is from SO : bootstrap-datetimepicker-is-not-a-function

Thanks for your report. It would be great if you reduce your issue to a small reproducible example. Best put this example into a github repository together with instructions how to get to the problem.

Thanks for your input, I have moved all the files to a temporary repository in GitHub.

While trying few things out, I noticed, If I bundle everything into one script, that works fine. Any solution on this?

It fails because main.js and custom.js both include jquery. The main entry contains its own version of jQuery with the plugin installed whereas custom.js does not. If you want to emulate CommonsChunkPlugin in webpack 4, use mode=production instead of creating additional entrypoints.

Well, the way i see this is that since datepicker.js is in another module, and you are using provide plugin, $ is replaced by jquery, but eonasdan-bootstrap-datetimepicker needs access to jquery to modify it maybe :grimacing:, since the version of jquery you used is not modified, the error occurs, a workaround to this situation can be by introducing a side effect in your datetimepicker.js file
import 'eonasdan-bootstrap-datetimepicker';
$(document).ready(function() {
...
};

Then it works as expected.

+1 to @nveenjain, importing the respected module solved the error. @ooflorent setting mode=production did not change the output error. it just made the package smaller in size. You can mark this issue as close on second answer.


redirectRegex doesn't seem to match on fully capital paths.

Welcome!

[x] Yes, I've searched similar issues on GitHub and didn't find any.
[x] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I have a vanilla k8s ingress, using annotations, I specify a middleware that should redirect any request to /license path (non-case-sensitive). ie. /license, /LICENSE, /liCEnsE, should all redirect to the base url.
What did you see instead?
All scenarios except for fully capital case paths, will redirect to the base path. I've been scratching my head as to why this could be, and I really can't figure it out at all.
What version of Traefik are you using?
traefik v3.1.2
What is your environment & configuration?
---
kind: Ingress
metadata:
  annotations:
    kubernetes.io/ingress.class: traefik
    traefik.ingress.kubernetes.io/router.entrypoints: websecure
    traefik.ingress.kubernetes.io/router.middlewares: harbor-license-redirect@kubernetescrd
spec:
  ingressClassName: traefik
  rules:
  - host: domain.com
    http:
      paths:
      - backend:
          service:
            name: harbor-core
            port:
              number: 80
        path: /api/
        pathType: Prefix
      - backend:
          service:
            name: harbor-core
            port:
              number: 80
        path: /service/
        pathType: Prefix
      - backend:
          service:
            name: harbor-core
            port:
              number: 80
        path: /v2/
        pathType: Prefix
      - backend:
          service:
            name: harbor-core
            port:
              number: 80
        path: /c/
        pathType: Prefix
      - backend:
          service:
            name: harbor-portal
            port:
              number: 80
        path: /
        pathType: Prefix
---
---
apiVersion: traefik.io/v1alpha1
kind: Middleware
metadata:
  name: license-redirect
  namespace: harbor
spec:
  redirectRegex:
    regex: ^(.*)/(?i)license
    replacement: ${1}

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
No response

Hello @Newman-Hui,
Thanks for reaching out!
To keep the repository focused, we ask that all questions be asked in the community forum. It is pretty active, so you might find that your question has already been answered there.  If not, you can ask and get help from other community members pretty quickly.


Proofread the Synfig Promo

Since the video is already out, I'm not sure if you want these subtitles or ones that match the video.
Closes #16

I am planning to make more releases of this video. So, these subtitles is what I need. Thank you very much for proofreading! ^__^

I have partially merged this pull request. Thank you so much for proofreading!


Add songs to the topic "Keerthanaigal"

Add existing songs to the topic "Keerthanaigal".

There are 23 songs now in this category.


Any chance of adding CoinGecko?

Quite a nice API available here: https://www.coingecko.com/en/api.

Why do you need CoinGecko?
Bitcoin Checker app interacts directly with exchanges API, but CoinGecko is not an exchange but a centralized aggregator.
Direct communication has various advantages: speed, availability, reliability, the ability to use the specifics of exchanges, and more.

Because it will add instant support for all coins listed there, including a lot of DEX. I saw you added Uniswap but there are uncountable other DEXes each with their own coins.
I'm not sure about alternatives, I know there is thegraph.com but it is also still missing a lot of exchanges.
Interesting problem, I'd be curious about other techniques/APIs of extracting price information from LPs for example! More and more of trading will take on-chain only.

I agree, CoinGecko is very useful for extracting price information from DEX.  But when I added Uniswap, CoinGeko was missing information on some significant tokens, and there were other problems, so I had to use a more complex but reliable TheGraph.
Perhaps the situation has improved now.
It seems like a good idea to implement support for top 20-30 DEX through CoinGecko. Will this solve the issue?
Full integration with CoinGecko doesn't fit into the app's concept.

Interesting problem, I'd be curious about other techniques/APIs of extracting price information from LPs for example! More and more of trading will take on-chain only.

Yes it's true. But what is "LPs"? :)

Liquidity pools. Problem is they don't use order books so price information is much more opaque. I find CoinGecko is somewhat of a solution here. This is their list of top DEXes: https://www.coingecko.com/en/dex.

:+1:  for this!
Will be awesome to have top 10/20/30 dex from coingecko on BitcoinChecker
In my point of view LPs  get out of the context of the APP, because the interface expect a pair(ex: BTC/BUSD)
For LPs we need to have a more explicit way to show this (as exemple, is often called BTC/BUSD LP)
An IDEA is to support the coins pair for these DEX on coingecko and put the LP on backlog for later

@jorijnsmit Unfortunately, I was unable to find an API to get ticker for coin pair. CoinGecko only allows you to get a list of prices for a token, not for a pair. This leads to traffic overhead.
Instead of CoinGecko , I plan to use a TheGraph.com for all DEX, as already implemented for Uniswap.

@jorijnsmit Unfortunately, I was unable to find an API to get ticker for coin pair. CoinGecko only allows you to get a list of prices for a token, not for a pair. This leads to traffic overhead.
Instead of CoinGecko , I plan to use a TheGraph.com for all DEX, as already implemented for Uniswap.

any plans to have thegraph working?

This turned out to be more difficult than I expected and will take longer, sorry. While the issue on hold.

@aneonex could you add pancakeswap similar to how you added uniswap in the meantime?


Error 500 when create pull request on Windows

I got error 500 when trying to do a pull request on a fresh install of gogs (built out of sources as of december 3rd). In the log file, I found this error :
2015/12/03 13:27:28 [...routers/repo/pull.go:632 CompareAndPullRequestPost()] [E] NewPullRequest: SavePatch: WriteFile: open MY_REPO_DIR\pulls\1.patch: The system cannot find the path specified.

Where MY_REPO_DIR is the folder of the repository. I found out that the subfolder pulls does not exists. I created it manually, and since then pull requests works.

Please see https://github.com/gogits/gogs/blob/master/CONTRIBUTING.md#bug-report

Yes I'm using Windows, and the problem is now fixed.
Thanks,

@bouffa thanks your confirmation!


Http client connection management is not optimal

When we dig into the CommonsHttpClientImpl, upon each request, it will create new http client, execute and then close
https://github.com/semi-technologies/weaviate-java-client/blob/85a837c61e096b8bab0512a14ba2c519d9ae421b/src/main/java/technology/semi/weaviate/client/base/http/impl/CommonsHttpClientImpl.java#L112
The http client created by HttpClients.createDefault() has a default connection manager with max=20 and maxPerRoute=2. However it should be put as the CommonsHttpClientImpl  class field and shared by multiple http calls and no need to do client close. Otherwise, we may met with following exception under a bit heavy load.
Mar 30, 2022 3:20:23 PM org.apache.http.impl.execchain.RetryExec execute
INFO: I/O exception (java.net.NoRouteToHostException) caught when processing request to {}->[http://<IP_ADDRESS>:80](http://<IP_ADDRESS>/): Cannot assign requested address (Address not available)
Mar 30, 2022 3:20:23 PM org.apache.http.impl.execchain.RetryExec execute

And going further we can use customized HTTP client connection pool manager to let client optimize the connections based on their loads.  And I tried locally this will increase the throughput especially under heavy load.
Creating the issue just for sync up in case you guys are already actively optimizing this. If not I could probably create a PR for this.

I think this has been fixed, @antas-marcin?


Handle edge-case for when User has invalid current_campaign

Gambit API would throw a 500 error in edge-case where User document is deleted, or the Campaign document of the saved current_campaign is deleted or closed.
Cleanest way to handle may be to implement a minimal CampaignBot msg_menu_select_campaign as a single field to update with keywords. A nicer way (but requires a little bit more work) could be to dynamically generate the menu content based on the current CampaignBot campaigns/keywords #745

This happened for https://secure.mcommons.com/profiles/223451936.
They signed up for a campaign, it failed, and then subsequent messages fail (as they've been opted into the Chatbot Mobile Commons campaign).
The addition of "we didn't get your photo" to the mData error message doesn't help this user in particular, as they haven't uploaded anything.

This one also gets handled by https://github.com/DoSomething/gambit-conversations currently.


Charles/phage filter

Adds an ad-hoc checkbox for phage to category filter.
Also changes the styles of the tags to match those on the sample page.
First, you need to run
PipelineRun.all.map(&:update_is_phage)

Until you do that, everything is classified as "not a phage" by default.

Ok, generalized it. It should now be easy to add Fungi filter without changing much frontend code.


Bug fix 2573

Pull Request type

[x ] Bugfix
[ ] Feature
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

#2573

After a discussion with @aravindanr it appears this entire block of code is entirely unnecessary as domain is evaluated as part of the tasks array where it is available and we should just use the value as-is returned by the API. Could you try if the following works?
Could you try if the following works?
  const handleSelectedTask = (task) => {
        setSelectedTask(null);
  }

Opend a new PR as this one got mixed with changes from other branch.


[1.9.x] Refs #27132 -- Added pylibmc to test requirements.

Backport of 5d978c46216df53884fbca590a9abe660a739774 from master
See: https://github.com/django/django/pull/7168#issuecomment-242986039

So we don't have to fix the pylibmc test failures on 1.8/1.9 (there are some conflicts when backporting your recent commits that I couldn't quickly resolve), I'm going to put this in the Jenkins settings file.
if django.VERSION < (1, 10):
    del CACHES['pylibmc']

Ok, sounds good to me :-)


Possible to add 1 more y-axis (Highest) label?

Hi there, e.g the highest data is 350, the bar chart will show the highest y-axis label as 350, is there a possibility to modify 350 into e.g 400 (e.g 0 > 116.67 > 233.33 > 400)? Or append 1 more layer to show 400 (e.g 0 > 116.67 > 233.33 > 350 > 400)? Thank You.

Another question was, currently by default it will automatic generate 4 (y-axis label). Currently Is there anyway to set to 5 or 6 labels & able to define y-axis by ourself?) Thank You

This would be great.

Agree, be great to define this, we only need 2 to display, would also be useful to be able to have some method to format the numbers, our customers wants to show thousand separators

This is not supported, but a Pull Request is welcome. Check out our contribution guide on how to get started.

Workaround if you just want to add a 'true' max value to your Y labels:
(e.g if you max data value is 350 but you want to show the max label as 400 anyway)
 <LineChart
    data={{
      labels: yourXLabels,
      datasets: [
        {
          data: yourMaxYValue, // <=== e.g: [400], the maximum you want, only one value in the array.
          color: () => `rgba(0, 0, 0, 0)` // <=== Here enable transparency of the rgba() to hide the max Y Value dot from your chart.
        },
        {
          data: yourChartValues, // your original data
          color: () => `rgba(255, 144, 182, 1)`
        }
      ]
  }}
/>

Preview:

But you can't add one more layer or choose yourself you y labels yet.

Workaround if you just want to add a 'true' max value to your Y labels: just add a fake dataset with a max value.
e.g if your max data value is 350 but you want to show the max label as 400 anyway
 <LineChart
    data={{
      labels: yourXLabels,
      datasets: [
        {
          data: yourMaxYValue, // <=== e.g: [400], the maximum you want, only one value in the array.
          color: () => `rgba(0, 0, 0, 0)` // <=== Here enable transparency of the rgba() to hide the max Y Value dot from your chart.
        },
        {
          data: yourChartValues, // your original data
          color: () => `rgba(255, 144, 182, 1)`
        }
      ]
  }}
/>

Preview:

It's just a workaround of course.
But you can't add one more layer or choose yourself you y labels yet.

I see, it works perfectly although it was a workaround, thanks for it buddy.


az iot central app device-twin   : Update documentation for Aug update

CLI improvements
To expand the capabilities of the CLI, we've re-organized the commands in the azure-iot extension. New commands include az iot central user, az iot central api-token, and az iot central device compute-device-key to generate a device SAS key for provisioning.
Two commands have moved to a new area:
az iot central app device-twin has moved to az iot central device twin.
az iot central app monitor-events has moved to az iot central diagnostics monitor-events.

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

ID: 8ca03657-212b-3c2e-aaf6-d7a1d3d91db6
Version Independent ID: e11da47a-6a74-e46e-baeb-5b347415f9d9
Content: az iot central app device-twin
Content Source: latest/docs-ref-autogen/ext/azure-iot/iot/central/app/device-twin.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

iot central


Add autodeploy CI step

Adds autodeploy step to Github Actions
This step depends on docker-publish which means that it will only be triggered once a new Docker image is published to DockerHub
This step executes autodeploy.sh which notifies AUTODEPLOY_URL that a new image is available

Coverage: 84.968% (+0.03%) from 84.937% when pulling 5988ca2c5117707292471a9f71d8b96836a703cc on add-autodeploy-ci into 7fb688f9f68c4a4cd8ebf7214b1eeb0962fd4c35 on main.


[Raffle] Ticket to DjangoCon EU 2023

Form here: https://docs.google.com/forms/d/17hQ3_LMTZJ9KxEv58JenXR9pIHkaNh6l8UoDAkh34P4/edit
We can use this repl.it to run the raffle: https://replit.com/@sleepypioneer/PyLadies-Raffle

[x] create form
[x] tweet raffle (see text below)
[x] share on slack
[x] share on linkedin
[ ] close form
[ ] run repl.it or similar program to determine winner from participants

⚡Flash Raffle!
Win an IN PERSON ticket to DjangoCon EU 2023 happening in Edinburgh (note travel & accommodation not included)
The conference happens next week 29th May - 2nd June. https://2023.djangocon.eu
✒https://forms.gle/eDFUoTFEm8ffju6Z9 Raffle ends end of the day tomorrow! (24th)

@marenwestermann could you take over the form and write the winner on Thursday??


Is Bash assumed as the default shell?

I use the Fish shell and the output I get from any yakuake-session call is the following:
 . '/tmp/yakuake-session-Z3JdLxrNNX'
[14:13:30]-[~]
[hp6-clima-10.ictp.it]->   . '/tmp/yakuake-session-Z3JdLxrNNX'
Unsupported use of '&&'. In fish, please use 'COMMAND; and COMMAND'.
/tmp/yakuake-session-Z3JdLxrNNX (line 2): true && cd /afs/ictp.it/home/a/afantini && exec ssh_marconi
                                                ^
from sourcing file /tmp/yakuake-session-Z3JdLxrNNX
        called on line 151 of file /usr/share/fish/config.fish

in function '.'
        called on standard input

source: Error while reading file '/tmp/yakuake-session-Z3JdLxrNNX'

This does not happen if I set Bash as the default shell to start in the Yakuake profile.

Yes. yakuake-session creates a script file which run inside the new Yakuake session. That file is compatible with bash. You can see it at:
https://github.com/aplatanado/yakuake-session/blob/master/yakuake-session#L278
But fish doesn't understand && operator. In fish the line:
$(profile_setup_command) && cd $(printf %q "$cwd") && $cmd

should be:
$(profile_setup_command); and $(printf %q "$cwd"); and $cmd

but it will not work in bash. So I don't know how to do that line works in fish and in bash compatible shell.
I guess I could detect the shell family and write a line or another for every case.

This bug is happening yet :

Welcome to fish, the friendly interactive shell
Type help for instructions on how to use fish
~   source '/tmp/yakuake-session-KyDy1hmH65'
/tmp/yakuake-session-KyDy1hmH65 (line 5): Unsupported use of '&&'. In fish, please use 'COMMAND; and COMMAND'.
cd /home/luis && true
^
from sourcing file /tmp/yakuake-session-KyDy1hmH65
called on standard input
source: Error while reading file '/tmp/yakuake-session-KyDy1hmH65'

cat /tmp/yakuake-session-KyDy1hmH65
clear
rm -f '/tmp/yakuake-session-KyDy1hmH65' 2>/dev/null
true
if [ -z "$FISH_VERSION" ]; then
  cd /home/luis && true
else
  cd /home/luis; and true
fi

More info :
Commit 5f83934
Fish-> fish --version
fish, version 2.7.1
kUbuntu 18.04.1


upgrade to 3.5.1

What does this pr do?

upgrade cloudreve to 3.5.1

Which issue is fixed by this pr?

Fixes # None

Does have any extra information?

None

Thanks for your contribution! The cloudreve 3.5.1 will be ready after GitHub Action finished on the master branch.


*: reduce max backoff time but don't count sleeps from busy errors

After TiKV replaces write stall with scheduler flow control, region errors hardly happen if the cluster works normally.
https://github.com/tikv/tikv/pull/10771 tries to guarantee that the client can receive the response before the RPC timeout. If a write request queues for too long, the client will receive a TiKV busy error.
After these changes, I think it possible to not count backoff times due to TiKV busy. If TiKV returns busy, it probably means the error is recoverable, so we can always retry. And other region errors are unlikely to exist for a long time, if we keep receiving the errors, it is likely that these errors are not recoverable, so we would better to return them to the user.

/cc @youjiali1995


Disable autocompletion on non login forms

This would be helpfull since for example edit fields wouldnt be autofilled by the browser.
For example in the edituser page when the username is recognized by the browser-autocomplete it fill prefill the password field.
In general password fields should not be prefilled or prefillable on registration and password edit pages.

I would appreciate if you submitted a pull request fixing that. ;) ... should be some additional HTML attributes on some formular elements.

is also in #37


Upgrade wheel, pip, and setuptools with make install_test

In the process of debugging the CI failure in #3442. We're now updating to the latest versions of wheel, pip, and setuptools at the beginning of the make install_test command.

I realized that we have to detect the pip upgrade and update the CI cache accordingly. Probably make more sense to pin the version in the CI configuration.

Pinning usually seems safer than upgrading to an arbitrary latest

@djrtwo
To make it a bit more flexible, I changed it to only set minimum version requirements here.


hasattr accepts non-string keys, unlike regular Python

class O(object):
    pass

o = O()

print delattr(o, None)               # TypeError in Python and Cython
print setattr(o, None, 42)           # TypeError in Python and Cython
print hasattr(o, None)               # TypeError in Python, returns False in Cython
print __builtins__.hasattr(o, None)  # TypeError in Python and Cython

Interesting. We're just calling PyObject_HasAttr. I guess https://docs.python.org/3/c-api/object.html lies.

Curious! PyObject_GetAttr does the string check, but PyObject_HasAttr clears the error and just returns NULL, so builtin_hasattr ends up doing a second string check to raise the TypeError.

Preparing a patch now.


[FEATURE] Longhorn SPDK Support

Using the information gained by the SPDK proofs of concept, begin working a version of the longhorn engine that uses SPDK.  This will work on a network on systems running the same customized SPDK software.
Describe the items of the task (DoD, definition of done) you'd like.

[x] Create SPDK JSON RPC command to add a volume, with 0 or more replicas.  This will make the volume available over NVMEoF.
[x] Create a SPDK JSON RPC command to create a replica.  This will make a new replica and make it available over NVMEoF.
[ ] Create a SPDK JSON RPC command to add a replica to a volume.  This will rebuild the replica to ensure the replica is consistent with the other replicas in the volume.
[x] Create a SPDK JSON RPC command to remove a replica from a volume.
[ ] Remove a replica automatically by timeout.  The longhorn bdev will basically proxy reads and writes to NVMe bdevs.
[ ] Create a SPDK JSON-RPC command to retrieve the state of a longhorn volume.
[ ] Create a SPDK JSON-RPC command to retrieve the state of each replica in a longhorn bdev.  This will include the IP address.

We can add commands and a gRPC interface to eventually integration with longhorn-manager.

@keithalucas moving this to the analysis pipeline, and also please help frequently update the status in the thread. Thanks.

I have the commands to create a replica and to create a volume from a list of replicas.  I'm working on implementing the consistent snapshot for adding new replica to an existing volume.

Is it possible to borrow some implementation from mayastor to speed up the progress?

Is it possible to borrow some implementation from mayastor to speed up the progress?

mayastor uses a Rust binding for SPDK's C API.  Our intention is to use C for the SPDK support.  Conceptually they do similar things in Rust to what we're doing in C.

I ran the kbench to compare a volume created with SPDK to a volume created with the existing longhorn engine, both with three replicas.  Using the default kbench settings, SPDK was this:
FIO Benchmark Summary
For: longhorn-spdk-10g
CPU Idleness Profiling:
Size: 10g
Quick Mode: disabled
=========================
IOPS (Read/Write)
        Random:         132,460 / 36,112
    Sequential:         127,961 / 70,484

Bandwidth in KiB/sec (Read/Write)
        Random:      1,163,133 / 351,267
    Sequential:      1,329,411 / 297,980

Latency in ns (Read/Write)
        Random:        197,335 / 155,719
    Sequential:        195,769 / 155,325

And the existing longhorn-engine was this:
=========================
FIO Benchmark Summary
For: longhorn-legacy-10g
CPU Idleness Profiling:
Size: 10g
Quick Mode: disabled
=========================
IOPS (Read/Write)
        Random:          45,178 / 16,775
    Sequential:          65,422 / 29,360

Bandwidth in KiB/sec (Read/Write)
        Random:        816,055 / 326,926
    Sequential:      1,050,622 / 341,601

Latency in ns (Read/Write)
        Random:        328,214 / 316,525
    Sequential:        324,732 / 315,977

However, I reran the test with a size of 100g.
SPDK:
=========================
FIO Benchmark Summary
For: longhorn-spdk
CPU Idleness Profiling:
Size: 100g
Quick Mode: disabled
=========================
IOPS (Read/Write)
        Random:         135,052 / 31,597
    Sequential:         124,419 / 70,430

Bandwidth in KiB/sec (Read/Write)
        Random:      1,133,688 / 370,263
    Sequential:      1,116,099 / 339,418

Latency in ns (Read/Write)
        Random:        197,643 / 154,698
    Sequential:        194,259 / 154,323

Longhorn-engine:
FIO Benchmark Summary
For: longhorn-legacy
CPU Idleness Profiling:
Size: 100g
Quick Mode: disabled
=========================
IOPS (Read/Write)
        Random:          54,756 / 11,468
    Sequential:          65,500 / 29,495

Bandwidth in KiB/sec (Read/Write)
        Random:      1,748,036 / 314,339
    Sequential:      1,762,729 / 339,428

Latency in ns (Read/Write)
        Random:        242,818 / 357,016
    Sequential:        320,200 / 317,861

The bandwidth results for longhorn-engine don't necessarily make since because they are more three times the bandwidth of the drive being used.  https://semiconductor.samsung.com/ssd/datacenter-ssd/pm883/mz7lh480hahq/   3*550 MB/S would be 1650 which is less that the random and sequential read.
If Linux's disk caching is causing this performance improve for the existing longhorn-engine, why doesn't it improve the SPDK version as well or as much?

Very interesting and promising results Mr. @keithalucas - is it possible to try out this SPDK longhorn volume already? I'm very eager as I'm seeing some less than desired perf. results currently on Longhorn v1.2.4 - see > https://github.com/longhorn/longhorn/issues/3037
Thanks

@keithalucas caching is done in kernel, spdk work in user space, could it be the cause?

Hi, is there any updates?

Closing, we will reorganize issues for SPDK.


Drag and drop is possible from file select dialog

Hi
It's currently possible to drag and drop files from system select file dialog. Is it a bug or a feature? If it's a feature is it possible to turn it off?
How to reproduce:

Open https://angular-file-upload.appspot.com/
click on the "Select File, Drop file or paste image" location
in the opened system file selection get file and drag and drop it on "drag and drop location"

Expected behaviour: drag-n-drop from system dialog is not handled, i.e. ignored
Actual behaviour: file start uploading + i can continue using system file selection dialog to select file. If so then it will replace one that was drag-n-dropped

There is not much we can do about that the browser popup and drop event all coming from the browser and are browser specific implementation. So there is no way to tell where the drop event comes from.


Manifest file's default Taskpane.Url is incorrect for the example Vue project (Fix issue with Add-in not side loading)

Having spent sometime attempting to side-load the basic Vue based Excel Office Addin created by following tutorial steps and not being able to figure out why it won't load, I have discovered that the Taskpane.Url is pointing to a .html file that doesn't exist.
To fix the issue it is necessary to update the manifest.xml file and find the entry
<bt:Url id="Taskpane.Url" DefaultValue="https://localhost:3000/taskpane.html" />

This entry refers to a non-existant taskpane.html file.  Change the entry so it points to you index.html file, thus
<bt:Url id="Taskpane.Url" DefaultValue="https://localhost:3000/index.html" />

Please update the tutorial to either include a step to help the user understand the manifest.xml file and check and update the entry where necessary.
Overall, the tutorial could do with more details on the key entries in the manifest.xml file, what they're used for, and when the should be changed.

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

ID: ea230e95-f483-6d19-8167-2341a03cf396
Version Independent ID: 3846765f-7587-fec4-711e-84edfce9c30c
Content: Build an Excel task pane add-in using Vue - Office Add-ins
Content Source: docs/quickstarts/excel-quickstart-vue.md
Product: excel
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Thanks for reporting this @jadjare.
@alison-mk, could you please take a look?

Hi @jadjare, thank you for reporting this issue and providing feedback. I've updated the article to incorporate your suggestions, and those changes are now in PR #2795. Once the PR is merged, this issue will automatically close. Let us know if you have any additional questions or concerns.
Cheers,
Alison


Fix Typo in Mirascope Integration Docs Page

@Kludex I noticed with this PR that there was a checkbox for "allow edits from maintainers" so I think it was because my previous PR was under the organization and not my personal account.
Will only use the personal one going forward so maintainers can make edits :)

Thanks. 👍


Disable the path filter for IM read.

Problem
Currently in master, we have enabled path filter for IM read, in spec, it is allowed to have multiple duplicated/overlapped path, there exists violation between code and spec, we would disable this path filter per @bzbarsky-apple 's suggestion.
At the same time, we file https://github.com/CHIP-Specifications/connectedhomeip-spec/issues/4045 to sort out right behavior.

done in https://github.com/project-chip/connectedhomeip/pull/9510


[core] Inspect subscription

Inspecting an actor system via actor.system.inspect(ev => …) now accepts a function or observer, and returns a subscription:
const actor = createActor(someMachine);

const sub = actor.system.inspect((inspectionEvent) => {
  console.log(inspectionEvent);
});

// Inspection events will be logged
actor.start();
actor.send({ type: 'anEvent' });

// ...

sub.unsubscribe();

// Will no longer log inspection events
actor.send({ type: 'someEvent' });

The subscription is welcome, but this also makes it easier to attach an inspector on-the-fly, because you don't need to use toObserver to do so.


Fix typing hinting for PHP7

In PHP7, most errors are now "exceptions" of type "class Error implements Throwable"
This is a quick fix to type hint for the interface \Throwable instead of the \Exception.

Your pull request will make php-console not working in PHP < 7. Sorry, I can't merge it.


feat: new deploy workflow for new ECR repo

Dependency of https://app.asana.com/0/584764604969369/1200734194931980/f. Adds a new deploy workflow which calls the reusable workflow and assumes that a different ECR repo secret is present, so we can test a dev deploy without altering prod. After prod is switched over to Terraform and the new repo this workflow will handle deployments to all environments.

While it's true we only have the new dev infrastructure set up (or will, shortly), can you add the input field (and updating the workflow call to use it) like, e.g., here? That way, we can move onto the prod terraform PR without having to do another PR over here first. Then, once both are done, we can remove the old workflow and update this one's secret to use DOCKER_REPO.


Find mutual from the overall ranking

Hi,

Can you make a function to find the mutual from the overall ranking?(Not a country ranking)

Can you make a function to save players who are checked and ignore them in the next finding?

My BN and GMT friends reported that they are stucked on email verification(wrong code), do you know why?

I can make it so that the program start will from the first id (1) and then checks everyone by incrementing the number but I don't think this is a good solution.

Actually this is a good idea, I can put a setting that will put numbers in the blacklist automatically. In the meantime, you can keep track of the page that the program left on and then use it to keep it from starting from 0.

I know about this issue but cause I don't have access to GMT account I don't know what is the problem. Probably with accounts that have extra perms there is extra verification stuff going on in the background.

I mean, can you make the program add friends starting from the #1 in the global ranking (Like sakamata1, aetrna, lifeline...)

Yeah, I can add this soon.

Released with https://github.com/sibyl666/osu-mutual-finder/releases/tag/v1.2.9


Is this toolbox suitable for robots with prismatic joints?

hi，Is this toolbox suitable for robots with prismatic joints?
such as  a robot with a revolute joint and a prismatic joint.
Looking forward to your reply!

hi，Is this toolbox suitable for robots with prismatic joints? such as a robot with a revolute joint and a prismatic joint. Looking forward to your reply!

https://github.com/cdsousa/SymPyBotics/issues/12


Stop adding timestamp to javadocs html

One of the causes of our repo bloat was the javadocs constantly changing due to a generated timestamp. The configuration seems to have no effect when set within the aggregate/site phase but works as a global setting.

Coverage remained the same at 62.61% when pulling 0b03d8b17e9e6da0fe7ea9325bd3c79a969b203f on no-javadoc-timestamp into 441e24b53930cb52294a31e0af77362985531dce on master.


figfam id.

Can you explain a little bit more? What do you think is wrong? What is expected in your mind?

Not sure what is the problem here, not enough details. closing the ticket.


discovery: flag flip no historical gossip

This PR adds the ignore-historical-filters CLI option, initially
defaulting to false. Users may use this option to prevent lnd from doing
historical gossip dumps to peers that set their gossip_timestamp_range
in the past. Enabling this option will result in lower bandwidth and
memory consumption. Down the road the plan is to make this default to
true.

@Roasbeef test added, stripped the existing test from #3315 so should make the code removal at that point simpler as well


Run from command line?

I like the way this extension presents coverage data, and I'd like to integrate it into my GitLab automated build pipeline.

Is there a way to generate this report as a standalone web page?
Can the report generation be run from the command line?

@SpencerRiddering I don't think this extension is able to generate a report. I think it only reads from a report and show the coverage info in VSCode.


when an address gets reused, establish a happens-before link in the data race model

Fixes https://github.com/rust-lang/miri/issues/3450

@bors r+

:boom: Test timed out

I excluded stack allocations from reuse, and made the reuse rate configurable.
@bors r+

:pushpin: Commit f3b31c3356be6492f28aad9c5e4956a98571b5b1 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit f3b31c3356be6492f28aad9c5e4956a98571b5b1 with merge ab25a3382736a8ca6de270a23f69f830c830cfae...

This actually still has a quite significant impact on data race detection; I suspect this is due to heap allocations in the thread spawning logic. I'm not sure what the best way forward is here... other than reducing the reuse rate drastically. With a reuse rate of 25%, the simple data race I checked is still found around 70% of the time (but it's 100% without reuse).
I think the rate will be better with real programs; the issue is that in these tests the threads exist only for a very short time so there's a real chance it never gets preempted and so the first thread is done before the second thread starts, and therefore allocations can be reused. If I add a simple for _ in 0..100 {} to the first thread, the data race is detected ~100% of the time even with address reuse.
We could of course also just not to any reuse across threads, then we also will never add spurious synchronization. But for heap allocations that's not real either.

:broken_heart: Test failed - checks-actions

OTOH, real programs will also do a bunch of stuff on the heap, which again increases the chance of address reuse inducting synchronization.
My current inclination is to add some extra randomness that makes address reuse across thread boundaries a lot less likely than address reuse within a single thread.

With the cross-thread reuse rate of 10%, the simple data race tests still detect the race 80% of the time when preemption is disabled (and ~100% of the time when preemption is enabled).

That seems fine to me. Anyone seriously looking to find data races should have preemption enabled.

Yeah... though it's hard to say what this will do for other cases. Not sure how many people run concurrency tests in a loop in Miri to give races more than one chance to appear.
But for now, let's go ahead with this, we can always re-adjust later if we realize that data race detection has become too non-deterministic.
@bors r+

:pushpin: Commit 2155a302a78ee0fc95aecf093ac35739d0cd562b has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 2155a302a78ee0fc95aecf093ac35739d0cd562b with merge 9b369148c6d60b8310f272312d36a12a716b1af2...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 9b369148c6d60b8310f272312d36a12a716b1af2 to master...


(cli) Allow disabling of public access block configuration during ddk bootstrap

Enhancement

(Allow disabling of public access block configuration during ddk bootstrap

Relates

#112

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

Agreed - I don't think we should support every little flag available in cdk bootstrap or other commands. Like for CDK, our underlying bootstrap template can be customised by the user if they require anything beyond what the vanilla template offers

@malachi-constant can't we parametrize the bootstrap template?

We can and we do already with some parameters. I just don't think that one is worth it. It also opens the door to similar requests when there is already the option of using a custom template

IMO it's fairly minimal to add an !If into the template. I do agree that we have to put the foot down somewhere though


Make quast a slurm-only tool

Pulsar is not transferring all of the files back for quast jobs. This needs further investigation but in the meantime it should not run on pulsar.

It never needed to be going to high-mem either... we put it there early on at somebody's request but 16 cores + 62G will be easily enough for the large jobs


Weights?

Thanks for sharing the code,  is there any training weights available for this work?

Unable to upload model now for some reason

@WenmuZhou Upload to Google Drive, or Microsoft OneDrive

I will update model before 11.11

is't ok

@WenmuZhou
The resnet18+FPEM_FFM that you uploaded, is it 1e-3 or 1e-4?

1e-3


chore: improve generated changelog

Codecov Report

Merging #79 (dc21ac3) into main (721e3f1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   71.75%   71.75%
=======================================
  Files           9        9
  Lines         570      570
=======================================
  Hits          409      409
  Misses        161      161

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 721e3f1...dc21ac3. Read the comment docs.


add clone docs for strategy (closes #520)

Summary

Checklist

[ ] ran cargo make fmt
[ ] using conventional commits to hightlight user-facing fixes and features

#534
#533  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @arlyon and the rest of your teammates on  Graphite


add yuwapat school file

add yuwapat school file
yuwapatschool.ac.th, thailand

@63warittochi Please provide us with:

the school official website URL
the school official full name in Thai
the school street address, including city and country.

Thank you.

@63warittochi You might miss my previous comment. Please react.

@63warittochi I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Use the lab builds which is what actually gets built when shipping

Use the lab builds, which would've caught the build break here: #325.
/cc @caslan @andrewcrawley

@mmitche Can you verify this is ok?

LGTM

As @andrewcrawley and I discussed yesterday, using Lab builds (both by explicitly specifying them or using Andrew's suggestion /p:Lab=true) doesn't run the unit tests.
Andrew suggested we could add a property to run unit tests, so in the end the option would be /p:Lab=true;Tests=true
@caslan, any objections to that?

@edumunoz what do we want to do with this PR?

I haven't had time to look into adding an argument to run the tests.
I was hoping @caslan would give the heads up.

Sorry, I missed the earlier communication. I don't have any issue with additional argument for tests.


Trouble matching tags if the same element is nested as self-closing

- atom-react v0.14.1
- Atom v1.4.3

Thanks for reporting. Will investigate asap.

Not sure if this was fixed before but it seems to have started again.

atom-react - 0.16.2
Atom - 1.14.4


Fix link to Get Started page in Flutter introduction page

closes #172
- What I did
Updated the link
- How I did it
By changing path
- How to verify it
Just to check the page
- Description for the changelog
Fix broken link of Flutter intro page

Hi, @TylerTrott
Not sure, how it happened. The thing, that you've mentioned was before my fix, and, can be found on https://dev3.atsign.wtf/sdk/flutter/introduction/
I've tried my changes locally, and, it works fine.
Untitled_ Oct 3, 2022 5_34 PM.webm

Hugo can be a little wonky. I think the easiest fix here is to check what the link address is on the page you want to be on, and simply copy and paste that address into the hyperlink.

guys, I have to fix the semantic check, but, dismissed your approvals @TylerTrott  @XavierChanth . Could you do that again, please?

@daniloff200
Hey! Thank you for your contribution. Your PR was accepted, merged, and it closed an issue. You could be eligible for a prize.
To be eligible, please send an email to<EMAIL_ADDRESS>with a screenshot of the PR approval email. This is to prove that your email corresponds to the correct GitHub account. Thank you!
When it comes time to distribute the prizes, we will look at all the emails we've received from that email so don't forget to do this!


Initial addition of documentation

This PR includes the following.

Ported entire documentation from mlrose's sphinx implementation to a markdown based SSG: mkdocs
A couple of edits were required as the port wasn't clean.
Some of the markdown files needed to be written from scratch as I could not port them directly.
Changed all occurrences of 'mlrose' to 'mlrose-ky'
Added better navigation

The docs can be viewed here: mlrose-ky docs

I can't see an issue section in the repo.
The only question I have is what else is required to add? Apart from covering a tutorial on hiive's runners.
Could you list out your enhancements so I can start working on it? @knakamura13

Could you list out your enhancements so I can start working on it?

Here's my list of tasks for this project. Task 1 is completed; tasks 2 and 3 are in progress.

Fix all Python warnings and errors (except for some unavoidable warnings such as "duplicate code") ✅
Add Python 3.10 type hints to all function and method definitions, plus all method properties (e.g., self.foo: str = 'bar')
Add NumPy Doc docstrings to all functions and methods, plus at least a one-line docstring at the top of every file summarizing the file's contents
Search for TODO, FIXME, or other comments and resolve their respective issues
Optimize vanilla Python loops that can be vectorized with NumPy
Improve any other sub-optimal code, bugs, code quality issues, etc
Remove any commented-out code


Fail to persist imports

Not sure if this is a Notepad++ or a Markdown++ issue, but here it goes:
I can successfully import Markdown++ syntax highlighter, it is listed in the combo box of the UDL dialog. However, it is not persisted nor listed among the language menu; thus I cannot apply it and when I restart Notepad++ it has disappeared.
Workaround for me consists in pasting the XML in
%APPDATA%\Notepad++\userDefineLangs
(in my case %APPDATA% = C:\Users\samuelm\AppData\Roaming\ )
Notepad++ v7.6.6   (32-bit)
Build time : Apr  3 2019 - 23:49:50
Path : C:\Program Files (x86)\Notepad++\notepad++.exe
Admin mode : OFF
Local Conf mode : OFF
OS : Windows 10 (64-bit)
Plugins : DSpellCheck.dll MarkdownViewerPlusPlus.dll mimeTools.dll NppConverter.dll NppExport.dll

Have re-written README.md that suggest downloading from latest release page.
Thankfully Notepad++ , from v7.6.4 , delegates a folder <userDefineLangs\> to collect your favorite UDLs.
I would consider this issue solved. Thanks for rising this question! 🍕


WIP: Refactor fixups

@sjanssen2 - please do not touch this PR.

@ebolyen, can you please review this PR when you get a chance? Thanks!


Update angular.rst

Angular 1.5.0 final release has happened  couple of days ago.

:shipit:

Thanks for the PR. Merged in 960601c04f239abd7fecf356ca8e1f8f7e067b51


Linux - App will not open after changing locale to ja_JP.UTF8

Acknowledgement

[X] I have read Getting-Started and FAQ

🐛 Describe the bug
i have changed my locale from from en_US.UTF8 to ja_JP.UTF8 with both my desktop environment and locale.gen, and after a reboot, OpenUTAU will not open. it seems to begin to load, and then crash.
i have manually gone into prefs.json and changed the language to ja-JP, and it still does not open, and i have uninstalled and reinstalled the program using yay.
it seems to be font-related.
Explains how to reproduce the bug

install openutau-installer using yay in the en-US.UTF8 locale, and then open it once.
change the locale to ja_JP.UTF8 and reboot.
try to open OpenUTAU, and it will load and then crash.

OS & Version
Arch Linux 6.3.2
Logs
log20230624.txt

update: changing back to en_US.UTF8 fixes the problem, but i would still look into this issue just in case. i am leaving the issue open for this reason. i'm willing to go back to ja_JP.UTF8 to help with troubleshooting!

also had this issue on debian 11!  seems to be an unhandled exception.
System.InvalidOperationException: Default font family name can't be null or empty.
   at Avalonia.Media.FontManager..ctor(IFontManagerImpl platformImpl) in /_/src/Avalonia.Base/Media/FontManager.cs:line 37
   at Avalonia.Media.FontManager.get_Current() in /_/src/Avalonia.Base/Media/FontManager.cs:line 59
   at Avalonia.Media.Typeface.get_GlyphTypeface() in /_/src/Avalonia.Base/Media/Typeface.cs:line 87
   at Avalonia.Rendering.Composition.Compositor.get_DiagnosticTextRenderer() in /_/src/Avalonia.Base/Rendering/Composition/Compositor.cs:line 58
   at Avalonia.Rendering.Composition.Compositor.CreateCompositionTarget(Func`1 surfaces) in /_/src/Avalonia.Base/Rendering/Composition/Compositor.Factories.cs:line 17
   at Avalonia.Rendering.Composition.CompositingRenderer..ctor(IRenderRoot root, Compositor compositor, Func`1 surfaces) in /_/src/Avalonia.Base/Rendering/Composition/CompositingRenderer.cs:line 53
   at Avalonia.Controls.TopLevel..ctor(ITopLevelImpl impl, IAvaloniaDependencyResolver dependencyResolver) in /_/src/Avalonia.Controls/TopLevel.cs:line 188
   at Avalonia.Controls.WindowBase..ctor(IWindowBaseImpl impl, IAvaloniaDependencyResolver dependencyResolver) in /_/src/Avalonia.Controls/WindowBase.cs:line 55
   at Avalonia.Controls.Window..ctor(IWindowImpl impl) in /_/src/Avalonia.Controls/Window.cs:line 223
   at Avalonia.Controls.Window..ctor() in /_/src/Avalonia.Controls/Window.cs:line 214
   at OpenUtau.App.Views.MainWindow..ctor() in /home/appveyor/projects/openutau/OpenUtau/Views/MainWindow.axaml.cs:line 45
   at OpenUtau.App.App.OnFrameworkInitializationCompleted() in /home/appveyor/projects/openutau/OpenUtau/App.axaml.cs:line 34
   at Avalonia.AppBuilder.SetupUnsafe() in /_/src/Avalonia.Controls/AppBuilder.cs:line 307
   at Avalonia.AppBuilder.Setup() in /_/src/Avalonia.Controls/AppBuilder.cs:line 291
   at Avalonia.ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime(AppBuilder builder, String[] args, ShutdownMode shutdownMode) in /_/src/Avalonia.Controls/ApplicationLifetimes/ClassicDesktopStyleApplicationLifetime.cs:line 219
   at OpenUtau.App.Program.Main(String[] args) in /home/appveyor/projects/openutau/OpenUtau/Program.cs:line 41```


Handle trigger activations with inactive rules

log trigger activation entry for disabled rules (but at least one active rule is needed to create a trigger activation record)
if trigger has no associated active rule(s); return a 204.  also no trigger activation record is created

@mdeuser can you take a look at #3256 I think we should get that one in first as it will benefit the enhancement in this PR.

rebased on top of #3256

Updated logs field in the trigger activation record will look like the following for a trigger with one active and one inactive rule:
    "logs": [
        "{\"statusCode\":0,\"success\":true,\"activationId\":\"a0e1a50ba6074eeca1a50ba6072eecdc\",\"rule\":\"guest/activerule1\",\"action\":\"guest/web-echo-env\"}",
        "{\"statusCode\":1,\"success\":false,\"rule\":\"guest/inactiverule1\",\"error\":\"Rule guest/inactiverule1 is inactive; action guest/web-echo-env was not activated\",\"action\":\"guest/web-echo-env\"}"
    ],

This will require a CLI change as well.


Support for Borland FBOV / VROOMM overlays for DOS MZ EXE

It would be nice to get support for the "FBOV" overlay extension for MZ EXEs. Also known as "VROOMM" (Virtual Run-time Object-Oriented Memory Manager), this was a feature provided by the Borland C/C++ toolchain from v2.0. It allowed for dynamic swapping of code segments to get around the 640K memory barrier. This approach was popular only briefly, as extenders such as DOS/4G soon became available. A section in Chapter 1 of the Borland C++ 4.0 reference manual describes the general design of VROOMM.
I've attached a sample executable in this ZIP file: nomad.zip
The .exe can be loaded by the existing Ghidra MZ loader, but with the following message:
File contains 0x29390 extra bytes starting at file offset 0x3e9c0.
This extra data is actually the entire FBOV section from the file, which starts with this header:
char[4] fbovMagic; // always "FBOV" -- should be used for detection
uint32 ovrsize;
uint32 exeinfo;
int32 segnum;

In the sample file, ovrsize is the size in bytes of the FBOV section excluding its header (0x29380). The segnum field is the total number of segments known to the overlay mechanism (0xa5). These segments are described by an array of structs at the file offset given by the exeinfo field. Borland gives this array the internal name _SEGTABLE_ in OVERLAY.LIB. In the sample, _SEGTABLE_ starts at 0x2e100 (which is actually in the middle of a segment containing the other VROOMM data.)
Each entry in _SEGTABLE_ is an 8-byte structure:
uint16 seg;
uint16 maxoff;
uint16 flags;
uint16 minoff;

The first few entries in the segment table of nomad.exe are:
  0, 21e2, 1, 0
21e, 5214, 1, 4  // loaded by IDA as seg001
73f, 1043, 1, 4  // loaded by IDA as seg002
843, ba,   1, 4  // loaded by IDA as seg003
84e, cb2,  1, a  // loaded by IDA as seg004

IDA Freeware 5.0 supports MZ/FBOV and can be used as a reference for desired behavior. In the case of nomad.exe, the code from the 0x29380 extra bytes at the end of the file is divided into overlay segments ovr075 through ovr153.

Hey! I made an utility, which pre-relocates the FBOV exes, so they could be loaded into Ghidra:
https://github.com/NancyAurum/devroomm/blob/main/tools/mzap/mzap.c
These exes will only work with Ghidra's analyzer, since IDA Pro gets confused by the cleared relocation table and doesn't resolve the far jumps properly.
As of now it can't handle the multisection FB file (beside FBOV there can be other FB sections) and ignores the function-reference relocation flag, since the exe I'm working on doesn't have them, so I can't test it. The overlay manager itself does the following to fixup the function references:
#define X86_MOD          0xF8
#define X86_RM           0x07
#define X86_MOV_IMM16    0xB8
#define X86_PUSH         0x50
void __OvrFixupFunref(uint16_t rseg, uint8_t *p) {
  //p - pointer at the fixup location

  //segment inside which the fixup location does a function call
  uint8_t *q = SEG2PTR(rseg);

  //Ensure our fixup site loads a far pointer, like the following
  //mov reg1,seg
  //push reg1
  //mov reg2,ofs
  //push reg2
  //Check the MOD part of the MODRM byte
  //it does some opcode magic, since 0x50 is both `push AX`
  //and when masked by 0xF8 is a general `push <register>` group
  //Same with 0xB8, which  is both `mov AX,<imm16>` and an opcode group
  if ((p[-1]&X86_MOD) != X86_MOV_IMM16) return; //moves segment?
  if ((p[ 2]&X86_MOD) != X86_PUSH) return;
  if ((p[-1]&X86_RM) != (p[2]&X86_RM)) return; //same RM operand?

  //now check the offset part...
  if (p[-1] != p[3]) return; //both are `move <reg>,IMM`?
  if (p[ 2] != p[6]) return; //both are `push <reg>`?

  uint16_t fofs = *(uint16_t*)&p[4]; //get raw offset of the function

  uint16_t rofs = sizeof(bosh_t);

  //go through the botrp_t entries
  for ( ; fofs != *(uint16_t*)&q[rofs+2]; rofs += 5);

  *(uint16_t*)&p[4] = rofs; //relocate the offset part of the far function ptr
}

Hey! I made an utility, which pre-relocates the FBOV exes, so they could be loaded into Ghidra: https://github.com/NancyAurum/devroomm/blob/main/tools/mzap/mzap.c

Just realized that instead of relocating, one can update the MZ relocation table and header to include the FBOVs segments, in addition to untrapping the trap segments. That way the resulting exe is properly loaded by both Ghidra and IDA, although IDA still detects it as a Borland overlayed exe and offers to load an external .ovr file (only Turbo Pascal 5.0 supported these), even though mzap erases the FBOV id after merge. Would be still nice if Ghidra does that properly because the FBOV SEGTABLE has proper segment starts and ends, for both normal and overlayed segments.

If this helps, I wrote a simple script that appends overlays from a selected executable to the program and wires them up: https://github.com/GeReV/ghidra_scripts/blob/main/LoadBorlandOverlays.java.
This is based almost entirely on @NancyAurum's work (thank you!).
It's currently just a simple script that asks for an executable explicitly because I'm working from a raw memory dump; Haven't sat down to learn how to integrate it as a loader or with analysis yet.
Side note, I got this working in my case, which required adding 0x2 to some segment so the correct traps are wired up. I'm not too sure why that was necessary (probably a bad calculation I missed), if anyone knows what I missed, please let me know. :)

Anyone have any official header files for these structures?

The SE.ASM was published as part of Turbo Pascal 7.01, it has the official field names for the overlay stub table header:
https://pastecode.io/s/7tr67h6q
; Overlay header record

ovSignature	EQU	(WORD PTR 0)
ovSaveReturn	EQU	(WORD PTR 2)
ovFilePos	EQU	(WORD PTR 4)
ovCodeSize	EQU	(WORD PTR 8)
ovFixupSize	EQU	(WORD PTR 10)
ovJumpCount	EQU	(WORD PTR 12)
ovLink		EQU	(WORD PTR 14)
ovSegment	EQU	(WORD PTR 16)
ovRetryCount	EQU	(WORD PTR 18)
ovNext		EQU	(WORD PTR 20)
ovEmsPage	EQU	(WORD PTR 22)
ovEmsOffset	EQU	(WORD PTR 24)
ovUserData	EQU	(BYTE PTR 26)
ovVectors	EQU	(BYTE PTR 32)
ovRecSize	EQU	32

There are also different versions of these overlay manager (the BC++ 3 one the most advanced) and Borland executable could have several sections beside the overlay one. I have decompiled the loader code:
https://github.com/NancyAurum/devroomm/blob/main/src/ovrman.c


feat: upgrade to swiper 5

Bug Report
Ionic version:
[x] 4.11.2
Current behavior:
i have those errors being generated on users devices:
undefined is not an object (evaluating 's.$imageEl.transition')
undefined is not an object (evaluating 't.$imageEl.transform')

they seem to be generated from here:

https://github.com/ionic-team/ionic/blob/master/core/src/components/slides/swiper/swiper.bundle.js#L5280
https://github.com/ionic-team/ionic/blob/master/core/src/components/slides/swiper/swiper.bundle.js#L5083

in both cases it can't be ensured that $imageEl is defined.
See also the issue on the swiper repo: https://github.com/nolimits4web/swiper/issues/3275

Thanks for the issue. Can you provide a code reproduction of the issue?

@liamdebeasi  i didn't manage to reproduce it, those errors come from production error logs. however looking at the code lines i posted it seems apparent to me where the errors come from ($imageEl can be undefined in both cases and needs to be guarded). Since this is an old swiper version and the library is bundled directly with ionic it's probably best to just change those lines in the ionic repo.

Makes sense. Thanks!

@liamdebeasi i made the fix to swiper.bundle.js but then realised that this file is re-created by an ionic build script (swiper.rollup.config.js) from the original<EMAIL_ADDRESS>package. So there either  an update needs to be made to the next major swiper version (which probably would be a major issue?) or it seems to me the swiper build script (which regenerates the checked in swiper.bundle.js) should be disabled, since it's unlikely that this old version will be fixed (the author also seems to indicate that on the issue on the original repo).
What do you think?

@chriswep  then shouldn't this be fixed in swiper first and then upgrade it in ionic?
It is reported, by the way, in the swiper repo too https://github.com/nolimits4web/swiper/issues/3275
the issue is closed due inactivity.
I am using Swiper 4.4.6 (from @ionic/angular 4.7.4)

@eamador sure that would be the right thing to do. However version 4 is deprecated and the author does not intend to work on the old version (which makes sense for this kind of project ..?). I'm also not sure why ionic decided to include the bundled coded directly into its repository.. Anyway, as i said above, Ionic should either upgrade to version 5 (and fix the issue in swiper if it's still there in this version) or "freeze" the code in the repo (by disabling the build script that overwrites it from the swiper-package) and fix it there.
any news @liamdebeasi ?  The issue appears randomly in certain edge cases (probably the user interacting with the slider in the exact (wrong) moment) - i (and others) see it regularly in remote logs, it now also happened one time for me locally.

@liamdebeasi bringing this up again after the ionic 5 release since the outdated swiper 4.5.1 is still used there. Any plans how to deal with this? As i explained earlier, the bundled version can be fixed in ionic however the build script that regenerates the bundle would need to be disabled then. Last time i checked the current swiper version still has the bug, so another route would be to submit a PR there however that would entail updating ionic to swiper 5.

I discussed this with the team today, and we decided to upgrade to Swiper 5 in an upcoming feature release of Ionic Framework. We are currently focused on getting bug fixes out to support the Ionic Framework 5 release, but I did create a dev build with the latest Swiper. Can people try it out in their apps and let me know if they run into any issues? Thanks!
Ionic Angular
npm i<EMAIL_ADDRESS>Ionic React
npm i<EMAIL_ADDRESS>

I'm also seeing this from production.
@chriswep any guesses at what the user's experience is when they hit this bug?

@chriswep any guesses at what the user's experience is when they hit this bug?

@josh-m-sharpe i'm not 100% sure, however as far as i understand the code, i think the user wouldn't notice.
@liamdebeasi i think upgrading to swiper 5 is the right call however please note that the bug yet has to be fixed there (https://github.com/nolimits4web/swiper/issues/3275#issuecomment-562113342)

Ok good to know that the bug still exists there. Hopefully the Swiper dev can fix the issue there.

My project would mainly be using the Swiper so I need to use v5, so just wondering if I could use Swiper 5 without using the slide component?

Hi everyone,
Here is a dev build with ion-slides using Swiper 5. I believe the issue mentioned https://github.com/ionic-team/ionic/issues/20033#issue-532110805 is still a problem, but at least we will be on a version that is maintained.
Ionic Angular
npm i<EMAIL_ADDRESS>Ionic React
npm i<EMAIL_ADDRESS>Feel free to test it out, and provide any feedback if things look broken. Thanks!

@chriswep Do you happen to have a reproduction of the original issue without ion-slides? (I.e. just using regular Swiper) It might be worthwhile to open a new issue - does not seem like the developer is watching that closed thread anymore.

@liamdebeasi unfortunately i can't reproduce the issue, i only once saw the error popping up in dev. i (and apparently others) can only say that it consistently shows up in production logs.
i haven't used slider outside of ion-slides. as it seems the issue on the original repo is also in the context of ionic.
someone already opened a new issue on the swiper repo to draw attention: https://github.com/nolimits4web/swiper/issues/3520

@liamdebeasi I also cannot reproduce this locally, but have active, ongoing, production occurrences of the issue.  I say just merge swiper5 (we should have that anyways... right?) and I'll be the first to upgrade/test/deploy it.

Thanks for the feature request. This has been resolved via https://github.com/ionic-team/ionic/pull/20917 and will be available in an upcoming release of Ionic Framework.

@liamdebeasi can you do one more update to from swiper 5.3.6 to 5.3.7? It looks like this bug was actually fixed in that version: https://github.com/nolimits4web/swiper/commit/d89a73cd63f3c0b0cfc7cb88f9e3120ec3f19930

All set https://github.com/ionic-team/ionic/pull/21055.

My app is entirely based on this component so just wondering whats the ETA for the next release?

We do not comment on exact timing for releases since they can sometimes change, but we plan to have this update out soon.


feat(Bazel): Support Bazel 7.2.0

Please have a look at the individual commit messages for the details.
@haikoschol
@sschuberth
FYI

The BazelFunTest is failing in the PR but interestingly it is working for me locally.
I guess the difference is that locally I installed Bazel through Bazelisk (which is the recommended way - https://bazel.build/install/bazelisk) and only Bazelisk supports this version mechanism with .bazelversion:
https://github.com/bazelbuild/bazelisk?tab=readme-ov-file#how-does-bazelisk-know-which-bazel-version-to-run
Should ORT install Bazel with Bazelisk in the Docker image?

Should ORT install Bazel with Bazelisk in the Docker image?

That's basically the same question I was asking over here.

@haikoschol
There is a third possibility if I understand correctly https://github.com/bazelbuild/bazelisk?tab=readme-ov-file#how-does-bazelisk-know-which-bazel-version-to-run.
We could use Bazelisk to install the latest Bazel in the ORT Docker image.
We define USE_BAZEL_FALLBACK_VERSION=7.1.0 in Bazel package manager.
If a repository to scan has a .bazerversion file, it will be honored. If it does not have a bazelversion file, 7.1.0 would be used. like now.

We define USE_BAZEL_FALLBACK_VERSION=7.1.0 in Bazel package manager.

Nice, didn't see that one. That's the best option IMHO.

@haikoschol I updated the PR as we discussed
@fviernau Can you please have a look at my comments ?
Sebastian is in holidays so he won't review this PR anymore.

Would it be an option to not immplement the interface ?
Then you can just return a nullable type and the need to fail hard is gone.

While yes it could be a reasonable solution, I fell that, when https://github.com/oss-review-toolkit/ort/issues/8801 will be implemented, all registry services will have to return null if the package is not found to allow trying more registries.
For instance currently the RemoteBazelModuleRegistryService fails with a 404 if the package is not present on the registry.
Hence, would is be ok to change the interface then ?


feat: Subet - Several improvements

Description

Added UDTs for Peering & Subnet
Removed subnet property from VNET deployment
Upgraded API versions
Added IPv6 test and made required changes
Adjusted diverse properties (to simplify interface)
Added WAF-aligned prefix to peering name

Beware: This PR contains breaking changes due to the interface updates

Closes

#2020
#2689
#1025
#2860

Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[x] Azure Verified Module updates:

[ ] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

Hey @AlexanderSehr, is there any eta on this merge?

Hey @johnlokerse, I did reach out to @elbatane last week so she is aware. Will ping her again, to get this PR in asap