Remove Job from History API

a part of issue  #12151
Signed-off-by: Hu Keping<EMAIL_ADDRESS>

Probably some integration test that needs to be modified here to clear up janky.

yeah, i am working on that :)

ping@LK4D4 updated

LGTM

Yes, it looks like that test wasn't really testing anything to begin with.... other than that you can create a job and pass in an arg to it.... definitely good to delete it.

LGTM


We need to sort the functors in the other order

otherwise Happy_eyeballs.t will refer to a different implementation (the Happy_eyeballs.t from the happy-eyeballs opam package).
sorry for the noise.

yes, so, if this is being merged and released... we will finally have something that can work :)


What is "Author protection"?

What do you mean by this? How does it work? From what specifically does it protect you and from what not?
I am interested because I am thinking about opensourcing my snakes as well :)

Author protected is a fields in the start json response.  So if someone use my code without changing anything, it won't be able to run on battlesnake.com.
But it easy to change to code, recompile and use it.


Update Dinos “maiasaura”

Automatically generated by Netlify CMS

👷 Deploy Preview for quizzical-galileo-bf682e processing.

Name
Link

🔨 Latest commit
4e4a75369766e2adc406183b23206509ab53f65d

🔍 Latest deploy log
https://app.netlify.com/sites/quizzical-galileo-bf682e/deploys/63ffa3243a1e0600082c0823


Rerun is extremely slow when updating a root pose

Describe the bug
rerun app is very slow and "laggy" when updating a root transform frame at 100 Hz.
To Reproduce

Use Rust 1.80, Linux and rerun 0.18.2
Publish a pose at 100 Hz: rr.log("root", rerun::Transform3D::from_translation_mat3x3(...))?;
Publish some other data attached to the pose at 10 Hz: rr.log("root/points", rerun::Points3D::new({...}))?;
It takes about 130 ms to render a single frame on a fairly powerful desktop.

Expected behavior
100 Hz pose update should be blazingly fast.
Screenshots
Some examples from the profiler widget:

Thanks for reporting!
Do you have a sample .rrd for us to take a look at? Or a full sample code to reproduce?
How many entities are there?
Are you compiling with --release?

I'm working a bit on transform performance right now and the rrd is indeed proving useful as a a case for slow transforms. That said it's by far not the only thing that is slow in that scene (and I plan to work on that as well! :)).
What is however a bit confusing to me is this bit

Note also that it took about 5 minutes to ingest this file into rerun ..

I'm running on an M1 Max (which tbf is a very powerful machine) but even though I'm getting nowhere near that slow ingestion time on either latest Rerun or 0.18.2 as described, the entire loading is way under a second.
I used the official python build, installed via pip install rerun-sdk==0.18.2.
@Danvil could you please share what exact build (or how you built it) of Rerun you used and on which machine you ran this?

For posterity  - mostly to keep better track myself:
On M1 Max @ rerun 0.20.2 (!!, that's not the originally posted version) I get about 23ms cpu time both while playing & at stills. Profiler shows about 8ms of that being in the transform context, but there's a ton of profiling overhead.
On a WIP branch (f170086c0ff4a1c861187e9ee64e69c6032e3480, as of writing that's tagged as 0.21-alpha.0; not known yet if it lands in 0.21!) I'm down to 14ms while playing & at stills so far. Profiler now with much less reduced overhead (but still significant in places) tells me that transform context is now 0.07ms


feat(#692): save filters to local storage

Description
Saving selected filters to the local storage.
discord username: İlker Sarica#7138
closes #issue
#692
Please describe your changes. Also describe your aim and content. Do not forget to list the dependencies required caused by those changes.
## Things to check before creating a PR

[ ] I have inspected my topic, checked.
[ ] If it is a core feature, executed detailed tests.

Creating PR rules

[ ] PR must be created for an issue with approved tag. Otherwise PR will be rejected.
[ ] Relevant issue number: The issue number related to PR must be attached to head of PR header, after prefix after prefix # must be attached in parenthesis. A header like this could be used "prefix(#issue_number): PR header" .
[ ] A descriptive and understandable title: The PR title should clearly describe the nature and purpose of the changes. The PR title should be the first thing displayed when the PR is opened. And it should follow the semantic commit rules. For example, a title like "docs(#issueId): Add README.md" can be used.
[ ] Related file selection: Only relevant files should be touched and no other files should be affected.
[ ] Format and Lint Suitability : The code should be made in accordance with a certain format standard and examined according to the lint rules.
[ ] Clean commit history: The commits where the changes are made should be clear and organized.
[ ] Opening PR when job is completed: PR should be opened when job is completed and sent for review by other team members.

Changes

[ ] A new feature (a change that adds a new feature, not a breaking change)
[ ] A new refactor (a change that is not a breaking change, that improves the readability or performance of the code)
[ ] A breaking change
[ ] Documentation change

How were these changes tested?
Please describe the tests you did to test the changes you made. Please also specify your test configuration.
Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Filtreleri store'a zustand-persist ile kaydedip geri okusak kod daha temiz olabilir

TR: PR'ların bundan sonra ingilizce olarak gönderilmesi gerekiyor ve halihazırda açık olanlara da bu talebimizi iletiyoruz. O nedenle sizden ricamız bu PR'ı ingilizce olarak düzenlemeniz. Lütfen ingilizce olarak düzenler misiniz?
EN: "PR" must be sent in English from now on, and we forward this request to those who are already open. Therefore, we request you to edit this "PR" in English. Could you please edit it in english?
https://github.com/acikkaynak/deprem-yardim-frontend/blob/development/.github/PULL_REQUEST_TEMPLATE.md

TR: PR'ların bundan sonra ingilizce olarak gönderilmesi gerekiyor ve halihazırda açık olanlara da bu talebimizi iletiyoruz. O nedenle sizden ricamız bu PR'ı ingilizce olarak düzenlemeniz. Lütfen ingilizce olarak düzenler misiniz?
EN: "PR" must be sent in English from now on, and we forward this request to those who are already open. Therefore, we request you to edit this "PR" in English. Could you please edit it in english?
https://github.com/acikkaynak/deprem-yardim-frontend/blob/development/.github/PULL_REQUEST_TEMPLATE.md

will do.

I think we should be using the URL as our source of truth here. Put the parameters to the URL as a query param, update the URL and listen for the URL params in the relevant components without extra state in between. We can put the logic into a useMapOptions hook and use that in our components. I can take this or help with if needed.
This is important because usability and ease of sharing is crucial. People can change the options and share the link with a friend or someone on the ground.

@batuhan, @puskuruk also has the same idea here https://github.com/acikkaynak/deprem-yardim-frontend/issues/707 maybe you can two pair up on Discord 👍

I think we should be using the URL as our source of truth here. Put the parameters to the URL as a query param, update the URL and listen for the URL params in the relevant components without extra state in between. We can put the logic into a useMapOptions hook and use that in our components. I can take this or help with if needed.
This is important because usability and ease of sharing is crucial. People can change the options and share the link with a friend or someone on the ground.

Makes sense. I think it's not that bad idea to also keep this. Could be useful for reaching the last state directly at https://afetharita.com/. Maybe we can reduce repetitive code but not that crucial.

I think we should be using the URL as our source of truth here. Put the parameters to the URL as a query param, update the URL and listen for the URL params in the relevant components without extra state in between. We can put the logic into a useMapOptions hook and use that in our components. I can take this or help with if needed.
This is important because usability and ease of sharing is crucial. People can change the options and share the link with a friend or someone on the ground.

I agree there are a lot of improvements can be done here, for example there are 2 states which can be just one, but I honestly didn't touch the parts of outside of the feature. So I didn't refactor anything.

We've tested this pr it seems to work as expected we didn't see any problem. Thank you :)

I agree with the comments, seems like keeping this in the URL is a better option. We have #740 for this but it seems to have some issues. So I'd prefer to focus fixing those 👍
Also instead of setting those here and there, it might be better to manage those states in a single place and sync the URL with the application. @puskuruk was working on such approach last time we talked.

Closing this, please reopen it when conflicts are fixed.

Closing this, please reopen it when conflicts are fixed.

is it necessary? I think another PR is in consideration instead of this one.


Workaround for godot empty array synced bug

There are some cases where I've noticed that somehow the export variable of one RoadPoint is somehow synced with that of another, even though they are distinct instances. This twitter post seems to have run into the same situation, and found an intersting workaround: (subbing in the offending variable name in RoadPoint)
export var traffic_dir:Array = []
func _init():
    traffic_dir = []  # clear the reference.

This does not seem to be an issue in Godot 4, so we would not need to keep this fix in Godot 3.5 if we do make this change.
Recording of the specific error in question:
https://user-images.githubusercontent.com/2958461/229102603-dedc8f96-313f-4c61-8f6c-ec894d31c6e7.mp4

Closing this as the bug has been resolved for a little while now (per PR).


The requested platform 'BroadlinkRM' was not registered by any plugin.

Hello,
I recently tried reconfiguring the Broadlink RM plugin after not using it for a period of time. I have verified that the plugin is installed (via npm install -g homebridge-broadlink-rm). Node is up to date. JSON is validated. However, I receive the error below whenever I try to start homebridge.
Error: The requested platform 'BroadlinkRM' was not registered by any plugin.
at API.platform (/usr/local/lib/node_modules/homebridge/lib/api.js:134:13)
at Server._loadPlatforms (/usr/local/lib/node_modules/homebridge/lib/server.js:327:45)
at Server.run (/usr/local/lib/node_modules/homebridge/lib/server.js:90:36)
at module.exports (/usr/local/lib/node_modules/homebridge/lib/cli.js:59:10)
at Object. (/usr/local/lib/node_modules/homebridge/bin/homebridge:17:22)
at Module._compile (internal/modules/cjs/loader.js:1144:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1164:10)
at Module.load (internal/modules/cjs/loader.js:993:32)
at Function.Module._load (internal/modules/cjs/loader.js:892:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
at internal/main/run_main_module.js:17:47

Rolling back to an earlier version of the plugin resolved the issue.

Rolling back to an earlier version of the plugin resolved the issue.

Hello there. how do you rollback to a earlier version? thanks.

I uninstalled the plugin via npm uninstall -g homebridge-broadlink-rm and then chose an older version and installed that via npm install -g<EMAIL_ADDRESS>(though you can specify another version of your choice after the '@').

This process doesn’t work when using HOOBS.
Lprhodes - is there a new version without this issue on the way? I have lost all Broadlink accessories since updating, and my json file is gone.

I'm in the same boat as Mattserati, I also can't rollback via HOOBS. Any thoughts?
{     "server": {         "port": 8080,         "origin": "*",         "autostart": 0,         "home_setup_id": "X-HM://0023RCC5CMF04",         "polling_seconds": 5     },     "client": {         "default_route": "status",         "inactive_logoff": 30,         "theme": "throwback-blue",         "locale": "en",         "temp_units": "celsius",         "country_code": "AU",         "postal_code": "2250",         "latitude": "-33.390517",         "longitude": "151.341377"     },     "bridge": {         "name": "XXX",         "port": 51826,         "pin": "XXXXXX",         "username": "XXXXXXXX"     },     "description": "XXX house-hold powered by HOOBS",     "ports": {},     "accessories": [],     "platforms": [         {             "platform": "TuyaWebPlatform",             "name": "TuyaWebPlatform",             "options": {                 "username": "username",                 "password": "password",                 "countryCode": "AU",                 "platform": "smart_life",                 "pollingInterval": 10             },             "plugin_map": {                 "plugin_name": "homebridge-tuya-web"             }         },         {             "platform": "Camera-ffmpeg",             "plugin_map": {                 "plugin_name": "homebridge-camera-ffmpeg"             },             "name": "Camera ffmpeg",             "cameras": [                 {                     "name": "Backyard",                     "videoConfig": {                         "source": "-re -i rtsp://<IP_ADDRESS>:7447/h8GRIZn4OBj0qMRc",                         "maxStreams": 2,                         "maxWidth": 1280,                         "maxHeight": 720,                         "maxFPS": 30                     }                 },                 {                     "name": "Frontyard",                     "videoConfig": {                         "source": "-re -i rtsp://<IP_ADDRESS>:7447/JzMFOEip1dlqDRn1",                         "maxStreams": 2,                         "maxWidth": 1280,                         "maxHeight": 720,                         "maxFPS": 30                     }                 }             ]         },         {             "platform": "Alexa",             "plugin_map": {                 "plugin_name": "homebridge-alexa"             },             "name": "Alexa",             "username": "username",             "password": "password",             "pin": "xxxxxxxxxxx",             "routines": true,             "debug": true,             "refresh": 15,             "oldParser": false         }     ] }

One thought for HOOBS users, does the fork homebridge-broadlink-rm-tv work for you?

One thought for HOOBS users, does the fork homebridge-broadlink-rm-tv work for you?

Maybe I can try tonight. But does that work with Air Con units?

Maybe I can try tonight. But does that work with Air Con units?

The fork is built from here so supports everything this does. We've only extended the existing functionality.  IMO, it works even better with Air Con units than this does.

Maybe I can try tonight. But does that work with Air Con units?

The fork is built from here so supports everything this does. We've only extended the existing functionality. IMO, it works even better with Air Con units than this does.

But the broadlink-rm-tv doesn’t work with the latest Broadlink mini 3 (device code 5f36)

Device support is dependant on the broadlinkjs-rm plugin which both plug-ins use.  If you're having issues with the device on broadlink-rm-tv and not broadlink-rm then please raise and issue on that fork and I'll take a look.

Device support is dependant on the broadlinkjs-rm module which both plug-ins use. If you're having issues with the device on broadlink-rm-tv and not broadlink-rm then please raise and issue on that fork and I'll take a look.

Thank you!
No way to use broadlink-rm

The requested platform 'BroadlinkRM' was not registered by any plugin

And broadlink-rm-tv does not support my device

Please raise an issue in the GitHub repository (https://github.com/lprhodes/homebridge-broadlink-rm/issues) with details of the type of device and its device type code: "5f36". The device is connected to your network with the IP address “x.x.x.x"

I guess I should use the latest version of broadlink-rm which added support for “5f36”.
I tried to star from the beginning with a fresh install of the system with the same result.

Device support is dependant on the broadlinkjs-rm module which both plug-ins use. If you're having issues with the device on broadlink-rm-tv and not broadlink-rm then please raise and issue on that fork and I'll take a look.

Thank you!
No way to use broadlink-rm
The requested platform 'BroadlinkRM' was not registered by any plugin
And broadlink-rm-tv does not support my device
Please raise an issue in the GitHub repository (https://github.com/lprhodes/homebridge-broadlink-rm/issues) with details of the type of device and its device type code: "5f36". The device is connected to your network with the IP address “x.x.x.x"
I guess I should use the latest version of broadlink-rm which added support for “5f36”.
I tried to star from the beginning with a fresh install of the system with the same result.

Thanks for your advice. Got the most process with that link (using home bridge now). However now I am getting the following error when I link on 'Learn' in HOME.APP.
[2/28/2020, 10:05:23 PM] [Broadlink RM] [ERROR] Learn Code (Couldn't learn code, device not found)

I uninstalled the plugin via npm uninstall -g homebridge-broadlink-rm and then chose an older version and installed that via npm install -g<EMAIL_ADDRESS>(though you can specify another version of your choice after the '@').

It worked. Thanks.

Sorry @josehipo, npm is a version or two behind github with the broadlink-rm-tv plugin so it's not pulling the latest broadlinkjs-rm module version. Hopefully we can get npm updated ASAP so the device support is the same.

Sorry @josehipo, npm is a version or two behind github with the broadlink-rm-tv plugin so it's not pulling the latest broadlinkjs-rm module version. Hopefully we can get npm updated ASAP so the device support is the same.

Bien! Now I understand! I will wait patiently
Thank you!

Maybe I shouldn't ask this here, but there is something that I don't understand:
To be able to use the TV type I need to install broadlink-rm-tv and for other devices (Switch type) I need broadlink-rm... Can I install both or do I have to choose?


Consider renaming tools-python to spdx-tools-python

The fact that the python package does not match the repository name and that the "tools-python" as a name does not say anything is causing confusions.
I already made the big mistake of cloning and installing another repository that happened to be named 'spdx-tools'... Basically the unfortunate decision to not use the same name on repository and python package name, will make victims.
While for someone working on spdx full time, that is implied but for almost everyone else 'tools-python' does say nothing.

yeah, that is a confusing situation. We are discussing how to make it clearer.


Handle empty POST data in Vendor API request log

Context
Some POST requests to the API do not require data, so there's no need to parse as JSON, currently this shows an error that the requests data was not valid JSON.

Changes proposed in this pull request
Handle empty request data for POST requests.

Guidance to review

Link to Trello card
https://trello.com/c/EtKnhZQM/3352-support-vendor-api-requests-screen-post-requests-with-no-request-body-display-incorrect-data

Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] New environment variables have been added to the Azure config

@benswannack I've added the QUERY_STRING value from Rack as a method-agnostic way of seeing the original querystring.


docs(demo): add hide fields with animations example

Codecov Report

Merging #916 into next will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             next     #916   +/-   ##
=======================================
  Coverage   93.61%   93.61%
=======================================
  Files          22       22
  Lines         940      940
  Branches      257      257
=======================================
  Hits          880      880
  Misses         60       60

Continue to review full report at Codecov.

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

need to update it into the latest version https://github.com/formly-js/ngx-formly/issues/800#issuecomment-386017066

the funniest thing is I don't like angular animation at all :laughing:

haha 😀. To be honest I didn't use them too much either, but those few times I had to use them they were quite easy to implement 🙂


Fix: Prevent "Uncaught RangeError: Maximum call stack size exceeded"

I made this pull request including code from this: http://trentrichardson.com/2011/07/29/many-updates-to-timepicker/#comment-32649

Thanks, I added this to the dev branch.


patch usage

I use patch/put like the document mentioned:
o('http://services.odata.org/V4/OData/OData.svc/Products(1)').patch({Name:'NewName'}).save(function(data) {
    console.log("Product Name changed");
});

but a exception throw:
Bulk updates are not supported. You need to query a unique resource with find() to patch/put it.
so, how can I use it correct?
thx

I can't rebuild your error. You are using it correctly. Does the get function work? The tests are working correctly.
Do you use integer or sting as id?

yes, the get function work.
I build it on jsfiddle:
https://jsfiddle.net/billskate/0Ltbvwwz/1/
I just copy and paste the patch's sample code, the same exception occured.

Thanks. This is a bug if you patch something without a endpoint configured. I sos updated the repo directly. Please try again with the new release.
One note: The Product-Service of odata.org seems to be read only. To test this, please use the TripPinService: http://services.odata.org/V4/(S(wptr35qf3bz4kb5oatn432ul))/TripPinServiceRW/


path exclusion in no longer os-dependent

Hey, I just made a Pull Request!
Description
Excluding path is no longer os-dependent, by using os.path.normpath.
Motivation and Context
Fixes #51
Have you tested this? If so, how?
I ran interrogate with these changes over relevant cases and it works for me.
No additional unit tests required.
Checklist for PR author(s)

[x] Changes are covered by unit tests (no major decrease in code coverage %).
[X] All tests pass.
[X] Docstring coverage is 100% via tox -e docs or interrogate -c pyproject.toml (I mean, we should set a good example :smile:).
[X] Updates to documentation:

[X] Document any relevant additions/changes in README.rst.
[X] Manually update both the README.rst and docs/index.rst for any new/changed CLI flags.
[X] Any changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives.  Find the appropriate next version in the project's __init__.py file.

Someone didn't install the pre-commit hook 😉

@oriash93 if @econchick is fine with the proposed feature in #59 , could you change
if any(
                [f.startswith(os.path.normpath(exc)) for exc in self.excluded]
            ):

to
if any( fnmatch(f, exc) for exc in self.excluded ):

That way this PR could kill two birds with one stone.

Whatever @econchick decides.
Also, on a different note - do you mind marking this PR or repo as eligible for hacktoberfest?

Thinking about it, the replacement should be
if any( fnmatch(f, exc + "*") for exc in self.excluded ):

or if the python 3.5 support was droped before
if any( fnmatch(f, f"{exc}*") for exc in self.excluded ):

That way it won't break configs that work with the current startswith implementation.
In [1]: fnmatch(r"project\tests\test_file.py", "project/tests")
Out[1]: False

In [2]: fnmatch(r"project\tests\test_file.py", "project/tests*")
Out[2]: True

In [3]: fnmatch(r"project\tests\test_file.py", "project/tests**")
Out[3]: True

In [4]: fnmatch(r"project\tests\test_file.py", "project/tests***")
Out[4]: True

In [5]: fnmatch(r"project\tests\test_file.py", "project/tests****")
Out[5]: True

Thanks @oriash93 @s-weigand (btw I labeled the repo w/ hacktoberfest - will that be enough?)
I'm cool with fnmatch if it makes #59 easier, although I'm not sure it does. I guess it depends on what's wanted: if we want to support regexes, then fnmatch may not be the best way since it does not support re regex. If #59 just wants to support unix-style wildcards, then it should be enough. That said, I'm not that familiar with fnmatch myself, so I could be wrong :)

fnmatch uses unix-style glob patterns, which I guess most people are more familiar with than RegEx.
At least I hear all the time "I'm scared of RegEx" or "I never understood RegEx" 😋
Also fnmatch is the way other tools like flake8 handle their file matching.
In #59 I was just asking for a less explicit/ more general way to exclude files, so glob patterns are totaly awesome for me. 😄

pinging @econchick

Thanks so much! Sorry for the delay - I was on vacation for the past week and a half 😄

FYI this has been released (finally) in 1.3.2 🎉  - thanks again for you're help @oriash93 & @s-weigand 🥇


Why is Tweet::text truncated sometimes?

I have just reposted a tweet and it's text is:
text: "RT @QuakeRussia: Quake Champions BETA countdown April 27: The next QC Beta will start at 16:00 CEST (10:00AM EST) Read More https://t.co/OT…",

However in the same structure if we try to see entities::retweeted_status it's text is:
text: "Quake Champions BETA countdown April 27: The next QC Beta will start at 16:00 CEST (10:00AM EST) Read More https://t.co/OTcq6H6sXU",

The full object looks like this:
bug.txt

That's a normal behavior in Twitter's spec. Twitter truncates text field of a retweet if the text (including RT @user_name: ) would exceed 140 characters. See the doc for Tweet.truncated field.
So, in case of a retweet, it's recommended to refer to text field of Tweet.retweeted_status, instead of Tweet.text.
Maybe I should write additional docs for this pitfall later.

@dmizuk Nice, thanks much for the explanation.


deleteCount param to splice() is not optional in es5 types

TypeScript Version:  3.5.1

Search Terms: splice deleteCount
Code
Set the target to ES5 then
[1, 2, 3].splice(0); // should error. deleteCount missing
[1, 2, 3].splice(0, 3); // ok

Expected behavior:
[1, 2, 3].splice(0); not allowed
Actual behavior:
[1, 2, 3].splice(0); allowed
Playground Link: this
deleteCount only become optional in ES6
ES5: https://www.ecma-international.org/ecma-262/5.1/#sec-<IP_ADDRESS>
ES6: https://www.ecma-international.org/ecma-262/6.0/#sec-array.prototype.splice

ES5 Array.prototype.splice (start, deleteCount, ...

Let actualDeleteCount be min(max(ToInteger(deleteCount),0), len – actualStart).

ToInteger

Let number be the result of calling ToNumber on the input argument.

ToNumber

Argument Type
Result

Undefined
NaN

ToInteger

If number is NaN, return +0.

max(ToInteger(deleteCount),0) is just max(ToInteger(undefined),0) is just max(+0, 0) is just zero

Unless I'm confused about something, the function's behavior is well-predictable when given one argument.
The spec is not authoritative as to argument optionality since it's not a well-defined concept in JS and ToInteger(undefined) gives 0 which is a good default argument in many cases.
For example Number.prototype.toFixed lists its argument as required, but TypeScript doesn't enforce this because (0).toFixed() is a well-formed and predictable result. Same for String.prototype.slice - its arguments are both listed as "required" but has predictable and expected behavior with one.

Hmm interesting. The reason I opened this is because we had a .splice(0), and then  our app broke in an old version of opera because there the second argument defaulted to 0 (correctly for es5) instead of the array length (explicit in es6).
The issue is the default value essentially changed between es5 and es6.
Given the the default becomes something else in es6, and the param is not marked as optional in es5, do you think it makes sense for there to be an exception in this case?
It's not safe to omit deleteCount unless you know for sure what environment you are running in.

The wording used in the spec is interesting:

When the splice method is called with two or more arguments [...]

Based on this, and the fact that the behavior changed between ES5 and ES6 (despite "don't break the web" being the unofficial ES motto), the default value of 0 therefore seems to just be an accident of the integer coercion rules and not directly intended in the design.  It definitely seems like the older spec wants to treat the second parameter as required, at any rate.
That said, even with the reasoning above, the fact that in practice there is a "default" and it isn't consistent between ES5 and ES6 is quite frustrating.

Yeah, after reading more, it seems like making the 2nd arg required for ES5 and optional for ES6 would be better in this particular case.
I don't think I've ever called splice without a second arg, though, because it's not very useful.

OK, that sounds pretty annoying. Accepting PRs

Thanks for the feedback on this BTW. When/if someone submits a PR it'd be nice if this issue had a clear and concrete description of what happens in ES5 vs ES6 with the various argument counts.

Awesome. I will put together a PR.
I know I have used splice(0) several times in the past. It would have been great to have typescript flag it then :)

I think this is what happens.
Maybe someone can correct me if I made a mistake.

ES
array.splice()
array.splice(start)

ES5
(0, 0)
(start, 0)

ES6
(0, array.length)
(start, array.length-actualStart) *

The actualStart is computed,

Let relativeStart be ToInteger(start).
ReturnIfAbrupt(relativeStart).
If relativeStart < 0, let actualStart be max((len + relativeStart),0); else let actualStart be min(relativeStart, len).

PR: https://github.com/microsoft/TypeScript/pull/32643

Hi! I've run into a related issue. I'm not sure if this is really a bug with TypeScript, the interpreters, or the spec itself. However, the typings are at the very least misleading. ES2022 deleteCount was clarified to be clearly optional, but if you pass undefined as deleteCount splice() will fail silently. I noticed this when I wrapped splice with another class I was working on, which should work according to the typings.
While deleteCount may or may not be optional (your guess is as good as mine), it cannot be undefined.
⏯ Playground Link
I added several cases to test behaviour at the following:
Playground link with relevant code
💻 Code

class FailingWrapper{
  arr = [1,2,3,4];

  splice(start:number, deleteCount?: number){
    // When deleteCount is 'undefined' method fails silently
    return this.arr.splice(start, deleteCount);
  }
}

const wrapper = new FailingWrapper();
const removed = wrapper.splice(2) // deleteCount is undefined

console.log(
  `Case 4: Arr length is now ${wrapper.arr.length}, and removed length is ${removed.length}`
);

🙁 Actual behavior
Case 4: Arr length is now 4, and removed length is 0
splice() failed silently here because undefined was passed as deleteCount.
🙂 Expected behavior
Case 4: Arr length is now 2, and removed length is 2
According to the typings splice() should have worked.
Addendum
It looks like the PR was reverted, but this edge case doesn't seem to be accounted for. @DanielRosenwasser and @RyanCavanaugh can you please take another look at this? It's easy enough to code around, but this is unexpected / undocumented behaviour, and was a serious pain to track down.

That's annoying as heck. We'll have to make it two overloads instead of having an optional parameter.

@RyanCavanaugh how would that work? I just tried adding a splice(start: number): T[]; separate to splice(start: number, deleteCount: number): T[]; and it still allows undefined

To be clear, as noted above, when deleteCount is assigned undefined it defaults to 0. It's just weird because:
arr.splice(number) // deleteCount is length - start
arr.splice(number, undefined) // deleteCount is 0

Behave totally different, when you'd expect they would be the same. I don't know if this should be handled by the transpiler, or simply just documented and warn users of weird behaviour of splice.

Even if it remains optional in ES5, could at least the documentation be clarified? The current documentation @param deleteCount The number of elements to remove. made me wonder what to expect. (Nothing is removed? One element is removed? All elements starting with start are removed?) MDN documents the ES6 behavior, without mentioning that the ES5 spec said it to be required, and that ES5-compliant browsers practically had a different default value. But even if MDN was 100% correct, having to leave the IDE takes time.


Storing data without cookies

[x] Review the documentation: https://docs.clerk.dev/
[x] Go through package changelog files.

Description
I am experimenting with using Clerk with an Ionic mobile app. There is no dedicated package for this so I was attempting to use @clerk/clerk-js but it uses cookies to store data.
Question
Is there a way to pass in a different way to store data?
E.g.
import Clerk from "@clerk/clerk-js";

const store = {
  set: (key, value) => {
    // ...some set code
  },
  get: (key) => {
    // ...some get code
  },
};

const clerk = new Clerk('fronted-api', { store });

Hi @lukeshay.
The Clerk long-lived __client JWT is stored in a cookie only in browsers. For native applications, the JWT can be transferred via the HTTP Authorization header. Our Expo package demonstrates how to use the Frontend API and set the _is_native=1 query parameter if necessary to instruct the API that it's a native application talking.
As for storage, you can build a custom Ionic storage similar to the Expo secure store and pass it to the  via the [tokenCache](https://github.com/clerkinc/javascript/blob/main/packages/expo/src/ClerkProvider.tsx#L14) prop.
Let me know if this helps.

@SokratisVidros Thank you! I will give that a try and close this issue if it works.

This is shown on app startup even with a custom token cache

@lukeshay Do you get that in the native application browser? Did you set the ?is_native=true query string parameters to all the frontend API requests?

@SokratisVidros I was using @clerk/clerk-expo. Do I still need to add that flag when using that package? If so, how do I do that?

If you are using the package you shouldn't get that page. Do you maybe have a sample repo I can take a look at?

Here is a sample repo, https://github.com/lukeshay/clerk-ionic

I see. I am not very familiar with Ionic but to the best of my knowledge the runtime is a webview powered by Capacitor that triggers Clerk browser flow.
In further detail, Clerk uses third party cookies for development instances in order to achieve seamless auth developer experience and avoid too much configuration in order to get started. To allow third party cookies in Safari the Storage access api is required. The modal asks for the user confirmation to enable the Storage access API.
This modal should appear only once in development until the Set up access button is clicked and will never appear in production instances.
Did you try to click on Set up access?

Hey @lukeshay. Any updates on this ☝ ?

I have not bee able to test this but I assume you are correct here

👍 Let me know if you need further assistance.


Identified Improvements for TOTP authenticator

Change the artifact ID to remove "connector" in the name as it is shipped with the product in plugins now
Change the feature ID to remove "connector" in the feature id
https://github.com/wso2/product-is/issues/10997
Improve UX of totp input page (https://github.com/wso2/product-is/issues/11045)
Enroll mobile device for TOTP without a computer (https://github.com/wso2/product-is/issues/10993)
TOTP issue for SaaS applications https://github.com/wso2/product-is/issues/11172

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Fix #483: Autocomplete unintuitive filter behavior with multiple=false

Change autocomplete filter behaviour when multiple=false
Adds a showAllSuggestions state, which is set to true initially.
It is set to false when the input is changed, and is used to control if
the shown suggestions should be filtered based when the multiple prop is false.
Once a value is set, the state will be reset to true.

Maybe this should be initialized via prop?

Maybe so, yeah. Do you imagine the current behavior would be wanted? Which behavior would you prefer to be default? Do you have any suggestions for the prop name?
Also, I just noticed that @grammka already had addressed the same issue in a very similar, but slightly different way in PR #455. It would be good to compare our solutions before merging.

I rly can't imagine when users need current behavior.. If u can give example plz. My thoughts about this: for example we selecting "My Country". I selected Russia and then I want to select other one, so I have to clear field and then type smth else or select from list.. Look at selectize - there is Single Item Select section.

I brought up two other libraries in #483. React select, which behaves similar to selectize and Angular Material, which does not show any suggestions until the input is changed and provides a clear icon.
Going through old issues, this was also brought up in #252:

When we have some selected option, Then user want to change this. Now he need to remove all text from input, then choose onother option from list. I think the best option is:
-Remove text from input, and show all options when user click on input (like a simple select)

I think the best option, is add one props to component which describe that clean input on click or not, and show all options in list when multiple = false.

I think the best option, is add one props to component which describe that clean input on click or not, and show all options in list when multiple = false.

Sure, It's good way. So how to be with my PR?)

I compared the solutions in a4810d8 and 77190b8, and the differences are:

77190b8 has added logic to clear the input on backspace, if the field has not been changed.
77190b8 resets the state on query blur and value change, a4810d8 only resets it on value change. What is the rationale for resetting it on query blur?
The suggestions() method has been refactored differently, but I believe the output is the same.

I thought some more about adding the prop, but couldn't really think of any intuitive way of explaining the different behaviours with a short name, so I'll hold off for now.

@lucaas I like ur solution with reseting state prop on queryChange, I used other way, more complex - set prop to true on focus and revert it on close(blur) / select..)) I think ur code much better ;) Then I'd like u add logic to clear the input on backspace.
I don't know about prop name.. but I have other idea - mbe it's better to add "select" component with such logic?

Ya so I created the fix with #399 cause the selected Item wasn't rendered even if a valid option is already selected. The way its implemented, it acts like a google search field component, where it filters it down and as you delete it, you'll get more options.
I do like the suggestion of populating the other entries of showing all the elements once an element is selected. I think we can do that with a prop, but I'm concerned what effect it will have on the multiple={true} type autocomplete list. I can probably look into this Sunday or sometime next weekend.

I've pushed some new commits with the following changes:

Add prop, showSuggestionsWhenValueIsSet, to control the behaviour.
Clear the query input on backspace, when a value is selected and the behaviour is true.
Refactor the suggestions method to make the different code paths more understandable
Add a new autocomplete to the spec page to show the two different behaviours.


Unprompted Exit

Describe the bug
With OP engaged, unprompted, it started to turn the car into taking the exit. I let it go hoping it would correct its error, but alas, it wasn't to be. I took over the wheel and brought it back into the lane OP should have stayed in.
How to reproduce or log data
I've had issues at this exit off and on in the past. I can't recall which laneful or laneless did what though or the versions at the time.
Device/Version information (please complete the following information):

Device: comma two
Dongle ID: f39975d916ed458a
Route: f39975d916ed458a|2021-08-10--07-05-15--41
Timestamp: 7:46::58 local time is about where it starts
Version: master-ci 0.8.8
Car make/model: 2020 Toyota Corolla Hatchback

Additional context
This particular part of the road has these deep grooves in the road on the left and a seam where the recently paved right size meets up with the older left side. Lighting was pretty poor as the sun was directly over the trees facing that eastern direction. The lead car signaled for 1/2 a mile and slowly took the turn, which maybe factored into the follow.

I am happy to report, at the same exit today, OP did not take the exit!
f39975d916ed458a|2021-08-11--06-51-19--43
7:34::42 local time
comma 2 on master-ci 0.8.8
2020 Toyota Corolla Hatchback
Additional context
I had just finished an automatic lane change. It hugged a little bit towards the exit (maybe a full tire width shifted over) and then stopped turning and just continued on pretty straight.

I was not able to hit that exit today. On a different exit today, it hugged and thought about going but didn't. This exit was extremely well marked and it still thought about taking it.
Device: comma two
Dongle ID: f39975d916ed458a
Route: f39975d916ed458a|2021-08-12--06-46-02--24
Timestamp: 7:10::50 local time
Version: master-ci 0.8.8
Car make/model: 2020 Toyota Corolla Hatchback
Of note, I took some automatic lane changes in the sharp curves before the exit... and it struggled a little bit but overall wasn't bad.

New model has had some fixes to the exit behavior, should be big improvement! Lemme know if it worked.

I'll try laneful tomorrow, but today at that troubled exit it took the exit again on laneless (I updated the master-ci right before the drive).
To be fair, there was a lot going on with a cut in, it's a two lane exit with a car taking the far right lane exit. It's probably an edge case. I had a car seemingly passing me on the right as they took the exit, so I couldn't let it go into that lane and took over pretty quickly. It might have caught that line at the last second and corrected but hard to say. I can switch between laneless and laneful if it would help?
comma 2
f39975d916ed458a|2021-08-16--09-09-32--5
9:14::45 local time the start of the exit process
2020 Toyota Corolla Hatchback

New model has had some fixes to the exit behavior, should be big improvement! Lemme know if it worked.

Took this exit in the POURING rain from Tropical Depression Fred... OP did it PERFECT on laneful!
comma 2
f39975d916ed458a|2021-08-17--06-59-58--50
7:50::39 local time the start of the exit process
2020 Toyota Corolla Hatchback
Looking at the high resolution, I think it was following the tire tracks in the rain. But, either way, it took this exit solid.

Great!

Great!

Even better one on Wednesday. So truly and utterly solved, it's like magic.
comma 2
master-ci 0.8.8
2020 Toyota Corolla Hatchback
f39975d916ed458a|2021-08-18--06-57-39--42
07:39::45 local time start of the exit

Great!

Unfortunately, on laneless, it took the exit =/
comma 2
master-ci 0.8.8 (updated right before the drive)
2020 Toyota Corolla Hatchback
08:35::00 Local time
f39975d916ed458a|2021-08-19--07-49-33--45
I also had a poor 'exit' take with laneless. Basically, it cut way to close to that other entrance and guardrail.
Same drive as above...
f39975d916ed458a|2021-08-19--07-49-33--29 and 30
08:19::30 local time


Value encrypted by RijndaelManaged via .NET Standard 2.1 cannot be decrypted by ASP.NET MVC .NET 4.7.2

Hi,
I have a weird of behavior to decrypt value that was previously encrypted by an asp.net core application.
I use this code in asp.net core to decrypt successfully the value :
private static byte[] DecryptBytes(this byte[] encryptedBytes, byte[] iv, byte[] key, byte[] encryptionKey)
{
  byte[] cipherBytes = encryptedBytes;
  byte[] keyBytes = new Rfc2898DeriveBytes(Encoding.UTF8.GetString(encryptionKey), key).GetBytes(256 / 8);
  var symmetricKey = new RijndaelManaged()
  {
    Mode = CipherMode.CBC,
    Padding = PaddingMode.None
  };

  var decryptor = symmetricKey.CreateDecryptor(keyBytes, iv);
  using (var memoryStream = new MemoryStream(cipherBytes))
  {
    using (var cryptoStream = new CryptoStream(memoryStream, decryptor, CryptoStreamMode.Read))
    {
      byte[] plainBytes = new byte[cipherBytes.Length];
      int decryptedByteCount = cryptoStream.Read(plainBytes, 0, plainBytes.Length);

      memoryStream.Close();
      cryptoStream.Close();

      return plainBytes;
    }
  }
}

The exact same code is use by an ASP.NET MVC .NET 4.7.2 application but the plainBytes return is not readable.
I had checked values of parameters in both framework and they are identicals. So my conclusion is that frameworks are not working the same. I have notice a difference of properties between symmetricKey.
Am I wrong in my conclusion ?
Thank you !

The last time I helped someone investigate a cryptography issue I was told that @bartonjs is the go-to expert in this area.

I made a test this morning.
I created a asp.net core .net 5 project with a controller that received all parameters that .NET 4.7.2 is using to decrypt. I copied the function showed in previous post in the new project to be used by the controller.
So in my .net 4.7.2, I made a called to the controller of the new project by passing all parameters. The controller has used the decrypt function successfully.
That's confirmed that RijndaelManaged has not the same behavior between framework.

The main thing I see out of the ordinary is you're assuming that CryptoStream.Read will fill your buffer completely.  (This is generally true on .NET Framework and .NET 5 and below, but is no longer true on .NET 6)
int decryptedByteCount = cryptoStream.Read(plainBytes, 0, plainBytes.Length);

should instead be
int read = 0;
int decryptedByteCount = 0;

while ((read = cryptoStream.Read(plainBytes, decryptedByteCount, plainBytes.Length) != 0)
{
    decryptedByteCount += read;
}

Other than that, we'd need some actual inputs to test to see what's going on.  The most suspicious thing is that you're turning a byte[] called encryptionKey into a string via Encoding.UTF8.GetString(byte[]); so you may be hitting differences in UTF-8 handling across versions of .NET.  (When debugging, check that keyBytes actually got the same value in both framework versions)

Hi,
@bartonjs you are completely right !!
I have replace
byte[] keyBytes = new Rfc2898DeriveBytes(Encoding.UTF8.GetString(encryptionKey), key).GetBytes(256 / 8);
by
byte[] keyBytes = new Rfc2898DeriveBytes(encryptionKey, key, 10000).GetBytes(256 / 8);
Now all is working like a charm !
Thank you help me to have good direction !!


Check Run comments

Need to check with Craig.
Runs 1,3 and 4 seem to have the standard comments.
Runs 2 and 3 have no comments.
Run 6 seems to have a different style of comment? using the trac barcodes.


Detect unpinned versions in GitHubCI

Let's update the ReadMe as part of this PR.

Let's update the ReadMe as part of this PR.

Done

Let's rebase this PR to get rid of the red CI.

Let's rebase this PR to get rid of the red CI.

Done


Fix tinkerbell IP in hegel urls controller

Issue #, if available:
Description of changes:
Fix tinkerbell IP in hegel urls controller
Testing (if applicable):

manual test
unit tests

Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm
/approve


Not able to create an aid package without changing its name first.

[ ] Not able to create an aid package without changing its name first. Will throw an error stating that 'something went wrong'.

Tracked at https://github.com/LSFLK/MedicinesforLK/issues/300
@anjula-sack Could you please make the inputs visible by default?

Tracked at https://github.com/LSFLK/MedicinesforLK/issues/354
Closing the issue


Cannot build app package due to nutc_driver.exe 1.7.6 out of memory error

Creating an App Package for publication in the store results in the following errors:
Error: NUTC300F:Internal Compiler Error: Native compilation failed due to out of memory error .App
Error ILT0005: 'C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\1.7.6\tools\x64\ilc\Tools\nutc_driver.exe @"C:\Users\myself<path><to><our>.App\obj\x64\Release\ilc\intermediate\MDIL<our>.App.rsp"' returned exit code 1 .App
We are targeting Surface Hub (Creators Update 1703) so we cannot use newer version of microsoft.net.native.compiler.
We have found not documented parameter for nutc_driver (/d2threads1) that helps building the solution on local machines.
Is there any way to pass this flag on build agents?
Are there any other workarounds?

Unfortunately, the option to use d2threads1 on build agents only became available in microsoft.net.native.compiler 2.0. (For completeness, the way to enable it in post 2.0 compiler is to set the <SingleThreadNUTC>true</SingleThreadNUTC> property in your main csproj.)
Can you try steps described here: https://github.com/dotnet/corert/issues/7408#issuecomment-490601593?

Setting  <Assembly Name="AssemblyX" Dynamic="Auto" /> for some of the assemblies solved the problem.
Thanks.


Revert "Remove all other version of debugpy"

Reverts microsoft/vscode-python#11689

Smoke tests passed for me when i ran them on my local GHA https://github.com/karthiknadig/vscode-python/runs/655479282?check_suite_focus=true
I am not sure why this is failing. lets try reverting and see if it addresses the GHA problem.

@karrtikr Did you run on local GHA after reverting?

@karthiknadig On it

Ongoing run: https://github.com/karrtikr/vscode-python/actions/runs/99513055

Codecov Report

Merging #11707 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #11707      +/-   ##
==========================================
- Coverage   60.71%   60.70%   -0.01%
==========================================
  Files         629      629
  Lines       33999    34007       +8
  Branches     4793     4795       +2
==========================================
+ Hits        20643    20645       +2
- Misses      12350    12354       +4
- Partials     1006     1008       +2

Impacted Files
Coverage Δ

src/client/debugger/extension/adapter/factory.ts
96.87% <100.00%> (+0.44%)
:arrow_up:

...ient/debugger/extension/adapter/remoteLaunchers.ts
100.00% <100.00%> (ø)

src/datascience-ui/react-common/arePathsSame.ts
75.00% <0.00%> (-12.50%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

Continue to review full report at Codecov.

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

@karthiknadig Smoke tests pass locally on GHA

Looks like the logging change broke the TS debugger:
C:\GIT\issues\s p\vscode-python>git bisect good 458f5456a8306c39126328edb786d6300fd79909
Bisecting: 8 revisions left to test after this (roughly 3 steps)
[bd22b95dd12c1d41f322f14932349c9946092ba3] Fix storage not being used (#11649)

C:\GIT\issues\s p\vscode-python>git bisect good
Bisecting: 4 revisions left to test after this (roughly 2 steps)
[226b8f8a64be919a6661c014a484b271bbc222fe] Fix scrolling (#11681)

C:\GIT\issues\s p\vscode-python>git bisect good
Bisecting: 2 revisions left to test after this (roughly 1 step)
[79840763ea52dcff4b5cd6f6b1926b10b93f22d1] Leave pipenv in a corner until the user decides to select an interpreter (#11654)

C:\GIT\issues\s p\vscode-python>git bisect bad
Bisecting: 0 revisions left to test after this (roughly 0 steps)
[5aaca27b5f5e058334963f8b4716e1be287a68f9] Logging cleanup. (#11606)

C:\GIT\issues\s p\vscode-python>git bisect bad
5aaca27b5f5e058334963f8b4716e1be287a68f9 is the first bad commit
commit 5aaca27b5f5e058334963f8b4716e1be287a68f9

    Logging cleanup. (#11606)

    For #9837

:040000<PHONE_NUMBER>403b6d01ae8ffbfa66a684852ff06e50c620 46e809c6add0f4723c35eb7e4ff31c2c1d1d6f9e M      src

we need to revert that one not this one.


Testing Searching algorithms

Could you please upload some working examples of binary search algorithm.

Made for sorting algorithms but you can use functions in this library like this.
if __name__ == "__main__":
    sizes: List[int] = [10, 100, 1000, 10000, 100000]
    maxIter: int = 5
    times: List[float] = []

    for size in sizes:
        arr = lib.genRandomArray(size)
        timeTaken = 0.0

        for i in range(maxIter):
            start = default_timer()
            result = binary_search(arr, 0, len(arr) - 1, target)
            end = default_timer()
            timeTaken += end - start
            if result != -1:
                print("Element is present at index", str(result))
            else:
                print("Element is not present in array")
        timeTaken /= maxIter
        times.append(timeTaken)

    complexity = lib._estimate(sizes, times)
    cplx = complexity._to_str()
    print(cplx)


Add sensor-bindings support, also optional fields

Summary

added sensor-bindings to the Layer struct

moved the EndBrace to a separate field since sensor-bindings is optional

make the Configs field optional
make the Combos field optional
add a test for the default sofle/keyhive keymap

tl;dr I tried to run on the default sofle/keyhive keymap, and it threw up because it didn't have &mt, combos within the keymap block, and included sensor-bindings in the layers. So I added logic to handle each of those, since they all seem to be optional for a functional keymap file.
In checking for those, I saw there are a few more fields that are optional that could get added later, but I've only handled for the ones I've run into and can test.
Thanks for the library!

Thank you so much for your contribution!
Just merged it to main branch and released a new version with your changes!


Multiple sources/targets in one config file

Hello,
Is it possible to have multiple source/target configurations in one config file?
I have use case where i need to sync multiple helm charts from different sources.
Best regards,

I think the problem will be that one might see the same helm chart name e.g. podinfo in multiple sources with different versions. This might lead to unwanted syncs.
I've tried putting multiple documents in one yaml, without success also. I think this would be the better solution. @tompizmor What do you think?


[shopping-cart]  Use red hat docker images for Java applications

https://access.redhat.com/containers/#/registry.access.redhat.com/redhat-openjdk-18/openjdk18-openshift
We are currently using adoptopenjdk/openjdk8.

This should probably use the new Universal Base Image, which can be used by both Red Hat customers and anyone else: https://www.redhat.com/en/blog/introducing-red-hat-universal-base-image
The original image linked in the description is only available to OpenShift customers, so we would have had to support two different base image options. Now we don't have to!
Unfortunately, they don't currently provide a base image with a JVM, but you can use the one that includes yum and use it to install the OpenJDK 8 package.


[release-1.10] Time.tickers that are no longer in use should be closed

This is an automated cherry-pick of #34133

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio_release-1.10
e23f1f4eaaa16072a3bde4a31e67a143b63a28bd
link
/test integ-security-multicluster-tests_istio_release-1.10

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@googlebot I consent


Resource server nginx fixes

K8s nginx ingress already has the headers-more module

This is no longer needed


-Scope Parameter on several Cmdlets not validated. Is this intentional?

While pestering around on one of my personal modules, I noticed that the scope parameter on Cmdlets like Get-Alias, Set-Variable is not Validated. It's just... open. Don't we have defined scopes, like "Local", "Script", "Global", "Private" and the numbers? In fact, after examining the Pester tests included in Powershell/test, those are the only valid values tested, and only up to scope 1. Who uses more than scope 2 anyway?? I'm going to rant and spitball here, so please bare with me.
Can we tag a ValidateSet attribute onto these parameters without breaking the intent of -Scope?
Seasoned users know the scopes. And if we forget, I think we know to Get-Help about_Scopes. But new users may not, and with the amazing discoverability PowerShell has built-in, they may see they can set scopes on their variables and aliases. But, what scopes they can define aren't immediately available to them in the terminal. With tab completion set from a validation, that's remedied.
The only issue I see is: Where's the limit on numbered scopes? How to we get them? Are the scopes defined by the session state so if you enter in a scope 3 when you only have a scope 2, do you get a null or an exception? How high do we go and still have a valid scope integer? If there is a limit that is determined by the session or stored in the ps session state, could we shift -Scope to be a Dynamic Parameter that pulls it's allowable values from "the list" and from the "current allowable numbered" scopes?
My current solution is to add tab completions for scope parameters. It will cycle through the list, and if I want to change it to a number, I can. They aren't used much anyway. This may be a big ol'nothingburger but I had to ask.

We could add a completer (not ValidateSet).

We should probably have a custom validator to check that it's either an integer value or a named scope.

@iSazonov A completer is my current solution. It works just fine. But, if a user really wants to put Some-Cmdlet -Scope Bork, they still can and the command will, well, bork. I don't know if that's an issue or not.
@vexx32 Are you speaking of a custom class derived from ValidateArgumentsAttribute? ...that seems easier than a dynamic validation and digging through the instance for scopes.

@endowdly yes, indeed. You could easily model off of ValidateSet and do something like that, except if it doesn't match any of the existing defined scopes it will still accept it if it's an integer. :)

I only see a value for UX with the completer. If an user do wrong input the user will get the same error message in both cases - without validateset and with one.

So I've been working on the Validator Attribute. It seems to work okay in testing so far, but I haven't had a lot of time with it. I really like this as a solution as it adds functionality to PowerShell that users can apply to their scripts. Are you building a cmdlet and want a -Scope parameter, where validation is kind of important? This can definitely help with that. Just throw a [ValidateScope()] attribute on it and continue on.
For now, I use TabExpansionPlusPlus and this completer:
$ScopeCompleter = {
    param ($commandName, $parameterName, $wordToComplete, $commandAst, $fakeBoundParameter)

    "Local", "Script", "Global" | ForEach-Object { New-CompletionResult $_ }
}
$ScopeCompleterArgs = @{
    CommandName = (Get-CommandWithParameter -ParameterName Scope)
    ParameterName = "Scope"
    ScriptBlock = $ScopeCompleter
}

Register-ArgumentCompleter @ScopeCompleterArgs

Validated scope parameters, like on Install-Module do not get overwritten by this.

To get a progress we should get PowerShell Committee approvement for new public API - ValidateScope vs CompleterScope.
/cc @SteveL-MSFT

So I have been learning lot about Scopes and here is a bit of clarification.
Scope Levels Limits
Currently the available scope levels numbers 0-2147483647
If the scope is a number above where global exists it will kick back an error which is why scoping seems to be such a confusing beast.
That being said. As far as a Validator is concerned I am not really certain why we would need a validator for this instead of using a custom ValidateScript because validation is still done at the scope level when you pass parameters through.
function Foo
{
    param(
        [ValidateScript({
            if ([Enum]::GetValues([System.Management.Automation.ScopedItemOptions]) -contains $_ ) {
                return $true
            }
            try {
                if (([int]$_ -ge 0) -and ([int]$_ -le [int]::MaxValue)) {
                    return $true
                }
            }
            catch { }
            throw "Invalid Scope of $_"
        })]
        [String]
        $Scope
    )
}```

I understand we all want tab completion however I feel like having<PHONE_NUMBER>+ Potential tab completions would slow down tab completion drastically where performance factors will be in play.

However I believe a ValidateSetAttribute would be great here if it allowed for Type Information as well.
Example:
[ValidateSet("Global","Script","Private","Local", [int]) ]

@romero126
The numbered scopes isn't an issue for tab completion, and there is no easy way to determine which of the number is a valid scope. I mean there is, if you did through the call stack, but that just seems like a folly. So the tab completion comes down to 4 real choices, and as I mention above, Jason Shirk's TabExpansionPlusPlus module adds completer for every command that uses a -Scope parameter (see here if you want to do it yourself for some reason).
Your ValidateScript solution is fine for a one-off script and for a single user. I'm talking about a systematic ... issue? I still don't necessarily think it's an issue nor an oversight. At this point, I guess I'm talking about an additional feature.
Like I mentioned above, A ValidateScope attribute just makes it easy for validating these scopes for anyone, PowerShell wide.
Here is a Gist of my ValidateScopeAttribute I've been toying with to see what I'm talking about.

@PowerShell/powershell-committee reviewed this and see no reason not to support tab completion for scopes


Update fr_fr.json

In French, colon must have a space before them, as exclamation marks do.
Uppercase only for the first letter.
Some typos are fixed.

Thank you! Can you add your name to CREDITS.txt before I merge?

Done!

Thank you! Merging now.


[WIP] Start of replication monitoring for Trtl sc-2365

This PR adds replication monitoring to Trtl with prometheus.
Done:

number of anti-entropy sessions per peer/per region
anti-entropy session latency per peer

Todo:

number of objects per anti-entropy push vs pull per peer/per region (gauge?)
number of versions stomped (e.g. number of conflicts)
also log on stomps (do we want to keep stomped version around for later merging?)
track number of versions per peer (is this a proxy for counting anti-entropy sessions per peer?)

Codecov Report

Merging #291 (c7b1461) into main (f7dd87b) will increase coverage by 0.03%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##             main     #291      +/-   ##
==========================================
+ Coverage   46.61%   46.65%   +0.03%
==========================================
  Files          57       57
  Lines        7182     7200      +18
==========================================
+ Hits         3348     3359      +11
- Misses       3208     3215       +7
  Partials      626      626

Impacted Files
Coverage Δ

pkg/trtl/replica.go
2.29% <0.00%> (-0.03%)
:arrow_down:

pkg/trtl/metrics.go
56.57% <71.42%> (+3.35%)
:arrow_up:

pkg/gds/certs.go
61.16% <0.00%> (+0.32%)
:arrow_up:

Continue to review full report at Codecov.

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

Closing since we ended up refactoring anti-entropy and the file structure pretty significantly in #299. Will open a new branch


[Bug] should not depend on react

Describe the bug
This package depends on react and react-dom, which both seem like they should be unnecessary
Steps to reproduce the behavior
Install project with Svelte. See React included
Expected behavior
No React for Svelte users
Screenshots and/or logs
N/A
Environment
latest code
Additional context
Tons of stuff is getting installed. Would love to see things slim down

I sent https://github.com/storybookjs/storybook/pull/18646


404 response to /regulator/{id}/ when not all ids could be mapped.

I am curious about the logic for using 404 as the response if not all ids were able to be attached.
404 is defined as "The server has not found anything matching the Request-URI".
If I am reading the docs correctly in the case that not all ids could be mapped the request URI /regulator/{id}/ is valid and a matching resource was found, the issue exists within the array of ids to map.
Since the /regulator/{id}/ transaction doesn't seem to follow ACID I think the request should return a 200 response code since it is possible that some ids were mapped successfully.
Then it is on the caller to check if the failures array has data / exists at all.
Thoughts on this?

This is a really great point. I am happy to change to a 200 with a failure array.

Nice. I think a 200 would result in the least amount of confusion.
That way 404 can be reserved for when the requested resource is not found.

#13

Closing issue as this will be handled with issue #13


CY8CKIT_064S0S2_4343W: Fix aws_tests for OTA tests

CY8CKIT_064S0S2_4343W: Fix aws_tests for OTA tests
Description
OTA PAL and Agent tests were failing to build due to a OTA demo
source file being included. This patch removes the source file
from aws_tests builds.
Signed-off-by: Raymond Ngun<EMAIL_ADDRESS>Checklist:

[x] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks


Add information on GitHub Actions and workflows

The GOV.UK Pay team have been trialling using Github Actions for continuous integration and image builds. We've captured some of the security concerns, and come up with a list of best practices around permissions (and the use of third party actions) for our own docs.
This feels like a useful thing to share more widely, especially given the potential issues around unversioned third party actions. Version tags like dodgyaction@v3 are mutable: it's recommended to pin to a Git SHA instead so that if the provider releases a compromised version under that tag, the workflow will not automatically pick up the bad action.
The other point that may not be obvious for new Github Actions users is that the output of workflow runs is public on public repos. A traceback from a failed unit test run is probably fine, but if a team is trying to do something more complex or sensitive, they should consider carefully whether Github Actions is the right tool for the job.
I've added this information to the existing section on GitHub - happy to move if there's a better place for it.

Also worth adding Github Actions to this list? https://github.com/alphagov/gds-way/blob/3fde1dffacad589e3583025105e8542030167d42/source/standards/source-code.html.md.erb#L40

Also worth adding Github Actions to this list?
https://github.com/alphagov/gds-way/blob/3fde1dffacad589e3583025105e8542030167d42/source/standards/source-code.html.md.erb#L40

Good spot! I've tacked it onto the last commit.

We have found it quite useful, where we have many repos with similar workflows to use Workflow Templates
How about adding something like

Consider creating a Workflow Template in https://github.com/alphagov/.github/tree/main/workflow-templates if you need to share a similar workflow between many repositories.

I've added a line about Workflow Templates, thanks @gidsg.
As per the conversation above, managing secrets is probably a section in itself. I'm not 100% sure what's best to recommend there for the organisation as a whole yet, so I'll leave that out for this PR. @louzoid-gds I think we're ready for review 👍

Of course, the other thing we should really do is review the Github Actions workflows and settings on this repository, and follow our own advice 😹


Add scripts to run vue client using docker

Are we missing anything that's not already in the /docker folder? https://github.com/bcgov/orgbook-bc-client-vue/tree/main/docker

Are we missing anything that's not already in the /docker folder? https://github.com/bcgov/orgbook-bc-client-vue/tree/main/docker

I guess I'm just blind :-D


Adding identity to RegistryCredentials

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@vturecek , the PR contains breaking change, please follow http://aka.ms/bcforapi for the review.

@lirenhe the resource type and version that is changed in this PR has not shipped to customers yet.

@lirenhe the resource type and version that is changed in this PR has not shipped to customers yet, nor has an SDK been generated yet, so this will not be a breaking change.

Could you ping @JeffreyRichter on this? He is responsible for review and signoff in this case.

@lirenhe Jeff is unavailable this week, is anyone else available that can approve?

Hi @vturecek (long time). I was able to approve the PR

Thanks @JeffreyRichter, much appreciated!
@lirenhe can you please merge (or let me know if there is anything else blocking)? I noticed CredScan failed in PostAnalysis, but none of the errors appear to be related to this change or even to this namespace.


Zoom not working when using Fade effect

Hello
it seems that when using { effect: fade, zoom: true } the pinch to zoom isn't working correctly: it zooms on double tap but when moving the zoomed image the positions are wrongly calculated (it seems that zoom functionality works only with default 'slide' effect).
Maybe when a slide is zoomed in, calculations should go to the ones of 'slide' effect, and when a slide is not zoomed in, swiper should apply the user selected effect, but I don't know how to force this behavior.

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 3.4.1.

Platform/Target and Browser Versions: Windows/Chrome.

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Add: hover function is fired to the active element

Please update TypeDefinition

It looks like it's working correctly.
@cicarulez, would you add your new option in the documentation in the readme?  I noticed that it doesn't activate the hover function for disabled menu items.  That may be the most sensible option, but that edge case should be documented in the readme so that users of the extension aren't surprised by how it works.

@cicarulez, are you still interested in adding the readme documentation to this PR for your feature?


Modify README to be more explicit about not mixing binary / source install?

We could perhaps add a disclaimer not to mix source / binary installs, to prevent accidental build :fire:  .

Good idea. Would it be better to add the note to the installation instructions in the open-rmf/rmf repository since the README here is more focused on running the demos?

ah yes, i should change this, thank you!


Invalid topic assignment N from word proposal , something different compared with the former issues

I have the same problem of Invalid topic assignment N from word proposal as the former issues .The former issues all use their owns corpus with the mistake of the TF of words,  But I use the example corpus of nytimes . When I run it by only one machine , it runs well. When I run it by two or four machines using MPI , it will get into the trouble of Invalid topic assignment N from word proposal . If you can tell me why , I will appreciate it very much.
the following is my input order:
OMP_NUM_THREADS=1 mpirun -n 2 -perhost 1 -host node1,node2 $bin/lightlda -num_vocabs 111400 -num_topics 1000 -num_iterations 100 -alpha 0.1 -beta 0.01 -mh_steps 2 -num_local_workers 1 -num_blocks 1 -max_num_document 300000 -input_dir $dir -data_capacity 800
the following is the logging info:
/home/danyang/mfs/lightLDA/example
[INFO] [2016-09-29 12:49:01] INFO: block = 0, the number of slice = 1
[INFO] [2016-09-29 12:49:01] INFO: block = 0, the number of slice = 1
[INFO] [2016-09-29 12:49:01] Server 0 starts: num_workers=2 endpoint=inproc://server
[INFO] [2016-09-29 12:49:01] Server 1 starts: num_workers=2 endpoint=inproc://server
[INFO] [2016-09-29 12:49:01] Server 0: Worker registratrion completed: workers=2 trainers=2 servers=2
[INFO] [2016-09-29 12:49:01] Rank 0/2: Multiverso initialized successfully.
[INFO] [2016-09-29 12:49:01] Rank 1/2: Multiverso initialized successfully.
[INFO] [2016-09-29 12:49:09] Rank 1/2: Begin of configuration and initialization.
[INFO] [2016-09-29 12:49:11] Rank 0/2: Begin of configuration and initialization.
[INFO] [2016-09-29 12:49:34] Rank 0/2: End of configration and initialization.
[INFO] [2016-09-29 12:49:34] Rank 1/2: End of configration and initialization.
[INFO] [2016-09-29 12:49:34] Rank 0/2: Begin of training.
[INFO] [2016-09-29 12:49:34] Rank 1/2: Begin of training.
[DEBUG] [2016-09-29 12:49:34] Request params. start = 0, end = 101635
[DEBUG] [2016-09-29 12:49:34] Request params. start = 0, end = 101635
[INFO] [2016-09-29 12:49:37] Rank = 0, Iter = 0, Block = 0, Slice = 0
[INFO] [2016-09-29 12:49:38] Rank = 1, Iter = 0, Block = 0, Slice = 0
[INFO] [2016-09-29 12:49:40] Rank = 0, Alias Time used: 6.88 s
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 13989076 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 25504855 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 341954549 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 510688998 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment<PHONE_NUMBER> from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 72048999 from word proposal
[INFO] [2016-09-29 12:49:40] Rank = 1, Alias Time used: 7.31 s
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 13989076 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 25504855 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 341954549 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 510688998 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment<PHONE_NUMBER> from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 72048999 from word proposal
[FATAL] [2016-09-29 12:49:40] Invalid topic assignment 324590367 from word proposal
if my way of running it by multiple machine is wrong , please show me a right example.Thanks very much!

any answer?

@sdy1106 Sorry for the late. I suppose you run two machines, each of which contains the whole data. Then the training data shall be twice than the original one, which causes the number of TF double too. Then the wrong TF causes the unexpected error. If you run with two machine with data-parallel way, you need to partition the dataset firstly.

@feiga  Thank you for your reply. I'm sure that I've done the data partition. I splite the libsvm file to two parts and get block.0 and block.1 by  dump_binary successfully. When I run it , I think block.0 should be dealed by rank0 and block.1 should be dealed by rank1.  But in a iteration, rank0 and rank1 both deal block.0 and block.1 which I think is unreasonable.
This is my input:
mpirun -np 2  $bin/lightlda -num_servers 2  -num_vocabs 111400 -num_topics 1000 -num_iterations 10 -alpha 0.1 -beta 0.01 -mh_steps 2 -num_local_workers 2 -num_blocks 2 -max_num_document 1000000 -input_dir $dir -data_capacity 800
someting wrong ?

This is the MPI problem. You're running two processes in one machine. Both processes run with same command line arguments. That's why they deal with both blocks. Please also note that you're not necessary to run multiple processes in one machine, since LightLDA has done optimization for multiple threads. Just use one process for one machine.
If you would like to try distributed training. Then you need to partition the data and distribute the data to different machines. Then run mpirun -machinefile machinelist.txt lightlda ... -num_blocks num_of_local_blocks
I think this would solve your problems.

@feiga I've tried your solution and it also failed. I used the -machinefile and edit two ip in the machinefile. I'm a little confused about "partition the data and distribute the data to different machines".  I've done the data partition in a share file system and I'm sure very node can access it , should I do something else? How do I make sure that specific node access specific block and vocab? Is it sure by the LightLDA?
Debugging log below:
[INFO] [2016-11-02 16:06:29] INFO: block = 0, the number of slice = 1
[INFO] [2016-11-02 16:06:29] INFO: block = 1, the number of slice = 1
before multiverso init!
Multiverso init!
[INFO] [2016-11-02 16:06:30] INFO: block = 0, the number of slice = 1
[INFO] [2016-11-02 16:06:30] INFO: block = 1, the number of slice = 1
before multiverso init!
Multiverso init!
[INFO] [2016-11-02 16:06:30] Server 1 starts: num_workers=2 endpoint=inproc://server
[INFO] [2016-11-02 16:06:30] Server 0 starts: num_workers=2 endpoint=inproc://server
MPI !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
MPI !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
[INFO] [2016-11-02 16:06:30] Server 0: Worker registratrion completed: workers=2 trainers=4 servers=2
[INFO] [2016-11-02 16:06:30] Rank 1/2: Multiverso initialized successfully.
after multiverso init!
[INFO] [2016-11-02 16:06:30] Rank 0/2: Multiverso initialized successfully.
after multiverso init!
[INFO] [2016-11-02 16:06:31] Rank 1/2: Begin of configuration and initialization.
[INFO] [2016-11-02 16:06:32] Rank 0/2: Begin of configuration and initialization.
[INFO] [2016-11-02 16:07:04] Rank 1/2: End of configration and initialization.
[INFO] [2016-11-02 16:07:04] Rank 1/2: Begin of training.
[INFO] [2016-11-02 16:07:04] Rank 0/2: End of configration and initialization.
[INFO] [2016-11-02 16:07:04] Rank 0/2: Begin of training.
[INFO] [2016-11-02 16:07:06] Rank = 1, Iter = 0, Block = 0, Slice = 0
[INFO] [2016-11-02 16:07:08] Rank = 1, Alias Time used: 4.65 s
[FATAL] [2016-11-02 16:07:08] Invalid topic assignment 519631042 from word proposal
[FATAL] [2016-11-02 16:07:08] Invalid topic assignment<PHONE_NUMBER> from word proposal
[FATAL] [2016-11-02 16:07:08] Invalid topic assignment<PHONE_NUMBER> from word proposal

Don't use a shared file system, use the local disk instead. Each machine store one block, and both blocks is named with block.0. The num_block equals to 1

OK! It works! Thanks!

Hi,did you install mpicc in both computers? what's the machine_file about?  Each machine have the same dir to store the training file? could you tell me the details how you succeed to run distributed training?@ sdy1106


New secure setup is getting errors Errno::EHOSTUNREACH: connection refused

I am getting the following errors when I try to configure the Hostname (registry.cloudkb.net) in registry settings.
Trying to configure the secure registry with portus.
I have not modified the .env file, only uploaded the SSL certificates in /secrets folder and did "docker-compose up". the hostname is resolving globally.
docker-compose version 1.21.2, build a133471
Docker version 1.13.1, build 94f4240/1.13.1
=============================================
portus_1      | Registry not reachable:
portus_1      | #<Registry id: 1, name: "registry.cloudkb.net", hostname: "registry.cloudkb.net", created_at: "2018-06-11 11:52:48", updated_at: "2018-06-11 11:52:48", use_ssl: true, external_hostname: "">
portus_1      | Errno::EHOSTUNREACH: connection refused
background_1  | Errno::EHOSTUNREACH: connection refused
background_1  | Exception: Errno::EHOSTUNREACH: connection refused
background_1  | Errno::EHOSTUNREACH: connection refused
background_1  | Exception: Errno::EHOSTUNREACH: connection refused
background_1  | Errno::EHOSTUNREACH: connection refused

It has fixed. The problem with firewalld.


Improve readability and glancability of docs in README.txt

Expected Behavior
That it's easy to search text, and see the whole structure, of the docs on the front page:
https://github.com/boost-experimental/ut
Actual Behavior

Most sections on the front page are collapsed, and some require two levels of expansion of triangles to see the content
Searching in the browser (Ctrl+F) doesn't find text in collapsed sections
I've seen someone who is more familiar with UT than me still click on 2 or 3 triangles before finding the code example they wanted to show me

Specifications

Testing in both Firefox and Chrome (Windows 10)

Suggestions

Perhaps provide a button at the top that opens all the triangles all the way down
Or just remove the triangles and collapsing altogether

Thanks, @claremacrae for pointing that out.
Fixed by https://github.com/boost-experimental/ut/commit/c74fb338c353e86dcc01df93f06993578476c8ee

Thanks @krzysztof-jusiak - having seen the result, I think it would really benefit from a table of contents now, ideally including levels 1 and 2, so users can quickly see the structure, and still jump to bits they are interested in.

I don't really understand the intention of 02ca5a40a904ff792f1c94e502ac79781dcdd784 - but it seems to have reverted the fix for this issue - almost all the sections are closed again, as far as I can see, so I think this issue should be re-opened.

Here are some examples of tables of contents in Markdown pages - so the full text is searchable on the page, and users can quickly see the structure and type of content, and also jump to a particular section very quickly:

https://github.com/catchorg/Catch2/blob/master/docs/tutorial.md
https://github.com/approvals/ApprovalTests.cpp#top

I don't really understand the intention of 02ca5a4

Ah right, now I understand - it has almost reverted to the original behaviour, but it has opened the sections that have sub-sections...
So users can see the sub-section titles, but the original issue of not being able to search the whole page (unless you expand every single section and sub-section) has returned.

What's the benefit of the expanding sections?
Could they be replaced with a traditional <h1>, <h2> etc, with a <ul> for Table of Contents at the top?

Thanks for your input @claremacrae.
Yeah, I'm still experimenting with it with the following goals in mind :thinking:

Easy to search (basically one page, it's usually more difficult to find things where they are spread around multiple Markdown files)
Easy to navigate (:new: top menu)
Easy to hide details (sections)

I pushed a change to expand all sections (they can still be collapsed) and added a top menu.

I pushed a change to expand all sections (they can still be collapsed) and added a top menu.

I like it!!!


document hawk/dove convergence logic

[x] update readme to include brief description of convergence logic
[x] add status to model data collection (converged/running) so it is easy to check if batch runs completed
[ ] see if there's an easy way to display convergence status in mesa interface (near the steps in the sidebar if possible)
[ ] try to determine typical convergence for the longer + more complicated runs included in our batch runs, so we can set a reasonable upper bound

@LaraBuchak I looked into options for displaying the status of the simulation (running or converged) so you'd know why it stopped. I can't put it where I'd prefer to; could you let me know if the easy option would be useful for you?
I would much prefer to display the status in the sidebar, something like this:

That's not currently supported by the mesa experimental jupyterviz display we're using. I could open a pull request with the approach I used to generate this to get feedback on it (and I probably should do that), but I worry about how complicated the jupyterviz interface is already without adding more.
What I was recommended to do, which is possible with current mesa, is include the text as a block with the other "measures" (charts), like this (look under the blue bar chart):

Would this option be useful?

Discussed with @LaraBuchak at meeting 2024-02-21; this convergence logic is fine for the base hawk/dove game, but it doesn't get at what we care about for the hawk/dove multi risk attitude - what we really want there is equilibrium. Merged in #65 but will revisit by testing new convergence logic for the hawk/dove multi game based on agents not changing on adjustment rounds.


Fix flexible members crash in clang 7.1

Fix flexible members crash in clang 7.1 (Fixes #42468)
Defining data as data[] causes compiler crash in clang 7.1 when defining
const TfLiteIntArray kZeroLengthIntArray = {0, {}};
in micro_allocator.cc

Note for @gbaned:
Per this comment, let's wait until @yair-ehrenwald has had a chance to make a github issue and link to it from this PR before we merge it.

this is the relevant github issue: https://github.com/tensorflow/tensorflow/issues/42468

@yair-ehrenwald -- thanks. I have updated the PR description comment to annotate that this PR should fix the issue that you created.
I think we are ready to merge.