NodeAuditAnalyzer fails when hitting a 400 Bad Request where npm audit succeeds

Describe the bug
[DependencyCheck] [ERROR] NodeAuditAnalyzer failed on <redacted-path>/package-lock.json
[DependencyCheck] [WARN] An error occurred while analyzing <redacted-path>/package-lock.json' (Node Audit Analyzer).
[DependencyCheck] [INFO] Finished Node Audit Analyzer (0 seconds)
...
Could not perform Node Audit analysis. Invalid payload submitted to Node Audit API.

There are some similarities with #1632 and #1634, but those have been closed and we're still having issues with version 5.2.0, hence this ticket.
Version of dependency-check used
The problem occurs using version 5.2.0 of the CLI (but it was originally spotted using the Jenkins plugin).
To Reproduce
Steps to reproduce the behavior:

Put the [package-lock.json|https://gist.github.com/oliverlockwood/22c5862746af242a21f8a713573560a0] and [package.json|https://gist.github.com/oliverlockwood/43369c2ef88135914d4c47e423a3013e] into a folder.
Run npm audit -registry=https://registry.npmjs.org.  It succeeds and finds 81 vulnerabilities.
Point the NodeAuditAnalyzerIT.testAnalyzePackage() test to use the package-lock.json file in this folder, and run.  Boom.

Expected behavior

the Node Audit should succeed
the behaviour should be consistent between dependency-check and npm audit!

We just hit this issue too with version 5.2.1. Any solutions since reporting?

Sadly not. We worked around in a decidedly unsatisfactory manner, by disabling dependency checking for projects which suffer this bug :disappointed:
It would be nice to see an opinion from the project maintainers...

@oliverlockwood Agreed. We have hit this bug as well and are having to turn off the dep check in a few repos as well.

@oliverlockwood @jpeterso did 5.2.3 fix the issue for you? Because I'm still getting it with 5.2.4.

@skjolber 5.2.3 did indeed fix the issue for us.  But I haven't tested with 5.2.4 yet 😬

@oliverlockwood any chance you can test 5.2.4?  The only change between 5.2.3 and 5.2.4 that would have affected the node.js analysis was wrapping the caching in some error handling:
+++ b/core/src/main/java/org/owasp/dependencycheck/data/nodeaudit/NodeAuditSearch.java
@@ -39,6 +39,7 @@ import org.slf4j.LoggerFactory;
 import javax.json.Json;
 import javax.json.JsonObject;
 import javax.json.JsonReader;
+import org.apache.commons.jcs.access.exception.CacheException;

 import static org.owasp.dependencycheck.analyzer.NodeAuditAnalyzer.DEFAULT_URL;

@@ -99,8 +100,13 @@ public class NodeAuditSearch {
             LOGGER.debug("Not using proxy");
         }
         if (settings.getBoolean(Settings.KEYS.ANALYZER_NODE_AUDIT_USE_CACHE, true)) {
-            final DataCacheFactory factory = new DataCacheFactory(settings);
-            cache = factory.getNodeAuditCache();
+            try {
+                final DataCacheFactory factory = new DataCacheFactory(settings);
+                cache = factory.getNodeAuditCache();
+            } catch (CacheException ex) {
+                settings.setBoolean(Settings.KEYS.ANALYZER_NODE_AUDIT_USE_CACHE, false);
+                LOGGER.debug("Error creating cache, disabling caching", ex);
+            }
         }
     }

@skjolber Any chance you can provide a package.json and package-lock.json that we can use to test?

@jeremylong apologies for my last message which turned out to be incorrect. I have just double-checked and it turns out we moved straight from 5.2.0 to 5.2.4, which resolved the issue we had seen.  So I guess @skjolber's issue must be a different variant.

Ok, this can be closed. We generated a package-lock.json via yarn and npm audit failed because we had not run npm install. The error message was exactly the same as above, so I guess there is something to be gained by propagating a more exact error message.


feat: host opentelemetry memory utilization

Relevant information
This is adding the memory golden metric definition for hosts monitored by the opentelemetry collector.
Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

Comments
In Linux systems, the resulting value will be different than the one provided by the infra-agent host. The agent uses the MemAvailable and MemTotal fields from /proc/meminfo to compute the memory usage, while the Open Telemetry Collector relies on a sum of some already provided fields. We consider MemAvailable more fine-grained as it is computed in the kernel and it takes into account memory low watermarks.
Issue proposing MemAvailable as a new state for the Open Telemetry Collector: https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/7417

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


Theme Endpoint Broken

I tried to run resume serve and received an error stating that There was an error downloading your generated html resume from our server: Error: Unsuccessful HTTP response
I also tried to go to https://themes.jsonresume.org/theme/ and it wouldn't load.

Just came here to report that themes.jsonresume.org is down, but seems like this already known :)

Working on new serverless theme renderer here -> http://us-central1-jsonresume-58a79.cloudfunctions.net/theme/paper

https://github.com/jsonresume/registry-server/issues/105
https://themes.jsonresume.org/kendall


Bluetooth: controller: Fix XTAL advanced feature

Fix XTAL advanced feature by adding the missing
implementation to calculate and, retain or release the XTAL
clock source after a Bluetooth state or role is stopped.
Fixes #15817.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

Issue reproduced using modified samples/bluetooth/scan_adv:
0001-samples-bluetooth-scan_adv-modified-reproduce-HFCLK-.patch.txt

HFCLK is retained after bt_le_scan_stop() for a started continuous scanning.
After the fix in this PR:

The HFCLK is released gracefully after the call to bt_le_scan_stop().


Select language, not ordered either ascending or descending

How about ordering languages by ascending instead of this? It seems pretty random.

Spelling error

It's ordered to what I originally thought the popularity would be, based on some old analytics data from before we actually had any translations, plus keeping related languages together.
I could see going for alphabetization, but I could also see for just updating it to be in order of real CodeCombat popularity now. It's not very convenient for those languages lower on the list, and I wouldn't want to put Chinese (zh) or Russian (ru) all the way toward the bottom when we have so many players needing to choose those, especially since English locale code alphabetization isn't as easily grokked by some players using different character sets.
@Imperadeiro98 @nixel2007 @ArthurLu @mogsie @Martin005 what do you guys think about ordering by popularity versus ordering by alphabetical versus some other ordering?

I think that it would be important to have languages ordered by popularity. E.g. if some Russian (as mentioned) goes to your site and have to sroll all the way down for selecting his language, he is not going to be much happy about that.

I agree with @Martin005! It would be nice if it was ordered according to CodeCombat popularity, plus everyone could see if they have many people playing from their country.

Solid points here guys. Just throwing out ideas tho; how about a pre-selected top of, let's say top 5, and then alphabetical, divided by --------------- or something. As a player from a country with not that many players, I might get the idea that my language is not present. Have used this approach previously with pretty good feedback from users.
English (by populatiry)
Chinese
Russian
-----------
Albanian (alphabetical)
Latvian
Norwegian
Swedish

All this considering a smaller number of very popular languages.

:+1: for @matifol's suggestion about having the top 5 languages at the top, and then the rest.  I also find the non-alphabetical (seemingly random) order to be quite odd! :)

Yea, I think the @matifol suggestion for top 5 used languages and other ordered alphabetically.

Just wanted to offer variant as @matifol did. And I think the "5" number for number of languages is just what we need.

It seems like we all agree :)

Here are the language popularities for registered users:
74.20%	en-US
5.04%	zh-HANS
4.77%	ru
3.98%	es-ES
2.91%	fr
1.11%	en-GB
0.90%	pt-PT
0.84%	es-419
0.74%	ja
0.69%	de-DE
0.65%	pl
0.61%	zh-HANT
0.41%	nl-NL
0.29%	it
0.28%	ko
0.21%	hu
0.19%	th
0.18%	sv
0.18%	tr
0.18%	pt-BR
0.17%	uk
0.14%	da
0.14%	ro
0.13%	cs
0.10%	en-AU
0.10%	id
0.10%	vi
0.09%	bg
0.09%	ar
0.09%	nb
0.09%	fi
0.07%	sk
0.07%	el
0.06%	he
0.04%	lt
0.03%	ca
0.03%	sl
0.03%	de-AT
0.02%	nl-BE
0.01%	sr
0.01%	nl
0.01%	fa
0.01%	es
0.00%	de-CH
0.00%	ms
0.00%	hi
0.00%	zh-WUU-HANS
0.00%	pt
0.00%	nn
0.00%	zh-WUU-HANT
0.00%	de
0.00%	gl
0.00%	ms-BA
0.00%	ur
0.00%	mk-MK

Following (with difficulty) a similar alphabetization scheme to what Wikipedia's language sidebar uses, and keeping similar language groups together, then gives us 93.56% coverage with the top five language families (which is actually 8 choices):
English (US)
English (UK)
简体中文
繁体中文
русский
español (ES)
español (América Latina)
français
------------------------
العربية
български език
Català
Čeština
Dansk
Deutsch (Deutschland)
Deutsch (Schweiz)
Deutsch (Österreich)
Ελληνικά
فارسی
Galego
한국어
Bahasa Indonesia
Italiano
עברית
Język polski
Lietuvių kalba
Limba română
Magyar
Македонски
मानक हिन्दी
Bahasa Melayu
Nederlands (België)
Nederlands (Nederland)
日本語
Norsk bokmål
Norsk nynorsk
Português (Portugal)
Português Do Brasil
Српски
Slovenčina
Slovenščina
Suomi
Svenska
ไทย
Tiếng Việt
Türkçe
Українська Мова
اُردُو
吴语
吳語

We can probably delete English (AU), especially since we do very little localization for flavors of English. The biggest loser would be pt-PT, which is the 7th-most-popular individual language, but which would be down in spot #36. But overall, it's much better. Cool?

Yea, looks great.

Hm... Alphabetical list looks strange a little, but if it like wiki does, it's OK.
Also we could change one list of languages to 3-4 lists, like translate.google.com does - it can resolve the scroll-scroll-scroll problem.


Turn QTS academic year question into radio buttons

Screenshot

Change QTS award year to radio buttons - radio buttons are considered more accessible than selects
Add an ineligible option to give ineligible claimants a way to fail at this step

Just checking, have we considered how this change impacts the CSV export?

I just checked the CSV import, it looks like it's using the string value:


feat: disable activity workers

Reason for This PR
https://github.com/roadrunner-server/roadrunner/issues/2089
Description of Changes
Disable activity workers
License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.
PR Checklist
[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

[ ] All commits in this PR are signed (git commit -s).
[ ] The reason for this PR is clearly provided (issue no. or explanation).
[ ] The description of changes is clear and encompassing.
[ ] Any required documentation changes (code and docs) are included in this PR.
[ ] Any user-facing changes are mentioned in CHANGELOG.md.
[ ] All added/changed functionality is tested.

Changes need to be provided to https://github.com/roadrunner-server/pool/blob/master/pool/config.go#L42

Hey @cv65kr 👋🏻 Thank you for the PR 👍🏻

Changes need to be provided to

To skip workers allocation, after the initializing the pool in the config.go, check if that option to disable activity workers is active (set to true) and set NumWorkers to 0: link.

We can't change the default behavior for the pool.
In the pool (NewPool), last arg are options. You may introduce a new option, like, WithOverrideWorkersNum (something like that) and pass this option in the pool in the rr-temporal plugin. Options are applied after InitDefaults and before actual workers allocation.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 40.21%. Comparing base (c5c2e9a) to head (6cccd21).
Report is 33 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #592      +/-   ##
==========================================
+ Coverage   40.06%   40.21%   +0.14%
==========================================
  Files          14       14
  Lines         936      940       +4
==========================================
+ Hits          375      378       +3
- Misses        518      519       +1
  Partials       43       43

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


matrix-appservice-irc: init at 0.3.1

Motivation for this change
Provides IRC bridging for the Matrix chat network.
Things done

[ ] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Is it really necessary to check in more than 10000 lines for a single package?

I have very poor understanding of the npm packaging infrastructure, but this is certainly not the only package that includes a npm2nix generated file. I've experimented some but haven't been able to work out a way to get it to work with the default top-level one. If there's a better way, I'd love to hear about it.

If nodejs stuff requires 10,000+ lines per package we seriously need to reconsider how things are done on that front.

It looks like the NPM/nodejs infrastructure is being reworked in #16886 although recent activity is limited. Should this be blocked by it?

Should this be blocked by it?

As far as I'm concerned, yes. I'm not checking in 10,000+ lines of code for a single package. Somebody else might do it, however, my opinion is by no means authoritative.

No progress for over a year. We will not merge this in the current form but feel free to reopen if things have improved.


Separate Post Tags section

Describe the Feature
Similar to the way Tumblr allows users to tag posts, separate from the post's main text and character limit.
Currently, the low character limit discourages people from using tags to label things like content warnings or relevant information [like #art, #illustration, etc] because they cut into the post's character limit. It also can be a bit ugly to have a whole string of hashtags, but it's necessary in today's world if you want to get noticed.
An alternative would be to have a [still character limited] 'tag section' that would go beneath the post, allowing people to still mark content with relative hastags that can be muted or browsed, but it doesn't cut into the main post or look as unsightly.
This would greatly expand the capabilities of people to self-moderate, self-tag for content warnings, etc.
To avoid people abusing the tags to make the post longer, just implement another character limit in the tagging section. If you can't fit all your tags in 240 characters, that's too many tags.
Here's how posts currently look:

Here's how they COULD look:

Art post example:

Attachments
No response
Describe Alternatives
An alternative would be that Hashtags don't count for your character limit, but that's way more easily exploitable and doesn't change the fact that a bunch of hashtags on the main post still looks kinda gross.
Additional Context
No response

We do have outlined tags, it does exactly what you describe, but the use of hashtags has been rather mixed for curation that it hasn't exactly been a focus.

We do have outlined tags, it does exactly what's described here. It's been implemented but bsky.app just doesn't have any UIs for it. Mainly because the use of hashtags for curation has been rather mixed so it's not been a focus.

Wait- it does??? Could you direct me where to find it? :0

I like this as a way of separating metadata from prose content, a bunch of hashtags on the main post does look gross. If implemented it would be nice to add a user preference to hide hashtags on posts if you don't care about them.

Exactly- the tags would still apply to your filters/moderation, but you don't have to see them if you don't want. [Could also put a 1-2 line visual limit so it doesn't clog things up too much if you have them turned on]

We do have outlined tags, it does exactly what's described here. It's been implemented but bsky.app just doesn't have any UIs for it. Mainly because the use of hashtags for curation has been rather mixed so it's not been a focus.

That may be a chicken-and-egg problem. Many people avoid too many inline hashtags because they feel cluttered and mess with accessibility, but outlined tags generally have much higher uptake.
I think a good way to drastically increase uptake would be to have the UI present some of the user's most frequent recent (hash)tags to be added with one click/tap, and if that doesn't fill the space then the most frequent among the last ~100 they've seen while scrolling. I think it's fine to go by UI space rather than absolute count for something like this, but there also needs to be a way to indefinitely exclude certain tags from the suggestions overall.
Especially for e.g. gallery curation, being able to edit posts to add these retroactively would be very important, though, so I'm pretty sure that if it's a choice between this and editing then the latter would be more welcomed.

We do have outlined tags, it does exactly what's described here. It's been implemented but bsky.app just doesn't have any UIs for it. Mainly because the use of hashtags for curation has been rather mixed so it's not been a focus.

Wait- it does??? Could you direct me where to find it? :0

Certain alternative clients like deck.blue support creating and displaying them.
Even if a post has outlined tags, they won't be visible on bsky.app though, even though they do make a post show up in hashtag searches and direct tag searches.

Adding this feature to the app UI sooner better than later would do a lot for artists.
A lot of them avoid hashtags for aesthetic reasons, or due to being used to other social apps where too many tags in the caption penalizes their posts algorithmically. Causing them to smuggle their hashtags in the alt text instead, which backfires on accessibility. Outlined tags are in the post schema, 3rd party apps are making use of them being there, would love to see it first party as well.

Note that Mastodon has this feature, too, for an additional point of comparison


Sign up and login form responsive

Issue that this pull request solves
Closes: #182
Proposed changes
Brief description of what is fixed or changed
Sign up and login form are now not compressed by reducing screen width. adjusted the screen likewise.
Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface
Other information
Any other information that is important to this pull request

@sakshikhachane can you please ping me on slack once. Will have to discuss few stuffs related to the PRs

@sakshikhachane can you please ping me on slack once. Will have to discuss few stuffs related to the PRs


Postman collections generated from Swagger specs (2020-03-23 20:40)

AEP Swagger Specs to Postman Collections conversion (2020-03-23)
Note that this PR should now stabilize the Postman IDs (Collections, Folder, Request) so they no longer add noise to the request, unless something about the actual entity changed in the Swagger spec (for example, if the name of a Folder changes, or the path of Request). Changes in parameters, headers, accepts, etc. will not result in a change of postman id.

@tsilver  can you review when you have a moment.
As noted in the PR description, hopefully this will be the last one w/ the noise of the ever-changing postman id's which were previously random UUID's - now they are UUID's generated from Strings that represent canonical aspects of each Postman element (ex. getUuid(this.collection.name)) .. so unless these core attributes change for the element, this noise will not be present.


Partially load collection based on WHERE or LIMIT clause

Is your feature request related to a problem? Please describe.
Only load part of a collection with query instead of loading all and then filtering them by looping.
For example, only loading the dutch translations from table below:
Posts -> ContentTranslation (with column language) -> only load rows related to post WHERE language = en.
Describe the solution you'd like
A solution like this for ordering the collection:
https://github.com/mikro-orm/mikro-orm/issues/184#issuecomment-537825266, passing an init() argument to the createCondition method (https://github.com/mikro-orm/mikro-orm/blob/972c6e405738805b7204b783f0f609ce087a7b91/lib/entity/Collection.ts#L108).
Maybe have a seperate method to partially load a collection next to the initialize function. As well as setting a property that contains the total count and the count of data currently initialized.
Describe alternatives you've considered
Loading multiple queries seperately but would be great if it could be included in the initialize phase of a collection.

I am planning to support both partial loading of collections as well as propagating the orderBy to collections (and other relationships).
Will definitely fix the orderBy propagating issue from #184 before v3 release as now it really behaves unpredictably. Partial loading might come later in one of 3.x releases, as I would like to implement filter/sort collection API that would be compatible with both populated and not initialized collections.

Thank you for your quick response (again)!
That would be a really great addition. Especially for collections that may get very big; it would be great to filter them in advance with a query.

where and orderBy propagation available for testing in 3.0.0-alpha.24. I will also allow where and orderBy in Collection.limit() soon.

Test this fix now. Seems to work properly, will keep testing.

Collection.init() now supports where and orderBy too in 3.0.0-alpha.26.


Why is “ALLSKY_EXTRA” on the SD card

Discussed in https://github.com/AllskyTeam/allsky-modules/discussions/37

Originally posted by Adler6907 December 17, 2023
Modules like allsky_dewheater.py constantly write to /home/pi/allsky/config/overlay/extra which is not in RAM like /home/pi/allsky/tmp.
Wouldn't it be better to change ALLSKY_EXTRA="${ALLSKY_OVERLAY}/extra" to ALLSKY_EXTRA="${ALLSKY_TMP}/extra" in /home/pi/allsky/variables.sh?
Then in the example the file allskydew.json in /home/pi/allsky/tmp/extra would be constantly written.

Agree.  Anything that isn't needed across reboots should be in ALLSKY_TMP.
Who put "extra" directory where it currently is????  :-)


Read triggerAuthentication values for service bus

Closes #336

Since this PR will close the issue, can you update the following as well please?

https://github.com/kedacore/keda/blob/master/spec/triggers/azure-service-bus.md
https://github.com/kedacore/keda/blob/master/examples/azureservicebus_scaledobject.yaml

For the rest it looks good to me!

Sorry for the delay @tomkerkhove. I updated the docs

Thanks!


grab any usful links (if any) from my list

Grab any usful links (if any) from my list:
https://github.com/10UWP/The-UWP-Tools-List
Pick through them to meet your "Asesome" criteria since I am not testing or verifying anything on my list so there could be a lot of useless or marginal projects there

The "awesome" is not really a criteria, as I was just following the naming pattern :) Feel free to submit a PR with the additions you'd like to see.


Enable Processing of Single Burst Interferograms

Purpose
As ASF's Ingest & Access team works to create a service that can provide users a way to download Sentinel-1 SLC data for a single burst, we want to create ways for users to create single-burst interferograms. This PR adds functionality and a new entrypoint to DockerizedTopsApp that allows it to interact with the alpha-stage burst API and create single-burst interfograms.
Additions
What this PR adds:

Utilities to interact with the burst API
Utilities to download metadata and swath data not currently supported by the API
An alternate entrypoint that allows for burst processing
Tests for these utilities

What this PR does NOT add:

A stable way to interact with the burst API
Packaging for the burst products beyond what is output from ISCE2

Comparison Data
This link contains data that can be used to compare the outputs generated by ISCE2 for an entire SLC, and the new single-burst option within DockerizedTopsApp. Data in the bursts directory ISCE2 geocoded single-burst outputs from the 2020 Iranian and the 2020 Greecian earthquake. The slc directory contains full slc outputs for the same test cases.
Within a correctly configured DockerizedTopsApp environment, the burst test cases can be run using these commands:
For the Iranian test case:
gunw_burst --username $EDL_USERNAME \
           --password $EDL_PASSWORD \
           --reference-scene S1A_IW_SLC__1SDV_20200604T022251_20200604T022318_032861_03CE65_7C85 \
           --secondary-scene S1A_IW_SLC__1SDV_20200616T022252_20200616T022319_033036_03D3A3_5D11 \
           --image-number 5 \
           --burst-number 8 \
           --azimuth-looks 4 \
           --range-looks 20

For the Greecian test case:
gunw_burst --username $EDL_USERNAME \
           --password $EDL_PASSWORD \
           --reference-scene S1B_IW_SLC__1SDV_20201115T162313_20201115T162340_024278_02E29D_5C54 \
           --secondary-scene S1A_IW_SLC__1SDV_20201203T162353_20201203T162420_035524_042744_6D5C \
           --image-number 5 \
           --burst-number 1 \
           --azimuth-looks 4 \
           --range-looks 20

The SLC test cases were run directly with topsApp.py using the topsApp.xml configuration files that were generated for each respective burst test case where the swath and region of interest paramaters had been commented out. These configuration files can be found in the same directory as this readme.

On the entry_points
This makes what should be completely transparent changes to the DockerizedTopsApp entrypoint.
Previously, either via docker run or python -m isce2_topsapp the main entrypoint would look like:
$ python -m isce2_topsapp --help
usage: gunw_slc [-h] [--username USERNAME] [--password PASSWORD] [--bucket BUCKET] [--bucket-prefix BUCKET_PREFIX] [--dry-run] --reference-scenes
                REFERENCE_SCENES [REFERENCE_SCENES ...] --secondary-scenes SECONDARY_SCENES [SECONDARY_SCENES ...]

optional arguments:
  -h, --help            show this help message and exit
  --username USERNAME
  --password PASSWORD
  --bucket BUCKET
  --bucket-prefix BUCKET_PREFIX
  --dry-run
  --reference-scenes REFERENCE_SCENES [REFERENCE_SCENES ...]
  --secondary-scenes SECONDARY_SCENES [SECONDARY_SCENES ...]

In this PR, that behavior stays intact, but we use a argparser and entrypoints trick to allow alternate entrypoints. Instead, if you run:
$ python -m isce2_topsapp ++help
usage: __main__.py [+h] [++process {gunw_slc,gunw_burst}]

optional arguments:
  +h, ++help            show this help message and exit
  ++process {gunw_slc,gunw_burst}
                        Select the HyP3 entrypoint to use (default: gunw_slc)

You can see how to select an alternative entrypoint. So this will run the GUNW process on full SLCs (the default, and only previously):
$ # same as python -m isce2_topsapp --help
$ python -m isce2_topsapp ++process gunw_slc --help
usage: gunw_slc [-h] [--username USERNAME] [--password PASSWORD] [--bucket BUCKET] [--bucket-prefix BUCKET_PREFIX] [--dry-run] --reference-scenes
                REFERENCE_SCENES [REFERENCE_SCENES ...] --secondary-scenes SECONDARY_SCENES [SECONDARY_SCENES ...]

optional arguments:
  -h, --help            show this help message and exit
  --username USERNAME
  --password PASSWORD
  --bucket BUCKET
  --bucket-prefix BUCKET_PREFIX
  --dry-run
  --reference-scenes REFERENCE_SCENES [REFERENCE_SCENES ...]
  --secondary-scenes SECONDARY_SCENES [SECONDARY_SCENES ...]

and this will instead invoke the single burst processing entrypoint:
$ python -m isce2_topsapp ++process gunw_burst --help
usage: gunw_burst [-h] [--username USERNAME] [--password PASSWORD] [--bucket BUCKET] [--bucket-prefix BUCKET_PREFIX] [--dry-run] --reference-scene
                REFERENCE_SCENE --secondary-scene SECONDARY_SCENE --image-number IMAGE_NUMBER --burst-number BURST_NUMBER
                [--azimuth-looks AZIMUTH_LOOKS] [--range-looks RANGE_LOOKS]

optional arguments:
-h, --help            show this help message and exit
--username USERNAME
--password PASSWORD
--bucket BUCKET
--bucket-prefix BUCKET_PREFIX
--dry-run
--reference-scene REFERENCE_SCENE
--secondary-scene SECONDARY_SCENE
--image-number IMAGE_NUMBER
--burst-number BURST_NUMBER
--azimuth-looks AZIMUTH_LOOKS
--range-looks RANGE_LOOKS

tl;dr: By adding a ++process argument you can select a different entrypoint.

Note tests are failing due to being a fork because:

Pytest is failing because it doesn't have access to the ACCESS-Cloud-Based-InSAR secrets
Docker build is failing because it can't push to ACCESS-Cloud-Based-InSAR container repository


[Feature Request]: Pending Payments

While on a trip me and my friend had an idea,
When paying using Apple Pay on an iPhone you can run iOS Shortcut Automations. The Shortcut input offers the following properties of the Transaction: "Card or Pass", "Merchant", "Amount" and "Name".
It would be as great addition to add pending payments via an API (#88) which would result in the following flow:

Pay something
The Automation adds it to Split Pro as an Pending payment
In the evening the user can then check all the pending payments, assosiates them with a group or a person or delete it if it was a not shared transaction.

this is such a cool idea, sadly didn't have a bandwidth to do this right now. i'll keep this open if someone wants to pick it up


Select within a Drawer does not close onBlur

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior

A Select element inside of a layer element such as Drawer or Dialog will not close onBlur
Expected behavior

Select elements will close onBlur by default
Minimal reproduction of the problem with instructions
Made a little codesandbox to demo this. The select shown on the main page closes just fine onBlur, but the select in the Drawer does not:
https://codesandbox.io/s/cranky-monad-s0ptgj?file=/src/App.tsx

What is the motivation / use case for changing the behavior?

Environment

Libs:
- react version: X.Y.Z
- reablocks version: X.Y.Z

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

Its because of this ... https://github.com/reaviz/rdk/blob/master/src/ExitListener/useExitListener.tsx#L21
It was added because if you have a select inside of a menu it would close the select and menu rather than just close the menu. Any ideas for a better fix?

🤔 let me think


IPV6 Ready !

Edit sql shema:

Field ip is now a varbinary(32) , that allow ipv6

Use INET6_ATON instead of INET_ATON.
IP is now encoded in Hexa in BDD
/!\ Warning MySQL 5.6 or more is requiered /!\

This update require mysql 5.6 to use mysar-ng? or only to import logs on ipv6?
This update break mysar-ng on mysql < 5.6 ?

I have rewritten all INET_ATON with INET6_ATON .
INET6_ATON was implemented in MySql 5.6.
So Mysql 5.6 is required for import all logs and for display.
This upgrade break mysar-ng with older MySQL server.

It would then be interesting to do a project versioning 1.0 would be the current mysar-ng and 2.0 would be the new version with support for MySQL 5.6 or higher. It can be done this way?

If you want to process with this way, You can merge your current master branch in a new branch named v1 and marge my pull request in Master.

Done, please test. :D

All seem to be correct ;)


App Store Translations

Once DOH Marketing team has provided copy, need to ensure that App Store and Play Store is translated appropriately.
Will get final copy over to translators once provided.
Logging this ticket as a placeholder.

@kevin-sheldon @fahd-arshad
I'm attaching a preview of all the files generated in the 7 languages.
Each preview shows:
5 screenshots for 5.5 inch devices.
5 screens for 6.5 inch devices.
5 screens for android screens.
And one Android promo screen.
Easy. I couldn t attach them here but you can download them here:
https://drive.google.com/file/d/1CIH8EJkE5f1e_0GumdxcCmh2FrGavWSV/view?usp=sharing

@helloantoine @kevin-sheldon I believe this is done now? Can it be closed?
We may need to open another issue for the translations of boilerplate text for describing app updates but I think that's best done in its own issue.


Applying license changes

Description of your changes
This PR applies license changes.
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

/test-examples="examples/azure/resourcegroup.yaml"


Implement ply batch iterator.

How is the octree generation going to work with batches?

How is the octree generation going to work with batches?

As far as I saw it, the iterator is used at two places:

Bounding box generation, which is pretty easy to change from points to batches
Subsampling of children: Here all points are read into memory anyway. So we need to add functionality to merge batches and then split up batches modulo 8, which should be not too difficult.

At a high level, couldn't the buf inside PropertyReader be an AttributeData of the right type? Then we wouldn't need to cast and copy.
Maybe we don't even need all those macros then?

I changed everything to work on AttributeData, so no vector casting is needed any more. I think the macros are still needed because of the simultaneous assignment and casting while reading, as mentioned in the comment above create_and_return_reading_fn.

I think we should be able to get rid of the match in read_casted_property! (at which point we can probably remove that macro). In push_reader! we have $dtype, from which we can construct the LittleEndian::read_xyz function name and get the $num_bytes by using std::mem::size_of::<xyz>().

Please be aware that $dtype is not equal the type we read in LittleEndian::read_* for x, y, and z. Furthermore, there is no LittleEndian::read_u8 function.

Thanks for your reviews!


Local test suite is broken because of mismatched MySQL/MariaDB version with CircleCI

Describe the bug
MySQL and MariaDB versions are not the same between .circleci/config.yml and tests/docker-compose.yaml, and as such, the local test suite fails.
The output from go test -v ./... run from the repository root is attached here,
jet-test-verbose.log
And the output of make jet-gen-all if needed,
jet-make-gen-all.log

Environment (please complete the following information):

OS: linux (Ubuntu 22.04.5 LTS, 6.8.0-49-generic # 49~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Wed Nov  6 17:42:15 UTC 2 x86_64 x86_64 x86_64 GNU/Linux)
Database: mysql/maridb
Database driver: N/A
Jet branch/version: master / v2.11.1

Code snippet
N/A
Expected behavior
The test suite would PASS on the master branch as it does on CircleCI
Possible Solution
Replacing the MySQL and MariaDB versions in the tests/docker-compose.yaml with the ones from .circleci/config.yml fixes the test suite. Only changing both versions to the ones from CircleCI's config seems to fix it, although they should be in sync regardless.
I already have a local branch fixing this. Should I open a PR?

Yep, go ahead.


Faster dot product

Uses the ARM smlal instruction to improve performance of neuron update step.

[ ] Fit model from IJCNN paper
[ ] Update partitioning to account for this change in cost

These graphs are comparable to those in the IJCNN paper: the left hand bars show current master, the right hand bars show for this PR.

After 9725565: (bars in order: master, after a06676e, after 9725565)

Hah, take that less than operator :-/
On 18 August 2015 at 10:46, Andrew Mundy<EMAIL_ADDRESS>wrote:

After 9725565
https://github.com/project-rig/nengo_spinnaker/commit/97255658ddefac31597c6d6875f1b981b5267407:
(bars in order: master, after a06676e
https://github.com/project-rig/nengo_spinnaker/commit/a06676e6469f442b09e13319cf25388c650785cc,
after 9725565
https://github.com/project-rig/nengo_spinnaker/commit/97255658ddefac31597c6d6875f1b981b5267407
)
[image: profile_16_dimensions]
https://cloud.githubusercontent.com/assets/3439837/9327083/48c0be20-4596-11e5-8ef1-82373c730966.png
[image: profile_100_neurons]
https://cloud.githubusercontent.com/assets/3439837/9327085/4bc5d6c8-4596-11e5-9c32-2b6869dee940.png
—
Reply to this email directly or view it on GitHub
https://github.com/project-rig/nengo_spinnaker/pull/58#issuecomment-132146632
.

At this point we're here vs master (left hand bar is master, right hand is now).

I slightly changed to model I was profiling against to give a better indication of reasonable firing rates.  That said there's still some space for someone to look into the distribution of neuron update costs.

For reference, profiling model:
with nengo.Network() as model:
    # Create the input to the network.
    ins = [nengo.Node(WhiteNoise(), label="inp") for _ in range(n_dims)]

    # Create the ensemble which we'll be profiling
    a = nengo.Ensemble(n_neurons, n_dims, label="A")

    # Create an ensemble to act as the end of the communication channel
    b = nengo.Ensemble(n_neurons, n_dims, label="B")

    # Connect the input to the first ensemble, and the first ensemble to
    # the second.
    for i, inp in enumerate(ins):
        nengo.Connection(inp, a[i], transform=1.0/np.sqrt(n_dims))
    nengo.Connection(a, b)

# Specify that we want to generate the input on chip and that we want to
# profile one of the ensembles.
nengo_spinnaker.add_spinnaker_params(model.config)
for node in model.all_nodes:
    model.config[node].function_of_time = True
model.config[a].profile = True

OK, profiling!

Cost
IJCNN Paper
Master
This PR

Input filtering
43D + 245
42D + 181
40D + 196

Neuron update
13ND + 69N + 188
15ND + 88N
9ND + 69N

Transmit output
702D + 100
123D + 46
125D + 45

Where N is the number of neurons and D is the number of dimensions.

Input filtering Has changed little, as expected.
Neuron update

I've removed the constant term as it wasn't significant
Between Master and This PR the neuron update cost decreases significantly
Between IJCNN and this PR the neuron model was improved and voltage recording added

Transmit output between IJCNN and Master I removed a 1us delay between transmitting each packet, hence the huge reduction in compute time.

As a side note: that's around 2500 neurons per core in 1D or (the sadly unreachable due to memory boundedness) 925 neurons per core in 16D.

925 neurons per core in 16D.

Awesome.  :) :)

All done.

Merging after checking with @neworderofjamie.


Fix key error when no links are present

Closing this since it was made irrelevant by the watermark rewrite


Add ThreeTickTeaks Plugin, Add name to PluginDescriptor

Woohooo first contribution to a open source project! :)
I have tested it out a bit and it seems to work good so far, got a woodcutting level with it!

Should be enough QOL implemented (Only the How to use Section is still bugged for me)


Mandd/mutator changes

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
What are the significant changes in functionality due to this change request?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Mingw Test on b005a9b : invalidated by @mandd

@mandd The following test failed, I guess you need to re-gold the outputs:
FAILED:
Diff tests/framework/Optimizers/GeneticAlgorithms/GAwithEnsembleModel

@mandd The following test failed, I guess you need to re-gold the outputs:
FAILED: Diff tests/framework/Optimizers/GeneticAlgorithms/GAwithEnsembleModel

Test has been regolded

Job Test CentOS 8 on 2b3d0db : invalidated by @joshua-cogliati-inl
mystery failure

Job Test Ubuntu 20-2 Optional on 2b3d0db : invalidated by @joshua-cogliati-inl
mystery failure

Job Test Ubuntu 18-2 Python 3 on 2b3d0db : invalidated by @joshua-cogliati-inl
mystery failure

Job Test Fedora 31 on 2b3d0db : invalidated by @joshua-cogliati-inl
mystery failure

Job Test Fedora 32 on 2b3d0db : invalidated by @joshua-cogliati-inl
mystery failure

Job Test qsubs sawtooth on cf931e9 : invalidated by @mandd

Test are green, checklist is good. PR can be merged.


"$emit native events"

What problem does this feature solve?
Hi team,
Maybe there should be a way to let components emit caught native events.
In my case I'm writing a music player component, I want to allow users of my component to use .native modifier to listen native Media Events on my component, like
<music-player @play.native="handleOnPlay" />

We know that .native modifier listen to native events on the root element of the component, but in this case <audio> apparently can't be a root component.
There're also other common cases facing similar issue:

A form field component providing @input.native from an internal <input>
A file upload component providing @progress.native.

I know I can catch internal native events and emit a custom event like <audio @play="e => $emit('play', e)"> then let user do <music-player @play="handleOnPlay">, but I just feel using .native directly is more intuitive.
What does the proposed API look like?
Possible forms

Another modifier that lets .native modifier listens to not only the root element
A directive like maybe v-emit-native on a non-root HTML element that lets the element emit its native events to outer .native listeners
An $emit.native(e) method on Vue instance

You can use $listeners to capture all v-on bindings of the component and selectively forward them to the native HTML element. See Binding Native Events to Components.

@Justineo Oh I haven't learned about that. It's a great feature.
I just tried it out but it didn't seem to work.
<music-player @timeupdate.native="handleTimeUpdate" />

I logged $listeners in mounted hook like this:
mounted() {
  console.log(this.$listeners)
}

and the result is only an empty object without a timeupdate property.
However if I remove .native modifier from @timeupdate, I can see a timeupdate function in the $listeners.
Is this as expected?

You should not use .native modifier here.
See https://vuejs.org/v2/api/#vm-listeners:

Contains parent-scope v-on event listeners (without .native modifiers).

@Justineo I don't understand. As explained in Binding Native Events to Components, this is just how you use .native to listen to inner elements. Then comes the doc stating you can't use .native here.

https://github.com/vuejs/vue/releases/tag/v2.4.0

@Justineo Thanks. I'm closing this issue.


python312Packages.fastapi: 0.110.2 -> 0.111.0

python312Packages.fastapi-cli: init at 0.0.4
Description of changes
Routine update; add recommended dependency which is also heavily used in tests.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Mass ping, please resubmit.


Fix NPC reloads

@ZoldorfTheWizard

wait was this never fixed

wait was this never fixed

It got fixed with the old NPC tasks but popped up again through entirely unrelated means.


Readme Clarification

Add instructions in the Readme to check off "Copy Items if Needed" when dragging and dropping the framework into the Demo project. Project won't compile otherwise, frameworks need their own code, not just references, to be independent.

Thanks for the input, we'll update our README file accordingly this week.

README file updated accordingly now. Thanks again!


Create AzureIdentity.framework

Create an identity library with functionality comparable to the azure-identity library in other SDK languages, with only features that make sense for mobile.

[ ] Define which types of authentication are part of the scope for this effort. Tentatively:

OAuth2 (bearer token)
SAS Token

Azure Identity was moved to "on hold" pending MSAL for iOS adding support for SwiftPM, which requires us & customers to be on Xcode 12 (GA expected in September). MSAL auth is not needed for Spool, and is low priority for Storage. Once MSAL team adds SwiftPM support we will treat as a target of opportunity.
Based on the above, I would remove this from the above milestones and add to the backlog.


Support Iterable Interface for Resolvers

Is your feature request related to a problem? Please describe.
GraphQL supports the iterator protocol via iterall. It only supports sync iterables right now for regular returns though, but it'd be nice if the generated TypeScript interfaces also accepted iterator returns.
Describe the solution you'd like
Changing all arrays generated by GraphQL to TypeScript resolvers into iterables.

Implement by @kamilkisiela in the recent refactor, it will be available soon.

I think I found the change but an array is iterable, it seems redundant to specify array or iterable.

@kamilkisiela ^ ?

@blakeembrey So if Array is Iterable then why to use Iterable in a first place?

Because Array implements Iterable, Iterable does not implement Array.

So basically, @blakeembrey we can use just Iterable? are you sure it's compatible with all TypeScript configurations?
@kamilkisiela

Available in 1.0.0 👍

I was looking for this exact feature, but discovered that the changes (https://github.com/dotansimha/graphql-code-generator/pull/1471) were reverted only a few days later (for the release of 1.0.3 it seems) with https://github.com/dotansimha/graphql-code-generator/pull/1534:

Remove ArrayOrIterable at the moment, until we'll figure out if it's really needed.

Can this issue be reopened?

I recently stumbled upon this issue when trying to speed up our subscriptions where we try to avoid excess memcopies by returning Iterables instead of Arrays.
Is there a way to explicitly make one resolver return e.g. Iterable, so that we could gradually transition from Arrays to Iterables?


Document the permissions needed by the security audit tools

What are the least required permissions needed for the security audit tools to run? For example, does it need certain permissions with Windows, PI Data Archive, PI AF, etc to run its audit checks?

PI Data Archive: Read access to PIDBSEC, PIMAPPING, PITRUST, PIUSER and PITUNING.

Addressed with wiki page: Required Permissions and Modules


Feature: Snowpack compatibility

Snowpack is built using Rollup, and even offers the ability to pass Rollup options in its config file, similar to Vite.
Its plugin API is good too.

The API looks like Rollup, but actually not very compatible (no resolveId, transformResult use contents instead of code, etc.) We will need to do a compatible layer for it. PR welcome if you managed to make it work at least for transform. Thansk.

Snowpack does allow direct Rollup configuration through packageOptions, and Unplugins can be registered already using packageOptions.rollup.plugins. This is why I'm thinking full compatibility shouldn't be too hard.

https://dev.to/fredkschott/5-more-things-i-learned-building-snowpack-to-20-000-stars-5dc9
Snowpack is planning to wind down - so I think we could skip the support for it then.

RIP 🪦


Refactored into a single function and added some additional tests.

Hi,
I refactored the unique,unique_eq and unique_pred functions into a single function.  Also added some more tests.
I did run it through eslint and "fixed" some coding errors related to missing semicolons but I know this is more a stylistic opinion rather than incorrect code.
Thanks,
Gary

I'll take the test cases, but I don't see the point of refactoring into a single functions.  Also please no semicolons.

Well if you compare the current unique_eq and unique_pred functions they only differ by a few characters, the majority of it is duplicated code.  By combining them into a single function you remove the duplication and get a simpler flow.  Also you save 20 line of code which is a 40% reduction.  I take your point about the semicolons and can remove them if you want.

My concern would be any performance impact from combining the functions.  I would like to see benchmarks that prove this change does not make the comparison slower for the predicate case.

http://jsperf.com/uniq-multi
Testing in Chrome 41.0.2272.76 on Linux i686
I've ran this repeatedly and they seem to run in the same time  and trade places as to which is fastest.

Some of those benchmarks and tests are calling with the wrong semantics.  Why would you pass Array.sort into the predicate?
Also even though jsperf is not a great platform for benchmarking, it is pretty clear that the older version is faster:
http://jsperf.com/uniq-multi/3
You should not declare the function constructors in the benchmark loop since that will trigger lots of extra GC and give inaccurate results.

Yea I threw the jsperf together. I think the compare function i create when we are not supplied one is slower than just using !==
I'll look at this again in a few days.

If you split out the tests into a separate PR, I'll merge them in.

Also passing in Array.sort was just a way of testing the the behaviour was correct when it is passed a compare.

Managed to find a few minutes to revisit this, I reworked things a little although I can't really say it has simplified things all that much.  I stripped out the creation of a default compare function when one is not supplied but that makes the code harder to shorten.  The changes do move everything into a single function which may make it marginally faster according to some test results.
http://jsperf.com/uniq-multi/4


Added installation video #172

What's this PR do? (related issue if exists)
This PR solves issue #172.

@Creatoon where you have added the video?
Also I can see that your this PR has some changes from some other PR in review. Reset your current branch with the master and then make the changes. So that changes come for only the issue you are working upon.

@Signior-X  this was an


feat: modulo[wip]

@joshmoore, here's the beginning of a modulo type
from ome_types.model import OME, modulo

m = modulo.ModuloAnnotation()
m.value.modulos.modulo_along_c = modulo.Modulo(
    type=modulo.ModuloType.ANGLE, start=0, end=360
)

ome = OME()
ome.structured_annotations.append(m)

print(ome.to_xml())

<OME xmlns="http://www.openmicroscopy.org/Schemas/OME/2016-06" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.openmicroscopy.org/Schemas/OME/2016-06 http://www.openmicroscopy.org/Schemas/OME/2016-06/ome.xsd">
  <StructuredAnnotations>
    <XMLAnnotation xmlns:ns2="openmicroscopy.org/omero/dimension/modulo" ID="Annotation:0" xsi:type="ns2:ModuloAnnotation">
      <ns2:Value>
        <Modulo>
          <ns3:ModuloAlongC xmlns:ns3="http://www.openmicroscopy.org/Schemas/Additions/2011-09" Type="angle" Start="0" End="360"/>
        </Modulo>
      </ns2:Value>
    </XMLAnnotation>
  </StructuredAnnotations>
</OME>

as you can see, the namespaces are still a bit messed up... but it's a start

This is great, thanks, @tlambert03! As I mentioned on Zulip, https://github.com/ome/bioformats/blob/bc1fcddd5329d900853d55c6f8378e9235a26a53/components/formats-api/src/loci/formats/Modulo.java#L39 is probably the best definition of truth for the moment. But if you have any other questions, @jburel, @dgault, @sbesson, and @melissalinkert can hopefully help us out.

seealso: https://github.com/ome/omero-cli-transfer/pull/74


Example code should be fixed for adc.begin

First, many thanks for this great library!
I'm not a experienced arduino user, but it seems like the example code doesn't work:
adc.begin(2,0);
Should be replaced with:
adc.begin();
it requires 3 arguments in order to work with SDA and SCL, the chip address should be provided.

This was because of a hack I was doing in order to make it work with both a software serial library, the esp wire library, as well as the default wire library.
I fixed the hack I did a while ago, so now it handles the begin function properly depending on what board Arduino is compiling for, and updated the readme appropriately.
Also added more functions in the functions list. Thanks!

Sir,
Please send me nau7802 source code header file

It's available with the source, here is the link for the current master branch.
https://github.com/Sawaiz/nau7802/blob/master/NAU7802.h


VMSingle add status field

It will be great to have an actual status of pods for single node installation and tracks it's availability.
It could use the same status as cluster has and poll for pod status (health/unhealthy) during upgrades and periodic resync.
Main goal - simplify VMSingle usage for programmatic access, user could subscribe for vmsingle.status and make some action of it's not healthy/upgrade failed instead of listing pods.

Added at v0.35.0 release


[Feature Request] Support for vim-ruby

Hi, I'd like to be able to see syntax-highlighting support for ruby strings:
  describe 'with default arguments' do
    let(:query) do
      '
      query QueryMeals {
          meals {
             id
          }
      }
      '
    end
  end

This looks a bit harder as there is no convention to tag them (as opposed to javascript where usually you have gql/graphql).

You could key off a HEREDOC like GRAPHQL or GQL
<<~GRAPHQL
  query QueryMeals {
    meals {
      id
    }
  }
GRAPHQL

This is similar to the proposal raised in #11, and I think I'd apply the same reasoning here, as well.
I'm open to considering a patch for this, but I'll be weighing its inclusion based on code complexity and fragility. (Sorry to be vague in this regard; I don't know the Ruby grammar well enough to know how tricky this might be to be done well. Please educate me if this is straightforward!)

The RubyMine plugin uses the <<-GRAPHQL heredoc for this.  I wouldn't mind taking a stab at this (as I very much want it).

@sodapopcan That sounds good! Please read my comment in #11 before you get too far, though. I’d like to offer this functionality, but only if the implementation is straightforward and testable.


⚠ CodePush has degraded performance

In a0e6b30, CodePush (https://codepush.appcenter.ms/v0.1/public/codepush/update_check?deployment_key=Fzqu71x4FXhOaKcAiwpqXkFQddjgYl2dIUW2K&app_version=4.4.1) experienced degraded performance:

HTTP code: 200
Response time: 21369 ms

Resolved: CodePush performance has improved in e8daade.


fix(clients): highlight and snippet results e2e

🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-2767 https://algolia.atlassian.net/browse/DI-2776 https://algolia.atlassian.net/browse/DI-2779
Changes included:
reported in https://github.com/algolia/algoliasearch-client-kotlin/issues/423, should close https://github.com/algolia/algoliasearch-client-kotlin/issues/423
also enables e2e for kotlin, can't remember why it was missing but... it was, and highlights some problems

awesome thanks a lot for the help @millotp 😍


from __future__ import annotations and then Union -> |

Just to be consistent, let's move to the pipe syntax everywhere it doesn't cause problems.

It is supported if you use from __future__ import annotations. However you still have to be careful, as this can cause some weird edge cases with Sphinx if you use it in conjunction with TypeVars.

Originally posted by @Shoeboxam in https://github.com/opendp/opendp/pull/1604#discussion_r1641861373

Not just sphinx: Trying to make this change:
- Chain = Union[tuple[Domain, Metric], Transformation, Measurement, "PartialChain"]
+ Chain = tuple[Domain, Metric] | Transformation | Measurement | "PartialChain"

produces this error:
TypeError: unsupported operand type(s) for |: 'types.UnionType' and 'str'

Sorting this out isn't feeling like a high priority.


Fix save bread when ajax calls are not done

Every relationship set in BREAD made three ajax calls to collect table fields.
In a BREAD with many relationships and/or slow server this could result in some relationship not correctly populated when hitting Submit button.
This PR prepopulates selects with already saved values and leaves ajax calls only when table is changed or modal is opened.
Fixes #3960
TODO

[x] Fix select width in modal

Everything is working so far but in modal select2 doesn't keep the correct width, I think it can be fixed changing the parent element to:
display: flex

and select2 span to:
flex: 1

Then we can't use jQuery show() because it will set again to block.

@emptynick looks good to me, thanks.
Why the QA label, did you need any clarification about what this PR is solving or how it's doing it?

Just added it so I don't blindly approve it without you approving it first :+1:


Setting up doom-emacs as a IDE for python.

I'm very new to emacs and doom-emacs and Python as well, I'm trying to setup doom-emacs as a vscodium replacement, I got most of the things setup like color scheme, linting and auto completion, but I can't seem to figure out how do I execute the .py file, like in vscodium I just press the play button, and it executes it in a console window below. Do I need some package or plugin to enable such feature in doom-emacs. Sorry if this has been already asked.

Look at #1997
I personally use elpy to run the python code by C-c C-c, you can also use an eshell and manually run python foo.py

Look at #1997
I personally use elpy to run the python code by C-c C-c, you can also use an eshell and manually run python foo.py

C-c C-c opens python interpreter, is that elpy? eshell seems like a good option. Is there no native way of running/executing the .py files, like Vscodium has, just press some key combination or button or something to run the program.

C-c C-c opens python interpreter, is that elpy? eshell seems like a good option. Is there no other way of running/executing the .py files, like Vscodium has, just press some key combination or button or something to run the program.

Apologies, I forgot to mention that you need to add elpy to packages.el to install it, check the documentation for elpy.
C-c C-c should then evaluate the code instead of just opening the interpreter, this should be your key combination code execution, Doom Emacs doesn't really have buttons.

C-c C-c opens python interpreter, is that elpy? eshell seems like a good option. Is there no other way of running/executing the .py files, like Vscodium has, just press some key combination or button or something to run the program.

Apologies, I forgot to mention that you need to add elpy to packages.el to install it, check the documentation for elpy.
C-c C-c should then evaluate the code instead of just opening the interpreter, this should be your key combination code execution, Doom Emacs doesn't really have buttons.

I added these lines to end of the of packages.el
(use-package elpy
  :ensure t
  :init
  (elpy-enable))

and it gave me errors
❯ ~/.emacs.d/bin/doom sync
> Synchronizing your config with Doom Emacs...
  > Regenerating envvars file at "~/.emacs.d/.local/env"
    - Ignoring DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/1000/bus
    - Ignoring TERM=alacritty
    - Ignoring HOME=/home/apoorv
    - Ignoring PWD=/home/apoorv
    ✓ Successfully generated "~/.emacs.d/.local/env"
Ignoring ’:ensure t’ in ’elpy’ config
  x There was an unexpected error
    Message: Symbol's function definition is void
    Data: (void-function . elpy-enable)
    Backtrace:
      (elpy-enable)
      (condition-case err (elpy-enable) ((debug error) (funcall use-package--war
      (progn (condition-case err (elpy-enable) ((debug error) (funcall use-packa
      (condition-case err (progn (condition-case err (elpy-enable) ((debug error
      (eval-buffer #<buffer  *load*-365026> nil "/home/apoorv/.doom.d/packages.e
      (load-with-code-conversion "/home/apoorv/.doom.d/packages.el" "/home/apoor
      (load "/home/apoorv/.doom.d/packages.el" noerror nomessage nosuffix)
      (if (not noeval) (load file noerror 'nomessage 'nosuffix) (if (file-exists
      (progn (if (not noeval) (load file noerror 'nomessage 'nosuffix) (if (file
      (unwind-protect (progn (if (not noeval) (load file noerror 'nomessage 'nos
  ! Extended backtrace logged to .emacs.d/.local/doom.error.log
~

According the documentation, it says to add the lines to init.el, but adding the lines to init.el does nothing.

According the documentation, it says to add the lines to init.el, but adding the lines to init.el does nothing.

Installing packages in Doom is different, just add (package! elpy) to packages.el then run M-x doom/reload
Here's what I am doing
I have this in my config.el for configuration
(use-package elpy
  :ensure t
  :init
  (elpy-enable)
  )
(setq elpy-rpc-virtualenv-path "~/Environments/emacs_main/")
(setq elpy-rpc-python-command "python3")
(setq python-shell-interpreter "~/Environments/emacs_main/bin/python3") ;; This is what `C-c C-c` will use

Don't forget to run pyvenv activate and select your environment

According the documentation, it says to add the lines to init.el, but adding the lines to init.el does nothing.

Installing packages in Doom is different, just add (package! elpy) to packages.el then run M-x doom/reload
Here's what I am doing
I have this in my config.el for configuration
(use-package elpy
  :ensure t
  :init
  (elpy-enable)
  )
(setq elpy-rpc-virtualenv-path "~/Environments/emacs_main/")
(setq elpy-rpc-python-command "python3")
(setq python-shell-interpreter "~/Environments/emacs_main/bin/python3") ;; This is what `C-c C-c` will use

Don't forget to run pyvenv activate and select your environment

Ah! I see what is happening, it also opens python interpreter. I was expecting more like console execution. And also it opens in a horizontal split rather than vertical split.


Complexe code evaluation fails

With some long and complexe code, evaluation seems confused in the python namespace and loose some variable names.

It seems it was a scope issue and adding globals() arg to exec() fixes it.


Add support for Ruby

@rohitpaulk I'm opening this as a draft PR so that you can review the progress as well.
I've added the starter template files for now. Will continue to work adding the rest of the code and diff files as I work my way through the Build your own Docker course with Go, and understand what we are doing in each step.

@sreeram-venkitesh awesome!
I've enabled GitHub actions on this PR so you should see errors as they appear. Let me know once you have the starter templates ready and compiled (i.e. compile_and_test.sh courses/docker ruby passes), and I can deploy this on production so you can test the rest of the stages there. We have the ability to make languages as "alpha", so they won't be accessible to other users until we decide to.
There's one go_lint error that is unrelated to your changes, that I'm fixing in this PR.

@rohitpaulk I think the verify_templates_compiled test failed since I hadn't created the files under compiled_starters directory. How can I create these files? Can I just copy the changes I made in starter_templates? I thought this would be generated somehow.
On a second viewing, I notice that the README.md file is not present in the files inside starter_templates and is only there inside compiled_starters. So I'm assuming I can copy the code files and add the appropriate content in the README.md file for Ruby, right?

@sreeram-venkitesh the README here mentions how to develop courses: https://github.com/codecrafters-io/course-sdk#developing-courses.
The "compile & test" command is what you're looking for 🙂

On a second viewing, I notice that the README.md file is not present in the files inside starter_templates and is only there inside compiled_starters.

It is present, just that it is common for all languages: https://github.com/codecrafters-io/build-your-own-docker/blob/main/starter_templates/README.md. When you run the compile_and_test command, it'll be automatically generated and placed inside compiled-starters 👍

@rohitpaulk Thanks! I've moved my version of the Build your own Docker repo to inside courses/ in the courses-sdk repo locally so that I can run the tests and generate the files into my repo. I tried running compile and test, but I was getting an error with docker in the Executing starter repo script step:
docker: invalid reference format: repository name must be lowercase.

I'll look into this and let you know here once I've fixed it. Right now I'm going through the courses-sdk code and understanding how everything works.

@sreeram-venkitesh interesting, haven't seen that one before. Might be worth confirming your docker version is the latest + ensuring no stray files exist anywhere that contain uppercase letters. (I'd do git clean -fd, which'll remove all untracked files).
The same error isn't present on CI, here's what we're seeing there:

(Tried this on my local machine, and I see the same error as what CI reports)

@sreeram-venkitesh just realized the GitHub actions requires me to approve workflows every time you push - I've added you as a collaborator so that this doesn't happen

I'll look into this and let you know here once I've fixed it. Right now I'm going through the courses-sdk code and understanding how everything works.

Awesome - curious to get to the bottom of this. If there's a way we catch the error early and display a friendly message to the user that'd be great

@rohitpaulk I was able to get around the docker: invalid reference format: repository name must be lowercase error locally by adding quotes around the options passed to docker run in the assert_script_output method in starter_repo_tester.rb, like so
command = [
  "docker run",
  "-v '#{File.expand_path(tmp_dir, ENV["HOST_COURSE_SDK_PATH"])}:/app'",
  "-v '#{File.expand_path(tester_dir, ENV["HOST_COURSE_SDK_PATH"])}:/tester:ro'",

I had tried enclosing everything after -v in single quotes, after reading this post. I'm not sure if this was failing earlier due to something specific to my local dev setup, but worth looking into anyway.
Right now I'm getting the same error in test_course_definition / test (ruby) as in the GitHub Actions pipeline locally, which I'm looking into fixing now.

@sreeram-venkitesh great find! Fixed in https://github.com/codecrafters-io/course-sdk/pull/9.
Looks like you've got the GitHub actions tests passing now?

@rohitpaulk Yes, all the tests are passing now.
I noticed that for all the languages other than Go, Build your own Docker only has the first stage right? I saw that we are showing Go's solutions for all the other languages as well. If this code is merged right now, I'm guessing the same would be the case for Ruby right? (for the people who are able to see languages marked as alpha that is)

@sreeram-venkitesh yep! We'd show Go for the official solution.
As users try this out using Ruby (after it's in Beta), their solutions will automatically appear in the "Solutions" tab btw (released last week):

@sreeram-venkitesh so this is good to review? (PR is still a draft - let me know when you want me to take a look 🙂)

@rohitpaulk I'm done from my end, please take a look and let me know if I can proceed in the same manner for the rest of the parts.

Hey @sreeram-venkitesh! Sorry, this got lost in a sea of notifications - taking a look soon 🙂

Is there any other way I can test if the solution is correct other than by running the tests locally?

The same local tests are run on CI too (and it looks like those are passing), so I think we're good (will review in detail soon)!

Quick update - going to take a look at this tomorrow (sorry, past week was super busy)

Deploying this in a couple of mins...

@sreeram-venkitesh can you head over to https://app.codecrafters.io/courses/docker?fresh=true and see if Ruby is now listed for your account?

@sreeram-venkitesh wouldn't worry about further "official" solutions, we don't need those to release this to the public 🙂  Just get through the challenge using your account, and your solutions will automatically appear in the "solutions" tab:

If you get through till stage 5 (process isolation), I think it's safe to release for others to try. Stage 6 only requires wiring up with the API, and that's straightforward in Ruby. The part I'm unsure about is process isolation (using PID namespaces).

Ah okay, so once I attempt the course in Ruby, my code would show up in the Solutions tab and even though official solutions aren't there for Ruby, that's enough for releasing it to the public, right? Anyway I'll let you know here once I'm done with all the stages.

Ah okay, so once I attempt the course in Ruby, my code would show up in the Solutions tab and even though official solutions aren't there for Ruby, that's enough for releasing it to the public, right? Anyway I'll let you know here once I'm done with all the stages.

Yep, exactly!


Deal with multiple replacements in one file

Currently, rustfix will explode (or wreck your source file to be more precise) when it applies multiply suggestions to the same file -- The file changes but we still use the original byte offsets given by rustc.
We could instead use a rope-like data structure and remember the original indices. Also, it might be nice to only write the file once, after all suggestions for a file were dealt with by the user (i.e., ≥ 0 were accepted).

I'm no expert on ropes – but I may have another idea. Please tell me if this is something that already exists (even better if there is a crate as well!)
I'm imagining storing this in a map of Ranges of line numbers mapped to Cow's (exercise for the reader: Make a joke with 'field of cows').
Reading the file, you begin with a map with one node that maps start...end (first to last byte in the file) to a Cow that borrows the whole slide of the input data. When you want to change some byte range x..y, you replace that one node with three: The first maps 0..x to a Cow that borrows 0..x, the second one maps x...y to a Cow that owns the new content, and the third one maps y+1...end to the Cow that borrows the y+1...end.
The important bit is this – you can replace the x...y slice with a new slice that is not of length y-x.
As we are unlikely to do many replacements per file (≥100 seems unlikely in practice, so we are probably dealing with ≤100 ranges to look at), a simple VecMap<Range<usize>, Cow<'input, str>> could suffice for this (instead of a tree we need to balance). Additionally, we might even be able to keep track of a cursor and never even need to scan through all the previous ranges while inserting at all – we are almost certainly only adding/splitting ranges with increasing byte offsets.(Lints go linearly through the file, or we can collect all changes to apply per file, sort them and do them at once.)

@eddyb mentioned I should look at xi-rope, and adjust it to not track index updates. (This might also be interesting to xi-rope upstream if it has similar use cases.)

You could just load the file in a String and replace the suggestions from the last to the first, that way the byte offsets given by rustc stay valid all along and you don't need anything as fancy as ropes.

That is actually brilliant :O

Am 07.07.2016 um 23:06 schrieb Martin Carton<EMAIL_ADDRESS>You could just load the file in a String and replace the suggestions from the last to the first, that way the byte offsets given by rustc stay valid all along and you don't need anything as fancy as ropes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Using a trivial suggestions.rev() version right now.


Fix number vs unit

Have a script that looks at all the times "number" is used as a type and confirm that:

number vs unit is correct
If it should be unit, make sure that there's actually a unit in the list (like percentage)

I think this is done. Although it was not done via script because it's a little hard to do since most of the time this was us writing something in the comments. We might still be able to look at this though


delete component ERROR strconv.Atoi: parsing "8+": invalid syntax

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind feature

What happened:
error occurred while trying to delete component with ks
What you expected to happen:
component deleted
How to reproduce it (as minimally and precisely as possible):
feixue@fei-kubernetes-test:~/my-kubeflow$ ks delete cloud -v -c serveInception
DEBUG Validating deployment at 'cloud' with server '[https://*.*.*.* https://*.*.*]'
DEBUG Retrieving all environments
DEBUG Found environment 'cloud', with server 'https://*.*.*.*' and namespace ''
DEBUG Found environment 'default', with server 'https://*.*.*.*' and namespace ''
DEBUG Overwriting --cluster flag with 'gke_*****-kubernetes-test_us-central1-c_kubeflow-large'
DEBUG Overwriting --namespace flag with ''
DEBUG Adding jsonnet search path /***/my-kubeflow/lib
DEBUG Adding jsonnet search path /***/my-kubeflow/vendor
DEBUG Adding jsonnet search path /***/my-kubeflow/environments/cloud/.metadata
DEBUG jsonnet result is: {
   "serveInception": {
      "apiVersion": "v1",
      "items": [
         {
            "apiVersion": "extensions/v1beta1",
            "kind": "Deployment",
            "metadata": {
               "labels": {
                  "app": "inception"
               },
               "name": "inception",
               "namespace": ""
            },
            "spec": {
               "template": {
                  "metadata": {
                     "labels": {
                        "app": "inception"
                     }
                  },
                  "spec": {
                     "containers": [
                        {
                           "args": [
                              "/usr/bin/tensorflow_model_server --port=9000 --model_name=inception --model_base_path=gs://kubeflow-models/inception"
                           ],
                           "command": [
                              "/bin/sh",
                              "-c"
                           ],
                           "image": "gcr.io/kubeflow/model-server:1.0",
                           "imagePullPolicy": "IfNotPresent",
                           "name": "inception",
                           "ports": [
                              {
                                 "containerPort": 9000
                              }
                           ],
                           "resources": {
                              "limits": {
                                 "cpu": "4",
                                 "memory": "4Gi"
                              },
                              "requests": {
                                 "cpu": "1",
                                 "memory": "1Gi"
                              }
                           }
                        }
                     ],
                     "securityContext": {
                        "fsGroup": 1000,
                        "runAsUser": 1000
                     }
                  }
               }
            }
         },
         {
            "apiVersion": "v1",
            "kind": "Service",
            "metadata": {
               "labels": {
                  "app": "inception"
               },
               "name": "inception",
               "namespace": ""
            },
            "spec": {
               "ports": [
                  {
                     "port": 9000,
                     "targetPort": 9000
                  }
               ],
               "selector": {
                  "app": "inception"
               },
               "type": "ClusterIP"
            }
         }
      ],
      "kind": "List"
   }
}
ERROR strconv.Atoi: parsing "8+": invalid syntax

Anything else we need to know?:
on GKE
Environment:

ksonnet version (use ks version): 0.8.0
Kubernetes version (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.6", GitCommit:"6260bb08c46c31eea6cb538b34a9ceb3e406689c", GitTreeState:"clean", Bu
ildDate:"2017-12-21T06:34:11Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"8+", GitVersion:"v1.8.7-gke.1", GitCommit:"192ccad06d24af9828cbf42330e1d915cb586406", GitTreeState:"cle
an", BuildDate:"2018-01-31T21:39:04Z", GoVersion:"go1.8.3b4", Compiler:"gc", Platform:"linux/amd64"}

Thanks for raising this issue! It's actually fixed in master (https://github.com/ksonnet/ksonnet/pull/254) already, so you have two options:

You can check out that branch and build the binaries yourself.
Or you can wait for the official 0.9.0 binaries, which will be released in two weeks.

i will wait, thanks for the info. closing this issue.


FIX: add remove-stream function to stream_engine

Codecov Report

Merging #79 (7ae8176) into master (b4dd516) will decrease coverage by 0.92%.
The diff coverage is 55.55%.

:exclamation: Current head 7ae8176 differs from pull request most recent head ec0bcc0. Consider uploading reports for the commit ec0bcc0 to get more accurate results

@@            Coverage Diff             @@
##           master      #79      +/-   ##
==========================================
- Coverage   95.53%   94.61%   -0.93%
==========================================
  Files          16       16
  Lines         381      390       +9
==========================================
+ Hits          364      369       +5
- Misses         17       21       +4

Flag
Coverage Δ

unittests
94.61% <55.55%> (-0.93%)
:arrow_down:

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

Impacted Files
Coverage Δ

kstreams/exceptions.py
62.50% <55.55%> (-4.17%)
:arrow_down:

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


Hello, I am a beginner and I am encountering some problems that are causing my code to not run. Could you help me take a look at it? Thank you.

I followed the instructions you provided, but I still encountered an error.

Hi, maybe in SMT/config.py the line 3 should be "from yacs.config import CfgNode as CN" instead of "from config import CfgNode as CN"


[stdlib] Resolve the ambiguity for Int32.init(bitPattern:)

When this initializer is invoked with an integer literal, the expression is ambiguous, since both UIn32 and Float are RepresentableByIntegerLiteral.
Fixes: https://bugs.swift.org/browse/SR-5176

@dabrahams is it too ugly? It affects code completion, but the API, I believe, is used rarely enough so that it does not seem to be a huge problem. What do you think?
/cc @airspeedswift

@swift-ci Please test

@moiseev Might it be more elegant to call it init(_floatingPoint:bitPattern:), for when it's inevitably printed?

@xwu It turns out, that the code completion already generates 2 suggestions in this case: one with just the Float argument, and one with the dummy.

@moiseev Yeah, to be expected.
I meant only that, when it is encountered (in code completion, say), the name init(bitPattern:_disambiguationMarker:) might seem a little cryptic.
By moving the dummy parameter first, people who type init(b might be spared the sight of it, and people who do see it might appreciate a more grammatical phrase such as "floating point bit pattern" :)

@xwu sure. I wasn't arguing against it at all. Using an underscore in the first argument label actually moves the declaration down the list of suggestions, which is also not a bad thing.

@swift-ci Please test

@swift-ci Please Test Source Compatibility

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - b4be724152cdcd4ac71d59869cdf44fcbe1674f7
Test requested by - @moiseev

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - b4be724152cdcd4ac71d59869cdf44fcbe1674f7
Test requested by - @moiseev

@swift-ci Please Test Source Compatibility

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - acf0ef11227c8f48519f0bc3861089a61ea663a6
Test requested by - @moiseev

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - acf0ef11227c8f48519f0bc3861089a61ea663a6
Test requested by - @moiseev

@swift-ci Please Test Source Compatibility


remove unused shell_escape dep

Good catch, thanks!
@bors r+

:pushpin: Commit 5ed0f1454a49fb970afd45ee6158acad4e59c1b6 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 5ed0f1454a49fb970afd45ee6158acad4e59c1b6 with merge 4ca3e276448ee5555f419d2a006ace4a60988388...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 4ca3e276448ee5555f419d2a006ace4a60988388 to master...


Why apply to the predicted depth with the min-pooling and one-hot operation? thx.

Because it can generate the best result. For more details, please refer to our paper.

Yes, it is.


-Insert SKU here-

What do you mean with -Insert SKU here- ? Probably a documentation mistake?

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

ID: fb68dee1-01d8-94cf-976a-5dd31d408f4d
Version Independent ID: 1deea8b5-5241-c349-0d17-3a6a55187e58
Content: Bluetooth developer FAQ - Windows UWP applications
Content Source: windows-apps-src/devices-sensors/bluetooth-dev-faq.md
Product: uwp
Technology: peripherals-sensors-and-power
GitHub Login: @eliotcowley
Microsoft Alias: elcowle

Hi @maexsp, thanks for pointing this out. While an odd title, it is intentional. However, I've revised it to be less confusing.


Migrate to Typescript

Closes https://github.com/ealush/emoji-picker-react/issues/217

Let me know if you need help on this one


Does power management plan applied by ConfigMgr affect computer performance?

It's not been mentioned anywhere in the document but I would like to find out.
Does applying power plan from SCCM affect the CPU / GPU performance especially on a laptop?

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

ID: 50e4ee97-90b8-7ffb-7aa0-c6b81f18e6e2
Version Independent ID: 02cc3bf5-e48d-8e08-6fff-60e6dd24ba26
Content: Introduction to power management - Configuration Manager
Content Source: sccm/core/clients/manage/power/introduction-to-power-management.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

It will depend upon the settings of the power plan. It's like any Windows power plan, but the SCCM power plans don't have as many properties. Looking at the list in SCCM, there aren't any settings for CPU throttling or other performance-related aspects. The only possible property to impact performance is Turn off hard disk after (minutes). All of the others relate to power state and power-related actions. Also note that these settings aren't applied by default; you have to turn on power management in SCCM and then configure a power plan on a collection.
I'm closing this issue


Nagios : Amend Nagios/Ansible Config Role To Create Host Groups Config File

Part of epic : #2445 ( Task 6.2 )
Update the Nagios_Config playbook to create the hostgroups.cfg file
The script should parse the ansible inventory ( similar to issue #2629 )
And create a config file which contains a hostgroup entry for each of the providers in the inventory, ie the second stanza for the build, test & dockerhost level 1 stanzas.
define hostgroup{
       hostgroup_name  Ibmcloud
       alias           Ibmcloud
       members         ,test-ibmcloud-rhel7-x64-1,test-ibmcloud-rhel6-x64-1,test-ibmcloud-ubuntu1604-x64-1,build-ibmcloud-win2012r2-x64-1,build-ibmcloud-win2012r2-x64-2,test-ibmcloud-win2012r2-x64-1,test-ibmcloud-win2012r2-x64-2
       }
The playbook should then add the relevant machines to the members list as shown in the example above.

this is a task next in line,  could you assign me ?


Lazy-sync local chunks with the network

User story
Story
I'm an node operator that is maintaining a set of data on Swarm. This data set has periodic updates, and any update replaces the previous one.
Each update contains data chunks that can be useful or unuseful for users; a data chunk is useful when at least an user requests it, it's unuseful when no users requests it, and a successive update discard it.
As a data maintainer I've a cost on upload chunks to the network, so I want to minimize traffic of unuseful data chunks. What I want is to publish with each update only the reference of the last data set, and use Global Pinning to make it reachable.
When an user request a chunk, if it has already been requested and it has not been dropped by caching nodes, the network will reply with it. If it doesn't exists, the Global Pinning implementation will require to me to upload it. In this way only useful chunks will be uploaded.
Acceptance criteria
When I upload a file to a Swarm gateway, I need a way for say "use lazy-sync", or something like this. The effect must be that the file (and relative chunks) will be local pinned, and the sync of the file with the net never starts. Using node address I will be able to use Global Pinning, and make data accessible on the network on a successive step.
Background
At Etherna we need this solution for implement MDBee: https://github.com/Etherna/mdbee
MDBee wants to keep synced a MongoDB instance with a Swarm copy of data. Documents evolves in time, but some changes could come before that previous document has ever been read by someone. Any upload of unwanted documents would be a waste of traffic. More information on the paper.

Hi @ttmm360!
First of all, thank you for taking the time to make this issue. I see the use case and how it would provide value to your users.
To support this feature as you have in mind, the Bee node must support an upload, but don't sync operation. Additionally, global pinning must be extended with issue #891 to efficiently request upload a big amount of chunks for global pinning.
Unfortunately, the Swarm team does not have the bandwidth at this moment to dive into this specific issue before our 1.0 release.
I do believe, however, that you will still be able to have this feature, even without any changes to the Bee node:

You register a special handler to your PSS inbox, looking for a recovery message, requesting the root chunk of the file. Normally this recovery message would trigger the recovery protocol of your global pinning node, but since you didn't upload anything this doesn't happen
The message triggers a process outside the bee that would upload everything under that specific root hash
The user will be able to download the download the file when everything is uploaded. He/she might need to retry once.

Let me know what you think about this!

Hi @ttmm360!
First of all, thank you for taking the time to make this issue. I see the use case and how it would provide value to your users.
To support this feature as you have in mind, the Bee node must support an upload, but don't sync operation. Additionally, global pinning must be extended with issue #891 to efficiently request upload a big amount of chunks for global pinning.
Unfortunately, the Swarm team does not have the bandwidth at this moment to dive into this specific issue before our 1.0 release.
I do believe, however, that you will still be able to have this feature, even without any changes to the Bee node:

You register a special handler to your PSS inbox, looking for a recovery message, requesting the root chunk of the file. Normally this recovery message would trigger the recovery protocol of your global pinning node, but since you didn't upload anything this doesn't happen
The message triggers a process outside the bee that would upload everything under that specific root hash
The user will be able to download the download the file when everything is uploaded. He/she might need to retry once.

Let me know what you think about this!

Thank you @Eknir!
This can be a good workaround and I appreciate your suggestion, but luckily this isn't a blocking feature for us. The MDBee project is currently considered an experiment, and we can postpone this optimization for when feature will be ready, also after 1.0 release.

Thank you @Eknir!
This can be a good workaround and I appreciate your suggestion, but luckily this isn't a blocking feature for us. The MDBee project is currently considered an experiment, and we can postpone this optimization for when feature will be ready, also after 1.0 release.

ping


Microsoft.Insights/scheduledQueryRules displayName always marked for removal

Describe the noise
Resource type (i.e. Microsoft.Storage/storageAccounts)
Microsoft.Insights/scheduledQueryRules
apiVersion (i.e. 2019-04-01)
2018-04-16
Client (PowerShell, Azure CLI, or API)
Azure CLI 2.10.1
Relevant ARM Template code (we only need the resource object for the above resourceType and apiVersion, but if it's easier you can include the entire template
{
 "name": "myRule",
 "type": "Microsoft.Insights/scheduledQueryRules",
 "apiVersion": "2018-04-16",
 "location": "northeurope",
 "properties": {
   "enabled": "true",
   "source": {
     "query": "['AzureDiagnostics| where ResourceProvider ==\"MICROSOFT.KEYVAULT\"]",
     "dataSourceId": "<WORKSPACE_ID>",
     "queryType": "ResultCount"
   },
   "schedule": {
     "frequencyInMinutes": "15",
     "timeWindowInMinutes": "60"
   },
   "action": {
     "odata.type": "Microsoft.WindowsAzure.Management.Monitoring.Alerts.Models.Microsoft.AppInsights.Nexus.DataContracts.Resources.ScheduledQueryRules.AlertingAction",
     "severity": "1",
     "aznsAction": {
       "actionGroup": "<ACTION_GROUP_ID>"
     },
     "trigger": {
       "thresholdOperator": "GreaterThan",
       "threshold": "1"
     }
   }
 }
}

Expected response (i.e. "I expected no noise since the template has not been modified since the resources were deployed)
No noise after first deploy of the resource
Current (noisy) response (either include a screenshot of the what-if output, or copy/paste the text)
~ Microsoft.Insights/scheduledQueryRules/myRule [2018-04-16]
    - properties.displayName:         "myRule"

Got a 404 during issue creation but it acutally was created. Duplicated of #112 closing.


Automated cherry pick of #11998 upstream release 1.0

Updates the version of heapster to include at least one important bug fix (https://github.com/GoogleCloudPlatform/heapster/pull/409). I'm not convinced this is better than cherry-picking the needed fix into a heapster 0.16.1, but @vishh is in favor of this.
@roberthbailey @brendandburns

LGTM.  Let's talk tomorrow about the risk profile here.
Thanks
--brendan

@timstclair @eparis @smarterclayton FYI

We're going to cherry-pick the heapster fix into a heapster 0.16.1 rather than pulling in 0.17.0, which includes too many changes. Abandoning this PR.

@timothysc


Add supported identifier

Same like with https://github.com/dotnet/docs/pull/15471

#sign-off

@tdykstra @Youssef1313 There is a merge conflict suggestion noted in the build validation report. Can you take a look and fix?

@Jim-Parker, Can you confirm if it's fixed now ?

@Youssef1313 The build still shows conflicts in
Contribute/validation-reference/multiple-h1s.md
and
Contribute/validation-reference/h1-in-moniker.md

@tdykstra, Strange for me, I didn't even edit these files. And GitHub UI doesn't show that there is conflicts. However, I updated them to be the same as the current master. Can you re-check please ?

@Youssef1313 @tdykstra The suggestions are still showing in the build report. I've asked @meganbradley to help clarify since this is related to several PRs. Thanks.


Market page analytics

LIVE-601
Specs : https://ledgerhq.atlassian.net/wiki/spaces/WALLETCO/pages/2626453631/MARKET+-+Live+specifications
Type

Context

Parts of the app affected / Test plan

Please fill the template for the PR correctly 🙏🏼


Added unlimited spam mode

Your program now has 2 modes

Define Number of messages mode
-> Enter message
-> Enter delay
-> Enter number of times spammer should send the message

Unlimited messages mode
-> Enter message
-> Enter delay
->  Program keeps spamming until user presses Cntrl + Shift + W

Code has changed significantly, so if documentation is required, please let me know so I can add it to the code.

Added unlimited spam mode, thanks to z404