verify consumer side tests

currently we are creating a lot of interactions that are not used in the tests
running verify() will show the issue
we need to reduce the  created interactions to match what is really sent
see POC: #640

Tests to be verified

[x] -unauthenticated (nothing need to be done, in this tests no request is send to the server)

[x] - appTest.js
[x] - capabilitiesTest.js
[x] - filesTest.js
[x] - groupTest.js
[x] - sharingTest.js
[x] - userTest.js

[x] -unauthorized

[x] - appTest.js
[x] - capabilitiesTest.js
[x] - filesTest.js
[x] - groupTest.js
[x] - sharingTest.js
[x] - userTest.js

[x] - appTest.js

[x] - capabilitiesTest.js

[x] - filesTest.js

[x] - filesTrashTest.js

[x] - groupTest.js

[x] - loginTest.js

[x] - publickLinkTest.js

[x] - requestsOcsTest.js

[x] - shareRecipientTest.js

[x] - sharingTest.js

[x] - sharingWithAttributesTest.js

[x] - signedUrlTest.js

[x] - signedUrlIntegrationTest.js

[x] - userTest.js

[x] - xmlParserTest.js  (nothing need to be done, in this tests no request is send to the server)

Tests to be verified

[x] -unauthenticated (nothing need to be done, in this tests no request is send to the server)

[x] - appTest.js
[x] - capabilitiesTest.js
[x] - filesTest.js
[x] - groupTest.js
[x] - sharingTest.js
[x] - userTest.js

[x] -unauthorized

[x] - appTest.js
[x] - capabilitiesTest.js
[x] - filesTest.js
[x] - groupTest.js
[x] - sharingTest.js
[x] - userTest.js

[x] - appTest.js

[x] - capabilitiesTest.js

[x] - filesTest.js

[x] - filesTrashTest.js

[x] - groupTest.js

[x] - loginTest.js

[x] - publickLinkTest.js

[x] - requestsOcsTest.js

[x] - shareRecipientTest.js

[x] - sharingTest.js

[x] - sharingWithAttributesTest.js

[x] - signedUrlTest.js

[x] - signedUrlIntegrationTest.js

[x] - userTest.js

[x] - xmlParserTest.js  (nothing need to be done, in this tests no request is send to the server)

done :+1:

done :+1:


Animal Classifier using CNN

This project is an implementation of an Artificial Neural Network (ANN) to classify images of cats and dogs. The dataset used contains images of cats and dogs and is obtained from Kaggle. The implementation is done in Python using popular machine learning libraries.

@ananyag309 go ahead.


Episode 049: Missing from TGIK playlist and updated Table of Contents

@kris-nova I just noticed episode 049 is missing from the TGI Kubernetes playlist on YouTube.
And also I'd like to suggest this updated Table of Contents for the video description:

00:00:00 - Welcome to TGIK!
00:07:00 - Week in Review
00:29:00 - Introduction - Looking at Flannel
00:31:15 - How to turn on --pod-network-cidr
00:39:57 - Deploying a pod that doesn't work
00:42:47 - Installing Flannel
00:51:50 - Looking at various types of Flannel backends
00:58:28 - Trick to run CNI Linux-compiled binaries on a Mac
01:00:07 - Looking at the Flannel source code
01:12:25 - Observing iptables and route tables mutations
01:16:48 - Looking at how Calico, Flannel and Canal fit together
01:25:05 - Back to iptables and route tables mutations
01:30:35 - Wrapping up

Thanks for the great episode!

Thanks for the note? Would you be interested in opening a PR to the repo? Let me know either way and I can add it too if needed!

I think PR https://github.com/heptio/tgik/pull/104 should be good, let me know if there's anything wrong, I don't git so much.


Update Composer dependencies (2021-11-25-00-06)

Visual regression test passed!
View the visual regression test report


[bug] tests aren't 100% working

It seems like there's a timezone offset issue with running the tests versus the original author's timezone (which are hardcoded).  There might be other issues too, e.g. with the original codebase.  I did notice one major issue which I thought stood out, search for "TODO" in the src/index.js file to see the comment.

Does later take into consideration timezones? Or is it just set to the hardcoded ones like you mentioned above?

It seems like this fork of later is the culprit behind the issues found in https://github.com/breejs/bree/issues/29.

 If you see anything let me know.

I got it figured out, will release shortly.

This is now resolved as of v4.0.0.  I am also releasing a new version of bree now.


Feature request: right-click file in Search pane to "Open to the Side"

As per the title, when I perform a search in the Search pane, I would like to be able to right-click a file in the results section and be provided with the option to open the file to the side. I couldn't find anything to suggest this is already possible with a setting or extension, so apologies if a solution is already provided :blush:

Hi @mikehurl - that would be pretty handy! But I think you're right, in that this functionality isn't provided by the VSCode extension model. There's no menus for search here: https://code.visualstudio.com/api/references/contribution-points#contributes.menus

Wrong repo 🤦♂


Access Array Data with Indexes problem needs a little tweak

Challenge Name
Access Array Data with Indexes

https://www.freecodecamp.com/challenges/access-array-data-with-indexes#?solution=
%2F%2F Example
var ourArray %3D [1%2C2%2C3]%3B
var ourData %3D ourArray[0]%3B %2F%2F equals 1

%2F%2F Setup
var myArray %3D [1%2C2%2C3]%3B

%2F%2F Only change code below this line.

Instructions
Create a variable called myData and set it to equal the first value of myArray.
Issue Description

It is not an issue rather we can get solution by not following steps
here is the wrong solution which is accepted
var ourArray = [1,2,3];
var ourData = ourArray[0]; // equals 1

// Setup
var myArray = [1,2,3];

// Only change code below this line.
var myData = 1;

solution to be accepted

var ourArray = [1,2,3];
var ourData = ourArray[0]; // equals 1

// Setup
var myArray = [1,2,3];

// Only change code below this line.
var myData = myArray[0];

@dimahali thanks for catching this! We should update the tests so this incorrect solution isn't allowed.

+1!


[MRG] proposal for validator

in relation to #4

adds a WIP version of the validator for the input directory

LGTM except a general term thingy...Please see comments.

Took care of the comments. Ready for merging if everything ok.

Great, thx. I'll merge and close.


helm finds namespace inside description

I discovered this with 3.7.2, but reproduces with 3.11.1 and HEAD as of today (e007c900).
If the argument to --description starts with a - and has an n in it somewhere that doesn't follow an h, it will try to install into the namespace named by the rest of the description after the n. For example:
❯ ./bin/helm upgrade -i --dry-run --description '- watsupnhello' my-release my-chart | head
Release "my-release" does not exist. Installing it now.
NAME: my-release
LAST DEPLOYED: Tue Mar  7 13:17:01 2023
NAMESPACE: hello
STATUS: pending-install
REVISION: 1
TEST SUITE: None
HOOKS:
MANIFEST:

The n and h are special presumably because they're the two short options recognized by the root helm command. How they end up being parsed as flags, I don't know.
It wasn't obvious to me whether this was a bug in Helm or cobra, or even something else, but I didn't spend much time looking at the code.

That is indeed cobra/viper related. You can fix this by removing the ambiguity: helm upgrade -i --dry-run --description='- watsupnhello' my-release my-chart

Ah, thanks. I guess I'll go file the issue over there and see if they consider it a problem or not.

Since there is a workaround and not a bug in helm, I'm going to go ahead and close this. If you get traction upstream and need upstream changes brought down, feel free to open a new issue at that time.


pepeline (ticdc): clean uselesss codes

What problem does this PR solve?

Issue Number: ref #5189
What is changed and how it works?
Check List
Tests

No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change

Side effects
none
Related changes
none
Release note
None

/run-integration-tests

/run-all-tests

/run-integration-tests

/run-integration-tests

/run-integration-tests


Test finishing as succeeded, if array of dependencies has an extra dependency

Hey, guys.
Recently I found strange issue with RamblerTyphoonUtils/AssemblyTesting.
For example:
I have some assembly with components. Some component has one dependency, for example some Service. When I writing test for this component and create array dependencies with one extra dependency, for example:
NSArray *dependencies = @[service, secondExtraService];

This test finished as succeeded, however this component need one dependency in this case.
This is normal behaviour?

After watching code of AssemblyTesting, I am convinced that function
[self verifyTargetDependency:result withClass:expectedClass dependencies:dependencies];

doesn't verify case if count dependencies of component less or greater than dependencies array.
Maybe it makes sense to implement this case?

Thanks for reporting!
It's possible to verify that all of the passed dependencies exist in the target class, we've just overlooked this case. Do you want to make a PR? :)
As for another part of your question - currently it's not possible to check if we are verifying all of the object dependencies. That's because we can't determine which of the properties should be injected by Typhoon, and which - manually in runtime.

@igrekde It will be great if you add case that verify passed dependencies exist in the target class.
What   can I do for PR? However, It will be nice for my growing experience, if you let me implement this case and make pull request.
Thank for explanation of my another second part of my question.

Look at the class, especially at lines 35-40. You can do something similar - check if passed dependencies selectors exist in allProperties NSDictionary.

btw, I'm going for a vacation, so I'll hardly be able to make code review before the January 10.

Thanks for your hint for me.
Can you make code review after New Year vacation, if I'll be make pull request?

Yes, of course! Maybe @mogol will do it even sooner.

Great!
I wish you nice vacation ;)


Got error when running "python3 app.py"

Traceback (most recent call last):
File "app.py", line 1422, in
inference_talkinghead()
File "/opt/conda/lib/python3.8/site-packages/gradio/blocks.py", line 1285, in exit
self.config = self.get_config_file()
File "/opt/conda/lib/python3.8/site-packages/gradio/blocks.py", line 1261, in get_config_file
"input": list(block.input_api_info()),  # type: ignore
File "/opt/conda/lib/python3.8/site-packages/gradio_client/serializing.py", line 44, in input_api_info
types = api_info.get("serialized_input", [api_info["info"]["type"]] * 2)  # type: ignore
KeyError: 'type'
I have change my gradio to 3.25.0, and got this error.

@wwdok please check on this when you are free

@wczqhd Do you solve the problem now ?
If not, could you please provide deeper infomations like what is the final line of code in inference_talkinghead() that throws this error ?
I search api_info.get("serialized_input", [api_info["info"]["type"]] * 2) in google, find there are same issues, it seems ralated to version of gradio and gradio_client:

pip install gradio==3.50.2
解决问题

please try out the newest train-free, 10s inference version facechain-fact.


Warning received while installing webpack in windows to global

Restarted the machine;
running cmd prompt as administrator
Windows 7 professional
npm installed and other packages installed
Warning received while installing webpack in windows to global:
npm WARN optional dep failed, continuing<EMAIL_ADDRESS>C:\Users\username\AppData\Roaming\npm\webpack -> C:\Users\username\AppData\Roa
ming\npm\node_modules\webpack\bin\webpack.js
npm WARN package.json<EMAIL_ADDRESS>No repository field.
<EMAIL_ADDRESS>C:\Users\username\AppData\Roaming\npm\node_modules\webpack
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>4<EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

npm WARN optional dep failed, continuing<EMAIL_ADDRESS>fine, it's an OSX only module.
npm WARN package.json<EMAIL_ADDRESS>No repository field. Send a PR to commandjs and add an repo field to fix it. But it doesn't hurt.


Marketplace Deployment - Error when creating a new service account

The current deployment allows a user to create a new service account or select an existing one.  However, when I attempt to create a new service account I get the following error.  I'm a project owner on my project so I have all permissions necessary to create a new service account.
I also attempted to assign this role (- roles/resourcemanager.projects.setIamPolicy)to the new service account in the jinja.schema file but that generates a different error stating that this role cannot be include.

I attempted the deployment with the following set of roles and it did not generate an error.
 roles:
        - roles/container.admin
        - roles/iam.securityAdmin
        - roles/iam.roleAdmin
        - roles/serviceusage.serviceUsageAdmin
        - roles/storage.admin
        - roles/resourcemanager.projectIamAdmin
        - roles/runtimeconfig.admin


Add coverage configuration file. | Fixes #8

Codecov Report
Base: 98.22% // Head: 100.00% // Increases project coverage by +1.77% :tada:

Coverage data is based on head (e7ada5e) compared to base (c4c03b2).
Patch has no changes to coverable lines.

:exclamation: Current head e7ada5e differs from pull request most recent head 308aeb3. Consider uploading reports for the commit 308aeb3 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             main        #9      +/-   ##
===========================================
+ Coverage   98.22%   100.00%   +1.77%
===========================================
  Files           4         3       -1
  Lines         282        67     -215
===========================================
- Hits          277        67     -210
+ Misses          5         0       -5

Impacted Files
Coverage Δ

tests/test_public_api.py

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Generation of RouteConfigs from redux store data.

Hello, how to generate a new RouteConfigs for TabNavigator when opening nested navigation? And how do I obtain data for the generator from the redux store? It would be logical to wrap TabNavigator as/into a component, but then this connection with the parent StackNavigator would break. Thanks!

I support this issue, I've switched back to react-native-router-flux because my TabNavigator nested into StackNavigator breaks when specifying the initial navigation state.

React Nav is designed to handle routing before your redux store has had a chance to load your screen's data. This allows the app to load faster. If the redux store loads and decides that the nav state is somehow invalid, you can fire a navigate action, or your screen can render an error.
Or, if you do want to have custom logic define your inner navigators, feel free to create the navigator after your data has loaded. But then you will be responsible for providing it with the navigation prop (including the current nav state and a navigation dispatch handler).

@ericvicenti Can you help to configure redux state for nested routes? I have same error situation

my TabNavigator nested into StackNavigator breaks when specifying the initial navigation state.
The error looks like:
There is no route defined for key MainTabs. Must be one of: 'MainTabs'


Every url in discovery document has to start with the issuer url - but why?

When using strictDiscoveryDocumentValidation (the default), all urls in the discovery document must start with the issuer url, but this does not seem to be a part of any OAuth, or OpenID specification that I can find. Why is this a requirement? It is incompatible wth eg. Googles OpenID configuration: https://accounts.google.com/.well-known/openid-configuration.

It's also incompatible with WSO2-IS.

Based on what I know about the library, and digging through the commits that introduced that option, I would say that it's a little overzealous and you can safely set the strictDiscoveryDocumentValidation option to false for Google OpenID and WSO2-IS, and move on.
Assuming I'm right, we could leave this GH issue open as a request to add this note (that it can be safely set to false for some IDServers) to the property's documentation. But that's probably something only the original author / maintainer can answer for us.

The discovery document for PingOne relying parties also fails the validation check, and as such requires setting strictDiscoveryDocumentValidation = false;
I'm opening a PR to reflect this configuration in the "Getting Started" section, as it seems relevant enough to multiple ID Providers and it took a little bit of digging to find


chore(travis): include dart doc generation

Installs Dart SDK
Installs ng2/dart sources from the pub package site.
Re-enables doc build on Travis (which runs pub and dartdoc over ng2/dart).

Note: npm install -g gulp --no-optional has been removed since it does not appear to be needed.
Implements most of #1907.

@filipesilva @Foxandxss @wardbell : ready for review. Note that the extra wait for the doc generation only seems to be necessary when full logging is enabled; currently the logging level is set to warn, and doc generation takes ~20 min. (Otherwise I've seen it take ~40 min, though I'm not sure why there is such a difference).
cc: @kwalrath

There's a couple of things that I would like to understand better about this PR:

The new WAIT env var strikes me as a red flag, why is it needed? can that value change? Having hardcoded waits is usually a brittle breaking point, or at the very least something that will overall delay CI.
What are all these travis_fold commands? Won't this immediately break if I try to run the scripts locally?  That sounds like a problem for using these scripts anywhere but travis.
Seems like a lot of environment variables need to be set, which also sounds like it can be problematic for local runs.
Dart now seems a .hard requirement for running deploy-install.sh, which means that I wouldn't be able to run it right now whereas before I could.

Thanks for your feedback.

The new WAIT env var strikes me as a red flag, why is it needed?

See my previous comment.

Having hardcoded waits is usually a brittle breaking point,

It is no more brittle than the current 10 min max.

or at the very least something that will overall delay CI.

Yes, it may.

What are all these travis_fold commands?

They help make the output manageable by having the output between start and end sections contained within a "fold" (headed by triangles in the left margin of the output).

Won't this immediately break if I try to run the scripts locally?

No, a value is always defined for the function, see env-set.sh.

That sounds like a problem for using these scripts anywhere but travis.

I suggest we follow what is done for the angular project and actually put all travis scripts in a separate folder so that it is understood that they are meant to be used for travis ... and might work locally, but that is not their primary purpose.

Seems like a lot of environment variables need to be set,

Not any more than those for angular.

which also sounds like it can be problematic for local runs.

These scripts are in support for Travis CI. I had assumed that when testing locally, doc developers would already have the necessary repos available as peers to the ng.io repo.

Dart now seems a .hard requirement for running deploy-install.sh, which means that I wouldn't be able to run it right now whereas before I could.

It runs fine in a Travis CI environment. Of course, I could make some parts be conditionally applied in a Travis environment only, and/or the presence of Dart tooling.
Currently, angular.io is the home to ng2 docs of all flavors, and under that premise, it seems essential to build all the docs as part of CI. Maybe I've been misdirected in my goal, any thoughts about this, @naomiblack?
The next steps, IMHO, include adding checks for broken links, etc. For that we want to do a whole site analysis.

I haven't checked out A2's scripts and was only comparing with the scripts we already had. That's probably why I found everything so strange.
I like the idea of a travis scripts folder, but would be worried that it would lead either repetition or having some scripts only there. Do you suppose we can have some middle ground on that, where is it possible to run locally run scripts for TS/Dart (separately, as not all authors do both), and still have all the extra tooling in travis only scripts?

I agree that we should build all docs on travis btw, just wanted to preserve scripts as a way to setup/reset local environments as well. Maybe it's too hard to have both at the moment.

I like the idea of a travis scripts folder, but would be worried that it would lead either repetition or having some scripts only there. Do you suppose we can have some middle ground ...

Yes, agreed. It would help to know which scripts are run locally. I was under the impression that scripts/deploy-install.sh would be used only for CI (given that most doc developers, IMHO, would tend to manage repo cloning by a separate mechanism). If not, I can add some conditional logic.

The script that I care about as a TS/JS author is examples-install.sh - because it's currently just for ts/js. Maybe the specific bits should be factored out.
The script that I care about as someone who does TS/JS releases is examples-install.sh and deploy-install.sh, again because of the ts/js focus. If the dart part of a site deploy fails my ability to fix it is limited... I can fix jade level stuff but anything that touches the examples I try not to touch.

Ok, thanks for the input on that. Let's see what can be done ... .

@filipesilva I have updated the scripts so that they can be run from the command line. I have moved out as much as I can from the env-set.sh script. Note that the examples-install scripts have not be touched. Please have a look.

LGTM Patrice, thank your for all the hard work in getting dart doc gen working on travis!
I'm seeing doc generation taking 7-8mins on Travis, whereas you mentioned 20m. Did you manage to cut it down further?

Ah, that is because it isn't actually building the Dart docs. I needed to adjust the gulpfile to switch to the default of all languages for the tasks: fullSiteBuildTasks = ['build-compile', 'check-serve', 'check-deploy']. Done.

@wardbell : ok, this is good to go! :)

I'm out of my depth. Since @filipesilva approves, in it goes

Great!
Btw, @filipesilva @wardbell : if ever the extended timeout becomes an issue, we can always adjust the number. At least now the basic mechanism is is place so that all we need to do is change a number.


There is no information on whether a package requires a config file

Bug Report
This might be a doc issue but I actually think this is a deficiency in the package listing command.
There is no easy to find information on whether a config file is required for a package to work properly, let alone what contents of that file would be required.
This page, https://tanzucommunityedition.io/docs/main/ref-unmanaged-cluster/#installing-packages-during-bootstrapping, talks about a config file being optional.  The bottom of that section has this text
For further information on packages and configuring packages, read the documentation.
But that link should probably be to https://tanzucommunityedition.io/docs/main/package-management/#configuring-a-package
That doc takes a few reads for me to understand - but that should be a doc issue.
Docs aside, there should be guidance in the tanzu cli when I list packages that says whether a configuration file is required or recommended for proper functioning. Or maybe there should be a details page for a package obtained through:
tanzu package available list contour.community.tanzu.vmware.com/1.20.1
which contains

Is a configuration required
Example how to get the --values-schema
Link to the page in the doc for that version - https://tanzucommunityedition.io/docs/main/package-readme-contour-1.20.1/

Expected Behavior
Self-discovery of configuration and detailed information for a package

This has also been a challenge I have faced during my first few Tanzu package deployments.
Related to this, in an ideal world, no package would need configuration values and should deploy for lab/demo purposes with sane default values. For example, Harbor requires setting a username and password. As an end-user, I would expect that to at least have simple values like "admin" and "admin" or, better yet, have the password be random and auto-generated.
As it is today, this all makes approaching Tanzu packages for newcomers a not-so-great experience.


Really slow polyfills when building nontrivial site

OK, I know it's a developer preview, but currently the site I'm building for my customer takes 20-25 seconds to load in either latest FF or chrome < 36. In Chrome 36 -> super-snappy.
So my question is; what is the timeline in speeding up polyfills? Or at least make the run in chunks, asynchronously or something.
Or are they as tuned as can be and we just have to wait until the world catches up?

Closing this issue due to age and the release of version 1 of Polymer - please feel free to re-open if this is incorrect.


How to add authentication

We are using BasicAuthentication for Authentication.

I found it in your documentation. Sorry for raising this issue


also resolves #9, channelized interfaces

on top of https://github.com/kellydunn/go-art/pull/10 adds a more idiomatic interface to the iterators, using channels as opposed to callbacks.  kept the original api for backwards compatibility

Heya @fredim!  Thanks for the PR, I really appreciate it :)
And my apologies on the slow response time!  Things have been a bit crazy for me lately.
I've left a few comments on your PR if you don't mind.  Would love to hear back from you!

thanks @kellydunn ! i appreciate the look.  i made the mods requested, and added some comments.


Setup CI for the project

Travis seems like the obvious CI choice since it's integrated with GitHub. Ideally, it needs to...

Do a validation deployment for all pull requests - all unit tests should be executed. If the validation deployment fails or has not run, the pull request should be blocked from being merged
(Unsure) Any changes pushed to master should be deployed to the Nebula sandbox

Build badges should also be added to README.md, like this...
Branches

Name
Build Status

master

dev


[vim] Fix default label problem

Update only utils/vim/syntax/swift.vim
Fix the problem that the subsequent lines are not highlighted when using default without colon.
Problematic patterns

Expects

@swift-ci please smoke test and merge

@swift-ci please smoke test Linux platform.


REDIRECT-INFO-ABOUT-APPROVAL-STATUS-CHANGES - testsuite should allow new tcp client to be created

SERVICELAYER / INDIVIDUALPART / /V1/REDIRECT-INFO-ABOUT-APPROVAL-STATUS-CHANGES / ACCEPTANCE / ATTRIBUTE CONFIGURED? / SUBSCRIBER-PROTOCOL UPDATED? - 5 REQUESTS IN THE FOLDER
This operation (always) changes attributes of registry office. The test proposes protocol of HTTPS, which is for RO not configured. Per openBackhaul/ApplicationPattern#858 both protocols are supported for RO,therefore the testsuite should not expect existing tcp client to be changed, but a new tcp client to be created.
This also impacts:

SERVICELAYER / INDIVIDUALPART / /V1/REDIRECT-INFO-ABOUT-APPROVAL-STATUS-CHANGES / ACCEPTANCE / UPDATE LTP NOTIFIED? / SUBSCRIBER-PROTOCOL UPDATE - 9 REQUESTS IN THE FOLDER
PostTermination - 2 Requests in the folder

As per specification of subscriber-protocol of /v1/redirect-info-about-approval-status-changes (update [/core-model-1-4:control-construct/logical-termination-point=tar-2-0-2-tcp-c-ro-2-0-2-000/layer-protocol=0/tcp-client-interface-1-0:tcp-client-interface-pac/tcp-client-interface-configuration/remote-protocol]'), the tcp-client instance shall be expected only to be updated and a new instance is not expected to be created.
Closing the issue assuming no change is required in test-suite.
@DanaSunal , kindly reopen the issue if my understanding is conflicting


[rsm-mmio] Add new port

Describe the pull request

What does your PR fix?
Adds a new port mmio

Which triplets are supported/not supported? Have you updated the CI baseline?
Only supports Linux/Windows + static linkage

Does your PR follow the maintainer guide?

Yes

@Ryan-rsm-McKenzie, thanks for your PR!
Could you change the 'tab' to 'space' in portfile.cmake?


Handling 1 of N dependencies

In various specs there are sections that references to other specifications in which the exact choice is not specified but typically one of multiple of those specs would be in use. An example is when an encryption or signing scheme is used- there may be multiple schemes that are supported by the spec but only 1 is used. The relationship to the signing scheme is neither only a reference nor a pure dependency.
The map currently is inconsistent in how it treats those. The relationships may be treated as references, dependencies or handle specially with an intermediate element representing 1 of n relationship.
How to represent these relationships should be done consistently. Some current ideas are:

make the connection type "depends" but make the label for the connection something like "depends 1 of a"

where all the connections for one set would be '1 of a' and if there were additional sets of relationships from the parent it would be represented as '1 of b' etc.
this is likely confusing to observe in the map and error prone to maintain

create an intermediate element that represents a dependency that then has child references connections to the set/sub-set of other specs

[spec a] ---depends -> [intermediate] -- references -> [spec y ] and [spec z]
This will create a clean separation in the map and not be too difficult to maintain
This intermediate element type may need a new element type since it is not a specification but a relationship element but a ; something other than core, non-core

V1.31+ has the proposed schema applied to

well-known-did
did-spec-registries

did-spec-registries has two 1-to-many relationships, one for signatures and the other for did-method
One variation was made to the proposed scheme. The same relationship label was carried through to the specs  so instead of
[spec a] ---implements -> [intermediate] -- references -> [spec y ] and [spec z]
it was applied as:
[spec a] ---implements -> [intermediate] -- implements -> [spec y ] and [spec z]

V1.31+ has the proposed schema applied to

well-known-did
did-spec-registries

did-spec-registries has two 1-to-many relationships, one for signatures and the other for did-method
One variation was made to the proposed scheme. The same relationship label was carried through to the specs  so instead of
[spec a] ---implements -> [intermediate] -- references -> [spec y ] and [spec z]
it was applied as:
[spec a] ---implements -> [intermediate] -- implements -> [spec y ] and [spec z]

The wiki page on how to populate the CSV has been updated to document this scheme as it is expected to be a final change. This issue will be left open for an indefinite period to allow for more input if needed.

The wiki page on how to populate the CSV has been updated to document this scheme as it is expected to be a final change. This issue will be left open for an indefinite period to allow for more input if needed.

closing issue. An implementation was performed in an earlier release and additional issues for improvements have been made e.g. #35


fix(DeploymentsModal.js): change of the first letter of location brea…

…dcrumb to uppercase S                                           Change of the string services to Services with uppercase S
Testing

Trade-offs

Dependencies

Screenshots

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

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


Any updates about the python version of the code?

I am very interested in your work and I would like to regenerate the results on my data. A python version is the most suitable for me. Any date of release fixed yet?

I am also interested in the python version. Any update?

No updates yet? I am trying to reproduce the experiment, but also the MATLAB code seems incomplete, and can't make it work


Bind g_utf8 / g_unichar APIs

std doesn't have almost anything of that, and the crate implementing all that seems is using icu, which is huge and annoying to build. The utf8 APIs should probably take &GStr / GString, otherwise we end up creating lots of copies.

Should this be closed now? cc @sdroege

When binding the g_utf8 APIs I didn't include the ones that I thought were already offered by the rust stdlib through str (which GStr gets through its Deref<Target = &str> impl), IIRC that was one of them.
However, the g_unichar APIs are still missing (some of them are offered by the stdlib already, but not all), so this issue should stay open for the time being.


tab size is set to 8 in latest version

Issue Type: Performance Issue
as title, default tab size is set to 8 in latest version, which is an unbelievable modification.
VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:28:07.755Z)
OS version: Windows_NT x64 10.0.19041
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz (8 x 1896)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.85GB (7.04GB free)

Process Argv
--crash-reporter-id ffd39a9b-3af7-4188-ba19-d2e6963bbef3

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    3	   123	  4488	code main
    0	    19	  1680	   "C:\Program Files (x86)\SogouInput\<IP_ADDRESS>46\SGTool.exe" -daemon
    0	   180	  4156	   gpu-process
    1	    82	  6240	   window (Issue Reporter)
    0	   175	  9580	   window (● Untitled-1 - Visual Studio Code)
    0	   144	   504	     extensionHost
    0	    68	 10432	       electron_node serverMain.js
    0	    70	 12252	       "C:\Users\epuxphi\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\epuxphi\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\node\jsonServerMain" --node-ipc --clientProcessId=504
    0	    24	 13272	       c:\Users\epuxphi\AppData\Roaming\Code\User\globalStorage\tabnine.tabnine-vscode\binaries\3.5.5\x86_64-pc-windows-gnu\TabNine.exe --client=vscode --no-lsp=true --client-metadata clientVersion=1.57.1 pluginVersion=3.4.6 t9-vscode-AutoImportEnabled=true t9-vscode-TSAutoImportEnabled=true t9-vscode-JSAutoImportEnabled=true vscode-telemetry-enabled=true vscode-remote=false vscode-remote-name=undefined vscode-extension-kind=1 "vscode-theme-name=Monokai Pro" vscode-theme-kind=Dark vscode-machine-id=7d4fab3af147874b92eb51e5fbd6216226a77ceaa96b61c71985c9c94c96fc47 vscode-is-new-app-install=false vscode-session-id=d205af03-cfa4-4f78-8621-68ab5b495d721624242258023 vscode-language=en "vscode-app-name=Visual Studio Code" vscode-status-customization=unknown ide-restart-counter=0
    0	    12	  7112	         console-window-host (Windows internal process)
    0	    23	  9640	         "C:\Users\epuxphi\AppData\Roaming\Code\User\globalStorage\tabnine.tabnine-vscode\binaries\3.5.5\x86_64-pc-windows-gnu\WD-TabNine.exe" --client vscode --client-metadata clientVersion=1.57.1 pluginVersion=3.4.6 t9-vscode-AutoImportEnabled=true t9-vscode-TSAutoImportEnabled=true t9-vscode-JSAutoImportEnabled=true vscode-telemetry-enabled=true vscode-remote=false vscode-remote-name=undefined vscode-extension-kind=1 "vscode-theme-name=Monokai Pro" vscode-theme-kind=Dark vscode-machine-id=7d4fab3af147874b92eb51e5fbd6216226a77ceaa96b61c71985c9c94c96fc47 vscode-is-new-app-install=false vscode-session-id=d205af03-cfa4-4f78-8621-68ab5b495d721624242258023 vscode-language=en "vscode-app-name=Visual Studio Code" vscode-status-customization=unknown ide-restart-counter=0
    0	   104	 24320	         "c:\Users\epuxphi\AppData\Roaming\Code\User\globalStorage\tabnine.tabnine-vscode\binaries\3.5.5\x86_64-pc-windows-gnu\TabNine.exe" --client=vscode --no-lsp=true --client-metadata clientVersion=1.57.1 pluginVersion=3.4.6 t9-vscode-AutoImportEnabled=true t9-vscode-TSAutoImportEnabled=true t9-vscode-JSAutoImportEnabled=true vscode-telemetry-enabled=true vscode-remote=false vscode-remote-name=undefined vscode-extension-kind=1 "vscode-theme-name=Monokai Pro" vscode-theme-kind=Dark vscode-machine-id=7d4fab3af147874b92eb51e5fbd6216226a77ceaa96b61c71985c9c94c96fc47 vscode-is-new-app-install=false vscode-session-id=d205af03-cfa4-4f78-8621-68ab5b495d721624242258023 vscode-language=en "vscode-app-name=Visual Studio Code" vscode-status-customization=unknown ide-restart-counter=0 --no_bootstrap
    0	   456	  9616	           "c:\Users\epuxphi\AppData\Roaming\Code\User\globalStorage\tabnine.tabnine-vscode\binaries\3.5.5\x86_64-pc-windows-gnu\TabNine-deep-local.exe" --settings C:\Users\epuxphi\AppData\Roaming\TabNine\local.settings --settings-version 1 --client vscode --client-metadata clientVersion=1.57.1 pluginVersion=3.4.6 t9-vscode-AutoImportEnabled=true t9-vscode-TSAutoImportEnabled=true t9-vscode-JSAutoImportEnabled=true vscode-telemetry-enabled=true vscode-remote=false vscode-remote-name=undefined vscode-extension-kind=1 "vscode-theme-name=Monokai Pro" vscode-theme-kind=Dark vscode-machine-id=7d4fab3af147874b92eb51e5fbd6216226a77ceaa96b61c71985c9c94c96fc47 vscode-is-new-app-install=false vscode-session-id=d205af03-cfa4-4f78-8621-68ab5b495d721624242258023 vscode-language=en "vscode-app-name=Visual Studio Code" vscode-status-customization=unknown ide-restart-counter=0
    0	    44	 13028	   utility
    0	    27	 16180	   crashpad-handler
    0	    91	 19876	   shared-process
    0	    65	 19096	     ptyHost

Workspace Info
;

Extensions (81)

Extension
Author (truncated)
Version

rainbow-brackets
2gu
0.0.6

better-comments
aar
2.1.0

html-snippets
abu
0.2.1

codesnap
adp
1.3.2

include-autocomplete
ajs
0.0.4

increment-selection
alb
0.2.0

Bookmarks
ale
13.1.0

auto-add-brackets
ali
0.11.0

All-Autocomplete
Ati
0.0.23

vscode-django
bat
1.6.0

vscode-log-viewer
ber
0.12.2

unique-lines
bib
1.0.0

color-info
bie
0.6.0

xml2json
bui
1.2.4

path-intellisense
chr
2.3.0

highlight-line-vscode
cli
0.0.1

bracket-pair-colorizer
Coe
1.0.61

compulim-vscode-closetag
Com
1.2.0

transformer
dak
1.11.1

python-extension-pack
don
1.6.0

xml
Dot
2.5.1

LogFileHighlighter
emi
2.11.0

vscode-relative-line-numbers
ext
0.0.5

vscode-highlight
fab
1.5.0

file-icons
fil
1.0.29

auto-close-tag
for
0.5.10

auto-complete-tag
for
0.1.0

auto-rename-tag
for
0.1.6

code-runner
for
0.11.4

sort-json-array
fvc
3.0.1

beautify
Hoo
1.5.0

vscode-power-mode
hoo
2.2.0

rest-client
hum
0.24.5

output-colorizer
IBM
0.1.2

highlight-icemode
ice
0.0.6

compareit
in4
0.0.2

path-autocomplete
ion
1.17.1

latex-workshop
Jam
8.19.2

better-cpp-syntax
jef
1.15.6

vscode-insertdatestring
jsy
2.3.0

auto-comment-blocks
kev
1.0.1

vsc-python-indent
Kev
1.14.2

vscode-format-context-menu
lac
1.0.4

vscode-smart-column-indenter
lmc
0.0.13

MagicPython
mag
1.1.0

python-path
mge
0.0.11

HTMLHint
mka
0.10.0

theme-monokai-pro-vscode
mon
1.1.19

python
ms-
2021.6.944021595

vscode-pylance
ms-
2021.6.2

jupyter
ms-
2021.6.999406279

cpptools
ms-
1.4.1

reload
nat
0.0.6

color-highlight
nau
2.3.0

autodocstring
njp
0.5.4

indent-rainbow
ode
7.5.0

subtle-brackets
raf
3.0.0

vscode-commons
red
0.0.6

vscode-xml
red
0.16.1

vscode-sort-json
ric
1.20.0

vscode-statusbar-json-path
ric
1.4.5

datetime
rid
2.0.3

ActiveFileInStatusBar
Ros
1.0.3

partial-diff
ryu
1.4.3

text-marker
ryu
1.11.0

trailing-spaces
sha
0.3.1

indenticator
Sir
0.7.0

vscode-fileutils
sle
3.4.5

vscode-smoothtype
spi
1.2.0

guides
spy
0.9.3

rewrap
stk
1.14.0

tabnine-vscode
Tab
3.4.6

selectline-statusbar
tom
0.0.2

sort-lines
Tyr
1.9.0

errorlens
use
3.2.7

vscodeintellicode
Vis
1.2.14

colonize
vms
2.2.2

vscode-icons
vsc
11.5.0

vim
vsc
1.21.3

vscode-jumpy
wma
0.3.1

clang-format
xav
1.9.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt700cf:30270857
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551cf:30311713
vspre833:30321513
pythonptprofiler:30281270
vshan820:30294714
pythondataviewer:30285071
vscus158:30321503
pythonvsuse255cf:30323309
vscorehov:30309549
vscod805cf:30301675
binariesv615:30323119

Same here, seems like it doesn't respect .editorconfig settings anymore

It looks like the issue is caused by one of your installed extensions. In order to narrow down the possible problematic extension, please try the following:

run F1 > Help: Start Extension Bisect
(this will restart VS Code with half of your extensions disabled)
try to do the steps which reproduce the problem
if the problem reproduces, click "This is bad" (this narrows down the problematic extension to the half which was enabled)
if the problem does not reproduce, click "Good now" (this narrows down the problematic extension to the half which was disabled)
these steps will repeat until the problematic extension is found

initial
step
final

hello, @alexdima
it ends here, seems that clang-format is the "crminal':

@alexdima I can reproduce this with no extensions enabled

Open workspace which has an .editorconfig file at its root (or in one of the workspace's parent directories, that also used to work)
Command palette, "Developer: Reload with extensions disabled"
Create a new file with a file type specified in .editorconfig.

In my case: SomeComponent.tsx

Observe "Tab size: 8" in the bottom right corner

In my case: should be "Spaces: 4"

@volksvagen Please report directly against the extension
@larskinn VS Code does not honor .editorconfig files in any way. It is the editorconfig extension that reads those files and then uses vscode APIs to change the indentation settings of the current opened file. Can you check your user settings, folder settings and workspace settings for editor.tabSize, editor.insertSpaces and editor.detectIndentation ?

VS Code does not honor .editorconfig files in any way [...]

@alexdima Okay, sorry for the wild goose chase. I actually don't have the editorconfig extension, but somehow the default for new files, which used to be "Spaces: 4", has changed to "Tab size: 8" with the latest update. Not a huge problem for me, as it is fixed by saving (autoformats with prettier, which reads .editorconfig) and closing/reopening the file (VS Code autodetects the correct indentation)
I checked my user settings:

editor.tabSize

was changed to 8, from the default 4

editor.insertSpaces

was changed to false, from the default true

editor.detectIndentation

was unchanged from default true

I can honestly say that I have never changed these settings myself, and the default indentation for new files until the latest VS Code update was always 4 spaces. So either VS Code or some extension somehow changed my user settings immediately after the latest update, or these settings were not being respected before.
After resetting the above settings to default, it works as expected.

:+1: AFAIK we have no code that would change these particular user settings.
But it is possible that an extension decided to change these settings on your behalf by editing the user settings file.


Change taproot functions

bip340 was not very descriptive, so that was changed to be xonly
The idea of a tweak was not very clear, instead the merkle root of the script path is a lot easier to reason about, so the code reflects this change. The terminology of internal pubkey and external pubkey is used to make it conceptually simpler. Similarly, a Tap Root is also not very clear, so we now use methods of a public key or a tapleaf/tapbranch to construct the needed external public keys and control blocks.

Nice @jimmysong!


Not working with new cullable property from PIXI 6.3.0.

Hi, I'm making a game using pixi-tilemap but I can't use the new cullable property, I see no performance differences when setting it to on and off and zooming.
I'm also using pixi-viewport but it seems that it's not because of it, see https://github.com/davidfig/pixi-viewport/issues/381.

Tilemap overrides render instead of _render (and renderCanvas instead of _renderCanvas). That's why not only cullable doesn't work but also visible, renderable, filters, and mask.

i should really make breaking changes to it and name 3.0

Yup it would be better

Any update on this ?


Scatter plot: incorrect tooltip for 'zoom and filter'

v1.16.7
Steps to reproduce:

Add a scatter plot
Open properties panel
Hover 'zoom and filter' option

Expected results: rename the last entry into 'pack and zoom by filter' in the tooltip
Actual results: two last options have same names in the tooltip:

1.17.0


Windows Demo throws arcane error

Windows Demo throws arcane error. I tried different streams that are accessible via web browser. Please see screenshot.
May be I am doing something wrong?
Screenshot contains everything. Ran on Windows Server 2016 with Open JDK 8.

Hello, try to enter the room number without "https://appr.tc/r/". Just "3631010101"

Thanks @vria !


Support specifying ADM file path via Java system property at backend startup

Support specifying ADM file path via Java system property like -Datlasmap.adm.path or whatever at backend startup so that the client like VSCode can provide a ADM file and have it preloaded in AtlasMap UI.
Only for initial loading purpose. User need to export explicitly to get adm file updated.

that sounds like a great idea

it will preload all files provided in this folder? And how to open the UI with a specific file? There is a specific uri parameter to provide?

"so that the client like VSCode can provide a ADM file and have it preloaded"

started some work in this branch: https://github.com/apupier/atlasmap/tree/755-providepropertyToInitAtlasMapWithAdmFile

started some work in this branch: https://github.com/apupier/atlasmap/tree/755-providepropertyToInitAtlasMapWithAdmFile

I started a bit of work with some tests that might be reusable but it is far from working.
I was expecting to have the DefaultAtlasContextFactory.createContext(URI) supporting having an .adm file passed to it. In fact it seems not the case and the upload of .adm file from the UI has some splitted work between UI and server (or I have not found the correct entrypoint)
I'm wondering if I started to look at the right place then.
@igarashitm @pleacu Have i understood well? Do ou think that we (Fuse Tooling dev) can implement the functionality in a reasonable time? Can you provide some guidance please?

It looks this is not trivial to achieve, need some refactoring around startup/working-directory/adm-file handling. Would be better to leave it to @pleacu

I'm tracking down an odd timing issue - I can look at this after that... :)
--Paul
On Wed, Feb 20, 2019 at 12:50 PM Tomohisa Igarashi<EMAIL_ADDRESS>wrote:

It looks this is not trivial to achieve, need some refactoring around
startup/working-directory/adm-file handling. Would be better to leave it to
@pleacu https://github.com/pleacu
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/atlasmap/atlasmap/issues/755#issuecomment-465685010,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC1-LbyUepjNCgGu-CfnFGtQGw7rtHOjks5vPYr9gaJpZM4a0afl
.


Health checks persist after successful deployment

I enabled health checks with deis config:set HEALTHCHECK_URL=/_health for an app and once a deploy successfully completes (the health check endpoint returns a 200 response), I am still seeing periodic (10s) requests on /_health from Go-http-client/1.1. Aren't health checks supposed to stop once one of them passed?

Nope! health checks occur continuously to ensure the health of the container. If it fails, publisher will remove that container from etcd, effectively removing it from the router.

Oh right, makes sense thanks.

I should have tested this more carefully but I'll write down my story here for posterity.
I setup a health check on my app which only returns a 200 status code if the database is reachable.
Unfortunately, on Friday evening, the AWS machine running our database had a hardware failure which caused the database to go down for about 20 minutes. In the meantime, the health check started failing and our app was eventually completely removed from the router, making it display an Nginx error page. When the database eventually went back up, our app was still down because it had failed to many health checks I presume and I had to manually boot it back up.
I have now removed the health check to prevent this sort of situation from happening.


Overhaul State Management

This PR accomplishes a couple of things

Consolidates the graph state management for nodes and edges together with the state management for the config.
Introduces a GraphProvider context and two hooks for working with it called useGraphStore and useGraphApi.
Adds builtin methods for saving and loading graph state, along with a story demonstrating persisting the graph to local storage.
Changes that will support the future development of #10 like our own state management, a NodeInheritingOutputField component, and more.
Support for TailwindCSS styling in preparation for better custom theming

GraphProvider Context
The GraphProvider needs to wrap the Flow component. This automatically happens if you're using the NodeGraphEditor component.
<GraphProvider
  config={props.config}
  initialNodes={defaultNodes}
  initialEdges={defaultEdges}
>
  <ReactFlowProvider>
    <Flow {...props} ref={ref} />
  </ReactFlowProvider>
</GraphProvider>

This context gives you access to two state hooks. Use useGraphStore to subscribe to state changes including changes to the config, or even to get access to state methods like serialize and deserialize.
const config = useGraphStore((store) => store.config)

Use useGraphApi when you want to get point-in-time access to the state without dealing with subscriptions.
const graphApi = useGraphApi()
const node = graphApi.getState().nodes.find(node => node.id === 'foobar')

@sroussey FYI I'm going to merge this soon before it gets too far behind main. It has breaking changes so I'm happy to hear any feedback before I merge the change.


Chapter2/homework/1

I re-coded everything from scratch.
Is (dot) at line 949 after Barbie Pankhurst meant to be there or it's just an error? I treated it as an error and removed it manually in all files.

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 4
- Added 9

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/BalanceStats.cs  9
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/ArrayOps.cs  3
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/StringOps.cs  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 4
- Added 6

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/BalanceStats.cs  9
- Src/BootCamp.Chapter/ArrayOps.cs  3
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/Account.cs  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 4
- Added 5

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/BalanceStats.cs  9
- Src/BootCamp.Chapter/ArrayOps.cs  3
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/Account.cs  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 4
- Added 10

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/BalanceStats.cs  9
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/ArrayOps.cs  3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 23

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/BalanceStats.cs  6
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/Conversion.cs  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 27

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/BalanceStats.cs  8
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/StringOps.cs  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 14

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/BalanceStats.cs  8
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/TextTable.cs  3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 11

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/BalanceStats.cs  5
- Src/BootCamp.Chapter/FileCleaner.cs  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 9

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/BalanceStats.cs  5
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/FileCleaner.cs  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 7

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/AccountOps.cs  3
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/BalanceStats.cs  5

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 5

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/BalanceStats.cs  6
- Src/BootCamp.Chapter/AccountOps.cs  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 5

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/AccountOps.cs  4
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/BalanceStats.cs  6
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/StringOps.cs  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 7

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/BalanceStats.cs  6
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/AccountOps.cs  4

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 7

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/BalanceStats.cs  6
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/AccountOps.cs  4
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/Test.cs  5

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 6

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/AccountOps.cs  4
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/TextTable.cs  3
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/BalanceStats.cs  6

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 5

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/AccountOps.cs  4
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/BalanceStats.cs  6
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/TextTable.cs  3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 7
- Added 4

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/AccountOps.cs  4
- Src/BootCamp.Chapter/ArrayOps.cs  4
- Src/BootCamp.Chapter/FileCleaner.cs  4
- Src/BootCamp.Chapter/Account.cs  2
- Src/BootCamp.Chapter/StringOps.cs  4
- Src/BootCamp.Chapter/Test.cs  5
- Src/BootCamp.Chapter/Conversion.cs  1
- Src/BootCamp.Chapter/BalanceStats.cs  7
- Src/BootCamp.Chapter/TextTable.cs  3

See the complete overview on Codacy

I think it looks good overall and this was what we expected.

Thank you!


Add Get[ChildResources] methods to each parent [Resource]Operations

/azp run autorest.csharp - ci


A PSA about the "world border resetting" bug

Hello,
A while back, my admin posted about a bug that involved my world border resetting back to a 10k radius every time the server restarted (https://github.com/Brettflan/WorldBorder/issues/110). You had responded saying that you have never seen this issue before, and concluded later that perhaps it was an issue with the server version and plugin version. This did not solve the bug, and we stopped using world border and began using the default, though less featureful, Minecraft world border. This problem has continued throughout numerous versions since that post, with multiple versions of the plugin, Minecraft, and Spigot. While attempting to pregenerate my world today, the same bug occurs. If the server restarts, the world border size is reset to 10k.
In fact, I found other users that have had experience with this bug as well

https://www.planetminecraft.com/forums/help/javaedition/worldborder-keeps-resetting-size-580673/

https://www.spigotmc.org/threads/worldborder-settings-reset.338196/

The issue, made apparent to me by a comment on this Spigot thread, states that the problem is likely because the server host Shockbyte is throttling the WorldBorder plugin to have a maximum radius of 10k without telling anyone that this is what is occuring.

Are you using shockbyte as your host? I was getting this issue because shockbyte started restricting the size people set using the world border plugin totally without warning. Glad I'm not using them anymore... - ceruleanReverie

Upon viewing the Shockbyte terms of service, there is an excerpt that mentions WorldBorder, and this is the only mention of the host about this plugin

We do not allow the following plugin(s) to be used on our services: PerfectBackup, BukkitHTTPD, BukkitTelnet, WorldBorder

So, there you have it. Instead of sending an automatic reminder to the server owner that this plugin is not allowed, they sneakily throttle its use to end users.
I wanted to make this apparent to you so that you can now properly respond to these bug reports that involve this situation. Thanks for your previous help with this bug, and I hope this is a help to you and your plugin

Thanks for posting about this, with the info that it is the host ShockByte intentionally covertly causing this problem. I know others with the same problem which is created by ShockByte will find this useful.

Hi, I'm from Shockbyte.
The reason the radius is limited to 10K is because many users mis-use the pre-generation feature (/wb fill command). This results in users generating extremely large worlds, because they did not understand the effects of the command and set the radius too large.
Restrictions like this must be implemented to protect other customers.
As far as I am aware there is no other way to limit or disable the /wb fill command - this is why the restriction is being applied to the border as well.
We have now updated the system so it is possible to exclude individual servers from this restriction.  Any Shockbyte customers wishing to remove the radius limit can now submit a ticket to have the limit removed from their server.


KAFKA-5433: Close SimpleAclAuthorizer in test to close ZK client

@ijuma Can you take a look, please? Thank you...
We need a better way to catch these and fail tests, but for now this just fixes the test to avoid transient build failures.

@rajinisivaram my LGTM was faster than your request for review. ;)

@ijuma Thank you for the review. I will wait for the PR builds to complete before merging. Should I add this to 0.11.0 branch as well?

Yes, please merge it to 0.11.0 as well.


Add support to render custom entities

Inspired by #21
This allows you to add custom renderers for entities like:
let options = {
  entityRenderers: {
    COLOR: {
       attributeMap: {color: 'color'},
       render: (({color}), content) => {
            return `<span style="color: ${color}">${content}</span>`;
        },
     },
  },
};
let html = stateToHTML(contentState, options);

I have adjusted the existing LINK and IMAGE entities to use this new format.

Hello @bramdevries
you make a really good job
can I use your pull request code in my fork project?

@DerekHung Sure. But full disclosure after making this PR and not wanting to wait for it to be merged and released I stumbled on https://github.com/HubSpot/draft-convert which also features importing/exporting entities.

Hi @bramdevries. Thanks for your contribution! I will take a slightly different approach for this one. Similar to #37. I'll try to get it landed and then hopefully that will suite your use case (I see you are already using draft-convert but this is still a good feature to support).


resource templates do not have any logs when archived

Summary
When you have a workflow that uses a resource template and it is archived, you do not see any logs

-> you cannot find the name of the created resource (especially frustrating when it is another workflow and that you have use workflow-of-workflows with dozens of "children")
"Additional" bug (or maybe related) : the error message is not recovered in the UI
instead, there is Error (exit code 1): interface conversion: error is *exec.Error, not *exec.ExitError which seems like a bug in Go

Additional improvement : would be great to have a link to the workflow and to the archived workflow when the created resource is a workflow -> does that need a separate issue ?

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

I think @alexec is fixing Error (exit code 1): interface conversion: error is *exec.Error, not *exec.ExitError in https://github.com/argoproj/argo-workflows/pull/6091

@terrytangyuan thanks for the info, what about the missing logs : is this related to the error or is this completely an issue on its own ?

I am uncertain at this point yet. I think once #6091 is merged, we will see a much clearer error message.

ok, I fallback to v3.1.0-rc12 and the logs are also missing on the archive : no way to find the name of the workflow which has been submitted ...
It happens when the pod is in success or error, indifferrently
Note that if I go to the non archived workflow (TTL is 1h), I can see way too much logs :)

It would be good if the resource could be logged in debug only and keeping info to the "relevant" evaluations like :
time="2021-06-07T15:55:23.163Z" level=info msg="failure condition '{status.phase in [Error Failed]}' evaluated false"
time="2021-06-07T15:55:23.164Z" level=info msg="success condition '{status.phase == [Succeeded]}' evaluated false"
time="2021-06-07T15:55:23.164Z" level=info msg="0/1 success conditions matched"
time="2021-06-07T15:55:23.164Z" level=info msg="Waiting for resource workflow.argoproj.io/dataset-k6h2h in namespace staging-argo resulted in retryable error: Neither success condition nor the failure condition has been matched. Retrying..."

Also note that these "temporary" logs (because they disappear in the archive) always end by a "Retrying" (whereas it is clear that something has been evaluated to true to move on ...) -> it seems that the latest logs are not captured correctly
For a resource template, do not you think that argo should automatically create an output with some infos about the created resource ?
at least, the archived logs should tell the id of the created resource (again especially when this is another workflow, it maybe could provide a link to the workflow and to the archive ...)

Note that if I go to the non archived workflow (TTL is 1h), I can see way too much logs :)

I agree this is too much log. I've adjusted it to debug level in https://github.com/argoproj/argo-workflows/pull/6100.

For a resource template, do not you think that argo should automatically create an output with some infos about the created resource ?

For this, you can use output parameters + jsonPath/jqFilter to obtain: https://github.com/argoproj/argo-workflows/blob/master/examples/k8s-jobs.yaml#L44-L56

I tested the outputs.parameters from jsonpath, it worked great and so I documented this behavior in the appropriate section

  outputs: {
    parameters: [
      { name: "name", valueFrom: { jsonPath: '{.metadata.name}' } },
      { name: "namespace", valueFrom: { jsonPath: '{.metadata.namespace}' } },
      { name: "uid", valueFrom: { jsonPath: '{.metadata.uid}' } },
    ]
  }

Do you have any idea to generate a direct link to the workflow archive ?
This is basically just the concat of the namespace and the uid, something like :
see archive


Variants within a gene should be sorted according amino acid or genome coordinate

Sorting of variants is important to an intuitive display.  For example:
Sorted by position:
L8Q, G12D, V600E
Is not as intuitive as sorted alphabetically:
G12D, L8Q, V600E
We can wait on fixing this until we have coordinates entered for all variants.  Then we can use chromosome position to accomplish the sorting...

I'll just need the sort order sent over from civic-server w/ any endpoint that returns multiple variants. I've created an issue in civic-server, github.com/genome/civic-server#41, for this enhancement.

Further refinement of this idea.  Sort should be based on Variant Start coordinate for all variants where it is defined.
If Start is not defined, these will go at the bottom (sorted by name).

Could this get tricky with fusions? Since the second coordinate is for the 3' partner, which is usually the important gene which it should be filed under. I guess a workaround for that would be to prioritize secondary coordinates, when available.

Yeah, you are right, fusions would need to be handled as a special case.  This will be hard because sometimes the fusion is entered under the 5' partner and sometimes under the 3' partner.  And which is the case is not recorded in the database...  As long as the name uses official gene names, you could use that to figure it out...  e.g. BCR-ABL1 is entered under ABL1, therefore use the coordinates for the 3' partner.  But that seems a bit clunky.

I guess you could look for a match to the mygeneinfo chromosome and pick the coordinates. DNAJB1:PRKACA would be an obvious exception, but for the most part maybe that would work. Or use the secondary coordinate unless the chromosome doesn't match?

For this to be implemented on the client, server endpoints returning multiple variants should include a field the client can use for sorting by amino acid or germline coord.

Issue of variant sorting also ties into larger issues of variant display and there is user feedback on this to be considered, it seems a V2 issue

This is a subissue of https://github.com/griffithlab/civic-client/issues/630

This has been implemented


Development

add button to switch between light and dark mode, changed login background image to webp, add portfolio website to footer + some fix


All return type hints should be fully-qualified

If return type hints are not fully-qualified, they can require the returned class to be imported in the calling code, which may not even know about the underlying type hint. This is the simplest case where I can cause this to occur with the current code base:
(require 'metrics.timers)
(.stop (metrics.timers/start (metrics.timers/timer "t")))
;; => CompilerException java.lang.IllegalArgumentException: Unable to resolve classname: Timer$Context

I realize this is a nonsensical example, however it can occur in real circumstances where it is impossible for the caller to fix. For example, when using metrics-clojure in tandem with core.async and wishing to wrap a timer around the body of a go block, you could do this:
(require '[clojure.core.async :as async])
(require '[metrics.timers])

(def my-timer (metrics.timers/timer "t"))
(async/go
  (let [c (metrics.timers/start my-timer)]
    (+ 1 1)
    (metrics.timers/stop c)))

;;=> ExceptionInfo Class not found: Timer$Context

The error here is in relation to the non-qualified return type on the metrics.timers/start function. In this case, the place where the class would need to be imported to fix the error is inside a third-party, specifically clojure.tools.analyzer.jvm. So there is no practical way for a user to fix this.
To fix, all return type hints in this library should be fully-qualified.
Here is a comment from Rich Hickey about this behavior: http://dev.clojure.org/jira/browse/CLJ-1232?focusedCommentId=34819&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-34819
Regarding that ticket, I would like to be noisy about this problem when it is created rather than silent as it is now, but this is the current behavior.
Eastwood (https://github.com/jonase/eastwood) has a check for this problem to find it in static analysis under the https://github.com/jonase/eastwood#wrong-tag linter.

Thank you, this is indeed something I've run into myself.

Thanks!

It seems like the pr creates an invalid type of ^clojure.core$long - that should just be ^long as it was. Also, the change does not fix a number of other cases I see browsing the code:

metrics.core/new-registry - ^MetricRegistry
metrics.timers/timer - ^Timer
metrics.utils/snapshot - ^Snapshot
metrics.reporters.console/reporter - ^ConsoleReporter
metrics.reporters.csv/reporter - ^CsvReporter
metrics.reporters.jmx/reporter - ^JmxReporter
metrics.reporters.ganglia/reporter - ^GangliaReporter
metrics.reporters.graphite/reporter - ^GraphiteReporter

Sorry, I went off the :wrong-tags linter on eastwood.  I'll go back through it.

I didn't see any that @puredanger didn't point out.  Let me know if there's anything I need to take a closer look at.

@jwhitlark @puredanger this is in 2.5.0.


Add RecordingSshTool clears to selected tests

Noticed test output from SCMTPT failing with
the tool output referring to the commands from RHSCST in a
brooklyn-server build failure.

@tbouron @aledsage could you have a look?

retest this please

Looks sensible @geomacy but it seems adding but another test failed: https://builds.apache.org/job/brooklyn-server-pull-requests/org.apache.brooklyn$brooklyn-software-base/2912/testReport/junit/org.apache.brooklyn.feed.jmx/RebindJmxFeedTest/testJmxFeedIsPersistedWithPreCreatedJmxHelper/
I'm wondering if it's releated

The error is 2018-04-30 11:11:04,475 WARN  - JmxService couldn't start test mbean (java.rmi.server.ExportException: Port already in use: 40124; nested exception is:  	java.net.BindException: Address already in use (Bind failed)); definitive; throwing java.rmi.server.ExportException: Port already in use: 40124; nested exception is:
can't see that it could be related to the RecordingSshTool change.
Retest this please.

Just to see what happens.

retest this please

Tests passed: https://builds.apache.org/view/B/view/Brooklyn/job/brooklyn-server-pull-requests/2916
Thanks @geomacy, merging


When a wallet account receives a transfer from tz1YtKsJMx5FqhULTDzNxs9r9QYHBGsmz58o

Display a lmessage "You ghave receive a  reward of .....USD"

J'en parle avec Taleb ce matin pour savoir si on a déjà l'info, ce que je ne crois pas... A part quand on affiche l'activity.

A timer which check last  UNO and XTZ received
When wallet receive tokens from tz1YtKsJMx5FqhULTDzNxs9r9QYHBGsmz58o we display message to user in popup: ""
When wallet receive from tz1YtKsJMx5FqhULTDzNxs9r9QYHBGsmz58o  we store transaction id as lastNotifiedReward
Timer is launch with the app for addresses of Tezotopia vouchers, on mainnet.

M Size

it does not work cotrrectly when you send xtz from temple for instance (with the correct address)

request private key to romuald to test it

It is ok for UNO transfer but it does not work with XTZ

@ThierryThevenet  Can you provide TransactionId? I want to check again

@ThierryThevenet  Can you provide TransactionId? I want to check again


[GH-334] Check error message when genesis/last block too old.

https://github.com/input-output-hk/cardano-node/issues/334
The transformers were left alone since I see no reason to transform anything in here.
Wasn't tested with the local node.

Erm. Hydra failed?

bors try

Chairman is failing.


startingDirectory formatting note

Summary of the Pull Request

References

PR Checklist

[ ] Closes #xxx
[ ] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Accidentally reversed them, you can only put a forward slash instead of a backslash (default windows directory) otherwise you get this

That's actually just because you need to escape the \ as \\. That's just JSON.


Converting a time zone should be done by changing region, not with UTC and time zone funcs

Currently the funcs fromTimeZone and inUTCDate complicate conversions. Converting should be as easy as:
let date1 = NSDate()

// I want to know the time here locally and in China
let china = Region(timeZone: chinaTimeZone)
let date2 = date1.inRegion(china)

// Now let us see what we have
print(date1.hour)  // prints the local hour
print(date2.hour)  // prints the hour in china in our calendar and locale

Mind that this is pseudo code to illustrate the issue.

So should be like current inRegion() function but it takes directly the timezone to create a default region without passing other optional params?

Correct, as implemented in fd73ef7bb9b28d69859260407acb6c01858b48f2

Fixed in v3.0.0


API review: LoggerStructureBase and LoggerStructureFormat polish

LoggerStructureBase

Rename to ReflecionBasedLogValues (Check with Glenn)
Remove Message (another work item tracks removal of this from the interface)

LoggerStructureFormat

Move to .Internal namespace
Rename class to FormattedLogValues
Should follow pattern similar to LoggerStructureBase
Remove Message (another work item tracks removal of this from the interface)

@glennc ReflecionBasedLogValues?
Also, LoggerStructureFormatter is only used in the Logging repo so I'm thinking move it to .Internal namespace too (and renaming, LogValuesFormatter?)

Yes to both of those questions.


build warning "This async method lacks 'await' operators"

connector.ActivityReceived += async (sender, activityReceivedEventArgs) => ...
I get a build warning:
Warning	CS1998	This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.	helloworld	E:\Workplace\2019-DevEx\CC-Docs\How-to-guide\helloworld\MainPage.xaml.cs	172	Active
Please fix the code so there are no build warnings.

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

ID: e692d4bf-af38-e13e-baf8-553fe3ee3ca8
Version Independent ID: 917e33a6-cbb6-0759-1330-8c7b2cfc978b
Content: Send Custom Commands activity to client application - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/how-to-custom-commands-send-activity-to-client.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @xiaojul
Microsoft Alias: xiaojul

Note that this is BEFORE I paste the code in section 'Handle the received payload'. The build warning is for the version of the ActivityReceived method that just plays audio, not handles JSON activity to display on/off state.

@dargilco thanks for your feedback. I have referenced this issue above and will proceed to close. Thanks.


Different jobs are supported in preheat job, it would be nice the job type can be selected in console

Bug report:

It would be nice to support select & configure different job types in preheat job creation pages.
Expected behavior:

How to reproduce it:

@TommyLike  The console currently only supports file type preheating. We will consider the issues you mentioned.


Add a note about not using postinstall to npm package developers

prepare could be suggested as an alternative to postinstall in this case.


Added support for installing the npm package with JSPM

Small addition to package.json to allow users of JSPM installing the prismjs npm package to experience a smooth install experience.
JSPM installation Instructions:
Installing the prismjs with jspm
jspm i npm:prismjs

loading prism with systemjs/ jspm:
import "prismjs";

All required files will be installed, but the theme and/or plugins will not be loaded by default.
To load a theme (requires systemjs plugin-css jspm i css):
import "prismjs/themes/prism.css!"

or
import "prismjs/themes/prism-dark.css!"

Plugins are not loaded by default, but can be loaded by performing:
import "prismjs/plugins/show-language/prism-show-language";
import "prismjs/plugins/show-language/prism-show-language.css!";

Would love to see the main npm package support JSPM.
Saves me time to keep a scoped npm package up-to-date :-).
Thanks,
Erik

Thanks!


Udpate glide.yaml and glide.lock to new kubernetes/service-catalog

Fix #670

@dgolovin, I also faced similar issues,
go build -o odo ./main.go
# github.com/redhat-developer/odo/vendor/github.com/kubernetes-incubator/service-catalog/pkg/client/clientset_generated/clientset/fake
vendor/github.com/kubernetes-incubator/service-catalog/pkg/client/clientset_generated/clientset/fake/clientset_generated.go:52:18: o.Watch undefined (type testing.ObjectTracker has no field or method Watch)

I am trying to use dep and will see whether this solves the problem

@dgolovin I tried the glide update thing yesterday and it fails with the same error mentioned above by @surajnarwade . Also the CI is failing with the same error

@surajnarwade for kubernetes-catalog build should also fail, but it is not because (I guess) they are re-generating that code during the build before compiling it.

Closing this in favour of https://github.com/redhat-developer/odo/pull/716


Bug in stitching module when applying composePanorama multiple times

Attempting to use composePanorama multiple times from the stitching module, yields very strange results. First time composePanorama is used, the resulting panorama is perfect, however the next time composePanorama, from the same stitch object is used, the resulting panorama is very strange.
The code that triggers the bug, can be boiled down to:
Stitcher stitcher = Stitcher::createDefault(try_use_gpu);
Mat pano; stitcher.composePanorama(pano);
Mat pano2; stitcher.composePanorama(imgs, pano2);

this blog post describes and illustrates the problem very well: http://www.scriptscoop.net/t/8d0bf668c48b/opencv-stitcher-class-with-overlapping-stationary-cameras.html

The stitching module is in the main repository, please move this issue there.


Keyword argument constructors, show types, README update

modified:   README.md	Example with (suppressed) output
modified:   src/HTTP.jl Keyword arguments: ServerWS, serve
modified:   src/show_ws.jl  Drop "application/juno+inline"
Base.show for ServerWS
modified:   test/show_test.jl Test all this
modified:   test/throttling_test.jl Test serve keywords
Refer issue #128

Coverage increased (+0.9%) to 78.25% when pulling d9a355e2bd0b9a50275d58f2f54e6bb489610e72 on issue128 into 89fb3ec89c13ce0d29bc4ab09d686a8aec7453a6 on master.


The depth smooth loss goes to zero

Dear Bolian, thank you for providing this code. While I was training on KITTI, the depth goes to zero. This is the screen cut of my trainig results.

Do you have any idea about it?
Regards,
Fei

@feipan664 Dear Feipan,
I just launched trainings on KITTI to find clues about the reported issue. I will give you feedback soon.
Best,
Bolian

Thank you for your quick reply. I have set up the depth smooth loss with 1e-3, and re-started the training. Now after 1 epoch, I can get this disparity map:

@feipan664 Thank you very much for the feedback. It is a very good reference.
I tried trainings with --learn_intrinsics option being activated and not and couldn't reproduce your results. If you followed the code  to generate the training data, I guess the optimal hyper-parameter value might be affected by the difference in GPUs or PyTorch versions.  I tried (pytorch 1.5 + torchvision 0.6 + rtx2080Ti) and (pytorch 1.7.1 + torch 0.8.2 + rtx2070) and was able to produce depth while keeping smooth loss weight as 1e-2.

python train.py --data_path $FULL_KITTI  \
                --png  --overwrite_old --boxify \
                --learn_intrinsics \
                --model_name "20220430/full_kitti_learn_cam"

python train.py --data_path $FULL_KITTI  \
                --png  --overwrite_old --boxify \
                --model_name "20220430/full_kitti_learn_cam"

Thank you so much for the help. I follow your data generation code to prepare my KITTI data. My environment is pytorch1.7.0 + torchvision0.8.1 + RTX 2080 Ti. With smooth loss weight set up to 1e-3, I got these results so far:

I would like to upgrade my PyTorch ver and run it again.
Thank you again, Bolian!


Do not expose mutex methods

expectedBase embeds a mutex and its methods are exposed. Confusing UX :)

yes i hide them


cue-py: CI fails on Windows

See e.g: https://github.com/cue-lang/cue-py-trybot/actions/runs/9697210765/job/26760829584
Strange that CI regeneration fails on Windows, it works on other platforms, e.g. Linux: https://github.com/cue-lang/cue-py-trybot/actions/runs/9697210765/job/26760828869
Git is not clean after CI regeneration for some reason.

I am not sure why @cueckoo closes this issue, it's being actively worked on.
CL 1196827 fixes the CR/LF normalization issue: https://review.gerrithub.io/c/cue-lang/cue-py/+/1196827
Next step is to add cue.dll to PATH: https://github.com/cue-lang/cue-py-trybot/actions/runs/9697854160/job/26762999555

CL 1196833 adds cue.dll to PATH.
The next step is identifying why dlopen doesn't like cue.dll: https://github.com/cue-lang/cue-py-trybot/actions/runs/9699260585/job/26767792648
This seems the same issue as #3016.


Can not get wallet balance

Hello.
I am running the node since yesterday, daemon is also running and shows "Wallet is synced, ready for queries" but when I run the command for get the balance I get: "Total balance,  KLS"
Anyone know how to fix this?
Thanks in advance.

Yes habe the Same Problem....--Diese Nachricht wurde von meinem Android Mobiltelefon mit GMX Mail gesendet.Am 01.12.23, 16:43 schrieb SpindlerJ @.***>:
I'm having the same problem. On github, when you download the wallet, it says there's a requirement to have Go installed. I'm going to try this, but I doubt, that this will fix the issue. I've also noticed, that the node doesn't give you a message whether it is synced or not.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

same problem here, don't show balance, i've been trying since yesterday
 and this second one, always syncing, i already closed a few times, but no balance

I already fixed, anyone who has problem, go to discord e download database and replace yours.
https://discord.gg/tvJKR4T5

hello, i have the same problem, balance dont show, i tray to install the datadir2 but i dont know where to put it ,i dont see C:\Users\Username\AppData\Local\   "Karlsend\karlsen-mainnet\datadir2"

same problem here, don't show balance, i've been trying since yesterday
 and this second one, always syncing, i already closed a few times, but no balance

I already fixed, anyone who has problem, go to discord e download database and replace yours. https://discord.gg/tvJKR4T5
the link is not showing anything please help

Can we check our balance WHILE the mining program is running??? OR do we need to shut that down then run all the commands to see the balance?????     someone PLEASE READ AND ANSWER

PLEASE ELABORATE WHEN DROPPING LINKS ITS NOT HARD TO WRITE A SENTENCE

I unziped the datadir2 into my karlsen folder.  shows datadir2 folder with the karlsen exe and bat files.  do i just leave datadir2 there and run the 3 commands as mentioned above.  I do not see a dir file to be replaced as mentioned above

This isn't a support chat, please report bugs with some minimal reproduction scenario please. To give an answer to the question about balance.
Wallet Sync != Node Sync, so please check node sync status with RPC call GetInfo, response should be something like:
{
    "getInfoResponse": {
        "p2pId": "f9d4cf4b697e4c2a2befcd2736270b08",
        "mempoolSize": "1",
        "serverVersion": "1.0.0",
        "isUtxoIndexed": true,
        "isSynced": true,
        "error": null
    }
}

If it is not sync, checks the virtualDaaScore via RPC call GetBlockDagInfo, response should be something like:
{
    "getBlockDagInfoResponse": {
        "networkName": "karlsen-mainnet",
        "blockCount": "231154",
        "headerCount": "4133883",
        "tipHashes": [
            "a4bf24cfdfb3b7108af2ae92295f36ffb52d7de97ac8aba78595303a0d3407e1"
        ],
        "difficulty":<PHONE_NUMBER>11804.88,
        "pastMedianTime": "1702543041181",
        "virtualParentHashes": [
            "a4bf24cfdfb3b7108af2ae92295f36ffb52d7de97ac8aba78595303a0d3407e1"
        ],
        "pruningPointHash": "3a069a08e9b774d24995caecec6de1d33c0fae50869633035f6f63e4e6336a0a",
        "virtualDaaScore": "3353047",
        "error": null
    }
}

RPC calls can be run via ./karlsenctl <RPC-command>.


Constant value fields

It's often useful to associate key-value pairs at the level of a RecordSet. We propose doing that by adding a "value" property to Field that allows it to have a fixed value.
This is similar to the "data" property of RecordSet, that makes it possible to specify the values (records) contained in a RecordSet directly in the Croissant definition.
The "value" of a Field can be atomic, for Fields with a simple dataType, or it can be structured, in case the Field has SubFields. For the latter case, a JSON string can be used to represent the value.

Do you have a specific example use case where this would be helpful?


added 'python-task' to registered task types for pod plugin

TL;DR
Python tasks are annotated with "python-task" for the task type. In FlytePlugins we do not have a Handler that registers with that task type, so in every case it falls back to the container handler which is registered as the default for everything that does not match an existing handler. However, this results in log messages like No plugin found for Handler-type [python-task], defaulting to [container] everytime that propeller evaluates a "python-task" node. This can result in a very large number of unuseful log messages, effectively reducing the utility of logging. Adding "python-task" as a registered type will remove this message from the logs.
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[ ] Unit tests added
[x] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
^^^
Tracking Issue
NA
Follow-up issue
NA

cc @nicklofaso

Very helpful! Thanks @hamersaw


REEF 151 tests

Opening another PR to trigger a build.

Yay! It built. Closing this PR to clean up the commits and open the final one.


fix(api): pass user agent when proxying news api

Description

Fix missing user-agent error when proxying news-api
How has this been tested?

Verified that the API does not return any error by starting React app

Types of changes

[ ] New feature - feat
[x] Bug fix - fix
[ ] Refactor - refactor
[ ] Test - test
[ ] Other(s):

Checklist:

[ ] Test have been written for utilities.
[ ] I have test my code on different platforms and devices.

Remarks

N/A

Codecov Report

Merging #4 (ba7ce23) into master (691682d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           17        17
  Lines          139       139
  Branches        24        24
=========================================
  Hits           139       139

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 691682d...ba7ce23. Read the comment docs.


Supervised speech tokenizer

Is your feature request related to a problem? Please describe.
The paper includes methods about "Supervised speech tokenizer", but the repository does not contain the training and reconstruction test code for this part.
Describe the solution you'd like
Did you perhaps forget to open-source this part of the code? Will you open-source this implementation in the future?

we have no plan to opensource speech tokenizer, we only release its onnx file

we have no plan to opensource speech tokenizer, we only release its onnx file

This message is quite regrettable, but thank you for your quick reply!
However, I have another question I would like to consult with you about. When you were conducting experiments with the speech tokenizer, how did you determine that "using ASR as backbone and having the tokenizer provide only semantic information (without acoustic information)" was feasible?
Thank you！

we have no plan to opensource speech tokenizer, we only release its onnx file

This message is quite regrettable, but thank you for your quick reply!
However, I have another question I would like to consult with you about. When you were conducting experiments with the speech tokenizer, how did you determine that "using ASR as backbone and having the tokenizer provide only semantic information (without acoustic information)" was feasible?
Thank you！

we did some experiement and verified that such token can recover original mel info. @ZhihaoDU can tell you more details

we have no plan to opensource speech tokenizer, we only release its onnx file

This message is quite regrettable, but thank you for your quick reply!
However, I have another question I would like to consult with you about. When you were conducting experiments with the speech tokenizer, how did you determine that "using ASR as backbone and having the tokenizer provide only semantic information (without acoustic information)" was feasible?
Thank you！

@ZhihaoDU Could you provide some detailed and specific information? I would be very grateful.

we have no plan to opensource speech tokenizer, we only release its onnx file

This message is quite regrettable, but thank you for your quick reply!
However, I have another question I would like to consult with you about. When you were conducting experiments with the speech tokenizer, how did you determine that "using ASR as backbone and having the tokenizer provide only semantic information (without acoustic information)" was feasible?
Thank you！

@ZhihaoDU Could you provide some detailed and specific information? I would be very grateful.

Thanks  for your attention. I believe, we have not claimed there is only semantic information (without any acoustic information) in the S3 tokens. In fact, decoupling semantic and acoustic information is still an open problem to my best knowledge. What we can do is modeling semantic information in the token as more as possible. That is also the motivation, we employ the ASR task to learn the tokenizer in a supervised manner. We think such training process can maintain more semantic information into the tokens than self-supervised or unsupervised methods, such as Encoder, SoundStream, HuBert, wave2bert and so on.

@ZhihaoDU
Thank you for your explanation of the motivation behind the speech tokenizer in the paper; it has been very helpful to me.
I will close this issues.


question: how to add a component dynamically

If I have some code like the following. how would I add any component at the given location? Having a bunch of if/switch statements is cumbersome.
<button onclick="@addcomp1">add comp1</button>
<button onclick="@addcomp2">add comp2</button>
<div style="border:dashed 1px black;width:500px;height:100px">
     /// component goes here
</div>
@functions{
 protected void addcomp1()
    {
       // what goes here?
    }
}

@Flearz Not yet. The above advice still applies.


Fix compilation issues against crystal nightly

Nightly specs are failing, these should be resolved ASAP

Resolved by crystal-lang/crystal/pull/10857


Apply maybe_convert_array to all class functions in Timber\Twig

As created in #1788, implement the maybe_convert_array method throughout Timber\Twig
https://github.com/timber/timber/blob/2.x/lib/Twig.php#L91-L93
$twig->addFunction( new Twig_Function( 'Attachment', function( $post_id, $attachment_class = 'Timber\Attachment' ) {
	return self::maybe_convert_array( $post_id, $attachment_class );
} ) );

@jarednova Think this might be a decent first contribution? If so, want to keep running with pid as opposed to post_id for argument naming convention?

def. post_id please! clarity > brevity

I’m going to work this out tonight! Assuming the PR should be against the 2.x branch too 😉

Created that PR whenever you have a chance @gchtr and @jarednova, thanks!


Copy - JPB Review Page (French)

Description
On the French Job Poster Review page, under "Durée", instead of "Months", it should be "mois".

I hope this is fixed already, please fix if not.


jQuery mousedown event does not get triggered once the zoom lense is active

I have tried a lot but the mousedown event does not fire when the zoom lens is active. We need to track the mousedown event so that we are able to find the mouse position and then use RaphaelJs to draw lines on the image.
Can you please take a look at this and let me know if there is a workaround.

It should work if you use event delegation in jQuery and bind it to lens container.

@thdoan thank you for your reply I will try this

@thdoan I have been struggling with the solution posted by you. Would be really grateful if you have a look at the Stackoverflow question I have created. I have a link to the jsFiddle as well in there for a clear picture on the problem I am facing.

https://stackoverflow.com/questions/54454260/a-way-to-bind-the-mousedown-event-on-magnifier-to-raphael-canvas

As a quick fix, try changing line 232 from $container.off().on(...) to $container.on(...).


Bugfix/two col features

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #172
Test URLs:

Before: https://main--bitdefender--hlxsites.hlx.page/drafts/scoman/features
After: https://bugfix-two-col-features--bitdefender--hlxsites.hlx.page/drafts/scoman/features

Reopened with the code being based on main: https://github.com/hlxsites/bitdefender/issues/172