[KAFKA-5212] Consumer ListOffsets request can starve group heartbeats

Through the identification of the poll method in ConsumerCoordinator as the place where the heartbeat is sent, I modified the metadata so that it polled periodically every interval for a heartbeat.

Retest this please.

@hachikuji This should be ready for review.

@ConcurrencyPractitioner Thanks for the patch. I'm not totally sure this addresses the issue. The problem is here: https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/clients/consumer/internals/Fetcher.java#L456. The Fetcher and the heartbeat thread are sharing the same underlying NetworkClient, so if we are blocking in poll() in Fetcher. retrieveOffsetsByTimes, then we cannot send heartbeats. One way to solve this is to pass the Heartbeat object into Fetcher and use timeToNextHeartbeat to limit the amount of time that we block in poll().

I came across that line during my search, but I thought that allowing Fetcher to have access to {{HeartBeat}} is not allowed, thus I abandoned that approach. However, from your suggestion, that is not the case, so I would go ahead with your proposal.

Yeah, I wouldn't say it's not allowed; "not ideal" is nearer the mark, but I guess that's one of the unfortunate consequences of sharing the NetworkClient.

Retest this please.

Most recently, I've moved towards staggering the poll times in retrieveOffsetsByTimes to allow the heartbeat thread the neccesary intermission to send a heartbeat.

@hachikuji Is this the right approach or is this wrong?

@hachikuji This should be close right?

@hachikuji  Is this the expected result?

@ConcurrencyPractitioner Sorry, been a bit busy. I'll look shortly.

@ConcurrencyPractitioner I was thinking about this problem a bit more and I'm wondering if we can get away with a simpler solution. Basically all we have to do is prevent the fetcher from blocking poll for longer than the heartbeat duration. The simple way to do that is to ensure that the poll timeout is never larger than a heartbeat interval. For example: https://github.com/hachikuji/kafka/commit/bd55d33f5cfdc3dcd164f17f70268d9718d95f6f. What do you think?

@hachikuji Just one thing, do I rebase on your commit or should you take over? Thanks.

@ConcurrencyPractitioner Feel free to rebase. I think we arrived at that solution only after the iterations here, so you should get the credit. We should also consider if there is a test case we can write which proves that it actually works. Maybe we can add a test case in FetcherTest which sends the ListOffsetRequest and delays the response; then in another thread, we can verify that we can still send requests through the network client.

I ran ResetIntegrationTest with patch locally which passed.


add: solidity and foundry support

This PR:

adds support for Solidity (languages.solidity, https://soliditylang.org/) with package option. Default package is pkgs.solc from nixpkgs.
adds support for Foundry (languages.solidity.foundry, https://book.getfoundry.sh/) with package option. Default package is foundry-bin from foundry.nix.

Trying out
If you want to try this out, you will need to change your devenv.yaml:
inputs:
  devenv:
    url: github:gabr1sr/devenv/new/modules/solidity?dir=src/modules
  foundry:
    url: github:shazow/foundry.nix
    inputs:
      nixpkgs:
        follows: nixpkgs

And change your devenv.nix:
{ ... }:
{
  languages.solidity = {
    enable = true;
    foundry.enable = true;
  };
}

Related to shazow/foundry.nix#34

Need to run pre-commit run -a


Video on mobile view

How can we get the  background video to run on mobile devices ?

Since mobile browsers don't support the autoplay feature in any widespread way, this is intentionally the case with this theme. Hopefully this helps explain things!


[NOMERGE] 103 extension group api

Progress:

[X] PG role management for #103.
[X] Group name escaping. @javisantana, @rochoa, suggestion for group role naming? Escaping? Hashing? Restricting input to valid role?
[X] Cache values with SD.
[X] Create org admin role on extension install.
[X] Extension --> Editor updates.

cc @rochoa , @javisantana , @rafatower

@juanignaciosl about the group role naming:
I would not use CDB_Organization_Member_Group_Role_Member_Name becuase it's already returning a long name. What about 'g_%s_%s' % (md5(current_database()), group_name)?
That allows us to control the group_name length and raise error on too long names (AFAIK role names are truncated at 63 chars with just a warning) because g_%s % md5(current_database() has a fixed 34 chars length.

I've just pushed some changes based on your comments. Pending changes:

Role: what do you think about creating an org-owner role for group creation? Or org role is enough for this?
group_name for group role. md5 group name would be too long and cryptic. What do you think about escaping? Or just force a valid string?

about the org role: I don't think so, I think we need an extra role (the admin user will inherit from it) with permissions to call group management functions
what do you mean with "group_name", you mean the role for the group name?

@rambo:

Yep, I'm already working on admin role.
With group_name I'm speaking about a suggestion on how to "concatenate" group name info in order to generate the role. Right now cartodb._CDB_Group_GroupRole returns 'g_' || md5(current_database()) || '_' || group_name;, but it won't work if group_name has special characters on it. That's why I'm asking about restricting valid group_names or escaping them inside the extension.

I'd not do any md5, I'd just restrict group_name to the rules that make sense for a group name, like [a-zA-Z0-9-_]

for reference
SQL identifiers and key words must begin with a letter (a-z, but also letters with diacritical marks and non-Latin letters) or an underscore (_). Subsequent characters in an identifier or key word can be letters, underscores, digits (0-9), or dollar signs ($). Note that dollar signs are not allowed in identifiers according to the letter of the SQL standard, so their use might render applications less portable. The SQL standard will not define a key word that contains digits or starts or ends with an underscore, so identifiers of this form are safe against possible conflict with future extensions of the standard.

http://www.postgresql.org/docs/9.2/static/sql-syntax-lexical.html#SQL-SYNTAX-IDENTIFIERS

:+1:

@javisantana : take a look at this limitation, please. Other than that, now org admin is the one who can run group management functions.
@rochoa take a look at the changes at the test function (well, and at all the code if you can :) ), I've changed error reporting because I wanted some warnings about permission granting to be handled like errors.
After taking into account your following suggestions I'll move forward to trigger updates from the extension to the editor. If I get no easy workaround for the limitation I'll probably wait until the end in order to unblock frontend work.
Thank you all!

I've just seen a couple of comments in the mail about roles:
http://www.postgresql.org/docs/9.4/static/extend-extensions.html

Notably, objects that are database-cluster-wide, such as databases, roles, and tablespaces, cannot be extension members since an extension is only known within one database.

That basically means that the extension installation script can do whatever but while other objects are owned by the extension the global ones are not. A DROP CASCADE will not delete the role for instance. Anyway, it hasn't been a big deal so far and don't think it'll be.

@rafatower I think it's not a problem for us because we don't use clusters for user databases where these roles belong.

@javisantana @rochoa I'm sorry from bringing an old discussion back, but now I'm testing with actual databases instead of test ones and I've realised something that might be a issue later. CartoDB user database names are like this: "cartodb_dev_user_6950b745-5524-4d8d-9478-98a8a04d84ba_db". This has 56 characters, thus after appending "g" separator we only have 4 characters left for the group name.
Important note: prefix depends on the environment (cartodb_dev_user_ / cartodb_staging_user_ / cartodb_test_user_ / cartodb_user_). Staging would have no room for identifiers U_U.
Alternatives:
A) Raised the limit by changing the NAMEDATALEN constant in src/include/pg_config_manual.h (see http://www.postgresql.org/docs/9.1/static/sql-syntax-lexical.html). Probably not something we want to do.
B) Apply md5 to the database name, which goes down to 32+3 characters, allowing 27 characters.
C) remove everything that is not a number and prepend db_, 35+3 characters, allowing 25 characters.
D) Any other idea?
Both B and C generates a almost unreadable name, so I'd use B, simpler. We could provide a CDB_ShortDatabaseName() that hides that implementation detail.
What do you think?

We are using B) for other parts so I'd use that for consistency

:+1: to B)

B then, thanks!

I suppose name group needs to be unique, right? Will we perform any search
by name or similar?
If so, maybe it's better using SHA-1 for the hashing which will produce 40
chars and we are still within the limit. It would reduce chances of
collisions. However it would require loading pgcrypto extension, which is
not cool.
On Fri, Aug 14, 2015 at 9:44 AM, Juan Ignacio Sánchez Lara <
<EMAIL_ADDRESS>wrote:

B then, thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb-postgresql/pull/104#issuecomment-131008655
.

--
Luis Bosque
@luisico
www.cartodb.com
Map, analyze and build applications with your data

Forget it. The variable part of the db name, which is the UUID, seems small
enough to make a collision with md5 almost impossible.
Sorry for the noise
On Fri, Aug 14, 2015 at 9:52 AM, Luis Bosque<EMAIL_ADDRESS>wrote:

I suppose name group needs to be unique, right? Will we perform any search
by name or similar?
If so, maybe it's better using SHA-1 for the hashing which will produce 40
chars and we are still within the limit. It would reduce chances of
collisions. However it would require loading pgcrypto extension, which is
not cool.
On Fri, Aug 14, 2015 at 9:44 AM, Juan Ignacio Sánchez Lara <
<EMAIL_ADDRESS>wrote:

B then, thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb-postgresql/pull/104#issuecomment-131008655
.

--
Luis Bosque
@luisico
www.cartodb.com
Map, analyze and build applications with your data

--
Luis Bosque
@luisico
www.cartodb.com
Map, analyze and build applications with your data

@rochoa @javisantana some unfinished work again, but now I'm able to create and drop groups from the extension after configuring the parameter (see scripts-available/CDB_Groups_API.sql). Do another CR when you can, please.

Just pushed another batch of improvements from the CR, still waiting for the final configuration storage format.

Troublesome scenario (just happened to me):

Extension receives CreateGroup and triggers request to the editor.
Server request takes longer than expected and extension http client timeouts it (but server works).
Now the server has the group but database doesn’t, so it can’t be neither created nor dropped, since extension rollbacks on remote request error.

Solution alternatives:
A) create group endpoint returns 409 (Conflict) error code for existing groups. Extension doesn't fail on 409.
B) create group endpoint returns 409 for existing group, and drop has a force option for deleting remote groups without failure.
I'd prefer B if an org admin could drop groups, but since it currently can't because of drop owned limitations (see https://github.com/CartoDB/cartodb-postgresql/pull/104/files#diff-ef5cb2082378f503285f1c707bdb49afR14) the user that gets the error message on create (the org owner) can't fix it with the drop.
Because of that, and since group management is done by org admins, I think there's no name collision or role highjacking risk so A is safe.
@javisantana , @rochoa , opinions?

We have discussed about that a lot, this issue is the same for layers and other stuff.
A question: if the connection is timeout shouldn't the rails part realize ?

@javisantana maybe you've discussed it, but I wasn't aware of those discussions, what conclusion did you embrace? :sweat_smile: Current code doesn't realizes about it: data is changed by the editor and extension doesn't notice. I'll take a look, but I wouldn't trust that's something we can guarantee.

haha, we didn't figure out a solution

Ok :smile: . Since there's an additional problem here (drops permission) I'll do A (if you and @rochoa agree) and we'll see how that goes.

But Rails should be able to know if request is being cancelled/aborted, shouldn't be?
By using A) we agree that we will be consolidating names. I mean:

User tries to create group wadus
Request to Rails timeouts but internally it succeeds.
Extension code fails so group creation fails, but it actually succeeded.
User gets error about failed group creation.
User tries to create group wadus again.
If there is no code until calling the extension group creation, group creation will call again Rails and will get a 409 error, the extension will ignore it and return as group creation went well.

I see a couple of problems:

Last point seems easy to guarantee but I guess we, at some point, will hit some validation in some model or something similar.
What happens if I try to create a group through the SQL API with an existent group name. Will I get a OK message because we are ignoring the 409 although the name was already created?

I will try to investigate how Rails handles aborted/cancelled requests and see if we can rollback on those cases.

First of all, I don't know if Rails can know whether a request has been aborted. I know the opposite is true, since I can reproduce that no exception is currently raised on Rails when Python has already closed the session, but I can't find in the documentation whether we can somehow get an exception when that happens. Unless you or @lbosque tells how can I do it I think we can't trust that request has been successfully sent to the client.
About your concerns:

409 will only be raised if both group name and role name matches, I think that's a good validation. Keep in mind we're retrying, as you requested in a previous CR. I think we can retry 3 times and rollback if no request is 200 or 400, commit otherwise. What do you think?
If name and role name matches (within your organization, of course) SQL API request will get a 409. If role name doesn't match you'll get a 500 because of the unique (organization, group_name) constraint.

Ok. Call for final CR here, @javisantana, @rochoa @rafatower
Notes:

Currently an org admin can't drop groups. See #114.
Conf and http client are cached in SD.
A 409 return code is considered valid to allow retries. I haven't managed to make Rails side realize connection has been closed by a client timeout (either statement timeout or http timeout).
Editor returns 409 if the same data is found. For example, creating an existing group with different database role name would trigger a 500. Branch.

I'll fix the code until I get all yours +1, but this won't be merged after the new cartodbfication is merged and some group UI is working.

:+1: once the config table backup thing is resolved

OK, I misunderstood the 409 handling.
So we are finally going with (param text, conf text) for the table? I can't find that part in this conversation anymore.

@rochoa there's no final decision on this. Alternatives for CDB_Conf.conf:
A) Different entry for each value.
B) Force JSON type for value.
C) Keep text type for value and let clients store whatever they need.
X) other
@javisantana , at your command.

I've removed version changes because of a suggestion by @rafatower on not changing it until release because it's convenient for development.

@rochoa :eyes: 97dd8e572048a8c2d630881d8118d139028e3e4a, please

Ok, after 551e09ff6f632c9c90a55d934104140f529d11e5 cartodb._CDB_Group_API_Request can only be run by a superuser. I've added SECURITY DEFINER to API Sync functions so they can use it. In spite of it, non-org-admin users can't create groups (because they can't create roles) or grant privileges on other users tables (because they don't have permissions on those schemas), so I think configuration table and functions are save now, and nobody can use cartodb._CDB_Group_API_Request to perform http requests.

One more thing U_U.
Previously I was using an existing superadmin controller for http basic authentication. While changing controller to use its own credentials (see CartoDB/cartodb#5124, commit d6d727032224ef7170a256467686f31617285576) I detected some misbehaviour of configuration functions: values were being cached but not cleared. That's because both SD and GD are session-scoped, so caching works but changing a parameter won't, so I've removed configuration caching (commit 43d41e5c26725ef4023544910bc1a696afc13c96).
Already discussed with @javisantana . cc @rochoa @rafatower

@javisantana @rochoa take a look at last commit, a4952f6a1e2b009f1e960725c332684a0c6297f3. It's needed for tables with default values depending on sequences. After this, I've been able to open a table my group has RW permissions and insert a new row, but I can't without it.

better call Paul cc @pramsey

@pramsey I've replaced all %s with %I, thanks!

@rafatower could you do a last CR at this, please? There've been some minor refactor commits since the last time you saw this, and it'd be nice that you have a last look before final merge when QA is over. No need to hurry :-) Thanks!

Adding @rochoa, as platform will be in charge of a large part of the extension.

@rafatower about your comment on EXECUTE (which I can't find at GitHub), I've fixed cartodb._CDB_User_RoleFromUsername and cartodb._CDB_Group_ShortDatabaseName.


Grid is not visible on computers with multiple screens

With computers that use multiple screens, the grid jumps out of the field of view, probably because the initial grid coordinates are calculated off of screen size.
This was reported by a Facebook user and confirmed on my machine with a secondary monitor hooked up.

What about using window.screenTop and window.screenLeft to set the defaults based on viewport instead of screen?

So I tried window.screenTop and window.screenLeft, but they have the same problem that window.screenX and window.screenY have.
I'm curious what the purpose of the following code is from page.tsx, lines 129-133:
  useEffect(() => {
    const dy = windowScreen.y + window.outerHeight - window.innerHeight;
    const dx = windowScreen.x + window.outerWidth - window.innerWidth;
    setCanvasOffset({ x: -dx, y: -dy });
  }, [windowScreen.x, windowScreen.y]);

Was this to maintain the same location of the grid regardless of where the browser was on the screen?
If I set the canvasOffset to: setCanvasOffset({ x: 0, y: 0 }); the issue goes away.

Yes, that exactly it. Except it doesn’t work well because some people have toolbars to the right or bottom. I was trying to stop the grid from moving when toggling full screen. Also, if someone moves their browser window. I’m not sure if it’s possible to reliably set the grid based on screen coordinates.

Here's what I'm thinking about that. Intuitively, I know there's a difference between full screen and non-full screen.I think most people will use fullscreen when calibrating/projecting, as it's the easiest. Even when it exits fullscreen to open a PDF, I immediately set it back to full screen to project (the calibration grid did not stay in the same place on my computer between full-screen and normal window).
I say we remove the complexity and just use the default offset. Let's make things easier on ourselves 😅 it's fast enough to recalibrate that even if this becomes an issue, I think it's only a minor issue at best.

I think you're totally right! I just overcomplicated things out of a fear of someone not realizing their calibration was off but people seem very careful about that and always have a calibration square in sight. It's not working anyway!

I've fallen victim to overcomplicating things more than I can count 😂 so you are in good company! I'll get this one knocked out and PR it here soon.


Source- github

Issue Type: Bug
Please someone help me i tried to send all files to github , but nothing was sent and all my data is lost, please help
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:13:20.174Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7400 CPU @ 3.00GHz (4 x 3000)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.97GB (6.95GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (11)

Extension
Author (truncated)
Version

add-reference
adr
1.0.2

php-debug
fel
1.13.0

php-intellisense
fel
2.3.14

vscode-solution-explorer
fer
0.3.10

discord-vscode
icr
3.14.0

amxxpawn-language
Kli
0.7.5

csharp
ms-
1.22.1

cpptools
ms-
0.28.3

format-html-in-php
rif
1.6.2

sourcepawn
sel
0.0.4

html-preview-vscode
tht
0.2.5

I really need that files

Can you explain exactly where you clicked and what happened? Can you reproduce the issue?

Thanks for response, i connected VS code with my github and send my folder to github repository and since this, folder is lost

I think my data  is forever lost :(

I clicked on Source Control and Publish Branch

and all data lost , i hate github from this time

Can you find your files somewhere in https://github.com/RaidoNek?
Did you have any backup of your files?
Did the files simply disappear after you published them to GitHub? How exactly did you publish them? Which button did you press?

Nothing at my discord, i have backup of some files, but actually 5 months ago... I just logged VS code with github and it idk how just put my folder into this source control and i have clicked to publish branch..  and files disappeared.. folder is still here, but files not.

Can you show me a screen recording, by doing the same on another, sample folder?

Now i cant even login to github from VS code, thanks for your time i have to pay for data recovery and blocking every interaction with gits. thanks.

Sorry but I really can't help if you can't provide additional information. Please let me know if you find out more.


Separate internal representation of symbols and aliases during execution

We can reduce the use of MetadataTools by giving distinct internal representations to symbols and aliases. The representation for symbols will also cache the symbol attributes. Ultimately this will make it easier to keep the pattern attributes up-to-date.
Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock
[ ] Style conformance: stylish-haskell

TODO

[ ] Extract Symbol helpers from Test.Kore.Unification.Unifier

The unit tests pass, but the integration tests will still fail because we are bypassing the constructorFunctions special cases, particularly for the kseq and dotk symbols. I have inquired about giving those symbols the constructor attribute.


Add PlayerGameModeChangeEvent

This pull requests adds a PlayerGameModeChangeEvent.

Closed due to mass conflicts. Feel free to reopen and fix them.


[2.0.3-beta] Right context menu on a marker doesn't work anymore

Looks like contextmenu string replacement on a marker is broken, I get the standard "Delete" button (it used to work in 1.5.39).
Right contextmenu replacement in the map (not on a marker) is ok, though.

brain fade here... sorry... do you have an example / screenshot so I can recreate ?

closed by 2.0.4-beta


Error with OpenGL when opening Godot

Tested versions
Godot 4.2.1
System information
Windows 10 - NVIDA Geforce MX450 - Godot 4.2.1
Issue description
When trying to open Godot, I get this error. I have not updated or changed anything, but started getting this error out of nowhere

Steps to reproduce
In my case, just try to open Godot
Minimal reproduction project (MRP)
No response

Have you tried updating your drivers? Or rebooting? I'm asking because you say you haven't updated and unclear if you've followed those instructions 🙂

Have you tried updating your drivers? Or rebooting? I'm asking because you say you haven't updated and unclear if you've followed those instructions 🙂

I have rebooted my computer, reinstalled Godot, and updated drivers. still have the error

Why not try to download NVidia's game ready drivers? I have searched the most latest one for your GPU: https://www.nvidia.com/en-gb/geforce/drivers/results/217060/
Worked for my GT 740M considering the fact its so old

@Fajrero50 Which NVIDIA driver version are you using? You can check this by pressing Windows + R, entering dxdiag, clicking OK, accepting the signature check and going to the Display tab. Look on the right for the Drivers box.


extend method missing options param

Typescript definition is missing options as a third parameter. Therefore it is not possible to bind target parameter for custom validations such as custom confirmed validator. This fix is to solve this issue in typescript.

thanks for the pr, much appreciated!


make big numbers pretty (and add K, M, B etc.) and round decimals

should probably enable directly changing x/y tick text


Allow partial override of the ubtu22cis_sshd struct

Overall Review of Changes:
My use case is simple:
When configuring SSHD, I need to set AllowUsers to a specific value, while leaving the rest of the configuration for SSHD as it is. More specifically I want to keep using the default ciphers (macs, kex_algorithms, etc.) as recommended by CIS.
I was only able to do that if I defined in my own variable the whole structure ubtu22cis_sshd as defined in defaults/main.yml which is really cumbersome.
It would be more user friendly if I was able to overwrite just on specific settings in my variable and use the recommended values for the rest of the settings, like this:
---
- name: ubuntu22-base
  hosts: all
  become: yes
  vars:
    # SSHD configuration, only overriding the allow_users, and keep the rest to its recommended values.
    ubtu22cis_sshd:
      allow_users: "{{ansible_user}}"
  roles:
    - mindpointgroup.ubuntu22_cis

Issue Fixes:
NA
Enhancements:
NA
How has this been tested?:
Tested on a pristine VirtualBox VM straight from installation.
ansible [core 2.15.6]
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['~/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python3/dist-packages/ansible
  ansible collection location = ~/.ansible/collections:/usr/share/ansible/collections
  executable location = /usr/bin/ansible
  python version = 3.10.12 (main, Jun 11 2023, 05:26:28) [GCC 11.4.0] (/usr/bin/python3)
  jinja version = 3.0.3
  libyaml = True

pre-commit.ci autofix

Not sure if that is a good idea. The combine_filter might be a better solution

PR unnecessary as this can be achieved by:
---
- name: ubuntu22-base
  hosts: all
  become: yes
  vars:
    ubtu22cis_sshd: "{{ ubtu22cis_sshd | ansible.builtin.combine({'allow_users': '{{ansible_user}}'}) }}"
  roles:
    - mindpointgroup.ubuntu22_cis

=> Closing the PR.

BTW, this whole PR might be simply solved by making sure that the AllowUsers contains not only vagrant and ubuntu, but also {{ansible_user}}

Hey @rostskadat,
Thank you for opening a PR and helping with the project, community input/support/help is always greatly appreciated and encouraged.  I'm looking at this PR and I might need some more explanation of the issue it is solving. You mention the issue is with defining the allow users and keeping the rest of the sshd variables set as they are in defaults/main.yml. The allow users variable is its own variable, ubtu22cis_sshd. allow_users. If you use the extra vars pretty anywhere you can set variables that will override the default value(s) we have in defaults/main.yml. So you can set ubtu22cis_sshd. allow_users to whatever you need it to be in -e at run time, host/group vars in inventory (setting on inventory is the preferred way), extra vars in templates/projects in AAP/Tower/AWX, etc. to have that newly defined value used where the allow users are applied. The defaults/main.yml is generally not intended to be modified, the goal is to modify the values by overriding via extra var areas. Since this adds a fair bit of complexity to the var values I want to make sure I fully understand the situation being addressed.
The other issues is can you remove the .editorconfig file?

Hi @georgenalen ,
Thanks for your reply. The idea behind the PR was to be able to override just one part of the ubtu22cis_sshd structure (in this case the allow_users leaving the rest as it is defined in the role. However I've had trouble managing on doing that, and it might be to my limited experience with all the different ways to set facts in Ansible...
From your comment I understand that it should be possible to override just one of the ubtu22cis_sshd attributes leaving the rest to their default value. Allow me to boil down the playbook to its minimal form:
---
- hosts: all
  vars:
    ubtu22cis_sshd:
      allow_users: "ubuntu"
      other_key: "default_value"
  tasks:
    - debug:
        msg: "{{ubtu22cis_sshd}}"

with the following inventory
---
all:
  hosts:
    target:
      ubtu22cis_sshd.allow_users: admin

When running this I still get ubtu22cis_sshd.allow_users=ubuntu
When running with the -e ubtu22cis_sshd.allow_users=admin on the command line I still get ubtu22cis_sshd.allow_users=ubuntu

My question is whether I understood correctly your suggestion.
As for the .editorconfig, it has been removed.

Just wanted to add a +1 for this exact scenario and PR solution.
I'm trying to customize ubtu22cis_sshd with the following pattern:
role_vars/ubuntu22_cis.yml
---
ubtu22cis_sshd.log_level: "VERBOSE"

Playbook: some_server.yml
---
- hosts: target
  become: yes
  vars_files:
    - role_vars/ubuntu22_cis.yml
  roles:
    - role: MindPointGroup.ubuntu22_cis
      tags:
        - level1-server
        - level2-server

I'm customizing many other, directly assigned variables using the same pattern, without problem.
However, because ubtu22cis_sshd is a dictionary structure instead of individual variables, I'm unable to follow this pattern here.
I took a look at the PR and it would seem to solve my needs, so it has my +1 on merging this.

Just wanted to add a +1 for this exact scenario and PR solution.
In my project, I'm customizing UBUNTU22-CIS using the following pattern:
role_vars/ubuntu22_cis.yml
---
ubtu22cis_purge_apt: true
ubtu22cis_time_sync_tool: "chrony"
# ...

Playbook: some_server.yml
---
- hosts: target
  become: yes
  vars_files:
    - role_vars/ubuntu22_cis.yml
  roles:
    - role: MindPointGroup.ubuntu22_cis
      tags:
        - level1-server
        - level2-server

(This does NOT work if, e.g., I add ubtu22cis_sshd.log_level = "VERBOSE" to role_vars/ubuntu22_cis.yml, as I believe you were suggesting in your comment here.)
Otherwise, I'm customizing many other, directly assigned variables using this pattern and it's working great for my needs.
But because ubtu22cis_sshd is a dictionary structure instead of individual variables, I'm unable to use this pattern for it.
I took a look at the PR and it would seem to solve my needs here, too. If there's anything I can do to help this PR along, I'd be glad to lend the help!

Hello @uk-bolly,
Thanks for your feedback. I fixed both problems.
Please let me know if there is anything missing.
Regards

hi @rostskadat
Thank you for the quick turnaround.
All looks good in the pipeline, the final commit is gpg signed, but it is still not happy with the signing. You will see that the merge is still blocked as not all commits are signed.
Many thanks
uk-bolly

Hello @uk-bolly
Sorry to bother you about that, but now I'm confused on how to proceed.
Allow me to give you a summary of the situation. As you mentioned the last commit (aa21b5b3 a merge commit) was not signed off.
I tried to rebase the history of my rostskadat:devel branch by signing off the last commit with git rebase --signoff HEAD~1.

Which might be more sign off than I might want to do.
I'm not really sure how to proceed and I would really appreciate if you could give me an indication on how to solve this issue.
Regards

@rostskadat
Just taking a guess here. git commit --amend --no-edit -S <commit hash> should allow you to amend a signature to a previous commit.
Since there's only 10 commits in this PR, maybe it wouldn't be too difficult to iterate individually through all 10 commit hashes using the above command to sign them?

Also, just a heads up that it seems like your key might be expired (note the blue 'Expired' bubble beneath the key hash).
You may need to address that prior to actually signing the commits.

On a recent PR of my own, it seemed like the CI system requires all of the following:

GPG signed commits
Sign-off commits

@uk-bolly is probably the best person to lead this conversation, as they're the maintainer of the CI system.

hi @rostskadat and @joshavant
Thank you for the time and conversations on this thread, this was merged a while back and i assume it is as expected.
I will therefore close this PR (Note that v2.0.0 has been released to subscribers and will be available in the next few of months).
Many thanks once again
uk-bolly


Animation not working on sets?

I can't seem to get animation to work for sets...
a simplest example:
const growAnimation = Raphael.animation({ transform:'"S5"' }, 300);

<Set animate={growAnimation} >
  <Circle x={100} y={200} r={5} />
  <Circle x={200} y={200} r={15} />
</Set>

but if I put the animate={growAnimation} direction on each of the <Circle>s then they do animate immediately on component load.

please see http://dmitrybaranovskiy.github.io/raphael/reference.html#Element.transform


Failed to execute 'matches' on 'Element': ''[name' is not a valid selector.

Getting this error when using paper-time-picker as a child of a custom defined component that uses webcomponents, but not polymer.
Uncaught SyntaxError: Failed to execute 'matches' on 'Element': ''[name' is not a valid selector.
DomApi.queryDistributedElements @ polymer-mini.html:702
Polymer.IronSelectableBehavior._updateItems @ iron-selectable.html:221
Polymer.IronSelectableBehavior.attached @ iron-selectable.html:147
Polymer.Base._addFeature._invokeBehavior @ polymer-micro.html:392
Polymer.Base._addFeature._doBehavior @ polymer-micro.html:385(anonymous function) @ polymer-micro.html:196
Polymer.RenderStatus.whenReady @ polymer-micro.html:110
Polymer.Base.attachedCallback @ polymer-micro.html:194
Polymer.Base._addFeature.attachedCallback @ polymer-mini.html:100

I'll add the the value of the selector is correct ([name]) for a few calls on init, but two of them fail with the strange mangled version.

Can you see if you can reproduce this using plunker, jsbin, jsfiddle, etc? You can use the following plunker as a starting point: http://plnkr.co/edit/gVQluG0GrFP3RzCPZPIi

Hey Ben, we have dropped paper/polymer stuff for the time being.  But I believe the problem for this one might have been that we are using virtual-dom, and html-to-vdom, the latter of which removes attributes it knows to be builtins for standard elements.

In that case @naturalethic I'm going to go ahead and close this issue out, seams like it's really an interop problem that we won't have much control over until v1 compliance across all browsers (which will be very soon!)


traefik v1.7  Error when creating error page

Do you want to request a feature or report a bug?

Bug

What did you do?
It is a docker swarm,
A TOML file to define a backend as an error page.
A traefik server as edge router, a dummy server to handle request.
When the swarm services setup,
I visit http://a.fake.jd.com/traefik/dashboard/ can be displayed successfully,
However when I visit http://a.fake.jd.com/a-long-funding-api/401, I didn't see the wanted error page.

What did you expect to see?
I wanted to see errors from 400 to 499 will trigger https://an-external-url/error-page.html to show.
What did you see instead?
No custom error page show. When I check the traefik.log I found:
Error when creating error page \"networks\" for frontend \"frontend-web-dummy-funding-web\": the backend \"backend-error\" doesn't exist.","time":"2020-09-16T11:05:03Z"}
In fact, I've configured backend error in the TOML file. But why it shows doesn't exist?
Output of traefik version: (What version of Traefik are you using?)

Version:      v1.7.16
Codename:     maroilles
Go version:   go1.12.9
Built:        2019-09-13_01:12:20PM
OS/Arch:      linux/amd64

What is your environment & configuration (arguments, toml, provider, platform, ...)?
file provider: traefik_11 (TOML file)
logLevel = "INFO"

[respondingTimeouts]
idleTimeout = "360s"

[file]
[accessLog]
  format = "json"
[entryPoints]
  [entryPoints.http]
  address = ":80"

[backends]
  [backends.error]
    [backends.error.servers.server0]
    url = "https://an-external-url"

[traefikLog]
  filePath = "/traefik/traefik.log"
  format   = "json"

docker provider: a traefik service
version: "3.6"

networks:
  traefik:
    external: true

configs:
  traefik_cfg:
    name: traefik_11
    external: true

services:
  reverse-proxy:
    image: traefik:alpine # The official Traefik docker image
    networks:
      - traefik
    command: "--api --docker --docker.watch --docker.swarmMode --respondingtimeouts.idletimeout=360s --respondingtimeouts.writetimeout=360s" # Enables the web UI and tells Traefik to listen to docker
    ports:
      - "80:80" # The HTTP port
      - "443:443"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock # So that Traefik can listen to the Docker events
    deploy:
      mode: global
      placement:
        constraints:
          - node.role==manager
      labels:
        - "traefik.port=8080"
        - "traefik.enable=true"
        - "traefik.passHostHeader=true"
        - "traefik.docker.network=traefik"
        - "traefik.frontend.entryPoints=http"
        - "traefik.backend.loadbalancer.swarm=true"
        - "traefik.frontend.rule=PathPrefix:/traefik;PathPrefixStrip:/traefik"
    configs:
      - source: traefik_cfg
        target: /etc/traefik/traefik.toml

docker provider: a dummy server to answer request:
version: "3.6"

networks:
  traefik:
    external: true

services:
  funding:
    image: jasonrm/dummy-server:latest # The official Traefik docker imageports:
    networks:
      - traefik
    deploy:
      replicas: 2
      placement:
        constraints:
          - node.role!=manager
      labels:
        - "traefik.domain=a.fake.jd.com"
        - "traefik.port=8080"
        - "traefik.enable=true"
        - "traefik.passHostHeader=true"
        - "traefik.docker.network=traefik"
        - "traefik.frontend.entryPoints=http"
        - "traefik.backend.loadbalancer.swarm=true"
        - "traefik.frontend.rule=PathPrefix:/a-long-funding-api;ReplacePathRegex:^/a-long-funding-api/(.*) /$$1;Host:a.fake.jd.com;"
        - "traefik.frontend.errors.networks.backend=error"
        - "traefik.frontend.errors.networks.status=400-599"
        - "traefik.frontend.errors.networks.query=/error-page.html"

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
{"level":"error","msg":"Error when creating error page \"networks\" for frontend \"frontend-PathPrefix-a-long-funding-api-ReplacePathRegex-a-long-funding-api-1-Host-a-fake-jd-com-1\": the backend \"backend-error\" doesn't exist.","time":"2020-09-17T11:17:27Z"}

Hi! I'm Træfiker :robot: the bot in charge of tidying up the issues.
I have to close this one because of its lack of activity :disappointed:
Feel free to re-open it or join our Community Forum.


improve login form

initial focus, clicking on button, etc.

@igiveadamn please re-open if initial work is required on this.


Sanitize log output

This sanitizes the output of the log file to remove password and
security code from the envisalink subscribe GET call.

Feel free to reopen/merge at will.


New status column

I added the status column for the features in order to prepare for the approval/decline flow for admins.

I'm closing this and opening a new one with squashed commits from @Kazaag.


host: Some job events are being dropped

In this CI failure, DeployerSuite.TestOmniProcess failed because omni jobs were stopped but the events didn't reach the scheduler, so the jobs were only marked as down in the controller when a sync happened, and that happened after the deployment timed out.
Take job 176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 for example:
Scheduler requests it to be stopped:
t=2016-02-07T04:41:38+0000 lvl=info msg="stopping omni job" component=scheduler fn=stopJobOfType app.id=57dee68e-f69b-448d-bad9-7a88a32eb127 release.id=3c798065-e30a-43f3-9746-06f5b2a78595 job.type=omni
t=2016-02-07T04:41:38+0000 lvl=info msg="stopping job" component=scheduler fn=stopJob job.id=04bff354-c570-4de8-afaf-b824c89c2b18 job.type=omni job.state=running
t=2016-02-07T04:41:38+0000 lvl=info msg="requesting host to stop job" component=scheduler fn=stopJob job.id=04bff354-c570-4de8-afaf-b824c89c2b18 job.type=omni job.state=running host.id=176903e8

Host stops the job (note the rpc error, may be related, but may just be because the container died):
t=2016-02-07T04:41:38+0000 lvl=info msg="request started" component=host req_id=ecda0eab-7184-4f59-b84a-fb0df03df2e2 method=DELETE path=/host/jobs/176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 client_ip=<IP_ADDRESS>
t=2016-02-07T04:41:38+0000 lvl=info msg="acquiring state database" app=host pid=1123 host.id=176903e8 fn=StopJob job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18
t=2016-02-07T04:41:38+0000 lvl=info msg="getting job" app=host pid=1123 host.id=176903e8 fn=StopJob job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18
t=2016-02-07T04:41:38+0000 lvl=info msg="stopping job" app=host pid=1123 host.id=176903e8 fn=StopJob job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18
now=2016-02-07T04:41:38+0000 backend=libvirt-lxc fn=watch_container job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 at=change state=exited
now=2016-02-07T04:41:38+0000 at=exited status=0 backend=libvirt-lxc fn=watch_container job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18
2016/02/07 04:41:38 rpc: client protocol error: unexpected EOF
now=2016-02-07T04:41:38+0000 backend=libvirt-lxc fn=waitExit job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 at=start
now=2016-02-07T04:41:39+0000 fn=waitExit job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 backend=libvirt-lxc at=done
now=2016-02-07T04:41:39+0000 job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 at=start backend=libvirt-lxc fn=cleanup
now=2016-02-07T04:41:39+0000 fn=unbind_mounts job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 at=start backend=libvirt-lxc
now=2016-02-07T04:41:39+0000 fn=unbind_mounts job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 at=unmount file=.containerinit status=error err="invalid argument" backend=libvirt-lxc
now=2016-02-07T04:41:39+0000 status=error err="invalid argument" fn=unbind_mounts job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 backend=libvirt-lxc at=unmount file=resolv.conf
now=2016-02-07T04:41:39+0000 job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 at=finish backend=libvirt-lxc fn=unbind_mounts
time="2016-02-07T04:41:39Z" level="error" msg="Removing active id tmp-176903e8-04bff354-c570-4de8-afaf-b824c89c2b18"
now=2016-02-07T04:41:39+0000 fn=cleanup job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 at=finish backend=libvirt-lxc

Observe subsequent job events in the scheduler for host 176903e8 but for other jobs:
t=2016-02-07T04:41:40+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=176903e8-9f880306-f63f-4e69-acad-1a9f54ff3ed1 event.type=create
t=2016-02-07T04:41:41+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=176903e8-9f880306-f63f-4e69-acad-1a9f54ff3ed1 event.type=start
t=2016-02-07T04:41:41+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=176903e8-a8a8ebd7-943e-4de2-ad1d-2fd53fa21c03 event.type=stop
t=2016-02-07T04:41:54+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=176903e8-29f794d6-c674-4def-9036-cde8f03f67c1 event.type=create
t=2016-02-07T04:41:55+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=176903e8-29f794d6-c674-4def-9036-cde8f03f67c1 event.type=start

And then the job is only marked as stopped after a sync:
t=2016-02-07T04:41:59+0000 lvl=info msg="syncing jobs" component=scheduler fn=SyncJobs
t=2016-02-07T04:41:59+0000 lvl=info msg="handling job status change" component=scheduler fn=handleJobStatus job.id=176903e8-04bff354-c570-4de8-afaf-b824c89c2b18 app.id=57dee68e-f69b-448d-bad9-7a88a32eb127 release.id=3c798065-e30a-43f3-9746-06f5b2a78595 job.type=omni from=stopping to=stopped

This is now happening frequently in CI since moving to running jobs on a single VM (so the load is quite high).
Interestingly, events which happened at almost the same time in the host appeared a minute apart in the scheduler:
In the host log:
t=2016-11-28T13:15:53+0000 lvl=info msg="state change" app=host pid=1108 host.id=f719a789 component=backend backend=libcontainer fn=watch job.id=f719a789-d5ebae3c-96c1-41ec-bf78-764aad4b567d state=initial
t=2016-11-28T13:15:53+0000 lvl=info msg="state change" app=host pid=1108 host.id=f719a789 component=backend backend=libcontainer fn=watch job.id=f719a789-d5ebae3c-96c1-41ec-bf78-764aad4b567d state=running
t=2016-11-28T13:15:53+0000 lvl=info msg="state change" app=host pid=1108 host.id=f719a789 component=backend backend=libcontainer fn=watch job.id=f719a789-d5ebae3c-96c1-41ec-bf78-764aad4b567d state=exited

In the scheduler log:
t=2016-11-28T13:15:50+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=f719a789-d5ebae3c-96c1-41ec-bf78-764aad4b567d event.type=create
t=2016-11-28T13:15:53+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=f719a789-d5ebae3c-96c1-41ec-bf78-764aad4b567d event.type=stop
t=2016-11-28T13:15:53+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=f719a789-d5ebae3c-96c1-41ec-bf78-764aad4b567d event.type=cleanup
...
t=2016-11-28T13:16:50+0000 lvl=info msg="handling job event" component=scheduler fn=HandleJobEvent job.id=f719a789-d5ebae3c-96c1-41ec-bf78-764aad4b567d event.type=start

So the host emitted the start event around the same time as the exiting and cleanup events, but the scheduler only received the start event a minute later.
Also note that the scheduler synced with the hosts a few times in that time:
t=2016-11-28T13:15:57+0000 lvl=info msg="syncing hosts" component=scheduler fn=SyncHosts
t=2016-11-28T13:16:07+0000 lvl=info msg="syncing hosts" component=scheduler fn=SyncHosts
t=2016-11-28T13:16:17+0000 lvl=info msg="syncing hosts" component=scheduler fn=SyncHosts
...
t=2016-11-28T13:16:47+0000 lvl=info msg="syncing hosts" component=scheduler fn=SyncHosts

I'm leaning towards the scheduler loop lagging behind (perhaps the sync hosts call is taking longer than 10s) and not actually processing the job events.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


help2man: 1.47.10 -> 1.47.11

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/help2man/versions.
meta.description for help2man is: '"Generate man pages from `--help' output"'.
meta.homepage for help2man is: '"https://www.gnu.org/software/help2man/"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.47.11 with grep in /nix/store/cmccd8029yp3vgjysm7x7pyx8p779nrs-help2man-1.47.11

directory tree listing: https://gist.github.com/bee9de9bb91435b3480a84d1b1170d9c

du listing: https://gist.github.com/6efbae80e0f6a867a5afa423941fdcf5

Rebuild report (if merged into master) (click to expand)

99490 total rebuild path(s)
28585 package rebuild(s)
28424 x86_64-linux rebuild(s)
26060 i686-linux rebuild(s)
19862 x86_64-darwin rebuild(s)
25144 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
AMB-plugins
AgdaSheaves
AgdaStdlib
CoinMP
DisnixWebService
EBTKS
EmptyEpsilon
FIL-plugins
Fabric
LASzip
Literate
MMA
NSPlist
OVMF
OVMF-CSM
PlistCpp
QuadProgpp
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer
SDL2_net
SDL2_ttf
SDL_gfx
SDL_gpu
SDL_image
SDL_mixer
SDL_net
SDL_sixel
SDL_sound
SDL_stretch
SDL_ttf
SkypeExport
Sylk
TotalParserCombinators
Xaw3d
_0x0
_1password
_2048-in-terminal
_20kly
_2bwm
_389-ds-base
_90secondportraits
_915resolution
_9pfs
a2jmidid
a2ps
a52dec

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cmccd8029yp3vgjysm7x7pyx8p779nrs-help2man-1.47.11 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A help2man https://github.com/r-ryantm/nixpkgs/archive/bee67d463f6f16c3e6f805abbe69c277655984c4.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cmccd8029yp3vgjysm7x7pyx8p779nrs-help2man-1.47.11
ls -la /nix/store/cmccd8029yp3vgjysm7x7pyx8p779nrs-help2man-1.47.11/bin

cc @pSub for testing.

Here is the changelog :
help2man (1.47.11) unstable; urgency=medium

  * Adjust maintainer sanity check again, this time to handle binary NMUs
    (closes: #934601).

 -- […] <[…]>  Mon, 12 Aug 2019 22:52:07 +1000

The new version seems to be for a debian related issue (https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=934601). It should be safe to merge in staging.
I am building it for tests at the moment.

Built and tested locally. Works fine


layout shift when editing a Text

See below. It is tiny, but certainly annoying
https://user-images.githubusercontent.com/7490180/226419219-99cc2653-39e5-4efa-a831-b4148121892b.mp4

Looks like a regression introduced in https://github.com/excalidraw/excalidraw/pull/6187 /cc @ad1992

@sedghi can you test in https://excalidraw-git-aakansha-baseline-excalidraw.vercel.app/ if its working fine ?


Key repeat broken for uppercase letters

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS 10.11.6 and macOS:
Hyper.app 1.0.0:
Link of a Gist with the contents of your .hyper.js:
Relevent information from devtools (CMD+SHIFT+I on Mac OS, CTRL+SHIFT+I elsewhere):
The issue is reproducible in vanilla Hyper.app:

Issue
Key repeat broken when using shift + letter

Do you have a picture?

I'm having the same issue and I can report that there's nothing to take a picture of. You just don't get key repeat when holding down a letter key and shift at the same time.

ok, I see what you mean. Thanks

I think this may be related to the OS character viewer you get when holding down keys. I took a quick screengrab of what it looks like when I hold down different keys: https://media.giphy.com/media/l0MYAzQDK9xPFSm5O/giphy.gif
It's kind of a lousy quality gif, so I'll summarize what's happening:

a: repeating character, but slower than in Terminal.app and the character viewer flashes up for an instant on each repeat.
d: repeating character, but slower than the normal key repeat in Terminal.app.
E (shift+e): only character viewer—no repeating character.
D (shift+d): no character viewer or key repeat.

Here's what happens with the same characters in this GitHub text editor. I'm assuming this is the OS behavior in text editing contexts:

a: character viewer only.
d:  nothing at all (no repeat or character viewer).
E: character viewer only.
D: nothing at all.

And finally, here's what happens with the same keys in Terminal.app, which is the functionality I was expecting:

a: repeating character, no character viewer.
d:  repeating character, no character viewer.
E: repeating character, no character viewer.
D: repeating character, no character viewer.

related to #1137
For now, you can disable this feature system-wide with the following command:

defaults write -g ApplePressAndHoldEnabled -bool false


Unable to see image after upload

Trying to integrate ContentTools in an Angular app.
When trying to add an Image, the upload dialog opens, I can select a file, but the uploaded file doesn't become visible
can't see any error or info in the console
Any suggestion ?
D.

Already found part of the reason. Seems I installed a new version and the event-binding has changed.
bind( xx.fileReady ) > addEventListener(xx.fileready) : events are no longer camel-case

Yeah the event system got a complete overhaul in release 1.2.0 are you still having issues, can you provide an example of you code?


[Hub Generated] Review request for Microsoft.AzureStackHCI to add version stable/2023-02-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

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

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month. 21 Feb 2022
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month. 15 Feb 2022
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

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

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

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

Change to data plane APIs
Adding new properties
All removals

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

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

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

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

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

These changes for a new api version 2023-02-01 have already been reviewed and merged in the private repo in PR https://github.com/Azure/azure-rest-api-specs-pr/pull/10950/

Hi @prateekprshr-nith API Readiness Check fails. Is the new api-version deployed in ARM?

Hi @prateekprshr-nith API Readiness Check fails. Is the new api-version deployed in ARM?

@msyyc, we are an RPaaS based RP, so this doesn't apply to us.

Hi @prateekprshr-nith API Readiness Check fails. Is the new api-version deployed in ARM?

@msyyc, we are an RPaaS based RP, so this doesn't apply to us. It was ignored in our last PR as well - #21717

add @jianyexi to confirm.

merge since https://github.com/Azure/azure-rest-api-specs/pull/21717 which has same content has been reviewed.


Fix typos and make easier-to-understand text.

Well lag does kinda mean delay, but I'll merge.

good mookan


Fixed transition delay for carousel fade

Closes https://github.com/twbs/bootstrap/issues/28227.
Transition duration was getting 0 value. Fixed the issue by setting transition delay value to 0 and setting transtion duration to $carousel-transition-duration.

This will fade fade out the old carousel item before the new item is faded in, which isn't the behaviour we want. Compare these carousels to check out what I mean:
Your implementation:
https://deploy-preview-28278--twbs-bootstrap.netlify.com/docs/4.3/components/carousel/#crossfade
Current implementation:
https://getbootstrap.com/docs/4.3/components/carousel/#crossfade

Hi @MartijnCuppens : Actually the Issue states that the images with transparent background in that case disappears immediately and looks ugly. If we persist old slide to stay there before the new one fades in, it will eventually be the same. Can you please suggest me the desired behavior for this case so that I can update my PR? Thanks!
This will be my first contributing PR for bootstrap and want to contribute more :)

@jahanzaibsuleman07 I believe you need to set a background-color or something else instead of modifying our source files for all images. I'm in agreement with @MartijnCuppens—the current version in our docs is the desired implementation here.


[stable/sonarqube] change wget to curl in install-plugins.yaml

Hello.
In my company env, we use http/https proxy.
in my test,
# docker run --rm -it alpine:3.10.3
/ # export HTTP_PROXY="http://proxy:port"
/ # export HTTPS_PROXY="http://proxy:port"
/ # apk update
fetch http://dl-cdn.alpinelinux.org/alpine/v3.10/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.10/community/x86_64/APKINDEX.tar.gz
v3.10.3-22-g9655dce427 [http://dl-cdn.alpinelinux.org/alpine/v3.10/main]
v3.10.3-20-g9fd7505b58 [http://dl-cdn.alpinelinux.org/alpine/v3.10/community]
OK: 10338 distinct packages available
/ # apk add curl
(1/4) Installing ca-certificates (20190108-r0)
(2/4) Installing nghttp2-libs (1.39.2-r0)
(3/4) Installing libcurl (7.66.0-r0)
(4/4) Installing curl (7.66.0-r0)
Executing busybox-1.30.1-r2.trigger
Executing ca-certificates-20190108-r0.trigger
OK: 7 MiB in 18 packages
/ # curl -o test.jar "https://binaries.sonarsource.com/Distribution/sonar-java-plugin/sonar-java-plugin-
<IP_ADDRESS>88.jar"
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 8152k  100 8152k    0     0   195k      0  0:00:41  0:00:41 --:--:--  213k

/ # export http_proxy="http://proxy:port"
/ # export https_proxy="http://proxy:port"
/ # wget "https://binaries.sonarsource.com/Distribution/sonar-java-plugin/sonar-java-plugin-<IP_ADDRESS>88.jar" --no-check-certificate
Connecting to proxy:port (proxy:port)
wget: server returned error: HTTP/1.1 400 Bad Request

curl does not have a problem,
but I think wget has some problem and I tested alpine:3.10.x 3.9.x 3.8x all same.

I am digging refered https://github.com/Yelp/dumb-init/issues/73,
and find out.
# docker run --rm -it alpine:3.10.3
/ # export HTTP_PROXY=http://proxy:port
/ # export HTTPS_PROXY=http://proxy:port
/ # apk --no-cache add openssl wget
fetch http://dl-cdn.alpinelinux.org/alpine/v3.10/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.10/community/x86_64/APKINDEX.tar.gz
(1/2) Installing openssl (1.1.1d-r0)
(2/2) Installing wget (1.20.3-r0)
Executing busybox-1.30.1-r2.trigger
OK: 7 MiB in 16 packages
/ # export http_proxy=http://proxy:port
/ # export https_proxy=http://proxy:port
/ # wget "https://binaries.sonarsource.com/Distribution/sonar-java-plugin/sonar-java-plugin-<IP_ADDRESS>88
.jar" --no-check-certificate
--2019-11-12 11:22:40--  https://binaries.sonarsource.com/Distribution/sonar-java-plugin/sonar-java-plugin-<IP_ADDRESS>88.jar
Connecting to <IP_ADDRESS>:8080... connected.
Proxy request sent, awaiting response... 200 OK
Length: 8348625 (8.0M) [application/x-java-archive]
Saving to: 'sonar-java-plugin-<IP_ADDRESS>88.jar'

sonar-java-plugin-5.14.0. 100%[=====================================>]   7.96M   215KB/s    in 40s

2019-11-12 11:23:21 (205 KB/s) - 'sonar-java-plugin-<IP_ADDRESS>88.jar' saved [8348625/8348625]

after re-install openssl and wget, it works like charm.

so if you want to use curl, please update install-plugins.yaml

I would also support changing it to curl...

@tsiddique @rjkernick please could you take a look on this

@tsiddique @rjkernick bump


a guard priority question

Hi. Can you advise me what should I do. I need to create an auth service for an api before guards start. What a priority of my listener should be or an event? Or how can I achieve this?

Also make sure that your use-case can't be covered with a custom provider: https://github.com/ZF-Commons/zfc-rbac/blob/master/docs/03. Role providers.md


feat: only and skip support

Fixes issue 5
WIP
Developments so far

Implemented only usage

Only a single suite or case can be only'ed, if more than 1 is, theen rhum throws an error
Will show all other tests as ignored

added skip support

same as above but as many test *'s can be skipped

Temporarily changed the output of rhum, see below

What's left

Complete and fix integration tests for only
add unit and integration tests for skip
add unit tests for only method
correct the output format, and adjust tests for this

Different types of output

Developments so far:

Implemented only usage

Only a single suite or case can be only'ed, if more than 1 is, theen rhum throws an error
Will show all. other tests as ignored

Temporarily changed the output of rhum which hasn't been best received, so it needs to be changed again - i'm proposing we ditch the current format but keep the test  bit in, so it's easier to write tests for

Decided to add skip support as well, because the only way only or skip will land, is if we ditch current output format, so may as well do it in one pr

can we rework this PR to be pointed to v2? or should we trash it and start over with a new branch, but taking the logic from this branch as a starting point?

Oh we can trash this PR, I was only keeping it open as a reference really until we landed your pr, if this PR is no use to you then feel free to close it and delete the branch 👍

i think it's still useful. we can keep it open as a reference for .only(). i like the edge cases so let's keep it open for now

Alright

Do we need only and skip properties for every test interface? It seems like it just overcomplicates things. For example, what happens if both only and skip are set to true?

Hey, yes because it means any test suite or case can be skipped, and i believe i added checks for if both are used but i can't remember, this PR was from a while ago :P Tbh, we're only keeping this PR up for reference purposes for the rhum v2 PR, none of this code will end up in master and when rhum v2 is in master, this pr will be closed and the code lost forever 😢

I know. I just realized that this getting this to work is probably not possible if we keep using deno.std/testing as a base, unless we find some way to change the message format for deno.std/testing. The only way I can see to get this work is just to write our own base for testing instead of working off of deno.std/testing.

So basically we would have to write our own "deno:cli/rt/40_testing.js".

it might be that we don't need this feature now that i think about it. i say this because rhum v2 CLI has the --filter-test-case and --filter-test-suite options

I think those options achieve the same functionality as skip, but I'm not sure those options achieve the same functionality as only. I'm pretty sure ebebbington's code would have worked if Deno's testing api had more customizable format options.

My pr was going to use a different test definition so we could use skip and only, but this meant the output would have changed in the shell for users
Where as now with Rhum v2, we have good output and can still ignore and only tests

For reference, this PR is only open to aid in eric's v2 PR, should any of the work i have done come in handy.
Once the v2 pr is merged, this will be closed without merging

not doing this anymore. v2 will be a test double framework


Question: Is there any way to inject memory into Tcmalloc?

Hi,
If I have a blob of memory that wasn't allocated by tcmalloc, is it possible to insert it into tcmalloc so that the allocator can use it as available memory?
It would have to be marked such that tcmalloc never unmaps it if it's empty.
Thanks!

Ok, I learned that I can do that by extending the SysAllocator and adding it. And also seems that tcmalloc never really unmaps anything, but just tell the kernel to drop the backing of regions of the virtual memory, which is great.


Prerequisites

Remove Visual Studio, not actually required to compile or run
Suggest Visual Code
Add ngrok
Node.js server is required, maybe add a link?

updated


touchpoint.lua and reactorController.lua does not have a version variable

After running pastebin run kSkwEchg

Fixed, this was due to a change in the pastebin API.
Unfortunately it can't be automatically fixed as the update script isn't versioned but wiping the computer and re-running the pastebin run command should fix.

Fixed by #19 , as stated above, a wipe + reinstall is required for the fix


CORTX-29179: Update RGW endpoints

Adjust RGW http/https ports to reflect updated CORTX defaults.

RGW services (container) listen on http/https ports 22751/23001.
External IO service updated to expose ports 80/443 by default. These are still configurable.
RGW settings config.yaml ConfigMap updated to match latest sample
S3 settings in config.yaml removed as it's no longer used

Signed-off-by: Keith Pine<EMAIL_ADDRESS>

These changes are not really needed functionality-wise. Aside from hard-coding the ports in the Chart templates, everything is configured correctly. This just brings a new deployment up-to-date with the CORTX defaults, and cleans up the cluster.yaml file.


KB shortcuts changes

What I added/modified:

Changed all instances of 'extended manifest JSON' to 'shortcuts JSON' for simplicity
Added links to the PnP sample at the beginning of each article.
Added a section for the limitation around the user's focus.
Removed per platform customization for public preview docs (will be included as part of GA)
For the tutorial, consolidate the entire JSON sample together to make it simpler for developers to copy and paste.
Add some keyboard mapping clarifications between Mac and win32.

Docs Build status updates of commit e3d5308:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/design/keyboard-shortcuts.md
:white_check_mark:Succeeded
View

docs/tutorials/keyboard-shortcuts-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Added FileTypes values

Issue #10214

Learn Build status updates of commit cc095ae:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-MalwareFilterPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-MalwareFilterPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Implement module.parent so Socket.io can work

Socket.io doesn't work right now because it uses module.parent.
Would it be possible to implement module.parent in Browserify?

@rattrayalex You'll run into larger issues, simply because of their Contextify dependency, which contains about 350 lines of C++ for creating a new V8 context to run code in. Good luck.

+1

Warning: this would be near impossible without runtime boilerplate, and lots of it. Where Browserify statically compiles the requires down to a single file, much of the filename information is lost, and the current module is largely unknown at runtime. Sourcemaps only know about them because they are only built to deal with line-to-line mappings, and can translate those mappings across files. On the other hand, those mappings are also statically generated, and cannot be retrieved during runtime.
Consider this:
// a.js
require('./b');
require('./c');

// b.js
var parent = require('./d');
process.nextTick(function () {
  console.log(parent());
});

// c.js
var parent = require('./d');
process.nextTick(function () {
  console.log(parent());
});

// d.js
module.exports = function (x) {
  return module.parent;
};

You would have to do some serious code analysis along with some actual evaluation to be able to tell what module.parent is. This is harder to accomplish than this (see my comment explaining why it's so difficult).
 Then look at this:
module.exports = function () {
  var mod = path.resolve(SOURCE_ROOT, module.parent.filename);
  console.log('Log from ' + mod + ': ');
  console.log('`this`: ' + this);
  console.log('Arguments:');
  for (var i = 0; i < arguments.length; i++) {
    console.log('  ' + i + ': ' + arguments[i]);
  }
  console.log();
};

It may look rather simple, but there is no safe way to bake in module.parent regardless of call site, since it is dynamically generated by Node.js. It's practically impossible to implement this safely without a implementation for each call of this function, given it is dependent on every way you could think of silently baking in an extra parameter for the function.

It would still help to make it non-falsey, so that scripts using if (!module.parent) as their main switch don't run their main.


Controller 'mdDataTable', required by directive 'mdDataTableHeaderRow', can't be found!

Hi, thanks for creating this module!  Unfortunately, I'm having a little trouble using it.  I took your example usage of the mdDataTable and put it in my view and (after including your module in my project), and get the following error.  Any ideas of what I might be missing?  Thanks!
Error: [$compile:ctreq] Controller 'mdDataTable', required by directive 'mdDataTableHeaderRow', can't be found!

Hi Samster, you're welcome, however still a lot to do with the project.
I think u missed wrapping the md-data-table-column with md-data-table-header-row.
Can you please make a screenshot or copy your code and show me?

I'm using the example code you had on your README as shown below.   My project is for work, so I'm unable to give you more of the source.
Could it have anything to do with the way bower is pulling in the library or the way I'm minifying using gulp etc?  It's just weird that it recognizes the md-data-table-column directive but then can't find the parent controller.
I'm not sure if it has anything to do with it, but I have an Angular project following the general structure outlined by John Papa's style guidelines.  In particular, I'm using ui-router, and "controller as" syntax etc.
Thanks for the help!
            <md-data-table
                   selectable-rows="true"
                    table-card="{title: Nutrition, actionIcons: true}">

                <md-data-table-header-row>
                    <!-- defining column descriptions, align content to the left -->
                    <md-data-table-column
                            align-rule="left"
                            column-definition="The total amount of food energy in the given serving size.">
                        Dessert (100g serving)
                    </md-data-table-column>

                    <!-- in case of inline menu -->
                    <md-data-table-column inline-menu="[ {iceCream: 'Ice Cream', pastry: 'Pastry', other: 'Other'} ]">Type</md-data-table-column>

                    <!-- inline text editing -->
                    <md-data-table-column editable-field="textInput">
                        Calories
                    </md-data-table-column>

                    <!-- in case of sortable columns, we can set the defaultly sortable column -->
                    <md-data-table-column sortable-rows-default>
                        Fat (g)
                    </md-data-table-column>
                    <md-data-table-column>Carbs (g)</md-data-table-column>
                    <md-data-table-column>Protein (g)</md-data-table-column>
                </md-data-table-header-row>

                <md-data-table-row ng-repeat="nutrition in nutritionList">
                    <md-data-table-cell>Frozen Joghurt</md-data-table-cell>
                    <md-data-table-cell>159</md-data-table-cell>
                    <md-data-table-cell>6</md-data-table-cell>
                    <md-data-table-cell>24</md-data-table-cell>
                    <md-data-table-cell>4</md-data-table-cell>
                    <md-data-table-cell>87</md-data-table-cell>
                </md-data-table-row>

            </md-data-table>

Hi!
Thanks for this module, it looks fantastic however I'm having the exact same issue as samster here, my simple table looks like this:
<md-data-table
    table-card="{visible: true, title: 'Users'}"
                  selectable-rows="true"
                    sortable-columns="true">
              <md-data-table-header-row>
                  <md-data-table-column>User Name</md-data-table-column>
                  <md-data-table-column>Role</md-data-table-column>
              </md-data-table-header-row>

              <md-data-table-row ng-repeat="user in users | orderBy: 'role'">
                  <md-data-table-cell>{{user.username}}</md-data-table-cell>
                  <md-data-table-cell>{{user.role}}</md-data-table-cell>
              </md-data-table-row>
</md-data-table>

Any help would be appreciated

I'm having the same issue, and after a lot of trial and error I have been able to reproduce it by simply removing the reference to jquery from the demo app. This implies there is a dependency on jquery, but I couldn't figure out why.
Maybe one of the calls to angular.element isn't supported by jqlite? Is that really necessary?

FYI: Working on the issue, give me 1 day and it will be solved.

Sorry guys. Today I'll have a look at it.

@samster @joansanfeliu @samchristie
I cannot reproduce the issue. For me it was working locally.
Can somebody provide me a demo plunkr where I can spot the error?
Thanks.

Tried to create a sample app to reproduce.   Hope it helps.
Example App

Thanks for looking into this. I have been able to recreate it many times but only in a very complex application. I'll see what I can do with the sample app you created. Thanks a lot.
Sam Christie
Sent from<PHONE_NUMBER>
-------- Original message --------From: samster  Date:07/21/2015  11:19 PM  (GMT-05:00) To: iamisti/md-data-table  Cc: samchristie  Subject: Re: [md-data-table] Controller 'mdDataTable', required by directive 'mdDataTableHeaderRow', can't be found! (#24)

Tried to create a sample app to reproduce.   Hope it helps.
Example App

Reply to this email directly or view it on GitHub:
https://github.com/iamisti/md-data-table/issues/24#issuecomment-123550897

I was having this same issue as well.  We use jade and compile to HTML and I discovered that this issue appeared when everything was not nested correctly.  For example, your md-data-table-header-row and md-data-table-row elements need to all be a child of md-data-table.  While this seems obvious, in our case, jade made it less obvious.  As soon as we fixed this, our errors went away and it functions as expected.

Good to hear that. But the others unfortunately had the right structure but it's not working for some reason. I'm did some investigations but I didn't find the cause. I'd be glad if someone of you guys can help me with that issue. Pull requests are welcome here as well.

I ran into the same issue.
I come to the same conclusion as samchristie: from this plunker http://plnkr.co/GhIJYp23Y3XqLAKzqSiL, just comment out the jquery script tag in index.html to reproduce the error.

I ran into the same issue, however if you include jquery before angular it works fine. If you are using bower_concat just add jquery as a dependency to angular.
like this:
dependencies: {'angular':['jquery']},

Make sure to include jQuery and lodash in your index.html - it seems that md-data-table relies on these two libraries.
I added these two lines to my code and the table worked
  <script src="https://code.jquery.com/jquery-2.1.4.js"> </script>
  <script src="https://cdnjs.cloudflare.com/ajax/libs/lodash.js/3.10.1/lodash.min.js"> </script>

@samster Is that solved your issue?

+1

add this line on top i will work

Fixes also electron with v1.8.1


add music sample

generated using https://github.com/rsxdalv/tts-generation-webui

Please tall me, M-GEN and A-GEN can use small checkpoint models? (small-med-large music audio) instead of HUGE (many GB!) diffusers in HF/Hub

Can you help me? Why TTS-generation-webui-with RVC Beta inside can work gradio==3.35.2
but Retrieval_based_....  cannot work gradio>=3.34.0 how to run it with gradio 3.35?

My flight arrives in about 18 hours, I can investigate after that.
On Tue, Dec 5, 2023, 11:09 PM JohnTesla @.***> wrote:

Can you help me? Why TTS-generation-webui-with RVC Beta inside can work
gradio==3.35.2
but Retrieval_based_.... cannot work gradio>=3.34.0 how to run it with
gradio 3.35?
—
Reply to this email directly, view it on GitHub
https://github.com/rsxdalv/musicgen-prompts/pull/1#issuecomment-1840983329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTRXIYOFLRD7UJ36TI5NM3YH42I5AVCNFSM6AAAAAAZDORN2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBQHE4DGMZSHE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Can you help me? Why TTS-generation-webui-with RVC Beta inside can work gradio==3.35.2 but Retrieval_based_Voice_Conversion_WebUI .... cannot work gradio>=3.34.0 how to run it with gradio 3.35?

I remember that I always did some work for adapting the other projects (here - RVC) but it might have been quite, maybe as easy as just installing the newer gradio version. In your case, I'd recommend opening an issue on the RVC project and maybe tagging me there since my project uses RVC with 3.35.2 gradio.

Can you help me? Why TTS-generation-webui-with RVC Beta inside can work gradio==3.35.2 but Retrieval_based_Voice_Conversion_WebUI .... cannot work gradio>=3.34.0 how to run it with gradio 3.35?

I remember that I always did some work for adapting the other projects (here - RVC) but it might have been quite, maybe as easy as just installing the newer gradio version. In your case, I'd recommend opening an issue on the RVC project and maybe tagging me there since my project uses RVC with 3.35.2 gradio.

Thanks! My idea was: to integrate Full RVC inside the TTS-GEN, Because I wanna train RVC also in TTS-GEN Webui.
So RVC full cannot use the same ENV form TTS because of Gradio Ver, so Just need to use a new ENV with gradio 34 for run RVCfull in separated UI

I like your idea. Please create this issue in the tts project
On Sat, Dec 9, 2023, 7:05 AM JohnTesla @.***> wrote:

Can you help me? Why TTS-generation-webui-with RVC Beta inside can work
gradio==3.35.2 but Retrieval_based_Voice_Conversion_WebUI .... cannot work
gradio>=3.34.0 how to run it with gradio 3.35?
I remember that I always did some work for adapting the other projects
(here - RVC) but it might have been quite, maybe as easy as just installing
the newer gradio version. In your case, I'd recommend opening an issue on
the RVC project and maybe tagging me there since my project uses RVC with
3.35.2 gradio.
Thanks! My idea was: to integrate Full RVC inside the TTS-GEN, Because I
wanna train RVC also in TTS-GEN Webui.
So RVC full cannot use the same ENV form TTS because of Gradio Ver, so
Just need to use a new ENV with gradio 34 for run RVCfull in separated UI
—
Reply to this email directly, view it on GitHub
https://github.com/rsxdalv/musicgen-prompts/pull/1#issuecomment-1848235160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTRXI2XQBWY4UUZJ5W6TEDYIPWTLAVCNFSM6AAAAAAZDORN2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBYGIZTKMJWGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Please tell some more Man. If I want to TTS using trained RVC model (pth+index) how can I do this directly, avoid of generating some speech then transfer to RVC gen

Was planning for that, but you can make a feature request for it in the
main repo.
For background - before I went on a holiday I was busy with migratong away
from gradio, which I need to build more complex pipelines and better UX;
now, I'm still returning.
On Sat, Dec 9, 2023, 10:07 AM JohnTesla @.***> wrote:

Please tell some more Man. If I want to TTS using trained RVC model
(pth+index) how can I do this directly, avoid of generating some speech
then transfer to RVC gen
—
Reply to this email directly, view it on GitHub
https://github.com/rsxdalv/musicgen-prompts/pull/1#issuecomment-1848319143,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTRXIYEPRZ52QRSJUVIQXDYIQL3ZAVCNFSM6AAAAAAZDORN2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBYGMYTSMJUGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Automatic metadata fetching

Would be cool to automatically fetch title and description fields, as well as preview image.
Similar to how the Android app is doing this.

No preview file?
$ unzip link.zip
Archive:  link.zip
 extracting: link.json

No description:
$ cat link.json | jq
{
  "title": "Solana TL;DR",
  "desc": "",
  "url": "https://2501babe.github.io/posts/solana101.html?accessToken=eyJhbGciOiJIUzI1NiIsImtpZCI6ImRlZmF1bHQiLCJ0eXAiOiJKV1QifQ.eyJhdWQiOiJhY2Nlc3NfcmVzb3VyY2UiLCJleHAiOjE2NDQ0ODUzOTUsImciOiI4QXcyeDJNQTJrUVVYdnYwIiwiaWF0IjoxNjQ0NDg1MDk1LCJ1c2VySWQiOi0xNzQ4NDU1ODc5fQ.poTnBB48mQjZhiun-HoSCPNG5pXLBoup42GxUMuSiMI",
  "created_time": {
    "secs_since_epoch":<PHONE_NUMBER>,
    "nanos_since_epoch": 598406590
  }
}

Solved in #11


Documentation: New extended tutorial for Liqo.

Description
This pull request provides a more in-depth tutorial showing how to use Liqo's main features.
The tutorial is inserted in the /docs/pages/gettingstarted section with the already present "hello world" tutorial.

/ok-to-test

/doc-build

/doc-build

/test


Feedback on rrtools

I was excited to see that you featured rrtools in your workshop! I would love to know what feedback you have about it. For example, if there were any points of confusion or friction experienced by the participants? I often make minor changes to the documentation after giving workshops myself to try and make it a smoother experience, especially for beginners. If you have any suggestions I'd be happy to know about them. Thank you so much.

I don't remember specific feedbacks about rrtools. I remember opening a few issues about friction points we found; but I don't remember which. I guess if you filter issues opened by me and see a few clustered together, those are them.  We'll probably revise a few things and I will let you know if we have any feedback after the course.

Hi! Last month we lead a second version of this workshop. The feedback around rrtools was that learners didn't get why it was useful. It might've been that I we didn't explained it correctly
I did followed the steps myself and I found a few confusing issues. For example, creating the readme also creates a dockerfile for the beamer thing. But then creating a dockerfile is a second step, which creates a different dockerfile? The Readme also seems to have hardcoded the location of the analysis folder, so it sometimes it's not consistent with the real location.
I can run the steps again and open specific issues if you want me to.


Omit mkPolyline from tvOS 8, watchOS builds

Fixed the following compilation errors on tvOS and watchOS introduced by #39:
/path/to/Polyline/Polyline/Polyline.swift
/path/to/Polyline/Polyline/Polyline.swift:59:28: 'MKPolyline' is only available on tvOS 9.2 or newer
/path/to/Polyline/Polyline/Polyline.swift:59:28: Add @available attribute to enclosing var
/path/to/Polyline/Polyline/Polyline.swift:59:28: Add @available attribute to enclosing struct
/path/to/Polyline/Polyline/Polyline.swift:61:26: 'MKPolyline' is only available on tvOS 9.2 or newer
/path/to/Polyline/Polyline/Polyline.swift:61:26: Add 'if #available' version check
/path/to/Polyline/Polyline/Polyline.swift:61:26: Add @available attribute to enclosing var
/path/to/Polyline/Polyline/Polyline.swift:61:26: Add @available attribute to enclosing struct

/path/to/Polyline/Polyline/Polyline.swift
/path/to/Polyline/Polyline/Polyline.swift:59:28: Use of undeclared type 'MKPolyline'
/path/to/Polyline/Polyline/Polyline.swift:61:26: Use of unresolved identifier 'MKPolyline'
/path/to/Polyline/Polyline/Polyline.swift:59:16: Did you mean 'mkPolyline'?
/path/to/Polyline/Polyline/Polyline.swift:41:15: Did you mean 'Polyline'?

/cc @tomtaylor @frederoni

@tomtaylor @raphaelmor, can you take a look? (For what it’s worth, I’d be happy to help out as a collaborator, since I maintain some libraries that depend on this library.)

I'm adding you as a collaborator, if @tomtaylor is fine with, the more the merrier (and I can't contribute anymore :( )

Absolutely!

On 9 Mar 2017, at 14:39, Raphaël Mor<EMAIL_ADDRESS>wrote:
I'm adding you as a collaborator, if @tomtaylor is fine with, the more the merrier (and I can't contribute anymore :( )
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks!


Can't get constant interpolation

When I'm using a bone to adjust the UVs, I don't want that bone to interpolate between keyframes. But the interpolation is unavoidable with the current importer.
Example: Say I want to make a bone position decides the expression of a character. If the bone doesn't jump exactly to the position of the next UV, the texture will be displayed in an incorrect position.

As a workaround, I've created a script to help me remove interpolation. See AnimationEditTools:
https://github.com/guzuligo/PlayCanvas-Tools/tree/master/AnimationTools
It is a simple script that adds a frame in-between to prevent the interpolation. It was a suggestion by "Steven Yau".


Added Tapas

This PR adds an icon from Tapas, weighed at 700 bytes! SVG has already been shrunk down into one path and has been rounded to, at most, two decimal points.

Sorry for the delay. Looking at their website, they seem to use a word-logo, or this one https://tapas.io/favicon.ico


Scream does not compile with Jack support when enabled

Issue description
Scream does not compile with jack support when it is explicitly enabled through an overlay.
@arcnmx @r-ryantm @AndersonTorres
Steps to reproduce
  nixpkgs.overlays = [     (final: prev: {       scream = prev.scream.override { jackSupport = true; };     }) ]

Error message?

Error message?

There is no error, it just skips compiling it.
someone else ran it and i think they said that it skips compiling it with jack support because soxr is missing
-- Checking for module 'jack' --   Found jack, version 1.9.19 -- Checking for module 'soxr' --   No package 'soxr' found

See #224493. In the meantime, you can try:
nixpkgs.overlays = [ (final: prev: {
  scream = (prev.scream.override { jackSupport = true; }).overrideAttrs (old: {
    buildInputs = old.buildInputs ++ [ final.soxr ];
  });
}) ]


Broken link

The gnomAD v4.0.0 README link on this page:
https://gnomad.broadinstitute.org/downloads#v4-variants
is broken.
It points here:
https://storage.googleapis.com/gcp-public-data--gnomad/release/4.0/README.txt
Which gives some kind of fancy 404 ;-)

Hiya, thanks for filing this issue!
This was my bad, we actually don't have a README, favoring the blog post instead. This link will be removed from the site since a README doesn't exist.
Thanks!


Update endpoints

Description
Logs rest endpoints changes From logs/rest/singles → logs/v1/singles
How Has This Been Tested?

Checklist:

[ ] I have updated the relevant component changelog(s)
[ ] This change does not affect any particular component (e.g. it's readme or docs change)

@royfur should you maybe change all these, or would it be too much?
λ git grep 'logs/rest'
aws-integrations/ecs-fargate/ecs-fargate-cf.yaml:              URI: /logs/rest/singles
aws-integrations/ecs-fargate/ecs-fargate-cf.yaml:              URI: /logs/rest/singles
fluentbit/ecs-fargate/README.md:            "URI": "/logs/rest/singles",
fluentbit/ecs-fargate/template.yaml:              URI: "/logs/rest/singles"

λ git grep 'private_key'
aws-integrations/ecs-fargate/ecs-fargate-cf.yaml:              Header: !Sub "private_key ${PrivateKey}"
aws-integrations/ecs-fargate/ecs-fargate-cf.yaml:              Header: !Sub "private_key ${PrivateKey}"
aws-integrations/lambda-integrations/archive-reader/template.yaml:          private_key:
aws-integrations/lambda-integrations/athena-s3/template.yaml:          private_key:
aws-integrations/lambda-integrations/cloudtrail-sns/template.yaml:          private_key:
aws-integrations/lambda-integrations/cloudtrail/template.yaml:          private_key:
aws-integrations/lambda-integrations/cloudwatch-logs/README.md:IF you want to use **AWS Secrets** to store the private_key, first you need to deploy Coralogix SecretLayer form AWS Serverless Repository.
aws-integrations/lambda-integrations/cloudwatch-logs/template.yaml:          private_key:
aws-integrations/lambda-integrations/cloudwatch-metrics/template.yaml:          private_key:
aws-integrations/lambda-integrations/ecr/template.yaml:          private_key: !Ref ApiKey
aws-integrations/lambda-integrations/elasticsearch-reporter/template.yaml:          private_key:
aws-integrations/lambda-integrations/kafka/template.yaml:          private_key:
aws-integrations/lambda-integrations/kinesis/template.yaml:    Description: store coralogix private_key as a secret. True/False
aws-integrations/lambda-integrations/kinesis/template.yaml:          private_key:
aws-integrations/lambda-integrations/lambda-secretLayer/README.md:## Coralogx Layer for SSM private_key
aws-integrations/lambda-integrations/msk/template.yaml:    Description: store coralogix private_key as a secret. True/False
aws-integrations/lambda-integrations/msk/template.yaml:          private_key:
aws-integrations/lambda-integrations/resource-metadata/template.yaml:          private_key:
aws-integrations/lambda-integrations/resource-tags/template.yaml:          private_key:
aws-integrations/lambda-integrations/s3-sns/template.yaml:          private_key:
aws-integrations/lambda-integrations/s3/template.yaml:          private_key:
aws-integrations/lambda-integrations/sns/template.yaml:          private_key:
aws-integrations/lambda-integrations/vpc-flow-logs/template.yaml:          private_key: !Ref ApiKey
fluentbit/ecs-fargate/README.md:            "Header": "private_key <YOUR PRIVATE KEY>",
fluentbit/ecs-fargate/template.yaml:              Header: !Sub "private_key ${PrivateKey}"
opentelemetry/ecs-ec2-windows/template.yaml:                      private_key: "$PRIVATE_KEY"
opentelemetry/ecs-ec2/components.md:    private_key: '<your-product-key>'
opentelemetry/ecs-ec2/components.md:    private_key: '<your-product-key>'
opentelemetry/ecs-ec2/template.yaml:            private_key: "$PRIVATE_KEY"
opentelemetry/ecs-ec2/template.yaml:            private_key: "$PRIVATE_KEY"
prometheus/ecs-ec2/autodiscovery/README.md:            private_key: "${CX_TOKEN}"
prometheus/ecs-ec2/autodiscovery/template.yaml:            private_key: "${CX_TOKEN}"

@zhrebicek
the change was done for aws-integrations/ecs-fargate/ecs-fargate-cf.yaml in this PR already
fluentbit/ecs-fargate/ is deprecated
aws-integrations/lambda-integrations* currently we cant change since it use the SDK once the new lambda will be release this will be solved
opentelemetry/* not sure what change is needed this is otel and not using the rest API
prometheus/ecs-ec2/ not sure what change is needed this is not using the rest API


Fail to load the weather-app, city-service, gateway, weather-service pages

Description
After running azd up, click the link of the weather-app service, the page is not as expected as follows:

weather-app:

In addition, the pages of the other three services cannot be loaded successfully, as follows:

city-service:

gateway:

weather-service:

Repro steps

Run azd auth login.
Run azd init --template Azure-Samples/java-on-aca-quarkus.
Ensure that Docker is running in your environment.
Run azd up.
Click the service link.

Environment

azd version 1.10.3 (commit 0595f33fe948ee6df3da492567e3e7943cb9a733)
OS: Windows

Expected behavior
The weather-app, city-service, gateway, weather-service pages can be loaded successfully.
@majguo and @hemarina for notification.

@zimuli157 The root cause is that the shell scripts cloned to the windows system have a different end-of-line and can't be executed in a Linux container. Fixed in commit b65b6b0 which will force shell scripts cloned to windows system keep Linux required end-of-line.

It has been fixed and we will close this issue.


Fix Gunicorn Concurrency Deadlock

It appears that something with how Gunicorn is configured is giving a deadlock on the app. We need to determine how to allow for multiple requests at once as currently it's not possible.

This was fixed via https://github.com/Justintime50/harvey/commit/dcd7497dc57cd82042608c1ea4898d06f0b88d6a

This appears to still hang after all...

The plot thickens. I switched over to uwsgi and the deadlocking still happened which leads me to believe that maybe this has nothing to do with that? Very odd. The only other thing I can think of is that nginx is the only other thing different in production vs locally (as this problem only happens in prod).


Inconsistent capitalization of file names in notebooks.

Chapter 2.4 images do not show up in linux, because markdown image links are uppercase, files are lowercase.

We should change all filenames to be all lowercase.


Fri talks take 2

Waiting on a smaller version of Joel's talk, but go ahead and merge for now.

All talks added @osg-cat @brianhlin.


Cleanup resources and add OpenShift specific test configs.

Also fix all pep8 reported warnings.

Acked-by: Lorenzo Bianconi<EMAIL_ADDRESS>


Add insecure mode for wip-catalog chart

To help move things along until we have SSL stuff in place working with the aggregator.

Why?

I ask, as I was told not to even consider using the insecure options.

@MHBauer good question. This is just for development only, and will be removed as soon as we can do so and have an easy in-cluster dev/test workflow

I restarted the stalled build in travis

I'm merging this now, as it's needed to unblock @pmorie and my testing efforts for crucial MVP1 functionality


Fix script bracket parsing and add line number to usage list.

Hi, this is part two of the CPP files patches to fix problems when running the unittests in a QMLUI environment under Linux.
This patch addresses the new scriptv4 "functionlist" handling which lacked the second information in the list, which is the line number. The doc.cpp failed with an index out of bound error, which is now leading to a debug message. The keywords and translation from legacy to v4 is now more explicit between the versions.

Looks good to me, thanks :+1:


Fix cwd handling/binding, including fugitive buffers

Includes/based on https://github.com/neomake/neomake/pull/1177.

@Neki
Sorry, clicked the suggestion from GitHub for reviewing accidentally.

Is it possible to specify cwd as a maker property?

@tpict
Yes, but not really related to this PR, is it?


CircleCIビルド廃止

issue #163

READMEからバッジ削除
CIrcleCI設定削除
.github/runner.dからCircleCI向け設定ファイル生成削除

Codecov Report

Merging #178 (d1e88c7) into master (3833af4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #178   +/-   ##
=======================================
  Coverage   97.28%   97.28%
=======================================
  Files          39       39
  Lines        2099     2099
=======================================
  Hits         2042     2042
  Misses         57       57

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Added job-role-prediction to the projects folder

This project predicts the job-role based on the resume that is uploaded

Thanks for your contribution ❤