add basic authentication and fix logformat bug

Since the logsearch website is a public site that everyone can access. The basic authentication is used for kibana login, which denies someone not related from reading our log information.
Add more log format in the nginx.config.erb, so that the nginx.access.log can show the logs of nginx accessing status.

Basic Auth is good enhancement to the logsearch boshrelease, even if minimalist. Logsearch for Cloudfoundry extension offers an even more compelling option see : https://github.com/logsearch/logsearch-for-cloudfoundry
The solution proposed by @mrdavidlaing includes a cf app, integrated with UAA / cf login, and multitenant logsearch access.
Maybe this PR could be merged with a PR for cloudfoundry logsearch ?
Pierre

@yuanzhao,
Firstly, thank you for your PR.
Unfortunately, we're not going to be able to merge it in its current state, because it contains 2 separate changes that we would like to discuss separately.

Change to logformat - this is a change we would like to merge, subject to some further discussion.  We've been doing some work with Elasticsearch's slowlog logging, and would like to make sure this change works well with that one.

Basic Authentication.  We've explicitly avoided adding authentication up til this stage, prefering to use IP firewall rules to limit access.  We understand the requirement; but are not quite sure the correct way for it to be implemented, especially given the fact that Kibana 4 comes with its own "basic" authentication.

So, to move forward please could you separate the changes, and raise 2 separate PRs.
Before we can merge, we will also need signed Contributor License Agreements from all the committers on the PR - please see https://github.com/logsearch/licensing for details.
Thanks again; and apologies for not being able to merge these changes at this time.

Hi David,
Here is the signed contributor license agreements for the log format pull
request.
1.pdf
https://docs.google.com/a/pivotal.io/file/d/0B-vhTsorAqZzUE56dTVxM0pHQk0/edit?usp=drive_web

2.pdf
https://docs.google.com/a/pivotal.io/file/d/0B-vhTsorAqZzeElRUk9YSnpkZEk/edit?usp=drive_web

3.pdf
https://docs.google.com/a/pivotal.io/file/d/0B-vhTsorAqZzREdDWVlpMktqZ0U/edit?usp=drive_web

4.pdf
https://docs.google.com/a/pivotal.io/file/d/0B-vhTsorAqZzNGprR3VQMTB4akU/edit?usp=drive_web

2015-05-05 18:34 GMT+08:00 David Laing<EMAIL_ADDRESS>
Closed #120 https://github.com/logsearch/logsearch-boshrelease/pull/120.
—
Reply to this email directly or view it on GitHub
https://github.com/logsearch/logsearch-boshrelease/pull/120#event-297086057
.

--
Thanks & regards,
Yuan

@yuanzhao - thanks for the CLA.  All looks good to me.


Add 'hubble observe' to bugtool

We should figure out a sane default for adding hubble observe to bugtool / sysdump.

I think this is a duplicate of https://github.com/cilium/cilium/issues/11841.

This has been implemented already for a long time.


Hide Remove filtered reports button

Show Remove filtered reports button at the bottom of the filter bar.
Hide Remove filtered reports button from the filter bar if the user doesn't have storage permission for the current product.
Show the number of results which will be deleted on the confirmation dialog.

This report removal works for me. I have just one note: when the reports are removed, then the filter options remain the same. This results that I get an empty page, since the filter options refer to the deleted reports. Shouldn't the filter options be reset to the default options except for the run name?

I've been out of the loop regarding CodeChecker — is this a webapp only feature?

@whisperity Yes, this is only a web app feature. We discussed it with @gyorb that the user must see what what kinds of reports will be removed.


GlfwKeyCallback does not register F11 key on QWERTZ keyboard

I am using a PS/2 QWERTZ keyboard on windows and the F11 key does not get registered by GLFW. When I press it, the callback does not get triggered. Are there any workarounds regarding this issue and is it known to more people?
I am using the current version glfw 3.4 and a Windows 11 machine
I suspect that it has something to do with the second nature of some F(X) keys on windows, since some of them might emit an action that is caught by the windows platform. I am very sure that WIN32 allows you to query for any F(X) key though, so this issue is very bizarr to me.

I just tested this on my machine (Windows 11 with QWERTY USB keyboard) and the F11 key is registered:
00000153 to 1 at 4.756: Key 0x012c (F11) Scancode 0x0057 (with no mods) was pressed
00000154 to 1 at 4.833: Key 0x012c (F11) Scancode 0x0057 (with no mods) was released

I get the same result setting the keyboard input to AZERTY and QWERTZ (i.e. it works with the above output). So perhaps this is related to the keyboard hardware?

If you're able to run the events.c test program you should be able to see all events that GLFW can process.
What I see when I use special function keys (increase volume for example) is the same key & scancode:
00000169 to 1 at 25.140: Key (UNKNOWN) Scancode 0x0100 (with no mods) was pressed
0000016a to 1 at 25.217: Key (UNKNOWN) Scancode 0x0100 (with no mods) was released

I will check it with this file tomorrow. For now "Fn + F[]" seems to be necessary. At the moment I am using a keyboard callback in my application. It does not get called when I press F11 and some other F[] keys alone. So if events.c triggers something else then the keyboard callback, I might not catch it. For me there are two possibilities at the moment:

The Lenovo hardware/keyboard bindings I am using, do something weird
The function set by glfwSetKeyCallback is not triggered by certain function-keys

Alright. I tried it with events.c and still got the same problem. The F11 key did not get registered, so no glfw callback catched the key press.

I experimented with lParam and the window Message system. There is NO unique event/flag existing for F11 and other function keys. WIN32 registers every F key as move movement, which is very weird. The conclusion is, it seems that it is impossible to detect specific function key presses on lenovo hardware. I would even say it is impossible, if WIN32 does not even get a unique flag for function keys...

I'm not sure there's that much we can sensibly do with unusual OEM keys without potentially API breaking changes to GLFW, but if you are interested in following up on this then it might be worth debugging this via the code in GLFW's main windowProc, along with the documentation for WM_KEYDOWN and WM_SYSKEYDOWN. The documentation on scan codes may also be helpful.
For example when I switch my keyboard to use the function keys for controls and I press the volume up button (F11 with FN lock) I get a WM_KEYDOWN message with scancode 0x100 (KF_EXTENDED) and wParam of VK_VOLUME_UP from virtual-key-codes.
After some debugging I found that for my keys a helpful addition (breaking change) is to modify the windowProc function's keyboard handling as follows:
// lines 705 onwards...
        case WM_KEYDOWN:
        case WM_SYSKEYDOWN:
        case WM_KEYUP:
        case WM_SYSKEYUP:
        {
            int key, scancode;
            const int action = (HIWORD(lParam) & KF_UP) ? GLFW_RELEASE : GLFW_PRESS;
            const int mods = getKeyMods();

            // store scancode plus extended bit
            scancode = (HIWORD(lParam) & (KF_EXTENDED | 0xff));
            // actual scancode is only in first byte, if this is zero map virtualkey
            if (!(scancode & 0xFF))
            {
                // NOTE: Some synthetic key messages have a scancode of zero
                // HACK: Map the virtual key back to a usable scancode
                // we | with the original to add the extended bit if present
                scancode = scancode | MapVirtualKeyW((UINT) wParam, MAPVK_VK_TO_VSC);
            }

The change here is in how extended key codes are handled. This could break previous user code because GLFW_KEY_RIGHT_CONTROL now gets a different scancode.
I'm considering a PR on this. My changes may not help your case, but it definitely maps more of the keyboard space to a unique scancode. More changes may be needed to get this fully working though.


"serverpod_runtime_settings" does not exist

Describe the bug
Failed to connect to the database. Retrying in 10 seconds. PostgreSQLSeverity.error 42P01: relation "serverpod_runtime_settings" does not exist
Expected behavior
The server should up and running
Screenshots

Serverpod versions
Serverpod version: 1.1.1

Library version: Check the pubspec.yaml file for the version of Serverpod you are using.

Platform information
Windows 10 pro 64 bitbuild 19045
How to solve that part???

This is because you haven't created the tables in the database!
You have to run the script setup-tables.cmd that you have in your root folder of the project. You can also do this manually like described here: https://stackoverflow.com/questions/76180598/serverpod-sql-error-when-starting-a-clean-project/76180599#76180599.


runMCMC does not reset the WAIC state when rerun

The result is that the WAIC reported in subsequent runs is actually the value from the initial run. We should reset it as we do the overall MCMC state, but only for the first chain, as we want an overall aggregate WAIC from running multiple chains in one call to runMCMC.

@paciorek Nice eye.  If you're handling this, then I'll certainly look over the modifications, whenever that is.

They're in PR #1256 .

@danielturek  Oops, I forgot you might look at this. I merged it in. But let me know if you have any comments. Feel free to close this if not.

@paciorek Reviewed PR #1256 regarding resetting WAIC only on (and always on) the onset of the first chain.  LGTM.


[Question] vp8 on iPhone Safari

Hi!
I am sorry that I created bug for question. It's because I can't ask in Slack due the sanctions for Russian citizens :(
Thank you for the great product. I tested it and it works perfectly. But I can't understand how it supports forwarding vp8 streams to iPhone devices. Is livekit transcode vp8 to h264 under the hood? For example I start sharing screen on Desktop Chrome, so it will be encoded with vp8 codec which is not supported on iPhone Safari. But with livekit I was able to see this stream. Does it mean that livekit not just forwards but also converts the video stream? I don't have any other idea how it can work.
I looked in sources code, but does not find anything related to vp8 -> h264 conversion.
I saw a benchmark on livekit docs page. According to it livekit on 16 core server process 5 publisher and 1k subscribers with 90% of load. Is it because of video stream conversion? if it just forwarding, why are the numbers so small? I believe that just for forwarding only the network is the matter. Am I wrong?
Thank you.

iPhone devices support decoding VP8 via software. RFC7742 indicates all WebRTC clients must support both codecs.
We do not convert video on the server side, but instead utilize simulcast

Wow thank you. So the information from caniuse is wrong?

Partial support in Safari refers to being limited to macOS 11.3 or later.
When I tried to forward vp8 to iPhone with plain ion-sfu it never worked for me.
However this is great news. Thank you.


component $onChanges behavior change in 1.5.5

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
$onChanges in a component is immediately triggered
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (template: http://plnkr.co/edit/tpl:yBpEi4).
http://jsbin.com/yuyabutalu/edit?html,js,output
Change to 1.5.3 to see different behavior
What is the expected behavior?
$onChanges should not be triggered until there is a change
What is the motivation / use case for changing the behavior?
Breaking change introduced
Which versions of Angular, and which browser / OS are affected by this issue? Did this work in previous versions of Angular? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
1.5.5, working as expected in 1.5.3
Other information (e.g. stacktraces, related issues, suggestions how to fix)

This is (sort of) a duplicate of #14433.
The current (1.5.5) behavior is the correct one - the fact that it wasn't called before was actually a bug (see https://github.com/angular/angular.js/issues/14433#issuecomment-210194853 for a little more context). This is not a breaking change imo, because it wasn't documented to work in any particular order and in fact it was never intended to.
I'm going to close this, since it works as intended, but feel free to continue the discussion below.
(BTW, it might be a good idea to add a note in the docs about the expected behavior - PRs welcome :smile:)


Vue 3 upgrade - some components get focus twice on tab

Certain components get focused on twice when tabbing through the application.  This is potentially because of SkillsButton components being wrapped by router-link components.
Examples:

Level component in the header
Manage button on Project cards
Preview button for projects
Configure button on Self Report page
Achievements, subjects, skills buttons on the Metrics page (both the top and bottom buttons)

Testing results: The ticket does address components/pages mentioned in the description but there are  a number of buttons in the SkillsDisplay that also exhibit the same issue:

My Rank View Button
My Badges View Button
Subject's View Button
Learn More Button on skill page

:+1:


API Updates - Users, Groups and Roles

Qlik is updating their API with the following changes:

New API: Roles
New API: Groups
API Deprecations: Users

Source: https://qlik.dev/changelog/api-updates---users-groups-and-roles

closed by #55 #56 and #57


KETTLE-46:  Move kettle.dataSource.URL to its own file...

See KETTLE-46 for details.

In separate conversations we discussed the huge (200k+) dependencies introduced in using kettle.dataSource.URL with the browserified libraries required to make it work.  We agreed to focus on building something backed by jQuery instead.  I'll close this one, but keep the branch around for reference.


Promise warning thrown

You are returning a Promise without returning anything inside of the promise so bluebird throws a warning.
https://github.com/db-migrate/mysql/blob/master/index.js#L388
Warning: a promise was created in a handler but was not returned from it
    at Object.Base.extend.close (/opt/application/node_modules/db-migrate-mysql/index.js:388:12)
    ....

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34989231-promise-warning-thrown?utm_campaign=plugin&utm_content=tracker%2F12292998&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F12292998&utm_medium=issues&utm_source=github).

You might be able to take advantage of bluebird's Promise.fromCallback functionality here too.
http://bluebirdjs.com/docs/api/promise.fromcallback.html


[Jenkins] Resource calculation and request for a higher tier memory/cpu plan

User Story

As a [user role below]:
Carbon for IBM.com developer

I need to:
run up to 20 Jenkins build jobs at once

so that I can:
Test and merge multiple pull requests in a shorter time period

Additional information

Jenkins pods are currently tuned in a way that is not ideal, and is capping the current tier plan
Calculations should be done in order to achieve the optimal configuration for pods https://taas.cloud.ibm.com/guides/jenkins-build-environment-resource-tuning.md
May have to do the upgrade during slow period

Acceptance criteria

[ ] Upgrade Jenkins service plan to a more optimal tier

Resource calculation complete, though there does appear to be an incremental cost for increasing the size of the cluster.
Making a request for cost increase.


Allow forcing a parameter value to string

Adds ability to force a paramter value to string using the --as-string
flag.
e.g.
[I] mule · value-string-type-hint± ⟩ ks param set comp aDigit 1
[I] mule · value-string-type-hint± ⟩ ks param list comp
COMPONENT PARAM    VALUE
========= =====    =====
comp      aDigit   1
comp      bindings [{"name":"name","role":"role","users":["user1","user2"]}]
[I] mule · value-string-type-hint± ⟩ ks param set comp aDigit 1 --as-string
[I] mule · value-string-type-hint± ⟩ ks param list comp
COMPONENT PARAM    VALUE
========= =====    =====
comp      aDigit   "1"
comp      bindings [{"name":"name","role":"role","users":["user1","user2"]}]

Fixes #496
Fixes #492
Signed-off-by: bryanl<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 842

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 4.567%

Totals

Change from base Build 841:
0.0%

Covered Lines:
19

Relevant Lines:
416

💛  - Coveralls

It doesn't work in ks version 0.13.1
params.libsonnet:
local params = import '../../components/params.libsonnet';
params + {
components+: {
sample+: {
env: 'qal',
assetId:<PHONE_NUMBER>474420594,
},
},
}
When I run, it still use integer,
/usr/local/bin/ks param set sample assetId<PHONE_NUMBER>474420594 --as-string --env qal
ks version
ksonnet version: 0.13.1
jsonnet version: v0.11.2
client-go version: kubernetes-1.10.4

It doesn't work for boolean values too.
I tried the following
ks param set sample_var "true" --as-string -env <evn>
expecting the value to be saved as string, but sets as bool type.
ks version
ksonnet version: 0.13.1


TAJO-2165: Add 'ALTER TABLE UNSET PROPERTY' statement to Tajo DDL

Thank you for your patch. I'll review soon. Would you please check the test failure?

Updated! Thanks for your immediate response.
Followings are additional comments on this issue:

During updating the (potentially) broken test, one question arose: is there any list of properties that should not be unset? If there are, it seems like we have to implement an additional feature to make them unset-proof. (for example, show error message when the user tries to unset some reserved properties.) Do you have any opinion on that? Or, is it better to query about this problem on dev mailing list?
I omitted implementation for HiveCatalogStore - as you remember, it is a deprecated feature. (TAJO-1775)

Thank you for the quick update. Here are my answers.

Basically, table properties are used for optional settings per table. So, every table property is optional, and tajo should be able to work with default property values if they are absent.
However, among table properties, timezone is recently changed as a mandatory property of a table. So, currently I think it is the only one which should not be removed.
The best solution seems to store the timezone value in the TableMeta or TableDesc instead of in the optional KeyValue set. But this would be better to be done in a separate ticket, and it would be enough to force timezone as unsettable here.

The deprecated one is HCatalogStore, and HiveCatalogStore should be implemented. But, as you see, many functions of HiveCatalogStore are not implemented yet. I think it also would be better to do in a separate ticket.

Oh, my mistake. I confused HCatalogStore and HiveCatalogStore.
Let's sum up. The following is the list of tickets that should be opened. Some of them are from this discussion, and the others are what I found during this work. (I think it is better to implement disabling unset timezone feature on separate issue, not here.)

[Major] Move timezone property from KeyValue set and disable to unset it. (Related: TAJO-1185, TAJO-1241)
[Major, Umbrella] Implement missing HiveCatalogStore features.
[Trivial] Remove AlterTableNode#clone - it has been commented out.

If you have any opinion, don't hesitate to comment to me. If you merge and close this request without any comments, I will regard it you accepted my proposal. Thanks again for your guidance! :)

Hi @dongjinleekr, I looked over your patch. It looks good to me in overall. I left a single trivial comment.

@jihoonson @jinossy Thanks for your comments. I applied all your comments. (You can see the link to the comment in the log message of each commit.) Thanks for reviewing!

As soon as you accept this pull request, I will request another one - the fix for the glitches discussed in this issue. I already completed that work. :)

@dongjinleekr thank you for the quick update! I left my last comment.
Thanks!

+1. The latest patch looks good to me.
@dongjinleekr thank you for your contribution!

+1
@dongjinleekr Thanks for your contribution!!


Upgrade to netty-tcnative 2.0.57.Final

Motivation:
There was a new netty-tcnative releae which upgraded boringssl
Modifications:
Update to the next version
Result:
Use latest boringssl / netty-tcnative

This needs a new netty-tcnative release... working on it


server not run

[root@localhost server]# npm start

<EMAIL_ADDRESS>start /var/www/html/server
node server.js

/var/www/html/server/node_modules/ws/lib/websocket.js:354
...options
^^^
SyntaxError: Unexpected token ...
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:549:28)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
at Module.require (module.js:504:17)
at require (internal/module.js:20:19)
at Object. (/var/www/html/server/node_modules/ws/index.js:3:19)
npm ERR! Linux 3.10.0-1160.31.1.el7.x86_64
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "start"
npm ERR! node v6.17.1
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: node server.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node server.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the csgo-matchmaking-server package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node server.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs csgo-matchmaking-server
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls csgo-matchmaking-server
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /var/www/html/server/npm-debug.log
[root@localhost server]#

You are running node 6 but node 8 is required for this project. As explained in the Readme.

myfriend can i ask you this
this script is working now ?
there is website template or no ?
is it similar like faceit or popflash?
how i will understand is it working or not :D
Glenn de Haan @.***>, 21 Haz 2021 Pzt, 11:48 tarihinde
şunu yazdı:

You are running node 6 but node 8 is required for this project. As
explained in the Readme.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/glenndehaan/csgo-matchmaking/issues/4#issuecomment-864853608,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADU6MEMOTJU45TRNPDPUR5DTT34HPANCNFSM47BDQSBA
.

Yes the project is working fine. But is a bit old.
Also it's more like a prototype then a full featured application.
There is no website it's just a terminal application.
And no it doesn't even come close to a faceit or popflash service. But that was ever the idea.
It's a terminal application that connects to a server via websockets. So no HTTP here.... the server has a pool of csgo server ip's and ports. When enough players are ready. So for competitive that will be 10. A free server IP is given to the group. Then a timer starts that roughly takes 110 minutes. Since a match can't be longer then that and then puts the server back in the ready pool.

thank you man u explaining very well and very fast.  Myfriend lots people
looking like this script for own matchmaking system for own country csgo
servers. why u dont keep do this full finish and update like faceit ? no
need tournament mode just need 5vs5 match script teams balance with elo
can u complete this with web site also ?
Glenn de Haan @.***>, 21 Haz 2021 Pzt, 12:07 tarihinde
şunu yazdı:

Yes the project is working fine. But is a bit old.
Also it's more like a prototype then a full featured application.
There is no website it's just a terminal application.
And no it doesn't even come close to a faceit or popflash service. But
that was ever the idea.
It's a terminal application that connects to a server via websockets. So
no HTTP here.... the server has a pool of csgo server ip's and ports. When
enough players are ready. So for competitive that will be 10. A free server
IP is given to the group. Then a timer starts that roughly takes 110
minutes. Since a match can't be longer then that and then puts the server
back in the ready pool.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/glenndehaan/csgo-matchmaking/issues/4#issuecomment-864867033,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADU6MELSJ2KXT6Y7TOGM3EDTT36MHANCNFSM47BDQSBA
.

I had 2 reasons why I stopped this project and uploaded the files to GitHub just for others to maybe continue what I started:

I stopped playing CSGO after the CS matchmaking got destroyed by cheaters and farm bots.
My intention was always to use the Steam API to verify a users account. That way the system is less prone to cheating (since we can monitor and track a users account) and thus is able to implement/track ELO correctly. Also this meant I didn't have to build a user management/account system since I could use the Steam API. But just as I started the project the Steam API got shutdown and later a much stripped down version was but back (this was during the whole CSGO skin gambling time).

This was one of the primary reasons that I stopped the project. I did setup one other project: https://github.com/glenndehaan/csgo-rcon-nodejs
This project focuses more on the LAN Party type of CSGO tournaments. Where an admin can control the entire server base and monitor/manage match info (partially automated since this is a webportal that communicates to and from the csgo server, something this application doesn't do). But this project didn't manage to get a lot of attention by others in the community so I also spend less and less time on it at the end.

Myfriend in turkey we have 1 million csgo player and they hate faceit esea
because no turkish csgo server. İf i open smilar matchmaking system it will
be very populer. By the way there is easyanticheat project its cool for
csgo server i will add that anticheat to my servers man pls finish match
find queue system for website plsss
21 Haz 2021 Pzt, saat 12:25 tarihinde Glenn de Haan <
@.***> şunu yazdı:

I had 2 reasons why I stopped this project and uploaded the files to
GitHub just for others to maybe continue what I started:

I stopped playing CSGO after the CS matchmaking got destroyed by
cheaters and farm bots.
My intention was always to use the Steam API to verify a users
account. That way the system is less prone to cheating (since we can
monitor and track a users account) and thus is able to implement/track ELO
correctly. Also this meant I didn't have to build a user management/account
system since I could use the Steam API. But just as I started the project
the Steam API got shutdown and later a much stripped down version was but
back (this was during the whole CSGO skin gambling time).

This was one of the primary reasons that I stopped the project. I did
setup one other project: https://github.com/glenndehaan/csgo-rcon-nodejs
This project focuses more on the LAN Party type of CSGO tournaments. Where
an admin can control the entire server base and monitor/manage match info
(partially automated since this is a webportal that communicates to and
from the csgo server, something this application doesn't do). But this
project didn't manage to get a lot of attention by others in the community
so I also spend less and less time on it at the end.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/glenndehaan/csgo-matchmaking/issues/4#issuecomment-864880937,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADU6MEKQ53I4DJ62FTYGNG3TT4AR3ANCNFSM47BDQSBA
.


Add example select item that start with key pressed

What:
Added an example that uses setHighlightedIndex() to highlight an item in the dropdown that starts with the letter that the user presses.  Also made a very unique dropdown UI that showcases the flexibility of Downshift.
Why:
There doesn't seem to be any example of using setHighlightedIndex() and the implementation is a bit complicated.
How:
Checklist:

[X] Documentation
[ ] Tests N/A
[X] Ready to be merged
[X] Added myself to contributors table

Sure!
On Mon, Aug 13, 2018 at 12:09 PM, Kent C. Dodds<EMAIL_ADDRESS>wrote:

@kentcdodds requested changes on this pull request.
I LOVE this. Ha! It's fantastic <3 Thank you.
If you don't mind, we're actually moving downshift examples to a single
codesandbox
https://codesandbox.io/s/github/kentcdodds/downshift-examples. Do you
mind making a PR to that repo
https://github.com/kentcdodds/downshift-examples instead? Here's how
https://www.youtube.com/watch?v=3PAQbhdkTtI&list=PLV5CVI1eNcJgCrPH_e6d57KRUTiDZgs0u.
Thanks again!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/paypal/downshift/pull/541#pullrequestreview-145726457,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXW_xKm_PhGcwz3SylbGbfsjkdKj8Ahjks5uQaTNgaJpZM4V6x-M
.

@kentcdodds Here's the PR to that repo: https://github.com/kentcdodds/downshift-examples/pull/8
Let me know if there's anything else I need to add :)


Origin and Delta is missing in Metron Counter Event

Hi,
we are updating our landscapes from CF v5.5 to v6.5.
During testing we discovered that some counter events have changed.
Using the CF nozzle plugin, we see that the origin and delta fields are not populated correctly anymore:
CF 5.5 (loggregator 104.0 / loggregator-agent 2.3)
origin:"loggregator.metron"
eventType:CounterEvent timestamp:1545055762713495887
deployment:"cf" job:"diego-cell" index:"4aa468d9-3a5b-4192-8bd2-8dd0ad93c626" ip:"<IP_ADDRESS>"
tags:<key:"metric_version" value:"2.0" > tags:<key:"source_id" value:"metron" >
counterEvent:<name:"egress" delta:2646 total:11207103 >

CF 6.5 (loggregator 104.1 / loggregator-agent 3.2)
origin:""
eventType:CounterEvent timestamp:1545054696439141513
deployment:"cf" job:"diego-cell" index:"f48acf1b-e325-41e2-8b2a-34738c655062" ip:"<IP_ADDRESS>"
tags:<key:"source_id" value:"metron" >
counterEvent:<name:"egress" delta:0 total:304995 >

Can you check if this is bug or did we miss something in the release notes?
Thanks and best regards,
Jochen & Johannes

@johha I am going to close this issue for now. Please feel free to re-open if you have further concerns about the bug or the change to the metrics not sending the delta.

@johha Just to expand a bit on dropping the delta:
Most downstream consumers (Prometheus, Infux, Datadog, OpenTSDB...) have built-in functions to calculate the rate/derivative of counters and most metric libraries don't do well with the emission of deltas. Therefore, we are slowing moving to a system where we discourage the usage of deltas and prefer totals.
Do you have a specific setup or use-case that would require the deltas? If yes, I would be very interested in hearing about it.

Thanks for the quick clarification! We currently use the delta to sum up all CF.loggregator.metron..*.egress values so that we get the an overall sent messages/sec value. This is used to detect unusual high logging loads (e.g. caused by chatty apps) in a CF installation. If the delta is not available any more, we'll have to calculate it using Riemann's derivative functions.


How to position the left/right controls the right way

I have looked at this https://stackoverflow.com/questions/56988717/how-to-target-a-component-in-svelte-with-css and I ended up doing this in my CSS
.c-left {
  position: absolute;
  left: -2vw;
  top: 0;
}

<span class="control c-left" slot="left-control">
   <ChevronLeftIcon />
</span>

is there a recommended way to modify the left/right control positioning ?

The way I do it is by using :global() in the consuming project

@antony Thanks, you were right
I had to either use important or use an extra element selector to get to the take precedence
  :global(.carousel .left) {
    left: 0 !important;
  }
  :global(.wrapper .carousel .right) {
    right: 0;
  }

You need the extra selector anyway because :global descopes your selector string, so you'd affect css across your entire project which matched the same selector ;)


Possibility of factoring out the Windows Store stuff into its own repository

Right now, I feel all the Windows Store stuff makes the repo a bit confusing for someone who is not used to all the Windows-specific things.
I am wondering if it is possible to have a pure "juliaup" repository (just the command line tools) and then a separate repository for all the additional stuff that is required to do the bundling etc for the Windows Store. I feel this might simplify the development and makes it a bit clearer what is specific to the Windows Store and the CLI tool.

Maybe we just come up with a better folder structure, so that we have one folder per deployment target? There is a lot of value in the one GitHub Action workflow that handles the entire deployment story for everything…

Yeah, I guess what i mean is to have them as separate Cargo "targets" or whatever it is called. Like now, the stuff for Microsoft store like with the bundling of the Julia binary here:
https://github.com/JuliaLang/juliaup/blob/19e7e8ab0eeffc78c4e9596bfb848377cd085783/build.rs#L509-L525
is in the same build file as all the other stuff. I think it would be good if the CLI could be completely independent of the store, and then the store stuff is layered "on top" of the CLI.

I think we've done what can be done on this one. The msix specific files are all moved into a subfolder now. But there will always be conditionally compiled code in the cli because some of the commands interact with platform specific APIs (for example, juliaup self update on a Windows Store installed version just has to run some Windows Store specific code).


fix Login issue #102

use a login activity instead of a login fragment
Update the User profile when the user is connected
Add check on both login and password text fields in the loginView
fix #102

the build is successful on my local environment.

Maybe an issue with the travis file config


Usage issues related to the "win" key in the "keyboard manager" 与“win”键有关的“键盘管理器”使用问题

Microsoft PowerToys version
0.86.0
Installation method
GitHub, WinGet
Running as admin
None
Area(s) with issue?
Keyboard Manager
Steps to reproduce
I enabled the keyboard manager and swapped the mapping between the win and alt keys on the left side (There is no problem using it at that time. When locking the screen, the mapping is invalid and you still need to press Win+L). Then I switched back to the normal keyboard, but I cannot cancel the mapp
ing (which is manifested as the following three methods being ineffective: opening the keyboard manager then delete the mapping; closing the keyboard manager function; closing the software process).
At this point, the functions of the win and alt keys are completely swapped, and alt fully possesses the function of win. When locking the screen, you need to press alt+L
May it be related to whether to run PowerToys as an administrator?
But I didn't turn on this feature at the time (see attached picture)
✔ Expected Behavior
Because I purchased a small wireless keyboard with Apple key positions, I enabled the keyboard manager and swapped the mapping between the left win and alt keys (there was no problem using it at the time. When locking the screen, the mapping was invalid, and you still need to press win+L on the keyboard）
When I open the keyboard manager and delete the mapping; Turn off the keyboard manager function; When closing software processes, the functionality of win and alt should be restored.Actually, there is none
❌ Actual Behavior
Because I purchased a small wireless keyboard with Apple key position, I enabled the keyboard manager and swapped the mapping between the win and alt keys on the left side (There is no problem using it at that time. When locking the screen, the mapping is invalid and you still need to press Win+L). Then I switched back to the normal keyboard, but I cannot cancel the mapping (which is manifested as the following three methods being ineffective: opening the keyboard manager then delete the mapping; closing the keyboard manager function; closing the software process).
At this point, the functions of the win and alt keys are completely swapped, and alt fully possesses the function of win. When locking the screen, you need to press alt+L

After that, I can only use this function for reverse mapping to ensure the normal use of the keyboard. (still swapping the mapping between the win and alt keys on the left side, but the keyboard is actually normal, and only after mapping can the functions of win correspond to the "win" key)To lock the screen at this time, i need to press alt+L.
Other Software
No response

The problem has been resolved (delete the newly created 'Scancode Map' in the registry)


Update Docker image

I wrote the initial Dockerfile over a year ago. Time to revisit it and work to make the container size as small as possible.

Hey, your project description promises a cool project.
I tried to test it but
$ sudo docker run -v $PWD/input:/Open-Publisher/Source -v $PWD/output:/Open-Publisher/Books -it open-publisher rake build --trace
rake aborted!
Don't know how to build task 'build'
/usr/local/lib/ruby/2.1.0/rake/task_manager.rb:49:in `[]'
/usr/local/lib/ruby/2.1.0/rake/application.rb:148:in `invoke_task'
/usr/local/lib/ruby/2.1.0/rake/application.rb:106:in `block (2 levels) in top_level'
/usr/local/lib/ruby/2.1.0/rake/application.rb:106:in `each'
/usr/local/lib/ruby/2.1.0/rake/application.rb:106:in `block in top_level'
/usr/local/lib/ruby/2.1.0/rake/application.rb:115:in `run_with_threads'
/usr/local/lib/ruby/2.1.0/rake/application.rb:100:in `top_level'
/usr/local/lib/ruby/2.1.0/rake/application.rb:78:in `block in run'
/usr/local/lib/ruby/2.1.0/rake/application.rb:165:in `standard_exception_handling'
/usr/local/lib/ruby/2.1.0/rake/application.rb:75:in `run'
/usr/local/bin/rake:33:in `<main>'

I did the following steps in zsh shell:
$ git clone https://github.com/chrisanthropic/Open-Publisher.git
$ cd Open-Publisher
$ sudo docker build -t open-publisher .
$ sudo docker run -v $PWD/input:/Open-Publisher/Source -it open-publisher rake skel
$ sudo docker run -v $PWD/input:/Open-Publisher/Source -v $PWD/output:/Open-Publisher/Books -it open-publisher rake 'all[all]'

...
Info(prcgen):I1036: Mobi file built successfully

$ evince output/Example/Example-5x8-print.pdf

The Example worked fine, but creating a new book fails  ... can you help?

Sorry for late reply, but I am just joining the fan club. The solution is to call create instead of build. Wiki needs updating, but I somehow cannot do it.
sudo docker run -v $PWD/input:/Open-Publisher/Source -v $PWD/output:/Open-Publisher/Books -it  ctarwater/open-publisher rake create

Its a simple fix in docs.


Maven-like rolling output when the build happens to be linear

Originally discussed in https://github.com/mvndaemon/mvnd/issues/210.
In situations, when the build happens to be linear, because the dependency graph does not allow any concurrency, noBuffering could be activated automatically. (This is trivially true for single module builds)
@gnodet :

It should be possible to enhance the smart builder to compute the maximum concurrency of a project, so that we could also use it as a maximum for the number of threads to display.

I had work on computing the maximum width of the maven project, see https://github.com/gnodet/mvnd/commit/dadf524e13a8bd79cc245ccf22a2b8b5010e1742, but this can be part of a later enhancement to the automatic maven-like rolling output.


Installing on OpenWrt?

Hi
I wonder if it would be possible to create a OpenWrt package.
Does anyone have ideas or already tried?

See #350

I don't think I want to create an official OpenWrt package for ntfy. I am happy to link to it if somebody else wants to create one. So for now I'm closing the ticket.


503 - Service Unavailable - How to troubleshoot?

Hi,
I need to test the nginx docker containers on my OSX computer before moving to production.  This involves  the docker-machine setup with a virtual machine.
I start the nginx and nginx-dev servers and get the default nginx message for my host name, which I set up on my etc/hosts configuration (on the mac). (http://mydockerhost1)
Then I start my image, which I can reach on the host name reach if I add ports (-p) arguments (http://mydockerhost1:3000 works).  When i console into the nginx-volumes I can see that it has modified the config to something like this, when I started my image:
# mydockerhost1
upstream bdacd495f8d155c80c0f8ddad3353987c0203bf0 {
}
server {
	server_name mydockerhost1;
	listen 80 ;
	access_log /var/log/nginx/access.log vhost;
	location / {
		proxy_pass http://bdacd495f8d155c80c0f8ddad3353987c0203bf0;
	}
}

However the nginx site now displays the 503 Service Temporarily Unavailable error.  I can't find any errors around the place, but does not know where to look?  The logs of nginx and docker-gen looks ok.
What should I do to figure out what's wrong?  Is there issues with running this on the docker-machine on a max?
I start the dockers the following way:
docker run -d -p 80:80 -p 443:443 \
    --name nginx \
    -v /etc/nginx/conf.d  \
    -v /etc/nginx/vhost.d \
    -v /usr/share/nginx/html \
    -v /home/docker/nginx-test/certs:/etc/nginx/certs:ro \
    nginx

docker run -d \
    --name nginx-gen \
    --volumes-from nginx \
    -v /home/docker/nginx-test/nginx.tmpl:/etc/docker-gen/templates/nginx.tmpl:ro \
    -v /var/run/docker.sock:/tmp/docker.sock:ro \
    jwilder/docker-gen \
    -notify-sighup nginx -watch -only-exposed -wait 5s:30s /etc/docker-gen/templates/nginx.tmpl /etc/nginx/conf.d/default.conf

docker run -d \
   --name website \
   -e "VIRTUAL_HOST=mydockerhost1" \
   --expose 3000 \
   tvedtorama/website

This setup is from the letsencrypt-compainon, which I intend to use - but I guess I have to get the basics running first?

Shouldn't there be a VIRTUAL_PORT=3000  in your dockerhost1?

Thank you, I'm not sure.
My node.js application listens on port 3000, and I expose that port in the commands above.  I want nginx to map it to standard web ports, however.  So that the user will find it on the url without any port specifications.
Do I have to map the port like this "-p 80:3000" on my application?  So nginx can find it there?
As I said, I can access the app using domain and port specification, when I do "-p 3000:3000", but that must be the app docker directly, I guess?

a few things to try in order to troubleshoot:
docker logs nginx
docker exec nginx cat /etc/nginx/conf.d/default.conf
docker exec nginx nginx -s reload

ok, so the readme says it should automatically find the only exposed port and map that into nginx. Can you try changing your command like this anyway?
docker run -d --name website -e "VIRTUAL_HOST=mydockerhost1" -e VIRTUAL_PORT=3000 --expose 3000 tvedtorama/website

Thank you both!
Putting the VIRTUAL_PORT argument in there did not seem to do anything.
However, I finally got the basics to run, by mangling the example from the letsencrypt-container and in this site.  I can't tell what the minor differences does, but it seems to work now.
 And you're only mapping the template itself and not the entire directory? Interesting. 🤔

I think these args are the only difference, while I don't know what they do.
I did lots of trial and error yesterdays - so it could be that I have confused something in the process.
Not sure what you mean by the "template" comment?

No, sorry, the other way around, you're mounting now the entire folder, where the nginx.tmpl file is in. You changed
-v /home/docker/nginx-test/nginx.tmpl:/etc/docker-gen/templates/nginx.tmpl:ro
to
-v /home/docker/nginx-test:/etc/docker-gen/templates

Yes, that's right.  I think I tested both ways yesterdays - but this could be the issue.  I think there was some other github issue that concluded that the folder should be mounted, not the file.


Stop the Oak Grove from spawning underwater

Is your feature request related to a problem? Please describe.
I keep seeing the oak trees from Occultism spawning in rivers and oceans, they also keep bisecting other underwater structures. Could you make a blacklist so it does not spawn on ocean and river biomes?
Describe the solution you'd like
Just make a biome blacklist so that the oak grove stop spawning underwater.
Additional context

I've seen so many of these in the ocean...


MAINT: remove dead code in interpolate.py

Those functions were written by Travis and haven't been touched since 2007.

should be god to go now

Thanks Ralf, merged.


Users were little confused by the info analyzer message in the exercise

Few participants were a little confused when they saw an info message before and/or after they passed the test. They were unsure if they were supposed to fix it in the exercise.

As a short term fix in the Dart Future codelab https://github.com/dart-lang/site-www/pull/1659, @legalcodes added the note below.

To improve exercises in Dartpad, we may want to add an tooltip to info message or add a note if users pass tests with info messages.
Feedback welcome!

Status of this issue is also tracked in https://github.com/orgs/flutter/projects/3.

As part of the new, simplified version of the DartPad frontend, the exercise functionality this was related to was removed.
If anyone would like to see similar functionality brought back, please check out https://github.com/dart-lang/dart-pad/issues/2702 or open a new issue. Thanks so much!


Deploy Otomi with k8s cluster at Scaleway

Description
Acceptance criteria
AS A developer
I WANT TO deploy Otomi on Scaleway cluster
SO I company can save 1000 euro/month by using free credits
Acceptance criteria
GIVEN Scaleway credentials are already set as secrets in Github repo
WHEN I go to Github Action at otomi-core repo AND I select Deploy Otomi pipeline AND click Run workflow
THEN k8s cluster is created at Scaleway provider in Dev project
GIVEN I am selecting just major and minor (X.Y) version of Kubernetes
WHEN version is selected
THEN underlying sac command choose supported Kubernetes version (X.Y.Z)
GIVEN Deploy Otomi pipeline is running
WHEN cluster_persistence is set to destroy
THEN k8s cluster must be always destroyed at the end of pipeline run
Definition of done

[ ] Relevant PRs are merged
[ ] Tested by peer
[ ] Updated documentation reviewed by peer

@ElderMatt Some preliminary work (PoC) has been done here: https://github.com/redkubes/otomi-core/tree/jeho/feat/int-test, so we know that integration between Scaleway and Github works fine


[#1529] feat(spark-connector): support table operation

What changes were proposed in this pull request?

support basic table DDL and DML operation.  (alter table are not supported)
add basic table operation to GravitinoCatalog
add GravitinoBaseTable class which get schema from Gravitino and do IO with internal spark catalog table.

Why are the changes needed?
Fix: #1529
Does this PR introduce any user-facing change?
no
How was this patch tested?
add spark SQL IT

@jerryshao @qqqttt123 @yuqi1129 @diqiu50 ,  it's ready to review now, please help to review when you are free.

We could get ResolvedTable from the dataset. ResolvedTable contains SparkBaseTable returned by GravitinoCatalog with following interfaces. If we test with SparkBaseTable, we could get a structured table metadata but not 100% end2end, such as:  comment, location is hidden in properties when testing table properties,  we could only check an expected transform object when testing partition. do you think it's a better way than parsing spark SQL result? @jerryshao @qqqttt123
public interface Table {

  /**
   * A name to identify this table. Implementations should provide a meaningful name, like the
   * database and table name from catalog, or the location of files for this table.
   */
  String name();

  /**
   * Returns the schema of this table. If the table is not readable and doesn't have a schema, an
   * empty schema can be returned here.
   * <p>
   * This is deprecated. Please override {@link #columns} instead.
   */
  @Deprecated
  StructType schema();

  /**
   * Returns the columns of this table. If the table is not readable and doesn't have a schema, an
   * empty array can be returned here.
   */
  default Column[] columns() {
    return CatalogV2Util.structTypeToV2Columns(schema());
  }

  /**
   * Returns the physical partitioning of this table.
   */
  default Transform[] partitioning() {
    return new Transform[0];
  }

  /**
   * Returns the string map of table properties.
   */
  default Map<String, String> properties() {
    return Collections.emptyMap();
  }

  /**
   * Returns the set of capabilities for this table.
   */
  Set<TableCapability> capabilities();
}

@FANNG1 can you please fix the conflicts.

The package should be consistent with the trino.
com.datastrato.gravitino.spark.connector

The package should be consistent with the trino. com.datastrato.gravitino.spark.connector

ok

The package should be consistent with the trino. com.datastrato.gravitino.spark.connector

I will propose a separate PR to fix this, see #2442


python312Packages.netutils: 1.9.0 -> 1.9.1

Diff: https://github.com/networktocode/netutils/compare/refs/tags/v1.9.0...v1.9.1
Changelog: https://github.com/networktocode/netutils/releases/tag/v1.9.1
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 332571 run on x86_64-linux 1

15 packages built:

napalm
napalm.dist
peering-manager
python311Packages.napalm
python311Packages.napalm-hp-procurve
python311Packages.napalm-hp-procurve.dist
python311Packages.napalm.dist
python311Packages.netutils
python311Packages.netutils.dist
python312Packages.napalm
python312Packages.napalm-hp-procurve
python312Packages.napalm-hp-procurve.dist
python312Packages.napalm.dist
python312Packages.netutils
python312Packages.netutils.dist


[SPARK-33509][SQL] List partition by names from a V2 table which supports partition management

What changes were proposed in this pull request?

Add new method listPartitionByNames to the SupportsPartitionManagement interface. It allows to list partitions by partition names and their values.
Implement new method in InMemoryPartitionTable which is used in DSv2 tests.

Why are the changes needed?
Currently, the SupportsPartitionManagement interface exposes only listPartitionIdentifiers which allows to list partitions by partition values. And it requires to specify all values for partition schema fields in the prefix. This restriction does not allow to list partitions by some of partition names (not all of them).
For example, the table tableA is partitioned by two column year and month
CREATE TABLE tableA (price int, year int, month int)
USING _
partitioned by (year, month)

and has the following partitions:
PARTITION(year = 2015, month = 1)
PARTITION(year = 2015, month = 2)
PARTITION(year = 2016, month = 2)
PARTITION(year = 2016, month = 3)

If we want to list all partitions with month = 2, we have to specify year for listPartitionIdentifiers() which not always possible as we don't know all year values in advance. New method listPartitionByNames() allows to specify partition values only for month, and get two partitions:
PARTITION(year = 2015, month = 2)
PARTITION(year = 2016, month = 2)

Does this PR introduce any user-facing change?
No
How was this patch tested?
By running the affected test suite SupportsPartitionManagementSuite.

Test build #131472 has started for PR 30452 at commit 3f20ee8.

@cloud-fan Could you take a look at this PR, please.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36079/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/36079/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36079/
Test PASSed.

Test build #131472 has finished for PR 30452 at commit 3f20ee8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131472/
Test PASSed.

The partition can be a transform like year(ts_col), shall we just partition index in the API instead?

As the changes are related to listPartitionIdentifiers() added by https://github.com/apache/spark/pull/28617. @stczwd @rdblue @RussellSpitzer @emkornfield @dongjoon-hyun May I ask you to review this PR.

... shall we just partition index in the API instead?

@cloud-fan Sure. I see at least two variants of the implementation:

Pass an array of indexes, or
a BitSet

The second one could save some memory, I guess.

Since SupportsPartitionManagement already have the API partitionSchema, which means that the implementations will pick a name for partition transforms, I think it's OK to use String[] in the listPartitionIdentifiers API parameter.

@cloud-fan @HyukjinKwon Can we continue with this PR or do you have some objections?

I'm merging it to unblock the following work. @stczwd  @rdblue please leave comments if you have any concerns, so that we can address them.

merging to master, thanks!

Since SupportsPartitionManagement already have the API partitionSchema, which means that the implementations will pick a name for partition transforms, I think it's OK to use String[] in the listPartitionIdentifiers API parameter.

Yeah, I prefer extend the listPartitionIdentifiers instead of add new API listPartitionsByNames.

The partition can be a transform like year(ts_col), shall we just partition index in the API instead?

If I remember correctly, there should be a schema exposed by the table that describes these. We should get the name from that schema.

I removed listPartitionIdentifiers() and renamed listPartitionsByNames() to listPartitionIdentifiers(): https://github.com/apache/spark/pull/30514


test and Trigger is not working

Hello
when I test this and I try the
HTTP Request body to include the following parameters:
{
'permission': '<Signed permission for shared access signature (Required)>',
'container': '<Container name to access (Required)>',
'blobname': '<Blob object name to access (Optional)>'
'ttl': '<Token time to live period in hours. 1hour by default (Optional)>'
}
I get the following errors
Invalid blob path specified : '{"body":" {'permission': "rwl",\r\n 'container': "sample-workitems\logomalek.jpg",\r\n 'ttl': 2\r\n }"}'. Blob identifiers must be in the format 'container/blob'.
please can you let us know what should we do? i am testing with python 3.7 - Azure function v2

@mkhazen
if you want to specify blob file name, you need send a request body like this:
{"body":" {'permission': "rwl",
 'container': "sample-workitems",  'blobname': "logomalek.jpg",
 'ttl': 2
 }"}

You need to specify container name and blob file name with container and blobname field respectively
Please try again and let me know how it will work

@mkhazen I'll close the issue but please feel free to open an issue if you have any other issue


Custom keywords?

Does this library give any mechanism or plugin API to specify custom keywords, like AJV for javascript? https://ajv.js.org/custom.html

Hi, yes, please have a look at the docs for extend.


Config options seemingly not being populated

Hello and thanks very much for your time and effort in this project!
I've got a bit of a stumper: I have the following config.json in .cognito as directed by the README:
{
  "LambdaClient": {
    "credentials": {
      "accessKeyId": "local",
      "secretAccessKey": "local"
    },
    "region": "local"
  },
  "TokenConfig": {
    "IssuerDomain": "http://localhost:9229"
  },
  "TriggerFunctions": {},
  "UserPoolDefaults": {
    "Id": "us-east-1_xxxxxxxxx",
    "UsernameAttributes": ["email"],
    "MfaConfiguration": "ON"
  }
}

when I load up cognito-local, however, the user pool config that gets created is named 'undefined.json'. MFA is also not enabled during auth, even though I've edited the user pool options to this:
  "Options": {
    "MFAOptions": [
      "SMS"
    ],
    "UsernameAttributes": [
      "email",
      "email"
    ]
  }

Any insights you have would be helpful. I'm on Windows 10, Node 10.18.1

Hey @Carrington Were you able solve it? can please share a working .cognito/db/$userPoolId.json idpool config

@sedhuait I'm afraid not. I don't think this repo is useable as a local emulator for a real cognito setup. Sorry to disappoint.

@Carrington Sorry to hear you're having trouble. cognito-local does work, but as the readme says it's for a limited set of Cognito features that I'm personally using, anything outside of that is uncharted territory.
What are you doing with the pool once cognito-local has started? What API calls are you making against the pool?

Yes its working for me.. I just forked it to make it work for my use cases with my own docker image.

No response, closing. Please re-open if needed.

https://github.com/sedhuait/cognito-local/commit/0d603d7e12bcb6c84af64be7aece9956da2b1369#diff-971637d96d35f6340524535e2fc62c59186e6bc1c3f1d28361f60ef9f9ed3833
@jagregory Thanks a lot for ur work. I just added to support few custom parameters


no support needs called out

The csi driver repo states no support from Microsoft. This needs called out because this page is a tutorial on how to deploy this, but doesn't state anywhere about no support. It could impact SLA's required by a customer.
https://github.com/Azure/secrets-store-csi-driver-provider-azure#support

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

ID: eee999c1-32dd-d014-40df-8027adee1258
Version Independent ID: 19a29c09-79f8-580a-841f-873e2801da56
Content: Integrate Azure Key Vault with Kubernetes
Content Source: articles/key-vault/general/key-vault-integrate-kubernetes.md
Service: key-vault
GitHub Login: @taytran0
Microsoft Alias: t-trtr

@jrudley
Thanks for your feedback! We will investigate and update as appropriate.

Thanks! I made a pull request to add the support note to the document https://github.com/MicrosoftDocs/azure-docs-pr/pull/126101


Rewrite Snackbar without componentWillReceiveProps

It won't be supported in future react versions. See if there are any sideeffects.

Done in #127


Deletion of Horreum Label returns not found error.

Describe the bug
Some Labels won't be deleted.
Removal of the label and then saving changes shows this error
Failed to update one or more labels.
Label 221 not found

Checking the exported Schema there is no Label with the id 221.
The Schema id is 221.
To Reproduce
tbc
Version
0.12.2

Java
17

the request to delete the Label is this http request
DELETE
https://acme.com/api/schema/4061/labels/221

On Hold due to https://github.com/Hyperfoil/Horreum/discussions/1603


Remove specific NumPy pin

Remove the not v1.15.2 NumPy pin from the Iris requirements, as the problem with this build has apparently been fixed.

@QuLogic I'm going to remove this from the v2.3.0 milestone - we can benefit from this now; no need to wait for v2.3 to do so.

Oh, I didn't realize there's no v2.2.x branch yet; shouldn't there be? The 2.2.0 release has been made already.

Oh, I didn't realize there's no v2.2.x branch yet; shouldn't there be?

This seems like an oversight, normally there should be a v2.2.x branch for maintenance fixes. @dkillick what's the plan here, are we doing something different this time?

This seems like an oversight

Yup - it's exactly that. Will correct soon! Apols.

Right, the v2.2.x branch exists. Thanks for pointing out that it was missing @QuLogic @ajdawson.


Feat: update spoon core

For this PR
1 test fail. Any tips ? :)

Thanks for the great help, Martin :) .

updated the assertions.
@khaes-kth note that this will impact a little bit your current paper.
@martinezmatias merge?

Merged. Thanks!


WinGet Settings export and import user settings

Description of the new feature / enhancement
Provide a way to export the existing user settings so that it can be imported into another device or VM.
Proposed technical implementation details
Winget Settings Export --path C:\WinGet\Settings.json
Winget Settings Import --path C:\WinGet\Settings.json
GPO: Import Winget Client settings

The export functionality is planned via PowerShell DSCv3.
After:

https://github.com/microsoft/winget-cli/issues/3676

We will work on implementing "export" in the DSC resources for WinGet.
The configuration is available today via the Microsoft.WinGet.DSC PowerShell module for both user settings and administrator settings.


Verification key for "armory-drive-log" not easily discoverable

The public key string for armory-drive-log should be discoverable so that checkpoints can be verified. Ideally this would be hosted on an external site with a link from here, though for now just popping it in a README would work.
Long term it would be best not to include it in the README as an attacker inside github could presumably then change both the checkpoint and the verification key.

The log public key is stored in the keys directory.
The checkpoint signatures are performed by the push_to_master action which has access to the log private key in its secrets envirionment. A github attacker wouldn't just use such private key instead of changing the public key?

Wow, I totally didn't find those despite the obvious name of the directory. Mea culpa - thanks! You're also right about the threat model here. I'm not having a smart morning :-)

The installer also highlights where keys are taken from:
$ ./armory-drive-install
...
Downloaded verified release assets
Downloading manifest authentication key from f-secure-foundry/armory-drive-log/keys/armory-drive.pub
Downloading transparency log authentication key from f-secure-foundry/armory-drive-log/keys/armory-drive-log.pub
...


Fails to build on arm

As mentioned in the docs for std::sync::atomic, AtomicI64 isn't portable because not all systems have 64-bit atomics.  At least 32-bit ARM, PowerPC, and MIPS do not, and I'd like to both use Flurry and support those architectures (my supported architectures are those of Debian where Rust is available).
Is it possible that the AtomicI64 could be an AtomicIsize here to better accommodate 32-bit platforms?

So, maybe. The problem with doing that is that a 32-bit counter here could much more reasonably overflow. I would potentially recommend falling back to a RwLock<HashMap> on those platforms instead.


support network_mirror configuration of .terraformrc

I implemented a rather rudimentary mirror script to get versions of used providers and their signatures and store them in /providers directory of the boring registry. Sadly this doesn't match the expectations of terraform. (the needed files are present)
The key difference is the <namespace>/<provider>, which gets extended by terraform by registry.terraform.io
.terraformrc  is
provider_installation {
  network_mirror {
    url = "https://terraform-registry.<client>/"
    include = ["*/*"]
  }
}

Error: Failed to query available provider packages

│ Could not retrieve the list of available versions for provider hashicorp/random: provider registry.terraform.io/hashicorp/random was not found in any of the search locations
│
│   - provider mirror at https://terraform-registry.<client>/

❯ curl -H "Authorization: Bearer xxx" https://terraform-registry.<client>/v1/providers/registry.terraform.io/hashicorp/aws/versions
404 page not found
❯ curl -H "Authorization: Bearer xxx" https://terraform-registry.<client>/v1/providers/hashicorp/aws/versions
{"versions":[{"version":"5.15.0","platforms":[{"os":"darwin","arch":"arm64"},{"os":"linux","arch":"amd64"}]}]}

see: https://github.com/TierMobility/boring-registry/issues/21
filesystem layout /providers
.
├── hashicorp
│   ├── archive
│   ├── aws
│   ├── cloudinit
│   ├── helm
│   ├── kubernetes
│   ├── null
│   ├── random
│   ├── time
│   └── tls
└── registry.terraform.io
    └── hashicorp
        ├── archive
        ├── aws
        ├── cloudinit
        ├── helm
        ├── kubernetes
        ├── null
        ├── random
        ├── time
        └── tls

Using the beta build from PR#143 I can confirm the feature is working. Waiting for the 0.12.0 release to close the issue

Closing this issue as I released v0.12.0 yesterday :)


refactor Devise.secure_compare

method becomes a bit clearer

I do not necessarily agree it is clearer and I would prefer to avoid "style" changes on code that can be sensitive in terms of security. Thank you though!


WIP: Enable race checker in CI and fix some race conditions

Fixes: #117

Wait for controller manager to exit before exiting the test (prevents calls to t.Log after the test has finished)
Add Lock around the setting and reading from the fake Etcd client (prevents the controller using a MembershipAPI which has been replaced in the test)
Return a copy of the members list (prevents controller using a members list which might be mutated during the test)

The controller-runtime manager.Start doesn't wait for all its go-routines to finish,
which leads to another race when the controller runtime attempts to log (via t.Log) after a test has finished.
See:

https://github.com/uber-go/zap/issues/687#issuecomment-473382859
https://github.com/kubernetes-sigs/controller-runtime/issues/350

Fixes #117

You can see the leaked goroutine in the test results;
Goroutine 270 (running) created at:
  sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start()
      /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.3.0/pkg/internal/controller/controller.go:157 +0x4ee
  sigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).startLeaderElectionRunnables.func1()
      /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.3.0/pkg/manager/internal.go:326 +0x74

--- https://app.circleci.com/jobs/github/improbable-eng/etcd-cluster-operator/668/parallel-runs/0/steps/0-102

I've removed the -race parameter now.
We will have to enable the race checks once the bug has been fixed in controller-runtime.
Meanwhile this fixes three of the race conditions.


HAWQ-1358. Refactor gpfdist library in featuretest.

extract gpfdist code(which exists in exttable test case) as a common library, to be used by both exttable and ranger feature test.
Main functions:
gpfdist_init: to setup gpfdist environment and start server.
gpfdist_finalize: to stop gpfdist server.

+1


Chore: workflow for running levitate on gcom plugins

Closing this in favour of https://github.com/grafana/levitate-grafana-ci/pull/1.
(After some thinking we have decided to move these workflows in a separate PR as they are more Grafana-related and we would like to keep this tool generic).


DockerHub handle images without a latest tag

Description
When importing an image from the DockerHub marketplace, if there is no tag specified, the driver downloader will assume the default tag is latest. This can lead to issues with the import process if the image has no such tag, like what happens with RockyLinux

Either handle the default tag smartly or prompt the user with an error on both CLI and Sunstone when the specified tag and/or the default one are missing on the marketplace.
Use case
Simplify export process and do not trigger sorta complicated processes if they are going to fail anyway.
Interface Changes

On Sunstone there could be a pop up if the default latest tag is missing on the list of detected tags
On the CLI have the missing tag error to be sent to STDERR

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

Fixed by https://github.com/OpenNebula/one/commit/dbf1e882b14d0b0b49b0ad41a2987ab628047043


retryOptions.retryableErrorFn called with 200 ok for resumable upload

Environment details

OS: node14.18.2-alpine3.12 (docker)
Node.js version: 14.18.2
npm version: 6.14.15
@google-cloud/storage version: 5.18.3

Steps to reproduce
retryOptions.retryableErrorFn is often called with a plain object containing code: 200.
const storage = new Storage({
  retryOptions: {
    autoRetry: true,
    retryDelayMultiplier: 2,
    totalTimeout: 600,
    maxRetryDelay: 60,
    maxRetries: 10,
    idempotencyStrategy: IdempotencyStrategy.RetryAlways,
    retryableErrorFn: (err) => {
      console.log(err)

      // !!
      // err is often a plain object:
      // { code: 200, message: 'OK', name: 'OK' }

      return true
    },
  },
})
const cloudFile = storage.bucket('our-bucket').file('somename.zip')
const result = await cloudFile.save(buffer, {
  // `resumable: true` seems to be the key to triggering this.
  // We only set `resumable: true` with files > 10mb.
  resumable: true,

  gzip: true,
  metadata: {
    cacheControl: 'no-cache',
  },
  predefinedAcl: 'projectPrivate',
})

If we ignore the 200s in retryableErrorFn, cloudFile.save ultimately succeeds, but it took a while for us to discover we could safely ignore them as we could only reproduce this in production. retryableErrorFn is called in the upload stream's error event, obviously extremely confusing that it's called with a code 200.
If we retry the 200s, it retries until it eventually fails with:

The offset is lower than the number of bytes written. The server has 0 bytes and while XXXX bytes has been uploaded - thus XXXX bytes are missing. Stopping as this could result in data loss. Initiate a new upload to continue.

Hi @benogle ,
This is happening here https://github.com/googleapis/nodejs-storage/blob/72f7992ee1b85f2dcb3060e7464dc2d7086cb3e1/src/gcs-resumable-upload.ts#L1014-L1031
We check all responses and if the retry function returns true, we retry the request. I've confirmed that for the default retry function, the 200 response is not retried. The custom function you've passed in here returns true for every type of response, so it will keep attempting to retry the request. Since 200 is a success response, it should never be retried.

Since 200 is a success response, it should never be retried.

Right, this is the issue. It feels like the retry function shouldn't be called at all for success responses, and it isn't when resumable: false, which added to the confusion when we started seeing these in the logs. Everything around this indicates it will be called on errors only, the function name, function arg type, etc.

Is there a reason you're passing in a custom function instead of using the default?

Basically to track when it is retrying, what errors it's retrying, and to ensure it did actually retry if the upload results in an error.
For more context, Google storage has been pretty unreliable on upload for us lately. We've been getting a number of 502s and 408s that come out as promise rejections / errors. We had the default retry configuration, but it was unclear if the lib was actually retrying the uploads. After digging into the code, it looks like you possibly need to specify a generation-related param or two to get the lib to actually retry with the default IdempotencyStrategy.RetryConditional. We weren't totally clear how that worked, and it's tough to repeatably test this against a real failure, so we set the strategy to RetryAlways and added the function to log retries for traceability.

I understand that this can be confusing. Thank you for calling it out and explaining a little bit. I've opened a PR that will prevent an accidental retry of a 200 response.
As for IdempotencyStrategy, the default behavior prevents data corruption by only retrying operations that are idempotent. An upload without a generation number is not idempotent. Retrying this operation could result in data corruption. For more information on preconditions, see the request preconditions documentation page. For more information on which operations are idempotent, see the retry strategy documentation page.


Silex error handler and Symfony\Component\Debug\ErrorHandler misleading param description

Hi all,
In one of my productions env. I run multiple front-end servers and use the php ssl2 lib. to move stuff around. I saw some errors didn't end up in my (central database) error log and start looking what was going on.
My Silex code (when stripped from all not relevant stuff) is something like this:
<?php
use Symfony\Component\Debug\ErrorHandler;
use Symfony\Component\Debug\ExceptionHandler;

require_once __DIR__.'/../vendor/autoload.php';

$app = new Silex\Application();
//$app['debug'] = true;

ErrorHandler::register(null, $app['debug']);

$app->error(function (\Exception $e, $code) {
	echo 'Write exception to database: ' . $e->getMessage();
	// serve some error page to the user
});

$app->get('/image/{id}', function () use ($app) {

	// if not on this server, fetch image from other server
	$connection = ssh2_connect('x.x.x.x');
	ssh2_auth_password($connection, 'lemming', 'xxxxxx');
	ssh2_exec($connection, '/usr/local/bin/php -i'); // some scp command
	return 'Image page';
});

$app->run();

In short, the Silex error function is not called for an  (fatal) error if the Errorhandler  is registered without the $displayErrors argument set to true. However the description states:
Display errors (for dev environment) or just log them (production usage).
I would like to log the errors, should I change to some other hook then $app->error() to log the error using a custom logger?

Fatal errors can be handled by the ExceptionHandler class -- see http://silex.sensiolabs.org/doc/cookbook/error_handler.html

:+1:   thanks, sorry for keeping this open so long ..

thanks! sorry for keeping this one open for so long, it kind of slipped my mind


Look into interop with Memo

Memo is a note taking app that persists its data to gist. We should also investigate whether there’s anything needed to ensure GistPad can interoperate with it.

Closing this since no one has mentioned wanting this interop support. We can always re-open it later if/as needed.


'Adjustment' mapping for XRBodyTracker::JOINT__SHOULDER appears borked?

Was having some trouble getting good results on a humanoid rig, so stripped things back today and hacked together a scene which just spits out marker meshes for all the (75) joints found in XRServer.get_trackers(32).get("/user/body_tracker"). It showed JOINT_RIGHT_SHOULDER and JOINT_LEFT_SHOULDER behaving very weirdly... as if their order of rotation-translation was messed up or something?
Digging into the specs* and openxr_fb_body_tracking_extension_wrapper.cpp, I see we don't map the scapula - which makes sense enough - but then ALSO that there's an adjustment made to those very same two joints: JOINT__SHOULDER? Suspicious. Then came across a magic number (-0.07) and couldn't get my head around how the adjustment is actually meant to work beyond that, I'm afraid :-/
I'll be first to admit... issue could easily be on Meta's side, my headset, even my big bushy beard getting in the way! Or just my imagination. However, don't seem to have any tracking issues of the ilk on any other platforms, and all signs point to this as the biggest anomaly on the rigging I was attempting.
Are we confident the 'adjustment' of shoulder_offset, along with the mapped starting quaternions, are working as intended?
*https://developers.meta.com/horizon/documentation/native/android/move-ref-body-joints
https://registry.khronos.org/OpenXR/specs/1.1/html/xrspec.html#_conventions_of_body_joints

Yep. Definitely something off. It appears we can force a shoulders worse case scenario by tracking a character / avatar at 45deg to ref. IE, shoulder placement may seem passable when avatar basis matches XROrigin / World, as the green capsules indicate here.

But reference the world 45deg 'clockwise', and you find both shoulders appearing on the same side of the torso... whereas all the non-adjusted trackers - where the vendor plugin simply maps them 1:1 - are where they should be!

It's a tricky little sneaky chestnut, for sure.

Hopefully this vid now makes sense?
https://youtu.be/JLssQrFttUo


fix config for URL

MDN for URL
caniuse for URL.createObjectURL()
Are screenshots needed for this? I can test for everything except safari.

Also MDN for searchParams as iterator.

Looks good to me!


Shell substitution

Hi,
ssh-pool uses child_process.exec for running an ssh command, and I think it does not handle all character substitution correctly, e.g:
const sshPool = require('ssh-pool');
const pool = new sshPool.ConnectionPool(['localhost']);
pool.run('X=1; echo $X').then(res => {
  console.log(res[0].stdout);
});
```

will print a blank line, while `1` would be expected. This is because $X is substituted from the local shell. Also on Windows shell is executed with `cmd.exe` where substitution rules are different (e.g. commands containing new lines do not work).

Replacing `exec` with `execFile` (which accepts an array of arguments) in `connection.js` seems to work fine on Linux and Windows, but I am not sure if it does not have any unforeseen consequences.

A preliminary pull request #17 contains the modifications, but apparently many tests would need to be updated too.

Came across this issue as well. Is there a workaround? I'd like to read the current user uid and pass it along to another command. The only way I can think of is executing multiple commands and remembering the value within the script context. That, however, seems cumbersome. Concerning your pull request, a more conservative approach perhaps would be to add a separate method.


Distributed Training is failing

I was trying to launch a training with DDP and got this error:
TypeError: ensure_nccl(): incompatible function arguments. The following argument types are supported:
    1. (arg0: torch.distributed.distributed_c10d.ProcessGroupNCCL, arg1: torch.Tensor) -> None

Invoked with: <torch.distributed.distributed_c10d.ProcessGroup object at 0x7fa630e71cb0>, tensor([[ 0.8793,  1.1869, -1.2402,  ...,  0.7247, -0.2372, -0.3371],

This is how I setup the training, starting from the distributed process (I use mpirun):
rank = int(os.environ["OMPI_COMM_WORLD_RANK"])
local_rank = int(os.environ["OMPI_COMM_WORLD_LOCAL_RANK"])
world_size = int(os.environ["OMPI_COMM_WORLD_SIZE"])

dist.init_distributed(dist_backend="nccl", rank=rank, world_size=world_size)

This is how I wrapped the model:
model.to(device)
ddp_model = DistributedGroupedDataParallel(model)

And the MoE layer is defined like this:
self.mlp = FMoETransformerMLP(
            num_expert=2,
            d_model=dim,
            d_hidden=int(dim * mlp_ratio),
            activation=act_layer,
            expert_rank=os.environ.get("OMPI_COMM_WORLD_RANK", 0),
            world_size=2,
            top_k=2,
            gate=gate,
            expert_dp_comm="world"
        )

And this is how I update the parameters:
loss = loss / num_grad_acc
scaler.scale(loss).backward()
model.allreduce_gradients()

if (i + 1) % num_grad_acc == 0:
    scaler.unscale_(optimizer)
    clip_grad_norm_(model.parameters(), grad_clip)
    scaler.step(optimizer)
    scaler.update()
    scheduler.step()
    optimizer.zero_grad()

Ok thank you! So the rest of the code is defined correctly?

Yes. I suppose this is a separate issue of ensure_nccl. You can try if simply calling it produces an error. (It takes a torch comm group and a tensor on CUDA as input)

As mentioned here: https://discuss.pytorch.org/t/processgroup-not-converted-to-processgroupnccl/180529/3
Maybe we can just make this small change and then it should work, what do you think?

No. We are using a protected member function of ProcessGroupNCCL, i.e. broadcastUniqueNCCLID, so we cannot do that conversion directly. I am afraid that we need to figure out another way to get the instance of ProcessGroupNCCL.
I am sorry but I am dealing with some really tough staff this week. I will look into this problem later. Maybe you can check the ProcessGroup code of newer PyTorch and see if there is any solution if you are interested.

Ok sure no problem, I'll try! Good luck

Should be addressed by #157 . Feel free to re-open this thread if there is any further issue.

Thank you very much! I'll try it as soon as I can.

Should be addressed by #157 . Feel free to re-open this thread if there is any further issue.

I tried the code with DistributedGroupedDataParallel and it works perfectly fine now but it is extremely slow, I wonder if this is normal or there is a problem with my code.
I paste here for context the model, loaders and update step definition:
# build model
train_sampler = DistributedSampler(dataset=train_ds)
train_dl = DataLoader(dataset=train_ds,
                      batch_size=batch_size,
                      sampler=train_sampler,
                      num_workers=cfg.train.data.num_workers
                      )

# build model
model = build_model(...)
model.to(device)
ddp_model = DistributedGroupedDataParallel(model)

# update step (using AMP)
loss = loss / num_grad_acc
scaler.scale(loss).backward()
model.allreduce_gradients()

if (i + 1) % num_grad_acc == 0:
    scaler.step(optimizer)
    scaler.update()
    scheduler.step()
    optimizer.zero_grad()


Option for Expo please (no spinkit)

Hi there,
Spinkit is used just for the spin animation of the arrow is this true? I was hoping to use this in Expo, so we can't install custom native modules. Is it possible to request a quick and dirty expo option?
And then for long term i can help you rewrite the animations in reanimated, expo has v1. And will have v2 soon.

Hello,
Sure I will make it a expo version today :) I will inform you ASAP

Thank you brother that is way too kind of you! I will also help asap to use reanimated for whatever you were using spinkit for! If you can guide me there when you have time afterwords I will hook it up. Maybe i can pr to allow people to use custom spinners too!

@Noitidart I'm using this library in Expo without any errors.

@Noitidart I'm using this library in Expo without any errors.

Oh thank you Mario!

What kind of error do you get?

I get this error, screenshot below. Thanks for asking!
I think the spinner is only shown while data is an empty array.
Here is a snack - https://snack.expo.io/@noitidart/react-native-single-select

Well you just need to install the spinkit package 😊 then it will work.

Oh I didnt realize we can install it but not link it, and it will work. Thanks! I will try.

Perfekt - so the issue can be closed @WrathChaos


Azure cloud shell is pointing to Old helm version. It should be updated to latest version 3.9.3.

execute the command in azure cloud shell
"rpm -qa"
It says "helm-3.4.1-5.cm2.x86_64".
This is very very old release of helm.
We are already in helm 3.9.3 as per release https://github.com/helm/helm/releases/tag/v3.9.3
Due to this, helm push command is not available in cloud shell.

The extensive details are mentioned here:-  https://github.com/Azure/CloudShell/issues/236
Expected result:-

I see that @chalamalasetty has just updated helm in #3586. I'm guessing you synced with him offline but I'm mentioning it here just in case. The new version should be available at the latest with our next release in 2-3 weeks.

Closing issue as per PR:

#3586

Please re-open if there are any further clarifications/questions :)


[release-1.14] conditionally set multi-arch selector in the gateway templates

This is an automated cherry-pick of #40068

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

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio_release-1.14
c3119539e2eeb251df6171f909d9fbb70c1d4009
link
true
/test integ-distroless_istio_release-1.14

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

/retest


HamburgerMenu - Visibility issue

My setup initial Window.Current.Content = Views.LoginPage(nav) where nav == INavigationService (even before the release of 1.1.1), this worked as desired. Which was to prevent from showing the Hamburger menu (split content) until the user was authenticated.
Now when I show the window, work thru the login process, it does not sure the HamburgerMenu at all, it will show the Location when you resize down to where you would expect it at a really small screen size.  The location is just a 48x48 box nothing more with background its as if the control doesn't want to show up or never existed.
The login page also facilitated account creation and a potential for password reset flows.
Would there be a better way...  btw figured out the validation after finally paying attention to the DataContext :(

How can I repro this to test?

Here ya go... This is using current source (not nuget) since I needed Services and Validation.  This http://1drv.ms/1OTOBsv  is still work in progress.  One thing I did notice was a flash of the HamburgerMenu and it disappearing.  For the UserName as long as it starts with 70 and 5 more digits following you will satisfy the Regex..

I did test out whether or not that the Shell being the starting content would make a different and it does!

@mvermef Have you tried the IsFullScreen property of the HamburgerMenu?

It wasn't present but it shouldn't make any difference related to this situation since shell vs login shouldn't cause a difference whether I was going to shell directly or login. This is the case of login > shell content assignment for current window, hamburgermenu should've worked correctly on Shell being assigned to the current content once login logic was satisfied.

Deeper look via the debugger and WindowWrapper.Current() still shows LoginPage as the main content even after assignment of the Shell() to the current content.  This is actually correct, ironically :(.  Based on logic in the BootStrapper for the Inclusion of "Existing Content" the developer wins, so by starting off with LoginPage and trying to "inject" Shell as the new content, I lose.  Something changed from version 1.0x to 1.1x where it worked previously.

I am only following your problem 90% here. But hey, could you take a look at the Search and Login sample and see if that approach would help you? It's far simpler than swapping content.

I have started to coverted over to this, actually but came across a little confusion with how to handle initial startup and how to handle if user was already logged in.. getting closer every minute I play with the logic...


Utilize pytest fixtures to create reusable test setup code for the BCO API tests, enhancing test organization and efficiency.

https://realpython.com/testing-in-django-part-1-best-practices-and-examples/
https://realpython.com/testing-in-django-part-2-model-mommy-vs-django-testing-fixtures/

@HadleyKing . I have completed reviewing them. Finishing the three tutorials was week 1 task that has been completed. I can take more time and review other django pytest resources/tutorials to gain more understanding or can take up a new task. Whatever you suggest


Replace AsyncBackpressuredStream with updated implementation

Motivation
As a follow up to #27, we noted it would be good to align on the latest draft implementation of SE-0406 (AsyncStream with backpressure) to both pickup the latest improvements in performance and correctness, and to minimise the churn if/when this lands in the standard library or standalone package.
Modifications
In order to simplify reviewing the following modifications have been made in independent commits:

Add updated SE-0406 implementation as BufferedStream, incl. tests: Skim over this—it's vendored in wholesale.
Port the custom watermark support to BufferedStream: Skim over this—it's a 1:1 port of the logic that was added to AsyncBackpressuredStream.
Switch from AsyncBackpressuredStream to BufferedStream in delegate: Review this—it's a minimal change.
Remove AsyncBackpressuredStream and its vendored locks: Skim over this—it's removing the old implementation.

Result
No functional change, but the internal async sequence we're using should be more robust, performant, and more likely to match a future standard library type.
Test Plan

The new vendored BufferedStream actually comes with a much greater number of vendored tests than the previous revision.
Also ported the tests from this repo for the custom watermark logic.
All our URLSessionTransport-specific tests continue to pass.

lgtm. Please once this is released, double check that NIO isn't brought in during package resolution when depending on the transport as a library.

So here's an example package that depends on the source branch for this PR, showing that it doesn't clone NIO. The only new dependency this release will bring in is swift-collections.
❯ tree
.
├── Package.swift
└── Sources
    └── OpenAPIURLSessionAdopter
        └── Example.swift

❯ cat Package.swift
// swift-tools-version: 5.9
import PackageDescription

let package = Package(
    name: "swift-openapi-urlsession-adopter",
    platforms: [.macOS(.v12)],
    dependencies: [
        .package(
            url: "https://github.com/simonjbeaumont/swift-openapi-urlsession",
            branch: "sb/update-async-stream-implementation"
        ),
    ],
    targets: [
        .target(
            name: "OpenAPIURLSessionAdopter",
            dependencies: [
                .product(name: "OpenAPIURLSession", package: "swift-openapi-urlsession"),
            ]
        ),
    ]
)

❯ cat Sources/OpenAPIURLSessionAdopter/Example.swift
import OpenAPIURLSession

func f() {
    _ = OpenAPIURLSession.URLSessionTransport()
}

❯ swift build
Fetching https://github.com/simonjbeaumont/swift-openapi-urlsession from cache
Fetched https://github.com/simonjbeaumont/swift-openapi-urlsession (0.53s)
Fetching https://github.com/apple/swift-collections from cache
Fetching https://github.com/apple/swift-openapi-runtime from cache
Fetching https://github.com/apple/swift-http-types from cache
Fetched https://github.com/apple/swift-http-types (0.52s)
Fetched https://github.com/apple/swift-collections (0.54s)
Fetched https://github.com/apple/swift-openapi-runtime (0.65s)
Computing version for https://github.com/apple/swift-collections
Computed https://github.com/apple/swift-collections at 1.0.5 (0.50s)
Computing version for https://github.com/apple/swift-openapi-runtime
Computed https://github.com/apple/swift-openapi-runtime at 0.3.6 (0.43s)
Computing version for https://github.com/apple/swift-http-types
Computed https://github.com/apple/swift-http-types at 1.0.0 (0.52s)
Creating working copy for https://github.com/simonjbeaumont/swift-openapi-urlsession
Working copy of https://github.com/simonjbeaumont/swift-openapi-urlsession resolved at sb/update-async-stream-implementation
Creating working copy for https://github.com/apple/swift-openapi-runtime
Working copy of https://github.com/apple/swift-openapi-runtime resolved at 0.3.6
Creating working copy for https://github.com/apple/swift-collections
Working copy of https://github.com/apple/swift-collections resolved at 1.0.5
Creating working copy for https://github.com/apple/swift-http-types
Working copy of https://github.com/apple/swift-http-types resolved at 1.0.0
Building for debugging...
[84/84] Compiling OpenAPIURLSessionAdopter Example.swift
Build complete! (17.54s)

❯ cat Package.resolved | jq .pins[].location
"https://github.com/apple/swift-collections"
"https://github.com/apple/swift-http-types"
"https://github.com/apple/swift-openapi-runtime"
"https://github.com/simonjbeaumont/swift-openapi-urlsession"

❯ swift package show-dependencies
.
└── swift-openapi-urlsession<https://github.com/simonjbeaumont/swift-openapi-urlsession@unspecified>
    ├──<EMAIL_ADDRESS>    │   └──<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>

@swift-server-bot test this please


Only build a package if it is an upgrade

Rebuilding every package every time is a bit costly and it causes some issues with checksums. If you reinstall a package after a repo update, you will probably see an error at the moment. This PR is made to address this, but it's not yet ready.
This PR makes it so packages are downloaded from the repo before being build. This makes it so if there is no version change, the package will not be build at all and the existing package will be added to the new release.

This is still a draft, because it requires an update to the docker container used to build.

I'm not going to work more on this for now.


Footer logo height and width not being set when href not present

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[ ] I have tried the npm run clear or yarn clear command.
[ ] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[ ] I have read the console error message carefully (if applicable).

Description
The height and width props are not being set on the FooterLogo here: https://github.com/facebook/docusaurus/blob/main/packages/docusaurus-theme-classic/src/theme/Footer/index.tsx#L182
Steps to reproduce
Can be reproduced with the following configuration:
footer: {
  logo: {
    alt: "Logo",
    src: "img/logo.png",
    height: 50,
    width: 50,
  }
}

Expected behavior
Should use configured height and width even when the footer logo is not a link.
Actual behavior
The height and width props are not being set on the FooterLogo here: https://github.com/facebook/docusaurus/blob/main/packages/docusaurus-theme-classic/src/theme/Footer/index.tsx#L182
Your environment

Public source code:
Public site URL:
Docusaurus version used: 2.0.0-beta.15
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Reproducible demo
No response
Self-service

[X] I'd be willing to fix this bug myself.

Sure! PR welcome 👍


Add BingWallpaper

Microsoft Reviewers: Open in CodeFlow

syntax error, mail to yao

syntax error, mail to yao

The InstallerType in the manifest is incorrect, it’s not an inno installer. Since the manifest does not declare its own Silent and SilentWithProgress switches, the BingWallPaper.exe failed with our default Inno type SilentWithProgress parameter “/silent”.
Fix: Please use exe as InstallerType and add Silent and SilentWithProgress switches with proper values.

syntax error, mail to yao

The InstallerType in the manifest is incorrect, it’s not an inno installer. Since the manifest does not declare its own Silent and SilentWithProgress switches, the BingWallPaper.exe failed with our default Inno type SilentWithProgress parameter “/silent”.
Fix: Please use exe as InstallerType and add Silent and SilentWithProgress switches with proper values.

Strange, the interface looked like inno or installshield, and if it is one of them, it should work.#1161

However, if i use exe as InstallerType add Silent and SilentWithProgress with the value "\Q"(which it tell me), it failed too.

what value should i add?

syntax error, mail to yao
The InstallerType in the manifest is incorrect, it’s not an inno installer. Since the manifest does not declare its own Silent and SilentWithProgress switches, the BingWallPaper.exe failed with our default Inno type SilentWithProgress parameter “/silent”.
Fix: Please use exe as InstallerType and add Silent and SilentWithProgress switches with proper values.
Strange, the interface looked like inno or installshield, and if it is one of them, it should work.#1161
However, if i use exe as InstallerType add Silent and SilentWithProgress with the value "\Q"(which it tell me), it failed too.
what value should i add?

Regarding InstallShield support, unfortunately we don't have full support for automatic handling of InstallShield type yet, as InstallShield can be built as both exe and msi. And we don't support installer config files like for .iss to be used with InstallShield yet. So currently, InstallShield installers need to declare exe or msi type in the manifest respectively.
But regardless of the above, I took a quick look at the BingWallpaper.exe and I could not relate it to inno or Installshield immediately. Here is what I found. The exe manifest says it's IExpress extraction tool. It indicates this exe is a self extract archive. So I renamed the exe to .cab and it can be successfully extracted. In the cab, there's a exe and msi. By examining the extracted installer exe, I did not find specific installer info in exe resources.
Anyway, the BingWallpaper.exe and the underlying installer does not accept default inno or installer switches, so it's appropriate to use exe type for it. The /Q switch is irrelevant here since it's a switch for IExpress archive, not the underlying installer. I tried a couple of commonly used silent switches and passed to the underlying BWInstaller.exe like BingWallpaper.exe /Q /C:"BWInstaller.exe /quiet" but unfortunately none of them does unattended installation.
Currently we require all installers support unattended installation. If the BingWallPaper installer does not support it(or maybe we did not find correct args for it), we'll have to put this installer on hold for now(we can revisit this in the future when we can handle interactive-only packages appropriately).
Yao

Unfortunately this installer with the /Q switch fails.  We are going to need to have the path for supporting non-silent installs.