Custom Number Spinner/Stepper

Description
The browser number spinner/steppers are not the greatest, visually:

It would be nice if we could leverage our design patterns and language to build one out that is better
Design

[ ] Design is finalized. Link to mockups/prototypes:
[ ] Design is being finalized by feature team. Link to any in progress mockups:
[ ] Assistance requested in finalizing design

Component API
I believe the API that is given for the number input field is sufficient, but visually needs to be upgraded. Full API is here
Usage
This would be used for any location where a consumer needs to input or adjust number input fields
Implementation
Please describe how you would like this component to be built. Design systems is happy to help with the implementation of new components (as time/roadmap allows) but also recognizes that sometimes feature teams prefer to implement components themselves.

[ ] Feature team would like to implement this component
[ ] Feature team would like the Design Systems team to implement this component
[ ] This component is already built, feature team requests assistance moving it to Primer React Components from outside application
[x] Other: (please describe below)

I would be happy to implement such a control outside of Primer React and then upstream it. Or, I could partner in building it inside of Primer React. I would love to discuss any roadmap that you might have for new components
Timeline
No timeline constraints
What to expect next
A Design Systems engineer will read through your component proposal and get in touch! We'll gather any additional information needed and set up a Zoom call only if necessary. Once we have all relevant information, we'll add the component to our Primer React Components Roadmap. If you opt to build the component yourself, we are happy to help with pairing, brainstorming, etc. Generally, component work requested from internal feature teams receives higher priority over general (non-requested) improvements to the library.

I'm so sorry this never got a response @dusave! Our automation must have failed for a short while so this didn't end up on our project board for triage.
If this suggestion is still of interest, I'd suggest we engage Primer designers on this request. @tallys how would you suggest we proceed?

This would be useful in repos as well: https://github.com/github/github/pull/265294

Hot take: primer/css has global style that hides the spin buttons!
https://github.com/primer/css/blob/main/src/forms/form-control.scss#L224-L231

We should talk to a11y about how this should be implemented.

Pinging @alliethu putting this on your radar

Pinging FR! cc/ @primer/accessibility-design

Not technically stale, but also not high on the priority list at the moment. Let's leave it open, but keep in the backlog unless there is a reason to reconsider the priority.


Different "direction" attribute error in Zurich data when loading with torchIO

In order to test a weighted sampling approach in torchIO, I was trying to load data patches and that resulted in an error shown below.
RuntimeError: More than one value for "direction" found in subject images:
{'label': (0.9981166689418295,
           -0.05645437422859611,
           -0.024005560303328167,
           0.05643535204519909,
           0.9984051800899562,
           -0.0013494249498021132,
           0.024045087232130596,
           0.0,
           0.9997109142781367),
 'sag_t2w': (0.9981165875284604,
             -0.05645307696797697,
             -0.024007068248685814,
             0.056436713381733866,
             0.9984052534421318,
             -0.0013574772136438992,
             0.02404527153536297,
             0.0,
             0.9997108671659608)}

This is one of the checks that TorchIO performs automatically to check the consistency of the data. For e.g. some of the checks ensure the spacing, orientation and the gradient direction (see 4th paragraph here) are correct.
To see whether it is an issue with TorchIO, I tried the same data loading procedure with sci-colorado data and it worked without errors. This suggests two things: (1) since both datasets are SCT-preprocessed in the same way, there is no issue with the SCT preprocessing output, and (2) the issue is with sci-zurich data and not on TorchIO's side.
@jcohenadad Would you have any suggestions as to how we can ensure that the directions for the input image and the label are the same?

well to me it just seems like a precision issue, right?
0.9997109142781367 vs. 0.9997108671659608
so you can simply change the precision threshold

Well, I looked into that as well i.e. lowering the threshold tolerance to 1e-2. I still run into this error despite doing that. Looking into it a bit more.


Add Simons Foundation to ENUM of Institutions

ror id = 01cmst727

Is there any abbreviation for this institution?


doc: Updated README.md to include uninstall instructions

Proposed changes
The adds instructions on how to uninstall the RKE2 cluster, as per the instructions found in https://docs.rke2.io/install/uninstall/.
This fixes #19

CI will fail until #26 is merged in... :rage2:


Add an option to hide the footer so that users cannot file an issue or access forums or Slack

Remove this

Useful
Info

Versions
Rancher v2.0.2 UI: v2.0.48

Access
`` admin

Route
global-admin.clusters.index

@vincent99 has mentioned this was possible using the ui-pl setting. By using a private label, it's supposed to hide these things.

@dxxxxxg, this is indeed correct.  If there is anything other than Rancher in the ui-pl in the settings page, the Help & docs, Forums, Slack, File an issue links go away.  Closing this issue.


What is the best JAVA SDK to be used for the Storage APIs

I am planing on using AZURE Cosmos DB and use the Table API (My use case is simple enough not need for indexing). Am confused about which SDK to use.
The Quick start points to the old Java SDK that seems to be deprecated as per this page https://github.com/Azure/azure-storage-java.
Thus my question what is the recommended SDK or library to use going forward. Thanks

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

ID: 2d2765b0-159f-49ed-4815-148af8ad8836
Version Independent ID: 8493309b-bfb6-f406-c2ae-7b8eed598650
Content: Use the Table API and Java to build an app - Azure Cosmos DB
Content Source: articles/cosmos-db/create-table-java.md
Service: cosmos-db
Sub-service: cosmosdb-table
GitHub Login: @SnehaGunda
Microsoft Alias: sngun

@b-slim Thank you for reaching out. We are actively investigating and will get back to you soon.

@b-slim The Java SDK referenced above is still the best way to work with Table API.
The Java V12 SDK has support for Blobs, Queues and Files.

@b-slim We will proceed to close this issue now.
If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Update according to php 7.2

Use new language features in the php source code

Please sync with goridge


Added feature support

IDispatcherOptions adds Assembly[] to avoid the problem of inconsistent Assembly[] in multiple EventBus
IRepository adds an AnyAsync method
IntegrationEventLog supports modification time
Attributes in Event in domain should not be public set
Support localization（Time zone issues and language issues）
Added IdGenerator support

IRepository adds an AnyAsync method
This is not the responsibility of IRepository

IRepository 添加了一个 AnyAsync 方法

这不是 IRepository 的责任

Understand, IRepository only provides basic functions of warehousing, others are extended by users and removed later


WIP: util: net: Change cached_download and derivatives to functions.

Fixes: #1056

Codecov Report

Merging #1092 (d6c9a0d) into master (a4fa4fe) will increase coverage by 0.04%.
The diff coverage is 95.45%.

@@            Coverage Diff             @@
##           master    #1092      +/-   ##
==========================================
+ Coverage   82.96%   83.00%   +0.04%
==========================================
  Files         150      150
  Lines        9825     9766      -59
  Branches     1659     1651       -8
==========================================
- Hits         8151     8106      -45
+ Misses       1326     1315      -11
+ Partials      348      345       -3

Impacted Files
Coverage Δ

dffml/util/net.py
72.83% <90.90%> (+5.91%)
:arrow_up:

dffml/service/dev.py
62.87% <100.00%> (ø)

tests/source/test_idx.py
100.00% <100.00%> (ø)

tests/util/test_net.py
98.36% <100.00%> (-1.64%)
:arrow_down:

dffml/util/data.py
64.08% <0.00%> (-2.77%)
:arrow_down:

dffml/base.py
84.38% <0.00%> (-0.34%)
:arrow_down:

dffml/df/memory.py
79.64% <0.00%> (-0.26%)
:arrow_down:

dffml/util/testing/consoletest/commands.py
75.16% <0.00%> (-0.06%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Bump Dependencies #1888062664

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/1888062664

 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.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


GeoWave as a data source for MrGeo

Implement a vector data provider for GeoWave to allow MrGeo to make use of vector data stored in GeoWave. Initially, we should support reading data out of GeoWave and being able to specify a GeoWave data source as an input format for a map/reduce job in MrGeo. An initial use case should be the ability to run a RasterizeVector operation against a GeoWave vector source.

Completed work for a GeoWave vector data provider for reading data. It is not able to write data. That can be done in the future if the need arises.


[UI] Add dividers between tabs

Is your feature request related to a problem?
With the UI overhaul released, it's rather difficult to tell your tabs apart, due to a lack of elevation or indication of where one tab ends and the other begins.
Describe the solution
Add small dividers between inactive tabs (similar to those seen on chrome and edge).
Describe alternatives you've considered
Slightly elevate inactive tabs in color to better distinguish between them.
Additional comment
Current Design:

Proposed Design:

That's a perfect idea. I vouch for it.

Great idea :+1: It's very subtle yet it helps to distinguish between individual tabs.

I think it should be more noticeable if its going to be implemented. I could barely tell that they were there with my display.

If this does get implemented, I don't think the separators need to be around the active tab, similar to Edge.

This change was implemented in #4612.


C#: Fail callp silently if script is not valid

Fixes https://github.com/godotengine/godot/issues/83351

Thanks!


Using Meson Build System

Currently we are setuptools like any other python project, which is fine, but I am strangely attracted to Meson, as a build system.
Meson is fast, other than that only feature I see, which is so hard to do with setuptools is that we can create C programs to for the testsuites. What I mean is we can write C programs to generate SVG files and compare it with what we generate. That should surely work because we are generating things in the same environment and testing the same.
While doing this would mean maintaining two different build system, one setuptools and other Meson, but I think Meson setup scripts are so easy to maintain than what we do at setuptools, also meson has subprojects, which in a way helps the users to build pango while installing manimpango, say they are on old linux, this may help.
Thoughts about it @PhilippImhof ?

I have absolutely no opinion on this topic, because I have never used Meson. Having two build systems to maintain seems bad to me, however.

Having two build systems to maintain seems a bad idea to me, however.

Indeed yes, meson is so new to python, so we can't use it to create wheels. But still has its own advantages though.

I think this would help a lot. Let me see if I can build wheels from meson itself. If I can do that this would be the best approach in long run.

I am planing to implement this in #48.

Feels like there are lot of things that are not going to work with this setup, so I think I am not going to do that.

Looks like scipy has moved to Meson now. https://github.com/scipy/scipy/pull/14847

Looks like scipy has moved to Meson now. https://github.com/scipy/scipy/pull/14847

Wheel creation still has sharp edges, but we plan to fix that over the next weeks/months. Unless you're in a hurry I would recommend to wait for a bit till we've tested everything for SciPy.

Wheel creation still has sharp edges, but we plan to fix that over the next weeks/months. Unless you're in a hurry I would recommend to wait for a bit till we've tested everything for SciPy.

Wheel creation still has sharp edges, but we plan to fix that over the next weeks/months. Unless you're in a hurry I would recommend to wait for a bit till we've tested everything for SciPy.

Thanks, yeah we would probably wait for SciPy to actually make a release with meson support and wheels built with meson. Also, wish you a happy new year 🎉 .

Wheel creation still has sharp edges, but we plan to fix that over the next weeks/months. Unless you're in a hurry I would recommend to wait for a bit till we've tested everything for SciPy.

Thanks, yeah we would probably wait for SciPy to actually make a release with meson support and wheels built with meson. Also, wish you a happy new year 🎉 .

You too, happy new year!

You too, happy new year!


Fix whitespace in day for asctime

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

Cc https://github.com/hyperium/hyper/pull/372

Cc https://github.com/hyperium/hyper/issues/387

Thanks!


Move adjustments for Movesets

See: https://www.reddit.com/r/TheSilphRoad/comments/6icwu4/type_effectiveness_changes/
We don't have the immunity in our stats, but as we don't fight gyms, I don't think we need it?
Noticed these changes when looking into #6069 but these changes are unrelated with the fast dps...

@davidakachaos you should have a pending invitation from pokemongof github team?

@solderzzc I haven't seen it, where do you see such an invite?

@solderzzc Never mind, found it and joined the team 😸

@davidakachaos nice, then you can do the pull approve for other PR, I guess the first invitation was sent on year 2016. lol

👍


Windows 10 22H2 10.0.19045.2788 not supported

termsrv.zip

Is it possible to just replace the term service DLL in build 19045.2846?  Doesn't seem to work in V 22H2


Just some compliments and a question

Wow, I've been using your colorscheme for a couple of days now and I'm just blown away with how good it is.
Good job man, this has to be the best colorscheme for Vim/NeoVim in existence right now.
Also I've been wondering, since I've been working on my material.nvim scheme. It has been using colorbuddy.nvim which I've wanted to ditch for some time now, because it has been holding me back in some aspects.  would it be okay if I used your source code as a guide for rewriting my theme?
Of course, not to make an exact clone with different colors, but just the basic stuff like loading highlight groups and figuring out different options like that darker nvim-tree background.

Hi, yes of course! Pick away :)
Great work on material btw. Love it

Thanks, I'm making a new branch called pure-lua.
If you want to add something later on, feel free to jump in ;)


Grafana upgrade 2.0.2

In this branch I have migrated grafana to version 2.0.2. These are the most relevant changes:

Grafana is no more a static site, but a webapp with a proper backend, so:

I have configured nginx to pass all the requests form / to the appropriate address
I have removed elasticsearch as it isn't used anymore for dashboard persistence. This has been replaced with a sqlite DB (can be configured to use other DBs).
The dashboard JSON structure is slightly changed, so I have updated the bundled welcome.json and system-metrics.json files.
Grafana is no more configured with a config.js file, instead a custom.ini file is provided

The dashboard-loader.js script has been heavily changed:

It now uses the Grafana HTTP API
The script has been moved out of the grafana/dashboards folder
Directories are now accepted as command line parameters and all the contained files (recursively) will be loaded
It's now possible to specify the command line option -w, so the files (or directories) specified will be watched for changes. This in my opinion is really handy because the entire /src/dashboards directory can be mounted as external volume in docker, so that the dashboards can be added/updated/removed on the fly from outside the container.
Removed misused server at the end of the script and adapted the supervisor configuration

I hope you will find my contribution useful and pull it. If you have any suggestion to improve what I have done, please let me know.

Hello Marco! Hope you are doing fine :).. man, I just pulled the changes to give them a try and I find that even while the Grafana loads and all, the default welcome dashboard is not available and the system metrics dashboard is there, but it only has random walks.. was that on purpose or maybe I'm missing something?
Overall all the efforts seem to be going in the right direction, we just need to ensure everything is working nicely and then publish a new version of this image, looking forward to your feedback, regards!

Hi Ivan,
I'm fine, thanks. I hope it's the same for you.
About the two issues you mentioned:

The default dashboard is not available: grafana seems to have a problem during the insertion of the first dashboard, however in the grafana code the error is not checked, so the API returns a "200 OK" anyway. The problem should be solved by the fix for issue grafana/grafana#1929. What I suspect is that it's related to the fact that when the app is booting the DB is not immediately available. As a workaround, in commit 8ccd87c597269723a297664a42013b1d4017f279 I have added a retry mechanism for the dashboards insertion: when the insert is completed I try to GET it back and if I receive a 404 I repeat the insertion. Is it a valid solution in your opinion?
The dashboard shows only a random walk: in grafana 2 you can configure one or multiple datasources. By default the only datasource available is a "test datasource" that always provides random walks. If you think it's necessary, I can commit a different config without the "test datasource" and with a graphite datasource that points to the internal installation. Please let me know what you think. Could you please also point me to the toy app you use to test the docker image?

Regards
Marco

Hello @mmojana ! Hope you are doing fine, Ivan and I we think that is good idea merge you PR and let to the users download the Dashboards from our Site. Can you please update to the Latests Grafana and we are good to go!

:+1:

Hi @dpsoft! How are you? I have updated Grafana to 2.1.3 and merged the commits from master. IMHO the grafana API is still not mature and reliable enough, so I had to add a couple of workarounds to cope with the misleading status codes returned.
The branch is ready to be merged.
Please let me know if I can help you further.
Regards
Marco

@mmojana  Just need to ask you please squash your commits and amend the message to something like "Grafana upgrade 2.1.3" and will merge right away  and we are good to go!

@dpsoft Sorry I'm not a git expert... are my commits "squashed" enough? I had squashed everything in a single commit, but then my branch was behind your master, so I had to pull again and that resulted in another commit. Is it possible/necessary to combine the two commits? Should I just amend the merge commit message so it reads "Grafana upgrade 2.1.3" too? Thank you for the support!
Regards,
Marco

@mmojana Thanks for Awesome contribution :), hope to see more coming!!!


allow gamemode mods to use banners as mode images and patches as icons

(Opening this PR now so i can test through flightcore no review pls)

Modifies menu_mode_select to add GetPlaylistBannerImage and RegisterPlaylistBannerImage, also modifies ModeButton_GetFocus to show/hide correct rui since basic_menu_image.rpak cannot show banner images, also adds mode_select.menu, only difference here is NextModeImageCallsign which is just an identical panel using the "ui/callsign_icon_button.rpak" rui instead of basic menu image
ImageTest.zip
test mod

Seems to work but i would test, some funky stuff happened trying to this is work locally with it just not updating some files and causing random stuff to crash

This is an old PR fixed up, Unfortunately, i did not include the original unmodified mode-select menu file in a separate commit :boowomp:

@JMM889901 merge conflicts atm

@JMM889901 if you resolve merge conflicts I can add the necessary vanilla files to get this PR merged o7


Sprite Overlay Error - HEX #306850 to HEX #071821

Bug Description
Not sure what causes this, but it seems to happen when a sprite using HEX #306850 is placed on top of a background tile that uses HEX #071821, the overlaid sprite turns black besides the color HEX #e0f8cf which stays the same. This happens all the time even if only a half or even a fourth of the sprite is overlaid. This ONLY affects HEX #306850 on the sprite, not HEX #86c06c, HEX #071821, or HEX #86c06c.
To Perform the Error
Steps to reproduce the behavior:

Create a background using the Gameboy color palette, making sure HEX #071821 is used on a tile for at least a fourth of the tile's size.
Go to '+'.
Click on 'Scene'.
Add a scene.
Select the created background containing HEX #071821.
Create a sprite using the Gameboy color palette, making sure HEX #306850 is used. Size: (16 x 16).
Go to '+'.
Click on 'Actor'.
Place the actor on a background tile using HEX #071821.
Go to 'Sprite Sheet'.
Click on the created sprite using HEX #306850.
Make sure when the game boots, you get shown the screen with the background using HEX #071821 with the sprite using HEX #306850
Go to 'Game'.
Click 'Run'.
There is the issue.

What the Expected Result is
The expected behavior was that when the sprite using HEX #306850 is overlaid on a background tile using HEX #071821, the HEX #306850 on the sprite should remain the same and not become HEX #071821.
Screenshots

Platform

OS: [Windows 10 x64 Build: 1714.1006]
App version [v.1.1.0]

For anyone else that comes across this issue, dark green is not a supported color for sprites.
Alternatively, you can bake the sprite into the background and use a 2 frame sprite, one being transparent for interaction, and another as base background/revealed color.
This will not let you walk through as the sprite must be enabled to be visible, a request for disabling collision is already open. #205


Switch from support library APIs to AndroidX

With the launch of Android Jetpack and AndroidX, the support library APIs have been superseded. We should drop the support library APIs and replace them with AndroidX.

@erikghonyan thanks for bringing this up and getting the ball rolling on it.


[region-isolation] Add a special error for when a closure captures a non-Sendable box.

The reason that I am modifying this error is that in situations like the following one can have a Sendable type that triggers this error since the box containing the value is non-Sendable.
func methodConsuming(x: consuming SendableKlass) async throws {
  try await withThrowingTaskGroup(of: Void.self) { group in
    group.addTask { // expected-tns-warning {{passing closure as a 'sending' parameter risks causing data races between code in the current task and concurrent execution of the closure}}
      useValue(x) // expected-note {{closure captures 'x' by reference in a non-Sendable box which is accessible to code in the current task}}
    }

    try await group.waitForAll()
  }
}

rdar://133813644

@swift-ci smoke test

@swift-ci smoke test


Data Type Inconsistency for Degree

Please take a look at the method which pass in the parameter for the start degree and layout degree
XXXRoundMenuButton.h
- (void)animatedLoadIcons:(NSArray<UIImage*>*)icons start:(CGFloat)start layoutDegree:(CGFloat)layoutDegree oneByOne:(BOOL)onebyone;
XXXRoundMenuButton.m
- (void)animatedLoadIcons:(NSArray<UIImage*>*)icons start:(CGFloat)start layoutDegree:(CGFloat)layoutDegree oneByOne:(BOOL)onebyone;
- (void)animatedLoadIcons:(NSArray<UIImage*>*)icons start:(double)start layoutDegree:(double)layoutDegree oneByOne:(BOOL)onebyone

There's inconsistency on the data type for the layoutDegree and startDegree which causes conflicting parameters types.
It's causing issue when I try to run on iphone4/5 , the icon didn't fan out circling the center of it, but instead it stick at one location, or at the wrong location.
When I try to nslog the layoutDegree parameter, it shows 0.000000 instead of a proper value.
But once I change the "double" to "CGFloat" , then it manage to pass in proper value for the layoutDegree and perform the animation properly.

I hope this can help you, sorry that I didn't has 4 / 5 device to fix it .
https://github.com/zsy78191/XXXRoundMenuButton/issues/6

I already did that by standardize all the data type before I see this ^_^ . But thanks.


Adds solution for 981. Time Based Key-Value Store in Kotlin

File(s) Modified: 981-Time-Based-Key-Value-Store.kt.
Language(s) Used: kotlin.
Submission URL: _https://leetcode.com/submissions/detail/831142575/_

Thanks, @t3chkid!


automate release packaging

@maxsam4 @cwsatpolymath PTA(onther)L.

Thanks for the detailed review @cwsatpolymath :)


Unable to change font size in excel vba editor

Unable to change font size. In courier western its set at 7 , and i cannot increase it. Arial cannot increase it more than 8. Every time it resets to font 7 ot 8.

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

ID: 98489ee2-e346-4b57-4c54-b7537a93c29d
Version Independent ID: 86458361-267d-82bf-8068-9638f9a3eb3a
Content: Editor Format Tab (Options Dialog Box)
Content Source: Language/Reference/User-Interface-Help/editor-format-tab-options-dialog-box.md
Product: office
GitHub Login: @o365devx
Microsoft Alias: o365devx

@gstazure Thanks for your feedback. We use Github issues for documentation bugs, not for developer support. Please try the VBA developer community on Stack Overflow (https://stackoverflow.com/questions/tagged/vba) where you can get help from actual developers. If you find a bug in our documentation, please let us know! Thanks!

Hi, I manage the Rubberduck OSS VBIDE add-in project (written in C# 7, if you want to contribute), which you may want to watch, since we have it on our roadmap to implement a custom code pane with enhanced syntax highlighting and more font options. That custom code pane feature is scheduled for v3.0 (we're about to release v2.3, so little is done towards custom code panes at this point, but we'll definitely get there).
I don't think Microsoft has any plans to enhance the VBE in any way, or fix any problems with it - so 3rd-party IDE add-ins are your only sure shot at getting new or improved functionality into the VBA editor.


Why e2function aliases/duplicates have smaller cost?

.
For example, above we see the following functions in E2 helper.
Now, I wonder why the bottom four e2functions cost 5 while the first four cost 25?

I think you're looking at this wrong. nns and nnsn functions with optional entity have a higher cost due to the isValid and isOwner calls. Now, what I don't understand is where the cost changed from 25 to 5 since there was only one __e2setcost(25).

@oldmud0 Yeah, well that's what I mean.. Just where is the line that set the cost to 5? I though I was blind for a moment, apparently not.

Pretty sure all soundPlay functions are supposed to have the same cost. A
simple isvalid check isn't slow enough to warrant such a huge difference in
cost.
On Tue, Jun 14, 2016 at 2:15 AM, CaptainPRICE<EMAIL_ADDRESS>wrote:

@oldmud0 https://github.com/oldmud0 Yeah, well that's what I mean..
Just where is the line that set the cost to 5? I though I was blind for a
moment, apparently not.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/wiremod/wire/issues/1147#issuecomment-225746270, or mute
the thread
https://github.com/notifications/unsubscribe/ABaroMbafAH14dugp5ohg0AjAzIYya9Bks5qLfKegaJpZM4I0z4X
.

The only reason I can think of why there is such a difference in cost for aliases, is that when alias e2function is interpreted it calls __e2setcost(5) before an alias function is defined.. So, I went to try out my theory (simply by checking the cost of other e2function aliases). I have found 39 e2functions that are alias/duplicate. Here are the results:

Alias Definition
Cost
Real/Should Cost

e2function void findExcludePlayer(entity ply) = e2function void findExcludeEntity(entity ent)
5
10

e2function void findAllowPlayer(entity ply) = e2function void findAllowEntity(entity ent)
5
10

e2function void findIncludePlayer(entity ply) = e2function void findIncludeEntity(entity ent)
5
10

e2function void findDisallowPlayer(entity ply) = e2function void findDisallowEntity(entity ent)
5
10

e2function string glonEncode(table data) = e2function string glonEncode(array data)
5
15

e2function string vonEncode(table data) = e2function string vonEncode(array data)
0
15

e2function void soundPlay(string index, duration, string path) = e2function void soundPlay(index, duration, string path)
5
25

e2function void entity:soundPlay(string index, duration, string path) = e2function void entity:soundPlay(index, duration, string path)
5
25

e2function void soundPlay(string index, duration, string path, fade) = e2function void soundPlay(index, duration, string path, fade)
5
25

e2function void entity:soundPlay(string index, duration, string path, fade) = e2function void entity:soundPlay(index, duration, string path, fade)
5
25

I don't want to post an entire list..
Guess what. My theory was true :D
It seems like there is a bug in e2function alias/duplicate interpreter ;)

vonEncode is supposed to cost 0, because it dynamically adds more ops depending on the length of the string.

Because I messed up, apparently. Try #1148.
It might fix it or it might blow up.

Btw, if i read this correctly, you also can't alias vararg functions.

Btw, if i read this correctly, you also can't alias vararg functions.

meh who cares. We can just do this if we really need to
local function do_stuff(A,B,C) end

e2function void whatever(A,B,C) do_stuff(A,B,C) end
e2function void whatever(A,B:vector,C) do_stuff(A,B,C) end

@TomyLobo Just tested it, #1148 fixes it.

Go ahead and merge that.

@Divran ^

Just noticed this affected a lot more than aliased functions.
--[[---------------------------------------------------------------------------
Broadcast messages
-----------------------------------------------------------------------------]]
__e2setcost(50)

e2function void broadcastMessage(...)
  if not check_permission(self) then return end

  send_color_print_VarArgs(self, nil, typeids, ...)
end

e2function void broadcastMessage(array args)
  if not check_permission(self) then return end

  send_color_print_Array(self, nil, args)
end

--[[---------------------------------------------------------------------------
Player methods
-----------------------------------------------------------------------------]]
__e2setcost(20)

e2function void entity:printMessage(...)
  if not IsValid(this) then return end
  if not this:IsPlayer() then return end
  if not check_permission(self) then return end

  send_color_print_VarArgs(self, this, typeids, ...)
end

e2function void entity:printMessage(array args)
  if not IsValid(this) then return end
  if not this:IsPlayer() then return end
  if not check_permission(self) then return end

  send_color_print_Array(self, this, args)
end

The first function in the above code would be set at 50 ops, while the last 3 would be set at 20.

yes, it also affected vararg functions


Improve web design

I think that present web design need to be update
Would you like to work on the issue?
Yes if someone would accept this issue

hi, before we get back to working on the web design, the current design should work with the latest API of eventyay.com. Therefore I am closing this.


Remove duplicate versions of Bootstrap

Removes additional style sheets and the require statements in the application.css file and moves to css.scss and uses imports to add the resources in the application.css.scss style sheet.
This change eliminates multiple copies of Bootstrap from being added to the application style sheet from multiple dependencies.
Fixes #1460

Coverage remained the same at 55.635% when pulling 6de310cb8ea7b062f75b0ff7b6f3da9074d85a76 on i#1460-dupe-boot into 60e9ee1f18405b2ee607f2aa57f0da1a19121ddd on develop.


[FAIL] Linting scripts/bootstrap.sh...

The command "./build/build.sh" exited with 1.
Raw Log File:
https://api.travis-ci.org/jobs/137483887/log.txt?deansi=true

Changes to linting is causing it to pick up something new. But I think the error message may be misleading. I'm parsing it line by line to check

Yes, this update brought out a linting mistake previously overlooked. So, thanks for the resource!


persist_docs fails on materialized_view

Describe the bug
Seems related to #104 - Setting persist_docs on a materialized_view causes the execution of an ALTER TABLE statement, but this fails on materialized views, ALTER VIEW doesn't support comments, so would need to be added during creation.
Steps To Reproduce

Set model config.materialized = 'materialized_view
Set persist_docs = true
Run model

Expected behavior
The MVW to be created with comments on each column from the dbt yml
Screenshots and log output
[EXPECT_TABLE_NOT_VIEW.NO_ALTERNATIVE] 'ALTER TABLE ... CHANGE COLUMN' expects a table but <mvw location> is a view
System information
The output of dbt --version:
Core:
  - installed: 1.8.2
  - latest:    1.8.2 - Up to date!

Plugins:
  - spark:      1.8.0 - Up to date!
  - databricks: 1.8.1 - Up to date!

Python 3.10.12
Additional context
Happy to contribute if someone could point me towards a preferred approach for this!

I believe this is blocked right now.  When I was testing column comments on MVs, I hit the same issue as you, and in fact, couldn't get them to stick on create either (this might have subsequently been fixed, since that was 4 months ago).

I've reached out internally to find out what the plan is about this.

Hey @benc-db - any luck with this? thanks!

Let me check again.


deleted cvode and replaced by cvodes, still with coupled ode system; all

Submisison Checklist

[X] Run unit tests: ./runTests.py test/unit
[X] Run cpplint: make cpplint
[X] Declare copyright holder and open-source license: see below

Summary:
swap cvode for cvodes library
Intended Effect:
only change library, not any ode stan functions
How to Verify:
check source
Side Effects:
none
Documentation:
none
Reviewer Suggestions:
Anyone
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Sebastian Weber
By submitting this pull request, the copyright holder is requiring to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

cvode tests still work

From what I can see this breaks now because the build systems needs to point to CVODES instead of CVODE in the upstream dependencies stan and cmdstan. On both repos I have opened branches which adjust the build system. If we merge these in sequence then thinks should work again.

Thanks, Sebastian. That explains a lot. What order does it need (pull request numbers please)?
I just realized that there should be a way to do this without breaking the builds. Let me look at that. It should be in the makefile in math for the libraries.

On Apr 15, 2016, at 2:31 AM, wds15<EMAIL_ADDRESS>wrote:
From what I can see this breaks now because the build systems needs to point to CVODES instead of CVODE in the upstream dependencies stan and cmdstan. On both repos I have opened branches which adjust the build system. If we merge these in sequence then thinks should work again.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

Yes, the way things are now is not ideal... I created the pulls you ask for:
stan: 1863
cmdstan: 370

forgot your order: first math, then stan, then cmdstan

I meant just add CVODES without removing CVODE; that would have not caused a build problem. But it should be ok, I think.

On Apr 15, 2016, at 8:44 AM, wds15<EMAIL_ADDRESS>wrote:
Yes, the way things are now is not ideal... I created the pulls you ask for:
stan: 1863
cmdstan: 370
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

Which math pull request? There are 3 open ones now.

On Apr 15, 2016, at 8:47 AM, wds15<EMAIL_ADDRESS>wrote:
forgot your order: first math, then stan, then cmdstan
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

sorry, if I misunderstood. What I did is to remove the CVODE library and I replaced it with CVODES. However, integrate_ode_cvode still works; just now with CVODES sources.
The pull for math is this one, 282.
The outdated pull request on math is 279, but I would keep it for now as the comments are there.
The pull request which adds the integrate_ode_cvodes function in a clean way (and leaves integrate_ode_cvode intact) is 283, but there I still have to add some doc, feedback is very welcome there of course.

Great. I looked through it. This is known to break upstream builds. I'm merging.


Vimplugins vim-simpledb and lightline-gruvbox-vim

Motivation for this change
Add the vim-simpledb and lightline-gruvbox-vim vimPlugins.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 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.

Result of nixpkgs-review pr 126009 at 7c1f9cb4 run on x86_64-linux 1

1 package failed to build:

vimPlugins.vim-clap

35 packages built successfully:

spacevim
vimPlugins.aurora
vimPlugins.chadtree
vimPlugins.defx-nvim
vimPlugins.denite-nvim
vimPlugins.deol-nvim
vimPlugins.deoplete-fish
vimPlugins.deoplete-khard
vimPlugins.deoplete-nvim
vimPlugins.dhall-vim
vimPlugins.gina-vim
vimPlugins.git-messenger-vim
vimPlugins.gitsigns-nvim
vimPlugins.lightline-gruvbox-vim
vimPlugins.ncm2-neosnippet
vimPlugins.neosnippet-vim
vimPlugins.nvim-autopairs
vimPlugins.nvim-bqf
vimPlugins.nvim-dap-ui
vimPlugins.nvim-hlslens
vimPlugins.nvim-jdtls
vimPlugins.nvim-lightbulb
vimPlugins.nvim-toggleterm-lua
vimPlugins.nvim-tree-lua
vimPlugins.nvim-treesitter-context
vimPlugins.sideways-vim
vimPlugins.telescope-frecency-nvim
vimPlugins.telescope-fzf-writer-nvim
vimPlugins.telescope-fzy-native-nvim
vimPlugins.telescope-nvim
vimPlugins.telescope-symbols-nvim
vimPlugins.telescope-z-nvim
vimPlugins.vim-autoformat
vimPlugins.vim-fugitive
vimPlugins.vim-simpledb

2 suggestions:

warning: missing-phase-hooks
configurePhase should probably contain runHook preConfigure and runHook postConfigure.
Near pkgs/misc/vim-plugins/build-vim-plugin.nix:28:14:
   |
28 |       inherit unpackPhase configurePhase buildPhase addonInfo preInstall postInstall;
   |              ^

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near pkgs/misc/vim-plugins/build-vim-plugin.nix:28:14:
   |
28 |       inherit unpackPhase configurePhase buildPhase addonInfo preInstall postInstall;
   |              ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

Thanks for submitting your first PR :)

Thanks for submitting your first PR :)

Thank you! 😄


Discriminative Layer Training

See #958
I will continue to add more tests and setup distributed tests as well.
If you have any feedback, please let me know

I believe the code now works as intended. A separate optimizer for each learning rate multiplier ensures correct behaviour with momentum based optimizers.
Please review and provide feedback

I'll look into the buildifer later today.

@seanpmorgan is there anything I can do to help? I believe I made the requested changes, but I can also see that this optimizer is complicated. Would more documentation explaining how it works or maybe a tutorial help with the review?

@seanpmorgan is there anything I can do to help? I believe I made the requested changes, but I can also see that this optimizer is complicated. Would more documentation explaining how it works or maybe a tutorial help with the review?

Hi @hyang0129 a tutorial would be great.
Fully intend to merge this in, but it's a bit of a busy week for me personally. If anyone else would like to review that'd be encouraged.

I'm having trouble setting up a tutorial because the files are not in the current release of tfaddons. @seanpmorgan do you have any recommendations on bypassing this issue?

@seanpmorgan  I have created a tutorial, but I am not sure if I should include it in the PR. The tutorial has to duct tape the DiscriminativeWrapper in because it is not including the tfa.
Here is a link to the tutorial. It should demonstrate how to use it and why it is useful.
https://colab.research.google.com/drive/1FS3byiMxIlwBqnDmW0Wl7k28uyq2QmJE

@seanpmorgan I have created a tutorial, but I am not sure if I should include it in the PR. The tutorial has to duct tape the DiscriminativeWrapper in because it is not including the tfa.
Here is a link to the tutorial. It should demonstrate how to use it and why it is useful.
https://colab.research.google.com/drive/1FS3byiMxIlwBqnDmW0Wl7k28uyq2QmJE

@facaiy @WindQAQ @Squadrick Please review if you are available

I'm having trouble setting up a tutorial because the files are not in the current release of tfaddons. @seanpmorgan do you have any recommendations on bypassing this issue?

Sorry for the delay, I'll try to have a review in a few hours. Yeah you're right that gets a bit hacky. After this merges please submit a tutorial PR that installs tfa-nightly

@seanpmorgan I believe I have applied all of the changes for the main file. The changes I applied to the testing file may need further discussion, as some parts were removed or reworked significantly.

@seanpmorgan I believe I have applied all of the changes for the main file. The changes I applied to the testing file may need further discussion, as some parts were removed or reworked significantly.

Thanks @hyang0129 ! I will review as soon as time allows.

Could you git pull --rebase from origin/master? It'll update all the GitHub actions.

@Squadrick I've made the proposed changes. However, the previous CI run failed only on building the wheels for linux (other wheels build fine). Please see https://github.com/tensorflow/addons/actions/runs/44669335
I tried merging tfaddons repo to my repo, but I'm not sure if that will actually fix the wheel build issue.

@googlebot I consent

I've merged master into your branch to update it and fixed any formatting/conflicts it might have. If you need to do some more modifications, please do git pull beforehand.

@googlebot I consent

I believe the requested changes have been made. Is there something that I missed on my end?

@Squadrick @seanpmorgan  could you confirm if the requested changes have been addressed?

@Squadrick the GPU run seems to have failed while trying to create the virtual devices.
Here is the error:
tensorflow.python.framework.errors_impl.UnknownError: Not enough GPUs to create virtual devices. num_gpus_to_use: 1 #virtual_devices: 2
I did a quick search in the repo and I believe that I am the only one using run_distributed. I could not find another test that uses run_distributed or run_all_distributed.
From what I can tell, all tests that ran in graph mode and distributed failed. 14 of my tests run distributed graph+eager. I have 1 test that runs only in distributed eager.

I've created a new issue: #1171 to track this issue. This PR will be blocked until that is resolved.

I've created a new issue: #1171 to track this issue. This PR will be blocked until that is resolved, or you can remove run_distributed and add it in a later PR. The choice is yours.

Due to issue #1171 and the complications #1209, I would like to remove the distributed tests from this PR and proceed without distributed tests for purposes of merging with tfa master.

Is it possible to throw a warning (or an error message, NotImplementedError would be the best) to users who are trying to make this work with in distributed mode?

Is it possible to throw a warning (or an error message, NotImplementedError would be the best) to users who are trying to make this work with in distributed mode?

That can be arranged. I believe that this actually works in distributed based on the tests so far. Let's go with a warning to users to inform them that this may not behave as expected in distributed mode.

Totally agree. +1

@Squadrick I believe the requested changes were made?

@seanpmorgan @gabrieldemarmiesse It seems that @Squadrick is busy. How can I help move this forward?

@hyang0129 sorry for the delay. I'll take a look.

@hyang0129 I checked the TF docs for how to make custom optimizers and it does not have enough material for me to properly understand what is going on. Same for the source code of tf.keras.optimizers.Optimizer. I don't understand well what is going on. So I can help you with the tests and convert to pytest and eager mode, but I won't be able to review. Sorry about that.

@hyang0129 Sorry about the delay, taking a look right now.

@Squadrick  there seems to be an error in /tensorflow_addons/layers:layers_wrappers_test on the bazel test for the latest commit. What should I do about that?

@gabrieldemarmiesse is there more information on the build release wheels step? The addons-release / Build release wheels (macos-latest, 3.5, 2.2.0rc3) is not failing, but instead getting cancelled after 32 minutes.

@Squadrick I believe the requested changes have been made

坐等

Any news on this? I current need this and am considering migrating to PyTorch because its not yet available in TF 2.0.

@nicholas-leonard  if you plan on using this for TPU it is not tested for TPU. I recommend using PyTorch if you plan on using GPU training.
@Squadrick @gabrieldemarmiesse what are the next steps for moving this forward?

@nicholas-leonard I just wrote some code to do discriminative training on a TPU. you can find it here

Hey @hyang0129 -- I'm in the same boat as @nicholas-leonard , and very excited to see this PR coming together! Do you have any sense of how close this is to being merged? Also, happy to help in any way that I can (even if it's just a second set of eyes).
Thanks for all of your time working on this!

@rforgione I believe it should be good to go once someone reviews the code. @Squadrick @seanpmorgan @gabrieldemarmiesse @WindQAQ @facaiy ?
You can also use the simplified TPU compatible version here. Copy the class definition for the multi optimizer and try it out in colab.
I think going forward the simplified version is better because it is fully compatible with how graphs work in TF. However, I don't want to risk investing more time into this to have it sit in limbo.

I can review it recently. Given many demands from users, I think it's worth to have this in addons.

@WindQAQ I believe there have been some changes on how tests are run and how code should be structured. I will make those changes first then request a review.

@WindQAQ I believe there have been some changes on how tests are run and how code should be structured. I will make those changes first then request a review.

Sure, feel free to ping me if needed.

@WindQAQ please review

Also, I would like to have some tests on saving this optimizer
https://www.tensorflow.org/api_docs/python/tf/train/Checkpoint
Probably we should use self._track_trackable(optimizer, name="xxx") to keep track on those optimizers.

@WindQAQ I'll look into that for optimizer saving.

@WindQAQ what kind of info would be helpful when calling print(multioptimizer)?

@WindQAQ what kind of info would be helpful when calling print(multioptimizer)?

I guess users may want to see the optimizers and their corresponding variables. This is optional, just depend on your choice if you also think it is useful.

@WindQAQ should be good for review

@WindQAQ I've made the changes

LGTM! Sorry for the very long delay and thanks for the contribution to this good feature! Thank you!

I'm glad I could help! Thanks for the feedback to make this possible.

@hyang0129 Oops, We forgot to update code owners, would you mind filing another PR to update https://github.com/tensorflow/addons/blob/master/.github/CODEOWNERS ?

@hyang0129 Another thing is that MultiOptimzer -> MultiOptimizer.

@hyang0129 Another thing is that MultiOptimzer -> MultiOptimizer.

I'm not sure what you mean by that? Could you elaborate?

@hyang0129 Another thing is that MultiOptimzer -> MultiOptimizer.

I'm not sure what you mean by that? Could you elaborate?

We misspell the word "Optimizer". We should correct it in next PR.

I'll fix that

@hyang0129 Thanks so much for your great contribution! Happy to see it merged, apologies for the delay.


Why MountainCarContinuous-v0 will converge to 0 ?

I used the code to train MountainCarContinuous-v0 directly.
But reward converges to 0.

why this was closed? I got same results. Is it expected behavior?

I got the same results, but it shouldn't be. Because if the robot reaches the goal, it will get a reward of 100 I think.

I got the same results, but it shouldn't be. Because if the robot reaches the goal, it will get a reward of 100 I think.


docs/concepts/containers/images.md kubernetes version changed

"This was tested with a private docker repository as of 26 June with Kubernetes version v0.19.3. "
v0.19.3 changed  to v1.9.3

@bradtopol Go for it. :+1:

/approve
/lgtm


waitQueue.compare_exchange_strong in the true loop can be weak

weak is more suitable for this while loop than strong.

@DougGregor Can we please test and benchmark?

@swift-ci please test

I think this is still functionally correct and may yield better performance on certain platforms. When I initially changed these two compare-exhanges to strong in this PR, I was a bit overzealous.
If you could also simplify the one hasSuspended check then that would be great.
@ktoso for a second opinion.

@swift-ci please test windows platform


Cloudflare (continuously) rate limiting API calls

Hi,
It seems the Cloudflare API is over solicited by the bouncer.  One of the multiple log lines:
level=error msg="you have been ratelimited please wait and try again (10040)"
That would be fine if it would be occasional; but on the contrary each and every single call made is rate limited without a single call being successful, for multiple succeeding days.  That is rather not sustainable, especially since data is transferred continuously (1 Mb/s, all day, every day -- please see below a screen grab of my Linode IPv6 transfer)

The only way to give the API a rest:
systemctl stop crowdsec-cloudflare-bouncer
After restarting it, an hour or so of peace and quiet.  Then, back to the 10040 error code.
I'd suggest some sort of exponential backoff.  I would try to tackle the issue myself but I'm barely a script kiddie in Go.
Thanks.

Hi, what is your update_frequency for cloudflare on your bouncer ?
Try to increase it. I have 1min on my bouncer.
If you have really a lot of request to do, try even more if needed.

Hi, what is your update_frequency for cloudflare on your bouncer ? Try to increase it. I have 1min on my bouncer. If you have really a lot of request to do, try even more if needed.

It's at 10 seconds...  The default.
I'll try 120s and see what happens.
Thanks.

Happy to report that changing the update_frequency to 3600 seconds finally solved the problem. 3 hourly calls were rate limited and the 4th was successful.  Please note that the first 3 failed hourly calls only peaked at a few Kb/s instead of a perpetual 1+ Mb/s: don't understand why.
Since I use both my box local firewall and Cloudflare (Cloudflare alone can only block port 80 and 443 and I use it mainly to reduce the stress on the local firewall), the (very) low frequency is not much of a problem.
Thank you for a very simple fix.

This hasn't worked for me so far and is only one of many problems with crowdsec for me.   I am trying to span the wait times out a bit longer to see if that helps.  I am not seeing anything useful at all from cloudflare either.  ¯(°_o)/¯
May have to throw this project back in the water.

Hi,
I am experiencing the same problem but I don't understand if the issue is fixed or not..
Based on this comment it seems that GET requests shouldn't count against rate limitations.
However, I've disabled the CAPI functionality in my Crowdsec LAPI server, but when I try to ban a single IP, the Cloudflare bouncer gets rate limited

You will need to use that for now if you want to ban on Cloudflare side : https://docs.crowdsec.net/u/bouncers/cloudflare-workers/
Using list is totally broken as Cloudflare rate limit you even if you only make 1 request / hour...

Thanks for letting me know. Unfortunately, I am on the free Cloudflare plan, so based on the first banner in the page you linked I guess I'll need to wait..

Hello @marcogiorgio,
You can technically use the new cloudflare bouncer on the free plan, but with some limitations (including a very big one, hence the warning in the documentation):

You are allowed 100k worker requests per day. If your website needs more than that, the bouncer will stop working for the rest of the day
You are allowed 100k read requests to the KV store: same thing as above.
You are allowed 1k write requests to the KV store: This is the biggest one. Using the community blocklist (or 3rd party lists) can easily make you make over 50k writes (and only the first 1k will be actually written to the KV), this means that you won't be able to use the community blocklist content in a free plan and have to set crowdsec_config.only_include_decisions_from to ["crowdsec", "cscli"] in the bouncer configuration.

Hello @marcogiorgio,
You can technically use the new cloudflare bouncer on the free plan, but with some limitations (including a very big one, hence the warning in the documentation):
* You are allowed 100k worker requests per day. If your website needs more than that, the bouncer will stop working for the rest of the day

* You are allowed 100k read requests to the KV store: same thing as above.

* You are allowed 1k write requests to the KV store: This is the biggest one. Using the community blocklist (or 3rd party lists) can easily make you make over 50k writes (and only the first 1k will be actually written to the KV), this means that you won't be able to use the community blocklist content in a free plan and have to set `crowdsec_config.only_include_decisions_from` to `["crowdsec", "cscli"]` in the bouncer configuration.

Hi @blotus,
thanks for the info! I am trying to use this bouncer but it doesn't seem to work (I can't see any IP added in the KV store when a ban is triggered).
I can see from the LAPI logs that the bouncer periodically performs GET /v1/decisions/, but the bouncer logs doesn't show anything (the last line of its logs is "Starting turnstile rotator" for the zone I am interested in.
This is the config I am using:

Bouncer config
# Config generated by using /etc/crowdsec/bouncers/crowdsec-cloudflare-worker-bouncer.yaml as base
cloudflare_config:
  accounts:
      - id: <id>
        ban_template: "" # template to use for ban action, set empty to use default
        zones:
          - zone_id: <id>
            actions: # supported actions for this zone. eg value ["ban", "captcha"]
              - ban
            default_action: captcha
            routes_to_protect:
              - '*<my_domain>/*'
            turnstile: # Turnstile must be enabled if captcha action is used.
              enabled: true
              rotate_secret_key: true
              rotate_secret_key_every: 168h0m0s
              mode: managed
          - zone_id: <id>
            actions: # supported actions for this zone. eg value ["ban", "captcha"]
              - ban
            default_action: ban
            routes_to_protect:
              - '*<my_domain>/*'
            turnstile: # Turnstile must be enabled if captcha action is used.
              enabled: true
              rotate_secret_key: true
              rotate_secret_key_every: 168h0m0s
              mode: managed
          - zone_id: <id>
            actions: # supported actions for this zone. eg value ["ban", "captcha"]
              - ban
            default_action: ban
            routes_to_protect:
              - '*<my_domain>/*'
            turnstile: # Turnstile must be enabled if captcha action is used.
              enabled: true
              rotate_secret_key: true
              rotate_secret_key_every: 168h0m0s
              mode: managed
          - zone_id: <id>
            actions: # supported actions for this zone. eg value ["ban", "captcha"]
              - ban
            default_action: ban
            routes_to_protect:
              - '*<my_domain>/*'
            turnstile: # Turnstile must be enabled if captcha action is used.
              enabled: true
              rotate_secret_key: true
              rotate_secret_key_every: 168h0m0s
              mode: managed
        token: <my_token>
        account_name: <my_account_name>
crowdsec_config:
  lapi_url: http://crowdsec-engine:8080/
  lapi_key: <my_lapi_key>
  update_frequency: 10s
  include_scenarios_containing: [] # ignore IPs banned for triggering scenarios not containing either of provided word
  exclude_scenarios_containing: [] # ignore IPs banned for triggering scenarios containing either of provided word
  only_include_decisions_from: ["cscli"] # only include IPs banned due to decisions orginating from provided sources. eg value ["cscli", "crowdsec"]
  key_path: ""
  cert_path: ""
  ca_cert_path: ""
daemon: false
log_level: info
log_mode: ""
log_dir: /var/log/
prometheus:
  enabled: true
  listen_addr: <IP_ADDRESS>
  listen_port: "2112"

Am I missing something obvious?


Add dependabot settings for automated Maven dependency upgrades

I'd like to work on this, can you assign me please? @rmgrimm


Updates for Cloud SaaS Team Change

Updates that reflect the Cloud SaaS team change to IAM and Admin Experience team.

Some pending TODOs to be tracked in the next PR(s):

[ ]  In content/company-info-and-process/onboarding/product-onboarding-videos.md, add video
[ ] In content/departments/product-engineering/engineering/cloud/index.md, add mission
[ ] In content/departments/product-engineering/engineering/cloud/iam-and-admin-exp/index.md, update slack channel #cloud-saas-jira
[ ] In content/departments/talent/tools/resources_for_recruiting_operations.md, there's a todo/question


No close field in vscode.d.ts interface TaskPresntationOptions

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.59.0
OS Version: Linux JD 5.13.10-arch1-1

Although there is an option to close terminal after task completes running in tasks.json, the option doesn't exist in vscode.d.ts interface TaskPresntationOptions
Steps to Reproduce:

Create VS Code extension
Create a task in extension
Try to set presentation field with close:

Steps to Fix:

Add close field to interface TaskPresntationOptions in vscode.d.ts:

No TypeScript error:

Behaves as expected, task closes after run:

Adding this issue for a pull request

@jackos can you please describe why having the presentation options close is important to have as extension API? We do not like to add API just for the sake of having it, so I want to understand that it will be useful for multiple extension authors before adding it.

Hi @alexr00
I added details of why it's important in pull request 131127

@jackos sounds like you have a real use case for it. Thanks for the details!

Reopening for finalization


not able to run test_cmdline

Am I missing something below? The "-vv" very_verbose mode is also not printing all the infos.

       Code: 1
       Path: "C:\Users\BTEM7\workspace\mbedOS\mbed-os-example-blinky"
       Command: "c:\python27\python.exe -u C:\Users\BTEM7\workspace\mbedOS\mbed-os-example-blinky\mbed-os\tools\run_icetea.py -m DISCO_L475VG_IOT01A -t GCC_ARM -n test_cmdline -v --application-list"
       Tip: You could retry the last command with "-v" flag for verbose output
---

I also tried to re-install packages if there was missing any with requirements.txt. Below is the output:

C:\Users\[User]\Desktop>pip install -r requirements.txt
Collecting<EMAIL_ADDRESS>(from -r requirements.txt (line 20))
  Cloning https://github.com/armmbed/manifest-tool.git (to revision v1.4.6) to c:\users\btem7\appdata\local\temp\pip-req-build-bawebr
Requirement already satisfied (use --upgrade to upgrade): manifest-tool==1.4.6 from<EMAIL_ADDRESS>in c:\python27\lib\site-packages (from -r requirements.txt (line 20))
Requirement already satisfied: colorama==0.3.9 in c:\python27\lib\site-packages (from -r requirements.txt (line 1)) (0.3.9)
Requirement already satisfied: pyserial<=3.4,>=3 in c:\python27\lib\site-packages (from -r requirements.txt (line 2)) (3.4)
Requirement already satisfied: prettytable==0.7.2 in c:\python27\lib\site-packages (from -r requirements.txt (line 3)) (0.7.2)
Requirement already satisfied: Jinja2<=2.10,>=2.7.3 in c:\python27\lib\site-packages (from -r requirements.txt (line 4)) (2.10)
Requirement already satisfied: intelhex<=2.2.1,>=1.3 in c:\python27\lib\site-packages (from -r requirements.txt (line 5)) (2.2.1)
Requirement already satisfied: junit-xml==1.8 in c:\python27\lib\site-packages (from -r requirements.txt (line 6)) (1.8)
Requirement already satisfied: pyYAML==3.13 in c:\python27\lib\site-packages (from -r requirements.txt (line 7)) (3.13)
Requirement already satisfied: urllib3[secure]==1.23 in c:\python27\lib\site-packages (from -r requirements.txt (line 8)) (1.23)
Requirement already satisfied: requests<2.21,>=2.20 in c:\python27\lib\site-packages (from -r requirements.txt (line 9)) (2.20.1)
Requirement already satisfied: intervaltree<3,>=2 in c:\python27\lib\site-packages (from -r requirements.txt (line 10)) (2.1.0)
Requirement already satisfied: mbed-ls<1.7,>=1.5.1 in c:\python27\lib\site-packages (from -r requirements.txt (line 11)) (1.6.2)
Requirement already satisfied: mbed-host-tests<=1.5,>=1.1.2 in c:\python27\lib\site-packages (from -r requirements.txt (line 12)) (1.1.2)
Requirement already satisfied: mbed-greentea<=1.5,>=0.2.24 in c:\python27\lib\site-packages (from -r requirements.txt (line 13)) (0.2.24)
Requirement already satisfied: beautifulsoup4<=4.6.3,>=4 in c:\python27\lib\site-packages (from -r requirements.txt (line 14)) (4.6.3)
Requirement already satisfied: fuzzywuzzy<=0.17,>=0.11 in c:\python27\lib\site-packages (from -r requirements.txt (line 15)) (0.17.0)
Collecting pyelftools<=0.25,>=0.24 (from -r requirements.txt (line 16))
  Using cached https://files.pythonhosted.org/packages/fa/9a/0674cb1725196568bdbca98304f2efb17368b57af1a4bb3fc772c026f474/pyelftools-0.25.tar.gz
Requirement already satisfied: jsonschema==2.6.0 in c:\python27\lib\site-packages (from -r requirements.txt (line 17)) (2.6.0)
Requirement already satisfied: future==0.16.0 in c:\python27\lib\site-packages (from -r requirements.txt (line 18)) (0.16.0)
Requirement already satisfied: six==1.11.0 in c:\python27\lib\site-packages (from -r requirements.txt (line 19)) (1.11.0)
Requirement already satisfied: mbed-cloud-sdk==2.0.1 in c:\python27\lib\site-packages (from -r requirements.txt (line 21)) (2.0.1)
Collecting icetea<1.1,>=1.0.2 (from -r requirements.txt (line 22))
  Using cached https://files.pythonhosted.org/packages/4a/1f/c9252b4599862a1d86cf599e1f067cbc880448a17bbd4752608d8aa99e86/icetea-1.0.2-py2-none-any.whl
Requirement already satisfied: ecdsa>=0.13 in c:\python27\lib\site-packages (from manifest-tool==1.4.6->-r requirements.txt (line 20)) (0.13)
Requirement already satisfied: cryptography>=2.0.0 in c:\python27\lib\site-packages (from manifest-tool==1.4.6->-r requirements.txt (line 20)) (2.1.4)
Requirement already satisfied: pyasn1<0.3.0,>=0.2.1 in c:\python27\lib\site-packages (from manifest-tool==1.4.6->-r requirements.txt (line 20)) (0.2.3)
Requirement already satisfied: asn1ate>=0.5 in c:\python27\lib\site-packages (from manifest-tool==1.4.6->-r requirements.txt (line 20)) (0.6.0)
Requirement already satisfied: pyparsing>=2.1.0 in c:\python27\lib\site-packages (from manifest-tool==1.4.6->-r requirements.txt (line 20)) (2.3.0)
Requirement already satisfied: protobuf<3.6.0,>=3.5.0 in c:\python27\lib\site-packages (from manifest-tool==1.4.6->-r requirements.txt (line 20)) (3.5.2.post1)
Requirement already satisfied: MarkupSafe>=0.23 in c:\python27\lib\site-packages (from Jinja2<=2.10,>=2.7.3->-r requirements.txt (line 4)) (1.1.0)
Requirement already satisfied: idna>=2.0.0; python_version <= "2.7" and extra == "secure" in c:\python27\lib\site-packages (from urllib3[secure]==1.23->-r requirements.txt (line 8)) (2.7)
Requirement already satisfied: ipaddress; python_version <= "2.7" and extra == "secure" in c:\python27\lib\site-packages (from urllib3[secure]==1.23->-r requirements.txt (line 8)) (1.0.22)
Requirement already satisfied: pyOpenSSL>=0.14; python_version == "2.7" and extra == "secure" in c:\python27\lib\site-packages (from urllib3[secure]==1.23->-r requirements.txt (line 8)) (18.0.0)
Requirement already satisfied: certifi; extra == "secure" in c:\python27\lib\site-packages (from urllib3[secure]==1.23->-r requirements.txt (line 8)) (2018.11.29)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in c:\python27\lib\site-packages (from requests<2.21,>=2.20->-r requirements.txt (line 9)) (3.0.4)
Requirement already satisfied: sortedcontainers in c:\python27\lib\site-packages (from intervaltree<3,>=2->-r requirements.txt (line 10)) (2.1.0)
Requirement already satisfied: fasteners in c:\python27\lib\site-packages (from mbed-ls<1.7,>=1.5.1->-r requirements.txt (line 11)) (0.14.1)
Requirement already satisfied: appdirs>=1.4 in c:\python27\lib\site-packages (from mbed-ls<1.7,>=1.5.1->-r requirements.txt (line 11)) (1.4.3)
Requirement already satisfied: lockfile in c:\python27\lib\site-packages (from mbed-greentea<=1.5,>=0.2.24->-r requirements.txt (line 13)) (0.12.2)
Requirement already satisfied: mock in c:\python27\lib\site-packages (from mbed-greentea<=1.5,>=0.2.24->-r requirements.txt (line 13)) (2.0.0)
Requirement already satisfied: functools32; python_version == "2.7" in c:\python27\lib\site-packages (from jsonschema==2.6.0->-r requirements.txt (line 17)) (3.2.3.post2)
Requirement already satisfied: python-dateutil>=2 in c:\python27\lib\site-packages (from mbed-cloud-sdk==2.0.1->-r requirements.txt (line 21)) (2.7.5)
Requirement already satisfied: python-dotenv>=0.8.2 in c:\python27\lib\site-packages (from mbed-cloud-sdk==2.0.1->-r requirements.txt (line 21)) (0.10.1)
Requirement already satisfied: semver in c:\python27\lib\site-packages (from icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (2.8.1)
Requirement already satisfied: yattag in c:\python27\lib\site-packages (from icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (1.10.1)
Requirement already satisfied: pyshark-legacy in c:\python27\lib\site-packages (from icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (0.3.8)
Requirement already satisfied: mbed-flasher==0.9.* in c:\python27\lib\site-packages (from icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (0.9.2)
Requirement already satisfied: jsonmerge in c:\python27\lib\site-packages (from icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (1.5.2)
Requirement already satisfied: cffi>=1.7; platform_python_implementation != "PyPy" in c:\python27\lib\site-packages (from cryptography>=2.0.0->manifest-tool==1.4.6->-r requirements.txt (line 20)) (1.11.5)
Requirement already satisfied: enum34; python_version < "3" in c:\python27\lib\site-packages (from cryptography>=2.0.0->manifest-tool==1.4.6->-r requirements.txt (line 20)) (1.1.6)
Requirement already satisfied: asn1crypto>=0.21.0 in c:\python27\lib\site-packages (from cryptography>=2.0.0->manifest-tool==1.4.6->-r requirements.txt (line 20)) (0.24.0)
Requirement already satisfied: setuptools in c:\python27\lib\site-packages (from protobuf<3.6.0,>=3.5.0->manifest-tool==1.4.6->-r requirements.txt (line 20)) (40.6.3)
Requirement already satisfied: monotonic>=0.1 in c:\python27\lib\site-packages (from fasteners->mbed-ls<1.7,>=1.5.1->-r requirements.txt (line 11)) (1.5)
Requirement already satisfied: funcsigs>=1; python_version < "3.3" in c:\python27\lib\site-packages (from mock->mbed-greentea<=1.5,>=0.2.24->-r requirements.txt (line 13)) (1.0.2)
Requirement already satisfied: pbr>=0.11 in c:\python27\lib\site-packages (from mock->mbed-greentea<=1.5,>=0.2.24->-r requirements.txt (line 13)) (5.1.1)
Requirement already satisfied: lxml in c:\python27\lib\site-packages (from pyshark-legacy->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (4.2.5)
Requirement already satisfied: py in c:\python27\lib\site-packages (from pyshark-legacy->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (1.7.0)
Requirement already satisfied: trollius==1.0.4 in c:\python27\lib\site-packages (from pyshark-legacy->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (1.0.4)
Requirement already satisfied: logbook in c:\python27\lib\site-packages (from pyshark-legacy->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (1.4.1)
Requirement already satisfied: pyOCD!=0.13.0 in c:\python27\lib\site-packages (from mbed-flasher==0.9.*->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (0.14.3)
Requirement already satisfied: pycparser in c:\python27\lib\site-packages (from cffi>=1.7; platform_python_implementation != "PyPy"->cryptography>=2.0.0->manifest-tool==1.4.6->-r requirements.txt (line 20)) (2.19)
Requirement already satisfied: futures in c:\python27\lib\site-packages (from trollius==1.0.4->pyshark-legacy->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (3.2.0)
Requirement already satisfied: websocket-client in c:\python27\lib\site-packages (from pyOCD!=0.13.0->mbed-flasher==0.9.*->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (0.54.0)
Requirement already satisfied: pyusb>=1.0.0b2 in c:\python27\lib\site-packages (from pyOCD!=0.13.0->mbed-flasher==0.9.*->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (1.0.2)
Requirement already satisfied: pywinusb>=0.4.0 in c:\python27\lib\site-packages (from pyOCD!=0.13.0->mbed-flasher==0.9.*->icetea<1.1,>=1.0.2->-r requirements.txt (line 22)) (0.4.2)
valinor 0.0.15 has requirement pyelftools==0.23, but you'll have pyelftools 0.25 which is incompatible.
Installing collected packages: pyelftools, icetea
  Found existing installation: pyelftools 0.23
    Uninstalling pyelftools-0.23:
      Successfully uninstalled pyelftools-0.23
  Running setup.py install for pyelftools ... done
  Found existing installation: icetea 1.1.0
    Uninstalling icetea-1.1.0:
      Successfully uninstalled icetea-1.1.0
Successfully installed icetea-1.0.2 pyelftools-0.25

Please help me with above issue as I cannot say exactly what is the problem.
Thanks in advance!!

It seems you want to test from the example. This is not currently supported (multiple mains collision). If you fetch only mbed-os release , you should be able to run the test.
There's an issue in mbed-cli repository for this support, I can find the tracking issue if that would help?

Hi , Thank you for the reply. I'm not able to view the below support ticket.
https://jira.arm.com/browse/MBOCUSTRIA-359
"If you fetch only mbed-os release , you should be able to run the test." --> Can you please explain me how to do this... Sorry I'm not still clear!!

@SeppoTakalo @OPpuolitaival Could y'all help here? This appears to be an issue with running icetea.
@mukhia88 Correct. The URL is internal only for tracking purposes.
Could you humor me and copy the mbed-os directory so that it's standalone instead of being present in a project, and rerun the command again?

@cmonr I don't know why but now I'm able to run the command after copying the ".mbed" file from the root directory to "mbed-os" folder of "mbed-os-example-blinky". Is there any reason why it worked and if there is any other solution that would still help. Below is the mbed-os directory:
.astyleignore
.astylerc
.coveragerc
.git
.gitattributes
.github
.gitignore
.mbed
.pylintrc
.travis.yml
astyle-branch.out
BUILD
cmsis
components
CONTRIBUTING.md
docs
doxyfile_options
DOXYGEN_FRONTPAGE.md
doxygen_options.json
drivers
events
features
hal
Jenkinsfile
LICENSE
log
logo.png
mbed.h
platform
README.md
requirements.txt
rtos
targets
TESTS
TEST_APPS
tools
UNITTESTS
Thanks!!

mbed test --icetea is searching TEST_APPS folder from current path. If that folder founds then it can compile and run the test. Test_cmdline is very simple tests and do not test application. Test application code https://github.com/ARMmbed/mbed-os/blob/master/TEST_APPS/device/exampleapp/main.cpp and test script https://github.com/ARMmbed/mbed-os/blob/master/TEST_APPS/testcases/example/test_cmdline.py

@mukhia88 This problem sounds familiar.
A related issue was logged in mbed-os back when icetea was introduced: https://github.com/ARMmbed/mbed-os/issues/8064
If memory serves, mbed-cli's behavior is to move to the project's root before running commands. I think copying the .mbed file tricks mbed-cli to thinking that the mbed-os directory is the project root instead of mbed-os-example-blinky.

Is there now difference on behaviour of mbed test --greentea and mbed test --icetea and if so, this has to be fixed.
If Icetea tests are limited to be running in mbed-os root folder, then the tool should take care of switching the directory to that, before trying the tests.

@OPpuolitaival is there any BLE example applications which runs on handler mode? I see most of there are in thread mode and I'm not able to do much from icetea test cases available. I wanted to write some icetea test cases and run on BLE example sample applications but not able to do so. I took the applications from below location:
https://github.com/ARMmbed/mbed-os-example-ble
@cmonr i saw the ticket and the issue looks similar. I don't know why it did not print error message like it was not in a root folder or something. Then I would have come to know. Anyways I'm able to run icetea test cases for now.
@SeppoTakalo I have not tried with --greentea yet as the requirement was only icetea.

@donatieng can you answer to the ble question?

It seems the original issue is about "mbed test". It then morphed into a question about BLE application.

I am closing this, if there are questions about BLE/testing, it should be done in support forums or Mbed OS repository.


mri, trans_fname, trans parameter name consistency.

see the problem in the SPM example where mri == trans_fname

I still get confused by this issue. We should unify before 0.9

yeah.  If it's stored as -trans.fif let's call it trans everywhere.


Chromium parameters not being passed correctly

System information:

Arch Linux with kernel 5.4.21-1-lts
Go version go1.13.8 linux/amd64

Steps to reproduce:

Create a new go file and paste the example code from https://github.com/zserge/lorca#example
Run the file using go run or build and run (it does not seem to make any difference)
Get the pid of the running application and take a look to all of it's child processes (this can be done manually using htop with tree view, or by running ps --forest -o pid,tty,stat,time,cmd -g $(ps -o sid= -p PID_OF_PARENT))

Current behavior:
There is a whitespace that gets inserted in between parameters, it appears randomly between arguments, for example sometimes it turns --disable-background-networking into --disable-background- etworking or --disable-background-timer-throttling into --disable-ba kground-timer-throttling. The whitespace is not a visual issue as it appears even if redirecting the output to a text file.
See the whitespace affecting --disable-ba kground-timer-throttling In the example below:
$ ps --forest -o pid,tty,stat,time,cmd -g $(ps -o sid= -p 67930)
    PID TT       STAT     TIME CMD
  51417 pts/10   Ss   00:00:00 /bin/bash
  67834 pts/10   Sl+  00:00:00  \_ go run hello.go
  67930 pts/10   Sl+  00:00:00      \_ /tmp/go-build688287313/b001/exe/hello
  67936 pts/10   Sl+  00:00:02          \_ /usr/lib/chromium/chromium --disable-background-networking --disable-ba kground-timer-throttling --disable-backgrounding-occluded-windows --disable-breakpad --disable-client-side-phishing-detection --disable-default-apps --disable-dev-shm-usage --disable-infobars --disable-extensions --disable-features=site-per-process --disable-hang-monitor --disable-ipc-flooding-protection --disable-popup-blocking --disable-prompt-on-repost --disable-renderer-backgrounding --disable-sync --disable-translate --metrics-recording-only --no-first-run --safebrowsing-disable-auto-update --enable-automation --password-store=basic --use-mock-keychain --app=data:text/html, --user-data-dir=/tmp/lorca366230083 --window-size=480,320 --remote-debugging-port=0
  67939 pts/10   S+   00:00:00              \_ /usr/lib/chromium/chromium --type=zygote --user-data-dir=/tmp/lorca366230083
  67941 pts/10   S+   00:00:00              |   \_ /usr/lib/chromium/chromium --type=zygote --user-data-dir=/tmp/lorca366230083
  67970 pts/10   Sl+  00:00:00              |       \_ /usr/lib/chromium/chromium --type=renderer --disable-background-timer-throttling --disable-breakpad --enable-automation --disable-webrtc-apm-in-audio-service --remote-debugging-port=0 --field-trial-handle=intentionally-hidden --disable-features=site-per-process --lang=en-US --user-data-dir=/tmp/lorca366230083 --disable-client-side-phishing-detection --disable-oor-cors --enable-auto-reload --num-raster-threads=4 --enable-main-frame-before-activation --renderer-client-id=4 --shared-files=v8_snapshot_data:100
  67960 pts/10   Sl+  00:00:00              \_ /usr/lib/chromium/chromium --type=gpu-process --field-trial-handle=intentionally-hidden --disable-features=site-per-process --disable-breakpad --user-data-dir=/tmp/lorca366230083 --gpu-preferences=KAAAAAAAAAAgAAAgAAAAAAAAYAAAAAAAEAAAAAAAAAAAAAAAAAAAAAgAAAAAAAAA --shared-files
  67962 pts/10   Sl+  00:00:00              \_ /usr/lib/chromium/chromium --type=utility --field-trial-handle=intentionally-hidden --disable-features=site-per-process --lang=en-US --service-sandbox-type=network --disable-webrtc-apm-in-audio-service --user-data-dir=/tmp/lorca366230083 --shared-files=v8_snapshot_data:100

Expected behavior:
No whitespace gets inserted between parameters

Could you please upload the output of cat /proc/<pid>/cmdline? This would show the actual command line with \x00 used as a delimiter between the arguments. I'm pretty sure the CLI arguments do not contain the whitespace in Go code of Lorca, and I see that in your case the whitespace is not inserted but replaces the letter in the CLI flags, which is even more suspicious.

I did a cat /proc/<pid>/cmdline > proc.txt, please see attached file
proc.txt

Huh... There clearly is a \0 symbol in the command line (which is rendered as a white space). Still, I'm unable to reproduce it. May I ask you to play around with exec.Command to debug it further on your end? Have a look at chrome.go, where all args are passed, maybe add some logs to see how they look like before they are sent to exec.Command.

I updated the original post (I realized it is happening with kernel 4.19 not 5.4 sorry for the wrong kernel version).
Tested with linux kernel 4.19, 5.4 and 5.5, go 1.14 and two different machines (a regular laptop and an iMac), and the only scenario when I was able to reproduce this issue was in the iMac running Arch Linux with kernel 4.19.
For testing sake I also modified the newChromeWithArgs method to print all of the arguments before running exec.Command (I did this in the iMac+kernel 4.19), and all of the arguments were OK. e.g.
for i := 0; i < len(c.cmd.Args); i++ {
	fmt.Println(i, "\t", c.cmd.Args[i])
}

0        /usr/bin/chromium
1        --disable-background-networking
2        --disable-background-timer-throttling
3        --disable-backgrounding-occluded-windows
4        --disable-breakpad
5        --disable-client-side-phishing-detection
6        --disable-default-apps
7        --disable-dev-shm-usage
8        --disable-infobars
9        --disable-extensions
10       --disable-features=site-per-process
11       --disable-hang-monitor
12       --disable-ipc-flooding-protection
13       --disable-popup-blocking
14       --disable-prompt-on-repost
15       --disable-renderer-backgrounding
16       --disable-sync
17       --disable-translate
18       --metrics-recording-only
19       --no-first-run
20       --safebrowsing-disable-auto-update
21       --enable-automation
22       --password-store=basic
23       --use-mock-keychain
24       --app=data:text/html,
25       --user-data-dir=/tmp/lorca157224168
26       --window-size=480,320
27       --remote-debugging-port=0

Feel free to close the issue if you think a fix is not necessary, I'm happy running a newer kernel version.


Unable to install nuget for XF Android project

I am able to install the MediaManager nugets for iOS and my portable project but the install fails for Android. XF is at the latest stable version <IP_ADDRESS> using VS 2017.  I get this error.

I did not update any android libraries. I am not sure why this would pop up. Any help will be appreciated. Thanks!

This is because Xamarin Forms currently only supports the Android Support libraries in version 23.3.0 and the new MediaManager plugin wants 25.1.1 or higher. I think you have to use the MediaManager plugin in version 0.3

See https://github.com/martijn00/XamarinMediaManager/issues/163#issuecomment-289018449

I am new to XF, please bare with me, but, I am using <IP_ADDRESS> and the requirement is as follows:

for <IP_ADDRESS> as follows:

The 2 dependencies look exactly the same to me. So, I am not sure what I am missing.

So the difference is with MonoAndroid 1.0 it is = 23.3.0 and with 7.0 it is >= 23.3.0 which means you need to set your Android project to target v7.0 or higher, after that you can update to v25.1.1 or higher support packages.

So do you mean that we need to downgrade xamarin.form to install this plugin?


--batch-file URL enumeration improvements

These four minor improvements apply to reading the list of URLs from the file supplied via the --batch-file (-a) command line option.

Exclude blank and empty lines in the file from being returned as enumerated URLs. Currently, lines with leading whitespace are only skipped when that whitespace is followed by a comment character ('#', ';', ']'). This means that each empty line or line consisting only of whitespace currently appears in the list of URLs to process as an extraneous (trimmed) empty string which gets excluded later on.

[bug fix] Detect and correctly remove the Unicode BOM when the file descriptor is already decoding Unicode text lines. With Python 3, the batch_fd enumerator returns the lines of the file as Unicode. For UTF-8 BOM, this means that the raw BOM bytes from the file \xef \xbb \xbf show up converted into a single \ufeff character prefixed to the first enumerated text line.
This fix solves several buggy interactions between the presence of BOM, the skipping of comments and/or blank lines, and ensuring the list of URLs is consistently trimmed. For example, if the first line of the file is blank, the BOM would be returned as a standalone URL, but if the first line contains a URL, it will be prefixed with this unwanted single \ufeff character--which will have therefore inhibited the proper trimming of any leading whitespace. Currently, the UnicodeBOMIE helper attempts to recover from such error cases, but that approach is inherently flawed, because clearly it is illogical for a BOM to appear in a (non-batch) URL(s) specified directly on the command line... and for that matter, on any URL after the first one in a batch listing, as well.
Regardless of any subsequent BOM handling, this PR prevents the error from happening in the first place (at least on Python3). So unless UnicodeBOMIE is somehow relevant to Python 2.7 (which I wouldn't know about), I would suggest getting rid of that kludgy extractor entirely.

Per section 2.4.3 of IETF RFC2396, the # cannot appear in a URL "because it is used to delimit a URI from a fragment identifier in URI references." In other words, it delimits what is interpreted by the browser, so it cannot affect what gets downloaded online in any way. There is no special treatment in the read_batch_urls code, but because URLs have that design requirement, it has been previously possible to use the # character for end-of-line comments in the batch file. This PR adds proper trimming of the # character into the read_batch_urls processing so that the URLs it enumerates are cleaned and trimmed more consistently. For example, the current "accidental" use of # interferes with the proper end-trimming of any whitespace that precedes it.

Having fixed read_batch_urls so that it more consistently enumerates only properly trimmed URLs, it becomes sensible and trivial to do a quick on-the-fly elimination of exact duplicates so that the same URL is not returned multiple times. Of course this is done without disturbing the original order in which the URLs are listed.

Blank and whitespace lines are excluded just fine. Provide concrete examples that does not work, add tests.
Add tests.
# is an indispensable part of URI and must not be touched. Lots of extractors already use it.
Duplicate entries may be intentional.


Removed convince initializers

Since this is a first party image solution by Apple, there is no need to have it use convince initializers. Additionally, because there is a test case for all symbols we know they all work.

Didn't realize that deleting this branch in my fork would close the pull request. I have restored the branch and reopened the pull request.

@Rspoon3 Thanks for opening this PR, I made a few comments and its ok to take your time to respond but for now I will close this PR since its been a few months. Please feel totally free to open a new PR if you want to address the concerns I brought up. I still think that this approach might be too opinionated since the user should decide if they want to force unwrap or not.


Feature Request for read_fwf

I typically work with fixed width files and read_table looks to only handle delimited files.  There doesn't appear to be a way to read fixed width files.

This should be doable with a bit of effort. Are you interested in working on it?

If I thought I was qualified I would love to take a stab at it.  Unfortunately, I'm a rather new Python coder and don't feel that I could adequately handle such a task (not understanding/knowing what's under the hood of Dask).  However, another few months under my belt at development in this language I may try if it's not already being developed.  I haven't used Dask all that much yet as I couldn't figure out how to shove a fixed width file into it without going through pandas first and instead have been bench-marking my solutions using multiprocessing.  I just didn't find anyone suggesting this feature and wanted to put it out there for others to consider.

I'll try working on this issue.


Update docstrings and examples (part 9)

Codecov Report
Patch coverage: 85.71% and project coverage change: -0.02% :warning:

Comparison is base (702f95a) 100.00% compared to head (5662bc4) 99.98%.

Additional details and impacted files
@@             Coverage Diff             @@
##              main     #378      +/-   ##
===========================================
- Coverage   100.00%   99.98%   -0.02%
===========================================
  Files          311      311
  Lines         9980     9984       +4
  Branches       975      975
===========================================
+ Hits          9980     9983       +3
- Misses           0        1       +1

Files Changed
Coverage Δ

src/gravitorch/optimizers/factory.py
100.00% <ø> (ø)

src/gravitorch/rsrc/base.py
100.00% <ø> (ø)

src/gravitorch/rsrc/distributed.py
100.00% <ø> (ø)

src/gravitorch/rsrc/logging.py
100.00% <ø> (ø)

src/gravitorch/rsrc/pytorch.py
100.00% <ø> (ø)

src/gravitorch/rsrc/sysinfo.py
100.00% <ø> (ø)

src/gravitorch/optimizers/noop.py
92.30% <50.00%> (-7.70%)
:arrow_down:

src/gravitorch/optimizers/__init__.py
100.00% <100.00%> (ø)

src/gravitorch/optimizers/utils.py
100.00% <100.00%> (ø)

src/gravitorch/rsrc/__init__.py
100.00% <100.00%> (ø)

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


Add support for PATCH operation for the entities Queue, Topic, Subscription and SharedautorizationRule.

This https://github.com/Azure/azure-rest-api-specs/pull/1036 PR contains PATCH for namespace entity, ARM required this operation to be enabled for all top level resources.

Thank you for bringing this to our attention. We are acknowledging the request for this feature. We have added it to our backlog and it should be addressed in a future release (date TBD). I am closing this issue, but feel free to reopen it if you have any additional questions about this feature.


RISC-V/QEMU: Use ROMFS image to load user applications in kernel mode and update rv-virt board documentation

Summary

risc-v/qemu-rv/rv-virt: Add support to load ROMFS image:

Use ROMFS to load user applications in kernel mode instead of using hostfs.

Documentation/rv-virt:

Add a section on how to debug RISC-V emulator using QEMU and GDB.
Remove README.txt and create a specific documentation for it, updating the description of the available configurations.

Impact
Allow to use romfs instead of hostfs to load user applications in kernel mode.
Testing
Testing can be performed with rv-virt:knsh32_rofms and QEMU as described in the documentation.

@masayuki2009 , can you take a look, please? (specially regarding the description of the defconfigs in the docs)

@tmedicci good idea @tmedicci !!! It is better to avoid complexity and simplify users' life


maxComments should not be a shared variable

As noted in PR #15, the maxComments variable being shared opens up race conditions between users since it is shared among all users. Should come up with a way to either make maxComments unique to each user.

One idea may be to use session storage (this article explains why you would use it pretty well). What you could do is have a default maxComments value and then whenever there is a new GET request on the page, you either take the new value they selected or the value stored in session storage.This way, the user will never use a value that is entered from another user and there is no need to do a server request and it is specific to each user. I am pretty sure that it works this way but I would do a little extra research as I am not 100% sure.


Last section needs updated to reflect the New Exchange Admin Center.

The Exchange Online section of this tutorial needs updated to reflect the steps to take inside the New Exchange admin center.

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

ID: 4ecd4f58-0a50-05f2-426d-6bc8ca8f5940
Version Independent ID: b3344450-204e-3bb4-54ad-5fa4f85a2695
Content: Step 1 - Stop an employee from logging in to Microsoft 365 - Microsoft 365 admin
Content Source: microsoft-365/admin/add-users/remove-former-employee-step-1.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@michaelvgibson Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5452
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @michaelvgibson Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


CI for Node

create node.js.yml

Contains the compiled JavaScript (5 ms)
Initializes with two players (2 ms)
Starts the game with a random player (1 ms)
Sets the current player to be whoever it is not (1 ms)


python3Packages.monty: fix tests

Motivation for this change
noticed it was failing in #84463
upstreamed to:
https://github.com/materialsvirtuallab/monty/pull/121
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

[12 built, 6 copied (14.5 MiB), 2.9 MiB DL]
https://github.com/NixOS/nixpkgs/pull/84503
1 package marked as broken and skipped:
python38Packages.sunpy

12 package built:
python37Packages.dftfit python37Packages.lammps-cython python37Packages.monty python37Packages.pymatgen python37Packages.pymatgen-lammps python37Packages.sumo python38Packages.dftfit python38Packages.lammps-cython python38Packages.monty python38Packages.pymatgen python38Packages.pymatgen-lammps python38Packages.sumo

[12 built, 6 copied (14.5 MiB), 2.9 MiB DL]
https://github.com/NixOS/nixpkgs/pull/84503
1 package marked as broken and skipped:
python38Packages.sunpy

12 package built:
python37Packages.dftfit python37Packages.lammps-cython python37Packages.monty python37Packages.pymatgen python37Packages.pymatgen-lammps python37Packages.sumo python38Packages.dftfit python38Packages.lammps-cython python38Packages.monty python38Packages.pymatgen python38Packages.pymatgen-lammps python38Packages.sumo

since editing my post isn't working


nü as nu: not supported

In the CC-CEDICT nǚ is written as nu:3 , but prettify returns nǔ:

Resolved in the PR you opened (#7).


time changes influence on agent.ps1

i was tring to debug a connection problem between my server and agent and i found out that if i have set the clock to a wrong date the agent will not get tasks from the server
in debug i found this on agent.ps1
if the epoch counter isn't within a +/- 10 minute range (600 seconds)
    #   skip processing this packet
    if ($counter -lt ($ServerEpoch-600) -or $counter -gt ($ServerEpoch+600)){
        return
    }

this condition dont pass the tasks if the time zones arent correct
why so?
can i remove this condition?
what does it spouse to do anyway?

sure, you can remove this condition- the original idea was to introduce some basic anti-replay logic that prevents third parties from replaying packets between both parties.

Like @HarmJ0y said, you can always remove it if it is causing issues with agent comms.