ARM M1 support? :)

See https://www.electronjs.org/blog/apple-silicon and https://www.electronjs.org/blog/electron-11-0
We're currently using Electron 9, so we'd have to consider upgrading Electron first: https://github.com/Kong/insomnia/blob/develop/packages/insomnia-app/package.json#L200

Luckily, upgrading electron is on the roadmap, currently slated for this quarter!

any updates here ?

waiting for updates here, or at least a roadmap.

can I hope that I'll able to download insomnia for my Mac on m1?

can I hope that I'll able to download insomnia for my Mac on m1?

Until 2023? Yep 🤓

Meh, off to other clients, who don't take years to transition.

+1 for this upgrade.
Would be nice have this amazing app working nicely in M1, keep good work dudes.. thanks!

Waiting for this... 🙏🏻

Hey everyone.  Thanks for the support.  This is on my mind, I promise, but the "+1" comments don't help and just spam our GitHub inboxes.
If you would like to voice your support for this, please throw a :+1: on the issue description.
This will definitely come in time, since it comes for free with an electron upgrade.  The team is super focused on spaces right now, so although I can't say when this upgrade will land, I can say that it won't land for a few weeks, at least.

Hey everyone: 😄
Just to update, we just merged in Electron 11 a few minutes ago.
If you want M1 support, the best thing you can do to help that along is go test the electron upgrade: https://github.com/Kong/insomnia/releases/tag/core%402021.4.1-alpha.1.  We'll be looking into M1 support soon now, no doubt, but it will be of great assistance if you could help us test the Electron upgrade.
Thanks for your snarky and non-snarky patience! :)

Still waiting...

Sorry for all of the delays, everyone. Just to let you know, we will be looking into this again. I've taken a look at this a couple times before, so I'd like to add some technical details to make it clearer why this has not happened yet.
One of the bigger blockers in our way was GitHub Actions, but unbeknownst to me it seems like GitHub Actions is unblocked and now has Big Sur runners generally available. This is important, because our CI is based on GitHub Actions, and the previous generally-available macOS runners were running a version of macOS too old to install a new-enough version of XCode to cross-compile for the M1/ARM64 architecture.
We're not totally unblocked, however. We have been running into some confounding issues getting things stable on Electron 12. This is relevant because we need to upgrade Electron Builder, which needs Node 14, and our Node version is currently synchronized to the Node version that ships with Electron (which makes syncing NAPI versions and possibly some other issues a bit easier to deal with.) Electron Builder needs Node 14 because it began to refactor to use fs/promises (https://github.com/electron-userland/electron-builder/pull/5880) which is only available in Node 14.
This is all a bit unfortunate, but the end result is that we may need to do some work arounds to enable M1 binaries for the time being.
Just to be clear, for the time being, Insomnia does run just fine on M1 macOS devices by virtue of Rosetta 2. So you needn't hold off on switching to machines with the M1 architecture if this is blocking you. Nothing is required to use Rosetta 2, it is fully automated by macOS:

That having been said, we do understand that this is unideal, and do not intend to lean on it for any longer than necessary. We don't have any exact timeline right now since there are too many unknowns, but it is not being ignored by any means.

Thanks for all the work you're doing on this! Excited for when you're finally able to get this out

Hello Guys! What about progress on this issue?

We were unable to ship an M1 build using Electron 11 because of a bug in Electron 11 that accidentally violates the W^X restrictions in some situations in hardened runtime and crashes immediately. Unfortunately, no release rectified this issue, and the official recommendation is to upgrade to Electron 12. This was a huge setback for getting an M1 build out quickly, but of course we will upgrade to Electron 12 and beyond.
What we have done, so far:

Upstream node-libcurl now has M1 builds as v2.3.4-0.
The #4305 PR replaced Spectron with Playwright, which was needed to unblock Electron upgrades.
The #4232 PR will update us to Electron 12. (to be merged)
The #4140 PR provides a port of Insomnia that builds and runs on M1. (WIP)

We were unable to ship an M1 build using Electron 11 because of a bug in Electron 11 that accidentally violates the W^X restrictions in some situations in hardened runtime and crashes immediately. Unfortunately, no release rectified this issue, and the official recommendation is to upgrade to Electron 12. This was a huge setback for getting an M1 build out quickly, but of course we will upgrade to Electron 12 and beyond.

@johnwchadwick
So, any ETA for that? :)

Barring any unforeseen circumstances, M1 support should ship in the next release.


[autocomplete] Autocomplete starts open then quickly closes after it or its field is hidden then shown

Describe the bug:
If the autocomplete component or its field is hidden while the dropdown is open, when it is shown again, the dropdown from before the hide will quickly be shown then hidden.
To Reproduce:
Steps to reproduce the behavior:

Open the dropdown on an autocomplete
Hide the autocomplete or its field (can add 'hidden' in devtools)
Show the autocomplete again

Expected behavior:
The dropdown from before the hide will not be shown again.
Please complete the following information:

Forge version: 3.3.5
I have searched existing issues before creating this report? Y
Browser: All
Platform: All
OS: All

@DRiFTy17 This issue was fixed for Chrome with your PR, but still exists in Firefox

@klschwartz I just created #765 with a fix for this, which is more reliable across browsers anyway. Thanks.


Can't craft the ceilands portal activator

Maybe I'm missing some step?

The mod is working tho:

You're missing a stick at the bottom middle slot.

cool! then the problem is with EMI

I realize that the portal activator item is crafted using forge:rods/wooden and Fabric uses c:rods/wooden.
Like, why did the mod loader systems have to use different tags? I will have to fix parity in 0.6.2 at a point!


More world generation parameters

This is a personal preference, but I'd like to scarce the generation of silverwood and greatwood trees. Also I'd like to scarce amber generation but not crystals. I think i can also be useful for modpack makers to have more control on world gen with more detailed parameters in the config file.
Thanks for reading!

This issue was moved to Azanor/thaumcraft-suggestions#383


[Godot 4.0 Beta 6] GDScript properties break in release mode only, when combining type hints and match getters

Godot version
4.0 beta 6 official
System information
Windows 10, also noticed on M1 MBP Mac OS 13
Issue description
I noticed my game broke on export, but only with release builds, debug builds worked fine. The errors I get from the console are:
SCRIPT ERROR: Parse Error: Not all code paths return a value.
          at: GDScript::reload (res://script.gd:26)
SCRIPT ERROR: Parse Error: Not all code paths return a value.
          at: GDScript::reload (res://script.gd:40)
SCRIPT ERROR: Parse Error: Not all code paths return a value.
          at: GDScript::reload (res://script.gd:54)
ERROR: Script does not inherit from Node: res://script.gd
   at: (main/main.cpp:2755)

The errors show that the script does not inherit from Node (even though it extends Node), and that not all code paths return a value on lines 26, 40, and 54. These lines are property getters that return a match, all with a default _: block, so they all do return a value. Sample starting at line 25 below:
var Interact: String:
	get:
		match ActiveDevice:
			DEVICE.KEYBOARD_MOUSE:
				return "Left Mouse"
			DEVICE.SONY:
				return "X"
			DEVICE.NINTENDO:
				return "B"
			DEVICE.MICROSOFT:
				return "A"
			_:
				return ""

As a consequence of these errors, most of my scripts stop working and the game completely breaks. This happens when using the official release export templates, or when compiling my own release templates, and does not happen when running from the editor or using any debug template. In debug builds, everything works exactly as expected.
I've narrowed down the script to the minimal that breaks. The following is broken:
extends Node

var Foo: String:
	get:
		match 1:
			0:
				return "Zero"
			1:
				return "One"
			_:
				return "Something Else"

and by only removing the type hint, the following works fine in release mode
extends Node

var Foo:
	get:
		match 1:
			0:
				return "Zero"
			1:
				return "One"
			_:
				return "Something Else"

All of the above involves the script being an autoload, however attaching the script (with the type hint) to a node in a scene still breaks, showing a parse error (but not showing a "does not inherit from Node" error), and failing to execute any further, e.g. a _ready() function that prints Foo will not run.
Steps to reproduce

Create a script with property that has a type hint, and a getter that uses a match statement to return a value.
Ideally, add a _ready() function that prints the value of the property.
Either add this script to a node in the scene, or set it to be an autoload.
Export with any release template, ideally with a console script to notice the errors, and the lack of a printed value from step 2.
Export with any debug template and notice the opposite behavior.

Console output should look like this (release on top, debug on bottom)

Minimal reproduction project
PropertyRelease.zip

For some additional context, I went back and recompiled a release template for 4.0 alpha 1, and this was occurring then as well. This bug isn't specific to beta 6.

This looks to be the same as #55045.


Add env var for dev to set the doc repository to use

If the directory does not exist:

clone the repository into it

What would be the use of this static directory?
I'd actually think it's more useful to have an env var that determines where to clone from, just like we have for the Booster Catalog.

What would be the use of this static directory?

The goal is for devs to avoid having to clone the reposioty at every server start. Though it may be a good idea to wait and see if it's transparent enough and doesn't slow us down in which case it wouldn't be necessary at all.

I'd actually think it's more useful to have an env var that determines where to clone from, just like we have for the Booster Catalog.

I didn't propose it because I liked the fact that the repository clone is IT tested and cannot be changed without a new build.
If you think some developers/docs people may have a need to change it often I definitively think we should add this env.
Thinking about it more globally; It seems we have a lot env for testing only, it may be a good idea to add a safety so that those cannot be used in prod preview or production. The goal would be to be sure that what is tested is what is used and the test envs only affect developers.

Well the env for the booster catalog is definitely not for testing, it's there so a user can use their own catalog if they want to. The fact that we hard-code the docs repo is just wrong IMO.
And the second advantage is that you could actually point it to a local clone and have the advantage of a static directory. Two birds with one stone!

@quintesse That is not not making sense so I have to not not not disagree :)
I changed the title


Adapt csv files for release version 1.2.2

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.92%. Comparing base (271f583) to head (1362745).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##           release-1.2    #3599   +/-   ##
============================================
  Coverage        58.92%   58.92%
============================================
  Files              323      323
  Lines            18200    18200
============================================
  Hits             10725    10725
  Misses            6301     6301
  Partials          1174     1174

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


Add auto-rebuild feature for webpack project

Followed with #141 , now we need to introduce webpack-dev-server maybe for auto-rebuilding on development.

On the root directory, type the following command to start the webpack-dev-server:
npm run start

The server will be run on localhost:9000.
Press Ctrl + C to stop the webpack-dev-server.
Due to the complexity of HMR, we only enable the auto-rebuild feature, ie. you still need to refresh the page manually to see the updated change.
But you can combine with other live server tool to fulfill the need of hot-reload feature, if you like.

@cwpeng Please review.

@cwpeng Please review agian.
The auto re-bundled problem is solved by adding writeToDisk setting to webpack config.
This setting seemed to be changed from devServer.writeToDisk to devServer.devMiddleware.writeToDisk without further detailed description.
But anyway IT WORKS! And magically the hot reload function takes effect too with no more settings changed!!
And with the function filter for the writeToDisk setting like filePath => !/.hot-update./ .test(filePath), we can further avoid hot-update files being written to the disk and save the clean step!
Now we don't need to manually build the bundle for pushing the updated file to GitHub. Just run the webpack dev server, update the code, check it right on the webpage, then we can just do the git push safe and sound. The world is saved and everyone is happy ever after, without any curiosity to know WHY :p.
(You can try to figure it out if you want: https://webpack.js.org/configuration/dev-server/#devserverdevmiddleware)

Thank for saving my world. XDDDD


milestones?

we should prioritise the backlog, any traffic just sees chaos

Agreed. I'll remove milestones and we'll re-organise them.


Low req per second, any way to fix this?

Hello!
I'm trying this amazing tool against a lot of domains and i noticed that in some domains it goes as a really high number of request per second (sometimes 300 or 2000 or 3000) and sometimes as low as 2 req/sec which makes my attack completely slow.
i noticed that i would have to add the -maxtime 360 option to not make it slow as fuck but i wanted to ask. Is there any way to speed this up? would speeding it up make more errors? i usually get 300-400 errors out of 6000 requests.
This is not a bug, it is a question.

@joohoi Would it go faster in general if this parameter in simple.go would be in the kilobyte range? //Download results < 5MB
const MAX_DOWNLOAD_SIZE = 5242880

The behavior described in the original post is extremely context specific. It's more likely that there are some other issues causing the slowdown than large downloads. It's very easy to check and verify using curl / ffuf -od option. Also lowering the MAX_DOWNLOAD_SIZE will render many of the matchers / filters unreliable or unusable (because you cannot reliably use regex matching for content, cannot determine lines / words amounts from the content).
That said it might be beneficial to add a new CLI flag to a always skip downloading content, but it should come with a proper warning (and maybe erroring out if -mw / -fw, -ml / -fl is in use).

I could have a look at the implementation of that cli switch. If ok, what would the switch be called?

I could have a look at the implementation of that cli switch. If ok, what would the switch be called?

I created a new issue #178 to properly keep track on this work

FYI, I think this can be closed.

Change the VM adapter to bridged.


Change text of cancel and ok buttons

Hi, I hope everything is ok, I'm using this package and I was asked to change the text of the "cancel" and "ok" buttons, but I can't find a way. Could you tell me if there is an option available?

Hi! Are you asking because of localization or because you'd like to use completely custom texts?
At this moment the buttons are using native Flutter translations from MaterialLocalizations.of(context) - .okButtonLabel and .cancelButtonLabel.


Porting apps: Porting your libraries

Issue tracking new topic at http://dotnet.github.io/docs/porting/libraries.html

same content as #301

Although I was tracking an existing PR with #212, I think I'm going to break that up until multiple PRs and track with these issues.
Here's what I'm thinking:

Prerequisites

Getting 3rd party dependencies in order
a. When using NuGet, what folders to look for
b. When not using NuGet, use ApiPort (this will involve jumping around the article a bit)
c. Recommendations for when your 3rd party dependencies don't run on .NET Core

If applicable, porting to .NET Framework 4.6.1
a. Using ApiPort and analyzing the results
b. If there are any incompatibilities, recommendations for fixing (this is pretty unlikely)
c. How to upgrade to next version in VS

Using ApiPort to analyze .NET Framework 4.6.1 ---> .NET Core
a. Using ApiPort
b. Guidelines for dealing with incompatible APIs used
c. Guidelines for a plan of attack if you think you can move forward

Before you deal with code: .NET Core project model, project system in VS, netstandard
a. Quick overview of the project model and project.json, which is going to be the center of things as far as building and dependencies are concerned (link to further docs)
b. How best to pick between xproj and csproj
c. Understanding what netstandard version to use and how this is influenced by your 3rd party dependencies and .NET Framework APIs used.  (Link to further docs on netstandard)

If applicable, getting your tests ported over
a. If using xUnit, how to set that up and run tests
b. If using NUnit, how to set that up and run tests

Getting your project to the point where all you need to deal with is your own code
a. Based on your needs, pick the correct project type in VS and the correct netstandard version
b. Entering the correct project.json incantations so everything works
c. Referencing this project in the relevant test project(s)
d. Copy code over, work out compile errors, run tests, repeat!

Getting the library up on nuget.org?
Link to the next article, which is getting a .NET Core library into a NuGet package that can then be distributed via NuGet.

/cc @terrajobst @richlander

@cartermp Can we close this issue? It looks like all but the NuGet PR is merged or closed.
And, I'd like to track the NuGet PR on a different milestone (this is the only issue not completed for the //build milestone)

Yep, good to close!


Shapes sometimes draw in constrained mode despite lack of Shift key

history-779.txt

thanks for these bug report files! super useful

I think this might get solved with a new keyboard-extra solution, attempted with this branch:
https://github.com/thebritican/goat/tree/use-experimental-keyboard-extra

I've not been able to repro after this PR was merged. Can you confirm it's good now, @alanhogan ?


Model weights on Hugging Face

Hi! This is really amazing work! It would be awesome to have the model weights shared on Hugging Face. Here's some more information how to do so: https://huggingface.co/docs/hub/models-uploading
Happy to help with any questions!

Thanks for the suggestion. Huggingface is doing a great job with hosting machine learning models. However, I think this model is a bit different from your typical pytorch model. Just hosting the weights on huggingface will not really help any of our users.

Thanks for your response! Curious to hear why this model is different from your typical pytorch model? I get asked quite frequently about medical image segmentation models, and I think even having this model searchable within the hub would be useful for others in the ML community who might not yet be aware of this awesome work. If it's okay with you, I'd be happy to help put the weights for the TotalSegmentator models on HF (of course, giving you full and proper attribution, linking back to this repository, including the appropriate license, etc). Let me know what you think.

@katielink  to my knowledge, inference of all organs requires running inference for several models, further, a bit of a post processing is involved (see https://github.com/wasserth/TotalSegmentator/blob/master/totalsegmentator/nnunet.py#L331 ).
On top of that, you'll need a a decent GPU, or else inference will take a while (i'm guessing, 20 to 40min, depending on your CPU spec)  So if people are going for the huggingface inference endpoint route, they will need to up the instance spec 💸.  Most people/researchers using this have a tight budget and/or likely already have on-prem gpu/HPCs.
That said, I've gave it a quick shot (see below sample huggingface space/gradio impl), since I'm curious if the free hardware can execute totalsegmentator without error.  I think you can convert it to a "huggingface model repo" with custom handler. One lacking logic is getting the weights to be cached in the deployed docker container / model repo.  Also, I have to add the "--fast" flag, or else it will timeout at 30min when deployed to huggingface.co using the free instance (2cpu ram16gb).
--
requirements.txt: #TODO add version
totalsegmentator

app.py:
import os
import sys
import tempfile
import subprocess
import gradio as gr
import totalsegmentator

PREPARE_EXEC_PATH = os.path.join(os.path.dirname(totalsegmentator.__file__),"download_pretrained_weights.py")
if os.environ.get("WEIGHTS_CACHED") != "TRUE":
    print('downloading pretrained weights...')
    subprocess.call(f"python {PREPARE_EXEC_PATH}",shell=True)
    os.environ["WEIGHTS_CACHED"]="TRUE"

THIS_DIR = os.path.dirname(os.path.abspath(__file__))
EXAMPLE_NIFTI_PATH = os.path.join(THIS_DIR,'files','sample-image.nii.gz')

def myfunc(file_obj):
    file_list = []
    input_file_path = file_obj.name
    with tempfile.TemporaryDirectory() as tmpdir:
        output_folder_path = os.path.join(tmpdir,"segmentations")
        cmd_str = f"TotalSegmentator -i {input_file_path} -o {output_folder_path} --fast"
        subprocess.call(cmd_str,shell=True)
        if os.path.exists(output_folder_path):
            file_list = os.listdir(output_folder_path)
        return {"status":file_list} # TODO: maybe add papaya js element to view nifti

if __name__ == "__main__":
    demo = gr.Interface(myfunc, ["file"], "json",examples=[EXAMPLE_NIFTI_PATH],cache_examples=True)
    demo.queue().launch(debug=True,show_api=True,share=False)


Move peer discovery into an isolated plugin

What was wrong?
Currently the peer discovery runs in the networking process among a lot of other things. Moving it into an isolated plugin should:

increase efficiency of syncing as the peer discovery probably takes a fair share of computing power
increase decoupling as it breaks up the networking code into smaller pieces
improve reusability as the new discovery can be accessed from any other interested party since it is integrated with the event bus

How was it fixed?
Not yet. Just opening a PR with junk code.
Cute Animal Picture

Another reason to wait on a fresh Lahja release before this gets merged (apart from it still being WIP)
https://github.com/ethereum/lahja/pull/24

Needs to be rebased now that https://github.com/ethereum/py-evm/pull/1530 is merged to use logger.debug2 instead of logger.trace

This should definitely wait until we have more clarity around #1583


Documentation for SwaggerToTypeScriptClientGenerator is missing

It seems that someone edited out all documentation for SwaggerToTypeScriptClientGenerator.
Is is possible to revert back to older revisions to have it available for users?
https://github.com/RicoSuter/NSwag/wiki/SwaggerToTypeScriptClientGenerator#extended-classes-and-extension-code

Any chance of getting this documentation back?  We are trying to integrate NSWAG into our Angular2 App and could use some guidance and documentation.

I'm wondering more and more whether it's a good idea to keep the wiki open...


Save event raised once per region

The editors 'save' event is called once per region on the page, rather than once overall, which means that any saving strategy needs to take this into account.
If I have three regions on the page, the save event handler is called three times.
I am using the following code:
    function link(scope, element, attrs){

            // Initialise the editor
            var editor = new ContentTools.EditorApp.get();
            editor.init('[ng-editor]', 'ng-editor');

            // Bind a function on editor save
            editor.bind('save', function(regions, autoSave){
                console.log("saving");
                ContentService.update($stateParams.pageId, regions);
            });
    }

Also, if I change the above code to use the 'saved' event instead, the event handler is never called.
Any ideas?

It occurs to me on the walk home that the reason its firing three times is because I am binding three times - but I cant find any docs on clearing or checking existing bindings...?

Hi @richardprice,
As you've highlighted this behaviour is caused by binding to the save event multiple times. It looks as if you are using an older version of the library as bind is no longer the correct method call and addEventListener should be used (this changed occurred in release 1.2.0).
In the releases prior to 1.2.0 you can use the unbind(eventName, func) to remove a previous binding however I'd recommend you update your library and that you checkout the documentation for the new event methods http://getcontenttools.com/api/content-tools#component-ui, see:

component.addEventListener(eventName, callback)
component.createEvent(eventName, detail)
component.dispatchEvent(ev)
component.removeEventListener(eventName, callback)

Heh, typical, started using CT about a month ago :)
Is there anyway to check if a particular component has a particular event listener for a given event? Or will I just have to clear the event of any listeners just to make sure?

Hi @richardprice - there's no officially documented way but actually it's pretty simple to do at the moment:
if (editor._binding['eventName']) {
    ...has event binding for ...
}

If you do upgrade to the latest version of the library then take a look at the getting started and saving strategies pages again.

@anthonyjb Thats fantastic, thank you very much for that!  I'm using the example angular directive code, so that might be a worthwhile addition to that as Im sure many people will run into the same issue (multiple bindings).
Just a small correction to the code:
if (editor._bindings['eventName'] && editor._bindings['eventName'].length > 0) {
    ...has event binding for eventName...
}

(the binding collection/object is component._bindings)

@andyjbgm
Where I need to write my event listener , please see my code , i have done this way. Its not working, please tell me where I am going wrong. Also, ev.regions is giving undefined. How can I use 'this' inside $scope.apply.
app.directive('ngEditor',  function(){

    function link(scope, element, attrs){

        // Initialise the editor
        scope.editor = new ContentTools.EditorApp.get();
        scope.editor.init('[ng-editor]', 'ng-editor');

        // Bind a function on editor save
        if (scope.editor._bindings['saved'] && scope.editor._bindings['saved'].length > 0) {

        }
        scope.editor.addEventListener('saved', function (ev) {
            scope.$apply(function(){
                scope.regions = ev.regions;
                console.log(scope.regions);
                console.log(ev.detail().regions);
            });
        });
    }
    return {
        link: link
    }
})


Allow multiple games per server

This will require support from olmod to send the server stats to include a game ID so that the tracker can tell the games apart.

Add port = Server.GetListenPort() to TrackerPostStats() calls

@tobiasksu Does this look right to you?
        public static void TrackerPostStats(JObject body)
        {
            body.Add("port", Server.GetListenPort());
            TrackerPost("/api/stats", body);
        }


Create Report Class

Provide a module for the output of results.
(Gitlab Issue 99, created by @newLabAspect)

#88 introduced a minimal version for representing reports. This can and should be extended, but I'm closing this for now.


Update main.go remove chin.Dots

I removed chin.Dots from chin.New() function, When i copy and paste your main file for test, this give me error, and after deleting chin.Dots it work's fine. Approve this if you think it's all right. Thank you! :)

just rechecked, Dots is working. the chin.New() supports optional variadic of type chin.Set and chin.Dots matches the type. so it must work


[FEAT] User Deactivation

Description
This endpoint allows an authenticated user to deactivate their account.
Acceptance Criteria
AUTHENTICATION

The endpoint must only be accessible to authenticated users.
Requests without a valid JWT token should return a 401 Unauthorized error.

{
    "Authorization": "Bearer <jwt_token>"
}

HTTP METHOD AND URL

The endpoint must accept only PATCH  requests.
The URL endpoint should be /api/v1/users/deactivate

EMAIL

Users should receive an email that their account has been successfully deactivated if the response is a success response.

REQUEST BODY

The request body contains a JSON object with:

reason (optional): A string indicating the reason for deactivation.
confirmation (required): A boolean that must be true to confirm the deactivation request.

Purpose
The purpose of the user deactivation endpoint is to allow users to deactivate their accounts, enhancing user control and data privacy.
Requirements

[x] Ensure the user is authenticated with a valid JWT token
[x] The endpoint should accept only PATCH requests
[x] The user status on the database should change to deactivated
[x] Users should receive an email that the account has been deactivated.

Expected Outcome
A user should be deactivated and should receive an email confirming that the account has been successfully deactivated.
Example
[PATCH] /api/v1/users/deactivate
Success
Request Body
{
    "reason": "No longer need the account",
    "confirmation": true
}

Response
{
    "status_code": 200,
    "message": "Account Deactivated Successfully"
}

Errors
Request Body
Missing data
{
    "reason": "No longer need the account",
}

Response
{
    "status_code": 422,
    "error": "Invalid input",
    "detail": [
        {
            "confirmation": "Deactivation confirmation is required"
        }
    ]
}

Request Body
Confirmation set to false
{
    "reason": "No longer need the account",
    "confirmation": "false"
}

Response
{
    "status_code": 400 ,
    "error":  "Confirmation needs to be true for deactivation"
}

Request Body
If the user has already been deactivated and they try to deactivate their account again.
{
    "reason": "No longer need the account",
    "confirmation": "true"
}

Response
{
    "status_code": 400 ,
    "error":  "User has been deactivated"
}

Request Body
Invalid or expired access token.
{
    "reason": "No longer need the account",
    "confirmation": "true"
}

Response
{
    "status_code":  401,
    "error":  "Could not validate user credentials"
}

Request Body
Unexpected error or server error
{
    "reason": "No longer need the account",
    "confirmation": "true"
}

Response
{
    "status_code":  500,
    "error":  "An unexpected error occured"
}

TEST
Unit Test

[ ] Test 200 status code and success message for successful deactivation
[ ] Test missing or invalid confirmation field
[ ] Test for unauthorized access (No JWT token)
[ ] Test email is sent to the user upon deactivation

End To End Test

[ ] Test the full flow of the deactivation process.

This is the link to the Approved issue


Can I adjust priority of diagnostics of harper-ls?

Thank you for your efforts ,
Currently the diagnostics of harper-ls is displayed first than other lsp and gitsigns,
Can I adjust this diagnostics priority of harper-ls?

I found the answer
local create_custom_handler = function (priority)
  return function (err, result, ctx, config)
    config = config or {}
    config.signs = config.signs or {}
    config.signs.priority= priority
    config.severity_sort = true -- if true, priority is set automatically
    config.virtual_text = false
    vim.lsp.diagnostic.on_publish_diagnostics(err, result, ctx, config)
  end
end

lspconfig.lua_ls.setup({
  -- ...
  handlers = {
    ['textDocument/publishDiagnostics'] = create_custom_handler(3)
  }
})

It can set sign priority of each lsp independently.


feat(remix-dev): add extraPathsToWatch option

This adds the possibility to watch extra directories/files/globs in dev mode.
This should make integration with mono repos lix NX easier.
Will spend more time documenting and testing if necessary and if feature is approved.
Closes: #2983

[x] Docs
[ ] Tests

Yes, I don't mind both option will fulfil my needs :)

Chiming in from the Nx side, I think the function might work better for us as well. That way, we could use the project graph to determine what other directories need to be watched instead of having a static list of directories we'd need to update.
I'm curious how often this config is re-evaluated. If a dev imports from a new lib that's not yet added to the watch, are we able to supply new directories to watch over time? Or do you need to re-start the dev script entirely to get the updated config?

Thinking through this a little more, I have a specific, rather than a hypothetical example of where the function would have advantages over the array.
We can look at how @nrwl/react-native does something similar working with metro-resolver in with-nx-metro.ts
let watchFolders = config.watchFolders || [];
  watchFolders = watchFolders.concat([
    join(workspaceRoot, 'node_modules'),
    join(workspaceRoot, workspaceLayout().libsDir),
    join(workspaceRoot, '.storybook'),
  ]);
  if (opts.watchFolders?.length) {
    watchFolders = watchFolders.concat(opts.watchFolders);
  }

  watchFolders = watchFolders.filter((folder) => existsSync(folder));

To provide all of the directories to watch, we use some utility functions from Nx to determine where the libs folder is actually located. We do this same sort of thing with a static array, it's just that we could introduce issues if the libs location ever changes or sin't set correctly to begin with. We also include changes in node_modules and .storybook directories to rebuild on those changes.

Make sense, to me, maybe @christophertrudel can you create a PR with your commit? Then I can close this one. Just make sure to link https://github.com/remix-run/remix/issues/2983 to it :)
Would be nice to have some remix developer feedback on that too :)

This solution works for my project. Hope this one or this https://github.com/christophertrudel/remix/commit/763b6acdabda6a855c3776df698159cba95f68d4 would be merged soon.

Make sense, to me, maybe @christophertrudel can you create a PR with your commit? Then I can close this one. Just make sure to link #2983 to it :)
Would be nice to have some remix developer feedback on that too :)

https://github.com/remix-run/remix/pull/3188

Closed by #3188


M3-5946: Cypress 10 clean up - refactor plugin file

Description 📝
What does this PR do?
This refactors our Cypress plugin code by removing plugins/index.js and moving that functionality to cypress.config.ts as described in Cypress's Migration Guide. It also removes some plugin logic that isn't needed, and upgrades the dotenv dev dependency for TypeScript support.
How to test 🧪
What are the steps to reproduce the issue or verify the changes?
yarn cy:run

You don't necessarily have to run the entire set of tests; the plugin code is responsible for loading the .env file, so if there is a problem you will quickly observe either an error loading the file, or test failures due to API 403 responses.

Ran this command and successfully kicked off the tests without any immediate errors. Changes look like they're in line with the migration guide. I'd make an approving review, but don't have the access to do that at the moment.


Announcement of SRE SIG

Thank you for blog post. I updated the formatting a little to align it with the README in the SRE Git repository. I'm now reviewing the changes in the SRE Git repository before approving this one.

@stianst - please review when you have the time. Does it align with your plans around the SRE SIG? Thanks!


How to De-Register device from AMAZON Account in AVSSDKV1.10

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

Hi, I would like to de-register the device from amazon account. But when i do de-register in the web, i am not getting any communication to the AVS SDK. We have enabled the debug logs in SampleAPP, but it is not showing any prints, when the device is de-registered.
What is the expected behavior?

We should get some communication to the AVS SDK, when device is de-registered, so that we can shutdown the app?
What behavior are you observing?

Provide the steps to reproduce the issue, if applicable:

Run the SampleApp, and go to the amazon account and de-register your device.
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <x.y.z>

AVS SDK V 1.10
Tell us what hardware you're using:

[ ] Desktop / Laptop
[ x] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ x] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi, We would like to de-register the device from amazon account. When we de-register, we need to delete the database and re-authorize the device. But we are not seeing any prints or state change when de-registered on Amazon Account. (DEBUG9)
Do we need to enable any flags, to get the communication from Account to AVS SDK ?
Please suggest.

Hello ShivaKrishnaCharka,
Thank you for your question.  No special flags are needed to enable this notification.
What version of the SDK are you using?  1.10 was enhanced to infer de-authorization by providing notification through AuthObserverInterface::onAuthStateChange() with a status of AUTHORIZATION_EXPIRED.
Note that this notification may not be immediate.  The client only discovers that it has been de-registered when making a request to AVS fails with a 403 and subsequent attempts to refresh the authorization token indicate that the refresh token is invalid.  So, for example, if the client is not actively interacting with AVS it may take about five minutes for the failure of a keep-alive ping to AVS to trigger this notification.
Best regards,
-SWH

Hi @ShivaKrishnaCharka , I hope that answers your question. I'll be closing down this thread but feel free to reopen if you have other questions regarding this.


Move request sync logic into GoogleConfig

Description:
This moves the request_sync logic into same place as cloud version has the request sync logic.
I have been forced to extend base class to send along the useragent identifier. Since that is supported in the local case.
Checklist:

[X] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.
[X] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

I've dropped return value on request_sync calls. Nothing ever used it. Also cloud could raise a timeout error, so exception returns are already in place for error situations. Seems cloud also should use raise for status.

Looks good.

@balloob had to do small refactor of the way scheduled sync is handled to allow for the case of required agent_user_id. It now keeps track of scheduled sync calls per agent as well. I'll merge in a few days if you have no objections.


[LC0005] Does not trigger on global objects

The wrong casing correctly detects when things like Codeunit::X and Page::X are incorrectly capitalised, but this doesn't seem to work when they are used as objects instead of enums (for example, when the .Run() function is called on those objects).
For example:

Codeunit.Run(Codeunit::X) - correct casing, correctly no warning
Codeunit.Run(CODEUNIT::X) - incorrect casing on the parameter, correctly shows warning
CODEUNIT.Run(Codeunit::X) - incorrect casing on the "variable", but no warning

Thank you reporting this and should now be resolved in the release version of v0.30.20 of the LinterCop.


Support latest go and js ipfs versions

summary of changes

reconnects go-ipfs nodes on after checking for suspect peer stream config
makes tests only use localhost for their swarm addresses
removes an ipfsConfig option that doesnt play nice with js-ipfs 0.52.0

this pr will

allow support for js-ipfs 0.52.0
test ipfs nodes will only use localhost for swarm address
reduce frequency of go-ipfs test timeouts significantly

@tabcat It looks like this might be obviated in part by #31 but there are other improvements here we should consider. I'm thinking I merge #31 first and then we revisit this?

I agree; the only thing needed to be changed for the new versions was the ipfs node's start option. The two other changes here are either not actually needed or can be changed in a separate pr.


Example texts are included in django.po

There are example texts in static/locale/ja/LC_MESSAGES/django.po.
https://github.com/zulip/zulip/blob/4e33b00/static/locale/ja/LC_MESSAGES/django.po#L45-46 :
#: docs/_build/html/_sources/translating.txt:91
msgid "English text"

https://github.com/zulip/zulip/blob/4e33b00/static/locale/ja/LC_MESSAGES/django.po#L49-L50
#: docs/_build/html/_sources/translating.txt:103
msgid "This string will have"

https://github.com/zulip/zulip/blob/4e33b00/static/locale/ja/LC_MESSAGES/django.po#L53-L54
#: docs/_build/html/_sources/translating.txt:103
msgid "inside"

https://github.com/zulip/zulip/blob/4e33b00/static/locale/ja/LC_MESSAGES/django.po#L57-L60
#: docs/_build/html/_sources/translating.txt:110
#: docs/_build/html/_sources/translating.txt:112
#, python-format
msgid "This string will have %(value)s inside."

I think that we don't need to process docs/ directory.

Agreed, we should skip the docs/ directory.  Thanks for noticing this @kou!
@umairwaheed do you want to fix this one as well?

sure.

Fix merged, thanks again @kou for the report and @umairwaheed for the fix!


Fix- Image alignment in table widget cells

Description
Fix image alignment in table widget cells
Fixes #3967
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Cypress test added to test table cell alignment

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
No changes to code coverage between the base branch and the head branch

@vicky-primathon as mentioned in the issue the fix still does not cover type URL, The alignment doesn't apply. However, it works fine for the type image now.

We will be talking up URL alignment issue with https://github.com/appsmithorg/appsmith/issues/4135


Correction for raw affiliation CRAL, Observatoire de Lyon, CNRS UMR 5574, 69561 Saint-Genis Laval, France

Correction needed for raw affiliation CRAL, Observatoire de Lyon, CNRS UMR 5574, 69561 Saint-Genis Laval, France
raw_affiliation_name: CRAL, Observatoire de Lyon, CNRS UMR 5574, 69561 Saint-Genis Laval, France
new_rors: 03j307b25;029brtt94;02feahw73;0084x3h80
previous_rors: 03j307b25;029brtt94;02feahw73
works_examples: W3101089313;W3100601934
contact: dc8adc8b4d1d329e955f78b5ef2fc373:e0635e84cdaede69fa2a2908e630 @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


register disposable when listen didChangeTextDocument

Thanks for the contribution! I just published the updated version 1.10.2.


EEGLAB import: empty channel positions were ingnored, which could lead to a shift in channel positions

Although empty channel positions were turned to np.nan such channel positions were later ignored leading to a shift in channel positions (in my case for example the O2 channel moved to where EOG should be).
On master (look at the eog channels and the occipital channels):

on this PR:

I'll add a test tomorrow and whatsneew, I'm just poking the CIs now to see whether the check I removed was actually needed anywhere.
Followup: should nan-position channels be plotted at 0, 0 when using inst.plot_sensors()?

Tests are ok, apart from one weird error on linux / pipi / py3.9:
X Error of failed request:  BadWindow (invalid Window parameter)
  Major opcode of failed request:  25 (X_SendEvent)
  Resource id in failed request:  0x400002
  Serial number of failed request:  71
  Current serial number in output stream:  71

Tests are ok, apart from one weird error on linux / pipi / py3.9:
X Error of failed request:  BadWindow (invalid Window parameter)
  Major opcode of failed request:  25 (X_SendEvent)
  Resource id in failed request:  0x400002
  Serial number of failed request:  71
  Current serial number in output stream:  71

Ignore that, that test bombs sometimes. We can restart if this happens once in MRG state, ignore it in the meantime (the other runs should be sufficient)

Ignore that, that test bombs sometimes. We can restart if this happens once in MRG state, ignore it in the meantime (the other runs should be sufficient)

I'll finish this today, sorry for delays.

I'll finish this today, sorry for delays.

@mmagnuski Care to finish this one up? :)

Followup: should nan-position channels be plotted at 0, 0 when using inst.plot_sensors()?

I think they shouldn't appear at all …

@hoechenberger Yes, my "tomorrow" seems to have been an approximation of 1.5 month :/. But I finally have more time this and the following week so I'm going to be more active here (meaning: mne, not just this PR). :)

Hi, I just got bit by it again, so I will remember to finish this now. :)

So, "tomorrow", I hear you say? 🤣😅 Just kidding, it's great you're picking this one up again!!! 💪💪💪

I didn't request the reviews, this is due to change in files from the catastrophic PR update. Is there anyway easy way to save this PR or is it just best to open a new one?

I'd just open a new one instead of spending 20+ min trying to salvage this one :)

this should probably work for fixing:
git checkout main
git pull upstream main
git checkout -b bak
git cherry-pick <cherry pick your commits that you actually want from fix_eeglab_chan_pos>
git checkout fix_eeglab_chan_pos
git reset --hard upstream/main
git cherry-pick <cherry pick the commits you put on "bak">
git push --force
<verify all is well>
git branch -D bak

CI failures are due to timeouts on Azure.

Thanks @mmagnuski !

🚀


Support GHC 9.2.*

When built with GHC 9.2.3 and 9.2.4, encountering error
[ 83 of 104] Compiling Monomer.Widgets.Containers.Dropdown ( src/Monomer/Widgets/Containers/Dropdown.hs, dist/build/Monomer/Widgets/Containers/Dropdown.o, dist/build/Monomer/Widgets/Containers/Dropdown.dyn_o )

src/Monomer/Widgets/Containers/Dropdown.hs:423:3: error:
    • Illegal equational constraint Control.Lens.At.Index a2 ~ Int
      (Use GADTs or TypeFamilies to permit this)
    • When checking the inferred type
        scrollListInfo :: forall {s} {a1} {b} {a2} {p}.
                          (L.HasWidgetId s a1, L.HasPath s b, Control.Lens.At.Index a2 ~ Int,
                           L.HasChildren p a2, Control.Lens.At.Ixed a2,
                           L.HasInfo (Control.Lens.At.IxValue a2) s) =>
                          p -> (a1, b)
      In an equation for ‘makeDropdown’:
          makeDropdown widgetData items makeMain makeRow config state
            = widget
            where

This looks like the issue that was fixed in #159.
There is not currently a release version with the fix, but you can might be able to use the unreleased code. E.g. with stack, put something like in your stack.yaml:
extra-deps:
  # for the latest fixes
- git: "https://github.com/fjvallarino/monomer"
  commit: "7b1d59d754d75588aae1097a231712264cf61719"


[npm] missing dependencies

Seems like the setup of the Jenkins build introduced new dependencies which are not listed in package.json. As a result, I can no longer run the tests locally.
Missing packages are

jenkins-mocha
tsc-watch

Also, the test coverage script config has gone.

I think this is not due to missing dependencies, but rather about running the locally installed executables instead of relying on the globally installed ones. We should fix the scripts in package.json.

Seems to have been an invalid intermediate state on my side.
Also checked that npm prepends ./node_modules/.bin on the PATH when runnign scripts, such that binaries from the global config are shadowed.


CopyFilesOverSSH:  Update nested dependencies to fix copying to Solaris 11

Currently, the CopyFilesOverSSH task uses scp2 which depends on ssh2 which uses ssh2-streams.  About 2 years ago, a change went into ssh2-streams to fix a bug that caused the SSH connection with Solaris 11 to hang.  Unfortunately, scp2 is not well maintained and it still uses an older version of ssh2 that doesn't use the updated ssh2-streams package.  This PR uses npm shrinkwrap to specify the nested dependencies for our task to force scp2 to use the updated ssh2 0.8.2 used elsewhere in the task.  Local testing shows compatibility and successfully copies files over SSH from Windows to a Ubuntu VM on Azure.

Any action on this? I am coming across the same issue with wpengine.


Outdated images in opendatahub-operator breaks ISVC reconcile

Summary
I am encountering errors while running a simple smoke test against KServe. Specifically, I am seeing reconcile errors from the ODH Model Controller and am unable to access the model.
Details
It appears that the manifests for odh-model-controller are referencing an outdated image (v0.12.0-latest) created 2 months ago. I suspect this might be related to the issue (see attached screenshot for reference).
Steps to Reproduce

Deploy the latest incubation of odh-operator sources using manifests from here.
This deployment uses the image referenced here.
Run the smoke test against KServe.
Observe the reconcile errors.

Observed Behavior
The following error is logged during the reconcile process:
{"controller": "inferenceservice", "controllerGroup": "serving.kserve.io", "controllerKind": "InferenceService", "InferenceService": {"name":"sklearn-v2-iris","namespace":"kserve-test"}, "namespace": "kserve-test", "name": "sklearn-v2-iris", "reconcileID": "b23200b9-879d-4075-a88c-a7f363e2cd47", "error": "2 errors occurred:\n\t* servicemeshmemberrolls.maistra.io \"default\" is forbidden: User \"system:serviceaccount:opendatahub:odh-model-controller\" cannot update resource \"servicemeshmemberrolls\" in API group \"maistra.io\" in the namespace \"istio-system\"\n\t* failed to load serviceHost from InferenceService status\n\n"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	/opt/app-root/src/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:329
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	/opt/app-root/src/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:274
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	/opt/app-root/src/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.14.6/pkg/internal/controller/controller.go:235

Workaround
Switching to the v0.12.0 image (which is actually newer than v0.12.0-latest as of the time of writing this issue) resolves the reconcile errors, and the SMM is created successfully.

AFAIK, CI was fixed, and the image reference was also fixed.
Closing as done.


fix: avoid installing the registry multiple times

While the fill-test-ci-values target's dependency on install-registry was correctly manipulated by the SKIP_INSTALL_REGISTRY variable, the install-lagoon-build-deploy target had a hard-coded dependency on install-registry. The result was that the registry would get installed multiple times anyway via the fill-test-ci-values target via an indirect dependency.
Fix that by removing the dependency that install-lagoon-build-deploy has on install-registry when SKIP_INSTALL_REGISTRY=true.

running make TESTS=[nginx] k3d/test in uselagoon/lagoon. Not hugely noticeable difference. I think it is probably worth merging though
before:
real	21m31.993s
user	0m46.142s
sys	0m8.614s

with this branch:
real	20m11.619s
user	0m44.983s
sys	0m8.093s

Only shaved off a small amount of time


Fix incorrect comparison

Modify to use the specified method for comparison between BigNumbers.
Using >= may return incorrect values.
> a
BigNumber { _hex: '0x2386f26fc10000', _isBigNumber: true }
> b
BigNumber { _hex: '0x3d570965c1', _isBigNumber: true }
> a >= b
false
> a.gte(b)
true

Thanks @haryu703, I have the same fix in #76 and will merge that instead due to the CLA stuff.


Elasticsearch

More simple, updated version of #254
Still a few outstanding issues from the checklist in #254 which I'll open as separate issues.

Closes #250

Latest commit closes #263

I think we should either keep the TaskGetter or TaskReader interface. There is no need for both to exist.


"Version History" button cannot be localized

Describe the bug
The string for the "Version History" button shown is excluded from all localization files, and thus always stays in English:

Steps to reproduce
Menu → Check for Updates…
or alternatively:
Menu → Preferences → About → Check for updates…
Expected behavior
The "Version History" button string should be localized, according to the espective lproj.
Environment Information
- macOS version: macOS 12.4
- Mac model: MacBook, Late 2020
- MonitorControl version: 4.1.0 Build 7034
- Monitor(s): —
- Apple Silicon/M1 (yes or no): yes

The reason for this is that it is part of Sparkle which is a third party component. You might want to contribute to that project: https://github.com/sparkle-project/Sparkle
If they release an update, it will eventually find it's way to MonitorControl. :)


pdf-commons: Exclude commons-io dependency from batik-transcoder

The pdf-commons library has a dependency to org.apache.xmlgraphics:batik-transcoder:jar:1.12. This dependency brings in commons-io:commons-io:jar:1.3.1. We want to use the 2.4 version. Exclude commons-io from batik-transcoder and explicitly include commons-io v 2.4.

Fixed in https://github.com/idsec-solutions/signservice-commons/pull/20.


Fixed opam-depext command example

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

opam depext without the dash works for me since opam-depext is some sort of opam plugin (I'm not familiar with opam plugins). perhaps it is like git, where git-foo commands can be run as git foo?
what version of opam are you using?

Thanks for the contribution!
The opam docs do use opam depext so I might leave it be for now. If you think I'm wrong, let me know why and I'll reopen!


Is there an easy way to store password based key derivation parameters?

In case I'd like to store the information for verifying a password, there's a nice function crypto_pwhash_str.  My use case is I'd like to store parameters required to derive a key to decrypt a particular secret key.  There does not seem to be anything "ready-to-use" available.
My question: is there anything that would make such a helper unreasonable  (for an arbitrary definition of "unreasonable")?

There's nothing unreasonable about a small helper like that, however it's a wuite simple function to write do and for that reason I'd like to not add it to the sodiumoxide API (in order to not diverge too much from the libsodium API)

If it's that simple a function, why there crypto_pwhash_str exists?  The latter stores:

salt
opslimit
memlimit

And this helps a lot while just verifying a password...

The simple answer is that crypto_pwhash_str exists in sodiumoxide since it exists in libsodium. If libsodium were to add a similar function for key derivation I would add bindings to it in sodiumoxide. As you say yourself though, the only parameters needed to be able to derive the same key is salt, opslimit and memlimit, so a function like (in pseudo-rust)
fn store_stuff(salt, opslimit, memlimit) -> Vec<u8> {
   let mut v = Vec::with_capacity(size of salt + size of opslimit + size of memlimit);
   serialize_salt(salt, &mut v);
   serialize_opslimit(opslimit, &mut v);
   serialize_memlimit(memlimit, &mut v);
}

should do the trick...

@dnaq: I'm sorry, I should have realised that this issue should have been raised with the upstream library.
Thank you for the snippet.


Bittrex api >> key

In generated json Bittrex needs to be "key" instead of "api"
Example from exchange section:
	"bittrex": {
		"key": "******************",
		"secret": "******************"

Thanks! Fixed


implement expontential backoff and retry for 'HttpError 500 ... Internal Error'

Hello,
I experience HttpError 500 ... Internal Error backtraces when using duplicity.
The backtrace (below) indicates that the exception comes from google-api-python-client .
While duplicity might be able to work around the problem in its code, if possible it would be better to fix the problem in google-api-python-client (so that every api user does not have to make their own implementation).
Nivco (apparently of "Google Developer Relations) on stackoverflow suggested implementing an exponential backoff and retry for HTTP Error 500 for others with the same problem:
http://stackoverflow.com/a/12640475
I see a fork which may be of interest called 'implement exponential backoff and retry for BatchHttpRequest':
https://github.com/jay0lee/google-api-python-client/commit/60eb5f4c520e9e91c6fc1daafabe8559309ae493
Thanks for considering,
C.

Backtrace:
Attempt 1 failed. HttpError: <HttpError 500 when requesting https://www.googleapis.com/drive/v2/files?q='0B-PLx8xVPvcDS1N1eFQ5QlYwbVk'+in+parents+and+trashed%3Dfalse&fields=items(title%2Cid)%2CnextPageToken&alt=json&maxResults=1000&pageToken=EAIanwELEgBSlgEKhAEKXPjz6gX_____wxSDokj8DkAA_wH__kNvc21vLnVzZXIoMDAwMDAwY2E5YzRjODRkMlUpLmRpcl9lbnRyeSg4NzAyMDU2NTQyMjZcLjdcLlJFYzdjZGNfTThRKQABEOgHISRTjeXk3XctOQAAAAD6FQwASApQAFoLCSjx1l1vCHwLEAEgATIGCIHrtLAF8ISVfAEMQAAiCwnShEycygAAACAG returned "Internal Error">
Backtrace of previous error: Traceback (innermost last):
File "/usr/lib/python2.7/dist-packages/duplicity/backend.py", line 365, in inner_retry
return fn(self, *args)
File "/usr/lib/python2.7/dist-packages/duplicity/backend.py", line 564, in list
return [tobytes(x) for x in self.backend._list()]
File "/usr/lib/python2.7/dist-packages/duplicity/backends/pydrivebackend.py", line 143, in _list
'fields': 'items(title,id),nextPageToken'}).GetList()
File "/usr/local/lib/python2.7/dist-packages/pydrive/apiattr.py", line 154, in GetList
for x in self:
File "/usr/local/lib/python2.7/dist-packages/pydrive/apiattr.py", line 138, in next
result = self._GetList()
File "/usr/local/lib/python2.7/dist-packages/pydrive/auth.py", line 54, in _decorated
return decoratee(self, *args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/pydrive/files.py", line 56, in _GetList
self.metadata = self.auth.service.files().list(**dict(self)).execute()
File "/usr/local/lib/python2.7/dist-packages/oauth2client/util.py", line 142, in positional_wrapper
return wrapped(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/googleapiclient/http.py", line 729, in execute
raise HttpError(resp, content, uri=self.uri)
HttpError: <HttpError 500 when requesting https://www.googleapis.com/drive/v2/files?q='0B-PLx8xVPvcDS1N1eFQ5QlYwbVk'+in+parents+and+trashed%3Dfalse&fields=items(title%2Cid)%2CnextPageToken&alt=json&maxResults=1000&pageToken=EAIanwELEgBSlgEKhAEKXPjz6ZL_____wxSCIAvxt4AA_wH__kNvc21vLnVzZXIoMDAwMDAwY2E5YzRjODRkMlUpLmRpcl9lbnRyeSg4NzAyMDU2NTQyMjZcLjdcLlJFUzF1QWR2NkRrKQABENAPISRTjeXk3XctOQAAAABtFgwASBlQAFoLCSjx1l1vCHwLEAEgATIGCLfrtLAF8ISVfAEMQAAiCwnShEycygAAACAG returned "Internal Error">
Giving up after 2 attempts. HttpError: <HttpError 500 when requesting https://www.googleapis.com/drive/v2/files?q='0B-PLx8xVPvcDS1N1eFQ5QlYwbVk'+in+parents+and+trashed%3Dfalse&fields=items(title%2Cid)%2CnextPageToken&alt=json&maxResults=1000&pageToken=EAIanwELEgBSlgEKhAEKXPjz6ZL_____wxSCIAvxt4AA_wH__kNvc21vLnVzZXIoMDAwMDAwY2E5YzRjODRkMlUpLmRpcl9lbnRyeSg4NzAyMDU2NTQyMjZcLjdcLlJFUzF1QWR2NkRrKQABENAPISRTjeXk3XctOQAAAABtFgwASBlQAFoLCSjx1l1vCHwLEAEgATIGCLfrtLAF8ISVfAEMQAAiCwnShEycygAAACAG returned "Internal Error">
Releasing lockfile <LinkLockFile: '/root/.cache/duplicity/90d3651b4aa73abc6c07294d68123219/tardis01.277403715235550991278502' -- '/root/.cache/duplicity/90d3651b4aa73abc6c07294d68123219/lockfile'>
Removing still remembered temporary file /tmp/duplicity-ZP1AOI-tempdir/mkstemp-wvnVvO-1

I think that pull request #160 fixed this.

I'll test eventually.  Isn't it about time for a version bump and new release ?

@lickdragon: I suspect it's long past time. We'll try to get to it soon.

btw, this also applies to batch calls, it gets slightly more complicated there as I believe and sub-request can throw a 5xx

btw I think retry for regular execute is now implemented with num_retries parameter

ugh, this ALSO applies to oauth, I've seen a 502 on token refresh

Related:  https://developers.google.com/api-client-library/python/guide/media_upload#resumable-media-chunked-upload


As a volunteer, be able to edit submitted requests for clarification

We don't have time to finish this before our presentation.


ocamlPackages.janestreet: 0.16 -> 0.17

Description of changes
Add version v0.17 of janestreet libraries.
Release announcement: https://discuss.ocaml.org/t/ann-v0-17-release-of-jane-street-packages/14717
Additional changes required for some packages to compile against new v0.17 libraries:

Updated ocamlPackages.bistro to latest unstable
Updated ocamlPackages.dbf to v0.2.0
Have older ocamlformat library / binary compile against v0.15 to avoid compatibility issues
Compatibility patches added for:

flitter
merge-fmt
ocamlPackages.camlimages
ocamlPackages.cfstream
ocamlPackages.genspio
ocamlPackages.tdigest

Notes:

krb library (https://github.com/janestreet/krb) has been archived, and is thus not included.
Version 0.17 of pythonlib library (https://github.com/janestreet/pythonlib) has yet to be released, and is thus not included.

Build log:
15 packages added:
ocamlPackages.async_log (init at 0.17.0) ocamlPackages.capitalization (init at 0.17.0) ocamlPackages.codicons (init at 0.17.0) ocamlPackages.gel (init at 0.17.0) ocamlPackages.legacy_diffable (init at 0.17.0) ocamlPackages.ocaml_intrinsics_kernel (init at 0.1
7.0) ocamlPackages.ocaml_openapi_generator (init at 0.17.0) ocamlPackages.ppxlib_jane (init at 0.17.0) ocamlPackages.ppx_diff (init at 0.17.0) ocamlPackages.ppx_embed_file (init at 0.17.0) ocamlPackages.ppx_quick_test (init at 0.17.0) ocamlPackages.ppx_string
_conv (init at 0.17.0) ocamlPackages.uopt (init at 0.17.0) ocamlPackages.versioned_polling_state_rpc (init at 0.17.0) ocamlPackages.virtual_dom_toplayer (init at 0.17.0)

547 packages updated: <snip>

3 packages removed:
ocaml5.1.1-diffable (†0.16.0) ocaml5.1.1-krb (†0.16.0) ocaml5.1.1-pythonlib (†0.16)

<snip>

  2 packages marked as broken and skipped:
  ocamlPackages.biocaml ocamlPackages.torch

  2 packages failed to build:
  dune-release opam-publish

  565 packages built: ...

Both dune-release and opam-publish seem to be broken in master.
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

[x] 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

[x] Fits CONTRIBUTING.md.

I should squash all commits into a single commit before merging.

Cc @anmonteiro since you did the same thing in the overlays


SoftBreakAddsNewLinePlugin don't work.

Markwon version: 4.6.2

The text still show 2 lines with code as below
`public class MainActivity extends AppCompatActivity {
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    final Markwon markwon = Markwon.builder(this)
            .usePlugin(new AbstractMarkwonPlugin() {
                @Override
                public void configureVisitor(@NonNull MarkwonVisitor.Builder builder) {
                    builder.on(SoftLineBreak.class, new MarkwonVisitor.NodeVisitor<SoftLineBreak>() {
                        @Override
                        public void visit(@NonNull MarkwonVisitor visitor, @NonNull SoftLineBreak softLineBreak) {
                            visitor.forceNewLine();
                        }
                    });
                }
            })
            .usePlugin(SoftBreakAddsNewLinePlugin.create())
            .build();
    TextView txt = findViewById(R.id.text);
    markwon.setMarkdown(txt,"12\n\n\n\n\n\n\n\n\n\n\n34");
}

}`
Could you let me know if there is any missing?

Hello @maitolab ,
your image cannot be displayed. But I think I know what is your issue. As per commonmark specification additional hardline breaks are collapsed to produce at most one hard break. Current specification does not allow multiple empty new lines. To overcome this you can use HtmlPlugin along with
 tags inside your markdown.
There is also an experimental solution that involves custom inline parsing processor. But due to planned commonmark-java version update this experimental solution might not be valid soon. Would you be still interested in such solution?

@noties Thanks for your answer. This is a simulator.
We have a edittext to enter text then format and display in textview. Then we can not insert  to text.
If you use enter in keyboard, it will be "hardline breaks"?

Two of more of new line characters ("enter" in keyboard) will be considered a "hardline break"

@noties @maitolab can you guys gave the example of using HtmlPlugin with

@shubham696 We need to add plugin
 private static final Markwon markwon = Markwon.builder(WAAFI.getContext())             .usePlugin(StrikethroughPlugin.create())             .usePlugin(HtmlPlugin.create())             .build();
and then replace break line as below
String body = body.replace("\n", "
"); CharSequence result = markwon.toMarkdown(body);


Compiled Vue 2.7 Script Setup Components crash when mounted in Vue 3 Compat

Vue version
3.4.27 (happens in 3.3 as well) with both Vue Compat and Vue
Link to minimal reproduction
https://github.com/jssullivan/vue-3-not-running-compiled-script-setup
Steps to reproduce
1.Pull down the linked repository
2.Run pnpm i && pnpm dev
3. Open the app, Click the first button of a Compiled Vue 3 component working correctly
4. Click the second button, this is a Compiled Vue 2 component that uses the Composition Api with defineComponent
5. Click the last button, this is a Compiled Vue 2 component that uses script setup, see this crashes when trying to read the ref hello_vue_team
What is expected?
All three components render correctly
What is actually happening?
It crashes when reading the variable hello_vue_team defined in the script setup of the compiled Vue 2.7 component.
System Info
No response
Any additional comments?
Looking from a debugger it appears that the variable hello_vue_team is being correctly evaluated and attached to this. However _vm._self._setupProxy is undefined (which also happens for a non script setup component in that example).

This only caused a warning for the defineComponent compiled component, as the template as its reading directly from _vm

Naively in this minimum viable repro, it seems like thisProxy._self._setupProxy needs to be set to thisProxy inside renderComponentRoot. I can submit a PR with that change, but don't know if there are other impacts. I can alternatively add this to compatRender in @vue/compat.
Appreciate the assistance!

If others hit this, you can apply this pnpm patch to compatRender in @vue/compat to address this crash.
https://github.com/jssullivan/vue-3-not-running-compiled-script-setup/commit/7dcda5b5f0b2db551cd86c848ed8558aae75f9f0
I can create an MR to add this, or create a new @vue/compat flag to add this. Looking at the Vue 2 code _setupProxy  only contains values returned from the setup function vs the whole component instance. But this should handle most cases, and atleast create warnings instead of a crash.

Technically pre-compiled components have no compatibility guarantees as it relies on Vue 2 internals and we strongly recommend you to use the source SFC instead of Vue 2 compiled code.
That said if you can submit a PR to get these components working I'd be happy to merge it.


Accessibility testing

Adds automated pa11y test to be run by Travis CI.
Additions

Adds npm test script.
Adds config file for Travis CI.
Adds Travis CI badge to readme.

Testing

Pull down changes.
npm install
npm test

Notes

This component's demo page had some accessibility problems so I corrected the relevant CSS comments which, in turn, generated new HTML for the demo page via topdoc. Hence all the /demo changes below.

Review

@anselmbradford

Weird, getting the same issue as https://github.com/cfpb/cf-core/pull/35#issuecomment-103274010


fix(tools): include email when generating JWT token

ISSUES CLOSED: #152

Thanks for the feedback, will amend the commit today.


How can I add sticky button to keyboard?

Hi,
How can I add a button to stick to the top of the keyboard always when the keyboard shows up?

You should be able to make the button sticky by just putting it outside the scrollView component, inside a view.
<View>
<KeyboardAwareScrollView>
</KeyboardAwareScrollView>
</Button>
</View>

It' pretty strange...
In android can help this trick:

You should be able to make the button sticky by just putting it outside the scrollView component, inside a view.

<View>
<KeyboardAwareScrollView>
</KeyboardAwareScrollView>
</Button>
</View>

but in ios its doesen't work

It' pretty strange...
In android can help this trick:

You should be able to make the button sticky by just putting it outside the scrollView component, inside a view.

<View>
<KeyboardAwareScrollView>
</KeyboardAwareScrollView>
</Button>
</View>

but in ios its doesen't work

I am also looking for a solution.
it is not working in IOS.

I am also looking for a solution.
it is not working in IOS.

Any news?


TINKERPOP-575 RemoteGraph

https://issues.apache.org/jira/browse/TINKERPOP-575
This is a brute of a PR that introduces RemoteGraph. The tests execute as part of gremlin-server with regular old mvn clean install. There's lots of optimizations and improvements we could make all around, but we at least have something fully functional at this point with this PR.
Docs generate nice...you can find the RemoteGraph docs here:
http://tinkerpop.apache.org/docs/3.2.0-SNAPSHOT/reference/#connecting-via-remotegraph
The upgrade docs are here:
http://tinkerpop.apache.org/docs/3.2.0-SNAPSHOT/upgrade/#_remotegraph
The developer/provider docs are here:
http://tinkerpop.apache.org/docs/3.2.0-SNAPSHOT/dev/provider/#remoteconnection-implementations
VOTE +1 - phew!

Wonderful!
VOTE: +1

The insane PR. VOTE +1.

Awesome to see progress on this! I had some trouble getting it to work though. Here's what I id:
build it
git checkout TINKERPOP-575 #currently e01a71b
mvn clean install -DskipTests

start gremlin-server
unzip $TP_SRC/gremlin-server/target/apache-gremlin-server-3.2.0-SNAPSHOT-distribution.zip
cd apache-gremlin-server-3.2.0-SNAPSHOT
bin/gremlin-server.sh conf/gremlin-server-modern.yaml

connect and run traversal
cd gremlin-console/target/apache-gremlin-console-3.2.0-SNAPSHOT-standalone
bin/gremlin.sh
         \,,,/
         (o o)
-----oOOo-(3)-oOOo-----
plugin activated: tinkerpop.server
plugin activated: tinkerpop.utilities
plugin activated: tinkerpop.tinkergraph
gremlin> graph = RemoteGraph.open('conf/remote-graph.properties')
==>remotegraph[DriverServerConnection-localhost/<IP_ADDRESS>:8182 [graph='graph]]
gremlin> g = graph.traversal()
==>graphtraversalsource[remotegraph[DriverServerConnection-localhost/<IP_ADDRESS>:8182 [graph='graph]], standard]
gremlin> g.V().valueMap(true)

No output :(
PS1: works fine for local TinkerGraph instance
PS2: Wireshark tells me that there is some communication between the REPL and gremlin-server (three TCP packets).

@mpollmeier that was a nice bug find. i never unpackaged the zip and my assembly files had a bug in them. the scripts directory wasn't being included so the graph couldn't get initialized with data which so you were executing your remotes against an empty graph and getting no results.  Should work for you now if you try again. Reviewing the distributions now to be sure nothing else is amiss.

works fine now, thanks!


Lita always looks offline

I'm finding since we upgraded to HipChat Server v1.1.5, the Lita bot no longer appears to be online. Strangely though, it still joins rooms and answers queries, just appearing to be offline.
Lita 4.1.0
lita-hipchat 2.1.1
HipChat Server 1.1.5

This is a confirmed bug with the HipChat client. Hopefully it will be resolved soon!
Dupe of #21.

I think its not bug in HipChat client. The presence implementation in hubot is slightly different: https://github.com/hipchat/hubot-hipchat/blob/master/src/connector.coffee

FYI, internally this is being tracked as an issue with the Hipchat Mac OSX client, HC-7228.

Also I just installed the latest lita and lita-hipchat and the bot is active but in Hipchat it shows as offline. I am not using Hipchat Server.

I can confirm this is totally fixed now.


This project depends on KituraNet

Currently, this project depends on the KituraNet API for outbound HTTP requests.  As far as I can tell, this is for historical reasons:  the KituraNet API runs on libcurl and worked reliably on Linux back when URLSession on Linux didn't.
We intend to deprecate and remove the KituraNet API as there is no reason why it should provide an API for outbound HTTP requests now that URLSession on Linux is functional.
This issue is to discuss how to achieve that.  Here are a few options:

Change this repo so it uses RestKit instead
Change this repo so it uses SwiftyRequest instead
Change all the users of this repo so they use RestKit instead, and deprecate this repo
Change all the users of this repo so they use SwiftyRequest instead, and deprecate this repo

My preference is for options 3 or 4, as I don't see why the users of this repo can't be updated to use one of the alternatives directly.

I asked Anton about this repo awhile ago. He said it's up to you guys (Swift team) to decide if you want to keep it. I'm pretty sure that this can be deprecated, so option 3 or 4 is probably best.
@aal80 Do you know of any other projects that would be affected by this?

These are all the references to this library on github.com, there might be others on GHE. https://github.com/search?q=bluemix-simple-http-client-swift&type=Code

I thought of an option 5.

Change this repo so it uses URLSession directly.

I kind of like that, what do you think @AnthonyOliveri @aal80 ?

That might be an easier solution, since there would be no need to coordinate updating the dependencies of several projects that depend on this. This is a fairly simple framework, and there is already some commented-out URLSession code that can be used as a starting point.
Once that's done, it's probably worthwhile to put a bold notice on the README that this repo is no longer being maintained, and URLSession should be used instead.

One of the biggest dependencies we own is the old Object Storage SDK:
https://github.com/ibm-bluemix-mobile-services/bluemix-objectstorage-serversdk-swift/blob/master/Package.swift

Option 6.

Change all the users of this repo (bms-pushnotifications-serversdk-swift and bluemix-objectstorage-serversdk-swift) to use URLSession directly.

Thoughts?

@ianpartridge When you said that "we intend to deprecate and remove the KituraNet API", do you mean you will be deleting the repo too? If the repo is going to stay up, then I think the most practical solution would be to simply put a deprecation notice to the README in this SDK, and leave everything else alone. As long as KituraNet repo exists, then there should be no issue.


Update README.rst

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Documentation (typos, code examples or any documentation update)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Operating System:
Python version:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

ggzfdzfgdfgzdfgzdgfzd


Better unit tests for resume

Make sure that the following are tested:

Removing link
Removing hardlink
Check file with existing checksum
Re-copy file that did not have checksum

Fixed in 328c2ca5c0dd5ed67ddc2c5d76102cee16235f20.


Rename to Character

As the app will be using a new endpoint, all the Source code must be renamed from Product, the previous endpoint business, to the current one

Closed by PR #85


Fix IModelHost onAfterStartup event and automatic shutdown.

It seems that raising onAfterStartup was broken on imodel02 branch in https://github.com/imodeljs/imodeljs/commit/3bc045f1756d4bde2c5e73b0d1eee3f7bbdfee82
Also added some tests for these events.

@wgoehrig I'm probably dumb but can you explain how @khanaffan's commit broke this and how this fixes it?

@wgoehrig I'm probably dumb but can you explain how @khanaffan's commit broke this and how this fixes it?

That commit replaced the call to IModelHost.onAfterStartup.raiseEvent(); at the end of IModelHost.startup with process.once('beforeExit', IModelHost.shutdown);.
@khanaffan can correct me if I'm wrong, but I think removing the raiseEvent() was inadvertent - it really has nothing to do with the rest of those changes.

@Mergifyio backport release/2.14.x


Allow application developers to restart alertmanager

Current behaviour
Currently when an application developer updates the alertmanager configuration secret they are not allowed to restart the pod to apply that configuration.
Expected behaviour
An application developer can update the user-alertmanager config and restart the pod to apply that config.
Steps to reproduce

As an application developer
Update alertmanager config
Try to restart the pod to apply the config
Fail

Compliant Kubernetes Apps Version
v0.33.1
Kubernetes Version
v1.27.5
Additional context
No response

Worth considering some mechanism that automatically reloads user-alertmanager when the config has changed?

Worth considering some mechanism that automatically reloads user-alertmanager when the config has changed?

I'll update the issue to have the one doing this testing if this is possible first

Wait what, alertmanager at least has a container that is called config-reloader. And since this hasn't been an issue in the past it should work just fine.

Wait what, alertmanager at least has a container that is called config-reloader. And since this hasn't been an issue in the past it should work just fine unless something has been broken in a recent release.

Hmm 🤔 I guess this will become a investigation task that it still works then. The alertmanager I saw this for was also broken in another way so it might have been that

Let's hope that it was just that particular alertmanager that was broken 🤞🏻
If it's a wide-spread issue it feels like we should have seen more people complain about it.


demo gif minify

Hi, thanks for your contribution. Though I realise this is needed, we will soon be getting an all-new gif. So I'm closing this.


Cannot run the 'install R bindings script'

Hello,
using Arcgis 10.3.1
R 3.2.2
I'm having trouble running the 'Install R bindings script'. Here comes the message I get.
Any help appreciated.
Julien
Traceback (most recent call last):
File "", line 177, in execute
File "C:\Program Files (x86)\ArcGIS\Desktop10.3\ArcToolbox\Toolboxes\r-bridge-install-master\rtools\install_package.py", line 62, in install_package
download_url = release_info()[0]
File "C:\Program Files (x86)\ArcGIS\Desktop10.3\ArcToolbox\Toolboxes\r-bridge-install-master\rtools\github_release.py", line 67, in release_info
json_r = parse_json_url(latest_url)
File "C:\Program Files (x86)\ArcGIS\Desktop10.3\ArcToolbox\Toolboxes\r-bridge-install-master\rtools\github_release.py", line 44, in parse_json_url
url, e))
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe9 in position 58: ordinal not in range(128)
Failed to execute (InstallBindings).

Julien,
Thank you for the report. It looks like a character encoding issue, I'm guessing your locale is set to one that includes non-ASCII characters, and the code didn't anticipate that. I'm looking into the issue now.

I got an error running the install R bindings script and was able to solve this by running ArcMap as Administrator.

@mhogeweg the current release (2 days ago) should notify you immediately if you need to install as admin, unfortunately for 10.3.1 it needs to write to the users' Program Files directory which is protected.

Same for me. So  I added Full Control privileges in the security settings for  the R folder. But my problem with import rtools persists: 'module' object has no attribute 'install_package'
After running “Install R Bindings”:
Executing: InstallBindings false
Start Time: Mon Dec 21 07:45:27 2015
Running script InstallBindings...
Traceback (most recent call last):
File "", line 164, in execute
AttributeError: 'module' object has no attribute 'install_package'
Failed to execute (InstallBindings).
Failed at Mon Dec 21 07:45:27 2015 (Elapsed Time: 0.00 seconds)
After running “Update R Bindings”:
Executing: UpdateBindings
Start Time: Mon Dec 21 07:48:37 2015
Running script UpdateBindings...
Traceback (most recent call last):
File "", line 47, in execute
AttributeError: 'module' object has no attribute 'update_package'
Failed to execute (UpdateBindings).
Failed at Mon Dec 21 07:48:37 2015 (Elapsed Time: 0.00 seconds)
After running “Print R Version”
Executing: RVersion
Start Time: Mon Dec 21 07:51:05 2015
Running script RVersion...
Traceback (most recent call last):
File "", line 74, in execute
AttributeError: 'module' object has no attribute 'r_version'
Failed to execute (RVersion).
Failed at Mon Dec 21 07:51:05 2015 (Elapsed Time: 0.00 seconds)
It seems that rtools isn’t importing fully into “R integration.pyt”
The debugging recommendation suggests checking these folders. I don’t have this and so what should I do from here?
C:\Users\Documents\R\win-library) into the requested place, C:\Program Files (x86)\ArcGIS\Desktop\Rintegration

when I ran the install 'as Administrator' it worked for me.

@julienrenaudleca Sorry for the long delay getting back to you. I think this issue has been solved, let us know if you have any further issues using the installation.

Hello ,mine is arcgis 10.4 ,R 3.2.5
**I received the folllowing information when I install R bindings.Hope you can help me!!
Executing: InstallBindings true
Start Time: Wed Apr 27 16:28:29 2016
Running script InstallBindings...
Saving URL to 'c:\users\duaen\appdata\local\temp\tmpmvkipw\arcgisbinding_<IP_ADDRESS>.zip'
Traceback (most recent call last):
File "", line 164, in execute
File "E:\ESRI\R_arcgis\r-bridge-install-master\rtools\install_package.py", line 196, in install_package
save_url(download_url, package_path)
File "E:\ESRI\R_arcgis\r-bridge-install-master\rtools\github_release.py", line 33, in save_url
f.write(r.read())
File "E:\ESRI\python\ArcGIS10.4\Lib\socket.py", line 355, in read
data = self._sock.recv(rbufsize)
File "E:\ESRI\python\ArcGIS10.4\Lib\httplib.py", line 612, in read
s = self.fp.read(amt)
File "E:\ESRI\python\ArcGIS10.4\Lib\socket.py", line 384, in read
data = self._sock.recv(left)
File "E:\ESRI\python\ArcGIS10.4\Lib\ssl.py", line 734, in recv
return self.read(buflen)
File "E:\ESRI\python\ArcGIS10.4\Lib\ssl.py", line 621, in read
v = self._sslobj.read(len or 1024)
error: [Errno 10053]
Failed to execute (InstallBindings).
Failed at Wed Apr 27 16:51:57 2016 (Elapsed Time: 23 minutes 27 seconds)

Hello @CodeQL,
I've made a new release of the ArcGIS binding package which supports offline installation (see instructions). This should also serve to solve the problem you described, it looks like in your case the call to download the bridge library itself from Python is failing. This is likely to do a proxy or firewall which may restrict access to resources via Python. If you follow those instructions, and with your browser place the downloaded arcgisbinding_<IP_ADDRESS>.zip file into the same directory as the "R integration" python toolbox, the installation should proceed without needing to use Python to download the file.
Cheers, Shaun


Search on membership table behaves confusingly when on page 2 or higher

In EDA, GEMS1, browse + subsetting, variable "Reason not evaluated after rehydration"
First search for "lunch" in the table. You'll see there are six results.
Cancel the search and navigate to page 2 of the table.
Now search for "lunch" and it looks like there are no results. Actually there are six results as before, but the table thinks it's on page 2 of the results, and six results don't spill over onto page 2, so nothing is shown (and neither are the pagination controls, which is the confusing bit).
If you cancel that search and instead search for "hurry" you do see some results, because there are 2 pages of "hurry" results, it looks and behaves fine (unless you're on page 3 or higher).
Not entirely sure of a clean solution!

perhaps related to https://github.com/VEuPathDB/web-eda/issues/1412


Updation of DEMO Pics

Hi, In the readme, these pics should be replaced with the screenshots of the current website.
These pics are old and adding new ones that are current would give users a better idea and overview of the project.
Would like to change these. Kindly assign me this issue.

Hello @neeru24,
Thank you for bringing this up! I've been considering this for a while, and while updating the demo pictures is a valid point, it may not provide long-term value. Given the high volume of incoming changes, the UI will evolve frequently, making such updates quickly outdated.
It might be more practical to revisit this task at a later stage.


Response next previous mark

Refactoring next/previous

removed rest/utils.rb
created rest/response.rb to act as response builder
changed signature of response.rb to remove summary and add connection

@allcentury @johaywood @jaysonlarin what are your thoughts?

I like the changes in here.  What will the API be to use next and previous?  I can see it in the object as:
[11] pry(main)> client.get_fitnesses.summary
=> #<Validic::Summary:0x007f9d2430a980
 @end_date="2016-01-27T23:59:59+00:00",
 @limit=100,
 @message="Ok",
 @next="https://api.validic.com/v1/organizations/our_id/fitness?access_token=our_token&page=2",
 @offset=0,
 @previous=nil,
 @results=110,
 @start_date="2016-01-25T00:00:00+00:00",
 @status=200,
 @timestamp=nil>

@allcentury see https://github.com/Validic/validic/blob/response_next_previous/spec%2Fvalidic%2Fresponse_spec.rb#L21 for an example usage

So in order to get all the records from a paginated response we'd encourage them to do something like this?
client = Validic::Client.new(organization_id: 'our_id', access_token: 'token')

# arg: client
# return: array
def get_all_fitness_records(client)
  total_records = []
  current_resp = client.get_fitnesses
  loop do
    break if current_resp.nil?
    total_records += current_resp.records
    current_resp = current_resp.next
  end
  total_records
end

puts get_all_fitness_records(client).size

You can obviously make that handle all the objects instead of just fitness but this was just an example.

correct. just like paging 'by hand' using a rest client
On Tue, Jan 26, 2016 at 10:03 AM, Anthony Ross<EMAIL_ADDRESS>wrote:

So in order to get all the records from a paginated response we'd
encourage them to do something like this?
client = Validic::Client.new(organization_id: 'our_id', access_token: 'token')
arg: client# return: arraydef get_all_fitness_records(client)
total_records = []
current_resp = client.get_fitnesses
loop do
break if current_resp.nil?
total_records += current_resp.records
current_resp = current_resp.next
end
total_recordsend
puts get_all_fitness_records(client).size
You can obviously make that handle all the objects instead of just fitness
but this was just an example.
—
Reply to this email directly or view it on GitHub
https://github.com/Validic/validic/pull/33#issuecomment-175062874.

--
Mark Phelps
Senior Software Engineer
http://validic.com

Alright - I think your changes look good.


RaZer Viper Ultimate with charging station

Please add support for the following device:
Device Name: Razer Viper Ultimate
Device Website:  https://www.razer.com/pc/gaming-mice/viper-family

Hello, this repository is no longer maintained. Can you check the aurora on https://github.com/Aurora-RGB/Aurora


swc() rule breaks copying files after first build

Issue description in commit message here: https://github.com/mdittmer/bazel-sandbox/commit/03b685b866ab15971d9b8bcc60efda8645970a75 (commit contains code to reproduce issue).
This issue was observed running bazel on mac with rules_swc-0.5.1.
Description copied here for convenience:
swc() rule that compiles transitive dependencies succeeds on first build, but fails on second build while copying files.
$ cd js && npm install && bazel clean --expunge && bazel build //:swc && bazel build //:swc
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

added 14 packages from 66 contributors and audited 14 packages in 1.092s
found 0 vulnerabilities

INFO: Starting clean.
Starting local Bazel server and connecting to it...
INFO: Analyzed target //:swc (68 packages loaded, 403 targets configured).
INFO: Found 1 target...
Target //:swc up-to-date:
  bazel-bin/node_modules/aws-sdk/scripts/services-table-generator.js
INFO: Elapsed time: 10.869s, Critical Path: 0.83s
INFO: 60 processes: 33 internal, 1 darwin-sandbox, 26 local.
INFO: Build completed successfully, 60 total actions
INFO: Analyzed target //:swc (1 packages loaded, 25 targets configured).
INFO: Found 1 target...
ERROR: /path/to/bazel-sandbox/js/BUILD.bazel:3:4: Copying file bazel-bin/node_modules/aws-sdk/scripts/services-table-generator.ts failed: (Exit 1): bash failed: error executing command /bin/bash -c 'cp -f "$1" "$2"' '' bazel-bin/node_modules/aws-sdk/scripts/services-table-generator.ts bazel-out/darwin-fastbuild/bin/bazel-bin/node_modules/aws-sdk/scripts/services-table-generator.ts
cp: bazel-bin/node_modules/aws-sdk/scripts/services-table-generator.ts: No such file or directory
Target //:swc failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 0.758s, Critical Path: 0.27s
INFO: 8 processes: 6 internal, 2 local.
FAILED: Build did NOT complete successfully

Copying file bazel-bin/node_modules/aws-sdk/scripts/services-table-generator.ts is the clue here. Your glob to gather .ts inputs is descending into the bazel output folder. You should probably add bazel-bin/node_modules/ to .bazelignore.

Please reply if there's still a bug here :)


v0.5.0

更换git repo

Codecov Report

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

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   52.19%
=========================================
  Files             ?        9
  Lines             ?      889
  Branches          ?        0
=========================================
  Hits              ?      464
  Misses            ?      383
  Partials          ?       42

Continue to review full report at Codecov.

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


User titles do not get rendered when @syntax is used in Published Dendron web pages.

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[ ] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[x] Publish.  External Site Publish
[ ] Markdown.  Markdown features, such as syntax support and features

Describe the bug
Lets say we have a note user.jon with title Jonathan
If we reference this in another note as [[user.jon]] publishing will show the note as Jonathan
However, if we reference this note as @jon publishing will show this as @jon and will not render the title of the note.
Expected behavior
Show the title of the note when published.
Desktop (please complete the following information)

OS: [e.g. iOS]
Browser (if with published websites; include version): [e.g. chrome, safari]
Versions (all that apply to bug):

Dendron Extension: 0.91.0
VS Code version:
Dendron CLI:

Additional context
Add any other context about the problem here.
Dendron Log file
Please attach the output of > Dendron:Dev: Open Logs here

Thanks for the feedback! This is by design, similar to how @... on websites like Twitter don't get replaced by the user name. It would be a good for some use cases, so we can add this as an option.


r-bh is a build-time-only dependency

…and also a quite heavy one.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Strange ... GPL-3.0 should be a valid short identifier according to https://spdx.org/licenses/GPL-3.0.html ... seems like conda-smithy wants GPL-3.0-only: https://github.com/conda-forge/conda-smithy/blob/master/conda_smithy/licenses.txt#L168

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender