SSD classification loss in Object Detection API is different from the original paper .

In the paper they have used a softmax cross entropy loss. But here in the model is using sigmoid cross entropy loss .
Why is has changed ? What is the effect ?

I think in this project softmax is used, why do you think sigmoid is used?

Check this configuration file . This says it is weighted sigmoid

Where is the configuration file used in this project? Are you sure you raise the issue in the right repository?

OMG! I am extremely sorry  . I  really beg your pardon .  Sorry .


SNOW-1787415 Fix dataframe join with sort deterministic

SNOW-1787415 Fix dataframe join with sort deterministic

If this patch doesn't have a hard dependency on AST logic, please consider merging into main instead.


fix sentry

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/retest


Visual studio version required  for CORE 2.0.3

Unable compile VS 2017 15.3.2 project with core adi 2.0.3
Could you please confirm if it is required to
upgrade VS 2017 to 15.4 .

Can you elaborate on what you mean by "core adi 2.0.3"?

I'm assuming adi == sdk :) I don't think 15.4 is required but if you can share your error, we'd be able to help more. I'm going to close this now because we need more information but will re-open if you provide more info.


Enhance node sizing docs

This PR adds more details on the node sizing docs.

@tobim can you take a look again at the new https://tenzir-node-sizing.streamlit.app/ and whether you think that works wrt. the text?


Can view unpublished game when not logged in.

I can't decide what do to about this. There are good reasons for/against it.
Keep it:

Can easily share unfinished posts/articles/games with anyone by sending them the link

Remove it:

Privacy

We're not really a community designed around private communication though. The entire point of what we do here is to share, so I'm leaning towards not changing this.

I think we keep it after all and no-one has complained about it so if it becomes a problem we make an issue of it then. I think it may be quite useful if you are a team working remotely and not everyone is registered..


Redundant view updates?

If something has an effect, and the effect dispatches another update, the view effectively ends up getting updated twice.
If that effect has an effect, three updates, and so on.
Is this by design?

@mindplay-dk Generally an effect is asynchronous, so when you dispatch a message it won't cause an immediate update again. If you find that you're modeling something synchronous as an effect, I think that's a good time to step back and consider if it ought not be a pure helper function called from your update(msg, model) instead.
Hope this helps! Given how simple Raj is, it definitely could be abused to thrash renders and updates. Like anything it'll need some consideration if you start running into performance bottlenecks but I've not had a problem here in particular.

I'm just getting reacquainted, and I think where I got confused again is by the return tuples with the effect in there - it makes it look like there might be something asynchronous going on in the library itself, which I know there's not, but... whenever you have to return a function, it's natural to wonder, so when does it get executed? In this case, it's synchronous - it gets executed immediately, it's just not obvious from the calling code.
I have this other little thing I've been playing with:
https://gist.github.com/mindplay-dk/135124139217f08bf2903c2553cf1aac
It's different from Raj, using an object as state, allowing partial updates - which works better for my use-case.
But if you ignore that...
Instead of returning effects, I went with simply passing the run function, both to actions and to the view - so you can queue up as many action "effects" as you want. Similarly to Raj (I think?) this guarantees that one action will finish before another runs - if an action queues up effects with run, these aren't run immediately, they run after the action that queued them has returned; but you don't have to wait until the end of the action to queue up effects, you can put them wherever you want.
It has similar behavior, but doesn't require tuples (which still to this day confuse me) and doesn't limit you to one effect.
I don't know if that's really a meaningful difference - I think I just like the idea of functions returning one thing, and if I can pass a run function that achieves the same thing (even if it's called more than once) that just seems to produce slightly more readable code to me? After all, when you return an effects, you're not really returning a value or something you created - it's really just a mechanic to make sure things execute in full, and in order, right? My run function seems to do that, but it just helps my understanding, seeing the word "run" in the function.
It's probably mostly a matter of preference, I guess. Anyhow, feel free to carry on the discussion if you like, or just close the issue, since there's no issue with your library here. 🙂
(and I still like it, I'm just concerned about future maintainers being able to read and understand the code - it's still quite difficult for me, and I've spent quite a bit of time with it, which is why I'm looking for ways to make things more readable without adding substantial complexity...)

@mindplay-dk I'll respond to only a few of these bits for time's sake, but I'd be wary of your action queuing pattern. It's certainly an option I've seen in other frameworks (Ember has a "run loop" everyone hates, for example) that can grow to be quite complicated to manage.

Similarly to Raj (I think?) this guarantees that one action will finish before another runs

Raj guarantees the effects which are processed by the runtime will be kicked off synchronously (which gives us this order), but effects can finish asynchronously and are not awaited.

I think I just like the idea of functions returning one thing, and if I can pass a run function that achieves the same thing (even if it's called more than once) that just seems to produce slightly more readable code to me?

Definitely a personal preference. I find code where every meaningful operation needs to be wrapped in a function call to be very noisy. What I want is plain, testable logic without needing to provide a fake run method to exercise everything. A framework shouldn't dominate the syntax for your business logic, it always ends up bad when you go to test/refactor it.

and doesn't limit you to one effect.

You're definitely not limited to one effect! Check out raj-compose/batch-effects for example. Effects are just functions which take a dispatch, that function can call other effects/functions with that dispatch.

I don't know if that's really a meaningful difference

Returning effect values makes it easier to test which is the focus of Raj. Take this snippet from your gist as an example:
setTimeout(() => {
  run(state => {
    console.log("ACCEPT MARKETING", state);
    return { acceptMarketing: true };
  });
}, 1000);

This is really hard to test, we've now got asynchrony in our update function which means we'd have to do some weird mocking tricks. Instead, Raj forces you to return an effect that can be called independently at Raj's (or when under test, your own) choosing. But you don't have to call it to get access to the next state.


Implement VALUES function for Map type

Description
resolve #1276
implement VALUES to get list of all values of Map.
Example
CREATE TABLE USER (id INTEGER, data MAP);

INSERT INTO USER VALUES
  (1, '{"id": 1, "name": "alice"}'),
  (2, '{"name": "bob"}'),
  (3, '{}');

SELECT VALUES(data) as result FROM USER WHERE id=1;

| result      |
|-------------|
| [1,"alice"] |

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

49 of 53 (92.45%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 98.726%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

test-suite/src/function/values.rs
23
27
85.19%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
44259

Relevant Lines:
44830

💛  - Coveralls


[Question] How do I use this with IRedMail

From what I understand is going on, this communicates to containers over HTTP.  I am using:  https://github.com/lejmr/iredmail-docker
When NGINX tries to communicate over HTTP, IRedMail redirects to HTTPs but NGINX can only communicate to containers over HTTP so it makes an endless loop.  I tried:
VIRTUAL_PROTO=https
VIRTUAL_PORT=443
in the IRedMail container.  This not only did not work and gave a wrong certificate error.  It broke the other working containers like the NextCloud container this ships with even though the only setting that was changed was in the IRedMail container.  You can see which Nextcloud container here:
https://blog.ssdnodes.com/blog/installing-nextcloud-docker/
I am not sure why this is happening and how to fix it.
edit:
I figured out most of the problems and I got  to the point of a login page.  What I don't know is what ports I need open.

You would want to make the NGINX container your HTTPS termination point, not iredmail if possible.


missing type definitions for ibm specific fields

When I import this package using import AWS = require('ibm-cos-sdk'); then typescript complains about some of the extra IBM specific parameters I have to pass it.   For example:
In:
new AWS.S3({
      endpoint: '<endpoint>',
      apiKeyId: '<key>',
      ibmAuthEndpoint: '<auth>',
      serviceInstanceId: '<service-instance>',
    }

The fields apiKeyId, ibmAuthEndpoint and serviceInstanceId do not exist in type 'ClientConfiguration'.
So I have to import it like this for it to work : const AWS = require('ibm-cos-sdk'); but then I lose types.
Perhaps the ServiceConfigurationOptions interface needs to be updated to include these new fields.

@hiro5id I'll look into this.
Are you using release 1.3.2?

Yes, 1.3.2

I have created a pull request : https://github.com/IBM/ibm-cos-sdk-js/pull/35

Hi @hiro5id ,
I've created an internal issue and will get this fixed in the next release. Thanks for the PR.

@stephenskinner Can we get this PR merged? Would resolve some issues for me as well. Thanks :)

Hi @hiro5id, @virkt25,
This is fixed in our upcoming 1.4.1 node sdk release.
Thanks for the PR, unfortunately we merge the PR due to the build release process in place.


Pressing the shortcuts are doing nothing for me

Pressing the shortcuts are doing nothing for me. I've been trying both control and command for shortcuts, but they simply aren't taking. I've removed all other extensions to avoid conflicts.

I've been trying both control and command for shortcuts...

 It's the Cmd key.
First you have to start Quick Mark "mode" (Edit > Quick Markup Mode). Note that shortcut used to be Cmd+M, but recently changed to Cmd+Shift+M. You should see a bar at bottom of editor to indicate that it's on.
Second, it only works on pages that Brackets considers HTML, so look at Language indicator on statusbar.
Third, Cmd+Enter, Cmd+Delete, and Cmd+Backspace only work inside tags.
Does that fix it?

Thank you for the quick reply! I found what was causing the issue. I needed to wrap the content in an HTML tag. I was just editing content for a WordPress widget, so it wasn't wrapped to begin with.


Migrate nonce to getAccountNonceAndTimestamp() and updateAccountNonceAndTimestamp()

Now we have nonce and timestamp in state db, so we can migrate blockchain node's nonce to them.

Done.


Missing steps in Regicide

After filling the barrels with tar, it should prompt you to pick up some sulphur, which is only obtainable in the quest area (and is a pain to return to).

https://i.imgur.com/EalGgBq.png
Photograph of the issue.
Starting step is that of the second quest step. Final is mentioning the sulphur.

Also not included in required items, since it's tradable (but not available on GE).


Create Gemfile.rspec-3.1

testing against rspec 3.1

Coverage remained the same when pulling bb324d9b0f8a56a94848f9b69e58fb711e11be85 on lucianosousa:patch-3 into 42fb4b72c5c1abb503398a3c9144e12c3a316388 on guard:master.

Coverage increased (+0.07%) when pulling dc6d0d6b35d87e7a9cfa14cb3b94b80c044e92dd on lucianosousa:patch-3 into 42fb4b72c5c1abb503398a3c9144e12c3a316388 on guard:master.

@e2 any idea here?

I don't remember exactly - it was something about RSpec not yet supporting original locations of shared_example calls, see the url in this comment: https://github.com/guard/guard-rspec/blob/master/spec/lib/guard/rspec_formatter_spec.rb#L114

broke just using ruby 1.9.3
suggestion?

"no spec file found for ./spec/support/breadcrumbs.rb:75"
I think I remember this a bit. The specs aren't really good - the file/path operations are not stubbed correctly, so the tests may be failing without a good reason.
You can try reproducing locally with the same RSpec seed: 3825
I think it's just a matter of stubbing File#* Dir#* or Pathname#* methods so that this output/error stops occurring: "no spec file found for ./spec/support/breadcrumbs.rb:75".
I think spec/requests/breadcrumbs_spec.rb was supposed to use a shared example defined in spec/support/breadcrumbs.rb - so I'm guessing the existence of spec/requests/breadcrumbs_spec.rb isn't properly stubbed. But I'm not sure.

If you rebase this against master, it should work (especially since Ruby 1.9 is EOL and I've removed it).

I'm closing this, since there were too many RSpec and Ruby versions tested - until RSpec 4.x comes out, it doesn't make sense to support more than the latest available 3.x version (and 2.14 and 2.99 for special reasons).


[Constraint solver] Remove yet more of TypeCheckExprFlags

Sink more information down into SolutionApplicationTarget so we can compute per-target two more bits of information that were previously flags passed down:

ExpressionTypeMustBeOptional: used to indicate when we want an optional type (e.g., for an if let). This can be recovered by storing the pattern in SolutionApplicationTarget (we need it anyway)
ConvertTypeIsOpaqueReturnType: used to note that the expression produces the concrete type for an opaque return type.

@swift-ci please smoke test

@swift-ci please test source compatibility


Gradle plugin jackson annotations not working correctly

Hi,
I started using the library recently, both with and without the gradle plugin, but noticed that the gradle plugin has an issue.
When generating the docs via the gradle plugin, all jackson annotations are ignored.
For example: "enumValues" in the ServerVariable class gets serialised as "enumValues" instead of "enum". The "lamps-asyncapi.json" file that is used for the unit tests is wrong, as it also contains "enumValues" instead of "enum".
I assume this is because the annotation info is not carried over between modules.

@DavyHubrecht hi.
Can you check that everything works correct? I forgot to initialize gradle classloader in similar manner of maven's classloader.
Brief bug description:
Problem was in a wrong initialization of class loader. I initialized it without current thread class loader context, so all classes were loaded correctly, but jackson annotations were ignored while deserialization.
Solution:
Initialize gradle's classloader in manner of maven's.
URLClassLoader(classPathUrls.toTypedArray(), Thread.currentThread().contextClassLoader) instead of URLClassLoader(classPathUrls.toTypedArray())
This #5 fixes this bug. If all is correct, I will merge it


Fixes issue with text clipping from page turning.

With certain window sizes the left text gets cut off from rounding
on the page slide size.

Waffle: connects to #283

Competing solution? https://github.com/readium/readium-shared-js/pull/188


fix: modify quickstart.js sample code

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@benqYannick thank you for the contribution :+1: I think this was actually just fixed here:
https://github.com/googleapis/nodejs-translate/pull/355/files#diff-40c733658e76667e905c6a7dd45a1f55R33
As part of promoting our v3 API to the default API surface.

Thank you for your teaching 👍  Looking forward to v3 API  : )


Switching between projects is extremely slow

Every time I switch between projects, it takes about 5-10 seconds before I can start working in the switched project. Not sure what you will need from me to get some data on that?

same issue here, origin open project folder took less than 1s.

super slow for me as well any thoughts?

Can I have some details like:

How many files are open in Project A (switching away from) and Project B (switching to)
How many files in both, period
Installed packages

At first it runs fast. Seems to become slower each time it's used until reaches the point where it freezes atom.

Mac OS,
Atom 1.21.0
Project Plus 1.0.0
After switching projects 4-5 times, the process of switching projects takes 4-5 seconds
After switching projects 14-15 times, one switch takes about 50-60 seconds...
After switching projects 20-21 times, one switch takes about 7-10 minutes...
So it's imposible to use this plugin after switching projects 8-9 times...

Any ideas on this @mehcode?
It kind of seems like the memory goes up and up and up each time you switch projects within the same window.


Automatic generation of ApiVersion as a query param even when not in Swagger

Not sure if bug or feature request but in my Swagger file I didn't specify any:
"ApiVersionParameter": {       "name": "api-version",       "in": "query",       "required": true,       "type": "string",       "description": "Client Api Version."     }
But in my generated code I still get:
query_parameters['api-version'] = self._serialize.query("self.api_version", self.api_version, 'str')
In every operation, but I don't want it.
I'm using<EMAIL_ADDRESS>"^4.0.67",         "autorest": "^3.0.5165",

Fixed in autorestv3 version


Default allowed number of connections?

With a vanilla install how many users can connect to a single site?
How do you change that setting?
How do you set a max amount of concurrent users?
Can anyone provide details on how the system scales out?
How much RAM/CPU power is needed for say 50 users, 100 users?

With a vanilla install how many users can connect to a single site?

The default size of the pool is 10 containers. This is defined here.

How do you change that setting?

You can change the pool_size variable when you pass it as a command line argument (--pool-size) when running python orchestrate.py.

How do you set a max amount of concurrent users?

Same as above. You can set the --pool-size parameter. If you'd like the number of containers available and the number of concurrent users to be different, I think you might be able to configure something in the proxy server, but I'm not entirely sure if this is possible or how it would be done.

Can anyone provide details on how the system scales out?

tmpnb acts an orchestrator for Docker containers that contain images of the Jupyter Notebook. When a user visits the tmpnb endpoint, a new container is allocated for them in the pool at a special endpoint (/user/123456789abc).

How much RAM/CPU power is needed for say 50 users, 100 users?

You can limit the amount of memory that is available in each container (--cpu-shares) that is allocated to the user as well as the CPU shares (----mem-limit ). You can't really say in advance how much memory or processor power is allocated to a container because it does depend on what is being executed in the container.
Let me know if you need further explanation on any of the above! 😄

When I set the --pool-size=20 in my command, it loads, but I get a 500 error from the proxy.
command:
docker run --net=host -d -e CONFIGPROXY_AUTH_TOKEN=$TOKEN --name=tmpnb -v /var/run/docker.sock:/docker.sock  jupyter/tmpnb python orchestrate.py --image='arcrest/scipy-notebook:v1' --command="ipython notebook --NotebookApp.base_url={base_path} --ip=<IP_ADDRESS> --port {port} --pool-size=20"
Any suggestions?

I put the --pool-size in the wrong spot
docker run --net=host -d -e CONFIGPROXY_AUTH_TOKEN=$TOKEN --name=tmpnb20 -v /var/run/docker.sock:/docker.sock  jupyter/tmpnb python orchestrate.py --pool_size=20 --image='arcrest/scipy-notebook:v1' --command="ipython notebook --NotebookApp.base_url={base_path} --ip=<IP_ADDRESS> --port {port}"

Awesome! Do you have any other questions or can I close the issue?

all good


sanitycheck: key error issue while trying to access the key in dict

While generating json report, status key does not exist and throws
keyerror in some specific scenarios.get() method is used to avoid
error.
Signed-off-by: Spoorthy Priya Yerabolu<EMAIL_ADDRESS>

FYI: assignee is the maintainer, not the contributor :)

this PR needs to be resubmitted from a  fork.


extract automatic market maker from farcasterd

I don't think there is an automatic market maker in farcasterd, so this sounds more like a service that would use farcasterd to be a market maker and is therefore out of scope.


How can i get the speech text and response text

for example when I say "alexa,hello", avs response "hi"。both two is sound, how can i get text

Hey @SpringSea28 thanks for the post.
Speak Directives only contain an audio file to be played and no text of the audio being played, there is no accompanying text.
I will be closing this issue, feel free to reopen or create a new issue if this has not been fully addressed.


Error when start the keybase

I start the keybase and show me this error:

my log id: 0c740a46f893c3534b54671c

What version of MacOS are you running? Are you using Docker on Mac (uses the kernel virtual machine management which may 'take a slot')?

MacOS: 10.13.1 Beta (17B35a)
Yes... I use docker but isn't running when fails.

It doesn't really matter if it's running or not, it still grabs the slot.

What maxtaco said, Docker will occupy the slot whether it is running or not I believe. Docker runs a "daemon" which lives in the background just waiting for you to give it something to do. You could try stopping the Docker service and see if that helps, but since you are running a beta you may also be running into a security feature where you need to reboot into the less strict "signing" mode in order to load the kext, been a while since I've pushed the bleeding edge with a Mac because it usually left me bleeding and running back to Linux.


 We have a blog for a community with different authors, and also new authors from time to time, and it would be great, if I can get all news from the blog.
Thanks for this plugin, I love it!
Greetings
Sven

duplicate of #1


A Javascript error occured in the main process

Im getting an error A Javascript error occured in the main process when I try to run this project using npm run start. This occurs when the dotnet core project sends a message back to electron

I am using the following packages and versions:
electron-cgi: ^1.0.6
electron-cgi-dotnet: 1.0.4-beta
Have I got something wrong in the project?

I'll have to update the nuget package to use .net core 3.1. Nothing you did wrong. I'll let you know when it's live you'll just have to install the new version of electron-cgi-dotnet to fix that

I'll have to update the nuget package to use .net core 3.1. Nothing you did wrong. I'll let you know when it's live you'll just have to install the new version of electron-cgi-dotnet to fix that

Can you confirn that the console project you created (where you installed the ElectronCgi.DotNet nuget package) is running netcore 3.0 ? If so you need to upgrade it to 3.1: https://devblogs.microsoft.com/dotnet/net-core-3-0-end-of-life/

Can you confirn that the console project you created (where you installed the ElectronCgi.DotNet nuget package) is running netcore 3.0 ? If so you need to upgrade it to 3.1: https://devblogs.microsoft.com/dotnet/net-core-3-0-end-of-life/

The console project was running on dotnet core 3.0. Switching over to 3.1 resolved the issue.
Cheers

The console project was running on dotnet core 3.0. Switching over to 3.1 resolved the issue.
Cheers


Best practise for maintaining document state on a language server?

This is not an issue, so feel free to close this down and direct me to the correct forum for questions, but I have a question regards the tracking of document state by the language server.
I have attached some message listeners to the LSP and see that text deltas are being sent by the protocol correctly. But my question is what is the general mechanism for handling document state on the server side. I would have assumed (in my ignorance) that each time a document was switched on a client that a full version of the document would be supplied to the server, and as the foreground document (of the active window) was edited, the deltas would be supplied. But, by observing the protocol, it seems that only opening of a document on a client will send the initial document, then potentially you could have 1,000 documents open on the client at the same time, and the server must be ready to apply deltas (assuming that the text document synch kind is set to incremental - which I like).
So, is the server meant to maintain state for every opened document, or is there a way for a server to ask for a complete version of a document state when documents are switched (so that only one active document requires its state tracking on the server)?
I'm just trying to figure out best practise for maintaining state. My apologies if this is (and likely is) the wrong place to ask this question, or for not understanding the spec well enough.

The server should maintain content state between textDocument/open and textDocument/close calls. There is currently no way to read document content of arbitrary files through LSP, so you can either read from disk or use an LSP extension: https://github.com/sourcegraph/language-server-protocol/blob/master/extension-files.md

@felixfbecker explanation is correct. However we do have requests to allow servers to fetch the content of a document from the client so that the server could skip open documents.
I will close this issue as a question.


Empty response for powershell env vars

Code of Conduct

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

What happened?
in env block adding global env vars resulting in empty response.
env:
  - name: USER_NAME
    value: $ENV:USERNAME
    if: match .Shell "pwsh"

This works
but
env:
  - name: ALIAE_CONFIG
    value: C:\Users\$($ENV:USERNAME)\.aliae.yaml
    if: match .Shell "pwsh"

this resulting in C:\\Users\\\\.aliae.yaml
is this even supported?
What OS are you seeing the problem on?
Windows
Which shell are you using?
powershell

Nevermind. Should've followed templating.


WIP hyperopt: init at 0.2.3

Motivation for this change
hyperopt was not part of nixpkgs.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Additional information
Following https://github.com/NixOS/nixpkgs/pull/74783 because I deleted my forked repo.
Todos from previous PR:

[ ] make tests work

[ ] make it work without a pinned networkx (hyperopt specifies 2.2, current nixpkgs networkx is 2.4)
[x] some issues about missing files in tests

Seems like the missing file issue disappeared. Now I get two main errors:
AttributeError: module 'bson' has no attribute 'BSON'

ImportError: You must install lightgbm and sklearn in order to use the ATPE algorithm. Please run `pip install lightgbm scikit-learn` and try again. These are not built in dependencies of hyperopt.

The first one is fine, I'll investigate the code.
The second is weird because I've put the packages in checkInputs:
https://github.com/NixOS/nixpkgs/pull/82144/files#diff-2020c7eb4c24313c689126b065cec464R77

hyperopt is now in nixpkgs


README Additions

A few things to add to the README

Documentation on Docker tags
Link to docker hub
Example commands should use full docker hub image name (declanisbritton/dashi rather than dashi)

Also, add a drone build badge


investigate possibility to move driverfactory functionality into separate module

it seems like for 3rd party integrations like browserstack/saucelabs/amazon etc better to have separate DriverFactory.
So in additions to DesktopFactory and MobileFactory we can register BrowserstackFactory with functionality for this custom client.
The root cause: each tool use their own capabilities toolset which differs from default selenium/appium.
For example browserstack need:
device=value
when standard is
deviceName=value

closing as out-of-dated requirement. All custom logic will be organized via driver listeners


Support node 7

Currently we do not support node 7. We should support it and add a Travis task as well.

ERR<EMAIL_ADDRESS>postinstall: node scripts/build.js`` I'm using Node 6.9.1
angular-cli: 1.0.0-beta.20-4
node: 6.9.1
os: linux x64 Ubuntu 14.04

@jkuri you want to take this?

yup, I'll take this.

So it seems to work with node 7.2.1

Great Job.... ¯_(ツ)_/¯

Closing this as done.


Logging Command Support

Added in commit https://github.com/sleipnir/cloudstate-cli/commit/425da2622fe5db5edf8b47b7e659f9609d4df08f


Can't download videos from Dailymotion

Hi,
I can't download videos from Dailymotion, so let's do a recap:

OS: Debian 8 (Jessy)
youtube-dl installed from: pip
youtube-dl -v output:
[debug] System config: []                                                                                [debug] User config: []                                                                                [debug] Custom config: []                                                                                [debug] Command-line args: [u'-v']                                                                                [debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8                                                        [debug] youtube-dl version 2017.07.02                                                                                [debug] Python version 2.7.9 - Linux-4.9.29-8-osmc-armv7l-with-debian-8.8                                               [debug] exe versions: none                                                                                [debug] Proxy map: {}

Why doesn't it download any video?

Carefully read new issue template and provide all requested information.

You could have at least tried to reproduce the issue instead of closing it...


Object still draggable after setting it to false

Version

Phaser Version: 3.11.0
Operating system: Arch Linux

Description
It seems to me that drag is not disabled once an object has already been dragged. When I enable and disable drag before I have touched the object, disable is working. Once the object has been dragged, the disable drag commands, executed from inside the 'dragend' event, seem to have no effect, my object is still draggable, even though no drag events are fired. Could it be that something is leaked to DOM?
Example Test Code
How I initialize drag:
export default class MyObject extends Phaser.GameObjects.Sprite {
  constructor(config) {
    super(config.scene, config.x, config.y, config.key);

    this.setInteractive();
    this.scene.matter.add.mouseSpring();
    this.scene.input.setDraggable(this);
}

When I disable drag just after that, I can no longer drag. But my code does this:
    this.scene.input.on('dragend', function(pointer, gameObject) {
      gameObject.scene.input.setDraggable(gameObject, false);
      gameObject.setInteractive(false);
      gameObject.input.draggable = false;

Essentially all I think I need is gameObject.scene.input.setDraggable(gameObject, false); but I've put the rest there just to show I've tried them. I have also checked that gameObject.input.draggable is still false while dragging is happening.
Additional information

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@strahius https://photonstorm.github.io/phaser3-docs/Phaser.GameObjects.Sprite.html#disableInteractive__anchor or even this https://photonstorm.github.io/phaser3-docs/Phaser.GameObjects.Sprite.html#removeInteractive__anchor

Thanks @alexeymolchan, forgot to mention I had also tried these, tried again now, same result.

@strahius i tried here https://labs.phaser.io/edit.html?src=src\input\dragging\enable%20for%20drag.js
just added gameObject.disableInteractive() in dragend handler and image has become non draggable
this.input.on('dragend', function (pointer, gameObject) {

        gameObject.clearTint();
        gameObject.disableInteractive();

    });

Thanks alot for looking at this @alexeymolchan! Then it must be something else in my setup, there's a few things going on. Not sure what's the best way of sharing this, but I kind of reproduced it on that example:
var config = {
    type: Phaser.WEBGL,
    parent: 'phaser-example',
    scene: {
        preload: preload,
        create: create
    },
    physics: {
        default: 'matter',
        matter: {
            gravity: { y: 0.8 },
            debug: false,
        }
    },
};

var game = new Phaser.Game(config);

function preload ()
{
    this.load.image('eye', 'assets/pics/lance-overdose-loader-eye.png');
}

function create ()
{
    // var image = this.add.sprite(200, 300, 'eye').setInteractive();
    var image = this.matter.add.image(200, 300, 'eye', null, {friction: 0.9});
    image.setStatic(true);

    image.setInteractive();
    this.matter.add.mouseSpring();
    this.input.setDraggable(image);

    var constraint = Phaser.Physics.Matter.Matter.Constraint.create({
        pointA: {x: 200, y: 300},
        bodyB: image.body,
        stiffness: 0.0001,
        damping: 1,
    });
    this.matter.world.add(constraint);

    this.input.on('dragstart', function (pointer, gameObject) {
        gameObject.setStatic(false);
        gameObject.setTint(0xff0000);
    });

    this.input.on('drag', function (pointer, gameObject, dragX, dragY) {
        if (Phaser.Math.Distance.Between(200, 30, dragX, dragY) > 200) {
            constraint.stiffness = 1;
            constraint.length = 200;
        } else {
            constraint.stiffness = 0.0001;
        }

        gameObject.x = dragX;
        gameObject.y = dragY;
    });

    this.input.on('dragend', function (pointer, gameObject) {
        gameObject.clearTint();
        gameObject.disableInteractive();
    });
}

I think it's the constraint...

But removing that does not change anything:
function create ()
{
    var image = this.matter.add.image(200, 300, 'eye', null, {friction: 0.9});
    image.setStatic(true);

    image.setInteractive();
    var spring = this.matter.add.mouseSpring();
    this.input.setDraggable(image);
    var scene = this;

    ...

    this.input.on('dragend', function (pointer, gameObject) {
        gameObject.clearTint();
        scene.matter.world.remove(spring);
        gameObject.disableInteractive();
    });
}

@strahius just call spring.destroy();

Ah yes! Had not seen that on the API doc. That did it!
For the constraint, I don't think I want to destroy it, as I need it again later. So I create it once, and add/remove from the scene.


CI: Identify where time is being spent across all tests

Looking at our history of CI runs and runs that are performed locally, we should produce an outline of where time is being spent. Time should be represented both as a percentage of the whole, and in actual minutes and seconds to give us a rough idea of real time taken.
For example, we could produce something like this:

rails61: 30 secs (1%)
rails52: 30 secs (1%)
background_2: 2.5 mins (4%)
infinite_tracing: 1.5 mins (3%)

For multiverse, we can start by listing suites and then go more granular with individually nested test stats after.
Once we produce this outline, we should work as a team to produce new issues to:
a) Address performance concerns through test and/or code refactoring
b) Address no-way-around-it slowness by re-evaluating how we group the tests for better parallelism

@fallwith - It looks like there's an existing reporter that can provide us with the information we're looking for: https://www.rubydoc.info/gems/minitest-reporters/1.5.0/Minitest/Reporters/MeanTimeReporter
The only downside is that this reporter isn't available on the version of minitest-reporters compatible with minitest 4.7.5
As minitest 5.0 was released in 2013, are we comfortable spending the time we need to get the tests that fail on minitest 5.0 fixed as part of this ticket?

We can get class-level timing data using the minitest-reporters gem's MeanTimeReporter:

When chatting with James, this seems to be enough to get us over the hurdle of where to start to get the best return on investment when trying to speed things up
I'm going to spend a bit of time on trying to get the timing information for individual tests by creating my own reporter, but not too much.

Some of the code needs to be pulled out in order to get this to pass on the CI. I'm hoping to do that this afternoon.


Abuse of setup_requires for the non-runtime dependency pytest-runner

The setup.py script has as a setup requirement a test-time one, pytest-runner.
According to the docs, setup_requires is for dependencies of the setup-tools themselves.
Such requirements typically are placed in the tests_require argument.
If this is currently impossible due to some installation script, there is additionally the test_suite argument for facilitating the launching of tests.
Thank you for your time.

PR has been merged


Improve namespace prefixes configuration

Feature Request
Allow custom prefixes to be configured with EDC configurations without requiring a custom extension.
Which Areas Would Be Affected?
JsonLDExtension
Why Is the Feature Desired?
Currently it's possible only to configure cached document but if users wants to support custom namespace prefixes an extension is required.
Solution Proposal
Implement a new configuration in JsonLdExtension that read a configuration and register custom namespaces.
Example of configuration:
edc.jsonld.namespace.customns.scopes=MANAGEMENT_API
edc.jsonld.namespace.customns.iri=http://www.w3.org/ns/odrl/2/
edc.jsonld.namespace.customns.prefix=customprefix
edc.jsonld.namespace.customns.enabled=true

the enabled flag seems useless, but it can be used to override registered context via extensions.
This allow having a clean response for people who wants to bring their own context definition and remove the usage of prefix namespace mapping.
For example currently when using the extension management-api-json-ld-context is used, which is an experimental context definition for the EDC management API all the response will contain a @context like this:
{
    "@context": [
        "https://w3id.org/edc/connector/management/v0.0.1",
        {
            "@vocab": "https://w3id.org/edc/v0.0.1/ns/",
            "edc": "https://w3id.org/edc/v0.0.1/ns/",
            "odrl": "http://www.w3.org/ns/odrl/2/"
        }
    ]
}

which is not ideal.

Not relevant, the improvements will be done manually via #4466


Use it on Intune

What it needs to use it on Intune (no Jamf, no IBM notifier in place)?

Did u find a solution for your case?
my idea ist to install IBM notifier or use another notifier.

Did u find a solution for your case? my idea ist to install IBM notifier or use another notifier.

No. I tried with a script running every 30-60 mins from Intune and demoting the privileged account. But the IT sec department preferred a commercial solution, with approvals on every request, etc

It should be easier to deploy on Intune with version 3, which is currently in pre-release.
The new version no longer relies on Jamf to run a policy.


TagEvent id is string instead of number[]

react-native-nfc-manager: 3.1.0
Running on Android (do not know if the problem exist on iOS yet).
Reading tag in TypeScript using NfcManager.getTag()
EXPECTED:  As the TagEvent id is defined like this in index.d.ts
  export interface TagEvent {
    ndefMessage: NdefRecord[];
    maxSize?: number;
    type?: string;
    techTypes?: string[];
    id?: number[];
  }

I expect it to be a number array.
OBSERVED:
However if I dump the attribute tyoe with typeof it is "string"

Hi @raydenvoldeskine your observation is correct! The typing is wrong, I will fix it soon.

Fixed in v3.11.3

Is the change sufficient ? I mean the issue happens "between the worlds"  - conversion from native to type script, would simple change of type script type solve it?

@raydenvoldeskine yes, it’s just a typing issue and native side will always set the .id field to string if it presents.


Issue #2 solved: Stripe API keys are no longer hard-coded

Expected:
Keys to be loaded from a secure place.
The two keys that I found: STORAGE_KEY & STRIPE_API_KEY were renamed to GATSBY_STORAGE_KEY & GATSBY_STRIPE_API_KEY respectively and can be accessed with process.env.GATSBY_STORAGE_KEY or process.env.GATSBY_STRIPE_API_KEY.
Following the Gatsby Docs, I added a file named .env.development in the root folder with the needed keys:
GATSBY_STORAGE_KEY="ABC"
GATSBY_STRIPE_API_KEY="ABC"
I renamed the variables because in gatsby to access the env variables on the client-side JS they need to be prefixed with GATSBY_.

Thanks @jose-donato for getting this started 👍
All .env* files are being ignored in the gitignore, which is good, but it would be good to have a .example.env file that lists the var names that are being set without the actual values.
Also, in the Gatsby docs: https://www.gatsbyjs.org/docs/environment-variables/#server-side-nodejs
Project environment variables that you defined in the .env.* files will NOT be immediately available in your Node.js scripts. To use those variables, use NPM package dotenv to examine the active .env.* file and attached those values, It’s already a dependency of Gatsby, so you can require it in your gatsby-config.js or gatsby-node.js like this:
require("dotenv").config({
  path: `.env.${process.env.NODE_ENV}`,
})

So we'll need to add that in the gatsby-config file.
Lastly, we should include the publishable Stripe key also: https://github.com/jamstack-cms/jamstack-ecommerce/blob/master/src/pages/checkout.js#L20
Let me know if you need any help with these. There's also a Gatsby eCom Tutorial that outlines these steps in case helpful for others :) https://www.gatsbyjs.org/tutorial/ecommerce-tutorial/#advanced-import-skus-via-source-plugin

Didn't saw your answer, sorry.
Already fixed what you suggested in this last commit.

Lastly, we should include the publishable Stripe key also: https://github.com/jamstack-cms/jamstack-ecommerce/blob/master/src/pages/checkout.js#L20

I was a bit confused after reading this, so I added an additional STRIPE_PUBLISHABLE_KEY, should it be the same as STRIPE_SECRET_KEY but called in both lambda.js and checkout.js or they are different keys?


watchPaths not working for Turborepo Monorepo

What version of Remix are you using?
1.7.5
Steps to Reproduce
I have created a simple example monorepo that shows this problem. I may be doing something wrong with Remix, or, I may have something wrong in the setup of my Typescript library. I am fairly new to Typescript. I appreciate your help in figuring this out.
Note that I am using the basic Remix example and chose Vercel as the deployment target.
Clone the following repo...
https://github.com/kevinswarner/remix-turborepo-issue

OS: Mac Monterey 12.6
Node: 18.12.0

Run "npm i" from the root of this project.
Run "npm run dev" from the root of this project.
Open browser to localhost:3000
You should see the text Hello World
Edit helloWorld function in ./packages/hello-world/src/hello-world.ts
Check console. It should show that the hello world library rebuilt and the
remix app rebuilt.
Refresh the page at localhost:3000

The message does not update.
Notes

If you check the ./packages/hello-world/dist directory, it does reflect the
change.
If you check the ./node_modules/hello-world directoty, it does reflect the
change.

Expected Behavior
When changes are made to the typescript library, the change is reflected in the Remix app.
Actual Behavior
It does not update.

The issue is that Remix App Server will purge the require cache for code that was in the /app folder. However, you are importing a component that was built into the shared node_modules folder. RAS is not purging those automatically, so there is no way to update them while it's running. Essentially, you need to make sure the web server process is restarted when updating node_modules.
I tried various techniques to get this to work. nodemon wasn't picking up the changes correctly. I ended up using pm2 but had to switch to an actual express server file.
It works and will trigger a rebuild/refresh for both app route changes as well as external package workspaces.
I'm still experimenting with various things. I will keep you posted.

@kiliman thanks for the note here. I don't suppose you can share what you did with pm2 to make this work?

With v2_dev and a custom app server (i.e. server.{js,ts}) you have full control over which files are watched to trigger rebuilds (via watchPaths) and control over which modules get purged from the require cache in CJS or from the import cache in ESM.
You should be able to just use watchPaths to trigger rebuilds when your node_modules/<my other package> changes, but you can bust the require/import cache for that other package too if need be.


yamerl_parser: Fix detection of a block scalar's end

When a block scalar was ended by a document end marker, it was considered an error because of the unexpected indentation (the indentation was less than the body of the block scalar).
The problem was that this error condition was verified before checking this was a document or directives end marker.
Simply swapping the two code blocks fixes the issue.
Fixes #41.

Coverage increased (+0.05%) to 77.453% when pulling bd7f4fab9319de3b8d2df69de1b8f6587c8bfc33 on fix-block-scalar-followed-by-end-of-doc into f26748aaa84636a85695ce15ea9032fb6ddcf344 on master.


Added custom rule description function

Added possibility to pass a rule description in the easy-rule script module

In an effort to improve the tracking of changes and bug/feature requests, we have decided to require an entry on the pfSense Redmine issue tracker associated with every pull request, and likewise the Redmine entry should also have a link back to the pull request.
If you could, please associate this PR to a Redmine issue either by locating an existing issue at https://redmine.pfsense.org or by creating a new issue. Add a link to the Redmine issue that points to this PR, and also add a link on the PR to the Redmine issue.
For more information, see https://doc.pfsense.org/index.php/Submitting_a_Pull_Request_via_Github
Thanks!


Expired Options Holdings Not Set to Zero

Expected Behavior
When an option expires, its quantity should be set to zero.
Actual Behavior
When an option expires, its quantity is not set to zero.
Potential Solution
N/A
Reproducing the Problem
from datetime import timedelta

class BullCallSpreadAlgorithm(QCAlgorithm):

    def Initialize(self):
        self.SetStartDate(2020, 11, 30)
        self.SetEndDate(2020, 12, 4)
        self.SetCash(600000)

        option = self.AddOption("HPQ")
        option.SetFilter(lambda universe: universe.IncludeWeeklys().Strikes(1, 2).Expiration(0, 7))

        self.symbol = option.Symbol
        self.Schedule.On(self.DateRules.EveryDay(self.symbol),
                         self.TimeRules.AfterMarketOpen(self.symbol, 30),
                         self.EveryMarketOpen)

    def EveryMarketOpen(self):
        if not self.Portfolio.Invested:
            for i in self.CurrentSlice.OptionChains:
                if i.Key != self.symbol: continue
                chain = i.Value
            # sorted the optionchain by expiration date and choose the furthest date
                expiry = sorted(chain,key = lambda x: x.Expiry, reverse=True)[0].Expiry
            # filter the call options from the contracts expires on that date
                call = [i for i in chain if i.Expiry == expiry and i.Right == 0]
            # sorted the contracts according to their strike prices
                call_contracts = sorted(call,key = lambda x: x.Strike)
                if len(call_contracts) == 0: continue
            # buy call option contract with lower strike
                self.Buy(call_contracts[-1].Symbol, 1)
            # sell call option contract with higher strike
                self.Sell(call_contracts[0].Symbol , 1)

    def OnOrderEvent(self, orderEvent):
        self.Log(str(orderEvent))

    def OnEndOfAlgorithm(self):
        for holding in self.Portfolio.Values:
            if holding.Invested:
                self.Log(f'{holding.Symbol} :: {holding.Quantity}')

OnEndOfAlgorithm logs:
2020-12-04 16:00:00 HPQ   201204C00022000 :: -1.0
2020-12-04 16:00:00 HPQ   201204C00023000 :: 1.0

and the unrealized profit is $-103.00 where it should be $0 since there are no open positions.
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

Closed. The options are liquidated on the day after the delisting warning for equities and equity options. Thanks, @gsalaz98!


Update Issue - 60967360

default description
URL tested: http://ci.bsstag.com/welcome
Open URL on Browserstack

Property
Value

Browser
Chrome 77.0

Operating System
Windows 10

Resolution
1024x612

Screenshot Attached
Screenshot URL

Click here to reproduce the issue on Browserstack

Updated Screenshot Attached


useGoogleLogin no success no error , no response

 const handleSuccess = (codeResponse: any) => {
    console.log('Login Success:', codeResponse);
  };

  const handleError = (error: any) => {
    console.log('Login Failed:', error);
  };

  const login = useGoogleLogin({
    onSuccess: handleSuccess,
    onError: handleError,
    scope: 'profile email https://www.googleapis.com/auth/drive',
  });

  return (
    <>

{!!user?.googleCalendarAccessToken ?
          user?.googleCalendarAccessToken :
          < Button onClick={() => login()}>
            Sign in with Google 🚀{' '}
          </Button>
        }

    </>
  );
};

+1

Same, although the same behavior is observed with the GoogleLogin component.
When I call login(), the popup loading google.com prompting login does show up, and I'm able to choose the account and "login". However, afterwards, the popup just closes, and there is no sign of activity from the library, i.e. as stated, the onSuccess and onError do not get called

i am also having the same issue after deployed it to develop.

I´m having the same issue, did you find any solution?

Try wrapping login() call in setTimeout(). It helped me, though i have another issue - redirect not working on IPhones.

Update: i have changed setTimeout timeout from 0 to 10ms and now redirect is working for me, it appears that issue revolved around mobile browsers' specific characteristics of some internal delays. Reference: https://stackoverflow.com/questions/18038602/settimeout-on-mobile

+1

Try to add http://localhost/ to the authorized origins in the Google cloud console

The issue was related to CSP. If you are using helmet please comment and
try..
On Wed, 17 Jan, 2024, 4:06 am Vinícius Valle, @.***>
wrote:

Try to add http://localhost/ to the authorized origins in the Google
cloud console
—
Reply to this email directly, view it on GitHub
https://github.com/MomenSherif/react-oauth/issues/310#issuecomment-1894632067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOSXVM6C2WDFYYHFG6DUTUTYO36G3AVCNFSM6AAAAAA5LLR77GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJUGYZTEMBWG4
.
You are receiving this because you commented.Message ID:
@.***>


chore: testing pipelines

junk PR

why am I a reviewer? :D

@fryorcraken you are the only reviewer we need :D


use codecs module to decode quoted printable email bodys

We're using Python 3.5 and we kept getting this error when parsing emails:
Documentation:
Returns all links found in the email body from given email_index.
Start / End / Elapsed:	20170609 14:05:28.333 / 20170609 14:05:28.500 / 00:00:00.167
14:05:28.334	TRACE	Arguments: [ b'21' ]
14:05:28.499	FAIL	LookupError: 'quoted-printable' is not a text encoding; use codecs.decode() to handle arbitrary codecs
14:05:28.499	DEBUG	Traceback (most recent call last):
  File "/usr/local/Cellar/python3/3.5.2_3/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/ImapLibrary/__init__.py", line 147, in get_links_from_email
    body = self.get_email_body(email_index)
  File "/usr/local/Cellar/python3/3.5.2_3/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/ImapLibrary/__init__.py", line 135, in get_email_body
    decode('quoted-printable')

The attached code fixes that by using the codecs module [https://docs.python.org/3.6/library/codecs.html] to parse the quoted printable email bodies

@peritus Actually, I've noticed that for my emails, the encoding is 'latin-1' instead of 'utf-8'. Any chance you could add an encoding='utf-8' default parameter to the method and then change the decoding step to read:
body = decode(encoded_body, 'quopri_codec').decode(encoding)

Thanks,
Andrew

Thank you, @peritus. Would you please fill in the CLA located here: https://goo.gl/forms/QMyqXJI2LM. Thanks.

Is there any update since May 21?

Is there any update since May 21?

If you need this fix, you can try using
https://github.com/lasselindqvist/robotframework-imaplibrary2 / https://pypi.org/project/robotframework-imaplibrary2/


Add ColumnToArrayTransformer

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (325914a) to head (4f74bff).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #66   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           24        25    +1
  Lines          385       405   +20
  Branches        35        35
=========================================
+ Hits           385       405   +20

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


increased test coverage of generated code

Current coverage is 91.52%

Merging #815 into master will increase coverage by +0.46% as of a236c87

@@            master    #815   diff @@
======================================
  Files           77      77
  Stmts         7815    7815
  Branches      1580    1580
  Methods          0       0
======================================
+ Hit           7117    7153    +36
+ Partial        235     202    -33
+ Missed         463     460     -3

Review entire Coverage Diff as of a236c87

Powered by Codecov. Updated on successful CI builds.

Towards #477


Improve menu bar

Reduce height

Note: No icon buttons have been introduced, since Stock List Icon next to Menu Icon did not look good

Codecov Report

Merging #145 (628a42f) into main (04f7589) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #145   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           43        43
  Lines          850       850
  Branches       348       348
=========================================
  Hits           850       850

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix: allow validate input to handle multiple source groups

Resolves https://github.com/enterprise-contract/ec-cli/issues/1688.
Turns out that the NewInput function only returned the final source group in the policy. This patch makes the function return ALL source groups and changes the caller to evaluate each one, then append all to the output.

I created https://github.com/enterprise-contract/ec-cli/pull/1858 which includes the changes from the PR and addresses the comments.

@lcarva's patch (#1858) is the way to go here, but I added some follow-up comments to clarify a few things just for the sake of completeness.

https://github.com/enterprise-contract/ec-cli/pull/1858 was merged. Let's close this.


PyTree: Pass node_data to tree walkers

Add an optional argument to PyTreeDef::Walk which causes node_data to be passed to the node callback, allowing that callback to inspect the node type.
In particular, it allows the definition of custom printers (see https://github.com/google/jax/issues/5047), which can e.g. print a tree with path labels like:
/a1: leaf-a1
/a2/b1/0: 111
/a2/b1/1: 2
/a2/b1/2: 3
/a2/b2: [[1. 1. 1.]\n [1. 1. 1.]]
/a2/b3/c1: 33
/a2/b3/c2/0: 44
/a2/b3/c2/1: 55
/a3/x1/x11: a
/a3/x1/x12/y1/y11: ya
/a3/x1/x12/y1/y12: yb
/a3/x1/x13: b
/a3/x2/x21: 2a
/a3/x2/x22/y1/y11: ya
/a3/x2/x22/y1/y12: yb
/a4: fred

corresponding to tree obj in the new tests (see below).
Having just seen the comment that Walk is to be deprecated, perhaps an alternative is to make PyTreeDefs a little more introspectable, e.g. add something like treedef.child_names().
The downside of the latter idea is that it's not clear how to define it for Custom nodes,
whereas the node_data is exactly the auxiliary data supplied by the programmer.
I would be happy to implement any proposal that enables printing as above.
@hawkinsp you might be willing to offer a view?

Hi @r4nt Can you please review this PR ? Thank you!

I'm sorry I didn't even see this PR until now! There's a much higher risk I will miss PRs sent to the TF tree since I don't check for those in my email.

No worries, I had a workaround.

I might be tempted to add your new tests there instead.

Will do.

Do I need to do anything on this?  I'm guessing not, but just checking...


Support user props

User get device id,
User get auth provider type
User get profile data

fixes #421 #422 #424


Addressing the Differing Data Distributions in the GAP Test Set

As the authors mentioned, the instances in GAP have certain differences between the female and male groups, e.g., the correct candidate is on average further away from the pronoun in the female group than in the male group, the female instances also contains more candidates on average than the male instances. These would incorrectly make unbiased models appear biased.
In our work, we address this issue. Check it out if you are using the GAP test set as a gender bias diagnostic dataset.
@misc{kocijan2020gap,
      title={The Gap on GAP: Tackling the Problem of Differing Data Distributions in Bias-Measuring Datasets},
      author={Vid Kocijan and Oana-Maria Camburu and Thomas Lukasiewicz},
      year={2020},
      eprint={2011.01837},
      archivePrefix={arXiv},
      primaryClass={cs.CL}
}

right


Support for NC and NV Series VMS on Kubernetes Cluster

I have tried to create a Kubernetes cluster and use NC VMS (Standard_NC6) for the Agents, however I got the following error :
error": {
"code": "InvalidParameter",
"message": "The value 'Standard_NC6' of parameter 'agentProfile.vmSize' is not allowed. Allowed values are 'Standard_A0, Standard_A1, Standard_A2, Standard_A3, Standard_A4, Standard_A5, Standard_A6, Standard_A7, Standard_A8, Standard_A9, Standard_A10, Standard_A11, Standard_D1, Standard_D2, Standard_D3, Standard_D4, Standard_D11, Standard_D12, Standard_D13, Standard_D14, Standard_D1_v2, Standard_D2_v2, Standard_D3_v2, Standard_D4_v2, Standard_D5_v2, Standard_D11_v2, Standard_D12_v2, Standard_D13_v2, Standard_D14_v2, Standard_G1, Standard_G2, Standard_G3, Standard_G4, Standard_G5, Standard_A1_v2, Standard_A2_v2, Standard_A2m_v2, Standard_A4_v2, Standard_A4m_v2, Standard_A8_v2, Standard_A8m_v2, Standard_D15_v2, Standard_F1, Standard_F16, Standard_F2, Standard_F4, Standard_F8, Standard_H16, Standard_H16m, Standard_H16mr, Standard_H16r, Standard_H8, Standard_H8m, Standard_DS1, Standard_DS2, Standard_DS3, Standard_DS4, Standard_DS11, Standard_DS12, Standard_DS13, Standard_DS14, Standard_GS1, Standard_GS2, Standard_GS3, Standard_GS4, Standard_GS5, Standard_DS1_v2, Standard_DS2_v2, Standard_DS3_v2, Standard_DS4_v2, Standard_DS5_v2, Standard_DS11_v2, Standard_DS12_v2, Standard_DS13_v2, Standard_DS14_v2, Standard_DS15_v2, Standard_F16s, Standard_F1s, Standard_F2s, Standard_F4s, Standard_F8s'."
}
}
I have tried to create the ACS cluster in South Central US (NC Series are available there), I have used the latest version of the Python CLI
az acs create --orchestrator-type=kubernetes --resource-group $RESOURCE_GROUP --name=$CLUSTER_NAME --dns-prefix=$DNS_PREFIX --master-count $MASTER_COUNT --agent-vm-size $AGENT_SIZE --agent-count $AGENT_COUNT  --location $LOCATION
Are the NC and NV Series supported ? What about NC_r (RDMA)
Adding support for NC will enable a broad set of scenarios like deep learning with TensorFlow.

This is how the list of allowed VM sizes is generated: https://github.com/Azure/acs-engine/blob/master/pkg/acsengine/Get-AzureConstants.ps1
Maybe there's something missing? I'm not familiar with the N skus though.
@anhowe Maybe we need to get your subscription access to the GPU skus so that they are present in your generated constants file.

@GONZALORUIZ You need to ask to the Azure Support to unlock cores to be able to use NV or NC series. Did you do that, single VM deployment is working ?

@theobolo yes, I have used NC and NV both as standard VMS and with Azure batch. I will open a support ticket and check whether I have to request it again for ACS, but I would be surprised if that is the case.

@GONZALORUIZ Yep you right it should be surprising that you need to ask twice, it should be an issue related to ACS, as Cole said maybe that they need to modify how ACS retrieve the NC and NV series.

@colemickens @anhowe I just double checked this with Azure support, I can NOT yet create Kubernetes clusters using N-Series , is this supported?

Is this an issue about ACS or about ACS-Engine? ACS is not supported via this repo. If this is about ACS, please close this Issue and open an Issue named "GPU SKU support in ACS" on the https://github.com/Azure/ACS repo.
If this is about ACS-Engine, I previously linked to the location that needs to be updated to accommodate the N* serises SKUs.

This is not supported in the RP, but is supported in ACS-Engine per constants: https://github.com/Azure/acs-engine/blob/master/pkg/acsengine/azureconst.go#L209-L215.  Closing.


Update total amount spent state in slice after finishing buy squeak request

https://github.com/squeaknode/squeaknode/pull/2151


Migrate to Android Architecture Components

This is a suggestion to migrate the app to the Android Architecture Components which include ViewModels, LiveData and Data Binding. It is a part of Android Jetpack.
Migrating to MVVM has many advantages like it solves the issue of tight coupling as well as improves testability along with other benefits like the use of lifecycle aware components. MVVM is recommended by Google and is preferred over MVP.
https://android-developers.googleblog.com/2018/05/use-android-jetpack-to-accelerate-your.html
This can be included as a project for GSoC 2020.
Some reference links:
https://developer.android.com/topic/libraries/architecture
https://developer.android.com/topic/libraries/architecture/viewmodel
https://developer.android.com/topic/libraries/architecture/livedata
It would be great if maintainers can share their views on this.

I think the existing architecture is supporting the app well enough and there is no need to migrate the app to a whole new architecture.
@nicolas-raoul @maskaravivek @ashishkumar468 @misaochan
What's your take on this?

I also believe the same. Even though Google promotes MVVM. They never suggested, it is the most suitable architecture for every android application. The current MVP architecture looks pretty robust and I dont think we really need to migrate from it.

I think the existing architecture is supporting the app well enough and there is no need to migrate the app to a whole new architecture.

@GearGit You won't notice much difference in the working of the app even if we don't use any architecture pattern and put all the code into the activities without having presenters. But the actual reason why architecture patterns are used is to make the code more efficient and the development process better. That's why I think migrating to MVVM would be a good idea and would be helpful in the future too since most modern applications follow MVVM.

Your take is right @ShridharGoel . The entire point of an architecture is to improve scalability.
That is why we are following MVP architecture.
Unless there are specific cases which MVP fails to prove efficient, there should not be a need to switch to MVVM.

MVP fails to be efficient when testability is concerned. Since there is dependency of views in case of MVP, unit testing becomes difficult. Similarly, there are other things too where MVVM is much better like it solves the issue of tight coupling and the use of lifecycle aware components have their own advantage.

Yes. MVVM does makes testability more effecticient and I think of the main goals of 2020 for commons app was to improve on the unit tests. Should we discuss more on this @maskaravivek @ashishkumar468 @nicolas-raoul

@ShridharGoel Thank you for suggestion. We recently moved to MVP after a lot of discussion in #888. Still if you think some of the issues are not handled by our current architecture then it would be great if you take a look a look at our current codebase and list down specific examples where MVVP could have been more efficient.

This discussion should be revived, in my opinion.
MVVM is the standard recommended approach since 2017 and has fantastic support from the modern androidx libraries.
The MVP solution currently has a good few issues with logic not contained to presenters, boolean traps in the api, Presenters named as Listeners, views not passed to constructors and associated errors of persistence across config changes as well as confusing potential states from not being data driven UIs from observable sources.
It would not be necessary to completely remove MVP but a gradual shift and commitment to anything new being in MVVM has great potential for benefit to the project in my opinion.

@macgills If the team is interested, then I would like to take this up as a GSoC 2020 project. It would be great if this can be added as a project.


Pinboard panel - hamburguer bug

Replace the hover function to show the menu with a click to prevent hidding the radial annotator when its open

Removed the radial annotator from the pinboard


first time use Claudiajs create in region China error NetworkingError

Please use GitHub issues only to report bugs. To ask a general question or request assistance/support, please use the Claudia.js Gitter Chat instead.
To report a bug or a problem, please fill in the sections below. The more you provide, the better we'll be able to help.

Expected behaviour:

What actually happens:

Link to a minimal, executable project that demonstrates the problem:

Steps to install the project:

Steps to reproduce the problem:

$ claudia create --profile default --region cn-northwest-1 --handler lambda.handler
packaging files npm install -q --no-audit --production
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
initialising IAM role   iam.createRole  RoleName=claudia-test-executor
Error: getaddrinfo ENOTFOUND iam.cn-northwest-1.amazonaws.com.cn
at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:67:26) {
errno: -3008,
code: 'NetworkingError',
syscall: 'getaddrinfo',
hostname: 'iam.cn-northwest-1.amazonaws.com.cn',
region: 'cn-northwest-1',
retryable: true,
time: 2020-12-29T03:10:28.971Z
}

when I change the region to cn-north-1 is ok, but cn-north-1 is Beijing


[WIP] RFFGaussPreproc

Addressing #5009

Proposal: Modify set_randomcoefficients
Reason: It takes in lots of parameters and set internal members value to that.
Rather
a) Take const ref and extract values from it
b) set each internal member value manually, there aren't many.
Also, rename to extract_randomcoefficients because it takes values from the arguments and set internal values from there and not vice versa.

Few changes:

Kept it pass-by-value for now, we would need to change everything in one swoop (DensePreprocessors and all it's derivatives)
instantiated() should be protected method I believe

If we're clear on this, let's start working on the docstring and unit tests already ^_^. (And sorry for the force push again xD)

Unit test passes, wooho!!!

The fluctuation in error we observed seems to be normal, to confirm that I tested the same with sklearn and compared the results. Below I present the graphs comparing Sklearn and shogun

Clearly, both Shogun and sklearn show erratic behaviour when the target_dim is increased linearly. But lets look things from a higher perspective

Et Voila !. It seems when the target_dim difference is large enough the overall effect is increased accuracy. To further prove my point, I'll show the results in even grander perspective

@karlnapf , what sayeth thou?

Questions

are these the same kernel bandwidths? they should be
How many samples are you using? (number of input vectors), the error depends on that

One thing you need to do in order to make the plots more meaningful, you would need to repeat the projection many times (for a single number of random features), and then plot the mean of those runs, plus some sort of error bar or yet better quantiles...something like this
def rff_curve(rffs): # this produces the y values of the curve you show, a 1d array with one error for each rff (array with number of random features)
  ...

# repeat many times
results = np.array([rff_curve(rffs) for _ in range(num_repeat)])

# and then plotting mean and percentiles
plt.plot(rffs, np.mean(results, axis=0))
plt.fill_between(rffs, *np.percentile(results, [5, 95], axis=0), alpha=0.5)

"A picture speaks a thousand words"

@karlnapf The problem with the previous erratic results seems to be due to fixing the seed. When I removed that, the results are proper. I've made proper conversion of log_width to width for sklearn but you can check the notebook to further confirm that.

Can we change the class name to something shorter? Like "RBFSampler" , I'm insisting because the clang-formatted code messes up its generation in class_list.cpp.

I'd suggest to rename only when we refactor the class structures...not yet. This is just a clean up
We should rather fix class_list.py do allow for differently formatted class names
Not sure what you mean by the seed and erratic results.

sorry accidentally closed

Once this issue is solved, all will be merry and this should be good to merge(I think :D)

Once this issue #5027 is solved, all will be merry and this should be good to merge(I think :D)

the issue doesnt need to be solve to merge this, just put stuff in a single line and all good

Once this issue #5027 is solved, all will be merry and this should be good to merge(I think :D)

the issue doesnt need to be solve to merge this, just put stuff in a single line and all good

Linux libshogun clang is bound to fail, because it clang formats it due to which it isn't registered in class_list.cpp. (as was pointed out in the issue)

Once this issue #5027 is solved, all will be merry and this should be good to merge(I think :D)

the issue doesnt need to be solve to merge this, just put stuff in a single line and all good

Linux libshogun clang is bound to fail, because it clang formats it due to which it isn't registered in class_list.cpp. (as was pointed out in the issue)

that is fine, we can ignore a failed format build this time ;)

Restoring getters and setters if there are no objections.

Did you run the tests locally? CI doesnt seem to agree...

Did you run the tests locally? CI doesnt seem to agree...

This is weird, all test passed(except clang,gcc on tests) before, all I changed was width parameter. I haven't done a rebase since a long time, maybe that's causing it. I'll look into it rest assured.

No need to rebase, that won't cause issues (as there is no merge conflict)
The width changing might expose some problem, the rffs might have been wrong but it went unnoticed as the kernel was essentially constant due to the extreme width.

@karlnapf  You were right, I made a mistake in unit-tests the implementation is correct. I've updated the notebook as per your request https://gist.github.com/NanuSai/cf0222c5b7eb5813478c881a1d220a5c. Have a look, and if all good, merge! :D

Sorry, why do you add the data file here? There are no changes to the example ... the implementation of the rff shouldnt have changed, so I dont understand that
I made another comment re memory requirements and additional matrices that I didnt see before.
Otherwise (again), this is very close to being mergable, and it is very nice work, trimming down the number of lines by quite a bit :)

Actually @gf712 pointed out the reason for the CI failure, it had to do with the .dat update. I sent a PR to data repository, I added the .dat just to see if the CI works or not(it works :) ).

I see with the data file, it just wasnt there before. That is good to add :)
Nevermind then. Just let's see if we can get the memory usage down via inplace operations on the projection matrix

Here's what I'm planning

Merge the cos PR
Merge the .dat PR
Remove .dat from this PR
Push here

To answer your question, the .dat file existed before but has been modified by my code. I've sent the .dat file in data submodule PR. I only added .dat here just to check CI accepts it or not(which it does) which confirms the data submodule PR is good to merge.

RE the dat file, why would it change if you haven't changed the implementation?

the CI tests still fail?

the CI tests still fail?

yes, the data file has to be updated, because @jonpsy switched the rng matrix generation from row to column major (to reduce cache misses). He just has to amend the data commit right?

ah of course, yes!
BTW imo there really should be a method that fills random arrays in this way, that takes care of the row/column major, to prevent people shooting into their feet ...

BTW imo there really should be a method that fills random arrays in this way, that takes care of the row/column major, to prevent people shooting into their feet ...

But we would just be wrapping around std::generate? I mean we could potentially extend fill_array with an overload for fill_array(matrix, generator)/fill_array(matrix.begin(), matrix.end(), generator)

maybe you are right, just thinking about how many times I asked someone to use std::generate or std::transform rather than a loop, and how many times people don't respect linearity in memory by for example iterating row-wise....

@gf712 if you don't have any further concerns, I will merge this beast ...

@gf712 if you don't have any further concerns, I will merge this beast ...

sounds good, I think this is ready to be merged! Great work :)


Fix issues with empty aggregates and Bars

Fixes issues aggregating empty Datasets and displaying empty Bars.

Ready to merge.

Looks good. Merging.


[CI] ui-integration tests are unable to start (Error: error:0308010C:digital envelope routines::unsupported, ERR_OSSL_EVP_UNSUPPORTED)

Current Behavior
There's a recurrent issue during running meshery cypress integration tests ('ci-test-integration' npm script called by our buld-ui-and-server.yml ci workflow) in which we get the following error (see here for example):
><EMAIL_ADDRESS>dev16
> eslint . --fix && node ui_dev_server.js

info  - Using webpack 4. Reason: custom webpack configuration in next.config.js https://nextjs.org/docs/messages/webpack5
node:internal/crypto/hash:71
  this[kHandle] = new _Hash(algorithm, xofLen);
                  ^

Error: error:0308010C:digital envelope routines::unsupported
    at new Hash (node:internal/crypto/hash:71:19)
    at Object.createHash (node:crypto:133:10)
    at module.exports.__webpack_modules__.15660.module.exports (/home/runner/work/meshery/meshery/ui/node_modules/next/dist/compiled/webpack/bundle4.js:111680:62)
    at NormalModule._initBuildHash (/home/runner/work/meshery/meshery/ui/node_modules/next/dist/compiled/webpack/bundle4.js:85092:16)
    at handleParseError (/home/runner/work/meshery/meshery/ui/node_modules/next/dist/compiled/webpack/bundle4.js:85146:10)
    at /home/runner/work/meshery/meshery/ui/node_modules/next/dist/compiled/webpack/bundle4.js:851[78](https://github.com/meshery/meshery/actions/runs/3835743870/jobs/6529301044#step:5:79):5
    at /home/runner/work/meshery/meshery/ui/node_modules/next/dist/compiled/webpack/bundle4.js:[85](https://github.com/meshery/meshery/actions/runs/3835743870/jobs/6529301044#step:5:86)033:12
    at /home/runner/work/meshery/meshery/ui/node_modules/next/dist/compiled/webpack/bundle4.js:51096:3
    at iterateNormalLoaders (/home/runner/work/meshery/meshery/ui/node_modules/next/dist/compiled/webpack/bundle4.js:50937:10)
    at Array.<anonymous> (/home/runner/work/meshery/meshery/ui/node_modules/next/dist/compiled/webpack/bundle4.js:50928:4) {
  opensslErrorStack: [ 'error:030000[86](https://github.com/meshery/meshery/actions/runs/3835743870/jobs/6529301044#step:5:87):digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}

Node.js v***.12.1
Error: server closed unexpectedly
    at ChildProcess.onClose (/home/runner/work/meshery/meshery/ui/node_modules/start-server-and-test/src/index.js:75:14)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:10[91](https://github.com/meshery/meshery/actions/runs/3835743870/jobs/6529301044#step:5:92):16)
    at ChildProcess._handle.onexit (node:internal/child_process:302:5)
Error: Process completed with exit code 1.

Desired Behavior
cypress tests should not error out due to starting Meshery UI in development mode throwing a webpack 4 error coming from changes in internal open-ssl lib from Node (webpack issue #14532).
Implementation
According to StackOverflow "Error message "error:0308010C:digital envelope routines::unsupported" question there are several ways to solve it but the wisest one is not to downgrade nor use legacy openssl flag because it leaves several security vulnerabilities unpatched and according to reply "For me adding this in the script once is better than restricting the node version, the long term fix would be to migrate to webpack@5, I don't see additional value in sticking with node16" in webpack issue #14532, long term fix is to migrate to webpack v5.
Acceptance Tests

???

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Closing issue for now.


[BUG] It only zooms into the center of the picture

Describe the bug
PhotoView is only zooming to the center of the image. All gallery applications allow to you zoom anywhere in the picture. However, with this library, wherever I zoom into the image with a double tap or pinch, it only zooms to the center.
To Reproduce
Open an image or gallery, zoom to the right bottom corner of the image and you will see it will be zoomed to the center. Also, if you double-tap to the same part of the image it will be zoomed center again.
What is the current behavior?
Wherever I zoom into the image with a double tap or pinch, it only zooms to the center.
Expected behavior
Users have to see image zoomed with wherever user zoomed with a double-tap or pinch.
Screenshots

Current
Expected

Which versions of Flutter/Photo View, and which browser / OS are affected by this issue? Did this work in previous versions of Photo View?
All versions and all OS.

Any workaround ?


Training

I tried to follow the training documentation. When I run the
cd data
make all
cd ..

It gives the following error.
python ../tools/spellcheck.py < ../tools/jfleg/dev/dev.src > jflegdev.err
Traceback (most recent call last):
  File "../tools/spellcheck.py", line 30, in <module>
    main()
  File "../tools/spellcheck.py", line 19, in main
    print(d.suggest(w)[0], end=" ")
IndexError: list index out of range
make: *** [Makefile:54: jflegdev.err] Error 1
make: *** Deleting file 'jflegdev.err'

I have the file dev.src in tools/jfleg/dev/
I'm using Python 2 for this experiment as it seems it does not support Python 3.

It seems this is a bug in spellcheck.py code. For example, there is a line in dev.src
the carebendiocside is very harm full for the people .

enchant gives an empty array for word carebendiocside and the script throws the error.

Here is the fix I am using
def main():
    args = parse_user_args()
    d = enchant.Dict(args.dict)
    for line in sys.stdin:
        for w in line.strip().split():
            if d.check(w) or not w.isalpha():
                print(w, end=" ")
            elif d.suggest(w):
                print(d.suggest(w)[0], end=" ")
        print()

else case can be built. I don't the logic and context of the script so I'm not using it.


Figure out how to deploy example-nextjs to vercel

We want to be able to automatically deploy changes to vercel for the example-nextjs folder.
The flow for local dev is:

Compile the code into esm / cjs via the command yarn build in the root directory
cd example-nextjs
Run yarn in the example-nextjs project. This uses the link method for the run-wasm dependency
yarn dev

When deploying to vercel, we're unable to use tsc to compile the code in the root src directory into the appropriate modules.
The setup-vercel branch has an attempt at this. Check it out here
Need to find a way to build the package at the root, and properly link, build and run the example-nextjs project
The error message from vercel:
21:21:04.198  	Cloning github.com/slipHQ/run-wasm (Branch: setup-vercel, Commit: f867b10)
21:21:04.784  	Cloning completed: 586.229ms
21:21:04.800  	Analyzing source code...
21:21:05.910  	Installing build runtime...
21:21:07.777  	Build runtime installed: 1.867s
21:21:10.232  	Looking up build cache...
21:21:10.294  	Build Cache not found
21:21:11.412  	Running "install" command: `yarn --cwd ./example-nextjs/ install`...
21:21:11.655  	yarn install v1.22.11
21:21:11.671  	warning package.json: "dependencies" has dependency "typescript" with range "^4.4.3" that collides with a dependency in "devDependencies" of the same name with version "4.4.3"
21:21:11.738  	warning<EMAIL_ADDRESS>"dependencies" has dependency "typescript" with range "^4.4.3" that collides with a dependency in "devDependencies" of the same name with version "4.4.3"
21:21:11.739  	[1/4] Resolving packages...
21:21:12.138  	[2/4] Fetching packages...
21:21:22.142  	info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
21:21:22.142  	info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
21:21:22.142  	info<EMAIL_ADDRESS>The CPU architecture "x64" is incompatible with this module.
21:21:22.142  	info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
21:21:22.142  	info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
21:21:22.143  	info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
21:21:22.143  	info "<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
21:21:22.143  	info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
21:21:22.143  	info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
21:21:22.151  	[3/4] Linking dependencies...
21:21:22.154  	warning "next > styled-jsx ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>21:21:27.393  	[4/4] Building fresh packages...
21:21:27.647  	success Saved lockfile.
21:21:27.651  	Done in 16.00s.
21:21:28.050  	yarn run v1.22.11
21:21:28.093  	$ tsc
21:21:28.105  	/bin/sh: tsc: command not found
21:21:28.115  	error Command failed with exit code 127.
21:21:28.115  	info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
21:21:28.131  	Error: Command "yarn build:esm && yarn build:cjs && yarn --cwd ./example-nextjs/ build" exited with 127

Looks like this can be fixed by
using this as the build command in vercel
yarn install && yarn build && yarn link && cd example-nextjs && yarn link run-wasm && yarn install && yarn build && next export

Tried the same it works. but its only static export. nextjs apis will not work.


Group description is cut off Maps and Apps Gallery

The group description is not fully visible when expanded.  When the window is resized a scroll bar appears but still does enable all the text to be displayed.  This issue occurs when the text exceeds the length of the length of the browser window. This issue occurs in IE, firefox, and chrome. it's most apparent in Chrome.

Example Gallery: https://sharedtheme.maps.arcgis.com/apps/MapAndAppGallery/index.html?appid=dde4717a64844db8b57815cdef11f871

closing - app will be moved to mature support at 6.1. Use Minimal Gallery app instead.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
13 seconds
sanity-inspect-image

✅ Succeeded
13 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
18 seconds
clair-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
8 seconds
sanity-label-check

✅ Succeeded
8 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
13 seconds
deprecated-base-image-check

✅ Succeeded
13 seconds
sanity-inspect-image

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
18 seconds
clair-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
8 seconds
sanity-label-check

✅ Succeeded
8 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
show-summary


[Obsolete] 'pregnancy disorder with abortive outcome'

Mondo term (ID and Label)
MONDO:0041526 'pregnancy disorder with abortive outcome'
Reason for deprecation
Editor note: consider obsoleting
Suggested term to consider
MONDO:0024575 'pregnancy disorder'

I'm not sure of the reason for obsoleting this. I will close this for now.


LCS_WINDOWS_COLOR_SPACE and LCS_sRGB are missing for LcsCsType enum constants

Suggestion
LOGCOLORSPACEA struct has lcsCSType field. The field expects LCS_CALIBRATED_RGB or LCS_sRGB or LCS_WINDOWS_COLOR_SPACE enum constants.
https://learn.microsoft.com/en-us/windows/win32/api/wingdi/ns-wingdi-logcolorspacea
windows-rs only defines LCS_CALIBRATED_RGB. Can we add LCS_sRGB and LCS_WINDOWS_COLOR_SPACE constants?
winapi crate defines these constants:

https://docs.rs/winapi/latest/winapi/um/wingdi/constant.LCS_WINDOWS_COLOR_SPACE.html
https://docs.rs/winapi/latest/winapi/um/wingdi/constant.LCS_CALIBRATED_RGB.html
https://docs.rs/winapi/latest/winapi/um/wingdi/constant.LCS_sRGB.html

Looks like we're missing these from the Win32 metadata. Will transfer this issue for their consideration.


⚠ UdeS website (prod) has degraded performance

In 93909ef, UdeS website (prod) (https://www.usherbrooke.ca/) experienced degraded performance:

HTTP code: 200
Response time: 957 ms

Resolved: UdeS website (prod) performance has improved in 335d91b.


CLI: command to execute arbitrary SQL

I'm surprised this hasn't been requested on the issue tracker yet.
If you want to just run a SQL snippet on your database, currently you have to reach for something database-specific like psql. I personally use CLion's Database tool, but that can't be used in scripts and is a bit daunting to configure.
Of course, it would also be super convenient to have a CLI SQL client that uses the DATABASE_URL that's already set in your .env, which you don't get with psql.
I'm thinking, for an MVP of this:
sqlx exec: connect to DATABASE_URL and read from stdin a single statement terminated with a semicolon or EOF. This will probably be the most convenient way to directly invoke it in CLI.
sqlx exec -m: connect to DATABASE_URL and read stdin until EOF and issue the whole input as a multi-statement command. For sanity, sqlx exec should assume this flag in piped input mode instead of terminating on the first semicolon.
sqlx exec -c <quoted-statement> [bind-params...]: execute quoted-statement with CLI-delimited bind parameters (optional)
sqlx exec -f <file-path> [bind-params...]: load file-path as a SQL statement and execute with CLI-delimited bind parameters (optional)
All these options should support overriding the database URL, of course. We already have precedent for that with other subcommands.
Not sure what I want the output format to be by default. Similar to psql's, I guess? As a bonus feature we should be capable of outputting CSV or JSON as well, maybe with -o csv or -o json.
Also not really thinking about turning this into a REPL just yet, I'd like to keep this simple with an aim towards scripting and composability.

Looks like there's not really a portable way to check if stdin is attached to a TTY or piped in. So maybe sqlx exec should just exit on the first semicolon?

It would be convenient if the CLI could launch a psql command line and connection to database using the parameters defined in the DATABASE_URL.

My ideal is to not even need psql installed for most commands you'd want to issue via CLI. For more advanced dbadmin stuff I personally prefer to reach for a tool with a nice GUI like DataGrip.
We could support subcommands to wrap the native database CLI clients, which I could see being pretty useful, but that's an orthogonal issue, IMO. Go ahead and open one if you like.


Various cosmetic improvements

Related to the larger effort of https://github.com/rust-lang/rust/pull/58036.

There are some test failures:
Mismatch at tests/source/itemized-blocks/wrap.rs:52:
 /// All the parameters ***except for
 /// `from_theater`*** should be inserted as sent
 /// by the remote theater, i.e., as passed to
-/// [`Theater::send`] on the remote actor.
+/// [`Theater::send`] on the remote actor
 fn func1() {}

 /// All the parameters ***except for
test test::self_tests ... FAILED
test test::verify_check_works ... ok
test test::verify_config_test_names ... ok
test unit_tests::test_format_code_block ... ok
test unit_tests::test_format_code_block_fail ... ok
test unit_tests::test_format_snippet ... ok
test unit_tests::test_no_panic_on_format_snippet_and_format_code_block ... ok
test utils::test::test_remove_trailing_white_spaces ... ok
test utils::test::test_trim_left_preserve_layout ... ok
test test::system_tests ... FAILED
failures:
---- test::self_tests stdout ----
internal error: line formatted, but exceeded maximum width (maximum: 100 (see `max_width` option), found: 102)
  --> src/matches.rs:28
   |
28 |     /// `true` if the arm is the last one in match expression. Used to decide on whether we should add
   |                                                                                ^^
   note= set `error_on_unformatted = false` to suppress the warning against comments or string literals
internal error: line formatted, but exceeded maximum width (maximum: 100 (see `max_width` option), found: 144)
    --> src/comment.rs:1813
     |
1813 |     // This is probably intended to be a non-test fn, but it is not used. We should keep this around unless it helps us test stuff to remove it.
     |                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
     note= set `error_on_unformatted = false` to suppress the warning against comments or string literals
internal error: line formatted, but exceeded maximum width (maximum: 100 (see `max_width` option), found: 101)
   --> src/config/file_lines.rs:214
    |
214 |     /// Returns `true` if `self` includes all lines in all files. Otherwise runs `f` on all ranges in
    |                                                                                ^
    note= set `error_on_unformatted = false` to suppress the warning against comments or string literals
internal error: line formatted, but exceeded maximum width (maximum: 100 (see `max_width` option), found: 102)
  --> src/patterns.rs:22
   |
22 | /// Returns `true` if the given pattern is short. A short pattern is defined by the following grammar:
   |                                                                                ^^
   note= set `error_on_unformatted = false` to suppress the warning against comments or string literals
warning: rustfmt may have failed to format. See previous 4 errors.
Ran 5 self tests.
thread 'test::self_tests' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: 1 self tests failed', src/test/mod.rs:249:5
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
---- test::system_tests stdout ----
Ran 346 system tests.
thread 'test::system_tests' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: 1 system tests failed', src/test/mod.rs:109:5

@scampi Thanks for the review. I think I've fixed them now, but we'll see soon...

Thanks!

No problem. Appreciate the quick review & merge.


Add ActivityTypeProperty to TelemetryLoggerMiddleware

Fixes https://github.com/microsoft/botbuilder-dotnet/issues/5023

Pull Request Test Coverage Report for Build 281541

0 of 0   changed or added relevant lines in 0 files are covered.
171 unchanged lines in 13 files lost coverage.
Overall coverage decreased (-0.1%) to 78.647%

Files with Coverage Reduction
New Missed Lines
%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs
1
93.94%

/libraries/Microsoft.Bot.Connector.Streaming/TaskExtensions.cs
1
82.61%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SendActivity.cs
2
65.71%

/libraries/Microsoft.Bot.Connector.Streaming/Application/TimerAwaitable.cs
2
85.71%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/ActionScope.cs
3
96.34%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/LogAction.cs
3
46.43%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/UpdateActivity.cs
3
51.52%

/libraries/Microsoft.Bot.Connector.Streaming/Application/WebSocketClient.cs
4
71.82%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Input/OAuthInput.cs
11
52.21%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Input/InputDialog.cs
12
86.67%

Totals

Change from base Build 279732:
-0.1%

Covered Lines:
24850

Relevant Lines:
31597

💛  - Coveralls

/azp run BotBuilder-DotNet-CI-PR-(MacLinux)-yaml


fix double deploy isues

Changes in subfolders of function parent folder triggered double deployment actions. These actions failed because of duplicated function external_id. Kudos to @pavelzubarev for the fix :)

This change looks fine to me 👍
But to be honest I didn't understand what is causing a duplication issue? Could you provide a more specific example?

The Actions for this PR would fail:
https://github.com/cognitedata/bootcamp-functions/pull/36/files


Date Range Presets

Hey Thanks for this Awesome plugin for React, it is Really outstanding! I wasn't sure how to file this since it is not a request or an issue, I just wanted to know if what I'm trying to accomplish is really posible with the current state of the calendar
Is your feature request related to a problem? Please describe.
I'm trying to create Range presets for the Date Range, and see the preset reflected on the calendar, these presets would be applied via buttons, I've tried to set startDate and endDate state but this doesn't seem trigger the range color highlight, this highlight is only applied when interacting via clicks
Describe the solution you'd like
No real preference, I'm want to know if it is possible, or no
Additional context
Here is a picture of what I'm trying to achieve

This is an sketch rather than a design, the buttons bellow the calendar would select presets, and would be based on current date.
Thanks!, Any insight would be much appreciated!

any update?


📌 Upgrade GitHub Script action

Currently we are using a mixture of version v6 and v5.2.0 (code search) but the latest (as of 17/10/23) is v6.4.1
The warning:
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/github-script@211cb3fefb35a799baa5156f9321bb774fe56294. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/ (blog referenced)
Somehow this hasn't been picked up by Dependabot.
This ticket is to track aligning with the latest release.
Related to https://github.com/ministryofjustice/data-platform/issues/1869
Definition of done

[x] github scripts in workflows updated to resolve the deprecation error

Bot resolved - https://github.com/ministryofjustice/data-platform/commit/37388b85d240a9e441a683a4db6115acfe433888