feat: plugin update dialog

Motivation
Fixes #102
Description
After the plugin has been updated or installed, the next time a project loads the user will be given a update dialog. The dialog will display opened in the top right hand corner of the IDE.
Any of the links in the update notification will open in the user's default browser.
Screenshots

Notes
I figured you would just squish the commits on merge, that way way your commit history will be consistent.

I figured you would just squish the commits on merge, that way way your commit history will be consistent.

Don't worry I do squash commits for this project 🙂

This PR is on top of #114 .

Created a new issue for the content (#118).
We can now merge this after rebase --onto wmp-1.2 61f828c62cbfc31d9f815a9feb104f6589fd14a7.

Wow, I messed that up.

No wait, I think I am okay.
I wanted to be based at head c793feecd31bcca005d917ceb24bce22b816c1c2

False alarm, I did mess it up :facepalm:
You are really testing my Git-fu with your git flow setup, bro. :sweat_smile:

Sorry for that, I should have merged this one first before doing the rebase to make the branch on top of the master branch.
I pulled your branch earlier and shared the command to rebase it with
rebase --onto wmp-1.2 809c21d1cddda870864dc129751bd579aa0e14be
It didn't work on your end?
I just pulled it again and I think it looks good now:
g cherry -v wmp-1.2                                                                                ─╯
+ 912d6c9e78d3de5e472171560422af3f67f2e2a1 Added kotlin support
+ af43baccd095e29de2e0f41ad74f34879faf1f58 Able to show notification on startup
+ 0450030c71634a27630f46b7f047c837c5935c6e Cleaning up
+ 0a2c24803efe8e693c87bc74e6aa949581d808ef Bump indents to 4 for now
+ 2690275dde4312f1ab02bafabef31f71639fa098 Added final new line rule
+ 4da1f3d46f380f120a07101b44b71789757ef26e Clamping to release that has all available APIs
+ 4acc701ce77c9de97972b50cf1b4e1523c497b35 Moving since build compatability to plugin.xml
+ 8dc60799ca1032b89ebaf3d108577e68e9880592 I'm bad at rebasing

There seems no breaking changes or missing commits from the rebase. I think we can now merge this.

I pulled your branch earlier and shared the command

It did not, for whatever reason the b43bbdf941a531b8a073c4abe6a9a82c29eeb190 commit was included as part of the commits to be applied during the rebase. It was already in wmp-1.2 and had conflicts with a3268259f6655c3b2040e134c7a59ea57074f124. So I just did an interactive rebase and dropped b43bbdf941a531b8a073c4abe6a9a82c29eeb190, and it worked for the most part (I had conflicts with kotlin state change stuff). I'll get better at this tho


Require ContentSize on UiImage again

Objective
The ContentSize requirement on UiImage got lost during merge conflict fixes, causing some images such as the icons on the game_menu example to disappear.
Fixes #16138
Solution
Require ContentSize on UiImage again.

Looks like the wrong ticket got linked in the description.


Extensions Localization

Conversion from xlf to single json for Schema Compare and Dacpac extension. Once I get a sign off for this - will send a PR for the same for other extensions too.

@kburtram please let me know if these pattern seems okay and I should do this for all extensions.


heimer: 1.18.0 -> 1.19.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/juzzlin/heimer/releases.
meta.description for heimer is: "Simple cross-platform mind map and note-taking tool written in Qt"
meta.homepage for heimer is: "https://github.com/juzzlin/Heimer"
meta.changelog for heimer is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

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

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

found 1.19.0 with grep in /nix/store/7d0n9dvm2yigd9cf6rvdvhc6km34by30-heimer-1.19.0

directory tree listing: https://gist.github.com/f6aab191c34473c7f1e00884c5d722f1

du listing: https://gist.github.com/d505b5f598d29ff83eca06d9a0ba6813

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

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
heimer

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7d0n9dvm2yigd9cf6rvdvhc6km34by30-heimer-1.19.0
ls -la /nix/store/7d0n9dvm2yigd9cf6rvdvhc6km34by30-heimer-1.19.0/bin

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

1 package built:

heimer

Maintainer pings
cc @dtzWill for testing.

LGTM, builds,runs,functions as expected on NixOS.


Upgrade building tools

This is still work in progress, opening the PR so we can track the progress.
Linked issues #2168 #1866
Summary
The build mechanism has not been updated with the v2. Now that version is live, we will upgrade the building tools in order to have the most productive development environment 🚀
I did an experiment by replacing node-sass with postcss but it's too much overhead, we will sync about this when we want to provide a single shared theme in between instantsearch libraries.
I also did an experiment with rollup instead of webpack, but there's no real advantages of doing this. It adds more dependancies and now webpack3 has reached the state of rollup, maybe goes further with scope hoisting 🌟

[x] Upgrade dev & staging build to use webpack v3
[x] $ yarn dev and $ yarn staging shares now the same dev/webpack.config.js
[x] Remove the .babel.js from webpack config to load faster since we use now nodejs7+ we can use ES6 features without transpiling through babel
[x] Upgrade docgen to use webpack3
[x] Upgrade final build to use babel CLI for ES6 modules
[x] Use babel-preset-env with modules: false (Tree shaking)
[x] Upgrade final build to use webpack3 for browsers builds
[ ] Simplify and clean every scripts/*.sh (since we use almost webpack to do all the stuff, we can have one liner commands into package.json scripts)
[ ] Document usage with rollup/wepback2+ for tree shaking

More detailed changes for the dev env:

use of webpack style-loader + sass-loader instead of building and watching with node-sass CLI
-> This introduce a speed gain as well of having the "hot-reload" of styles
use of html-webpack-plugin to generate an index.html with hashed assets
-> Cache buster when deploying to staging
use of autodll-webpack-plugin which build vendors seperatly
-> Speed up re-build on dev (big external libraries not re-builded)
use of happypack which allow to transform multiple files in parallel
-> Speed up initial build and re-build

Results
Staging build is 50% faster (removed the minification on new branch to have a fair comparison):
BEFORE:
➔ time (node-sass -o ./dev/ ./src/css --output-style expanded && webpack --config dev/webpack.staging.config.babel.js --hot --inline --no-info)

~12s

AFTER:
➔ time NODE_ENV=production webpack --config dev/webpack.config.js

~6s

Production build stays quite the same, no build size changes but everything is bundled by webpack.

Is there any feature added for package consumers? (Out of curiosity).
What's the dev reload time before/after?

@vvo
I will come with real stats in the end, but it's already a big win to have a single webpack handling all the stuff instead of having two CLI tool running for the dev environment (webpack & node-sass) separately.
It might not gain speed in term of bundling, but the end experience of hot reload in the browser is completely worth it while working on styles.
Features for the end users:

we will ship real ES6 modules so it will be easier for them to have tree-shaking
webpack3 scope hoisting should speed up the load of the library itself in the browser
lower the build size of CSS & JS (not sure, investigation at the end)

@vvo I did a little comparison of build time for staging, dropped from 12s to 6s 💪

I did a little comparison of build time for staging, dropped from 12s to 6s

What's staging (vs dev and prod)? What's the time for the actual dev?

dev is build of library itself + stories + documentation
staging is a static build of the dev without documentation
prod is browser and modules build of the library itself

Will add more metrics when I'll be done with upgrading the documentation tools 👍

I can't upgrade easily the docgen build, ms-webpack is explicitly requiring webpack v1 (https://github.com/10xjs/ms-webpack/blob/master/package.json#L21) which is altering the configuration on the fly and make it incompatible with webpack 3 😩
So, I'm removing upgrade of docgen build from the scope of this PR.

@iam4x ms-webpack is like 100 lines. Ditch it clearly :)

@vvo Thanks with ms-webpack alpha.1 which does not require a specific webpack version it works like a charm 🙌

We also now provide an ES6 build to be picked by webpack2+/rollup for tree shaking 🎉

We agreed with @vvo on an export strategy:

import instantsearch from 'instantsearch.js';
// provides same instantsearch object as es5 or <script> build
// useful for non-breaking changes on actual apps already using webpack or rollup

import { instantsearch } from 'instantsearch.js';
// provides instantsearch object without connectors and widgets attached.
// only `version` and `createQueryString` are available

import { connectXXX } from 'instantsearch.js'
// provides every available connectors

import { searchBox, menu, refinementList, ... } from 'instantsearch.js'
// provides every available widgets

.babelrc file has now only environments, default one picked will be development

@vvo @bobylito
Some more stats using the new ES6 build with etch-instantsearch which is only using the connectors API:

before: instantsearch.js: 438.42 KB
after: instantsearch.js: 160.11 KB

I reduced the size of the instantsearch.js dependency up to ~60% !!

But in order to have this level of optimisation, I had to change the export strategy.
Importing import main from './src/lib/main.js' into index.es6.js will automatically require React and all the widgets.
Even if on your side you import instantsearch.js this way:
import {instantsearch} from 'instantsearch.js'

In order to resolve this issue, the new exporting strategy now includes breaking changes:
import instantsearch, {connectMenu, searchBox} from 'instantsearch.js';

// throw an error when trying to access to `.connectors` or `.widgets`
instantsearch.connectors.connectMenu(...);
instantsearch.widgets.searchBox(...);

// does not throw
connectMenu(...);
searchBox(...);

The error displayed is pretty dumb for now, will add link to the docs after I updated them in this PR:

There is also something we did not think about with @vvo is the use-case of importing all the connectors by once which is not intuitive right now:
// require every connectors available
import * as connectors from 'instantsearch.js/dist-es6-module/src/connectors';

Something easier would be:
import {widgets, connectors} from 'instantsearch.js';

What do you think?

I am not sure to get it but:
import instantsearch from 'instantsearch.js';

instantsearch.connectors.connectMenu(...);
instantsearch.widgets.searchBox(...);

Should still work, otherwise it's a breaking change (and major version). If it does not work because we tried to have import {connectSearchBox} from 'instantsearch.js'; instead of having import {connectSearchBox} from 'instantsearch.js/connectors'; Then we should do the later instead (instantsearch.js/connectors).
import * as connectors from 'instantsearch.js/connectors';

This should work for all envs but the use case may be edgy.

So to comply with a non-breaking changes update and we want the tree shaking to be working, we have to separate all the "main" modules this way:
// instantsearchjs object without .connectors/.widgets property
import instantsearch from 'instantsearch.js/instantsearch';

import * as widgets from 'instantsearch.js/widgets';
import * as connectors from 'instantsearch.js/connectors';

This will imply documentation and won't work "out of the box" for existing projects, but it's non breaking and will provide really good results when you are not importing widgets or connectors.

This will imply documentation and won't work "out of the box" for existing projects, but it's non breaking and will provide really good results when you are not importing widgets or connectors.

Seems fine, in another major version we could ditch the version where we allow for importing the whole instantsearch + instantsearch.widgets* ... So users (of webpack/browserify) would be forced to uniquely ask for each method they want to use. Which is fine when you think about it (react is does that, nobody complains)

I've added a paragraph about tree-shaking into usage.md
I've published a v2.1.0-beta.1 version with tag beta on npm in order to test correctly

HI @iam4x,
how are you?
My name is Asaf, and I'm the author of autodll-webpack-plugin.
I'm here because I searched GitHub for projects which currently implement my plugin to learn from their use cases, and see if they encountered any issues with it.
I noticed that the Plugin's context is incorrect, it should point to the root of your project.
So instead of:
new AutoDllPlugin({
    inject: true,
    context: __dirname,
    filename: '[name].[hash].js',
    entry: {vendor: Object.keys(require('../package.json').dependencies)},
})

It should be:
new AutoDllPlugin({
    inject: true,
    context: path.join(__dirname, '..'),
    filename: '[name].[hash].js',
    entry: {vendor: Object.keys(require('../package.json').dependencies)},
})

Incorrect context will result in having the same modules in DLL bundle and in the main bundle too.
That silent error comes from webpack's own DllPlugin itself.
I plan to add warnings about it soon.
Stats before the change:
Time: 8901ms
                                Asset       Size  Chunks                    Chunk Names
       bundle.bcfcc724970cd0307929.js    5.96 MB       0  [emitted]  [big]  bundle
instantsearch.bcfcc724970cd0307929.js     4.1 MB       1  [emitted]  [big]  instantsearch
                           index.html  690 bytes          [emitted]
       vendor.df506d98aa0c53bd74ce.js    1.43 MB          [emitted]  [big]
Child html-webpack-plugin for "index.html":
         Asset   Size  Chunks  Chunk Names
    index.html  27 kB       0

Average Rebuild:  ~950ms
Stats before the change:
Time: 5516ms
                                Asset       Size  Chunks                    Chunk Names
       bundle.6873c9409851d5449e9d.js    3.96 MB       0  [emitted]  [big]  bundle
instantsearch.6873c9409851d5449e9d.js     2.1 MB       1  [emitted]  [big]  instantsearch
                           index.html  690 bytes          [emitted]
       vendor.df506d98aa0c53bd74ce.js    1.43 MB          [emitted]  [big]
Child html-webpack-plugin for "index.html":
         Asset   Size  Chunks  Chunk Names
    index.html  27 kB       0

Average Rebuild:  ~550ms
You can optimize it even more by adding the "dev-novel" module to the DLL entries too.
Average Rebuild:  ~450ms
Also, please update to version 0.0.8 as it fixes a critical bug.
Tip: add debug: true to the Plugin's settings to see more clearly what the plugin is doing.
Please let me know if you need any further help.
Asaf.

Thanks @asfktz that's awesome. This PR looks almost complete. Only @iam4x should be able to say so, let's wait a bit (he's currently not available).
Once we have more news, this is a great chore AND feature

@vvo 👍
Let me know if you need me.

To keep fix easy to deploy, I propose that we reopen this PR targeting feat/2.1 instead of develop. Thanks :)

Thank's a lot for your advices and code review @asfktz 👍

Closing it in favor of #2240


Fix SKI computation in verification tests

Currently, we are computing the subject key identifier as the hash of 0x04 || pubkey.x || pubkey.y and checking that it matches what's in the cert in the go verification tests. The pubkey is also pulled from the cert and we get its byte representation through big.int::Bytes.
For our purposes, this function is error-prone as it returns the absolute value of the pubkey coordinate. If the coordinate's first bit is 1 (i.e. the coordinate is negative), we would get the wrong byte representation and our hash would be incorrect. So instead of pulling the pubkey from the cert, in this PR we just pull it from the certifyKeyResponse since it's guaranteed to always have the correct pubkey representation. This should fix this flaky test failures in Caliptra.

Just to clarify the diagnosis, I think the issue is really just that big.Int will remove any leading zeros when writing to a byte slice. To fix this, the general solution is to use FillBytes(). This will ensure any padding zeros are kept.

Just to clarify the diagnosis, I think the issue is really just that big.Int will remove any leading zeros when writing to a byte slice. To fix this, the general solution is to use FillBytes(). This will ensure any padding zeros are kept.

But anyway, this PR does fix the problem as well.


chore: bump ring to 0.17

 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.


Fix multiple objects in extension! macro

Codecov Report
Attention: Patch coverage is 42.85714% with 4 lines in your changes missing coverage. Please review.

Project coverage is 81.78%. Comparing base (0c7f83e) to head (9135e8a).
Report is 189 commits behind head on main.

Files with missing lines
Patch %
Lines

testing/checkin/runner/ops.rs
0.00%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #993      +/-   ##
==========================================
+ Coverage   81.43%   81.78%   +0.34%
==========================================
  Files          97       98       +1
  Lines       23877    25787    +1910
==========================================
+ Hits        19445    21090    +1645
- Misses       4432     4697     +265

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


try to reuse the channel instead of opening new channel everytime this endpoint got hit

This issue has been automatically created by todo-actions based on a TODO comment found in pkg/apiserver/util/healthcheck.go:48. It will automatically be closed when the TODO comment is removed from the default branch (main).

fix e1ea9bce8093a1931d2ef3d1507e52a6b6a5adb6


Tag v1 doesn't exist

uses: dotnet/code-analysis@v1
Doesn't work as no tags exist in this project.

Like in the examples, you have to use dotnet/code-analysis@main. But yes, the documentation is not correct here.


Cannot set worker-processes to auto

The nginx.conf support setting worker_processes as auto as following:
worker_processes auto;

But ingress controller only support using  int for worker_processes from here https://github.com/kubernetes/ingress/blob/master/controllers/nginx/pkg/config/config.go#L255
We should support auto for this case.

The default is set to the recommended method of estimating the value.
From the docs:
When one is in doubt, setting it to the number of available CPU cores would be a good start (the value “auto” will try to autodetect it).

Yes, so that's why I want to enable auto for worker_processes, as sometimes the worker nodes may have different CPU cores and we may need to set it as auto for auto detect. @cmluciano

@gyliu513 @cmluciano this is the actual behavior. The controller checks and uses the number of CPUs as default.
Just to allow the setting I opened #508

Thanks @aledbf for the fix!


Dates not translated when autodetect

When a user's language is set to auto detect, the text correctly changes, but the dates do not. They should also auto detect the language. They work as expected when a language is explicitly set.

 Localisation Support

Caused by a race condition. For slower connections, the page is drawn before moment's locale is set. Setting moment's locale doesn't refresh the dates.


ephemeral_disk_type: local-ssd is not compatible with persistent disks

As reported by Tim Leong:
The BOSH Agent incorrectly looks for /dev/sdc for the persistent disk instead of looking for /dev/sdb (which would normally be the ephemeral disk, but the ephemeral disk is using /dev/nvme, so the persistent disk is now /dev/sdb).
Stemcell 1.360
When deploying to GCP, we witness the following failure:
          Task 956 | 18:57:47 | L executing post-stop: worker/3ab61249-2d9b-41b7-83c0-6630c017824e (0) (canary) (00:04:07)
                              L Error: Action Failed get_task: Task b3bdddd4-f917-4740-795b-cf9c9ee62152 result: Adjusting persistent disk partitioning: Getting real device path: Resolving mapped device path: Timed out getting real device path for /dev/sdc
          Task 956 | 19:01:52 | Error: Action Failed get_task: Task b3bdddd4-f917-4740-795b-cf9c9ee62152 result: Adjusting persistent disk partitioning: Getting real device path: Resolving mapped device path: Timed out getting real device path for /dev/sdc

The VM_types:
vm_types:
            - cloud_properties:
                accelerators:
                - count: 1
                  type: nvidia-tesla-t4
                ephemeral_disk_type: local-ssd
                machine_type: n1-standard-8
                on_host_maintenance: TERMINATE
                root_disk_size_gb: 25
                root_disk_type: pd-ssd
              name: gpu-t4
            - cloud_properties:
                accelerators:
                - count: 1
                  type: nvidia-tesla-v100
                ephemeral_disk_type: local-ssd
                machine_type: n1-standard-8
                on_host_maintenance: TERMINATE
                root_disk_size_gb: 25
                root_disk_type: pd-ssd
              name: gpu-v100
            - cloud_properties:
                machine_type: g2-standard-8
                on_host_maintenance: TERMINATE
                root_disk_size_gb: 50
                root_disk_type: pd-ssd
              name: gpu-l4

That wasn't the problem.


docs: add Scope community baseline documentation

Stolen and adapted from the CNI project https://github.com/containernetworking/cni

In general: great work! I left a couple of comments: some of it is just typos or stuff which can be easily clarified. A few other bits probably deserve some discussion.

In general this looks good to go as a r1.
In general these bits still might need figuring out though at some point.

Any DCO signing steps necessary? Either way, document.
Figure out who is member of the set of maintainers who are "grandfathered in".
Consider pointing to examples of good tests, the list of currently endorsed plugins and other places of interest.
Figure out acceptance criteria for plugins.
Update mission statement of the scope community.

As I haven't been too closely involved with the Scope community yet, I'm a bit hesitant giving a +1 on a document which defines voting processes, etc. I'd be happier if the process portions of the docs wer indicated as a draft and discussed and approved in a meeting soon.
In general this is great work and I'm happy with it to get more of a governance around scope started.
I went ahead and filed issues for the other action items out of this PR and used the governance label.

As I haven't been too closely involved with the Scope community yet, I'm a bit hesitant giving a +1 on a document which defines voting processes, etc.

There hasn't been a true community, which is what we are trying to change. My idea is to provide a baseline we can evolve from.

I'd be happier if the process portions of the docs wer indicated as a draft and discussed and approved in a meeting soon.

The governance document is explicitly tagged as Work in Progress in its first paragraph.

I went ahead and filed issues for the other action items out of this PR and used the governance label.

Thanks! I think we should merge this and use those tickets to fill in the gaps. Feel free to create another ticket to review the voting scheme if you think that's necessary.

Cool. WFM. I'll file the ticket. Just to make sure people can have their say.


[CUR-940] Switch curriculum visualiser to newer MV

Description
Music year: 2005

List of changes

This change consolidates the curriculum visualiser and docs to one MV rather than two.
I've run this through the curriculum visual testing tool and can see no visual changes to units.
How to test

Go to https://deploy-preview-2853--oak-web-application.netlify.thenational.academy/teachers/curriculum
Check the units are correct

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

:tada: This PR is included in version 1.580.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Frequency and time meshes

Implemented changes in GW, BSE, DMFT using the new FrequencyMesh and TimeMesh.
Fixed Wannier90 to populate Method.k_mesh and not Projection.wannier.k_mesh

I think I don't need review for this, as it is a refactoring, but @ladinesa can you check everything is good? Thanks!

@ndaelman-hu this interests you.

Everything looks good.

sec_k_mesh.grid = self.out_parser.get('k_grid', [1, 1, 1])

So, in the FHIaims parser, you default the k_mesh.grid to the origin.
To my understanding, AIMS does not have a guaranteed PBC (check the atom_frac tag in the manual).
Actually, this would be a case with variable dimensionality, as is CP2K.
From how I understood @ladinesa , we should for example write (1, 1, 0) in the 2D case. Anyhow, the default seems too restrictive.

@JosePizarro3 Okay, done with the review. Minor comments, overall, please resolve the threads and we can merge.

sec_k_mesh.grid = self.out_parser.get('k_grid', [1, 1, 1])

So, in the FHIaims parser, you default the k_mesh.grid to the origin. To my understanding, AIMS does not have a guaranteed PBC (check the atom_frac tag in the manual). Actually, this would be a case with variable dimensionality, as is CP2K. From how I understood @ladinesa , we should for example write (1, 1, 0) in the 2D case. Anyhow, the default seems too restrictive.

I can leave it empty to be None

@JosePizarro3 Do we merge?

I think not. I should do the minor changes. Furthermore, we should try to coincide with #79 and then update the ref in gitlab right? Otherwise, the days diff will cause a lot of troubles.

Please feel free to add the changes that you feel need adding.
Regarding merging inot #79 , it's not that big of a deal: I can do the merge cleanly in VS Code, so you don't have to worry about that.

Ok, done. Please, double-check it before merging.


Add operator>= and operator<= on Vector

Add operator>= and operator<= on Vector
Signed-off-by: Bensuperpc<EMAIL_ADDRESS>

Thanks @bensuperpc! Should we instead compare their magnitudes instead? Comparing their individual coordinates may not get the desired results.

Thanks @bensuperpc! Should we instead compare their magnitudes instead? Comparing their individual coordinates may not get the desired results.... In raylib, the lengths.

I improved this PR, What do you think ?


width: 0

you define "width: 0" in css, so we can't take % tables..

If you want to use percentages you should do the calculation and apply the widths accordingly.

but it works perfectly fine if i change it to "100%" , i can define the colums per percent.

Can you send an example on jsfiddle?


Error happened on calling GetblockInfo RPC

loginfo: Error happened on calling GetblockInfo RPC
run the example has error, what can i do for it?

@opi-smccoole

Hi, sorry for the delay.  It is the Easter holiday weekend here in the states, so I can only give a quick answer.  The 1.0.0 versions of the SDKs and the examples are meant to align with the 2.1 release of EOS, which added get_block_info as a more efficient way of getting information for calculating TAPOS for transactions.  Older versions of EOS and the SDKs used get_block which was much less efficient.  We are looking at a change to the SDKs to detect the EOS version and act accordingly but do not have an ETA for that yet.  In the mean time we are recommending that either you use an older version of the SDKs or work around the problem by getting the block information yourself via get_block and setting the TAPOS fields in the transaction yourself before trying to sign and send it.  I'll see in the mean time if we can't find a way to show an example of that workaround.

Thank you for your reply and look forward to your new version of Android demo
Happy holidays


write up explanation of translation of simple example to python in notebook

write up explanation of translation of simple example to python in notebook

subtask of #2


typo

fixed minor typo

@sveinrune : Thanks for your contribution to the Azure documentation! The author, @CarlRabeler, has been notified to review your proposed change.

#sign-off


Add influx reporter support

I'm creating pull request with influxdb reporter, however it is not ready for merge. It is waiting for https://github.com/iZettle/dropwizard-metrics-influxdb/pull/48 , description: https://github.com/iZettle/dropwizard-metrics-influxdb/issues/47 . I am creating PR so that it would be visible that influx reporter is already "in progress" :)

@jhorwit2

I'd prefer you call it prefix so it matches ganglia/graphite.

It is a map now, however there is an issue with prefix naming. If I would like to use this name, then I have to resign from using AbstractHostPortReporterConfig because of course types do not match. Again there are a few options:

Use tags name instead of prefix, since prefix might be misleading for influx users :)
Resign from using AbstractHostPortReporterConfig as parent and refactor AbstractHostPortReporterConfig so that "resolving" logic will be accessible for new class
Resign from using AbstractHostPortReporterConfig as parent and resign from "resolving" logic for tags values.

Although dependency was updated, not yet ready for merge.

@jhorwit2 please review latest changes. dropwizard-metrics-influxdb updated, additionally they have added "prefix" in the constructor.

LGTM.
@yuesong PTAL.

ping @yuesong @jhorwit2


Symfony Flex compatibility (namespaced syntax)

@pixocode Thanks! Can you take a look to the falling tests?

@egeloen I forgot the test file that tested the accuracy of the parameter I changed :)

Do you confirm that twig (shipped with Symfony 2.7.0) already supports namespaced templates? or do we need to bump the twig version? (The composer.json currently uses "twig/twig": "^1.12")

Supported on version 1 and 2
"The filesystem loader also supports namespaced templates. This allows to group your templates under different namespaces which have their own template paths."
"Namespaced templates can be accessed via the special @namespace_name/template_path notation"
https://twig.sensiolabs.org/doc/1.x/api.html
https://twig.sensiolabs.org/doc/2.x/api.html

:+1 Thanks!: Can't release a new minor version right now but I will do it later this week.


RFC: Go 2: user definable index operators

This is just an draft for discussion, not a formal proposal yet.
Author background

experienced, already touched toolchain code (also packagjng it for various distros)
other languages: too many to list'em all ;-)

Related proposals

haven't found similar proposal yet
no impact on error handling or generics

Precedence in other languages

python translates index operator to get() and set() methods
any class can implement them and so provide index operator
IOW: the [xxx] notation is just syntactic sugar for method calls
c++ allows custom operator implementations, which also go down to function/method calls, under the hood

Proposal

add support for user defined index operations ([foo]), these are routed to function/interface calls
helpful for reducing boilerplate for typical index-like operations
by employing interfaces, consumer code can be made agnostic of the actual type its operating on, while still using the compact slice/map-like syntax
by implementing corresponding interface on maps or slices themselves, those can be directly used whereever the that interface is demanded

How does it work:

when compiler encounters an index access on some custom type (or interface) it looks for a matching getter/setter function (exact naming scheme yet to be defined)
the compiler just rewrites it to a direct getter/setter call (ast transformation) and tries to resolve the function call the usual way.
so now, instead of foo.Get(bar) one can just write foo[bar], as well as foo[bar] = x instead of foo.Set(x)
the getter actually returns the value type as well as an ok flag
Yet leaving out iterators, since that's orthogonal to index operator and IMHO should be done independently.

Compatibility:

existing code remains uneffected
side effect: previously broken (non-compilable) code could become valid (if there happens to be an existing function that looks like a getter/setter by accident), but the practical risks should be quite zero.
error messages on broken code (trying index access on type that doesnt support it) might look differently - might complain on missing function now (depending on actual implementation). OTOH, that might be actually more useful than harmful, by giving the dev better hints on what he shall do, if he really wants index access

Costs

adds a little bit more complexity to the language itself
readers of source, who dont know this feature yet, might wonder how code using it could ever work/compile at all
affected tool: only those who actually with compiling (not just formatting etc)
tiny increase of compile time for code using that feature: at the point build breaks now, we'll have little ast transformation and continue compiling
no runtime overhead

while it doesn't specifically mention indexing, I think operator overloading should be folded into #27605

while it doesn't specifically mention indexing, I think operator overloading should be folded into #27605

No, this is quite different. The one you've mentioned deals with arithmetic operators. And explicitly demands the operants to be immutable, which totally contradicts the index operator, since changing elements belongs to its sole purpose.
Oh, and closing the ticket (whose sole purpose is starting a discussion) without any discussion is quite the opposite of opensource culture.
I wonder how @ianlancetaylor or @rsc, @robpike have to say about this.

We are not going to support overriding the [] operator without support overriding of other operators.  All the same problems arise for both kinds of overriding.  So I think that de-duping this proposal is correct.
I appreciate that you are looking for a discussion.  But we've found that GitHub issues are not a good place for that.  They lack threading and if many people comment then comments are hidden by default and difficult to find in practice, leading to constant repetition.  The proposal process is really for specific proposals.  For discussions, please use a forum like golang-nuts.  Thanks.


feat: support whole config as env

Follow-up for #6143

Uses custom decoders for complex data structures, so all config properties can be configured via env variables
Fixes some wrong env var descriptions

Definition of Ready

[x] I am happy with the code
[x] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[x] Acceptance criteria are met
[x] All open todos and follow ups are defined in a new ticket and justified
[x] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[x] My code has no repetitions
[x] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[x] Documentation/examples are up-to-date
[x] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

@adlerhurst I resolved the conflicts


Update to new Version

Hi there,
i´m using PhpStorm with your Theme. Its incompatible to the newes Version :(  (2019.3)
Thanks a lot
Robin

Have you installed version 1.1.3 of the plugin?

Ohhhh :D i removed it, my plugin manager said its incompatible when i had a second look for it... but... reinstalled it, perfect! Perhaps some spooky caches in plugin manager shows incompatibily. Thanks a lot for support!


feature/Backend-Settings-Password-Admin-App-Sprint

@askinss
@peculiaruc
Feature Name: Password Management
Description:
The password management feature is an essential component of the backend system that allows users to securely change or reset their passwords. This feature ensures that users have control over their account security and can update their passwords as needed to maintain the confidentiality and integrity of their account information.
The password management feature includes the following functionalities:
Password Change: Users can change their existing password by providing their current password and specifying a new password. The backend system verifies the current password for authentication purposes and then securely stores the new password using industry-standard encryption techniques to protect it from unauthorized access.
Password Reset: In case a user forgets their password, they can request a password reset. The backend system generates a secure password reset link or token and sends it to the user's registered email address or phone number. The user can then use this link or token to reset their password by providing a new password. This helps users regain access to their account while maintaining the confidentiality and security of their password.
Password Strength Requirements: The password management feature enforces strong password policies to ensure that user accounts are protected with robust passwords. This may include requirements such as minimum length, a mix of uppercase and lowercase letters, numbers, and special characters to enhance password security.
Password History: The backend system may also keep track of users' password history to prevent password reuse. This helps users maintain unique and secure passwords, reducing the risk of unauthorized access to their account.
Error Handling and Security Measures: The password management feature includes robust error handling mechanisms to prevent any security loopholes or vulnerabilities. It may include measures such as rate limiting, CAPTCHA, and multi-factor authentication to further enhance the security of password-related operations.
Overall, the password management feature is a critical part of the backend system that ensures users can securely change or reset their passwords, promoting strong account security and protecting against unauthorized access to user accounts.

@askinss @peculiaruc Feature Name: Password Management
Description: The password management feature is an essential component of the backend system that allows users to securely change or reset their passwords. This feature ensures that users have control over their account security and can update their passwords as needed to maintain the confidentiality and integrity of their account information.
The password management feature includes the following functionalities:
Password Change: Users can change their existing password by providing their current password and specifying a new password. The backend system verifies the current password for authentication purposes and then securely stores the new password using industry-standard encryption techniques to protect it from unauthorized access.
Password Reset: In case a user forgets their password, they can request a password reset. The backend system generates a secure password reset link or token and sends it to the user's registered email address or phone number. The user can then use this link or token to reset their password by providing a new password. This helps users regain access to their account while maintaining the confidentiality and security of their password.
Password Strength Requirements: The password management feature enforces strong password policies to ensure that user accounts are protected with robust passwords. This may include requirements such as minimum length, a mix of uppercase and lowercase letters, numbers, and special characters to enhance password security.
Password History: The backend system may also keep track of users' password history to prevent password reuse. This helps users maintain unique and secure passwords, reducing the risk of unauthorized access to their account.
Error Handling and Security Measures: The password management feature includes robust error handling mechanisms to prevent any security loopholes or vulnerabilities. It may include measures such as rate limiting, CAPTCHA, and multi-factor authentication to further enhance the security of password-related operations.
Overall, the password management feature is a critical part of the backend system that ensures users can securely change or reset their passwords, promoting strong account security and protecting against unauthorized access to user accounts.

Noted. Good job!


Trademark Symbol

The trademark symbol on the navigation item 'Hypothesis Engine' should use the HTML Name Code of ™. This will prevent any issues with the superscript positioning while using icons.

@almighty/uxd can you please confirm with @qodfathr if we are going to keep the "TM" as I know the phrase existed as early as 2009.

It was just a joke. That tab isn't even part of the current UX plan. All of the names on the tabs, and the # of tabs, are just placeholders.
For now, we just need to work towards a set of "extensions" which are discovered at runtime and via their manifest/api/whatever provide a name to the system for display. Until we've got that, the "fake" placeholder tabs aren't too important (although having them be thematically correct helps).
In other words, don't stress out about this.

Can we close this for now, in that case?

Fine with me to close.

FYI... I will be updating the tab names this week.... and they will
probably continue to change... :)
On Wed, Nov 30, 2016 at 8:46 AM, Sudipta Sen<EMAIL_ADDRESS>wrote:

Closed #366 https://github.com/almighty/almighty-ui/issues/366.
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/almighty/almighty-ui/issues/366#event-876869076, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARTwqw26Sh381AcpkaoyvIUhnByCV3Zyks5rDX4ggaJpZM4K-QuT
.


Adding card manager class

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

555 of 555 (100.0%)  changed or added relevant lines in 24 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
826

Relevant Lines:
826

💛  - Coveralls


Twisted/Occult Item Aliases

Added Twisted Ancestral and Occult necklace (or) aliases, so you can use monster boosts with those instead

These already work, but only when they're in your bank. I fixed the bug that prevents the match on equipped items in my slayer PR, so the update is on the way already.

fixed in cyr’s slayer pr


Close #26: Filter PRs by updated date instead of commit date

This supersedes #205.
This resource can inadvertently miss Pull Requests due to out-of-order
commits across PRs. If PR#2 is opened after PR#1, but the head commit
of PR#2 is older than the head commit of PR#1, the resource will not
include PR#2 in the list of new versions provided to Concourse.
In #205, I removed the date filter entirely.  This ensures that the PR
resource will find all PRs that match the explicitly-configured filters.
While Concourse can detect and ignore duplicate versions, it has to run
a database query for every version returned by a check, so removing
the date filter entirely would increase load on a Concourse database.
(That said, I'm not sure whether this increased load is a particular
concern, and other resources don't seem to make much effort to avoid
returning duplicate versions from a check.)
To avoid that extra load on a Concourse database, this change instead
replaces the filter by commit date in check.go with a filter by updated
date in the GraphQL query to list pull requests.  This should reduce the
number of duplicate versions returned by a check while still allowing
the PR resource to detect PRs with out-of-order head commits.

Hi,
Thanks for the contribution, when I get the time, I'll see if these changes resolve our issues. I had the same implementation in private branch of this, but I made the mistake of digging myself a hole trying to make some of the queries optional. But going back to the basics is a great idea.
Do you have an image available of this ready to be tested?

@rickardl Can this be addressed sooner, I have bunch of PRs opened some time back and label added today not triggering build.

I'm somewhat waiting for a confirmation that this resolves the issue, I haven't seen a result from the test suite being run here.

GitHub says it's not going to run the tests for this PR until a maintainer approves them.

Hello, I made my own docker image based on this PR, and it solved our issue, the PR that wasn't seen by the CI, even with fake git commit is now seen correctly and has been processed by our CI :tada:
Thanks for this patch ! :heart:

Hello, any news on this one?

I'm tired of waiting, people are yelling. Started to write my own PR resource. 😂

Is there any way a maintainer can allow the test jobs to run so we can confirm if this PR is mergeable?  I ran the tests locally and they passed, but they won't run in GitHub unless a maintainer approves them.

Any updates on this?  I'm pretty sure this passes all of the tests, but I the PR checks won't run unless a maintainer allows them to run.  We continue to run into this issue on a regular basis and would prefer to have an official release with the fix rather than running a fork.

@ctreatma
Thank you for this fix, I decided it will be faster to just get your commit and push own image to docker repository and use that instead of telia-oss. And now PR''s are fetched correctly


syntax supporting a sequence of declarations of same type?

Is there no syntax to allow a sequence of fields to be declared,
Eg, I would like to write
    actor Client
         var   i1, i2 : I32
but this is a syntax error.
Is there an alternative way of doing this, rather than
    actor Client
         var   i1: I32
         var   i2: I32

Thanks, I was wondering whether it was an “issue”
Sophia
From: Sean T Allen<EMAIL_ADDRESS>Reply-To: ponylang/ponyc<EMAIL_ADDRESS>Date: Saturday, 4 March 2017 at 19:33
To: ponylang/ponyc<EMAIL_ADDRESS>Cc: "Drossopoulou, Sophia"<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ponylang/ponyc] syntax supporting a sequence of declarations of same type? (#1635)
Hi Sophia, please address questions of this sort to mailing list. You'll find a link to it in the README.
Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ponylang/ponyc/issues/1635#issuecomment-284175313, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGixcfFawE6dj5PFDtKpctHbpkrrh0Zgks5ribyUgaJpZM4MTJr3.

@sophiaIC - Pony doesn't currently have that kind of abbreviated syntax for multiple references of the same type.  If we were to add it, it would have to go through the RFC process.


EN server support

Added EN server support

Thank you for contributing! I thank you on behave(?) of all the EN server players.
By the way, beware not to post this on reddit / popular sites. Only smaller ones or private msg maybe.

Another reason is that Delightworks can simply blocked the daemon process used here, and I'll have to ask the author of Ankulua framework to deal with it - and it might be impossible to do so just like ANRC's development currently on hold.
Ahh, but I guess EN server is free from this root blocking crap? : p

I agree. Let's keep this low profile. I think they are heavily against automation, cuz emu has the highest chance to automate the game, hence they block them. EN server is the same in blocking root/jailbreak. Can't run the game even just with USB debug mode on...
I used to play on JP server until last year when they decided to block jailbreak devices. My customization on Ipad has more values than the game so I just quit :p

@ryuga93
Hi, I noticed the option setImmersiveMode(true) has been set by you in the English version.
How's the virtual button row affecting FGO? I'd like know the behavior, and if setImmersiveMode(true) affects the script performance - since this makes every compare action done twice.

I haven't test it without setImmersiveMode. When I first used your script, all of the coordinates are wrong, but that could be because I changed the setScriptDimension to my resolution instead of 2560.
However, my phone (Xperia Z2) has virtual button row and the game will fill up the whole screen. Using the Ankulua calculator script, the yellow touchable region will only cover 1776 pixels instead of the full 1920 pixel without setImmersiveMode. So I just remain it there even after changing back setScriptDimension to 2560.
A better solution is to create a menu for user to select, but I think since it is open source, anyone can change it once in the code and run the script directly without going through the menu every time.

Thank you for the info, it helps when I encountered virtual button issue in the future!
By the way you don't really have to change scriptDimension - it's gonna auto adjust to your screen resolution, that's only useful for script writing - you don't want to calculate for coords everytime you grab a new pic for recognition.

Yup, thanks for that. I noticed the fact after looking at the Chinese doc. English doc did not have much detailed documentation.


Missing Sack in MIA @ SEA (Week 1, 2016)

Giving my fair share of issues lately whilst I develop my fantasy scoring system hahaha (sorry in advance and future). And I've nearly got there with lining up all the player's scores with the current league.
One issue that's cropped up is that in this particular game (could be for others at the same time), is that the final play of the game (a sack missing from Seattle) isn't in the game's drive.
The sequence in nflgame for the final drive:
(SEA, SEA 35, Q4) S.Hauschka kicks 69 yards from SEA 35 to MIA -4. J.Grant to MIA 18 for 22 yards (K.Pierre-Louis).
(MIA, MIA 18, Q4, 1 and 10) (:26) (Shotgun) R.Tannehill pass incomplete deep left to K.Stills.
(MIA, MIA 18, Q4, 2 and 10) (:20) (Shotgun) R.Tannehill sacked at MIA 9 for -9 yards (C.Marsh). FUMBLES (C.Marsh), ball out of bounds at MIA 11. Ball marked on the 9 because fumble forward not allowed.
END GAME
Here's what it should be (looking at game center http://www.nfl.com/gamecenter/2016091109/2016/REG1/dolphins@seahawks?#menu=gameinfo&tab=analyze&analyze=playbyplay )
(SEA, SEA 35, Q4) S.Hauschka kicks 69 yards from SEA 35 to MIA -4. J.Grant to MIA 18 for 22 yards (K.Pierre-Louis).
(MIA, MIA 18, Q4, 1 and 10) (:26) (Shotgun) R.Tannehill pass incomplete deep left to K.Stills.
(MIA, MIA 18, Q4, 2 and 10) (:20) (Shotgun) R.Tannehill sacked at MIA 9 for -9 yards (C.Marsh). FUMBLES (C.Marsh), ball out of bounds at MIA 11. Ball marked on the 9 because fumble forward not allowed.
(MIA, MIA 9, Q4, 3 and 19) (:00) (Shotgun) R.Tannehill sacked at MIA 1 for -8 yards (C.Avril).
END GAME
Not sure if this is a known issue or not?

Once the game goes final, nflgame flips the switch on that game and makes no attempt to ever see if the game has been updated in anyway.
One thing you can try (heavy emphasis intended) is to delete that game from the gamecenter-json and see if the game downloads again with the complete set of plays.  But your mileage may very on this since I don't believe the game center json feed is actually the source of play-by-play data; it only feeds the game center.  In most cases, especially stat corrections, the NFL will modify the play-by-play found on pages like the one you linked without modifying the game-center json that we rely on.
import nflgame

games = nflgame.games(2016, 1, 'SEA', 'SEA', 'REG')
plays = nflgame.combine_plays(games)

print [game.eid for game in games]
print 'Final 5 plays:'
for play in list(plays)[-5:]:
    print play

[u'2016091109']
Final 5 plays:
(SEA, MIA 15, Q4) S.Hauschka extra point is Blocked (J.Jones), Center-N.Frese, Holder-J.Ryan.
(SEA, SEA 35, Q4) S.Hauschka kicks 69 yards from SEA 35 to MIA -4. J.Grant to MIA 18 for 22 yards (K.Pierre-Louis).
(MIA, MIA 18, Q4, 1 and 10) (:26) (Shotgun) R.Tannehill pass incomplete deep left to K.Stills.
(MIA, MIA 18, Q4, 2 and 10) (:20) (Shotgun) R.Tannehill sacked at MIA 9 for -9 yards (C.Marsh). FUMBLES (C.Marsh), ball out of bounds at MIA 11. Ball marked on the 9 because fumble forward not allowed.
END GAME

With<PHONE_NUMBER> in hand, I go and find the corresponding<PHONE_NUMBER>.json.gz file in my nflgame\gamecenter-json directory and delete it.
Running the above again will force download the json for that game.  Unfortunately in this case, it results in the exact same set of plays being displayed since the NFL never bothered to go back and update the gamecenter json file.
I sympathize as I battle this exact problem as well.  For this reason I maintain a nasty looking "manual updates" section in my code that handles these.  In fact, if you were to review my livescoring script, you would find the following:
        if year == 2016 and season_type == 'REG':
            if week == 1:
                # (:00) (Shotgun) 17-R.Tannehill sacked at MIA 1 for -8 yards
                # (56-C.Avril).
                defense_results['SEA'][8] += 1
                defense_results['SEA'][10] += 20

Where I had to manually credit the Seattle defense with an additional sack and the 20 points that was worth in my league.  (Since no one was playing Tannehill I didn't bother manually adjusting his score in my league)

So deleting the JSON file won't give the correct plays? Assuming I would only be doing what you did which didn't get the change.
Funnily enough, if it was an issue that may well not be solved I had the same idea to add a table in my database for manually modifying scores etc. (Was likely to be needed at some point, because this library is amazing but unfortunately not perfect).
Thanks for the help @ochawkeye

So deleting the JSON file won't give the correct plays?

Not in this case, no, since the gamecenter json source doesn't have the correct plays.  I have encountered cases in the past, though, where deleting a game and re-downloading did change the plays that were contained.


Docs: Add related processes

Pull Request Test Coverage Report for Build 276

0 of 0   changed or added relevant lines in 0 files are covered.
22 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.6%) to 96.279%

Files with Coverage Reduction
New Missed Lines
%

openregistry/lots/loki/tests/base.py
1
100.0%

openregistry/lots/loki/models.py
1
97.56%

openregistry/lots/loki/tests/blanks/auction_blanks.py
20
100.0%

Totals

Change from base Build 275:
-0.6%

Covered Lines:
3778

Relevant Lines:
3924

💛  - Coveralls


Add/update api route template variable documentation

Updating the task rendering documentation to pair with https://github.com/RackHD/on-tasks/pull/218, and also removing some old, inaccurate information.
@RackHD/corecommitters @heckj @zyoung51

:+1:

:+1:


Create Contribute.md file

Create Contribute.md File For This Repo

Hi @nemynigam What is to be written exactly in this MD file?
Also, can you please assign to me

see my pull req.


Update build status

The pipelines listed in the README have all been disabled. This PR is removing them so that customers are not misled by the reported status here. I have also added the build status for the current branches in use.

@castrodd can we replace these build statuses with the pipelines from azfunc/public?

Hey, I do not see corresponding builds in azfunc/public (except for v4.x/preview): https://azfunc.visualstudio.com/public/_build?definitionId=939&view=branches

Yes, replace with whatever is in azfunc/public for this repo

Abandoning this PR so we can open these changes on the fixed main branch


Setup Changes

Installs a virtualenv for Empire. Moves bomutils and xar to a local
bin/ directory instead of installing globally. Uses #!/usr/bin/env foo
on shebangs to get the right path for the environment.

I'm not sure if these changes are wanted, but the current installation installed lots of artifacts globally that I didn't want.

@varnerac Is there a reason for the change from empire to run_empire.py?

@xorrior Yes. empire is now a shell script that activates the virtualenv and then call run_empire.py. The setup script creates the virtualenv so we don't install the dependencies globally.

Also, please don't merge this. I'd need to remove the Linux native package calls, remove the sudo requirements from the reset script and add a sanity check after installing the pip requirements to tell users what local deps they may be missing (e.g. OpenSSL).

@varnerac preferably we would like to keep the empire main script as "empire". Do you want me to close this out until you are ready to submit? Or keep it open?

@xorrior So, I just want to clarify what you mean by "main script"? In this PR, if you type
./empire

you'll start Empire. Before this PR, empire ran as a shebang'd Python script. After this PR, it runs as a shebang'd shell script that calls start_empire.py. The PR is incomplete, because the new empire doesn't copy the args to start_empire.py.
Is there an appetite for a non-sudo, non-global install of Empire? If that's not a desired outcome, I can close the PR.
I have a few questions:

Do you want me to break the PR it into smaller pieces? For example, the non-global install of xar and bomutils could be its own PR. The shebang fixes could be a separate one. The virtualenv piece could go in its own PR. I
How do we start Empire with a virtualenv? My preference is to seamlessly activate the virtualenv and start Empire using the same commands the user already does, i.e. ./empire or even ./empire arg1 arg2 ....

I'll experiment with starting the virtualenv from empire as a Python script instead of a bash script.

Ah ok, sorry if I misunderstood. You can keep the pr open and just let us know when your done so we can test.On Jun 1, 2017 12:56 PM, Andrew Varner<EMAIL_ADDRESS>wrote:@xorrior So, I just want to clarify what you mean by "main script"? In this PR, if you type
./empire
you'll start Empire. Before this PR, empire ran as a shebang'd Python script. After this PR, it runs as a shebang'd shell script that calls start_empire.py. The PR is incomplete, because the new empire doesn't copy the args to start_empire.py.
Is there an appetite for a non-sudo, non-global install of Empire? If that's not a desired outcome, I can close the PR.
I have a few questions:
Do you want me to break the PR it into smaller pieces? For example, the non-global install of xar and bomutils could be its own PR. The shebang fixes could be a separate one. The virtualenv piece could go in its own PR. IHow do we start Empire with a virtualenv? My preference is to seamlessly activate the virtualenv and start Empire using the same commands the user already does, i.e. ./empire or even ./empire arg1 arg2 ....
I'll experiment with starting the virtualenv from empire as a Python script instead of a bash script.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.

I'll try to finish this evening.

@varnerac any reason why this was closed?

I'm gonna decompose it. I'll do one PR each for:

improved shebangs
local binary installs
virtualenv

Oh ok, sounds goodOn Jun 7, 2017 7:25 PM, Andrew Varner<EMAIL_ADDRESS>wrote:I'm gonna decompose it. I'll do one PR each for:
improved shebangslocal binary installsvirtualenv
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.


Enum ApplicationDeviceTypeCodeType.php not correct?

What I see on the [eBay](http://developer.ebay.com/DevZone/XML/docs/Reference/eBay/SetNotificationPreferences.html#Samples site are these possible values:
ClientAlerts
CustomCode
Platform
SMS
and you have Browser, CustomCode, Desktop, SetTopTVBox and Wireless in the Enum file mentioned in the subject.
Or maybe Im looking in the wrong place? Im trying to get a notification type sent to eBay so that I can get notifications, but Im struggling to figure out all the code and then I ran into the above.. and wasn't sure if you hadn't updated the file or if Im looking in the wrong file those particular enums.
If you have time could you help me with figuring out NotificationEnable->EventType  and EventEnable? I must be doing it wrong since things are just not working
$request->UserDeliveryPreferenceArray = new DTSTypes\NotificationEnableArrayType();
$request->UserDeliveryPreferenceArray->NotificationEnable = new DTSTypes\NotificationEnableType();
$request->UserDeliveryPreferenceArray->NotificationEnable->EventType = DTSEnums\NotificationEventTypeCodeType::C_END_OF_AUCTION;;
$request->UserDeliveryPreferenceArray->NotificationEnable->EventEnable = 'Enable';

I realize the NotificationEnable is supposed to be multiple arrays, so Im sure what I am doing wrong is simple, but Im struggling :) Once Im done with all of this, I need to submit a lot of this code for your example pages :)

Ok I figured out the notifications thing:
$request->UserDeliveryPreferenceArray = new DTSTypes\NotificationEnableArrayType();

// set End of Auction notification
$notification = new DTSTypes\NotificationEnableType();
$notification->EventType = DTSEnums\NotificationEventTypeCodeType::C_END_OF_AUCTION;
$notification->EventEnable = DTSEnums\EnableCodeType::C_ENABLE;
$request->UserDeliveryPreferenceArray->NotificationEnable[] = $notification;

// set Item Listed notification
$notification = new DTSTypes\NotificationEnableType();
$notification->EventType = DTSEnums\NotificationEventTypeCodeType::C_ITEM_LISTED;
$notification->EventEnable = DTSEnums\EnableCodeType::C_ENABLE;
$request->UserDeliveryPreferenceArray->NotificationEnable[] = $notification;

Instead of the ApplicationdeviceTypeCodeType I used the DeviceTypeCodeType


Some Method implementation

Adding some method.

@jaderdev Seems like the issue is not solved by the assignee. If you solve the conflicts I'll merge your PR :)

Finaly ready to merge.


Netflix-proxy in other region (Germany)

I set up a netflix-proxy in a host in Germany. However, DNS resolution seems not to work correctly. An address (e.g. zdf.de) configured in zones.override does not resolve. If it is not configured in zones.override (e.g. ard.de), it does resolve.
I have both zdf.de and ard.de configured in OSX in /etc/resolver with the same dns IP address of my server in Germany.
How can I debug/investigate further, where the problem is?
By the way, I am running a similar server in the US which works fine for the US hosts.

Perhaps start by doing some lookups on the server itself? (e.g. dig foo.bar @localhost).

Please register for updates on the mailing list (http://netflix-proxy.belodedenko.me/) RE: August update. An alternative (paid subscription based) unblocking solution is close to being completed and should be going to trial soon.


mean & standard deviation

Hi David Hall,
Im quite new to breeze, please how do I find the mean and standard deviation of a single column DenseMatrix?
like in numpy X[:,0].mean()
thanks.

Hi,
can anyone help out?


My quests have disappeared but only in game

Where else have you looked for answers to this question?

I've googled and searched the spigot forum but thus far have been unable to find anyone with an answer to my dilemma.

Share your question with as much detail as possible?
So, basically, I've been making quests on my server for many months now and on different servers for years but somehow, as I was finishing up this one quest and ran /questadmin reload it deleted all of the quests. Forced me and others on the server out of already active quests and has wiped all contents of /quests list. However, in the console all of the quest information is still there in the quest.ytml thing. The only missing data in console is player data i.e. who has taken what quests. I don't know if this is an unsolvable issue or a bug or what. I'm happy to give more information if need-be but I'm not sure what else might be needed!
Thanks.

Please re-submit this as a Bug Report instead of a Question. Include all relevant information.


GPU usage

hi there,
is it possible to detect if the DNC is using the GPU when is running the train.py
thanks

Well you can log the device placement of ops in your graph:
https://www.tensorflow.org/tutorials/using_gpu#logging_device_placement
namely using
tf.Session(config=tf.ConfigProto(log_device_placement=True))
this prints out all op -> device placements. If the device ends in /gpu:0 (or /gpu:X where X is some integer) then it has been placed on the GPU.

this is my Output:
Device mapping: no known devices.
any advice please?
thanks

There is probably a problem with TensorFlow locating your nvidia driver, e.g. this is discussed in: drorlab/tf-singularity#3.


我居然安装不了

太失败了，按照教程安装结果出现500服务器错误
我用的的vestacp控制面板，我的VirtualHost配置和你的有点不一样

发php error log

应该是这里吧
[root@fuaer public_html]# curl -sS https://getcomposer.org/installer | php
All settings correct for using Composer
Downloading...
Composer successfully installed to: /home/admin/web/kiv.pw/public_html/composer.phar
Use it: php composer.phar
[root@fuaer public_html]# php composer.phar  install
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for aws/aws-sdk-php 3.15.1 -> satisfiable by aws/aws-sdk-php[3.15.1].
- aws/aws-sdk-php 3.15.1 requires php >=5.5 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 2
- Installation request for guzzlehttp/guzzle 6.1.1 -> satisfiable by guzzlehttp/guzzle[6.1.1].
- guzzlehttp/guzzle 6.1.1 requires php >=5.5.0 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 3
- Installation request for guzzlehttp/promises 1.0.3 -> satisfiable by guzzlehttp/promises[1.0.3].
- guzzlehttp/promises 1.0.3 requires php >=5.5.0 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 4
- Installation request for illuminate/container v5.2.7 -> satisfiable by illuminate/container[v5.2.7].
- illuminate/container v5.2.7 requires php >=5.5.9 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 5
- Installation request for illuminate/contracts v5.2.7 -> satisfiable by illuminate/contracts[v5.2.7].
- illuminate/contracts v5.2.7 requires php >=5.5.9 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 6
- Installation request for illuminate/database v5.2.7 -> satisfiable by illuminate/database[v5.2.7].
- illuminate/database v5.2.7 requires php >=5.5.9 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 7
- Installation request for illuminate/pagination v5.2.7 -> satisfiable by illuminate/pagination[v5.2.7].
- illuminate/pagination v5.2.7 requires php >=5.5.9 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 8
- Installation request for illuminate/support v5.2.7 -> satisfiable by illuminate/support[v5.2.7].
- illuminate/support v5.2.7 requires php >=5.5.9 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 9
- Installation request for slim/csrf 0.5.0 -> satisfiable by slim/csrf[0.5.0].
- slim/csrf 0.5.0 requires php >=5.5.0 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 10
- Installation request for slim/slim 3.1.0 -> satisfiable by slim/slim[3.1.0].
- slim/slim 3.1.0 requires php >=5.5.0 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 11
- Installation request for slim/twig-view 2.0 -> satisfiable by slim/twig-view[2.0].
- slim/twig-view 2.0 requires php >=5.5.0 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 12
- Installation request for symfony/translation v3.0.2 -> satisfiable by symfony/translation[v3.0.2].
- symfony/translation v3.0.2 requires php >=5.5.9 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 13
- Installation request for zeuxisoo/slim-whoops 0.4.0 -> satisfiable by zeuxisoo/slim-whoops[0.4.0].
- zeuxisoo/slim-whoops 0.4.0 requires php >=5.5.0 -> your PHP version (5.4.45) does not satisfy that requirement.
Problem 14
- symfony/translation v3.0.2 requires php >=5.5.9 -> your PHP version (5.4.45) does not satisfy that requirement.
- nesbot/carbon 1.21.0 requires symfony/translation ~2.6|~3.0 -> satisfiable by symfony/translation[v3.0.2].
- Installation request for nesbot/carbon 1.21.0 -> satisfiable by nesbot/carbon[1.21.0].

requires php >=5.5.0 -> your PHP version (5.4.45) does not satisfy that requirement.
php版本太低

嗯 我升级一下版本试试  谢谢


resuming the harvest

Hi!
Did anyone of you try to implement resuming the harvest process after fail (f.e. connection lost)?
We are now harvesting big set - which is failing on "cURL error 56" (Failure with receiving network data.)
It would be great, if it would be possible to initialize the harvest just with resumption token.
Or any other idea?

Hi @igor-kamil -- that is a good idea.  Let me look into this weekend and get back to you.  At first glance, I think this would be pretty easy to implement.

I'm looking forward! Thank you @caseyamcl

Well, @igor-kamil .. first glances can be misleading :).  Let me make sure I understand the issue and possible remedy.
Are you using Guzzle or cURL to do the request?  I'm guessing cURL since you're getting a raw cURL error.  But, I need to make sure.
What I believe is happening (and you can verify) is that the client is timing out before receiving all of the data per request.  This would happen if each page of data from your OAI-PMH provider is large.  If this is the case, then upping the timeout for the cURL adapter would probably work.  It is hard-coded to 60, but I can make it configurable.
If you are using Guzzle, this is easier to fix.  Also, I can provide some assistance with retrying requests (possible for the Guzzle client; not so easy with cURL).

So, a couple of things.

I just released v2.4 with ability to set custom request parameters.  Refer to the Customizing Default Request Parameters section in the README.md file for details.  I'd suggest trying to increase the request timeout from the default of 60 seconds and see if that makes a difference.

If that doesn't work, there is a way to retry requests if you are using Guzzle.  You can use the following code to recover from errors and attempt the last request again.  I'll work on making a simpler API for this in v3, or you are welcome to submit a PR if you want.  In the meantime, this should work:

First, make sure you've installed the Guzzle Retry-Subscriber in your composer.json:
require: {
   "guzzlehttp/retry-subscriber": "~2.0"
}

Then, you can do something like this:

// Create a Retry Guzzle Subscriber that retries the last request on any failure event
$retrySubscriber = new \GuzzleHttp\Subscriber\Retry\RetrySubscriber([
    'delay' => function($numRetries, \GuzzleHttp\Event\AbstractTransferEvent $event) {
        $waitSecs = $event->getResponse()->getHeader('Retry-After') ?: '5'; // default try again in 5secs
        return ($waitSecs * 1000) + 1000; // wait one second longer than the server said to
    },
    'filter' => \GuzzleHttp\Subscriber\Retry\RetrySubscriber::createIdempotentFilter(),
]);

// Manually create a Guzzle HTTP adapter
$guzzleAdapter = new \Phpoaipmh\HttpAdapter\Guzzle();
$guzzleAdapter->getGuzzleClient()->getEmitter()->attach($retrySubscriber);

// Build PHPOAIPMH client and endpoint
$client  = new \Phpoaipmh\Client('http://some.service.com/oai', $guzzleAdapter);
$endpoint = new \Phpoaipmh\Endpoint($client);

HI @igor-kamil  - I'm going to close this issue, since I haven't heard back from you.  If you still haven't resolved this issue, let me know, and we can re-open it.  Thanks.


Imlpement Swagger UI API documentation and create full template

Implement Swagger UI
Create full examples to continue building all new APIs off.

Goals
Tags

Swagger is implemented, some progress on examples

Implemented with Tags as full example
URL: /api-docs/


dev goes out of bounds on devices

In Simulation.cpp this line appears twice: auto dev = jobs[job].tasks[jobs[job].cur].device; followed by some form of devices[dev]. When stepping though however, dev is sometimes outside the bounds of devices.
 So I can confirm its happening.

fix:
Task.cpp : 10
std::uniform_int_distribution<> dist(low, high - 1);
Task.cpp : 37
auto max = type == cs3100::Task::Type::CPU ? maxPage : maxDevice - 1;

In my program your second fix does the trick but implementing the first fix in any combination causes seg faults in all cases. Can anybody else confirm?

I changed it, only the one from line 37 is needed


Get-Item returns invalid FileName when listing alternate data streams

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Whenever trying to list alternate data stream names and filenames with Get-Item the stream name is removed from FileName property value.

Create a file named test.txt with stream name t
Set-Content -Path C:\test.txt -Stream t -Value testvalue
List available data stream using Get-Item
Get-Item -Path C:\test.txt -stream t
Notice that whenever the stream name is in FileName it gets truncated

Get-Item test.txt -Stream t | Format-List PSChildName, FileName, Stream

PSChildName : test.txt:t
FileName    : C:\es.x
Stream      : t

Both Windows PowerShell and Powershell Core are affected. Tested on Windows 11 as well as Windows Server 2022.
Expected behavior
PS C:\> Set-Content -Path test.txt -Stream t -Value testvalue
PS C:\> Get-Item -Path test.txt -Stream t | Format-List PSChildName, FileName, Stream

PSChildName : test.txt:t
FileName    : C:\test.txt
Stream      : t

Actual behavior
PS C:\> Set-Content test.txt -Stream t -Value testvalue
PS C:\> Get-Item test.txt -Stream t | Format-List PSChildName, FileName, Stream

PSChildName : test.txt:t
FileName    : C:\es.x
Stream      : t

Error details
No response
Environment data
PS> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.2.6
PSEdition                      Core
GitCommitId                    7.2.6
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

PS> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.22000.832
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22000.832
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

PS> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.20348.859
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.20348.859
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Visuals
No response

I've managed to find the reason for this behavior.
File
PowerShell/src/System.Management.Automation/namespaces/FileSystemProvider.cs
Class
AlternateDataStreamUtilities
Method
GetStreams
Line
data.FileName = path.Replace(data.Stream, string.Empty);
Description
The path variable holds a path to the alternate data stream i.e. C:\path\to\file.ext:streamname. Running a replace obviously replaces all instances of stream name from the path.
Probably the following change would fix the issue. That's how its handled elsewhere in FileSystemProvider.cs.
-- data.FileName = path.Replace(data.Stream, string.Empty);
-- data.FileName = data.FileName.Trim(Utils.Separators.Colon);
++ int firstColon = path.IndexOf(':');
++ data.FileName = path.Substring(0, firstColon);

@ovdeathiam Won't firstColon return the colon in C:? You should use LastIndexOf instead. Alternatively, more precise solution would be data.FileName = path.EndsWith(data.Stream) ? path.Substring(0, path.Length - data.Stream.Length) : path;.

Having tested both suggestions, the first did as @dkaszews suggested, however @dkaszews suggestion is oddly a single character off as shown below

However the actual fix is much simpler
data.FileName = path
Which I will be submitting a PR for shortly

@kilasuit Hah, gotta love off-by-one errors 😄 . Should be -1 there because Substring is exclusive I guess? That's why you always have to test index arithmetic.


Scholomance Portal is open from the start

I think this was done as part of fixing #409
In maps/source.w3x/war3map.lua
u = BlzCreateUnitWithSkin(p, FourCC("n04B"), -21056.0, 17536.0, 88.511, FourCC("n04B"))
WaygateSetDestination(u, GetRectCenterX(gg_rct_Scholomance_Entrance), GetRectCenterY(gg_rct_Scholomance_Entrance))
WaygateActivate(u, true)

Haha yep. I got a bit overzealous with making waygates.

Closed by https://github.com/AzerothWarsLR/WarcraftLegacies/commit/28cd138295a0366e38b02d92d6497ece473531be.


Document list/map/structType and listMapKeys

xref: https://github.com/kubernetes/website/issues/16725
These markers were introduced to describe topology of lists, maps,
structs - primarily in support of server-side apply.
Secondarily, a small typo fix:)

/cc @jennybuckley @apelisse
/wg apply
/hold
(just want to double-check the versions I've put up there are correct.)

mapType was indeed added in 1.17: https://github.com/kubernetes/kubernetes/pull/84113
and listType, listMapKeys in 1.16: https://github.com/kubernetes/kubernetes/pull/77354
/hold cancel

That's great, thanks Maria!
/lgtm

/approve


Add TS type definition

Codecov Report

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

@@           Coverage Diff           @@
##           master      #10   +/-   ##
=======================================
  Coverage   52.17%   52.17%
=======================================
  Files           1        1
  Lines          23       23
=======================================
  Hits           12       12
  Misses         11       11

Continue to review full report at Codecov.

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

@minhducsun2002 thank you for interest but not in plans to support typescript :)


Upgrade to tokio 1.20

https://github.com/tokio-rs/tokio/releases/tag/tokio-1.20.0

Are there any relevant changes we want? Else I'm not sure whether we want to accept such PRs.

Ah sure, I must have missed that in the contribution guidelines - it was just maintenance as it didn't seem like there is any dependabot automation setup.

Are there actually contribution guidelines? :D
I'm actually not sure what to do about updates. Generally we will need full-upgrades from time to time.
I think for any non-major updates we should update dependencies as needed.


Implementation of the curled-wake model in FAST.Farm

Not ready to merge
Feature or improvement description
Implement the curled wake model into FAST.Farm

Uses a Cartesian grid as output of WakeDynamics and within AWAE. for both polar and curled wake
UpdateStates of WakeDynamics is modified to accommodate curled wake using if statements throughout
Outputs of WakeDynamics are in cartesian coordinates and on cartesian grid
FAST.Wrapper returns Cq, psi_skew and chi_skew

Impacted areas of the software
FAST.Farm, AWAE, WakeDynamics
Additional supporting information

Original curled wake paper: https://wes.copernicus.org/articles/6/555/2021/
Paper regarding this implementation: TODO

Test results, if applicable
We expect small changes in the r-test sine we don't use the filtered velocity anymore to compute dx and convect, and beasue we use a Cartesian grid to communicate with AWAE.
Checklist

[ ] Document input file changes. Add link to paper
[ ] Understand why elimination of velocity filter created issues at last time step of r-tests
[ ] Review the filtering of skew angle and azimuth angle
[ ] Decay of wake curl needs tuning and default value. (Elliminate the need for deflection parameters, decay should take care of it)
[ ] Set default value of f_c based on the a=1/3, Uinf = 8, and a rotor radius based on the wake plane radius
[ ] Finalize implementation of plane outputs (- base name of VTK files, - Comment out the old wake-based write outputs for the velocity deficits and eddy viscosity values, - Ensure that the VTK-based output for the velocity deficits and eddy viscosity values also works for polar)
[ ] Add OpenMP parallelization of WD update states (and perhaps calculate output?)
[ ] Add an r-test for the curled wake  (based on the yaw steering test case with one of the existing turbulent wind data files)

FYI: at the moment it looks like some of the FAST.Farm source code in this PR has white-space changes (different line endings?) on every line. For example, see FAST_Farm_Subs.f90.

Damned, I just saw that, I think it's just in the last commit. I'm not sure how it happened and how I can easily reverse that, let me know if you have a good hint. I'll try to revert and force push this commit.

Fixed... I used set ff=unix in vim on those files. I might have mingled with git config --global core.autocrlf true before this problematic commit, but that was to actually try to avoid this issue as I could see vim not being happy with the file endings... I'll keep an eye out in my next commits to make sure it doesn't happen again. Thanks for spotting it!

Change of convection velocity slightly affect the test results (for LESInflow):

Test cases will be rebaselined


aws subnet reconciliation fails

What happened?
When an AWS subnet managed by Crossplane is deleted and when the controller reconciles it fails with an error.
How can we reproduce it?

Apply the below YAML spec

YAML spec:
---
apiVersion: ec2.aws.crossplane.io/v1beta1
kind: VPC
metadata:
  name: frezbo-vpc
spec:
  forProvider:
    cidrBlock: <IP_ADDRESS>/16
    enableDnsSupport: true
    enableDnsHostNames: true
    instanceTenancy: default
    tags:
      - key: Name
        value: frezbo-vpc
  reclaimPolicy: Delete
  providerRef:
    name: aws-provider
---
apiVersion: ec2.aws.crossplane.io/v1beta1
kind: Subnet
metadata:
  name: frezbo-pub
spec:
  forProvider:
    cidrBlock: <IP_ADDRESS>/24
    tags:
      - key: Name
        value: frezbo-pub
    vpcIdRef:
      name: frezbo-vpc
    mapPublicIPOnLaunch: true
  reclaimPolicy: Delete
  providerRef:
    name: aws-provider

Wait for the subnet to be created
Delete the subnet from the AWS console

Error:
❯ k describe subnet frezbo-pub
Name:         frezbo-pub
Namespace:
Labels:       <none>
Annotations:  crossplane.io/external-name: subnet-04632929ad60075a9
API Version:  ec2.aws.crossplane.io/v1beta1
Kind:         Subnet
Metadata:
  Creation Timestamp:  2020-09-09T16:11:48Z
  Finalizers:
    finalizer.managedresource.crossplane.io
  Generation:  3
  Managed Fields:
    API Version:  ec2.aws.crossplane.io/v1beta1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .:
          f:kubectl.kubernetes.io/last-applied-configuration:
      f:spec:
        .:
        f:forProvider:
          .:
          f:cidrBlock:
          f:mapPublicIPOnLaunch:
          f:vpcIdRef:
            .:
            f:name:
        f:providerRef:
          .:
          f:name:
        f:reclaimPolicy:
    Manager:      kubectl-client-side-apply
    Operation:    Update
    Time:         2020-09-09T16:11:48Z
    API Version:  ec2.aws.crossplane.io/v1beta1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          f:crossplane.io/external-name:
        f:finalizers:
          .:
          v:"finalizer.managedresource.crossplane.io":
      f:spec:
        f:forProvider:
          f:assignIpv6AddressOnCreation:
          f:availabilityZone:
          f:availabilityZoneId:
          f:tags:
          f:vpcId:
      f:status:
        .:
        f:atProvider:
          .:
          f:availableIpAddressCount:
          f:subnetId:
          f:subnetState:
        f:conditions:
    Manager:         crossplane-aws-provider
    Operation:       Update
    Time:            2020-09-09T16:15:39Z
  Resource Version:  65529
  Self Link:         /apis/ec2.aws.crossplane.io/v1beta1/subnets/frezbo-pub
  UID:               543e5d76-0a20-47b9-94d5-1ce15e209f52
Spec:
  For Provider:
    assignIpv6AddressOnCreation:  false
    Availability Zone:            us-east-1e
    Availability Zone Id:         use1-az3
    Cidr Block:                   <IP_ADDRESS>/24
    Map Public IP On Launch:      true
    Tags:
      Key:    Name
      Value:  frezbo-pub
    Vpc Id:   vpc-03218d0c7d6f9e6c7
    Vpc Id Ref:
      Name:  frezbo-vpc
  Provider Ref:
    Name:          aws-provider
  Reclaim Policy:  Delete
Status:
  At Provider:
    Available Ip Address Count:  251
    Subnet Id:                   subnet-04632929ad60075a9
    Subnet State:                available
  Conditions:
    Last Transition Time:  2020-09-09T16:15:39Z
    Message:               create failed: failed to create the Subnet resource: InvalidParameterCombination: The parameter availabilityZone cannot be used with the parameter availabilityZoneId
                           status code: 400, request id: 96acc599-75fa-4092-8141-b90d0e54ac31
    Reason:                Encountered an error during resource reconciliation
    Status:                False
    Type:                  Synced
    Last Transition Time:  2020-09-09T16:14:42Z
    Reason:                Resource is being created
    Status:                False
    Type:                  Ready

@frezbo Could you try with either availabilityZone or availabilityZoneId? I think the error is not really helpful but from what I can tell, you have to provide one of them.

@muvaf For creating a subnet, both availabilityZone and availabilityZoneId are completely optional. This happends if the subnet resource is manually deleted from the console. What happens is that the subnet resource in k8s already has both availabilityZoneId and availabilityZone retrieved from the API on first create and update operation. Now when it reconciles we sent a create operation with both fields which is not allowed by AWS. I'm thinking of dropping the availabilityZone when doing a subsequent create, WDYT?

Won't fix as explained in https://github.com/crossplane/provider-aws/pull/335#issuecomment-690982667


feat: 增强饼图标签跟随切片颜色

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[x] commit message follows commit guidelines

Description of change

背景 label 标签色跟随了 theme 设置，存在一个默认色。在 deepMix 下，undefined 会取默认色，null 则会导致标签渲染不出，因此可以在 fill 为 null 或空字符串的时候，处理为跟随切片颜色，若用户的确不想渲染 label，可以取 fillOpactity 为 0，或label: false

fill 为 undefined 时，跟随旧逻辑，取默认色
fill 设置为 null 时，映射为对应块的颜色

close: #3146

Pull Request Test Coverage Report for Build 436884807

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.07%) to 93.466%

Files with Coverage Reduction
New Missed Lines
%

src/component/labels.ts
2
87.55%

Totals

Change from base Build 435813675:
0.07%

Covered Lines:
10686

Relevant Lines:
11088

💛  - Coveralls


Node link in Workloads Flat List is not aligned properly

Version: master 9/20
Steps to Reproduce:

Open Workloads, select Flat List

Results:

Version - 2.1 master 9/20
Verified fixed


Sign up as a contributor

Thank you for your interest in the session "Health Data Sharing is Caring"! If you participated in the live session at Mozfest 2019, or shared your ideas with us during the festival, please leave your name below so we can credit you.
If you were not at the session but would like to contribute to any of the issues, please leave your name below along with comments, and your area of interest to credit you. We invite you to explore the issues section of the repo to learn more about the session and collaborate with us.
session participants and repository contributors:
Sakthi Anand
Yash Chaubey
Vishal Pandey
Bala Subramaniyan
Gopala Krishna
Anne Clinio
Athina Tzovara
Micah Vandegrift
Damian O.Eke
Stefano Vrizzi
Joscha Jäger
and you :)!

Sakthi Anand

Yash Chaubey
It was an amazing experience being a part of your session and for sure i wanna be a part of any of your future projects.

Here I am, Anne Clinio.

Great session Natalia. Would love to contribute to your RRI framework

George Ogoh

Here is a blog post that might interest some of you: https://www.ethicsdialogues.eu/2019/12/19/healthy-ai-at-mozfest-2019/ Your session at MozFest


How to update customer user id ?

Currently i have use case that need to update customer user id ( after login )
In the document i was noticed that  the function setCustomerUserId(userId, callback)
Setting your own Custom ID enables you to cross-reference your own unique ID with AppsFlyer’s user ID and the other devices’ IDs. This ID is available in AppsFlyer CSV reports along with postbacks APIs for cross-referencing with you internal IDs. NOTE: This must be set before initSdk is called.
So how can I update customer user id?

@amit-kremer93 It would be helpful if you posted an answer to how to approach it.
Like I wish to collect app installs AND after the user has signed it I want to enrich my data collection.

Landing screen
appsFlyer.initSdk(...)
AppsFlyer records install
User goes to sign in screen
User completes sign in and gets user id '123'
appsFlyer.setCustomerUserId(123)

According to the docs step 6 should not happen before step 2. So do I have to something like this for step 6:
appsFlyer.stop(true, res => {
        appsFlyer.setCustomerUserId('123')
        appsFlyer.initSdk(options)
        // OR since docs says "In any event, the SDK can be reactivated by calling the same API, by passing false.":
        // appsFlyer.stop(false) // ??
      })

Reply from customer support:

I saw your comment on the Github issue you opened and I would like to clarify that the note that we have under the setCustomerUserId method is not entirely correct nor mandatory.
You can call setCustomerUserId at any point of your application's flow.
The note simply (and after we will fix it) should mention that if you wish to see the CUID (Customer User ID) under your installs raw data reports, it should be called before starting the SDK.
There also some other dependencies, partners or integrations that require this to appear on the install event in order to match IDs to their BI system.
If the above is not the case and you simply would like to add additional user id to the events raw data reports, then you can freely call it anytime you need.


Model parameters

Hello! With the current neural network parameters presented in the README, can I get a decent result if I train AudioLM on Russian LibriSpeech? Or is it just a proof of concept?

There's nothing that makes this model specific to English other than perhaps the T5 embedding model if you're using it. (I'm not sure how much Russian is in the corpus that was used to train that model but you could look it up). So yeah, no reason that it shouldn't work with Russian LibriSpeech. How many hours of data is it?

About 100 hours of recording.

You're quite likely struggle to produce really coherent outputs with that amount of data. You might need to train it on a larger corpus and then fine-tune it on your corpus. Would love to know how you get on!


[AJ-1618] Report new cWDS version to Sherlock and deploy to dev on tag

https://broadworkbench.atlassian.net/browse/AJ-1618
Add jobs to the publish-app-version workflow (run as part of the tag workflow) to report a new cWDS version to Sherlock and deploy it to dev.
Followed instructions in Google Doc linked from ticket.

how are you thinking we test this? just run and see if it works I guess 😄
also, should we consider being able to push an update to cWDS but not WDS?

how are you thinking we test this? just run and see if it works I guess

Yeah, I don't know of another option for GitHub Actions.

should we consider being able to push an update to cWDS but not WDS?

Without thinking much about it, I think I'd vote no. One of the motivations for cWDS was having consistent behavior / the same code in Google and Azure.


Adapt integration tests to use generated test data in fixtures/

The test data is randomly generated so that it's "interesting". Every time I regenerate the test data it changes, and the integration tests are breaking. What I do in other projects is generate the same test data in a form the front end / integration tests can use.
I've put this data in .json form in the fixtures/ directory. It can be read with cy.fixture.

Added TODOs to the commented-out assertions in the .spec.ts's.


Change month by using arrow will trigger onDateSelected on frist day of month

I need to run some function in onDateSelected so i can save the date user pressed in a list, but when user press the arrow the onDateSelected will get called and insert first day of month into my list. Is this a bug ?

Technically it is not a bug. The navigation using these arrows leads to the selection of a new date, which then triggers the callback. But I get your point. There should be a way to differentiate wether the user clicked on a date, tapped an arrow or used the day picker to select a new date.

Technically it is not a bug. The navigation using these arrows leads to the selection of a new date, which then triggers the callback. But I get your point. There should be a way to differentiate wether the user clicked on a date, tapped an arrow or used the day picker to select a new date.

and select on any date will trigger the onMonthChanged too from my testing which is not suppose to happen I assume.

What could be a proper solution? Should the package provide the possibility to assign callback functions for each of the situations?

What could be a proper solution? Should the package provide the possibility to assign callback functions for each of the situations?

I tried to use the onMonthChanged callback to remove the first day of month from list when user press arrow but the onMonthChanged get call even the user select a date on the current month so I'm unable to fix that.
now I'm trying another package.

Sure, you are free to use whatever package you want. But this is open source code. I was just asking you for an idea, how it should work.

Hi, i think the solution is that, allow provide assign different callbacks

I think I'll add a parameter that controls the behavior of the callback. This parameter will let you select between two modes:

Trigger each time, the date updates (like today)
Trigger only on touch events, when the user explicitly selects a day

@cypie92 Do you think, that could work for you?


modern-ysu-thesis:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: Explain what the package does and why it's useful.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Removing the logo works. I have attached a review that shows how to do it - let me know if you want to apply the change!

well,the logo just a example to tell user how to add picture and is not necessary,i don't want user confuse by lib.typ,so just remove it ,maybe user can learn how to add picture to there thesis by google.

Okay, let's merge this then. Thank you!