the Component will unmount method

Hello ,
I initiated this package in index.js as per given steps.
I testes this.test() in app.js file , the output we are getting in error object is right , but when i created component folder added one button.js inside component .
I used this.test() inside button.js file , not getting proper error object , it's always saying "Component will unmount method " and is fatal getting undefined .
Can you please help me ?

I am not really sure if I get this right. Can you please give a link to a sample repo with the issue ?

Hey master-(atul),
Thanks for your quick response.
Could you please let me know how I can capture error from nested components?
E.g. App -> Header -> Navigation
If I have above hierarchy of component and there is an reference error in Navigation component. Then how library will capture that error?

"Uncaught Errors" are bubbled up. So setting an error handler on the root should catch them even if errors come from sub components.
But if you are catching these errors by means of a try catch or componentDidCatch etc, then those are no longer "Uncaught errors" and wont be picked up by the global error handler.
This is the expected behaviour of the module.
I am not really sure if that is what you were expecting?

Thank you so much for your help ..
I am sharing my sample to you ..
I have initialized react-native-exception-handler package in index.js , In home.js i used {this.test()} function...
For login you can used any id or password like -<EMAIL_ADDRESS>& 1234 password.
https://github.com/rahuljain88/React-Exception-Handling-Test
Thanks again

i have the same problem, how to solve it? can't got the right error and have the red error background

I saw this as well - and I believe this is due to warnings becoming red box errors. Not sure yet how to solve it.


Have the router behave relative to URL the root app is mounted from

What problem does this feature solve?
Right now, if you don't specify a base option, the router will overwrite whatever URL you're on.
Example: 'www.foo.com/bar/baz' is where the app is mounted, the router's '/' path goes to 'www.foo.com', instead of 'www.foo.com/bar/baz'.
This is problematic for when an app (or component) should be reusable on different pages (e.g. /foo/bar and /foo/baz).
A workaround is to supply the basepath in some way (like through a data attribute), but this is far from elegant:
const router = new VueRouter({
  mode: 'history',
  base: document.querySelector('#app').getAttribute('data-path') || '/',
  routes
})

What does the proposed API look like?
Add a relative Boolean option in the router config (or something similar).
const router = new VueRouter({
  mode: 'history',
  relative: true,
  routes
})

Wouldn't this be enough?
const router = new VueRouter({
  mode: 'history',
  base: window.location.pathName,
  routes
})

Also, this would be problematic with hinstory mode, because the app could be started from / just as well as /some/other/route, so making the base dependent on the current path will break your app everytime that it's booting from a subroute.

@LinusBorg Unfortunately that would indeed result in problems when started from a sub route, that's why it would be great if the router had something built-in for this.
Isn't the Vue router instance aware of it's root path it got created in?
The server should be handling the redirect to the root, the router should behave relatively to that i my opinion, but maybe I'm over simplifying this?

Unfortunately that would indeed result in problems when started from a sub route, that's why it would be great if the router had something built-in for this.

...but a built-in functionality would suffer from the same problem:

Isn't the Vue router instance aware of it's root path it got created in?

No. From the router's perspective, the "root path it got created in" is. e.g. /yourRoot/users/TheDutchCoder.
The fact that your nginx frontend served index.html for that route is happening without the client being able to tell.

Gotcha.
Well I guess it's just the hacky way then ;)

I'm facing the same issue. I serve 2 apps on different paths:
/panel and /.
The problem is that when serving the admin panel app, history mode replaces whe whole URL to:
/overview and /account instead of replacing it to /panel/overview and /panel/account.
How can I hack it?

Please use forum.vuejs.org or chat.vuejs.org for questions.


qt5: inherit from __splicedPackages to fix cross

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.05 Release Notes (or backporting 22.11 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.

The following overlay now causes an infinite recursion
  plasma5Packages = prev.plasma5Packages.overrideScope (final2: prev2: {
    okular = prev2.okular.override { withSpeech = false; };

    plasma-workspace = prev2.plasma-workspace.overrideAttrs (oldAttrs: {
      buildInputs = oldAttrs.buildInputs or [ ] ++ (with prev2; [ qtimageformats ]);
    });
  });

I only changed overrideScope' to overrideScope after seeing the warning and thought that is unexpected.

plasma5Packages is a different set and different file and it doesn't use makeScopeWithSplicing so the overrideScope you're using is the deprecated overrideScope which has the arguments flipped and thats why you're getting the infinite recursion

plasma5Packages is more or less just an alias for libsForQt5.
  # plasma5Packages maps to the Qt5 packages set that is used to build the plasma5 desktop
  plasma5Packages = libsForQt5;

I don't get the warning right now anymore but I forgot which shenanigans I did to get it.

@Artturin this change may cause murmur to rebuild QT5 unnecessarily again if set to environment.noXlibs = true;?

@Artturin this change may cause murmur to rebuild QT5 unnecessarily again if set to environment.noXlibs = true;?

it shouldn't


add rule to detect Unnecessary Parentheses

Resolves #270
Uses KtPsiUtil to figure out if parentheses are useless, which is the same that the IntelliJ warning and quick-fix use.
But it doesn't detect the case described in #270 itemView.setOnClickListener({ it.doSomething() })
For this we check arguments to see if only a single lambda is provided (and it's not a supertype constructor because otherwise this reports Spek classes as well...)

Looks good, have you tried running the rule on some other projects than detekt?

@arturbosch Just ran the rule on another project as well and the findings all look correct to me.

AppVeyor failed because of not finding Kotlin.Unit :/


update external elasticsearch image version to 7.9.3

Signed-off-by: Jeeva Kandasamy<EMAIL_ADDRESS>

Thanks Jeeva, but I think we should leave on 6.8.6 for now.


Doubt related to CSS Class to style an element.

Browser Information

Browser Name, Version: Google chrome
Operating System: Windows 10
Mobile, Desktop, or Tablet: laptop

Your Code

CatPhotoApp

This code is not working in my system.

@mannya89
Thanks for reporting this. But your code is incorrect.
GitHub Issues are for reporting bugs on the website only.
Please use the Help chat room for getting challenge related help.
Happy Coding.

Hi
I am student who is learning HTML for the first time . All I want to ask
you is I just copied the code given on the study material. In such a
situation to whom I should ask. So it means the code given on the website
is also wrong. Anyways thanks for ur reply.
On Monday, September 19, 2016, Lance<EMAIL_ADDRESS>wrote:

Closed #10813 https://github.com/FreeCodeCamp/FreeCodeCamp/issues/10813.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/10813#event-794583209,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVRV8VLE7Zsyoa0GbreqntT1b3efwrxiks5qruSQgaJpZM4KA3-7
.

Im stuck here.can someone help me?

@hardeesarmiento, Please use the Help chat room for getting challenge related help.
GitHub Issues are for reporting bugs on the website only.
Happy Coding.


Allow version 1.3.1 to work in strict mode

Hi guys - I work with @far-blue at Namesco. We are currently using the old-ish 1.3.1 version of NestedTypes and recently needed to get said version to run in strict mode. So, here's a PR that covers the changes to do that. Note that tests that previously passed still pass (no new fails).
Whether you want to do anything with this PR I'm not sure, given the age of the codebase 😄

Hi. I think I gonna merge this PR and release the update for 1.3. Thanks for the fixes.


Updated for 0.18.0 support

Don't think this belongs on master yet since 0.18.0 isn't GA but these updates, without any code changes, are working in my edge build.

Published as 2.0.1, I have no problem with master being ahead of current.


feature_names mismatch on sparse matrices

Hi,
I'm have some problems with CSR sparse matrices. I train the model on dataset created by sklearn TfidfVectorizer, then use the same vectorizer to transform test dataset.
During prediction following error occurs:
<ipython-input-48-bb14ec4ec14c> in <module>()
      2
      3 for cat, classifier in tqdm(classifiers.items()):
----> 4     pred[cat] = classifier.predict_proba(words)

/home/jkarwowski/anaconda3/lib/python3.5/site-packages/xgboost-0.6-py3.5.egg/xgboost/sklearn.py in predict_proba(self, data, output_margin, ntree_limit)
    475         class_probs = self.booster().predict(test_dmatrix,
    476                                              output_margin=output_margin,
--> 477                                              ntree_limit=ntree_limit)
    478         if self.objective == "multi:softprob":
    479             return class_probs

/home/jkarwowski/anaconda3/lib/python3.5/site-packages/xgboost-0.6-py3.5.egg/xgboost/core.py in predict(self, data, output_margin, ntree_limit, pred_leaf)
    937             option_mask |= 0x02
    938
--> 939         self._validate_features(data)
    940
    941         length = ctypes.c_ulong()

/home/jkarwowski/anaconda3/lib/python3.5/site-packages/xgboost-0.6-py3.5.egg/xgboost/core.py in _validate_features(self, data)
   1177
   1178                 raise ValueError(msg.format(self.feature_names,
-> 1179                                             data.feature_names))
   1180
   1181     def get_split_value_histogram(self, feature, fmap='', bins=None, as_pandas=True):

ValueError: feature_names mismatch: ['f0', 'f1', 'f2', 'f3', 'f4', 'f5', 'f6', 'f7', 'f8', 'f9', 'f10', 'f11', 'f12', ...
f38732', 'f38733', 'f38734', 'f38735', 'f38736', 'f38737', 'f38738', 'f38739'] []
expected f4057, f36350, f1683, f1914, f33121, f16637, f21443, f10995, f36221, f24340, f15968, f7863, f38732, ...
f19897, f33500, f37792, f30259, f20094, f27943, f5788, f14369, f9074 in input data

The dimensions are the same in training and prediction time, the second list appears to be permutation of the first (... symbol is mine, log is very long).
It may be related to #1238

You need transform sparse matrix to array, like this:
    xg_train = xgb.DMatrix(X_train.toarray(), label=y_train)
    xg_test = xgb.DMatrix(X_test.toarray())

@SpLin12 No, that would be ridiculous. Converting a sparse array to be dense is not an intended fix nor is it possible in the majority of sparse feature spaces.

It's definitely #1238, code works fine with csc matrices (although the performance is ~20% worse)

passing a dataframe solved the issue for me

@nazirmubbashir could you clarify?

@jpbm I used pandas dataframes and passed them directly, without converting to arrays.

Although using n_features, load_svmlight_file gives a sparse matrix with the right shape,
XGBClassifier.predict seems not recognize it.


3.0 translation update

[ x] created new branch from release-3.0-translations
[ x] rebased branch on release-3.0
[ x] updated pot/po files
[ ] hand over updated edge.pot to localization
[ ] receive updated updated Chinese and Japanese .po files
[ ] updated translated XML
[ ] merge into release-3.0-translations

Closing, do not nmerge, I messed this one up.


Add self playing CFR+ agent and historian

feat: counter factual regregret minimization historian
This PR adds a historian for a single agent that follows along creating a CFR tree for the current game path. This needs to be expanded but should serve as a starting place.

The current status is that this is making progress, but I need to see why it fails when the state is changed during exploration.


[bug] SteamDB.info blank page

Whenever SteamDB asks me to verify that I'm human it returns a black page after checking the box
https://github.com/user-attachments/assets/04d6b3ad-6e51-456b-80eb-64fc05c5e790

+1 I have the same issue

New update of millennium has been out for a moment and seems to fully fix this. Closing this issue.


Issue with build production for angular 4

When I am trying for build angular-4 app its giving  Template error.
ERROR in Template parse errors:
Can't bind to 'options' since it isn't a known property of 'ks-swiper-container'.

If 'ks-swiper-container' is an Angular component and it has 'options' input, then verify that it is part of this module.
If 'ks-swiper-container' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.
To allow any property add 'NO_ERRORS_SCHEMA' to the<EMAIL_ADDRESS>of this component. ("

I was thinking to use this library on my Angular 4 project as well. Let me know if you find any fix

its for angularjs, not angular 2/4


RuntimeError: Cannot pause_reading() when closing

i think pause_reading should be no-op during closing or for closed transport.
for aiohttp case parsing incoming data is separated from transport and it is not possible to control closing state of transport in current architecture. also closing process is asynchronous, it is very possible to get this exception even in simple asyncio libs.
issue KeepSafe/aiohttp#1211

Sounds like a good idea to me.

@1st1 if you move this ticket to cpython, you can assign it to me, I will work on this issue next during next couple days


add boot to installer

Seems, we miss files in boot needed for RPi4 boot process
#2021
Signed-off-by: Petr Fedchenkov<EMAIL_ADDRESS>

Open question for me here: do we need u-boot.bin in /boot for amd64? https://github.com/lf-edge/eve/blob/1af1a093b18dd9ceae8eded16455ce8d053f6675/pkg/u-boot/Dockerfile#L32

Interestingly enough @giggsoff -- this wasn't an oversight -- this was done on purpose. I am still not convinced we should be in business of managing firmware when doing the netboot route. Otherwise /boot content is going to end up on all those Packet ARM servers -- that's not good. We really need to treat RPi4 as an exception at this point and make presence of those files conditional in iPXE
Makes sense?

We can use this when booting on LAN, however without TFTP on LAN, Rpi4 won't be able to boot after installation, is this the right approach?
And when we make a live-raw image for ARM architecture, these files are present in EFI partition.

Live image is for RPi only @Insei -- so that's fine. I don't think I understood your other comment, tho.

Live image is for RPi only @Insei -- so that's fine. I don't think I understood your other comment, tho.

After network installation on Rpi4, we get not bootable device, if bootable blobs has been removed from TFTP.


Hang on starting cgroups cleanup

I have cgroup cleanup feature enabled and openrc hangs during cleanup for considerable amount of time for every service. This happens with version 0.31.2. Not sure how can I provide additional information or help to debug this. Booting works perfectly fine, though.

Previous known working version was 0.29

In the newever versions of openrc both cgroups v1 and v2 are now implemented. Does your cgroups cleanup feature use cgroups v1 or v2? Make sure to configure cgroups in the openrc rc.conf file. By default it has hybrid turned on which is a mixture of v1 and v2. If you just want legacy like it was before v2 was added, change cgroups to legacy in rc.conf.

In the newer versions of OpenRC, you can also use the rc_timeout_stopsec variable to shorten the amount of time the cgroup_cleanup delays.

@williamh thanks. What is the purpose of sending cont signal after sending term? The amount of time it'll sleep is determined by the mentioned variable, but it seems that openrc will always sleep this long. Maybe some polling solution would be better to detect cases when processes have exited earlier?

experiecing the same problem on a gentoo install.

This should be fixed in 0.34.10.

Tested it today in a Gentoo install, still an issue with xdm.

I believe the xdm issue is caused by this commit:
https://github.com/OpenRC/openrc/commit/6a5ca2ab368d0a85f51bb559672dba2e3ffcc6be
Don't know if it's poor signal handling by one of the child processes.

@saboya Is it happening with any services other than xdm?

I'm not sure, it may be that xdm is hanging first and, since it never shuts down, OpenRC just kills everything after the timeout.
I have a particular setup with Gnome 3.26 without systemd (dantrell overlay), which uses elogind. Another user with the same setup, but a different version (3.22) doesn't seem to have any problems.
3.26 uses elogind while 3.22 still uses consolekit, so the issue has something to do with elogind, which is probably a child process of gdm, which itself is a child process of xdm.
Using rc_send_sighup="YES" solves the issue.

In that case, I'll go ahead and close this issue.


Add test cases for examples

Hi @jdkanani
When can you deliver test cases for examples?
They are essential in practice.

would be really helpful to have some test examples.
i have no clue how to unit test something like receiving a bytes message from a child contract.
I'd be happy to mock out sender validation, but my workflow right now seems to require testnet deployment and manual testing


Add Workflow to generate GraphQL introspection json

What type of PR is this?
documentation
Which issue does this PR fix:
Part of #606
What does this PR do / Why do we need it:
This PR adds a few new steps that only run on linux-amd64. They start a zot instance and do a GraphQL introspection query, saves the result to a json and uploads it as an artifact.
I added a small comment in the GraphQL schema for testing purposes. For speed, I modified the workflow to only build the binary and not run tests. This will be reverted when the PR is ready to merge.
Example result, at the bottom of the page: https://github.com/project-zot/zot/actions/runs/3106672044
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@andaaron Is it the same PR you mentioned during our discussion that will update documents as per schema

This is the one I meant.


Paths from the WSL filesystem (bash) do not open on clicks

VSCode Version: 1.10.2
OS Version: Windows 10

Ideally, when clicking on files from WSL bash output, they should open in the editor. This is a basic functionality of IDEs for any language.
Steps to Reproduce:

Run bash as integrated terminal "terminal.integrated.shell.windows": "C:\\Windows\\sysnative\\bash.exe",.

Run make on a Makefile-based project that doesn't compile.

All compile errors will be printed as /mnt/c/Users/... which VSCode doesn't seem to understand.

Any solutions or workarounds for this? So close to WSL being a viable option for development within VSCode but this is really a killer.

For now links only work when you're connected the VS Code using the Remote WSL extension.

@dddejan
@benjamind
@dskachan
@adam-p
@benjamind
@n0v1
@acristu
@lsoica
@livius-ungureanu
@deroude
@xeurun
@noclat
@mehdi-benallegue
We need 20 upvotes in 60 days to put this feature request into backlog.
Please upvote this issue: https://github.com/microsoft/vscode/issues/148062#issue-1214779841
Explanation about why 20 upvotes are needed are in this comment: https://github.com/microsoft/vscode/issues/148062#issuecomment-1116429405


TOPK operator for RKNN export

Issue Type
Support
OS
Ubuntu
OS architecture
x86_64, aarch64
Programming Language
Python
Framework
ONNX
Model name and Weights/Checkpoints URL
movenet multi
Description
Is there anyway to swap out the topK operator ?

I am trying to export to RKNN; however, this operator doesn't seem to be supported.
Relevant Log Output
No response
URL or source code for simple inference testing code
No response

I am not interested in posts that have nothing to do with this repository.


Clear a Bullet Area's overlappingObjects vector when removing an area from a space.

Fixes #39765.

Thanks!

Cherry-picked for 3.2.3.


Customized proxies seems not work, how can we know the proxies's IP address send to server?

Take this as an example:
#!/usr/bin/env python
# -*- coding: utf-8 -*-

import requests

proxies = {"http":"http://111.222.333.444:80"}
r = requests.get("https://www.google.com", proxies=proxies, timeout=5)
r.status_code
r.content

Obviously, the proxies "111.222.333.444" is not correct, but why I still got status_code 200??
How can we check the proxies we used to send to server?

The proxies dictionary is defined as having the following form: the keys are the URL schemes for the URLs you are requesting, and the values are the URLs for reaching the proxy.
That means when we make a web request, we take the scheme of the URL you're requesting and look for a proxy for it. In your case, the scheme is https (because you're looking for https://www.google.com).
If you change your requests.get to be http://www.google.com, then you'll find that we attempt to reach the proxy and error out.

Thanks, so rapid reply!!
Can you also reply the second question: can we retrieve the proxies send to server? some time, I got error, and I am pretty sure the schemes are same but still got an error, like use free proxies, some of them may out of date.

@withr Unfortunately, not easily. Generally speaking you should look at the proxies dictionary you provided. Alternatively, you can write a custom Transport Adapter that will provide the information you need, though that's quite a lot of work.


BUG: grep doesn't take only the 1st available value of version

As per title, consider to have a pubspec.yaml like this:
name: aName
version: 1.0.1

# configs...

dependencies:
  # some deps

dev_dependencies:
  # some deps
  version: ^1.0.0

This will result in an output of 1.0.1_^1.0.0 which is not a valid tag.

Fixed by #8.


Add authorized key parameter in virtualnetworkgatewayconnection APIs

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Hi @dihan0604, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (dihan). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@azuresdkci add to whitelist


Select method on client

Problem
Overfetching data from database.
Suggested solution
Develop a select method, who already exists on original prisma for javascript.

This is a duplicate of #19. There is also a WIP implementation of the second solution mentioned in that issue that will be finished soon.


Support replacements capture groups

Is your feature request related to a problem? Please describe.
Currently, Replacement only allows you to replace with a hardcoded string or with ReplaceData when processing replacements.  I'd like to use the shortest possible regex since I'm covering multiple lines of text to find what I'm looking for.
Specifically, I'm trying to update versions of dependencies in a helm chart so I can't just do something like version: .* as there will be multiple lines in the Chart.yaml that have that.  I need to get the specific list item and update the version so my regex would look something like this when updating a single dependency:
(- name: chartName.+?version): \d+\.\d+\.\d+

I'd like to use the capture group to put whatever string I found back in place.  This helps with not having to format each of those files very specifically and ensures no mistakes are made.  It also keeps us from having to do a bunch tedious formatting as well as making sure we don't have to follow a specific format in the future.
Describe the solution you'd like
Supporting regex capture groups
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
I've tested out some changes and can post a PR if you think the feature is worthwhile

Hmm this is largely left to a package I use I believe. If you could give me an example of the helm chart and regex I can see what I can do.

Sure....
# Chart.yaml

version: 1.0.0
name: common
description: Common chart to be shared

dependencies:
  - name: other
    version: 2.0.0
    repository: https://fake-registry.com/other
  - name: dependency
    repository: https://fake-registry.com/dependency
    condition: dependency.enabled
    version: 1.0.0

Regex to find what I'm looking for and replace a version number:
(- name: other.+?version): \d+\.\d+\.\d+

This would find the dependency chart other in the Chart.yaml for common and replace it's version with the one that was just released.  By using the capture group it can just put back the piece it found in the replacement when you do something like this in changie.yaml
# changie.yaml

projects:
  - key: other
    label: Other
    replacements:
      - path: other/Chart.yaml
        find: "^version: .*"
        replace: "version: {{ .VersionNoPrefix }}"
    - path: common/Chart.yaml
      find: "(- name: other.+?version): \\d+\\.\\d+\\.\\d+"
      replace: "{{ index .CaptureGroups 0 }}: {{ .VersionNoPrefix }}"
      flags: s

The thing I tested out was adding a CaptureGroups field to ReplaceData and moving some of the code around to read the file and run a regexp.FindAllStringSubmatch on the current replacement then add that to the new CaptureGroups field but if there's another/better way to do it I'm all for it.

Hey there @Magic-Mayo I did some testing and found I don't actually need a package for this as the regexp has one already. Go uses $N for index based replacements, or $KEY for key value based replacements. Both will be supported with the next update.
# changie.yaml

projects:
  - key: other
    label: Other
    replacements:
      - path: other/Chart.yaml
        find: "^version: .*"
        replace: "version: {{ .VersionNoPrefix }}"
      - path: common/Chart.yaml
        find: "(- name: other.+?version): \\d+\\.\\d+\\.\\d+"
        replace: "$1: {{ .VersionNoPrefix }}"
        flags: s

^ is what you will be able to do, should have this up and pushed this weekend.


Text StrikeThrough

Hello,
I could not find an example
how can I implement strike-through property for texts ?

Oh i searched on issues then i found it
https://github.com/scanny/python-pptx/issues/574


file env/splitted_ovcamo.yaml is missing

It was excluded in the .gitignore file.

@zhoustan
It is a project-specific config file, so you need to generate it by yourself as stated in https://github.com/lartpang/OVCamo?tab=readme-ov-file#prepare-dataset


Added support for user creation and deletion

PR Type
enhancement, tests

Description

Added support for creating and deleting users, reflected in the CHANGELOG.
Introduced new test data files for creating an admin user, a non-admin user with extensive properties, and deleting a user.

Changes walkthrough 📝
Relevant filesEnhancement

CHANGELOG.mdUpdate CHANGELOG for User Management Features

CHANGELOG.md
Added entry for support of user creation and deletion in the changelog.

+1/-0

Tests

create-admin-user.yamlAdd Test Data for Creating Admin User

src/tests/data/create-admin-user.yaml
Added a new test data file for creating an admin user with specified roles and state.

+5/-0

create-non-admin-user.yamlAdd Test Data for Creating Non-Admin User

src/tests/data/create-non-admin-user.yaml
Added a new test data file for creating a non-admin user with multiple roles and properties.

+16/-0

delete-user.yamlAdd Test Data for Deleting User

src/tests/data/delete-user.yaml
Added a new test data file for deleting a user with specified username and state.

+3/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

Task linked: DUPLO-17365 create users in duplocloud console

Where are the code changes?

Pushed now please verify


hyper: 3.2.0 -> 3.2.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/hyper/versions.
meta.description for hyper is: "A terminal built on web technologies"
meta.homepage for hyper is: "https://hyper.is/"
meta.changelog for hyper is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/n720sxzv8w18v1jdvyygww9kvw52qs0r-hyper-3.2.1/bin/hyper had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 3.2.1 with grep in /nix/store/n720sxzv8w18v1jdvyygww9kvw52qs0r-hyper-3.2.1
found 3.2.1 in filename of file in /nix/store/n720sxzv8w18v1jdvyygww9kvw52qs0r-hyper-3.2.1

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

1 total rebuild path(s)

1 package rebuild(s)

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

First fifty rebuilds by attrpath
hyper

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

hyper

Maintainer pings
cc @puffnfresh @FabianGD for testing.

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

1 package built:

hyper

Did a quick testing, works fine.


Support parallel processing in TrackingEventProcessor

Currently, the TrackingEventProcessor only supports a single thread to process events. Instead, it should be able to use a pool of threads to process events concurrently.
A single thread will always be used to read events from the stream. That thread will calculate the segmentID for each incoming event, based on their sequencing policy and a hash function. Two events with the same sequencing policy value will always be processed by the same thread.
The other threads will process their (portion of) the message queue, updating the token belonging to their segment.

With regards to #392 this would mean, the ExecutorService shall be initialised with a cached ThreadPool using the ThreadFactory that could be injected/overwritten via the newly created TrackingEventProcessorConfiguration?

No, I actually mean that the TrackingProcessor will use the ThreadFactory directly to ask for new threads when necessary. In JEE environments, this would be a ManagedThreadFactory (https://docs.oracle.com/javaee/7/api/javax/enterprise/concurrent/ManagedThreadFactory.html), so that the JEE container can manage the threads that are created this way.

@abuijze I think we're on the same page here. I just wanted to elaborate where the ThreadFactory (and probably a reasonable default) will be configured.

created a Pull Request with a draft to configure ThreadFactory - does this look reasonable?


[RFC] Add the Summary

Branched from #77.
It's a mixup between the project plan, preface and introduction.

Todo: add all contributions made by this project.


Outdated bashrc command for Linux ARM Toolchain

https://github.com/electro-smith/DaisyWiki/wiki/1d.-Installing-the-Toolchain-on-Linux#arm-toolchain
The code is written for the version 9:
GCC_PATH=~/Developer/gcc-arm-none-eabi-9-2020-q2-update/bin
export PATH=$GCC_PATH:$PATH

But the folder/path from the tar file should be:
GCC_PATH=~/Developer/gcc-arm-none-eabi-10-2020-q4-major/bin
export PATH=$GCC_PATH:$PATH

As it was recommended for version 10-2020-q4-major

You can ignore the "This page is deprecated" message at the top of the website. Please scroll down and we recommend that you click on "Downloads: 10-2020-q4-major December 11 2020" and install "gcc-arm-none-eabi-10-2020-q4-major-x86_64-linux.tar.bz2".

This led to a bit of confusion when I just copied and pasted the whole thing from the wiki.

Hi Z-QIAO,
Thank you so much for pointing that out.
I just edited the folder/path on the wiki to match the version we recommended.


setCustomerIds command: Updating warning message on failed hashing

Updating warning message when browser fails to hash a customer id to inform the customer that the id will not be sent to Exp Cloud.
Description
Updating warning message from
Unable to hash identity **NAMESPACE** due to lack of browser support to
Unable to hash identity **NAMESPACE** due to lack of browser support. Provided **NAMESPACE** will not be sent to Adobe Experience Cloud
Related Issue
https://jira.corp.adobe.com/browse/CORE-39142
Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (non-breaking change which does not add functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[ ] I have submitted a documentation pull request or no changes are needed.
[ ] I have made any necessary test changes and all tests pass.
[x] I have run the Sandbox successfully.

We need tests for when we can't hash due to browser support. It will require you to inject convertStringToSha256Buffer instead of importing it, then mocking convertStringToSha256Buffer to simulate no browser support.


bridge

I tried to bridge at super bridge but didn’t know the gas fee was eth not base eth now my $every token has been withdrawn from my MetaMask pls how can I get it back to my MetaMask

Please reach out to<EMAIL_ADDRESS>


Broken bazel test: //test/core/client_channel:lb_policies_test

https://source.cloud.google.com/results/invocations/4fcdc1cb-4e7d-4b65-80a1-50c62a821266/targets
https://source.cloud.google.com/results/invocations/4fcdc1cb-4e7d-4b65-80a1-50c62a821266/targets/%2F%2Ftest%2Fcore%2Fclient_channel:lb_policies_test@poller%3Depoll1/log

Friendly ping?

This test will probably be removed. It has been disabled for about a year: https://github.com/grpc/grpc/pull/14284


fix: Relates to #474. Fix so Transaction Details shows missing inputs instead of incorrect values

Replaced by https://github.com/paritytech/fether/pull/476


Add assert() that throws an Error

Clojure's spec has s/assert that validates and then throws an Exception at runtime with explain's output. I had to write something similar:
export function validateSpec(data, aSpec) {
        if (!spec.valid(data, aSpec)) {
            throw new Error(spec.explain(data, aSpec));
        }
}

I guess that's reasonable.

Done with #16


add package: timezone + spelling fix for crone package

added package: timezone
fixed name of package that I misspelled: "cron" insted of "crone" from this pr

I’d split it into two PRs


C++ Clang Linter: 'iostream' file not found

When linting with ALE, I get 'iostream' file not found on #include <iostream>, but clang++ <file>.cpp works just fine.
:ALEInfo
 Current Filetype: cpp
Available Linters: ['clang', 'clangtidy', 'cppcheck', 'g++']
  Enabled Linters: ['clang', 'clangtidy', 'cppcheck', 'g++']

Linted file
#include <iostream>

using namespace std;

int main() {
  cout << "hello" << endl;
  return 0;
}

The linter probably needs to be informed where the headers are. See #324.

Invoking clang++ -S -x c++ -fsyntax-only <file>.cpp seems to work flawlessly though, as per the linter's command. Shouldn't system headers like iostream be included regardless of whether or not clang++ is invoked from Vim?

All my header file not found even though the .h file and .c file are in the same folder.

Yes, someone needs to implement this.

I have now pushed a commit which should support header files in the same directory by using -iquote with the directory path.

@metakirby5 Do you still see this issue? See if you can see which of the available linters reports the problem if you do.

@w0rp Looks like it works now! On some occasions Vim hangs for a bit when opening a file though, possibly because of clang trying to sift through recursively included headers.

Is editing blocked at all when "Vim hangs?"

Yes, none of the text renders until it finishes :(

Could you profile Vim when this happens and share the results? See the steps for doing this here: https://stackoverflow.com/a/12216578
If ALE is the cause of this, then I can fix that.


Login, password input isn't sliding down.

picture 4 proof.
input just slides down, when clicked at. || password label only slides up, when the input is click at.

I think this is browser bug. Filed a Chrome bug about it: http://crbug.com/544537


Feature: Implementing contribute screen on mobile app

Description

This feature is dedicated for implementing the contribute screen in the mobile app.
Tasks

[x]  Getting contributions.
[x]  Displaying contributions.
[x]  Filtering contributions.
[ ]  Optimizing ux.

Type of change

New feature (non-breaking change which adds functionality)

I'd also want to have some empty state svgs or lottie files if there is any to apply them when no contributions are not found.

@ZibanPirate I need your opinion on this one, should the filters be updated when checking one of them after getting the result? Like what happens in the website ? or should I not updated them and leave them as the initial state or when pulling to refresh the contributions?

@ZibanPirate I need your opinion on this one, should the filters be updated when checking one of them after getting the result? Like what happens in the website ? or should I not updated them and leave them as the initial state or when pulling to refresh the contributions?

good question, we should follow the website where:

filter options are dictated by the api response.
the state of each filter option however (check or not) should be preserved if the filter option is still there.
when clicking on a filter option, it should add to the already selected filter options, so we would be able to select multiple options.
when clicking on a Chip (label's on the card) however, we should reset the filters states and select only that option.

this may be challenging with the useState, but it will be piece of cake when using redux, so i suggest we address this as followup PR after we implement redux.

@ZibanPirate I need your opinion on this one, should the filters be updated when checking one of them after getting the result? Like what happens in the website ? or should I not updated them and leave them as the initial state or when pulling to refresh the contributions?

good question, we should follow the website where:

filter options are dictated by the api response.
the state of each filter option however (check or not) should be preserved if the filter option is still there.
when clicking on a filter option, it should add to the already selected filter options, so we would be able to select multiple options.
when clicking on a Chip (label's on the card) however, we should reset the filters states and select only that option.

this may be challenging with the useState, but it will be piece of cake when using redux, so i suggest we address this as followup PR after we implement redux.

You can say that I already implemented the half of those points. in each filters state change I am getting again the response from the api. when clicking the chip the filters are reset, but when checking a filter the response will change the filters as well, as the case for the web. and when pull to refresh it will reset all filters to the initial state, so that way it would give the initial result if the user want it.
There is something else @ZibanPirate  and that's the empty state, should we do some custom image or lottie file or what?

@ZibanPirate I need your opinion on this one, should the filters be updated when checking one of them after getting the result? Like what happens in the website ? or should I not updated them and leave them as the initial state or when pulling to refresh the contributions?

good question, we should follow the website where:

filter options are dictated by the api response.
the state of each filter option however (check or not) should be preserved if the filter option is still there.
when clicking on a filter option, it should add to the already selected filter options, so we would be able to select multiple options.
when clicking on a Chip (label's on the card) however, we should reset the filters states and select only that option.

this may be challenging with the useState, but it will be piece of cake when using redux, so i suggest we address this as followup PR after we implement redux.

I will do these changes on this PR


Component is missing template or render function

I'm using those deps
  "dependencies": {
    "bootstrap": "5.2.0",
    "pinia": "2.0.16",
    "vue": "3.2.37",
    "vue-router": "4.1.2",
    "vue3-google-login": "2.0.11"
  },

and typescript 4.7.4
When I've put Sign in with Google on App.vue - all cool.

It works whenever I remove import
import GoogleLogin from 'vue3-google-login';
Not sure yet why

@szwalkowski Sorry for the late reply, if you are initializing the plugin in main.js like this, you don't need to import GoogleLogin component seperately it will be available globally, if you don't intend to initialize it globally inside main.js, you can import it seperately like this

<template>

Login

  <GoogleLogin client-id="570008781991-5ghaikfc73jqqm8g5mohb295vcpqlmri.apps.googleusercontent.com" :callback="callback" />
</template>

Forgot to close :) But yeah, thank you for the response! :)


App crashing at splash screen on Huawei phones with HiSilicon Kirin chip

Using Kivy 1.9.1
I have an app on the play store, and have received 3 negative reviews recently from Huawei device users stating that the app does not start. Exits directly from splash screen.
Here are the three devices according to developer console:

Device: Huawei GT3 (HWNMO-H) CPU make: HiSilicon CPU model: KIRIN650
Device: Huawei Mate S (HWCRR) CPU make: HiSilicon CPU model: KIRIN930
Device: Huawei X2 (HWGemini) CPU make: HiSilicon CPU model: KIRIN930

All other factors (RAM, Android OS, Architecture, Language, GLES) seem as if they can be eliminated, based on successful usage by other reviewers.
The crash is happening prior to any logs being written, so I wonder if there is a difference in how the storage directory is accessed?
I don't have any Huawei devices, and there are hardly any around in the US, so I was wondering if anyone had any insight into what is going on?

This seems particularly instructive, but not sure how it would be best implemented in kivy / pyjnius: http://stackoverflow.com/questions/39895579/fileprovider-error-onhuawei-devices

same problem seen

I've nominated this as a feature request because it seems these devices do not follow the Android contract, and so specially handling them would be a feature.
I suspect this won't be fixed unless someone can provide access to one of these phones. In particular, have Android patches fixed the problem?


Added one use documentation and fixed Python 3 portnetial issue

Second attempt at pull request to fix #475
This fixes a python 3 potential issue, and adds documentation to snapshot to explain it should only be used once as on re-use variables are not initialised.

Coverage decreased (-0.4%) to 60.302% when pulling ca71078e99c7a8ce72adfc4da5f485f33b47ed42 on DPH:snap_upd2 into 1fe61bdf10aa9538f3eca38a0940144d02104c33 on SoCo:master.

Thanks for writing this up. Just a few minor comments.

Coverage increased (+0.01%) to 60.761% when pulling 0b2ca3fe510e3616287e22eae0abaee2f9a191b5 on DPH:snap_upd2 into 1fe61bdf10aa9538f3eca38a0940144d02104c33 on SoCo:master.

Coverage increased (+0.01%) to 60.761% when pulling 0b2ca3fe510e3616287e22eae0abaee2f9a191b5 on DPH:snap_upd2 into 1fe61bdf10aa9538f3eca38a0940144d02104c33 on SoCo:master.

Coverage decreased (-0.4%) to 60.315% when pulling f44998e0c0889a9af7f07529ed7f6f61ddb43762 on DPH:snap_upd2 into 1fe61bdf10aa9538f3eca38a0940144d02104c33 on SoCo:master.

Updated as requested - I should be able to spell in English!!

We could add a flag that raised an error if anyone tried to re-use - or is this being over cautious!

IMO, raising a ValueError on invalid state is better than giving wrong results if the user disregards the documentation.

@DPH @ghcs27 I tend to think that is being over cautious. I don't we should block it, Python is typically referred to as the "consenting adults" language, without locked doors. Of course checking for inconsistent state while trying to write the snap shot back could be done, but would be a bit of extra work. I think we should let it become a problem before doing that work. In any case it should be a different PR, so I think I'll pull this one as it is.

Merged. @DPH can you add a comment about the change in the release notes issue #440.


correctly reflect inconsistent option name for libxar

#145 didn't have the desired effect in https://github.com/conda-forge/lld-feedstock/pull/34, hence I raised an issue upstream and realized that I had been caught out by falsely assuming consistency with other file format options.

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.


CI was broken with new py2exe version

CI temporally fixed: I'm just rollback py2exe version to 0.10.*
But this problem still waits for full resolving
And, may be, I should sent issue to py2exe repo

Turn off optimizations also fix problem

py2exe developer will include fix into next release..

Fixed #120 in <IP_ADDRESS>


Add Units to Siac

I believe all units in siac right now are base units, we should add support for S, KS, MS, GS to be recognized.

:+1:
Right now /wallet/send accepts floating-point notation (e.g. 1e27) but 1 SC is clearly more natural.
What if you want to send less than 1 SC though?

mS and uS


fix bug allelic imbalance

PR checklist

[ ] fix bug allelic imbalance
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I have removed the parameter that restricts the regions where allelic specific expression is checked. I think I should probably add a filter somewhere so that vcfs only contain canonical chromosomes


Add support for flipped mode

Apparently there is already support for flipped mode (still gotta test this).
For instance if you have a tileset like this

Then you should add a flipped mode like this:

So I tested this out in my test game
I took this...

Mirrored and flipped it to make this...

And what I found is that it places tiles in the wrong place now...
Take this example.

Notice how you can see the floor inbetween the wallpaper and the black wall top.
It should looks like so

If I turn the mirror setting off in LDtk this issue isn't seen.

If you're like me, your first thought would be that I've flipped things incorrectly. But if that were the case, it will offset mistake would be filled with white. Not transparency.


Process JCK git repo and branch parameters in JCK build script

https://github.com/AdoptOpenJDK/openjdk-tests/issues/1155
Signed-off-by<EMAIL_ADDRESS><EMAIL_ADDRESS>

Since this PR only affects the internal Grinder-systemtest build, the grinder testing it has been shared via slack to the reviewer.


Extensions Stucked Installing Status

Issue Type: Bug
Some extensions are stucked in installation and can´t install new Extensions
VS Code version: Code 1.36.1 (2213894ea0415ee8c85c5eea0d0ff81ecc191529, 2019-07-08T22:59:35.033Z)
OS version: Windows_NT x64 6.1.7601
Extensions (17)

Extension
Author (truncated)
Version

rainbow-brackets
2gu
0.0.6

vscode-html-css
ecm
0.2.0

php-debug
fel
1.13.0

php-intellisense
fel
2.3.10

php-pack
fel
1.0.2

vscode-javac
geo
0.2.18

python
ms-
2019.3.6558

csharp
ms-
1.20.0

java
red
0.46.0

bootstrap4-vscode
the
6.1.0

vscodeintellicode
Vis
1.1.7

vscode-java-debug
vsc
0.18.0

vscode-java-dependency
vsc
0.5.1

vscode-java-pack
vsc
0.7.1

vscode-java-test
vsc
0.18.1

vscode-maven
vsc
0.17.1

vscode-icons
vsc
8.8.0

What are the "some extensions"?

#78783 seems to be a DNS issue.
Can you try to access this image? https://ms-python.gallerycdn.vsassets.io/extensions/ms-python/python/2019.8.29288/1565127378610/Microsoft.VisualStudio.Services.Icons.Default
If not, can you try changing your DNS setting, like pointing to <IP_ADDRESS> as suggested here? https://<IP_ADDRESS>/dns/

If you are able to repro it, can you please try with previous release.
https://az764295.vo.msecnd.net/stable/2213894ea0415ee8c85c5eea0d0ff81ecc191529/VSCodeUserSetup-x64-1.36.1.exe
Would like to understand what caused this change.


When i am Hosting it, it gives an error

SQLSTATE[HY000] [1203] User epiz_21421395 already has more than 'max_user_connections' active connections
Its running fine in localhost
Is this an Server Fault or a Hosting Server Fault?

I have the same issue on MacOS, php 7. I cant fix the problem till now.
Do you have a solutions found?

not yet

Did you use "localhost" as host in the "config/class/needy_class.php" file? I replace it with <IP_ADDRESS>. No more error now. ;-)

not actually, i used my host as sql113.byetcluster.com in that line


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix connections staying longer than needed

Only happens through nginx for some reason.

It appears that the 30 sec timer is not closing the sockets like they should.

Fixed in 5f2729249e007e679c239c14b7986679e55ba9ef.


Add growl flags to straycat

I have changed the flag names however i think the tremolo growl flag might have to remain as either GR or MG due to "G" already being a flag.


test_expandtabs error on Python-2.6

Hi all,
this error comes out by performing future tests with Python-2.6:
======================================================================
ERROR: test_expandtabs (test_future.test_buffer.BytearrayPEP3137Test)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/python-future-0.15.0/tests/test_future/test_buffer.py", line 181, in test_expandtabs
    if sys.maxsize < (1 << 32) and struct.calcsize('P') == 4:
TypeError: Struct() argument 1 must be string, not unicode
----------------------------------------------------------------------
Ran 992 tests in 22.542s
FAILED (errors=1)

I'm looking for packaging future on Fedora and EPEL6/7 ( Red Hat Enterprise Linux (RHEL), CentOS and Scientific Linux (SL), Oracle Linux (OL)) repositories.
This is full log: https://copr-be.cloud.fedoraproject.org/results/sagitter/preprocess/epel-6-i386/future-0.15.0-1.fc22/build.log.gz
Thanks.

Strange, I have a build of future on EPEL 6 that seems to have run just fine.

Indeed, i have used your patch, tested future by using pytest and rebuilt successfully.
Thank you.


Bugs in project.c's setcontrol function

The setcontrol function in project.c, called by the EN_addcontrol and EN_setcontrol API functions, fails to assign the units-adjusted control setting value and the head-adjusted tank setting value to the simple control whose parameters it assigns to a Scontrol struct. More specifically, in line 1222 setting  should be replaced with s  and in line 1224 level  should be replaced with  lvl.

My bad -- the dev branch has no setcontrol function in project.c. I added the function to a local branch I am working on to address issue #730 and got confused when I found some bugs in it. I'm therefore closing this bug issue since it doesn't apply to any code currently residing in the project's repository.


Is there any plan to support pytorch 2.2.0?

Anything you want to discuss about vllm.
Is there any plan to support pytorch 2.2.0?

Please refer to https://github.com/vllm-project/vllm/pull/2804 .


Allow automatic selection of Powershell for Windows

When running in shell mode automatically select RunPowershell document when the target is Windows.  Print stderr response from the shell command.

Great idea! Thanks for this, I'll review as soon as possible (travelling right now).

So I seemed to have broken the Manager mock as well... and I can't see why. I am a terrible person and commented out the test just for now. When I get the chance I'll work out where/how it's broken.

Hey! I think it would be nice to get this feature merged, but the tests are currently failing and I was wondering whether you think you'll have time fix it?

I'll close the PR now since it's stale, and also the AWS CLI has session manager baked in.


Unsupported depth of input image

I just ran the example from:

https://keras-ocr.readthedocs.io/en/latest/examples/using_pretrained_models.html

with several png and jpg files and always got the following error:

(keraspypi) retrohelix@retrohelix-P64-HJ-HK1:~/.virtualenvs/test$ python bla.py
Looking for /home/retrohelix/.keras-ocr/craft_mlt_25k.h5
2020-01-04 16:40:15.855346: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2020-01-04 16:40:15.877424: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2020-01-04 16:40:15.878135: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x3dfe2c0 executing computations on platform Host. Devices:
2020-01-04 16:40:15.878149: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): Host, Default Version
WARNING:tensorflow:From /home/retrohelix/.virtualenvs/keraspypi/lib/python3.6/site-packages/tensorflow_core/python/keras/backend.py:5783: sparse_to_dense (from tensorflow.python.ops.sparse_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Create a tf.sparse.SparseTensor and use tf.sparse.to_dense instead.
Looking for /home/retrohelix/.keras-ocr/crnn_kurapan.h5
Traceback (most recent call last):
File "bla.py", line 12, in
predictions = pipeline.recognize(image=image)
File "/home/retrohelix/.virtualenvs/keraspypi/lib/python3.6/site-packages/keras_ocr/pipeline.py", line 49, in recognize
**recognition_kwargs)
File "/home/retrohelix/.virtualenvs/keraspypi/lib/python3.6/site-packages/keras_ocr/recognition.py", line 398, in recognize_from_boxes
[cv2.cvtColor(crop, cv2.COLOR_RGB2GRAY)[..., np.newaxis] for crop in crops])
File "/home/retrohelix/.virtualenvs/keraspypi/lib/python3.6/site-packages/keras_ocr/recognition.py", line 398, in
[cv2.cvtColor(crop, cv2.COLOR_RGB2GRAY)[..., np.newaxis] for crop in crops])
cv2.error: OpenCV(4.1.2) /io/opencv/modules/imgproc/src/color.simd_helpers.hpp:94: error: (-2:Unspecified error) in function 'cv::impl::{anonymous}::CvtHelper<VScn, VDcn, VDepth, sizePolicy>::CvtHelper(cv::InputArray, cv::OutputArray, int) [with VScn = cv::impl::{anonymous}::Set<3, 4>; VDcn = cv::impl::{anonymous}::Set<1>; VDepth = cv::impl::{anonymous}::Set<0, 2, 5>; cv::impl::{anonymous}::SizePolicy sizePolicy = (cv::impl::::SizePolicy)2u; cv::InputArray = const cv::_InputArray&; cv::OutputArray = const cv::_OutputArray&]'

Unsupported depth of input image:
'VDepth::contains(depth)'
where
'depth' is 4 (CV_32S)

Any idea what I can do about this?

Thanks for reporting this! I had made a mistake that caused keras_ocr.tools.warpBox to return images as int64 arrays instead of uint8 arrays as required by OpenCV. It's fixed as of 2acb106f0f33aa37a83fc159df04a9c68d14e712 which also adds a unit test to make sure that example works in the future. I also published a fixed version to PyPi so you can get it using pip install keras-ocr>=0.5.1.
Let me know if this resolves the problem. Thanks!

Bingo!
Thank you. I used pip install --upgrade keras-ocr to upgrade to the new version and tested again and at least the "unsupported depth error" is gone. But now I get his error:

Successfully installed keras-ocr-0.5.1
(keraspypi) retrohelix@retrohelix-P64-HJ-HK1:~/Programming/test$ python bla.py
Looking for /home/retrohelix/.keras-ocr/craft_mlt_25k.h5
2020-01-04 18:45:38.643560: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2020-01-04 18:45:38.675349: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2020-01-04 18:45:38.676041: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x4cf4c40 executing computations on platform Host. Devices:
2020-01-04 18:45:38.676086: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): Host, Default Version
WARNING:tensorflow:From /home/retrohelix/.virtualenvs/keraspypi/lib/python3.6/site-packages/tensorflow_core/python/keras/backend.py:5783: sparse_to_dense (from tensorflow.python.ops.sparse_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Create a tf.sparse.SparseTensor and use tf.sparse.to_dense instead.
Looking for /home/retrohelix/.keras-ocr/crnn_kurapan.h5
2020-01-04 18:45:46.304798: W tensorflow/core/grappler/optimizers/implementation_selector.cc:310] Skipping optimization due to error while loading function libraries: Invalid argument: Functions '__inference_standard_lstm_5013' and '__inference_standard_lstm_5013_specialized_for_lstm_10_back_StatefulPartitionedCall_at___inference_keras_scratch_graph_8029' both implement 'lstm_fa72b496-e092-49d4-9523-2f51f2f55567' but their signatures do not match.

Tried uninstalling and reinstalling the newest version with pip uninstall keras-ocr but to no avail. Same outcome.
Shall I close this issue and open a new one?

I think we should be able to resolve quickly so no need for a separate issue yet. All of these look like typical TensorFlow warnings that come up depending on your hardware and how the TensorFlow distribution was compiled. Are you sure an exception was raised and that the recognize function didn't return results? If there is, in fact, an exception, I can try to reproduce if you let me know what the output of the following is:
import tensorflow as tf
print(tf.__version__)

This package works best with TensorFlow version >= 2.0.0.

Ok. It somehow works with the warning (text gets detected and recognized) but there is no image output. I tested by just printing the text in the example.
# Plot the results.
fig, ax = plt.subplots()
ax.imshow(keras_ocr.tools.drawBoxes(image, predictions, boxes_format='predictions'))
for text, box in predictions:
    ax.annotate(s=text, xy=box[0], xytext=box[0] - 50, arrowprops={'arrowstyle': '->'})
    print(text)

Ah, yes -- the example was tested in a Jupyter notebook where the image would appear in the cell output. You may have to add plt.show() at the end to get the figure to appear if you're running from the command line.
As for the warnings, they are just letting you know that TensorFlow could run faster if certain optimizations were made during compilation. It's a little noisy sometimes but they will only appear when you first start the session.

Oh, plt.show() is missing :D

Thank you very much for your help. I close this.

FYI, I've added plt.show() to the example code so others don't hit the same bump.


Optimize IdP requests to database on login

This PR addresses Issue #1439
It includes two optimizations w.r.t. retrieving the identity providers from the database:

Whenever a login_hint is provided, we do not need to retrieve a full list of all identity providers, but it is sufficient to try to retrieve the identity provider with the given origin. If that does not work an error is returned anyways. So we can optimize the login method in a way, that we only retrieve a single idp, when we have a login_hint. Note that this includes the case, where we click on an account in the account_chooser page.

On the pages for the account_chooser and the page for entering the email for idp discovery, the list of all identity providers is retrieved but completely unused. By first determining the parameters and configuration valid for the current request, we can decide if we need to retrieve the identity providers or not.

As the login method is called several times for each login flow, you will save the unnecessary requests also possibly multiple times per login.
The optimized scenarios already were covered by Unit tests. The mocks for those were changed, to represent the optimized database requests. But the asserts stay the same, as the visible behavior does not change.
The visible behavior does not change with this PR - that is also why the Integration Tests do not change. We only optimize the requests to the database (=performance improvement).
If you want to, we can also have a session where we discuss these changes.

@aramprice  any concerns?

@strehle I'll defer to the UAA team on this. I've been out of the code for too long to offer useful opinions at the moment 😞

Hi @tack-sap
I undid your merge commit and replaced it with a rebase:
git reset --hard HEAD~
git rebase develop
git push --force-with-lease

I wasn't able to find your commits on the command line because of the merge commit. Could you please try to do rebases in the future? AFAIU merges should only be used when merging a feature/PR branch into the main branch (and even then I personally often like to rebase), and not the other way around.

The visible behavior does not change with this PR - that is also why the Integration Tests do not change. We only optimize the requests to the database (=performance improvement).

How would you feel about some sort of performance test? We could add a performance test section/directory in our integration tests if it doesn't exist yet. I would like to see a test that shows that indeed in practice this change is useful.

@bruce-ricard Ok, I will use rebase in future - I was just not aware of this rule.
Do you have any preferences on the format of such a performance test? Should I add it here to the PR or are you just interested in having such a test to check manually?
I created a rather simple MockMvcTest today (using StopWatch and looping 1000 times over retrieving the login page with 10 Idps) - with that I see that the new implementation is constantly a few seconds faster compared to the same test executed against the develop branch (on my system: around 8s vs. constantly over 11,5s) - as it saves 10 unnecessary database reads per request. (with a growing number of idps the new version stays at the same time, while the time of the current coding keeps growing with that number).
That's only a few ms per request, but as every browser login does that, is adds up to a big amount (especially retrieving all identity providers over and over again add up to such an amount of DB requests, that the statements showed up in our monitoring as one of the most resource-consuming statements, although they are usually answered very quickly).
Should I add such a test to this PR or do you have something else in mind?

@bruce-ricard Ok, I will use rebase in future - I was just not aware of this rule.
Do you have any preferences on the format of such a performance test? Should I add it here to the PR or are you just interested in having such a test to check manually?
I created a rather simple MockMvcTest today (using StopWatch and looping 1000 times over retrieving the login page with 10 Idps) - with that I see that the new implementation is constantly a few seconds faster compared to the same test executed against the develop branch (on my system: around 8s vs. constantly over 11,5s) - as it saves 10 unnecessary database reads per request. (with a growing number of idps the new version stays at the same time, while the time of the current coding keeps growing with that number).
That's only a few ms per request, but as every browser login does that, is adds up to a big amount (especially retrieving all identity providers over and over again add up to such an amount of DB requests, that the statements showed up in our monitoring as one of the most resource-consuming statements, although they are usually answered very quickly).
Should I add such a test to this PR or do you have something else in mind?

Fixed a bug, that we found in our internal dev environment
Added the MockMvc test I mentioned in my last comment that shows the performance improvements

Fixed a bug, that we found in our internal dev environment
Added the MockMvc test I mentioned in my last comment that shows the performance improvements

LGTM


When you hit back it resets the citadel value

Hitting back or back button on browser it resets the view to the default citadel, not the one I was looking at.

Will look into this after https://github.com/angular/angular/pull/20030#event-1671919242 is released.


feat(rook): Don't deploy Ceph cluster if minimumNodeCount > 2 (instead of > 1)

What this PR does / why we need it:
When eligible for automatic storage scaling (local to distributed), the Rook add-on should not create a Ceph cluster if minimumNodeCount > 2. In such case, EKCO will create the Ceph Cluster.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Does this PR require documentation?

Not needed.


Readme: update Redux cheatsheet URL

http://ricostacruz.com/cheatsheets is now https://devhints.io :) See: https://devhints.io/redux
Btw, there's also many pages in the site that can fit into awesome-cheatsheet:

https://devhints.io/react
https://devhints.io/sass
https://devhints.io/go
https://devhints.io/css
...and a whole other bunch, really

Thanks for the contribution, you can create another PRs for the new addition 👍


fix(graph-cache-config): correct IntrospectionData

Description
In<EMAIL_ADDRESS>IntrospectionData is no longer exported, but can be accessed via CacheExchangeOpts's schema property.
https://github.com/urql-graphql/urql/issues/3027#issuecomment-1463386912
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as
expected)
[ ] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):
n/a
How Has This Been Tested?
Updated existing test & snapshot.
Test Environment:
CI
Checklist:

[x] I have followed the
CONTRIBUTING doc and the
style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
n/a

I fixed the formatting :)

You're right, sorry. It's hard to see which of the many scripts one needs to run. I now went through all ci scripts and hope I got everything. Maybe there could be a "ci" script in the package.json which runs all necessary other scripts that need to run fine so one does not have to wait for each approval or go through the ci definition yml :smile:

You're right, sorry. It's hard to see which of the many scripts one needs to run. I now went through all ci scripts and hope I got everything. Maybe there could be a "ci" script in the package.json which runs all necessary other scripts that need to run fine so one does not have to wait for each approval or go through the ci definition yml 😄

feel free to send that PR and we can also update the docs for it!

Updated to incorporate #314

@dargmuesli i want to keep EOL node 12 independent so let’s revert that here.

Will do once the current GitHub incident is resolved :+1: Do you plan to merge this PR without all tests being green then?

Done :heavy_check_mark:


Closed Dispute Blank at First

I closed my first dispute today and my discovered a tiny bug. Everytime I open the "closed" dispute the summary page is empty / blank / white. I need to click on Summary to display the content of the dispute.

The user "anyone" on Slack reported the issue above. It needs to be reproduced.

It would be useful if he could provide the case api payload and also if he could see if there are any errors in the console.

Here's the exception that's thrown (still waiting on "anyone" to provide the case payload):
VM631:10 Uncaught TypeError: Cannot read property 'amount' of undefined
    at _.forEach.type (eval at _.template (/usr/lib/openbazaar2/resources/app/node_modules/underscore/underscore.js:1454:20), <anonymous>:10:78)
    at Array.forEach (native)
    at eval (eval at _.template (/usr/lib/openbazaar2/resources/app/node_modules/underscore/underscore.js:1454:20), <anonymous>:9:36)
    at template (/usr/lib/openbazaar2/resources/app/node_modules/underscore/underscore.js:1461:21)
    at wrappedTmpl (/usr/lib/openbazaar2/resources/app/prod/utils/loadTemplate.js:52:14)
    at /usr/lib/openbazaar2/resources/app/prod/views/modals/orderDetail/summaryTab/DisputePayout.js:174:25
    at sendBackTmpl (/usr/lib/openbazaar2/resources/app/prod/utils/loadTemplate.js:54:5)
    at loadTemplate (/usr/lib/openbazaar2/resources/app/prod/utils/loadTemplate.js:73:5)
    at _class.render (/usr/lib/openbazaar2/resources/app/prod/views/modals/orderDetail/summaryTab/DisputePayout.js:173:34)
    at _class.setState (/usr/lib/openbazaar2/resources/app/prod/views/modals/orderDetail/summaryTab/DisputePayout.js:157:14)

{
timestamp: "2017-10-10T17:06:13.000Z",
buyerContract: {},
state: "RESOLVED",
read: true,
buyerOpened: true,
claim: "Magic Coin stated 3 days ago that this order was not yet funded. I explained to him to go to Settings --> Advanced, click Reload Transactions. No response since that time.",
unreadChatMessages: 0,
resolution: {}
}

This is happening because the server is returning an empty resolution object. The expectation is that when a case is in a state of RESOLVED a resolution object is provided with details about the final payout.
A server bug has been logged.


Language auto-detection interferes with snippet language

Testing #145929

Add the snippet in https://github.com/microsoft/vscode/issues/145929#issuecomment-1192532575.
Create a new empty text file.
Add in the css-template JS snippet using F1 > Populate File From Snippet. The language changes to JS.
:bug: Auto-detection changes the language back to CSS.

@TylerLeonhardt auto-detection shouldn't do that, how do I tell it to respect user-choice?


JavaScript: Add library for HTML sanitizers

Adds HtmlSanitizerCall, which models a call to a function whose purpose is to sanitize HTML.
CodeInjection has been updated to use the HtmlSanitizerCall class for additional taint steps. Unilke the previous version of this PR, no other queries are affected, since it cost more than expected.
I ran CodeInjection on default slugs to force evaluation of HtmlSanitizerCall. The performance results are noisy but after re-running the slowest bunch it looks okay.
@max you had some concerns about the name-based matching in the standard library. We could move that part back into CodeInjection if you prefer.

Addressed comments and rebased to avoid conflict in the change log.


Add support for certificate links

Add support for certificate links by providing update_certificate_links()

@mennodegraaf due to compatibility issues, as a feature that might not be always required, it would be best if we could have this configurable, and work for both ocpp1.6 and ocpp2.0.1. The security lib (with the exception of the GetKeyPair which will be refactored in everest-core) does not have the side-effects which libocpp can have.

@mennodegraaf due to compatibility issues, as a feature that might not be always required, it would be best if we could have this configurable, and work for both ocpp1.6 and ocpp2.0.1. The security lib (with the exception of the GetKeyPair which will be refactored in everest-core) does not have the side-effects which libocpp can have.

This symlink functionality is configurable in the sense that if the symlinks are not configured nothing will be done and the functionality is as it was before.

@mennodegraaf due to compatibility issues, as a feature that might not be always required, it would be best if we could have this configurable, and work for both ocpp1.6 and ocpp2.0.1. The security lib (with the exception of the GetKeyPair which will be refactored in everest-core) does not have the side-effects which libocpp can have.

This symlink functionality is configurable in the sense that if the symlinks are not configured nothing will be done and the functionality is as it was before.

I think it would be safer to have the function call to be configurable (if the user wants it or not), especially since it is a niche application, and there can always be side-effect. Except that, I can't find any other issues.

@AssemblyJohn I will it up to you if you want to prevent update_certificate_links() from being called with some config parameter. Note that this method will also make sure that symlinks are removed, e.g. when no certificate is valid anymore. So if symlinks are first created and then it is configured not to call the method anymore and then a certificate expires they will not be deleted.


Train on multiple GPUs with nn.DataParallel()

Hello,
I have issue trying to train an INN on multiple GPUs using nn.DataParallel().
I recreated a minimal example of the issue from the example at the beginning of the ReadMe. When initializing the inn using Ff.SequenceINN() (commented in the code below), all goes as intended and I can see all of my GPUs being loaded. But when using Ff.ReversibleGraphNet(), I get an error saying that there is a mismatch between several tensors' devices (note: this initialization works fine on a single GPU).
Since I am planning to implement a convolutional conditional INN I will need something more expressive than just a SequenceINN() that supports splitting.
Does anyone had success training such model on multiple GPUs?
Thanks.
import torch
import torch.nn as nn
from sklearn.datasets import make_moons

import FrEIA.framework as Ff
import FrEIA.modules as Fm

from tqdm import trange

BATCHSIZE = 1000
N_DIM = 2

device = torch.device("cuda:0" if torch.cuda.is_available() else "cpu")

def subnet_fc(dims_in, dims_out):
    return nn.Sequential(nn.Linear(dims_in, 512), nn.ReLU(),
                         nn.Linear(512,  dims_out))

# inn = Ff.SequenceINN(N_DIM)
# for k in range(8):
#     inn.append(Fm.AllInOneBlock, subnet_constructor=subnet_fc, permute_soft=True)

nodes = []
nodes.append(Ff.InputNode(2))
for k in range(8):
    nodes.append(Ff.Node(nodes[-1],
                Fm.AllInOneBlock,
                {'subnet_constructor':subnet_fc, 'permute_soft':True}))
nodes.append(Ff.OutputNode(nodes[-1]))
inn = Ff.ReversibleGraphNet(nodes)

inn = nn.DataParallel(inn)
inn = inn.to(device)

optimizer = torch.optim.Adam(inn.parameters(), lr=0.001)

for i in trange(1000):
    optimizer.zero_grad()
    data, label = make_moons(n_samples=BATCHSIZE, noise=0.05)
    x = torch.Tensor(data).to(device)
    z, log_jac_det = inn(x)
    loss = 0.5*torch.sum(z**2, 1) - log_jac_det
    loss = loss.mean() / N_DIM
    loss.backward()
    optimizer.step()

z = torch.randn(100, N_DIM).to(device)
samples, _ = inn(z, rev=True)

did you manage to find a solution?


[APM-73] Change result code for not-fulfilled write-concern

This implements a PM decision from APM-73:
If there are not enough replicas of a shard on-line for the configured
write concern, then we now return a special error with error code 503,
rather than "collection is read-only" with error code 403. This has the
important consequence that the coordinator can retry for up to 15
minutes.
There are several things missing for this PR. One is documentation and making sure that all drivers and driver tests are fine.
Some additional tests are still needed.
Second is that in all my tries Jenkins had something to complain about. Things seemed to be unrelated, but who knows?
And finally, there is a decision necessary as to if this hits 3.9 or will only appear with 3.10.
Scope & Purpose
(Please describe the changes in this PR for reviewers - mandatory)

[*] :hankey: Bugfix (requires CHANGELOG entry)
[*] :book: CHANGELOG entry made

Backports:

[*] No backports required
This is a breaking API change and should not be backported.

Related Information
(Please reference tickets / specification / other PRs etc)

[*] GitHub issue / Jira ticket number: https://arangodb.atlassian.net/browse/APM-73

Testing & Verification
(Please pick either of the following options)

[*] The behavior in this PR was manually tested
[*] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests

Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the HTTP API

This has bit rot and is not complete. There is not a lot in it to merit keeping it. Closing.


Release 0.8.0-rc

Changes for the 0.8.0 release.
Some small tweaks to release process as I work out the new website.

Hello


Pgbar improvements

When the rendering was slow, you could see the progressbar is empty before the first update() is called. This is not a problem anymore, just in case calling update() from init since the code that would set the init value would be the same as update().
Indicator vertical-align set to center.This fixes the drawing position of the indicator when the progress bar is a child of a Stack.

Thank you.


Clicking 'Go To Gallery' from 'Insights' page does not load all Workbooks listed in next screenshot.

Link: https://docs.microsoft.com/en-us/azure/azure-monitor/vm/vminsights-workbooks#creating-a-new-workbook
Under the section above the directions to see all of the Workbooks, mainly the networking Workbooks is incorrect as clicking on 'Go To Gallery' from that page doesn't load them as shown in the last screenshot. Need to update directions with either new screenshot or update directions to instead go to 'Monitor->Virtual Machines', then to Performance or Map tab and then click on 'Go To Gallery' from there to see the Workbooks.

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

ID: db9a46cf-56c5-8d9f-46d0-1339b0a5eb16
Version Independent ID: b4283585-299a-0863-cc4f-2534d1b3881c
Content: Create interactive reports VM insights with workbooks - Azure Monitor
Content Source: articles/azure-monitor/vm/vminsights-workbooks.md
Service: azure-monitor
Sub-service: virtual-machines
GitHub Login: @bwren
Microsoft Alias: bwren

@bdegs Thanks for your feedback. We will investigate and update the thread.

Thanks, looks like we have an out of date screenshot. I'll get that updated.
#please-close


[zh]Sync pull-image-private-registry.md

Manually synchronize the content/zh/docs/tasks/configure-pod-container/pull-image-private-registry.md  file.
After the ./scripts/lsync.sh script is run, the synchronization content is inaccurate. It should be caused by a previous incomplete synchronization of the file.

All feedbacks addressed.

Thanks.
/lgtm
/approve


Extension causes high cpu load

Issue Type: Performance
Extension Name: vscode-bicep
Extension Version: 0.2.59
OS Version: Windows_NT x64 10.0.20277
VSCode version: 1.52.0

:warning: Make sure to attach this file from your home-directory:
:warning:c:\Users\makolo\AppData\Local\Temp\ms-azuretools.vscode-bicep-unresponsive.cpuprofile.txt
Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load

Thanks for reporting this @mariuszdotnet!

Would you mind either attaching the requested file (c:\Users\makolo\AppData\Local\Temp\ms-azuretools.vscode-bicep-unresponsive.cpuprofile.txt) to this issue, or emailing it to<EMAIL_ADDRESS>if you don't want to share it publicly?
Does this occur intermittently or constantly?
Can you think of anything specific that you were doing when this occurred?

Was the extension functioning in this state? For example, were errors and warnings updating correctly as you were typing or was everything frozen?

@anthony-c-martin In the article linked above, one of the issues called out is code loading. Should we consider using webpack or a similar bundler to reduce load time regardless if this is the cause of this particular perf problem?

dump.txt

@anthony-c-martin  see attached.
Constantly at startup of VS code, but it take a few minutes before it comes up.
No.

@anthony-c-martin In the article linked above, one of the issues called out is code loading. Should we consider using webpack or a similar bundler to reduce load time regardless if this is the cause of this particular perf problem?

I think moving to webpack would be a good idea generally - looks like this extension was loading modules for ~5s. However I don't believe that explains the delay of a few minutes - unfortunately I don't think info about loading the language service is captured in the profiler.

@mariuszdotnet What kind of machine/environment are you running Bicep on?

@majastrz

Kicked off a build with webpack so that we can see if that improves things: #1147

@mariuszdotnet I'm testing with #1147, which uses webpack to efficiently pack all of the module files into one, meaning there's less initialization to do when the extension loads.
If you get a chance, would you mind manually installing the extension from https://github.com/Azure/bicep/suites/1668166397/artifacts/31081831 and seeing if that makes any difference to your load time?
You can install the above VSIX with the following:

Unzip vscode-bicep.vsix.zip
Open VSCode
In the Extensions tab, select the options (...) menu in the top right corner and select 'Install from VSIX'. Provide the path to the unzipped .vsix file.
Click "Install".
Reload VSCode and open a .bicep file.

The webpack fix from #1147 is now released. Going to close assuming this is resolved, but please reopen if that is not the case.


Transformations and slots

CreateSlot API
SlotComposer to handle hosted-slots within transformation particles
Test with multiplexing slots and views

Here is the one-pager description of the APIs: https://docs.google.com/document/d/19MjfUMu1N1to9scuw584lF5Pzb6L_MXUVDoQe6NpR3o/edit#heading=h.gq4ihdogp562

I refactored the slot-composer code a bit - moved the hosted slots to their containing transformation slots, and added unit tests.


suggestion: no https endpoints (on Linux)

https://github.com/dotnet/tye/blob/main/docs/tutorials/hello-tye/00_run_locally.md#troubleshooting describes how to manually make HTTPS certificates work on Linux.
Another option would be to opt-out of HTTPS by not including an https endpoint in ASPNETCORE_URLS.
Since these services are for localhost access, so HTTPS doesn't add much security.
It could be for any OS, or just for Linux.
@jkotalik what do you think?

Hm, it's an interesting idea, but I see a few drawbacks:

If you want to do service to service communication, differences between local and kubernetes for HTTP and HTTPS will be difficult. However, I believe we don't HTTPS for k8s today, so this is a mute point.
In general, ASP.NET Core promotes using HTTPS everywhere, even on local. Ideally, we'd like to follow that guidance if possible.

However, I agree that adding http and https by default is restrictive. Can you try adding just an HTTP binding to your service without a port and see if just an HTTP port is set in ASPNETCORE_URLS?

However, I agree that adding http and https by default is restrictive. Can you try adding just an HTTP binding to your service without a port and see if just an HTTP port is set in ASPNETCORE_URLS?

When I add this:
  bindings:
    - protocol: http

The app binds to http only on a random port.
I believe the bindings get added here:
https://github.com/dotnet/tye/blob/e6b6b06cd34414fa142b4aa62722d8ef0a8befb5/src/Microsoft.Tye.Core/ApplicationFactory.cs#L350-L358
The suggestion is to not add the "https" binding on Linux to avoid usability issues related to trusting the developer certificate.

@jkotalik I hope you'll consider not adding an https binding by default on Linux. I think it's a better alternative than https://github.com/dotnet/tye/blob/main/docs/tutorials/hello-tye/00_run_locally.md#certificate-is-invalid-exception-on-linux.


Add exchange in MemPool to reduce alloc/free operation

This PR will reuse allocated chunks when storage read operation.
#1726

This PR seems have a indent issue?

This PR seems have a indent issue?

Origin code is 2 spaces indent, I change it to 4 spaces which is consistent with most of our code

+1. LGTM.
The main change of this PR is add the MemPool::exchange_data method, other diff is because of indent.


Visualization tooltips

Now tooltips are displayed upon mouse over.
Here is the result:
http://batarypa-personality-insights.mybluemix.net/

Please @germanattanasio review this.


Code is generating links to undesired content

Hello,
I downloaded the latest version to date and uploaded to my hosting (OVH).
When I amchecking the code with developper tools and the Lighthouse analysis, I saw there was an advice to remove unused CSS.
I thought it was because I removed the portfolio sections and modals but I see that a code is generated that links to indesirable websites. The link is changing each time I run the analysis.
See here:

and here:

We aren't able to replicate this issue. There could be a browser extension or something else that you have running locally on your machine that is causing this issue. If you can provide more details I can check further, but I am unable to replicate this at the moment.


Automatic Vundle installation on first run of Vim

I've been using this code for ages, from the very first time I learned about Vundle. I can't even remember where I found it (hey, I was young, I didn't care about storing that type of information).
Anyway, I adapted it to work with the new synstax of Vundle (PluginInstall instead of BundleInstall)
and found out that everything works as expected.
I thought it could be added to the documendation. All that is required is to add the lines in ones .vimrc file and have git installed on the system.
" START - Setting up Vundle - the vim plugin bundler
let iCanHazVundle=1
let vundle_readme=expand('~/.vim/bundle/Vundle.vim/README.md')
if !filereadable(vundle_readme)
  echo "Installing Vundle.."
  echo ""
  silent !mkdir -p ~/.vim/bundle
  silent !git clone https://github.com/VundleVim/Vundle.vim.git ~/.vim/bundle/Vundle.vim
  let iCanHazVundle=0
endif
set rtp+=~/.vim/bundle/Vundle.vim/
call vundle#rc()

" let Vundle manage Vundle, required
Plugin 'VundleVim/Vundle.vim'
if iCanHazVundle == 0
  echo "Installing Bundles, please ignore key map error messages"
  echo ""
  :PluginInstall
endif
" END - Setting up Vundle - the vim plugin bundler

This is awesome! Works great!

Pretty cool. Thanks

Replace
:PluginInstall

with
:source $MYVIMRC
:PluginInstall

will automatically install plugins after installing Vundle.

use a script that will install Vundle, and run the extrinsec vim command (non interactive) once.
vim -c PluginInstall -c qall


Check if flag is "server-side only"

Hi,
is there a way to check if some feature is server-side only? I am trying to use this lib in my backend, but here for some business rules I need to know if some flag has the option "server-side only" set.
Thanks.

Hi @dhferreira, that's interesting. There isn't any way currently to do that, no. I would recommend perhaps using a naming convention to achieve this?


Baseline model implementation

Hello, I was reading your paper about MACNet recently.
According to MCANet you compared MCANet with MolTrans, TransformerCPI etc.
These baseline models are implemented by yourself or their official repositories?Because I have also reproduced these two benchmark methods recently, but found that the AUPR result in your paper is higher than the result written by the original author's paper.And I use the same evaluation metrics as yours in Davis and KIBA, but the result is much worse than yours.
If convenient, could you provide baseline model implementation about Davis and KIBA dataset?

Thanks for your attention!
All baselines are based on official code. The changes include:

converting the dataset into a format that these baselines can read
fixing a few deprecated API calls in the code to ensure that the same runtime environment can run these code.

Unfortunately, the code for the baseline is stored on a server and I currently do not have access to that server.
I hope to be able to release the code as soon as possible in the coming months.

Do you remember what modifications you made to the official repository? For example, whether the data division has been modified, whether cross-validation has been performed, whether hyperparameters have been modified, etc.

Regarding the hyperparameters, I don't remember making any changes to them. The dataset was split with 5 fold cross validation.

I got permission to access the storage again recently, but I found that the data had been deleted.
I found some code in the archive that may not be exactly the same as the code used for the baseline comparison in the paper.
I have posted these codes in another repository and hope this will be helpful.

thank you！


View proposed market changes

As a console user
I want to see when there are proposed  changes to market parameters that effect the market I'm looking at in console
So that I know what's being proposed, can understand potential impact and easily go vote for agains t the changes without having to keep an eye out for proposals somewhere else

Ideally need some API updates for this first https://github.com/vegaprotocol/vega/issues/6667

Looks like this might not be blocked any more as 6667 is closed.
I'd suggest we highlight this in the market header when there is a live change:

This would link to the proposal page on Token. But this does raise the question of where we link to if there are multiple proposals for this market - ideally we would link to a page that lists all the proposals for a given market. It doesn't look like this is currently possible - on the frontend you can filter by proposer ID, but not market ID - I don't know if that's something we want to consider adding?
We could also highlight the sections/fields that are proposed to be changed under the Market Info tab:

Coming back to this having worked on View proposed network changes, and suggest that we use the same solution - use toasts to let the user know about changes to the market/network:
This could also solve the issue of what to do when there are multiple proposals for a single market, as you could show a toast for each proposal.


How to fetch values from array?

How to fetch values from array, such as:
key1=["val1", "val2", "val3"]
key1 = [ ["txt1", "txt2", "txt3"], [1, 2, 3] ]

For the first:
auto config = cpptoml::parse_file("file.toml");
auto key1 = config->get_array("key1")->array_of<std::string>();
auto item1 = key1.at(0);

For the second:
auto config = cpptoml::parse_file("file.toml");
auto key1 = config->get_array("key1");

// getting the array of strings
auto array1 = key1->at(0)->as_array();
auto string1 = array1->at(0)->as<std::string>();

// getting the array of integers
auto array2 = key1->at(1)->as_array();
auto int1 = array2->at(0)->as<int64_t>();

Is there a function of get array size? And does cpptoml change the items order of array in config file?

I've just added some easier methods for getting and dealing with arrays. The above still works, of course, but here's how I would write it now.
For the first:
auto config = cpptoml::parse_file("file.toml");
auto key1 = config->get_array_of<std::string>("key1");
auto size = key1->size();

For the second:
auto config = cpptoml::parse_file("file2.toml");
auto key1 = config->get_array_of<cpptoml::array>("key1");

// getting the array of strings
auto array1 = key1->at(0)->get_array_of<std::string>();
auto string1 = array1->at(0);
auto size1 = array1->size();

// getting the array of integers
auto array2 = key2->at(1)->get_array_of<int64_t>();
auto int1 = array2->at(0);
auto size2 = array2->size();

(I've included how to get their sizes.)
cpptoml does not reorder the elements of an array.


[BEAM-9346] Improve the efficiency of TFRecordIO

I added the code requested in the Jira issue.
R: @mwalenia @kamilwu

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

retest this please

retest this please

@piotr-szuberski As a non-commiter, you are not allowed to trigger Jenkins tests on your own, unfortunately. Let me do this for you.

retest this please

Ok, thank you - I was confused that the job has not started yet. Good to know!

Run Java Precommig

run java precommit

Run TFRecord IOIT

Run Java TFRecordIO Performance Test

LGTM, thanks for the contribution!

If this is a common pattern (another one from: https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/io/WriteFiles.java#L412), should this be abstracted out to happen in a single place?
/cc @chamikaramj @kennknowles


Add function that submissions can call that can change the dropout value

Our current API has 2 dropout related limitations:

Currently, in the external tuning ruleset we read the dropout value from the hparam config and pass it to the model initialization functions. In the self-tuning ruleset there exist no convenient way to specify the dropout value in the model initialization.
Furthermore, there is no way to change the dropout value during training.

Having a workload function to change the dropout value that submitters can call will remove both of these limitations.

dedupe issue/753


Streamline tests

tl;dr: this PR makes better use of pytest features instead of reinventing them in multiple places. It also moves some testing code to a new ixmp.testing submodule so it can be reused by message_ix (e.g. this shouldn't need redefinition) or by users.
I started making some of these changes when writing tests for #126 and figured they could stand alone.

ixmp.testing: new submodule for reuse of testing utilities.

create_local_testdb(): moved from tests/testing_utils.py
dantzig_transport(): replaces code in tests/test_integration.py and
tests/testdb_setup.py.

consolidate test data in tests/data.
ixmp.Platform.init: ensure dbprops is string, for Java compat.
ixmp.default_paths: use Path objects.
tests/conftest.py:

new fixtures: test_data_path, tutorial_path.
existing test_mp* fixtures: add docstrings, use tmp_path_factory
fixture.
new hook: pytest_report_header (replaces test_admin.py).

other tests: use pytest & ixmp fixtures.

PR checklist:

[x] Tests added
[x] Documentation added
[x] Description in RELEASE_NOTES.md added

Okay, finally all the tests pass. This is ready to review & merge, from my POV.

Would it be possible to connect the tutorial-notebook tests to a temporary db?
As mentioned in Slack, this is possible, but it requires new functionality. Related to #124. I would prefer to merge this PR and then work on the other issue.