[BUG] flytekit module should hide imported packages/classes

Describe the bug
Historically flytekit has been pretty good at strictly hiding imported packages/classes in the shape of from foo import bar as _bar. This is a very good practice to avoid users of flytekit accidentally importing undesired stuff into their code space.
In recent code changes it seems this is not the case anymore.
Expected behavior
Imported packages and classes are hidden from users.
Flyte component

[ ] Overall
[ ] Flyte Setup and Installation scripts
[ ] Flyte Documentation
[ ] Flyte communication (slack/email etc)
[ ] FlytePropeller
[ ] FlyteIDL (Flyte specification language)
[x] Flytekit (Python SDK)
[ ] FlyteAdmin (Control Plane service)
[ ] FlytePlugins
[ ] DataCatalog
[ ] FlyteStdlib (common libraries)
[ ] FlyteConsole (UI)
[ ] Other

To Reproduce
N/A
Screenshots
N/A
Environment
Flyte component

[ ] Sandbox (local or on one machine)
[ ] Cloud hosted

[ ] AWS
[ ] GCP
[ ] Azure

[ ] Baremetal
[ ] Other

Additional context
N/A

@honnix so the new way we are trying to do is -
from flytekit import x we are trying to expose things are a module level. do you think thats ok?

@honnix so the new way we are trying to do is -
from flytekit import x we are trying to expose things are a module level. do you think thats ok?

I don't really have a strong opinion, so please feel free to close this.

I don't really have a strong opinion, so please feel free to close this.

closing since imports are now exposed at the module level


Think about search

General:

Algolia: should be easy to add this because its well documented and laravel-scout comes with and algolia driver; however its only free in the community version
Elasticsearch: should be more complex to integrate but there are several packages that aim to make it easy to integrate it
Laravel Scout: our starting point; should probably be used in combination with elasticsearch or algolia for indexing and searching our eloquent models, comes with a ready-to-go algolia driver

Packages:

laravel-scout-elasticsearch: aims to provides a simple way to use elasticsearch; looks promising and maintained
spatie/laravel-searchable

As discussed, we need to add simple mysql search. Later on we can have a look in how we introduce scout :-)
We need to introduce a property which columns can be searched and then we need to build up a simple query with where clauses for a model which is declared as Searchable.


Update bottle to 0.13-dev, use python3 in run and test_server

Using the dev version of bottle just because I copied it straight from github. It works, so no worries.
Also, I just "fixed" the test_server.py script without getting into detail as to why it needs to reload the sys module (it most probably does not). Again, it works in python 3.x just fine.

By the way, how about putting it into requirements.txt and pip installing it on Travis instead of keeping the bottle.py file in your repo? Can do that no problem.

@all3fox
This touches SUB: continuous integration so I'd like the opinion of the subject owner on this.
@AXGKI, I think positively of this pull request, how about you?
(And by the way, @AXGKI, how are you these days?)

Personal opinion: as python developer, one would expect either pip install -r requirements and then python setup.py test or pytest or nosetests or tox or whatever "classic" to do the job. Instead, you really have to read .travis.yml, then README.md in development/continuous_integration and then (which is the best actually) find the transcrypt youtube video which shows you how things work. That is too slow/discouraging for newcomers.
Some tests probably could do away with node.js instead of a full-blown browser environment. If you know how to set such things up (you run python code, then run js code, then compare the results. Preferably inside a known framework like pytest or unittests), please explain the basic components/layout.
The idea here is, for instance, how jquery does it: they have a healthy amount of their test base that just runs with a conventional npm run test and then, once you really need it, you can setup the LAMP (or whatever stack) to run the more involved unit tests.

Hey guys, I'm totally sorry for my silence, got pretty involved this year in other stuffs. In general I'm well though, thanks for asking Jacques @JdeH , glad to see Transcrypt does so well, it deserves it!!
@all3fox: Your personal opinion is for sure justified, the whole thing was a one night hack to have a real browser running through all the autotests of Jacques. Meaning: All was basically there, just the browser automation was missing, thats why I wrapped some javascript around them, to have the browser hit one test after the other and served them from bottle. Its awfully non standard but it got us over the important start of the second phase of the project when more people got involved with PRs.
I honestly don't know how far you get w/o a real browser, I just know I tried with phantomjs and failed miserably - but I'm really not good in javascript, maybe there are better possibilities these days.
All in all: I think in this phase of Transcrypt a rewrite of the test framework would be justified. Alone - this year I won't be able to contribute, timewise I'm afraid, but for sure I can explain you what I did in detail, if you intend to rewrite my stuff.

Well you don't need JS to run PhantomJS at all anymore.. Maybe use Selenium? That's why it was made for ? :P

@the1337guy : Yes, selenium would be the tool to go for - once we have browser js compat errors (do we?) - but does not get you around the need for the test server, guiding the browser through the autotests, no?. And getting firefox up in headless mode in travis was not that big an issue, so I took that road.

Well, Selenium does have error wrapping services plus you might not use Firefox at all, PhantomJS can be used with Selenium.

Ok, let's put a pin into the Selenium idea and into the "some common test framework" idea.
I would like to suggest to focus on the immediate issues this PR addresses:

It updates bottle framework and makes it use python3
By doing that, it eliminates the need to juggle between a python2-driven virtual environment to run tests and a python3-driven virtual environment to develop Transcrypt.

Merging this PR will make at least my experience more pleasant and should not break the existing workflow for others.

@all3fox: Alex I see me=was requested for review - what exactly you want here? I mean the print functions should work. Regarding bottle I fear I can't help much, in my company (and industry) we are still at py2, have no experience with 3. What I can is try explain you the mechanics, you're requesting that?

Hey @AXGKl to be honest I would like you to just say that this PR makes sense. Also, if you take a closer look, you will see that it was @JdeH who requested the review:
https://github.com/QQuick/Transcrypt/pull/344#event-1137701145
Finally, your youtube video explains a lot and makes a lot of sense, so I do not have any questions :)
Finally-finally, to accept a review (since it was requested of you) you need to follow these guidelines: https://help.github.com/articles/approving-a-pull-request-with-required-reviews/
Basically, you can just do steps 5 and 7 from that guide even without leaving any messages.


send image Buffer error

 const address = "0x235fF667f9F4eC9F84d886Bd4e9361d0409b0CC0" const url = await QRCode.toBuffer(ethereum:${address}?value=${rd(Number(0.1)*1e18, 0)}, { width: 400}) await ctx.replyWithPhoto(url, { caption: 👋 Hi, ${ctx.from.first_name}! \n\n🌟 Deposit To Upgrade: \n\n💳 Address: VGpLHmTqkGxcwkVzVypAUpyxccCqexiPgtmjQJWirSy\n🔸Chain: BSC\n💰 Amount: 0.1 BNB, parse_mode: 'HTML'})
Not working, please help

Please start reading documentation: https://grammy.dev/guide/files#uploading-raw-binary-data


hf dict cfg overrides

fixes issue: https://github.com/mosaicml/llm-foundry/issues/89

Should these same changes be incorporated into hf_prefix_lm.py and hf_t5.py?


Is there something in robovm to replace Object-c's  __weak?

This is not a support forum. Please post questions to https://groups.google.com/group/robovm.


No data if internet is down

So my roomba tried to eat the power cable for the fiber converter tonight, and the second the connection went down the data from the integration dissapeared. This is a major issue considering this data may be used to control critical things like heating. Not sure why it would need an internet connection other than when fetching prices at 1400 anyways? The nordpool integration had no issues.
Log:

Logger: custom_components.entsoe.coordinator
Source: helpers/update_coordinator.py:151
Integration: ENTSO-e Transparency Platform (documentation, issues)
First occurred: 04:31:45 (2 occurrences)
Last logged: 04:31:46

Error fetching ENTSO-e coordinator data: Unexcpected error when fetching ENTSO-e prices: HTTPSConnectionPool(host='web-api.tp.entsoe.eu', port=443): Max retries exceeded with url: /api?documentType=A44&in_Domain=10YNO-3--------J&out_Domain=10YNO-3--------J&securityToken=xxx&periodStart=202212012300&periodEnd=202212062200 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x9dd66340>: Failed to establish a new connection: [Errno -3] Try again'))
Error fetching ENTSO-e coordinator data: Unexcpected error when fetching ENTSO-e prices: HTTPSConnectionPool(host='web-api.tp.entsoe.eu', port=443): Max retries exceeded with url: /api?documentType=A44&in_Domain=10YNO-3--------J&out_Domain=10YNO-3--------J&securityToken=xxx&periodStart=202212012300&periodEnd=202212062200 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x9a88d238>: Failed to establish a new connection: [Errno -3] Try `again'))`

Screenshot of the history of a few of the sensors from entso-e, and some of the sensors used to control my water heater and electric heat when I'm not home.
Entsoe network down

The nature of this issue matches #86.

We'll continue this issue in #86


Passed final Payment deadline

The system is confused because the payment deadline is passed. It is still allowing people to pay 50%, but apparently telling them they paid in full. Reporting on the back end shows actual payment and amount due.
example:
Camper paid 50% on July 6th but was sent this confirmation-

Initial Payment: Full Payment
Amount you are paying now: $921
Due by June 20th: $0
Your total: $921

Ideally partial payment should not be an option past the final payment deadline. New registrants should be required to pay in full when they register after June 20th.

This was manually fixed, I just removed the deposit options from the DB


matrices not updated in bilateral model

While debugging my code I stumbled over an interesting problem which only arises when changing the modalities of a bilateral model.
Updating the modalities in any model should update the observation_matrix. This also works for the ipsilateral side, but not for the contralateral side. It seems as if callback for the contralateral side is not successful. I am not 100% sure whether this is due to the synchronization or if there is another problem.
I am working on a  fix, but I wanted to notify you since you probably can detect the problem faster and produce a better fix.
Edit: my fast fix was quite simple. Since the problem is that the contralateral side does not trigger callbacks anymore, I am applying a different syncing function for the modalities:
def init_dict_sync2(
    this: AbstractLookupDict,
    other: AbstractLookupDict,
) -> None:
    """Add callback to ``this`` to sync with ``other``."""
    def sync():
        other.clear()
        other.update(this)

    this.trigger_callbacks.append(sync)

Thus we also need to change the way modalities are synced in the init_synchronization function:
       # Sync modalities
        if self.is_symmetric["modalities"]:
            init_dict_sync2(
                this=self.ipsi.modalities,
                other=self.contra.modalities,
            )

I am sure that you can come up with a more elegant solution, as I am struggling a bit to get a good overview of all the callbacks and syncing functions.

I think I found the problem. Even though the documentation states, that the modalities are synchronized by default, there is no part in the code which would assure that. In lines 177-181 we use:
  delegated_attrs = [
      "max_time", "t_stages", "diag_time_dists",
      "is_binary", "is_trinary",
  ] + ["modalities"] if self.is_symmetric["modalities"] else []
  self.init_delegation(ipsi=delegated_attrs)

But the the dictionary is_symmetric is initialized empty. Thus we do not have a synchronization by default. (While the modalities are somehow synchronized at some point, the callback is not going through. Thus the observation_matrix is never updated for the contralateral side.)

I noticed the same issue occurs for the transition matrix.
Doesn't your solution create an infinite loop? Because when changing the ipsilateral modality, it triggers the contralateral callbacks, which, in turn, should trigger the ipsilateral callbacks. That's why I defined these update_without_trigger() methods.
Currently, I am thinking of an implementation similar to what I did with the transition_tensor for the Edge class that does not require us to delete these matrices. But it's a bit harder in these cases...

Interestingly, it does not create an infinite loop Seemingly, the change of the contralateral modalities does not trigger a second callback since only the ipsilateral should be triggering the callback. Which probably means, that when setting contralateral modalities, no callbacks are initialized and the ipsilateral side is not synced. I.e. we only have a one way syncing

Alright, then I'll work on that now:
A two-way sync for the modalities via the callbacks and globally cached functions for the transition and observation matrices.

Ok, this was a bit trickier than I thought 😅
Because the diagnose_matrices depend on three things:

the modalities
the data
and the T-category that was requested

If either changes, it should be recomputed. So, I had to compute a hash for all three quantities (ideally somewhat fast) and combine them into one hash to make sure you always get the correct diagnose matrix.
The changes are now in the dev branch. All tests (and some new ones) pass, but I still have the feeling that there could be something wrong with this new implementation.
So, it'd be great if you could double check that you can still reproduce your results. Thanks!

I'll go through it on Monday to double check that everything is fine. Thanks for the changes!

I tested the trinary and the binary model with bilateral sampling and risk calculations. The results were good --> seems like everything works as expected. The computation time is only minimally increased (might also just be some CPU fluctuations^^). Looks like a perfect fix!


add all functions about complex Fourier transform and others

Context: Yuan's leftover codes that may be useful but not in any branch now.
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

Falling too far behind. Will move to a new PR.


Use struct for #audit_formula args

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

As discussed in https://github.com/Homebrew/brew/pull/17643#discussion_r1667381705, adding type signatures to #audit_formula methods in formula cops would lead to verbose, repetitive signatures across the existing ~63 instances:
sig {
  override.params(
    _node:              RuboCop::AST::ClassNode,
    _class_node:        RuboCop::AST::ConstNode,
    _parent_class_node: RuboCop::AST::ConstNode,
    body_node:          RuboCop::AST::Node,
  ).void
}

This reworks #audit_formula to use a T::Struct for its arguments, which allows us to use a one-line signature for these methods. There may be room for improvement here but this seems to work as expected, at least.

Not a fan of the name "args" as everything is an argument but otherwise don't have a strong opinion either way.

Not a fan of the name "args" as everything is an argument but otherwise don't have a strong opinion either way.

I <3 this change, but agree that args could be improved. Maybe formula_nodes, if that seems an accurate description.

I updated this to use FormulaNodes as the struct name and formula_nodes as the #audit_formula parameter name. Per Bo's comment, we can always rename it to something else if we ever add a non-node argument, so the "nodes" language seems fine for now (it's certainly more descriptive than args).


Adding bit assignments in DBG_MANUF_SERVICE_REG description

I realized this is already being handled in https://github.com/chipsalliance/caliptra-rtl/pull/281


V greach horton deploy mods

From Manifest, create devices and modules, get images, create and run containers, then clean up.

def create_device_module(self, device_id, module_id, is_edge=False):

please remove this parameter.  This concept doesn't exist the same way for modules.

Refers to: horton_helpers/service_helper.py:85 in 1ec7494. [](commit_id = 1ec74940fcf95d644564270d8f6fe46cba6c8796, deletion_comment = False)

Overall: awesome!  I was about to comment on the fact that you're missing docstrings, but your code is self-documenting enough that i don't think it needs it.  Once you resolve the last few comments, pull it in.


react-scripts: not found

Hi,
I really like https://bkrem.github.io/react-d3-tree/ and I want to build something similar.
While trying to get the demo running I get an error. What am I doing wrong?
git clone https://github.com/bkrem/react-d3-tree.git
cd react-d3-tree/
npm i
npm link

then
$ npm run build && npm run build:docs && cd demo && npm run build && npm run deploy

><EMAIL_ADDRESS>build
> rimraf lib && npm run build:cjs && npm run build:esm

><EMAIL_ADDRESS>build:cjs
> tsc -p tsconfig.json

><EMAIL_ADDRESS>build:esm
> tsc -p tsconfig.esm.json

><EMAIL_ADDRESS>build:docs
> rimraf ./docs && typedoc

Using TypeScript 4.9.4 from /tmp/react-d3-tree/node_modules/typescript/lib
Warning: You are running in an unsupported TypeScript version! TypeDoc supports 3.9.x || 4.0.x
DeprecationWarning: 'isIdentifierOrPrivateIdentifier' has been deprecated since v4.2.0. Use `isMemberName` instead.
Rendering [========================================] 100%

Documentation generated at /tmp/react-d3-tree/demo/public/docs

><EMAIL_ADDRESS>build
> react-scripts build

sh: 1: react-scripts: not found

environment
$ node --version
v19.7.0
$ npm --version
9.6.2

Hey, sorry I didn't get back to you here in good time but the issue seems to have been that you also needed to run npm i in the demo directory since it has its own deps/node_modules


kde5.kdepim

Things done:

[x] Tested via nix.useChroot.
[x] Built on platform(s): NixOS .
[ ] Tested compilation of all pkgs that depend on this change.
[ ] Tested execution of binary products.
[ ] Fits CONTRIBUTING.md.

All dependencies of kdepim build. It seems as if akonadi is missing AkonadiConfig cmake file causing build of kdepim to fail.

This is not going to work. While all the components are released, they require different versions of frameworks. 5.16, 5.19, ...

Closing this for now.

Note that it requires kdewebkit which was removed in dd34c61097898c5de2ee286fc5df24ee0d12c0d5, likely because QtWebKit has been removed from Qt 5.6.


Import PureData Files

05a0258, c1e58bb, 92691d6, 8372c13, 65341e6...
In these commits, PureDat import was basically implemented and works properly at ed0e1a7.


Missing install steps in README

I had this project set up on my machine for demoing automated testing, then I tried replicating it and found that I couldn't! One problem: @ebay/makeup seems to have been renamed on npm to makeup-ebay. I tried updating to that devDependency but Lasso still fails, I think because that project changed in structure.
Can you provide install steps for this project? I was going to submit a PR but I didn't get it working again easily. Thanks!

@marcysutton Yeah, I'm sorry about that, there was quite a bit of churn with the makeup stuff and it's project structure did change as you say. I'm not sure when you did your clone, but if you pull in the latest browser.json and run npm start I think it should just work.
I also added some install steps to the readme if it's easier for you to just re-clone and start again.

That works, thanks so much for the prompt reply! I think we can close this issue now...

Also, FYI :) https://a11ywins.tumblr.com/post/144578706413/ebays-mind-pattern-examples

Wow! I'm incredibly honoured to appear on a11ywins! Thanks so much Marcy!


Add a direction option so on vertical orientation values can be bottom-to-top.

I need to add the slider but on vertical orientation with values from bottom-to-top. Currently it supports only top-to-bottom. Is it correct?
Update: I am able to workaround that by setting negative numbers for min/max/model-min but it would be nice to have an option to do that automatically.
<div range-slider
	min="-100"
	max="-30"
	model-min="model.zoomValue"
	orientation="vertical"
	on-handle-up="zoomChart()"
	pin-handle="max"
	show-values="false">
</div>

+1

+1

How is this still even an issue? It is absolutely unfathomable why this was even released to the public like this...

To those experiencing this issue -- Follow @cirilomegg 's instructions, and then add this css to your stylesheets to hide the negative sign.
.ngrs-value.ngrs-value-min,
.ngrs-value.ngrs-value-max {
    width: 33px;
    margin-left:auto;
    margin-right:auto;
    text-align:right;
    overflow:hidden;
    white-space: nowrap;
}

.ngrs-value.ngrs-value-min > div,
.ngrs-value.ngrs-value-max > div {
  float:right;
  font-size: 1.2em;
  color: #333;
  font-weight: 600;
}

@dsifford it is open source. Why not contribute a PR?

+1


Bot doesn't start when using start.sh

Expected Behavior
Bot should start by running the start.sh
Actual Behavior
Bot gave the error "There's no ./configs/config.json file", so I edited the path in the start.sh to the correct config.json. Now the terminal shows this error:
"/Users/k/PokemonGo-Bot/run.sh ; exit;
python: can't open file 'pokecli.py': [Errno 2] No such file or directory"
Note: It's possible to start the bot using
"$ cd PokemonGo-Bot
$ virtualenv .
$ source bin/activate
$ pip install -r requirements.txt
$ python pokecli.py"
Steps to Reproduce
Installed everything as recommended and started the start bash
Other Information
OS: Mac OSX
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)
Python Version: Python 2.7.12

python: can't open file 'pokecli.py': [Errno 2] No such file or directory"
ensure you are on the same level in the directory as pokeclip.py  Then run the following command
python pokecli.py

When I'm in the directory of pokecli.py (just inside the PokemonGo-Bot folder) there is another error:
Traceback (most recent call last):
File "pokecli.py", line 38, in
from pgoapi.exceptions import NotLoggedInException, ServerSideRequestThrottlingException, ServerBusyOrOfflineException
ImportError: No module named pgoapi.exceptions
I can start the bot just by "cd PokemonGo-Bot","source bin/activate" and then "python pokecli.py". Is there a way to implement those steps into the start.sh?

As there have been many updates since this issue was posted, I'm assuming this issue has been resolved. If I'm wrong, please reopen.


Only publish to npm on NodeV12

Creating a PR to test out the builds (does it run on PRs?) since my fork does not have Travis setup properly (plus the API keys won't work anyways). I have seen this work on documentations so it's worth giving a try.

Hmmm turns out PR builds do not test deploy anyways... Do you have a better idea on how to test this?


Chapter 5: configuration property value is wrong in this code repository

in chapter 5, the code in this repository has errors, e.g. the value of spring.cloud.config.uri  in  bootstrap.yml of licensing-service  should be http://localhost:8071  not  the http://configserver:8071.  The same error also could find in docker-compose.yml

I am wrong, sorry~


CCITT 4 decoding is too slow

See the letter page in the first 10 pages of the CHAM for an example.

Its much better now.


No UDID or UUID stored in SIGH_UDID environment variable

When running sigh from the command line or within Ruby script and after successfully downloading a provisioning profile and installing it, I try to print it out from command line
echo $SIGH_UDID
This prints nothing

puts ENV["SIGH_UDID"]
This prints nothing as well

Please verify that this environment variable is assigned with the UUID of the provisioning profile and if not could you please fix this bug.
Thanks kindly.!

Environment variables can't be set and transferred from a sub-process. If you use fastlane this works, but not when running sigh directly.

Hi Felix,
Thanks for this information. I am using the sigh tool standalone at the
moment actually, but this SIGH_UDID is a missing link for me between
fighting provisioning and automating the entire process start to finish. Do
you have any plans to port this functionality to the current standalone? I
am currently unable to switch our whole build process to Fastlane at the
moment, because there are some other factors and systems involved.
Best Regards,
Jacob
On Sat, Jul 25, 2015 at 2:22 PM, Felix Krause<EMAIL_ADDRESS>wrote:

Environment variables can't be set and transferred from a sub-process. If
you use fastlane this works, but not when running sigh directly.
—
Reply to this email directly or view it on GitHub
https://github.com/KrauseFx/sigh/issues/146#issuecomment-124896695.

Sorry, but that's not a problem of sigh, it's just the way environment variables work. You can't just pass environment variables from a child process to the parent process (please correct me if I'm wrong).
What you could do is to access the downloaded provisioning profile and just fetch the UDID from there. You can take a look at the source code here.

Ok, that looks simple enough. Thanks for your time and great job on all these tools.!

Thanks :+1:


Fix bugs in broadbean element and sequence

This PR fixes 2 bugs that are found when using Ripasso module:

Plotter function fails to plot elements processed by Ripasso RC filter

OutputForSEQXFile function fails for sequences processed by Ripasso RC filter

Unit tests are also added.

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

Comparison is base (d0b31ab) 79.09% compared to head (7a090b9) 82.13%.
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #751      +/-   ##
==========================================
+ Coverage   79.09%   82.13%   +3.04%
==========================================
  Files          16       17       +1
  Lines        2406     2441      +35
==========================================
+ Hits         1903     2005     +102
+ Misses        503      436      -67

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


fix: greeting api should be protected by roles.

Pull Request Test Coverage Report for Build 60

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.0%

Totals

Change from base Build 59:
0.0%

Covered Lines:
20

Relevant Lines:
21

💛  - Coveralls


fix: move to slack-sdk files_upload_v2

Randomly caught https://github.com/apache/superset/issues/28353 and decided to propose a fix. This seemed straightforward when I looked at the docs here https://github.com/slackapi/python-slack-sdk/releases/tag/v3.19.0
Confirmed we're on the latest client and that stuff is pretty well covered by unit tests

For now let me remove that line


Please update this package on npmjs.com..

@sokra
Sorry to be bothering you, but the fix #5 is not updated on npmjs.com.
Could you please update it? Thank you!

Closing since 0.1.5 was released (a while ago).


Define defaults for Vue

We might need some default set for each framework, in order to skip the post framework selections step with a flag.
This issue is about picking the defaults for vue

hello @haripajjuri ! Of course you can! Thank you  :D


Fix steering backlog link.

🤦♂ Used the old link.

Page preview:
https://deploy-preview-16683--kubernetes-io-master-staging.netlify.com/blog/

/lgtm

/approve

Much appreciated @kbhawkey. Thanks!


Add multitouch support

Hi, these changes should enable multi-touch support, so that elements can be individually dragged at the same time, when using a multi-touch device.
I've tested this successfully on Chrome for Android. On Firefox for Android, scroll and zoom aren't being prevented (a separate issue?) I unfortunately do not have an iOS device to test with.
This adds a few conditional statements in the event handlers, but it didn't seem to impact performance when testing on my laptop. The original tests all pass, and everything in the demo is functioning as I would expect it to (The "Active Drags" counter is fun to watch, now)
I would like to add unit tests for this, but I'm not sure how to simulate this situation - everything depends on how the state of each component changes.
There might be a more performance-savvy way to do this by having one set of touch event listeners per app, but that would take a significant refactoring and might not be in line with the philosophy of React.
Please let me know what you think!

Hey, this is pretty nice. Thanks for the contribution.
I agree that this is the right way to do it. Any other approach essentially relies on globals.

Thanks! If we were using ES6 classes instead of React.createClass, it might make sense to have the event listeners be static class variables, enforcing one instance of each for the whole class. The single listener could be added with the first drag from any component, and removed with a drop as long as no other component is dragging... but determining that would also probably require global state.
I guess in the worst case, one ten-fingered person will be dragging ten items at once. There will be ten listeners that each fire the handleDrag ten times. That's definitely not ideal, but... at least most people don't have more than ten fingers!


Convert type comments to JSDoc format

Many code editors support JSDoc comments, displaying their content while showing suggestions.
For example in src/api/chia/consensus/block_record.ts, if we replace:
export type BlockRecord = {
  header_hash: bytes32; // bytes32
  prev_hash: bytes32; // bytes32  # Header hash of the previous block
  ...

with
export type BlockRecord = {
  header_hash: bytes32; // bytes32
  /**
   * Header hash of the previous block
   */
  prev_hash: bytes32; // bytes32
  ...

or
export type BlockRecord = {
  header_hash: bytes32; // bytes32
  /** Header hash of the previous block */
  prev_hash: bytes32; // bytes32
  ...

Then the emitted declaration file will retain the JSDoc comment and we will get in-editor suggestions.

Since I don't know whether you extract the types from the chia code base manually or with a script, I am posting this as an issue first, but I will gladly contribute a PR in either case.

I manually check the chia code one by one.
JSDoc comments are good if what they say are correct and easy to understand.
What do you think should be a good source of those JSDocs?

To begin with, I was thinking to use the existing single-line comments appended after the data type property names in the chia-blockchain source code, which you helpfully copied over along with names of the original python types (which would also be helpful).
Then if we want to add docs to the API endpoint functions and their parameters, we can parse the descriptions in chia-docs:
https://github.com/Chia-Network/chia-docs/blob/main/docs/12rpcs/full_node_api.md
https://github.com/Chia-Network/chia-docs/blob/main/docs/12rpcs/did_rpcs.mdx
https://github.com/Chia-Network/chia-docs/blob/main/docs/12rpcs/nft_rpcs.mdx


[BUG] A value of 0 is not accepted as a valid number field value, minus sign cannot be entered as first character

Looks like numbers are incorrectly parsed in the input field. It has 2 manifestations:

When trying to enter 0, the field empties itself.
When trying to enter - to be followed with some numbers, the field empties itself (although you can overcome it by typing minus after typing the number)

Manifestation 1 is more severe, since it has no workaround.
Affected field:

Reproduction repository (small fork from test app):
https://github.com/jedzej/trpc-panel-bug-repro/commit/62857ae7f95bec78e540327a41b13a03cf6a3a79

Fixed in v1.2.6. Thanks for the issue report!


typings: Missing dependency react-native

The index.d.ts file references the react-native module but this module is not listed as a dependency.
Maybe we could use
import { CSSProperties } from "react";
// ...
style?: CSSProperties,

instead of
import { ViewStyle } from 'react-native';
// ...
style?: ViewStyle,

since react is already listed as a dependency?

Alright, changes published in v0.7.9. Hopefully that works.


Fixed minor bugs on the submission/upload page

Summary Change Description
Note that I also updated the submission google form (already live).

Changed the submission doc link (in the example env file) - Note, this will need to be changed in the .env file after deployment.
Changed the settings to get the submission details from the env file
Fixed the upload page background (light gray to white)
Fixed the expand/collapse icon updates

Affected Issues/Pull Requests

Resolves #941
Resolves #943
Resolves #944
Partially addressed #939 (links to a wiki page on the repo stating that the docs are under renovation)

Review Notes

See comments in-line.
Checklist

This pull request will be merged once the following requirements are met.  The
author and/or reviewers should uncheck any unmet requirements:

Review requirements

Minimum approvals: 1
No changes requested
All blocking issues resolved by reviewers
Specific reviewers: @add_username_here

Review period: 2 days

Associated issue/pull request requirements:

[x] All requirements in affected issues marked "resolved" are satisfied
[x] All required pull requests are merged (or none)

Basic requirements

[x] All linters pass
[x] All tests pass
[x] All conflicts resolved

Overhead requirements

[x] New/changed method tests implemented/updated (or no change)
[x] Updated Unreleased section of changelog.md (or no change)
[x] Migrations created & committed (or no change)

@lparsons, would you mind taking another look at just the new commit?  I elected for a javascript alert for when the env vars are not set.  Not all browsers support that from href=javascript:..., so I used a little template code to convert it to onclick.  I tested it and it works, but if you have a better suggestion, let me know.


information_schemamodify information_schema.tables rows column use cache rows.

Use the cached information and estimated information of the table in the rows column under information_schema.tables. Avoid querying information_schema.tables that will cause rpc timeout when there are a large number of tables in the catalog.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Objects split into base colors

Overview
When updated to newest version of Mixed Reality Toolkit, there appears to be an issue with displaying objects on HoloLens. Objects base colors separate when moving around.
Made a scene with a single cube using Mixed Reality Toolkit Toolbar button to set up the scene and project settings. The same issue appears on example scenes from Mixed Reality Toolkit.
Expected Behavior
Objects are stable and colors of an object do not separate
Actual Behavior
All objects within the scene split into base colors the same as described in "color separation" section in the documentation https://developer.microsoft.com/en-us/windows/mixed-reality/hologram_stability
Note: Already installed all essential software mentioned in "2017.2 Upgrade Guide for Window Mixed Reality"
Steps to reproduce
https://github.com/marjackarpinski/HL_Test
Unity Editor Version
2017.2.1f1
Mixed Reality Toolkit Release Version
2<IP_ADDRESS>

Is not fixed in Unity 2017.3.1f1, right?


Hardcode docs for compatibility

#![doc = include_str!("../README.md")] is still relatively new and does not compile on pre 1.54.0 versions

Should resolve https://github.com/vinted/elasticsearch-dsl-rs/issues/42


Array in "type" definition is considered error

Hello,
I have a schema that needs to validate a property that could be an integer or a string like this:
{
    "type": "object",
    "properties": {
        "x": {"type": ["integer", "string"]}
    }
}

As far as I know this is valid in json schema but the library throws an exception "Argument 2 passed to League\OpenAPIValidation\Schema\Keywords\Type::validate() must be of the type string, array given".
Is this valid in openapi spec?
Thanks,
Frank

OpenAPI 3.0 defines schema type as string, only in 3.1 type can be array. OpenAPI 3.1 is not fully supported yet.


 text not visible on light theme

Screenshot below:

Please, share your Atom version, UI Theme version, Syntax theme version, OS, and how to reproduce the bug and I'll reopen the issue.


The error returned in Azurite is inconsistent with the one in  Azure when use unauthorized queueClient to perform operation

Error Description:
The error returned in Azurite is inconsistent with the one in  Azure when use unauthorized queueClient to perform operation.
In Azure, the error is shown as following:

In Azurite, the error is shown as following:

Note: Azurite doesn't define error ResourceNotFound.
To Reproduce
The demo code is shown as following:
import { QueueClient,StorageSharedKeyCredential, QueueServiceClient } from "@azure/storage-queue"
import * as assert from "assert"

async function main(){
  const queueServiceClient = new QueueServiceClient("<queue-endpoint>",new StorageSharedKeyCredential("<account-name>","<account-key>"));
  const queueClient = queueServiceClient.getQueueClient("<queue-name>");
  const unauthorizedQueueClient = new QueueClient(queueClient.url);
  await unauthorizedQueueClient.create();
}

main().catch((err)=>{
  assert.ok((err as any).response.parsedBody.Code === "ResourceNotFound");
});

Expected Behavior
The error returned in Azurite is consistent with the one in Azure when use unauthorized queueClient to perform operation.
@jongio for notification.

Hi @zzhxiaofeng Thanks for this. We should update Queue to use 404 instead of 403.
FYI. Even for Azure Storage cloud. The blob service error code may diff between 404 or 403 for different storage account types. For Queue/Table, only 404.

Hi @zzhxiaofeng Thanks for this. We should update Queue to use 404 instead of 403.
FYI. Even for Azure Storage cloud. The blob service error code may diff between 404 or 403 for different storage account types. For Queue/Table, only 404.

azsdk-azurite


Implement verge spec feature

Motivation

Develop "verge" spec feature on top of Deneb. https://github.com/ChainSafe/lodestar/issues/4681
Spec: https://github.com/ethereum/consensus-specs/pull/3230

@gballet current spec is based on capella. Are there plans to rebase it on deneb?
Description
This fork currently only updates the execution payload type. There's no additional logic required in consensus side.
Closes https://github.com/ChainSafe/lodestar/issues/4681

superseded by

https://github.com/ChainSafe/lodestar/pull/5805


many html tags are not working as it should while using windicss, vite, vue3

all the heading tags from <h1> to <h6> are diplaying as normal p tag or div tag
even <a> is not showing blue hyperlink

all the heading tags from <h1> to <h6> are displaying as normal p tag or div tag
even <a> is not showing blue hyperlink

That's basically how a CSS framework like Windi CSS or Tailwind would work though (or is it different in WIndi ?), it resets the styles so every browser displays them the same way. You can use CSS classes to make them look however you want without any conflict.


Added colors for SHW and LAW

Hi, the lines you changed are longer than the 79 character limit given in the coding style guidelines. This is thre reason for the partial CI failure. Can you please fix this?

Sure! Fixing now.


Icon "placeholder" sometimes showing when pointing to an app

The exact same script on the same VM was used to generate both of these screenshots, however one time the icon drew and one time it didn't.
It seems to happen consistently the very first time I run the script on a device/VM, and only happens when using a /foo/bar/Something.app path (not a file path to an .icns or .png). Subsequent runs will load the .app icon properly.
This is the output (via set -x) from the failed run
2023-02-21 20:47:16.927 Dialog[2378:29496] XType: failed to connect - Error Domain=NSCocoaErrorDomain Code=4099 "The connection to service named com.apple.fonts was invalidated: failed at lookup with error 3 - No such process." UserInfo={NSDebugDescription=The connection to service named com.apple.fonts was invalidated: failed at lookup with error 3 - No such process.}
2023-02-21 20:47:16.927 Dialog[2378:29496] Font server protocol version mismatch (expected:5 got:0), falling back to local fonts
2023-02-21 20:47:16.927 Dialog[2378:29496] XType: unable to make a connection to the font daemon!
2023-02-21 20:47:16.927 Dialog[2378:29496] XType: XTFontStaticRegistry is enabled as fontd is not available.

This is the output (via set -x) from the successful run
2023-02-21 21:00:25.215 Dialog[1494:10410] XType: failed to connect - Error Domain=NSCocoaErrorDomain Code=4099 "The connection to service named com.apple.fonts was invalidated: failed at lookup with error 3 - No such process." UserInfo={NSDebugDescription=The connection to service named com.apple.fonts was invalidated: failed at lookup with error 3 - No such process.}
2023-02-21 21:00:25.215 Dialog[1494:10410] Font server protocol version mismatch (expected:5 got:0), falling back to local fonts
2023-02-21 21:00:25.215 Dialog[1494:10410] XType: unable to make a connection to the font daemon!
2023-02-21 21:00:25.215 Dialog[1494:10410] XType: XTFontStaticRegistry is enabled as fontd is not available.

I think the font errors are a red herring, i see those often when running SwiftDialog from a management tool or from a LaunchDaemon. In this case, the script is running from a LaunchDaemon.
This is the dialog command in question. You can see its calling a local macOS system app for the icon:
+/usr/local/Baseline/Baseline.sh:903> /usr/local/bin/dialog --title 'Your computer setup is underway' --message 'Feel free to step away, this could take 30 minutes or more. \n\nYour computer will restart when it'\''s ready for use.' --icon '/System/Library/CoreServices/Erase Assistant.app' --blurscreen --width 900 --height 550 --button1disabled --commandfile /var/tmp/baselineDialog.4cIrR8 --quitkey ']' --listitem 'Google Chrome' --listitem Firefox --listitem 'Omnidisk Sweeper' --listitem Zoom --listitem Dockutil --listitem 'Microsoft Office' --listitem Nextiva --listitem 'SSC Customization' --listitem 'User Settings'

I've had this issue happen to me too. In most of my cases, the icon initially shows up as the generic talk bubble icon that swiftDialog uses when no icon is specified or found, and then switches to the actual icon. The time it takes to make that switch is usually a couple of seconds. I've had a few cases where the icon doesn't appear at all, like above.
It seems the application has some trouble initially determining the icon from an app bundle and falls back to its defaults.

I am also seeing this issue particularly with App bundles when running an initial onboarding dialog. Also I have one of the icons set as a SF Icon from SF Symbols and it does not display in the list but displays when in the bigger side of a progress dialog.

I'm using Dialog version <IP_ADDRESS>48, and I keep running into this problem. It seems very consistent now that app icons from bundles don't appear correctly in the resulting dialog. I'm uncertain if the problem is when the script is called as root from a Jamf policy. This is often when I see it. If I just call Dialog directly from the shell and use an app bundle path for the icon, it appears just fine. It also seems to happen when using lists very consistently, but not necessarily with a standard dialog.
Is this being looked into? It's not causing the dialogs to fail, but it's messing up the dialogs from an aesthetic point of view, so it would be great to see this fixed at some point.

@mm2270 if you could provide an example where it happens consistently for you that would help. in my own testing here I can't replicate what you are seeing.

I've been seeing this as well with "icon": "/System/Library/CoreServices/Installer.app"
In our case it's running as root as part of our bootstrap process. Sometimes its there, sometimes its not 🤷

Other conversation about it on Slack as well:
https://macadmins.slack.com/archives/C01U5MXNGG6/p1679681658584889

as per this comment and also this post this is an identified bug in Ventura and looks like it will be fixed in macOS 14.

interested if this is fixed in 2.3

should be fixed in 2.3.1 - can re-open if it's still a problem


FacebookLoginPackage.java:20: error: method does not override or implement a method from a supertype @Override

node_modules/react-native-facebook-login/android/src/main/java/com/magus/fblogin/FacebookLoginPackage.java:20: error: method does not override or implement a method from a supertype     @Override     ^ 1 error :react-native-facebook-login:compileReleaseJavaWithJavac FAILED
this happened after updating to android build tools 25
Is there any quick fix for it?

Did you solve this ? Hitting the same error.

I was wrong in my assumption of the cause of the issue. Eventually some of the solutions in the other issues raised worked, but I no longer remember which one :/


feat: add optional CBOR transaction representation to SUBSCRIBE_ADDRESS

Ohh, could you also please add SUBSCRIBE_ADDRESS with enabled cbor as an example into the readme under the usage section (just add one more wscat message until we write proper documentation)


Multiple Y Axes

Description
Can we support multiple lines that render to different y-axes. Complexities here exist in setting y-min/max. Not sure D1.0 supports this?
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

Could have something like this?


Router onUpdate called twice

Issue - When we trigger any route change event, the onUpdate method getting invoked twice.
Sample code structure-
const appHistory = useRouterHistory(createHashHistory)({ queryKey: false });
var counter = 0;
ReactDOM.render(
    <Router history={appHistory} onUpdate={() => {
        //this will get invoked twice
        console.log("Router got updated and counter is " + counter);
    }}>
        ...
    </Router>,
    appDOM
);

When I was going through the flow, in createTransitionManager.js I had seen one comment in listen  method-
    // TODO: Only use a single history listener. Otherwise we'll
    // end up with multiple concurrent calls to match.

Does it relate to the same issue? Had anyone experienced this issue?

Establish your appHistory object as a singleton module, like we do with the provided histories. How are you triggering your route changes, via Links or push on the history instance?

This is probably because you're using a <a href="#foo"> instead of a <Link>. Use <Link> instead.

Hi @taion @timdorr , Thanks for the inputs.
I was using <Link> component only. But the issue was the to param. Earlier I was using it like to="bingo"(which was being converted to localhost/#bingo), after changing it to to="/bingo"(which is being converted to localhost/#/bingo) the issue was resolved. The two call to onUpdate was expected here because it was causing redirect.
Thanks again 😃

@taion @timdorr, I used  the way @codingdash mentioned and still have trouble with onUpdate being called twice.

I'm still having this issue and I'm using Link component. I'm using react-router 3.0.5.
I ended up using something like this:
<Router onUpdate={() => {         if (this.state.location.action == "PUSH") {            this.fireTracking()}         } } history={ hashHistory }>
I noticed the action is different the two times the onUpdate is being called: PUSH and then POP.

I'm still having this issue and I'm using Link component. I'm using react-router 3.0.5.
I ended up using something like this:
<Router onUpdate={         function(){           if (this.state && this.state.location.action == "PUSH") {             fireTracking();           }         }       } history={ hashHistory }>
I noticed the action is different the two times the onUpdate is being called: PUSH and then POP.


documentation for select! macro

https://github.com/rust-lang/futures-rs/blob/3601bb76708ea4bcc14c3491237abc33b685ba0b/futures-util/src/async_await/select_mod.rs#L32-L33
With this commit , is the above statement still accurate?  Is it true that select! macro only uses proc-macro-hack if rustc version older than 1.45?  Do we still need to worry about recursion_limit?

Good catch. It's definitely outdated; if you're using Rust 1.45+, you won't have the problem.


ping fails on wrong address

@rultor release, tag is 0.0.13

@rultor release, tag is 0.0.13

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.13

@yegor256 Done! FYI, the full log is here (took me 2min)


Update deploy-integration-services-ssis-projects-and-packages.md

The text should read "call stored procedures" vs "call stored"

@cloudgiant99 : Thanks for your contribution! The author, @janinezhang, has been notified to review your proposed change.

@cloudgiant99 Hello, Thank you very much for fixing this confusing documentation wording  :-) .
I am merging this change now. The fix should be on live Docs in a day or two.


How many rows are there?

It might have a place to display the number of rows (after filter etc.).

+1

+1

+1


Juju nrpe charm sets load alert threshold values incorrectly

There are 2 defects with Juju nrpe charm

Config's description of load was not updated to date and is misleading when set to auto
Definition per [1]:
if 'auto' is set, then NUM_CPUS0.7 is used for the warning threshold and NUM_CPUS for critical
Actuality per [2]&testing:
if 'auto' is set, then load average (1 minute,5 minutes,15 minutes respectively)
(NUM_CPUS4, NUM_CPUS2, NUM_CPUS1) is used for the warning threshold
(NUM_CPUS8, NUM_CPUS4, NUM_CPUS*2) is used for the critical threshold

If auto is not favourable in the cloud, hard-coded setting is rigid and inflexible
As the customer complained
"The other problem is that if I want to override these numbers in the juju config I can only specify it in the format "-w 8,8,8 -c 15,15,15" and this does not take into account numcpu. Since some units have 88 cores (e.g. nova-compute-kvm) and other have 2 (e.g. kibana) it makes it impossible to set one load number that is appropriate for all systems."

How to reproduce

Deploy nrpe and relevant relationships with Juju
$ juju deploy ubuntu
$ juju deploy nrpe
$ juju deploy nagios
$ juju add-relation ubuntu nrpe
$ juju add-relation nrpe:monitors nagios:monitors
$ juju config nrpe load=auto

reboot ubuntu node by allocating it with 1/2/3 CPU cores, and check the check_load configuration

First boot with 1 CPU core
root@6nova:~# more /etc/nagios/nrpe.d/check_load.cfg ;cat /proc/cpuinfo | grep processor
System Load (sub)
command[check_load]=/usr/lib/nagios/plugins/check_load -w 4,2,1 -c 8,4,2
processor	: 0
Second boot with 2 CPU core2
root@6nova:~# more /etc/nagios/nrpe.d/check_load.cfg ;cat /proc/cpuinfo | grep processor
System Load (sub)
command[check_load]=/usr/lib/nagios/plugins/check_load -w 8,4,2 -c 16,8,4
processor	: 0
processor	: 1
root@6nova:~#
Third boot with 3 CPU cores
root@6nova:~# more /etc/nagios/nrpe.d/check_load.cfg ;cat /proc/cpuinfo | grep processor
System Load (sub)
command[check_load]=/usr/lib/nagios/plugins/check_load -w 12,6,3 -c 24,12,6
processor	: 0
processor	: 1
processor	: 2
root@6nova:~#

hard-coded setting, below setting will be refreshed to all nodes without consideration of number of CPU cores
$ juju config nrpe load="-w 1.0,0.80,0.70 -c 5,2,1"

[1] https://jujucharms.com/nrpe/49
[2] https://git.launchpad.net/nrpe-charm/commit/?id=009a1ae855fe9e4d648ac800b5e7d78a4efb00a6

Imported from Launchpad using lp2gh.

date created: 2018-06-20T03:39:02Z

owner: hou-m

assignee: None

the launchpad url

(by aurelien-lourot)

Documentation of 'auto' has been fixed meanwhile.
Complaint about rigidity of the 'load' config is still valid.

(by brettmilford)
Check load has the parameter:
-r, --percpu
Divide the load averages by the number of CPUs (when possible)
Thus in the case of 2) one may still set thresholds a a proportion of the core count i.e.
'-r -w 0.9,0.8,0.7 -c 1.5,1.2,1'


flutter command doesn't follow Unix conventions for stdout and stderr

Unix commands should sent primary output to stdout and any progress or error messages should go to stderr. That way if we pipe the output to another command, we'll still see progress or errors on the terminal and the downstream command doesn't need to handle them.
We don't currently follow this convention for the flutter command. See update_dart_sdk.sh for example, which uses 'echo' to send errors to stdout.

This isn't planned. Rather, for the specific cases where we need machine readable output, the tool accepts the --machine flag and generates JSON output.


*: Remove type ContainerName

The type for the default name of the container is not needed, and the PR is to eliminate it.
/assign @jlewi  @ScorpioCPH
Signed-off-by: Ce Gao<EMAIL_ADDRESS>

This change is

/assign @jlewi @ScorpioCPH

Coverage decreased (-0.1%) to 27.475% when pulling 27946c6efe3c05992b8fb7ace84f933e0f768635 on gaocegege:name into 43677ccaa3d0da5b05e5b66ae730e6e6eac0fd25 on kubeflow:master.

/lgtm

@gaocegege I think you need to resolve the conflicts and then it will automerge.

/retest

@jlewi I think the bot could not auto-merge the code because of #433

/lgtm
/approve

@ScorpioCPH Could you approve the PR in GitHub review page and merge it manually? I think bot does not work now.


Preliminary replay parsing

Coverage decreased (-0.04%) to 89.252% when pulling 8acea74d7523d18632a91bab948f10009218b575 on dev-rec into 4a859acef00ad848574a158a7dfb1faa4ac67eec on master.


line breaks implementation change

Hi there!
I run this company iva-docs.com. The product is very straight forward, give us a template, we'll connect it to your data sources (Zapier, Typeform, Webhooks) and we'll deliver it where you want (Google drive, Email...).
I've looked at the current implementation of line breaks, and for some reason, it doesn't render properly in the Docx.
The output of this string:
const str = ` Love
Will
Always
Win`

is:
<w:r>
<w:t>
love <w:br/>will <w:br/> Always <w:br/> Win
</w:t>
</w:r>

for some reason, it doesn't display properly.
So my suggestion (and what worked for me) is:
<w:r>
<w:t>
love
</w:t>
 <w:br/>
<w:t>
will
</w:t>
<w:br/>
<w:t>
Always
</w:t>
<w:br/>
<w:t>
Win
</w:t>
</w:r>

I've then added this new implementation as an option in the CreateReportOptions. But I'm not sure if that's how you see it implemented.
I've setup this implementation in Iva and currently testing it in production. No Issues so far.
I'm available if you need anything

Coverage decreased (-0.5%) to 91.858% when pulling b7fcbc78e2ff03ddccdd3a4f81e37089fbdcc0ec on iva-docs:line-break-implementation-change into cd9bb79e9508f68f6ebeaafad73e97452b3c7fd6 on guigrpa:master.

@guigrpa Is there any chance this will be updated and merged? I can confirm that monkey-patching into a current release solves the problems mentioned e.g. in #143. Since this problem apparently appears since 2019 I'd consider it worth merging upstream, so that not everyone has to maintain a custom patch.

Thanks for this PR. I'll merge it in my work branch and do some more testing. This is pretty fundamental stuff, so I want to make sure it ends up being correct.


Offline installation

I started using your plugin and quite like it!
The problem I am facing right now is about offline installation.
I don't understand how I am able to not transfer the RUC to the target directly, but to transfer the right files to the BRsim folder. (Like I would do in Offline installation/Alt+F8 in brAS). Do you know how to achieve this?

Thanks for your request.
Currently there is no such feature implemented. I did the online transfer because of a former request, but you're the first person asking for the offline transfer. I myself usually use Automation Studio for the transfer and installation.
Currently you could build with the option 'generate RUC package' and then create your own Runtime Utility Center script for the offline install. That's the same approach I use for the online transfer, just that I generate and execute the script automatically.
I will think about adding this feature. Just doing the implementation will be quite easy, as it will work almost same as the online transfer.
For now I will add it to the backlog. If someone else is also interested in this, please upvote the request or leave a short comment.

Allright, thanks for your answer! I am not sure what files are generated with the offline installation, but I will figure out and create a script. I can update this post if I have it working, but not sure how quick it will be due to priority.

Did you manage to create an offline installation package? I can not find the right console commands to run this action.


Binding error channel

bind Spring Integration errorChannel as a message producer when there is at least one outbound endpoint exists in the SCSt context.
By default the errorChannel is bound with the destination name ${spring.config.name}.error and this can be overridden via spring.cloud.stream.errorDestination

This destination name can be used to subscribe to the error channel output messages.

Add test

This resolves #329

Updated the PR with the approach to bind error channel via implicit Bindable for the error channel.

Updated the PR based on comments and rebased as well.

Updated the PR to address the latest review comments.

Rebased, squashed, merged w/https://github.com/spring-cloud/spring-cloud-stream/commit/cae56385e0356011e64396df2245b8eeb963d5fa and additional changes in https://github.com/spring-cloud/spring-cloud-stream/commit/5e2e4ecd6796f965ae6405970e5ed2bb289cbcb3


All values in memory_sequence_length must greater than zero

What does it mean?
InvalidArgumentError (see above for traceback): assertion failed: [All values in memory_sequence_length must greater than zero.] [Condition x > 0 did not hold element-wise:] [x (IteratorGetNext:1) = ] [8 8 6...]
	 [[Node: dynamic_seq2seq/decoder/decoder/while/BasicDecoderStep/decoder/attention/assert_positive/assert_less/Assert/Assert = Assert[T=[DT_STRING, DT_STRING, DT_STRING, DT_INT32], summarize=3, _device="/job:localhost/replica:0/task:0/device:CPU:0"](dynamic_seq2seq/decoder/decoder/while/BasicDecoderStep/decoder/attention/assert_positive/assert_less/All, dynamic_seq2seq/decoder/decoder/while/BasicDecoderStep/decoder/attention/assert_positive/assert_less/Assert/Assert/data_0, dynamic_seq2seq/decoder/decoder/while/BasicDecoderStep/decoder/attention/assert_positive/assert_less/Assert/Assert/data_1, dynamic_seq2seq/decoder/decoder/while/BasicDecoderStep/decoder/attention/assert_positive/assert_less/Assert/Assert/data_2, dynamic_seq2seq/decoder/decoder/while/BasicDecoderStep/decoder/attention/assert_positive/assert_less/Less/Enter)]]

@bota7070 It means you have input sequence with zero length, which is an empty line.

I got this in the tutorial for blank lines in /tmp/nmt_attention_model/output_infer

I got this problem too, dead.


(Porting to 'master') Remove notion of predicates as types, move predicates back to signatures

Porting #7744 (which fixes #6328) to master.

:+1:


Messages sent and received, there is quite a long time delay

The current online equipment 1200, probably to send and receive a total of 50 messages per second, now found that there are quite a long time delay, delay one minute or more, from v3.0.13 upgrade to Version 3.0.14 - rc2, or have the same situation, is caused by a backlog of message queue, please? What should I do?
thank you
`using MQTTnet;
using MQTTnet.Server;
using SStore.Log.Core;
using System;
using System.Collections.Generic;
using System.Net;
using Microsoft.Extensions.Caching.Memory;
using System.Threading.Tasks;
using MQTTnet.Protocol;
using System.Runtime.InteropServices;
namespace SStore.IoT.MQTT
{
public class MQTTService : ServiceBase
{
private IMqttServer _mqttServer = null;
private MQTTStartParms _serviceParms;
static MemoryCache _cache = null;
static MQTTService()
{
_cache = new MemoryCache(new MemoryCacheOptions());
}
    public override void Init(ServiceStartParmsBase parms)
    {
        _serviceParms = parms as MQTTStartParms;
        Task.Factory.StartNew(async () =>
        {
            _mqttServer = new MqttFactory().CreateMqttServer() as MqttServer;
            //客户端消息事件
            _mqttServer.UseApplicationMessageReceivedHandler(MqttServe_ApplicationMessageReceived);
            //客户端连接事件
            _mqttServer.UseClientConnectedHandler(MqttNetServer_ClientConnected);
            ////客户端断开事件
            _mqttServer.UseClientDisconnectedHandler(MqttServer_ClientDisconnected);
            ////开启订阅事件
            _mqttServer.ClientSubscribedTopicHandler = new MqttServerClientSubscribedHandlerDelegate(MqttServer_ClientSubscribedTopic);
            //取消订阅事件
            //_mqttServer.ClientUnsubscribedTopicHandler = new MqttServerClientUnsubscribedTopicHandlerDelegate(MqttNetServer_UnSubscribedTopic);
            IMqttServerOptions serverOptions;
            if (_serviceParms.Port != null)
            {
                serverOptions = new MqttServerOptionsBuilder()
                .WithConnectionValidator(t =>
                {
                    if (t.Username != _serviceParms.Username || t.Password != _serviceParms.Password)
                    {
                        t.ReturnCode = MqttConnectReturnCode.ConnectionRefusedBadUsernameOrPassword;
                    }
                    t.ReturnCode = MqttConnectReturnCode.ConnectionAccepted;
                })
                .WithConnectionBacklog(20000)
                .WithPersistentSessions()
                .WithDefaultCommunicationTimeout(TimeSpan.FromSeconds(100.0))
                .WithMaxPendingMessagesPerClient(10000000)
                .WithDefaultEndpointPort(_serviceParms.Port ?? 500).Build();
            }
            else
            {
                serverOptions = new MqttServerOptions();
            }
            await _mqttServer.StartAsync(serverOptions);
            Logger.Event($"{this.GetType().Name}_Init", $"服务已启动{IPAddress.Any.ToString()}_{_serviceParms.Port ?? 500}", _serviceParms);
        });
    }

    public override void SendMsg(string client, byte[] buffer, string topic = null, MqttQualityOfServiceLevel level = MqttQualityOfServiceLevel.AtMostOnce)
    {
        Task.Factory.StartNew(async () =>
        {
            if (topic == null)
            {
                topic = $"{ _serviceParms.DefaultServiceTopic.TrimEnd('+').TrimEnd('/')}/{client}";
            }
            if (level == 0)
            {
                level = _serviceParms.ServiceLevel;
            }
            Logger.Event(this.GetType().Name, $"Mqtt异步发送开始_{client}", buffer);
            var message = new MqttApplicationMessageBuilder()
                .WithQualityOfServiceLevel(level)
                .WithTopic(topic)
                .WithPayload(buffer).Build();
            await _mqttServer.PublishAsync(message);
            Logger.Event(this.GetType().Name, $"Mqtt异步发送结束_{client}", buffer);
        });
    }

    private void MqttServer_ClientSubscribedTopic(MqttServerClientSubscribedTopicEventArgs e)
    {
        Logger.Debug($"{this.GetType().Name}_ClientSubscribedTopic", $"设备订阅_topic_{e.TopicFilter.Topic}_ClientId_{e.ClientId}", e);
        var clientId = e.ClientId;
        if (!ClientList.ContainsKey(clientId))
        {
            ClientList.TryAdd(clientId, new List<string>());//value 为topic
            //处理默认订阅主题
            if (_serviceParms.SubscribeTopicList != null && _serviceParms.SubscribeTopicList.Count > 0)
            {
                _serviceParms.SubscribeTopicList.ForEach(item =>
                {
                    _mqttServer.SubscribeAsync(clientId, new MqttTopicFilterBuilder().WithTopic(_serviceParms.DefaultClientTopic).Build());
                });
            }
            else
            {
                _mqttServer.SubscribeAsync(clientId, new MqttTopicFilterBuilder().WithTopic(_serviceParms.DefaultClientTopic).Build());
            }
        }
        if (ClientConnectedHandler != null)
        {
            ClientConnectedHandler.ClientConnected(clientId);
        }

    }

    private void MqttServer_ClientDisconnected(MqttServerClientDisconnectedEventArgs e)
    {
        try
        {
            object client;
            ClientList.TryRemove(e.ClientId, out client);
            if (ClientDisconnectedHandler != null)
            {
                ClientDisconnectedHandler.ClientDisconnected(e.ClientId);
            }
            Logger.Event($"{this.GetType().Name}_ClientDisconnected", $"设备断开连接_{e.ClientId}", e);
        }
        catch (Exception ex)
        {

        }
    }

    private void MqttNetServer_ClientConnected(MqttServerClientConnectedEventArgs e)
    {
        Logger.Debug($"{this.GetType().Name}_ClientConnected", $"设备已连接_{e.ClientId}", "");
        var clientId = e.ClientId;
        if (!ClientList.ContainsKey(clientId))
        {
            ClientList.TryAdd(clientId, new List<string>());//value 为topic

        }
        if (ClientConnectedHandler != null)
        {
            ClientConnectedHandler.ClientConnected(clientId);
        }
    }
    /// <summary>
    /// 数据接收
    /// </summary>

    private void MqttServe_ApplicationMessageReceived(MqttApplicationMessageReceivedEventArgs e)
    {
        try
        {
            if (string.IsNullOrEmpty(e.ClientId))
            {
                return;
            }
            Logger.Info("MqttServer_ApplicationMessageReceived", $"客户端:{e.ClientId} 主题:{e.ApplicationMessage.Topic}", e.ApplicationMessage.Payload.ToStringByHex());
            //缓存Client topic
            if (!string.IsNullOrEmpty(e.ClientId))
            {
                if (!ClientList.ContainsKey(e.ClientId))
                {
                    ClientList.TryAdd(e.ClientId, new List<string> { e.ApplicationMessage.Topic });
                }
                else
                {
                    var topicList = ClientList[e.ClientId] as List<string>;
                    if (topicList == null)
                    {
                        ClientList[e.ClientId] = new List<string> { e.ApplicationMessage.Topic };
                    }
                    else if (!topicList.Exists(i => i == e.ApplicationMessage.Topic))
                    {
                        topicList.Add(e.ApplicationMessage.Topic);
                        ClientList[e.ClientId] = topicList;
                    }
                }
            }

            if (RecvMsgHandler != null && !string.IsNullOrEmpty(e.ClientId))
            {
                RecvMsgHandler.RecvMsg(e.ClientId, e.ApplicationMessage.Payload);
            }
        }
        catch (Exception ex)
        {
            Logger.Exception(ex, "MqttServer_ApplicationMessageReceived", $"客户端:{e.ClientId} 主题:{e.ApplicationMessage.Topic}");
        }
    }

}

}
`

What client do you use? Also a MQTTnet client or a device?
Please try to move the code from the application message received handler to another thread.
With your approach all messages are processed in serial so that all have to wait until all pending messages are processed (RecvMsgHandler.RecvMsg).
Do you mean that v3.0.13 had no such issues?

Thank you for your reply
The same situation v3.0.13 also exist.
My client device iot module, written by C language
Isn't your UseApplicationMessageReceivedHandler multithreaded?
You mean to the following？
Task.Factory.StartNew( () =>
{
//This place is my specific business implementation
RecvMsgHandler.RecvMsg(e.ClientId, e.ApplicationMessage.Payload);
});

No it is not multithreaded. The reason is that some use cases require that the messages are processed in the right order. So the default implementation of MQTTnet is serial.
But when messages can be processed in parallel you must start your own thread in the handler.
You can do this via Task.Run(...) or (as you proposed) via Task.Factory.StartNew...

I will add a new handler to the library to that you can choose how the message interceptor deals with threads.

Thank you for your help
Now I added a message queue, where accept messages will be subject to the message in the queue, then program separate open multiple threads processing messages in the queue, pressure test of 500 per second transceiver should do not have what problem. At present basically stable.
At the same time also very much looking forward to you for the new version.
thanks again.

Some time ago to upgrade to the v3.0.15 version, just yesterday system peak, found again there is a relatively long time delay, the pressure test is on the personal computer simulation a frequent equipment with the proxy to send and receive, 500 do not have what problem, why to production, now 60 received per second, news began a backlog of feeling? Is related to upgrade the latest v3.0.15?
The current online equipment 3500 sets


refactor(storage): use the new Kotlin Destructuring Declarations

This PR should update our storage sample app to use the Kotlin Destructuring Declarations introduced in storage-ktx:19.2.0.
It also fixes a couple of warnings on MyUploadService.kt:

The Service import is redundant.
Log.d() can use String template instead of String concatenation.

@rosariopfernandes I missed this one too!


Call to undefined method V8\ObjectValue::value()

Trying to extend bundled example to work with object, I'm not able to get value. Consider following code:
$isolate = new \V8\Isolate();
$context = new \V8\Context($isolate);
$source = new \V8\StringValue($isolate, "var say = function() {return {foo: 'bar'};}; say();");

$script = new \V8\Script($context, $source);
$result = $script->run($context);

echo $result->value() . PHP_EOL;

I understand that return value is no longer StringValue but ObjectValue instead. How do I properly get it's value?
My use-case is rendering VueJS app on server to implement server side rendering. For this, I need to execute some wrapper javascript and app itself on the server. This js code results in ObjectValue just like my example above. If I try to do $result->toString($context)->value() I get [object Object] which is not usable. What is a correct way to do this?

Hi @qodeboy!
That's right, ::value() is a method which comes from PrimitiveValue abstract class, so only primitive values (non-objects) have this feature.
As converting object to string is not quite trivial thing and as it's a bit higher abstraction then this extension provides, you'll have to do it by yourself, probably, by iterating over properties and output key-value pairs. Alternatively, you can use JSON:stringify() method to get serialized object representation (it will work for any Value, actually).

I'm moving away from PHP world and all my PHP projects going to be abandoned too. Abandoning this project too as I have no intent to continue working on it unless there would be strong request from community and commercial interest. No more updates or documentation will be made. If someone is interested, feels free to contact me using email specified in my GitHub profile.


Optimize empty Sets which are initialized with array literals

For example:
func foo() -> Set<Int> {
  return []
}

Initializing a set with an empty array literal boils down to very minimal code - if the corresponding Set initializer is inlined.
I introduced a new underscored attribute: @inline(__forEmptyArray). It is placed on Set.init(arrayLiteral:) and lets this initializer inlined when it's called with an empty array literal.
Using this new attribute is a more reliable way to get the optimal result than tuning the inliner heuristics.
rdar://53509655

@swift-ci test

@swift-ci benchmark

Huh; is this much machinery really worth it for a one-off fix? I don't think this attribute would apply to any other case.
As a stdlib engineer, I admit my first instinct would be to try splitting this initializer in two, along the lines of:
@inlinable @inline(__always)
public init(arrayLiteral elements: Element...) {
     if elements.isEmpty {
       self.init()
      return
    }
   self.init(_arrayLiteral: elements)
}

@_alwaysEmitIntoClient // (bit of hand waving)
internal init(_arrayLiteral elements: Element...) {
  let native = _NativeSet<Element>(capacity: elements.count)
  // etc
}

Would something like this not work well enough?

@lorentey You are absolutely right! I was thinking too complicated. The empty-array check for array literals can always be constant folded. So there is no overhead in always-inlining this check.

@swift-ci benchmark

@swift-ci test

@swift-ci smoke test

@swift-ci smoke test


How to prevent scrollbar from appearing when using the tooltip?

Describe the bug
scrollbar appears when tooltip appears
Version of Package
v5.7.5
To Reproduce
https://codesandbox.io/s/eager-water-6tvblc?file=/src/index.js
(I use state with additional logic on my app, for not showing tooltip on mobile)
Expected behavior
scrollbar shouldn't appears
Screenshots
https://user-images.githubusercontent.com/52624465/220127295-9bdb45e4-1129-465a-8cca-9354f2d87422.mov

By default, the tooltip is placed on the DOM with absolute positioning.
You can use <Tooltip positionStrategy="fixed" /> to change that and prevent the scrollbar from appearing.

I am getting the same bug, and it is not fixed with <Tooltip positionStrategy="fixed" />. However, <Tooltip clickable /> seems to get rid of the scrollbar for some strange reason.

Which version are you using @wds4?

react-tooltip 5.8.2, this project


[WIP] Add profiling support

This is an initial attempt at adding profiling support to Daphne. As a first draft, I've added support for the PAPI library.
There's a profiling compiler pass that adds profiling 'tracepoints' at the beginning of each block, which call the Start/StopProfiling kernels.  This is more of a placeholder and we should decide exactly how / where this should be enabled. This can be exposed at the DSL / IR level via a pair of (start/stop) profiling operations.
The profiling kernels only support the HL (high-level) PAPI API at the moment, but we can switch to the low-level API if needed. PAPI also supports various plugins (e.g. for CUDA) which we can consider using / enabling.
Finally, we should also discuss whether PAPI is good enough or if we'd like to consider / evaluate other profiling tools / frameworks.
The documentation on how to use the new profiling infrastructure is also missing at the moment. You can try it by passing the   --enable-profiling CLI option to Daphne and settings the relevant PAPI HL environmental variables, e.g.:
PAPI_EVENTS="perf::CYCLES,perf::INSTRUCTIONS,perf::CACHE-REFERENCES,perf::CACHE MISSES,perf::BRANCHES,perf::BRANCH-MISSES" PAPI_REPORT=1 ./daphne --enable-profiling script.daph
This PR partly addresses #423

Is this ready to be merged now @psomas? You could confirm by removing the WIP in the PR title. Don't worry about the failing test - that's because the github-action docker image does not have libpapi.so (only libpapi.a). I'll fix that when merging.

LGTM - thx for your contribution @psomas. I added a few minor corrections and cleanups. Please add documentation about this (e.g., the text you have written in this PR description and possibly more) to the documentation section (doc sub directory) in another commit.


Consider counting attributes  on data and tracking as meta.attribute_count

Feedback from CS: this would help them a lot since they can measure how many fields teams usually use, but they don't know the total. Could also be other ways to get that, but it seems possible to count up on ingest.

something something sampling could complicate?

Sampling doesn't affect the set of attributes within a span, only the number of actual spans.
A meta. attribute is something that would decorate an individual span (i.e., in the customer's data). We could do it, but it requires sudo access to customer data to see it. Is that the desire here?
I don't actually understand the original statement "they can measure fields but don't know the total"? Total of what?
Please be more specific as to what's wanted.


[Microsoft.DigitalTwins] February 2023 release for azure-digitaltwins-core

Description
Updates azure-core and azure-identity packages.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

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


ZStoreArgs.weights should accept double.

ZUNIONSTORE and ZINTERSTORE allow floating point WEIGHTS. This should be reflected in ZStoreArgs.

Thanks, that's fixed.


Sync installation instructions on wiki and readme

or remove one of the two. I move the instructions to the wiki for faster editing while I update all the 447 rigs

I was thinking of this and already rearranged the wiki (in the behavior-blog) a bit.
It's also a bit frustrating that you need to go back and forth between blog and foraging when setting up PCs. Would you think it's ok to put all the installation instructions in the blog and then having a link to it in foraging?
As the blog repo is (will be) private, it might be an issue when disseminating dynamic-foraging-task widely in the future...though. @alexpiet any thought?

I'm fine with whatever is easiest to use


javax vs jakarta issue, using the application bom in Java EE 9/10 based app (quarkus 3)

Background:
A lot of frameworks Spring/Quarkus etc has been releasing major new versions with a breaking change to the Java Eco-system. javax->jakarta.  Quarkus 2.x is now on it's way out, and Quarkus 3.x is where most bugfixes go, ( and the recommended version ). For us, when currently using the signservie-application-bom, further down the dependency chain, the xml utils is used. https://github.com/idsec-solutions/signservice-commons/tree/master/commons/signservice-commons/src/main/java/se/idsec/signservice/xml, and with that comes javax. usage during runtime.
Our problems:
We have been testing the Quarkus 3.x releases but they are missing the javax. packages, during runtime (as intented by Quarkus 3x). Coexistence with javax is not an straightforward option either as Quarkus wont include all javax. libraries in the jar dependencies build results as they are changed to jakarta.
While, we could most likely hack around it a bit on our side , (bypass the filtering of deps on the quarkus side, add the missing jars to the build results and see if the classloader picks them up etc), it feels to hackish for us.
Solution proposal:
Is it possible to use the jakarta versions of the libs in the util artifacts, replacing javax?

See #162

Fixed in https://github.com/swedenconnect/signservice/pull/163


Prometheus Metrics接入问题

[x] 我已经检查过discussions
[x] 我已经搜索过issues
[x] 我已经仔细检查过FAQ

描述bug
官方文档里写了，暴露了Prometheus 格式的Metrics，实际怎么是个json？

复现
请求 /metrics，是个JSON，Prometheus无法接入
{"names":["application.ready.time","application.started.time","disk.free","disk.total","executor.active","executor.completed","executor.pool.core","executor.pool.max","executor.pool.size","executor.queue.remaining","executor.queued","hikaricp.connections","hikaricp.connections.acquire","hikaricp.connections.active","hikaricp.connections.creation","hikaricp.connections.idle","hikaricp.connections.max","hikaricp.connections.min","hikaricp.connections.pending","hikaricp.connections.timeout","hikaricp.connections.usage","http.server.requests","jdbc.connections.active","jdbc.connections.idle","jdbc.connections.max","jdbc.connections.min","jvm.buffer.count","jvm.buffer.memory.used","jvm.buffer.total.capacity","jvm.classes.loaded","jvm.classes.unloaded","jvm.gc.concurrent.phase.time","jvm.gc.live.data.size","jvm.gc.max.data.size","jvm.gc.memory.allocated","jvm.gc.memory.promoted","jvm.gc.overhead","jvm.gc.pause","jvm.memory.committed","jvm.memory.max","jvm.memory.usage.after.gc","jvm.memory.used","jvm.threads.daemon","jvm.threads.live","jvm.threads.peak","jvm.threads.states","logback.events","process.cpu.usage","process.files.max","process.files.open","process.start.time","process.uptime","spring.data.repository.invocations","system.cpu.count","system.cpu.usage","system.load.average.1m","tomcat.sessions.active.current","tomcat.sessions.active.max","tomcat.sessions.alive.max","tomcat.sessions.created","tomcat.sessions.expired","tomcat.sessions.rejected"]}
请求 /metrics/disk.free
{"name":"disk.free","description":"Usable space for path","baseUnit":"bytes","measurements":[{"statistic":"VALUE","value":3.29164107776E11}],"availableTags":[{"tag":"path","values":["/apollo-portal/."]}]}
期望
暴露标准的Prometheus格式
如：
#HELP go_gc_duration_seconds A summary of the GC invocation durations.
#TYPE go_gc_duration_seconds summary
go_gc_duration_seconds{quantile="0"} 1.0087e-05
go_gc_duration_seconds{quantile="0.25"} 2.2333e-05
go_gc_duration_seconds{quantile="0.5"} 4.3757e-05
go_gc_duration_seconds{quantile="0.75"} 9.1701e-05
go_gc_duration_seconds{quantile="1"} 0.000750728
go_gc_duration_seconds_sum 14.936457049
截图

额外的细节和日志

版本：2.1.0
错误日志
配置：
平台和操作系统   linux k8s

文档中写得是/prometheus，不是/metrics

文档中写得是/prometheus，不是/metrics

实在抱歉，跟我自己的东西搞混了。


Typing lag when using tsc watch

VSCode Version: 1.6.0
OS Version: Windows 10
TypeScript Version:  2.0.3

I'm not sure whether or not to post this in Typescript repo or this one.
I am getting quite bad typing lag when I've got my tsc transpile task running. The extent of the lag is if I type "hello world" it takes me about 2secs to finish the keystrokes but only fully on the screen after 7-8 seconds.
I've tried the same with Atom and the atom-typescript plugin which does not have the same issue, I'm not sure what they're using under the hood to do the watch and transpile. I am very happy to debug and provide further diagnostics but not sure what the best way forward is.
I also use the vim extension, but have got the same result with all extensions removed. The typing lag only happens if I initiate the Task > tsc.
Project size: around 200 files (.ts + .js)
tasks.json:
{
  // See https://go.microsoft.com/fwlink/?LinkId=733558
  // for the documentation about the tasks.json format
  "version": "0.1.0",
  "command": "tsc",
  "isShellCommand": true,
  "args": ["--watch", "--project", "./app"],
  "showOutput": "silent",
  "isWatching": true,
  "problemMatcher": "$tsc-watch"
}

tsconfig.json:
{
  "version": "2.0.0",
  "compileOnSave": true,
  "compilerOptions": {
    "target": "es6",
    "module": "commonjs",
    "isolatedModules": false,
    "jsx": "react",
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "declaration": false,
    "noImplicitAny": false,
    "removeComments": true,
    "noLib": false,
    "preserveConstEnums": true,
    "suppressImplicitAnyIndexErrors": true,
    "watch": true
  },
  "include": [
    "./**/*.ts",
    "./**/*.tsx"
  ],
  "exclude": [
    "!./node_modules/**/*",
    "!./.sass-cache/**/*",
    "!./.vscode/**/*",
    "!./logs/**/*",
    "!./public/dist/**/*",
    "!./public/lib/**/*"
  ]
}

@eetee are you able to share your project. That would be the easiest to track this down. If not could you do the following:

Help > Toggle Developer tools
In the Dev tools select Profiles
Take a JS CPU Profile
Do your typings...
Attach the profile here.

Sorry unable to share the project.
The typing lag didn't happen when I opened the project and started to do some typings. Thinking back it's entirely possible that it happens after a period of having the editor open.
Would it be best if I proceed with something like the investigation steps in #4114? i.e. take the heap snapshots and then also the profile when it starts lagging?

@eetee yes, that would be perfect.

Without the additional information we can't work on this issue. Please ping when the information is available and we will reopen the issue.


Making Leaderboard solution live

Docs Build status updates of commit a8209b8:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Articles/BusinessSolutions/Leaderboard/Leaderboard.md
:warning:Warning

Details

Articles/BusinessSolutions/Leaderboard/Leaderboard-SolutionPackage.zip
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard.csv
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/1.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/10.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/11.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/2.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/3.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/4.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/4.jpg.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/5.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/6.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/7.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/8.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/9.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard.md

Line 31: [Warning] Invalid file link:(~/BusinessSolutions/Leaderboard/Leaderboard-SolutionPackage.zip).
Line 42: [Warning] Invalid file link:(~/BusinessSolutions/Leaderboard/Leaderboard.csv).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit dcdfbb9:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Articles/BusinessSolutions/Leaderboard/Leaderboard.md
:warning:Warning

Details

Articles/BusinessSolutions/Leaderboard/Leaderboard-SolutionPackage.zip
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard.csv
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/1.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/10.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/11.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/2.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/3.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/4.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/4.jpg.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/5.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/6.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/7.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/8.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/9.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard.md

Line 31: [Warning] Invalid file link:(~/BusinessSolutions/Leaderboard/Leaderboard/Leaderboard-SolutionPackage.zip).
Line 42: [Warning] Invalid file link:(~/BusinessSolutions/Leaderboard/Leaderboard/Leaderboard.csv).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 1a930db:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Articles/BusinessSolutions/Leaderboard/Leaderboard-SolutionPackage.zip
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard.csv
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard.md
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/1.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/10.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/11.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/2.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/3.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/4.jpg
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/4.jpg.png
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/5.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/6.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/7.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/8.JPG
:white_check_mark:Succeeded

Articles/BusinessSolutions/Leaderboard/Leaderboard_Images/9.png
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


refactor: improve kubeflow docker image

What changes were proposed in this pull request?

Don't switch root when installing python packages (as this makes the files not owned by the jovyan user)
Update the base FROM image to: public.ecr.aws/j1r0q0g6/notebooks/notebook-servers/jupyter:v1.3.0
Removes explicit yarn install
Re-orders the install so that user-provided requirements.txt happen AFTER the jupyter lab build (which takes a LONG time)

fixes: https://github.com/elyra-ai/elyra/issues/2110
How was this pull request tested?

by running make kf-notebook-image at the root of the repository

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

cc @lresende


lookup: Update docs

As the branch name suggests, using sudo with yay is an anti-pattern and so this fixes that.

Good catch. Thanks again!