Google.ProjectGameface version 0.3.30

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Hi @upintheairsheep,
The installer did not complete:

Traceback (most recent call last):
File "run_app.py", line 19, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 352, in exec_module
File "src\gui_init.py", line 15, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 352, in exec_module
File "src\gui\main_gui.py", line 27, in
File "customtkinter_init.py", line 67, in set_default_color_theme
ThemeManager.load_theme(color_string)
File "customtkinter\windows\widgets\theme\theme_manager.py", line 21, in load_theme
with open(theme_name_or_path, "r") as f:
FileNotFoundError: [Errno 2] No such file or directory: 'assets/themes/google_theme.json'

Hi @upintheairsheep,
The installer did not complete:

Traceback (most recent call last):
File "run_app.py", line 19, in
File "", line 1007, in _find_and_load
File "", line 986, in find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 352, in exec_module
File "src\gui__init.py", line 15, in
File "", line 1007, in _find_and_load
File "", line 986, in find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 352, in exec_module
File "src\gui\main_gui.py", line 27, in
File "customtkinter__init.py", line 67, in set_default_color_theme
ThemeManager.load_theme(color_string)
File "customtkinter\windows\widgets\theme\theme_manager.py", line 21, in load_theme
with open(theme_name_or_path, "r") as f:
FileNotFoundError: [Errno 2] No such file or directory: 'assets/themes/google_theme.json'

Try on windows 11 and on a real machine first.

I'm seeing the same as Stephen:

Make an issue on Google’s GitHub repository

[Policy] Blocking-Issue

[Policy] Hardware

Supporting custom hardware in validation is "Not Planned".

https://github.com/microsoft/winget-pkgs/issues/199504


Improve EntityError and SectionError views

The ServerError view  view, SectionError view and EnityError view (from the DXA modules project) are not completely consistent:

view
Git project
location
uses customErrors mode

ServerError.cshtml
dxa-web-application-dotnet
Site/Views/Shared/
yes; is not used when customErrors is On

SectionError.cshtml
dxa-web-application-dotnet
Site/Views/Shared/
no; always shows HTML formatted message with error details

EntityError.cshtml
dxa-modules
Areas/Core/Views/Entity/
no;  always shows HTML formatted message with error details

I propose two changes to improve this.

Move all error views to Site/Views/Shared/ in the dxa-web-application-dotnet project. This way it is clearer to implementors that DXA expects those views to exist, even when the core module does not exist. Also, the EnityError is the only view in Areas/Core/Views/Entity/ which is not registered in the CoreAreaRegistration.
Do not render the technical error detail in the SectionError and EntityError when the customErrors mode is 'On' in the Web.config

Internal Issue ID (for tracking purposes): CRQ-12796


Where to define destination folder path in a bucket?

With given sample file gets uploaded to a bucket but what if we want to upload file in a specific folder in bucket. I simply want to know where to define destination folder path while uploading?

The key controls the folder in which the file is uploaded. Key can be folder1/folder2/file.txt which would make it nested in 2 folders.


fix: snyk HTML report in Azure Pipelines - HTML display issue [ROAD-1197]

updated license information in files
commit with scrolling fix is already on develop (as I supposed if there's a develop branch, we don't need a feature branch). Well we need it for CI/CD to run correctly, even though this means 2 PRs.

https://user-images.githubusercontent.com/20150761/198061032-e8eb3d1f-af05-4e0f-897f-aee064770303.mov
Screencast using 0.0.24 of plugin. https://dev.azure.com/snyk-dev/goof/_build/results?buildId=98&view=snyk-dev.dev-security-scan-test.snyk-report-tab

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

Tried out the fix (it's already on develop). Looks good, now scrolling works.

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

GitHub release
v1.1.1

Your semantic-release bot :package::rocket:


Give warning in cfa-cli if refinement might not support bitvectors

It would be nice to give a warning (with Level.RESULT) if the CFA contains bitvectors, but the refinement might not support it (e.g.,SEQ_ITP).

I agree. The question is where this functionality should be implemented in the code.
My argument is, that the interchangeable solver backend coming with the changes in branch solver-smtlib, the number of such incompatibilities will increase. For example:

Z3 does not support interpolation with bitvectors (in all circumstences)
CVC4 does not support Theta's function typ
MathSAT does not support tree interpolation (while Z3 and SMTInterpol does)
SMTInterpol does not support interpolation with bitvectors, rationals, ...
etc...

I think it would be beneficial to implement a general interface for solvers to describe the functionality they support, and than algorithms, formalisms could assert that.

True, I agree. I think we can never be complete in this sense, i.e., there will always be cases where an exception occurs in the middle of the algorithm). However, if we can at least detect the major cases that would be nice. I like this idea of letting solvers describe their capabilities via an interface. Let's discuss this in more detail on our next meeting.


SendObserver is not called using RabbitMq Transport, work with InMemory

ConsoleApp1.zip
I am replacing a transport in my tests from InMemory to RabbitMq, I use ISendObserver to see if message is sent (as a result of a business logic). To do that I use following code: SendNotificationSendEndpoint.ConnectSendObserver(testSendMessageObserver);
where testSendMessageObserver is ISendObserver implementation with breakpoints in.
It works fine, however when I switch transport to RabbitMq then PreSend/PostSend/SendFault is never called even tough I can see following lines in MT log (and actual messages through RabbitMQ Management website):
2017-04-11 13:21:36.252 +02:00 [Debug] SEND rabbitmq://localhost/SendNotificationQueue a67f0000-5dc7-0015-d170-08d480ccea86 Contracts.SendNotification
2017-04-11 13:21:36.253 +02:00 [Debug] RECEIVE rabbitmq://localhost/consignments N/A Contracts.IConTransaction Consumers.MyConsumer(00:00:00.0209910)

Flow is following:

Consumers.MyConsumer receives a message IConTransaction
Consumers.MyConsumer obtains send endpoint by context.GetSendEndpoint(UriProvider.GetSendNotificationQueueUri())
Consumers.MyConsumer uses that endpoint to send a message sendEndpoint.Send<SendNotification>

If I replace SendNotificationSendEndpoint.ConnectSendObserver(testSendMessageObserver); with BusControl.ConnectSendObserver(testSendMessageObserver); then observer is not called despite using InMemory.
I could provide simple code with reproduction
edit: sample in attachment

1

1

You connect your observer to one endpoint and send messages from another endpoint. This is why your send observer does not fire. You can connect your observer to the bus or to the endpoint, which you use to send messages from.

Yeah, it seems that when getting the send endpoint from a context, observers are not working. I recall there was a similar issue before?

Yeah, I've reproduced this and I know why it's happening. I need to rework how send observers are setup globally. IN reality, just using bus.COnnectSendObserver() should get you everything, and that isn't happening with the transports.

Well, bus.ConnectSendObserver() doesn't help. Same for publish observer. As soon as messages are sent or published from the consume context, observers are not invoked.

I know. It should - so I'm making sure it does.

Ok, should. I found that send and publish observers only connect to the ignore endpoint and not to the actual endpoints, which are used inside the receive context.

It's actually the XxxSendTransport that calls the observers, the wiring is
just broken. I'm fixing it.
On Mon, Apr 17, 2017 at 11:04 AM, Alexey Zimarev<EMAIL_ADDRESS>wrote:

Ok, should. I found that send and publish observers only connect to the
ignore endpoint and not to the actual endpoints, which are used inside
the receive context.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MassTransit/MassTransit/issues/856#issuecomment-294545141,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABOSTxdmhTeaGfvYZ2fDmG3o5tqgg51ks5rw6megaJpZM4M5-sk
.

@phatboyg hey, any progress on that?

Yeah, I am wondering too....

This issue is still present in v4.

Ok, what I found that there are multiple instances of RabbitMqPublishEndpointProvider, which are created. One instance is for the bus itself and one instance per receive endpoint. When we use bus.ConnectPublishObserver, it is only registered on the bus instance and not registered on endpoint instances. The context Publish method uses the provider from the current receive endpoint. There, no observers are registered and therefore no observers are being called.

@molszews if you can check with the latest preview build it would be great. Works for me.

@alexeyzimarev I recompiled the original sample code, now bus.ConnectSendObserver(new Observer()); is working fine, but connecting observer to a send endpoint is still not producing any results. I see why this is happening, but it is a bit misleading as I would expect that observer connected to a particular endpoint will somehow handle all communication that is going through that endpoint

I have not checked this scenario, I only took care that observers connected to the bus also connect to all endpoints. But observers, which are connected to endpoints, work fine, I am not sure about the issue you are having. If you compose a repro repository on Github, I can have a quick look at it.

Hmm, you are connecting your observer to a particular endpoint and in your case we are speaking about two completely different endpoints. The one you are connecting your observer to is not the one, which you get inside the consumer.
For RMQ you get one receive endpoint for the bus instance with non-durable queue, and one for the actual receive endpoint that you configure, with a durable queue. You are connecting your observer to the send endpoint you got from the first receive endpoint (the bus). And you are sending your message from inside the consumer, which is another receive endpoint.
The solution in your case is to use:
cfg.ReceiveEndpoint("input_endpoint", ec =>
{
    ec.Consumer<MessageConsumer>();
    ec.ConnectSendObserver(new Observer());
});

Ah, good to know, thanks!


Document how to use fields for navigation properties

See https://github.com/aspnet/EntityFrameworkCore/pull/10939 and
https://github.com/aspnet/EntityFrameworkCore/issues/11992

Does this ticket imply that only documentation part is remaining and the actual functionality is already in EFCore. If so, could you demonstrate some quick codes how to achieve this?

@KshitizGIT Yes, if you want to use field-only navigations you can configure them like this:
b.HasMany(typeof(PostNavFields), "_posts").WithOne("_blog");


update packages

:white_check_mark: Build TheIdServer 4.4.1-fix-dependencies1-0033 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/e8c0781d98 by @)

:x: Build TheIdServer 4.4.1-fix-dependencies1-0034 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/abecdeb258 by @aguacongas)

:white_check_mark: Build TheIdServer 4.4.1-fix-dependencies1-0035 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/a7b22c64d5 by @aguacongas)

:x: Build TheIdServer 4.4.1-fix-dependencies1-0037 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/0595496e40 by @)


Update "Optimizing bundle size" README section

Since we have es2015 modules support, current size optimization suggestion is outdated and gives not the best result.

cc @TrySound

Codecov Report

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

@@           Coverage Diff           @@
##           master     #367   +/-   ##
=======================================
  Coverage   92.98%   92.98%
=======================================
  Files          51       51
  Lines         328      328
=======================================
  Hits          305      305
  Misses         23       23

Continue to review full report at Codecov.

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

Looks good.

Tree shaking isn't working very well, especially with recompose. Every export makes a call to a function (createHelper). Your implementation is good but it will sadly not impact the final build size of users.

Thanks for your mention. There's a solution. Just to prevent createHelper.
But what is the purpose of your library? Performance? It's our purpose too. A few new utils? It can be different library without copying stuff.

@TrySound yes we have to do it, but there is a few issue, tree shaking isn't very reliable for now.
We use a lot of composition in React at Doctolib, we used recompose for a long time and contributed several times. Recompose was not updated during 6 months, the project was like abandoned. Also in the same time we encountered some performance issues and a lot of debug pain especially in the tree view. After one year of work on observables and with connectObs and withObs, I decided to create recompact to have the control on the core of our stack and being able to give it the right direction. I figured out that @acdlite had the same idea. That's how recompact was born.
After the release of recompact, some releases of recompose appeared and it sounds like going in the good direction now. We could consider merging the two projects at some time. On my side, I have a lot of ideas and improvements for the future.
The purpose of recompact is about performances and debug. Using observables under the hood we can avoid creating some React components in the majority of cases.

Fine. Let's start with treeshaking issue. I think it's more improtant to be consistent and have single source of truth.

The purpose of recompact is about performances and debug. Using observables under the hood we can avoid creating some React components in the majority of cases.

Just curious about real life example of performance benefit (not tests with 10s withState one after each other), as I use recompose a lot and even in projects with thousands of components I haven't seen visually the big difference when i used recompose or plain react classes.
Always I got the same result or bot solution are slow, or both works well.

Wanna add that initially I was also sure that recompose is slow, and it was in some cases - but only in dev mode, when I set NODE_ENV=production it had the same perf as I wrote above

@istarkov performances is not the first concern, I agree React is very fast and stacking components doesn't introduce a bottle neck in majority of cases. But I like the idea to be able to split my higher-order components into small pieces without thinking about it.
At Doctolib, in our application (similar to Google Agenda), we mount a lot of components and recompact permit us to win a few milliseconds.

@neoziro got it, thank you for comment about tree shaking, you are right. Seems like I need to read about how tree shaking works, as for now I cant get why call to identity function breaks all the things.

@istarkov I tested it in a deep way on recompact, also recompose, in fact it's very simple, tree shaking is safe. If a function is executed in the module, it will not drop it because of side effects.
Imagine the code:
export default alert('foo')

Dropping this module will change the behaviour of your application, so it is not dropped. In comparison, in recompose / recompact we do:
export default createHelper(mapProps, 'mapProps')

Tree shaking doesn't know if createHelper create a side effect or not, so the module is included.
There is no simple workaround for recompose / recompact to deal with it (except duplicating code in
createHelper or transpile it before give it to the bundler).
That's why I recommend you to recommend the usage of "babel-plugin-lodash" even if the tree shaking is effective in the project. By the way, good work on the bundle part!

Seems like babel-plugin-lodash still is the best way to reduce bundle size. I use it for a long time and it works quiet well.

Removal of createHelper call for production build will make Tree Shaking approach great again.
So let's wait a little.

@istarkov I think we can put both approaches into doc, right?

Agree with @wuct 's approach for common building system, and I use babel-plugin-import to do the same thing in my project:
// babel plugin
{
  libraryName: 'recompose',
  libraryDirectory: '/', // default: lib
  camel2DashComponentName: false, // default: true
},

https://github.com/MCS-Lite/mcs-lite/blob/master/packages/babel-preset-mcs-lite/index.js#L45-L49

@wuct  I'm not sure, having both you need to support both ;-)

Having both is the same as lodash entry points problem. Somebody like lodash/module, somebody lodash.module, somebody import { module } from 'lodash';

I've reopened this issue accidentally 😓

Let's try removing all createHelper and see if it works. I agree we should put only the best practice in the doc.

@istarkov You may reopen this PR now. Tree shaking works well.


fix for gnome shell 3.38

Thanks to https://extensions.gnome.org/accounts/profile/pemmoura

I simply removed the import since it was not required at all. This should not break the code for older gnome shell versions

Could you update it on extension.gnome.org please?

I submitted the new extension version, it takes a few days for the review process.

@Keridos  FYI: The new version is available on extension.gnome.org


Documentation missing AzureBlobLoggerOptions properties?

Please refer to my Stackoverflow question at: https://stackoverflow.com/questions/70992091/net-azureblobloggeroptions-properties-documentation
In the section entitled Azure App Service, reference is made to AzureBlobLoggerOptions, however, the documentation for the options seems to be missing some properties, such the FileNameFormat property (an example or two would be most helpful).  Could the documentation be updated to include the missing properties?
Thanks in advance.

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

ID: 7184a308-a254-9350-a5cd-5bced1f369ae
Version Independent ID: 726e3bf1-f367-d733-8933-bccc04da0e16
Content: Logging in .NET Core and ASP.NET Core
Content Source: aspnetcore/fundamentals/logging/index.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

It looks like FileNameFormat was added in 6.0. There doesn't seem to be an API docs page for anything after 3.1 for the AzureBlobLoggerOptions class, so that explains why it's missing. I don't know why there isn't a 5.0 or 6.0 version of the API docs...

@Rick-Anderson Should this issue be transferred elsewhere? How do we understand why there's no version > 3.1 for AzureBlobLoggerOptions?

I'll ping @v-alje offline.


Fixed a bug in AZD integration code

Fixed a bug where the appsettings.json file was not actually rewritten

API change check
API changes are not detected in this pull request.


Code blocks in the websites are not showing up

Just checked and they're showing up fine in Chrome/Firefox/Edge. Does your browser console show any errors?


Preserve RUSTDOCFLAGS so that custom --cfg settings can be applied.

Allow users to set --cfg options included when scanning a crate by invoking the tool as:
RUSTDOCFLAGS="--cfg your-cfg-value" cargo semver-checks

Thanks to @wmmc88 for suggesting it in #1012!


Question about sample topk of foreground points?

if self.training and self.train_cfg.get('disable_pretrain', False) and not self.runtime_info.get('enable_detection', False):
            seg_scores = seg_scores[:, cls_id]
            topks = self.train_cfg.get('disable_pretrain_topks', [100, 100, 100])
            k = min(topks[cls_id], len(seg_scores))
            top_inds = torch.topk(seg_scores, k)[1]
            fg_mask = torch.zeros_like(seg_scores, dtype=torch.bool)
            fg_mask[top_inds] = True

Does anyone know why here we need to filter the points with the top 500 highest score for each category by score?
After sampling, each class only have 500 points, Will it lose a lot of information?
In my understanding, this is the point that constitutes all virtual voxels. Why is such a small threshold needed?
Hope someone can give me some help, I'm very confused on this part, thanks a lot！

This is only enabled at the very beginning of training. You can refer to this hook for details: https://github.com/tusen-ai/SST/blob/main/mmdet3d/core/hook/fsd_hooks.py#L70C11-L70C11

Thank you！


Integrate GoReleaser

Good way to get my feet wet with GitHub Actions.

Version 1.0.0 successfully released. Easy enough.


Add dropdown to switch teams for issues

Switching the owner of an issue within the same team currently works well, but two use cases don't:
• Moving it as an unassigned item to a different team (important so they can find their own owner)
• Moving it to another team member (since we don't show a list for / can't search for people in all teams)

Updated to allow switching of project for tickets.  If you use 1 team per project, then this is an effective solution.  If not, we may still want to implement some team switching or alternate names in the assignee popup.


python error

Just installed to Ubuntu 16.04
Linux mail2 4.5.7-std-3 #1 SMP Tue Jul 12 09:56:30 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
as per instructions using pip and get this error :
$ cat file.html|tq -t title
Traceback (most recent call last):
  File "/usr/local/bin/tq", line 5, in <module>
    tq.main()
  File "/usr/local/lib/python2.7/dist-packages/tq/__init__.py", line 55, in main
    selected_els = get_els(args.selector)
  File "/usr/local/lib/python2.7/dist-packages/tq/__init__.py", line 50, in get_els
    input_stream = io.TextIOWrapper(sys.stdin.buffer, encoding='utf-8', errors='ignore')
AttributeError: 'file' object has no attribute 'buffer'

I should probably improve the documentation a little bit. Reading from with sys.stdin in python 2 is tricky and prone to portability issues like this one. I didn't think that many systems would still default their pip command to python 2. But I think that is still the case of ubuntu.
In your case, to get it to work, uninstall tq with pip uninstall. Then install using pip3. Should go something like this:
sudo -s
pip uninstall tq
pip3 install

if pip3 is not present install it from ubuntu repositories. The package is called python3-pip

I added a mention to this on the readme.
@redlegoman did it work?


Customer registration, login, forgotpassword

#6

And here is the my working copy of this changes.
All works fine for me.
https://cezerin2.slack.com/files/UFVAW9HA4/FGCL59ETV/cezerin-customer-register-login-account.zip

To test new email templates, features, i have local smtp server.
Setup your smtp server at /config/server.js to test account activation, password recovery feature.
New email templates and options added to the "npm run setup"
/src/api/server/setup.js

If you have error when you click register button, try to clear cache in Chrome - Developer Console - Application - Clear storage - Clear site data.
actions.js:774 Uncaught (in promise) TypeError: Cannot read property 'retrieve' of undefined
in /src/store/shared/actions.js
:
export const registerUser = (data, callback) => async (dispatch, getState) => {
const response = await api.ajax.register.retrieve(data);
dispatch(handleRegisterProperties(response.json));
};
this string:
api.ajax.register.retrieve(data);

Add a few missing files.

you can create an account from the checkout

Work is still required in:

Customer Login (have not tested)
Customer Register (have not tested)

I Verify all working

ok, merge this pr.

I Verify all working

Only verified working, but have not checked code.

When this PR will be merged?!
I test it on my local machine and it's ok.
Customer Registration.
Customer Activation.
Customer Order Checkout.
Customer Logoff.
Customer Account Page.
New settings for "npm run setup".
Al of this works fine at my local machine.


AppBar bottom widget, not necessarily a TabBar

The AppBar widget that appears below the toolbar no longer has to be a TabBar. It does have to implement AppBarBottomWidget, which just reports the bottom widget's height.
Removed the minimumHeight AppBar parameter.


The hardcoded FIPSKEY break the intergrity security

The hardcoded FIPSKEY is HMAC key, this is sensitive data but hardcode, the intergrity will loss when the attacker changed the HMAC key and the signature file together.
The hardcoded FIPSKEY in openssl/Configure:
$config{FIPSKEY} =
'f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813';

This is an ops problem

This is what is mandated by NIST.
The intention of the integrity test is not to stop an attacker, it's to detect corruption of the module.

So，this key was not regard as senstive data

No, not sensitive at all.


[Monorepo Builder] Fetch name and path from config, do not hardcode in .yaml

First PR continuing from #2715, to fix #2698.
Replaces packages/${{ matrix.package }} with ${{ matrix.package.path }} in the YAML workflows, as to enable additional packages in other locations (eg: projects/).
Also get package name with ${{ matrix.package.name }}

I guess this would be a BC break :/

I guess this would be a BC break :/

Closing for lack of feedback. We can't have BC break now

Closing for lack of feedback. We can't have BC break now

Closing for lack of feedback. We can't have BC break now

Closing for lack of feedback. We can't have BC break now

Closing for lack of feedback. We can't have BC break now

Closing for lack of feedback. We can't have BC break now


Hex colors Orange and Raspberry not registering

Challenge learn-about-tertiary-colors has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

@noone396 I don't think this is an issue, you're just using other colors than what's tested for.
Try using the colors from the instructions :blush:


Update compound series schema

Addresses discussion in #32.
https://github.com/choderalab/covid-moonshot-infra/issues/32#issuecomment-683371304
@jchodera I started implementing your suggested schema as types, with a few minor modifications so far, which are mostly a matter of preference (happy to change back if you have a strong feeling):

compounds and molecules are lists (of objects with a *_id field) instead of objects; I think is slightly more conventional for a stable schema that we might want to validate

changed some fields previously represented as strings of the form "value * unit" to floats with the unit explicit in the field name (e.g. "temperature": "300*kelvin" -> "temperature_kelvin": 300.0

Example JSON (updated)
{
  "metadata": {
    "name": "2020-08-20-benzotriazoles",
    "description": "Sprint 3: Prioritization of benzotriazole derivatives",
    "creator": "John D. Chodera",
    "created_at": "2020-09-08",
    "xchem_project": "Mpro",
    "receptor_variant": {
      "biological_assembly": "monomer",
      "protein_variant": "thiolate"
    },
    "temperature_kelvin": 300,
    "ionic_strength_millimolar": 70,
    "pH": 7.4
  },
  "compounds": [
    {
      "compound_id": "MAT-POS-f42f3716-1",
      "smiles": "Cc1ccncc1NC(=O)Cc1cc(Cl)cc(-c2ccc(C3CC3(F)F)cc2)c1",
      "experimental_data": {
        "pIC50": 4.324
      },
      "microstates": [
        {
          "microstate_id": "MAT-POS-f42f3716-1-1",
          "smiles": "Cc1ccncc1NC(=O)Cc1cc(Cl)cc(-c2ccc(C3CC3(F)F)cc2)c1"
        },
        {
          "microstate_id": "MAT-POS-f42f3716-1-2",
          "smiles": "Cc1ccn[H+]cc1NC(=O)Cc1cc(Cl)cc(-c2ccc(C3CC3(F)F)cc2)c1"
        }
      ]
    },
    {
      "compound_id": "MAT-POS-f42f3716-2",
      "smiles": "Cc1ccncc1NC(=O)Cc1cc(Cl)cc(-c2ccc(S(C)(=O)=O)cc2Cl)c1",
      "experimental_data": {
        "pIC50": 4.324
      },
      "microstates": [
        {
          "microstate_id": "MAT-POS-f42f3716-2-1",
          "smiles": "Cc1ccncc1NC(=O)Cc1cc(Cl)cc(-c2ccc(C3CC3(F)F)cc2)c1"
        },
        {
          "microstate_id": "MAT-POS-f42f3716-2-2",
          "smiles": "Cc1ccncc1NC(=O)Cc1cc(Cl)cc(-c2ccc(C3CC3(F)F)cc2)c1"
        }
      ]
    }
  ],
  "projects": {
    "complex_phase": 12345,
    "solvent_phase": 12346
  },
  "transformations": [
    {
      "run": 0,
      "initial_microstate": {
        "compound_id": "MAT-POS-f42f3716-1",
        "microstate_id": "MAT-POS-f42f3716-1-1"
      },
      "final_microstate": {
        "compound_id": "MAT-POS-f42f3716-1",
        "microstate_id": "MAT-POS-f42f3716-1-2"
      },
      "xchem_fragment_id": "x10789"
    },
    {
      "run": 1,
      "initial_microstate": {
        "compound_id": "MAT-POS-f42f3716-2",
        "microstate_id": "MAT-POS-f42f3716-2-1"
      },
      "final_microstate": {
        "compound_id": "MAT-POS-f42f3716-2",
        "microstate_id": "MAT-POS-f42f3716-2-2"
      },
      "xchem_fragment_id": "x10789"
    }
  ]
}

changed some fields previously represented as strings of the form "value * unit" to floats with the unit explicit in the field name (e.g. "temperature": "300*kelvin" -> "temperature_kelvin": 300.0

The Open Force Field Initiative folks (@j-wags, @SimonBoothroyd, etc) have some code to appropriately deal with things like this---might check in with them in case there is a well-supported idiom we can reuse.

Quantity serialization is hard. The OpenFF Toolkit currently implements quantities-as-strings, but we recognize the kludginess and are hoping to replace it in our next refactor
@simonboothroyd, @mattwthompson, and I discussed this a little on Monday, and kinda concluded "it's hard". My memory isn't perfect, but I think that Simon said if performance isn't a huge deal, the built-in Pint serialization is fine. But if doing I/O with thousands of unit-bearing fields, it starts to add up.

The Open Force Field Initiative folks (@j-wags, @SimonBoothroyd, etc) have some code to appropriately deal with things like this---might check in with them in case there is a well-supported idiom we can reuse.

There a several approaches that can be taken to serialize quantities each with their own shortcomings, and I go back and forth about which approach I support doing. Ultimately we don't have a consistent way of doing unit serialization yet.

if performance isn't a huge deal, the built-in Pint serialization is fine. But if doing I/O with thousands of unit-bearing fields, it starts to add up.

If your data is shaped as structures of arrays and you wrap the unit bearing array in a Quantity like object the performance isn't an issue - you only have to validate the unit type once upon deserialising, and when converting to something like a numpy array or pandas data frame you don't need to validate and unify the unit on each entry in the list as it's just attached to the list as a whole. In this case having one field for the unit and one for the value is probably an ok way to go.
The issue is when your data is shaped as an array of structures, where each structure has a (or multiple) unit bearing attribute(s) - think a data set object which stores a list of data set entries. In this case the overhead becomes large for large lists (hundreds of thousands to millions) of structures as you have to trawl through the entire list validating units when deserialising, and similar when you want to extract say all of the values into a list or make sure they have the same units (say when mass inserting objects into a database which should not have a heterogeneous set of units). In this case I much prefer the approach implemented in this PR.
There are numerous other issues associated with the serialisation of unit bearing things but I won't hijack this thread with them.
As of now, I currently serialize my unit bearing quantities as just floats (or arrays) with the attribute having a fixed unit.
In future pydantic (+1 for pydantic over dataclass) may allow us to do clever things like store only the unit-less value on the model but then attach / convert units with getters / setters when a user access / sets the value on the model, but this currently isn't possible.

@SimonBoothroyd thanks for the detailed response! pydantic looks cool!

Let's change these in metadata:
    "biological_assembly": "monomer",
    "protein_variant": "thiolate",

to a receptor_variant: dict section that contains info about the receptor structure.
Let's ditch
      "is_racemic_mixture": false,
      "has_multiple_protonation_states": true,
      "has_multiple_tautomers": false,

and use compound_id instead of cid throughout.
molecules: Add optional field state_penalty_kT which contains a float assumed to be 0.0 if not specified.

@jchodera given that molecules have a compound_id would it make sense nest them under compounds? E.g. something like
{
  "compounds": [
    {
      "compound_id": "MAT-POS-f42f3716-1",
      "smiles": "Cc1ccncc1NC(=O)Cc1cc(Cl)cc(-c2ccc(C3CC3(F)F)cc2)c1",
      "experimental_data": {
        "pIC50": 4.324
      },
      "molecules": [
        {
          "molecule_id": "MAT-POS-f42f3716-1-1",
          "smiles": "Cc1ccncc1NC(=O)Cc1cc(Cl)cc(-c2ccc(C3CC3(F)F)cc2)c1"
        },
        {
          "molecule_id": "MAT-POS-f42f3716-1-2",
          "smiles": "Cc1ccn[H+]cc1NC(=O)Cc1cc(Cl)cc(-c2ccc(C3CC3(F)F)cc2)c1"
        }
      ]
    }
  ]
}

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f550e6c). Click here to learn what that means.
The diff coverage is n/a.

@jchodera can we add compound_id as a property of Transformation? (i.e., will we ever want transformations between different compounds?)

@jchodera given that the compound to molecule relationship is one-to-many, would it make sense nest molecules under compounds?

Yes, that's totally fine! Let's call them microstates then, since they're different microscopic species that the compound can exist in, and then microstate_id to identify each microstate.

@jchodera can we add compound_id as a property of Transformation? (i.e., will we ever want transformations between different compounds?)
(The motivation is to make the code simpler, avoiding the need to parse out a prefix from the molecule_id)

Transformations can be between different compounds or different microstates of the same compound.
I'm happy for whatever scheme you think will lead to simpler code!

@jchodera one more thing I just thought of: I think we had discussed including project numbers for the complex and solvent. Is this a good place to add them?

If the RUN number is going to be included in the transformations section, let's include the project numbers in the metadata section

@jchodera, BTW, feel free to push to this branch; the schema and test case are isolated from the rest of the code, so it won't break anything and might be more efficient than going back and forth for minor changes.


Add more tests

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

Comparison is base (9ff748c) 93.38% compared to head (39a548a) 93.62%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #812      +/-   ##
==========================================
+ Coverage   93.38%   93.62%   +0.23%
==========================================
  Files          93       93
  Lines        9915     9927      +12
==========================================
+ Hits         9259     9294      +35
+ Misses        511      486      -25
- Partials      145      147       +2

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


Update copy on DT Toolkit homepage

Update tense of intro text to be present

Avoid duplication of ‘Digital Trust Toolkit’ in the two headings

Adjust style of Governance card description to match the other cards

Pause until research consolidation completed.

Ticket to draft new content exists in Delivery team backlog exists. Once ready, this can be tackled.


Face Swagger doesn't respect case for OperationStatus

JSON received:
{
  "CreatedTime": "2019-05-09T16:12:11.5891562Z",
  "LastActionTime": "2019-05-09T16:12:15.1138077Z",
  "Message": "Failed to retrieve the source object. This failure can be caused by deleting source object or training source object during snapshot taking.",
  "ResourceLocation": null,
  "Status": "failed"
}

But Swagger defined status, lastActionTime, etc.
Note that Swagger are case sensitive, hence this is a Swagger issue:
Reference:
https://github.com/Azure/azure-sdk-for-python/issues/5224

I can confirm Face API will return all these fields with lower case, and the Swagger file also uses lower cases.
{
  "createdTime": "2019-05-09T16:12:11.5891562Z",
  "lastActionTime": "2019-05-09T16:12:15.1138077Z",
  "message": "Failed to retrieve the source object. This failure can be caused by deleting source object or training source object during snapshot taking.",
  "resourceLocation": null,
  "status": "failed"
}

Hi @lebronJ
Sorry, but my own recording shows the capital letters case + a customer saw it. My own recording:
https://github.com/Azure/azure-sdk-for-python/blob/master/azure-cognitiveservices-vision-face/tests/recordings/test_snapshot.yaml#L67

Hi @lmazuel
Sorry, please ignore my last reply (deleted). I can reproduce that Face API returns upper case for operation status.
Our plan to fix this is to change Face API's behavior to return lower case, then this problem will be auto resolved for Python SDK. Moreover in this approach, there's no need to change the Swagger, thus no effect to API references or other SDKs. We will roll this change out globally with our next planned release, ETA at round Jun 7th.
What do you think about this fix? Is this super urgent so that we need a quicker hot fix?

Hi @lebronJ
First, yes, this would solve the problem and no, it's not urgent that you should urge your timeline.
I must mention that's a breaking change for customers that use directly your RestAPI today, their code won't work anymore. It's why ARM team forbids this kind of changes for instance. If your API is in preview however, or if the SDK are considered your top scenario, you might consider it as an acceptable side effect of a bug fix. Note that I'm not doing any recommendation about where it's the right solution or not, I just want to be sure you know the side effect of this one :)

Hi @lmazuel , thanks for the explanation. Per our team's internal discussion, we will treat this as a service bug fix and will change Face API's behavior to return lower case.
The reason behind this is the fact that our documentation and swagger are consistent (lower case), while response from service is different (upper case). Moreover, this API interface (Snapshot) is called in a quite low frequency since it was on board. So we think the potential side effect of fixing the service's response should be rather limited.
We have combined this fix in our ongoing release, if everything goes well, this issue will be solved by end of this week.


python312Packages.reolink-aio: 0.9.6 -> 0.9.7

Diff: https://github.com/starkillerOG/reolink_aio/compare/refs/tags/0.9.6...0.9.7
Changelog: https://github.com/starkillerOG/reolink_aio/releases/tag/0.9.7
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 333165 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.reolink
python311Packages.reolink-aio
python311Packages.reolink-aio.dist
python312Packages.reolink-aio
python312Packages.reolink-aio.dist


Henryiii touchup

Pinning gdk_pixbuf and librsvg, since old versions can crash ROOT when opening a TBrowser (now listed explicitly in deps to make sure they pin) (should fix #15 )
Rerender to get above pinning working (fixed minor typo in rerender)
Slightly better message for setting the SDK on macOS
Patch to fix C++ std flag from root-config on macOS (closes #13)
Fix #14, cannot be installed with root5 also
Better names for patches in some cases
Add symlinks to the compilers through the compilers feedstock

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@henryiii I think this is ready to merge, do you have anything else left to change?
(I'm happy to skip waiting given the Azure CI passed)

Everything passed. Sure, I'm ready for the merge.


multiple replication streams (client)

Is it already possible to do replicate multiple feeds over one instance of the client proxy?
I saw in the code that ProxyStream should already support this, but how would this work with the client?

Just found out that the native discovery-swarm does not support this as well.
(or at least dat-node uses one swarm per dat)

Hypothetically you could use hypercore-protocol to replicate as many feeds as you want over a single proxied stream to a client.


[WIP] fix(dag): ensure dag.put() allows for optional options

Note: Only review last commit as it's rebased on top of #785 to make test runner work
This is to align with API changes in
https://github.com/ipfs/interface-ipfs-core/commit/011c417b97e58aceafaa4e98fce8f32217bc5cf7
and
https://github.com/ipfs/js-ipfs/pull/1415
Temporarily rebased on top of #785
Note: this is a WIP as I have to run now :D will make tests green once I'm back

@alanshaw following your comment on https://github.com/ipfs/js-ipfs-api/pull/785#issuecomment-402197142, I've changed the base of this PR to your branch. If you approve, let's get this one merged there and then merge one clean PR to master.

@PascalPrecht I pushed a fix, I hope you don't mind, this PR is now on the critical path to getting the modular interface tests merged 😉
I think the problem was that hashAlg was being passed to dag.put but it was expecting hash. The API often allows you to pass the HTTP API querystring name instead of the interface-ipfs-core name for the argument (they differ sometimes) so I've preserved this functionality.


Allow the drawing and editing of all animation bezier curves

Based on godotengine/godot-proposals#3141.

Moves the bezier edit button from being a property of a track to the animation editor's toolbar
Draws all bezier curves simultaneously and allows them to be selected simultaneously. Only one track remains 'primary selected', which affects adding new keyframes, or prioritizing keyframes when they overlap.
Can click on a curve to select its track
F zooms in to encompass the selected key(s) and the previous + next (based on Maya's graph editor)
Tie the track editor's filter view to bezier editor, hiding all tracks that are not part of that object.
Ctrl+A selects all keys that can be selected (lock/hide/filter to prevent selection), Shift+Ctrl+A deselects all keys.
Tracks can be hidden and/or locked. Adds a solo-button as well to hide all but that particular track.
Fix editor visual bugs. Changes in the way 4.0 drew some stuff, like the introduction of TextLine, wasn't taken into account, so visually it was starting to break in a few places.

https://user-images.githubusercontent.com/2823241/142140926-6ebba3d3-a9a9-49ac-a9e7-66744223f1f4.mp4

Great work so far :slightly_smiling_face:

Ctrl+A selects all keys that can be selected. Lock/hide/filter to prevent selection

Could Ctrl + Shift + A be added to deselect all keys?

Could Ctrl + Shift + A be added to deselect all keys? This would complement Ctrl + A well.

Sure!

Some bugs I found testing. I'm not sure they're all related to this PR specifically.

First there's the issue when the icons overlap the layer names, the text should get clipped somehow.

After creating bezier tracks, say, for position, trying to insert a key from the inspector or the toolbar loc/rot/scale key icon gives the following error:

ERROR: Condition "arr.size() != 6" is true.
   at: track_insert_key (scene/resources/animation.cpp:1628)

Also, it only inserts the key for one of the channels at most. The picture below shows a key inserted only for the position:y component after clicking the key icon next to Transform -> Position in the Inspector.

The X and Y axis colors are inverted in 2D: Keying position:x draws a curve in green and keying position:y draws a curve in red.

First there's the issue when the icons overlap the layer names, the text should get clipped somehow.

Yeah, I wanted to grow the timeline to some minimum size, but I don't have access to the timeline width from the bezier editor. Clipping is likely the better solution in the interrim.

After creating bezier tracks, say, for position, trying to insert a key from the inspector or the toolbar loc/rot/scale key icon gives the following error:

ERROR: Condition "arr.size() != 6" is true.
   at: track_insert_key (scene/resources/animation.cpp:1628)

Weird. I'll look into that. The array size being wrong seems to imply it sent a key without handle_mode being taken into account.

Also, it only inserts the key for one of the channels at most. The picture below shows a key inserted only for the position:y component after clicking the key icon next to Transform -> Position in the Inspector.

Will have to take a look, thanks.

The X and Y axis colors are inverted in 2D: Keying position:x draws a curve in green and keying position:y draws a curve in red.

The colors are calculated from the hash of their property path rather than their axes, so it's always been the case. Would have to somehow detect that it's coming from position/rotation/scale to do axis coloring.

Highlighted bugs and some mentioned in private conversation should be fixed, and I swapped the lock icons around to act like the visibility icon.

Just tested everything again, it's all working well so far.

Note: Currently the Bezier track insertion from the Inspector is broken, but this PR might fix it.

Note: Currently the Bezier track insertion from the Inspector is broken, but this PR might fix it.

I've added keys from the inspector in the process of testing, so it should work.

looks good to me!

Needs a rebase to solve merge conflicts.

@akien-mga Gone ahead and added your suggestions. Thanks :)

Thanks!

Doesn't the "bezier_edit" signal in AnimationTrackEdit is no longer needed?

Doesn't the "bezier_edit" signal in AnimationTrackEdit is no longer needed?

You're right that it is no longer in use. I've gone ahead and removed it in #57920

Due to a change in the _key_selected() argument, key selection in AnimationTrackEditor is broken.


Disable Input when Chat is Starting

Part of https://github.com/WideChat/Rocket.Chat/issues/581

Running the code locally:
(.venv) ~/git/Rocket.Chat.Livechat $ git log
git log
commit 00ca3d39540fa8e2a201a421d0620185759853a8 (HEAD -> develop, shailesh/sb_starting_chat)
Author: Shailesh Baldaniya<EMAIL_ADDRESS>Date:   Thu Jun 24 19:56:22 2021 +0530

    Update build files

commit 70e41c596624ddd86a261d0c702ac02b17e25b12
Author: Shailesh Baldaniya<EMAIL_ADDRESS>Date:   Thu Jun 24 19:52:56 2021 +0530

    Disable Input when Chat is Starting

I

was able to get input in before it had received the text associated with the initial intent

@Shailesh351 ^^^

tested and verified


CB-5966. Configure DataBus (metering/logging/telemetry publisher) to use proxy.

Add proxy configs:

fluentd (dbus plugin)
telemetry publisher (CM component)

will require some more testing (manual)

Target version should 2.20

from squid access log with using fluentd databus plugin
1587122455.161 1115091 <IP_ADDRESS> TCP_TUNNEL/200 64141 CONNECT dbusapi.sigma-dev.cloudera.com:443 squid HIER_DIRECT/<IP_ADDRESS>

I think that one is validated
Next: validate with Telemetry Publisher

seems like Telemetry Publisher configured properly, and also it's started. I could not check every kind of hadoop tools around that, but if it will be any issues can be done later or TP can be disabled if it does not work with http only proxies


Make V3 accept workbench preamble queries

SET TRANSACTION ISOLATION LEVEL REPEATABLE READ
SET @@SESSION.wait_timeout=610
help 'SELECT' -- serverside help
It also complains about keyspace performance_schema not found in vschema. And same for mysql and sys.

@dveeden let me know if I've missed something.

It also tries to enable 'safe updates'.
On 3 September 2017 3:31:36 p.m. Sugu Sougoumarane
<EMAIL_ADDRESS>wrote:

SET TRANSACTION ISOLATION LEVEL REPEATABLE READ
SET @@SESSION.wait_timeout=610
help 'SELECT' -- serverside help
It also complains about keyspace performance_schema not found in  vschema. And same for mysql and sys.

@dveeden let me know if I've missed something.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/youtube/vitess/issues/3130

i connected mysql-workbench to a local mysql with general log turned on, then ran the queries through vitess. the following failed:
SELECT logfile_group_name FROM information_schema.FILES;
SELECT NAME FROM mysql.func;
SELECT tablespace_name FROM information_schema.FILES;
SET SESSION TRANSACTION ISOLATION LEVEL REPEATABLE READ;
SET SQL_SAFE_UPDATES=1;
USE `database_name`;
show charset;
show collation;
SHOW COLUMNS FROM `database_name`.`table_name`;
SHOW ENGINES;
SHOW FULL TABLES FROM `database_name`;
SHOW FULL TABLES FROM `database_name` LIKE '%';
SHOW FUNCTION STATUS WHERE Db='database_name';
SHOW GLOBAL VARIABLES;
SHOW PROCEDURE STATUS WHERE Db='database_name';
SHOW SESSION STATUS LIKE 'Ssl_cipher';
SHOW SESSION VARIABLES LIKE 'interactive_timeout';
SHOW SESSION VARIABLES LIKE 'lower_case_table_names';
SHOW SESSION VARIABLES LIKE 'offline_mode';
SHOW SESSION VARIABLES LIKE 'sql_mode';
SHOW SESSION VARIABLES LIKE 'version';
SHOW SESSION VARIABLES LIKE 'version_comment';
SHOW SESSION VARIABLES LIKE 'version_compile_os';
SHOW SESSION VARIABLES LIKE 'wait_timeout';
SHOW TRIGGERS FROM `database_name` WHERE `Table` = 'table_name';

and vitess didn't choke on the following query, but it showed all of them, instead of doing the like:
SHOW DATABASES LIKE 'database_name';

I will take this.


View on submission db to determine if changes are pending

The mobile application needs to know if, for a given document id, changes are pending in the submission DB.
Add a view to find submissions based on a document id. Add a reduce function to indicate if the document has pending changes.

Still need a call on the submission database servlet to return the view results

Removed the view submission-docId. Added views submission-info-by-id and submission-info-by-device-id. The concept of deviceId has to do with finding out information about all submissions for a given mobile device. To do so, the URL is contacted:
.../servlet/submission/_submission-info-by-device-id?key=
The device id is set on a submission
{
    nunaliit_submission:{
        deviceId: <deviceId>
    }
}

When submitting a document (or deleting) via the submission servlet, it is possible to add the parameter 'deviceId' to the URL (as query parameters). This device id is saved in the submission.


How should one go about adapting this to use with the flux dev model?

How should one go about adapting this to use with the flux dev model?

Please follow this issue https://github.com/rohitgandikota/erasing/issues/32
We are working on a solution for this :)


Feature request: Action for opening a new integrated terminal with a non-default shell

There is currently a keyboard shortcut mapping for workbench.action.terminal.new, which opens a new terminal instance of the default shell type. There's also a mapping for workbench.action.terminal.selectDefaultShell, although I can't find a setting in settings.json that this changes.
I'd like to see a new action, e.g. workbench.action.terminal.newNonDefault, which opens up the same select list as for selectDefaultShell but instead of setting the default, just opens an instance of the selected type. This would enable me to have PowerShell as my default, but open a bash or cmd shell every now and then without having to change the default back and forth.

I recommend checking out this extension which gives you a command to select the shell https://marketplace.visualstudio.com/items?itemName=Tyriar.shell-launcher


Remove classes with no references

I used UCDetector (http://www.ucdetector.org/) as a plugin for Eclipse.  With this tool, I discovered a lot of code without any reference (variables, methods and classes).
Following the work that was done at [https://github.com/apache/cloudstack/pull/1448]; this pull request had the goal of removing some of these classes. To check if I wasn't missing anything I searched for any file that could reference some of those classes. As I haven't found any way of these classes being used, they were removed. Note that some of them I found other references, but references such as commented lines or tests, nothing that could indicate their use (as XML files configuring beans or another class instantiating an object with "new").
Waiting for tests. Please tell me if I am missing something.
Removed Classes:

org.apache.cloudstack.framework.jobs.JobCancellationException (Note: removed
variable JobCancellationException in com.cloud.utils.SerialVersionUID)
org.apache.cloudstack.ldap.NoSuchLdapUserException (Note: removed test file
/cloud-plugin-user-authenticator-ldap/test/groovy/org/apache/cloudstack/ldap/NoSuchLdapUserExceptionSpec.groovy)
com.cloud.agent.api.storage.CreateVolumeOVAAnswer
com.cloud.exception.MissingParameterValueException
org.apache.cloudstack.api.response.StatusResponse
org.apache.cloudstack.api.response.VolumeDetailResponse
org.apache.cloudstack.api.response.UpgradeVmResponse
org.apache.cloudstack.api.response.AddIpToVmNicResponse
org.apache.cloudstack.api.response.TemplateZoneResponse (Note: at
org.apache.cloudstack.api.response.TemplateResponse, there is this
comment "To avoid breaking backwards compatibility, we still treat a
template at different zones as different templates, so not embedding
template_zone information in this TemplateZoneResponse set. private Set<TemplateZoneResponse> zones;" but right now it is not used)
org.apache.cloudstack.api.response.NicDetailResponse

I built, ran the unit tests and ran the server with the simulator. Thanks for cleaning up! LGTM
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache CloudStack Developer Tools - Checkstyle Configuration  SUCCESS [  1.481 s]
[INFO] Apache CloudStack ................................. SUCCESS [  2.319 s]
[INFO] Apache CloudStack Maven Conventions Parent ........ SUCCESS [  1.075 s]
[INFO] Apache CloudStack Framework - Managed Context ..... SUCCESS [  5.519 s]
[INFO] Apache CloudStack Utils ........................... SUCCESS [ 20.629 s]
[INFO] Apache CloudStack Framework ....................... SUCCESS [  0.035 s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [  6.274 s]
[INFO] Apache CloudStack Framework - Configuration ....... SUCCESS [  3.581 s]
[INFO] Apache CloudStack API ............................. SUCCESS [ 17.061 s]
[INFO] Apache CloudStack Framework - REST ................ SUCCESS [  2.250 s]
[INFO] Apache CloudStack Framework - IPC ................. SUCCESS [  5.209 s]
[INFO] Apache CloudStack Cloud Engine .................... SUCCESS [  0.043 s]
[INFO] Apache CloudStack Cloud Engine API ................ SUCCESS [  4.154 s]
[INFO] Apache CloudStack Framework - Security ............ SUCCESS [  1.968 s]
[INFO] Apache CloudStack Core ............................ SUCCESS [ 24.328 s]
[INFO] Apache CloudStack Agents .......................... SUCCESS [  7.392 s]
[INFO] Apache CloudStack Framework - Clustering .......... SUCCESS [  4.394 s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [  1.455 s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [ 14.251 s]
[INFO] Apache CloudStack Framework - Jobs ................ SUCCESS [  4.137 s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [  2.500 s]
[INFO] Apache CloudStack Server .......................... SUCCESS [ 44.828 s]
[INFO] Apache CloudStack Framework - Quota ............... SUCCESS [  5.957 s]
[INFO] Apache CloudStack Usage Server .................... SUCCESS [  5.625 s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS [  8.316 s]
[INFO] Apache CloudStack Cloud Services .................. SUCCESS [  0.029 s]
[INFO] Apache CloudStack Secondary Storage ............... SUCCESS [  0.230 s]
[INFO] Apache CloudStack Secondary Storage Service ....... SUCCESS [  4.525 s]
[INFO] Apache CloudStack Engine Storage Component ........ SUCCESS [  5.890 s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [  4.042 s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [  2.578 s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [  2.076 s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [  1.821 s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [  7.113 s]
[INFO] Apache CloudStack Cloud Engine API ................ SUCCESS [  1.391 s]
[INFO] Apache CloudStack Cloud Engine Service ............ SUCCESS [  4.426 s]
[INFO] Apache CloudStack Plugin POM ...................... SUCCESS [  0.300 s]
[INFO] Apache CloudStack Plugin - API Rate Limit ......... SUCCESS [  5.367 s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS [  2.396 s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS [  4.066 s]
[INFO] Apache CloudStack Plugin - API SolidFire .......... SUCCESS [  1.818 s]
[INFO] Apache CloudStack Plugin - API Discovery .......... SUCCESS [  3.680 s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [  1.571 s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS [  1.612 s]
[INFO] Apache CloudStack Plugin - Explicit Dedication Processor  SUCCESS [  1.848 s]
[INFO] Apache CloudStack Plugin - User Concentrated Pod Deployment Planner  SUCCESS [  1.717 s]
[INFO] Apache CloudStack Plugin - User Dispersing Deployment Planner  SUCCESS [  1.660 s]
[INFO] Apache CloudStack Plugin - Implicit Dedication Planner  SUCCESS [  5.252 s]
[INFO] Apache CloudStack Plugin - Skip Heurestics Planner  SUCCESS [  1.438 s]
[INFO] Apache CloudStack Plugin - Host Allocator Random .. SUCCESS [  1.754 s]
[INFO] Apache CloudStack Plugin - Dedicated Resources .... SUCCESS [  4.998 s]
[INFO] Apache CloudStack Plugin - Hypervisor OracleVM .... SUCCESS [  2.442 s]
[INFO] Apache CloudStack Plugin - Open vSwitch ........... SUCCESS [  2.689 s]
[INFO] Apache CloudStack Plugin - Hypervisor XenServer ... SUCCESS [ 22.957 s]
[INFO] Apache CloudStack Plugin - Hypervisor KVM ......... SUCCESS [ 10.390 s]
[INFO] Apache CloudStack Plugin - RabbitMQ Event Bus ..... SUCCESS [  1.770 s]
[INFO] Apache CloudStack Plugin - In Memory Event Bus .... SUCCESS [  3.110 s]
[INFO] Apache CloudStack Plugin - Kafka Event Bus ........ SUCCESS [  1.437 s]
[INFO] Apache CloudStack Plugin - Hypervisor Baremetal ... SUCCESS [  3.183 s]
[INFO] Apache CloudStack Plugin - Hypervisor UCS ......... SUCCESS [  2.155 s]
[INFO] Apache CloudStack Plugin - Hypervisor Hyper-V ..... SUCCESS [  4.373 s]
[INFO] Apache CloudStack Plugin - Hypervisor OracleVM3 ... SUCCESS [ 17.905 s]
[INFO] Apache CloudStack Plugin - Network Elastic Load Balancer  SUCCESS [  4.764 s]
[INFO] Apache CloudStack Plugin - Network Internal Load Balancer  SUCCESS [  7.849 s]
[INFO] Apache CloudStack Framework - Spring Life Cycle ... SUCCESS [  1.700 s]
[INFO] Apache CloudStack Plugin - Network Juniper Contrail  SUCCESS [  9.838 s]
[INFO] Apache CloudStack Plugin - Palo Alto .............. SUCCESS [  5.376 s]
[INFO] Apache CloudStack Plugin - Network Netscaler ...... SUCCESS [  3.538 s]
[INFO] Apache CloudStack Plugin - Network Nicira NVP ..... SUCCESS [ 11.095 s]
[INFO] Apache CloudStack Plugin - BigSwitch Virtual Network Segment  SUCCESS [  8.695 s]
[INFO] Apache CloudStack Plugin - Network Brocade VCS .... SUCCESS [  7.456 s]
[INFO] Apache CloudStack Plugin - Midokura Midonet ....... SUCCESS [  5.149 s]
[INFO] Apache CloudStack Plugin - Stratosphere SSP ....... SUCCESS [  5.350 s]
[INFO] Apache CloudStack Plugin - Network Opendaylight ... SUCCESS [  4.635 s]
[INFO] Apache CloudStack Plugin - Storage Allocator Random  SUCCESS [  1.480 s]
[INFO] Apache CloudStack Plugin - User Authenticator LDAP  SUCCESS [  2.706 s]
[INFO] Apache CloudStack Plugin - User Authenticator MD5 . SUCCESS [  3.360 s]
[INFO] Apache CloudStack Plugin - User Authenticator PBKDF2-SHA-256  SUCCESS [  4.444 s]
[INFO] Apache CloudStack Plugin - User Authenticator Plain Text  SUCCESS [  1.742 s]
[INFO] Apache CloudStack Plugin - User Authenticator SAML2  SUCCESS [ 33.553 s]
[INFO] Apache CloudStack Plugin - User Authenticator SHA256 Salted  SUCCESS [  3.433 s]
[INFO] Apache CloudStack Plugin - Dns Notifier Example ... SUCCESS [  1.689 s]
[INFO] Apache CloudStack Plugin - Storage Image S3 provider  SUCCESS [  1.950 s]
[INFO] Apache CloudStack Plugin - Storage Image Swift provider  SUCCESS [  1.686 s]
[INFO] Apache CloudStack Plugin - Storage Image default provider  SUCCESS [  1.754 s]
[INFO] Apache CloudStack Plugin - Storage Image sample provider  SUCCESS [  1.603 s]
[INFO] Apache CloudStack Plugin - Storage Volume Nexenta Provider  SUCCESS [  3.603 s]
[INFO] Apache CloudStack Plugin - Storage Volume CloudByte Provider  SUCCESS [  2.691 s]
[INFO] Apache CloudStack Plugin - Storage Volume sample provider  SUCCESS [  1.640 s]
[INFO] Apache CloudStack Plugin - SNMP Alerts ............ SUCCESS [  3.388 s]
[INFO] Apache CloudStack Plugin - Syslog Alerts .......... SUCCESS [  4.680 s]
[INFO] Apache CloudStack Plugin - Network VXLAN .......... SUCCESS [  3.897 s]
[INFO] Apache CloudStack Plugin - GloboDNS ............... SUCCESS [  6.082 s]
[INFO] Apache CloudStack Plugin - Quota Service .......... SUCCESS [  4.870 s]
[INFO] Apache CloudStack Plugin - Hypervisor Simulator ... SUCCESS [  4.056 s]
[INFO] Apache CloudStack Framework - Spring Module ....... SUCCESS [  5.767 s]
[INFO] Apache CloudStack Secondary Storage Controller .... SUCCESS [  2.372 s]
[INFO] Apache CloudStack Client UI ....................... SUCCESS [ 10.796 s]
[INFO] Apache CloudStack Console Proxy - RDP Client ...... SUCCESS [  9.548 s]
[INFO] Apache CloudStack Console Proxy ................... SUCCESS [  0.331 s]
[INFO] Apache CloudStack Console Proxy - Server .......... SUCCESS [  5.782 s]
[INFO] Apache CloudStack Framework - QuickCloud .......... SUCCESS [  0.116 s]
[INFO] Apache CloudStack Test ............................ SUCCESS [  0.405 s]
[INFO] Apache CloudStack Developer Mode .................. SUCCESS [  0.673 s]
[INFO] Apache CloudStack Developer Tools ................. SUCCESS [  0.223 s]
[INFO] Apache CloudStack apidocs ......................... SUCCESS [ 27.713 s]
[INFO] Apache CloudStack marvin .......................... SUCCESS [  7.112 s]
[INFO] Apache CloudStack DevCloud ........................ SUCCESS [  0.684 s]
[INFO] Apache CloudStack DevCloud4 ....................... SUCCESS [  0.366 s]
[INFO] Apache CloudStack DevCloud-KVM .................... SUCCESS [  0.364 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 09:44 min
[INFO] Finished at: 2016-03-30T10:39:18-05:00
[INFO] Final Memory: 121M/735M
[INFO] ------------------------------------------------------------------------

Thank you @pdube!

@GabrielBrascher I'm getting a merge error on this PR to master. can you have a look and rebase if needed?

@DaanHoogland, @GabrielBrascher,
The merge errors are occurring because the PR has merge conflicts.
A rebase should solve that.
@DaanHoogland, are you going to merge this PR?
It only has 1 LGTM and no functional tests executed for it.
Even though I think that when removing unused code (code without any reference at all), we do not need functional tests (a complete build and unit + integration tests are enough), that is how it has been done so far. So, I am just asking to avoid any future complications. @swill, what do you think here?

@rafaelweingartner yes, I will be running CI on this when the merge conflicts have been resolved.  nothing should be committed to master without CI run against it IMO...

@swill thanks
I did not know that you were also watching this PR.
Man, you are literally everywhere; that is very good.

haha, trying to be.  it is turning into a full time job...

@swill , @DaanHoogland ,
Now everything is ok here.
@GabrielBrascher was a little busy, so I did the rebase for him.

@rafaelweingartner the merge errors occured during testing as merging is part of the integration tests. I agree with @swill about the test being needed for this change. thought I think it makes no sense/adds no value for some other changes. The latter being UI and added tests. Every thing else should undergo the regression suite.

@DaanHoogland I understand your points on requiring functional tests and accept it.
Even after the rebase I did today, you had problems while testing this PR?

I am having other problems (after the merge) in my test environment, not related to the PR.

CI RESULTS
Tests Run: 85
  Skipped: 0
   Failed: 0
   Errors: 1
 Duration: 9h 04m 44s

Summary of the problem(s):
ERROR: Test VPC offering without port forwarding service
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs1/cloudstack/test/integration/component/test_vpc_offerings.py", line 799, in test_05_vpc_off_without_pf
    domainid=self.account.domainid
  File "/usr/lib/python2.7/site-packages/marvin/lib/base.py", line 4217, in create
    return VPC(apiclient.createVPC(cmd).__dict__)
  File "/usr/lib/python2.7/site-packages/marvin/cloudstackAPI/cloudstackAPIClient.py", line 2182, in createVPC
    response = self.connection.marvinRequest(command, response_type=response, method=method)
  File "/usr/lib/python2.7/site-packages/marvin/cloudstackConnection.py", line 379, in marvinRequest
    raise e
Exception: Job failed: {jobprocstatus : 0, created : u'2016-05-09T15:20:37+0200', jobresult : {errorcode : 530, errortext : u'Failed to create VPC'}, cmd : u'org.apache.cloudstack.api.command.admin.vpc.CreateVPCCmdByAdmin', userid : u'7762822f-15a0-11e6-91f9-5254001daa61', jobstatus : 2, jobid : u'b5b0ebc2-11b3-451f-ab08-5b45ce58d61e', jobresultcode : 530, jobresulttype : u'object', jobinstancetype : u'None', accountid : u'77626b72-15a0-11e6-91f9-5254001daa61'}
----------------------------------------------------------------------
Additional details in: /tmp/MarvinLogs/test_vpc_routers_I1C28S/results.txt

Associated Uploads
/tmp/MarvinLogs/DeployDataCenter__May_09_2016_06_45_25_CAQM5J:

dc_entries.obj
failed_plus_exceptions.txt
runinfo.txt

/tmp/MarvinLogs/test_network_MTB4IB:

failed_plus_exceptions.txt
results.txt
runinfo.txt

/tmp/MarvinLogs/test_vpc_routers_I1C28S:

failed_plus_exceptions.txt
results.txt
runinfo.txt

Uploads will be available until 2016-07-10 02:00:00 +0200 CEST
Comment created by upr comment.

The one problem that is showing up in my CI does not appear to be related to this PR, but this is the first time I have ever seen this error.  If someone else can confirm that this would not be related, I would appreciate it.  :)

Can I get one more code review on this one???  Thx...

LGTM @swill


AspnetCore 2.0 sample

Prerequisites

[x] I have written a descriptive pull-request title
[x] I have verified that there are no overlapping pull-requests open
[x] I have verified that I am following the Nancy code style guidelines
[x] I have provided test coverage for my change (where applicable)

Description

Added AspnetCore 2.0 hosted sample

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

@thecodejunkie, @khellang, thank you for your feedback, the amendments have been committed as per your comments.


Bug PingableDriverCompilerPass?

I have an issue in the PingableDriverCompilerPass.php.
The PdoPeriodicPing service is not working on 1.8.3, yet we have working code on 1.8.1.
On line 25 to 30 the following code is executed:
`
$sessionHandler = $container->getAlias('gos_web_socket.session_handler');
if (false === $sessionHandler instanceof PdoSessionHandler) {
return;
}
`
As you can see the if statement will always return false because $sessionHandler will be an instance of Alias instead of PdoSessionHandler.
If i change $container->getAlias to $container->get (as it was in 1.8.1) i receive the proper instance and my code will execute the PdoPeriodicPing.

Can you try with $sessionHandler = $container->get($container->getAlias('gos_web_socket.session_handler')->getId());
And said me if that solve your problem? I'll push the fix if it's the case. Thank you for reporting it @koenjonker

Thanks for the quick reply!!
There is no getId function on Alias, but when you parse it as string the above method works!
$sessionHandler = $container->get((string)$container->getAlias('gos_web_socket.session_handler'));

You can run composer update, it's fixed in 1.8.4. Give me feedback if any problem. Thank you!


cuda.h: No such file or directory

Description
I can not perform manual MXNet-R Installation on Ubuntu 18.04 LTS with cuda 10.1 and cudnn 7.6.4. The post-installation actions of cuda and cudnn had been performed successfully.
Environment info (Required)
What to do:
1. Download the diagnosis script from https://raw.githubusercontent.com/apache/incubator-mxnet/master/tools/diagnose.py
2. Run the script using `python diagnose.py` and paste its output here.

----------Python Info----------
('Version      :', '2.7.15+')
('Compiler     :', 'GCC 7.4.0')
('Build        :', ('default', 'Oct  7 2019 17:39:04'))
('Arch         :', ('64bit', ''))
------------Pip Info-----------
('Version      :', '19.2.3')
('Directory    :', '/usr/local/lib/python2.7/dist-packages/pip')
----------MXNet Info-----------
('Version      :', '1.5.1')
('Directory    :', '/usr/local/lib/python2.7/dist-packages/mxnet')
('Commit Hash   :', 'c9818480680f84daa6e281a974ab263691302ba8')
('Library      :', ['/usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so'])
Build features:
✔ CUDA
✔ CUDNN
✔ NCCL
✔ CUDA_RTC
✖ TENSORRT
✔ CPU_SSE
✔ CPU_SSE2
✔ CPU_SSE3
✔ CPU_SSE4_1
✔ CPU_SSE4_2
✖ CPU_SSE4A
✔ CPU_AVX
✖ CPU_AVX2
✖ OPENMP
✖ SSE
✔ F16C
✖ JEMALLOC
✖ BLAS_OPEN
✖ BLAS_ATLAS
✖ BLAS_MKL
✖ BLAS_APPLE
✔ LAPACK
✖ MKLDNN
✔ OPENCV
✖ CAFFE
✖ PROFILER
✔ DIST_KVSTORE
✖ CXX14
✖ INT64_TENSOR_SIZE
✔ SIGNAL_HANDLER
✖ DEBUG
----------System Info----------
('Platform     :', 'Linux-4.15.0-65-generic-x86_64-with-Ubuntu-18.04-bionic')
('system       :', 'Linux')
('node         :', 'machinelearning')
('release      :', '4.15.0-65-generic')
('version      :', '#74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019')
----------Hardware Info----------
('machine      :', 'x86_64')
('processor    :', 'x86_64')
Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian
CPU(s):              8
On-line CPU(s) list: 0-7
Thread(s) per core:  2
Core(s) per socket:  4
Socket(s):           1
NUMA node(s):        1
Vendor ID:           AuthenticAMD
CPU family:          21
Model:               2
Model name:          AMD FX(tm)-8350 Eight-Core Processor
Stepping:            0
CPU MHz:             1403.655
CPU max MHz:         4000.0000
CPU min MHz:         1400.0000
BogoMIPS:            8027.19
Virtualization:      AMD-V
L1d cache:           16K
L1i cache:           64K
L2 cache:            2048K
L3 cache:            8192K
NUMA node0 CPU(s):   0-7
Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf pni pclmulqdq monitor ssse3 fma cx16 sse4_1 sse4_2 popcnt aes xsave avx f16c lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs xop skinit wdt fma4 tce nodeid_msr tbm topoext perfctr_core perfctr_nb cpb hw_pstate ssbd ibpb vmmcall bmi1 arat npt lbrv svm_lock nrip_save tsc_scale vmcb_clean flushbyasid decodeassists pausefilter pfthreshold
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0046 sec, LOAD: 0.9385 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0035 sec, LOAD: 1.0756 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.8954 sec, LOAD: 0.3998 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.2478 sec, LOAD: 0.2311 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.3526 sec, LOAD: 0.2759 sec.
Timing for Gluon Tutorial(cn): https://zh.gluon.ai, DNS: 0.7199 sec, LOAD: 0.7465 sec.
----------Environment----------

I'm using R
R version 3.6.1 (2019-07-05)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 18.04.3 LTS

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/openblas/libblas.so.3
LAPACK: /usr/lib/x86_64-linux-gnu/libopenblasp-r0.2.20.so

locale:
 [1] LC_CTYPE=C.UTF-8       LC_NUMERIC=C           LC_TIME=C.UTF-8
 [4] LC_COLLATE=C.UTF-8     LC_MONETARY=C.UTF-8    LC_MESSAGES=C.UTF-8
 [7] LC_PAPER=C.UTF-8       LC_NAME=C              LC_ADDRESS=C
[10] LC_TELEPHONE=C         LC_MEASUREMENT=C.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

loaded via a namespace (and not attached):
[1] compiler_3.6.1

Build info (Required if built from source)
Compiler (gcc/clang/mingw/visual studio):
gcc --version
gcc (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0

MXNet commit hash: cf61364a2f38c90769098b330d5ecbafc34d4ed2
Build config for short:
# whether use CUDA during compile
USE_CUDA = 1

# choose the version of blas you want to use
# can be: mkl, blas, atlas, openblas
# in default use atlas for linux while apple for osx
UNAME_S := $(shell uname -s)
ifeq ($(UNAME_S), Darwin)
USE_BLAS = apple
else
USE_BLAS = openblas
endif

Error Message:
Makefile:360: WARNING: Significant performance increases can be achieved by installing and enabling gperftools or jemalloc development packages
g++ -std=c++11 -c -DMSHADOW_FORCE_STREAM -Wall -Wsign-compare -O3 -DNDEBUG=1 -I/home/nam/Downloads/incubator-mxnet/3rdparty/mshadow/ -I/home/nam/Downloads/incubator-mxnet/3rdparty/dmlc-core/include -fPIC -I/home/nam/Downloads/incubator-mxnet/3rdparty/tvm/nnvm/include -I/home/nam/Downloads/incubator-mxnet/3rdparty/dlpack/include -I/home/nam/Downloads/incubator-mxnet/3rdparty/tvm/include -Iinclude -funroll-loops -Wno-unused-parameter -Wno-unknown-pragmas -Wno-unused-local-typedefs -msse3 -mf16c -I/include -DMSHADOW_USE_CBLAS=1 -DMSHADOW_USE_MKL=0 -I/home/nam/Downloads/incubator-mxnet/3rdparty/mkldnn/build/install/include -DMSHADOW_RABIT_PS=0 -DMSHADOW_DIST_PS=0 -DMSHADOW_USE_PASCAL=0 -DMXNET_USE_MKLDNN=1 -DUSE_MKL=1 -I/home/nam/Downloads/incubator-mxnet/src/operator/nn/mkldnn/ -I/home/nam/Downloads/incubator-mxnet/3rdparty/mkldnn/build/install/include -DMXNET_USE_OPENCV=1 -I/usr/include/opencv -DMSHADOW_INT64_TENSOR_SIZE=0 -DMXNET_USE_BLAS_OPEN=1  -DMXNET_USE_NCCL=0 -DMXNET_USE_LIBJPEG_TURBO=0 -MMD -c src/operator/quantization/mkldnn/mkldnn_quantized_elemwise_add.cc -o build/src/operator/quantization/mkldnn/mkldnn_quantized_elemwise_add.o
In file included from /home/nam/Downloads/incubator-mxnet/3rdparty/mshadow/mshadow/tensor.h:16:0,
                 from include/mxnet/./base.h:33,
                 from include/mxnet/operator_util.h:43,
                 from src/operator/quantization/mkldnn/../../tensor/elemwise_unary_op.h:28,
                 from src/operator/quantization/mkldnn/../quantized_elemwise_add-inl.h:28,
                 from src/operator/quantization/mkldnn/mkldnn_quantized_elemwise_add.cc:27:
/home/nam/Downloads/incubator-mxnet/3rdparty/mshadow/mshadow/./base.h:173:12: fatal error: cuda.h: No such file or directory
   #include <cuda.h>
            ^~~~~~~~
compilation terminated.
Makefile:560: recipe for target 'build/src/operator/quantization/mkldnn/mkldnn_quantized_elemwise_add.o' failed
make: *** [build/src/operator/quantization/mkldnn/mkldnn_quantized_elemwise_add.o] Error 1

Minimum reproducible example
(If you are using your own code, please provide a short script that reproduces the error. Otherwise, please provide link to the existing example.)
Steps to reproduce
(Paste the commands you ran that produced the error.)

rm -rf build
mkdir -p build && cd build
cmake -GNinja
-DUSE_CUDA=ON
-DUSE_MKL_IF_AVAILABLE=OFF
-DCMAKE_CUDA_COMPILER_LAUNCHER=ccache
-DCMAKE_C_COMPILER_LAUNCHER=ccache
-DCMAKE_CXX_COMPILER_LAUNCHER=ccache
-DCMAKE_BUILD_TYPE=Release
..
ninja
cd ..
make

What have you tried to solve it?

Installed cuda 10.0, 9.2 and equivalent cudnn
Removed completely old cuda installation before installing new one.
Added two lines in to the ~/.profile file:

PATH=/usr/local/cuda-10.1/bin:/usr/local/cuda-10.1/NsightCompute-2019.1${PATH:+:${PATH}}
LD_LIBRARY_PATH=/usr/local/cuda-10.1/lib64${LD_LIBRARY_PATH:+:${LD_LIBRARY_PATH}}

Solved by using this:
make -j$(nproc) USE_OPENCV=1 USE_BLAS=openblas USE_CUDA=1 USE_CUDA_PATH=/usr/local/cuda USE_CUDNN=1 USE_MKLDNN=0


Nested groups in Azure AD are not supported within all scenarios.

Nested groups in Azure AD are not supported within all scenarios.

What does this mean? Which scenarios? Where can we expect it to fall down? Is this a bug? Is this by design?

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

ID: 42a0b4cb-1981-1cbc-ce2c-dbfd8af1527c
Version Independent ID: d49c873c-9843-8428-a6d1-a1995067d13b
Content: Service limits and restrictions - Azure Active Directory
Content Source: articles/active-directory/enterprise-users/directory-service-limits-restrictions.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @curtand
Microsoft Alias: curtand

@dougscc The same section describes the nested groups supported and unsupported scenarios. this is currently by design. As explained the article within the same section the limitations related to nested groups as as below. If you are testing a custom scenario that works without being listed here means that the scenario works but is not supported yet . For example , assigning licenses to nested groups is still not supported. That doesn't mean it won't work under any circumstance, but if you proceed, you should know that it's an unsupported configuration. Please find the list of supported/Unsupported scenarios.
At this time, the following scenarios are supported with nested groups: One group can be added as a member of another group, and you can achieve group nesting. Group membership claims. When an app is configured to receive group membership claims in the token, nested groups in which the signed-in user is a member are included.Conditional access (when a conditional access policy has a group scope).Restricting access to self-serve password reset.Restricting which users can do Azure AD Join and device registration.The following scenarios are not supported with nested groups: App role assignment, for both access and provisioning. Assigning groups to an app is supported, but any groups nested within the directly assigned group won't have access.Group-based licensing (assigning a license automatically to all members of a group).Microsoft 365 Groups. |

"It won't work, unless it does, so don't rely on it to work when it doesn't, unless it does, in that case refer to the list of situations where it absolutely won't work for a list of things that may work".
From which Charlie Stross novel was this answer lifted?

@dougscc Apologies for the comment . I have updated the comment to provide a better clarification . "For example , assigning licenses to nested groups works if you try however if there is an issue , Microsoft does not provide a guarantee to solve this issue as this is not supported configuration and we do not recommend it . The following are supported and unsupported scenarios." .
Hope this helps.


add .dockerignore to avoid COPY of node_modules

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.d061612 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix BootStrapToken API link

I found the typo in the latest issue.

kubeadm BootStrapToken API is changing

The link is wrong one.  This PR fixed the link with https://github.com/kubernetes/kubernetes/pull/102964.  I'm not sure it is the correct one tough.

@coderanger @jberkus  could you review this, please?

That's the right link, thanks for fix!


Creation of identical assets tag should continue without error

Currently, the generated tag for a commit to the assets rep is:
<TagPrefix>_<SHA>
Where SHA  is the commit sha of the assets repo for the new tag.
We are running into an issue where the proxy doesn't properly handle a situation where we've generated a new tag that already exists in the assets repo. This causes the migration script to set an empty tag within the assets.json.
@witemple-msft and the JS folks discovered this while implementing their migration tool.
Solution: if a tag we're attempting to push already exists, we don't have an error and can simply return the tag as-is.
Repro of this problem:

use the migration script against an existing package. It generates an assets.json, pushes new tag, and deletes recordings in place.
git reset --hard (still within the language repo that has the new assets.json + deleted recordings)
Run migration script again for same target. Assets is generated with no target tag.

Implementation suggestion:

I think we can pretty much do a git ls-remote origin ${generatedTagName} and see if it spits out the commit sha of the current branch.

Addressed in #5748


Development doc

@eguzki Comments/suggestions added.

Thank you very much @dfennessy for the review :medal_sports:

@3scale/apicast ready for review

https://github.com/orgs/3scale/teams/apicast ready for review


Fix displaying of status reporting to reflect the correct operations

Correctly reporting the operation type when printing summary instead of calling all operations "Put"

lgtm


Can CloudLAPS have a complexity requirement?

Hi
I am sure someone has run into this but we've only just run into this recently
Our intune policy has a local password complexity requirement that must include :
1 upper case letter
1 lower case letter
1 number
and a total of 8 characters ( could be more but lets say 8 as my memory is bad)
We had one device rotate their cloudlaps password and it didnt meet requirements due to having only letters ( pretty rare )
Is there a way to enforce cloudlaps password rotations to have the above complexity requirements?

You should move to Windows LAPS, which is a native solution in Entra ID / Intune. This solution is no longer being maintained since MS now has a native solution.
https://learn.microsoft.com/en-us/windows-server/identity/laps/laps-overview


Knife::SSH#run runs a Kernel.exit with non-zero exit code

Description
I'm utilizing Knife::SSH within some Ruby deployment code and when I use the #run method, it bails out of the entire code run because of this code block:
https://github.com/chef/chef/blob/a2a63b6ee82f7caabcda87cbcbb51e4fdc0fc9d9/lib/chef/knife/ssh.rb#L600-L605
To work around the problem, the command I pass to the instances targeted by Knife SSH looks like this:
command = 'sudo chef-client || true'

but that's problematic for a few reasons. It would be nice if this method threw an exception that I could capture and handle on my own. For instance:
if exit_status != 0
  raise Knife::SSH::NonZeroExitCodeError("instance abc123 failed with exit status #{exit_status}")
else
  # existing logic
end

Then, I could capture that exit code and do the retry logic, or whatever else I need. Having to capture a SystemExit event is pretty scary.
Chef Version
2.4.23
Platform Version
Docker image chef/chefdk:2.4.23
Replication Case
knife_ssh = Chef::Knife::Ssh.new
knife_ssh.name_args = ['machine1 machine2', 'eval "exit 2"'] # replace with two machines
knife_ssh.config = {}
puts "OK - running!"
knife_ssh.run
puts "I made it!" # you won't see this because the system exits

Client Output

OK - running!

Stacktrace

There are simple "fixes" that could be implemented here, but we haven't considered Knife to be a library in the past and there are endless issues like this if we did.
I argue this isn't a supported pattern and if you want to build scripting around knife ssh, it should be through the cli, e.g. use system('knife ssh') or Mixlib::ShellOut.


refactor: use predefined config rather than rpc config

refactor: use predefined config rather than rpc config

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@5680591). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #143   +/-   ##
==========================================
  Coverage           ?   47.24%
==========================================
  Files              ?       28
  Lines              ?     1344
  Branches           ?      196
==========================================
  Hits               ?      635
  Misses             ?      525
  Partials           ?      184

Continue to review full report at Codecov.

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


Touch Event Listener attached even if not needed

Hi All,
I have used following options:
touchMove: false,
swipeToSlide: false,
draggable: false,
swipe: false,

After these options, I still see in chrome devtools that touch event listener are attached. Is there something I am missing or it wasn't considered?

The referenced commit should resolve the issue.
Changes will be released soon.

Thanks @laveesingh for fixing it.


Add some tests

The project needs a set of tests

can you share which type of test and path.

you can write unit-test if you want.


build: make some linkage flags public

Some of the compiler and linker flags need to be used by dependent libraries
which requires making the flags public so that users of the libraries and the
export targets are setup properly.

@swift-ci please test

@ktopley-apple - okay to merge?


Python Pydantic improvements

Reason/Context
Field.alias
It seems that for Pydantic Field typing use of property.description as an alias is incorrect, and property.name or some other field should be used instead. Current implementation:
const PYTHON_PYDANTIC_CLASS_PRESET: ClassPresetType<PythonOptions> = {
  property(params) {
    const type = params.property.required
      ? params.property.property.type
      : `Optional[${params.property.property.type}]`;
    const alias = params.property.property.originalInput['description']
      ? `alias='${params.property.property.originalInput['description']}'`
      : '';

    return `${params.property.propertyName}: ${type} = Field(${alias})`;
  }
};

Possible fix:
    const alias = params.property.property.originalInput['name']
      ? `alias='${params.property.property.originalInput['name']}'`
      : '';

Field.description
Introduce support for setting a Field's description from property.description
Extra.forbid
For models declaring additionalProperties as false enable native validation:
from pydantic import BaseModel, Extra

class Model(BaseModel, extra=Extra.forbid):
    a: str

Description

Change from description to name when setting Field.alias will be a breaking change, but it's more of a misuse or a bug in the current state.

I might be wrong, but is it the .originalPropertyName you are looking for here? 🧐

There is another problem here: if the description used for the alias is a multi-line string containing single quotes, it will result in generating broken code:
class Model(BaseModel):
    a: str = Field(alias='Single-line description is fine')
    b: str = Field(alias='Unless it contains ' in it - that breaks the code')  # unescaped ' in the string
    c: str = Field(alias='It is even worse
    for a mult-line strings
    here ' breaks it also
    ')

I believe the fix will be to use triple quotes for the alias=... value generated, that way no escaping is needed, and multi-line strings are also fine and work better with syntax-highlighting in IDEs:
class Model(BaseModel):
    a: str = Field(alias='''Single-line description is fine''')
    b: str = Field(alias='''Contains ' in it - no longer breaks the code''')
    c: str = Field(alias='''It is better
    for a mult-line strings
    here ' or " do not break it
    ''')


Added documentation to infer

@alecgunny When you get a chance, could you give me some text to describe ChunkedSegmentLoader? I don't think I understand it well enough to do a complete job of it.

Yeah, that's awesome, thanks for writing it up

Actually, hold on, I'm gonna update the description of shifts based on what you said for datagen

Never mind, I think for the way shifts is used in this project, the description is fine as-is.


Don't mask errors when downstream parsing fails

If the client of this library mistakenly gets a NPE when parsing a status
update the exception is hidden in the logging. That makes debugging near
impossible.
So this is a 3-part fix.

Log the actual exceptions instead of the localizedErrorMessage.
Split apart the message parsing from the message handling. If a parse
error happens we'll close the connection. If the handler fails, just ignore
it and report the error.
Catch all exceptions in both phases - parsing and error handling and act
accordingly based on the phase you're in.

@jarlebh I took a bit of inspiration from your PR and came up with this.

@vitalidze can you take a look at this? I've been using this for a week+ and it's been working great. Would love to get this merged.
Also, not sure what's up w/ the build. Looks like the checkstyle plugin is expecting Java 1.8 to run. I imagine master is broken as I didn't change the pom.xml

addressed feedback

Merged. Thank you very much.

Right on!
Any chance you want to publish a release? It'd be nice to get off SNAPSHOT(s).

I have already published a v0.10.1 today with all these changes.
вс, 2 июля 2017 г. в 19:02, Jason Holmes<EMAIL_ADDRESS>
Right on!
Any chance you want to publish a release? It'd be nice to get off
SNAPSHOT(s).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/vitalidze/chromecast-java-api-v2/pull/81#issuecomment-312500361,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACMCNYpjzdMk6NTXut_ZRLSosqig9q8eks5sJ78UgaJpZM4OGFYe
.

--
Kind regards, Vitaly Litvak


Pr/fix extension for plain text json html content type

Hey there, with the recent death of JsonView (one of my favorite Chrome Extensions) I decided to take a look at yours, which to my disappointment didn't work. Later it did, and digging into it, looks like it wasn't handling the case where there was a plain text JSON response with a ContentType of text/html.
Made a quick fix, check it out!

Just confirming this works for me, thanks for the PR. Hope it gets merged.

+1 please merge this

Doesn't look like this is being merged, but I fixed the same issue in my fork: https://github.com/nikrolls/json-formatter


[PAXURL-326] Removed unneeded shaded dependencies: 5% size reduction.

Remove also the nekohtml exclusion because it is superfluous.

as in #18 I removed the shading process. the shorter directories alone had a minimal effect on jar size, but I consider consistency with the other projects and especially debuggability (which is increased when my IDE can find the class because it is no longer shaded) as the major driver.

Hi @CodingFabian, the shading process wasn't initially there in assembly I just initially added it in the first commit of #18 and then again removed it because it was unnecessary. The driving factor to use shading in url-aether was to avoid conflicts with versions of maven and aether when using url-aether in a non-osgi context. See https://ops4j1.jira.com/browse/PAXURL-251.
@ANierbeck what is your opinion?

The aether bundle is a special one, as it can be used outside the OSGi context, the shaded one is needed afaik, to make sure maven and aether don't get into a conflict.
But @hwellmann knows best, he implemented it for a reason :-)

I've reviewed the shade configuration for pax-url-aether with #417 here: 52dc4fa5179d71171a9d7dc7245a2bc894a573f8


Add env-expansion option to grafana agent and options to force or skip agent config overrides

@arul-opsverse , adding the following info section in docs to detail its usage:

We're also now passing in -config.expand-env to the grafana-agent startup in case customers push out their own agent configs which have environment variable references that need interpolation.
Testing
On test instance with already existing installation:
$ sudo ./installer.sh -- -m <redacted> -l <redacted> -p <redacted>
Verifying archive integrity...  100%   All good.
Uncompressing OpsVerse Agent Installer  100%
An agent config at /etc/opsverse/agent-config.yaml already exists...

There is already an existing agent config at /etc/opsverse/agent-config.yaml.
Please select one of the following options:
 (o) - to (o)verride it with the defaults
 (e) - to use the (e)xisting config
 (v) - to (v)iew the diff
Enter option:

Passing  --no-config-override to not prompt and use existing:
$ sudo ./installer.sh -- -m <redacted> -l <redacted> -p <redacted> --no-config-override
Verifying archive integrity...  100%   All good.
Uncompressing OpsVerse Agent Installer  100%
An agent config at /etc/opsverse/agent-config.yaml already exists...
--no-config-override option passed, so we'll use the existing /etc/opsverse/agent-config.yaml ...
Removed /etc/systemd/system/multi-user.target.wants/opsverse-agent.service.
Backing up existing service (/etc/systemd/system/opsverse-agent.service) file to /tmp
Removed /etc/systemd/system/multi-user.target.wants/node_exporter.service.
Backing up existing service (/etc/systemd/system/node_exporter.service) file to /tmp
Created symlink /etc/systemd/system/multi-user.target.wants/opsverse-agent.service → /etc/systemd/system/opsverse-agent.service.
Created symlink /etc/systemd/system/multi-user.target.wants/node_exporter.service → /etc/systemd/system/node_exporter.service.

Passing -f to not prompt and force override:
$ sudo ./installer.sh -- -m <redacted> -l <redacted> -p <redacted>  -f
Verifying archive integrity...  100%   All good.
Uncompressing OpsVerse Agent Installer  100%
An agent config at /etc/opsverse/agent-config.yaml already exists...
--force-config-override option passed, so we'll use  a default agent config file...
Removed /etc/systemd/system/multi-user.target.wants/opsverse-agent.service.
Backing up existing service (/etc/systemd/system/opsverse-agent.service) file to /tmp
Removed /etc/systemd/system/multi-user.target.wants/node_exporter.service.
Backing up existing service (/etc/systemd/system/node_exporter.service) file to /tmp
Created symlink /etc/systemd/system/multi-user.target.wants/opsverse-agent.service → /etc/systemd/system/opsverse-agent.service.

For now, merging... if any updates needed, we can do it in subsequent commits.


Select dropdown has transparent background in VSCode Settings

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

VS Code Version: 1.75.0 Insiders
OS Version: Win 10

Steps to Reproduce:
Initially discovered in https://github.com/microsoft/vscode/issues/166265#issuecomment-1398024284
The VSCode Settings show select dropdown's having seemingly transparent background, so the options' texts are mixing with the document flow beneath the dropdown. It is probably a CSS regression in 1.75.0 Insiders https://github.com/microsoft/vscode/issues/166265#issuecomment-1398111460 because 1.74.3 doesn't exhibit this problem

See already #171774