fix: updates ValueRecorder to allow negative values

Which problem is this PR solving?

Fixes: https://github.com/open-telemetry/opentelemetry-js/issues/1240
Fixes: https://github.com/open-telemetry/opentelemetry-js/issues/1241 (I believe.  Couldn't find any other usage of that configuration so finished removal process in this PR)

Short description of the changes

Removes absolute check for ValueRecorder to allow negative values
Removes absolute configuration since no-longer has a valid use
Removes unused record(..., correlationContext, spanContext) signature(s)

I think codecov is mad at me for deleting too much covered code? 🤷

Haha actually it's mad because only 75% of lines you added are tested. Since you mostly deleted and only added a very small number of lines, it is saying you only tested 75% of the diff, even though you increased total project coverage by 0.07%.

I believe the only actual new non-test code I added was appropriately referencing the api.BoundBaseObserver for the BoundObserver so not currently planning to try to get that codecov to pass but let me know if I need to try to force it to be happy.
Also, the related issue is flagged as required for GA so not sure if you want to flag this PR or not for the project board but figured I'd mention.

@dyladan merged in master. think this has the necessary approvals now (2 maintainers, 3 total reviewers)


[excercises.md] Fix link

Please also fix a link on http://mrl.cs.vsb.cz//people/gaura/spja_course.html, at

All the information for the Scripting languages course may be found in this GitHub repository.

The link https://github.com/geordi/skj-course/README_en.md is broken, please use https://github.com/geordi/skj-course/blob/master/README_en.md.

The link on my website is now fixed. Thanks for noticing!


Maintenance

Hi,
I've noticed that this package doesn't seem to be maintained anymore.
Would you be willing to transfer it? I've been using TypeScript to develop my addons and I'd like to update the package to the latest versions of the different components.
Appreciated.

Sure, that would be great. I'll see how I can do that.

So I tried to transfer it to you, but it failed with this error "locks already has a repository in the kimroen/ember-cli-coffeescript network"
Let me know what you'd like to do.

What about now? I sent a PR to another project that forked ember-cli-coffeescript but I've removed the repository.

@flcdrg thank you, sir :smile: If you can also add me as an npm owner I'll coordinate publishing an update soon.

No worries!
That wasn't me that published to npm. I think you'll have to contact the person who published it up there. Hopefully they'll add you in seeing as you've taken over the repo.


Adding more logs for the command state watcher

These were merged into another PR and then merged into main, closing.


Farmmgmt: any prices update should reflect only after saving

[x] Setting up farm prices doesn't show up the current farm prices in the edit form
[x] Changing values of a deal reflects immediately and doesn't rollback if you press cancel

Works interactively,
commit: latest development_vdc


Handling User Authentication in Firebase and Vue.js Applications

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #5452

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Hello @annemwangi ,
Thanks for submitting this PR.
Your PR has entered our reviewing queue. Please wait until a peer-reviewer pings you with the changes to be made.
Meanwhile, we suggest you to go through other approved PRs/published articles to see if you have missed out anything, which might avoid a potential PR closure.
Thanks.

@section-engineering-education/content-moderator-team

Plagiarism check complete

@adrianmurage, could you please explain to me what "nothing renders means"?
Because the code was working perfectly. I have also tested it right now and it's still working perfectly.

@annemwangi
after following through your article with 2 different machines, I end up with the same issue:
 ERROR  Failed to compile with 1 error                                                                                1:58:59 PM

/home/m/Code/reviews/firebase-vue-user-management/src/components/Navbar.vue
  1:1  error  Component name "Navbar" should always be multi-word  vue/multi-word-component-names

/home/m/Code/reviews/firebase-vue-user-management/src/views/Home.vue
  1:1  error  Component name "Home" should always be multi-word  vue/multi-word-component-names

/home/m/Code/reviews/firebase-vue-user-management/src/views/Login.vue
  1:1  error  Component name "Login" should always be multi-word  vue/multi-word-component-names

/home/m/Code/reviews/firebase-vue-user-management/src/views/Signup.vue
  1:1  error  Component name "Signup" should always be multi-word  vue/multi-word-component-names

✖ 4 problems (4 errors, 0 warnings)

You may use special comments to disable some warnings.
Use // eslint-disable-next-line to ignore the next line.
Use /* eslint-disable */ to ignore all warnings in a file.
ERROR in
/home/m/Code/reviews/firebase-vue-user-management/src/components/Navbar.vue
  1:1  error  Component name "Navbar" should always be multi-word  vue/multi-word-component-names

/home/m/Code/reviews/firebase-vue-user-management/src/views/Home.vue
  1:1  error  Component name "Home" should always be multi-word  vue/multi-word-component-names

/home/m/Code/reviews/firebase-vue-user-management/src/views/Login.vue
  1:1  error  Component name "Login" should always be multi-word  vue/multi-word-component-names

/home/m/Code/reviews/firebase-vue-user-management/src/views/Signup.vue
  1:1  error  Component name "Signup" should always be multi-word  vue/multi-word-component-names

✖ 4 problems (4 errors, 0 warnings)

webpack compiled with 1 error

I started again from scratch up to the point where you introduce Navbar.
Upon renaming Navbar.vue to NavBar.vue, I got it to render out.
I also found documentation that explains the error.
I am assuming if you fix this, the app should work as expected

@annemwangi
Good job with this article.
Do however take note of the value of working through your article from a reader's perspective to help catch any errors in your future work
Otherwise, keep up the good work and I look forward to seeing more of your work published by EngEd

@annemwangi Good job with this article.
Do however take note of the value of working through your article from a reader's perspective to help catch any errors in your future work.
Otherwise, keep up the good work and I look forward to seeing more of your work published by EngEd.

Thanks a lot for taking the time to review my article and for the advice :rocket:

Merged #6282 via #5452


Deploy/700 jan8 eth

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Convert to ESM

Feature description
The smallest jQuery implementation is the one that gets treeshaken :smirk:

Feature motivation

So I don't have 90% of my JavaScript bundled marked as unused because I want to import one method from cash-money over here.
Is this feature present in jQuery? Yes/No
No

I'm not very familiar with tree shaking, I have a few concerns with this:

Keeping our minified bundle here as small as possible is important to us, wouldn't exporting and importing all our components individually add a significant amount of useless require calls to the bundle?

We have a few methods exported as $.* (e.g. $.isFunction), I can see how those could easily be imported individually when needed, but everything else relies on selecting the elements to work on first with the main selector $() and is designed around providing a chainable API. For instance we can write this today:

import $ from 'cash-dom';
$('.foo').addClass ( 'asd' ).find ( 'li' ).remove ();

how would this piece of code translate to a tree shaking-enabled version of cash? The following looks awful to me, and most importantly it would be a very different API:
import {$, addClass, find, remove} from 'cash-dom';
const $foo = $('foo'); // We don't want to run this selector more times than necessary, so we cache it
addClass ( $foo, 'asd' );
const $li = find ( $foo, 'li' );
remove ( $li );

If I run import {each} from 'cash-dom'; is that supposed to import $.each or $.fn.each?

This doesn't seem practical to me, IMHO partial builds are a better options for cash.


fix(state): fail transition on exceptions in transition handler

closes #2772

This change broke all the unit tests.  Would you please:

Figure out how this broke the unit tests
Add a test case that throws an error in onEnter and successfully listens for $stateChangeError

Thanks @christopherthielen.
Hmm, the tests pass for me locally. I'll look into why Travis is unhappy. Assuming I find and fix the problem, do you prefer a series of separate commits in this PR, or squash (git commit --amend and git push -f) so this PR contains just one working commit?

I prefer one working commit, thanks!

$q's support for promise.catch seems to have been introduced in Angular 1.3. The travis test suite nicely runs regression tests with all supported versions of Angular. If I just invoke grunt locally I guess it picks one of the newer versions of Angular. The suite that failed in travis was "karma:ng108" (given the $q history, I would expect karma:ng115 and karma:ng1214 to also fail, karma:ng130 and karma:ng149 and karma:150 to succeed).

Note to self: grunt integrate is what I should have been running, not just grunt default aka grunt.

Ah, thanks for looking into that!
You can run the 'travis style' test suite using grunt integrate on legacy branch and npm run test:integrate on master.
Let's change from .catch(handler) to .then(null, handler)

Trying to write the requested unit test for the new fix and I'm getting hung up on another $q-ism. I don't understand the $q mocks (test/testUtils.js in ui-router, and qFactory in angular proper) very well. Presumably the ui-router tests are using qFactory to provide a different nextTick because I see it continuing synchronously but I don't know where that happens.
Anyway, the $q processQueue function has some special semantics for exceptions thrown in promise .then routines -- it catches the exception, rejects the promise, but also calls the exceptionHandler (specified to qFactory). And that results in my test failing with
	Error: negative onEnter
	    at /home/matt/src/open/ui-router/test/stateSpec.js:125
	    at invoke (/home/matt/src/open/ui-router/lib/angular-1.4.9/angular.js:4535)
	    at /home/matt/src/open/ui-router/src/state.js:1133
	    at processQueue (/home/matt/src/open/ui-router/lib/angular-1.4.9/angular.js:14991)
	    at /home/matt/src/open/ui-router/lib/angular-1.4.9/angular.js:15007
	    at /home/matt/src/open/ui-router/lib/angular-1.4.9/angular.js:16251
	    at /home/matt/src/open/ui-router/lib/angular-1.4.9/angular.js:16069
	    at /home/matt/src/open/ui-router/test/testUtils.js:11
	    at /home/matt/src/open/ui-router/test/stateSpec.js:567
	    at invoke (/home/matt/src/open/ui-router/lib/angular-1.4.9/angular.js:4535)
	    at workFn (/home/matt/src/open/ui-router/lib/angular-1.4.9/angular-mocks.js:2517)

where (at the top) stateSpec.js:125 is in my intentionally faulty onEnter callback, (at the bottom) stateSpec.js:567 is the call to $q.flush() in my test which is necessary to note the rejected promise, and (in the middle) angular.js:14991 is the $q glue code for promise.then that should catch the exception and convert it to a promise rejection (which it does, but it also essentially rethrows the exception).
Can you point me at where the ui-router tests set up qFactory?

I repushed the branch with the updated change (including the test that will fail as described above).

@metamatt  I'm in the same boat here, I'm not sure why it's set up to re-throw.  The mocks docs imply it's because a throw in a promise is probably a bug...
The key is to call $exceptionHandlerProvider.mode('log'), (instead of rethrow).
  beforeEach(module(function ($stateProvider, $provide, $exceptionHandlerProvider) {
    var x = this;
    var foo = jasmine;
    $exceptionHandlerProvider.mode('log'); // <--
    angular.forEach([ A, B, C, D, DD, E, H, HH, HHH ], function (state) {
      state.onEnter = callbackLogger('onEnter');
      state.onExit = callbackLogger('onExit');
    });
    stateProvider = $stateProvider;

    $stateProvider
      .state('A', A)
      .state('B', B)
      .state('C', C)
      .state('D', D)
      .state('DD', DD)
...

However, I'm worried that this could lead to false positives in other tests.  I think we should create a new top-level beforeEach(module()) block for just this test

Great suggestions. I'll try that next time I get a chance. Thanks.

merged


PrimeOption: take() function does not have an event emitted!

Description
The take() function will send the tokens on the balance sheet which are not internally tracked to the caller of the function. By internally tracked, we mean the values in the cache != balances.
The Missing Event
This take function should have an event emitted, because it will probably be called by parties whenever tokens are erroneously (or otherwise) sent to it.
Fix
event Take(address indexed from, uint256 outTokenU, uint256 outTokenS, uint256 outTokenR, uint256 outTokenP)
This might be a little gas intensive.

Doesn't need an event because it emits the transfer events.


New package: Animations v0.2.1

Registering package: Animations
Repository: https://github.com/jkrumbiegel/Animations.jl
Created by: @jkrumbiegel
Version: v0.2.1
Commit: 68ae0e0afa135403d882933871ae2d95d6601777
Reviewed by: @jkrumbiegel
Reference: https://github.com/jkrumbiegel/Animations.jl/issues/1#issuecomment-539620891

I've reset the version number to 0.1.0 as the bot recommended (it wasn't really meaningful before anyway) so now this pull request is obsolete, there is a newer one. for some reason it didn't update this one after the version number change


Add ideal state param back to rebalanceAndUpdate, so that the updated ideal state is returned to the API

Codecov Report

Merging #2643 into master will increase coverage by 0.11%.
The diff coverage is 57.14%.

@@            Coverage Diff            @@
##           master   #2643      +/-   ##
=========================================
+ Coverage   69.39%   69.5%   +0.11%
=========================================
  Files         861     860       -1
  Lines       39950   39925      -25
  Branches     5374    5372       -2
=========================================
+ Hits        27722   27750      +28
+ Misses      10416   10366      -50
+ Partials     1812    1809       -3

Impacted Files
Coverage Δ

...ore/rebalance/DefaultRebalanceSegmentStrategy.java
71.91% <57.14%> (+0.97%)
:arrow_up:

...in/pinot/core/query/pruner/ValidSegmentPruner.java
57.14% <0%> (-28.58%)
:arrow_down:

...minion/executor/BaseSegmentConversionExecutor.java
67.16% <0%> (-2.84%)
:arrow_down:

...e/operator/dociditerators/SortedDocIdIterator.java
61.11% <0%> (-2.78%)
:arrow_down:

...pby/NoDictionarySingleColumnGroupKeyGenerator.java
90.9% <0%> (-2.03%)
:arrow_down:

...ot/core/query/pruner/ColumnValueSegmentPruner.java
87.93% <0%> (-1.73%)
:arrow_down:

...inot/core/operator/docidsets/AndBlockDocIdSet.java
57.93% <0%> (-1.38%)
:arrow_down:

.../com/linkedin/pinot/common/data/TimeFieldSpec.java
91.13% <0%> (-1.27%)
:arrow_down:

.../pinot/core/common/datatable/DataTableBuilder.java
91.06% <0%> (-1.12%)
:arrow_down:

.../broker/routing/HelixExternalViewBasedRouting.java
90.1% <0%> (-1.07%)
:arrow_down:

... and 20 more

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 557e6ab...a9f0c8e. Read the comment docs.


search_after queries are not performing as expected

Disclaimer: the origin of this issue was a post in ElasticSearch discuss.
Is your feature request related to a problem? Please describe.
We began to use the search_after parameter for solving our issues related to pagination consistency. This works fine until we realized that our query slow log started to have a lot of entries. At some moments this lead us to high CPU usage and high search latency, which degraded our service.
All of these log entries show us queries without filters or search terms and use the search_after parameter. This kind of query is taking ~400ms to be resolved. The issue is that its equivalent using a simple from/size pagination is taking just ~20ms.
So, why ES is taking more time/resources to resolve the query with "search_after" than its equivalent with from/size?
Describe the solution you'd like
After some query profiling, the team of ElasticSearch came with the explanation:

I see what's happening.
From 7.6 we introduced an optimization 2 for sort queries, we rewrite a sort query as LongDistanceFeatureQuery that can skip over blocks of documents. This makes sort queries under certain conditions much faster. The optimization is currently not applied to search_after case. From 7.16 it will be applied to search_after as well, so sort queries with search_after also will be much faster under certain conditions.

So, this issue is just to ask for including this optimization to improve search_after queries in OpenSearch too.
Describe alternatives you've considered
In the meantime, the only thing we can do is add index sorting, which improves dramatically the issue, but just for one sort criteria.
Additional context

I will work on this

For search_after queries, sort optimization has not been applied. In some certain scenarios like @manuel-pancorbo said there is performance latency with search_after queries and some improvement needs to be done by applying sort optimization to search_after queries. The latest Lucene version has the sort optimization capability and I am planning to make use of this to optimize search_after queries. I will be comparing the benchmark results of OpenSearch using Lucene's capability for optimizing with current OpenSearch version. Based on results, we will further move forward with this optimization.


Suggestion: Include Predefined Checks for Supervision Tree Readiness

Hi,
First, I want to thank you for developing Existence. I have been exploring its features and it seems like an excellent tool for managing health checks in Elixir applications.
I have a suggestion for a predefined check that I believe could enhance the library: a built-in check to ensure that the entire supervision tree has fully started, utilizing a "terminator" process or similar mechanism. This would be particularly useful for readiness probes in Kubernetes deployments.
Proposed Feature:

Supervision Tree Readiness Check: A predefined health check that verifies whether the entire supervision tree is fully started. This could be implemented by adding a special "terminator" process as the last child in the supervision tree. The readiness check would then confirm that this process is running, indicating that all prior processes have been started successfully.

Benefits:

Reliability: Ensures that all necessary processes are up and running before marking the application as ready.
Ease of Use: Provides a ready-to-use solution for a common requirement, reducing the need for custom implementations.
Kubernetes Integration: Enhances the ability to use Existence with Kubernetes readiness probes, providing a more robust setup for production deployments.
Simplified Readiness Check: This check wouldn't be used on its own but could be useful in determining readiness without the need to implement numerous individual checks.

Considerations:

Flaws/Challenges: I am curious if there are any potential flaws or challenges you foresee with this approach. For example, are there edge cases or scenarios where this might not be reliable or could cause issues?

I appreciate your time and consideration of this suggestion. Looking forward to hearing your thoughts!
Best regards,

Hey @eric0fw,
Thanks for your kind words :) We are using the library in production and it is not causing any problems.
The additional supervision tree check obviously makes sense for environments running in a k8 cluster. Depending on the complexity of the application, checking just the last process (child) may not be sufficient to consider the application ready:

previous processes may have been started, but their operation may cause repeated restarts. In such a situation, Supervisor.which_children/1 returns a status of :restarting instead of pid for such a process.
previous processes may have been started, but when the callback GenServer.init/1 returns :ignore, initialisation of the supervision tree continues but the process is ignored:

Returning :ignore will cause start_link/3 to return :ignore and the process will exit normally without entering the loop or calling terminate/2. If used when part of a supervision tree the parent supervisor will not fail to start nor immediately try to restart the GenServer. The remainder of the supervision tree will be started and so the GenServer should not be required by other processes.

source: https://hexdocs.pm/elixir/1.17.2/GenServer.html#c:init/1
In both of these situations I would consider the supervision tree not ready, especially if all processes are required to run the app.
Nevertheless, the problem is interesting and we could look into it.
@andrzej-mag I would be happy to hear your thoughts on this topic.

I am sorry for late reply. I didn't receive any notification and I wasn't aware of this discussion.
Application supervision trees are created from stable and trusted processes usually. If some supervised OTP process would crash infrequently, process would be restarted so quickly that it would be way below any orchestration platform time resolution threshold. If supervised process would crash continuously, application supervisor should terminate itself and application - no need for any additional health-checks. I don't see any practical value in health-checking application supervision tree or single tree processes. Situation is different with health-checking unsupervised external dependencies which can fail due to uncontrolled reasons (eg. network issue) for extended time.
To health-check some important OTP process you can use for example is_pid(Process.whereis(:some_important_process)) in the check callback implementation. If checked OTP process is supervised, supervisor restart intensity and period in reference to orchestration timeouts should be considered probably.
Existence should be started as a last process in the supervision tree as all health-checked dependencies should be started first anyway. Library will act then as a "terminator" process mentioned in the @eric0fw comment.
To use Existence as a readiness check with dependencies requiring substantial time to initialize you can set initial_delay in the check configuration.


feat: made sensetive file optional

Enhance Code Review Functionality
Overview
This pull request aims to improve the code review functionality in the kaizen-cloudcode project. It introduces a new feature to optionally check for sensitive files during the review process and provides a way to customize the review context.
Changes

Key Changes:

Added a new parameter check_sensetive to the review_pull_request function, which allows the caller to enable or disable the check for sensitive files.
Refactored the review_pull_request function to only execute the check_sensitive_files method if the check_sensetive parameter is set to True.
Updated the pyproject.toml file to increment the version number from 0.4.8 to 0.4.9.

New Features:

Introduced the ability to provide a custom context string to the review process through the custom_context parameter in the review_pull_request function.

Refactoring:

No major refactoring changes were made in this pull request.

✨ Generated with love by Kaizen ❤

Original Description

!review


Changed to use FindFirstFileExW instead of FindFirstFile

This improves performance approximatelly by 2x
Now, on my home computer, the node_modules folder (and its subfolders) can be processed in ~35ms

@KrzysztofCwalina is this supposed to run on Unix too? If that's the case, we would need to add the FindFirstFileExW to PAL.

We wouldn't want to add it to the libcoreclr PAL, though; if/when we want to make this library support Unix, we'd want to introduce a mini-PAL layer into the library itself, as we've done elsewhere in corefx.

@stephentoub Makes sense


Fix for #971 - "Fatal error when Ethereum node going offline"

Overview
Fix for #971, added .once() into Events.
Review
@alaibe
Cool Spaceship Picture
spaceship.jpg

@hodlbank thanks for the PR and also reporting the issue!
While the code changes look good, I'm having a hard time reproducing the original issue so I can test whether you PR works. What else did you do apart from running embark console testrpc from within an Embark project?
Some additional thoughts:

For completeness sake, it'd be really really nice if you could add a comment on the issue about the original error that's being thrown by Embark. The Callback was already called is a very unfortunate characteristic of async.js, in which it swallows the actual error that has been thrown.
Once we know the original Error it probably also becomes more clear why switching to .once() API solves the issue

And a tiny minor thing: If we want this PR to close the original issue once it's merged, we need to put the Fixes #971 in the body of the commit message.

@PascalPrecht , the switch to once is right. As the even listened to is really more a "wait for ethereum node to be ready" than a "check back online". It should have been once all along.

the switch to once is right. As the even listened to is really more a "wait for ethereum node to be ready" than a "check back online". It should have been once all along.

This makes sense, thanks for clarifying!

@jrainville , I believe chnaging .on to .once in other places (e.g. run()) would break following use case working now:

embark run starts.
Handler for "check:backOnline:Ethereum" assigned.
Ethereum node check emits "check:backOnline:Ethereum" event, handler executed.
The handler initiates deployment.
Connection to Ethereum node lost, deployment aborted.
Connection to Ethereum node restored, deployment should start over.

Currently 6 works as expected. Replacing .on() with .once() in cmd_controller.js->run() would never initiate 6 after 5.
Flow is different in cmd_controller.js->console(), so replacing .on() with .once() makes sense there.
Please advise on how to proceed.


Add Node 22 to CI version matrix

Add Node.js 22 to CI version matrix, as it is the "current" version and LTS. Also moves single version actions to run in 22 instead of the older 18, which will be end-of-life in six months.
I've been running TestContainers in 22 for a while now without issue, but would be nice to have them in official test battery too. For successful CI run, see PR in this fork.

Thanks for the PRs @stscoundrel, much appreciated!


DEV TO DO LIST

TODO (for now):

[x] Change url from default www.arcgis.com to the actual portal site URL

Example: site.maps.arcgis.com instead of the default arcgis.com.  This comes from the portal self call from the portalSelf.urlKey, portalSelf.customBaseUrl items.

[x] add new User properties on community.User class.

[x] Handle create assignment functions

[ ] verify all properties captured on HFS/MapService/Feature Service, etc..

[ ] Improve addByParts() to handle memory better for very large files.

Add catch if file is less than 4 MBs, then do not add by parts, do standard upload of file.

[x] Add function to reset a user's password using the reset() on community.user class

[ ] Issue with python 2.7.10 and invalid SSL certificates.  Make this an optional flag on BaseWebOperation Module.  By default it will be set to True, but if it is False and Python is 2.7.9 or greater, this code at the 2.7 framework needs to be run:
  import ssl
  import urllib2
  ctx = ssl.create_default_context()
  ctx.check_hostname = False
  ctx.verify_mode = ssl.CERT_NONE
  urllib2.urlopen("https://sometestserver.com/portal/sharing/rest", context=ctx)

TODO: verify workaround in Python 3.4.
WISHLIST (for now):

[ ] local data visualizer

[ ] Get all samples to latest version

[ ] build out simpler front end for user easy of use

[ ] produce a tutorial

Please feel free to add your comments.  This is just some of the goals for the next release.  Remember this is fluid, and comments are welcome.
Additional Functionality:

Added to community.user update() input value of userType which will allow admins to enable or disable 'Esri Access' so organizational users can use the AGOL org account with the forums.

Would it be possible/useful to create a global environment setting (a la arcpy.env.overwriteOutput) for message logging? My naive thought is something like arcrest.env.verbose = 0|1|2 which would suppress all print functions, only show errors, and print everything, respectively.
I also plan on taking a stab at the Sphinx/jinja documentation.

Paul,
There are some good logging modules in python that we could look at leveraging - https://docs.python.org/2/howto/logging.html

We can try to create an htttphandler(debug=1) to print out urllib messages?
Also new in the 3.5.3 version there is a object.last_code, last_message, and last_url which can help in debugging as well.

Or just use fiddler ☺


Adding notification support for project setup issues

More and more issues come in about Lombok Plugin not working that boils down to AnnotationProcessing being disabled or dependency missing.
This is an initial step for doing on-the-fly detection of the issues with project setup.
Currently only supports checking for annotation processing being enabled.
What do you think?

Current coverage is 58.28%

Merging #221 into master will decrease coverage by -0.32% as of ea56bd2

@@            master    #221   diff @@
======================================
  Files          160     161     +1
  Stmts         4517    4535    +18
  Branches      1212    1214     +2
  Methods          0       0
======================================
- Hit           2647    2643     -4
  Partial        416     416
- Missed        1454    1476    +22

Review entire Coverage Diff as of ea56bd2

Powered by Codecov. Updated on successful CI builds.

Good idea!
If we add dependency checking too, this issue would be possible done as well: https://github.com/mplushnikov/lombok-intellij-plugin/issues/103

Yes, that was my thought as well. I'm not 100% sure at this point about the best and most reliable way of checking the dependency, simply because they could come from many different sources. But i'll play around with that.


Update Dockerfile user permissions on /tmp/export

Apply chown to /tmp/export with created user

Allows using named containers through docker-compose without forcing user back to root
Change created user to "overviewer-mc"

Thank you!


Fix for Unexpected struct type

In some circustmances, Twilio client return an object with some key(s) whose value is undefined. This somehow breaks Pulumi SDk.
Replace undefined with null for those object items.
Fix #6

@kaiquelupo can you verify and merge? I reproduced that with an IncomingPhoneNumbers resource. I verified that the fix works on my Twilio project. You may want to double check if there are any side effects.


Keep exif metadata when scale down

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description
Fix #2026.
Our SDWebImageScaleDownLargeImages options implementation just use UIImagePNGRepresentation and this will lose all the EXIF metadata(Because UIImage instance will not keep this information, it was kept in the CGImageSource).
Since we have ImageIO, it's much easier to do image encoding and keep all the EXIF metadata. So we can use this instead of that and keep cross-platform support.
To keep the code clean, I pick the commits from #1987 because it will move all that image decoding/encoding code into seperate class and keep this SDWebImageDownloadOperation easy.

Closed due to our refacor ImageIO coder will keep image orientation (decoding and encoding).
But there is maybe one behavior change. Previously, when use SDWebImageScaleDownLargeImages. After scale down the image. We use UIImagePNGRepresentation to recreate an data and set it to callback. I think this behavior is a little strange since it's useless in most cases.(In fact, if you ignore the NSData, it will be release after the completion block).
If someone think we should keep this behavior to modify the image data as well. We can reopen a PR and change back this one.


[API] API Errors throw different HTTP Status Code

Feature Request
Feature Information:
It would be awesome, if incorrect API calls (for example if you provide no sessionID) would return some status code different from the standard 200 - Successful.
It would make writing an API Wrapper significantly easier, if the answer wouldn't need to be parsed every time to check if it was successful or not...
I confirm:

[x] that I have searched for an existing feature request matching the description.

In the situation where AMP thinks you don't have permission to do something, it 'plays dumb' and doesn't acknowledge that the endpoint you're trying is valid but that you don't have permission. Not having a session ID falls under that.
Ideally you should use one of the official API wrappers (or use one of them as the basis for writing your own instead of starting from scratch) - there's official APIs for NodeJS and C#


Setup Generic Goldsky Dagster Asset

Describe the feature you'd like to request
An issue to cover the creation of a generic goldsky dagster asset. Specifically to split #1293
Closed with #1321
Describe the solution you'd like
Goldsky data should be loaded as a dagster asset
Describe alternatives you've considered
N/A

Accidental dupe of #1280


KIOST-ESM licensing update from CC BY-SA 4.0 to CC BY 4.0

From: 김영호 yhokim@pknu
Sent: Wednesday, September 28, 2022 3:40:40 AM
To: Briony Turner
Subject: Re:re Action requested: CMIP6 licence relaxation
Dear Briony,
Sorry for late reply.
I agree to the license relaxation. The date would be September 1st, 2022.
Best wishes,
YoungHo.

Young Ho Kim, Ph.D.
Assistant Professor
Department of Oceanography, Pukyong National University
45 Yongso-Ro, Nam-Gu, Busan, 48513, South Korea
Tel: +82-51-629-6xxx; Fax:+82-51-629-6xxx
E-mail: yhokim@pknu

----- Original Message -----
From: Briony Turner
To: "yhokim@pknu"
Date: Tue, 27 Sep 2022 22:05:32 +0900 (GMT)
subject: re Action requested: CMIP6 licence relaxation
Dear Dr YoungHo Kim,
Further to the correspondence in July from CMIP IPO, the WGCM Infrastructure Panel (WIP) workstream to relax the CMIP6 licences from the existing CC BY-SA 4.0 default license is being closed this month.  There are now 98 of 133 entries on WCRP-CMIP CMIP6_CVs which are CC BY 4.0. CMIP6 source_id license details (wcrp-cmip.github.io).
Action requested by 30 September
Please either:
Confirm the date by which license relaxation will take effect for KIOST-ESM
Or, if you do not intend to make the license relaxation, please supply alternative monitored email address or confirm that yhokim@pknu is the correct point of reference for the WGCM Infrastructure Panel (WIP) to list for all license waiver/exceptions enquiries for: KIOST-ESM
If we do not receive a reply to this email we will assume there is no intended license relaxation and that the email address listed in (2) is the correct correspondence address for license enquiries.
Best wishes,
Briony

@pandland-kiost @noah1004 that change is now in place, please review CMIP6_source_id_licenses.html and let me know of any remaining issues


EthStorage Devs Meeting #16 Agenda

Meeting Info

Jan 17, 2023, 10:00 UTC+8
Duration: 60 minutes
Meeting link
Anyone can comment on the agenda where they would like this to be discussed.

Agenda

Protocol/App/Research progress
Qi will present a summary of 2022 H2 and future plan for 2023 H1

I will present a summary of 2022 H2 and future plan for 2023 H1

Application

Helios

English version slides - sent
wasm integration in our extension - (todo: hook the URL response)

W3URL

w3bnb/w3evmos blacklist - (todo: submit the review request)
dashboard support other L1s
w3box on Nova - code review
pixel-war - draw the image on the frontend

Research

EOF - 70%

Application

Sharded storage with P2P support

Design doc (Updating)
Implementation: part 1 has check-in, and part 2 is waiting for review (https://github.com/QuarkChain/go-ethereum/pull/104)
Integration test (status: done, debug branch)

Efficient p2p discovery protocol

implementation (75%)

Application

research

Shared Tornado-Cash
Continue to learn plonk and halo2

Application
-Reoptimizing external-call module code

Research:

https://arxiv.org/pdf/1906.07221.pdf -> Halo 2 theoretical basis: A formalization of arithmetization of a particular 2nd order logic language.

Recording: https://drive.google.com/file/d/1e8I443SlnOlous5ZMT3bmKqL8DEcbw2s/view?usp=share_link


AHD fringing

AHD produces CA-like fringing on higher contrast edges which cannot be removed by color post-processing. These are not present in other software supporting AHD (e.g., RawTherapee) even when disabling false color suppression or using less sophisticated methods like "Fast". Images are otherwise identical so its likely a photosite offset.

Fixed as of 20f635a. Homogeneity of infilled pixels tends to be lower now probably because Gaussian filter was replaced by bilinear but fringing now matches other programs.


CircleCI 1.0 is to cease functioning on 15 March 2019

You are receiving this notification because you have recently triggered a build on the cloud-hosted version of CircleCI 1.0, which has been EOL since September 2018.
Starting March 15, 2019, all jobs referencing 1.0 in their configuration will cease to run.
In order to continue building your project, you must update your configuration file to 2.0.
To see which of your projects are still building on CircleCI 1.0, please follow the instructions here. If you identify projects on 1.0 that are no longer under active development, remove the webhook by clicking “Stop Building” from the project settings page.
Our goal in providing these notifications is to give you as much notice as possible before we deprecate the cloud-hosted version of CircleCI 1.0. Over the next six weeks, we will be sending frequent reminders to everyone with recent builds on CircleCI 1.0.
For more information including a guide to migrating to CircleCI 2.0, please visit our migration resource center.
Thank you for your attention to this matter,
The team at CircleCI

This is a follow-up reminding you that one or more of your projects uses CircleCI 1.0 configuration and will no longer build after March 15, 2019.
In order to address this issue and keep your project(s) building, please see:
Instructions on seeing which projects still use CircleCI 1.0 config
A guide to migrating from 1.0 to 2.0
CircleCI 1.0 EOL policy
We know we have sent you a number of notifications in recent weeks regarding 1.0 EOL. At the risk of over-informing you, our goal is to make sure that zero teams or projects are caught unawares when they can no longer build with CircleCI 1.0 configuration on March 15, 2019.
We are here to help in making this switch, please visit the migration center.
Thanks again,
The team at CircleCI


[MINOR] Fix some wrong assert reasons

What is the purpose of the pull request
Fix some wrong assert reasons
Brief change log

Fix some wrong assert reasons

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3248 (64ad40a) into master (3715267) will decrease coverage by 24.52%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3248       +/-   ##
============================================
- Coverage     27.40%   2.88%   -24.53%
+ Complexity     1287      85     -1202
============================================
  Files           381     281      -100
  Lines         15108   11620     -3488
  Branches       1305     952      -353
============================================
- Hits           4141     335     -3806
- Misses        10667   11259      +592
+ Partials        300      26      -274

Flag
Coverage Δ

hudiclient
0.00% <ø> (-21.06%)
:arrow_down:

hudisync
5.37% <ø> (ø)

hudiutilities
9.25% <ø> (-49.32%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Pipes in latest ionic2 version not working

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
import {TranslateService, TranslatePipe, TranslateLoader, TranslateStaticLoader} from 'ng2-translate/ng2-translate';
pipes: [TranslatePipe],
Please suggest how to use pipes in the latest version of IONIC2 since i used like above in older verison it is working fine.
What behavior are you expecting?
Steps to reproduce:
1.
2.
3.
insert any relevant code between the above and below backticks (```)

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Run ionic info from terminal/cmd prompt: (paste output below)

Hello, thanks for using Ionic! You should be importing the TranslatePipe in your app module and putting it in the providers array there. You can see our docs on how to use ng2-translate here.


Missing description of field 'UxdSettings'

UxdSettings is not defined

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

ID: 1b88c77d-d9a8-9cfe-7cea-d2a502fb9248
Version Independent ID: 7dbc8a6f-c34d-ba6b-dfc0-b4e12472e402
Content: _HUB_DEVICE_CONFIG_INFO_V1
Content Source: wdk-ddi-src/content/usbioctl/ns-usbioctl-_hub_device_config_info_v1.md
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Hi @BrianCatlin,
I’ve created an internal work item for myself to investigate this with the USB team, and I’ll make a note here when I complete the changes. In the meantime, I’m closing the GitHub issue while it’s tracked in the bug. Thanks again for helping to improve the docs.


Linking of rule '@protobuf//:internal/_api_implementation.so' failed:

I am trying to build tensorflow v0.12.0 from source with bazel 0.4.3 and failed with this error:
ERROR: /ui/ncsa/qiyuelu1/.cache/bazel/_bazel_qiyuelu1/ce8d1fb9ff1b7a9dbbd225fe7f0c6f52/external/protobuf/BUILD:579:1: Linking of rule '@protobuf//:internal/_api_implementation.so' failed: link_dynamic_library.sh failed: error executing command
  (cd /ui/ncsa/qiyuelu1/.cache/bazel/_bazel_qiyuelu1/ce8d1fb9ff1b7a9dbbd225fe7f0c6f52/execroot/tensorflow-0.12.0 && \
  exec env - \
    LD_LIBRARY_PATH= .......

I searched online and found github bazelbuild/bazel #1867 and tensorflow/tensorflow issue #2266, #5617 and #5616, but it seems the changes suggested in these issue have already been implemented in the source code of version 0.12.0. I am not sure why this problem still exists.
Environment info
Operating System: RHEL 6.8
CUDA: 8.0
cuDNN: 5.1
Java: 1.8.0_112
gcc: 4.9.2
bazel: 0.4.3
tensorflow: v0.12.0 (Latest commit c62a66b on Dec 19, 2016)
What other attempted solutions have you tried?
I tried tensorflow version 0.12.head and v1.0.1-rc0, both of them can be built successfully. However, we do need this specific version 0.12.0.

Have you tried changing the TF_BINARY_URL link to the version 0.12.0? Or do you need that specific commit?
Example:
`# Ubuntu/Linux 64-bit, GPU enabled, Python 3.4
Requires CUDA toolkit 8.0 and CuDNN v5. For other versions, see "Installing from sources" below.
$ export TF_BINARY_URL=https://storage.googleapis.com/tensorflow/linux/gpu/tensorflow_gpu-0.12.1-cp34-cp34m-linux_x86_64.whl`
The variable becomes:
$ export TF_BINARY_URL=https://storage.googleapis.com/tensorflow/linux/gpu/tensorflow_gpu-0.12.0-cp34-cp34m-linux_x86_64.whl
And then you just have to follow the installation with pip. ;-)

@MicaelCarvalho Thanks for your response.
Unfortunately, we need to build this specific version from source because, as we tested, keras, using tensorflow as backend, has some problem with newer (0.12.head) versions. I have to roll back older version like 0.12.0.

@qiyuelu1 the proposed approach will install version 0.12.0... if you only need "older version like 0.12.0" it should work; the only situation for which it won't do the job is if you really need that specific commit.

@MicaelCarvalho Thanks for your response. Actually, installing binary directly doesn't work for me. I am porting tensorflow on a cluster which is running RHEL 6.8, its default gcc is 4.4.7 (I use 4.9.2 in my build as a module). I have to build from source and make modifications on CROSSTOOL.tpl etc. files, otherwise, will get glibc version complaints.

@girving Assign to you mainly because you have comments in the mentioned issues. Feel free to re-assign. Thanks.

@jmchen-g Please use the who-do-I-notify sheet.  I do not know about build issues.  @martinwicke.

@qiyuelu1 - can you post the full link error you get if you build with -s?

@r4nt Thanks for your response. This problem has been bypassed through updating keras+tensorflow to newer versions.
Thanks all for your help. I'm gonna close this session.


Extend register layout to 3 dimensions

Currently, only 2 dimensions are supported.

Does it make sense to construct registers with flexible 1D/2D/3D coordinates? E.g. 1D could be represented by a map from qubit index to len(p)==1 tuple:
{
  0: (-1,)
  1: (0,)
  2: (1,)
}

and a 2D register with a tuple of length 2:
{
  0: (-1,1)
  1: (0,0)
  2: (1,1)
}```

Does it make sense to construct registers with flexible 1D/2D/3D coordinates? E.g. 1D could be represented by a map from qubit index to len(p)==1 tuple:
{
  0: (-1,)
  1: (0,)
  2: (1,)
}

and a 2D register with a tuple of length 2:
{
  0: (-1,1)
  1: (0,0)
  2: (1,1)
}

etc.

@nmheim Yes, the idea of this issue is exactly what you are writing here.

Good to have but low priority. Access if useful.
@pimvenderbosch - can you please check if hardware/protocol teams need this. If this is supported in the hardware.

I checked with Protocols. 3D layouts are not on the hardware roadmap, and therefor such features won't be required in future. I'll close this issue.


Update documentation in develop branch (version 1.6)

The documentation is out of date with the latest changes done in develp/version 1.6

Hello,
I am currently migrating CSP from v1.5 to v1.6 and I have noticed that my i2c driver is now broken due to the changes in the interfaces (csp_i2c_if.h).
I was wondering if there is a plan to add some documentation about drivers implementation and if somebody could provide a reference implementation of a i2c driver updated to the csp version.
Thank you
Marco Bergamin

hi Marco,
We will update the Wiki pages to libcsp version 1.6, they explains how to implement a driver.


Add null check when fetching line string

See Microsoft/vscode#34466

Coverage decreased (-0.006%) to 69.94% when pulling 13d46476c896c7794deacd89f4d012c5ae25ddce on Tyriar:vscode34466 into 313d1e6fecd98a4a5e8dddebb832a6c4c16a828d on sourcelair:v3.


updated doc for OSS docker engine version

Proposed changes

Unreleased project version (optional)

Related issues (optinoal)

FATA[0000] Your engine version 1.12.3 OSS is too old.  UCP requires at least version 1.12.1 CS or 1.13.0 OSS.
Also, can you publish 1.13.0 for https://yum.dockerproject.org/repo/main/centos/7/Packages/

@amitsehgal what version of UCP were you trying to install when you saw this message?

I think this is probably inaccurate; we don't require both CS and open-source Docker for UCP as this PR implies. It's easy to have your environment improperly configured, my guess is that's what happened here. No response from the author for over a month. Closing.


ch13 softmax update

Hello @rasbt! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file ch13/ch13_part2.py:

Line 225:44: E128 continuation line under-indented for visual indent

In the file ch13/ch13_part3_lightning.py:

Line 265:1: E303 too many blank lines (4)


Test of directly syncing specific tide model subfolders

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

Comparison is base (eb04f0c) 87.42% compared to head (6352b0a) 87.42%.
 Consider uploading reports for the commit 1be81aa to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #86   +/-   ##
========================================
  Coverage    87.42%   87.42%
========================================
  Files            4        4
  Lines          628      628
========================================
  Hits           549      549
  Misses          79       79

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


MIGS = minimally invasive glaucoma surgery

137760 GLAUCOMA, PRIMARY OPEN ANGLE; POAG
Pillunat LE, Erb C, Jünemann AG, Kimmich F. Micro-invasive glaucoma surgery (MIGS): a review of surgical procedures using stents. Clin Ophthalmol. 2017 Aug 29;11:1583-1600. doi: 10.2147/OPTH.S135316. Erratum in: Clin Ophthalmol. 2018 Feb 05;12 :287. PMID: 28919702; PMCID: PMC5587165.
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5587165/
Minimally invasive glaucoma surgery (MIGS) is a group of surgical techniques used to treat glaucoma, a condition that can damage the optic nerve and lead to vision loss. MIGS procedures are designed to be less invasive than traditional glaucoma surgeries, which typically involve creating larger incisions in the eye.
MIGS procedures typically involve the use of specialized devices or instruments to increase the outflow of aqueous humor, the fluid that circulates inside the eye, and lower intraocular pressure. These procedures can be performed alone or in combination with cataract surgery, a common procedure to remove the natural lens of the eye.
Some examples of MIGS procedures include trabecular micro-bypass stents, which are small, flexible devices that are inserted into the drainage canal (Schlemm's canal) to increase outflow, like iStent, XEN and Presaflow.
MIGS procedures are typically performed under local anesthesia and can have shorter recovery times and fewer complications than traditional glaucoma surgeries. However, they may not be appropriate for all patients or all types of glaucoma, and your ophthalmologist can help determine if a MIGS procedure is an appropriate treatment option for your specific case of glaucoma.

added. Will be available in next release


fix install-cli doc

What type of PR is this?
/kind documentation
/kind cleanup

What this PR does / why we need it:
fix install-cli doc
Which issue(s) this PR fixes:
fix https://github.com/karmada-io/karmada/issues/4012
Special notes for your reviewer:

/assign @RainbowMango

You are absolutely right, much appreciated the quick fix, Thank you!
/lgtm

sure, done.


feat: add reuse and order ignored packages

As title says

reuse has an APACHE license so it works for us


Unclear how to ingest an exported table from Azure Blob Storage

Are there some missing steps in the process I am following?
The code snippet is below:
.export async compressed to parquet 'https://stn.blob.core.windows.net/kustobackup?sv=2020-04-08&st=2021-05-31T06%3A11%3A19Z&se=2021-07-01T06%3A11%3A00Z&sr=c&sp=racwl&sig=XXX'
with (namePrefix=parquetpatients, fileExtension=parquet) <|phidocument
.ingest into table tempdocs 'https://stn.blob.core.windows.net/kustobackup/parquetpatients_1_88c60187376a4374be30f6fa0a7c6119.gz.parquet?sv=2020-04-08&st=2021-05-31T06%3A34%3A11Z&se=2021-06-01T06%3A34%3A11Z&sr=b&sp=r&sig=YYY'
The export succeeds and creates a blob file which I then proceed to try and import in the second line into a separate table called tempdocs (which already exists).
I get the following errors:
"OperationId": c3af2c8a-f109-4172-94be-75d0b40e073f,
"Database": deidpatientsm1,
"Table": tempdocs,
"FailedOn": 2021-05-31T06:51:26.9248833Z,
"IngestionSourcePath": https://stn.blob.core.windows.net/kustobackup/parquetpatients_1_88c60187376a4374be30f6fa0a7c6119.gz.parquet,
"Details": BadRequest_InvalidBlob: OS Error -2146233088 (FormatMessageW() returned error 15105) (os error -2146233088),
"FailureKind": Permanent,
"RootActivityId": c2662e75-cd59-43bd-bc71-e14598a7f6c8,
"OperationKind": DataIngestPull,
"OriginatesFromUpdatePolicy": 0,
"ErrorCode": BadRequest_InvalidBlob,
"Principal": aaduser=xxx,
"ShouldRetry": 0,
"User": ,
"IngestionProperties": [Format=Csv/not-mandatory, IngestionMapping=[ToStringEmpty], ValidationPolicy=[Options=ValidateCsvInputConstantColumns, Implications=BestEffort, IsDetailedErrorReportingEnabled=False], Tags=[ToStringEmpty], IngestIfNotExists=[ToStringEmpty], ZipPattern=[null]],

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

ID: ae4cba9e-9d0a-0650-317a-09806b9bb29e
Version Independent ID: f7fa7b58-5735-96e2-ce45-1659232579dd
Content: Export data to storage - Azure Data Explorer
Content Source: data-explorer/kusto/management/data-export/export-data-to-storage.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

@GeorgeJoy Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.
@YaelSchuster Could you please look into this issue?

@GeorgeJoy , the article you reference only talks about export to storage.
Ingest from storage is explained in https://docs.microsoft.com/en-us/azure/data-explorer/kusto/management/data-ingestion/ingest-from-storage and this article mentions and links to the document covering "ingestion properties".
In your case what's missing is format specification. Please let us know if you think the article on ingestion from storage is not informative enough

thanks for the response. I tried creating an ingestion mapping and using that and also specifying format="parquet" as below. I am still getting error BadRequest_InvalidBlob. I guess I'm looking for a way to serialize a table1 into Azure Blob and later ingest it into another table2. I expect table2 to be identical to table1.
.ingest async into table tempdocs
h'https://stdxx.blob.core.windows.net/kustobackup/parquetpatients_1_88c60187376a4374be30f6fa0a7c6119.gz.parquet?sv=2020-04-08&st=2021-06-05T12%3A16%3A02Z&se=2021-07-06T12%3A16%3A00Z&sr=b&sp=r&sig=xxx'
with ( format="parquet", ingestionMappingReference = "ParquetMapping")

hi
can you try naming the files *.parquet.gz
and try again?

@ohadbitt , tried that and got similar error. Reported full errror from .show ingestion failures below (xx modified for privacy):
"OperationId": 587df0e8-fd64-48e9-9577-85d015b519b5,
"Database": deidpatientsm1,
"Table": tempdocs,
"FailedOn": 2021-06-08T12:03:07.308657Z,
"IngestionSourcePath": https://stxx.blob.core.windows.net/kustobackup/parquetpatients_1_8xx.parquet.gz,
"Details": BadRequest_InvalidBlob: HResult: 0x80131501, source: (OS Error -2146233088 (FormatMessageW() returned error 15100) (os error -2146233088)),
"FailureKind": Permanent,
"RootActivityId": 96bd7814-7704-49d6-a0a3-9fc7cbeb96f1,
"OperationKind": DataIngestPull,
"OriginatesFromUpdatePolicy": 0,
"ErrorCode": BadRequest_InvalidBlob,
"Principal": aaduser=xxx,
"ShouldRetry": 0,
"User": ,
"IngestionProperties": [Format=Parquet/mandatory, IngestionMapping=[{"column":"PatientId","datatype":"","Path":"$.PatientId"},{"column":"SourceHCSName","datatype":"","Path":"$.SourceHCSName"},{"column":"InstanceId","datatype":"","Path":"$.InstanceId"},{"column":"lastModified","datatype":"","Path":"$.LastModified"},{"column":"Birth","datatype":"","Path":"$.Birth"},{"column":"PersonDetailChange","datatype":"","Path":"$.PersonDetailChange"},{"column":"Diagnosis","datatype":"","Path":"$.Diagnosis"},{"column":"RiskAssessment","datatype":"","Path":"$.RiskAssessment"},{"column":"MedicationExposure","datatype":"","Path":"$.MedicationExposure"}], ValidationPolicy=[Options=ValidateCsvInputConstantColumns, Implications=BestEffort, IsDetailedErrorReportingEnabled=False], Tags=[ToStringEmpty], IngestIfNotExists=[ToStringEmpty], ZipPattern=[null]],

The server side reports the gzip stream is invalid:
Error GZIP header, first magic byte doesn't match
ICSharpCode.SharpZipLib.GZip.GZipException: Error GZIP header, first magic byte doesn't match

Are you sure the parquet zip stream is valid?
Can you get the file and decompress it?

@yogilad - the last experiment suggested by @ohadbitt renamed the file from *.gz.parquet to *.parquet.gz. I am guessing that's why the GZIP exception resulted.  (By the way, are you getting that extended error by looking up the operationId on the ADX back-end?)

Hello
Could you please provide us with a sample file failing ingestion

@georgejoy ping on the above question from Ohad

Hello
Could you please provide us with a sample file failing ingestion

I will try to reproduce with a non-sensitive workload

While doing some more experiments to extract a sample file, I found that a parquet file exported from a smaller data set (20K rows instead of the full 42K rows) can be successfully imported.
.```
export async compressed to parquet
with (namePrefix=phisample10k, fileExtension=parquet) <| phidocument take 20000

So maybe if the blob is over a certain size ADX cannot import it using .ingest? It would be good if this threshold was documented.

hi
can you still provide us with a file that fails ingestion ?

The error doesnt indicates it was an issue with the size - therefore we are asking for the file for more investigation.
Anyway you can limit the size of an exported file (sizeLimit):
https://docs.microsoft.com/en-us/azure/data-explorer/kusto/management/data-export/export-data-to-storage
Read here the answer and referred docs for more info on size limitation
https://stackoverflow.com/questions/61276069/increase-the-maximum-file-size-to-be-ingested-in-the-azure-data-explorer-adx-k

We have not heard back from the customer in 6 days. This is clearly a product question, not a docs issue.
#please-close


feat: 6001 - cached impact-co2.svg

What

Cached impact-co2.svg

Fixes bug(s)

Closes: #6001

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

Project coverage is 6.50%. Comparing base (4d9c7fc) to head (3a6a2a9).
Report is 555 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #6005      +/-   ##
==========================================
- Coverage     9.54%   6.50%   -3.05%
==========================================
  Files          325     441     +116
  Lines        16411   24982    +8571
==========================================
+ Hits          1567    1624      +57
- Misses       14844   23358    +8514

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


deploy CSI conditionally if client is running in provider mode or standalone

if client-op is running along side with ocs-op and running in provider mode or client-op is in it's own cluster then only we should deploy CSI.
part of [RHSTOR-5487]

Testing
--- storagecluster crd exists but not running in provider mode, just wait
# ko logs ocs-client-operator-controller-manager-6c584888c9-rxtmh | head
2024-03-21T05:53:23Z    INFO    preflight-validation    StorageCluster CRD exist, awaiting events on storagecluster CR for preflight validation
2024-03-21T06:01:23Z    INFO    preflight-validation    Value of .spec.allowRemoteStorageConsumers is not set
2024-03-21T06:01:23Z    INFO    preflight-validation    Value of .spec.allowRemoteStorageConsumers is not set
2024-03-21T06:01:23Z    INFO    preflight-validation    Value of .spec.allowRemoteStorageConsumers is not set
2024-03-21T06:01:23Z    INFO    preflight-validation    Value of .spec.allowRemoteStorageConsumers is not set
[...]

--- storagecluster crd exists and running in provider mode, start controllers
# ko logs ocs-client-operator-controller-manager-54ffc5b4c5-t6gmd
2024-03-21T09:42:21Z    INFO    preflight-validation    StorageCluster CRD exist, awaiting events on storagecluster CR for preflight validation
2024-03-21T09:43:56Z    INFO    preflight-validation    Value of .spec.allowRemoteStorageConsumers is set to true, preflight validation is completed
2024-03-21T09:43:56Z    INFO    preflight-probes        Stopped preflight probes serving new connections.
2024-03-21T09:43:56Z    INFO    preflight-probes        Preflight probes HTTP server shutdown complete
2024-03-21T09:43:56Z    INFO    setup   starting manager
2024-03-21T09:43:56Z    INFO    controller-runtime.metrics      Starting metrics server
2024-03-21T09:43:56Z    INFO    controller-runtime.metrics      Serving metrics server  {"bindAddress": "<IP_ADDRESS>:8080", "secure": false}
2024-03-21T09:43:56Z    INFO    starting server {"kind": "health probe", "addr": "[::]:8081"}
[...]

--- storagecluster crd doesn't exist in the cluster, start controllers
# ko logs ocs-client-operator-controller-manager-76557ccb77-vcv9b
2024-03-21T11:59:56Z    INFO    preflight-validation    StorageCluster CRD doesn't exist in the cluster
2024-03-21T11:59:56Z    INFO    setup   starting manager
2024-03-21T11:59:56Z    INFO    controller-runtime.metrics      Starting metrics server
2024-03-21T11:59:56Z    INFO    controller-runtime.metrics      Serving metrics server  {"bindAddress": "<IP_ADDRESS>:8080", "secure": false}
2024-03-21T11:59:56Z    INFO    starting server {"kind": "health probe", "addr": "[::]:8081"}
[...]

another point, it seems the deployment of csv waiting for nginx configmap to get mounted for client console and so we need to make it as an optional mount

\lgtm

/lgtm


RTSP is not defined

Hi
I'm trying to follow the steps of https://github.com/Streamedian/html5_rtsp_player/wiki/HTML5-RTSP-Player (It must be updated)
After a long work, I'm really close to have my rtsp player, but I received the next console error:
Uncaught ReferenceError: rtsp is not defined
at example.js:5
let mediaElement = rtsp.attach(document.getElementById('test_video'));
Could you help me to solve this issue?
Regards
Edit: By the way, I just want to use the player functionality, I don't want the websocket in the serverside.

Hi
Try to use example/streamedian.js
example.js in root folder is outdated. I'll remove it

where did you get example with
let mediaElement = rtsp.attach(document.getElementById('test_video'));
line?
It is outdated and removed from main repository

@kreopt the line and all the guide is in the link:
https://github.com/Streamedian/html5_rtsp_player/wiki/HTML5-RTSP-Player
Should I follow another start guide?

@kreopt I tried using the "example.js" from the main repository, but it doesn't work for me.

I've tried using the example/streamedian.js. I receive these errors:
Client paused
logger.js:36 Client started
index.html:1 Not allowed to load local resource:
blob:null/ee91058b-d7b2-48a4-8032-e9b10304ad3f
websocket.js:247 WebSocket connection to 'wss://specforge.com/ws/' failed:
Error during WebSocket handshake: Unexpected response code: 502
Promise @ websocket.js:247
connect @ websocket.js:246
disconnect.then @ websocket.js:63
logger.js:36 [transport:ws] [ctrl] error
_log @ logger.js:36
error @ logger.js:45
ctrlChannel.onerror @ websocket.js:301
logger.js:36 [transport:ws] [ctrl] close. code: 1006 unknown reason
I just want to stream my RTSP.
Regards
El jue., 10 ago. 2017 a las 5:23, kreopt<EMAIL_ADDRESS>escribió:

Hi
Try to use example/streamedian.js
example.js in root folder is outdated. I'll remove it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Streamedian/html5_rtsp_player/issues/48#issuecomment-321499297,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOcELbtb4Xv7L2xnA7H_9dIfZ_Ea0W9ks5sWsv_gaJpZM4OxOzF
.

wss://specforge.com/ws is not very stable place to test cause it is public and can be temporary unavailable. It is better to get your copy of proxy and test locally.

Does anybody complete streamedian.js sample with RTSP ?
If yes, can you share the code please?

Does anybody complete streamedian.js sample with RTSP ?
What do you mean?

I've followed the instructions on Stremeadian page and struggled on a Server Side :
Server side
Install websocket proxy
**Sign up and install websocket proxy from http://streamedian.com
Register domain you want to use this player
Download license file to your server**
Note that this package depends on systemd and gcc5+ runtime so it can be installed only on recent distribution versions.
Configure port and license file path in /etc/ws_rtsp.ini
This port should be open in your firewall. Also you can pass request to this port from your proxy. (for example: http://nginx.org/en/docs/http/websocket.html)
Run it

I mean if somebody has already done that, can you supply a server side code ?
I want to test it locally

Any possible way to change the http port number


feat: use treesitter Query for building Positions

Playing with the tree-sitter is quite fun :)
I figured tree-sitter comes with a built-in query feature, I'm not sure it is more performant than the current implementation, I didn't test it, but I'd place my bet on that.
I used this for reference:
https://tree-sitter.github.io/tree-sitter/using-parsers#pattern-matching-with-queries
Also their unit tests https://github.com/tree-sitter/tree-sitter/blob/master/cli/src/tests/query_test.rs
I didn't remove the dead code, I just kept it as a comparison, but accordingly with the tests, they are behaving the same. This implementation also contains the fix for #11
This implementation doesn't address the suggesting when editing an attribute in the middle of other hx-*
For instance:
<div hx-get="/foo" hx-target="<cursor here>" hx-swap="next">

you are freaking awesome
i love queries (i just didn't want to have to relearn everything for them)  this looks perfect.

the tests look great and the queries look great too


[python] Edit pyenv virtualenv-init check to support non-brew installations

Fixes #1428
Proposed Changes
If the pyenv virtualenv plugin is installed using the pyenv-installer
app script or directly via a Git clone, then the pyenv-virtualenv-init
executable, that the pyenv virtualenv-init command uses, will not
exist in the user's PATH and therefore cannot be found using $commands[pyenv-virtualenv-init].
Installing the pyenv-virtualenv plugin in this manner is common among Linux users.
Using the pyenv commands command, which lists all commands pyenv can
run, we can find if the virtualenv-init command is available to pyenv
and subsequently the virtualenv plugin, without relying on
pyenv-virtualenv-init to exist in the user's PATH.

Personally, I am not a fan of | and grep noises since these can be mostly achieved directly using zsh.
That said, I am going ahead and merging the PR  because this is a decent attempt at fixing the initial problem (which is an obvious bug to me) with awesome analysis in commit log.
I'll make a followup PR little later with relevant tweaks.

One other change I'd like to suggest: the check for the :prezto:module:python:virtualenv' initialize zstyle seems to be wrong. Currently, it is using
zstyle -T ':prezto:module:python:virtualenv' initialize
This is returning true even if the relevant line in the .zpreztorc is commented out. It should be
zstyle -t ':prezto:module:python:virtualenv' initialize 'yes'
But this PR just got merged, so does this need to be a new PR now?

And thanks @zachwhaley for the first PR! ✨

@gpanders It's a bit confusing, but the values in zpreztorc are supposed to be the defaults, even when commented out, so I believe this is expected behavior. Thanks for mentioning it though!

@belak Interesting. So where is that value set, then, if not in zpreztorc?

As inlined doc says, it defaults to yes and thus enabled.
So to disable, you'll have to explicitly specify no in ~/.zpreztorc:
zstyle ':prezto:module:python:virtualenv' initialize 'no'

This has been done to stay compatible with previous behavior (when it wasn't backed by a zstyle setting).


Rename abstract classes to emphasize that they are abstract

We have various classes that are abstract that either show they are abstract in their name or hide that fact until one dives into the class. As such, we have 3 paradigms for abstract class names: 0) Abstract*; 1) Base*; and 2) Surprise Me!
Since the JoindIn-Web2 project has Base* as its manner of naming abstract classes, this PR follows suit.

Can one of the admins verify this patch?

ok to test

Yay, conflicts :D.
@ericpoe - do you think you'll have a chance to look through this? No worries if not, I'll try to find time to do so :)

Of course there are conflicts! Two git commits enter, only one shall be victorious. @liam-wiltshire I'll take a look this evening.

Build finished.

@liam-wiltshire merge conflicts disappear when merging master into this branch. Woo hoo!


VS Code: Cody allows blank message to post and also respond.

Version
v1.10.0
Describe the bug
Platform: MacOS
Steps to reproduce:

Have Cody plugin installed in your VS Code editor
Login to Cody with Free/Pro user
Open New Chat
Hit enter twice

Observed behavior:
Blank message will be posted and Cody will also respond.
Expected behavior
Blank message shouldn't be allowed.
Additional context
Video link:
https://drive.google.com/file/d/1XtBcy_orLbS2Gjm4W-C7LPTmlhJoc7eZ/view?usp=drive_link

I can't repro this, but I can repro a similar issue: if you press Enter in the empty textarea, it types a newline and continues doing so. I believe we are not properly preventingDefault the event in this case. That would also lead to this bug, and it's possible the solution is the same.

I can repro
https://www.loom.com/share/e1a0051027824d51b35cabeb977455ef?sid=d3bb489e-c1ca-4e71-8f31-44d12e0b6700


TextCreate and/or TextSetShown not working

I'm at the point of giving up on this library and going to roll my own DX9 injection.
I'm not sure why this DX9 Overlay API isn't working. I've back-tracked to even the simple "hello world" AutoHotkey script to test with the GFXTest app (GFXText.exe). The samples don't work.
1.) The dx9_overlay.dll is loading properly.
2.) The WatchProcess function is being called, and overlay_ids are getting the results from CreateOverlays function.
3.) The process name is correct. Verified in the sample code after compiling it using AutoHotkey ANSI 32-bit. And confirmed that process is running in the task manager.
4.) Set GFXTest to DirectX9, and confirmed it's running just fine.
5.) No other DirectX 9 hooks are active on my PC I think, but not sure how to confirm this?
6.) Just to be sure I downloaded and installed DirectX 9.0c runtime library (last released by Microsoft) since my system is using DX11. I confirmed that DX9.0C now exists on the system with a legacy DXText.exe tool (it's dxdiag.exe these days).
7.) I tried running GFXTest.exe in 2D mode, 3D mode, 640x480, 800x600, and 1024x768.
8.) The hello world.ahk script was compiled with ANSI 32-bit version of AutoHotkey and compiled and runs just fine as an executable. It includes overlay.ahk just fine. It loads the dx9_overlay.dll file just fine. I can use Alt-X to exit it. But Alt-T never shows the Hello World text in GFXTest.exe
Thoughts to troubleshoot this? Thanks.

Troubleshooting this more, and still no success.
1.) Extracted the zip archive into its own folder structure
2.) I tried most of the samples included for AutoHotkey. Using ANSI 32-bit compiler, compiled each.
3.) Downloaded the GFXTest app you tested this library with. Using that for process/window. Making sure it was set to DX9. Tried 2d and 3d modes, different resolutions, etc.
4.) Current user is part of Administrator group, but also tried running the compiled samples as actual Administrator.
5.) The executables for each sample loads, the key bindings (hot keys) work, and in the framerate one the dialog window appears.
6.) Nothing is every overlayed. No error message at all, no nothing. I think something has changed in DirectX or Windows that completely breaks this overlay from working. So is this project dead?

Hello,
you pointed out that the current Windows user is a part of the administrator group but did you try running the compiled executable with administrator privileges?

Thanks for the reply, good to know you're still around here. :-) The Administrator group has Administrator privs. I also tried running the compiler as the Administrator user itself.

Hm, this sounds really strange. Did you try running the example C++ or C# application? Maybe AutoHotKey is the problem.

Good idea. I'll try one of those to test with C# and report back.


Firebase ios, cloud message doesn't show when app in background, only if app in foreground,

I'm using flutter dependancies:
firebase_messaging: ^7.0.3
flutter_local_notifications: ^3.0.1+6
I send firebase cloud message like this:
{
	"to": "/topics/demo_ios"
    "notification" : {
      "body" : "Hi there now now",
      "title" : "Wow!",
      "sound": "default"

    },
     "priority": "high"
	"data": {
    	"title": "Hi there yes",
    	"body": "Wow",

   }
}

In ios the notification shows when the app is in the foreground, however when in the background it doesn't show at all. I tried reading and fixing many things, but is still doesn't work. I set background fetch, remote notificaitons, and background processing from xcode, and it still doesn't work.
I set FirebaseAppDelegateProxyEnabled to false in the plist file, but it didn't help.
I request permission:
    _messaging.requestNotificationPermissions(
      const IosNotificationSettings(sound: true, badge: true, alert: true, provisional: false)
    );
    _messaging.onIosSettingsRegistered.listen((event) {print('2124: ios Setting registered');});

Here is flutter doctor:
[✓] Flutter (Channel beta, 1.24.0-10.2.pre, on Mac OS X 10.15.7 19H2 darwin-x64, locale en-GB)
    • Flutter version 1.24.0-10.2.pre at /Users/reuvenberman/Developer/flutter
    • Framework revision 022b333a08 (12 days ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/reuvenberman/Library/Android/sdk
    • Platform android-30, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio 4.0
      Preview.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio 4.0 Preview.app/Contents
    • Flutter plugin version 50.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.48.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • Pixel 3a (mobile)          • 97KAY112F3                • android-arm64 • Android 11 (API 30)
    • Reuvens iphone 11 (mobile) • 00008030-000571380198802E • ios           • iOS 14.0.1

! Doctor found issues in 1 category.

Same Here in IOS 14

i am also facing same issue.

Hi @bermanapps
This works for me using the official example code on the latest master 1.24.0-8.0.pre.375 with
  firebase_core: ^0.5.0+1
  firebase_messaging: ^7.0.3

flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-8.0.pre.375, on macOS 11.0.1 20B29 darwin-x64, locale en-GB)
    • Flutter version 1.24.0-8.0.pre.375 at /Users/markus/development/flutter_master
    • Framework revision a7f5fd5360 (24 hours ago), 2020-11-30 13:14:13 +0100
    • Engine revision 20caf54969
    • Dart version 2.12.0 (build 2.12.0-76.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/markus/Library/Android/sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: /Users/markus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.6953283/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Users/markus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/201.6953283/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      � https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      � https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] Connected device (4 available)
    • Nevercode’s iPhone (mobile) • b668e524315069f3db3661ac11ff1f66afafebdb • ios            • iOS 14.2
    • macOS (desktop)             • macos                                    • darwin-x64     • macOS 11.0.1 20B29 darwin-x64
    • Web Server (web)            • web-server                               • web-javascript • Flutter Tools
    • Chrome (web)                • chrome                                   • web-javascript • Google Chrome 87.0.4280.67

• No issues found!

Could you check if the registered permissions are the same as the ones you requested?
Could you please provide your flutter pub deps -- --style=compact?
Thank you

Hi, I am using these permissions like you said:
firebase_core: ^0.5.0+1
firebase_messaging: ^7.0.3
Its still not working.
I'm sorry, what do you mean flutter pub deps -- style=compact
I'm new to flutter, what and where is that info?

Well, the solution was quite simple, and a mistake on my behalf.
As the documents at pub.dev state:
Generate the certificates required by Apple for receiving push notifications following this guide in the Firebase docs. You can skip the section titled "Create the Provisioning Profile".
Here is the link:
https://firebase.google.com/docs/cloud-messaging/ios/certs
I forgot to do these steps. After doing this, it worked

Thanks for the update
Closing this as solved based on your last comment.

@bermanapps I got the same error could you tell me how you can solve it?


feat (cheatcode) TOML support

Component
Forge
Describe the feature you would like
The TOML format is used by foundry itself and is quite an elegant configuration format.
it would be great if we could have TOML parsing facility like we do for json via vm cheatcode
This would allow us to also read the foundry toml file and this way we could extend it with new parameters
Additional context
No response

+1, would love this as well. Especially useful for interacting with Noir circuits.


Could not resolve host: github.com

I get the above error when trying to install pyenv on a new machine using
curl https://pyenv.run | bash

any idea why?
I could install pyenv successfully on other computers a few months earlier. Maybe it's because github.com changed some bash based authentication settings? (although one should not need login credentials here)

Could you post the error? I'm not seeing anything amiss here.

Could you post the error? I'm not seeing anything amiss here.

$ curl https://pyenv.run | bash
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   285  100   285    0     0    283      0  0:00:01  0:00:01 --:--:--   283
Cloning into '/Users/liming/.pyenv'...
remote: Enumerating objects: 751, done.
remote: Counting objects: 100% (751/751), done.
remote: Compressing objects: 100% (505/505), done.
remote: Total 751 (delta 390), reused 345 (delta 151), pack-reused 0
Receiving objects: 100% (751/751), 414.23 KiB | 461.00 KiB/s, done.
Resolving deltas: 100% (390/390), done.
Cloning into '/Users/liming/.pyenv/plugins/pyenv-doctor'...
fatal: unable to access 'https://github.com/pyenv/pyenv-doctor.git/': LibreSSL SSL_connect: SSL_ERROR_SYSCALL in connection to github.com:443
Failed to git clone https://github.com/pyenv/pyenv-doctor.git

Not a pyenv-installer problem.
https://github.com/openssl/openssl/blob/1c0eede9827b0962f1d752fa4ab5d436fa039da4/doc/man3/SSL_get_error.pod says that this could happen on an unexpected EOF on the encrypted stream -- i.e. if there are problems with transferring the data.
https://stackoverflow.com/questions/49345357/fatal-unable-to-access-https-github-com-xxx-openssl-ssl-connect-ssl-error/51433565#51433565 indicates that this can happen if you use a proxy and it has problems.


Alert user if they try to navigate away from PD

As a protocol designer, I would a warning before I navigate away from a protocol with unsaved changes, so that I don't accidentally lose my changes.
Acceptance criteria

Display a dialogue box is user attempts to navigate away from protocol design
Only display dialogue box if user has made changes since last file export
User has option to continue (and proceed away from PD) or cancel (and return to PD)

Design and Copy

Title: Unsaved changes
Body: Export your file to save changes before leaving protocol designer

FYI @howisthisnamenottakenyet added a 'continue' and 'cancel' button to this story.


[Event Request] Table 5767 "Warehouse Activity Line" - OnBeforeCheckInvalidBinCode

We are migrating our code from NAV 2018 to BC16.4. In the Table 5767 "Warehouse Activity Line", in the procedure CheckInvalidBinCode, we would like to condition the use of this procedure. To do so, we need a IsHandled flag and an event publisher, like this :

With this integration event :

Do you think it could be possible ?
Thank you for your work.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Linux (ubuntu) injection problem

Currently trying to inject the latest build of Osiris with the latest release of CS2 and upon injection the game immediately crashes. I haven't seen any issues similar and any help would be greatly appreciated.

might be duplication of #4169

works on my machine -fedora user


Search results disappear

Issue Type: Bug
Step to reproduce:

Search
Wait a bit after search finishes

For some reasons, the search results disappear after a little bit of time passes.
VS Code version: Code 1.26.1 (493869ee8e8a846b0855873886fc79d480d342de, 2018-08-16T18:38:57.434Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-2687W v4 @ 3.00GHz (48 x 3000)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
63.88GB (42.58GB free)

Process Argv
C:\Users\mmkakulp\AppData\Local\Programs\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

doxdocgen
csc
0.3.3

arm
dan
0.2.0

gitlens
eam
8.5.6

python
ms-
2018.7.1

cpptools
ms-
0.18.1

sublime-keybindings
ms-
4.0.0

Not sure what you're seeing, can you include a screenshot or something?

Before:

After:

I think it triggers when I start building my project.

Building is done through a Python script from within the source that executes the build and post-processing tools. It's a large chain of events, but I can tell you that it reads and modifies files within the source directory.

Not these files, no.
Would this happen if a single file gets deleted or modified?

Yes. That would cause this, I can't think of anything else.

I think that surely your build process is regenerating those files, or something similar.


How to integrate Anugalr LS with your package?

In 'the link bellow' there is a guide for installing Angular LS for sublime, but it states that I need a custom sublime plugin for completions in HTML files.
-> the link bellow <-
Question is how to get Angular suggestion in HTML files?

That documentation likely refers to Microsoft's Typescript package (called "Typescript" in Package Control). The instructions (install the angular service next to your typescript dependency) sound correct, give it a try!
The LSP package does work with the tslint language service in javascript/typescript-langserver, but my guess is no one has tested if the angular service works.

Well thank you for your time :)
Tried to set it up, it didn't work out. Not your fault, so I will close this.


Split jobs that contain many gates

Today we split jobs that contain too many circuits. We obtain the maximum possible number of circuits from the backend configuration.
Another, informal threshold on the job size, is the total number of gates. Job submission crashes if this number is too large (see https://github.com/Qiskit/qiskit-ibm-provider/issues/375). Specifically related to qiskit-experiments, we exceed this number for typical RB experiments, when executed in parallel on all qubits of a 27Q device.

This should solve #537. Do you know if the number of gates is the same for all devices, or is there a way to get this info for each device?

My understanding is that it's not a "formal" constraint, therefore you won't find information about it at the devices. In fact, the issue that I opened in qiskit-ibm-provider asks to formalize it.
I didn't check for different devices.
My guess is that it's a timeout exception, located around 6.5 minutes. As such, it probably even depends on user's connectivity, so certainly not something that we can predict exactly.

Is there any update from the provider side on the formal constraints? If not, perhaps we can close this since the user can customize splitting now.


Text input field aligment

Thank you @marcofugaro for maintaining this theme!
I notice since the last update that the text input fields in the bottom panel are not correctly aligned with the labels:

To fix the alignment, I set content_margin: 9 instead of 3 here https://github.com/kkga/spacegray/blob/master/Spacegray.sublime-theme#L807.
But it looks like this line has not been touch since 2004 (https://github.com/kkga/spacegray/blame/master/Spacegray.sublime-theme#L807).
I would be glad to create a PR but I am not sure if I correctly identified the issue.
I am using ST3 on MBP (Retina display if it matters) with macOS 10.12.1.
Here is Sublime settings:
{
	"color_scheme": "Packages/User/SublimeLinter/Otto Tomorrow (SL).tmTheme",
	"draw_minimap_border": false,
	"enable_tab_scrolling": false,
	"ensure_newline_at_eof_on_save": true,
	"folder_exclude_patterns":
	[
		".sass-cache",
		".tmp",
		".git",
		".nyc_output",
		"bower_components",
		"build",
		"docs",
		"dist",
		"node_modules"
	],
	"font_face": "DejaVu Sans Mono",
	"font_size": 10,
	"highlight_line": true,
	"ignored_packages":
	[
		"Vintage"
	],
	"indent_guide_options":
	[
	],
	"open_files_in_new_window": true,
	"scroll_past_end": true,
	"spacegray_tabs_font_small": true,
	"spell_check": false,
	"tab_size": 2,
	"theme": "Spacegray.sublime-theme",
	"translate_tabs_to_spaces": true,
	"trim_trailing_white_space_on_save": true,
	"word_wrap": true
}

Could you let me know how to make the same edit and apply it? I am interested in the same fix. If possible, could you provide the steps to apply the fix or the fixed theme?

To apply the fix:

Go to Sublime Text > Preferences > Browse Packages.
Open /Theme - Spacegray/Spacegray.sublime-theme
Locate this line https://github.com/kkga/spacegray/blob/master/Spacegray.sublime-theme#L807
Replace content_margin: 9 by content_margin: 3

Thanks for letting me know. I applied the change you mentioned, albeit differently because I was not able to find .sublime-theme. I am guessing that since this was installed with package control, I have Theme - Spacegray.sublime-package. What I needed to do was to convert that to a .zip, extract the sublime-theme and make the change. Then drop the changed file with a different name under packages.
Finally, I see the fix, however, this time the search bar has a white background. Did you notice that as well?
Anyway really interested in this fix since I am kind of OCD :)
Thanks!

Good catch! But that property has effect on the padding of every input field in sublime (say Goto line or Goto File or Quick Switch Project) so that's the thing to consider.
Also I've tried the fix, and found out that actually 7 is the number which correctly aligns the text for me. (Tested with the search bar more squashed possible)
Why instead did you choose 9 @pioug?
Here's how it looks with the fix:

This is how content_margin: 7 looks for me:

And content_margin: 9

Alright, found out 6 was the right number which aligns the baseline of the text when the search panel is at its lowest height. Thanks for your help, fix out in next patch release


Autorest Regen Preview to 3.0.0-alpha.20240223.2 by Michael Nash from refs/pull/4271/head

Triggered from https://github.com/Azure/autorest.csharp/pull/4271

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


Feature: Stereo Balance Presets

@ebaauw Thanks a million tons for this plug-in!!!
Would it be possible to add a feature to include 2 or 3 balance presents for a stereo pair?
Why I need this: I have two different listening positions in my living room.
I know this can be tricky given the fact that stereo pairs can be created and separated dynamically, plus it may require complicated changes in the config.json, and I see that you are (rightly) averse to changes/additions to the config.json. So, I am making this request hoping youll be able to come up with something sometime! and solve this annoying process, to have to change the balance every time I shift listening position, in the sonos app. Perhaps a siri command or scene would be stereo nirvana!

I've been wanting to support Balance for a long time, see #7, but it's quite challenging:

There isn't a Balance endpoint on the zoneplayers.  Instead they seem to expose three Volume "channels", Master, RF and LF.  The Sonos app adjusts the RF and/or LF channel volume when changing the Balance.  A balance of 0% (middle) sets both channels to 100%; a balance of -100% (left) sets the right channel to 0%; and a balance of 100% (right) sets the left channel to 0%.;
I still haven't been able to figure out how to determine if a zone supports Balance.  All zones seem to report the three Volume channels, irrespective of whether they support Balance.  It's not just supported for stereo pairs, but also for the Connect.  Counter-intuitively, it's not supported for the Playbar.

Ad 1) In line with the Sonos app, I've exposed Balance in the Speaker service as a percentage from -100% to 100%, in steps of 5%.  As this is a custom characteristic, neither Apple's Home app nor Siri support this.  However, you should be able to include this in a HomeKit scene using Eve, and then recall the scene from Siri.
Ad 2) I added logic to see if a zone is has slave zoneplayers.  Afaik this is only the case for stereo pairs and home theatre setups.  I already whitelisted the home theatre players (like PlayBase and PlayBar), so I won't expose Balance for home theatre setups.  Also, I added a whitelist for zoneplayers that natively support stereo channels, like the Sonos Connect.

In v0.3.14.

@ebaauw, thank you for the patient response and for pointing me in the right direction. Apologies for not writing in earlier.
I haven't tried any other home app except the one from apple. Will give it a try and report back.
And thanks for this great plugin!


jupyter and ipython code-blocks

Add support for >>> blocks outside of code-blocks
Add ipython as identifier for python blocks

Thanks


Error matching on a struct in a union

Use case: I'm working with FFI calls where an argument can either be a pointer to a struct or NULL (where it will then use defaults). I set up a variable as either (struct | None) but I'm running into a compiler error when trying to craft the appropriate MaybePointer (I get a similar error when trying to cast it with an as also). It is quite possible that I'm going about this the wrong way (in which case a pointer in the right direction would be appreciated).
Pony Playpen link: http://pony-playpen.lietar.net/?gist=4d388470321b9585cbb33f22f9578b74
Code:
struct Z
  var a: U32 = 0

  new create() => None

actor Main
  new create(env:Env) =>
    let arg: (Z | None) = None

    let y = match arg
      | let x: Z => MaybePointer[Z](x)
      else
        MaybePointer[Z].none()
      end

Error:
0.14.0 [release]
compiled with: llvm 3.9.1 -- cc (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609
Error:
main.pony:11:9: this capture violates capabilities
      | let x: Z => MaybePointer[Z](x)
        ^
    Info:
    main.pony:8:15: match type: (Z ref | None val)
        let arg: (Z | None) = None
                  ^
    main.pony:11:9: pattern type: Z ref
          | let x: Z => MaybePointer[Z](x)
            ^

structs can't be part of unions because they don't have a type descriptor. I'm not sure why the compiler isn't telling you that, though.

I'll keep the issue open for the error message.

@Praetonus Thanks for the quick response. I'll refactor my code accordingly.

What should the error message for this be?

It should be something like a struct cannot be part of a union type.


Fix some extensions and syntax highlighting on Razor Pages article

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@tdykstra  how do we format Razor?

@Rick-Anderson Good question.  I'm not aware of a prescriptive guidance doc for that.

@DaleGardner  thanks


Don't use fragile indentation size for string indexing

It's fragile to take into account the indentation size of the Diesel-generated (or human-generated) schemas as a string indexing component. Better to trim() any indentation and use direct word indexes to search for tokens.
The tests will pass with #48, they were broken yesterday, but anyway, it would be nice if you could check this against your real-life examples.

LGTM