Add scripts and instructions for static iOS builds

Details
This is something I figured out today while testing push notifications – running this script will build and install the app as it exists in your current branch. Because it's a static, or "release" build, it is like any other app on your phone where it can be closed and reopened without any connection to the metro process.
For contrast:

npm run ios
After it builds, close the app.
The metro process will die.
Try to open the app again, and you'll get a red screen saying that it could not reach the metro server.

v.s:

npm run ios-static
After it builds and installs, close the app.
Try to open the app again, and it will reopen.

Fixed Issues
Fixes n/a
Tests

Run npm i && npm run ios-static
The app should build and install.
Close the app.
Reopen the app. It should reopen and work.

QA Steps
None.
Tested On

[ ] Web
[ ] Mobile Web
[ ] Desktop
[x] iOS
[ ] Android

Oops, ios-deploy should be a dev dependency

@roryabraham Travis is very unhappy...

Oh ... it looks like this change means that all our GHA that run npm install would need to run on macOS runners, because ios-deploy depends on Xcode. Which means that I am going to close this. It's really just mean to be a convenience.


Is there an api to query ListOrchestratorsWithHttpMessagesAsync

Library name and version
Azure.ResourceManager.ContainerService 1.1.0
Query/Question
Hi,
I am trying to migrate this existing code from Microsoft.Azure.Management.ContainerService to Azure.ResourceManager.ContainerService package.
using (var containerServiceClient = new ContainerServiceClient(credentials))
{
    var result = await containerServiceClient.ContainerServices.ListOrchestratorsWithHttpMessagesAsync("westus2", "managedClusters"); // <-- This call

    //...
}

The API above calls subscriptions/{subscriptionId}/providers/Microsoft.ContainerService/locations/westus2/orchestrators?resource-type= managedClusters
I looked in the source code, and I cannot find anything there, help!
Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @johnsimons, currently the API version of Azure.ResourceManager.ContainerService is 2022-09-01 and unfortunately this API version doesn't support the request that you want. I don't have too much context here about why they removed this operation, so I will marked this issue as needs-team-attention to see if we can get some feedback from service team.


Upcoming Features and Future Directions of Chat Arena

Firstly, we would like to express our gratitude to all the contributors and users of Chat Arena. Over the last month, our community has grown significantly and we have received numerous feature requests, bug reports, and contributions. It is exciting to see the community thriving and passionate about our project.
Given the influx of feature requests, we believe it would be beneficial to list the most common ones we've received and share some insight about our future directions.
Common Feature Requests
Here are some of the most frequently requested features that we are considering:

[ ] OpenAI and Anthropic async query API support
[ ] Batch experiment support to run multiple games concurrently
[ ] Langchain backend support
[ ] Bard backend support

Future Directions
We're focusing on the following directions:

[ ] More game environments
[ ] Improved stability and reliability
[ ] Extensibility to more backends
[ ] Benchmarking different LLMs with in game environments
[ ] Improved documentation (homepage, notebook example)
[ ] Improved Web UI

We encourage you to share your thoughts on these upcoming features and future directions. It's through your feedback and contributions that Chat Arena can continue to grow and evolve.
Thank you again for your support and for being part of our growing community.
Best,
Chat Arena Team
Last updated: May 2023

Langchain backend support is a much needed thing . I believe we should work in that direction for sure.


Add a block intrinsic for Hash#fetch

Motivation

Performance, avoiding a call through the VM when a block is given to Hash#fetch.
Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_Kh33p0m0UnJjHS
→ https://go/builds/bui_Kh33mPbxhXlYqR


gettext: 0.19.6 -> 0.19.7

now using libxml2 instead of expat
Replaces #13072
Needs testing, still compiling on my machine.

It compiles fine but I'm not sure what to text with the XML version

gtk3 build uses it. Maybe @lethalman remembers what was the problem in there.

I made some cleanup which I was at it.
gettext, gettextWithXML and gtk3 compile fine.
After installing gettextWithXML and running ldd $(which gettext) I only get:
$ readlink $(which gettext)
/nix/store/wbpykariknr1qqdyl29f33gmj6yp1dpg-gettext-0.19.7-with-xml/bin/gettext
$ ldd $(which gettext)
	linux-vdso.so.1 (0x00007ffe517c0000)
	libc.so.6 => /nix/store/3xnbb8hkzv8nmndssknx413sd217bj56-glibc-2.21/lib/libc.so.6 (0x00007f19bb1ab000)
	/nix/store/3xnbb8hkzv8nmndssknx413sd217bj56-glibc-2.21/lib/ld-linux-x86-64.so.2 (0x00007f19bb54a000)

Well, it seems not detected, as it's not in closure and log says:
checking whether included libxml is requested... no
checking for libxml... no
checking whether to use the included libxml... yes

But I think it's OK for now to let it use bundled libxml. I think this gettext+XML stuff is all used only during build-time anyway.

I bet it needs pkgconfig to find it. I agree, let's use the bundled version for now and make gettextWithExpat a deprecated alias.

Compiling gtk3 to see if it works. If you have an idea of a gtk3 program that uses locales heavily let me know.

Compiled sonata which uses gtk3, ran LANG=de_DE.utf8 LC_CTYPE= sonata and got proper German in the UI. I'm not 100% sure this exercises gettext+xml or how to find a program that does.
Is this good enough for testing ?

I think so. IIRC expat was only needed during build time.

Okay. How do you usually merge a mass rebuild, is there anything different to do ?

They get merged to staging to save master from mass rebuilds without binaries.

Is staging merged back into master later or do I first merge the branch into staging manually, wait for hydra to catch-up and then merge this branch into master ?

Staging is merged to master periodically, after Hydra finishes (most) builds.

Just merged into staging manually: https://github.com/NixOS/nixpkgs/compare/8bfc5d7e724d...f05bc0444aac

Then we should close this.

It would be nice to have this in master as soon as possible -- we have git rebase broken on unstable channel now...

agreed. Still 26k packages to go: https://hydra.nixos.org/jobset/nixpkgs/staging

As a workaround with less rebuilding needed I've reverted patch c38eb526d59e2fcc985a0459e94a6ad30b1d3706 locally (changing it a bit to make sure when actual fixes hit master rebase will yell at me to skip it).

Yeah it might make sense to bundle these changes

Well, we might just wait for Hydra to finish most builds now -- reverting that patch is a good enough workaround for those who need it. I hope next unstable channel won't be so delayed ~_^.

Yeah next time. Hopefully the hydra+s3 stuff goes out soon as well so we can have faster builds.

...wait, I think git rebase might actually have worked for me, just spewed error messages and haven't shown other ones. Let me check...

Yes, that was a false call. git pull --rebase has shown only errors to me so I assumed that it's broken, but I have just been atop the latest master already. Sorry for the noise!


Detect PowerShell 7 (PWSH) more smart!

I have installed PowerShell 7 from the official msix package. But Windows Terminal cannot find it directly.
Please change the default detection path from "C:\Program Files\PowerShell\7-preview\pwsh.exe" to "pwsh.exe" or smart way just like Azure Cloud Shell.

pwsh.exe works perfectly:

We fixed this in 0.5. You might want to refresh your profiles.json to take advantage of things we fixed in the past year.
We can’t change the path in the file you already have because it’s your profiles file, we aren’t ever going to change what’s inside your existing profiles because we can’t know what you want.

To fix this easily without deleting your settings, just delete your whole powershell core profile... or just delete the icon and commandline!

We fixed this in 0.5. You might want to refresh your profiles.json to take advantage of things we fixed in the past year.
We can’t change the path in the file you already have because it’s your profiles file, we aren’t ever going to change what’s inside your existing profiles because we can’t know what you want.

Thanks for the quick response. I think I need more explanation.
I deleted the old profile according to your method and it does work. "Source": "Windows.Terminal.PowershellCore" is exactly what I want.
But in fact, I installed Windows 10 completely new yesterday, and downloaded Windows Terminal Preview from Microsoft Store. I saw his latest version 0.10.781.0, but why the problem I mentioned still happened?

Wow, that is strange! Have you ever used Terminal before? If so: we had a bug where sometimes it would copy a very old settings file from an old installation. :smile:

how do you get it to identify an install of PowerShell 7 that is not the default install. I put powershell 7, among other things, in a different folder for a variety of reasons. Adding it manually into Windows Terminal settings.JSON, quite frankly, is a bit confusing.

@dweddig01 for the record, adding a profile should be as simple as adding a blob like:
        {
            "name": "The name of your profile",
            "commandline": "C:\\the\\full\\path\\to\\pwsh.exe"
        },

to the list in the profiles object in the settings.json file. Of course, replace C:\\the\\full\\path\\to\\pwsh.exe with whatever the actual path is. Make sure to escape the backslashes in any paths! There are plenty of other settings to use, but that should be enough to get started ☺

NONCONFIDENTIAL // EXTERNAL
Thank you for the reply. I did end up finding out it was due to “source” being defined. Apparently “source” should only be used when you have default installs. Once I removed “source” it started working as expected. I have not circled back to the docs to see if I just missed that and should have known. It is weird though.
Anyway, thanks again!
FEDERAL RESERVE BANK OF MINNEAPOLIS
Pursuing an economy that works for all of us
Dennis Weddig
Platform Engineer, Technology and Risk Group
W<PHONE_NUMBER>
@minneapolisfed https://twitter.com/minneapolisfed  I  minneapolisfed.orghttps://www.minneapolisfed.org/
“01000111 01000001 01001101 01000101 00100000 01001111 01001110 00100001”
From: Mike Griese @.>
Sent: Tuesday, April 13, 2021 3:32 PM
To: microsoft/terminal @.>
Cc: Weddig, Dennis @.>; Mention @.>
Subject: [External] Re: [microsoft/terminal] Detect PowerShell 7 (PWSH) more smart! (#5411)
NONCONFIDENTIAL // EXTERNAL
PLEASE NOTE: This email is not from a Federal Reserve address.
Do not click on suspicious links. Do not give out personal or bank information to unknown senders.
@dweddig01https://github.com/dweddig01 for the record, adding a profile should be as simple as adding a blob like:
    {

        "name": "The name of your profile",

        "commandline": "C:\\the\\full\\path\\to\\pwsh.exe"

    },

to the list in the profiles object in the settings.json file. Of course, replace C:\the\full\path\to\pwsh.exe with whatever the actual path is. Make sure to escape the backslashes in any paths! There are plenty of other settings to use, but that should be enough to get started ☺
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/5411#issuecomment-819033345, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIN5D2EYCRFKUWQ4QNDTRXLTISS3HANCNFSM4MLPD73A.
This e-mail message, including attachments, is for the sole use of the intended recipient(s) and may contain confidential or proprietary information.  If you are not the intended recipient, immediately contact the sender by reply e-mail and destroy all copies of the original message.

In settings.json in C:\Users\User\AppData\Local\Packages\Microsoft.WindowsTerminal_8wekyb3d8bbwe\LocalState, I have "commandline": "%SystemRoot%\\System32\\WindowsPowerShell\\v1.0\\powershell.exe", "name": "Windows PowerShell". It looks for the systemwide powershell.exe , which seems good. Why does it nevertheless give the error as mentioned in this issue  and also on my pc?

Can you confirm the error message?
Please note: this issue is more than two years old. I would encourage you to file a new issue and fill out the bug reporting template.


A server with the specified hostname could not be found.

Hi,
Everything worked fine, but from few days, I have this error when I'm trying to upload a photo :
Object {
  "headers": Object {},
  "status": 0,
  "text": "A server with the specified hostname could not be found.",
}

My options are greats, what happen ?
thanks

same here, when I try to send into S3 compatible

same here too.


Launcher wIll not launch game

After pressing Launch Game button, I get an error message box:
"Error launching game"
"TypeError: Cannot set properties of undefined (setting 'lock_gui')
I rebooted; same issue.
I uninstalled Launcher and reinstalled; same issue.
Was working last night with seemingly normal startup and normal shutdown.

@davestockton Apologies, I missed this issue. Do you still experience this problem?


wg-bond: 0.2.0 -> 0.2.1

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I wouldn't do that. 0.2.1 breaks a lot of stuff, and I was better off not releasing it at all

Duly noted. Closing.


Migrate to new charging engine

The February OpenEye toolkit release introduces OEAssignCharges and OEAssignPartialCharges is basically deprecated (see https://docs.eyesopen.com/toolkits/python/quacpactk/OEProtonFunctions/OEAssignCharges.html#OEProton::OEAssignCharges) so we should migrate at some point.

How is this used in the "current best practices" manner?

@jchodera - I think it's a drop-in-replacement for the old one, but it's possible @cbayly13 has built some of his conformer stuff into it so that we can remove some of the machinery we had to use with the old one. I have this on the list to bring up in my next call with him but have not yet discussed it with him.

See https://github.com/choderalab/openmoltools/issues/252 -- we need to NOT do this for now as there is no partial bond order access in the new charging engine.


[IMPROVEMENT] Pre-condition check required for the snapshot purge

Problem
Snapshot Purge command is used to release space from the disk, but the purge command itself will execute the coalescing operation which will temporarily increase the disk space usage before removing the deleted snapshots. The temporary increase in disk space usage might result in the disk running out of space.
Describe the solution you'd like
The command should error out if there isn't enough space on the disk for the potential coalescing operation.
The logic should be mostly inside the engine:

Only the engine knows exactly how much space is needed before coalescing.
If the snapshot purge failed as a part of snapshot deletion, the information should be pass to the user via the manager and CRD resource (e.g. the future snapshot CR), and the user can safely retry after expanding the disk.

Also, when we have the feature sharding the volume across different disks, any failure in one disk should result in the whole operation failure. Make sure we won't have inconsistent states across the disks regarding the snapshots.
Describe alternatives you've considered
Reserve a big enough space and allow the manager to detect the potential out-of-space problem.
The downside is the manager needs to have logic built in to understand how the snapshot purge works internally.
Note
For e2e test, we can use Longhorn on Longhorn to simulate the disk full situation.

Hey team! Please add your planning poker estimate with ZenHub @jenting @PhanLe1010 @shuo-wu @joshimoo

ref: #3613

For snapshot purge, there are 2 scenarios:

The regular case. The removing snapshot is not the parent of the volume head and is removable. Here is the flow:

Copy data from the child snapshot (newer one) to the parent snapshot (older one, and it's also the snapshot being removed) ==> The cause of the extra space requirement
Replace the parent snapshot with the child snapshot:

Remove the child snapshot
Rename the parent snapshot to the child snapshot name

The special case. The removing snapshot is the direct parent of the volume head. For this case, Longhorn will blindly punch holes to the snapshot file based on the fiemap of the volume head. There is no extra space requirement.

After checking current purge command, SnapshotPurge API directly calls the engine binary to do purge, i.e. /.../longhorn  snapshot purge .... It looks there is no way to know

Disks' storage information, e.g. storage maximum, storage available and etc.
Other running snapshot purge tasks

Instead, we can do the pre-condition check in the snapshot controller (https://github.com/longhorn/longhorn/issues/3144 and https://github.com/longhorn/longhorn-manager/pull/1304), and then the logic is
Before calling SnapshotPurge

Find out the the disks/nodes participating in this snapshot purge
Then, we check each disk/node one by oneFor each replica in each disk/node {

      totalUsedSize = 0

      For each replica, find out its engine {
          usedSize = sum(each snapshot size) +(volume nominal size)

          If the snapshot of the replica is purging {
              usedSize += (reserved size for purging)
          }
    }
    if the there is enough storage for snapshot purge; insufficient space {
        return fail
    }
}

But, the first version of snapshot CRD do not change the HTTP API, so the implementation cannot take effect until the next version.
@shuo-wu @innobead @PhanLe1010 WDYT?

Extra space for a volume purge can be Min(VolumeSpecSize, The Actual size of the largest snapshot (or we can directly use volume actual size for convenience))
Used size of a disk can be directly read from the disk status.
This is a general check no matter if the caller is SnapshotController or not.

After discussing with Shuo, the general check will be added in both the API path and the snapshot controller.

Extra space for a volume purge can be Min(VolumeSpecSize, The Actual size of the largest snapshot (or we can directly use volume actual size for convenience))

For Min(VolumeSpecSize, The Actual size of the largest snapshot), is it possible that The Actual size of the largest snapshot is larger than VolumeSpecSize?

Used size of a disk can be directly read from the disk status.
This is a general check no matter if the caller is SnapshotController or not.

After discussing with @shuo-wu and @shuo-wu,  the async mechanism between engine and manager introduces uncertainties, e.g. we cannot know which snapshot is purging. The uncertainties may lead to the over-estimation of the reserved space and reject the purge request.
We can revisit the task after the snapshot CRD and the snapshot APIs refactoring  are ready.

Part of #6563


[Feature Request] Update the list of tags adding new ones

what the title says

Adding my voice to this. Not sure why the tag list isn't set to automatically update whenever Anilist refreshes their list. There are tons of tags missing, and it's ridiculous. Some of the ones I can't find have definitely been on Anilist for years, so I can't understand why they are not in this app.

@rev0lgenX if it still.not possible to auto-add them. what about running a simple bot to check a lot of popular entries in anilist website and gather all their tags? then compare them with the default list of anilist and see what is missing. you can modify the app itself to do so (maybe). OR you could modify the app itself to all of us so when users load an entry, tags will be loaded too, and in that moment you should make the app compare those tags with the default list, if a tag is not in the list then make it send the information somewhere else on the internet. just a suggestion. or you could add the tags manually

.


Loading content after changing renderer to "code" stucks

Describe the bug
Reformatting of pom.xml to code stucks. (waited > 15mins)
To Reproduce
Steps to reproduce the behavior:

Go to a directory with a pom.xml (e.g. this one https://github.com/jeremylong/DependencyCheck/blob/main/pom.xml)
Open file with "less pom.xml"
Click on "line settings" icon
Change Renderer from "terminal" to "code"

Expected behavior
The file content will be rendered in colors.
Screenshots

Desktop (please complete the following information):

OS: MacOS x64
Version: v0.5.0

Ah, the problem is that your less command never terminates and the code renderer only renders content after the command has finished running.  So, you can either use a command that will just dump the whole file and terminate like cat pom.xml, or what I'd recommend is just running codeedit pom.xml or codeview pom.xml which will avoid any weird character / terminal encoding issues.


Test suite improvements

Cache chef client keys between tests
Use a Logger class for test output rather than puts and pp
Add more diagnostic log messages throughout the test suite

Here is an example of the full debug log output:
    when the node nacks the quorum
[2016-07-27 11:49:28 +0100][INFO ]: Starting client DONKEY
[2016-07-27 11:49:28 +0100][INFO ]: Waiting 5 seconds for ["DONKEY"] to come online
[2016-07-27 11:49:28 +0100][DEBUG]: All nodes now online!
[2016-07-27 11:49:28 +0100][INFO ]: Waiting 30 seconds for ["DONKEY"] to enter availability == available
[2016-07-27 11:49:28 +0100][DEBUG]: Got job_stat_change: {:state=>:idle, :job_id=>nil, :command=>nil}
[2016-07-27 11:49:28 +0100][DEBUG]: Node DONKEY has availability = available
[2016-07-27 11:49:28 +0100][INFO ]: Waiting 30 seconds for ["DONKEY"] to enter availability == available
[2016-07-27 11:49:28 +0100][DEBUG]: Node DONKEY has availability = available
[2016-07-27T11:49:28+01:00] ERROR: [DONKEY] Received commit e5af89913daa803961f933b9e13df501, but command 'bad command' is not in the whitelist!
[2016-07-27 11:49:28 +0100][INFO ]: Waiting 30 seconds for https://api.chef-server.dev/organizations/pedant_testorg_api_15583/pushy/jobs/e5af89913daa803961f933b9e13df501 to finish
[2016-07-27 11:49:29 +0100][DEBUG]: Validating events. Expected 3: [#<struct Event name=:start, id="job-1", json={"timestamp"=>"2016-07-27 10:49:28.898527Z", "command"=>"bad command", "run_timeout"=>4, "quorum"=>1, "node_count"=>1, "user"=>"pedant_admin_user_api_15583", "capture"=>false}>, #<struct Event name=:quorum_vote, id="job-2", json={"timestamp"=>"2016-07-27 10:49:28.910443Z", "node"=>"DONKEY", "status"=>"failure"}>, #<struct Event name=:job_complete, id="job-3", json={"timestamp"=>"2016-07-27 10:49:28.910542Z", "status"=>"quorum_failed"}>]
[2016-07-27 11:49:29 +0100][DEBUG]: Got job_stat_change: {:state=>:idle, :job_id=>nil, :command=>nil}
[2016-07-27 11:49:30 +0100][INFO ]: Waiting 10 seconds for ["DONKEY"] to enter status == offline
[2016-07-27 11:49:30 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:30 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:30 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:30 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:31 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:31 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:31 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:31 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:32 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:32 +0100][DEBUG]: Node DONKEY has status = online
[2016-07-27 11:49:37 +0100][DEBUG]: Node DONKEY has status = offline
      the events should be: start,quorum_vote(failure),job_complete(quorum_failed)

I think we'll turn the loglevel to WARN before shipping, but my hope is that having timestamps and more structured messages will help us debug the unstable tests.


Update mdoc, sbt-mdoc to 2.2.15

Updates

org.scalameta:mdoc
org.scalameta:sbt-mdoc

from 2.2.14 to 2.2.15.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, semver-patch

Superseded by #83.

Superseded by #83.


Add /debug command

Most people tend to skip over the debugging part and ask questions that can be solved by themselves if they knew how to debug.
/debug should send a link: https://developers.google.com/web/tools/chrome-devtools/javascript

Just now getting back to this, I'm not opposed. If you'd like, you can make the PR.

Okay, Will make a PR on this. Thanks @twalton83


Qiskit-ify Furo theme: remove dropshadow from tables

The Furo data table generally look ok except for the drop shadow. The IBM carbon design system is a flat design system, so we shouldn't be using drop shadows for any components.

Another point that i'm not sure is as important: IBM tables don't have vertical lines only horizontal ones (see data table guidance here), we should consider removing the vertical lines

update: I realised we also have drop shadows cards and dropdowns, these should also use the flat carbon design system. Additionally, these 2 components have slightly rounded radius, the should have border-radius:0

same for admonitions. Dropshadow should be removed and border radius should be 0

these elements are "toasts", which means they are designed to pop up in front of the screen and the user can close them. The drop shadow here is an intentional design choice to show there is depth to the page and that there may be stuff behind it that the user might want to uncover by closing the toast.


Problem with .form-group-lg from bootsrap.css

Border from bootsrap css overlap border from admin-lte
.form-group-lg .form-control {
    height: 46px;
    padding: 10px 16px;
    font-size: 18px;
    line-height: 1.3333333;
    border-radius: 6px;
}

By default other inputs admin-lte have  border-radius: 0;
.form-control {
    border-radius: 0;
    box-shadow: none;
    border-color: #d2d6de;
}

Code for reproducing

     Year

          <input type="text" id="zoneparam-year" class="form-control">

Just wanted to add that this bug is a reason for overlapping forms when using the FieldList together with a FormField in WTForms

Why did you close?


@NonCPS annotation inside Jenkinsfile

Hi,
This is more of a question than an issue, and I couldn't find a better place to ask.
In some of our Jenkinsfiles we use functions outside of node() block,  I know it's not a best practice, but anyway it looks like this:
Jenkinsfile
node('ci'){...)
@NonCPS
String someRegexFn(String){ ... )

is there any way to test this function in jenkins-spock without having to add:
import com.cloudbees.groovy.cps.NonCPS;
To the Jenkinsfile?
I guess it is a similar issue to https://github.com/ExpediaGroup/jenkins-spock/issues/21#issuecomment-453290998

At the moment, I cannot think of such a way.
We'd probably have to modify the implementation of getPipelineScriptForTest(...) to automatically inject imports for all of the classes that Jenkins does. That would be a good thing for jenkins-spock to do, for completeness' and correctness' sake.


Update Database diagram

Update https://app.diagrams.net/#G1j2Vpq-zY63TT6Da3mUZLmzRTN_1TD3P5 with all recent changes.

I agree. Lets do it after #612 is solved

The old diagram is deprecated, long live the new diagram https://drive.google.com/file/d/1bKvqEKr6JzPWryVg-vYudQy_4KvuzHS9/view?usp=sharing
@davidzbiral probably take it and back up it on the DISSINET drive somewhere.


Fix #3134 - search_garbage_collect command does not work without wagtailsearchpromotions contrib

Changes include:

fixing the FieldError exception that can occur as described in #3134
adding tests for the "search_garbage_collect" command

Hi @ellmetha - this looks really good, thanks!
Unfortunately I don't think it's possible to modify the INSTALLED_APPS list using override_settings - if I revert the fix to wagtailsearch/models.py, test_garbage_collect_command_works_without_promoted_searches still passes (indicating that it's still behaving as if wagtailsearchpromotions is installed). As I understand it, override_settings can only affect settings that are read from django.conf.settings after override_settings has been called - and, of course, by that time INSTALLED_APPS has already been processed (and thus the editors_picks property already exists on the Query model).
Since I can't see a good way of testing the behaviour without wagtailsearchpromotions - other than having a completely separate test runner just for this one test - I'll merge this PR with this test omitted.

Merged in 51b4f0f53d93a69f3beaba67e3211cd001ff4087 - thanks!

Cool! Actually I think that we could've modified the test to properly test how the command behaves without wagtailsearchpromotions. We could've used the Django TestCase.available_apps attribute. ;-) I can provide a new PR adding the test if you want.

The list of "gotchas" at https://docs.djangoproject.com/en/1.10/topics/testing/advanced/#django.test.TransactionTestCase.available_apps makes me think that available_apps is going to be more of a liability than a benefit... also, from the description there it sounds like it's only designed to optimise away some database operations, rather than making any deep changes to the relations between apps/models. I think we're fine without a test here, really.

The available_apps attribute has an impact on the registry and the models' _meta fields cache. It turns out that if you remove the wagtailsearchpromotions app from the installed apps (using the available_apps attribute) the editors_picks fields won't be returned by the Query._meta.get_fields method. This would allow to test the desired behaviour.
I note that the test is not needed, though. :smile:


Use Godot4 Time in LocalTime

Describe What
On Godot3 i use OS time functions where are replaced by
https://docs.godotengine.org/en/latest/classes/class_time.html?highlight=Time
Describe Why
It must adapt to a new class of time that offers new functions.

done


Could modules be deployed in separate hosts?

Previously, the Collector, Query and Web are separate services and they can be deployed on different hosts. Now from the Readme, the quickest way to run is by java -jar command. I wonder if I can do the same thing in old style since I don't want all the modules are on the same machine. Thanks!~

Yes, you can java -jar 3 instances if you want to, on 3 servers, with one instance each. And then configure the instances to only behave as one of the "roles". (I think it's possible to disable roles)

Thanks for answering!

When we released the new server, we clarified this topic on the mailing list, too, which you are free to join, too!  https://groups.google.com/forum/#!topic/zipkin-user/VtyR3GD-0Fo

@eirslett @adriancole Thanks a lot!~


How can encrypt data that is already in the database?

 So my question is, how can I encrypt this fields with the same config as the plugin but without using it?

There isn't a seamless way to migrate data built into the library.  You'll need to build a migration yourself.  e.g. Maybe something like add a column called lastNameEncrypted, which is mapped to the encrypted field.  Then write some code that will populate the encrypted property from the unencrypted property's value and save the entity.

There was so much fields that what I did was to make a "temporal" table were I clone de real table. Ones this was done I truncate original table and make the fields as encrypt type fields. Then I populate the original with the temp one. And now all is working just fine!
Thanks for all, your plugin is great. (You can close de issue)

great - good to hear.


No need to poll with firebase

You don't need to poll. HN API uses Firebase. You can use the Firebase JS lib that uses a websocket connection. Firebase has methods to observe a node, like v0/topstories and v0/newstories for changes. See documentation for .on('value', callback) at doc
A simple proof of concept that I made to demonstrate this: jsfiddle
It has detection of change of position of an article in the top 500 list and detection of additions in the new stories list.

Thanks! Will see.

Done.


Documentation: Expand on installing under Docker

Expand on docker example to make it easier to persist configuration changes.
The current example is not a good example of 'installing' onto Docker because it is not at all suitable for production use.
There is no clear documentation elsewhere that helps users find this.  Adding it here helps point users in the right direction.
See issue #451

I would be happy to expand on this a bit as well. I am building an API Umbrella Swarm stack which I will document and include.

@vegasbrianc I would also be interested in knowledge on clustering API Umbrella

@vegasbrianc same, if there are any advances towards a docker stack, I'd be very much interested!


fix check if wal need recovery is not correct

Proposed changes

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


General improvements: Refactors & Docs.

These changes are focused on refactoring changes and documentation improvements.

Added a test and coverage Make commands.
Added some possible elements to improve in the ToDo section of the Readme.
[Refactor]: In the range() is not necessary to define 0 as the start position.
Defined a Protocol for the Block classes, the idea is to define the interface, the methods that need to be implemented by the Block classes.
[Refactor]: in general, is not necessary to compare with True (if X == True: ...), just do directly if X: ...
[Refactor]: For each Block class, I create a draw method. Now with that is not necessary the function draw_block_to_wait()
[Refactor]: Move the erase_sound and break_sound sound elements, instead of passing those as parameters, I moved to the Tetris class where those are used.
[Refactor]: Rename some variables, to a more descriptive name.

P.D: Sorry if this PR has too many changes.

Thank you so much!! There is nothing to be sorry.
Your PR is very helpful for me to study Python.
I got to know a lot of things that I didn't know, including coverage.
Thank you again for letting me know the parts I need to fix! 😄


deltas() function, closes #437

Shouldn't this have special handling for hues?

Shouldn't this have special handling for hues?

Yes, as I said earlier

I envisioned this as a low-level utility function, if users want to make this optimization for hue it's trivial to do so on the deltas returned.
That said, it should probably constrain the ΔH returned (possibly by also preserving the original).

Ok, I’ve made several improvements:

A hue interpolation setting, with "shorter" being the default.
Fixed a bunch of issues around none handling, and added more tests.

We could have another option for scaling the Δh by taking C/L into account, but not sure how that would work in a generic way.

Looks good to me. I would fix lint issues. related to the files in questions.


Support in Bash on Windows

I've been derping it up with PowerShell so I'd rather just run EmPyre in Bash on Windows instead of Empire. Why don't you support that yet?

Meh. I'll use pupy RAT. It's better than Empire anyway...


fix(backend-core): New Resource class structure

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/clerk-expo
[x] @clerk/backend-core
[ ] @clerk/clerk-sdk-node
[ ] @clerk/edge
[ ] build/tooling/chore

Description

[x] npm test runs as expected.
[x] npm run build runs as expected.

This is the example response object from the new and old versions, which is identical.

The input I would like is if we should consider this change breaking. The clerk-sdk-node re-exports these classes but we have no indication how they are to be used and actually they don't even have any methods except fromJSON attached to them. This is used only internally. WDYT ?


Allowing all certificates for local environment.

Update:

Boolean flag validateCertificates is introduced.
createSSLFactory method is modified to handle certificates of local & non-local environment. For now local configuration are set up.

@fantavlik,
I will make scope change from protected to public. But if we are thinking of removing above line of setting default SSLSockerFactory then it will give an error (unable to find valid certification path to requested target) if we are using https connection.
For now, changes are pushed.

Verified with the team that the error being thrown when validation is turned on cannot be caught for testing purposes and makes running other tests difficult/impossible but manual testing confirms that validation is being performed/erroring out when validation is turned on in CI (as expected because self signed cert is being used)


Format on save removes multiple cursors and moves main cursor

VSCode Version: Latest
OS Version: Windows 10 Home

Steps to Reproduce:

Is it possible to keep the cursor count and positions? This is a huge time-wasting factor for me.

This applies to all operations (also from extensions) that are performed on save. What is the recommended way to preserve the cursor position (ideally in all open and editors on a save all operation)?

fyi @jrieken This could be done if format on save would call editor.executeEdits and avoid the highly tuned EditOperationsCommand from formatCommand.ts that also runs with format on type.

Bump - this would be nice to have! (Sublime does this)


Migrate Accordion web component to now render using LitElement

Jira
http://vjira2:8080/browse/BDS-2125
Summary
Migrates our existing Accordion web component to now render using LitElement. Addresses catastrophic SSR hydration issues that can sometimes occur when asynchronously loading the component's JS / being lazily rendered (for example, as seen here).
Details
As far as the Accordion component goes, this is more or less a 1:1 upgrade with only minor logic tweaks required, plus a new interaction part of the existing Jest test added as well.
Note: As an added bonus, updating the spacing prop (when an accordion item is already open -- ex. going from small to large spacing, or vice-versa) now correctly recalculates the height of the open panel which doesn't happen in the current version...
How to test

Confirm that with these upgrades, the Accordion component renders correctly (despite it now being lazily loaded on this branch + renders using <ssr-keep>.

Also confirm that contextual props (ex. single, spacing, etc) also trigger the Accordion Items to re-render as expected.

I unfortunately don't have enough knowledge of the specifics of litHtml and litElement to give a meaningful code review of this one.  So I'm not going to approve since I think that needs someone else, but what I do understand seems reasonable, so I'll give it a 👍 instead.

This PR also splits up our @bolt/element decorators into separate files
It looks to me like this part of the PR description is not longer true (splitting up the decorators was handled in some other PR and then merged back into this one).

PR was released with v2.23.0


Date.toLocaleDateString() ignores country settings on Android

[x] Provide a minimal code snippet / rnplay example that reproduces the bug.

    let testDate = Date.UTC(2016, 6, 23, 10, 0, 0, 0);
    let output = new Date(testDate).toLocaleDateString();
    return         <Text style={styles.text}>{output}</Text>;

https://rnplay.org/apps/bGMmOA

[x] Provide screenshots where appropriate

go to phone settings and change the language to Deutsch (Deutschland)

launch the app

Expected:
23.06.2016
Actual:
07/23/16

[x] What's the version of React Native you're using?

0.28

[x] Does this occur on iOS, Android or both?

only Android, on iOS this works fine.

[x] Are you using Mac, Linux or Windows?

Mac

Is there any progress on this? I also have this issue on 0.30.

+1

I've got same issue

The issue is also reproducable with 0.40.0

same issue

Still have this issue with 0.43.3

this issue is really annoying and i can't understand how this issue can be open since nearly a year.
sure there are tons of date-format-npm-packages out there, but that's so unnecessary.
btw this is related: http://stackoverflow.com/questions/42218022/react-native-reacts-differently-when-not-debugging-remotely

Still reproducible with 0.46.3.

+1

This is because JSC on Android doesn't include the locale settings. It would increase the size of the APK by ~2MB so it's not clear whether we want to include this.

imho the issue should stay open, as far as i know it is still a problem in the latest release, and the issue is inactive for a while because people accept it as problem and use workarounds.

Is there a PR open that attempts to address the issue?

now seems use new Date().toLocaleDateString() crash directly in android huawei phone with 1.0.0, while it is ok in ios,
use new Date().toString() ok

Same here with @jayzhou215. Tried with physical Samsung Galaxy Note 4, Android Studio and Genymotion emulated  Google Pixel 7.1.0 API 25.

This definitely shouldn't be closed.
new Date().toLocaleDateString() still isn't working correctly, and is really a basic feature.

I also have the same problem. Using react-native version 0.55.4


Feature request: pragmas and replacement of build-config

Since the build-config has been removed I feel like there has been no proper replacement for some of its features, or I do not know about it.
Specifically it used to provide great customization possibilities in interactive mode via the  options line.
Using the latest F* version I can no longer (in interactive mode):

set a custom timeout for each file
set some debugging features (e.g. --print_implicits) etc.
is the --verify_module X options automatically set to the current file in interactive mode ?

The auto-deps feature is great for file inclusion but the build-config was not useful only for that but also to pass other options.
Here is typically how I progress into a file, using a previous version of F* and build configs:

I use the emacs interactive mode
I verify one function  at a time with a small timeout
When functions take more time to prove, I insert asserts or cuts and incrementally increase the timeout to test whether it really cannot prove something or merely needs more time
I put an admit(); at the very beginning of the functions that as just been proven an move to the next.
It is the most efficient pattern I have experienced.

Hence is it possible to:

restore the build-config for options so that the timeout and other options can be modified ?
otherwise provide a pragma to at least set custom timeout and options per file in interactive mode, set at the top of the file just like the build-config used to work ? AFAIK only the fuel is modifiable through the #set-options pragma in the file
I would love to have a pragma like #lax-on and #lax-off to be able to lax typecheck blocks while in verification mode, the same way it works with the --verify_module option typically. What already exists for module-per-module verification would be great to have function-per-function inside a module so that one does not have to assume or to put admit() in every function that has already been verified when a modification is made at the beginning of a partially verified file in interactive mode. It would also be super useful in CLI mode.

Thanks

+100
I really think that we lack those features to be efficient in producing a large amount of verify code.

I think the right way to bring back this functionality is via pragmas. There is already an issue on allowing any non-security-critical option to be set in pragmas (#439).

Such pragmas would be great.
It is too bad that introducing auto-deps led to removing useful features. It feels like taking a step back for the interactive mode.

is the --verify_module X options automatically set to the current file in interactive mode ?

I don't think so but I think it's a valid feature request. Thoughts?

@jkzinzindohoue  The functionality is still available for interactive mode via the fstar-subp-prover-args variable of the emacs mode.
I agree that controlling all non-security-critical compiler options via pragmas should be supported.
Adding --verify_module X for the current file in interactive mode is a good idea.

cf. 0b4d9b8: allowing reset-options with arguments to control parameters like z3timeout via pragmas - Nikhil Swamy
16a2f8b: snap - Nikhil Swamy
e2bd1eb: merging and re-snap - Nikhil Swamy
36ae901: removing spurious #set-options "--z3timeout n" in various places ... these were not being honored anyway. if you want to set the timeout, you need to use #reset-options "--z3timeout n" - Nikhil Swamy


[BUG] Volumes stuck in "in progress"/attaching/detaching state Longhorn 1.6.2

Describe the bug
Recently upgraded from Longhorn 1.6.0 to Longhorn 1.6.2 after which I started experiencing issues attaching nodes, also seeing issues with volumes completing their upgrade/running with a high amount of replicas(old/new).
To Reproduce
Start a new pod that requires a volume to be provided by Longhorn
Expected behavior
I expect to launch a pod and a volume attach to said pod from Longhorn.
Support bundle for troubleshooting
Support Bundle emailed referencing this issue #.
Environment

Longhorn version: 1.6.2
Impacted volume (PV): Any new volume joining the cluster. For example: pvc-33a6a321-9b90-41ad-986e-377680aed2c4
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Helm
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: RKE

Number of control plane nodes in the cluster: 3
Number of worker nodes in the cluster: 8

Node config

OS type and version: Rocky 9
Kernel version: 5.14.0-362.13.1.el9_3.x86_64
CPU per node: 256
Memory per node: 1007G
Disk type (e.g. SSD/NVMe/HDD): NVMe
Network bandwidth between the nodes (Gbps): 10

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal
Number of Longhorn volumes in the cluster: 283

Additional context
A node was also removed from the cluster during this time.
Workaround and Mitigation
Attempted to restart, force detach, delete pods and volumes, etc..

cc @james-munson for community coordinator

cc @mantissahz for community coordinator

Hi @dt-ab,
Could you reproduce a stuck volume and generate the support bundle immediately?
I think some logs related to the volume pvc-33a6a321-9b90-41ad-986e-377680aed2c4 are missing.

@mantissahz new support bundle sent, newest pvc is pvc-b6884406-8c68-4278-8257-08a8bb32d440

As an FYI, I am not seeing errors as described in the OKD issue.

I saw that some replicas of pvc-b6884406-8c68-4278-8257-08a8bb32d440 are not running and found the controller owns the replicas is busy deleting the ophan replicas (not sure it is related).
Try to figure out why the replicas are not running.


small sentence structuring errors

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[x] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

Replacing "surname" with "last name" is not technically correct for all cultures.  While this change is well-intended, I don't see the changes as a clear improvement.

@tkremenek That might be true. However, lastname is the most preferred and commonly used parameter online for name fields.

"lastname" isn't a word in English. Also removing 'the' in the first line you changed makes the sentence less clear in my opinion..

"lastname" could also be changed to "last name" if that is something which you may consider. The omission of 'the' looks good to me, however, either way it sounds fine.
So, we can have the extra white space omitted.

I'd say both of these sentences are correct as is.


Type of thisArg is not captured properly

lib Update Request
Configuration Check
My compilation target is ES5 and my lib is [ES7, DOM, es2019, es2020.promise, es2020.bigint, es2020.string].
Incorrect Definition
Various objects, mostly ArrayLike objects, have methods that accept a callback function and a thisArg.
For example:
interface Array {

    ...

    /**
     * Executes a provided function once per each key/value pair in the Map, in insertion order.
     */
    forEach(callbackfn: (value: V, key: K, map: Map<K, V>) => void, thisArg?: any): void;

    ...

}

However this completely ignores the fact that the type of thisArg can be used to determine the type of this in the callback function
Sample Code
function testNumbers (array:number[]) {
	const evenTester = {
		test: (a:number):boolean => (a & 1) == 0
	};

	array.forEach(
		/** callbackfn */
		function(number) {
			console.log(this.test(number)? `${number} is even` : `${number} is odd`)
		},
		/** thisArg */
		evenTester
	)
}

The above code should properly compile, since the type of evenTester is known at the time of calling array.forEach. Instead, the programmer has to specify the type of this manually.
function testNumbers (array:number[]) {
	const evenTester = {
		test: (a:number):boolean => (a & 1) == 0
	};

	array.forEach(
		/** callbackfn */
		function(this:{test: (a:number)=>boolean},number) {
			console.log(this.test(number)? `${number} is even` : `${number} is odd`)
		},
		/** thisArg */
		evenTester
	)
}

Of course the above could be simplified by adding an interface for evenTester, but a one-off interface is an inelegant solution that will end up cluttering the namespace.
Documentation Link
An example of how thisArg is treated in the standard:
Ecmascript 2015 Array.forEach documentation

Duplicate of #32512. This was once added, but then removed again in #16223 for performance reasons.


Update Windows-2016-Accurate-Time.md

Fix typo: Windows 10 Anniversary Update/Windows Server 2016 is Version 1607, not 1604

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Hi - thanks for the contribution! We're reviewing it with the author of the article but will get back to you as soon as we can.
Thanks!

Yep, you're right - we screwed up the version number. You'll see your change go live this morning. Thanks!


Add null value handling for kotlin TypesAdapters

PR for #109

@cortinico Any thoughts on this PR?
We spent a day and a half debugging this issue and @mikeyshean discovered that toJson needs to handle null values otherwise it triggers an issue in Moshi thinking there is a nesting problem.

The pr looks sane to me, but I need some time to verify it.
It looks surprising that we haven't face this issue before.
Something that for sure I'll ask is to have testing coverage over the change and ideally a unit test in samples/junit-test that ensures no further regressions around this.

We were able to trigger this with a swagger definition that had a deeply nested BigDecimal value. In the swagger specification the type was set as "number".

Any news on this?

Any news on this?

Sorry, the project we were using this on was canceled. @klaszlo8207 feel free to pick up this PR and add a regression test to it.

Sorry, the project we were using this on was canceled. @klaszlo8207 feel free to pick up this PR and add a regression test to it.


PoolCluster on('connection') event

Hi!
There is no connection event emitted, that worked well with just Pool like this:
pool.on('connection', function (connection) {
    var p_all = pool._allConnections.length;
    var p_fre = pool._freeConnections.length;
    var p_ack = pool._acquiringConnections.length;
    var p_que = pool._connectionQueue.length;

    console.log('MySQL connection established. Connections: ' + p_all + '. ACK: ' + p_ack + '. QUE: ' + p_que + '. FRE: ' + p_fre);
    // Mix-in for Data Access Methods and SQL Autogenerating Methods
    mysqlUtilities.upgrade(connection);
    // Mix-in for Introspection Methods
    mysqlUtilities.introspection(connection);
});

Hi! Indeed it does not! There is an open issue at #1161 regarding this, and it's just looking for someone to make a PR to implement the feature. Perhaps you can submit a PR :)


Add secrets management feature to feature status

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/cc @nakabonne


prepare release v0.3.0

[0.3.0] - 2019-10-18

unify the fork() and execute() methods. There is now a single API for initiating asynchronous execution, fork() which makes the mental model much simpler and the learning curve smaller. https://github.com/thefrontside/effection.js/pull/14

make the fork() function static. Prior to this, fork() was a method on the Execution class which made the API awkward and non-functional. https://github.com/thefrontside/effection.js/pull/12

make generators (not just generator functions) valid operations. Before this change, you needed to use a utility function call() in order to pass arguments to generator based operations. This lets you invoke generator functions directly and pass the resulting generator to yield and fork directly thus simplifying code greatly. https://github.com/thefrontside/effection.js/pull/13

A preview package of this pull request has been released to NPM with the tag prepare-v0.3.0.
You can try it out by running the following command:
$ npm install<EMAIL_ADDRESS>
or by updating your package.json to:
{
  "effection": "prepare-v0.3.0"
}

Once the branch associated with this tag is deleted (usually once the PR is merged or closed), it will no longer be available. However, it currently references<EMAIL_ADDRESS>which will be available to install forever.

  Generated by :no_entry_sign: dangerJS against 0767f0944c55d215f410ec547c64f2557df72973

A preview package of this pull request has been released to NPM with the tag prepare-v0.3.0.
You can try it out by running the following command:
$ npm install<EMAIL_ADDRESS>
or by updating your package.json to:
{
  "effection": "prepare-v0.3.0"
}

Once the branch associated with this tag is deleted (usually once the PR is merged or closed), it will no longer be available. However, it currently references<EMAIL_ADDRESS>which will be available to install forever.

  Generated by :no_entry_sign: dangerJS against 4de91271a8f3eb983797a7d9cecad0b39b9675a6


[Merged by Bors] - Fix 'Invalid Argument' when MemoryMappedFile::open called on empty file.

bors r+

version 0.3.15 is published!


🎉 5.0.1

ℹ About this release

Version: 5.0.1
Type: patch
Last Release: 5.0.0 (11/23/2023, 5:06:38 PM) [?]
Commits to merge: 35 [?]

📦 Dependencies

Update ical-generator from ^6.0.0 to ^6.0.1

📦 Development Dependencies

Update @qiwi/semantic-release-gh-pages-plugin from ^5.2.10 to ^5.2.12
Update @semantic-release/npm from ^11.0.1 to ^11.0.2
Update @types/mocha from ^10.0.4 to ^10.0.6
Update @types/node from ^20.8.9 to ^20.11.5
Update @typescript-eslint/eslint-plugin from ^6.10.0 to ^6.18.1
Update @typescript-eslint/parser from ^6.10.0 to ^6.18.1
Update c8 from ^8.0.1 to ^9.1.0
Update eslint from ^8.53.0 to ^8.56.0
Update eslint-plugin-jsonc from ^2.10.0 to ^2.13.0
Update ts-node from ^10.9.1 to ^10.9.2
Update tsup from ^7.2.0 to ^8.0.1
Update typedoc from ^0.25.3 to ^0.25.7
Update typescript from ^5.2.2 to ^5.3.3

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

npm package (@latest dist-tag)
GitHub release
Docker container (sebbo2002/fitness-first-ical-proxy:5.0.1)
v5.0.1

Your semantic-release bot :package::rocket:


Internalize reconnection mechanics and introduce connection pooling

After several false starts, I'm pleased to share this work-in-progress attempt at channel pooling. In the past, the model was that a single ApnsClient would have a single connection to the APNs server. In this pull request, each client now has an ApnsChannelPool which manages access to one or more connections. Connections are created on demand, which eliminates the notion of explicitly connecting (or reconnecting) an ApnsClient.
Because the model is changing significantly, this change will introduce lots of breaking API changes. I'm very interested in all of your feedback.
Again, I emphasize that this is a work in progress, but I do think it should be fairly representative of the shape of things to come. I'm also hopeful that this will resolve the host of reconnection issues folks have been reporting lately. In particular, I believe this:

Fixes #435
Fixes #454
Closes #455
Fixes #489

TODO:

[x] Restore/update metrics
[x] Restore exponential backoff when opening new connections
[x] Add factory/pool tests
[ ] Document everything

This one would be very useful.
I've managed to get only around 11K notifications/s from single connection with 8-threaded event loop group on machine with 8 cores. Neither network, nor CPU was fully loaded. So I switched to 8 single-threaded APNSClients and throughput improved to 35-44K notifications/s.

Aside from the part where I think the Travis CI team is trolling me by constantly fixing and then un-fixing a weird hostname bug for OpenJDK 7, I think this is ready to roll. I'll sleep on it before I merge it, but anticipate a release some time tomorrow.

For those of you waiting patiently for this change (and release): given the significance of the changes, we want to observe it in production for a little while on our end before we unleash it upon the wider world. I'm hoping for a release at the end of next week.

Our work in #515 prompted some testing with very high levels of leak detection, which in turn increased load and slowed everything down. That, in turn, revealed some sporadic test failures when sending many notifications, which hints at a race condition. We're tracking it down, but it's something we'll need to resolve before the next release.

Here's what I've figured out so far:

The "send many notifications" tests are sometimes failing because a "send" future fails because the stream was closed before the server sent a reply.
The stream appears to have been closed because the underlying channel was closed.
The underlying channel appears to have been closed because we tried to acquire it from the idle pool, but then noticed that it wasn't active.

The last part is really puzzling to me, and will be the focus of the investigation from here.

In all cases of failure, the client sends a GOAWAY frame to the server, and the last stream created by the client is 143.

Found it. We're attaching a response promise to streams in the client handler asynchronously, and in some cases, we can get a reply from the server before that promise is in place, which causes a NullPointerException. This was likely also a problem in 0.10, but we just didn't notice it until now.

Everything is fixed in ed1d764.


Web UI update

Attemps to fix issue 82 in the upstream ReproServer repo, which specifies a few UI changes for web capture.
This is what I got locally:

It seems that those changes were made from a version before I did my own rework of the dashboard form, so I am not sure how to reconcile them.

I am going to close this, let me know if there are changes you wanted to make that are not already in.


Add a singleton_map module for serde's 'with' attribute

#[derive(Serialize, Deserialize)]
struct Struct {
    #[serde(with = "serde_yaml::with::singleton_map")]
    e: Enum,
}

will represent that field as a 1-element map variant: {contents} instead of the default tag-based representation !variant {contents}.

I am already using #[serde(serialize_with = "some_function"] and #[serde(deserialize_with = "some_other_function"]. How can I still use this feature?


Website Loading on Mobile Browsers on Mozilla Firefox (latest version) on Android

Description: Verify the website loads correctly on Mozilla Firefox (latest version) on Android.
Preconditions: The website https://oportunitatisicariere.ro/ is live and accessible.

Step 1: Open Mozilla Firefox on an Android device.
Expected Result : The website should load without any visual or functional issues.

Step 2: Navigate to https://oportunitatisicariere.ro/
Expected Result : The website should load without any visual or functional issues on Mozilla Firefox (latest version) on Android.

Step 3 : Observe the page load time and any potential loading issues.
Expected Result: The website should load within 3 seconds.

Step 4 : Verify that the layout, images, text, and other elements are displayed correctly.
Expected Result: The page should fully render all elements correctly without visual issues (e.g., broken images, misaligned text).

Step 5 : Interact with the website (e.g., click links, fill out forms) to ensure functionality.
Expected Result: There should be no loading errors (e.g., 404 errors, server errors).


Set up a CDN (Cloudflare) for anonymous visitors, static content pages

Cloudflare would require migrating the entire boost.org DNS domain to Cloudflare.  Not merely the website, but everything, by pointing the NS records at the service.
Even if that move is ultimately approved it probably wouldn't be until after many more meetings.    Therefore, what are the alternatives?  Fastly, Cloudfront.. since we are already on GCP, their offering is "Cloud CDN". https://cloud.google.com/cdn/?hl=en
"Google Cloud CDN and Cloudflare are both great solutions. Google Cloud CDN is the fastest and will be a delight to use if you already utilize Google Cloud Services."
"Highly recommended to implement if you are already using other Google Cloud Services"
"It is one of the most secure and fastest CDN available globally."
"It drastically reduces the latency, so the loading time of the content is reduced as well."
I think the next step should be testing stage.boost.cppalliance.org with "Cloud CDN".  This week I was running load testing https://github.com/locustio/locust , and can direct web traffic at the site (before/after a CDN is in place.)

Normal "CDN" services aren't what we need, we need an edge side cache that is smart.  So Fastly is an option, but likely to be quite a bit more expensive than CloudFlare unless you can work a deal with them due to Boost being Open Source (for example they host all of Python's PyPI for free for some advertising).
The other option is to host Varnish ourselves in front of the Django app.
In either case, adding some Django level caching (and cached sessions for logged in users) would go a long way to improve performance at scale.

Fastly:   is the most customizable and smart edge cache. The pricing of $.12 per GB would be $120/month at 1TB of traffic for the website. Not extremely expensive. Boost archive downloads continue to be hosted on JFrog. The choice of CDN for the downloads is not required to be the same as the website.

"Cloud CDN":  to explore the idea, could the service do these tasks?

cache all anonymous content
cache html text in addition to images and js
not cache authenticated pages.

Currently Django is setting the header "Vary: Cookie, origin" on all html pages.  Those would not be cached. To fix this, unauthenticated users would need to have "cookie" removed from the Vary header.

Cloud CDN won't cache html text unless Cache-Control is included in the headers. So, Cache-Control would need to be added for unauthenticated users.

As long as "Vary: Cookie, origin" continues on authenticated pages, they would not be cached, which is correct.

Perhaps some forward progress could be made using Cloud CDN to implement edge-caching and remove traffic from the origin.
At the same time, Fastly is much more customizable and can implement any caching rules. So, let's try that. We could still switch CDNs in the future if necessary.   And if the Boost Foundation agrees to migrate all DNS to cloudflare.

The main performance concern is with the boost documentation.  Pages like https://www.preview.boost.org/doc/libs/boost_1_83_0/doc/html/accumulators.html
In the long term, configuring Edge-Side Includes (ESI) on the CDN would enable caching even when authenticated.
In the short term, if the documentation page headers could display an anonymous profile photo in the upper right corner instead of the actual profile photo, then  in exchange for this very small sacrifice all documentation pages would be cached and optimized by the CDN.   (Set up the caching policy to remove Vary: cookie on doc/ pages)
Add Edge-Side Includes (ESI) in the future.

This is basically completed. The Fastly CDN has been configured to cache pages.  It will still need to be adjusted if/when we have access to the main website domain.  Also fine-tuning later.


STYLE: Update Python files to 3.7 syntax

Accomplished by using https://github.com/asottile/pyupgrade:
C:\Dev\ITK-git>for /f %f in ('dir /s /b *.py') do pyupgrade --py37-plus %f
PR Checklist

[x] No API changes were made (or the changes have been approved)
[x] No major design changes were made (or the changes have been approved)

Thank you.  It looks like this would benefit from a post-pass with black:
for /f %f in ('dir /s /b *.py') do black %f

so that black wins out where there are disagreements with pyupgrade.

Sorry, we probably don't want to apply black within ThirdParty.

I noticed that black modified a lot of files in third party, but I didn't for pyupgrade. If you want to take over, please create a new PR (I don't think you can push to my fork). I will close this one in favor of that.

If you want to take over, please create a new PR

Can do.

Closing in favor of #3244.


Fixing UX of find a measure filters

Selections are all OFF initially
Fields are no longer disabled by default
Checkboxes are turned on whenever the user makes a selection or changes values

 Search / Find a Measure: all boxes should not be ticked by default

 [DIT TQ-46] Find measures page usability regressed (have to explicitly select checkbox before selecting criteria)


[Request] Reset all for xp tracker

Reset all for xp tracker, so there's no need to separately click on "reset others" and then reset the remaining one.

You can right click the overall section at the top.


ACE list does not return proper role name when "href" is not found.

Relevant Issue: https://github.com/personium/personium-client-java/issues/24

Released v1.5.10


Missing graylog_stream_rule value field in state when type=5

Environment

Graylog version: 3.2.4
OS (client side): 5.4.44-1-MANJARO
The result of terraform version:

Terraform v0.12.26
+ provider.graylog (unversioned)

Overview
After importing resource graylog_stream_rule when type = 5 (type "field must be present"), state shows no value.
Expected behavior
terraform state show should show value = "" even when type = 5
Actual behavior
Using the output of terraform state show to write the config into the .tf file, terraform plan says value is required.
Manually adding empty value fixes it.
Error: Missing required argument

  on streams.tf line 2541, in resource "graylog_stream_rule" "import_575010a93aca000fc197a38b_rule1":
2541: resource "graylog_stream_rule" "import_575010a93aca000fc197a38b_rule1" {

The argument "value" is required, but no definition was found.

Please correct me if this is not directly caused by the plugin.
Awesome plugin by the way :)

Please share the definition of graylog_stream_rule.import_575010a93aca000fc197a38b_rule1.
Do you specify value in stream.tf?

provider.graylog (unversioned)

And please download the latest provider from GitHub Releases.
https://github.com/terraform-provider-graylog/terraform-provider-graylog/releases
Otherwise, it is difficult to support.

Hi @suzuki-shunsuke
I was already using lastest release (1.0.1), terraform version is just not printing the version.
As I said, the definition of graylog_stream_rule.import_575010a93aca000fc197a38b_rule1 was written from the output of terraform state show graylog_stream_rule.import_575010a93aca000fc197a38b_rule1 which is missing value :
$ terraform state show graylog_stream_rule.import_575010a93aca000fc197a38b_rule1
# graylog_stream_rule.import_575010a93aca000fc197a38b_rule1:
resource "graylog_stream_rule" "import_575010a93aca000fc197a38b_rule1" {
    field     = "syslog5424_msg"
    id        = "575010a93aca000fc197a38b/57b72f023aca0003ac5d0fe9"
    inverted  = false
    rule_id   = "57b72f023aca0003ac5d0fe9"
    stream_id = "575010a93aca000fc197a38b"
    type      = 5
}

The definition is then translated to :
resource "graylog_stream_rule" "import_575010a93aca000fc197a38b_rule1" {
  field     = "syslog5424_msg"
  inverted  = false
  rule_id   = "57b72f023aca0003ac5d0fe9"
  stream_id = graylog_stream.import_575010a93aca000fc197a38b.id
  type      = 5
}

Adding value = "" solves the issue though.
Is it just normal behavior for terraform state show to not print empty field in this case?

I can't reproduce the problem.
Graylog version: 3.2.4+a407287
$ terraform version
Terraform v0.12.26
+ provider.graylog v1.0.1
+ provider.random v2.2.1

I imported the following stream rule.
        {
          "field": "tag",
          "stream_id": "5ea8eed22ab79c00129dfc3a",
          "description": "test",
          "id": "5ea8eed22ab79c00129dfc58",
          "type": 1,
          "inverted": false,
          "value": "5"
        }

$ terraform import graylog_stream_rule.test 5ea8eed22ab79c00129dfc3a/5ea8eed22ab79c00129dfc58
graylog_stream_rule.test: Importing from ID "5ea8eed22ab79c00129dfc3a/5ea8eed22ab79c00129dfc58"...
graylog_stream_rule.test: Import prepared!
  Prepared graylog_stream_rule for import
graylog_stream_rule.test: Refreshing state... [id=5ea8eed22ab79c00129dfc3a/5ea8eed22ab79c00129dfc58]

Import successful!

The resources that were imported are shown above. These resources are now in
your Terraform state and will henceforth be managed by Terraform.

$ terraform state show graylog_stream_rule.test
# graylog_stream_rule.test:
resource "graylog_stream_rule" "test" {
    description = "test"
    field       = "tag"
    id          = "5ea8eed22ab79c00129dfc3a/5ea8eed22ab79c00129dfc58"
    inverted    = false
    rule_id     = "5ea8eed22ab79c00129dfc58"
    stream_id   = "5ea8eed22ab79c00129dfc3a"
    type        = 1
    value       = "5"
}

I posted https://github.com/terraform-provider-graylog/terraform-provider-graylog/issues/43#issuecomment-658835709 before I looked https://github.com/terraform-provider-graylog/terraform-provider-graylog/issues/43#issuecomment-658834710 . I'm seeing your comment now.

The type is 1 in the rule that you imported.
Try with type 5 which means "field must be present" :
        {
          "field": "tag",
          "stream_id": "5ea8eed22ab79c00129dfc3a",
          "description": "test",
          "id": "5ea8eed22ab79c00129dfc58",
          "type": 5,
          "inverted": false,
          "value": ""
        }

Oh, I see. Sorry, I misunderstood. 🙇♂

I understood the problem.
Thank you for your report!
I'm considering how to solve.

Thanks for your super-fast reactivity and keep up the excellent work!

I created a pull request to fix the problem. #44
The idea is to make the field value optional.

I have released the prerelease version v1.0.2-0.
Please check.
And if there is no problem, I'll release the version v1.0.2 and close this issue.
Thank you.

https://github.com/terraform-provider-graylog/terraform-provider-graylog.github.io/pull/11

Prerelease is working as expected on my side, looking good to me.

Thank you for your confirmation.
I have released v1.0.2.
So I close this issue.


Fix UI tests in system_devices_controller

The tests & UI got out of sync in 914c88491. This fixes them.

+1


UnicodeDecodeError

When importing the some flashcards with utf8 formatting this error occurs:
Traceback (most recent call last):
  File "c:\Users\Philip\Documents\GitHub\ankdown\ankdown\ankdown.py", line 389, in main
    cards_to_apkg(card_iterator, pkg_arg)
  File "c:\Users\Philip\Documents\GitHub\ankdown\ankdown\ankdown.py", line 340, in cards_to_apkg
    for card in cards:
  File "c:\Users\Philip\Documents\GitHub\ankdown\ankdown\ankdown.py", line 328, in cards_from_dir
    for card in produce_cards(os.path.join(parent_dir, fn)):
  File "c:\Users\Philip\Documents\GitHub\ankdown\ankdown\ankdown.py", line 301, in produce_cards
    for line in f:
  File "C:\Python36\lib\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x81 in position 3504: character maps to <undefined>

Changing line 297 in ankdown.py to
    with open(filename, "r", encoding="utf8") as f:

helps.
I thought though, that the default import in python is utf8. Should I do a pull request for this little bug?

Weird! Yeah a PR would be great! (I'm traveling so it's a bit tricky to write code)


Thanks for update. Tried to launch the latest version but got this error:

Can it be fixed somehow?

@JohnClaw Oh. There is no hiplas(rocm) build in the workflow yet.
If you can change the backend, you can force it with startup arguments, for example:
sd_ui.exe -cuda
or
sd_ui.exe -avx
There is currently avx, avx2, avx512 and cuda builds available only.
I will update the build workflow today, but before I will manually build a rocm for testing.

Thanks for answer. Now app runs without that error message.


PNL - Test2

Hello.
I was applying the PNL algorithm to some data and was getting unexpected results.
Upon running the tests, I get an error in test_pnl_simulation_2, for the  p_value_forward_2 value.
Is there a known problem regarding this?

Thank you

Hello. I was applying the PNL algorithm to some data and was getting unexpected results. Upon running the tests, I get an error in test_pnl_simulation_2, for the p_value_forward_2 value. Is there a known problem regarding this?  Thank you

Thank you for bringing up this issue. I've run the second test myself, and it successfully passed.

I'm considering that the error might be raised from the update in some fundamental packages, such as scipSklearnlearn. Could you verify the precise values of p_value_forward_2 and p_value_backward_2? We're considering updating that asserting exact values to defining intervals, which we think will provide better resilience against potential minor issues arising from future updates to these core packages.

That seems to be the problem, there is a small difference in the p_value_forward_2 to the simulated one.

That seems to be the problem, there is a small difference in the p_value_forward_2 to the simulated one.  Third test passes and the real case also has a small deviation.

Thanks for the reporting results! We have encountered a similar issue in some other implementations before, where updates to the underlying computation library caused the test examples to fail. Thank you for your information; we will update these assert statements ASAP.


NPM package

Please add this repository to NPM.
I use meteor.js and the importance of versioning.

Hi,
Sure, I'll add package.json and maintain it in the future.
Cheers!
Sérgio

Please publish!

Bump! :)

Hi,
I don't have or use NodeJS at the moment. Can you guys publish it for me? Is the publication by the author a requirement?
Thanks!
Sérgio

Check this:
https://docs.npmjs.com/getting-started/publishing-npm-packages
Not complicitad, but requiremt installed node.js/npm.
Only publish owner (I think).

I would like to bump this as well.
NPM is the de facto dependency manager for front-end libraries for the time being.
Right now there is a fork of the plugin available, that is on v2.1.4.

Hi
Im using webpack and it cannot find your package (installed via npm) when building bundle.
I solved it by adding 'main' entry to your package.json file like this:
"main": "jquery.flexdatalist.js"
Cheers


ProcessException: Permission Denied

Steps to Reproduce

flutter build for Android any options

[✓] Flutter (Channel master, v0.5.8-pre.184, on Linux, locale C.UTF-8)
    • Flutter version 0.5.8-pre.184 at flutterRoot
    • Framework revision 3a6228bead (19 hours ago), 2018-07-30 18:07:23 -0700
    • Engine revision af855e063a
    • Dart version 2.0.0-dev.69.4.flutter-08f59e5de3

[!] Android toolchain - develop for Android devices (Android SDK 28.0.1)
    • Android SDK at androidSdkRoot
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.1
    • Java binary at: androidStudioRoot/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b04)
    ✗ Android license status unknown.

Logs

command
flutter build apk --debug -v
exception
ProcessException: ProcessException: Permission denied
Command: project/android/gradlew -v
#0      runAsync (package:flutter_tools/src/base/process.dart:227:33)
<asynchronous suspension>
#1      runCheckedAsync (package:flutter_tools/src/base/process.dart:242:34)
<asynchronous suspension>
#2      _initializeGradle (package:flutter_tools/src/android/gradle.dart:180:9)
<asynchronous suspension>
#3      _ensureGradle (package:flutter_tools/src/android/gradle.dart:161:37)
<asynchronous suspension>
#4      buildGradleProject (package:flutter_tools/src/android/gradle.dart:268:31)
<asynchronous suspension>
#5      buildApk (package:flutter_tools/src/android/apk.dart:41:10)
<asynchronous suspension>
#6      BuildApkCommand.runCommand (package:flutter_tools/src/commands/build_apk.dart:49:11)
<asynchronous suspension>
#7      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:347:18)
#8      _asyncThenWrapperHelper.<anonymous closure> (dart:async/runtime/libasync_patch.dart:77:64)
#9      _rootRunUnary (dart:async/zone.dart:1132:38)
#10     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#11     _FutureListener.handleValue (dart:async/future_impl.dart:129:18)
#12     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:642:45)
#13     Future._propagateToListeners (dart:async/future_impl.dart:671:32)
#14     Future._complete (dart:async/future_impl.dart:476:7)
#15     _SyncCompleter.complete (dart:async/future_impl.dart:51:12)
#16     _AsyncAwaitCompleter.complete.<anonymous closure> (dart:async/runtime/libasync_patch.dart:33:20)
#17     _rootRun (dart:async/zone.dart:1124:13)
#18     _CustomZone.run (dart:async/zone.dart:1021:19)
#19     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:947:23)
#20     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#21     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#22     _runPendingImmediateCallback (dart:isolate/runtime/libisolate_patch.dart:113:13)
#23     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:166:5)

I do not really know how I could help flutter build there. Running with sudo is not only discouraged, but does also not help. It definitely has permissions for the gradlew file. My guess is that it is another file that needs additional permissions, but I only get this from -v:
...
[ +289 ms] Initializing gradle...
[   +7 ms] Using gradle from project/android/gradlew.
[  +63 ms] project/android/gradlew -v
[  +57 ms] "flutter apk" took 732ms.

EOF

The project directory was on a FAT filesystem, where I could not change permissions.

so How i can change the filesystem


translated ch3 and ch4

Description
completed the translation of chapter 3 and chapter 4

LGTM!
I will remerge it buy using a local branch.


Suggestion - a very small code change would implement rate limiting

a lot of APIs have rate limiting but...
If you were to add:
$client_options = ['timeout', 'proxy', 'handler'];
then we could use this: composer require spatie/guzzle-rate-limiter-middleware and write apps that include (for example)
$stack = HandlerStack::create();
$stack->push(RateLimiterMiddleware::perSecond(8));
$provider = new Provider([
'clientId'     => MYOB_CLIENT_ID,
'clientSecret' => MYOB_CLIENT_SECRET,
'redirectUri'  => CALLBACK_URI,
'username'     => USERNAME,
'password'     => PASSWORD,
'scopes'       => ['CompanyFile'],
'handler'      => $stack
]);

The next major version of this library will support PSR-18 HTTP clients, which should allow for greater control over the HTTP client and all the options you might want to set on it. 🙂


Convoy / Rancher NFS plugin

There is no convoy plugin in integrated to rancher server / catalog? Is it replaced or integrated by another plugin? Is Rancher NFS a replacement? What is planned?

Hi, @pwFoo - rancher-nfs is the updated version of convoy-nfs. It will work for all of your NFS needs.
Do you have any specific questions about its use?

Some documentation like convoy github page / readme would be nice. Convoy was useable without nfs too. That was nice for simple local tests.
At the moment I'm searching for the best volume solution with (automated) backups, restore and optional snapshot features.
For databases I'll test rancher-backup from catalog. So I need a backup solution for volumes too. During search I found convoy but not rancher-nfs ;)

Dupe of https://github.com/rancher/rancher/issues/6859
rancher-nfs docs that should include migration path from convoy-nfs


Program details page is missing h1 heading

Found while adding axe accessibility testing.
Details from axe: https://dequeuniversity.com/rules/axe/4.4/page-has-heading-one?application=axeAPI
We currently have a level 2 heading. We should either bump this up to a level 1 heading, or evaluate if it makes sense to add a separate level 1 heading.

Which Program details?
I'm fixing an issue with a H1 following a H2 on the screen when an Admin views applications.  I'm removing the H1 fwiw.

Ah this is from the applicant view. Added more details and a screenshot to this issue.


A refactoring and an example of embedding Starlark in Rust

Four commits:

change Value::downcast_apply* parameters to accept only TypedValue for type safety
change Value::downcast_apply* to return Option<Return> instead of panicking
Function::new_native_n where n is 0, 1, 2 as easy easy setup of Rust callbacks and as example
an example how to embed Starlark into Rust

Hi, can this be made into 4 different PR for easier review? Unfortunately, @github does not support stacked review really nicely :/

Sure, I'll create separate PRs.


Shows up in wrong position in Safari (desktop and mobile)

The modal shows up too far down and to the right on safari, both desktop and mobile.

@acoreyj Thanks for creating issue and spotting the bug , it'll try to fix this problem ASAP ! Cheers :+1:

Wrong in chromium too.


Release to Master

This change is

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add support to update the developer portal between environments

The original extractor tool had the benefit of having all the developer portal templates, which made it easy to update the developer portal between environments
Investigating the ability to get the developer portal info with the standard REST API calls to APIM.

@waelkdouh May I ask why this issue is closed? I'm using the ApiOps project to deploy the API Management to different environments and I'm looking for the best option to deploy the developer portal content (including email templates) from my dev environment to test, acc and production environment. If I could use this tool for this that would be awesome :)

Hi @ericfitskieolyslager. Our tool is exclusively focused on the apis themselves. Having said that our colleague here at Microsoft created a separate tool that takes care of promoting the changes within the developer portal. We even endorsed it from our own repository which you can find here

Thank you for your response. I'll look at the repository to implement the developer portal part of API Management.


[WIP] Added fixer for PHPUnit's @expectedException annotation

In favor of #2200

[ ] Rename to PhpUnitExpectedExceptionFixer
[ ] Only fix sub classes from \PHPUnit_Framework_TestCase
[ ] Normalize setExpectedException('Some\Exception'); to '\Some\Exception'

Only fix classes extending \PHPUnit_Framework_TestCase

Nope, we don't do that assumption in any of our PHPUnit fixer.
PHPUnit_Framework_TestCase could not be used directly but custom child of it is used.
Eg we are using PhpCsFixer\Test\AbstractFixerTestCase in most of our tests files.

is_subclass_of? I mean.. we shouldnt just update every call to setExpectedException... right?

When you fix a code you have only file-level context.
If given class is defined outside of file, you have no idea what it is. It may be even defined in different repo

Yeah.. but we should verify the class/file we're updating is actually a test class/file or am i missing something..

Normalize setExpectedException('Some\Exception'); to '\Some\Exception'

I would do that in another fixer, it use will be limitted because of: https://github.com/sebastianbergmann/phpunit/issues/2074#issuecomment-183472385 (i.e. it will be removed from PHPUnit itself)

Only fix sub classes from \PHPUnit_Framework_TestCase

I agree with @keradus here, there is no way solid figuring out if a class (or some include / trait) is part of a PHPUnit_Framework_TestCase-class inherence by looking at the tokens of that single file.

Rename to PhpUnitExpectedExceptionFixer

only if your fixer also fixes the casing ;) (I would like that feature btw., but still think it is out of scope)

Rename to PhpUnitExpectedExceptionFixer

as we don't modifying ->expectException I would name it PhpUnitFqcnAnnotationFixer.
changing string passed to method is different thing and it should be separated fixer/PR.

:+1: lets keep things simple.

raname and ready to merge ?

Worst idea ever.. but i think it's ok :)

Worst idea ever..

the name? it was just a proposal to have it short and not connected to function call with same name that annotation.
propose better one ;)

I meant renaming via github :smile:
This is ready :tada:

:+1:

:+1: :ship: thanks @ro0NL

Thank you @ro0NL.


Dependency telemetry is not collected in Background Services

We're using Background Services for various background processing inside our Asp.Net Core web application.
For each background operation, we start a new operation like this:
using (_telemetryClient.StartOperation<RequestTelemetry>("operation name"))
{
   // ... Do some processing, database calls, http calls, storage calls
}

The operation appears in Application Insights, but there is no telemetry for dependency calls to the Database (Sql Server), or HTTP requests with HttpClient. However we can see dependency calls to Azure Queues.
Is there a way to activate dependency telemetry for database or HTTP calls? We can't use the WorkerService package as it does not work alongside AspNetCore package.

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

ID: 4b88d8c4-0013-70aa-7997-a2203afd9743
Version Independent ID: 0ab99d5e-4ebd-8cb9-3d27-efe58def249f
Content: Azure Application Insights for ASP.NET Core applications - Azure Monitor
Content Source: articles/azure-monitor/app/asp-net-core.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@Costo , Thanks for brining this to our attention. We will investigate it further and update you shortly.

@Costo, Hope you're doing great and thanks for being patient.
In order for us to look at how you have instrumented the app, we might need to repro the scenario.
Could you provide us with the repro instructions on how you are trying to achieve this, so we can replicate it and provide any guidance we may have on this.
Looking forward to hear back from you soon.

@Costo , Hope you're doing fine. Thought of checking to see if you're still having issues. And Since this is not a doc update related question, please open a question in Application Insights MSDN Forum.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Add user to groups

I have an install.sh on the pi that adds the user to several groups, e.g.:
# setup permissions: add user to video, lpadmin, input and gpio groups
echo "Creating permissions..."
sudo groupadd -f -r gpio
sudo usermod -a -G gpio,lpadmin,video,input $(whoami)

Would be nice to have a setting that adds you to specified groups (and creates the group if it doesn't exist).

Good idea!


Bug report: 403 forbidden on Flow Export

Description
When running m365 flow export --environment [env] --id [id]   I get the error "Error: Request failed with status code 403"
Running the command with --format json does work however.
If I grab the access token and run the command in postman I get same error but with more information:
{"error":{"code":"AuthorizationFailed","message":"The client certificate 'DF4D2C02B7F733BB44849D827A5BC6AFA11B316C' is not allowed."}}
Steps to reproduce
Export a flow with
m365 flow export --environment [env] --id [id]
Expected result
Success with a zip exported
Actual result
403
Environment
windows 10
m365 version: v3.4.0
node.js v14.15.4

Thank you for raising this @m-carter1 apologies for the trouble, we will take a look ASAP.

Thank you for raising this @m-carter1 apologies for the trouble, we will take a look ASAP.

Can I ask whether this is the first time you have used the command or has this just stopped working for you?

Can I ask whether this is the first time you have used the command or has this just stopped working for you?

This is the first time I've used it so it may be something I'm doing wrong but I was able to successfully get the flow json back just not the zip file. I get the same results on another tenant also.

This is the first time I've used it so it may be something I'm doing wrong but I was able to successfully get the flow json back just not the zip file. I get the same results on another tenant also.

When prepping my pnp demo I had the exact same issue for 15 out of the 20 flows I wanted to export. The weird thing is that it did work 15m later for some of those 15 that first failed.. Could it be that MSGraph has perf. degredation?

When prepping my pnp demo I had the exact same issue for 15 out of the 20 flows I wanted to export. The weird thing is that it did work 15m later for some of those 15 that first failed.. Could it be that MSGraph has perf. degredation?

@m-carter1 before we start investigating it, could you please check if you can still repro the issue?

@m-carter1 before we start investigating it, could you please check if you can still repro the issue?

@m-carter1 before we start investigating it, could you please check if you can still repro the issue?

@waldekmastykarz  yes still getting the 403 issue this morning
I'm authenticating using m365 login and then using the device code in a browser I assume that's correct and I'm not missing anything?

@m-carter1 before we start investigating it, could you please check if you can still repro the issue?

@waldekmastykarz  yes still getting the 403 issue this morning
I'm authenticating using m365 login and then using the device code in a browser I assume that's correct and I'm not missing anything?

That is correct. Are you the owner of the flow that you're trying to export? Can you open it in the browser?

That is correct. Are you the owner of the flow that you're trying to export? Can you open it in the browser?

Yes I am the owner and I'm able to open it in the browser.
The export works when setting --format=json  but without that it's a 403 error

Yes I am the owner and I'm able to open it in the browser.
The export works when setting --format=json  but without that it's a 403 error

@appieschot does that sounds like anything you've see in the past few days?

@appieschot does that sounds like anything you've see in the past few days?

@m-carter1 do you get this issue on all or some flows?

@m-carter1 do you get this issue on all or some flows?

@waldekmastykarz yes all of them, I've just tried creating a new flow and same issue with that one too

@waldekmastykarz yes all of them, I've just tried creating a new flow and same issue with that one too

@waldekmastykarz We're seeing the same issues in PnP PowerShell, which is calling exactly the same end points.

@waldekmastykarz We're seeing the same issues in PnP PowerShell, which is calling exactly the same end points.

I have the same issue on all flows on my dev tenant :(

I have the same issue on all flows on my dev tenant :(

Is the option in the UI available or gone? Could it be that the functionality has been removed from the service?

Is the option in the UI available or gone? Could it be that the functionality has been removed from the service?

@waldekmastykarz I am still able to export flows as a zip from the UI

@waldekmastykarz I am still able to export flows as a zip from the UI

Seems like there were some changes in the underlying API. Let's see if we can adjust the command to accommodate the changes.

Seems like there were some changes in the underlying API. Let's see if we can adjust the command to accommodate the changes.

@m-carter1 we've just released a new preview version where this issue should be fixed. Could you please give it a try by running npm i -g @pnp/cli-microsoft365@next?
Once again thank you for reporting the issue! 👏

@m-carter1 we've just released a new preview version where this issue should be fixed. Could you please give it a try by running npm i -g @pnp/cli-microsoft365@next?
Once again thank you for reporting the issue! 👏

thanks @waldekmastykarz I've updated and it now works as expected

thanks @waldekmastykarz I've updated and it now works as expected

Hi,
I have been working on this error since january 11th. Also open an MSFT incident ticket.
The bug is present on 3.4.0 and 3.5.0 since january 11th.
Don't know how is happened.

Hi,
I have been working on this error since january 11th. Also open an MSFT incident ticket.
The bug is present on 3.4.0 and 3.5.0 since january 11th.
Don't know how is happened.


Allow ringing sessions to be recorded and played back as test cases

This would provide an excellent mechanism for testing Wheatley's performance, since if we mocked the Tower class and make the sleep function just increment a counter then we can simulate Wheatley ringing for huge amounts of time.
If someone else wants to have a crack at this then they're very welcome, but if I had to do it I'd probably do something like:

Move all the knowledge of timing (i.e. time.time() and time.sleep()) into the Tower class so that it can be mocked easily.
Make some kind of running mode which records the events passing through the Tower class and records them to some format (probably JSON)
Make a mocked version of Tower which never interacts with RR but instead replays any given simulation using mocked sleep and time functions to go wayyyy faster than real time.

A record-and-playback feature could also be used for other purposes than just testing, such as for inclusion in pre-recorded worship services, or even semi-live demonstrations (playbacks) for (spatially-) distributed listeners of a single online tower.
It might be useful to somehow indicate visually to onlookers (casual, non-participating listeners, watching the playback ringingroom) a status message that a recording or playback is in session, when it is.

A record-and-playback feature could also be used for other purposes than just testing, such as for inclusion in pre-recorded worship services, or even semi-live demonstrations (playbacks) for (spatially-) distributed listeners of a single online tower.

Agreed.  Real-time playback wouldn't be super useful for testing Wheatley, but once we have some kind of recording format it would be really easy to play them back to Ringing Room (we probably don't even need Wheatley - we could just ring the bells directly using WebSocket or a library like belltower).


Gradients in Wang's dropout

Might be a problem with the way I've implemented Wang's Gaussian dropout; might need to explicitly set the gradient in Theano. There is a section in the older paper that shows setting that gradient as an expectation. Should just autodiff at the moment though, thanks to the parameterization.

Not necessary.


Position / orientation updates on object post-gravity does not move object if in non-flatcache mode

As we focus our efforts more on the flatcache mode this bug (confirmed with physx team) can now be ignored. In the QA script we make sure to articulate the object to use that interface.