Incorporate changes from fork

There is a fork of this module that has interesting additions.
https://github.com/robwi/xHyper-V
HELP WANTED
If anyone would like to review the changes, determine which would benefit the community, and drive them as a new PR, that would be a great project.  It is lacking tests and documentation, so this is not a small undertaking.

I don't mind looking into this. However, It would be my first contribution and any guidance and feedback will be very much appreciated.

Hi @Haelters - it would be awesome to have the help if you want to take this on! All help is gratefully accepted and appreciated :smile:
I'm happy to help you with the technical stuff of getting the resources in line to merge into the upstream repo.
There are a few logistical questions that need to be answered though:

Which resources do we want to merge from @robwi 's repo?
Which features can we copy across?
How do we go about taking these "chosen" resources, modifying them to meet guidelines and adding unit/integration tests and then submitting the PR to the @PowerShellTeam repo?

Looking at the two repos:

PowerShell/xVHD has similar function to robwi/xVHD
PowerShell/xVMHyperV has similar function robwi/xVM (there are some additional boolean flags here).
robwi/xVMProcessorMaximum - just contains a single parameter - couldn't this just be added to xVMHyperV?
robwi/xVMAddDisk - this is the most useful one that I think could be migrated. @rchaganti - do you have anything in your xHyper-V that you're planning on moving across that would conflict with this one?

Given the above I'd suggest the answers to the logistical questions might be:

xVMAddDisk.
some features of xVM
I'm not sure, but possibly:

@Haelters forks Powershell/xHyper-V
@Haelters Manually copy files from robwi/xHyper-V that we want to include into fork.
@Haelters makes changes to fork to meet guidelines, adds unit & integration tests.
@Haelters submits PR to PowerShell/xHyper-V

Sorry about the long response, but this sort of "merge" isn't something I've done before. So I'm sure others may have much better ideas than I!

I have a bunch of new resources in my cHyper-V module. I will be merging changes to xVMSwitch to add SET functionality very soon.
I will add other new resources as well.

xVMNetworkAdapterVLAN
xVMNetworkAdapterSettings
xVMNetworkAdapterMapping
xVMIPAddress
xWaitForVMGuestIntegration
xSimpleVM (very simple VM with no strings! :-))
xVMFile (over PowerShell Direct)
xVMCommand (over PowerShell Direct)

I will have these with HQRM guidance complete.

I'll start with the xVMAddDisk, as this might be an easy one to get started and learn about all the do/don'ts.
I've already forked the Powershell/xHyper-V and will copy/generate the files required for xVMAddDisk and adapt to guidelines (I will have to study them) and generate the unit and integration tests. I hope that at that point, some of you will be able to review what I've done and then we can launch the PR to PowerShell/xHyper-V.

@rchaganti I've also been thinking about the xVMFile, but in the context of xVhdFile.
The scenario is that I use DSC to ensure certain Vhdx exist on a host and make sure certain files are included in the vhdx. This works perfectly the first time as the VHDX is not yet in use. However, subsequent launches generate an error on the xVhdFile because the VHDX is in use by the VM.
What I intended to do was to modify xVhdFile to handle two cases transparently for the user:

The Vhdx is not in use: simply take the approach that is currently implemented: i.e. mount the vhd and copy.
The Vhdx is currently in use (in a running VM): there the approach would be to simply copy it over using PowerShell Direct.

A parameter could be used to determine if a file may be injected in a disk on a running VM (this would trigger the requirement for credentials).
Is this a good approach to avoid having two resources that in essence do the same? (if I should ask this somewhere else, please let me know - still learning)

Could someone take a look at the extra resources I've created:
https://github.com/PowerShell/xHyper-V/compare/dev...Haelters:Issue-66
As you can see, I've created two extra resources:

xVMHardDiskDrive: allowing to add/remove/change drives to a VM
xVMScsiController: allowing to add/remove SCSI controllers (See #64)

At first I started with quite some complexity in it (e.g. when adding a hard disk drive on a non existent controller, it was automatically added). However, I think that this complexity should be handled in the DSC configuration and not in the resource. I've also set it up in a way that adding additional features should be very easy (e.g. MaximumIOPS).
If you agree with the proposal, I'll continue with the other options and resources (e.g. xVMProcessor). It should go a lot faster now :-))
The only issue I'm currently having some issues with, are the tests. I find it quite useless as we may not use the actual Hyper-V cmdlets. I've started mocking them, but creating mocked functions that fully mimic the Get-VMHardDiskDrive is quite a heavy job. I'll continue looking into it.
@kwirkykat: could you place this one 'in progress' please?

@Haelters I am keen to get the xVMScsiController and xVMHardDiskDrive resources merged into this repository. Is there anything that you need to progress this PR? Presumably you are using the fork in your repository for internal purposes?

@Haelters, I'm also keen to see these new resources merged into the repo. Thank you for doing the work... I'll the new resources as soon as they're in place.

@Haelters If you're having problems with writing tests and mocking Hyper-V cmdlets then have a look at Stub Commands as a way of handling that.

@ChrisLGardner Ha - I was looking at that Pester internals to generate that functionality for one of my upcoming presentations! I shall now add a link to your repo instead 😃.
@Haelters @ScottArbeit I've started working on the change in this fork and will add unit test etc. assuming @Haelters is OK with this?

@Haelters Are you OK for me to take over your work in this fork? If you would rather continue to develop it, then that's OK by me too 😃!

Go ahead. Currently I don't have the time.
Best regards
On Jun 27, 2017 10:37 PM, "Iain Brighton"<EMAIL_ADDRESS>wrote:

@Haelters https://github.com/haelters Are you OK for me to take over
your work in this fork
https://github.com/iainbrighton/xHyper-V/tree/Issue66? If you would
rather continue to develop it, then that's OK by me too 😃!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/xHyper-V/issues/66#issuecomment-311477781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWaxdmVd73lZtisNHDOdhpWyOf_2db_Mks5sIWfqgaJpZM4KlBeJ
.


background elevation scene setting

What's the difference between the foundry setting and the terrain mapper setting? If both should hold the same value, what about extracting the foundry one for terrain mapper calculations?

How are you getting the Background Elevation setting in base Foundry? I am running Foundry v12.331 and dnd5e 3.3.1 and I don't see any such thing. I have never seen such a scene config in base Foundry.
I agree that if such a setting exists in base Foundry, it would be worth using instead of the one TM adds.

oof, you're absolutely right! That comes from the Levels Module of theripper93 x)
Might be an idea to special case if Levels is installed since Levels brings a lot to take into account for this module.
Levels Wiki regarding regions: https://wiki.theripper93.com/levels#regions


Navigating clients from the SW context?

In discussions with the Google team, a question has come up regarding the ability to navigate client objects. This would be the moral equivalent of window.location = ... and/or window.location.reload().
Would love to find a way to enable this.

Not against this, but would like to know why postmessage isn't a good solution here

because it means you need a client-side contract and can't "just do the
same thing" with regards to a server-sent document/redirect. This solution
makes it easier to keep documents ignorant of SWs and reduces duplicate
code paths.
On 16 Apr 2015 3:07 pm, "Jake Archibald"<EMAIL_ADDRESS>wrote:

Not against this, but would like to know why postmessage isn't a good
solution here
—
Reply to this email directly or view it on GitHub
https://github.com/slightlyoff/ServiceWorker/issues/681#issuecomment-93850512
.

Does not being able to do this with (shared)workers pose a problem?

We could have windowClient.navigate(url), which returns a promise. url could be optional and default to the current url (eg reload). Promise would reject if client no longer exists.
@slightlyoff can you give us a better idea of the use-case? When would they be calling this?

The use-case was a situation where you might otherwise want to do a
redirect, but from the perspective of the SW. In the case of document
creation, there's a step between starting the process of creation (which
generates a new name) and navigating to the actual named location;
/create -> /generatedname
On Wed, Apr 22, 2015 at 8:22 AM, Jake Archibald<EMAIL_ADDRESS>wrote:

We could have windowClient.navigate(url), which returns a promise. url
could be optional and default to the current url (eg reload). Promise would
reject if client no longer exists.
@slightlyoff https://github.com/slightlyoff can you give us a better
idea of the use-case? When would they be calling this?
—
Reply to this email directly or view it on GitHub
https://github.com/slightlyoff/ServiceWorker/issues/681#issuecomment-95234389
.

Another use-case for this: Say a site ships a broken ServiceWorker. It fixes it. The new version should be able to skipWaiting, clients.claim() and reload all the clients.
I'll spec windowClient.navigate(url), where url defaults to the current url, so windowClient.navigate() reloads.
url is relative to the SW.
If url only differs in fragment, or does not differ at all, the page will still reload. Therefore calling .navigate will always cause a fetch of the url.

I've taken a first stab at this, @jungkees, can you review?
I'm checking that the document hasn't been discarded (1.6.2), should focus() do the same?
I'm rejecting the promise if the client isn't controlled by this SW - else you can navigate parts of the origin "owned" by someone else.
I'm uncomfortable with the "source browsing context" bit, as it's a lie, but I think this is the first API that lets a non-browsing context navigate a browsing context.
I decided against:

I'll spec windowClient.navigate(url), where url defaults to the current url, so windowClient.navigate() reloads.

…as that clashes with other APIs where the default url is "" (eg getRegistration).

@jakearchibald Looks good. I just tweaked step 4 (links to concepts) and step 6.3.4 (variable). And added step 6.3.2 for https://github.com/slightlyoff/ServiceWorker/issues/694#issuecomment-103627523.

I'm uncomfortable with the "source browsing context" bit, as it's a lie, but I think this is the first API that lets a non-browsing context navigate a browsing context.

This seems fine to me since the browsingContext there is actually a browsing context gotten from a service worker client's global object.

I'm checking that the document hasn't been discarded (1.6.2), should focus() do the same?

FWIW, focus() still should work even if the browsing context discards its document?

FWIW, focus() still should work even if the browsing context discards its document?

Does that mean this can happen:

SW js looks at clients, finds one with url /chat/, which is correct
User focuses that client & navigates it to another URL, perhaps even another origin
SW js calls .focus on the client, which works

Does that mean I just focused a tab that had been navigated away from /chat/, maybe even to another origin?

Tracked at http://crbug.com/500911 for Blink

From the step 4 on navigate():

If the context object's associated service worker client's active worker is not the incumbent settings object's global object's service worker, return a promise rejected with a TypeError.

According to jake's comment, a client to be navigated should be controlled by the incumbent service worker, so "service worker client's active worker" should be "service worker client's controller".

@nhiroki I don't think "controller" is a defined thing, but active worker is defined as:

A service worker client has an associated active worker (an active worker) which currently controls it. It is initially set to null.

…which means the same.

@jakearchibald Thank you for the reply. That makes sense. I was thinking the case where a service worker is active but not a controller yet (eg. first load).

We implemented this!


Cosmetics suggestions to testing setups

I'm submitting an improvement request

Aurelia Skeleton Version
skeleton-esnext-webpack
skeleton-typescript-webpack

Framework Version:
1.1.5

Operating System:
Windows 10

Node Version:
8.6.0

NPM Version:
5.4.2

JSPM OR Webpack AND Version
webpack 3.6.0

Browser:
all

Language:
TypeScript and ESNext

Desired behavior:
Improvement (1):
Simplify following part of karma-bundle.js
// load and run tests:
const testModuleContexts = loadTestModules();
runTests(testModuleContexts);

function loadTestModules() {
  const srcContext = require.context(
    // directory:
    '../src',
    // recursive:
    true,
    // tests in /src folder regex:
    /\.spec\.[tj]s$/igm
  );

  const testContext = require.context(
    // directory:
    './unit',
    // recursive:
    true,
    // tests in ./unit folder regex:
    /\.spec\.[tj]s$/igm
  );

  return [srcContext, testContext];
}

function runTests(contexts) {
  contexts.forEach(requireAllInContext);
}

function requireAllInContext(requireContext) {
  return requireContext.keys().map(requireContext);
}

by replacing it with the following (less cryptic and more readable):
function requireAllInContext(requireContext) {
    return requireContext.keys().map(requireContext);
}

requireAllInContext(require.context('../src', true, /\.spec\.[tj]s$/igm));
requireAllInContext(require.context('./unit', true, /\.spec\.[tj]s$/igm));

Improvement (2):
Rename karma-bundle.js to karma-pretest.js.
This will make the purpose of the file clearer. Also, this will make it more aligned with the existing file jest-pretest.js naming convention.
Improvement (3):
Change the below line in protractor.conf.js from:
    onPrepare: function() {
        process.env.BABEL_TARGET = 'node';
        process.env.IN_PROTRACTOR = 'true';
        require('babel-register');
    },

to:
    onPrepare: 'protractor-pretest.js',

and create a new file under test folder:
// test/protractor-pretest.js
require('babel-register');
process.env.BABEL_TARGET = 'node';
process.env.IN_PROTRACTOR = 'true';

The user can add more stuff in this file.
With these changes, we will end up with 3 consistent testing setups:
jest.conf.js karma.conf.js protractor.conf.js
jest-pretest.js karam-pretest.js protractor-pretest.js
The reason I am not submitting a pull request is because I work on ESNext only (I don't use Typescript).

@nashwaan
Sad that there was no PR (-:
think you could close this one?


non-nullable arguments are being inferred as nullable

Problem
non-nullable arguments are being inferred as nullable
Given the following class
public class Query
    {
        public Task<User> GetUser(IResolverContext ctx, int id)
        {
            IUserRepository repo = ctx.Service<IUserRepository>();
            return repo.GetAsync(id);
        }
    }

results in the following
user(
    id: Int = null
): User

Expected behavior
user(
    id: Int!
): User

Additional context
The only way for me to specify that id should be non nullable is to override the field
descriptor.Field(f => f.GetUser(default, default)).Argument("id",d => d.Type<NonNullType<IntType>>());

I will look into that. We have recently added a lot functionality in that area so it might be a regression bug. I will write a test for that and check it out. A fix should be ready Sunday night. Thanks for reporting this issue.

This one is fixed now with 0.6.7.


chore: Wordle word redefinition

adding hyperlink to the word "Worlde" to redirect to the website Wordle. That way users can play the game Worlde from the website yesterdle.

needs work


Link to pre-trained models dead

Hi @ronghanghu. I'm trying to download the pre-trained models for clevr and vqa but the link provided (https://people.eecs.berkeley.edu/~ronghang/projects/snmn/pretrained_models/exp_vqa/tfmodel/) is dead. Would you still happen to have them available?

Hi @zachary-cauchi, sorry that the links have expired. I've uploaded the models here: https://drive.google.com/drive/folders/15EYBymzexMQ12GEq5cG6nabw7-CunaZ0?usp=sharing

Hi @ronghanghu. Thanks a lot for the new link! Closing the issue.


Add endpoint for purl lookup

fixes : #1357

@TG1999 I tried to test this new PURL lookup endpoint but cannot find the branch lookup_endpoint anywhere in the nexB repo.  How do I access that branch?  Or is that branch not in the nexB vulnerablecode repo (where I expect it should be)?  It almost looks like the branch is in a  fork on your own repo, but I don't know why that would be the case -- all of our work should be in the nexB vulnerablecode repo, right?

@TG1999   BTW, I've been doing some initial testing of our API bulk PURL search endpoint and other search endpoints/methods -- using pkg:nginx/nginx@0.8.9?os=windows which we know has two duplicate records with the same PURL -- and have found that:

my local .py single-PURL API search returns 3 PURLs: 2 dupes of pkg:nginx/nginx@0.8.9?os=windows plus<EMAIL_ADDRESS>

my local .py bulk-PURL API search returns 2 PURLs: 2 dupes of pkg:nginx/nginx@0.8.9?os=windows -- just what we want (except for the duplication) i.e., only the PURL I searched for -- works for a single-PURL search so maybe we don't need a new lookup endpoint?

public Package UI single-PURL search returns 3 PURLs: 2 dupes of pkg:nginx/nginx@0.8.9?os=windows plus<EMAIL_ADDRESS>

public API UI single-PURL search (GET /api/packages/) returns 3 PURLs: 2 dupes of pkg:nginx/nginx@0.8.9?os=windows plus<EMAIL_ADDRESS>

public API UI bulk-PURL search (POST /api/packages/bulk_search/): throws an error

Unable to get this to work.
The required input template differs from the structure that works in my local .py file.
{
  "purl": "string",
  "type": "string",
  "namespace": "string",
  "name": "string",
  "version": "string",
  "qualifiers": {
    "additionalProp1": "string",
    "additionalProp2": "string",
    "additionalProp3": "string"
  },
  "subpath": "string"
}

I tried this without success:
{
  "purl": "pkg:nginx/nginx@0.8.9?os=windows",
  "type": "nginx",
  "namespace": "",
  "name": "nginx",
  "version": "0.8.9",
  "qualifiers": {
    "os": "windows"
  },
  "subpath": ""
}

Response:
{
  "Error": "A non-empty 'purls' list of PURLs is required."
}

I also tried the following structure without success (but this DOES work in my local .py file AND is recommended in our RTD (https://vulnerablecode.readthedocs.io/en/latest/api.html#package-bulk-search)):
{
    "purls": ["pkg:nginx/nginx@0.8.9?os=windows"],
    "purl_only": False,
    "plain_purl": False,
}

Response:
{
  "detail": "JSON parse error - Expecting value: line 3 column 18 (char 70)"
}

Does the public API UI bulk-PURL search (POST /api/packages/bulk_search/) actually work and I just can't follow directions?  ;-)

@johnmhoran see https://github.com/nexB/dejacode/blob/2bdb96d1fefca88e10db4c4e0c8efdcecb05864f/dejacode_toolkit/vulnerablecode.py#L69 how bulk_search API is used here

@TG1999 I don't think you read my comments very closely.  I already have the bulk search API working in my Python code -- but I am unable to get the Public API UI bulk search working (here: https://public.vulnerablecode.io/api/docs/), and I explained what the experience was with that.  I also do not know where your lookup branch is.  All of that is detailed in my comments above, including my successful .py use of this structure:
{
    "purls": ["pkg:nginx/nginx@0.8.9?os=windows"],
    "purl_only": False,
    "plain_purl": False,
}

Added bulk_lookup endpoint with tests, please review

@TG1999 I've added your repo as a new remote and can now access your lookup_endpoint branch.  So far I've focused only on the new lookup endpoint.
Based on the tests, I expected this structure to work with my local instance:
headers = {"Authorization": f"Token {token}"}

request_body = {"purl": "pkg:nginx/nginx@0.8.9?os=windows"}

response = requests.post(
    "http://<IP_ADDRESS>:8001/api/packages/lookup",
    data=json.dumps(request_body),
    content_type="application/json",
    headers=headers,
).json()

However, this threw an error:
TypeError: Session.request() got an unexpected keyword argument 'content_type'

Not sure why that works in your tests but not for me.  Googling was unhelpful, and the variations I tried in the key 'content_type' did not resolve the error.  I finally deleted that line from the requests.post() content and instead modified my headers like this:
headers = {"Authorization": f"Token {token}", "Content-Type": "application/json"}

and the following returned the expected list containing two duplicate pkg:nginx/nginx@0.8.9?os=windows PURLs:
request_body = {"purl": "pkg:nginx/nginx@0.8.9?os=windows"}

response = requests.post(
    "http://<IP_ADDRESS>:8001/api/packages/lookup",
    data=json.dumps(request_body),
    headers=headers,
).json()

I'll continue my exploration of the lookup endpoint and then turn to the bulk_lookup endpoint.

@TG1999 Very nice!  👍


adding spot percentage to controll on-demand count

what

adding the optional spot_percentage to the spoitinst controller

why

By adding the spot_precentage you can control how many on-demand instances will be created instead of all spoint instances

references

This is the documentation about the spot_percetnage option

closing this and resubmitting from the org account so we can use it until this is merged.


Fix remaining API types

This cleans up the last changes for API types to generate and produce building code.

Alright, I think this is us with an openshift/api


Question on Schematron operation

I have a particular use case, and I would like to check if Xspec supports it.
This is that I want to run a Schematron test on a document. And then I want to look at the SVRL output, but I don't want to check the assertions. I want to check whether other things were generated, in particular  diagnostic and property elements.
I know that XSpec now provides special elements to check if assertions failed etc.  But can I use those at the same time as normal Xspec  tests on the data generated?

Yes, you can write expectations using <x:expect> to test the SVRL output from Schematron. For example:
<x:expect label="assert" test="exists(svrl:schematron-output/svrl:failed-assert)"/>
You can mix <x:expect> with Schematron-specific expect-* elements in the same scenario.
Within XSpec, the Schematron-specific expect-* elements actually test the SVRL output from Schematron XSLTs using tests like the example above.

Thanks!
On Tue, Feb 13, 2018 at 1:22 PM, vincentml<EMAIL_ADDRESS>wrote:

Yes, you can write expectations using <x:expect> to test the SVRL output
from Schematron. For example:
<x:expect label="assert" test="exists(svrl:schematron-
output/svrl:failed-assert)"/>
You can mix <x:expect> with Schematron-specific expect-* elements in the
same scenario.
Within XSpec, the Schematron-specific expect-* elements actually test the
SVRL output from Schematron XSLTs using tests like the example above.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xspec/xspec/issues/165#issuecomment-365130366, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AX3VKTtKYJgVToVt_Ar7INbsyB1b7EkSks5tUPH1gaJpZM4SA7pm
.


categorical supervised learning(strings only)

Model Builder currently supports two data sources, local files (.csv and .tsv) and SQL tables. We understand your data is in many different places and shapes. We'd like to understand where your data comes from so we can better support your scenario in Model Builder.
Where is your data located?
My dataset is located on my local c drive.
What does your data look like?
My data is text, format is CSV file as plain text data with (12 columns+ 1 label column) = 13 columns
Does your data require transformations?
No
Example
dataset-C.txt
Additional context
all numbers are categorical, I was able to produce a model in MATLAB, however, ML.net is not working with me.
The dataset is representing a organization chart, for each job title there is a grade and categorical job type(numbers).  I am prediction the grade of the supervisor position by providing all features(12 columns).

Hi @Gattan-Qbit
What kind of error do you get? No model produced within time budget or poor performace in accuracy?
In addition, you can set up a column as catagorical in advanced data option window

yes, it worked with Text classification scenario with advanced options with text format dataset, before that I was using recommendation scenario and it did not work.
Great thanks

Closing as this issue is resolved. Please @ mention us if there is anything else you need help with!


external_deps: some rewrites and package lists

more error detection
change directory if called from outside the directory
whitespace
base and all package lists
do not build curl with brotli and zstd

I added a commit making possible to do ./build.sh <platform> base and ./build.sh <platform> all, as testing the script without those were too much painful.
I actually rebuilt everything with -o pipefail without error. I haven't tried install, package, and wipe though.

I also edited curl build function to not build curl with brotli and zstd. If the host has brotli and zstd installed for the current architecture, one cannot cross-compile curl for another architecture, as headers would be found and then features will be enabled, but linking will fail.


Weird Unicode characters on Docker example on the Homepage

Do you want to request a feature or report a bug?
Bug
What did you do?
Open the Oficial site: https://traefik.io/
Find "Easy To Install", click on "Docker Image".
The example command there is filled with the Unicode char "Zero Width Space" (http://www.fileformat.info/info/unicode/char/200b/index.htm) and if you copy and paste that to a terminal the command wont work because of that weird char in the example.
Do an Inspect element to see.

What did you expect to see?
The command without the "Zero Width Space" after each "/"
What did you see instead?
A command that isnt possible to execute because of weird unicode chars in it.
Output of traefik version: (What version of Traefik are you using?)

Doesnt matter.
What is your environment & configuration (arguments, toml, provider, platform, ...)?
Doesnt matter.

If applicable, please paste the log output in DEBUG level (--logLevel=DEBUG switch)
Doesnt matter.

Screenshot

Ahh, and another problem there on that example. It is missing a "-v /var/run/docker.sock:/var/run/docker.sock"

Hi @falsovsky : Nice catch!
We were using the empty space so that the text would be nicely wrapped depending on screen sizes ... but we forgot the copy/paste issue in the process!
This is now fixed. Thanks for taking the time to point the issue.


How to change android applicationId dynamically?

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
Change applicationId dynamically using env files
Question Description
I need to export the same app with different applicationIds. I tried to set the gradle property, but has no effect, the app is always generated with same package (defined in gradle file). This is my gradle action:
gradle(
        task: "assemble",
        build_type: "Release",
        print_command: false,
        properties: {
            "applicationId" => ENV["APPLICATION_ID"],
            "android.injected.signing.store.file" => "#{ENV["ANDROID_KEYSTORE_KEYSTORE_NAME"]}",
            "android.injected.signing.store.password" => ENV["ANDROID_KEYSTORE_PASSWORD"],
            "android.injected.signing.key.alias" => ENV["ANDROID_KEYSTORE_ALIAS_NAME"],
            "android.injected.signing.key.password" => ENV["ANDROID_KEYSTORE_KEY_PASSWORD"],
        }
    )

I found a question similar to mine but the solution to change the applicationId (only in gradle file) isn't there.

@daraujofs Have you tried this plugin yet? 😊 https://github.com/MaximusMcCann/fastlane-plugin-android_change_app_name


Version 5.800 stopped recognizing user-installed CA

Describe the bug
When using TLS with a a server whose certificate is signed by a non-mainstream CA, despite the CA certificate is installed in Android's CA store, K9 throws a dialog saying that the signer is unknown, and offering to accept server's certificate individually.
To Reproduce

Have a CA.
Have an SMTP / IMAP server that has certificate signed by the above CA.
Install certificate of the CA into Android's "trusted CA" store.
Verify that Chrome and Conversations (XMPP client) connect to web and jabber servers that use certificates signed by the above CA without warnings.
Try to configure K9 / have previous version configured to talk to an email server that uses certificate signed by the above CA.
Observe error message saying "Trust anchor for certification path not found", offering to accept or reject "key".

Expected behavior
Like other TLS-capable Android applications, and like the previous version of K-9, TLS session with the server should be opened without complaint.
Screenshots

Environment (please complete the following information):

K-9 Mail version: 5.800
Android version: 10
Device: Sony H9436 (Xperia XZ-something)
Account type: IMAP

Additional context
Logs
Can provide upon request, I think it should easy to reproduce though.

We now probably need a custom network security configuration to support user-installed CAs. https://developer.android.com/training/articles/security-config

@crosser: Can you please check if the attached version restores support for user-installed CAs for you?
k9mail-5.801-SNAPSHOT-network-security-config.zip

Yes, it works now!
Thanks for such a quick reaction! :+1:

Thanks for the quick feedback. The fix will be included in 5.801.


Azure IoT Hub / DPS

Azure IoT Hub and DPS remain on Baltimore CyberTrust Root CA but their intermediate CAs will change. Explore other details provided in this Azure IoT blog post.

As far as I understand, this statement is currently out of date according to the blog-post.

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

ID: b3c4368c-82a4-f82e-b1a7-41cfa694fa58
Version Independent ID: 60a94729-02ad-0ef1-a642-63351d0171ea
Content: Azure TLS Certificate Changes
Content Source: articles/security/fundamentals/tls-certificate-changes.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@CIPop
Thanks for your feedback! We will investigate and update as appropriate.

@CIPop
I'm going to assign this to the document author so they can take a look at it accordingly
@msmbaldwin
Can you please check and add your comments on this doc update request as applicable.

@CIPop - Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know within two weeks. #please-close
UUF-Stale-Not Planned


Common CI fails

At the moment, we are experiencing some instabilities in our CI, which is annoying. Until now I always restarted builds but starting with this issue, I want to get an overview of the most common CI fails to help prioritize fixes. I link the failed builds in order to be able to gather additional information when needed.
Client With HttpClient Can paginate over all txs
HeadlessChrome 0.0.0 (Linux 0.0.0) Client With HttpClient Can paginate over all txs FAILED
	Expected 2 to equal 3.
	    at <Jasmine>
	    at eval (webpack:///./build/client.spec.js?:159:31)
	    at <Jasmine>
	    at fulfilled (webpack:///./build/client.spec.js?:4:58)

https://travis-ci.com/iov-one/iov-core/jobs/149914988
https://travis-ci.com/iov-one/iov-core/jobs/149929347
https://travis-ci.com/iov-one/iov-core/jobs/150532345

LiskConnection can get chain ID
Firefox 62.0.0 (Mac OS X 10.13.0) LiskConnection can get chain ID FAILED
	Error: Timeout - Async callback was not invoked within 5000ms (set by jasmine.DEFAULT_TIMEOUT_INTERVAL) in /Users/travis/build/iov-one/iov-core/node_modules/jasmine-core/lib/jasmine-core/jasmine.js (line 5100)
	<Jasmine>

https://travis-ci.com/iov-one/iov-core/jobs/149918069

LiskConnection can post transaction
HeadlessChrome 0.0.0 (Linux 0.0.0) LiskConnection can post transaction FAILED
	Expected undefined to be defined.
	    at <Jasmine>
	    at Object.eval (webpack:///./build/liskconnection.spec.js?:133:40)
	    at <Jasmine>
	    at fulfilled (webpack:///./build/liskconnection.spec.js?:4:58)
	Expected undefined to be greater than 6000000.
	    at <Jasmine>
	    at Object.eval (webpack:///./build/liskconnection.spec.js?:134:40)
	    at <Jasmine>
	    at fulfilled (webpack:///./build/liskconnection.spec.js?:4:58)
	Expected undefined to be less than 8000000.
	    at <Jasmine>
	    at Object.eval (webpack:///./build/liskconnection.spec.js?:135:40)
	    at <Jasmine>
	    at fulfilled (webpack:///./build/liskconnection.spec.js?:4:58)

https://travis-ci.com/iov-one/iov-core/jobs/149951799
https://travis-ci.com/iov-one/iov-core/jobs/150949064
https://travis-ci.com/iov-one/iov-core/jobs/151152521

Tx (long uppercase hex string here) not found
HeadlessChrome 0.0.0 (Linux 0.0.0) Client With HttpClient Can query a tx properly FAILED
	Error: {"code":-32603,"message":"Internal error","data":"Tx (8D665E7E6C9CBD65F9EB5ED25CC938EC0688C002) not found"}
	    at Object.exports.ifError (webpack:///./build/common.js?:16:16)
	    at Object.exports.throwIfError (webpack:///./build/common.js?:7:29)
	    at HttpClient.eval (webpack:///./build/rpcclient.js?:78:29)
	    at <Jasmine>
	    at fulfilled (webpack:///./build/rpcclient.js?:4:58)

https://travis-ci.com/iov-one/iov-core/jobs/150524298

Rise node not available
RiseConnection : can get chain ID
   Error: Timeout - Async callback was not invoked within 15000ms (set by jasmine.DEFAULT_TIMEOUT_INTERVAL)

https://travis-ci.com/iov-one/iov-core/jobs/151788054
https://travis-ci.com/iov-one/iov-core/jobs/151797930
https://travis-ci.com/iov-one/iov-core/jobs/152024717
https://ci.appveyor.com/project/webmaster128/iov-core/builds/19540179/job/07ogiiog3ntjapgs
https://ci.appveyor.com/project/webmaster128/iov-core/builds/19544811/job/p5qvq5jmcend2lwr
https://travis-ci.com/iov-one/iov-core/jobs/152061361
https://ci.appveyor.com/project/webmaster128/iov-core/builds/19543948/job/egpjm4jnkv1mk3pu

BovFaucet : can be used to credit a wallet
1. BovFaucet : can be used to credit a wallet
   Error: Error: Request failed with status code 500; response body: ""
       at faucet.credit.catch.error C:\projects\iov-core\packages\iov-faucets\src\bovfaucet.spec.ts:25:15
       at <Jasmine>
       at process._tickCallback internal/process/next_tick.js:189:7

https://ci.appveyor.com/project/webmaster128/iov-core/builds/19513928/job/hckpkij7sluo5e0o

The tx paginate issue only appears to be encountered in headless chrome, right?
Which is the only browser/karma test run against the blockchain, as other browsers are only tested on osx.
Maybe I should revisit the code there to ensure it is reentrant even in race conditions.

The lisk issue...
Well, I would kind of like to remove any dependency on lisk testnet (or any non-local infrastructure) in our ci. Ideally we could launch a simple node like we do with tendermint. If not, maybe make those integration tests manual (not automatic on ci).

Looks like Rise testnet node is taking the lead recently. Let's work on this as soon as 0.7.0 is out

Some still exist but this is mostly out of date. Closing.


Replay Error

when video finish and replay button appear, when we click replay it got error and no replay video. any solution?

What player are you using?

Im using jwplayer, but can change if u recommend any other
On Tue, 7 Jul 2020, 10:02 Andriy Lysnevych<EMAIL_ADDRESS>wrote:

What player are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Novage/wt-tracker/issues/24#issuecomment-654672889,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANFYIW6KMPES2AKFHIVQYXTR2LJANANCNFSM4LRBSFQQ
.


serialize to bytebuffer

This solves https://github.com/RoaringBitmap/RoaringBitmap/issues/322

Looks like great work to me. Thorough.

Thanks. I haven't benchmarked it but I'm sure this will soften the CPU spikes I see during serialisation of many sparse bitmaps so I am keen to pick up the next release. Please ping me when you get time to issue a release.

@richardstartin
Working on it right now. I just want to update the documentation before I issue a release.
Even if the performance was the same, the usage is now so much nicer...

I have release 0.8.0. I want with a new minor version because I think we have a much better API.

I notice that Spark may be picking up 0.7.45 for Kryo serialisation. It may be worth their while to use ByteBuffer in preference to DataOutput.

I can't share more details but this reduced a "CPU utilisation" spike from 50% to under 15%. It was a big efficiency gain.

@richardstartin
I fear that this long lasting inefficiency is largely my fault. I was assuming that if there were any performance differences with such functions, it would be small. I held to my assumption for too long without checking.


[WIP] First attempt at deboosting mlir-air

Please do not review this seriously yet, still being refactored and debugged (and I should add unit tests for the transitive reduction).
I need to understand why the test test/Util/Runner/segment_dataflow.mlir fails.
My current assumptions:

Dependency graphs do not contain cycles
Duplicate graphs can be ignored. i.e. if the boost graph has the edge a->b already, it's fine to not insert it again if add_edge(a,b) called.

Are these assumptions valid?
Error dump:

FAIL: AIRMLIR :: Util/Runner/segment_dataflow.mlir (2 of 206)
******************** TEST 'AIRMLIR :: Util/Runner/segment_dataflow.mlir' FAILED ********************
Exit Code: 1
Command Output (stderr):
RUN: at line 8: /home/jamesn/labs/mlir-air/build/bin/air-runner /home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir -f test -m /home/jamesn/labs/mlir-air/mlir/test/Util/Runner/arch.json | /home/jamesn/labs/mlir-air/llvm/build/bin/FileCheck /home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir

/home/jamesn/labs/mlir-air/llvm/build/bin/FileCheck /home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir
/home/jamesn/labs/mlir-air/build/bin/air-runner /home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir -f test -m /home/jamesn/labs/mlir-air/mlir/test/Util/Runner/arch.json
loc("/home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir":197:13): error: 'air.segment' op mismatch between graph and hierarchy op
loc("/home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir":76:12): error: 'air.segment' op mismatch between graph and hierarchy op
loc("/home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir":126:13): error: 'air.segment' op mismatch between graph and hierarchy op
/home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir:13:11: error: CHECK: expected string not found in input
// CHECK: "name": "ChannelGetOp@channel_5(L2<--L2)",
^
:1:1: note: scanning from here
[
^
:67:2: note: possible intended match here
"name": "WaitAllOp",
^

Input file:
Check file: /home/jamesn/labs/mlir-air/mlir/test/Util/Runner/segment_dataflow.mlir
-dump-input=help explains the following input dump.
Input was:
<<<<<<
1: [
check:13'0     X~ error: no match found
2: {
check:13'0     ~~
3:  "name": "process_name",
check:13'0     ~~~~~~~~~~~~~~~~~~~~~~~~~
4:  "ph": "M",
check:13'0     ~~~~~~~~~~~~
5:  "pid": 1,
check:13'0     ~~~~~~~~~~~
6:  "args": {
check:13'0     ~~~~~~~~~~~
.
.
.
62:  "args": {
check:13'0     ~~~~~~~~~~~
63:  "sort_index": "4"
check:13'0     ~~~~~~~~~~~~~~~~~~~
64:  }
check:13'0     ~~~
65: },
check:13'0     ~~~
66: {
check:13'0     ~~
67:  "name": "WaitAllOp",
check:13'0     ~~~~~~~~~~~~~~~~~~~~~~
check:13'1      ?                     possible intended match
68:  "cat": "layer",
check:13'0     ~~~~~~~~~~~~~~~~~
69:  "ph": "B",
check:13'0     ~~~~~~~~~~~~
70:  "ts": 0.001,
check:13'0     ~~~~~~~~~~~~~~
71:  "pid": 1,
check:13'0     ~~~~~~~~~~~
72:  "tid": 1,
check:13'0     ~~~~~~~~~~~
.
.
.

--

Thanks for working on this, @newling . I think both assumptions should hold true, although in the previous boost graph impl they were never checked.

The part which throws error is here: https://github.com/Xilinx/mlir-air/blob/c99190a1ad455b13f49c04107033aad94b7a4e46/mlir/lib/Util/Dependency.cpp#L1606 The air-runner expects a one-to-one mapping between the air.hierarchy ops (herd, segment and launch) and graphs. This error suggests that there is a mismatch.

Thanks @erwei-xilinx
I can't figure out what logic I have changed, so will decompose this patch step-by-step (expect some small PRs tomorrow!)

I still need to implement the sub-graph logic. Will see if I can before cleaning up and making more PRs.

The only remaining use of boost is program options in
examples/air_to_ipu/test.cpp


style: rename interface Accessory to IAccessory

i found that all interfaces have been named IName expect Accessory, so i renamed it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@dchaofei Please help review this PR.

@simpleapples Thank you very much pr, it would be better if wechaty/interface/wechaty.go:4 can be changed together.

@dchaofei You're welcome.
I have renamed Wechaty to IWechaty, please review the code, thx.

@dingdayu Please help review this PR.

@simpleapples HI，Sorry there is no immediate processing.
When I review the PR, I found some minor problems, the codeclimate  failed.
I read it because the code is not formatted by fmt and some exported functions are not commented.
If you need help, you can get my contact information directly by email<EMAIL_ADDRESS>

@dingdayu Hi, I have fixed all issues except the 'similar blocks' issue. Please review it.
Thank you.

Hi @simpleapples ,
Thank you for the contribution! I have invited you to join our Wechaty Contributors team on GitHub, please accept it by following the below message:

You've invited Zhiya Zang to Wechaty! They'll be receiving an email shortly. They can also visit https://github.com/wechaty to accept the invitation.

And we have a WeChat Group for our contributors, please contact @lijiarui and she will help you to join.
Cheers!

Hi, @huan ,
Got it, thank you.

Congratulations! 🎉 🎉 🎉
You become a wechaty contributor successfully! 👏 👏 👏
Please:

[ ]  Add my wechat: ruirui_0914 and send me this pr link. I will invite you into Wechaty Contributor Room
[ ] Update your profile here: Contributor List, then other contributors will know you better!
[ ]  Join wechaty org by clicking https://github.com/wechaty


Fix creation of AWs/Jobs with same name in different namespaces

Issue link

Resolves #433
And resolves #383
In the genericresource.go file there is a SyncQueueJob function which is used to create resources inside etcd. In this function, the identification of AppWrappers/Jobs was primarily based on a label (resourceName) which had a value of the AW name without taking into account the namespace. As a result, when 2 AWs/Jobs with the same name were created in different namespaces, the 2nd AW/Job was identified as a duplicate.
What changes have been made

Changed the resourceName label to account for the namespace too.
Changed the labelSelector and function accordingly.
Created and added this scenario as part of the e2e-test.

Verification steps

Have MCAD running.
Create two AWs/Jobs with the same name but in different namespaces.
See their pods created respectively, and running.

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] e2e test
[x] Manual tests

Do we have any plans to change the label name from appwrapper.mcad.ibm.com to something consistent with the group name, maybe appwrapper.worload.codeflare.dev?

Do we have any plans to change the label name from appwrapper.mcad.ibm.com to something consistent with the group name, maybe appwrapper.worload.codeflare.dev?

Yes, I agree it'll have to be changed, e.g., to worload.codeflare.dev/appwrapper. @tardieu is this the only purpose of that label? If yes, @ChristianZaccaria maybe you could tackle that change in the scope of that PR, WDYT?

Do we have any plans to change the label name from appwrapper.mcad.ibm.com to something consistent with the group name, maybe appwrapper.worload.codeflare.dev?

Yes, I agree it'll have to be changed, e.g., to worload.codeflare.dev/appwrapper. @tardieu is this the only purpose of that label? If yes, @ChristianZaccaria maybe you could tackle that change in the scope of that PR, WDYT?

Yes, sounds good to me. It's a good idea to include the change here.

@tardieu is this the only purpose of that label?

The label is used to select resources directly created by MCAD (in genericresource.go) during creation, deletion, and to decide if the job is complete. As discussed, this is probably redundant with just querying for the generic items one by one.
The same label is used to select pods directly or indirectly caused by an appwrapper (in queuejob_controller_ex.go) to look at pod statuses and decide if the job is healthy or not.
The change needs to be consistent across. The introduction of a separate label for the namespace should also be done consistently across. Otherwise, we will end up confusing pods resulting from appwrapper mynamespace/myname with the pods of appwrapper myothernamespace/myname.
This does mean that all tests and all appwrapper yamls in circulation have to be adjusted. I would rather sync this change with the apigroup change than have deal go back and forth between 3 CRD revisions (old group + old label, new group + old label, new group + new label).

If I read this correctly, the latest push adds a label with the namespace of the wrapped resource itself rather than the appwrapper namespace. I am not sure I understand the logic.

We should probably avoid unqualified label names like resourceName and replace them with names such as workload.codeflare.dev/resourceName.

If I read this correctly, the latest push adds a label with the namespace of the wrapped resource itself rather than the appwrapper namespace. I am not sure I understand the logic.

I think I understand your point. Would it make more sense to include labels for both wrapped resources and appwrappers? or just use the appwrapper name and namespace? I think that the issue with this is that i.e., if two jobs are created with same name and different namespaces, the 2nd job's pods are not created, so it's not originally the appwrapper where the issue lies. Let me know if I misunderstand, thank you Olivier

@astefanutti Hi Antonin, I'm wondering if you think updating the quotaManagement.rbac.apiGroup like this is correct? From ibm.com to quota.codeflare.dev. Thanks!
https://github.com/project-codeflare/multi-cluster-app-dispatcher/blob/bbb6871a4d79d1364cd04504557d0f1c4c671844/test/kuttl-test-borrowing.yaml#L8

I attempted to not use completely the resourceName label but caused the e2e tests to fail. I believe it has to do with certain scenarios where the resourceName differs from the AppWrapper name, such as this example: https://github.com/project-codeflare/multi-cluster-app-dispatcher/blob/41833f2bd925d39e7391165d402e4df76025a296/test/e2e-kuttl-deployment-01/steps/03-assert.yaml#L27-L30

Hello @ChristianZaccaria , a lot of changes seem to be unrelated, or indirectly at least, to the original issue:

shouldn't these change be merged in another PR? 🤔

Hello @ChristianZaccaria , a lot of changes seems to be unrelated, or indirectly at least, to the original issue:
shouldn't these change be merged in another PR? 🤔

Hi Kevin, it was suggested to include these changes in the scope of this PR: https://github.com/project-codeflare/multi-cluster-app-dispatcher/pull/652#issuecomment-1741082212


Route53 plugin dns validation

I'm looking to get automatic cert renewal working through route53 and I'm not seeing any documentation on the wiki about this.
I'm wondering where the config with the AWS credentials goes, and any other steps I might need to perform?
I need to use DNS validation (most of our certs are for machines that don't have port 80 forwarded externally), is this supported by the plugin that was added recently?

Yes it's meant for DNS validation. The way these plugins work is that you copy the .dll files from the package to the same folder as the main program. If you then create a new certificate "with advanced options", i.e. M from the main menu, the plugin will be offered as an extra option when the time comes to choose a validation method. Once chosen, it will ask for the "Access key ID" and "Secret access key". Now I'm not familiar with the specifics of AWS myself, but I guess those are the credentials that you will find/create somewhere in your control panel.

Does the plugin support IAM roles for instances running on EC2?  Having to provide credentials is an antipattern in this case.

Looking at the code I don't see anything related to that, but perhaps @ivanstus can comment.

This plugin requires "Access key ID" and "Secret access key" for the user with "route53:GetChange", "route53:ListHostedZones", "route53:ChangeResourceRecordSets"permissions. Currently it has no support for IAM roles, but I'll promise to look into when have some time.

#1113 adds support for IAM roles.
Now there are 3 options:

use default IAM role for current instance
use a custom IAM role
use access keys

@WouterTinus, I think this question can be closed now.

Yes, thanks for your quick response!


Fix invalid scene error

Fix invalid scene error when the scene switching system is setting the active scene.

Codecov Report

Merging #112 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           master    #112   +/-   ##
======================================
  Coverage    8.62%   8.62%
======================================
  Files          48      48
  Lines        8009    8009
  Branches     1241    1241
======================================
  Hits          691     691
  Misses       7260    7260
  Partials       58      58

Impacted Files
Coverage Δ

...rkingManagerComponents/Core/NetworkSceneManager.cs
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Fixed here https://github.com/MidLevel/MLAPI/commit/91e84f2f9642e64af6fd9f426d15e5eac5eebc95


Sending expected file name via parameter to override random_url

This is the fruit of the discussion #216

Normally users can decide resulting filename by sending filename on multipart request, as long as server does not use random_url
In case #217 is implemented, an extra argument like --file-name could be helpful to override random path result (on servers which explicitly allow overriding  the random_url setting). An example:
# normal operation with random_url
$ rpaste imgurdownload.jpg
http://example.com/some-random-string.jpg

# if overriding allowed
$ rpaste --file-name i-dont-know-what-i-expected.jpg imgurdownload.jpg
http://example.com/i-dont-know-what-i-expected.jpg

Such a parameter would also be useful no matter what. Right now it doesn't seem possible to set the filename if random url is not used either. So apparently that was an oversight earlier.

I looked into this, but it seems a bit more complicated than expected.
Firstly, the arguments are not available to the upload function. Thus one either has to refactor quite a bit, or use a workaround and use the Config struct. The latter is easier but makes less sense, since this would also allow people to set it in the config file, which makes no sense at all.
Just for getting ahead quickly, I implemented the workaround, but for some reason setting the header for the request does not work. You can have a look at the code here: https://github.com/tessus/rustypaste-cli/commit/9c33fd18f022259842927e4a509b3ec48ea6aa49
I haven't had the time to debug the request/response via tcpdump.

I was mixing up a few things in my last comment. I was able to code it. I still have to update the README. Will do that tomorrow and then I'll create a PR.


Add frankfurter.app (formerly fixer.io)

Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[x] Your submissions are formatted according to the guidelines in the contributing guide
[x] Your additions are ordered alphabetically
[x] Your submission has a useful description
[x] The description does not end with punctuation
[x] Each table column should be padded with one space on either side
[x] You have searched the repository for any relevant issues or pull requests
[x] Any category you are creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Hey, sorry for the delay.
Please link to API docs.

@yannbertrand I updated the link

Thank you


Local hosted fonts are not exported

We are using this great plugin to make all fonts local: https://wordpress.org/plugins/embed-google-fonts/
But not a single font file is exportet, even if i add them manually to the "Include additional URLs"-List, like so:

/wp-content/cache/embed-google-fonts/tangerine/tangerine-v12-latin-700.woff
/wp-content/cache/embed-google-fonts/ubuntu/ubuntu-v15-latin-ext_latin_greek-ext_greek_cyrillic-ext_cyrillic-700.woff
/wp-content/cache/embed-google-fonts/ubuntu/ubuntu-v15-latin-ext_latin_greek-ext_greek_cyrillic-ext_cyrillic-300.woff
/wp-content/cache/embed-google-fonts/ubuntu/ubuntu-v15-latin-ext_latin_greek-ext_greek_cyrillic-ext_cyrillic-500.woff
/wp-content/cache/embed-google-fonts/ubuntu/ubuntu-v15-latin-ext_latin_greek-ext_greek_cyrillic-ext_cyrillic-300italic.woff

The only file in those exported font folders still is the *.css file.

Just a note to say I'm also having the same issue.
My original assumption was that my CSS file was not being parsed for further URLs to crawl (@font-face).
So then I also added my font files to "Include additional URLs" too, and they are being ignored. It may be that the plugin doesn't attempt to crawl certain extensions?

Thanks for reporting, @lukearmstrong, @lukearmstrong. Leaving this issue open, as I think the Include Additional URLs function has some issues I need to resolve.
For a workaround, the Autoptimize plugin usually helps catch any missing fonts/styles, etc. I'm not sure if it will play nicely with the Embed Google Fonts plugin, but it should be able to replace it and provide the same functionality, plus more nice features.

Just to +1 on this - theme icons aren't being exported even if explicitly listed in additional URLs.

Just to +1 on this - theme icons aren't being exported even if explicitly listed in additional URLs.

I'm afraid, the same applies for CSS files and PDF files, though explicitly listed in additional URLs.
For example (names slightly adapted):
/wp-content/uploads/2021/02/XYZ_someDocument_210229.pdf
/wp-content/themes/theme-name/legacy.css
/wp-content/themes/theme-name/svgcache/base.svg

Same issue here, besides fonts reported earlier, I'm also having issues with svg, txt, etc... I tested a bunch of files and basically 'Include additional URLs' feature doesn't work at all.
I started to test back version by version and noticed that it breaks from v6.6.19 (when this was introduced: "no more txt files polluting uploads dir, slowing things down").
I also noticed that by default, v6.6.18 generates/discovers much more files than v6.6.19.
Another thing: from v6.6.19 the ability to select 'Target directory' is removed and always defaults to /wp-content/uploads/static-html-output, which to me is an important feature to have... not sure why it was removed.
In summary, I like how the latest version is faster... but unfortunately is too unreliable, so for now I think is better just to use v6.6.18 and hopefully @leonstafford can fix it some day  :)
This is such a useful and unique plugin... I really hope it can get more love as it is the only (open source) tool around.

Thanks @diegoteliz.
Most love is shifting to WP2Static in https://github.com/leonstafford/wp2static, but I want to bring in the good bits from this plugin to there.
It's a hard compromise between detecting too little and too much. I prefer to detect the minimum best-guess URLs by default, but allow the user to control adding more/less to this. Finally, allowing a crawler to discover new URLs as it crawls the initial list.
Hopefully, the best of all these processes will be merged into WP2Static upcoming versions. I've also started a new specific library for WordPress URL detection.
There is also the Simply Static plugin on GitHub (not sure if Pro features are included in that) and a few new entrants in the wp.org plugin repository.
I expect WP2Static to solidify it's spot as best open source tool for this, you can find my recent roadmap/plans: https://github.com/leonstafford/wp2static/discussions/795
For short term workaround, if your missing assets are part of WP site, you can try the Autoptimize plugin, which often makes it much easier for the plugin to crawl everything in the site.

(also, bit slowed down in development past few months, as I've taken on a fulltime role with Strattic. As I get more comfortable in that role, I'll be putting more time into my open source projects, such as WP2Static and Lokl


Show download progress of custom GE Proton

Make generic by taking a parameter for the text prompt, and allow passing progress via stdin.
Then reuse on the the chimera tweaks download splash screen.

Could use a similar approach as the password prompt or splashscreen. A simple template change should be all that is needed.

This doesn't appear to work with Gamepad UI. The download appears to happen during the Gamepad UI loading screen.

Mhhh... I'll look into it. Thanks for the testing.

This doesn't appear to work with Gamepad UI. The download appears to happen during the Gamepad UI loading screen. I think this is actually fine that way.

REading this again. Should we just drop the feature? Would seem odd to add a ChimeraOS splash in the middle of a game launching.

Is this one still relevant? I would close this as won't fix.

Is this one still relevant? I would close this as won't fix.

I think the message is general should be updated because a lot of people get confused thinking it actually is downloading updates every boot.


Starting Actioncable and Rails separately

It seems that one should not have to start the Actioncable and Rails portion of an app separately. Is this something to be fixed, or is this a conscious design choice?

Conscious, but not fixed in stone. Please take a look at our open issues before submitting a question. You would have found: https://github.com/rails/actioncable/issues/15 :grin:


fix: support ChainExpression in await-interactions rule

Issue: #86
What Changed
in existing use-cases of the await-interactions rule, the code is assumed to be valid when the CallExpression node is the immediate child of an AwaitExpression node.

await FirstStory.play(context); → AwaitExpression > CallExpression > …

await FirstStory.play!(context); → AwaitExpression > CallExpression > TSNonNullExpression > …

to support an expression with an optional chain, the logic to determine if the expression was properly awaited now checks if the parent of the parent of the CallExpression node is an AwaitExpression.

await FirstStory.play?.(context); → AwaitExpression > ChainExpression > CallExpression > …

Checklist
Check the ones applicable to your change:

[x] Ran yarn update-all
[x] Tests are updated
[ ] Documentation is updated

Change Type
Indicate the type of change your pull request is:

[ ] maintenance
[ ] documentation
[x] patch
[ ] minor
[ ] major

Thank you so much for your contribution @VinceMalone !! I added a small fix and this is good to go.
Before merging though, would you like to edit your author email from your commit? Seems as if it's not set and I think this might end up not adding you as a contributor to this project as a result.
Thank you!

thanks @yannbf. I think I fixed the email; lemme know if it's still not working.

thanks @yannbf. I think I fixed the email; lemme know if it's still not working.

It's perfect now! Thanks once again, for using this library and for making it better!


OnScroll, RefreshControl and onEndReached of nested ListView not working

Hi,
I'm trying to nest the following component inside the ParallaxScrollView:
                <ListView
                ref="listview"
                key={"postList"}
                dataSource={this.state.dataSource}
                renderSectionHeader={this.renderSectionHeader}
                renderRow={this.renderArticle}
                renderFooter={(this.state.connError === false) ? this.renderFooter : this.renderErrorView}
                scrollEnabled={this.state.scrollEnabled}
                onScroll={this.onScroll}
                onEndReached={this.onEndReached}
                refreshControl={
                    <RefreshControl
                    refreshing={this.state.isRefreshing}
                    onRefresh={this.onRefresh}
                    colors={['#f7913d', '#f7913d', '#f7913d']}
                    progressBackgroundColor={'#fff'}
                    />}
                    />

OnScroll, onEndReached and RefreshControl are not working.
I Also noticed that scroll properties are not changing by setting the OnScroll function on the ParallaxScrollView:
    onScroll = () => {
        console.log(this.refs.listview.scrollProperties);
        if (this.refs.listview.scrollProperties.offset + this.refs.listview.scrollProperties.visibleLength >= this.refs.listview.scrollProperties.contentLength){
            // this.onEndReached();
        }
    };

I also noticed the this.refs.ParallaxView.state is not updated at the OnScroll event, so I have no idea what is the new offset and what is the new scrollY values of the state.
Any ideas?

Hi @avishayil, I found it was better to use ListView#renderScrollComponent rather than mounting ParallaxScrollView as the parent of ListView:
<ListView
  renderScrollComponent={props => (
    <ParallaxScrollView
      {...props}
      .../>
  }}
  .../>

This gives the ListView a chance to pass its own onScroll handlers down to ParallaxScrollView, and I think this is how you can get refreshControl to work.
I haven't tested this exact approach myself, but I have something similar to this working in an app.

Awesome, thanks!

@avishayil onEndReached was working? when you test?


Add runtime accessor class

Re: #187, possibly some other issues as well.
This adds lib/get.js, a runtime accessor class for compiled MessageFormat objects. It's a new API, so the method and other names can and should be changed if we think of something else that's better. Here's how it would work:
First, presuming that we have the messages in a JSON file messages.json:
{
  "en": {
    "a": "A {TYPE} example.",
    "b": "This has {COUNT, plural, one{one user} other{# users}}.",
    "c": {
      "d": "We have {P, number, percent} code coverage."
    }
  },
  "fi": {
    "b": "Tällä on {COUNT, plural, one{yksi käyttäjä} other{# käyttäjää}}.",
    "e": "Minä puhun vain suomea."
  }
}

And that we've compiled that into JavaScript:
messageformat -l en,fi messages.json > messageData.js

Then we can use the new getter and this API:
var MessageFormatGet = require('messageformat/lib/get');
var msgData = require('./messageData');
var messages = new MessageFormatGet(msgData, 'en');

messages.hasMessage('a')                // true
messages.hasObject('c')                 // true
messages.get('b', { COUNT: 3 })         // 'This has 3 users.'
messages.get(['c', 'd'], { P: 0.314 })  // 'We have 31% code coverage.'

messages.get('e')                       // 'e'
messages.setFallback('en', ['foo', 'fi'])
messages.get('e')                       // 'Minä puhun vain suomea.'

messages.locale = 'fi'
messages.hasMessage('a')                // false
messages.hasMessage('a', 'en')          // true
messages.hasMessage('a', null, true)    // true
messages.hasObject('c')                 // false
messages.get('b', { COUNT: 3 })         // 'Tällä on 3 käyttäjää.'
messages.get('c').d({ P: 0.628 })       // 'We have 63% code coverage.'

Based on how I've used MessageFormat myself and seen it used by others, I believe that the above would provide a more developer-friendly interface, as well as being less likely to throw errors.
The code already has JSDoc comments, but here's an overview of the class's members:
constructor (locales: object, defaultLocale: string | null | undefined)

availableLocales: string[]  // readonly
defaultLocale: string | null
locale: string

addLocale (lc: string, data: object)
getFallback (lc: string | undefined): string[]
setFallback (lc: string, fallback: string[] | null)

get (key: string | string[], props: object | undefined, lc: string | undefined): string | object
hasMessage (key: string | string[], lc: string | undefined, fallback: boolean | undefined): boolean
hasObject (key: string | string[], lc: string | undefined, fallback: boolean | undefined): boolean

For getters, this.locale is used if lc is null or undefined. key can be a string for top-level objects, or a keypath for hierarchical objects.

I didn’t initially implement this because I assumed people would want to integrate their tooling in their own way with compiled messages and stuff, but:
a) we provide a lot more of that tooling now, especially with the cli stuff
b) this is super small and something everyone would have to end up implementing.
I assume adding this will be dead code for a lot of people, but I think it’s small enough to warrant the feature.
:+1:

I'll add some tests, and keep this open for at least a few days in case the API ends up warranting a bit of bikeshedding. I'm not sure I'm happy yet with the class name Get in particular.
Re: dead code, at least via require() and import this only gets included if it's implicitly asked for. And in any case it gzips to something like half a kilobyte, which is probably good enough. :)

Merged this to get it available in a messageformat@next release, but the API is still liable to change.

Noting some API changes here from the last week:

The accessor class is now called Messages, and it's been moved from lib/ to the root so that it'll be available via npm as messageformat/messages.
addMessages replaces addLocale, to allow for partial updates of a locale.
Setting locale and defaultLocale will match the given string to partial match of the available locales; they use the new method resolveLocale to do that.


subsys: logging: Add RTT logging backend

This commit adds SEGGER RTT logging support.
Signed-off-by: Olivier Martin<EMAIL_ADDRESS>

Codecov Report

Merging #9365 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9365   +/-   ##
=======================================
  Coverage   52.51%   52.51%
=======================================
  Files         201      201
  Lines       25352    25352
  Branches     5299     5299
=======================================
  Hits        13313    13313
  Misses       9944     9944
  Partials     2095     2095

Impacted Files
Coverage Δ

subsys/logging/log_core.c
69.93% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

FYI, I have updated my pull-request to address one of the comment and to follow the recent changes in UART LOG backend: https://github.com/zephyrproject-rtos/zephyr/commit/c859e2a254266bcd5ad2698c73e2b2b2f0d20e83#diff-17614056ba02859d097b017d64d3efc7
My last comment is still valid:
@nordic-krch, @pizi-nordic Thanks for your feedbacks. Using SEGGER_RTT_Write() would allow to not mask higher priority interrupts such as Bluetooth or USB (to be confirmed that their priority level is higher than the RTT one). I would also not need to change rtt_console.c that disables all interrupts (that's where my code was inspired) - even if rtt_console.c should probably also use SEGGER_RTT_Write(), no?
The current RTT library (ie: ext/debug/segger/rtt) does not expose a function to know if RTT is blocking or not. But LOG subsystem could in theory support partial block transfer, in practical it does not seem to do it - the returned number of written bytes  is never taken in account (look for ctx->func( in https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/logging/log_output.c).
Is it correct the changes I need to do are:

Fix LOG  to support partial write
Use SEGGER_RTT_Write() and return the number of written byte

@Olivier-ProGlove, there is a pending PR which fixes issue in log_output so that number of written bytes is taken into account: https://github.com/zephyrproject-rtos/zephyr/pull/9211/files#diff-94bdf76265c844bd2964a29c102c57a7R84

Thanks @nordic-krch I keep an eye on your pull-request. I will rework this pull-request once yours is merged. I also noticed your SEGGER_RTT_LOCK/UNLOCK() issue.

@pizi-nordic can you please re-review?

@Olivier-ProGlove could you add a commit to this PR implementing #9778? Then I think @pizi-nordic will be happy from what I understand.

@Olivier-ProGlove are you going to address #9778?

@nashif #10659 adress #9778.

I still not see selection of SEGGER write mode.
Without this, the SEGGER_RTT_Write() could just spin with interrupts disabled by SEGGER_RTT_LOCK() while waiting on data request from PC.

I still not see selection of SEGGER write mode.
Without this, the SEGGER_RTT_Write() could just spin with interrupts disabled by SEGGER_RTT_LOCK() while waiting on data request from PC.

@pizi-nordic Please see ##10659 I think this is a bit ahead of this PR.

@pkral78: Thanks.
@Olivier-ProGlove : Could this PR be closed due to #10659 ?

If you think https://github.com/zephyrproject-rtos/zephyr/pull/10659 is more advanced than my current PR, let's go for @pkral78's PR.
I could update mine with the missing bits but if @pkral78 has already done it, there is no point for me to duplicate work.

Could we close this PR as #10659 is merged?

Could we close this PR as #10659 is merged?


Update README.markdown

Scan submitted to Checkmarx

Checkmarx scan completed
Full Scan Details
Violation Summary

Severity
Count

High
77

Details

Lines
Severity
Category
File
Link

8
High
Command_Injection
website/users/check_pass.php
Checkmarx

18
High
Command_Injection
website/pictures/high_quality.php
Checkmarx

9
High
Command_Injection
website/passcheck.php
Checkmarx

18
High
Command_Injection
website/pictures/conflictview.php
Checkmarx

3
High
File_Inclusion
website/admin/index.php
Checkmarx

27 29
High
File_Manipulation
website/pictures/upload.php
Checkmarx

44
High
Reflected_XSS_All_Clients
website/comments/preview_comment.php
Checkmarx

19 22
High
Reflected_XSS_All_Clients
website/pictures/search.php
Checkmarx

22 35
High
Reflected_XSS_All_Clients
website/comments/add_comment.php
Checkmarx

144
High
Reflected_XSS_All_Clients
website/include/html_functions.php
Checkmarx

19
High
Reflected_XSS_All_Clients
website/submitname.php
Checkmarx

16
High
Reflected_XSS_All_Clients
website/comments/delete_preview_comment.php
Checkmarx

16 39
High
Reflected_XSS_All_Clients
website/users/login.php
Checkmarx

83
High
Reflected_XSS_All_Clients
website/pictures/upload.php
Checkmarx

49
High
Reflected_XSS_All_Clients
website/users/register.php
Checkmarx

5
High
Reflected_XSS_All_Clients
website/error.php
Checkmarx

35
High
Reflected_XSS_All_Clients
website/pictures/conflict.php
Checkmarx

9 36
High
Reflected_XSS_All_Clients
website/passcheck.php
Checkmarx

2 3 4 5
High
Reflected_XSS_All_Clients
website/test.php
Checkmarx

12
High
Reflected_XSS_All_Clients
website/piccheck.php
Checkmarx

3
High
Remote_File_Inclusion
website/admin/index.php
Checkmarx

24 30
High
SQL_Injection
website/users/view.php
Checkmarx

16
High
SQL_Injection
website/pictures/view_flymake.php
Checkmarx

14
High
SQL_Injection
website/guestbook.php
Checkmarx

14
High
SQL_Injection
website/comments/delete_preview_comment.php
Checkmarx

10
High
SQL_Injection
website/admin/login.php
Checkmarx

102
High
SQL_Injection
website/include/admins.php
Checkmarx

29 30 39 47
High
SQL_Injection
website/pictures/upload.php
Checkmarx

18 24
High
SQL_Injection
website/comments/preview_comment.php
Checkmarx

16
High
SQL_Injection
website/pictures/view.php
Checkmarx

31 71 104
High
SQL_Injection
website/cart/action.php
Checkmarx

16 25
High
SQL_Injection
website/pictures/conflict.php
Checkmarx

16
High
SQL_Injection
website/comments/add_comment.php
Checkmarx

18
High
SQL_Injection
website/pictures/high_quality.php
Checkmarx

18
High
SQL_Injection
website/users/register.php
Checkmarx

18
High
SQL_Injection
website/pictures/conflictview.php
Checkmarx

11
High
SQL_Injection
website/users/login.php
Checkmarx

15
High
SQL_Injection
website/pictures/search.php
Checkmarx

37 97
High
Second_Order_SQL_Injection
website/include/cart.php
Checkmarx

22 102 157
High
Second_Order_SQL_Injection
website/include/users.php
Checkmarx

117 197
High
Second_Order_SQL_Injection
website/include/pictures.php
Checkmarx

60
High
Second_Order_SQL_Injection
website/include/comments.php
Checkmarx

87
High
Second_Order_SQL_Injection
website/include/admins.php
Checkmarx

28
High
Stored_XSS
website/include/comments.php
Checkmarx

97 136
High
Stored_XSS
website/include/cart.php
Checkmarx

19 39 60 79 98 117 132 221
High
Stored_XSS
website/include/pictures.php
Checkmarx

15
High
Stored_XSS
website/include/guestbook.php
Checkmarx

22 132
High
Stored_XSS
website/include/users.php
Checkmarx

hgfg


Further optimize initial page load by leveraging preboot

Describe the bug
In https://github.com/DSpace/dspace-angular/pull/1627 we've used a number of strategies to mitigate the problem that pages appear to load twice: they come back from the server fully rendered, but then are constructed again when the client JS code starts up
That works well for the home page, because we can predict what will be needed there and load it immediately, rather than lazily load it afterwards. However we can't do that everywhere without increasing the initial JS download further (and it is already rather large). So as a consequence on most pages other than the homepage you'll still see a flicker on the first page load, though https://github.com/DSpace/dspace-angular/pull/1627 made it much quicker and less noticeable
We can likely further improve upon this by using preboot, a library to smooth the transition from server-side rendered pages to client-side rendering, by doing things like replaying server side events on the client, buffering, ensuring the focus is retained when re-rendering, etc. (not to be confused with the preboot option in our current config, which basically just enables SSR)
We've done a few initial tests, and the results look promising:
Without preboot
https://user-images.githubusercontent.com/1567693/179940500-9d3eff79-e1b3-4f92-a522-194af6aab75f.mov
With preboot
https://user-images.githubusercontent.com/1567693/179940540-64db2065-f326-46ad-a454-eb5ff5c8373c.mov
However simply adding and enabling it as we've done here creates a few other problems, such as unexpected page reloads when using the header search field for example. It also looks like we'll need at least some of the workarounds for https://github.com/angular/preboot/issues/75
We'd like to see if we can work out the kinks and create a PR. We'll take a max of 20h
Related work
https://github.com/DSpace/dspace-angular/pull/1627

@artlowel : This seems worth looking into.  I would caution though that I'd be good to ensure it doesn't take up too much time for 7.4... I feel the fixes in 7.3 are significant (over 7.2 and prior).  This seems like another good fix, but I don't want it to take away significantly from fixing other high priority issues on our board.
So, I'm fine with spending 20hrs on this. But, if it becomes more than that, I'd recommend we consider pushing this to 7.5.  In any case, for now I'll assign to you with the 20hr max on it, and move it to 7.4 board


2.7.x

Platforms affected
Motivation and Context

Description

Testing

Checklist

[ ] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[ ] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[ ] I've updated the documentation if necessary

Thank you for the PR.
There are a lot of changes, from various committers, and no context in the PR description
From the title and a quick glance at the commits, it seems you are updating deprecated versions of CLI which we no longer maintain.
I am not certain if anyone will have time to review or merge this.

This is an accidental PR it seems, when you open the branch the PR came from you get:

This branch is even with apache:2.7.x.


Controls not dismissed when unpausing programatically.

When we pause the video by tapping it. The controllers appear.
When calling programmatically
chewieController.play()

The controllers don't get dismissed.
Demo:

Clicked on pause
Scrolled down and up again which causes a chewieController.play() if within some boundaries.
The video plays again but the controls and overlay don't get dismissed.

The bug makes sense as I see the call just proxies the call to the videoPlayer
  Future<void> play() async {
    await videoPlayerController.play();
  }

It would be nice to trigger the same hiding the controllers as when tapping the play button.

True, do you mind open a PR with this?


Add instructions for build setup on macOS

The steps to get a working Kythe build setup on macOS are sufficiently different from Linux-based systems that I think they deserve a separate document.
I propose to write this doc, based on my recent experience of setting it up.
One question, though: Assuming you agree this is worth doing, would you like it to be inlined into the existing Getting Started doc, or put into a separate file and linked? The latter would be my preference, but I don't feel very strongly about it.

Thanks for doing this!
I'd prefer a separate file.

Agree with latter (separate file)

After #3316 I'll push the website and close this issue

The website should be live now.


admin

smaill

What is the purpose of this PR?

Closing, since there was never a response and no purpose was identified.


Custom entry point

Are there any ways to define custom entry points other than src/index.js?

Yup. See the example: https://github.com/palmerhq/backpack/blob/master/packages/backpack-examples/with-custom-webpack-config/backpack.config.js#L9

Link is broken. This is the right one. https://github.com/palmerhq/backpack/blob/master/examples/with-custom-webpack-config/backpack.config.js#L9.

Yeah I moved the examples to the top level directory


Fixes entry parse if there is no URL

Parsing was failing if an entry had no defined URL. The parser was assuming it existed, and using it to expand all relative URLs
While it probably always exists for any well-made RSS feed, it won't be true if we're using something like Kill The Newsletter to convert emails to RSS

@naps62 could you squash before I merge it?

@gabrielpoca done


Where is the documentation???

A fat lot of use an SDK is without documentation ...
Other than a few sparse examples I see no documentation ...
I cannot recommend to my company that we use this to integrate Docusign PDF signing to our 4,000 strong userbase with an undocumented SDK so ............

Yeah, this takes a fair bit of reverse-engineering and trial/error to get a handle on the classes and methods, but I can attest that it works really well. I don't have 4000 users, but we do send out about 800 docs a month using this sdk.
If you're familiar with the official DocuSign API documentation itself, it makes things a lot easier to figure out since at that point you're just figuring out how this sdk performs those raw functions.

Hi.
Thanks for the reply.
Well I'm not stupid and perfectly capable of, eventually, working out the answers to how to do something and what all the properties are called and used for etc but as the KDM for technology take-on in our organisation I've got a hell of a hard time recommending a company that can't be bothered to document its own SDK ...
If this was free then fair enough but presumably these guys want to charge our userbase somewhere down the line and in that context, it's not nearly good enough to warrant that commitment - I've lost count of the number of hours I've lost going round in circles on the (awful!) web site(s) ...
My spideysense is telling me to walk away because problems now are likely to be huge problems after the commitment has already been made and I'm too old and experienced to be making such mistakes ...

I definitely get where you're coming from. My fear is that you're going to run into this wherever you go. The majority of e-sign customers are not going to be doing advanced API stuff like this. We tried out RightSignature in particular before DocuSign, and they were worse in terms of what their API could do. Before that, I worked for a different company that had a Silanis implementation, and it was a total nightmare.
Granted, it sounds like my company is a good deal smaller than yours, so our commitment may be a lot less weighty than what you have to deal with, but I'd say if you want e-signed docs badly enough with the ability to do everything via API, this may be about as good as it gets right now.

The issue is one of being able to get out of the commitment - by commitment I mean having to lose the time you've put into it because it's a development dead-end with no way out.
Of course, I don't know that there are going to be such problems but failing to document your own SDK is an appalling message to send out to experienced developers like myself. It gives me that awful "open source" vibe where nothing works as it should and, inevitably, disappears after a year - I/we really can't afford that to happen with this dev ...
Perhaps I should give it another look after the holiday weekend when I've got my second wind :)
Thanks again for your responses - much appreciated :)
Cheers.

Hi @JF66666, thanks for your comments/feedback.  Our apologies that the SDK documentation is currently lacking, we recently started using Swagger to generate our SDKs and with limited resources have only been able to focus on functionality so far.
We do indeed have current work in progress to vastly improve the documentation and guides, which will go within 6 weeks from now.  Till then as @brbeaird mentions you should be able to accomplish all your desired workflow/use-cases using the SDK so feel free to log issues if you run into anything and we can help.  Sounds like @brbeaird is pretty familiar with the code as well :)
I'd also like to mention just in case you didn't see, the API Recipes demonstrate some of the most popular API workflows (using the SDKs) and should be a big help:
https://www.docusign.com/developer-center/recipes
Also check out the DocuSignAPI tag on Stack Overflow as there's a wealth of information throughout.
Hope this helps.
--Ergin

Hi.
Thanks for the reply - I look forward to seeing the C# SDK documentation ...
Recipes are all well and good and better than nothing but it's a matter of due diligence,
Ultimately, while I may well be able to do what needs to be done without proper documentation, if I/we get stuck and I'm seen to have chosen tech that is undocumented then that makes me look a right unprofessional muppet - hence the hesitancy ...
Cheers.

So.... how bout that documentation?

We have SDK-specific documentation on the roadmap, but for now, we have three sources of info:

The API reference manual. The SDK calls track the reference manual extremely closely since the SDKs are mechanically built using a Swagger file.
The code examples. For C# see https://github.com/docusign/eg-03-csharp-auth-code-grant-core Especially note the patterns of SDK vs API implementation which are consistent for the entire SDK.
The SDK source is open source

I agree that while the above is helpful, more should be provided. It is on our roadmap.
Thanks,
Larry


chore: update log4js to 3.0.0

Node.js 4 is no longer supported upstream. Also, log4js 2.x (through its loggly dependency) includes quite a few exploitable packages which users (like me) have started to get notices about.
Not that any of that helps the major version # bump, but it could serve to justify it.

Obselete by #3082


How to get frame rate (fps) of the playing video?

How to get the frame rate of the playing video?

Hi, it's not possible, unless the iframe player from YouTube adds that functionality.


feature(backport): Perform a once-off bulk backport of the full dataset

Add command bulk_backport that calls backport for backporting all datasets that are used in at least one chart and are not private to Walden. Next, flag --backport creates Backport steps for all backported datasets found in Walden. That step aligns grapher metadata with ETL metadata and yields a wide-table (or long-table in rare cases) with all columns from the backported dataset.
I've tested it on two cases - creating World Bank Indicators from backported data and for getting SDG indicators into ETL catalog.
Datasets & tables are currently named like dataset_5438_global_health_observatory__world_health_organization__2021_12. We have an issue for making them prettier.
Long vs wide tables
The initial plan was to convert every dataset to a wide Table, but it turned out there are a few datasets for which wide format wouldn't be effective (creating extremely sparse >4GB dataframes). I tried a few things - sparse dataframes, hardcoded steps for those datasets, better compression, ... but none of them worked well so I ended up keeping the table in long format and added helper function create_wide_table to convert them on-demand after pre-filtering (useful for SDG dataset). If this works for us we can wrap this functionality in a new LongTable class with to_wide_table method.
Running some benchmarks showed that long format is much less efficient than wide format. There are a few exception (extremely sparse datasets) that were autogenerated, but rather than supporting another format I've just split it into tables with 1000 variables. In the future we can revisit those datasets and split them logically into tables in the generating script.
Removed content-hash
Since field dataEditedAt is reliable, there's no need for calculating content hash of all values. Instead, we only calculate checksum of all metadata (which includes dataEditedAt).
Performance
Checksum comparison takes ~1s per dataset (we have 800 datasets in total), this could be speeded up by async / threading, but it's ok for now IMHO.

Looks good! I think we should store something else in walden for the source but that is my only issue for now. In the future we might want to do a bit more to make the very large datasets more digestable as you mention in a comment but this can be done later on.

Thanks!!! Gonna merge this and rebuild everything on analytics server... and then do more smaller PRs for backporting. Can't wait.


KAFKA-14978 ExactlyOnceWorkerSourceTask should remove parent metrics …

…(#13690)
Reviewers: Chris Egerton<EMAIL_ADDRESS>Viktor Somogyi-Vass<EMAIL_ADDRESS>

LGTM, @viktorsomogyi . I see 3 PRs with more or less the same changes. I am assuming those would be closed?

Thanks @viktorsomogyi . I am assuming there would be a PR against trunk as well(could find only against 3.3, 3.4, 3.5)?

@vamossagar12 yes, trunk and backports to 3.5 and 3.4 have been merged already.

Aah ok. sorry I missed that.


Diagrams do not render correctly (SVG / PNG)

Draw.io offloads PDF rendering to a webservice, https://exp-pdf.draw.io/ImageExport4/export, it may be possible to re-use this - or identify how it works to achieve the same result in-house.
Reproduction steps
http://localhost:8080/?url=https://laingsimon.github.io/render-diagram/Sample file.xml&format=png

actual
expected

POST https://exp-pdf.draw.io/ImageExport4/export HTTP/1.1
Host: exp-pdf.draw.io
Content-Type: application/x-www-form-urlencoded

format=png&allPages=1&bg=none&base64=0&embedXml=0&xml=<url encoded file content>

This method works for PDF & PNG exports, but doesn't support SVG

Java renderer now offloads PNG rendering to the ImageExport4 service however isn't able to do so for svg formatted images.
Options are:

update ImageExport4 via a pull request to JGraph
create a nodejs app to render the images in SVG format where requested - could easily be a fork of the ImageExport4 product

Issue raised to consider the introduction of a svg renderer in ImageExport4

The following is an option:


Issues in authentication (Portugal)

Hi there! I'm trying to put it to work in Portugal but I'm getting this issues:

This is my configuration:
securitas:   username: !secret securitas_username   password: !secret securitas_password   code: !secret securitas_code   country: PT
Thank you!

Thanks for the issue, but I'm going to close the issue because it's related to this one. #52 please follow updates in there.


context deadline exceeded

What did you do?
I was configuring Prometheus for Kong scraping
What did you expect to see?
I expected to see node UP in /targets
What did you see instead? Under which circumstances?
I saw DOWN in /targets with the error:
context deadline exceeded
Environment

System information:
Linux 4.4.0-1088-aws x86_64

Prometheus version:
prometheus, version 2.1.0+ds (branch: debian/sid, revision: 2.1.0+ds-1)
build user<EMAIL_ADDRESS>build date:       20180121-21:30:42
go version:       go1.9.2

Prometheus configuration file:

global:
  scrape_interval:     15s # By default, scrape targets every 15 seconds.
  evaluation_interval: 15s # By default, scrape targets every 15 seconds.
 scrape_configs:
  - job_name: 'prometheus'
    static_configs:
    - targets: ['localhost:9090']
  - job_name: 'node_exporter'
    static_configs:
    - targets: ['localhost:9100','my_public_ip_4:9100','my_public_ip_3:9100']
  - job_name: 'Kong_test'
    metrics_path: /metrics
    static_configs:
    - targets: ['my_public_ip:8001']
  - job_name: 'Kong_test_2'
    metrics_path: /metrics
    static_configs:
    - targets: ['my_public_ip_2:8001']

Logs:

context deadline exceeded

I'm able to reach the endpoint /metrics and it takes like 5s -10s to load over http. I tried to insert a scrape_timeout in global section without any positive effects. I also put a scrape_timeout under job_name section. The aws security group is correctly defined.

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


CLI tutorial Baysor segmentation creates different number of patches

Description
I'm running the CLI tutorial, and when I create patches for Baysor using
sopa patchify baysor tuto.zarr --config-path config.toml --patch-width-microns 1200 --patch-overlap-microns 50
it creates only a single patch, contrary to the 4 patches the tutorial suggests. Where can I actually find the conversion from pixels to microns?
I played around a bit and found that sopa patchify baysor tuto.zarr --config-path config.toml --patch-width-microns 200 --patch-overlap-microns 20 actually gives 4 patches.

Thank you @quentinblampey !

Can I keep asking a question @quentinblampey @Marius1311 So for sopa patchify image before cellpose, it runs --patch-width-pixel 6000 --patch-overlap-pixel 150, so this is based on pixel; whereas running sopa patchify baysor --patch-width-microns 1200 --patch-overlap-microns 50, so this is microns. Does it correct running like this pixel for cellpose patchify; microns for Baysor patchify? Thanks!!!

Yes @KunHHE, this is correct! Indeed, we create patches based on the pixels of the image for cellpose, while, for Baysor, we create patches based on the transcripts (whose coordinates are generally in microns)


could not parse global class "XRToolsPlayerBody"

hi,
i pulled the newest 4.0-dev and added the movement_direct.tscn scene to the right-hand controller. Then when i run it it returns

What do i wrong ?

like here described ( https://docs.godotengine.org/en/stable/tutorials/plugins/editor/installing_plugins.html) , enabled godot-xr-tools plugin removed this error


Windows: untag passing tests

Commit Message: Windows: untag passing tests
Additional Description:
Risk Level: Low
Testing: Enables tests to run on Windows in CI
Docs Changes: N/A
Release Notes: N/A

cc @wrowe @davinci26 @nigriMSFT
See https://github.com/envoyproxy/envoy/pull/11741/commits/e3b82771da042a4f3369b55ff0a61254e5a07438 for tests that are no longer failing we can remove the tag from

Hey folks, windows CI has been really flaky since this landed.  After some discussion amongst the maintainers, I think we're going to roll this back until you folks have root caused the timeouts we have been seeing.  Again I'm happy to help out over on #11904, or you can ping me on slack when you have some logs for me to look at.


Firebase error on IOS

I have been add GoogleService-Info.plist file like this:
ios:
bundleId: "com.example.myappHomolog"
firebase:
config: ".firebase/myapp/GoogleService-Info-MyAppHomolog.plist"
But, when i run flutter run flutter_flavorizr
And try to run my flavor, this error occurred:
Error (Xcode): Unable to load contents of file list: '/Target Support Files/Pods-Runner/Pods-Runner-frameworks-Debug-myapp_homolog-input-files.xcfilelist'

Error (Xcode): Unable to load contents of file list: '/Target Support Files/Pods-Runner/Pods-Runner-frameworks-Debug-myapp_homolog-output-files.xcfilelist'

Could not build the application for the simulator.
Error launching application on iPhone 14 Pro.

Looks like the flavorizr command not is generating necessary files.

@thiaagodev this helped me
https://stackoverflow.com/questions/55505991/xcode-10-2-update-issue-build-system-error-1-unable-to-load-contents-of-file-l/56966495#56966495

Hello @thiaagodev , you can't run the flavorizr process using this command: flutter run flutter_flavorizr. This command doesn't work and produce
Target file "flutter_flavorizr" not found.
How did you get that error?