Path.resolve does not work

Try:
Path.relative("/mydir", "/mydir/file");

Crashes with:
TypeError: Object #<Object> has no method 'resolve'

https://github.com/joyent/node/blob/master/lib/path.js#L504-L530

I was about to fix this by adding missing code, but then I notice that it might just be as simple as changing these to simply resolve vs. exports.resolve: https://github.com/filerjs/filer/blob/develop/src/path.js#L123L124

@gideonthomas any chance you're willing to try testing this in Bramble?  Just change https://github.com/humphd/brackets/blob/bramble/src/bramble/api.js#L28 to filer vs. filer.min and then change https://github.com/filerjs/filer/blob/develop/dist/filer.js#L2072-L2073 to resolve vs. exports.resolve.  If that doesn't fix it, I'll look for a different solution, but I'm pretty sure that's it.

So, I did what you said and it didn't work. But then I tried it in filer standalone itself. When I make the change in dist/filer.js, it still fails. But when I make the change in dist/path.js, it works.


fix:validate args with kosmos join, create ServiceExport/ServiceImport crd with Kosmos install

What type of PR is this?

What does this PR do?

Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

add Fixes #
/cc @ONE7live


IGNITE-21048 System property to configure default Storage Engine

https://issues.apache.org/jira/browse/IGNITE-21048
Thank you for submitting the pull request.
To streamline the review process of the patch and ensure better code quality
we ask both an author and a reviewer to verify the following:
The Review Checklist

[ ] Formal criteria: TC status, codestyle, mandatory documentation. Also make sure to complete the following:
- There is a single JIRA ticket related to the pull request.
- The web-link to the pull request is attached to the JIRA ticket.
- The JIRA ticket has the Patch Available state.
- The description of the JIRA ticket explains WHAT was made, WHY and HOW.
- The pull request title is treated as the final commit message. The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] Design: new code conforms with the design principles of the components it is added to.
[ ] Patch quality: patch cannot be split into smaller pieces, its size must be reasonable.
[ ] Code quality: code is clean and readable, necessary developer documentation is added if needed.
[ ] Tests code quality: test set covers positive/negative scenarios, happy/edge cases. Tests are effective in terms of execution time and resources.

Notes

Apache Ignite Coding Guidelines

Merged to main in commit 67601b5ba7f73d8e3f9bda96dc9cddc4942f49ac.


ERROR: the child emacs had the following errors

When I run esup, it gives such error:
ERROR: the child emacs had the following errors:
  ERROR(profile-sexp) at /home/chenli/.emacs.d/init.el:30 with sexp (unless (bound-and-true-p early-init-file) (load (concat (file-name-directory load-file-name) early-init) nil t)): error=(wrong-type-argument stringp nil)

Results will be incomplete due to errors.

Total User Startup Time: 1.188sec     Total Number of GC Pauses: 37     Total GC Time: 0.473sec

core.el:403  1.022sec   85%
(unless noninteractive
(doom-initialize-modules))

core.el:402  0.157sec   13%
(doom-initialize noninteractive)

core.el:397  0.006sec   0%
(require 'core-lib)

core.el:398  0.003sec   0%
(require 'core-modules)

......

This is my ~/.emacs.d/init.el(without any modification because I nearly config everything in .doom.d/:
(unless (bound-and-true-p early-init-file)
  (load (concat (file-name-directory load-file-name) "early-init")
        nil t))

(require 'core (concat user-emacs-directory "core/core"))

emacs --version
GNU Emacs 27.0.50

doom v
Doom v2.0.9 (Emacs v27.0.50)
Branch: develop
Commit: 37e80c48788f4e01ed582f4442965a37149bef45.

Ah, I suspect esup isn't setting load-file-name. I'll look into this soon.

Hi, any solution?

As of 25d9786 and b149dcf, esup should work again. Please update and give it a try.

Thanks! It works fine now.


Study Cacti nodes as solution to easily create consortia

The Cacti-core package provides interfaces and methods to create Cacti Consortia. We need to explore this to assess if it is a feasible technical basis to implement SATP consortia.
Requirements:

Study packages/cactus-api-client, cactus-cmd-api-server, packages/cactus-common, packages/cactus-core, packages/cactus-core-api.

You have examples of usage in the cactus-test-* packages, for instance packages/cactus-test-cmd-api-server/src/test/typescript/integration; packages/cactus-test-api-client/src/test/typescript/integration/api-client-routing-node-to-node.test.ts

hi

@RafaelAPB I'll try to tackle this one


Different Board/Multiple Board?

For example, I have one board for my work and one for my personal project (Or even multiple board for multiple projects). Right now, everytime I need switch (Ex. Going to office from home) I have to edit the config file to edit the board ID.
I just woundering if its possible to put board ID into command or make it easier to change/switch

OK, I think I found a way to solve it, its little bit hard to do but its possible.
In /bin/bash config, there is a line of code say:
php -f script.php -- "${token};${boardid};{query}";${boardid} this part. I can type boardid manually into Alfred input box and put it before everything
Such as

**board id;trello card name; description
Example: trello Do work!;seriously, do work;red;09/01/2015;Inbox

Do work! - Card title
seriously, do work - Card description
red - Card label
09/01/2015 - Card due date
Inbox - List Name


Setting page info while resizing or cropping page

Is it possible with Stirling-PDF to redefine the MediaBox, CropBox, BleedBox, TrimBox and ArtBox?
Possible use case:
Having a 40x50cm poster template PDF. This poster should be printed 80x100cm, so with the scale factor 2.
But the tricky part is the bleed: The printer only accepts a 80x100cm sized PDF document with 5mm bleed on each side.
Any idea leads on this? Thank you!

Not currently

This is very niche of a usecase suitable more for a PDF API not a PDF application
Will be closing for now!


Add test input java classes to jandex repository

Currently, it is hard to write tests since jandex repository defaults to source=target=1.6, and the test code that uses type annotations just does not compile.
It would be nice if there was a way to add test classes right into jandex repository (e.g. via Maven toolchains or Gradle toolchains)

For the moment you can add java 8 test classes here: https://github.com/wildfly/typeannotation-test
It has come time to move to 8, but this will probably mean declaring a 3.0 version.

For the moment you can add java 8 test classes here: https://github.com/wildfly/typeannotation-test
It has come time to move to 8, but this will probably mean declaring a 3.0 version.

For the moment you can add java 8 test classes here: https://github.com/wildfly/typeannotation-test

It feels weird to add classes in one place and debug them in another :-/
That would be super-weird for PR purposes as well.
I guess different modules can have different source/target versions if you still need to support Java 6/7.
Have you explored that possibility?
However, jandex issues hit Apache Calcite badly, and I guess it might be even easier to fix jandex than implement workarounds in Calcite :-/
On the other hand, I have no preference regarding  6-7-8 (Calcite is Java 8+ anyway), so if you migrate jandex to Java8, then it would indeed help with testing.

For the moment you can add java 8 test classes here: https://github.com/wildfly/typeannotation-test

It feels weird to add classes in one place and debug them in another :-/
That would be super-weird for PR purposes as well.
I guess different modules can have different source/target versions if you still need to support Java 6/7.
Have you explored that possibility?
However, jandex issues hit Apache Calcite badly, and I guess it might be even easier to fix jandex than implement workarounds in Calcite :-/
On the other hand, I have no preference regarding  6-7-8 (Calcite is Java 8+ anyway), so if you migrate jandex to Java8, then it would indeed help with testing.

For the moment you can add java 8 test classes here: https://github.com/wildfly/typeannotation-test

It feels weird to add classes in one place and debug them in another :-/
That would be super-weird for PR purposes as well.

The test cases you debug are only expected to be in the main jadex project, it's just the input byteocde that is elsewhere,. The pattern is like this since the project is forward bytecode compatible (it can read newer bytecode than the JVM it runs on. E.g you can read Java 11 bytecode running on a Java 6 JVM).  I agree its unusual, and means you have to build install in two separate steps.
.

I guess different modules can have different source/target versions if you still need to support Java 6/7.
Have you explored that possibility?

Historically, the only reason was to avoid requiring contributors to have extra ultra new JVMs around. So for example if we added early support for a pre-release of say Java 15, but you just wanted to fix something for the current stable VMs its a bit of a PITA.

However, jandex issues hit Apache Calcite badly, and I guess it might be even easier to fix jandex than implement workarounds in Calcite :-/

sorry about that!

On the other hand, I have no preference regarding 6-7-8 (Calcite is Java 8+ anyway), so if you migrate jandex to Java8, then it would indeed help with testing.

For the moment you can add java 8 test classes here: https://github.com/wildfly/typeannotation-test

It feels weird to add classes in one place and debug them in another :-/
That would be super-weird for PR purposes as well.

The test cases you debug are only expected to be in the main jadex project, it's just the input byteocde that is elsewhere,. The pattern is like this since the project is forward bytecode compatible (it can read newer bytecode than the JVM it runs on. E.g you can read Java 11 bytecode running on a Java 6 JVM).  I agree its unusual, and means you have to build install in two separate steps.
.

I guess different modules can have different source/target versions if you still need to support Java 6/7.
Have you explored that possibility?

Historically, the only reason was to avoid requiring contributors to have extra ultra new JVMs around. So for example if we added early support for a pre-release of say Java 15, but you just wanted to fix something for the current stable VMs its a bit of a PITA.

However, jandex issues hit Apache Calcite badly, and I guess it might be even easier to fix jandex than implement workarounds in Calcite :-/

sorry about that!

On the other hand, I have no preference regarding 6-7-8 (Calcite is Java 8+ anyway), so if you migrate jandex to Java8, then it would indeed help with testing.

This is now done on the smallrye branch.


Switch to use Mopidy-Local-SQLite

We should switch to using the Mopidy-Local-SQLite (instead of the built-in JSON local backend) to benefit from it's improved speed and features. In particular it's better support for large music databases and it's ability to automatically reload and use the latest scan result without restarting Mopidy.

This is done and 0.7 uses latest version.


Formatting of mmol/L eventualBG: two decimal places in app and on watch

I propose to reduce to one decimal place for both.

Possible fix for FAX app:
https://github.com/bjornoleh/freeaps/commit/a1507d7f5ca158bf0d887009239335acb3268108
Possible fix for watch app:
https://github.com/bjornoleh/freeaps/commit/4a5dbf1af36eeb4c8500d2fcb3450a075003d2b4
Closing this issue with the above hints for anyone else who might want to modify their fork of FAX.


Design/Develop GraphIR - XACC intermediate representation

We need a general intermediate representation in XACC that all compiler implementations produce. This IR should take on the form of a graph (nodes and associated edges) since our post-exascale accelerator machine instructions all take some graph representation as input (graph of tensors for gate model quantum, graph of ising parameters for adiabatic model quantum, neural network for neuromorphic computing)


Update screenshots for contact page with map plugin

The image /user/tutorial/images/structure-board-with-two-plugins.png needs to be changed, it shows lorem ipsum content.


Secretsmanager

Apologies for the large commit, but I didn't think it would be feasible to break this into smaller parts and still ensure it worked end-to-end.
There are two main things here:

The get_credentials function. This simply interacts with secretsmanager, intended to be called within the lambda.
An outline document for the critical requirements to make this work using GitHub secrets and terraform (documentation/secrets.md).

documentation/secrets.md makes reference to credentials on lines 14 -> 21
Are these real credentials?

You're right they are real credentials. Updated now. If possible might be worth squashing this branch on merge to mask them.


Fix back button issue with deeply nested routers

Coverage decreased (-0.5%) to 92.178% when pulling 7aae3d31383dce09bc373b215564a12d815888c8 on fix-back-button into ba4dcd17b615f47597d9015c0d6d1517ff15261c on master.


Set package manager versions

Description:
Added support for specifying package manager versions in the workflow file just as specifying node version.
This only added support for the package managers that are supported by this GitHub action and in this case these are npm, pnpm, yarn.
Related issue:
#529
Check list:

[ ] Mark if documentation changes are required.
[ ] Mark if tests were added or updated to cover the changes.

@gordey4doronin, how do we spec the version of the package manager to be consumed?


Duplicated declaration

Affected Puppet, Ruby, OS and module versions/distributions

Puppet:4.10.12
Ruby: ruby 2.0.0p648 (2015-12-16) [x86_64-linux]
Distribution: CentOS Linux release 7.5.1804
Module version: latest in master branch

How to reproduce (e.g Puppet code you use)
Just running puppet agent -t
What are you seeing

root@ip-10-189-229-189 ~]# puppet agent -t
Info: Using configured environment 'staging'
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Function Call, Duplicate declaration: Jenkins::Plugin[structs] is already declared in file /etc/puppetlabs/code/environments/staging/modules/jenkins/manifests/config.pp:9; cannot redeclare at /etc/puppetlabs/code/environments/staging/modules/jenkins/manifests/plugins.pp:9 at /etc/puppetlabs/code/environments/staging/modules/jenkins/manifests/plugins.pp:9:3 on node ip-10-189-229-189.internal
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run

What behaviour did you expect instead
Error free behaviour
Output log
Any additional information you'd like to impart

Found the bug in my code.
Thanks.

@isurugiu What was your bug?
I think I have the same one.


fix xref link errors

@tdykstra Curious ... Splitting the pattern out was approved back when it was discussed. It was decided internally to pull it?

@guardrex The Repository Pattern is covered in one of the eBooks over in dotnet/docs, so users will be redirected there if they try to view the old link.

Yes ... I see what you mean, but I don't think that the eBook coverage is nearly as good as what we had (for a beginner anyway). However, I worked on that topic 😅, so I can't objectively say.
Anyway ... It was just a nice topic imo (and MS paid 💰 for it), so I just hate to see that work vaporize. No worries. :+1:


feat: change type of class MediaUrlPlaceholderHandler

1. Why is this change necessary?

2. What does this change do, exactly?
Use the correct abstract class
3. Describe each step to reproduce the issue or behaviour.
Decorate the class, see crashing. https://github.com/FriendsOfShopware/FroshPlatformThumbnailProcessor/blob/main/src/Core/Media/MediaUrlGenerator.php#L11
4. Please link to the relevant issues (if any).
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

thanks for your contribution @tinect 👍🎉💙


7pm-3 at and jc - Fixed SearchByUser error when you are admin but not member

In this PR, team 7pm-3, we fixed the Search Messages by User by allowing to search messages on the Mapache app if you are a member OR an admin.

LGTM


Point to current repository in package.json

Having package.json point at an abandoned repository is going to trip people up.
(It'd also be good if the gitlab repository could come with a big warning about the project having moved here.)

@Rich-Harris Please merge this and stop leading people to the wrong repository!

Thanks! Have updated the README on the other repo


webview_bind does not take effect until next page load

What OS are you using (uname -a, or Windows version)?
$ uname -mrs
Linux 5.15.32_1 x86_64

What programming language are you using (C/C++/Go/Rust)?
$ clang --version
clang version 12.0.1
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /bin

What did you expect to see and what you saw instead?
When calling webview_bind I would expect that the function is immediately bound (using webview_eval to run the RPC init code), like how webview_unbind immediately unbinds functions, however the function is not bound until the next time the page is loaded.
The following example illustrates the issue - it should print bound \n Hello, world! but instead only prints bound
#include <stddef.h>
#include <stdio.h>
#include "webview/webview.h"

void say_hi(const char *seq, const char *req, void *arg) {
	puts("Hello, world!");
	webview_t w = arg;
	webview_return(w, seq, 0, "undefined");
}

void do_bind(const char *seq, const char *req, void *arg) {
	webview_t w = arg;
	webview_bind(w, "sayHi", say_hi, w);
	puts("bound");
	webview_return(w, seq, 0, "undefined");
}

int main() {
	webview_t w = webview_create(0, NULL);
	webview_bind(w, "doBind", do_bind, w);
	webview_set_html(w, "<script>doBind().then(() => sayHi());</script>");
	webview_run(w);
	webview_destroy(w);
}

The fix is simple - I can make a PR if that's easier but figured it'd be simpler to let a maintainer apply the patch:
diff --git a/webview.h b/webview.h
index 9de30e7..f549e15 100644
--- a/webview.h
+++ b/webview.h
@@ -1169,6 +1169,7 @@ public:
       }
     })())";
     init(js);
+    eval(js);
     bindings[name] = new binding_ctx_t(new binding_t(f), arg);
   }

I'm away from my computer, so a PR means you get the credit and I can click a button, but I don't mind making a one line change. Good catch - thanks!

Made a PR :)


Run-AlValidation produces warning unrelated to Extension being validated

Describe the issue
When running Run-AlValidation there is a warning output shown after compiling the extension as below
C:\ProgramData\BcContainerHelper\be303062-fac0-4094-8147-49eeb845071a\app.json(16,5): warning AL0667: 'Extension' is being deprecated in the versions: '4.0' or greater. The Extension compilation target  is being deprecated. Use Cloud instead. This warning will become an error in a future release.
I managed to capture the app.json in question during the Run-AlValidation.  It has all the key values from the extensions actual app.json but isn't a direct copy as it has additional fields that are not in the version of app.json I have in the repo used to create the .app file.
One of those additional fields is line 16
"target":  "Extension",
I can't figure out where that entry is coming from.  Does Run-AlValidation process generate the app.json itself?
Scripts used to create container and cause the issue
Run-AlValidation `
    -apps @("https://...preview/latest/apps.zip") `
    -affixes @("AAA") `
    -countries @("AU")

Additional context

does it happen all the time? Yes
did it use to work? Not sure, I noticed it while trawling the output of Run-AlValidation and went looking to fix the error in my app.json but couldn't see any instance of "target":  "Extension",

At this stage I'm more just wondering if app.json is generated by Run-AlValidation.  Can provide more detailed outputs if necessary, will attempt to recreate in a spike app.

Yeah, so Run-AlValidation uses Extract-AppFileToFolder with -generateAppJson to generate the app.json file based on the NavxManifest.xml. My NavxManifest has target = cloud

hence my app.json gets:

The code transferring the value to app.json is straightforward:

No translation is going on.
So, somehow this comes from your .app file - which must have target=extension - whether the compiler uses extension if your runtime or application has a specific version, I don't know.

OK, thanks for that.  I've done some addtional testing with a Hello World app from AL: GO! I suspect what might be happening is that when app.json does not contain a value for "target" it gets added and given a default value of "extension".
I created a new app using AL: GO! and accepted all the defaults.  App.json does not contain "target" entry (ms docs list it as optional).  Compiling this app in VS Code gave the following message
Microsoft (R) AL Compiler version <IP_ADDRESS>27
Copyright (C) Microsoft Corporation. All rights reserved

Compilation started for project 'TestAppJSONValidation' containing '1' files at '14:35:51.567'.

Compilation ended at '14:35:51.606'.

Success: The package is created.

When passing the app to Run-AlValidation the compile section shows the following warning
Microsoft (R) AL Compiler version <IP_ADDRESS>27
Copyright (C) Microsoft Corporation. All rights reserved

C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(16,5): warning AL0667: 'Extension' is being deprecated in the versions: '4.0' or greater. The Extension compilation target
is being deprecated. Use Cloud instead. This warning will become an error in a future release.
Compilation started for project 'TestAppJSONValidation' containing '1' files at '14:37:44.065'.

If I manually add a "target" entry to the app.json file with a value of "cloud" the error goes away during Run-AlValidation (which is expected).  If I leave it empty or with a value equal to "target": "" then RunAlValidation raises the same AL0667 warning (though vs code compilation still succeeds with no warning shown).
Full output of Run-AlValidation follows.. Search for " warning AL0667" to see the warning I'm referring to.
 If you experience problems, add -isolation hyperv.
Using process isolation
Using locale en-AU
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Downloading C:\ProgramData\BcContainerHelper\Extensions\bcserver\Microsoft.VisualStudio.Services.VSIXPackage.vsix
Additional Parameters:
--env customNavSettings=EnableTaskScheduler=True
Files in C:\ProgramData\BcContainerHelper\Extensions\bcserver\my:
- AdditionalOutput.ps1
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container bcserver from image mcr.microsoft.com/businesscentral:10.0.19041.1415-filesonly
4fe8d117299b51594d1f7c53c45b560dbd9dd347bf8fea4b13f956fd96170787
Waiting for container bcserver to be ready
Using artifactUrl https://bcartifacts.azureedge.net/sandbox/20.4.44313.45632/au
Using installer from C:\Run\150-new
Installing Business Central
Installing from artifacts
Copying Service Tier Files
c:\dl\sandbox\20.4.44313.45632\platform\ServiceTier\Program Files
c:\dl\sandbox\20.4.44313.45632\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\sandbox\20.4.44313.45632\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\sandbox\20.4.44313.45632\platform\WindowsPowerShellScripts\WebSearch
Copying Web Client Files
c:\dl\sandbox\20.4.44313.45632\platform\WebClient\Microsoft Dynamics NAV
Copying Client Files
c:\dl\sandbox\20.4.44313.45632\platform\LegacyDlls\program files\Microsoft Dynamics NAV
c:\dl\sandbox\20.4.44313.45632\platform\LegacyDlls\program files\Microsoft Dynamics NAV
c:\dl\sandbox\20.4.44313.45632\platform\LegacyDlls\systemFolder
Copying ModernDev Files
c:\dl\sandbox\20.4.44313.45632\platform
c:\dl\sandbox\20.4.44313.45632\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\sandbox\20.4.44313.45632\au\ConfigurationPackages
Copying Test Assemblies
C:\dl\sandbox\20.4.44313.45632\platform\Test Assemblies
Copying Extensions
C:\dl\sandbox\20.4.44313.45632\au\Extensions
Copying Applications
C:\dl\sandbox\20.4.44313.45632\platform\Applications
Copying Applications.AU
C:\dl\sandbox\20.4.44313.45632\au\Applications.AU
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Skipping restore of Cronus database
Modifying Business Central Service Tier Config File for Docker
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Installation took 21 seconds
Installation complete
Ready for connections!
Reading CustomSettings.config from bcserver

Creating container took 46 seconds
  _____                   _                                    _____                           _____
 |  __ \                 (_)                 /\               / ____|                         / ____|
 | |__) |   _ _ __  _ __  _ _ __   __ _     /  \   _ __  _ __| (___   ___  _   _ _ __ ___ ___| |     ___  _ __
 |  _  / | | | '_ \| '_ \| | '_ \ / _` |   / /\ \ | '_ \| '_ \\___ \ / _ \| | | | '__/ __/ _ \ |    / _ \| '_ \
 | | \ \ |_| | | | | | | | | | | | (_| |  / ____ \| |_) | |_) |___) | (_) | |_| | | | (__  __/ |____ (_) | |_) |
 |_|  \_\__,_|_| |_|_| |_|_|_| |_|\__, | /_/    \_\ .__/| .__/_____/ \___/ \__,_|_|  \___\___|\_____\___/| .__/
                                   __/ |          | |   | |                                              | |
                                  |___/           |_|   |_|                                              |_|
Default publisher_TestAppJSONValidation_<IP_ADDRESS>.app is not signed, result is NotSigned
Extracting C:\ProgramData\BcContainerHelper\1b391896-d36a-4f57-9191-8d96151bc66e\Default publisher_TestAppJSONValidation_<IP_ADDRESS>.app
Analyzing: Default publisher_TestAppJSONValidation_<IP_ADDRESS>.app
Using affixes:
Downloading C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\appsource.default.ruleset.json
AppSourceCop.json content:
{
    "mandatoryAffixes":  [

                         ]
}
Ruleset.json content:
{
    "name":  "Run-AlCops RuleSet",
    "description":  "Generated by Run-AlCops",
    "includedRuleSets":  [
                             {
                                 "path":  "C:\\ProgramData\\BcContainerHelper\\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\\appsource.default.ruleset.json",
                                 "action":  "Default"
                             }
                         ]
}
Using Symbols Folder: C:\ProgramData\BcContainerHelper\6c7d35b3-6d4e-4f8e-8264-483fc6af859e
Copying System.app from Container
Copying Microsoft_AMC Banking 365 Fundamentals_20.4.44313.45632.app from Container
Copying Microsoft_Any_20.4.44313.45632.app from Container
Copying Microsoft_Application_20.4.44313.45632.app from Container
Copying Microsoft_Base Application_20.4.44313.45632.app from Container
Copying Microsoft_Basic Experience_20.4.44313.45632.app from Container
Copying Microsoft_Business Central Cloud Migration - Previous Release (AU)_20.4.44313.45632.app from Container
Copying Microsoft_Business Central Cloud Migration - Previous Release_20.4.44313.45632.app from Container
Copying Microsoft_Business Central Cloud Migration API_20.4.44313.45632.app from Container
Copying Microsoft_Business Central Intelligent Cloud_20.4.44313.45632.app from Container
Copying Microsoft_Company Hub_20.4.44313.45632.app from Container
Copying Microsoft_Contoso Coffee Demo Dataset (AU)_20.4.44313.45632.app from Container
Copying Microsoft_Contoso Coffee Demo Dataset_20.4.44313.45632.app from Container
Copying Microsoft_Czech language (Czechia)_20.4.44313.45632.app from Container
Copying Microsoft_Danish language (Denmark)_20.4.44313.45632.app from Container
Copying Microsoft_Data Archive_20.4.44313.45632.app from Container
Copying Microsoft_Dutch language (Belgium)_20.4.44313.45632.app from Container
Copying Microsoft_Dutch language (Netherlands)_20.4.44313.45632.app from Container
Copying Microsoft_Dynamics GP History SmartLists_20.4.44313.45632.app from Container
Copying Microsoft_Dynamics GP Intelligent Cloud_20.4.44313.45632.app from Container
Copying Microsoft_Email - Current User Connector_20.4.44313.45632.app from Container
Copying Microsoft_Email - Microsoft 365 Connector_20.4.44313.45632.app from Container
Copying Microsoft_Email - Outlook REST API_20.4.44313.45632.app from Container
Copying Microsoft_Email - SMTP API_20.4.44313.45632.app from Container
Copying Microsoft_Email - SMTP Connector_20.4.44313.45632.app from Container
Copying Microsoft_English language (Australia)_20.4.44313.45632.app from Container
Copying Microsoft_English language (Canada)_20.4.44313.45632.app from Container
Copying Microsoft_English language (New Zealand)_20.4.44313.45632.app from Container
Copying Microsoft_English language (United Kingdom)_20.4.44313.45632.app from Container
Copying Microsoft_English language (United States)_20.4.44313.45632.app from Container
Copying Microsoft_Essential Business Headlines_20.4.44313.45632.app from Container
Copying Microsoft_Finnish language (Finland)_20.4.44313.45632.app from Container
Copying Microsoft_French language (Belgium)_20.4.44313.45632.app from Container
Copying Microsoft_French language (Canada)_20.4.44313.45632.app from Container
Copying Microsoft_French language (France)_20.4.44313.45632.app from Container
Copying Microsoft_French language (Switzerland)_20.4.44313.45632.app from Container
Copying Microsoft_German language (Austria)_20.4.44313.45632.app from Container
Copying Microsoft_German language (Germany)_20.4.44313.45632.app from Container
Copying Microsoft_German language (Switzerland)_20.4.44313.45632.app from Container
Copying Microsoft_Icelandic language (Iceland)_20.4.44313.45632.app from Container
Copying Microsoft_Image Analyzer_20.4.44313.45632.app from Container
Copying Microsoft_Intelligent Cloud Base_20.4.44313.45632.app from Container
Copying Microsoft_Italian language (Italy)_20.4.44313.45632.app from Container
Copying Microsoft_Italian language (Switzerland)_20.4.44313.45632.app from Container
Copying Microsoft_Late Payment Prediction_20.4.44313.45632.app from Container
Copying Microsoft_Library Assert_20.4.44313.45632.app from Container
Copying Microsoft_Library Variable Storage_20.4.44313.45632.app from Container
Copying Microsoft_Microsoft Pay Payments_20.4.44313.45632.app from Container
Copying Microsoft_Norwegian language (Norway)_20.4.44313.45632.app from Container
Copying Microsoft_Payment Links to PayPal_20.4.44313.45632.app from Container
Copying Microsoft_Performance Toolkit_20.4.44313.45632.app from Container
Copying Microsoft_Permissions Mock_20.4.44313.45632.app from Container
Copying Microsoft_Recommended Apps_20.4.44313.45632.app from Container
Copying Microsoft_Russian language (Russia)_20.4.44313.45632.app from Container
Copying Microsoft_Sales and Inventory Forecast_20.4.44313.45632.app from Container
Copying Microsoft_Send remittance advice by email_20.4.44313.45632.app from Container
Copying Microsoft_Send To Email Printer_20.4.44313.45632.app from Container
Copying Microsoft_Shopify Connector_20.4.44313.45632.app from Container
Copying Microsoft_Simplified Bank Statement Import_20.4.44313.45632.app from Container
Copying Microsoft_Spanish language (Mexico)_20.4.44313.45632.app from Container
Copying Microsoft_Spanish language (Spain)_20.4.44313.45632.app from Container
Copying Microsoft_Swedish language (Sweden)_20.4.44313.45632.app from Container
Copying Microsoft_System Application_20.4.44313.45632.app from Container
Copying Microsoft_Test Runner_20.4.44313.45632.app from Container
Copying Microsoft_Troubleshoot FA Ledger Entries_20.4.44313.45632.app from Container
Copying Microsoft_Universal Print Integration_20.4.44313.45632.app from Container
Copying Microsoft_WorldPay Payments Standard_20.4.44313.45632.app from Container
Copying Microsoft__Exclude_AnonymizedDataSharing__20.4.44313.45632.app from Container
Copying Microsoft__Exclude_APIV1__20.4.44313.45632.app from Container
Copying Microsoft__Exclude_APIV2__20.4.44313.45632.app from Container
Copying Microsoft__Exclude_Bank Deposits_20.4.44313.45632.app from Container
Copying Microsoft__Exclude_ClientAddIns__20.4.44313.45632.app from Container
Copying Microsoft__Exclude_Email Logging Using Graph API_20.4.44313.45632.app from Container
Copying Microsoft__Exclude_Microsoft Dynamics 365 - SmartList_20.4.44313.45632.app from Container
Copying Microsoft__Exclude_PlanConfiguration__20.4.44313.45632.app from Container
Copying Microsoft__Exclude_ReportLayouts_20.4.44313.45632.app from Container
Copying Microsoft__Exclude_SyncBaseApp__20.4.44313.45632.app from Container
INFO: You have to specify AuthContext and Environment if you are compiling in a filesOnly container in order to download dependencies
Compiling...
.\alc.exe /project:"C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375" /packagecachepath:"C:\ProgramData\BcContainerHelper\6c7d35b3-6d4e-4f8e-8264-483fc6af859e" /out:"C:\ProgramData\
BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\Default publisher_TestAppJSONValidation_<IP_ADDRESS>.app" /GenerateReportLayout- /analyzer:C:\build\vsix\extension\bin\Analyzers\Microsoft.Dynamics.Nav.
AppSourceCop.dll /ruleset:C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\ruleset.json /reportsuppresseddiagnostics /assemblyprobingpaths:"C:\Program Files (x86)\Microsoft Dynamics
 NAV\200\RoleTailored Client","C:\Program Files\Microsoft Dynamics NAV\200\Service","C:\Program Files (x86)\Open XML SDK\V2.5\lib","c:\Windows\Microsoft.NET\Assembly","C:\Test Assemblies\Mock Assemblies
"
Microsoft (R) AL Compiler version <IP_ADDRESS>27
Copyright (C) Microsoft Corporation. All rights reserved

C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(16,5): warning AL0667: 'Extension' is being deprecated in the versions: '4.0' or greater. The Extension compilation target
is being deprecated. Use Cloud instead. This warning will become an error in a future release.
Compilation started for project 'TestAppJSONValidation' containing '1' files at '14:37:44.065'.

C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(1,1): warning AS0092: The app.json file must specify an Azure Application Insights resource with the property 'applicationI
nsightsConnectionString' for monitoring operations related to this extension. See https://docs.microsoft.com/dynamics365/business-central/dev-itpro/administration/telemetry-overview for additional infor
mation.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(6,5): error AS0051: The manifest property 'brief' must be specified and contain a meaningful value.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(7,5): error AS0051: The manifest property 'description' must be specified and contain a meaningful value.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(14,5): error AS0051: The manifest property 'privacyStatement' must be specified and contain a meaningful value.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(13,5): error AS0051: The manifest property 'EULA' must be specified and contain a meaningful value.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(15,5): error AS0051: The manifest property 'help' must be specified and contain a meaningful value.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(11,5): error AS0051: The manifest property 'logo' must be specified and contain a meaningful value.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(1,1): error AS0051: The manifest property 'contextSensitiveHelpUrl' must be specified and contain a meaningful value.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(23,5): error AS0084: The ID range '[50100..50149]' is not valid. It must be within the range allocated to the partner for A
ppSource, within the range '[1000000..75999999]' allocated to AppSource applications, and outside the range '[50000..99999]' allocated to per-tenant customizations.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(29,5): error AS0015: The "TranslationFile" flag must be added to the "features" array in the app.json file.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\app.json(12,5): error AS0052: The property 'url' must be set to a valid URL.
C:\ProgramData\BcContainerHelper\b5bfbe2e-6a47-4587-9a80-eb3ea06f2375\AppSourceCop.json(1,1): error AS0054: The AppSourceCop configuration must specify one of the following properties: 'mandatorySuffix'
, 'mandatoryPrefix', or 'mandatoryAffixes'

Compilation ended at '14:37:50.208'.

 It must be within the range allocated to the partner for AppSource, within the range '[1000000..75999999]' allocated to AppSourc
e applications, and outside the range '[50000..99999]' allocated to per-tenant customizations.
app.json(29,5): error AS0015: The "TranslationFile" flag must be added to the "features" array in the app.json file.
app.json(12,5): error AS0052: The property 'url' must be set to a valid URL.
AppSourceCop.json(1,1): error AS0054: The AppSourceCop configuration must specify one of the following properties: 'mandatorySuffix', 'mandatoryPrefix', or 'mandatoryAffixes'
App generation failed with exit code 1

  _____                          ___      __   _ _     _       _   _               ______    _ _
 |  __ \                   /\   | \ \    / /  | (_)   | |     | | (_)             |  ____|  (_) |
 | |__) |   _ _ __ ______ /  \  | |\ \  / /_ _| |_  __| | __ _| |_ _  ___  _ __   | |__ __ _ _| |_   _ _ __ ___
 |  _  / | | | '_ \______/ /\ \ | | \ \/ / _` | | |/ _` |/ _` | __| |/ _ \| '_ \  |  __/ _` | | | | | | '__/ _ \
 | | \ \ |_| | | | |    / ____ \| |  \  / (_| | | | (_| | (_| | |_| | (_) | | | | | | | (_| | | | |_| | | |  __/
 |_|  \_\__,_|_| |_|   /_/    \_\_|   \/ \__,_|_|_|\__,_|\__,_|\__|_|\___/|_| |_| |_|  \__,_|_|_|\__,_|_|  \___|

13 errors found in Default publisher_TestAppJSONValidation_<IP_ADDRESS>.app on https://bcartifacts.azureedge.net/sandbox/20.4.44313.45632/au:
Default publisher_TestAppJSONValidation_<IP_ADDRESS>.app is not signed, result is NotSigned
app.json(6,5): error AS0051: The manifest property 'brief' must be specified and contain a meaningful value.
app.json(7,5): error AS0051: The manifest property 'description' must be specified and contain a meaningful value.
app.json(14,5): error AS0051: The manifest property 'privacyStatement' must be specified and contain a meaningful value.
app.json(13,5): error AS0051: The manifest property 'EULA' must be specified and contain a meaningful value.
app.json(15,5): error AS0051: The manifest property 'help' must be specified and contain a meaningful value.
app.json(11,5): error AS0051: The manifest property 'logo' must be specified and contain a meaningful value.
app.json(1,1): error AS0051: The manifest property 'contextSensitiveHelpUrl' must be specified and contain a meaningful value.
app.json(23,5): error AS0084: The ID range '[50100..50149]' is not valid. It must be within the range allocated to the partner for AppSource, within the range '[1000000..75999999]' allocated to AppSourc
e applications, and outside the range '[50000..99999]' allocated to per-tenant customizations.
app.json(29,5): error AS0015: The "TranslationFile" flag must be added to the "features" array in the app.json file.
app.json(12,5): error AS0052: The property 'url' must be set to a valid URL.
AppSourceCop.json(1,1): error AS0054: The AppSourceCop configuration must specify one of the following properties: 'mandatorySuffix', 'mandatoryPrefix', or 'mandatoryAffixes'
App generation failed with exit code 1

PS C:\WINDOWS\system32>

I can confirm that I can repro this.

Will report to the ModernDev team - I think that is their bug.

Please report on microsoft/AL and we'll take care of it :)


Quantity of Product doesn't get updated after creating a refund.

Bug report
Title
Quantity of Product doesn't get updated after creating a refund.
Issue Description
Product Quantity not updated after refund is placed.

Preconditions

framework Version-0.1.7

Steps to reproduce
1. step1-    Place and order and generate invoice and shipment for that order

2. step2 - Create refund for that order and check the inventory.

Expected result
Product Quantity should get updated

Actual result
  Product Quantity doesn't updated

Framework Version: 0.1.7

Fixed

Framework Version v0.1.8
still same issue for bundle product.

Framework Version v0.1.8
issue fixed


remove points after add?

Hi,
will it be possible to remove points as well in the future? Afaik no (kd|...)tree library for Rust does allow it. Would be nice to have.
Thanks

Hey I'm not actively working on this crate. If you're interested in submitting a PR, that'd be great!


Ripple in Tabs stuck when double clicked.

On any tabs with ripple, a double click would cause the ripple to stuck in place and won't fade away until the cursor moves out of the area.
Tested on latest Chrome and Safari on OS X El Capitan.

The core team is focusing on V2 work and only critical internal issues are going to be handled for V1. If someone from the community would like to contribute a patch here, mdl-1.x is open for pull requests.
Thank you.


Test About OpenImages

Dear Authors,
I have trained the model based on BDD. And I have evaluated the model on the COCO successfully. Now I use the same model to evaluate openimages. There is an error.

The command is:
CUDA_VISIBLE_DEVICES=4 python apply_net.py --dataset-dir /data1/user/vos/OpenImages --test-dataset openimages_ood_val --config-file BDD-Detection/faster-rcnn/vos.yaml --inference-config Inference/standard_nms.yaml --random-seed 0 --image-corruption-level 0 --visualize 0

hi, @AmingWu , the evaluation on openimages is not supported in the original version, i have updated the code, can you try again?
thanks for catching the bug!

There is an error. I only update evaluation_utils.py and keep other files unchanged

.

For the two files in OpenImages, i.e., coco_classes and ood_classes_rm_overlap, they all contain the file COCO-Format, could you give the accurate command about the evaluation on OpenImages? This is an another error:

Yes, the second step works. However, How to perform the last step to obtain the results of FPR95 and AUROC for OpenImages?

The current project may lose the file, i.e., bdd_openimages_plot.py and voc_openimages_plot.py.

can you please make these files yourself, I think it is very easy and you just change the folder name, right?

btw, thanks for catching these bugs on evaluation for OpenImages. Very cool! Thank you so much!

Thank you. I have solved this problem.


Remote Video Track Recording

Help Request
Related Issues: #123 and #286
Scenario:
Local user starts up a room with a VP8 Simulcast Video Track with a remote user joining the room with the same connection settings.  Local user wants to record the incoming video track. The remote user will start up a 2nd video track in which the local user will record into an MP4. The 2nd video track will not be viewed, it will all happen in the background with a notification going to the remote user to accept/deny the video recording.  Currently both users will have a main video track that ensures high quality using the VP8 connection.  The 2nd optional video track is created and destroyed on command.  While the main video track keeps a high quality connection by having a reduced frame rate or frame size, the 2nd video track can be dynamic.
There are two paths:

According to #286,  a custom render is created for the purpose of saving the video frames from the 2nd video track.  However, it seems that I420 isn't supported.  Using #123 as an example, a user was able to convert I420 into NV12 which then can be saved to the MP4 container (Would a MOV file also work?).   But this process seems overly complex with a lot of overhead. Even in putting libyuv into the project seems a bit complex for a single function call. Since that issue was from a couple of years ago, is there any updates on this conversion taking place in Twilio? Is this functionality exposed in any way in from the Twilio SDK?

If the 2nd video track can be configured to use the H264 codec, then using the custom render from #286 would work out of the box.  I don't see a way to force the 2nd video track to take on a different codec as the room was setup using the VP8 codec.  Is there a way to configure the codec of the video track without doing anything to the room settings or interfering with the 1st video track?

Hi @astaranowicz,
We haven't rushed to support this use case because recording on the subscriber side results in the lowest quality video (compared to the media server or publisher perspective). If you want to manage the I420 format conversion then you can record remote video (please be sure to get user consent of course).
If you are using Vp8 simulcast then you must be in a Group Room and I would recommend our Recordings API to record the Vp8 video track on the media server. Server recordings are going to be better quality than subscriber recordings since the server always has the best simulcast layers and doesn't transcode the video. Your subscriber might experience packet loss that would be missing in the server recording, and there is no guarantee that you are receiving the top quality layers (you can use a bandwidth profile + high priority to make high quality more likely).

According to #286, a custom render is created for the purpose of saving the video frames from the 2nd video track. However, it seems that I420 isn't supported. Using #123 as an example, a user was able to convert I420 into NV12 which then can be saved to the MP4 container (Would a MOV file also work?). But this process seems overly complex with a lot of overhead. Even in putting libyuv into the project seems a bit complex for a single function call. Since that issue was from a couple of years ago, is there any updates on this conversion taking place in Twilio? Is this functionality exposed in any way in from the Twilio SDK?

No we don't support converting from I420 -> NV12 natively. If you can stand some extra memory and CPU use then vImageConvert_420Yp8_Cb8_Cr8ToARGB8888 is built in and can convert I420 -> BGRA for AVAssetWriter with no 3rd party code.

If the 2nd video track can be configured to use the H264 codec, then using the custom render from #286 would work out of the box. I don't see a way to force the 2nd video track to take on a different codec as the room was setup using the VP8 codec. Is there a way to configure the codec of the video track without doing anything to the room settings or interfering with the 1st video track?

Setting codec preferences on a per-Track basis isn't supported yet so I wouldn't recommend this approach. Doing a format conversion is way cheaper than encoding, sending and decoding a second video track.
Best,
Chris

Great, Thank you for the awesome feedback and insight into Twilio SDK.
Do you have an example to share for the Recordings API?
I have been looking for non-merged pull requests and other items through the issues and github. I haven't seen this use-case in any of the examples.
Best,
Aaron


Remove AMP URL API Wrapper

no longer needed as API now supports CORS
rename view folder -> iframe
increase result view size to match default result
API key no longer needs to be encrypted as its access controlled via request host

@kul3r4 @juliantoledo PAL

LGTM.


Update gfm.js

Remove SHA hash from being treated specially.
Now this will require
// User/Project@SHA
// User@SHA
The existing regex matches arbitrary code that is a series of [a-f0-9]{7,40} not matter how they appear in a sentence.

Oh. Apparently github has this issue too where it marks anything with [a-f0-9]{7,40} as special.

Github uses CodeMirror. So it will obviously exhibit this behaviour.

GitHub does allow linking of stand-alone hashes (as in 67379b827d111c97), so I don't think this is correct.

Agreed. This pull request is incorrect.
I didn't know about this feature that allows commit messages to become
valid links.
On Thu, Sep 14, 2017, 02:00 Marijn Haverbeke<EMAIL_ADDRESS>wrote:

GitHub does allow linking of stand-alone hashes (as in 67379b8
https://github.com/codemirror/CodeMirror/commit/67379b827d111c97aaeebfca96f454b6ce0d3b23),
so I don't think this is correct.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/codemirror/CodeMirror/pull/4967#issuecomment-329392009,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGFgDn3ZgTOf_hL5R-m8WJZdIZrgfqCwks5siM8WgaJpZM4PW_w0
.


Throw input errors on build optional sources

This PR follows https://github.com/Azure/AppConfiguration-DotnetProvider/issues/318#issuecomment-1522874825 to fix #318 , it:

removes check for optional flag, always throw user input error like malformed connection string, even if for optional configuration sources.
adds some test cases guarding the behavior

Changes look good to me! However, we'll hold off on merging this PR because it's a breaking change. We can revisit it when we are ready for the next major release. Thanks @Eskibear !


copy-sqldatabase, no exclude parameter

In the comments and help both show a exclude command to limit which DBs are copied. However when running the function the exclude command is not a valid parameter.

Hey jmccollum-bw,
Thanks for filing a bug. I just tested this and it worked well
PS C:> Copy-SqlDatabase -Source sql2016 -Destination sql2016a -DetachAttach -Reattach -Exclude db2 -AllDatabases
Attempting to connect to Sql Servers..
Checking access to remote directories...
Resolving NetBIOS name for SQL2016...
Resolving NetBIOS name for SQL2016A...
######### Database: db2 #########
Skipping db2
Database migration finished
Migration started: 06/30/2016 05:27:27
Migration completed: 06/30/2016 05:27:28
Total Elapsed time: 00:00:00
(db2 is the only db on that server). Can you paste your results along with Test-SqlConnection to your source server? You can remove any sensitive info.

Thanks for responding so quickly!
I am using DBAtools version <IP_ADDRESS> from the PowerShell gallery.
PS C:\Users\jmccollum> Copy-SqlDatabase -Exclude "vra" -Source vrasql -Destination testsql
Copy-SqlDatabase : A parameter cannot be found that matches parameter name 'Exclude'.
At line:1 char:18

Copy-SqlDatabase -Exclude "vra" -Source vrasql -Destination testsql

             ~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [Copy-SqlDatabase], ParameterBindingException
FullyQualifiedErrorId : NamedParameterNotFound,Copy-SqlDatabase

Local PowerShell Enviornment
Windows    : 6.1.7601.65536
PowerShell : 5.0.10586.117
CLR        : 4.0.30319.42000
SMO        : <IP_ADDRESS>
DomainUser : False
RunAsAdmin : False
SQL Server Connection Information
ServerName         : vrasql
BaseName           : vrasql
InstanceName       : (Default)
AuthType           : SQL Authentication
ConnectingAsUser   : sa
ConnectSuccess     : True
SqlServerVersion   : 12.0.4100
AddlConnectInfo    : N/A
RemoteServer       : True
IPAddress          :
NetBIOSname        : Unknown
RemotingAccessible : False
Pingable           : True
DefaultSQLPortOpen : True
RemotingPortOpen   : True
--
Jeff McCollum  |   Consultant  |   burwood group, inc.
jmccollum<EMAIL_ADDRESS> | Work:<PHONE_NUMBER>  |   Fax:<PHONE_NUMBER>
From: Chrissy LeMaire<EMAIL_ADDRESS>Sent: Thursday, June 30, 2016 10:29 AM
To: ctrlbold/dbatools<EMAIL_ADDRESS>Cc: McCollum, Jeff<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ctrlbold/dbatools] copy-sqldatabase, no exclude parameter (#29)
Hey jmccollum-bw,
Thanks for filing a bug. I just tested this and it worked well
PS C:> Copy-SqlDatabase -Source sql2016 -Destination sql2016a -DetachAttach -Reattach -Exclude db2 -AllDatabases
Attempting to connect to Sql Servers..
Checking access to remote directories...
Resolving NetBIOS name for SQL2016...
Resolving NetBIOS name for SQL2016A...
######### Database: db2 #########
Skipping db2
Database migration finished
Migration started: 06/30/2016 05:27:27
Migration completed: 06/30/2016 05:27:28
Total Elapsed time: 00:00:00
(db2 is the only db on that server). Can you paste your results along with Test-SqlConnection to your source server? You can remove any sensitive info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ctrlbold/dbatools/issues/29#issuecomment-229695301, or mute the threadhttps://github.com/notifications/unsubscribe/APyLgguUowdwhGD1BX-TsGTKL38eQbstks5qQ-DTgaJpZM4JCRnB.

The results of Test-SqlConnection do not reflect the same connection type as the Copy-SqlDatabase. With the first one, you're connecting using Windows credentials, and the second one, you passed the SQL Credential that contained sa. What you'll want to do is Copy-SqlDatabase -Source vrasql -Sql -SourceSqlCredential $yourcred then tab until Exclude appears. If it never does, it's not logging in properly.

Sorry about that, I was just dropping off the credential in the sample I showed. So even when I specify the credential to be used, I am still getting the same result of
A parameter cannot be found that matches parameter name 'exclude'
Looking at the code, in the parameters section, I don't see a parameter for -Exclude

So uninstalling and re-installing the module fixed it. Thanks for the help!


Velum on Kubic

From the previous issue:

Hi, how can I activate the Velum dashboard in Kubic?

https://github.com/kubic-project/velum/issues/216

@tgeek77 the current image is not loading velum because i can't see any manifests in /etc/kubernetes/manifests. kubelet is supposed to load them and start the salt-api, dashboard and db - but i'm not 100% sure how this would work on kubic, so i might be wrong.
the docker images are installed via rpms and loaded with container-feeder in the docker daemon. but i cannot see that on kubic.
will try to find out tomorrow and open a bugzilla if it's an actual issue.

On Wed, Jul 05, Ionuț Victor Palade wrote:

@tgeek77 the current image is not loading velum because i can't see any
manifests in /etc/kubernetes/manifests. kubelet is supposed to load them and
start the salt-api, dashboard and db - but i'm not 100% sure how this would
work on kubic, so i might be wrong.

As told the in the presentation, the containers including velum are still
missing on Kubik, since they are not yet part of Factory/Tumbleweed.
--
Thorsten Kukuk, Distinguished Engineer, Senior Architect SLES & CaaSP
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nuernberg, Germany
GF: Felix Imendoerffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nuernberg)

ah, i remember now.

Thanks guys, I'll keep an eye on the newest builds and be patient.

Any news on the Velum container?

Already found more recent information on Bugzilla. Here is the link to the Bugzilla bug https://bugzilla.opensuse.org/show_bug.cgi?id=1058919

On Mon, Nov 13, Stefan Weiberg wrote:

Already found more recent information on Bugzilla. Here is the link to the
Bugzilla bug https://bugzilla.opensuse.org/show_bug.cgi?id=1058919

We have containers in a devel project:
https://build.opensuse.org/project/monitor/devel:CaaSP:kubic-container
We have images using this containers:
https://build.opensuse.org/project/monitor/devel:CaaSP:1.0
But, somebody needs to port velum to ruby 2.5 and fix the bootstrap
problems with generating certificates, at minimum.


XAPTC-62 url added to @import in "styles.css"

@CameliaFarcas Just watch out when you committing things to git and which branch your on.


windows-build-tools

added a lighter option to build the libpng dep on windows.

@justinegeffen i merged your suggestions (thanks!) and answered your question, unless I missunderstood something. Can you take a second pass?


misc(Makefile): add help target to display the help msg

What does this PR do?
This pr added a new target help to project Makefile for displaying which commonly used commands that the project Makefile supported with their descriptions.

Motivation

This is useful when a new developer goes to find a helpful tool for testing or building etc.
Additional Notes

Hi @Halimao ,
Thanks for your PR, could you please rebase your PR on v2.11 branch.

Hi @Halimao ,
Thanks for your PR, could you please rebase your PR on v2.11 branch.

Is it ok now?


Normalize all text file in the repo.

Normalize all text file's line ending in the repository to LF https://git-scm.com/docs/gitattributes

@shshaw can you please take a look at it?

I'm not the most educated on git. @simeydotme any thoughts?

umm, yeh... I've never seen this needed before, git is usually set to automatically change cr to lf when committing, unless we've run into instances where someone's changed the line endings of files and caused a huge diff-mess , then I don't think that this is needed? but equally; it won't cause any harm...

@simeydotme I just wanted to follow the standards like here.


Expose internals

expose some internals (both haskell and nix) so nix-thunk can be (more practically) used as a library.

Redone in #36 ; merge conflicts :woozy_face:


[missing website] Starbase

https://welcome.starbase/
I made this site after Elon Musk tweeted about starting a new city in Texas called "Starbase."

Added the website. Thank you very much. :))


build: drop support for python 3.8

would resolve #571

if merged we can also implement the changes requested here.


accept unqualified unique column name in getter for column value in RowRecord

If the column name is qualified in the results but the column name is unique for the columns in the results, the qualification should not be necessary in the getter method.
Per review by Sam

Formerly, in the RecordSet.get*() methods, the column name string had to be an exact match.
For instance, if both schema and view were specified in fromView(), the getString() call had to specify schemaName.viewName.columnName
Now, the call can specify viewName.columnName or even columnName so long as the name has a unique mapping to the full name.
The same principle applies regardless of how the column was named including for the other accessors.
This approach is consistent with the behavior of SJS and XQY Optic.
You can see a unit test in:
com.marklogic.client.test.RowRecordTest.aliasTest()


Creating a new Jekyll dev container leads to duplication in post create step

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:  1.73.1
OS Version: CentOS 7

Steps to Reproduce:

Execute "Create Dev Container" command
Select Jekyll

The command will create a devcontainer.json with the following content:
// For format details, see https://aka.ms/devcontainer.json. For config options, see the
// README at: https://github.com/devcontainers/templates/tree/main/src/jekyll
{
	"name": "Jekyll",
	// Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
	"image": "mcr.microsoft.com/devcontainers/jekyll:0-bullseye",

	// Features to add to the dev container. More info: https://containers.dev/features.
	// "features": {},

	// Use 'forwardPorts' to make a list of ports inside the container available locally.
	"forwardPorts": [
		// Jekyll server
		4000,
		// Live reload server
		35729
	],

	// Use 'postCreateCommand' to run commands after the container is created.
	"postCreateCommand": "sh .devcontainer/post-create.sh"

	// Configure tool-specific properties.
	// "customizations": {},

	// Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
	// "remoteUser": "root"
}

and a post-create.sh:
#!/bin/sh

# Install the version of Bundler.
if [ -f Gemfile.lock ] && grep "BUNDLED WITH" Gemfile.lock > /dev/null; then
    cat Gemfile.lock | tail -n 2 | grep -C2 "BUNDLED WITH" | tail -n 1 | xargs gem install bundler -v
fi

# If there's a Gemfile, then run `bundle install`
# It's assumed that the Gemfile will install Jekyll too
if [ -f Gemfile ]; then
    bundle install
fi

Newer versions of the Jekyll dev container already contain the settings for postCreateCommand (and in port forwarding) - the entry in the JSON and the script. The generated setup results in the bundler commands being executed twice.

Starting with version 0.1.9 of mcr.microsoft.com/devcontainers/jekyll, the dev container already contains the settings related to post-create.sh:
https://github.com/devcontainers/images/tree/v0.2.9/src/jekyll/.devcontainer

Version 0.1.9 of Jekyll is part of version 0.2.9 of the entire repo: https://github.com/devcontainers/images/blob/main/src/jekyll/history/0.1.9.md


remote image support

I'd love to be able to download and set a remote desktop image.
wallpaper http://imgur.com/a/DdV4w#VGgJD9O

Yeah, that would be nice. Belongs in https://github.com/sindresorhus/wallpaper-cli though. Can you move the issue there?

https://github.com/sindresorhus/wallpaper-cli/issues/2


Add a extendedRequest parameter to QMChatService's messagesWithChatDialogID:completion: and earlierMessagesWithChatDialogID:completion:

In our project we don't want QuickBlox to mark messages as read when we fetch them.
We use QMChatService and themessagesWithChatDialogID:completion: and earlierMessagesWithChatDialogID:completion: methods. But they don't allow us to provide our own extended request parameters (we need to fetch the messages with mark_as_read=0, see http://quickblox.com/developers/SimpleSample-chat_users-ios#List_chat_messages).
Made/Proposed changes:

Added two more methods: messagesWithChatDialogID:extendedRequest:completion: and earlierMessagesWithChatDialogID:extendedRequest:completion:
They set the defaults as in the previous implementation.
Call the new methods from messagesWithChatDialogID:completion: and earlierMessagesWithChatDialogID:completion: respectively, with the extendedRequest:nil

How should this be manually tested?

Check if messagesWithChatDialogID:completion: and earlierMessagesWithChatDialogID:completion: still work as expected.
Check if the default request parameters are overridable when using messagesWithChatDialogID:extendedRequest:completion: and earlierMessagesWithChatDialogID:extendedRequest:completion:

Does the documentation need an update?
I've added the new methods to the header file and documented the parameters.

Appropriate methods were added in 0.4.5. Thank you for your contribution.


Maintenance

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 91.544%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
177043

Relevant Lines:
193396

💛  - Coveralls


fix double tapping on back button bug (go back to twice)

Does this leave the top left button the same as it would normally be on iOS and Android? The two have very distinct and different styles and we want to keep them correct to the platform?

This appears to override the button content with one style that is used on both platforms. Can you send screenshots of how this looks on iOS and Android for the TransactionList, SendConfirmation, and Settings screens.

@koretskiyav Any comment on this?

@koretskiyav Any comment on this?

There are now conflicts to resolve. Please address them


rkt fails to run multi-arch image on Arm

Environment
Hi~ I was doing some hacking in arm(aarch64) and turned out that the rkt would fail when it came to running docker multi-arch image. However recently, Docker hub has completed the major change that most of the images have been transformed into Multi-arch format. I was digging into the codes relating to the pulling docker image(appc/docker2aci), right now they have just updated it to image manifest version 2, schema 2.
I knew from the ROADMAP that you are focusing on the appc -> OCI transition, so it may add complexity to this situation, but it's really a bummer for arm developers.
What did you do?
./rkt run --insecure-options=image --interactive docker://ubuntu:16.04 --exec=/bin/bash
What did you expect to see?
Running successfully
What did you see instead?
Error: Unable to determine interpreter for "/lib64/ld-linux-x86-64.so.2" (I think it falls to wrong image )

@weichen81

Specifically, in image manifest version 2, schema 2, for now, the media tyepe "application/vnd.docker.distribution.manifest.list.v2+json" hasn't been dealt with.

I reproduced this problem with rkt Version: 1.29.0 (CoreOS 1590.0.0).
I'll look into a fix, but maybe not until next week due to the US Holidays this week.
Error: Unable to determine interpreter for "/lib64/ld-linux-x86-64.so.2"
Container rkt-119f4e56-e71a-4295-bdfc-b604bb440e7c failed with error code 203.

Hi, i am also facing same error on raspberrypi 3. Any update or solution please ?

You could convert the images by yourself and import them directly. At least
this is scriptable so you can get your containers up and running.
Md Shaiful Islam<EMAIL_ADDRESS>schrieb am Mi., 6. März 2019,
12:44:

Hi, i am also facing same error on raspberrypi 3. Any update or solution
please ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rkt/rkt/issues/3856#issuecomment-470077285, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC3LqWe573hp2T8dgHptAxbtc8KBCCdXks5vT6oQgaJpZM4Qhkfc
.

hi @trusch, yes i converted from docker aarch64/ubuntu image and able to run it on raspberry pi 3 but last 4 days i am trying hard to solve another issue with dns. when i am using --net=host its fine to go internet  from container but while using --net=default then container getting private ip as expected but dns not getting as well as not internet. even i  passed --dns=<IP_ADDRESS>  and --net=default --dns =host ...all possible option. my /etc/resolv.conf is fine. couple you please give me any idea to solve it please ?


Changed core pool size to number of processors

See http://stackoverflow.com/questions/15888366/meaning-of-core-pool-size-in-scheduledthreadpoolexecutors-constructor#comment39019434_15888497

Unfortunately ScheduledThreadPoolExecutor does not treat these parameters the same as ThreadPoolExecutor. STPE acts as a fixed thread pool of size 'corePoolSize' and does not create any additional threads. maximumPoolSize does nothing

:+1:


add ignoreHash option to include all colors in colorList

@diurnalist see the test added. the issue being seen was that users would have a list of colors, and the final hashtag was being set as hashIndex in the constructor.
instead of http://graphite-example.com/render?colorList=#38AFEB,#EB3838,#38EB38,#EB38DC&decache=3, the url was being stringified to
http://graphite-example.com/render?colorList=#38AFEB,#EB3838,#38EB38,&decache=3#EB38DC (and breaking)
Is this how I should handle this issue?

@diurnalist ping

Yes, that will solve this problem. This used to "work" by looking at the "this is a graphite url" flag. If that was set, I believe it would use the ignoreHash behavior. Otherwise it would treat that character as denoting the beginning of the hash part of the URL.
This change sort of means that hashes are not supported in promdash URLs, unless I'm missing something?

yeah that was my concern. so I'll change it so that "this is a graphite url" means to ignore hash, which seems like the right solution

:fist:


How to configure HttpClient to use Default System Proxy

Some details
In this doc https://microsoft.github.io/reverse-proxy/articles/http-client-config.html#httpclient   is shows a way to configure a WebProxy, but not mention how to just set the HttpClientHandler.UseProxy to true,
I use a custom configuration to set it to use system proxy  for fiddler debugging

                .ConfigureHttpClient((c, h) =>
                {

                    h.UseProxy = true;
                })

and I hope I can use the build in configuration for that

When WebProxy is configured and Address is not null , handler.UseProxy = true
    private static IWebProxy? TryCreateWebProxy(WebProxyConfig? webProxyConfig)
    {
        if (webProxyConfig is null || webProxyConfig.Address is null)
        {
            return null;
        }

        var webProxy = new WebProxy(webProxyConfig.Address);

        webProxy.UseDefaultCredentials = webProxyConfig.UseDefaultCredentials.GetValueOrDefault(false);
        webProxy.BypassProxyOnLocal = webProxyConfig.BypassOnLocal.GetValueOrDefault(false);

        return webProxy;
    }

        var webProxy = TryCreateWebProxy(newConfig.WebProxy);
        if (webProxy is not null)
        {
            handler.Proxy = webProxy;
            handler.UseProxy = true;
        }

This seems like something we could call out in the docs - The default ForwarderHttpClientFactory will set UseProxy to false, so if you want to use the default proxy (not a manually specified one from the config), you can flip it back to true.
Another similar case with fiddler: https://github.com/microsoft/reverse-proxy/issues/1477#issuecomment-1343390586
I don't remember actually discussing why the default of disabling UseProxy was chosen though.


Feature/#40 save update vehicles

I believe I've completed the features intended to be implemented by @gwenf in her branch and PR. I created a new branch for my work, and now this fresh PR.
I have beaten on this pretty hard... so I think that nothing is broken. But of course, you'll tell me if that is true or not.
So please take a look and give me your most awesomest feedback!

Hey, thanks @camoore1979
This seems to be working great. There is no working delete function for vehicles? Maybe that's coming later.

Thanks, @camoore1979! It's a step in the right direction!
One bug I was able to replicate (usually):

Add a new vehicle, with or without the image.
The app will display the list of vehicles, including the new vehicle.
Click the VIN of the new vehicle. Crash:

TypeError: Cannot read property 'vin' of null
VehicleDetail.render
src/components/fleet/vehicles/VehicleDetail.js:16
13 |
14 | return (
15 |

16 |     {Vehicle Detail: ${vehicle.vin}}

Also, could you make Rent to Own a checkbox to match the spec?

Thanks @gwenf and @BWortman !! Working the requested changes now.

Thanks for the reviews @BWortman @gwenf !!!
I have made the requested changes by @BWortman and also fixed #48 (I think they are now per spec).
Please give it a try!

@BWortman thank you for the thorough reviews & UAT! OK, I think this is it.... ;-)
Take a look and let's see if we can pull the trigger this time. Thank you!

Just playing with tasks...

[] - incomplete
[x] - complete

ok, that's better


FEAT-#6048: add wait method for Dask/Ray/Unidist wrappers

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #6048
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

@YarShev could you take a look?


Icon Generator incorrect sizes for Xcode v 7.2

When generating icons, Xcode reports that the icon sizes generated from the plugin are never the correct size.

Never saw that message, can you post a screenshot?
Anyway, this plugin does not export icons, it's up to you to export everything in proper size.


why is this symlink here

https://github.com/kscalelabs/store/blob/master/store/app/frontend

it looks like the way we will run things In Reality is just off port 8080(?)


Fix vuln

Coverage remained the same at 98.333% when pulling d051dd3d3bdf93302593b82c13a59ffc96c47fd9 on brandon-binary:fix_vuln into 82cb253d5df66b115aeecc369940ca4f3e603e67 on binary-com:master.


stac-api feeder as a workflow

Random thoughts on turning the stac-api feeder in Cirrus into a Workflow
Input payload
This input payload contains no STAC Items, only task parameters. The STAC API Crawl
task will use the provided query to fetch all matching STAC Items from the API
and then publish each of the resulting Items as a single feature FeatureCollection
with the given process deinition attached.
Here is an example input payload. Note that the stac-api-crawl task contains a
process definition that will be used for the processing of all found Items.
{
    "id": "usgs-landsat/workflow-stac-api-crawl",
    "type": "FeatureCollection",
    "features": [],
    "process": {
        "workflow": "stac-api-crawl",
        "input_collections": ["usgs-landsat"],
        "tasks": {
            "stac-api-crawl": {
                "url": "https://landsatlook.usgs.gov/stac-server",
                "search": {
                    "intersects": "<geometry>",
                    "datetime": "2021"
                },
                "process": {
                    "workflow": "process-landsat",
                    "upload_options": {
                        "path_template": "s3://<my-bucket>/${collection}/${id}",
                        "public_assets": "ALL"
                    },
                    "tasks": {
                        "preprocess-landsat": {
                            "param1": "val1"
                        }
                    }
                }
            }
        }
    }
}

Each found STAC Item will be published as a Cirrus Process payload back to Cirrus.
There is typically no publish step in this workflow, unless it is desired to mirror the contents
of the STAC API to the local STAC API.
For each item found construct the Process payload and publish to the SNS topic. Assuming a given
STAC Item is fetched from the API there should be no possibility of error here.
Some questions:
- The ID should be the hash of the payload ?
- If an error happens during paging through the API, what happens?
Example output Process Payload
{
    "id": "usgs-landsat/workflow-stac-api-crawl",
    "type": "FeatureCollection",
    "features": [
        "<LandsatStacItem>"
    ],
    "process": {
        "workflow": "process-landsat",
        "upload_options": {
            "path_template": "s3://<my-bucket>/${collection}/${id}",
            "public_assets": "ALL"
        },
        "tasks": {
            "preprocess-landsat": {
                "param1": "val1"
            }
        }
    }
}

This doesn't completely remove the need for a "feeder"...something still needs to create the
original input payload(s) which, for a large number of items, should parition the API into
sets of query parameters.  e.g., a payload for 2015, 2016, etc.

@matthewhanson Do you need anything more for this on the cirrus-geo side? That is, can this issue be closed, or do we need to track further action items?

Nope, we can close this.  Implemented this as a workflow, which works like any other task except it queues payloads.  We may want to consider a generic class built on stac-task (or, more likely, a base class that gets factored out) that promotes this.


DDR Menu: Long Page name should be wrapped inside a tooltip box

Description
Long Page name should be wrapped inside a tooltip
If a user has a really long page name, the page name overflows outside the tooltip if a user hovers over the page name.
This has been previously addressed by dnnsoftware/Dnn.AdminExperience.Extensions#404 and #1947
Current result

Expected result
Should wrap within a certain width /etc..
Affected version

[x] 9.2

Fixed by #2028
@dnnsoftware/tag please close unless you have any concerns about this fix


Should Error type be public?

Hello, thanks for your work on this!
I ran in to an issue when using it though. Many of the api functions return a Result where the Err variant is of type ytmapi_rs::error:Error, which is private. This means that I can't match on the error type and react differently depending on what produced the error. Should this module be public?

Hi there, ytmapi_rs::error::Error should be re-exported as ytmapi_rs::Error for convenience, and then to get the kind for pattern matching you would call into_kind() to get the ErrorKind (similar to std::io::Error). However seems I haven't made ErrorKind public. Happy to accept a PR, otherwise I am hoping to have some time to work in this crate in a week or 2's time. Development note: ErrorKind should be marked #[non_exhaustive] to allow for future error kind's to be added without a breaking change (https://doc.rust-lang.org/src/std/io/error.rs.html#190).

@pianocomposer321 resolved on main, keep an eye out for ytmapi_rs 0.0.3 on crates.io soon.

@pianocomposer321 v0.0.3 now on cargo


Feature/tws update grace

Description
This PR adds the capability for Noah-MP v4.0.1 to assimilate Terrestrial Water Storage (TWS) data from the GRACE mission. TWS DA code is added to the Noah-MP v4.0.1 LSM, and the TWS DA smoother is updated to include this capability.
Resolves Issue #1144
Testcase
LIS is tested with TWS DA in the following directories:
/discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_multiVarDA_01km_TESTS/OUTPUT_CaliforniaColumbiaISRIC_TWSda_LSMupdate
lis.config File: /discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_multiVarDA_01km_TESTS/lis.config
Log Files: /discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_multiVarDA_01km_TESTS/TWSda_TESTupdate_LOG2
The 2-month TWS DA simulation may be compared to the control OL simulation for the same time-period:
/discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_01km_OL_LSM/OUTPUT_CaliforniaColumbiaISRIC_wIMERG_OL_LSM
The GRACE Data Files for the rest domain (and the corresponding ldt.config file) are available here:
/discover/nobackup/projects/eis_freshwater/lahmers/SHARE/OUTPUT_GRACE/
/discover/nobackup/projects/eis_freshwater/lahmers/SHARE/ldt.config_freshwaterRegional_CaliforniaColumbiaRegionWBD_ISRIC_GRACEda

@jvgeiger I did some more testing with this model configuration this week, and I am now testing it with multi-variate DA. Please let me know before you start this PR, as I am going to need to make a small modification to this PR after the CCI PR is complete to ensure that all of the DA configurations I am using are supported. I know exactly where the code needs to be changed (it is a plugin), so the update will only take a few minutes to push through.

@jvgeiger @sujayvkumar I added some updates to the branch used for the TWS PR, and I need to do some model tests to verify that the results are consistent with earlier multi-variate DA simulations that I have executed. I can confirm by Monday if the changes to this branch are ready to commit into the LIS master.

Hi @TimLahmers:
I do not have permission to look into /discover/nobackup/projects/eis_freshwater.  Any chance this could be opened up so I can test run this pull request?

Hi @TimLahmers,
When you have time, can you please rerun you testcase to see if it still works?  I am testing it out now and receiving an error from the file */LISF/lis/surfacemodels/land/noahmp.4.0.1/phys/module_sf_noahmp_glacier_401.F90 at line 284.
Thank you!

Hi @TimLahmers,
We have been able to run your pull request, however, we will need to set up a time to review some of the code changes. Specifically, many of the files in the da_tws directory need to be discussed.   Jim, Eric, or I will be in touch with you!
In the meantime, would you mind rerunning your testcase?  We noticed two commits have been made since the last time the testcase was run.
Thanks!

@cmclaug2 Thanks for the update, and that sounds good. Please reach out to me if you have any questions.
Regarding the test case, I will re-run the simulation today (it might take a few days due to the current issues with Discover). Since there are other changes to LIS, should I also re-run the control test-case?

@TimLahmers,
Great thanks! Yes, can you rerun the control (OL) testcase as well?  We'll only need to look at files within the time window of your DA run (10/2014 - 01/2015)

Thanks @cmclaug2 that sounds good! I just submitted the control run with the latest LIS version (waiting for Discover to start), and I brought the feature/twsUpdateGRACE branch up to date with the LIS master. Once I confirm that it runs, I will re-submit the TWS-DA sample job. That job is 12 hours with 720 CPUs, so it may take some time to get started. I will let you know when the new test cases are complete.

@cmclaug2 The TWS-DA simulations with updated LIS code are complete. Please let me know if you have any questions about these outputs or the model configuration.
Control:
Output Directory: /discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_01km_OL_LSM/OUTPUT_CaliforniaColumbiaISRIC_wIMERG_OL_ctrlTWS
lis.config File: /discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_01km_OL_LSM/lis.config_LSM_CaliforniaCascades_ISRIC_wIMERG_TWSctrl
Output Log Files: CA_wIMERG_LOG_OL_ctrlTWS
TWS DA:
Output Directory: /discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_multiVarDA_01km_TESTS/OUTPUT_CaliforniaColumbiaISRIC_wIMERG_DA_TWSudpate
lis.config File: /discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_multiVarDA_01km_TESTS/lis.config_RegionalTemplate_WestUSISRIC_daTWS_Update
Output Log Files: /discover/nobackup/projects/eis_freshwater/lahmers/RUN/REGIONAL_multiVarDA_01km_TESTS/TWSda_TESTupdate_LOG2

At a recent LIS monthly tag-up meeting, this PR was discussed and all parties feel that these changes were already merged in via #1305
I did a code comparison, and the master, public, and eis-freshwater2 branch all have the latest GRACE DA code for Noah-MP-4.0.1 LSM.
Thus, we will be closing and NOT merging this PR.
Leaving open for a few more days for any additional comments.


Support iocage develop branch as of 2016-03-25

Will be brought in as soon as ports/sysutils/iocage-devel has support for
https://github.com/xmj/iocage/tree/support_for_template_state or later.


Support variable unification in matche patterns

When given an existing variable name in a pattern, matche appears to treat it as a new variable. Example:
(run 2 (q r) (matche (r) (((Foo ,q))) (((Bar ,q)))) (== q 'Qux))
'((Qux (Foo _.0)) (Qux (Bar _.0)))

I expected the output to be '((Qux (Foo Qux)) (Qux (Bar Qux))).
I glanced at the paper describing the implementation of matche, lambdae, and defmatche and the problems encountered therein, but this didn't look like one of them. Is this feature doable?

Upon reflection (and reading an e-mail from B), I realize this is the expected and desirable behavior. Nevertheless, it would be nice to be able to support unification in matche patterns by referencing variables in the patterns themselves. Perhaps it would make sense to use different syntax when referencing a variable than when introducing one.


Build with circle

Description
[Describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves]
Check List

[ ] All tests pass. See https://github.com/sous-chefs/apache2/blob/master/TESTING.md
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable

@Xorima we have linting!!!


image: Add option for drawing smooth lines or add new method (like DrawSmoothLine)

Now it is impossible to draw anti-aliased line. I'll do it in the future.

See #137


Force node clones to get a copy of all fields

I observed the node reports were removed after some time.
I have traced this to
https://github.com/vespa-engine/vespa/blob/master/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/Node.java#L134

When Node wants to create a clone of itself with a modification, e.g. with
wantToRetire, it calls that Builder constructor. The problem is that
this.reports is not copied to the Builder's reports field, and so the new node
has lost the report.
A new field 'modelId' has been added to the node after 'reports', and it too
is not copied to the builder.
This is clearly too fragile.
This PR makes a public Builder() constructor taking in ALL fields of the Node
constructor, and forces all code outside to use that Builder constructor by
making the Node constructor private. This is just a refactoring.
However the Builder(Node) constructor calls the new Builder constructor, thus
forcing forwarding of all fields. This solves the problem.

This is why there wasn't a builder.

Let's revert this to the original pattern, we have had zero such issues with it to my knowledge. In its current form, it's more code and apparently easier to make mistakes.

It's not easier to make mistake with this PR. And, whenever a field is added, you add it in much fewer places. It is also much easier to build variations of nodes in tests by building from a node and modifying.

We don't add fields that often and I don't think making test code slightly less verbose is important enough to have this builder.
With this pattern, it's tempting to attempt to simplify the builder (as the code before this change showed). It also makes consistent refactoring harder as all fields and setters are duplicated.

Mentioned a 3rd alternative offline - removing all mutation-like methods on Node and make the Builder public. The benefit would be removing the redundant mutation-like methods. This would require updating all callers to Node.with* to use new Node.Builder(node).
Voted down as there weren't that many call sites on Node::new.

https://github.com/vespa-engine/vespa/pull/8505


Resolved clicking stop runtime on Terminal screen shows starting runtime msg [SATURN-1101]

ticket here

I did notice a bug separately when trying to test this. When deleting a notebook runtime and then starting one, we get this error Cannot read property 'clusterUrl' of null. Which appears already was happening in the code before my work as far I have examined. I think this is worth another ticket.

This would be easier using utils.cond.

I got the same Cannot read property 'clusterUrl' of null error when trying to open a terminal when I didn't yet have a runtime environment.

Added more cluster status options per @breilly2's comment and observed the correct messages for:
- Creating
- Stopping
- Stopped
- Starting

Getting the null error (that Cameron is working on in the other ticket) when trying to test for:
- Deleting
- Deleted
(so not able to fully see if those are working or not but can confirm once that null bug is fixed.)

Not able to get a cluster status of Updating (need to look into Leo code more for why)

Did not test Error or Unknown, assuming these are generally going to be handled by other mechanisms but added messages if all else falls through

Thoughts about the extra status' welcome. Thanks!

Clusterstatus messages have been confirmed with @sharmatime, all is good to go, ready for review

I am following up with Leo team to see if there is an issue about the clusterStatus of 'Updating' coming through.
At line 295 in this code there is a comment that changing an existing cluster's size should result in a status of Updating:
https://github.com/DataBiosphere/leonardo/blob/3e0879e2586e2ddc303a3491557d31a54a91b40a/http/src/main/scala/org/broadinstitute/dsde/workbench/leonardo/service/LeonardoService.scala#L295
However this is not happening.
I was able to test that the 'Updating' conditional messages does come through by setting it with the ajx.overrides, but neither Pete nor I were able to see an 'Updating' status after updating the cluster size in regular testing circumstances.

Clarification from @zarsky-broad is that currently we are doing deletes even when updating (except in super rare instances) so it is currently expected that we cannot get an 'Updating' status and instead are seeing the 'Creating' one.


Using Mediaflow Proxy Instead of Warp

I have a Mediaflow Proxy server that I would like to use instead of Warp. It'd be awesome to see more apps add this option. Thanks!

wtf
wdym replace warp with mediaflow proxy this makes no sense
warp is not used to proxy the stream but to bypass Torrentio/RealDebrid/AllDebrid server blacklists which mediaflow proxy cannot do
comet already has its own stream proxy and mediaflow proxy will never be integrated as it is useless


Repository example

Hi is there any example open source project using this? I'm curious for reference what is the setup inside the Dockerfile

+1 to this

My example (preserve /var/cache/apt):
https://github.com/reproducible-containers/repro-sources-list.sh/blob/v0.1.0/.github/workflows/main.yaml#L65-L104


Benchee 0.8.0 and dialyzer adjustments :)

Coverage remained the same at 100.0% when pulling 402a6cd052b7a7b5d3d44519f94bd2de22e991fe on benchee-0.8.0-adjustments into 7998eff0fc61361f1175d893e34b9f0627fbc8f8 on master.

Coverage remained the same at 100.0% when pulling 402a6cd052b7a7b5d3d44519f94bd2de22e991fe on benchee-0.8.0-adjustments into 7998eff0fc61361f1175d893e34b9f0627fbc8f8 on master.

Coverage remained the same at 100.0% when pulling 35a64c93501c20a020db4ccfc10d9c3e44bb5a88 on benchee-0.8.0-adjustments into 7998eff0fc61361f1175d893e34b9f0627fbc8f8 on master.

Coverage remained the same at 100.0% when pulling 35a64c93501c20a020db4ccfc10d9c3e44bb5a88 on benchee-0.8.0-adjustments into 7998eff0fc61361f1175d893e34b9f0627fbc8f8 on master.


Update to Olmsted

Also, cache seems old - should have about another 1,300 records

Not sure what happened here...I tested the REST query as detailed in the log and I can't get it to fail similarly.

Try updating the data url from http to https.  I noticed that when i put the http URL into the browser, it redirects to https.

good call! Trying now.

any chance this can get merged soon?


Use Category Patters for pure I/O

Pull Request Test Coverage Report for Build 252589499

58 of 95 (61.05%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-13.8%) to 67.153%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

assay/types.go
11
16
68.75%

assay/config.go
0
10
0.0%

http/types.go
47
69
68.12%

Totals

Change from base Build 251561966:
-13.8%

Covered Lines:
92

Relevant Lines:
137

💛  - Coveralls


Make the site mobile/tablet responsive

Use CSS media queries and material UI breakpoints.

I want to make the site mobile/tablet responsive.

Hey @daretobedifferent18 thanks for the help. The site is made using react and material-ui, so please make the changes and raise a PR if you’re comfortable working with them 😄.

Pls help me out in running this project locally.

@daretobedifferent18 I'm planning for some re-design to the site. As a part of that, I'll cover the responsiveness issue too. So, closing this issue now. Please feel free to work on other issues in repo. 😃


Programmatically display CursorLine

Hello,
I have a use case where some buttons aim to go forward/backward over time.
It would be great if there is a way to move/display the cursorline from code (like hook).
Thanks again for the lib.

Hi @tiste.
Great suggestion! How about an at property that can be used like:
<LineChart.CursorLine at={indexToDrawLine} />

Hey @honeybadger26, it's done :) #183


Make support for all the H7(2,3)(3,5) variants explicit

For now, the settings are all the same as the H735 because the security features that exist in some variants and not others are not supported for any variant, and the only other differences are in memory layout.
I have not exhaustively tested support for the added chips, but have been able to run blink and UART serial programs on the NUCLEO-H723ZG board: https://github.com/jlogan03/nucleo-h723zg .

Thanks for the PR! The change to the README looks good! As you say these parts can be supported with the same configuration as the stm32h735.
Although we didn't always do this in the past, I would prefer to only add new feature flags if they can be tested in Continuous Integration here and here, and if these test a meaningfully different configuration. The idea would be to keep the delta between the list of possible features and those tested in CI as small as possible. Because of that I don't see a big benefit in adding the new flags to Cargo.toml.
There are some tables in the documentation that currently refer to stm32h725/735 here here here and here. I guess that all of 723,725,733,735 (and 730) apply in these cases?

Makes sense - I removed the new feature flags and just updated the documentation instead. How's that look?

Looks good, thanks! I think you can remote the patch version bump now?

Done. I had added the patch version bump because there is a change to one of the feature flag gates to remove the stm32h725 flag, but since that flag did not exist to begin with, I don't think there are any cases where it could have a real effect.

Got it. I'll merge this now
bors r+


ENH: An improved error message when calling zipline api functions

outside of a running simulation. Previously, an AttributeError was
raised.
The goal here is to make the situation clearer for developers, for example: https://github.com/quantopian/zipline/issues/1541 and https://github.com/quantopian/zipline/issues/448#issuecomment-249312347
What do folks think of the message?

:+1: from me

The only change I could maybe see here is adding a specific exception type for this (e.g. NoSimulationRunning) rather than RuntimeError.  But I don't feel super strongly about it either way.

+1 also, I think the runtime error is fine because I don't expect anyone to want to catch this

Ok, cool, then I'm going to merge on travis passing.

Coverage decreased (-0.004%) to 86.992% when pulling 1d2d6d843f53eae96a83dcaa69fe027fbb7ff1b6 on nicer-message into 8b8b4a811859e07615eab3db790652553bed460e on master.


Locations provided outside of the scope of a Builder are ignored

These two tests should provide the same results:
with BuildPart() as test1:
    Box(10, 10, 10)
    with BuildSketch(*test1.faces()):
        with GridLocations(5, 5, 2, 2):
            Circle(1)
    Extrude(amount=1)

with BuildPart() as test2:
    Box(10, 10, 10)
    with GridLocations(5, 5, 2, 2):
        with BuildSketch(*test2.faces()):
            Circle(1)
    Extrude(amount=1)

but they don't (Blue is test1):

@bernhard-42 provided a more mathematically correct answer - there should be four sets of six cylinders as location multiplication is not cumulative.  This implies that Workplanes should follow existing Locations.

Thanks @jmcmullan, you're right they generate the same parts now with the nesting fix.  What about the "not cumulative" discussion above - does it make sense that they generate the same part?

I agree with you @jmcmullan. Although the more complex interpretation may ultimately be the correct one, I'm going to close this issues as I believe this is the behaviour that would be expected by most users.  Thanks for the help!


fixed note creation not respecting title case issue #344

removed the post-formatting for the title to remain true to the original input casing.

 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.

Ah, that makes sense. Just to make sure I understand:

if a title is 'Title example' it remains 'Title example'
If a title is 'title example' it changes to 'Title example'
If a title is 'Title Example' it remains 'Title Example'
If a tile is 'title Example' it remains as 'title Example'
Most unsure about case 4, seems like the convention before was to make sure that the first letter was capitalized.


Auto discovery embedded cache configurations #29

Looks good, I like the idea!
Could you please enhance your PR by adding two things:

The same type of discovery for Hot Rod client configuration: https://github.com/infinispan/infinispan/blob/master/client/hotrod-client/src/main/java/org/infinispan/client/hotrod/configuration/Configuration.java
At least a small example in the README.adoc: https://github.com/infinispan/infinispan-spring-boot/blob/master/README.adoc

Does the remote cache support multiple configurations ?
Or in this case is just to lookup the remote cache configuration and apply it ?

No no, it supports only 1 configuration. Just look it up and apply if
that's not a problem. Thanks for looking into it!
On Thu, May 25, 2017 at 3:43 PM Luca Burgazzoli<EMAIL_ADDRESS>wrote:

Does the remote cache support multiple configurations ?
Or in this case is just to lookup the remote cache configuration and apply
it ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/infinispan/infinispan-spring-boot/pull/30#issuecomment-304013260,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIBvhv-_wESGH4Ups-bpBZG75WbkgpOgks5r9YWPgaJpZM4NmMLM
.

--
SEBASTIAN ŁASKAWIEC
INFINISPAN DEVELOPER
Red Hat EMEA https://www.redhat.com/
https://red.ht/sig

Should the InfinispanRemoteConfigurer be deprecated ?
It could be nice to add also an InfinispanRemoteConfigrationCustomizer, like:
@FunctionalInterface
public interface InfinispanRemoteConfigrationCustomizer {
    void configure(final ConfigurationBuilder builder);
}

Then the Configuration bean lookup would happen as last step if none of the current steps succeed and finally the customizers are triggered.
Make any sense ?

+1, makes sense.

I've pushed a new version with requested changes

Thanks a lot @lburgazzoli! Awesome work!