Fields Incorrect

The PBI Template is looking at an old column for estimates.
Compute monthly cost USD in PBI, is now Compute monthly cost estimate USD and the same for storage

@JammyB Thank you for reporting the issue. Please provide the name of the sample or a link to the sample that you're experiencing issues with. Also, if an error message is generated when attempting to use the referenced sample, please provide it.

Hello both @mikefrobbins what @JammyB mentioned above is that the new labels that are produced by default at the exported assessment excel files have differences whit what PowerBI is configured for.
Differences
Sample files vs Actual files downloaded currently from Azure

On your sample files on the worksheet "All_Assessed_Machines" the Column G label is "Compute monthly cost USD" on the new exported files the column label is "Compute monthly cost estimate USD".
On your sample files on the worksheet "All_Assessed_Machines" the Column H label is "Storagemonthly cost USD" on the new exported files the column label is "Storage monthly cost estimate USD".
On your sample files on the worksheet "Assessment_Summary" the cells A16 label is "Compute monthly cost USD" on the new exported files the column label is "Compute monthly cost estimate USD"
The same happens for the labels on the cells A17 to A20 on the same worksheet.

Also there is an issue when the assessment is made in other currency. Like Euro.
What i did till now to manage run the PowerBI (cause I prepared only two assessments), was to go on my exported files and changed the label values to mach with the old values, and it works without the issues mentioned above.
It would be nice if you may apply the above changes.
One last question, when you want to process an AS-IS report and a Performance based report Average CPU remains blank most probably because the AS-IS report does not have a value but text "Not Applicable"

If the values changed to a number (I updated the non applicable to zero for both CPU and RAM) it works like a charm.

@ekleidaras or @JammyB, Which file in this repo is causing the problem? Is there a link or sample script in documentation somewhere that guided you to this repo?
The files in this repo are sample scripts and artifacts for Azure PowerShell documentation. Once I know which file is causing the problem, I'll contact the appropriate service team to take a look at it.

@mikefrobbins the file on this repo that is producing the issue is the "Azure Migrate Assessment Comparison PowerBI Template.pbit".

Azure Migrate Export (AME) replaced the Azure Migrate Assessment Comparison PowerBI Template found in this repo.
AME Documentation
Download the required Utility Package
Additional links:
AME Sample Presentation
Sample Reports
Demo and Training Video
I'll update this repo to reflect this information and search our docs for any outdated links pointing to the Azure Migrate Assessment Comparison PowerBI Template in this repo.

@mikefrobbins The additional links does not work
Additional links:
AME Sample Presentation --> May be this link should be https://aka.ms/amepresentation and not https://aka.ms/amepresentationAME
Sample Reports --> May be this link should be https://aka.ms/amesamplereports and not https://aka.ms/amesamplereportsAME
Demo and Training Video --> This is not accessible.
Event if i tried the above corrections on the links i got the same message for all of them

Could you please validate the url's again?
Thank you.

@ekleidaras Thanks for reporting the issues with the additional links. I've removed them for now and sent a message to the service team asking if they can make them available externally.

@mikefrobbins Is there any update on this? Thank you!

@ekleidaras No updates, but I'd recommend logging an issue in the azure-migrate-export repo since this is out of the scope of the samples located in this repo.


after service startup,  redisson occupies about 150 local ports

we config redisson like this
@Bean
  public RedissonClient redisson() {
    String[] nodeArray = clusterNodes.split(",");
    for (int i = 0; i < nodeArray.length; i++) {
      nodeArray[i] = "redis://" + nodeArray[i];
    }

    Config config = new Config();
    ClusterServersConfig clusterServersConfig =
        config.useClusterServers().addNodeAddress(nodeArray);
    if (password != null && !password.isEmpty()) {
      clusterServersConfig.setPassword(password);
    }
    return Redisson.create(config);
  }

the variable clusterNodes is redis ip and port, value like this clusterNodes   = "<IP_ADDRESS>:7000,<IP_ADDRESS>:7001,<IP_ADDRESS>:7002,<IP_ADDRESS>:7003,<IP_ADDRESS>:7004,<IP_ADDRESS>:7005".
and in out project, we  also use Lettuce to save value in redis. But after the service startup, there is a problem that we use netstat -nap  will find a large number of redis connections（about 150+）, and  a large number of port is occupied.

But after we remove redisson config bean, and remove the code dependency on redisson, the problem will not appear.
The redisson version is 3.17.4,

reduce connection pool size for both master and slave nodes.


chown resource executes every run, even when not changing anything

We commonly see an execute resource updated every chef run, despite nothing being changed:
Example from our env:
- execute[chown -R root:root /mnt/consul] (consul::_service line 60)
- execute[chown -R root:root /etc/consul.d] (consul::_service line 60)
- execute[chown -R root:root /var/lib/consul] (consul::_service line 60)

This was introduced in #141 - current source.
A better approach is to guard the resource and execute only if necessary, something like:
execute "fixup consul #{dirname}" do
  command "chown -R #{consul_user}:#{consul_group} #{dirname}"
  only_if { Etc.getpwuid(File.stat(dirname).uid).name != "root" }
end

This is untested example code, based on conversations in mailing list and jira ticket tracker from 6 years ago, and the current conditional only applies to the uid, not gid. That should be an easy-enough fix.
Another method would be to use Ruby to get all subdirectories and create resources for each one, but that could lead to unexpected resource creation, if dirname has 10k subdirs - unlikely but possible.
I'd be happy to submit a PR for this, if you're interested in this approach.
/cc: @darron

This seems reasonable to me. Feel free to drop a PR and I'll cut another release prior to merging #126.


Update showcase.js

Hey there and thanks for submitting a pull request! Please have a look at the following checklist so that others have enough information to review your pull request:
Explain the motivation for making this change. What existing problem does the pull request solve?
Example: When saying "Add a function to do X", explain why it is necessary to have a way to do X.
Test plan (required)
Describe exact steps you've done to test that the code works. This means exact commands you ran, screenshots / videos if the pull request changes UI.
Make sure tests pass on Circle CI.
Code formatting
See the simple style guide.
add zrcaifu to showcase

I've just merged another PR that adds an app to the end of the list. You might want to wait until that lands and then rebase.

@jxintang Can you rebase?

Hey @jxintang! Thanks for making the PR, but we are closing it due to inactivity (24 days with no actions). If you want to get your proposed changes merged, please rebase your branch with master and send a new PR.


Run linter on tests and docs

Fixes #768.

Description

This pull request changes the linter config to run on tests and docs.
Pull request checklist

[ ] Did you test this change locally?
[ ] Did you update the documentation (if required)?
[x] Did you add/update relevant tests for this change (if required)?

Additional information

How to test

@dcmcand PTAL. Let me know if you currently don't have cycles, then I'll assign someone else.


Error installing

Hello and Thank you for creating such a tool that can help adalo apps perform faster
but iam having a problem installing
U see i keep getting the same error
here is the repo
52 error code ERESOLVE
53 error ERESOLVE could not resolve
54 error
55 error While resolving: <EMAIL_ADDRESS>55 error Found<EMAIL_ADDRESS>55 error node_modules/react
55 error   dev<EMAIL_ADDRESS>from the root project
55 error   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>55 error   node_modules/react-art
55 error     dev<EMAIL_ADDRESS>from the root project
55 error     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>55 error     node_modules/react-native-web
55 error       dev<EMAIL_ADDRESS>from the root project
55 error   19 more (react-dom, react-datepicker, react-native-firebase, ...)
55 error
55 error Could not resolve dependency:
55 error peer<EMAIL_ADDRESS>from <EMAIL_ADDRESS>55 error node_modules/@protonapp/proton-runner
55 error   <EMAIL_ADDRESS>from the root project
55 error
55 error Conflicting peer dependency<EMAIL_ADDRESS>55 error node_modules/react
55 error   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>55 error   node_modules/react-native
55 error     peer<EMAIL_ADDRESS>from <EMAIL_ADDRESS>55 error     node_modules/@protonapp/proton-runner
55 error       <EMAIL_ADDRESS>from the root project
55 error
55 error Fix the upstream dependency conflict, or retry
55 error this command with --force, or --legacy-peer-deps
55 error to accept an incorrect (and potentially broken) dependency resolution.
55 error
Thank you in advance i hope this is an easy issue and you can help me sort it out

@PragmaFlow


Fixing memory leak fetching Enum vals each loop

The old way caused a memory leak to occur where the Enum.GetValues was creating array each loop and the GC was not cleaning it up once finished. Plus I don't think there is a need to fetch it each loop as the enum is fixed on execution.

Sorry for the messy commits... for some reason when I just wanted to amend a commit it said I was behind somehow

Can you explain how this caused a leak, is there a bug in Enum.GetValues in some CLR implementations? GC is supposed to take care of this at a language level. Is this working around some known bug? Do you have documentation this bug?

I just noticed that the memory was increasing unexpectedly. It may just be me with how I do development. I am very stringent with memory. It plateaus at 128MB which I think is when GC was kicking in. I may have misinterpreted it at the time as a memory leak. But wouldn't it make sense to call Enum.GetValues only once seeing that it's technically a constant?

It's another global to maintain. Really the optimizer should handle this. I'll think about it. I mostly wanted to know if it was a real memory leak or just GC batching.


Improved handling of channel pages

When a GET request is sent to the endpoint it should redirect to the channel page.

Need to:

Change the URL for this GET endpoint: https://github.com/paulgb/notify.run/blob/master/server/notify_run_server/app.py#L100
Add a redirect endpoint.
Change api.ts to hit the newly renamed endpoint https://github.com/paulgb/notify.run/blob/master/site/src/api.ts#L35


Update jitpack.yml

This PR is for the Add jitpack.yml issue

Will we have to remove Java 11 from https://github.com/elimu-ai/analytics/blob/main/.github/workflows/gradle.yml as well?

After looking into it, I don't think that is necessary. The 11 and 17 in that file simply makes it so that it tests against both Java 11 and 17. The matrix.java variable dynamically sets the java version for each run so I don't necessarily think it is necessary but I am willing to do it if you think it should be done.


Change DATA segment processing

Current processing of DATA_CONST and DATA segments is ok, no need to change.


Executor race condition and ethereum account param

This pull request introduces the following features:

New eth_account_address ROS parameter, which can be passed to all nodes, requiring Ethereum account for operation. Empty value by default means, that a node should use 0-th account, as it was before this improvement
New enable_executor ROS parameter in liability.launch, which can be used to start Liability-related nodes without Executor. Default value is true.

This pull request fixes the following:

Race condition, which took place in Executor rosbag recording
Liability execution result was not sent in the separate topic, suitable for signing and sending to the IPFS

Nice, merge.


Headlamp doesn't send correct resource name parameter for CRDs to the SelfSubjectAccessReview API

Hello, this issue is similar to https://github.com/headlamp-k8s/headlamp/issues/1490 but a little bit different and related to CustomResourceDefinition objects. I've tried to edit a Namespace scoped CustomResource (like ServiceMonitor or Certificate) with my namespace scoped service account that is bound to the cluster role admin.
Currently headlamp send incorrect resource name for CRD resources in the SelfSubjectAccessReview API call:
{"kind":"SelfSubjectAccessReview","apiVersion":"authorization.k8s.io/v1","spec":{"resourceAttributes":{"verb":"update","name":"my-servicemonitor","namespace":"my-namespace","group":"monitoring.coreos.com","version":"v1","resource":"crds"}}}
Response:
"status": {     "allowed": false   }
This request returns incorrect false response for a user who is admin in the namespace only and should have ability to edit these resources.
Note that resource is crds for all requests to all CustomResource objects. Instead it should send correct CustomResource name, like:
{"kind":"SelfSubjectAccessReview","apiVersion":"authorization.k8s.io/v1","spec":{"resourceAttributes":{"verb":"update","name":"my-servicemonitor","namespace":"my-namespace","group":"monitoring.coreos.com","version":"v1","resource":"servicemonitors"}}}
This request returns correct response:
"status": {     "allowed": true,     "reason": "RBAC: allowed by RoleBinding \"namespace-admin" of ClusterRole \"admin\" to ServiceAccount \"namespace-admin\""   }

Thanks @Denis220795 . We will look into that in the coming days hopefully.


color-scheme and font improved

please resolve conflicts and add ss

delete the repo and fork again to do your changes


Move allele database schemes to their own section

Description of feature
Allele database schemes parameters --allele_scheme and --lx_allele_database should be in the same section.

fixed in: PR 104


Zoom, Scale Issues

I am working with a very large flow chart, 6000 down x 750 wide. When using the zoom feature it is cutting off part of the diagram when I display it. When I use the scale option is is extremely small even when I change the iFrame size to match the diagram it is still not filling the space and stays very tiny. This has made it very hard to view large flow charts. I have also changed the size of the page in the xml to match the large flow chart size, but still am not able to view whole chart correctly.
This is part of the xml:

Hi Seana36,
I'll try to add zoom with button or mouse wheel for next versoin.
it could satisfy your need ?
BR
Arnaud

Hi Arnaud, yes actually that would be great. Thanks!

Hi Emily,
Try last version, first version with zoom
https://algenty.github.io/flowcharting-repository/CHANGELOG.html

I have installed the latest version of the plugin. I can see that the right click to drag is working fine. For the zoom, when holding down control and using the mouse wheel in Chrome the browser zoom in/out is taking over, even when making sure to have clicked inside the panel. Is there any specific setting I need to be using?

no, just use ctrl + mousewheel on panel and esc to restore

Looks like i was still using version 0.3.0, once 0.4.0 is available for download I will test out the zoom. Thanks.


Move SlowDownMonitoringAsync.

Description
Previously we were calling this method late in the monitoring loop however after some earlier refactoring we made it less effective. By moving it up to where we are in a steady state it has the chance to have an effect (it internally checks for healthy state and falls through quickly if it isn't in one.
Fixes #6134
Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[x] Yes
[ ] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

These tests would be MUCH better if we did https://github.com/dotnet/aspire/pull/6068


Collections Error

First off, I’m so sorry about opening this issue here.
I’m an avid user of your site devmarks.io and I’ve run into a few issues:

I can’t access my profile. It returns a 500 error
I can’t add new collections, it returns a 500 error

I thought maybe these were related to me not having a paid account, so I splurged for one month of the ‘Developer’ level account. It looked like my transaction was successful, but I haven’t gotten a receipt yet (that might just be a timing thing though)
Anyway, I wanted to let you know about the 500 errors that I’m getting (even after the paid account)
Keep up the good work.
Also, loved your appearance on DjangoChat with Will and Carlton. Great conversation! And thank you for all of your work in the Django Community 😁

Hello! No worries. 😅 I'm glad you like devmarks, but I'm sorry you are getting 500s -- I didn't notice the error emails from Sentry somehow. I'm digging into the error and will hopefully fix it soon. I will let you know when it's cleared up. 👍


fix query_country_alsi_storage

There seems to be a copy paste issue in the query_country_alsi_storage which makes it give the same result as for the AGSI query.

Hi @nttoshev,
any chance this gets fixed some times?


Retrieve Markdown parser from Application

Changing the markdown provider could be quite an API change, but instead I elected to add a small PSR Container layer inbetween, to leave the functioning identical to the current behaviour.
I didn't add a commonmark parser yet, but that's one extra commit away, if that's deemed required.

Helps with #252

I changed the base to 2.x, since this is a change that can be implemented in v2 without serious issues (no method signatures were changed)


no body returned for response

i'm using token for vk-oauth2 authentication, and every time i send post request to api/login/social/token_user/  i get 400 error. what am i doing wrong? first method with token works just fine

@benzom do you solve the problem? Please say how for other programmers with the same problem :)


Rake commands to ping DynamoDB and to create tables from the models

I've created a couple of simple rake commands for Rails to:

test the connection to the DynamoDB (local or remote) using the current Dynamoid.configuration settings

Create the tables in advance of using them which kinda works around the lag Dynamo has when creating tables

Coverage decreased (-0.04%) to 95.212% when pulling 6b233ef6cfda090d314c351b309d112498896932 on NrBT:master into 5dfff232e57688d083b591d5ac3631b2afebab25 on Dynamoid:master.

Please rebase on latest master - Spec suite is fixed!!!

@NrBT still interested in this?


Put contribution guidelines into CONTRIBUTING.md

GitHub facilitates the communication of contribution guidelines, if they're put in a file CONTRIBUTING or CONTRIBUTING.md in the root of a repo. Since you've made guidelines, it would be appropriate to put these in a CONTRIBUTING file.
More info: https://github.com/blog/1184-contributing-guidelines

Added


Add Hungarian provider Hosszupuska

Hi!
I see that test are runnig for lxml and native parser also.
But it seems for me that in case of this provider the results are different for them.
Which one I need to use?


Onflow to merge with ledger hq

Copy of https://github.com/LedgerHQ/app-flow/pull/15 .
Required for the CI to be properly triggered before merge.

@relatko please update the tests that fail because of the version bump so that the CI can pass again before the merge. Thank you.

@relatko please update the tests that fail because of the version bump so that the CI can pass again before the merge. Thank you.
Seems i do not have write access to this branch.
I fixed the tests in the original PR branch https://github.com/onflow/ledger-app-flow/tree/to-merge-with-ledgerHQ

:heavy_check_mark: CI is ok.
:heavy_check_mark: Version bump.
:heavy_check_mark: No pending review warning.
:heavy_check_mark: Merge to develop for P4 deployment.
Ok for merge.


Using @Component Functions in Spring Cloud Stream still broken in 3.2.8/v2021.0.5

It's still not possible to use Functions/Consumers etc. defined as @Component in the latest release 3.2.8 (or spring-cloud-dependencies v2021.0.5).
There was a similar issue with @Component annotated Functions that was resolved in #925 by @olegz.
It's not possible to cast a Class to ParametrizedType as it is done in this line:
https://github.com/spring-cloud/spring-cloud-function/blob/b90dd7e1498c1f132709a651410ce018e79e3c81/spring-cloud-function-context/src/main/java/org/springframework/cloud/function/context/config/KotlinLambdaToFunctionAutoConfiguration.java#L160
The difference is, when using a Function Bean like
@Bean
fun myTestConsumer(): (String) -> String {
    return {...}
}

the actual Type is kotlin.jvm.functions.Function1<java.lang.String, java.lang.String> which can be cast to ParametrizedType.
whereas a Component like
@Component
class MyTestConsumer : Function<String, String> {
    override fun apply(t: String): String {
        TODO("Not yet implemented")
    }
}

resolved to the actual class, in my example (see below) class com.example.springcloudfunctiontest.MyTestConsumer which cannot be cast to ParametrizedType
Sample
The sample I prepared for #925 is still valid and not working with the latest version. I am sorry that I did not tested it earlier.
https://github.com/fschlag/SpringCloudFunctionTest

May relates to #940 as well

Hi @olegz , I tested the latest changes using the Snapshot Release of spring-cloud-dependencies:2021.0.6-20221115.024836-32 from repo.spring.io (which includes your latest change).
Unfortunately this did not fix the issue. The same error still occurs when casting  at this line
https://github.com/spring-cloud/spring-cloud-function/blob/ace6c16dd5f51cdd112181504c4fb205b96bdae5/spring-cloud-function-context/src/main/java/org/springframework/cloud/function/context/config/KotlinLambdaToFunctionAutoConfiguration.java#L160
The adjustments you've made regarding detecting the functionType did not work. At the line above the Type is still the Test Consumer com.example.springcloudfunctiontest.MyTestConsumer

Hi @olegz, I am not sure if you saw my previous comment. But this is not fixed with your previous change same for #940.


Proof of concept integrating checkpoint simulation into scheduler

After #1189 , we want to use checkpoint simulation in the scheduler, and see if it can help with #1187

Currently blocked by an issue in the Clipper model

Issue in Clipper model has been fixed so this is unblocked :)


CustomCollection passed instead of CustomCollection

Description
An incorrect test is generated:
Array<Object> is passed as a parameter to the method, that expects Array<Int>.
The problem is present at the engine level.
To Reproduce
Steps to reproduce the behavior:
Example:

Generate tests for org.utbot.examples.stream.BaseStreamExample#customCollectionStreamExample
Check if there is a test named like testCustomCollectionStreamExampleReturnsZero
In that test the described effect is present

Expected behavior
CustomCollection<Int> is supposed to be created instead of CustomCollection<Object>.
Actual behavior
CustomCollection<Object> is used.
Visual proofs (screenshots, logs, images)
    @Test
    @DisplayName("customCollectionStreamExample: customCollection = CustomCollection(Object[]) -> return 0")
    public void testCustomCollectionStreamExampleReturnsZero() {
        BaseStreamExample baseStreamExample = new BaseStreamExample();
        Object[] objectArray = {};
        CustomCollection customCollection = new CustomCollection(objectArray);

        long actual = baseStreamExample.customCollectionStreamExample(customCollection);

        assertEquals(0L, actual);
    }

Environment
Not applicable.
Additional context
Also might be present in other cases with collections, especially if the timeout is long enough.

The problem looks similar to #281, @CaelmBleidd take a look, please

I think it is a different problem. We have the following problem: there is a need to construct Iterable, but that's impossible. Therefore, we chose some of its inheriters independent of their generic parameters, so we chose the wrong Scanner<? extends String> for Iterable<Object>. Here we don't have a need to choose between types, so it is just missed constraints somewhere, I guess

Cannot reproduce it at this moment, perhaps #253 should be fixed first.

This issue is a part of a big feature that should possibly be implemented in the future, so it should be fixed after resolving #924.


feat: add a tool to convert AVD inventory to ANTA inventory

Add a CLI entrypoint to convert a YAML ansible inventory from AVD to ANTA
anta get from-avd <path to yaml file> --container-name fabric

Resolved with PR #295


Login with Facebook account

As the Product Owner I think this Feature is totally and utterly completely implemented.
The Story is well satisfied.
It can be closed now.

Teambeschluss 20.09.2022: Wird nicht ungesetzt und geschlossen.


top k logic issue

I think it should be x.sort(1, True),   and the results should be the same with torch.topk API.
https://github.com/IBM/MAX-Scene-Classifier/blob/0927fc5788c3bc4ede43edf6bc0e626f40976f9b/core/model.py#L93
Sample codes:
>>> import torch
>>> probs = torch.tensor([[0.0297, 0.3163, 0.4335, 0.2205],
                [0.3554, 0.2583, 0.3467, 0.0396],
                [0.3997, 0.4566, 0.0879, 0.0557]])

>>> probs.sort(1,True)

orch.return_types.sort(
values=tensor([[0.4335, 0.3163, 0.2205, 0.0297],
        [0.3554, 0.3467, 0.2583, 0.0396],
        [0.4566, 0.3997, 0.0879, 0.0557]]),
indices=tensor([[2, 1, 3, 0],
        [0, 2, 1, 3],
        [1, 0, 2, 3]]))

>>> torch.topk(probs, 4)

torch.return_types.topk(
values=tensor([[0.4335, 0.3163, 0.2205, 0.0297],
        [0.3554, 0.3467, 0.2583, 0.0396],
        [0.4566, 0.3997, 0.0879, 0.0557]]),
indices=tensor([[2, 1, 3, 0],
        [0, 2, 1, 3],
        [1, 0, 2, 3]]))

By default, topk performs on the last dimension. Since your examples are all 2-dimensional, it makes sense that it corresponds to sort(1, True). Since the line you cited is on a 1-dimensional tensor, sort(0, True) would correspond to a topk call. Nevertheless, I made a PR to fix this part of the code to use topk, which would be less confusing.


dontmerge: stdenv: test regression on staging

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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build stdenv

@ofborg build stdenv

The title of this commit is wrong; I don't see any of my commits in here.

I've squashed your commits in the commit that references your PR. I guess it's still OK since this PR is for testing.

It's impossible for anybody else to figure out what you've done.
Changing the authorship on other peoples' commits is not OK.

Okay, I'll rebase.

@ofborg build stdenv

I still don't see https://github.com/NixOS/nixpkgs/pull/206907 anywhere in the commit history of this PR, as you claimed here.
As I pointed out, https://github.com/NixOS/nixpkgs/pull/206907 isn't rebased past https://github.com/NixOS/nixpkgs/pull/213185 so there is no way to make a clean comparison without mangling one PR or the other.
That's why we wait for them to once again have a common ancestor.  This will have to happen in order for one of them to merge and the other to be merge-eligible.


from .rdBase import rdkitVersion as _version and ImportError: DLL load failed: The specified module could not be found

Configuration:

RDKit Version:  '2019.09.3'
Operating system: win 10
Python version (if relevant): '3.7.4 (default, Aug  9 2019, 18:34:13) [MSC v.1915 64 bit (AMD64)]'
Are you using conda? YES
If you are using conda, which channel did you install the rdkit from? conda-forge
If you are not using conda: how did you install the RDKit? using pycharm

Description:
I wrote a script on Pycharm using conda interpreter. when running the scrip on Pycharm it runs fine.
However, when the API of Compound Discoverer (software) runs the script, it shows those warnings and finally fails.
Time	Processing Node	Level	Message
11:19 AM	(49): Scripting Node	Error	The executable ran for 242.6 ms and finished with error(s), exit code was :1
11:19 AM	(49): Scripting Node	Warning	ImportError: DLL load failed: The specified module could not be found.
11:19 AM	(49): Scripting Node	Warning	    from .rdBase import rdkitVersion as __ version __
11:19 AM	(49): Scripting Node	Warning	  File "C:\Users\BCDD\AppData\Local\Continuum\anaconda3\lib\site-packages\rdkit_init_.py", line 2, in < module >
11:19 AM	(49): Scripting Node	Warning	    from rdkit import Chem
11:19 AM	(49): Scripting Node	Warning	  File "D:\BCDD\Documents\Tal\CD\Scripting\cd-scripting-node-python\node\ScriptingNode1.py", line 5, in
11:19 AM	(49): Scripting Node	Warning	Traceback (most recent call last):
11:19 AM	(49): Scripting Node	Info	Starting executable C:\Users\BCDD\AppData\Local\Continuum\anaconda3\python.exe with arguments D:\BCDD\Documents\Tal\CD\Scripting\cd-scripting-node-python\node\ScriptingNode1.py "C:/ProgramData/Thermo/Compound Discoverer 3.1/Scratch/Job42/Script(49)/node_args.json"

Your code sample here

I checked other discussions  :
http://rdkit.org/docs/Install.html#cross-platform-under-anaconda-python-fastest-install
https://github.com/rdkit/rdkit/issues/1634

@TalWac please try googling for "rdkit pycharm" and trying out the solutions you find there.
If those don't work, you can provide more information and reopen this issue.


DID Parsing Checks

Should add more checks when parsing DIDs, not just nullity checks. The next step could be DID length.

This has been solved in #54 by checking that the DIDs used when creating/editing/buying/selling/swapping bonds are using a ledgered DID.

This issue has been reopened as it needs to be tackled for the new modules. Also, I have noticed that the approach used by the Cosmos SDK to create module accounts is the same as the approach used here to convert a DID to a Cosmos address.
This is good news (as it means we're using a standard approach) but this makes it that much more important to confirm that we're creating addresses for DIDs and do not allow sending of tokens to module accounts.

Criteria for closing of this issue:

No module or message handler should allow sending of funds to module accounts
DIDs used in any module should conform to a standard format
DIDs used in any module should be ledgered in the DID module (or should they?)

Since https://github.com/ixofoundation/ixo-blockchain/issues/55#issuecomment-619019963 has been satisfied, this issue can be closed.


InternalError:cudnn poolforward launch failed

i have installed tf-gpu 12.0,   cuda9,  tf-compression 10.0 , and when used  the code below for  test:
python OpenDVC_test_video.py --path BasketballPass --model PSNR  --metric PSNR --l 1024
the information is :

"cudnn  poolforward launch failed"
does anyone has good idea to solve it?

Hi, What kind of GPU are you using?-------- 原始邮件 --------发件人： weihua04<EMAIL_ADDRESS>2020年9月18日周五 08:12收件人： RenYang-home/OpenDVC<EMAIL_ADDRESS>Subscribed<EMAIL_ADDRESS>  题： [RenYang-home/OpenDVC] InternalError:cudnn poolforward launch failed (#5)
i have installed tf-gpu 12.0,   cuda9,  tf-compression 10.0 , and when used  the code below for  test:
python OpenDVC_test_video.py --path BasketballPass --model PSNR  --metric PSNR --l 1024
the information is :
![Uploading<PHONE_NUMBER>368395475.jpg…]
does anyone has good idea to solve it?
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Hi, What kind of GPU are you using?-------- 原始邮件 --------发件人： weihua04<EMAIL_ADDRESS>2020年9月18日周五 08:12收件人： RenYang-home/OpenDVC<EMAIL_ADDRESS>Subscribed<EMAIL_ADDRESS>  题： [RenYang-home/OpenDVC] InternalError:cudnn poolforward launch failed (#5)
i have installed tf-gpu 12.0,   cuda9,  tf-compression 10.0 , and when used  the code below for  test:
python OpenDVC_test_video.py --path BasketballPass --model PSNR  --metric PSNR --l 1024
the information is :
![Uploading<PHONE_NUMBER>368395475.jpg…]
does anyone has good idea to solve it?
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

GeForce RTX 2080

I am not sure whether it is because of insufficient memory. Is there an
error if running on CPU?
On Fri, 18 Sep 2020 at 09:07, weihua04<EMAIL_ADDRESS>wrote:

Hi, What kind of GPU are you using?-------- 原始邮件 --------发件人： weihua04
<EMAIL_ADDRESS>2020年9月18日周五 08:12收件人： RenYang-home/OpenDVC
<EMAIL_ADDRESS>Subscribed<EMAIL_ADDRESS>题： [RenYang-home/OpenDVC] InternalError:cudnn poolforward launch failed (
#5 https://github.com/RenYang-home/OpenDVC/issues/5)
i have installed tf-gpu 12.0, cuda9, tf-compression 10.0 , and when used
the code below for test:
python OpenDVC_test_video.py --path BasketballPass --model PSNR --metric
PSNR --l 1024
the information is :
![Uploading<PHONE_NUMBER>368395475.jpg…]
does anyone has good idea to solve it?
—You are receiving this because you are subscribed to this thread.Reply to
this email directly, view it on GitHub, or unsubscribe.
GeForce RTX 2080
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/RenYang-home/OpenDVC/issues/5#issuecomment-694697427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AES6XUFLODER3SDE5LHPUBLSGMBL5ANCNFSM4RRPXXTA
.

--
Ren Yang
Doctoral student
Computer Vision Laboratory | ETH Zürich
Sternwartstrasse 7 | ETF D 114.2 | 8092 Zürich, Switzerland

I am not sure whether it is because of insufficient memory. Is there an
error if running on CPU?
…

CPU is ok, I have tried tf 12.0, but tf-gpu12.0 have problem

Maybe try on a GPU with larger memory (e.g., 12GB or more).
I haven't met that problem, so I am not sure what's wrong it is.

Maybe try on a GPU with larger memory (e.g., 12GB or more).
I haven't met that problem, so I am not sure what's wrong it is.

you run the test.py in docker or the host machine?

I run on the host machine, not on docker-------- 原始邮件 --------发件人： weihua04<EMAIL_ADDRESS>2020年9月18日周五 11:59收件人： RenYang-home/OpenDVC<EMAIL_ADDRESS>Ren Yang<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>  题： Re: [RenYang-home/OpenDVC] InternalError:cudnn poolforward launch failed (#5)
Maybe try on a GPU with larger memory (e.g., 12GB or more).
I haven't met that problem, so I am not sure what's wrong it is.
you run the test.py in docker or the host machine?
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.

thanks for reply. I think maybe it's the version problem, Could u tell me what's your minor version of CUDNN CUDA, and tensorflow?  thank u very much.
mine:
CUDA: 9.0.176_384.81
CUDNN:  V7.1.4
tensorflow: 1.12.0

I meet the same problem.

thanks for reply. I think maybe it's the version problem, Could u tell me what's your minor version of CUDNN CUDA, and tensorflow? thank u very much.
mine:
CUDA: 9.0.176_384.81
CUDNN: V7.1.4
tensorflow: 1.12.0

Mine is:
CUDA:  9.0, V9.0.176
CUDNN:
#define CUDNN_MAJOR 7
#define CUDNN_MINOR 6
#define CUDNN_PATCHLEVEL 0
#define CUDNN_VERSION (CUDNN_MAJOR * 1000 + CUDNN_MINOR * 100 + CUDNN_PATCHLEVEL)
tensorflow: 1.12.0

I meet the same problem.

I solved  the problem by change  version of cudnn，my environment is in docker ,
the environment of docker is :
CUDA: 9
CUDNN: 10.0 V7.4.2

I meet the same problem.

I solved the problem by change version of cudnn，my environment is in docker ,
the environment of docker is :
CUDA: 9
CUDNN: 10.0 V7.4.2

Happy to hear that :-)


Create additional bind functions

https://github.com/kjblanchard/lua_c_dash/blob/b27dd13ed6ba3b03ddf538a5e0dfc8978c591e65/scripts/controller.lua#L37-L44

This will stay in place, but create higher level bind functions so that it is easier to bind functions.


Web Application template has 15 analyzer issues (3.1)

Creating a new project from the 3.1 Web Application template results in 15 analyzer warnings and errors in the Dependencies section of Solution Explorer. Configuration shown below (HTTPS, no Docker, no auth/identity), I didn't check other templates/configurations but I imagine results would be similar. Double-clicking them doesn't lead anywhere (I don't know for sure that would work) so I'm guessing it's something in the SDK rather than the template code itself.
VS 2019 16.4.1

Expand for dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   3.1.100
 Commit:    cd82f021f4
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.18362
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.100\
Host (useful for support):
Version: 3.1.0
Commit:  65f04fb6db
.NET Core SDKs installed:
1.1.10 [C:\Program Files\dotnet\sdk]
3.0.100 [C:\Program Files\dotnet\sdk]
3.1.100 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.14 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.14 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.0.0-preview7.19365.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 1.0.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 1.1.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.14 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.0.0-preview7-27912-14 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.0.0-preview7-27912-14 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

That's the list of the analyzers \ code fixes being run on a project. Do you see errors on warning in the Error list as a result of running these analyzer rules?

I wondered if it might be, but I don't remember the yellow triangle showing, which seems to indicate something is wrong. (So perhaps my report focused on the wrong thing...)

I see the yellow triangle if I have NuGet restore in VS disabled:

Do you happen to have it turned off in your VS too?

@pranavkm I almost always have the yellow triangles showing on all the analyzers, and I have all the Nuget stuff set to default options (Restore is certainly enabled).
I have a vague suspicion that when I create a new project, the analysers have 'nice' names (not just file paths) and no exclamations.  After a while they change to showing a file path with a yellow triangle.  That's just a feeling though.
I know this is the wrong place to complain about VS in, but it's maddening that VS can sprinkle yellow triangles all over the solution tree without any explanation of what problem they're supposed to be indicating.   It should be impossible/forbidden to add a yellow triangle without also having to supply some text somewhere to say what it's supposed to mean.
Even just a hover tooltip would be a good start.

Package restore is on. I don't think I've ever had a reason to disable it, although I have a LOT of package sources (local drive, network drives, in-house servers, and public NuGet ... maybe 20 or so).

Sounds like this is a VS specific issue and not an issue with the analyzers we ship. This would be better handled as a VS feedback issue. Using the feedback tool - https://docs.microsoft.com/en-us/visualstudio/ide/how-to-report-a-problem-with-visual-studio - would include additional diagnostics that should help the relevant VS team to diagnose this.

I face the above problem when I open a solution from a recent project and folder context menu and especially when a project or solution created by Rider but when I open a solution from double click then analyzer load correctly and no more analyzer issue.


Update dependencies

This updates dependencies and removes term crate since it's not used.
Fixes #201
Fixes #224

Is it possible to merge this so that the tiny-http dependency can be updated, and then cut a new release?  0.7 is a big win over 0.6, and being stuck on tiny-http 0.6 means that crates have to carry [patch] sections around, which is not great.

multipart and base64 can be updated to 0.18 and 0.13 respectively without any changes to rouille, just tried it in a local fork. This removes dependency on version_check and other very old dependencies

@MightyPork I was able to update multipart, time, base64 and rand and those are now available in 3.2.1
Postgres is more difficult as anything above 0.15 breaks our 1.41.1 test suite and since it's a dev-dependency I don't think it's as big a deal.
@froydnj tiny-http has been up at 0.8 since Rouille 3.1, hope that helps tidy up your [patch] sections!

Excellent, thanks for looking into it!


Error

twitter-contest-js-bot-master>node index.js
Error retrieving blocked users: 400 - {"errors":[{"code":215,"message":"Bad Authentication data."}]}

Hi @Andsoeg . That's the first API call to Twitter that's being done. Bad Authentication data means your auth. data is not valid. See the config.js file


Update pulumi version

@stack72 any idea what the release process is for kube2pulumi?
Fixes: https://github.com/pulumi/kube2pulumi/issues/58

@komalali just cut a tag (without the sdk) and the magic will happen - https://github.com/pulumi/kube2pulumi/blob/master/.github/workflows/release.yml#L4

@komalali just cut a tag (without the sdk) and the magic will happen - https://github.com/pulumi/kube2pulumi/blob/master/.github/workflows/release.yml#L4


Broken after Loxone update to release version <IP_ADDRESS>

Loxone changed their auth scheme: https://www.loxone.com/dede/wp-content/uploads/sites/2/2016/08/0900_Communicating-with-the-Miniserver.pdf
This one is not functional anymore.

Duplicate of https://github.com/alladdin/node-lox-ws-api/issues/10


Server side processing

Hi guys,
We are planning to use this plugin for our production system. Just want to confirm that this plugin supports server-side processing similar to https://datatables.net/examples/data_sources/server_side.html ?
Thanks

Might not...

Thanks for quick reply.
So that means if we would like to load records from server, and add server-side pagination and search ability, it is not possible right now?

Actually I haven't tried yet
You can have a try

I said 'might not' since this plugin relies on jQuery etc

Ic, ok.

server-side rendering is not necessary for datatable pages, IMHO


[centos 7] Issue with Vim-plug, the theme doesn't work

Hello,
Maybe I've done something wrong, idk.

I installed rust with rustc 1.62.1 (e092d0b6b 2022-07-16), (start new session to reload zsh)
then I add this line Plug 'shaunsingh/oxocarbon.nvim', { 'do': './install.sh' } to add the theme with vimplug.
Added this line too colorscheme oxocarbon
Install plug ; everything is ok on screen
But nothing is happening. No theme loaded.

I tried to start ./install.sh and I have this error:

The pictures shows that « target/release/liboxocarbon.so » and « lua/oxocarbon.so » targeted the same file
I'm not sure what to do next ? Or if I did something wrong during the first steps
Thanks for your help :)
have a nice day

Go to directory ~/.local/share/nvim/plugged/oxocarbon.nvim.
Manually delete file  rm lua/oxocarbon.so.
Run ./install.sh to generate file again.
Reopen terminal or neovim gui. Now, it's work.


Changelog: build next stable (aka minor) version

Sorry, but can we not use exactly the same method we use for the master branch but for the stable branch? And use the same method to generate the file name, except increment the minor version number instead of the major one.

Sure, but I am not sure whether this is useful because at the moment the manual text changelog files are only intended for major releases, so the page will only included a list of bugs from Bugzilla, which might introduce "semi-random" changes in the DAutoTest output.
OTOH it's nice to have a preview for this as well and we already have this Bugzilla bug problem for the generated major version.
Hence leaving this up to @MartinNowak

It seems like this isn't important to @MartinNowak, so I am closing this ;-)


[MRXN23-531]: invalidates cost-surface fetch a second later of creation

Substitute this line for a meaningful title for your changes
Overview
This PR delays the query invalidation of cost surfaces up to one second so it avoids falling in limbo when the API processes the cost surface between the POST callback and the first /status polling. If the API takes more than that second into processing the cost-surface, the /status will in charge of telling the application when the query invalidation should occur.
Designs
Link to the related design prototypes (if applicable)
Testing instructions
Please explain how to test the PR: ID of a dataset, steps to reach the feature, etc.
Feature relevant tickets
https://vizzuality.atlassian.net/browse/MRXN23-531

Checklist before submitting

[ ] Meaningful commits and code rebased on develop.
[ ] If this PR adds feature that should be tested for regressions when
deploying to staging/production, please add brief testing instructions
to the deploy checklist (docs/deployment-checklist.md)
[ ] Update CHANGELOG file

@andresgnlez thanks for all your patient explanations off-band today.
All looks good to me, and I can reliably get cost surfaces to display once the async upload job has completed, with this change applied.
What puzzles me a bit is that:

on marxan23.vercel.app (so, without this change applied until now), I can't trigger the behaviour whereby a cost surface is not added to the sidebar and legend once its async upload job is done, even with super-simple grids where the limbo situation will very often happen - basically there the undesirable behaviour doesn't seem to happen already 🤷🏼
using the same develop branch as marxan23.vercel.app but running it locally, and with the marxan23 API as backend, I cannot trigger the undesirable behaviour either 🤷🏼
same as above (develop frontend running locally) but using api.marxanplanning.org as backend, I can consistently trigger the undesirable behaviour

So I guess there's some PEBKAC somewhere on my end 🤷🏼 but nevertheless, since local app with this PR's branch and api.marxanplanning.org as backend always avoids the undesirable behaviour, I'd say that we can consider this fixed. Why on earth a different backend would make any difference is 🤯 to me, but I also think that trying to understand this at this stage is not a battle worth fighting. In case this issue raises its head again somehow, then hopefully we'll remember what I observed while checking this PR in case that could help to shed any light. But hopefully we won't need any of this once merging this PR - which I'm going to do in a few moments.
thanks @andresgnlez!


Cannot run Tasklist or Taskkill in Nanoserver 2022 in Docker

Newbie question here. I'd like to debug a process running in a Nanoserver Docker container (I'm running nanoserver:ltsc2022). But to do that I need to know the process ID to attach to and Tasklist.exe is not available. Other than copying Tasklist.exe and any necessary DLLs in from outside, are there any native options? Similarly, if I'd like to kill a task with Taskkill, what can I do? I noticed here  that I there's a Nanoserver 1809 instance with PowerShell, is that my best option? Are there any others? I'm hoping I've overlooked some useful CMD command. Already tried netstat -aon, but my process isn't listening on a port. Thanks!

Hey @Sudoite. I'm going to assume you're running a process isolated container, and you're trying to get the process id of the process so the debugger can connect to it.
The easiest option would be to run the debugger from the host. You can:

launch taskmgr
go to the details tab
right click on of the column headers to select a new column to be displayed (the session id column)
Then I sort by session id.

If you only have one container running, you can find the session your container is running in by looking for the CExecSvc.exe process. Whatever session ID it is in, the process should be in the same session. Now you can find the process id for the process, and launch the debugger on the host and connect to the process.

Hi @fady-azmy-msft , thanks so much! Yes you are right, and I got that to work for me. I can also infer from your response that I can similarly kill the process from outside the Docker container if needed. And indeed that is true. Awesome! I'm a happy camper. I can also infer that I should be able to use Powershell from the host to automate the process of killing troublesome processes in the Docker container. I'll post another question if I can't figure it out. Super helpful answer, thank you.


Suggesting to include an architecture diagram

Before you describe your issue, idea or feature, please consider using this issue template.
Type of issue:

[ ] Bug / Error
[ ] Idea / Feature
[x] Improvement detail

Short description on the issue
In the readme, under architecture section, better to add an architecture diagram rather than explaining the architecture in words
Possible fix
Adding an architecture diagram to the readme

@psnmissaka how about this high level diagram? Any alternations to do?

LGTM. Open a PR with this

#117 @rehrumesh Opened a PR


A.torch_matvec() changes the shape of tensors

When A is a KFACLinearOperator or KFACInverseLinearOperator for model parameters given by a list of tensors, the expected behaviour when applying A to a list of tensors of the same shape would be that it returns a list of the same shape. This is not the case:
params = [torch.zeros(5,4), torch.zeros(5, 3, 3, 3)]
A = KFACLinearOperator(
    ...
    params,
    ...
)
v = [torch.randn_like(p) for p in params]
y = A.torch_matvec(v)
print([y_elem.shape for y in y_elem])
# Expected output: same shapes as `v` and `params`:
>>> [(5, 4), (5, 3, 3, 3)]
# Actual output: flattened to a 2D tensor:
>>> [(5, 4), (5, 27)]

Hi,
that's indead weird. KFACLinearOperator expects params to be used by whatever you supply as model_func argument. What neural network are you using? Anyways this seems like the checks on params are not sufficient to catch this.

Yup, exactly! This was just an illustrative example, I noticed this in a larger resnet where every parameter ended up being flattened to at most 2D.

Could you set up a PR to fix this? I guess you already have a small example to reproduce this, which we could use as test in the future.

Yeah, I could have a look. I'll tag @runame because he might have already looked into this.

Sorry about the delay, I know where the bug is, just haven't fixed it yet. Will create a PR later today.


will support webUI?

Thanks for your comments.
We have to find out how compatible our project is with Gradio.
This may take some time, but I will share it with the developer team.

Thanks for your comments.
We have to find out how compatible our project is with Gradio. This may take some time, but I will share it with the developer team.

Thanks for your comments.
We have to find out how compatible our project is with Gradio. This may take some time, but I will share it with the developer team.

in general, it would not be bad to write a plugin for Sd automatic and comfyUI?

This is a PURELY system paper instead of ML paper, which means that it would be non-trivial to support it in any variation of WebUI. Also I believe that we almost certainly need to build a new WebUI (certainly with A1111 backend) and get rid of Gradio, to really have a good real time experience.


(AzureCXP) Adding note regarding Alternate IDs

Adding note regarding Alternate IDs  reference git : https://github.com/MicrosoftDocs/azure-docs/issues/36404

@FrankHu-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@jborsecnik : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Feat/regex options

Summary
Add dotAll and sticky regex flags.
Test plan
Added automated tests

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

Project coverage is 100.00%. Comparing base (49e8aa4) to head (97eb4f1).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #84   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           16        16
  Lines          190       192    +2
  Branches        42        44    +2
=========================================
+ Hits           190       192    +2

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


STYLE: Class derived from TransformBase need explicit instantiation

Move definition code from hxx to cxx file.
These classes must be explicitly
instantiated for each valid template type to support
dynamic casting robustly (one-definiton-rule).
As a consequence, the only place that the definitions
for these class member functions is needed is in the
cxx compiled files.

Total Test time (real) = 102.89 sec
	Add file: /home/circleci/ITK/.circleci/config.yml
	Add file: /home/circleci/ITK-dashboard/circleci.cmake
	Add file: /home/circleci/ITK-dashboard/itk_common.cmake
Submit files (using http)
   Using HTTP submit method
   Drop site:http://open.cdash.org/submit.php?project=Insight
   Error when uploading file: /home/circleci/ITK-build/Testing/20190305-2049/Build.xml
   Error message was: Failed to connect to open.cdash.org port 80: Connection timed out
   Problems when submitting via HTTP
Exited with code 255


Bayer images not supported

Environment

OS Version:Ubuntu 22.04
Ignition Gazebo, version 6.11.0
ROS Humble

Description
I am trying to simulate bayer images using the BAYER_BGGR8 format.
<camera>
  <image>
    <height>${camera_height}</height>
    <width>${camera_width}</width>
    <format>BAYER_BGGR8</format>
  </image>
</camera>

It seems the bayer format is missing from the switch case here:  https://github.com/gazebosim/gz-sensors/blob/8e8dfa84371d865f54223aa9585e8b255c0b6605/src/CameraSensor.cc#L210-L226
This results in the following error:
[ruby $(which ign) gazebo-1] [Err] [CameraSensor.cc:221] Unsupported pixel format [16]

not supported yet in gz-rendering. One option is to port it from gazebo-classic: https://github.com/gazebosim/gazebo-classic/blob/gazebo11/gazebo/rendering/Camera.cc#L1381

Hey @iche033,  I would like to work on this issue and have been working on implementing an example for a custom rendering sensor.
I had quite a few clarifications to make about what I understood till now about the way rendering sensors present in gz-sensors use the gz-rendering library with ogre.
I apologize for asking such a silly question, but I am just getting started with understanding the implementation of rendering sensors and would like to confirm it so I do not make a mistake.
To find where exactly in gz-rendering I need to make changes to add bayer conversion function, I explored the codebase.
I started with the normal camera sensor to get an initial understanding of the implementation of rendering sensors. Here's what I understood from the exploration:

Camera sensor ("gz/sensors/CameraSensor.hh"): This is used to load the specific inputs given by the user from SDF elements into an instance of camera rendering sensor(present in gz-rendering), create an instance for a scene which updates the scene graph, calls appropriate methods for camera rendering sesnor to generate data and publishes image data on the protobuf image msg-type

Camera rendering sensor ("gz/rendering/Camera.hh" and gz/rendering/base/BaseCamera.hh): This acts as connecting link between rendering engine generated scene graph and gz-sensors camera. This uses a pointer instance of the RenderTarget class to write the generated rendered image into a given image buffer and to define methods such as PostRender and Update.

BaseRenderTarget("gz/rendering/RenderTarget.hh" and "gz/rendering/base/BaseRenderTarget.hh"): This declares a class to represent RenderTarget, RenderTexture, and RenderWindow.
This class has definitions for pre-rendering and post-rendering procedures.

ogreRenderTarget: This defines inherited classes of the above BaseRenderTarget class, BaseRenderTexture class and BaseRenderWindow class. This is where Ogre::RenderTarget is actually called to render the scene. It overrides the Copy method(OgreRenderTarget::Copy) that actually converts the underlying image to the specified format listed in the given Image using the ogrePixelBox function. I see that this function is also called under the OgreRenderTexture::Buffer method.
Henceforth, I concluded that after ogrePixelBox is called, there could make a call to another function that converts rgb pixels to bayer pixels, which would be either in OgreRenderTarget::Copy or OgreRenderTexture::Buffer. I also found a comment stated under OgreRenderTarget::Copy for Bayer conversions to be implemented.

Here’s what I wanted to ask:

Am I headed in the right direction to infer the above ideas behind the implementation?
Further, I couldn't find where the ``OgreRenderTarget::CopyorOgreRenderTexture::Buffer` methods are called and where it actually converts the image format to, say, RGB to grayscale when simulation starts in gazebo. Could you please help me with that?
I also found some OgreSensors like OgreCamera and OgreDepthCamera. What are they used for?

thanks for working on this! Let me answer your questions below:

yes your understanding is correct. One thing to note is that by default gazebo launches with the ogre2 render engine. Nonetheless, we can come up with a generic solution that can be re-used by different render engines.
OgreRenderTarget::Copy is called by BaseCamera::Copy which is in turn called by CameraSensor here https://github.com/gazebosim/gz-sensors/blob/gz-sensors7/src/CameraSensor.cc#L577. As for converting from RGB to grayscale, the underlying render engine does that for us. As an example, in ogre render engine when we create a render texture, we give it an image format here (ogreFormat).
BaseCamera as its name suggests is the base class for camera. You will find that its functions are virtual and overriden by derived classes such as OgreCamera and Ogre2Camera which is doing most of the work. OgreDepthCamera renders depth data stored in floating point image data.

So going back to Bayer conversion. I think one implementation option would be to create a generic function convertRGBToBayer, e.g. in Utils.cc and call that in OgreRenderTarget::Copy / Ogre2RenderTarget::Copy.
One thing to watch out for is that if user specifies a format like BAYER_RGGB8, currently gz-rendering does not know about this format, so the image format conversion functions like here will not work and we will need to catch that. We can just create RGB render texture in the underlying render engine and do Bayer conversions when we read back the values in Copy.

Also, shall I create a draft PR with what I have implemented so far?

Nice!

Presently, to display the image with Image Display Plugin under gz-gui, I have added the following snippet inside a switch statement under ImageDisplay::ProcessImage() in ImageDisplay.cc.

That's sounds good

Further, I found that under PixelUtil::channelCounts[PF_COUNT], the no of channels assigned to the bayer formats are 4. I understand that each channel corresponds to one color among RGGB. Could you please help me understand if four 8-bit channels are required, or could we work with just a single 8-bit channel?

oh good catch, I think we should be consistent with gazebo-classic and change it to 8 bit.

I couldn't find image.format() being used with the Optix engine. Does it use image format? Though from its OptixRenderTarget::Copy(Image &_image) function, it seems to create a 3-channel image(might be RGB).

Let's skip optix implementation. The code is outdated.

Also, shall I create a draft PR in each affecting repository with what I have implemented so far since there are files in different repositories that would require some small additions/ modifications?

Sure, I can take a look at the PRS once they are up. Something to watch out for is that If you had to modify enums, it could break ABI and the changes would need to be targeted at main branch.

Hey @iche033, I have opened PRs in four repositories. Here's a summary of the same.
The functionality reads the user input and renders an RGB image, which is later converted into a single channel 8bit Bayer image using ConvertRGBToBayer() added to Utils.cc inside gz-rendering.

gz-sensors : Adds a switch case for RGGB bayer format inside CameraSensor.cc and passes the R8G8B8 format to render the image.
gz-redering : Adds ConvertRGBToBayer() to Utils.cc, modifies OgreRenderTarget.cc to call the conversion function, and handles image format conversion functions with if-else statements.
gz-gui : Adds switch cases to display Bayer images in Gazebo GUI inside. It treats them as single-channel 8-bit images.
gz-common : Modifies Image::SetFromData inside Image.cc to support saving of Bayer images. In order to save it It treats them as single-channel 8-bit images.

I also had a few points to discuss with you:

I found that there is a little discrepancy in the 2 BAYER formats (GRBG and GBRG) specified at two different places:PixelFormat.hh in gz-rendering and Image.hh in gz-common.
I have implemented it with GRBG and GBRG which were also the formats used in gazebo-classic. These Bayer formats would require changing enum elements inside PixelFormat.hh.
I also tried converting the no of channels assigned to the bayer formats inside PixelUtil::channelCounts[PF_COUNT] from 4 to 1.
Upon modifying, I received error double linked list corrupted. I am still working on debugging it.

thanks for the PRs!

These Bayer formats would require changing enum elements inside PixelFormat.hh. Could you please confirm if I should go ahead with GRBG and GBRG instead of GRGB and GBGR?

hmm that's a difficult one as we usually we don't change enum values in a released version. On the other hand, no code is using that particular enum (since it never worked). Can you try make the change so that it is consistent with gazebo-classic. That'll trigger a new ABI build, and if the build show that ABI compatibility fails, we would need to think of an workaround.

I also tried converting the no of channels assigned to the bayer formats inside PixelUtil::channelCounts[PF_COUNT] from 4 to 1.
Upon modifying, I received error double linked list corrupted. I am still working on debugging it.

I tried changing them from 4 to 1 in you branch but didn't run into the crash. If it helps, you can debug gz with gdb with instructions from this page:
https://gazebosim.org/api/gazebo/7.0/debugging.html
You'll just need to replace ign with gz.

Hey, @iche033, I am done with adding support for bayer formats to ogre and ogre2, along with all the modifications required in visualization plugins. Sorry, I was busy with some of my semester exams last week and therefore couldn't complete it.
The only thing that needs to be catered is the no of channels assigned to the bayer formats inside PixelUtil::channelCounts[PF_COUNT]. I tried changing it to 1 or 2, but the program crashed. Interestingly it doesn't crash if I keep it 3.

thanks for iterating! Were you get to get a backtrace or find out where the crash is coming from?

@iche033, Yup it works perfectly! Thank you!
I am writing to request your help in understanding how you were able to debug the segmentation fault issue in Gazebo that I was struggling with. I could only reach the point where I found that segmentation fault when trying to update entities.

I would be grateful if you could explain to me how you approached the debugging process and any techniques or tools that you used. Specifically, I am interested in understanding how you made breakpoints and used Valgrind (incase you used it) on an SDF file. As I am not very familiar with using a debugger, I would greatly appreciate any help you offer.

I didn't have a quick and easy way for debugging this one. Backtrace wasn't that useful but I know it's a memory corruption issue since we're change the number of bytes. So I started looking at places that could be impacted by the change. The more obvious one is here when an Image object is created. A Bayer image now has a a much smaller but correct size. Then I started taking a look at the new Bayer conversion function and how it's used. That's when I noticed the bug. We're passing an image of size h*w to the conversion function that expects an RGB image. Hope that helps


feat: add sachaos/go-life

#5699 sachaos/go-life: Terminal based Conway's Game of Life. Implemented in Go
$ aqua g -i sachaos/go-life

Thanks.

Test failed, but I don't know how to avoid adding .exe suffix.

complete_windows_ext: false

https://aquaproj.github.io/docs/reference/registry-config/complete-windows-ext

Thanks!

Released https://github.com/aquaproj/aqua-registry/releases/tag/v3.37.0


Alexa discovery

Hi
I have the bridge running and seems to be available in the network as the hue app so discovery seems to be on the bridge side.  I'm running 3.2.
I'm not sure of what logs I should be attaching so switched on a few.  Would really appreciate some guidance on how to get this working:
12-29-2016 22:31:13.248	INFO	Traceupnp: sendUpnpResponse discovery responseTemplate1 is <<<HTTP/1.1 200 OK
HOST: <IP_ADDRESS>:1900
CACHE-CONTROL: max-age=100
EXT:
LOCATION: http://<IP_ADDRESS>:8085/description.xml
SERVER: Linux/3.14.0 UPnP/1.0 IpBridge/1.15.0
hue-bridgeid: 000088FFFE00BBEE
ST: upnp:rootdevice
USN: uuid:2f402f80-da50-11e1-9b23-00008800bbee::upnp:rootdevice

com.bwssystems.HABridge.upnp.UpnpListener
12-29-2016 22:31:13.248	INFO	Traceupnp: sendUpnpResponse discovery responseTemplate2 is <<<HTTP/1.1 200 OK
HOST: <IP_ADDRESS>:1900
CACHE-CONTROL: max-age=100
EXT:
LOCATION: http://<IP_ADDRESS>:8085/description.xml
SERVER: Linux/3.14.0 UPnP/1.0 IpBridge/1.15.0
hue-bridgeid: 000088FFFE00BBEE
ST: uuid:2f402f80-da50-11e1-9b23-00008800bbee
USN: uuid:2f402f80-da50-11e1-9b23-00008800bbee

com.bwssystems.HABridge.upnp.UpnpListener
12-29-2016 22:31:13.255	INFO	Traceupnp: sendUpnpResponse discovery responseTemplate3 is <<<HTTP/1.1 200 OK
HOST: <IP_ADDRESS>:1900
CACHE-CONTROL: max-age=100
EXT:
LOCATION: http://<IP_ADDRESS>:8085/description.xml
SERVER: Linux/3.14.0 UPnP/1.0 IpBridge/1.15.0
hue-bridgeid: 000088FFFE00BBEE
ST: urn:schemas-upnp-org:device:basic:1
USN: uuid:2f402f80-da50-11e1-9b23-00008800bbee

com.bwssystems.HABridge.upnp.UpnpListener
12-29-2016 22:31:13.255	INFO	Traceupnp: isSSDPDiscovery found message to be an M-SEARCH message.	com.bwssystems.HABridge.upnp.UpnpListener
12-29-2016 22:31:13.255	INFO	Traceupnp: isSSDPDiscovery found message to be valid under strict rules - strict: true	com.bwssystems.HABridge.upnp.UpnpListener
12-29-2016 22:31:13.255	INFO	Traceupnp: SSDP packet from <IP_ADDRESS>:56264, body: M-SEARCH * HTTP/1.1
Host: <IP_ADDRESS>:1900
Man: "ssdp:discover"
ST: ssdp:all
MX: 5

com.bwssystems.HABridge.upnp.UpnpListener

12-29-2016 22:31:13.422	INFO	Traceupnp: sendUpnpResponse discovery responseTemplate1 is <<<HTTP/1.1 200 OK
HOST: <IP_ADDRESS>:1900
CACHE-CONTROL: max-age=100
EXT:
LOCATION: http://<IP_ADDRESS>:8085/description.xml
SERVER: Linux/3.14.0 UPnP/1.0 IpBridge/1.15.0
hue-bridgeid: 000088FFFE00BBEE
ST: upnp:rootdevice
USN: uuid:2f402f80-da50-11e1-9b23-00008800bbee::upnp:rootdevice

com.bwssystems.HABridge.upnp.UpnpListener
12-29-2016 22:31:13.424	INFO	Traceupnp: sendUpnpResponse discovery responseTemplate2 is <<<HTTP/1.1 200 OK
HOST: <IP_ADDRESS>:1900
CACHE-CONTROL: max-age=100
EXT:
LOCATION: http://<IP_ADDRESS>:8085/description.xml
SERVER: Linux/3.14.0 UPnP/1.0 IpBridge/1.15.0
hue-bridgeid: 000088FFFE00BBEE
ST: uuid:2f402f80-da50-11e1-9b23-00008800bbee
USN: uuid:2f402f80-da50-11e1-9b23-00008800bbee

com.bwssystems.HABridge.upnp.UpnpListener
12-29-2016 22:31:13.427	INFO	Traceupnp: sendUpnpResponse discovery responseTemplate3 is <<<HTTP/1.1 200 OK
HOST: <IP_ADDRESS>:1900
CACHE-CONTROL: max-age=100
EXT:
LOCATION: http://<IP_ADDRESS>:8085/description.xml
SERVER: Linux/3.14.0 UPnP/1.0 IpBridge/1.15.0
hue-bridgeid: 000088FFFE00BBEE
ST: urn:schemas-upnp-org:device:basic:1
USN: uuid:2f402f80-da50-11e1-9b23-00008800bbee

For one, download the latest version and use that. and Two, search the issues that are closed in this list, most all questions have been answered.

Any update on this?

I haven't had an opportunity to troubleshoot today but I'll be back on it shortly and will try and search for some of the prior issues to see if there is anything I've missed....

All resolved.  I had changed my IP address but I had changed it to the IP address of my rm tasker bridge rather than the ip address the ha bridge was working on #idiot!  Anyway, gotta say this is a cracking little solution and will help to get around the current 16 device limit on the RM Tasker app (or at least I hope it will!).


Fixed Column and Row

Hello,
is there anyway to get the first column and first row fixed?
Thank you

Hi @AlbertAssaad
I think your issue is related to #72. You can check @ISchwarz23's reply on that issue.

Closed because this is a duplicate of #72


Fix doc path.

Codecov Report

Merging #7 (1f7b5bb) into main (cdd7abe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   63.38%   63.38%
=======================================
  Files          17       17
  Lines        2395     2395
=======================================
  Hits         1518     1518
  Misses        877      877

Continue to review full report at Codecov.

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


Feature/git annex remote google

Motivation for this change
add git-annex-remote-googledrive. This remote has more features then the rclone remote and allows you to export a repository to google drive.
Things done

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

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

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

@GrahamcOfBorg build gitAndTools.git-annex-remote-googledrive python27Packages.annexremote python27Packages.pydrive python27Packages.tenacity python36Packages.annexremote python36Packages.pydrive python36Packages.tenacity

Failure on x86_64-darwin (full log)
Attempted: gitAndTools.git-annex-remote-googledrive, python27Packages.annexremote, python27Packages.pydrive, python27Packages.tenacity, python36Packages.annexremote, python36Packages.pydrive, python36Packages.tenacity
Partial log (click to expand)
reading manifest file 'git_annex_remote_googledrive.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'git_annex_remote_googledrive.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
error: build of '/nix/store/cb1l4nglrwn75a39b93si8zy7wyc3sxn-python2.7-tenacity-5.0.2.drv' failed

Failure on x86_64-linux (full log)
Attempted: gitAndTools.git-annex-remote-googledrive, python27Packages.annexremote, python27Packages.pydrive, python27Packages.tenacity, python36Packages.annexremote, python36Packages.pydrive, python36Packages.tenacity
Partial log (click to expand)
reading manifest file 'git_annex_remote_googledrive.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'git_annex_remote_googledrive.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
error: build of '/nix/store/291qgryh7ydbjsgyx3wlxr5090vb3cbj-python2.7-tenacity-5.0.2.drv' failed

Failure on aarch64-linux (full log)
Attempted: gitAndTools.git-annex-remote-googledrive, python27Packages.annexremote, python27Packages.pydrive, python27Packages.tenacity, python36Packages.annexremote, python36Packages.pydrive, python36Packages.tenacity
Partial log (click to expand)
reading manifest file 'git_annex_remote_googledrive.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'git_annex_remote_googledrive.egg-info/SOURCES.txt'
running build_ext

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
error: build of '/nix/store/1zlfj2iz218g6drjndd9xchvci4gf8xm-python2.7-tenacity-5.0.2.drv' failed

this should be fixed:
Processing ./tenacity-5.0.2-py2.py3-none-any.whl
Collecting monotonic>=0.6; python_version == "2.7" (from tenacity==5.0.2)
  Could not find a version that satisfies the requirement monotonic>=0.6; python_version == "2.7" (from tenacity==5.0.2) (from versions: )
No matching distribution found for monotonic>=0.6; python_version == "2.7" (from tenacity==5.0.

Any updates on this pull request, please?

Has been added in pkgs/applications/version-management/git-and-tools/git-annex-remote-googledrive/default.nix, see also linked PR.


Fix live reloader

Compile and watch dev.exs so it can be live reloaded.
~Note that page/hello.html.heex is not live reloading neither compiling, ie: run mix dev, change that file, and I don't see the changes even if I reload the server. Need to debug a bit more to figure out the issue.~ Nevermind... that template is actually duplicated on both dev.exs and hello.html.heex (https://github.com/mcrumm/phoenix_profiler/pull/45/commits/fea1369e0115c621063a12ca5dcb365dea3e201c)

I think we want to avoid adding dev.exs to the compiled paths, and afaik it will not exactly play nicely with the code reloader either (fwiw I cribbed the dev setup from LiveDashboard).
I do not expect the dev templates to change frequently, so I would rather leave it out for now. :)


Iteration Not Working

Thanks for this module.  It is just what I need for text highlighting, I think.  Unfortunately, I can't get it to iterate.  My code is nearly identical to the code in the docs.  I never see any console messages (unless I put one in prior to the for loop):

Any ideas on what I might be missing or doing incorrectly?

Sorry, just saw this. Hard to tell what's wrong from the given code sample; it looks like it should work. Perhaps try cloning the repo and seeing if the unit tests run on your machine. If those fail, let me know your OS, JS engine version, etc. Otherwise, next step, try adding a unit test which captures your particular error scenario.


Improve message Access Denied: insufficient view owner privileges

If a user A creates a view and user B wants to access the view and then user A loses access to the original data then B should not be able to access data in the view.
If B gets and access denied, they are told whether it's because they don't have access to the view or if the view creator doesn't have access to the underlying data.
This PR is only about enhancing information in the error message.

@electrum @kokosing all comments are addressed.

I am concerned that after the change, it's harder to know which permissions need to be changed in case of "Access Denied".
Imagine a situation when a BI user gets this error for a query and they turn to their sys admin / data engineer for help. Will sys admin know what permissions would need to be change for the user?
cc @willmostly

@findepi when a BI user gets this error for a query and they turn to their sys admin / data engineer for help. Sys admin will see the very same information they see right now, but in caused by part of the exception.

@findepi when a BI user gets this error for a query and they turn to their sys admin / data engineer for help. Sys admin will see the very same information they see right now, but in caused by part of the exception.

I would not assume BI tools are nice enough to retain full stacktrace, unless someone with more hands on experience tells me I should not be concerned.

@kokosing @electrum all comments are addressed

@kokosing @electrum @findepi all comments are addressed

The very same pull req to the forked master:
https://github.com/zaz968m/presto/pull/1
Some checks were not successful
37 successful and 2 cancelled checks

Unfortunately I don't have permissions to restart jobs manually. I can immitate it only with force push. So I did one.
Before last force push there was the very same diff https://github.com/prestosql/presto/commit/679b861429a37e333d69127ac209fb5628a9d734
with only one failing test https://github.com/prestosql/presto/runs/1134114335
Running command ./mvnw test -B -fae -Dair.check.skip-all -pl presto-raptor-legacy for that test locally I have
[INFO] Results:
[INFO]
[WARNING] Tests run: 641, Failures: 0, Errors: 0, Skipped: 4
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS

restarted the checks

@electrum @findepi @kokosing are there any open questions left that require actions from my side?

are there any open questions left that require actions from my side?

https://github.com/prestosql/presto/pull/5086/commits/9e03d752aea95aff37d303b4c40a981e03649ad6#r490878495

@zaz968m There are legitimate test failures:
2020-09-28T12:41:41.5110410Z tests               | 2020-09-28 18:26:41 INFO: FAILURE     /    io.prestosql.tests.hive.TestSqlStandardAccessControlChecks.testAccessControlCreateView (Groups: authorization, profile_specific_tests) took 0.5 seconds
2020-09-28T12:41:41.5140795Z tests               | 2020-09-28 18:26:41 SEVERE: Failure cause:
2020-09-28T12:41:41.5142271Z tests               | java.lang.AssertionError: Query failed with unexpected error message: 'java.sql.SQLException: Query failed (#20200928_124141_00912_zcajp): Access Denied: View owner does not have sufficient privileges: View owner 'bob' cannot create view that selects from default.alice_owned_table'
2020-09-28T12:41:41.5143874Z tests               |  Expected error message to contain 'Access Denied: View owner 'bob' cannot create view that selects from

Failed with: https://github.com/prestosql/presto/issues/5354 and https://github.com/prestosql/presto/issues/4973

Merged, thanks!


Unable to achieve synchronized behaviour with kivy textfield with alphabets only filter and android keyboard suggestion shown and thus causes discrepancy

Software Versions

Python:3
OS:android
Kivy:2.1.0
Kivy installation method:pip installation

Describe the bug
It seems that when we select keyboard suggested word, it produces the word along with trailing space at the ending side of word. So I modified the substring with strip() function. But there's another issue with it. As per keyboard functionality, It still assumes that the word delivered was with a space and it was messing up.
Let me throw an example. Let us say I type string "tes" and the keyboard suggestions shows "test "(with trailing space at end only) and I touch the suggested word, now the word "test"(without a space) as strip is used  was populated and when I do backspace 4 times the word in text field holds "" but as per keyboard functionality memory it holds "t" (as it still considers space) and shows another suggestion word say "thanks", the text field only receives the text "hanks" as the keyboard considers "t" was already there.
Expected behavior
Reconciliation with keyboard functionality and textfield functionality
To Reproduce
keyboard_textfieldBug.zip
Code and Logs and screenshots
from kivymd.app import MDApp
from kivymd.uix.textfield import MDTextField
from kivymd.uix.floatlayout import MDFloatLayout
class Word(MDTextField):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        self.input_filter = self.letters_only
    def letters_only(self, substring, from_undo):
        if (substring.strip()).isalpha():
            return substring.strip()
class TestTextFieldApp(MDApp):
    def build(self):
        self.root = MDFloatLayout()
        self.word = Word()
        self.word.input_type = "text"
        self.root.add_widget(self.word)
        self.word.pos_hint = {"center_x":0.5,"center_y":0.5}
        self.word.size_hint = (0.75,0.125)
        return self.root
if __name__ == '__main__':
    TestTextFieldApp().run()

**Additional context**
Already had some discussions related to this issue in https://mail.google.com/mail/u/0/#inbox/FMfcgzGrbRQtcvpZwvjkBFQGmkDBclrW with @ElliotGarbus and @RobertFlatt and @misl6 was mentioned by them to have better understanding of this issue.

Please help me resolve this issue :)

@tshirtman @misl6 @RobertFlatt Created an issue to track what can be done as part of conversation : https://groups.google.com/g/kivy-users/c/yQIuLZPf-jg . Can anyone please acknowledge this issue ?

@tshirtman @misl6 @RobertFlatt Created an issue to track what can be done as part of conversation : https://groups.google.com/g/kivy-users/c/yQIuLZPf-jg . Can anyone please acknowledge this issue ?

This is a pretty uncommon way to use the Android IME when set in text mode.(https://developer.android.com/develop/ui/views/touch-and-input/creating-input-method)
Our TextInput should stay in sync with the hidden one on Android side for the IME to work correctly and this is not happening, nor we are exposing a method to re-sync it when (as your example) the developer filters the input and changes it programmatically.

@misl6 Can kivy framework introduce/expose a method or something to sync the behavior ?

It looks like there is no plan to implement this feature, as it is an unsual use-case and there are other solutions.
Closing as Not Planned.
If you think this is in error (particularly if you want to submit a PR), please let us know.


Refactor demo notebook storyline

We should define a recommended best practice of handling branches and tags to best reflect the experiments done in data science projects.
We should then refactor the storyline of the notebook such that it reflects these recommendations. The current storyline is only written with the aim to show features of lakeFS-spec, not applicable best practices.
If possible, we can refactor the storyline in a way such that we do not need a hidden cleanup cell to ensure idempotency of the notebook execution.
Also align the tutorial notebook with the blog post (take good practices from the blog post) and PyData slides good practices.

Now the Repo containing the best practices is here

@maxmynter after installing requirements.txt in a vanilla fresh env I was able to reproduce the notebook smoothly. I also like the story (it is clear and informative).

We have the other Repo and storyline finished, so it sits there if we ever need it (and should handle issues etc. there as well). The PyData presentation is a separate thing.
So I am closing this here


[enzyme-directed rRNA 2'-O-methylation]

Please provide as much information as you can:

GO term ID and Label
enzyme-directed rRNA 2'-O-methylation

Reason for deprecation
molecular function in BP

"Replace by" term (ID and label)
If all annotations can safely be moved to that term

"Consider" term(s) (ID and label)
Suggestions for reannotation

Are there annotations to this term?

How many EXP:
4 EXP (IMP)
https://www.ebi.ac.uk/QuickGO/annotations?goUsage=descendants&goUsageRelationships=is_a,part_of,occurs_in&goId=GO:0000453&evidenceCode=ECO:0000269&evidenceCodeUsage=descendants

Are there mappings and cross references to this term? (InterPro, Keywords; check QuickGO cross-references section)

https://www.uniprot.org/unirule/UR000086060

https://www.uniprot.org/unirule/UR000166071

Is this term in a subset? (check the AmiGO page for that term)

Any other information

[ x] obsoletion notice sent


⚠ SceneTime has degraded performance

In adb176f, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1414 ms

Resolved: SceneTime performance has improved in 35ce8a7 after 6 minutes.


Cursor flickers its state

Describe the bug
If decorations is set to true, the cursor is flickering when hovering over text or elements that have a non-default CSS cursor property set. This behavior only happens in frameless mode and I could only reporduce this on Windows.
To Reproduce
Steps to reproduce the behavior:

Create a new Tauri app using create-tauri-app
Set decorations to true in tauri.conf.json
Run the app with npm run tauri dev
Hover over text
See the cursor having weird behavior

Expected behavior
The text cursor should be applied permanently when hovering over text
Screenshots
https://user-images.githubusercontent.com/15818698/128110525-86c6224c-da0c-4de0-b17b-5544ea053a3c.mp4
Platform and Versions (required):

Operating System - Windows, version 10.0.19042 X64
Webview2 - 92.0.902.62

Node.js environment
  Node.js - 14.17.3
  @tauri-apps/cli - 1.0.0-beta.6
  @tauri-apps/api - 1.0.0-beta.5

Global packages
  npm - 6.14.13
  yarn - 1.22.11

Rust environment
  rustc - 1.54.0
  cargo - 1.54.0

App directory structure
/.git
/build
/node_modules
/public
/src
/src-tauri

App
  tauri.rs - 1.0.0-beta.5
  build-type - bundle
  CSP - default-src blob: data: filesystem: ws: wss: http: https: tauri: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'
  distDir - ../build
  devPath - http://localhost:3000
  framework - React

Additional context
I am using the React template of create-tauri-app, however I believe that it applies to any template.

Set decorations to true in tauri.conf.json

I am assuming you mean decorations: false
This is has been fixed in https://github.com/tauri-apps/wry/pull/350 and will be available in the next release.

Yes, I indeed meant false. Thank you that you have already made a PR and fixed the issue. I unfortunately couldn't find it before making this.

No worries, the bug was reported on discord and I fixed it without making an issue about it so it was kinda hidden.


Github Actions for DSpace

This will be to update our local fork with upstream github actions.

appears to be already created and in use here:
https://github.com/DSpace/DSpace/blob/main/.github/workflows/build.yml
https://github.com/DSpace/DSpace/actions?query=workflow%3ABuild

appears to be already created and in use here:
https://github.com/DSpace/DSpace/blob/main/.github/workflows/build.yml
https://github.com/DSpace/DSpace/actions?query=workflow%3ABuild

The build.yml workflow can either be brought in individually and require merge conflicts to be resolved or we can wait until 6.4 is released and upgrade.


Change Background color.

Are you looking for a new color palette to change the whole project to? Or like Trello has where you can select your own background color on a per-board basis?

This issue is about implementing the per board background color, @yasaricli plan to work on this on Sunday (it's possible that I'll do it tomorrow). Personally I'm also interested in a new palette to distinguish visually from Trello. Maybe http://flatuicolors.com?


Fix #54 to add DevServices for Artemis JMS

Hi @middagj ,
Can you review these changes? and I'd like to include them in a new release if it makes sense.
Thanks!

Sure, it can take a few days

We should be able to also use testcontainers on core integration test. If you have time, could you add that, otherwise I will add it myself. Also we don't need the broker.xml files.

yeah, I will add one in core integration test. And broker.xml is needed if we still use ArtemisTestResource.

yeah, I will add one in core integration test. And broker.xml is needed if we still use ArtemisTestResource.

correct, but only one

Thanks for the PR

Could you give it a test, and confirm it works as expected in your project. I will then release.

Thanks @middagj and yeah I confirm it works in my camel-quarkus -example project. Please go heading for the release.

@middagj I just realized that ArtemisTestResource is useful on windows where docker is not supported currently. So it should be better to remove @Deprecated . I'm sorry for the inconveniences.

I didn't realize you added it, will remove it.

I raise https://github.com/quarkiverse/quarkus-artemis/pull/60


[Custom request] Build in-depth documentation

Replace the main README.md with a proper in-depth documentation hosted on gh-pages.
Bounce: enable versioning

One docs framework that can be used: https://squidfunk.github.io/mkdocs-material/

Tasks:

[x] Bootstrap docs framework
[x]  Initiate gh-pages hosting
[ ]  Fix to out of docs references

WARNING  -  Documentation file 'index.md' contains a link to '../kube/crd/locust-test-crd.yaml' which is not found in the documentation files.
WARNING  -  Documentation file 'index.md' contains a link to '../kube/sample-cr/locust-test-cr.yaml' which is not found in the documentation files.
WARNING  -  Documentation file 'index.md' contains a link to '../CONTRIBUTING.md' which is not found in the documentation files.
WARNING  -  Documentation file 'landing-page.md' contains a link to '../kube/crd/locust-test-crd.yaml' which is not found in the documentation files.
WARNING  -  Documentation file 'landing-page.md' contains a link to '../kube/sample-cr/locust-test-cr.yaml' which is not found in the documentation files.
WARNING  -  Documentation file 'landing-page.md' contains a link to '../CONTRIBUTING.md' which is not found in the documentation files.

[ ] Getting started section
[ ] Document how to use HELM
[ ] Update main README with needed info and reference to docs on gh-pages


Enable Amendments from config file or static data

The rippled.cfg file has a new section called "amendments"

Each line in this section contains two white-space separated items
** The first item is the id of the amendment (a 256-bit hash)
** The second item is the friendly name

Replaces config section name macros with variables

Make addKnown arguments safer

Added lock to addKnown

@scott @tomswirly

Commit messages should not drill down to the nuts and bolts of the language, i.e.:
"Changed function signature of addKnown from two char*'s  to a hash and a string, makes it less likely to call incorrectly."
Consider instead
"Make addKnown arguments safer"

Got it. Will change.

@nbougalis addKnown was an existing function - I didn't want to change the signature without a good reason (tho I'd rather it didn't return a pointer as well). Will do w.r.t. boost::format. What's rippled's preferred formatter? (I saw boost::format used elsewhere, that's why I picked it). As to return type on line by itself: will do.

@seelabs don't worry about the boost::format bits - we can tackle that at another time. Please consider changing addKnown to not return a pointer - a simple bool seems sufficient to me with the code as is. If we ever need something more, we can add it.

@nbougalis Agree about addKnown returning bool. Will do.

I think I addressed all the feedback. The only two I did add to the code were changing the amendment id from a uint256 to a tagged base_uint type and a  comment about asserting. I left comments above as to why. If you think I missed your feedback point me to the comment and I'll gladly look again.

I'm wondering if anyone has done a lock audit on AmendmentTableImpl?.  I have the sense that the get() method should probably hold a lock since it might be searching m_amendmentMap while the map is mutating.

@nbougalis I added all the changes except changing the asymmetry of the return types. Left a note above as to why.

@seelabs: I'll give it a thumbs up if you write a comment why AmendmentTableImpl::get doesn't need a lock or you put the lock in place.  Sometimes David says that locks aren't needed in certain places, but we should always document why that's the case.  I doubt this is one of David's special cases, however.

@scottschurr get is a public function that iterates through the table. You're right, it needs a lock. Done. I also looked at other code that might check out a lock and call get, and there wasn't any.

:+1:
Looks great!

Does the unit test instantiate the class and exercise its public members?

Writing unit test now. Needed to change the interface a little so I can pass in a config.

:+1: conditional on a unit test.  Good work on catching the lock and on eliminating that bool parameter altogether!  One less thing to have to know about...

Oh, one more comment.
I really believe that there is no use case for naked pointers any more - at least in rippled's flavor of C++11.
Often in the rippled codebase, when we return a naked pointer it's because we're creating that pointer - so the caller owns the pointer.  We're converting these as we see them to std::unique_ptr - that's a no-brainer.
Now, look at AmendmentState* getExisting (uint256 const& amendment); - who owns the returned AmendmentState*?  is it nullable?
Now I can read the answers by reading the code but returning boost::optional<AmendmentState&> getExisting (uint256 const&); makes it unnecessary to even ask the question in the first place.
Just my $0.02 on this issue!

Pointers are part of the language, and since we require C++11 , the assumption that naked pointers do not imply ownership is reasonable. There are use-cases for naked pointers, this looks like one of them.

Then add a comment saying that the caller does not own the returned pointer, and that nullptr indicates that an Amendment wasn't found, please!

Or really, let's just leave it alone.  :-)

I checked in a unit test for addInitial, but I have not checked in the supporting files (to come, want to review locally first). Does run and tests do pass.

@rec on the topic of naked pointers, my thoughts are that if you need to participate in object lifetime management, then use std::unique_ptr and std::shared_ptr as appropriate. Otherwise, passing a pointer is acceptable although I'd prefer a reference if the thing pointed to is guaranteed to never be null.

that last change seemed to cause a legitimate compilation error in clang... :-(

I added unit tests for some of the public interface to AmendmentTable. The unit test and the way I'm doing dependency injection into the class needs to be reviewed. After that, I'd like to mark this as ready to merge.
There are some public functions of Amendment table that do not yet have unit tests, however, I would like to make those a separate ticket and submit them with another PR.
I will squash all the small commits into one commit when we mark this as ready to merge.

"Mock" is correct. Will change ("MOC" ate into my brain from QT programming).

:+1:
Looks great.  Nice job with the unit tests too.

There's a merge commit at the top of the branch

Thanks Scott. I'll clean up all the small git commits into one commit after I get the other thumbs up.

I'm not really feeling these names like AmendmentTableInjections

@vinniefalco I needed to merge after I pushed a minor cleanup that it couldn't automatically merge (probably due to the way I brought in develop into the repo). I'll figure out how to clean up the mess and make a clean commit after I get the other thumbs up.

@vinniefalco I could split the class into two: one dealing with the walletDB interface and one dealing with getValidations. Then naming would be easier: AmendmentTableWalletDB and AmendmentTableValidations or somesuch.
I'd prefer to keep the single class tho. Injection isn't great, I agree, but tough to come up with a name for a class like this.

I needed to create a similar mock in TransactionSign.h.  That mock provides even more scattered functionality than this one does.  We ended up naming the mock TxnSignApiFacade (I believe a suggestion from Vinnie).  I'm pretty satisfied with that name.  You might consider something similar.

Why does the facade need to be wrapped in shared_ptr?

@vinniefalco You mean a unique_ptr? Or was that comment directed at ScottS?
I'm not sure what you're suggesting.

@vinniefalco I got rid of the unique_ptr. The facade is not a regular object in the class. I'm guessing that's what you meant with your comment about shared_ptr.

@vinniefalco I did all the minor cleanups you suggested. For now, I'd rather keep the implementation in  a cpp file instead of moving it to a header (as suggested elsewhere). That means keeping the make_AmendmentTable with a bool parameter. I might go back and refactor this later - especially as we add more testing. I'm anxious to close this ticket.

Me too, where are we?
I was pretty pro-this the last time.  Vinnie, are all your objections satisfied so I can give it a last once-over?

@rec Its about to be merged so you better be quick, I'm ok with it. Although we will almost certainly need to revisit it later.

Great, that's good news.  No further comments!

@vinniefalco Want me to squash all those small commits now?

@rec, @seelabs This has already been squashed down and is about to be merged as 0.28.0-b1
https://github.com/ripple/rippled/pull/829
@seelabs did you change it since yesterday morning?

@vinniefalco No, I haven't touched it.

Merged as 44450bf6448c56ca3f218ba726a9b4e33d422e51


reduce the loading time for analytics page by saving lstm model

now saves model every day for each stock the user owns.