updated getting started commands

Checklist
🚨 Please review this repository's contribution guidelines.

[ ] I've read and agree to the project's contribution guidelines.
[ ] I'm requesting to pull a topic/feature/bugfix branch.
[ ] I checked that my code additions will pass code linting checks and unit tests.
[ ] I updated unit tests (if applicable).
[ ] I'm ready to notify the team of this contribution.

Description
updated getting started commands
Thank you!

PR is too old , closing it. If needed rebase the with current master branch and raise PR again.


Use new /draft-service POST endpoint for creating drafts

/draft-service/<framework-slug>/create is deprecated in favour of
using /draft-service and passing frameworkSlug in the payload, similar
to lot and supplierId
(https://github.com/alphagov/digitalmarketplace-api/commit/dd2ed8)

:+1:


Auto generated docs

Make docs auto generated or better yet, make an auto generated option search like home-manager and nixpkgs, it could be auto deployed to github pages
This could prevent sync issues between the modules and the doc, and would mean less work for the maintainers

Do you recommend any specific tool for this process. I looked around when I started the project, but gave up and wrote the docs myself (I agree, not optimal), but I would like to move to auto generating them at this point.

I just came across this which seems simple enough to add to a project:
https://github.com/NuschtOS/search/
which is used by this repo for example:
https://github.com/NuschtOS/nixos-modules/
which unlike search.nixos is a static site deployed with gh actions+pages. It seems ideal for a project at this scale
for extra customization such as themes, site name or logo, we could make a fork of the repo or simply add it to a folder in this repo.
I'm willing to try to implement it 🙂


Add forecast api endpoint

Hi, these changes add the forecast 5 day 3 hour api endpoint:
https://openweathermap.org/forecast5
I tried copying your style as much as I could.
But you may want to look at how I'm exporting more stuff from  Web.OpenWeatherMap.Client and how I implemented the testing code in cmd/Main.hs
I wasn't sure if my solution or simply extending the location sumtype would be preferable. Your thoughts would be appreciated.

My style is just running hindent :)

Here https://github.com/ip1981/openweathermap/commit/0b1981ef40404802a828f2a91ecadba3c2453034 :)

Hi I appreciate the effort you put in, but I would've preferred if you simply replied to my question because now I have to relink my app against these changes and I also don't get the github credits.


[Feature request] character limit and on-hover effect for custom widget and adjusting the position of process widget

Hi, I noticed that my current process widget is usually overlapped with my widgets on the right side.

I'm wondering if it is possible to let the custom widgets to have a similar on-hover effect as the music widget (i.e., animation to show the rest of the string that exceeds the character limit).
It would be very useful for most of user widgets like calendar or todo list ...etc
Also, it would be nice to be able to adjust the position of the process widget to avoid the overlap. Currently, it is right at the middle by default, however, there is usually more space on the left side.
Thank you!

Hi, this would be indeed usefull, I'll implement it soon!

Similar as #233


Error when computing MS-SSIM in images with size different than 144x144

When computing the ms-ssim for images with different size than 144 x144, for example in:
from generative.metrics import MSSSIM
import torch

metric = MSSSIM(data_range=torch.tensor(1.0))

metric(torch.ones([3, 1, 256, 256]), torch.ones([3, 1, 256, 256]))

returns
RuntimeError: stack expects each tensor to be equal size, but got [3, 1] at entry 0 and [1, 1] at entry 4

After investigating more, I found that the implementation from MONAI is wrong. Working to fix it.


How to set option/setting for later use?

1. For now, the only way to set option, is when using it
var sanitizeHtml = require('sanitize-html');

var dirty = 'some really tacky HTML';
clean = sanitizeHtml(dirty, {  // must pass in dirty HTML
  allowedTags: sanitizeHtml.defaults.allowedTags.concat([ 'img' ])
});

2. How do I set option and use it later?
Maybe something like this:
var sanitizeHtml = require('sanitize-html');

// set option
sanitizeHtml.options({
  allowedTags: sanitizeHtml.defaults.allowedTags.concat([ 'img' ])
})

and later just
   sanitizeHtml(dirty)

In short
from this
sanitize(HTML, option);

to this
sanitize.option({...});
sanitize(HTML)

Why?
So I can put sanitizeHTML into one file like custom-sanitize-html.js and use it in multiple place later
I don't have to copy setting every time
I already read README, and search in Github issue, can't find answer
Thank you

This is really more of a general javascript question. This module gives you a function that can take an options object and some HTML, and sanitize according to those options. You can create a function yourself that does what you want:
function mySanitize(html) {
  return sanitize(html, { options the way you want them } );
}

Now call mySanitize(html) anywhere you like. You're not finding this in the README or github issues because it's more of a general question about javascript programming and how to write functions that call other functions in useful ways.

@boutell  Thank you!


Fix README instance name

name - The name of your instance. The default is "i-yyyyMMddHH". Example 2014/10/01 10:00:00 is "i-2014100101".
This is not the case anymore.

Fixed by #101


Is synchronous function available?

Hi @jorangreef:
Node.js has both synchronous and asynchronous functions to execute commands. In your package, is the synchronous function available?

Hi @junjunruan, no there are no synchronous functions for sudo-prompt.


Add a table of contents

I think many devs read this documentation in a way where they know what they are looking for. For example, people know that they need to upgrade their build system and want to find that section as fast as possible. Since this documentation is quite extensive, I think the format could benefit from a table of contents.
The proposed change in this commit only considers level 2 headings (equivalent to ## resp. <h2>).

People actually read the README.md? ;D

😂
Well, I do :)


SpringMvcContract does not support query string defined in interface's @RequestMapping

Proposed fix for #1023.
I've signed the contributor's agreement. Thanks!

I had to revert this because there were test failures. If you can rebase and fix them, that would be great.

My fear (see https://github.com/spring-cloud/spring-cloud-netflix/issues/1023#issuecomment-218629178) is that this is just not compatible with spring cloud netflix. I just haven't had time to look.

I'll take a look at it, sorry about that, I was sure I ran the tests.

Hmm, mvn clean install completes succesfully with maven 3.3.9 and openjdk 1.8_91 on ubuntu 14.04 LTS. Same with ./mvnw clean install. Do you have any input on this? I see that the travis build fails with an OOM error so I'm not sure where to look.
Thanks

Hmm. I was getting test failures in ribbon bits.
Travis is failing because of some weird javadoc thing. I would ignore it for now.

Well, I had to disable the Build Automatically in eclipse while building but that's pretty much it.

Did you have the chance to take a second look at it? Thx :+1:

Wondering if you can rework the tests to leave the _Simple tests as they are and add a new set with @RequestMapping on the interface. That way we know you haven't broken existing functionality.

Definitely, that's a good suggestion. I'll work on that next week.

PR updated. Do you prefer squashed commits or you'd rather do it on your end? Thx again

So @spencergibb, you're not worried about compatibility any more?
@jebeaudet if you squash them it will save us time. They get squashed anyway usually, but most people don't seem to know how.

Squashed.

@dsyer yes I am, see #1079

OK I see, does extending the interface for your client as mentioned in http://cloud.spring.io/spring-cloud-static/spring-cloud.html#spring-cloud-feign-inheritance would help?

I don't think so. Spring MVC automatically creates controllers if a bean has @RequestMapping at the class level.

I'll give it a shot with @FeignClient, as I said in the issue, we're not using it. Thanks for the hint, it might not be possible after all.

I may be missing something but I've made a sample in https://github.com/jebeaudet/feign-client-sample reproducing exactly the pattern defined in http://cloud.spring.io/spring-cloud-static/spring-cloud.html#spring-cloud-feign-inheritance with a class level @RequestMapping with a path parameters. The only thing I've added is that the resource implementing the api also needs the @PathVariable annotations.
The repo should be self contained, I've used the Brixton release and I've created the SpringMvcContract class with the code from this PR in the same package as the dependency so my class has precedence.
To test, spawn the eureka server, run UsersClientApplication and UsersResourceApplication. The resource expose an endpoint on http://localhost:8081/organizations/{organizationId}/users/{username} which simply prints out the path parameters. The client expose an endpoint on http://localhost:8080/ that simply proxy the resource endpoint with the path parameters hello and from feign client proxy.
I tried reproducing the issue mentioned in the link you gave @spencergibb without success, even when using @RequestMapping over @FeignClient (though I don't see why someone would do that), it didn't work but it didn't throw. I've tried with both my modified SpringMvcContract and the current one.
Thanks again!

I think the problem @spencergibb was worried about occurred when people put @RestController on their feign interfaces (which is clearly wrong but some people want us to support it). I think a @RequestMapping at the class level is fine. Isn't that what this PR is partly about?

Yes, this PR is about supporting @PathVariable defined in the class' @RequestMapping (I just noticed that I've used query string all along when I meant path parameters, I'll update the commit message to be clearer).
I'm struggling to reproduce the problem mentioned in the linked issue. Also, since @RequestMapping is already supported at the interface level to prepend a constant path, I don't see how this PR breaks anything (see my explanation in the first comment of #1023), it just needs to be in the MethodMetadata before the parseAndValidateMetadata is called.
I'd be happy to dig around a little more but I'm not sure where to look next.

I think we can probably merge this as long as the tests are fine again. Spencer can weigh in when he gets a chance.

This is what we've had problems with before https://github.com/spring-projects/spring-framework/blob/master/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerMapping.java#L169-L172
If you share an interface annotated with a top level @RequestMapping between feign and a real @Controller in the same project, spring will try and create a controller out of it even if it is not annotated with @Controller.

OK so the client and the resource have to be in the same project which doesn't make any sense to me, why would you use a client to access a resource in the same application? Even if you have multiple application instances, it's most likely always better to directly use the service than to go through an http request...
I just want to point out that :

class level @RequestMapping is already supported in SpringMvcContract, only the path params defined in it are not, are we planning in deprecating the support for it?
Separating the interface+client+resource in different projects with correct dependency management (as shown in my example) works fine.

I don't really mind if we drop this PR, we already use our custom SpringMvcContract to build our feign clients. It's not blocking for us but I think this improvement, if the class level @RequestMapping is to be still supported in future releases, should make its way into the codebase to support path params.
Once again, thanks for your inputs :+1:

I agree it doesn't make sense :-). I think this is ok to go.

Great news, happy to contribute!

Closed with a little polish (formatting) via b3ef770819e121dfbbbda371b3e7fe6ecf587b96


nightly: Checkout with tags and use hash in case nothing can be found

This is a followup of #3334.

Shouldn't we also disable the CI that @zyedidia has? So that it is not confusing.
Also, I think we can add this
 on:
+  workflow_dispatch: # Allows manual trigger
   schedule:
    - cron:  '0 0 * * *'

so that we can test it immediately instead of needing to wait for the next day.


Support for custom input producers

This is a feature request for being able to create custom functions to feed examples into input pipelines. Existing ops are designed to read pre-defined formats. Although it is entirely possible to pre-process all the data beforehand, I felt like it shouldn't be necessary, especially in realtime environments. I am sure, a lot of people would be glad if they could pass TensorFlow a lambda to use to generate the batches directly instead of either pre-processing them into the standard tensorflow format a having to set up all of the IO nonsense or using the not very scalable option of python dicts (according to the performance guide).
[Please tell me if there already exists such a solution, I didn't find a way.]

I think the performance guide already mentions that the solution is to use queues.

@ppwwyyxx I am referring to how the data is introduced, not how it is processed afterwards.

What currently can be done: Loading TFRecords, CSV files and whatever format has support
What currently isn't as easy: Loading a unknown/custom format and/or applying custom pre-processing before enqueuing the data

You can load whatever custom formats and apply custom pre-processing, and feed them into queues.

Thanks for helping our friend @ppwwyyxx.
If queues don't work for your use case, let me know and I'll reopen.


export settings/ user profile (feature request)

I was using Mac for development for almost 5-6 months with VSCODE and now I'm moving to a new laptop. Its really helpful if I can import/export or sync my installed extensions with user profile so I do not need to install all of them one by one and other customisation I have done.
Note: It is a feature request and not a issue

You can use https://marketplace.visualstudio.com/items?itemName=Shan.code-settings-sync at this moment.

Thanks @bang88. @manasjoshi Please use the above extension for sync.


Possible infinite loop

See https://github.com/weaveworks/weave/blob/master/nameserver/dns.go#L45

Nevermind.


added check for "audioWorklet"

fixes https://github.com/GoogleChromeLabs/audioworklet-polyfill/issues/19

CLA signed

Hi @developit, are you able to review this one line change? It fixes an issue on insecure origins like debugging localhost on an Android. More info at https://github.com/GoogleChromeLabs/audioworklet-polyfill/issues/19 and https://github.com/processing/p5.js-sound/issues/409
Thanks for your work on this library! And thank you @JohnWeisz for identifying this issue!

Thanks folks, sorry for the slow merge!

@developit thanks for merging this in! Is this PR going to be included in a release soon? It would be great to get this PR in a release, especially since there hasn't been a release since 2018!


fix: removed truthy as obsolete and unsupported

[x] - removed truthy as obsolete and unsupported

coverage: 94.71% (+0.03%) from 94.684%
when pulling d50b9fb7fb6e5b71aa8d1e668dce6e0744bb2ff3 on majioa:no_truthy
into a76a4517cb98f1ccb403335a4c7f4357587eef30 on tagliala:main.

Thanks!


Create an example project

Nice work here! Thanks for that lib

Would be great to see a working example project of this!

@ciathyza I agree, I've used it in conjunction with ReSwiftRouter in a project I can't use an example.


ivshmem-doorbell test: create test for ARM64 and x86_64

Hello everyone,
this patch series introduces a test for the ivshmem-doorbell that support both ARM64 and x86_64 running on virtualization. We also demonstrate how to build a project to support doorbell on ARM64, as it requires quite a few of dependencies.
This patch series is a continuation of other two. Initially we proposed the MR https://github.com/zephyrproject-rtos/zephyr/pull/52283 where we tested some features using doorbell. During the discussion, the reviewers requested a sample for doorbell. Then, internally, we decided it was best to split into two: MR https://github.com/zephyrproject-rtos/zephyr/pull/52823 adapted the existing ivshmem-plain test to support ARM64, and prepared the test directory for doorbell. Now, in this current MR, we are proposing the doorbell test.
Commit 1: bump the highest IRQ value when GICv3 ITS is enabled (for LPI support)
Commit 2: imply the need for dynamic memory when using ITS
Commit 3: doorbell sample with readme, auxiliary tools and code

@SgrrZhf whenever you have time, I would appreciate another review


python37Packages.pyscreenshot: 2.2 -> 2.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.pyscreenshot is: "python screenshot"
meta.homepage for python37Packages.pyscreenshot is: "https://github.com/ponty/pyscreenshot"
meta.changelog for python37Packages.pyscreenshot is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.3 with grep in /nix/store/i641510vp000q0gprl0zw86lcv6c0ls3-python3.7-pyscreenshot-2.3

directory tree listing: https://gist.github.com/004ef7552bec6db4dd4fdce5757fd31b

du listing: https://gist.github.com/1b50e87c8ea00905559cbe1bf7cab6c1

Rebuild report (if merged into master) (click to expand)

9 total rebuild path(s)

3 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
3 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python37Packages.pyscreenshot
python38Packages.pyscreenshot
python39Packages.pyscreenshot

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i641510vp000q0gprl0zw86lcv6c0ls3-python3.7-pyscreenshot-2.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.pyscreenshot https://github.com/r-ryantm/nixpkgs/archive/05f8794b0ff7c78df76c3a940dc84f2d2ebbc87e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i641510vp000q0gprl0zw86lcv6c0ls3-python3.7-pyscreenshot-2.3
ls -la /nix/store/i641510vp000q0gprl0zw86lcv6c0ls3-python3.7-pyscreenshot-2.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python37Packages.pyscreenshot
python38Packages.pyscreenshot

Maintainer pings
cc @costrouc for testing.

Result of nixpkgs-review pr 107862 run on x86_64-darwin 1

3 packages built:

python37Packages.pyscreenshot
python38Packages.pyscreenshot
python39Packages.pyscreenshot


Fix thrift and ssl sniffing

This fixes: https://github.com/elasticsearch/elasticsearch-net/issues/1189
It also alters the thrift connection to generate thrift connections for endpoints on demand instead of in the constructor this allows endpoints that are discovered through sniffs to be also be initialized at any time.
I ran the indexation profiler a dozen time without hiccups and added two integration tests to ensure sniffing works and new nodes are being utilized.

I was running into a NullReferenceException in Sniffer.FromStream() because the credentials I was testing with existed, but weren't part of a role that had permission to _nodes/_all.  In this scenario, apparently ES returns a 403 Forbidden instead of a 401.
I amended the PR with the above commit to throw an ElasticsearchAuthenticationException on both 401 and 403.
Other than that this LGTM :+1:
Feel free to merge if you're good with my last change.


bug: Radio and checkbox problem in alert

Type: bug
Ionic Version: 2.x
Platform: desktop browser
Radio and checkbox not showing correctly when i add them with prompts in alert
https://forum.ionicframework.com/uploads/default/original/3X/b/6/b68c9c6ecbd4774899b82a29c36371bef461ff75.png

Would you be able to provide a code example of how you're writing this?
https://github.com/driftyco/ionic/blob/2.0/ionic/components/alert/test/basic/index.ts#L122

Yes , I do this example, It's work when I add radio buttons only, but it doesn't work when I add  them with inputs
let alert = Alert.create({
title: "Alert",
inputs:
[
{
name : 'FirstName',
placeholder: "FirstName"
},
{
name: "LastName",
placeholder: "LastName"
},
{
name: "Phone",
placeholder: "Phone"
},
{
name: "E-mail",
placeholder: "E-mail"
},
{
type: "radio",
label: "male",
value:"m"
},
{
type: "radio",
label: "female",
value:"f"
},
],
buttons: [
{
text: 'Cancel',
handler: data => {
console.log('Cancel clicked');
}
},
{
text: 'Save',
handler: data => {
console.log('Saved clicked');
}
}
]
});
this.nav.present(alert);

And the same think when I add buttons and inputs with alert.addInput({})

The alert does not allow mixing different types of inputs, like radio and checkboxes, or radio and inputs. I just updated the docs with that info, and also created a warning within the framework. Thanks
https://github.com/driftyco/ionic/commit/74a48f3b64f9613d9ff8992c6bd17de0f743bbed

Thanks for reply, but is there a feature to mix different types of inputs?


We need a mechanism to set GlycoCT sequence field externally

Is your feature request related to a problem? Please describe.
We need a mechanism to set GlycoCT sequence field externally and then name and diagram needs to be drawn based on the GlycoCT sequence value.
Describe the solution you'd like
A window load event should call a function (with timeout delay) which should internally render diagram and generate name based on GlycoCT sequence.

We currently don't have a method to convert GlycoCT to name. To do this we would need a mechanism to convert GlycoCT -> glycan JSON. Then we have a method to parse the JSON directly to either the name and the diagram.

Adding Rene to the discussion.
On Thu, Apr 7, 2022 at 12:53 PM Akul Mehta @.***> wrote:

We currently don't have a method to convert GlycoCT to name. To do this we
would need a mechanism to convert GlycoCT -> glycan JSON. Then we have a
method to parse the JSON directly to either the name and the diagram.
—
Reply to this email directly, view it on GitHub
https://github.com/akulmehta/GlycoGlyphPublic/issues/37#issuecomment-1091979735,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADJ7PGSUSDA6ZNYX74XJYO3VD4HIXANCNFSM5R3U25AA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Enable PostGIS extensions on default database

The postgres container implicitly creates a database for the postgres user (or the user specified at container creation by setting $POSTGRES_USER to something other than postgres). It might be nice if that database had the PostGIS extension installed automatically so that this image could work out of the box.

It should be possible to fix this once #9 is merged.

See https://github.com/appropriate/docker-postgis/pull/11 for a proposed fix


[maybe bug]

https://github.com/jpivarski/AwkwardArray.jl/blob/4f97922ded63937d45f4b4c3dd474f379d95fd3f/src/AwkwardArray.jl#L1320-L1323
certainly layout.array and layout.at would error because it's a Tuple not a NamedTuple

I did get tied up in a knot about this one. In Awkward's typesystem, tuples are records without field names. In Python, field names are referenced by strings and in Julia, field names are referenced by Symbols. In Python, tuple slots are referenced by integers-in-strings and in Julia, ... ?

record
tuple

Python
"x"
"2"

Julia
:x
???

So in AwkwardArray.jl, TupleArray fields can't be selected by getindex, and we have an additional AwkwardArray.slot function to do that instead. (The same use of the word "slot" was used in Awkward 1.x, as a method.) That's a loss because
some_record_array[:field_name][123] == some_record_array[123][:field_name]

is a nice way of thinking about row-like things (indexed by integers) and column-like things (indexed by Symbols). The above has no equivalent for TupleArray (because slot is defined for TupleArray but not for Tuple, a scalar item from a TupleArray).

Oh, keep in mind that, because the Awkward type is named "Tuple", uses of Tuple in this source file means a scalar item from a TupleArray. The Julia Tuple is Base.Tuple. In case that's what you were thinking.
Both a scalar Record and a scalar Tuple are just pointers to an element from the corresponding RecordArray and TupleArray (a columnar thing—better to index than to copy data). Here, layout is an AwkwardArray.Tuple, which has array and at fields to point to its TupleArray. getitem is defined on this scalar Tuple; it picks one column, so layout.array.contents[f][layout.at] is correct.

Oh, keep in mind that, because the Awkward type is named "Tuple", uses of Tuple in this source file means a scalar item from a TupleArray. The Julia Tuple is Base.Tuple. In case that's what you were thinking.

oh yea that's what confused me and JET.jl, is there no way to change a name, that's gonna be kinda confusing haha.

why don't we use 2 to replace "2" in Python? Or you can do one of
julia> a = (1,2,3)
(1, 2, 3)

julia> a[2]
2

julia> getfield(a, 2)
2


Chapters fade when all its challenges are completed

It wasn't doing it unless all the chapters in a same certification had all the others challenges completed too. fixed. :)

Also, removing the decoration after click.
closes #6648

@BerkeleyTrue can you take a look at Travis's error https://travis-ci.org/FreeCodeCamp/FreeCodeCamp/builds/106269480#L2714?

@hallaathrad ?

This PR passes all tests in my local. Looks good.

@bugron Did you try with a fresh npm install?

@BerkeleyTrue no, should I?

Yes, in order to find the cause of the break in travis.

:+1:


Add Licensing API

Add an API to install our licenses.
Those who directly install the WindowsAppSDK packages (i.e. don't run WindowsAppRuntimeInstaller.exe and don't use the Deployment API) need to install the *.msix packages AND their licenses.
Easiest way to do that is to
#include <WindowsAppRuntime-Licensing.h>
...
const uint32_t c_majorMinorVersion{ WINDOWSAPPSDK_RELEASE_MAJORMINOR };
PCWSTR c_versionTag{ WINDOWSAPPSDK_RELEASE_VERSION_TAG_W };
const auto hr{ WindowsAppRuntime_InstallLicenses(c_majorMinorVersion, c_versionTag) };
if (FAILED(hr))
{
    ...error...
}

NOTE: There's also WindowsAppRuntime_InstallLicensesInPackage() but that's primarily for testing.
See WindowsAppRuntime-Licensing.h for more details.

Gah! Pointing to wrong branch, and no way to alter once created :-(
Abandoning this PR and cooking a corrected one

@DrusTheAxe FYI for next time, click the Edit button then change the target branch.
Example:

click the Edit button then change the target branch. (Then rebase your changes onto the new target branch.)

Hmmm. I hit Edit and saw the title was editable but didn't notice the branch too. My bad. I doubt it'll be the last time ;-)
Thanks for pointing it out


Vulnerabilities detected in container

To replicate:
make build
docker scan ibmosquito/web-hello-python:1.0.0

Results found four vulnerabilities:
Testing ibmosquito/web-hello-python:1.0.0...

✗ Medium severity vulnerability found in sqlite/sqlite-libs
  Description: Use After Free
  Info: https://snyk.io/vuln/SNYK-ALPINE312-SQLITE-1300270
  Introduced through<EMAIL_ADDRESS><EMAIL_ADDRESS>  From<EMAIL_ADDRESS>  From<EMAIL_ADDRESS>><EMAIL_ADDRESS>  Fixed in: 3.32.1-r1

✗ Medium severity vulnerability found in sqlite/sqlite-libs
  Description: Out-of-bounds Write
  Info: https://snyk.io/vuln/SNYK-ALPINE312-SQLITE-1300271
  Introduced through<EMAIL_ADDRESS><EMAIL_ADDRESS>  From<EMAIL_ADDRESS>  From<EMAIL_ADDRESS>><EMAIL_ADDRESS>  Fixed in: 3.32.1-r1

✗ High severity vulnerability found in krb5/krb5-libs
  Description: NULL Pointer Dereference
  Info: https://snyk.io/vuln/SNYK-ALPINE312-KRB5-1533463
  Introduced through<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>  From<EMAIL_ADDRESS>  From<EMAIL_ADDRESS>><EMAIL_ADDRESS>  From<EMAIL_ADDRESS>><EMAIL_ADDRESS>  Fixed in: 1.18.4-r0

✗ High severity vulnerability found in apk-tools/apk-tools
  Description: Out-of-bounds Read
  Info: https://snyk.io/vuln/SNYK-ALPINE312-APKTOOLS-1533753
  Introduced through<EMAIL_ADDRESS>  From<EMAIL_ADDRESS>  Fixed in: 2.10.7-r0

Package manager:   apk
Project name:      docker-image|ibmosquito/web-hello-python
Docker image:      ibmosquito/web-hello-python:1.0.0
Platform:          linux/amd64
Base image:        python:3-alpine3.12

Tested 37 dependencies for known vulnerabilities, found 4 vulnerabilities.

This is a good first issue.  The way to fix this would be to modify the Dockerfile
FROM python:3.10-alpine

@joewxboy  can I be assigned this issue please?

@WilliamLi10 I've assigned this issue to you.  Thanks!


Reactions with ContextMenu

Need to do some cleanup, but this looks pretty neat

Love it!
Sent with GitHawk

I'll try and sneak this in for the latest release, why not. The whole UIMenu stuff seems to break a lot of things w/ the UICollectionView anyways.

#yolo

Didn’t see this. Looks amazing! 🤯
Sent with GitHawk

Was ReactionsMenuViewController added?

We should add the UIAccessibilityTraitButton for these :)
Sent with GitHawk


feat: add pre-commit automatic code formatting tool

Add .pre-commit-config.yaml configuration file for pre-commit tool.
This will run some code formatting automatically: clang-format, black, formatting cmake files, etc.
The pre-commit bot can be configured to automatically enforce this style in PRs.
I have not formatted the whole repo using this configuration file as it would be a very big diff. If you agree on using pre-commit, we could let the CI bot do it.

Thanks, I'm not sure about using all those formatters but clang-format as pre-commit hook is definitely a nice addition.


Convert to Python3

using : find . -name '*.py' | xargs 2to3 -w

Hi! Marcus is on holiday but he should be back in a weeks time.
What's the idea behind adding the .bak files? If it is backup then the git history will be enough to undo changes if need be


Fix building with multiple jobs for BSDmake

BSDmake does not accept the --jobs long-form flag and requires the use of -j.
The issue can be reproduced on a Ubuntu docker container by installing bmake and invoking that instead of make.
Fixes php/pie#80

Thank you @TimWolla ! 🙌


Windows install

For anyone trying to install under windows this may help to get aqua running:
When running install if you have bcrypt or other issues withing compiling try using the "--msvs-version=2013" flag: npm install --msvs-version=2013  (if you have that installed or try a later version if that is what you have).  I had so slog though all the bcrypt install solutions find something that worked.
"npm run setup"  threw an error on ./setup.js.  I had to remove the "./" on line 7 ("setup" : "./setup.js") of package.json so windows would not complain.
"npm run setup" also just kept opening setup.js instead of prompting. I got it to run setup just using: node setup.js
I have it running, but have not tried the testing.  I assume it may complain about the test paths also using "./"  instead of "."  in package.json. These may need to be changed too.

Thanks for opening an issue and sharing these details.
Do you think we should add some of these details to the bcrypt-Installation-Trouble wiki page?

Yes adding some of the items to the wiki may be helpful.  I started with
drywall and went through the links along with other items I found.  It took
a few hours to get bcrypt installed.  I actually had to use another
work-around (cheat actually). see below.  When I installed aqua i was able
to use the --msvs-version switch.  Would you like me to edit it?
I don't think the path issues in package.json and the npm run setup issues
should go on the bcrypt page though.  I don't know why npm does not handle
the ./ in the path to setup.js.  Just removing the ./ from the package.json
should suffice.
I also don't understand why running 'npm run setup' just kept opening
setup.js in my editor. A note about that in the install instructions to try
running 'node setup.js' directly should suffice.
Alternate bcrypt install work-around (ran into this installing bcrypt for
drywall):
I had some VS 2010 redistributables installed, but I also had VS 2013
Express and VS 2015 community. I kept getting an error message looking for
MSBuild in a folder call V110 in 'C:\Program Files
(x86)\MSBuild\Microsoft.Cpp\v4.0'.  I only had V120 and V140.  Out of
frustration I renamed V120 to V110 to get the install to work and renamed
it after.  Later found the --msvs-version=2013 parameter.
Allan Dumaine
480 342-9045
On Mon, Aug 24, 2015 at 1:25 PM, Reza Akhavan<EMAIL_ADDRESS>wrote:

Thanks for opening an issue and sharing these details.
Do you think we should add some of these details to the
bcrypt-Installation-Trouble
https://github.com/jedireza/aqua/wiki/bcrypt-Installation-Trouble wiki
page?
—
Reply to this email directly or view it on GitHub
https://github.com/jedireza/aqua/issues/55#issuecomment-134366582.

Yes adding some of the items to the wiki may be helpful. I started with drywall and went through the links along with other items I found. It took a few hours to get bcrypt installed. I actually had to use another work-around (cheat actually). see below. When I installed aqua i was able to use the --msvs-version switch. Would you like me to edit it?

Sure. Let me know when it's been updated.

I don't think the path issues in package.json and the npm run setup issues should go on the bcrypt page though. I don't know why npm does not handle the ./ in the path to setup.js. Just removing the ./ from the package.json should suffice.

Bummer. I took the ./ out of the setup script and it fails on a mac. Keep in mind that the setup.js file is executable (ex: chmod +x).


Auto-convert all available balances in different currencies to a given coin at the optimal rate

Is your feature request related to a problem? Please describe.
Routine and time-consuming, which needs to be done every time you need to withdraw free assets.
Describe the solution you'd like
All balances in the allocated account, the size of which allows conversion, are converted to a given coin. Depending on the volume, the maximum possible number of orders is set. If the price has gone after a specified time, the grid shifts after the price. The purpose of the operation is to convert at a better price at the moment.
Additional context
The implementation option is an additional, independent strategy.

As prepared stage - before starting strategy checking free asset volume and transferred it to main account. The main account not for trading strategy. Only for consolidate assets and manage it. On main account wold be started control and conversion bot.


[Feature Request] Add revocation EventSub Message Type

I think it would be useful to be able to test for when an EventSub message is of type revocation.  This is the only one currently missing from the three types.
Going off of the current design, it seems like we'd want an additional command like twitch event revoke <event_type> ...etc, but it could also be as simple as adding a flag like -R and --revoke to the trigger command.
See here for relevant doc

It would be very useful for me right now. Looking forward for this feature.


USHIFT-180: add version config map

Add ConfigMap called microshift-version to the kube-public namespace with version details of the MicroShift service so that API consumers can tell that they are talking to a MicroShift instance.
Which issue(s) this PR addresses:
Closes USHIFT-180

/cc @derekwaynecarr

/hold
This isn't the only way to solve the problem, and we should discuss the security implications of making the version details freely available.

/cc @fzdarsky

/cc @deads2k

/hold cancel

/lgtm

would you like to get a review from a RHACM server foundation engineer to ensure that the bits involved will do what we need?  I get that this is a 'generic' goal, but lets ensure it fits also the immediate need.

@berenss sure, we'd be happy to have any specific feedback or requirements. This solves the immediate need to have a way to tell what version is in place from a workload, but it's not necessarily the only info that would be useful


fix issue #731

fix issue #731

could you add a test case please

ideally we'd benchmark it, but i think that's a good idea.
Thanks,
David
On Thu, Nov 29, 2018 at 10:18 PM Boleyn Su<EMAIL_ADDRESS>wrote:

@BoleynSu commented on this pull request.
In
math/src/main/java/breeze/linalg/operators/DenseVectorSupportMethods.java
https://github.com/scalanlp/breeze/pull/732#discussion_r237756740:

  * @param a

  * @param b
  * @param length
  * @return
  */
 public static double smallDotProduct_Double(double[] a, double[] b, int length) {

   double sumA = 0.0;

   double sumB = 0.0;

   double sum = 0.0;

I removed sumA and sumB to make smallDotProduct for both double and float
the same. If I revert this, should I use the same optimization for float?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/scalanlp/breeze/pull/732#discussion_r237756740, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAlocF5A20AVA_zD62s4tSN_kry9DcLks5u0M3HgaJpZM4YWPP9
.

@dlwh I have added tests and reverted sum back to sumA and sumB.

Thanks!


Create a page allocator

Summary
Before we can create a loader that will be able to set up memory properly for processes about to run, we need a way to allocate a stack, heap, text, and data section for each process. To do this, we will use a pager to request blocks of memory, called pages, from the kernel in order to fill our memory section needs. Your task involves creating this pager, which will serve users blocks of memory of fixed size from a specified area in memory.
Tasks

[ ] Create global variables that will hold the state of the pager
[ ] Implement page_init()
[ ] Implement page_alloc()
[ ] Implement page_free()

i am totally qualified to do this


Allow to ignore symlinks

:zap: Summary

Some CI tooling, such as Prettier (https://github.com/prettier/prettier/pull/14627) fails on symlinks. Currently, Lefthook has no way to skip paths changed under Git that are of a certain type/mode, e.g., a symlink .
Value

More control over paths specified to commands requires fewer work-arounds or complex configurations of the tools themselves.
Behavior and configuration changes

Allow a per-command option to filter symlinks, default off.

Closed as duplicate of https://github.com/evilmartians/lefthook/issues/76


When to Swift 4?

Is there a compatible branch for swift3.2: in that case, it would work with Swift 4?

there is no branch for 3.2

where is the swift branch named 3.2

Give them a break guys! Or do it and PR the changes :-)

@ParanoiaWan @KarlZeo Alamofire builds under Xcode 9 beta 1 just fine in Swift 3.2 mode. In addition, you can use the Alamofire framework built in 3.2 mode in an app that builds using Swift 4, according to Apple, so a Swift 4 port isn't strictly necessary at the moment. If you really want to build Alamofire using Swift 4, the required source changes are very minor. We'll be announcing our future plans soon!
@Vinzius Thanks! 🍻

Did the future plans have an announcement yet?

Any updates on Almofire with Swift 4 guys?

Alamofire has supported Swift 4 for months now. Check our README.


Add processing mode to default error handler

Description
Add processing mode to default SAX error handler. Instead of always treating errors as fatal errors by throwing an exception, the handler can throw the error only in strict processing mode.
Motivation and Context
Per SAX interface documentation, default behaviour for error should be to ignore them. We don't want to ignore them, but we don't want to treat all errors as fatal errors either. By using processing mode to decide how to handle errors, we allow parsing to continue in lax mode and only log the error.
How Has This Been Tested?

Type of Changes

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

Just to see if I understand what this means for DITA OT users, there are two types of SAX exceptions:

Exceptions when the processed XML is not wellformed. And such exceptions break the parsing anyway.
Exceptions when the XML does not adhere to the DTD/XML Schema/RNG. For example the XML has an invalid element somewhere. And in these cases the processing of a topic would stop on the first error and the topic would be ignored but after your fix the processing of the topic might proceed until the end if there is processing-mode=lax?


inline debuging shows incomplete json parse result

Testing #18670
Steps to Reproduce:

Debugging https://github.com/rebornix/vscode-project-snippet
Add breakpoint to Line 23 in extension.ts
Debug and hit breakpoints.

The value of JSON parse is incomplete as "{

This is how inline css values are treating \ncharacters in string.
Forwarding to @aeschli I would like the inline css to preserve my escaped new line character
If I double escape it on my side I get an n character which is also not good
Good old escaping

Not sure what you mean by inline css...

@isidorn Isi, do you have more information?

Yes, sorry for not making it clear in my previous comment.
Have the following string 'hello \n martin' as the contentText for an end of line decoration. Add that decoration to the editor and notice that only the 'hello' is decorated.
I would expect the whole string to be contained in the decoration (the newline escaped)

You should do that on your side. Maybe replace all new lines with spaces, or cut it.
The before/after decorator can only render one line. Passing multiple lines is an invalid use of the API and the implementation now just takes the first line.

@aeschli But if I escape the \n with \n you show just the n, not \n.
I could just remove it, but showing \n would be better

PRs accepted
Code pointer https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/debug/electron-browser/debugEditorContribution.ts#L48

Passing more than one line to the end of line decoration is an "invalid use of the API" which is a bug.
I suggest to fix it by replacing the newline by a space (and to file a new issue "backslash not shown in end of line decoration")


Bring in master changes

Coverage decreased (-1.02%) to 97.53% when pulling b23c2bc3f88a37ccca200b4fc47304f6df10881c on master into fe282108bc66a9a8318f22e57e191cd6b969eede on revise.


Fix @type and @id of AnalysisTechnique instances

This fixes the first part of #28. The question of whether all of these are actually analysis techniques still has to be addressed.

We need to do the other part first. Otherwise we move instances between types twice. I will comment in the PR which ones I think need to move back to Technique

Replaced by #150


Create interface for hand

In order to get better at OOP best practices we think it would be good to create a hand interface

has been implemented but needs more work


Locked drops not always removing PDC data causing item stacking issues

"Was not an issue prior to latest release"

Looking at an alternate approach to how this is handled but will likely require Paper / Spigot API changes to implement in a nice way.

Will be fixed by #115


"startGrabbing: Node is not writable." in pylon_camera package

When I try to launch the pylon_camera node, I receive the following error:

startGrabbing: Node is not writable.

I traced through the code and found that an exception is thrown when the node is trying to set the PixelFormat to Mono8 using the setValue function in pylon_camera_base.hpp in the startGrabbing function.
I'm using the basler ace acA4600-7gc GigE camera with pylon 5 sdk. I can view the camera images and set camera parameters just fine using the PylonViewerApp.
http://answers.ros.org/question/237591/startgrabbing-node-is-not-writable-in-pylon_camera-package/

Thanks for reporting! I could reproduce the error using a acA4600-10uc and will try to provide a fix within the next days.

Great, thank you! If you need any more info from me, please let me know!

Done! Please leave a feedback if your problem could be solved.

That seems to have done the trick! Thank you for such a quick response!


main.tf line 366, in resource "kubernetes_config_map" "sas_iac_buildinfo":

PLease, would really appreciate ur help I am close.
Sorry, just learning as i go, seems like i am almost there.
module.jump[0].azurerm_linux_virtual_machine.vm: Creation complete after 1m49s [id=/subscriptions/6bea75b7-34c4-477e-bd72-02c58414d151/resourceGroups/kube5-rg/providers/Microsoft.Compute/virtualMachines/kube5-jump-vm]
╷
│ Error: Post "https://kube5-aks-1b02fa42.hcp.westus2.azmk8s.io:443/api/v1/namespaces/kube-system/configmaps": dial tcp <IP_ADDRESS>:443: i/o timeout
│
│   with kubernetes_config_map.sas_iac_buildinfo,
│   on main.tf line 366, in resource "kubernetes_config_map" "sas_iac_buildinfo":
│  366: resource "kubernetes_config_map" "sas_iac_buildinfo" {
│
╵
╷
│ Error: Post "https://kube5-aks-1b02fa42.hcp.westus2.azmk8s.io:443/api/v1/namespaces/kube-system/serviceaccounts": context deadline exceeded
│
│   with module.kubeconfig.kubernetes_service_account.kubernetes_sa[0],
│   on modules/kubeconfig/main.tf line 45, in resource "kubernetes_service_account" "kubernetes_sa":
│   45: resource "kubernetes_service_account" "kubernetes_sa" {
│

From what I can tell here, you did not provide a default_public_access_cidrs as define here for your network.

Yes
I got it fixed. I put the wrong cidr. Now it looks good.
Any documentation as to how to access the cluster once created?
Henry
From: Thomas S. Pangborn @.>
Sent: Thursday, June 10, 2021 8:17 AM
To: sassoftware/viya4-iac-azure @.>
Cc: Henry Mora @.>; Author @.>
Subject: Re: [sassoftware/viya4-iac-azure] main.tf line 366, in resource "kubernetes_config_map" "sas_iac_buildinfo": (#165)
From what I can tell here, you did not provide a default_public_access_cidrs as define herehttps://github.com/sassoftware/viya4-iac-azure/blob/main/examples/sample-input.tfvars#L15-L17 for your network.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sassoftware/viya4-iac-azure/issues/165#issuecomment-858570367, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUNPLY3GYANSSFCMKAJDROTTSCULTANCNFSM46OMMEQQ.

I have logged in to the vm created by terraform, how can I get access to the Kubernetes?
HEnry
From: Henry Mora
Sent: Thursday, June 10, 2021 9:15 AM
To: sassoftware/viya4-iac-azure @.>; sassoftware/viya4-iac-azure @.>
Cc: Author @.***>
Subject: RE: [sassoftware/viya4-iac-azure] main.tf line 366, in resource "kubernetes_config_map" "sas_iac_buildinfo": (#165)
Yes
I got it fixed. I put the wrong cidr. Now it looks good.
Any documentation as to how to access the cluster once created?
Henry
From: Thomas S. Pangborn<EMAIL_ADDRESS>Sent: Thursday, June 10, 2021 8:17 AM
To: sassoftware/viya4-iac-azure<EMAIL_ADDRESS>Cc: Henry Mora<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [sassoftware/viya4-iac-azure] main.tf line 366, in resource "kubernetes_config_map" "sas_iac_buildinfo": (#165)
From what I can tell here, you did not provide a default_public_access_cidrs as define herehttps://github.com/sassoftware/viya4-iac-azure/blob/main/examples/sample-input.tfvars#L15-L17 for your network.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sassoftware/viya4-iac-azure/issues/165#issuecomment-858570367, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUNPLY3GYANSSFCMKAJDROTTSCULTANCNFSM46OMMEQQ.

You should have a kube config that was generated. The file is the *.conf file where you ran terraform. The name will be based on your prefix and have a name like <your_prefix>-azure-kubeconfig.conf
You can then run the following:
export KUBECONFIG=<terraform_directory_where_you_created_your_cluster>/<your_prefix>-azure-kubeconfig.conf
kubectl cluster-info

If this shows info on your cluster your golden.


Add Vale rule to require title-case H1s

The style guide calls for title case H1s, in contrast to sentence case H2s elsewhere. Speculatively, I've written a Vale rule to require this and fixed up the handful of cases where this convention wasn't already followed.
If this is desirable, I can create a corresponding rule for sentence case on H2+ headings.
Prompted by @karensawrey in https://github.com/buildkite/docs/pull/1615#discussion_r911041368.

I actually thought I'd done this already, but maybe it's one of the things I fiddled with and didn't push.
Yeah, let's do it for H2 as well. I'd also like to mark which rules in the style Guide are validated by linters, and try to slowly increase that number.

@plaindocs I'd like to do the other headings in incremental follow up PRs, if that's OK with you. A quick look shows that the H2s are a bit more complicated and will require some actual decision making, since they don't as widely follow the Chicago style.

Sure thing :heavy_check_mark:


notification analysis finished in X seconds must be cleared when a new analysis is initiated

Because otherwise we have potentially contradicting information on what is actually going on.

closed in 625ef985e799df4737d4d5cf2f3cfea3e2132f3c, but I forgot to tag the commit


Initial review

Since the first commit wasn't created via a PR, I'll post any review notes here.

https://github.com/beeware/briefcase-linux-flatpak-template/blob/86958cb98afc1a06b01d4a0a771b014d3f86a129/{{ cookiecutter.formal_name }}/briefcase.toml#L1-L4
If this template doesn't use support_path, then the line should just be omitted rather than commented out.

@freakboy3742: It doesn't look like you're watching this repository, or is there some way to watch an entire GitHub organization?

It looks like the template doesn't currently use app_packages_path either. However, for Chaquopy I had the idea of making the template use briefcase.toml to give the path of the requirements.txt file (including filename). That way, the path doesn't need to be hard-coded in briefcase itself, and if it ever changes, only the template will need to be updated.
In fact, if we use a different briefcase.toml variable, such as app_requirements_path, then the template can control whether to generate a requirements file or to run pip directly, so that doesn't need to be hardcoded in briefcase either.
I haven't implemented this yet, so if you want to do it for Flatpak then I'll reuse it. What I have in mind is merging the install_app_dependencies override in flatpak.py into the base class impementation in create.py, as follows:

If app_requirements_path exists in briefcase.toml, then write a requirements file to that location.
Otherwise, run pip to install to app_packages_path.

That's a good idea - it means one less special case in the Flatpak backend, and it provides a clear flag that a template can use to opt-in to requirements generation rather than direct pip installation. I'll add this to my flatpak branch.
As for watching the repo; that was an oversight - I've adjusted my watch configuration (and done the same on a couple of other repos I've added recently).

#2 addresses the template-related parts of this review.


Add new validation rule: VALUE_OBJECT_CANNOT_CONTAIN_UNDEFINED

By @warpech in https://github.com/PuppetJs/PuppetJs/issues/21
Fair enough!  PuppetJs in debug mode (currently ON by default) validates outgoing patches:
   if(this.debug) {
      this.validateSequence(this.remoteObj, patches);
    }

in which case the error should be reported. I am glad that we both agree :)

@miyconst: Current validation is implemented here: https://github.com/Starcounter-Jack/JSON-Patch/blob/master/src/json-patch-duplex.js#L632

And it is throwing OPERATION_VALUE_REQUIRED for undefined values, which should be replaced with VALUE_OBJECT_CANNOT_CONTAIN_UNDEFINED.

It should not be mixed. The current check for undefined value (OPERATION_VALUE_REQUIRED) should be kept. A new check for undefined property of the value (if the value is an object) should be added after it (VALUE_OBJECT_CANNOT_CONTAIN_UNDEFINED).
@miyconst could you prepare a solution for this on a separate branch in https://github.com/Starcounter-Jack/JSON-Patch? Create a new issue for it, publish code on a separate branch for review. The changes should include test and update to README.md

@warpech I don't access to that repository.

Unfortunately I cannot give you access there... I will ask @Starcounter-Jack. In the meantime, could you do it as a pull request?

You should have access now

Please recreate this issue in JSON-Patch issues board


Service port gets quoted with a puppet 6.10 catalog server and consul rejects it

With this service configuration:
consul::service { 'node_exporter':
address => $::hostname,
checks  => [
{
'http'     => 'http://localhost:9100',
'interval' => '5s',
}
],
port    => 9100,
tags    => $consul_tags;
}
consul module version 6.0.0, and a puppet 6.10 catalog server I get a service json file with a quoted port number:
{ "service": {"address":"cadws002","checks":[{"http":"http://localhost:9100","interval":"5s"}],"enable_tag_override":false,"id":"node_exporter","meta":null,"name":"node_exporter","port":"9100","tags":["node_exporter","metrics","eng"],"token":null}}
which results in a consul error: * 'service.port' expected type 'int', got unconvertible type 'string'
The sorted_json() function doesn't seem to do this when the same code is run through a puppet 5.5 catalog server.

Additionally, the major version of the client doesn't matter. A 5.5 client pointed at a 6.10 catalog server will get the stringified integer, and a 6.10 client pointed at a 5.5 catalog service will get a bare integer.
Removing 'Integer' from 'when Integer, Float, TrueClass, FalseClass' in sorted_json allows it to fall through and not get quoted - though I'm still not sure why 5.5 wasn't hitting the same code path unless it was stringifying it on the way in and having it end up in the else clause.

It's looking more like code to turn on the quoted boolean for everything inside node_meta|meta|tags hashes in the 2 sorted*generate functions is instead turning it on for everything that happens after it in the loop.

I'm running into this issue as well

Hi people. I need to say that I'm unable to reproduce this. I'm running version 6.1.0 of this module with puppetserver 6.12.1 and puppet agent 6.17.0. example consul code:
    consul::service { 'node_exporter':
      checks  => [
        {
          name     => 'node_exporter health check',
          http     => 'http://<IP_ADDRESS>:9100',
          interval => '10s',
          timeout  => '1s'
        },
      ],
      port    => 9100,
      address => $consul_address,
      tags    => ['node_exporter'],
    }

results in:
{
    "service": {
        "address": "***",
        "checks": [
            {
                "http": "http://<IP_ADDRESS>:9100",
                "interval": "10s",
                "name": "node_exporter health check",
                "timeout": "1s"
            }
        ],
        "enable_tag_override": false,
        "id": "node_exporter",
        "name": "node_exporter",
        "port": 9100,
        "tags": [
            "node_exporter"
        ]
    }
}

The simple fix of excluding Integer from the list fixes the quoted port issue, but it does not fix any more complicated configs with nesting because the recursion causes quoting to get turned on for all enclosed parameters, even if they should not be quoted. I tried with https://github.com/solarkennedy/puppet-consul/pull/527 but never pushed it over the finish line with an RH6 TravisCI issue that I could not replicate locally.

I was running version 6.0.1 of the module, with puppetserver 6.12.0 and puppet agent 6.17.0.
After upgrading to version 6.1.0 of the module I'm now getting the proper, unquoted integer for the port number.
# cat /etc/consul/service_node-exporter.json | jq
{
  "service": {
    "address": "host.example.com",
    "checks": [
      {
        "http": "http://localhost:9100",
        "interval": "10s"
      }
    ],
    "enable_tag_override": false,
    "id": "node-exporter",
    "name": "node-exporter",
    "port": 9100,
    "tags": [
      "node_exporter"
    ]
  }
}

I should have made sure I was on the latest version of the module. Thank you @bastelfreak and @rtkennedy !


Changing Mesh Name in SSBH Not Reflected in-game or in StudioSB (potentially deprecated)

Sorry if this is a bugfix you've already implemented, but this issue is from a while ago and I want it up just in case it might point to something fixable:
I changed the name of a mesh in ssbh. The name change is reflected in ssbh editor, but not in studio sb. I also have reason to suspect that the fact it isn't changed in studiosb reflects the fact that the hair mesh still acts as if it is a highmesh instead of a regular, unnamed mesh (ie a mesh that will be visible whether it's docked or undocked)

Did you save the mesh with File > Save in the mesh editor? Is the change reflected in SSBH Editor after you reload the workspace or open the folder again? StudioSB has a lot of bugs, so I wouldn't use it as a reference. You should also see the change using ssbh_lib_json or ssbh_data_json.

Solved in updated version of SSBH Editor (the editor shows the name with the _highshape suffix)


update examples for jobs-run-to-completion.md

This PR updates the below content.

https://kubernetes.io/docs/concepts/workloads/controllers/jobs-run-to-completion/

Some examples in the content is old and there is a little difference between it and current version. Therefore, we should update it.
I confirmed that the output looks like as the below in v1.15 and v1.18 alpha.
$ cluster/kubectl.sh version
Client Version: version.Info{Major:"1", Minor:"18+", GitVersion:"v1.18.0-alpha.0.1321+f6337c762483e6-dirty", GitCommit:"f6337c762483e60e41dd775166afd536b130455e", GitTreeState:"dirty", BuildDate:"2019-12-02T05:58:25Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18+", GitVersion:"v1.18.0-alpha.0.1321+f6337c762483e6-dirty", GitCommit:"f6337c762483e60e41dd775166afd536b130455e", GitTreeState:"dirty", BuildDate:"2019-12-02T05:58:25Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"linux/amd64"}

$ cluster/kubectl.sh describe jobs/pi
Name:           pi
Namespace:      default
Selector:       controller-uid=c9948307-e56d-4b5d-8302-ae2d7b7da67c
Labels:         controller-uid=c9948307-e56d-4b5d-8302-ae2d7b7da67c
                job-name=pi
Annotations:    kubectl.kubernetes.io/last-applied-configuration:
                  {"apiVersion":"batch/v1","kind":"Job","metadata":{"annotations":{},"name":"pi","namespace":"default"},"spec":{"backoffLimit":4,"template":...
Parallelism:    1
Completions:    1
Start Time:     Mon, 02 Dec 2019 15:20:11 +0200
Completed At:   Mon, 02 Dec 2019 15:21:16 +0200
Duration:       65s
Pods Statuses:  0 Running / 1 Succeeded / 0 Failed
Pod Template:
  Labels:  controller-uid=c9948307-e56d-4b5d-8302-ae2d7b7da67c
           job-name=pi
  Containers:
   pi:
    Image:      perl
    Port:       <none>
    Host Port:  <none>
    Command:
      perl
      -Mbignum=bpi
      -wle
      print bpi(2000)
    Environment:  <none>
    Mounts:       <none>
  Volumes:        <none>
Events:
  Type    Reason            Age   From            Message
  ----    ------            ----  ----            -------
  Normal  SuccessfulCreate  46m   job-controller  Created pod: pi-5rwd7

Thanks for updating the example and providing the output in the description.
/lgtm
/approve


feat(eds/store): cache proofs after first eds recompute on sampling

Overview
Before this change eds was recomputed twice on sampling:

inside sampling client to verify data integrity
On storing EDS to storage to calculate merkle tree proof nodes.

This PR adds proofsAdder cache, that will store proof nodes for later use with key features:

cache could be optionally added to context and will be respected by ipld/shrex getter to populate on eds recompute.
cache is safe from second recompute, since it will only allow single recompute write  and all consequent recomputes over same cache will be noop.

Also this PR reworks writeEDS to use use proofsAdder instead of creating new blockstore and iterating over all keys in it. This approach speeds up writeEDS even without pre collected cache.
Since there are no more need for blockstore being created, I have simplified writeEDS and removed a lot of redundant code including whole struct of writeSession.
The PR also adds benchmark that uses disk for badger store (existing benchmark uses in-memory datastore instead of badger).
This PR brings significant performance increase. On disk store badger results are:

before PR store.Put operation: ~1.5s
after PR store.Put with cached proofs: 800ms
after PR store.Put without cached proofs: 900ms

Does this solution works with bridge?

Needs support for bridges by passing nmt options through app's block extension code

Added support for bridges and also made it to not pass proofs inside context when it is possible to.

Codecov Report

:exclamation: No coverage uploaded for pull request base (v0.11.0-rc9@d256425). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             v0.11.0-rc9    #2429   +/-   ##
==============================================
  Coverage               ?   52.53%
==============================================
  Files                  ?      156
  Lines                  ?    10042
  Branches               ?        0
==============================================
  Hits                   ?     5276
  Misses                 ?     4302
  Partials               ?      464


Add GPU support with cupy and dask

I have added support for GPU processing for CUDA-enabled GPUs through cupy. Calling ddm.processing.dmm will automatically determine whether cupy is installed properly.
Installation
In order to use cupy, you will need the following requirement: https://docs.cupy.dev/en/stable/install.html#requirements
I have excluded cupy from the environment.yml as not everyone has a GPU and the cuda-toolkit is ~1Gb. In order to install it, add the cuda option to the pip installation of the package.
pip install -e .[dev,cuda]

I used these references:

https://blog.dask.org/2019/01/03/dask-array-gpus-first-steps
https://stackoverflow.com/questions/56776385/how-to-create-a-dask-array-from-cupy-array

TODO
In order to make use of the dask dashboard, you will need to install dask-cuda. However, this package is currently only available for Linux.

Benchmark
Dataset
Simulated dataset with 100 frames of 512x512, totalling 512 MB.
GPU
_Nvidia Geforce GTX 1070 with 8GB of dedicated GPU memory. _

Chunk size = (50,512,512)
Calling dask.compute on every tau separately

Result: Calculating ddm for taus=np.arange(1,100) takes ~40 seconds
CPU
Intel i7-8665U @ 1.9GHz (TU Delft laptop)

Chunk size = (5,512,512)
Calling dask.compute on entire tau range
Hyperthreading enabled (2 threads per core)
Boost enabled @ 3GHz

Result: Calculating ddm for taus=np.arange(1,100) takes ~270 seconds


Update futf to 0.1.2

This is necessary for cargo update -Z minimal-version builds.

@bors-servo r+

:pushpin: Commit 3367e11 has been approved by jdm

:hourglass: Testing commit 3367e112087d159591cce856348997d424ced788 with merge e379f8547241d84865aa45f44802feeff1126726...

:sunny: Test successful - checks-travis
Approved by: jdm
Pushing e379f8547241d84865aa45f44802feeff1126726 to master...


feat(input-text): add readonly

This work is done as part of LEARNER-6116.
Adds an optional readOnly attribute to the inputText component since Firefox does not allow users to copy text from a disabled field.
Storyboard
If the reviewer decides that this doesn't need its own story, I'm happy to remove it.  Added just to give an idea of the UI.

Coverage remained the same at 99.551% when pulling 283a74cfbdcf2b8ebff60a1416ca51068624ec9e on bbaker/inputText-readOnly into a4bc543678e03243a1d0ed35d6fd7801b89850fa on master.

Coverage remained the same at 99.551% when pulling 283a74cfbdcf2b8ebff60a1416ca51068624ec9e on bbaker/inputText-readOnly into a4bc543678e03243a1d0ed35d6fd7801b89850fa on master.

Added new test to verify that the input is readOnly only when it is explicitly set

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How committed are you to this?

Title.

I am seriously committed. I will be hunting contributors who are willing to help build this project, report bugs and work on its documentation.
I also want to extend this project by creating great tooling, including gatsby plugin, nextjs helpers, apollo link plugin and so on.
We are in great need for a GraphQL-first type of headless CMS.
So, yes, seriously committed 🏗

relationships between models ???

Yep, Prime has one-to-one and one-to-many.
It's the Document Field

Hey, there! I've been writing my own headless GraphQL CMS based on GraphQL Modules but I was thinking that creating the API + the whole admin UI + a plugin system + writing tests + writing documentation was a bit too much for one man, so I gave it up. Until I saw this project, which definitely matches my needs.

Hey @Laegel !
Still just one man over here, so any help appreciated!!
I have been busy in Gatsby+Prismic land for the past 2 months. Prime will be ramping up soon enough.


Focus should not reach

Testing #2145
setInterval(function() { console.log(document.activeElement); }, 2000);

put last editor in Tab moves focus mode via Ctrl+M
tab in last editor:

@alexandrudima yeah, I am not sure I can disable this actually. maybe I can redirect the focus to the first control somewhere.

@alexandrudima tried a while now to see if I can prevent keyboard to navigate into the  element and failed to find anything useful. The only super hack I can come up with is to listen globally to Tab-KeyUp and find out if the active element is the body and then move focus to some element within.
If anyone has ideas, they are most welcome.

@bpasero 'focus' listener on <body> ?

or there was another one 'focusin' - I forgot the exact difference. You can check if the e.target === body

@alexandrudima best I can come up with is this:
window.document.body.addEventListener('blur', function(e) {
   setTimeout(function() { console.log(document.activeElement);});
}, true);

I do not get any focus event on the body element, so listen on blur in the capture phase and use timeout because without, activeElement is === body. Still a bit ugly.

I am not really able to do this nicely, moving to backlog.

Maybe this will help in Chrome 50: https://developers.google.com/web/updates/2016/03/focus-start-point

@bpasero
This looks like it also relates to what I'm trying to do with some list  navigation improvement with
PR #24628 in dealing with tabindex issues
you still have help-wanted tag shall I take a look at it or is somebody on this?

@cleidigh the issue is that when you tab long enough through the UI you end up in the body element at some point without any visual indication. If possible I wanted to disallow jumping to this element because it does not really help for navigation.

Closing as I am seeing no way of doing this currently, if someone wants to fix this, PR me.


Assert that links work in Functional tests

To avoid issues like this we should Assert that our links are correct (or at least valid) in our functional tests.

@ryanbrandenburg, please spend some time to first confirm whether this is feasible approach. One scenario I think about where this won't work is when an Click here tag is wired up with javascript onclick event handler. Not sure we have that though, or not.


where is the pasta_weights

Do you provide the pasta_weights? And where is it? Or should I train it by myself.

Thanks for using.
Which pasta_weights are you referring to?

In activaty2vec, when I use main.py to do inference, it needs the pretrained model "pasta_weights". Also, in the config.py it requires: __C.WEIGHT_DIR = "/Disk2/hongwei/hat-weights/"#__C.ROOT_DIR + "/Weight/"

Thanks for playing with HAKE-Activity2Vec! We've prepared the pretrained model and will add the URL of model to our README file. We will inform you instantly about these changes if we update the file.

In activaty2vec, when I use main.py to do inference, it needs the pretrained model "pasta_weights". Also, in the config.py it requires: __C.WEIGHT_DIR = "/Disk2/hongwei/hat-weights/"#__C.ROOT_DIR + "/Weight/"

We've uploaded the pretrained model weights, and the URL is https://1drv.ms/u/s!ArUVoRxpBphYgtRugeMQVT-ncVq0OA?e=J6aBJn

Thanks a lot!


Implemented a method to properly format custom URLs

Motivation
To ensure user-defined URLs remain consistent, this PR introduces a method to ensure that is possible.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan

Run tests
Add a call to the newly added function, with a missing end slash. Log the output of the call.
You should see a slash added.

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@yangshun This one is still in draft due to https://github.com/facebook/docusaurus/issues/2748

Hi,
wondering if you have an example where not formatting the urls is a problem?
Asking because I also have a need to sanitize urls here: https://github.com/facebook/docusaurus/pull/2771/files/492962e99841069b87966a3d6e04430a851a88a6#r428105564

@slorber This PR is a result of the discussion that took place over at https://github.com/facebook/docusaurus/pull/2694.
It was agreed that permalink structures would get a trailing slash. That didn't happen yet, hence why I made this PR.

Hi, considering how things have changed, this PR seems to be already implemented because URLs are not naively augmented.


improve access to annotations of Document

If annotation type is a variable then currently only way to access annotations by their name is by iterating over a loop like in clear_predictions method. For instance, in code below we add annotations to the doc by annotation name (field) but it is done one by one.

  for doc in docs:
      for field in overwrite_annotations_with_predictions:
          for base_type, collection in doc.predictions.typed_collections:
              if field in collection:
                  doc.add_annotation(field, collection[field])

Maybe we need a get method to fetch all the annotations by their name.
Similarly, currently add_annotation or add_prediction methods add annotation one by one. Would it make sense if we can also add a list of annotations directly ?

This is implemented.


Can't find the code for Progressive Webserver

Download the Code ( I can't Locate this Code ) Thank you!
Click the following button to download all the code for this codelab:
Unpack the downloaded zip file. This will unpack a root folder (your-first-pwapp-master), which contains one folder for each step of this codelab, along with all of the resources you will need.
The step-NN folders contain the desired end state of each step of this codelab. They are there for reference. We'll be doing all our coding work in a directory called work.

Download Code button is missing from
https://developers.google.com/web/fundamentals/getting-started/codelabs/your-first-pwapp/#download_the_code

lines 905 - 906:
`Click the following button to download all the code for this codelab:

`

Hi!
Thanks for the feedback. I've rewritten the codelab from scratch and believe I have fixed this issue. The code has been updated, and the new guide is now live.
If you find any issues with the new codelab, please let us know!
Thanks!


Add workaround for --net=host for mac and windows users

Based on https://docs.docker.com/network/host/, --net=host only works for linux hosts. The workaround for mac and windows users seems to be to use the hostname host.docker.internal which will resolve to the host IP. https://docs.docker.com/desktop/mac/networking/#use-cases-and-workarounds
Feedback from a support case is that we should add that note to the examples for new users.

closed by ^


How to use with Travis?

Is anybody already using this gem for deploying a GitHub project or user/organization page via TravisCI?
I'm trying to set one up (Parallel-in-Time/parallel-in-time.github.io) but failing hard with Travis' default behaviour of running bundle install --path=${BUNDLE_PATH:-vendor/bundle}. It somehow assumes one is using rake for all the CI tasks and not a binary provided by a gem.

@imathis / @torbjoernk considering this is an issue from last year, not really sure if you're still having trouble with it, but i've written a blog post (designed for 2.0, but the general bit that you'll need to work with remains the same.
blog: http://plttn.me/blog/2015/01/17/yet-another-octopress-2-dot-0-plus-travis-ci-guide/
Short version:
Use something like this for your Travis config: https://gist.github.com/plttn/655c1a3bce8ab3275a4b.
(I deploy in script rather than after success, as a failure in after_success won't flip the build over to failing)
Generate a SSH keypair, and add the public to github's settings, and then edit the private key as follows: replace all newlines with \n, and then escape that to \\n. In your Travis repo settings, make a new environment variable named GITHUB_PRIVATE_KEY, and paste the private key, which should now be one line, into the variable box.
I've got a proof of concept on travis-ci.org (at the moment it's suffering from an ssh-agent bug, which I had resolved on my primary site, but i can't seem to access my build history right now to determine what I did to fix it.) at https://travis-ci.org/plttn/octopress3.

Realized my bug.
Ensure that ssh-agent is not running before generating the key you intend to deploy with. For some reason, leaving it running and generating a key causes ssh-agent on Travis to ask for a passphrase.
Commit on Github: https://github.com/plttn/octopress3/commit/520ce15af5362e880ef6ca16c37ee8f4133926bc
Build on Travis:
https://travis-ci.org/plttn/octopress3/builds/47474897

Thanks so much for sharing this with me. The last time I looked at it, either it wasn't possible (with public projects) or I'd given it up because it didn't seem to work. I'll give it a try and if I'm successful, I'll update the documentation with a guide. Thanks!

No problem. I went through around 50 builds on my private repo trying to get this all worked out, so I figured I'd help everyone else out.


Make custom password retrieval script option separate from --vault-id

Currently the --vault-id option checks if the argument following contains the @ character, then if the file given after is marked executable. If it is executable, it runs the file and uses the stdout printout as the vault password. If it is not executable, it reads the file and uses the contents as the vault password.
This is too fragile and can lead to incorrectly encrypted secrets if the user forgets to add the executable bit to their script. The script can appear to work as intended, while the intended behaviour is actually silently failing and ansible-vault is using the script contents as the password. (Yes, I did this.)
I suggest separating the script functionality to an explicit separate option which unambiguosly always tries to run the script, and if it fails to do that, it reports the error and stops. Something like --vault-id-script.

Apologies, I just realised I posted this in the wrong repo. Closing.


Extract single frame from video

Is it possible to extract a single frame from a video using this trimmer? If not, is that an enhancement you would consider adding? Regardless, thanks for your excellent package!

I think what you want to get is a single image of a particular frame from the video. Then, for that purpose, there is already a plugin available: video_thumbnail.

Hi Souvik, thanks for your response! I don't actually need to create video thumbnails (though thank you for pointing me toward that package). I'm currently using the trimmer to allows users to extract multiple short GIFs from a single long video. I'm interested in allowing them to extract many images from that same video just like I'm allowing them to extract GIFs. Does that make sense?

Yes, I understood. I believe that can also be done using the video_thumbnail plugin. You can define the time (in ms) from where you want to take the image, and set the image quality to 100%, so it will generate a high-quality image. So, you will get images generated from your defined timestamp.
If there is already an implementation of this, I think it's not wise to do it again.
Actually, I am also using that plugin to show the preview of the video (in form of images) in the VideoEditor of my plugin.

What about the picker? Can we configure VideoEditor to be a picker for the single frame?

@qbait @jmmaxwell okay, I understood what you guys want to tell. The video_thumbnail plugin provides the functionality but there is no UI component like VideoEditor in that.
Currently, this plugin can't be configured to pick a single frame, because that will also require some changes in the UI of the TrimEditor.
Though I didn't have any such enhancement in the timeline. I will try to implement that once I get some time (probably by the end of December)

Hi @sbis04, I was just curious if you had a chance to work on this at all? Again, thanks for your excellent work on this!

Similar problem here... it would be great if was implemented a way to show a specific frame of the video in the videoViewer.


Retrieve Zoom level via ROS Topics or services.

Hello there, I am currently using MapViz in ROS Noetic. I noticed that Visualisation Markers don't scale based on zoom. In my particular use case, the user would have to zoom in and out  from few metres of screen area to several hundreds of Kilometres frequently. And the markers I publish are dynamic. So I was wondering if there is no built-in way to resize Visualisation Markers based on zoom, then if some how i could get the zoom level, I would re-publish all markers with updated scale. I could then set a fixed frequency for updating the marker or update them when zoom changes.
Are there any way to get the current zoom level?
I was thinking if its not possible them I'll have to monitor the Tile Map URL to extract the zoom level but that would be too much for me.
Thankyou in advance.

Right now there's not a way to resize that I know of, but I think that makes a lot of sense. I think I'd rather have Mapviz scale the markers itself, based on a user setting, than publish the zoom level and have the markers themselves change. If we had the markers re-publish, it would change it for all listeners of the message, which doesn't seem great.

So are there any work arounds, atleast for my particular case?

Okay thanks.


[Bug]: Profile freezes after entering PIN on macOS

Issue description
Cannot get into wallet on macOS. When I enter my passcode all I see is the loading screen for the firefly application.
Expected behaviour
Once I enter my passcode into the firefly app I expect to see my wallet and the balance of all my coins and tokens I have in there.
Actual behaviour
The firefly application is just showing the loading screen with the logo.
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
Open the app and then enter the passcode for your wallet and then this problem happens.
Diagnostics
Firefly v1.3.3
Platform: macOS
Platform Version: 12.2
Platform Architecture: x64
CPU Count: 8
Total Memory: 8192.0 MB
Free Memory: 26.5 MB

Errors
No response
Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

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

No I am not using a Ledger. I am using the firefly app on the macOS and for some reason I can see what in my wallet.

Can you try adding a new profile with "Add Profile" --> "Migrate or restore wallet" --> "I have a Firefly recovery phrase".
This has solved the issue for some users. Your funds are still there, the Firefly wallet acts as just a GUI to the tangle.
If that works, then you can delete the profile causing the freeze by navigating to ~/Library/Application\ Support/Firefly/__storage__/<profile> and deleting the profile.

@ianboyd22 out of curiosity, would you be able to try these steps before adding a profile or resetting the wallet? This has sometimes fixed keychain-related issues on macOS

@rajivshah3 I am not able to lock my keychain on my Mac.

Ah yikes, I should have double-checked that. It seems like Apple removed the ability to lock and unlock the default keychain after that message was written. I think you'll need to try what @kraftjs recommended in that case

I have tried what @kraftjs recommended and it seems to have worked. Thank you both for your help on this issue.

Can you try adding a new profile with "Add Profile" --> "Migrate or restore wallet" --> "I have a Firefly recovery phrase". This has solved the issue for some users. Your funds are still there, the Firefly wallet acts as just a GUI to the tangle.
If that works, then you can delete the profile suffering from the freeze by navigating to ~/Library/Application\ Support/Firefly/__storage__/<profile> on your system and deleting the profile.

If I migrate to a new wallet profile in Firefly will I lose the staked amounts I have collected so far?

Nope, your funds will still be staked


Fatal Error

Hi,
I got the following error while running the pipeline for the test dataset. Please advise on how to overcome this issue. Thanks a lot.
Samar
Error executing process > 'batch_statistics (Example/1000G_controls)'

  Caused by:
    Failed to pull singularity image
    command: singularity pull  --name jkaessens-ikmb-gwas-qc-assoc-assets.sif library://jkaessens/ikmb-gwas/qc-assoc-assets.sif > /dev/null
    status : 255
    message:
      [33mWARNING: No default remote in use, falling back to: https://library.sylabs.io
      FATAL:   While pulling library image: image jkaessens/ikmb-gwas/qc-assoc-assets.sif:latest (amd64) does not exist in the library
```

Hi,
thanks for reporting. We had an issue with the library provider, that should be fixed now. Please try again.

Hi,
We are still encountering this issue. Is there a work-around to download the container?
singularity pull  --name jkaessens-ikmb-gwas-qc-assoc-assets.sif library://jkaessens/ikmb-gwas/qc-assoc-assets.sif
FATAL:   While pulling library image: image jkaessens/ikmb-gwas/qc-assoc-assets.sif:latest (amd64) does not exist in the library

Hi cemmeydan,
sorry, that it took us so long to find a way to solve the issue. We encountered the same problem again with oour library hoster, but as a workaround you can download the container now from one of our servers.
Please follow the instructions in the updated Readme. (Note that the name of the container has changed slightly.)
Thank you and sorry for the inconvenience.
Lars

Thank you, it is working with the new image.


Are pre hooks and schema options inherited?

Hey, thanks for the new plugin!
I derive a schema from a "base schema" that has pre hooks on it and "timestamps".
Are they inherited?
See here: https://github.com/Automattic/mongoose/issues/5897

You're welcome!
Hooks are not inherited. You extend the copy of original object passed to the schema constructor. Use Schema plugins for hooks: http://mongoosejs.com/docs/plugins.html

Ah, got it. Just had a look at the source and it's very basic ;-) Could've seen this myself....


Only run a PR if the sender is part of the org or repo owner

If PR sender is a public member (because we can't otherwise with github
app to have that info) on the organization where the pr has been sent.
If user that submitted the PR is owner of the repo then allow it too.
There is going to be a bit more logic here for OWNERS but let not go too
ahead of ourselves.
partial addressing #39
Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>

Codecov Report

Merging #43 (5402b4d) into main (9b6809f) will decrease coverage by 1.36%.
The diff coverage is 59.57%.

@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
- Coverage   66.05%   64.68%   -1.37%
==========================================
  Files           7        8       +1
  Lines         489      521      +32
==========================================
+ Hits          323      337      +14
- Misses        129      145      +16
- Partials       37       39       +2

Impacted Files
Coverage Δ

pkg/cmd/pipelineascode/pipelineascode.go
15.62% <0.00%> (-0.25%)
:arrow_down:

pkg/pipelineascode/pipelineascode.go
59.50% <11.11%> (-3.89%)
:arrow_down:

pkg/webvcs/github.go
78.43% <65.51%> (-4.31%)
:arrow_down:

pkg/pipelineascode/acl.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Update agent sign out redirect

in the "SignOutControllerISpec"

on row 36: can you please change the text to redirect to gg sign out instead of sign in.
can you also add another scenario for the "unauthenticated user" just to make this test more completed. The unauthenticated user should be redirected to sign-in page.  you can refer to my PR, Alex has helped me with mine. thanks


[ALLUXIO-2339]Improve error handling in NetworkAddressUtils#getLocalIpAddress

https://alluxio.atlassian.net/browse/ALLUXIO-2339
remove the redundant log statement

Can one of the admins verify this patch?

ok to test

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12005/
Test PASSed.

Merged build finished. Test PASSed.


Reworded first bullet in "Requirements for cloud management gateway" …

…section

Looks good, thanks @cliffhobbs


fix: Handle new file creation in the same folder

This is a fix for https://github.com/paul-gauthier/aider/issues/1222. Currently, if you have an existing file and Aider tries to create a file with similar name, the new file doesn't get created since it's fuzzy-matched to an existing file here
This PR checks if edit block tries to create a new file and if so it passes None as valid_fnames

Nice improvement, thanks for this!

Thank you! Was unable to work with aider because of that, great job!


The Typescript Typings in DefinitelyTyped are missing Graphics.StrokeDash

We are trying to get the ZIM Typings onto DefinitelyTyped at https://github.com/DefinitelyTyped/DefinitelyTyped.  Not sure who is maintaining the CreateJS typings there...  The Graphics.StrokeDash seems to be missing.  The Graphics.Stroke and others are working.  We will see if we can solve the issue there - but... if anyone here has been working on the Typings, please let us know.  Cheers.

I'd be happy to help fix this.

@danzen ZIM's StageGL may also have an invalid bug with {antialias: true} when displaying a scale= 1 appears. Do you have any good solutions

@danzen Bug rotation can also occur

@ajex-china - just saw this - sorry for the delay.  And saw your other issue.  Will ask about it there.


Orientation bug iOS

Like your menu very much. Looks good and easy to install. Only when using it with iOS (iPad and iPhone) once you have used the menu and change orientation it stays the same and does not scale. It is also with your demo. Can this be fixed?

anyone?

this line seems to cause the problem:
https://github.com/Glitchbone/jquery-fatNav/blob/master/src/js/jquery.fatNav.js#L62
I guess it is used to avoid a little movement of the body content when opening the menu, however it also cause to set a fixed body width, so subsequently it won't scale on window resizing or orientation change. I guess we should find an alternative way to avoid the body offset movement.

whell the body movement is caused by the scrollbar... so a very quick and dirty fix would be to substitute hidden on https://github.com/Glitchbone/jquery-fatNav/blob/master/src/js/jquery.fatNav.js#L63
with auto. this way however you will have a visible scrollbar when the menu is open

Sorry guys for being so late, I was really busy.
I've just removed the hack to prevent the body movement for now, until I find a good fix


Removes apprenticeship link

I think we are bringing the apprenticeship back /cc @rmdmachado

@gabrielpoca maybe, but that hasn't been decided yet. In the meantime, laura asked me to remove this, so that it is more consistent with the main website

Not maybe, I was going to remove the link and he said that he wanted to add the link back to the main site. I think we can wait a couple of hours for him to come online.

Let's remove for now, mainly because the applications link is not working. I
canceled formkeep, was costing us $29/month.
Cheers,
[Roberto Machado](https://link.nylas.com/link/98s06a8os7wlfjgynv76h3jr4/local-
de449f68-6ffb/0?redirect=https%3A%2F%2Ftwitter.com%2Frmdgb&r=cmVwbHkrMDAwM2M4Z
WI0NGU3ZTM3NDI2MTczNDZkMTU4NmExZWE5MjgwMDUyODQwMzIwZGM4OTJjZjAwMDAwMDAxMTRhMmY
yNzA5MmExNjljZTBjMjI5YTJjQHJlcGx5LmdpdGh1Yi5jb20=)
_CEO at _[Subvisual](https://link.nylas.com/link/98s06a8os7wlfjgynv76h3jr4
/local-de449f68-6ffb/1?redirect=https%3A%2F%2Fsubvisual.co%2F&r=cmVwbHkrMDAwM2
M4ZWI0NGU3ZTM3NDI2MTczNDZkMTU4NmExZWE5MjgwMDUyODQwMzIwZGM4OTJjZjAwMDAwMDAxMTRh
MmYyNzA5MmExNjljZTBjMjI5YTJjQHJlcGx5LmdpdGh1Yi5jb20=)
_+351 916748994 | skype:
_[roberto@subvisual.co](https://link.nylas.com/link/98s06a8os7wlfjgynv76h3jr4
/local-de449f68-6ffb/2?redirect=mailto%3Aroberto%40subvisual.co&r=cmVwbHkrMDAw
M2M4ZWI0NGU3ZTM3NDI2MTczNDZkMTU4NmExZWE5MjgwMDUyODQwMzIwZGM4OTJjZjAwMDAwMDAxMT
RhMmYyNzA5MmExNjljZTBjMjI5YTJjQHJlcGx5LmdpdGh1Yi5jb20=)
MeetMe: [http://doodle.com/rmdmachado](https://link.nylas.com/link/98s06a8os7w
lfjgynv76h3jr4/local-de449f68-6ffb/3?redirect=http%3A%2F%2Fdoodle.com%2Frmdmac
hado&r=cmVwbHkrMDAwM2M4ZWI0NGU3ZTM3NDI2MTczNDZkMTU4NmExZWE5MjgwMDUyODQwMzIwZGM
4OTJjZjAwMDAwMDAxMTRhMmYyNzA5MmExNjljZTBjMjI5YTJjQHJlcGx5LmdpdGh1Yi5jb20=)
![](https://link.nylas.com/open/98s06a8os7wlfjgynv76h3jr4/local-de449f68-6ffb?
r=cmVwbHkrMDAwM2M4ZWI0NGU3ZTM3NDI2MTczNDZkMTU4NmExZWE5MjgwMDUyODQwMzIwZGM4OTJj
ZjAwMDAwMDAxMTRhMmYyNzA5MmExNjljZTBjMjI5YTJjQHJlcGx5LmdpdGh1Yi5jb20=)
On Jan 27 2017, at 11:35 am, Gabriel Poça<EMAIL_ADDRESS>wrote:

Not maybe, I was going to remove the link and he said that he wanted to add
the link back to the main site. I think we can wait a couple of hours for him
to come online.

—
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](https://github.com/subvisual
/blog.subvisual.co/pull/147#issuecomment-275646516&r=cmVwbHkrMDAwM2M4ZWI0NGU3Z
TM3NDI2MTczNDZkMTU4NmExZWE5MjgwMDUyODQwMzIwZGM4OTJjZjAwMDAwMDAxMTRhMmYyNzA5MmE
xNjljZTBjMjI5YTJjQHJlcGx5LmdpdGh1Yi5jb20=), or [mute the
thread](https://github.com/notifications/unsubscribe-auth/AAPI6
-rPbvQJPA8Bys8jryl-QhdrdlFaks5rWdZwgaJpZM4LvloE&r=cmVwbHkrMDAwM2M4ZWI0NGU3ZTM3
NDI2MTczNDZkMTU4NmExZWE5MjgwMDUyODQwMzIwZGM4OTJjZjAwMDAwMDAxMTRhMmYyNzA5MmExNj
ljZTBjMjI5YTJjQHJlcGx5LmdpdGh1Yi5jb20=).![](https://github.com/notifications/b
eacon/AAPI66XNjRzkiLyxdupYj6D80z05yPDFks5rWdZwgaJpZM4LvloE.gif)

@gabrielpoca bump, then


Issue Pairing to HomeKit

I am unable to get Example 01-SimpleLightBulb to pair to HomeKit.  I added the optional LED and SWITCH to observe the behavior of the ESP32.  I selected my WiFi network and added the password in the Serial Monitor.  The LED is blinking twice showing it is connected to my WiFi. My iPhone is connected to the same WiFi network.
In the Home App under My Home I am clicking on Add Accessory, More Options, My Accessory Isn't Shown Here, Enter a Setup Code, Enter Code...   I am using 46637726.  After entering the code, the Home App just defaults back to the Select an Accessory to Add to My Home.
I am using the following software versions.
Arduino IDE 1.8.19
Library: HomeSpan by Greg Version 1.6.0
Sketch: Example 01-SimpleLightBulb
esp32 version 2.0.4 board manager
My iPhone 13 Pro is running iOS Version 16.0.2

![IMG_4810](https://user-images.githubusercontent.com/11469813/192805796-5914127a-ad0d-46a0-abf2-1289015675fe.PNG)
![IMG_4812](https://user-images.githubusercontent.com/11469813/192807056-64e3b279-b397-4ed5-92ef-05f11f06923a.PNG)
![IMG_4817](https://user-images.githubusercontent.com/11469813/192809475-c4d5ed40-49df-49cc-93e5-5f382f444a77.jpeg)

sercontent.com/11469813/192810267-e4aedfe7-6514-4b90-a143-0566c1c2ad6d.png">

The HomeSpan device should show up in the list - you should not have to press "My Accessory Isn't Shown Here."  If it does not show up there are a few things to try.

Open a terminal window and try pinging your device (ping <IP_ADDRESS>).  If no reply, that suggests a WiFi connectivity problem.  If there is a reply, that ensures WiFi is properly connected.

Download a bonjour discovery app on your iPhone.  I use "Discovery DNS".  It's free from the App Store.  Once downloaded, open the app and click on the _hap._tcp. local service.  This should show all the devices broadcasting HomeKit compatibility.  If you don't see your Accessory that suggests a problem with the MDNS service for your chip.  If you do see your Accessory, click on it and verify that sf=1.  The Home App looks for this value to decide which Accessories to show when pairing.

If everything above looks correct, it may be an issue with your router.  Does your router support both 5GHz and 2.4GHz networks?  If so, the iPhone will usually connect via the 5GHz.  However, the ESP32 chips only have a 2.4GHz radio, so the Home App will NOT be able to find the Accessory UNLESS your router is configured to cross-route all traffic between the 5GHz and 2.4GHz bands.  This is usually something that can be set in the router config.


pub accepts array of arguments to pass to subscriber as parameters

https://github.com/rafde/PrioritizedSubPub/commit/08aa3019f462f000fc774f0f648c956133b197a7