fix(upgrade): Migrate 1.24 users to 2.13 (update message of upgrade notification) EE-2167

closes #0
closes [CE-0]
Changes:
update message of upgrade notification

/azp run


[deepcast] .. setTargetLanguage to last selection

Extension
https://github.com/raycast/extensions/tree/a66477346699916635254ff66b88e76b338514c8/extensions/deepcast
Description
Why is the target language set to EN-US? I want to select another target language. It would be best to activate the last selection again. A super feature would be, if I could set the target language in the settings.
 const [targetLanguage, setTargetLanguage] = useState<TargetLanguage>("EN-US");
Steps To Reproduce
https://github.com/raycast/extensions/blob/a66477346699916635254ff66b88e76b338514c8/extensions/deepcast/src/index.tsx
Current Behaviour
No response
Expected Behaviour
No response

Good morning!

Why is the target language set to EN-US

I guess as it's default in Raycast.

A super feature would be, if I could set the target language in the settings.

This doesn't sound very complicated — just another preference.
However, we should choose between default target language in settings or the language from last translation. I don't currently see a way of combining these two approaches.

But the target language is defined in the code and not in Raycast?
const [targetLanguage, setTargetLanguage] = useState<TargetLanguage>("EN-US");
at line 29: https://github.com/raycast/extensions/blob/a66477346699916635254ff66b88e76b338514c8/extensions/deepcast/src/index.tsx

That’s true, yep.
Are you thinking about selecting default target language in settings?
Or using the previously selected language?
Maybe there’s a way of combining these two approaches, but I don’t see it now.

I wouldn't really care. Both options are good. I was only surprised that since the extension update the target language is now always fixed and then also in US English. Before, at least the last selection was always used and the behaviour was wonderful for me.


Fix test for Create a bootstrap row challenge.

This commit fixes the tests to check if the
 with row class is
actually nested below the
 tag.
Tested locally.
closes #6728

I can still pass with the following code that does not match what the test case claims to check for:

jQuery Playground

Sorry, I had pushed an incorrect local branch. Thanks for pointing out
Should be fixed now :sweat_smile: !
Tested for the below cases:
Should Fail for:

jQuery Playground

jQuery Playground

jQuery Playground

And Pass for:
The correct case :wink:

Woot! Looks perfect :+1:


How to integrate this with reactjs

I am trying to integrate neo4jd3 with my reactjs app.  i also tried this
https://github.com/eisman/neo4jd3/issues/10.
but getting lots of error.
please help me

Any news about that?
any one have codepen example?


PLANNER-285 Benchmark Aggregator: Replace 'Expand/Collapse all' by 'Expand/Collapse' (only the current selected item)

Issue link: https://issues.jboss.org/browse/PLANNER-285

Thanks Matej! Works like a charm :)


YAML files generated from xml2yml.m

Hello,I am a student who want make my own data set like VOC2007, but the Annotations of VOC2007 is
xml, I need transform xml to yml, like the follow form ,
%YAML:1.0
annotation:
folder: VOC2007
filename: "000001.jpg"
source: {database: The VOC2007 Database, annotation: PASCAL VOC2007, image: flickr,
flickrid: '341012865'}
owner: {flickrid: Fried Camels, name: Jinky the Fruit Bat}
size: {width: '353', height: '500', depth: '3'}
segmented: '0'
object:

bndbox: {xmin: '48', ymin: '240', xmax: '195', ymax: '371'}
name: dog
pose: Left
truncated: '1'
difficult: '0'
bndbox: {xmin: '8', ymin: '12', xmax: '352', ymax: '498'}
name: person
pose: Left
truncated: '1'
difficult: '0'

Can you tell me how to transform,Thank you very much

Prepare all your info about the box of objects in picture. And thento put the info in XML or YAML file by APIs.
For example, opencv has such APIs

How to run wy own data?


Workbook with veryHidden Sheets can't be edited.

Hi Xuri,
We are editing Third Party Excel files and there is one that has a veryHidden sheet. If we try to open/save this workbook with Excelize we get an error 'sheet mod01EssentialInfo is not exist' (where mod01EssentialInfo is the veryHidden sheet).
See attached the file, and screenshot from Libre Office to show the veryHidden sheet. Is this a limitation of Excelize?
veryHidden.xlsx

I don't think this is related to the veryHidden sheet, as I just made a spreadsheet with a veryHidden sheet and it worked fine, but because the veryHidden sheet is listed in the workbook.xml but not in the worksheets directory.


lsp_zero.cmp_format() hides important information for rust_analyzer

Hi all,
First, thanks for this great module. It makes life with Neovim way easier! :)
I noticed that the default lsp_zero.cmp_format() completely overwrites the menu field.
In rust_analyzer, the menu field sometimes contains essential information, e.g., from which crate a struct would be imported. Without this information, you are occasionally left with multiple identical entries.
local cmp_format = lsp_zero.cmp_format()

cmp.setup({
  formatting = cmp_format,
..
}

On the other hand, without configured formatting, I get the expected information.

Could this be documented/mentioned somewhere? Or is this related to something else, and should the menu not contain this information?

Okay. I've added an extra option in .cmp_format() to show that information. Now you can do this.
local cmp_format = lsp_zero.cmp_format({details = true})

The default behavior will stay the same, but I've added a note in the documentation.

That was quick, thanks!


Android: play can only be initiated by a user gesture

Hello,
The youtube player on mobile isn't recognizing the playVideo() method as an user input and It is blocked with the following warning:
"Failed to execute 'play' on 'HTMLMediaElement': API can only be initiated by a user gesture."
I've also tried { autoplay: true 1 } in the opts without success.
In my app, it is initialized by a "user gesture" but the player can't know that. Do you know any workarounds ?
By the way, thank you for this amazing project !

Seeing as you wrote {autoplay: true} can you confirm that your opts object is what the Youtube IFrame API expects?
More information about the opts object can be found here:
https://developers.google.com/youtube/player_parameters?playerVersion=HTML5
and a working example of the autoplay option can be found in the readme of this project:
https://github.com/troybetz/react-youtube#example

Oh sorry, that's a typo in my post, i've fixed it. I've tried, every combinaisons of "autoplay" option and "playerReady" method. This is a sample of the code I used:
 <YouTube videoId={video.id}
          onReady={this.onPlayerReady}
          opts={{
            height: '200',
            width: '200',
            playerVars: {
              autoplay: 1
            }
          }}
/>

The playerReady method:
onPlayerReady(event) {
  this.setState({
    isFetching: false,
    player: event.target,
  })
  event.target.playVideo()
}

This issue seems related to this.

Hello, any inputs about this ?

Hey since there was a long period of inactivity on this issue (and no response since my last suggestion) i figured it would be okay to close it... The documentation of the youtube iframe api now contains a note regarding this issue: https://developers.google.com/youtube/iframe_api_reference#Mobile_considerations

Okay no problem! Thanks for this, I'll have a look.
Cheers!


Fixed IDENTITY-4367

Carbon Jira: https://wso2.org/jira/browse/CARBON-16033

Reviewed. Can merge.


Document the need to include Primer::ViewHelper in order to use the helpers

The Octicon docs mention possible use of the primer_octicon helper and the PrimerOcticon cop corrects all usage of the octicon helper to be primer_octicon instead, but the docs don't make it clear how to get the primer_octicon helper to work.
It appears that one must either have include Primer::ViewHelper in their ApplicationHelper or helper Primer::ViewHelper in their ApplicationController (or any other more specific controller). I'd suggest the Getting Started page is updated to include this info as part of the Installation

👋🏻 Hi @kjg, thanks for reporting this! We would welcome a PR to the docs; I believe this is the file to edit. Let us know if we can help!


Cert.pem and Key.pem Files Purpose

Hi, what are the additional files in the zip packages for?
They are called Cert.pem and Key.pem.
Thank you for the support.

Its to set up a local ssl server for use of Ledger locally. Knowledge base article coming soon. Sorry for the delay

Thank you for the clarification. This will be a great feature.


Improvements and fixes in filter select and form select

Added model method in Grid/Filter/Presenter/Select to reload selected data.
Now it's possible to modify the configuration of select2 in Select filter.
Fixes in Form/Field/Select to keep the API consistent.

Thanks!


Can't access some parts of u18chan.

Any board under the section labelled "The Basement" gives an invalid server response.

It seems that they've implemented a multiple files posting so it causes a parsing error. I'll fix it.


CMSIS-NN LSTM issue fixes

Remove read of non-initialized buffer
Adds back integer unidirectional_sequence_lstm_test for CMSIS-NN
Adds back dtln example using CMSIS-NN
Reverts "Add intermediate tensors to LSTM unit test to enable CMSIS-NN (#1996)"

BUG=#2449

@Mergifyio refresh

@Mergifyio refresh


CAMEL-15496: Add support for additional headers and properties

The additional headers will be added on top of the existing headers
from the message.
The properties added are only basic AMQP properties as defined in the
{@link com.rabbitmq.client.AMQP.BasicProperties.Builder}. When the
message contains already these properties then will be considered and
these additional properties will be ignored.
Introduced a SimpleDataHolderBean to hold a hashmap for adding any
header. Changed the RabbitMQConstants to Enum so that we can also add
the property description in that.
The rabbitmq AMQP basic properties constant keys should not
have dot in their prefixes. This gives problem in trying to
retrieve the value due to fix of CAMEL-15394.
As with this fix though we can add any header, only static/hard-coded
values could be associated.

I am using IntelliJ and I couldn't format the files..
I imported the code-style from buildingtools/src/main/resources/camel-eclipse-formatter-config.xml
mvn install -Psourcecheck always fails.

[ERROR] Failed to execute goal org.apache.camel:camel-format-plugin:3.6.0-SNAPSHOT:validate (validate) on project camel-core: File '/home/ganesh/IdeaProjects/github/camel-unifly/core/camel-core/src/test/java/org/apache/camel/util/EndpointHelperTest.java' has not been previously formatted.  Please format file and commit before running validation! -> [Help 1]

Can you please let me know which code-formatter is used by all ?

I am using IntelliJ and I couldn't format the files..
I imported the code-style from buildingtools/src/main/resources/camel-eclipse-formatter-config.xml
mvn install -Psourcecheck always fails.

[ERROR] Failed to execute goal org.apache.camel:camel-format-plugin:3.6.0-SNAPSHOT:validate (validate) on project camel-core: File '/home/ganesh/IdeaProjects/github/camel-unifly/core/camel-core/src/test/java/org/apache/camel/util/EndpointHelperTest.java' has not been previously formatted.  Please format file and commit before running validation! -> [Help 1]

Can you please let me know which code-formatter is used by all ?

you just need to build the entire project and it will be auto-formatted.

Even after repeated attempts to rebase and merge I am getting failures in the checkstyle step.
I will close this PR and open a new PR from master.


get on github marketplace

currently blocked, because multi-action repos are not yet supported

I just tried to poke this by adding a release, but it's still not working, so I guess there's still no support for multi-action repos on GitHub actions marketplace.

I aksed GitHub on Feb 2 about this:

thanks for the invite to the "staff ship" thing. I did publish my deploy to gh-pages action: https://github.com/marketplace/actions/github-pages-deploy
The E-Mail you send mentioned that there has to be a Dockerfile at the repo root. I'm wondering how this would work for repos that have multiple (interrelated) actions.
Specifically, I'm building another repo with lots of actions for the R platform for statistical computing, and there, the individual actions and their Dockerfiles are all in repo/action-foo/. I vaguely remember that the gh action docs recommend that such related actions be grouped in one repo (sorry, can't find the source).
The latter seems to make more intuitive sense to me; otherwise I worry I'd need to deal with submodule (and who wants that?).
Some guidance on this would be much appreciated.

Response was that this was actively being worked on.
Alternative is, I guess, to use submodules (arghh :()


Tidying drawing of header

Prevent one thing you can not see drawing over something else you can not see. Part of a tidy up ahead of introducing user colour customisation option.

@mattbrejza looks good to me, you're happy to merge?

Yes, please merge away.


Duplicate cells during incremental update with HEAD + META

When I add another <div> tag, it duplicates the section:
https://github.com/observablehq/cli/assets/37343722/22beb08e-902c-449f-8b2d-8870001d3754

Fixed in #843.


How to set git to English separately for vscode and use localized languages for others

How to set git to English separately for vscode and use localized languages for others

git translate It's difficult to read

I don't believe that is possible. @TylerLeonhardt, please correct me if I am wrong.

Yes, this is not possible. We'd need some sort of an extension specific override.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Data Hub Tutorial Video

Add the Data Hub Overview tutorial video to the portal.

This task has been completed.


413 Entity Too Large

Referring to: https://docs.sentry.io/clients/javascript/sourcemaps/#uploading-source-maps-to-sentry
indicates that we should be able to upload up to 40mb. Currently, i am getting a 413 error when uploading more than 20mb. I have narrowed the issue down to this:
When using https://docs.sentry.io/api/releases/post-organization-release-files/ , the request limit seems to be lower than when using https://docs.sentry.io/api/releases/post-project-release-files/ (which is what the initial link at the top of this post seems to use). I believe this is a recent issue as a result of https://github.com/40thieves/webpack-sentry-plugin/pull/36

Note that version 1.11.0 of this plugin is working correctly.

@kjanoudi Thanks for the report. Using the organization release endpoint was necessary for the plugin to support Sentry's release commits. You might consider opening a support ticket with Sentry to ask whether they intended for the endpoints to have different max upload sizes; that seems like a strange behavior to me. Otherwise I would be amenable to having a configuration option for which endpoint ought to be used.

Related / Unrelated Q: Does Sentry have an API-wide max Entity size? (for artifacts or incident reports)
Is there a way to increase it on the dashboard?

@drewlustro I'm not sure. Note that this is an unofficial community project, I/we don't have any formal affiliation with Sentry. I'd recommend writing into their support with your questions.

@kjanoudi Closing the issue as from https://github.com/getsentry/sentry/issues/6331 it sounds like this was a misconfiguration on sentry.io that's been resolved.

@schneidmaster - my man, thanks for the info.
Love the plugin, thanks for authoring; using it in production at @capsulehealth FWIW

@drewlustro Thanks! Most of the authorship credit goes to @40thieves though, I just help maintain the plugin these days.

No all credit to @schneidmaster (& @rhys-vdw) for maintaining :)


Add uninstall subcommand

Part of #3.
This errors if the tool isn't in any discovered manifests, e.g. if it's installed locally in another directory. Is this desired behavior or should it still uninstall as long as the tool binary exists?

I didn't see that. Great!


Support @media CSS queries

As far as I can tell, capybara-webkit doesn't currently support @media queries. These are needed in order to properly test mobile workflows.
Here's someone else that has the same issue: http://stackoverflow.com/questions/37768656/capybara-webkit-not-acknowledging-media-query-for-min-width-in-css

This is most likely an issue with the version of Qt used.


[RFC] Evented IO mode as the only mode

We're working on an IO backend on top of the great libuv library (the one that powers nodejs). The current results are very promising and it's already possible to compile a HTTP server for example, capable of answering multiple calls at the same time.
The implementation makes heavy use of fibers to give control back and from the libuv event loop. New lightweight processes can be started with spawn and every time an async IO operation is started the fiber yields the control back to the loop. The event callbacks are set by the implementations so the feeling is very much like multithreaded programming.
In order to make use of this new implementation a flag "evented" must be passed to the compiler. The std classes are being populated with tons of "ifdef evented" to switch the implementation.
At the beginning we thought that maintaining both implementations could be worth to someone looking for a language that is able to make quick synchronous IO on a single thread/fiber, but so far I cannot think on any real use case of this, besides having awesome numbers on some benchmarks.
The startup overhead is really small (currently the main thread hast to be running inside a fiber, so at least one must be created and there is also the libuv initialization and loop running) and if no IO is performed by the application there shouldn't be major differences.
So, the question is... is it really worth the effort of maintaining both implementations, or should we focus on having a single one well tested and stable? Does someone see any scenario where synchronous IO makes a real difference in performance? And besides performance, is there any other reason to use sync IO?

I believe switching to async only is a wise decision.  This would give rubyist a familiar language to migrate to with the promise of node like concurrency.  Which, if your willing to make the trade offs for the performance of crystal, would probably be a big selling point.  I can see that playing a major role in language adoption.

A fully concurrent language is the way to go. But an evented concurrency with libuv that must run on a single thread... is no fun at all.
If crystal multithreads fibers, then it would be incredible and, yes, please make -Devented the only concurrency.
I know libpcl supports multi threading, which is great, but apparently libuv isn't thread safe, and all uv functions must be called from one, unique, thread.

... should we focus on having a single one well tested and stable?

This is a big plus, especially if there is not enough resources to maintain different I/O models.
Because of the limited resources, good features (TLS, gzip compression, static file handling, ...) would be implemented faster as well.
Some libraries show that evented I/O works very well (Node.js, Ngnix, ...). You can handle virtually unlimited amount of I/O concurrency. Slow clients will not to be a problem, thereby eliminating the need for a buffering reverse proxy. Of course its a little bit harder to implement a good evented model compare to a blocking I/O model.
I would say "yes" for evented I/O "only" mode. Maybe a sync version is provided as a library?!

@weskinner That's the idea. Crystal is really fast already, but adding native support for concurrency would make it suitable to make scalable systems.
@ysbaddaden I thought we could start with a single thread and move later to a multi worker thread model. In Go the network poller loop is run on a single thread. I wonder how they manage to register new network file descriptors asynchronously into the loop.
Maybe we should reconsider if libuv is the best choice before moving any forward. Maybe libevent has a better design for multithread environments although I read some benchmarks in which libevent is much slower that libuv. Or... maybe forget about any framework and get our hands dirty with epoll/kqueue directly.
@akaufmann The idea is the apis remain synchronous. Even when the runtime would be making async calls and running evented IO underneath.

I see. But whats the point to use libuv async functions underneath if my fs operations are sync?
If I have File.read it will block (your event loop runs empty, because no other event was registered after the fs_read) until a result is returned. Is it my job to call spawn on sync functions to get an async behavior?
Btw, I think Go used a lock to allows runtime poll updates to be called synchronously from the loop thread.

Yes, you basically have to use spawn to get real concurrency. That's pretty much how Go works, using the go keyword to start new "goroutines". When the File.read blocks (the current fiber) the scheduler would give control to another job, or just wait for the result if there isn't anything else to do.

I generally support the idea of this being the primary or only mode going forward. I think if this concept gets implemented right, Crystal potentially wins the next decade. (no pressure, huh?)
A thought:
This is about 90% messaging and 10% technical, but we might want to consider overhauling the concept of a Fiber. As they exist in Ruby, they're essentially useless as a high-level concurrency construct. Instead, Fiber could move away from being a co-routine and move toward being a true light-weight thread like Erlang's "process" or (increasingly) Go's goroutine. Threads without the overhead of POSIX threads and schedule themselves around I/O and/or other events (channels? timeouts? course-grained pre-emption?)
Essentially that means dropping the yield, resume etc methods so that scheduling is handled entirely in the runtime.

The whole concurrency thing is so complicated. I long for Erlang-like processes (actors) too, and I would love that concurrency/parallelism would be just like that (or something along those lines):
process = spawn do
  receive do |message, caller|
    puts message  # message
    caller.send "got it"
  end
end

process.send "message"

receive do |message|
  puts message # got it
end

That being said, about the current RFC, if making the evented IO (libuv) the only concurrency (for now) means that I can still do IO within different threads without segfaults like this, then I'm fine with it :+1:

@brasten Many thanks for your words! Let's try to do our best.
Probably as the design evolves there's going to be a separate entity for coroutines, but I still think fibers could be useful in some cases.
@ysbaddaden We're thinking something more like Go channels instead of just a process inbox, but that's the idea. You can already try to use the HTTP server with libuv if you compile with -D evented flag. It's not multithreaded but it works pretty well so far.

If you get a segfault remember to wrap your main code in a spawn. This will be done automatically later, but for now you have to do it manually.

@ysbaddaden Why are you getting a segfault there? Isn't it because the "socket" variable is always the same so every thread you create will refer to that same variable?

@asterite I didn't meant that it segfaulted, but that I should still be able to do IO in many threads without libuv segfaulting on me —until Crystal goes for a great de-facto concurrency/parallelism scheme.
I just tried, but Prax won't compile with -Devented right now: crystal/src/io.cr:82: undefined method 'fd' for TCPServer.

Some progress on this... I changed my mind and it doesn't use libuv anymore. Instead libevent2 is used that should have better support for threads in the future. The implementation it's less intrusive and easier to replace with something else (libev? direct calls to epoll/kevent/etc...)
I started a new branch (evented) where this is the default and only implementation of IO in Crystal. It works pretty well and all the specs are passing (there isn't many IO specs anyway but at least the compiler can still build itself).
I have a problem I couldn't figure out yet with the TCPServer. After some successfully attended requests, the "accept" fails and errno == EINVAL. Do someone have any idea of what's going on there?

Ok... I figured out the problem. There was a bug with the fibers and GC.

That sounds very good! I read about the different solutions, and I think libevent or libev will require more work to integrate, but should prove far more flexible than libuv, which should fit better to Crystal (I hope).
I'll try to find some time to play with the branch and to discuss/propose enhancements :-)
About libevent/libev the former has some weird naming, but I don't know about libev and thread safety.

We will push this idea forward :-)


[vm/ffi] Pointer

What do we do with Pointer<Null> in the FFI? (This is allowed by the Dart language.)

This type could be useful for the nullptr constant, so that we don't need to cast it to use it in other contexts:
Pointer<Null> nullptr = Pointer<Null>.fromAddress(0);

// ...
void takesPtr(Pointer<Int8> ptr);
// ...

takesPtr(nullptr);  // Actually works!

linking https://github.com/dart-lang/sdk/issues/35756

What needs to happen? AFAICT, everything is working as intended.

We don't actually use Pointer<Null> as nullptr:
/// Represents a pointer into the native C memory.
final Pointer<Void> nullptr = Pointer.fromAddress(0);

If we can, we should change it to that maybe.
We don't have any tests containing Pointer<Null>, we should probably add one.


Wrong items alphabetical ordering in server/database view

SQL Operations Studio Version: sqlops 0.23.6 (8a3011756318a2ded1e6e84332230a2b2bf9c544, 2017-11-06T21:51:43.232Z)
OS Version: Windows_NT x64 6.1.7601

Steps to Reproduce:

Create a table named "ATable"
Create a table named "_ATable"

What I see:
Table "ATable" appears first in the list

What I expect:
Table "_ATable" should appear first.

this bug is fixed and will be in the next release


[workers] Add type/external URL to tutorials

Adds the type column in tutorials. In addition, since we're experimenting with adding the new "Video" category (linking to our egghead course), this PR also includes support for setting a URL for a Markdown file, which will cause it to link directly to external URLs.

Updated from category->type via @rita3ko's request

Updated from category->type via @rita3ko's request


Added ACLs for mutation and query

I have added ACLs for mutation and query on Loopback models

any update on this ?

Any update on this?


typings: Add is_public to the application model

Resolves: #676
Change-type: minor
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>
Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

@thgreasi yeah, that makes more sense, thanks for clarifying.


Unable to resolve module AccessibilityInfo

Installing this package breaks react-native@^47:
This is the error that pops up:
error: bundling failed: "Unable to resolve module `AccessibilityInfo` from `$ROOT/node_modules/react-native/Libraries/react-native/react-native-implementation.js`: Module does not exist in the module map\n\nThis might be related to https://github.com/facebook/react-native/issues/4968\nTo resolve try the following:\n  1. Clear watchman watches: `watchman watch-del-all`.\n  2. Delete the `node_modules` folder: `rm -rf node_modules && npm install`.\n  3. Reset packager cache: `rm -fr $TMPDIR/react-*` or `npm start -- --reset-cache`."

Uninstalling nachos-ui DOES NOT fix this error.

This issue is being tracked now in #34. Closing.


Extension README content should be wrapped to 80-100 characters

#8215

VSCode Version: Code - Insiders 1.3.0-insider (d0c2b891903d73e83dcdc7786398ebfa764fc4ee, 2016-06-28T05:01:27.563Z)
OS Version: Linux x64 4.4.0-24-generic, Ubuntu 16.04

Extension README content should be wrapped to 80-100 characters to significantly improve readability/presentation (just like how this GitHub issue is wrapped).
This is bad:

Related: https://github.com/Microsoft/vscode/issues/8251

Bad build...


CSP option does not handle styles in a way compatible with CSP

Vulcanize allows inline styles even when the CSP option is enabled.  However inline styles are not allowed under CSP, unless excepted.
Just as the CSP option extracts inline scripts, it should also extract inline styles.
This can be demonstrated with the following html file:
<script src="bower_components/webcomponentsjs/webcomponents.min.js"></script>
<link rel="import" href="bower_components/polymer/polymer.html" />

Running vulcanize with CSP over the above the resulting html file contains a <style shim-shadowdom=""> element that causes CSP violations.

This causes functional problems when trying to consume elements.  As such, it is not sufficient to just write one's own elements to not contain inline styles.
Here is are examples of a paper-input element using vulcanize on Chrome Canary (43.0.2314.0 canary (64-bit)):
with CSP:

without CSP:

<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
  <title>Polymer Demo</title>
  <script src="bower_components/webcomponentsjs/webcomponents.min.js"></script>
  <link rel="import" href="bower_components/paper-input/paper-input.html" />
</head>
<body>
  <label>Count:</label><paper-input></paper-input>
</body>
</html>

Unfortunately, the style machinery for Polymer when running without native ShadowDOM requires polyfilling styles, and has to then inline them.
Preventing inline styles seems like a rather heavy-handed CSP rule to use in practice. Can you tell me more about the environment that requires this? Even Chrome Apps do not have this restriction: https://developer.chrome.com/apps/contentSecurityPolicy

Could the the polyfilling be accomplished with an explicit inclusion of a CSS file, such as how one must include angular-csp.css for angular?
I believe the inlining you are talking about is an issue, but a different one than the <style>s provided by individual components. Otherwise, I am not sure why my own custom elements (written with the style in a separate file) seem to avoid the issue.
These are the relevant requirements:

CSP must be used
Make the application as secure as reasonably possible

In my case I can probably get unsafe-inline approved for styling as my requirements are relatively informal and the relevant stakeholder is amicable to change.
If Polymer's design intention is that it will work with CSP but only with unsafe-inline I will create a new issue over at Polymer/docs since that is not at all clear from the section on CSP.  Coming from AngularJS I would expect CSP to work with out anything being relaxed unless otherwise specified.

Unfortunately, disabling inline style is not heavy handed at all. There are numerous clever attacks that use inline style (see http://scarybeastsecurity.blogspot.com/2009/12/generic-cross-browser-cross-domain.html for an example), so for any sensitive page, it is quite important to not use unsafe-inline for style-src. It would be really fantastic to find a way resolve this, since as it stands, Polymer is not compatible with a secure and useful CSP :-(


Refactor the storage of comments to have identity be dependent on the user id

Present implementation stores the identity of the user thus making changes in identity tougher to be propagated.
Ideally we would have a function that hooks onto a central map that associates a user id to the identity of the user and thus having a centralized model for identity.

Hi @srirag-vuppala, I want to say I have some idea about this. Would love to connect, and see if i can help :)

Definitely! I'll send over an email and we can go from there 👍🏾

Hi @srirag-vuppala, I want to say I have some idea about this. Would love to connect, and see if i can help :)

Definitely! I'll send over an email and we can go from there 👍🏾

Thx @srirag-vuppala for the email. I am on also on gitter https://gitter.im/jess-x, and might be more responsive there


Modernize the report.html when using the --modern-ui Flag

Prerequisites

[X] I am using the latest version of Locust
[X] I am suggesting a new feature, not asking a question

Description
Is your feature request related to a problem? Please describe.
Following suite with the new modern web UI #2396, we should modernize the report.html as well
Describe the solution you'd like
When the --modern-ui flag is active, return a modern looking report.html using the same technologies as the new modern ui. When the flag is not active, the report.html should be unchanged
Additional context
Already have a PR ready for this!

Fixed in #2420


✨ Add EIP-2098 fallbacks

Description
Everyone uses nonces. I think.
With the spread of smart wallets on L2s, which use bytes for signature due to 4337 and 1271, this makes sense.
Checklist
Ensure you completed all of the steps below before submitting your pull request:

[x] Ran forge fmt?
[x] Ran forge snapshot?
[x] Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

Thanks @Philogy for the re-review.

Looks good to me, just some small notes:

you need to update the comments on the 4 updated methods that EIP-2098 is now accepted
I'd switch the normal/EIP-2098 check blocks to optimize for EIP-2098, in practice I think this is likely to be the more commonly used one

Generally I'm not sure if it's a good idea to add both EIP-2098 and normal signature checking code for the same methods, I'd create separate variants that allows devs to choose, allowing each to be more compact and gas efficient.


Mask labels in a track

Is there a way to identify the original mask labels  in a track? Thanks for making this package available and please let me know if there's a preferred way to ask for features or get guidance.

Hi Mark,
You should be able to use the assign_class_ID flag in the segmentation_to_objects function to do this. From the tracks you can then get the original mask labels using:
original_mask_labels = track.properties["class_id"]

Let me know if you have any issues

I'm going to close this


Non-string template variables (alt: template overrides from git)

I have a bunch of applications loaded from git.
Sometimes we need to stop syncing per application.
It seems there is currently no way to achieve this? If I got it right:

You cannot set syncPolicy.automated.selfHeal to '{{ autoSync }}' as selfHeal needs to be a boolean. Having only strings as variables is quite a heavy limitation

Another place where I have this issue is valueFiles for Helm, which are also app dependent, but this can be worked around by providing a single concatenated string as the list item

You cannot load generated templates when using git generator?

have that substituted into the ApplicationSet template

Looking at the example from the thread, yes - it could work for the use cases I'm thinking of.
Thanks!

We are investigating using ApplicationSets to provision helm applications. Helm supports list templating and we use this to generate n amount of resources from a helm values list. Since ApplicationSets currently only supports strings we cannot use ApplicationSets to provision our applications.
Since json/yaml is used as the substitution configuration it makes sense to me that ApplicationSets should support the full typeset of json/yaml. Otherwise json/yaml does not make sense as a config file type.
The docs also currently do not make it clear that only strings (or nested objects, as long as eventually the value is a string) are supported so it is easy to make the mistake that since the config file is json, the assumption can be made that basic valid json would work (I would argue json lists are a pretty basic json feature).

@rouke-broersma Very well written as you say exactly my experience with this;
First being surprised that a json list didn't work and then  trying to understand what kind of limitation the templating part has (documentation doesn't specify, and only have very basic examples).


Memory leak?

please fix.

According to the docs,   is deprecated and has been replaced by  .
This should definitely be fixed, since apparently   can lead to memory leaks when still used.

I totally agree with @cybrox, except sometimes you can even get away by doing  , rather than  .
It's all up to you though.

Don't forget to call   or you might end up with memory corruption, be careful. And remember,   mutates, so don't forget to clone  .

Look thoroughly, this leak has negative sign, so each nocode run gives you additional memory. Effect is persistent. When your memory achieves 64GB, memory will be added to your neighborhoods.

It seems your memory is too low. Be aware that the project contains nothing.
I suggest clear cache, then restart your device. Working for me...

Cannot reproduce this problem, please buy a new MacBook

Dzenly//no code = peace and love, let's kill codes

@hanseul1795
The peace contains the idea of hatred, the 'no code' contains the idea of code.
I will just watch and meditate.

@Dzenly  Wise word from the master, I'll meditate it also. ZEN!


feat(nodes): organise shared node classes/objects

What type of PR is this? (check all applicable)

[x] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[x] Yes
[ ] No, because:

Description
fix(nodes): create app/shared/ module to prevent circular imports
We have a number of shared classes, objects, and functions that are used in multiple places. This causes circular import issues.
This commit creates a new app/shared/ module to hold these shared classes, objects, and functions.
Initially, only FreeUConfig and FieldDescriptions are moved here. This resolves a circular import issue with custom nodes.
Other shared classes, objects, and functions will be moved here in future commits.
Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings
Add a custom node or two to your $INVOKEAI_ROOT/nodes/ dir and run the app. Should start up.
This node is known to cause the issue: https://github.com/JPPhoto/ideal-size-node/

Added/updated tests?

[ ] Yes
[x] No 😅

Tested and works!


Fix json.loads

Prevent crash from call to json.loads. Use vm.get_attribute to get the attribute json.JSONDecodeError instead of get_item.

Codecov Report

Merging #876 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #876   +/-   ##
=======================================
  Coverage   63.62%   63.62%
=======================================
  Files          88       88
  Lines       14775    14775
  Branches     3285     3285
=======================================
  Hits         9401     9401
  Misses       3216     3216
  Partials     2158     2158

Impacted Files
Coverage Δ

vm/src/stdlib/json.rs
63.93% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for contributing!


Is this project still active

Hi,
I want to use it in another big project of mine and I want to know if its still under active development and support.
Amazing product!
Thanks

Hi there,
I do hope so too :)
@fzaninotto could you provide some status about the plans for ng-admin?

+1 Hope so too

It is active. But mid-July to mid-August is traditionally a calm period 😄

Thanks!
We love this project!


When creating a shared library (dll) on WIndows, a small static library is also created

When creating a shared library (MathLib.dll) on Windows using premake4, and running the generated makefile, in fact 2 targets are created: MathLib.dll (25 KB) and a file (libMath.a of 2 KB size). The libMath.a file seems to be a dummy (empty) static library, since it is only of 2K size. Also it isn’t cleaned up when executing premake clean.
What's happening is that when I create a shared library using premake4, it creates it alright, but it also creates a dummy static library! In fact, this is more serious than it sounds, because if there had been a (genuine) static library of that name in the directory containing the libraries, then when I run premake to create a shared library, the new (dummy/empty) static library erroneously created by premake, OVERWRITES AND REPLACES the genuine static library of the same name that was already existing in the dir.
A workaround may be to avoid creating the libraries in a common dir initially, but instead create in a separate (local) dir, perhaps the same as the one containing the (shared library creation) source files and then copy the (genuine) library to the final common library dir.
Is there any way to prevent creation of this spurious static library?
The premake file contains:
`solution "MathLib"
configurations { "Debug", "Release" }
-- A project defines one build target
project "MathLib"
kind "SharedLib"
language "C++"
files { ".h", ".cpp" }
-- Create target library in Files > C > SW >
-- Applications > ProgramLibraries
targetdir "../../../ProgramLibraries/"
  configuration "Debug"
     defines { "DEBUG" }
     flags { "Symbols" }

  configuration "Release"
     defines { "NDEBUG" }
     flags { "Optimize" }

-- Register the "runmakefile" action.
newaction
{
trigger = "runmakefile",
description = "run the generated makefile to create the executable using the default (debug config)",
execute = function()
os.execute("make")
end
}
-- Register the "runmakefilerelease" action.
newaction
{
trigger = "runmakefilerelease",
description = "run the generated makefile to create the executable using the 'release' config)",
execute = function()
os.execute("make config=release")
end
}`

That's the import library. You can turn it off with the NoImportLib flag.
Feel free to re-open if that's not clear.


Migrate PipelineResource in their own package… 📦

Changes
Extracts PipelineResource in their own package while still being exposed in tekton.dev. This is require to support embedded PipelineResource in v1alpha2 with PipelineResource still being alpha (aka v1alpha1).

This adds a pkg/apis/resource/v1alpha1 package which holds PipelineResource struct and related.
This adds a pkg/client/resource package (in addition to the pkg/client package) for the PipelineResource generated clients, lister, … code.

… and API group (resource.tekton.dev). This is mainly done to
separate the PipelineResource from the rest of the API and remove
the dependency cycle that we get by keeping PipelineResource in
alpha.
This doesn't change yet the current webhook and controller, and does
not expose PipelineResource.resource.tekton.dev (current exposed are
PipelineResource.tekton.dev).
We can expose this and support this once we expose v1alpha2 (or
v1beta1). We will need to auto-convert PipelineResource.tekton.dev
to PipelineResource.resource.tekton.dev, the easiest way would be to
have a controller handling that.
This comes from this comment https://github.com/tektoncd/pipeline/pull/1725#discussion_r356158794 :angel: .
This is an experiment, seeing if it make sense. Without this, it is very difficult to support embedded PipelineResource in v1alpha2 (or I am missing something really obvious :stuck_out_tongue_closed_eyes: )
/hold
/cc @bobcatfish
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.

/test pull-tekton-pipeline-integration-tests

I think it makes a lot of sense to move the code, but I'm a bit confused by why we want to add a new API group - looking at our other types, and even looking at triggers types such as trigger bindings they are still in the tekton.dev group (or maybe this is a mistake and the group should be different?), so I don't quite understand why we want pipelineresources to be an exceptional case thinking

Right, on triggers, I would really like having them in a different group (make it easier to detect the precense of trigger without having to know which types it exposes — right now..  | grep tekton.dev returns pipeline and triggers :stuck_out_tongue:).
The main problem I've got, so far, is that I did not manage to make the codegenerator(s) work with two distinct package serving the exact same api group (the clients are from one package only making the thing useless). I am still trying to figure out how to do that, as it would definitely simplify that pull request.
That said, I think we may discuss apiGroup in pipeline and in tekton in general :angel:.

Right, on triggers, I would really like having them in a different group (make it easier to detect the precense of trigger without having to know which types it exposes — right now.. | grep tekton.dev returns pipeline and triggers stuck_out_tongue).

Ah okay, so in that case maybe we should have pipeline.tekton.dev and triggers.tekton.dev - i.e. mirror our repo layout?
I'm still not sure it would make sense to put resources in it's own group :thinking: on the other hand that might fit well with some of the plans @sbwsg and I have been scheming :smiling_imp:

The main problem I've got, so far, is that I did not manage to make the codegenerator(s) work with two distinct package serving the exact same api group (the clients are from one package only making the thing useless). I am still trying to figure out how to do that, as it would definitely simplify that pull request.

I think it'd be better if we could get this to work, esp. if the main problem is that the tool doesn't work. If you want to write up an issue and get any of the rest of us to take a stab at it just say the word!! happy to help :D

@bobcatfish should be fixed now, no more different apiGroup.

@bobcatfish @sbwsg this should be ready for review :angel:

/hold cancel

/lgtm
thanks again for all your hard work @vdemeester 🙏 😭 🙇♀

/approve


Need help with this

Describe your problem here.
PS C:\FNF-PsychEngine-main> lime test windows
source/psychlua/CallbackHandler.hx:55: characters 18-33 : Class<llua.Lua_helper> has no field sendErrorsToLua
export/release/windows/haxe/ApplicationMain.hx:153: characters 5-36 : ... Called from macro here
C:/HaxeToolkit/haxe/lib/flxanimate/3,0,4/flxanimate/frames/FlxAnimateFrames.hx:142: characters 21-27 : Field concat should be declared with 'override' since it is inherited from superclass flixel.graphics.frames.FlxAtlasFrames
C:/HaxeToolkit/haxe/lib/flxanimate/3,0,4/flxanimate/frames/FlxAnimateFrames.hx:142: characters 21-27 : Field concat overrides parent class with different or incomplete type
C:/HaxeToolkit/haxe/lib/flixel/5,8,0/flixel/graphics/frames/FlxAtlasFrames.hx:472: characters 18-24 : ... Base field is defined here
C:/HaxeToolkit/haxe/lib/flxanimate/3,0,4/flxanimate/frames/FlxAnimateFrames.hx:142: characters 21-27 : ... Different number of function arguments
source/backend/Paths.hx:605: characters 7-18 : flxanimate.FlxAnimate has no field loadAtlasEx (Suggestion: loadAtlas)
source/objects/Character.hx:304: characters 87-101 : flxanimate.animate.FlxAnim has no field lastPlayedAnim
source/objects/Character.hx:335: characters 20-26 : flxanimate.animate.FlxAnim has no field resume
PS C:\FNF-PsychEngine-main>
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
before i got this message i got the "source/psychula/HScript: characters 89-96 :Type not found: TeaCall so i changed the TeaCall to Tea instead in notepad but now this message pops op and i dont know what to do, please help

ik this is old and you might've already figured it out but i'll still comment
you should have a folder called "setup" and there'll be a file called "setup-windows.bat"
open it and then it should try to install the stuff it needs, which should remove most of the errors
for source/backend/Paths.hx:605: characters 7-18 : flxanimate.FlxAnimate has no field loadAtlasEx (Suggestion: loadAtlas), you should change it to the suggestion, save, and then change it back which should remove that error
still haven't figured out the first error yet tho


Incremental sync not working at all

I've read through the documentation found here
https://docs.microsoft.com/en-us/azure/app-service-mobile/app-service-mobile-offline-data-sync#how-sync-works
Something tells me I'm doing something wrong though.
My pull async looks like
await Results.PullAsync ("Results", Results.CreateQuery());
The records in the results table have a "meaningful" value. As far as I know that means a not null value of DateTimeOffset.UtcNow
It seems that none of the records are synced when I try to use incremental sync. If I pass in null
as the queryId then I get the records to come down from the server. Is there something I'm missing or is this a bug?

We need more information - offline sync is verified working in our tests, so it's likely something in your code.  What does your model look like?  What about the initializer?  Have you changed the model in the interim?  More code is required.

No response, so closing.

Sorry I we've been busy and implementing incremental sync hasn't been a priority, but I'd still like to get this working at some point. Is there an example you can point me to that has this working so I can see if my code differs in a significant way?

Plenty of examples in the book - http://aka.ms/zumobook

I am facing the same issue in iOS
[self.syncTable pullWithQuery:query queryId:nil completion:^(NSError *error) {
}];

This function can get all the records from the server. But when i used the queryId like
[self.syncTable pullWithQuery:query queryId:@"AllRecords" completion:^(NSError *error) {
}];

Its getting partial data from the server. Any help here?

Did anyone manage to resolve the issue?


Allow exporting in HTML

Currently, carbon-app provides export in PNG, SVG, and other formats. It would be great if carbon-app provides exporting the written code in HTML so that users can it in their blog apps.

where is description  ?

@ariannargesi description added. Thanks for letting me know 👍

@ilyaskarim would embedding work for your use case? The reason we support embedded iframes is for exactly what you are describing: including the snippet in your own blog
You can copy the Iframe under the copy menu, or else simply go to carbon.now.sh/embed?<your query string here>

@mfix22 Hey thanks for the comment. I was able to embed https://jsfiddle.net/8kubocLg/7/. One thing. I cannot copy the content inside. Do you know? Because I want my readers to copy the content as well. Is this a known bug? If the copy is not possible then definitely export HTML will be useful for me. For others as well.

Hey @ilyaskarim I haven't noticed that issue before! One thing you can do is remove the wc=false from your query string to enable the copy button, which will let users super easily copy the code.
I will have to investigate further to enable direct copying — potentially this is a Chrome bug

@ilyaskarim opened up a new issue to track this bug (https://github.com/carbon-app/carbon/issues/972). If we fixed that bug, would iframes work for your use case?

@mfix22 That did the job. Thank you. That will work for me(https://jsfiddle.net/L6xantj0/). But one thing. I am giving you the option to decide. Will you keep this as feature-request? I am pretty sure that users will love this new feature. What do you think?

Sharing jsFiddle so that it will help other readers.

No problem if you can't.

@ilyaskarim thank you so much for posting this! This issue actually uncovered 2 bugs in the embedded Carbon editor — both of which have been fixed.
Copying the text directly works again 👍
To answer your question, we do not want to enable exporting the HTML directly, because then we are unable to fix issue on sites that have embedded the carbon snippet as HTML, which we can do as an iFrame.

@all-contributors add @ilyaskarim for bug


Bug in data flow inference sequence

Pattern PDOP-S+fS creates a self-referential 'fromSource' link at an Output Pool (a Data Access subclass representing data produced by a process that has no other data source).
This is incorrect - an Output Pool (being a process output) would already be marked as a data source by that point. The pattern should have acted on an Input Pool with no other source.

Turns out the problem isn't that Output Pools don't need to be tagged at this point. The problem is that the tag should be added a little earlier, and Input Pools should have the same treatment.
Now fixed in branch 85, although there are still some unresolved corner cases arising from #40.


can you please attach pictures to understand how it looks ?

[can you please attach pictures to understand how it looks ? ]

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

ID: 237cc665-0041-6374-bd81-56dcad94926f
Version Independent ID: cfddf72f-0ee5-df62-3c29-3e8af4e6b26c
Content: addNotification (Client API reference) in model-driven apps - Power Apps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/controls/addNotification.md
Service: powerapps
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk

@Kaviarasi This is how it looks on model-driven app forms when you try the sample code provided in the docs. We have also updated the docs with the image. Closing the issue. If you need further help, please reopen or create a new issue.


[Feature] Satellite mode

Support satellite mode, so enable config for:

music directory
database plugin: simple by default, proxy available, host/port to be provided

Available in release 2022-11-30.2


how to add NodeJS to the project? we use nodejs for backend. Someone can help?

You could add the proxy parameter to devServer in webpack.dev.js:
        devServer: {
            proxy: {
                '/api/*': 'http://<YOUR_BACKEND_HOST>:<YOUR_BACKEND_PORT>',
            },
            port: METADATA.port,
            host: METADATA.host,
            historyApiFallback: true,
            watchOptions: {
                aggregateTimeout: 300,
                poll: 1000
            },
            outputPath: helpers.root('dist')
        },

This way, all requests starting with '/api/' will be redirected to your backend server.

@vesteraas, thanks.
I fixed it with your method.

Hi @vesteraas ,
i need your further help....
with the config as below, angular2.0.0 project works with node as backend, but it failed in angular1.5 project, may i know where it was wrong?
....
 devServer: {
        port: 5000,
        host: 'localhost',
        historyApiFallback: true,
        watchOptions: {
            aggregateTimeout: 300,
            poll: 1000
        },
        outputPath: '/',
        proxy: {
            '/api/*': 'http://localhost:8000'
        }
    },
....

@abbymrs I guess your question is off-topic here (as well as this answer) but have a look at http://stackoverflow.com/questions/35538472/webpack-dev-server-with-backend-api

thanks, i rechecked again, it was my mistake that forgot to add '/api/' in my front end codes when make requests from backend.
Thank you so much!


memcached SCAN always fail

Hello world!
I am now running YCSB to benchmark memcached, and I have tested all the workloads (A to F) so far.
I noticed that workloade, which performs SCAN operations, will fail, mostly because all SCAN operations fail. As a result throughput is meaningless, and I can not assess this operation latency on my architecture.
Did anyone experience this before?
Thank you in advance.

@cimballihw - The Memcached binding doesn't implement SCAN. Check the source code here https://github.com/brianfrankcooper/YCSB/blob/master/memcached/src/main/java/com/yahoo/ycsb/db/MemcachedClient.java#L189-L194 You are probably getting not implemented instead of actual failures. Memcached and SCAN doesn't really make sense in most cases. Memcached is basically a cache for key/value lookups.

@cimballihw - did the above answer your question? Thanks!

@risdenk - yes, indeed I had a look at the source code and scan is not implemented for Memcached. However, because why all other key-value databases would use SCAN, since the search using key list is by far more efficient? is it because of memcached partitioning in slabs, which makes it slower than others?


Also check for Java libs in new Java 9 directory structure

Prior to Java 9, the macOS version of Java placed the libjvm.dylib file at $JAVA_HOME/jre/lib/server. As of Java 9, it now lives at $JAVA_HOME/lib/server. This breaks the JPL build under Java 9, as it's looking for the libjvm.dylib file in the wrong place. (For details on why they moved it, see JEP 220.)
This may also move it on other OSes, as $JAVA_HOME/lib/server should now be the "standard" location for the libjvm library on macOS, Solaris, and Linux. However I have not checked that this is the case.

Thanks!


Update onCrossChainCall with zContext

### Tasks
- [ ] https://github.com/zeta-chain/example-contracts/pull/43
- [ ] https://github.com/zeta-chain/docs/pull/106
- [ ] https://github.com/zeta-chain/toolkit/pull/28
- [ ] https://github.com/zeta-chain/toolkit/pull/32
- [ ] Release an updated version of `@zetachain/toolkit`
- [ ] https://github.com/zeta-chain/template/issues/22

Relevant: https://github.com/zeta-chain/node/pull/912

@lucas-janon @andresaiello I might need your help reviewing PRs above.


[Feature Request] yaml configs for Index State Management

Is your feature request related to a problem? Please describe
Now almost all settings can be managed in 3 ways - yaml configs, API, opensearch-dashboard. But ISM policy can be managed in only 2 ways - API, opensearch-dashboard. It would also be convenient to be able to manage via yaml configs and ISM policy.
Describe the solution you'd like
yaml configs for ISM policy
Related component
Plugins
Describe alternatives you've considered
No response
Additional context
No response

[Triage - attendees 1 2 3 4 5 6 7 8]
@opensearch-project/admin Please transfer this to the index management repository


S3: PUT request should fail when no 'Content-Length' header is present, but doesn't

I've discovered an inconsistency between the AWS behaviour and the moto one when trying to upload an empty file to S3 with a PUT request: as stated in http://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectPUT.html#RESTObjectPUT-requests, the Content-Length header is required in such request, and should fail with a 411 Length Required response if not provided. This is happening if I contact the "real" AWS API using the requests Python library (which, for some reason, is not setting this header when the file is empty), but the moto mocked one just returns 200 OK.
My version of moto is 0.4.27.

@CianciuStyles I've addressed this in #924 - that branch should fix your problem!

Yes, I think your changes will fix the problem. Thanks @JackDanger!


[Bug]: Messages don't show up ingame

What happened?
It just says "[DISCORD] <username>", the message the user sent doesn't show up.
Game Version
Other
What platform are you using?
Forge
Platform version
1.18.2
Config File
# General options for the bot
[general]
  # Insert your Bot Token here!
  # DO NOT SHARE IT WITH ANYONE!
  botToken = "MTAzNTExNzk5NzkwMTIyMTk1Mg.GolnXY.YrYje1FIMUuXZ0uEX9-W6HTqxRS7yrG7gJO0Xo"
  # The channel ID where the bot will be working in
  botChannel = "1035119318456860682"
  # The bot's status message
  #
  # PLACEHOLDERS:
  # %online% - Online Players
  # %max% - Maximum Player Amount
  botStatusName = "%online% players Online"
  # Type of the bot's status
  # Allowed Values: DISABLED,PLAYING,WATCHING,LISTENING,STREAMING,COMPETING
  botStatusType = "PLAYING"
  # URL of the bot's stream when using the status type 'STREAMING'
  # Has to start with https://twitch.tv/ or https://www.youtube.com/watch?v=
  streamingURL = "https://www.youtube.com/watch?v=dQw4w9WgXcQ"
  # Enable checking for updates?
  # Notification will be shown after every server start in log when update is available
  enableUpdateChecker = true
  # The minimum release type for the update checker to notify
  # Allowed values: release, beta, alpha
  updateCheckerMinimumReleaseType = "beta"
  # Set to false to disable config migration from other mods/plugins to this one
  # This does not prevent updating of this config after mod updates
  allowConfigMigration = true
  # Attempt to parse id-based mentions to names in in-game chat
  parseMentionsIngame = true
# Configuration options for commands
[commands]
  # The Role IDs of your Admin Roles
  # Now supports multiple roles which can access admin commands
adminRoleIDs = [  "1035119581569757225"  ]
  # Add your custom commands here
  # You can also generate some on https://erdbeerbaerlp.de/dcintegration-commands/
  # You must op this UUID in the ops.txt or some custom commands won't work!
  senderUUID = "8d8982a5-8cf9-4604-8feb-3dd5ee1f83a3"
  # Enable help command?
  # Disabling also removes response when you entered an invalid command
  # Requires server restart
  helpCmdEnabled = true
  # Enable the list command in discord
  # Requires server restart
  listCmdEnabled = true
  # Enable the uptime command in discord
  # Requires server restart
  uptimeCmdEnabled = true
  # Set to false to completely disable the "Unknown Command" message
  showUnknownCommandMessage = true
  # Set to true to enable the "Unknown Command" message in all channels
  showUnknownCommandEverywhere = false

    [[commands.customCommands]]
      name = "kick"
      description = "Kicks an player from the Server"
      mcCommand = "kick %player% %reason%"
      adminOnly = true
      hidden = false

        [[commands.customCommands.args]]
          name = "player"
          description = "The player to be kicked"
          optional = false

        [[commands.customCommands.args]]
          name = "reason"
          description = "Reason for the kick"
          optional = true

    [[commands.customCommands]]
      name = "stop"
      description = "Stops the server"
      mcCommand = "stop"
      adminOnly = true
      hidden = false
args = [    ]

    [[commands.customCommands]]
      name = "kill"
      description = "Kills an Player or Entity"
      mcCommand = "kill %target%"
      adminOnly = true
      hidden = false

        [[commands.customCommands.args]]
          name = "target"
          description = "The target(s) for the kill command."
          optional = false
# Toggle some message related features
[messages]
  # Enable formatting conversion (Markdown <==> Minecraft)
  convertCodes = true
  # Send formatting codes from mc chat to discord
  # Has no effect when markdown <==> Minecraft is enabled
  formattingCodesToDiscord = false
  # Should /say output be sent to discord?
  sendOnSayCommand = true
  # Should /me output be sent to discord?
  sendOnMeCommand = true
  # When an /say command's message starts with this prefix, it will not be sent to discord
  sayCommandIgnoredPrefix = "§4§6§k§r"
  # Should tamed entity death be visible in discord?
  sendDeathMessagesForTamedAnimals = false
# Advanced options
[advanced]
  # Custom channel ID for server specific messages (like Join/leave)
  # Leave 'default' to use default channel
  serverChannelID = "default"
  # Custom channel ID for death messages
  # Leave 'default' to use default channel
  deathsChannelID = "default"
  # Custom channel for for ingame messages
  # Leave 'default' to use default channel
  chatOutputChannelID = "default"
  # Custom channel where messages get sent to minecraft
  # Leave 'default' to use default channel
  chatInputChannelID = "default"
  # Allows you to change the target URL for the API to make it usable with custom discord instances like fosscord
  # DO NOT CHANGE if you don't know what you are doing!!
  baseAPIUrl = "https://discord.com"
# Config options which only have an effect when using forge
[forgeSpecific]
  # A list of blacklisted modids
  # Adding one will prevent the mod to send messages to discord using forges IMC system
IMC_modIdBlacklist = [  "examplemod"  ]
  # Show item information, which is visible on hover ingame, as embed in discord?
  sendItemInfo = true
# Configuration for linking
[linking]
  # Unlink players when they leave the discord server for whatever reason (ex. leave,kick,ban)?
  unlinkOnLeave = true
  # Should discord linking be enabled?
  # If whitelist is on, this can NOT be disabled
  # DOES NOT WORK IN OFFLINE MODE!
  enableLinking = true
  # Role ID of an role an player should get when he links his discord account
  # Leave as 0 to disable
  linkedRoleID = "0"
  # Enable discord based whitelist?
  # This will override the link config!
  # To whitelist use the whitelist command in the bot DMs
  whitelistMode = false
  # Adding Role IDs here will require the players to have at least ONE of these roles to link account
requiredRoles = [    ]
settingsBlacklist = [    ]

    # Allows you to configure the default values of some personal settings
    [linking.personalSettingsDefaults]
      default_useDiscordNameInChannel = true
      default_ignoreReactions = false
      default_pingSound = true
# Webhook configuration
[webhook]
  # Whether or not the bot should use a webhook (it will create one)
  enable = false
  # The avatar to be used for server messages
  serverAvatarURL = "https://raw.githubusercontent.com/ErdbeerbaerLP/Discord-Chat-Integration/master/images/srv.png"
  # The name to be used for server messages
  serverName = "Minecraft Server"
  # The URL where the player avatar gets fetched from
  #
  # PLACEHOLDERS:
  # %uuid% - Returns the player's UUID with dashes
  # %uuid_dashless% - Returns the player's UUID without dashes
  # %name% - Returns the player's name
  # %randomUUID% - Returns an random UUID which can be used to prevent discord cache
  playerAvatarURL = "https://minotar.net/avatar/%uuid%?randomuuid=%randomUUID%"
  webhookName = "MC_DISCORD_INTEGRATION"
# Configuration for the in-game command '/discord'
[ingameCommand]
  # Enable the /discord command to show an custom message with invite URL?
  enabled = true
  # The message displayed when typing /discord in the server chat
  message = "Join our discord! http://discord.gg/myserver"
  # The message shown when hovering the /discord command message
  hoverMessage = "Click to open the invite url"
  # The url to open when clicking the /discord command text
  inviteURL = "http://discord.gg/myserver"
# The command log channel is an channel where every command execution gets logged
[commandLog]
  # Channel ID for the command log channel
  # Leave 0 to disable
  channelID = "0"
  # The format of the log messages
  #
  # PLACEHOLDERS:
  # %sender% - The name of the Command Source
  # %cmd% - executed command (e.g. "say Hello World"
  # %cmd-no-args% - Command without arguments (e.g. "say"
  message = "%sender% executed command `%cmd%`"
  # A list of commands that should NOT be logged
ignoredCommands = [  "list", "help", "?"  ]
# Configure votifier integration here
# (Spigot only)
[votifier]
  # Should votifier messages be sent to discord?
  enabled = true
  # Custom channel ID for Votifier messages
  # Leave 'default' to use default channel
  votifierChannelID = "default"
  # The message format of the votifier message
  #
  # PLACEHOLDERS:
  # %player% - The player´s name
  # %site% - The name of the vote site
  # %addr% - (IP) Address of the site
  message = ":ballot_box: %player% just voted on %site%"
  # Name of the webhook title
  name = "Votifier"
  # URL of the webhook avatar image
  avatarURL = "https://www.cubecraft.net/attachments/bkjvmqn-png.126824/"
# Configure Dynmap integration here
[dynmap]
  # The message format of the message forwarded to discord
  #
  # PLACEHOLDERS:
  # %sender% - The sender´s name
  # %msg% - The Message
  dcMessage = "<%sender%> %msg%"
  # Custom channel ID for dynmap chat
  # Leave 'default' to use default channel
  dynmapChannelID = "default"
  # Name of the webhook title
  name = "Dynmap Web-Chat"
  # URL of the webhook avatar image
  avatarURL = "https://static.wikia.nocookie.net/minecraft_gamepedia/images/9/91/Map_Zoom_0.png/revision/latest?cb=20200311153330"
  # The name format of the message forwarded to the dynmap webchat
  #
  # PLACEHOLDERS:
  # %name% - The discord name of the sender (including nickname)
  # %name#tag% - The discord name with tag of the sender (without nickname)
  webName = "%name% (discord)"
  # Name shown in discord when no name was specified on the website
  unnamed = "Unnamed"
# Configure some plugin-specific BStats settings here
# Everything can be seen here: https://bstats.org/plugin/bukkit/DiscordIntegration/9765
#
# Does not apply to fabric yet, as there is no bstats for it
[bstats]
  # Allow sending of installed addon stats (Name and version of installed addons)
  sendAddonStats = true
# Settings for servers running as Bungeecord-suberver
[bungee]
  # Set this to true if the server is running as an subserver of an bungeecord network and therefore needs to be in offline mode
  # Setting this will force account linking in offline mode
  # Do NOT use for actual offline mode servers, as this will break the linking feature because of the UUIDs!
  #
  # Currently no support for floodgate running on bungee
  isBehindBungee = false

Crash report / Error log (if applicable)
No response
Additional Information
No response

Emojiful Mod issue

Emojiful Mod issue
I dont have that mod tho

Also reset your token
Can't help here without log tho


Using Langchain::Assistant with MistralAI throws an error when tool output is returned

Describe the bug
I believe when the role is equal to "tool" the Langchain::Assistant::Messages::MistralAIMessage#to_hash() is generating incorrect hash.
To Reproduce
llm = Langchain::LLM::MistralAI.new(api_key: ENV["MISTRAL_AI_API_KEY"])

assistant = Langchain::Assistant.new(
  llm: llm,
  instructions: "You're a helpful AI assistant",
  tools: [Langchain::Tool::NewsRetriever.new(api_key: ENV["NEWS_API_KEY"])]
)

# Add a user message and run the assistant
assistant.add_message_and_run!(content: "What's the latest news about AI?")

Actual behavior
Error is thrown:
/Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/faraday-2.12.0/lib/faraday/response/raise_error.rb:30:in `on_complete': the server responded with status 422 (Faraday::UnprocessableEntityError)
	from /Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/faraday-2.12.0/lib/faraday/middleware.rb:57:in `block in call'
	from /Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/faraday-2.12.0/lib/faraday/response.rb:42:in `on_complete'
	from /Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/faraday-2.12.0/lib/faraday/middleware.rb:56:in `call'
	from /Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/faraday-2.12.0/lib/faraday/rack_builder.rb:152:in `build_response'
	from /Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/faraday-2.12.0/lib/faraday/connection.rb:452:in `run_request'
	from /Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/faraday-2.12.0/lib/faraday/connection.rb:280:in `post'
	from /Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/mistral-ai-1.2.0/controllers/client.rb:75:in `request'
	from /Users/andrei/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/mistral-ai-1.2.0/controllers/client.rb:48:in `chat_completions'
	from /Users/andrei/Code/langchain/lib/langchain/llm/mistral_ai.rb:40:in `chat'
	from /Users/andrei/Code/langchain/lib/langchain/assistant.rb:347:in `chat_with_llm'
	from /Users/andrei/Code/langchain/lib/langchain/assistant.rb:302:in `handle_user_or_tool_message'
	from /Users/andrei/Code/langchain/lib/langchain/assistant.rb:269:in `process_latest_message'
	from /Users/andrei/Code/langchain/lib/langchain/assistant.rb:251:in `handle_state'
	from /Users/andrei/Code/langchain/lib/langchain/assistant.rb:142:in `run'
	from /Users/andrei/Code/langchain/lib/langchain/assistant.rb:161:in `add_message_and_run'
	from /Users/andrei/Code/langchain/lib/langchain/assistant.rb:169:in `add_message_and_run!'
	... 3 levels...

Expected behavior
An error is not thrown.
Note
From the Mistral AI docs:

I believe we're sending content: [...] but it expects a string back.

I think it would be great to also split to_hash methods into:
def to_user_hash
end

def to_system_hash
end

def to_assistant_hash
end

def to_tool_hash
end


add a receiver to mute alert

https://issues.redhat.com/browse/AAP-4478
Signed-off-by: Song Song Li<EMAIL_ADDRESS>

/hold

/unhold


add dark mode

Should respect system/browser settings, i.e. detect automatically.
Perhaps this needs coloring the share button and possibly the favicon correct, both of which currently use opacity.

@mo271 Can you assign this issue to me??

Shall i add a button which changes the page to its dark mode ?

No, I don't think we want to make this a button, but rather pick up the settings of the browser, like the page github.com does for instance.

fixed by #5


Rework Error Messages

Description
The error messages need to be reworked
Acceptance criteria

Tasks

[ ]

Is this still the case?

I'm sorry this issue is older then I'm part of the team. The description isn't giving much information and I could not match if with something I'm aware of. Sorry. :speak_no_evil:

@weilbith Don't worry :) I have been asking you because I have remembered, that you were working on CLI/API errors.
Let's wait for @andrevmatos :)

Yes, but I see a big difference there. The CLI was meant to be as dump as possible, acting as a flat gate to the SDK. Therefore we decided that the SDK should do more sensitive validation of the users input parameters. The most smart thing the CLI does then is to categories the different errors to assign them to a certain HTTPS status code. So after all we talk about machine-to-machine communication here. The clients should make sure they use correct (formatted) inputs.
On the dApp side we should not rely on the SDKs parameter validation and forward its error messages. It would be a horrible user experience, since it would be slow, not precise enough and would be more a try-and-error approach rather then a interactively supportive approach.
But after all I recognize that I discussesd about parameter validation in first place. Of course this a way wider topic. So waiting for Andre :grimacing:

I don't have anything strong to say: @weilbith already gave a good explanation about it. Error messages can ALWAYS be reworked to be more explicit, informative and whatnot, but I'm happy with the current state of it. It'd be more UX, if something isn't good enough for dApp, than actual errors, as we're already quite covered on this matter. Afterall errors aren't something we expect often if things are happy, if they're not, for the logic, we mostly only care if there's an error, the message itself only matters for the users.

Example:

Closing, we create separate issues for theses kind of error messages when they appear.


Make Vega work offline in REPL

Hi,
I've downloaded the asset files required by vega, put them in an assets folder, from where it is read to output as HTML in the render.jl file for the REPL output. This let's Vega work offline from the REPL.
I tried the same approach in IJulia, but the Javascript emitted was causing issues. Will look into how to resolve that!
Regards,
Rohit

Thanks for starting this @rohitvarkey. The files you added already exist in the deps folder, so there isn't a need to add them again.
The only thing I would say is that if we're going to implement this now, we should modify the filename structure to include the version, so that it's easy to see when versions get out of date. Vega updates pretty much once a week, so if we move to providing the JS code with Vega.jl, we'll need to commit to providing updates as well.
Related: #31

Randy I would personally default to using the internet to always use a
latest version, but have this as a backup if there's no internet, or if the
versions match.  Then there's not as much pressure to maintain a weekly
release schedule.
On Mon, Oct 12, 2015 at 9:19 AM, Randy Zwitch<EMAIL_ADDRESS>wrote:

Thanks for starting this @rohitvarkey https://github.com/rohitvarkey.
The files you added already exist in the deps folder, so there isn't a
need to add them again.
The only thing I would say is that if we're going to implement this now,
we should modify the filename structure to include the version, so that
it's easy to see when versions get out of date. Vega updates pretty much
once a week, so if we move to providing the JS code with Vega.jl, we'll
need to commit to providing updates as well.
Related: #31 https://github.com/johnmyleswhite/Vega.jl/issues/31
—
Reply to this email directly or view it on GitHub
https://github.com/johnmyleswhite/Vega.jl/pull/55#issuecomment-147397317
.

Maybe the design should just be to have a non-exported function called Vega.use_offline(;flag::Bool = false) and just wrap the JS calls in that. I don't want to add another package dependency to find out whether there is an Internet connection.

Oh. I put the files in assets because Escher expects the files to be there. I've been working on getting Vega and Escher to play well and I'm really close to a solution.
To solve the dependency issues, we could use bower and then just run bower update to automatically update the asset files.

If you make Vega work with Escher, you can put the files anywhere you want ;)

On Oct 12, 2015, at 3:17 PM, Rohit Varkey Thankachan<EMAIL_ADDRESS>wrote:
Oh. I put the files in assets because Escher expects the files to be there. I've been working on getting Vega and Escher to play well and I'm really close to a solution.
To solve the dependency issues, we could use bower and then just run bower update to automatically update the asset files.
—
Reply to this email directly or view it on GitHub.

Haha. I think I've got it to work!

So is this PR still needed given #56? Meaning, are they co-mingled?

This works using my not-yet-implemented battery of tests. In Jupyter Notebook, there are no changes to behavior, in the REPL it generates pages with all of the JS code in them.
It's amazing how fast everything generates, as the REPL now generates 20k line HTML pages (due to readall for the JS files)

Shoot. I forgot to close this. I'll just remove the asset files I added here and update paths to the bower components introduced in #56 to remove duplicate asset files.


Fails to parse october 20 2017 with languages set to [sv, en]

Failing to parse some dates with languages set to ['sv'. 'en'] and date_formats=['%B%d%Y]
These dates works fine:
"oktober202017"
"october20th2017"
However dateparser.parse returns None for
"october202017"

>>> dateparser.__version__
'0.7.2'
>>> dateparser.parse("october202017", languages=['sv', 'en'], date_formats=["%B%d%Y"])
datetime.datetime(2017, 10, 20, 0, 0)


fix validation errors and enable additional checks in CI

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:
Include additional ci checks and fix reconcile issues with device plugins.

Related to: https://github.com/harvester/harvester/issues/3524
Solution:

Related Issue:
Test plan:

To test device plugin fix can only be tested with multiple devices of the same type.
Easiest option would be to create VF's on a nic or use the SriovNetworkDevices crd.

create 4 VF's
enable pcidevice passthrough for all 4 VF's.
check node status kubectl get nodes -o yaml | yq .status.allocatable, user should see 4 devices available for the VF's just created.
delete pcideviceclaim pod or reboot node.
check node status kubectl get nodes -o yaml | yq .status.allocatable, without this fix there is a high chance of only seeing 3 devices available for the VF's created from step above.
build a new image with fixes, and update pcidevices-controller
once controller initialises kubectl get nodes -o yaml | yq .status.allocatable should reflect the right number of VF's.

https://github.com/ibrokethecloud/pcidevices/blob/b6653138864acb5fe4842f2d0864eb71d6c0da2e/pkg/controller/pcidevice/pcidevice_controller.go#L69
maybe not directly related to this PR, but it logs with wrong information?

good spot @WebberHuang1118. fixed it.


Uncaught TypeError: Cannot read property 'func' of undefined

react-router3.x,react16.0.0

Not enough info provided.


Be more precise with unscope-ing

Hi @fabn, I was wondering if you could travel back in time and give a bit more detail as to your intention with this commit?:
https://github.com/swanandp/acts_as_list/commit/4a45dca9bf9059ce5491d2bb576e7f7eed9cd82c
I've been having deadlock issues around the :order scope being cleared in aal updates (e.g. shuffling the table) vs other updates where the default_scope is kept intact.
What do you think about just calling: .unscope(:where) instead? Are there other scope types we'd need to unscope? I don't think there is as we really only want to make sure that our .where clauses are the only ones in there.
What do you think @swanandp?

Hi @brendon, I think you are right, .unscope(:where) should be enough, but it will mean to drop compatibility with rails 3 since that method is only available with rails >= 4.0.
Maybe except could help to preserve rails 3 compatibility.

I will get to this shortly.

I've had good success with this PR. Let's see how the CI runs for all scenarios. I had to .reorder instead of .order but again, that's less of a blunt instrument.


images

Vault Sealed Status upon docker-compose up

Consul Dashboard after Vault is unsealed

After Storing Secrets using Vault CLI:


Finding type info for functions can cause huge delays in displaying inline documentation if type hints aren't given

Environment data

Language Server version: v2021.9.4
OS and version: Windows 10 21H1
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.10 (WSL Ubuntu)

Expected behaviour
While type info is loaded for a function, inline documentation (docstrings) are displayed without the type annotations. If/when type info is determined, the hint is updated to include the information.
Actual behaviour
If type info isn't provided for a function, all inline documentation can take minutes to load as PyLance tries to determine return types for the function.

Error 2021-10-01 16:29:28: Failed to check if file needs to be fixed [EntryNotFound (FileSystemError): Unable to read file 'vscode-remote://wsl+ubuntu/home/migue/.config/Code/User/settings.json' (Error: Unable to resolve non-existing file 'vscode-remote://wsl+ubuntu/home/migue/.config/Code/User/settings.json')
	at _handleError (/home/migue/.vscode-server/bin/7f6ab5485bbc008386c4386d08766667e155244e/out/vs/server/remoteExtensionHostProcess.js:94:160087)
	at processTicksAndRejections (internal/process/task_queues.js:93:5)
	at async y.readText (/home/migue/.vscode-server/extensions/ms-python.python-2021.9.1246542782/out/client/extension.js:9:314351)
	at async p.doesFileNeedToBeFixed (/home/migue/.vscode-server/extensions/ms-python.python-2021.9.1246542782/out/client/extension.js:59:823969)
	at async /home/migue/.vscode-server/extensions/ms-python.python-2021.9.1246542782/out/client/extension.js:59:823096
	at async Promise.all (index 0)
	at async p.getFilesToBeFixed (/home/migue/.vscode-server/extensions/ms-python.python-2021.9.1246542782/out/client/extension.js:59:823042)
	at async p.updateTestSettings (/home/migue/.vscode-server/extensions/ms-python.python-2021.9.1246542782/out/client/extension.js:59:822669)] {
  code: 'FileNotFound'
}
> conda --version

Code Snippet / Additional information
Steps to reproduce:

Install SymPy (a complex mathematical toolkit that doesn't currently have type annotations
In a new file, import sympy, and make a call to the function sympy.solve()
Mouse over the function. Observe that loading the documentation for the function takes minutes, as PyLance parses nearly the entirety of SymPy's codebase to try to find the return types.

It seems that I'm not the first to notice this issue: #946
I hope this performance issue can be fixed soon!

I would normally close this as a dupe of #946, but you're suggesting the concrete change of only showing the type information once it's available, and to show things like docstrings if we have them faster.
The general problem is that there's no way to do this in VS Code or the LSP; it asks for a hover, then waits for the full result, and then that's it. Pylance will cache the info so it's fast the next time, but otherwise, there's no way to update it once we send it back. We could send a limited response and then have a more complete one later, but it would only appear if you exit the hover and come back again.
sympy is a really complicated library, so it tends to be slow the first time you interact with it; stubs help.

I'd say that although it'd be possible to get stubs for libraries, it'd effectively be a game of cat and mouse trying to find or create stubs for any library that happens to have the same issue. I know that I've had issues for some of my own personal projects where certain situations have caused similar lag, so it's not like this issue is isolated to SymPy (even if in my case it was due to bad design :P ).
I think a good compromise is to set a time-out (eg 1 second) for the types to load and then just return the information without type hints if it isn't loaded at that point, and then update it on later occasions if necessary, although it could be an idea to actually add the ability to update the hints after they've displayed in VS Code.

I've checked and Eric's change for #1890 also significantly improves sympy performance. In this case, typing sympy.solve froze for minutes (as you noted), but after, it's practically instant, at the cost of an imperfect return type.
I'm going to mark this one as fixed, but going further to improve the type will definitely take a stub or upstream to annotate their code.

This issue has been fixed in version 2021.10.0, which we've just released. You can find the changelog here: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#2021100-6-october-2021


Support calling Rem with empty request parameters

If you write a REM which uses the RemService to access RESMI outside the context of an HTTP request what RequestParameters should you pass?
This change allows to invoke the REM with an empty parameters without causing a NPE.

Interesting :dart:
Thanks Alex! :smiley:


FIX: update prometheus agent storage template to allow empty value

FIX:
When using the default values which include storageSpec: {},
it set the storage as {} inside the final yaml file, which is causing the following error:
PersistentVolumeClaim "prometheus-prometheus-agent-tst-db-prometheus-prometheus-agent-tst-0" is invalid: spec.resources[storage]: Required value

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


Release google-cloud-java v0.82.0

This pull request was generated using releasetool.
03-04-2019 10:03 PST
AutoML

Regenerate automl client (#4563)

Asset

Regenerate asset client (#4562)

Bigtable

Regenerate bigtable client (#4564)
Bigtable: integrate OpenCensus tracing into the bigtable data client (#4493)

BigQuery

Fix useAvroLogicalTypes option in LoadJobConfiguration And WriteChannelConfiguration. (#4615)
Fixes When Table,Dataset and Job is not found  (#4389)

BigQuery Data Transfer

Regenerate bigquerydatatransfer client (#4600)

BigQuery Storage

Regenerate bigquerystorage client (#4601)
Regenerate bigquerystorage client (#4586)
bigquerystorage: address doc nits (#4544)

Compute

Regenerate compute client (#4587)

Container Analysis

Regenerate containeranalysis client (#4565)

Dataproc

Regenerate dataproc client (#4602)

Datastore

Regenerate datastore client (#4566)

Dialogflow

Regenerate dialogflow client (#4567)

Data Loss Prevention

Regenerate dlp client (#4603)

Error Reporting

Regenerate errorreporting client (#4568)

Firestore

Regenerate firestore client (#4604)

IAM

Regenerate iamcredentials client (#4605)

IoT

Regenerate iot client (#4569)

KMS

Prepare for KMS GA release - upgrade versions to 1.0.0. (#4581)
Regenerate kms client (#4570)

Language

Regenerate language client (#4606)

Logging

Regenerate logging client (#4571)

Monitoring

Regenerate monitoring client (#4572)

OS-Login

Regenerate os-login client (#4607)

Pub/Sub

Clean up MessageDispatcher by changing processOutstandingBatches to explicitly loop instead of while(true) with breaks.  There is now only 1 explicit return and 1 runtime error. (#4619)
Extract the single message processing functionality from processOutstandingBatches. (#4618)
Regenerate pubsub client (#4608)
Scale the system executor provider with the number of pull channels opened. (#4592)
Change MessageDispatcher to own its own queue. (#4590)
Remove package private dead code from the subscriber client library.  None of this code is ever called, and none of it is usable outside of the library. (#4589)
Regenerate pubsub client (#4588)

Redis

Regenerate redis client (#4573)

Scheduler

Regenerate scheduler client (#4609)

Security Center

Regenerate securitycenter client (#4610)

Spanner

add propagateTimeout in SpannerExceptionFactory (#4598)
Regenerate spanner client (#4574)

Speech

Regenerate speech client (#4575)

Tasks

Regenerate tasks client (#4611)
Regenerate tasks client (#4550)

Text-to-Speech

Regenerate texttospeech client (#4576)

Trace

Regenerate trace client (#4577)

Video Intelligence

Regenerate video-intelligence client (#4612)

Vision

Adding vision v1p4beta1 (#4584)
Regenerate vision client (#4613)

Web Security Scanner

Regenerate websecurityscanner client (#4614)

Dependencies

Upgrading grpc in google-api-grpc (#4593)
Using the latest version of truth (#4597)
Using the google-cloud-http-bom (#4594)

Internal / Testing Changes

[Storage] Set the isDirectory flag appropriately (#4616)
Update Issue Templates to match other google-cloud repositories (#4591)

Release is at https://github.com/googleapis/google-cloud-java/releases/tag/v0.82.0


Create sample config for summary and definition updates

Basically it's a loop since the config will live in puppet, but let's have our local copy too for review and testing.
cc @wikimedia/services

Coverage remained the same at 95.259% when pulling d4654771cd4eb6a4d363c78ade1d7ac531543f24 on Pchelolo:summary into fcbcc4f5ff1c68b91d6e5e506bbd7708419aa3aa on wikimedia:master.

Coverage remained the same at 95.259% when pulling b30562da035c6fa7062f25a28f6e4aabd30a4a53 on Pchelolo:summary into fcbcc4f5ff1c68b91d6e5e506bbd7708419aa3aa on wikimedia:master.


Skip constructing Time for transactions

Before contributing, please read our contributing guidelines and code of conduct.
Overview
The time parameter was never used, but each call constructed the current time
to fill out the default value.

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

I see the same infinite_tracing tests failing on dev branch, so they don't seem specific to this PR.

@viraptor -- thank you for this PR -- Looks like google-protobuf released 3.15.0 with a change that breaks the infinite tracing tests.  We'll open a separate ticket for that.
Meanwhile, very nice catch on finding this unused parameter.  I would be in complete favor of removing the parameter altogether and removing it where given in the test suite.
lib/new_relic/agent/sql_sampler.rb:
  67        # This is called when we are done with the transaction.
  68:       def on_finishing_transaction(state, name, time=Time.now)
  69          return unless enabled?

test/new_relic/agent/pipe_channel_manager_test.rb:
  223        sampler.notice_sql("SELECT * FROM table", "ActiveRecord/Widgets/find", nil, 100, state)
  224:       sampler.on_finishing_transaction(state, 'noodles', Time.now)
  225      end

test/new_relic/agent/sql_sampler_test.rb:
  440
  441:   def test_on_finishing_transaction_with_busted_transaction_state_does_not_crash
  442      state = NewRelic::Agent::Tracer.state
  443:     @sampler.on_finishing_transaction(state, "whatever", Time.now)
  444    end

test/new_relic/agent/transaction_sampler_test.rb:
  445        end
  446:       @sampler.on_finishing_transaction(@state, @txn, Time.now.to_f)
  447      end

  460        @sampler.notice_pop_frame(state, "a")
  461:       @sampler.on_finishing_transaction(state, @txn, (stop || Time.now.to_f))
  462      end

I split it in two - removing the use in rest of the code, and removing the param, just in case you want to keep the API.


Enable SRP for the cc26x2x7

Problem
The CC26x2x7 platform did not advertise CHIP functionality with SRP on the Thread network.
NOTE: During TE#2 it was necessary to delay starting the SRP client until
attached to a Thread network to avoid a deadlock on the CHIP stack lock.
Summary of Changes

Add configuration to enable the OpenThread SRP client
Add default GN option to add the CHIP platform mDNS code

Fixes #6375

@srickardti Can you please restyle per the instructions in https://github.com/project-chip/connectedhomeip/pull/6379

NOTE: During TE#2 it was necessary to delay starting the SRP client until
attached to a Thread network to avoid a deadlock on the CHIP stack lock.

Does this mean this PR will lead to a deadlock?

Does this mean this PR will lead to a deadlock?

I'm still working on the TE#2 environment and that branch, I haven't been able to test this on the mainline. I had made this PR, not realizing that starting the SRP client had moved. It had moved to the end of the rendezvous flow so I do not believe it will deadlock, but I will report back when I've made sure it doesn't.


stdbuf: fix cargo --git build (#1276) (Attempt 2)

Closes #1276.
I have tested this with
cargo install --git file://<path to local repo> uu_stdbuf

Let's hope it also works correctly with crates.io.

Could you please add this to the CI ? for example, the build task could test this.
thanks

Could you please add this to the CI ?

That's a bit tricky. It causes a full build of the project, even if the project has been built already, so having it on all targets may be a bit excessive. I could add a separate job that just installs via git on ubuntu with all the unix features enabled?

sounds good :)

I can't get it to work. The CI creates a merge commit that cargo does not recognize and therefore it cannot clone the repository to install it. I've removed it again for now.


feat(delayWhen): add index to the selector function

#2388
This is for re-merge https://github.com/ReactiveX/rxjs/pull/2409, all credit goes to @martinsik .

Description:
Related issue (if exists):

Coverage increased (+0.0007%) to 97.69% when pulling 688ea189a13bf0272588a15c8916490f289d8e73 on kwonoj:pr-2409 into e2696db8f67b03854555bb8c0754ed39a5520763 on ReactiveX:master.

Ugh. @kwonoj, can we get this updated? Perhaps a new PR if necessary?

This needs refactored add added to v6 :\ . Ugh, the backlog... haha.. sorry @kwonoj

@benlesh I made the original commit so I can have a look at it.

@benlesh This should be rebased and ready for review/merge.


dblclickToNavigate

i want to make dblclickToNavigate navigate to day not to narrow view

@HaniOurEdu  You can do this by registering the @cell-dblclick and turning off narrow view
https://antoniandre.github.io/vue-cal/#ex--other-event-creation-methods


istio-policy always crash

I use  istioctl manifest apply --set profile=demo  to install istio 1.4.10 on my kubernetes 1.13.
After install, the pod Istio-policy alway crash. plesase see the following image

the pod event show as following image

can you tell my how to solve this problem, thank you

@kengan istio 1.4 has reached end of life.
Please try it on a supported version of Istio. (1.5 +) at this time.
Please also include logs from the istio-policy pods. Thanks.