Issue with password

I think this is due to our usual password?

TASK [Add undercloud_host after setting python interpereter (RHEL 8)] ********************************************************************************************************************************************************************************************************************
fatal: [localhost]: FAILED! => {"msg": "The conditional check ''interpreter' in ping_output.msg' failed. The error was: error while evaluating conditional ('interpreter' in ping_output.msg): Unable to look up a name or access an attribute in template string ({% if 'interpreter' in ping_output.msg %} True {% else %} False {% endif %}).\nMake sure your variable name does not contain invalid characters like '-': argument of type 'StrictUndefined' is not iterable\n\nThe error appears to have been in '/home/jtaleric/Code/Work/jetpack/tasks/prepare_inventory.yml': line 14, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Add undercloud_host after setting python interpereter (RHEL 8)\n  ^ here\n"}

This is due to some python3 nuances. This will be addressed in issue #35


fresh player spawning separate window

ever since the 100x100 px version fresh player has been spawning a separate window instead of getting full-screen resolution
seems to do it when i switch out of a firefox tab then switch back.
see screenshot
http://i.imgur.com/ZjcMBMF.jpg

fresh player has been spawning a separate window instead of getting full-screen resolution

This is how it's actually done. I create window, set some properties for it. Then window manager detects those changes and makes the window full screen, i.e. maximizes it, and removes decorations. I set the window transient for browser window, just like open file dialogs do. So it feels like browser window itself goes fullscreen.
But for some reason Compiz removes _NET_WM_STATE_FULLSCREEN property from a window occasionally. I still don't understand why it does so, and what actual reasons are.

will there be a fix for this?

will there be a fix for this?

Yes, as soon as I discover magic sequence of actions that Compiz will accept :-)
By the way, check recent version, I've changed sequence in 3a9747e42cb9009c036a6104e57ae9d7392530e1 and 4788328f4247e513d00d3cf7872f632ab3e5df7c.

most recent version seems to be fixed.
thanks very much.


Warnings on fresh install

I'm trying out Elder.js and on the default setup of a project there is a lot of noise, including build warnings (errors?) at build:
> npm run build

><EMAIL_ADDRESS>build
> node ./src/cleanPublic.js && npm run build:rollup && npm run build:html

 Clearing out public folder.

><EMAIL_ADDRESS>build:rollup
> rollup -c

CSS file not found in /home/alan/src/elderjs-testing/public/_elderjs/assets
Elder.js using rollup in production mode.

src/layouts/*.svelte, src/routes/**/*.svelte, src/components/**/*.svelte → ___ELDER___/compiled...
Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db

Why you should do it regularly:
https://github.com/browserslist/browserslist#browsers-data-updating
(!) Plugin replace: @rollup/plugin-replace: 'preventAssignment' currently defaults to false. It is recommended to set this option to `true`, as the next major version will default this option to `true`.
(!) Plugin rollup-plugin-elder: Layout has unused export property 'settings'. If it is for external reference only, please consider using `export const settings`
src/layouts/Layout.svelte
1: <script>
2:   import style from '../../assets/style.css';
3:   export let templateHtml, settings;
                              ^
4: </script>
5:
(!) Import of non-existent export
src/layouts/Layout.svelte
1: <script>
2:   import style from '../../assets/style.css';
            ^
3:   export let templateHtml, settings;
4: </script>
created ___ELDER___/compiled in 1.1s

src/components/**/*.svelte → public/_elderjs/svelte...
created public/_elderjs/svelte in 1.8s

><EMAIL_ADDRESS>build:html
> node ./src/build.js

[░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░] 0% | ETA: 0s | 0/19 | Errors: 0 | P/sec: 0
Build Completed Successfully: Built 19 pages in 1.36 seconds
No SEO issues detected.

I installed using npx degit Elderjs/template elderjs-testing/ and I'm using:
node v18.10.0
npm 8.19.2

import style from '../../assets/style.css';

This thing is really frustrating and should be fixed in the template:

Why do we have to import a default member from CSS file? What does it even mean?
It is already frustrating enough to import a file outside of src.
There is already a hook copying files from the assets folder into the dist folder. Why do we have to import it?

I suggest:

Rename the style to src/layouts/global.css.
Change the import statement to import './global.css';.
Put some pictures in assets/ and display them on the homepage to demonstrate copyAssetsToPublic hook.


installed_extras npm "Failed to validate the SSL certificate for deb.nodesource.com:443"

#844 was closed citing Drupal Console, but the issue is still occurring and isn't related to Drupal Console.
If you add npm to installed_extras using ansible_local on 3.2.1 with geerlingguy/ubuntu1404, you get the following error:
TASK [geerlingguy.nodejs : Add Nodesource apt key.] **************************** fatal: [nyu-wagner.local]: FAILED! => {"changed": false, "failed": true, "msg": "Failed to validate the SSL certificate for deb.nodesource.com:443. Make sure your managed systems have a valid CA certificate installed. If the website serving the url uses SNI you need python >= 2.7.9 on your managed machine or you can install the urllib3, pyopenssl, ndg-httpsclient, and pyasn1python modules to perform SNI verification in python >= 2.6. You can use validate_certs=False if you do not need to confirm the servers identity but this is unsafe and not recommended. Paths checked for this platform: /etc/ssl/certs, /etc/pki/ca-trust/extracted/pem, /etc/pki/tls/certs, /usr/share/ca-certificates/cacert.org, /etc/ansible"}
Based on that error, the Python version and/or lack of python modules looks to be the culprit.
python --version
Python 2.7.6

python-urllib3 is already installed, python-pyasn1 and python-openssl aren't installed, but python-ndg-httpsclient doesn't look like it's available as a package in Ubuntu 14.04 LTS.

I am having same issue, I am running geerlingguy/ubuntu1204.

Working on a fix in upstream role https://github.com/geerlingguy/ansible-role-nodejs — apparently Nodesource recently switched how they were hosting and delivering SSL using SNI, which broke the key distribution on older distros like CentOS 6 and Ubuntu 12.04.

Ubuntu versions are fixed... I might mark the CentOS < 7 support as temporarily deprecated because I can't find a quick way to fix it. I'll tag a new upstream release. See for CentOS 6: https://github.com/geerlingguy/ansible-role-nodejs/issues/46


Linechart of percentages with max value 100%

I'm doing a line chart with percentages values, and I'd like to set 100% as maximum value in the Y axis, but when using forceNiceScale, the maximum value is auto generated and sometimes is higher than 100%, which for this kind of visualization is nonsense.
Without forceNiceScale, if I set the maximum value, the values of the ticks in the Y axis get weird values, in an irregular scale.
Example of a percentage visualization in codesandbox
I'd like to know if there is a property for working with percentage visualizations, that fixes some of this issues.

yaxis: {
  forceNiceScale: false,
  max: 100,
  labels: {
    formatter: (value) => value.toFixed(0) +'%',
  },
}

Your answer solves the problem with the amount of decimals in the tick values. But the following issue continues, by doing the fix you proposed, the ticks in the y axis have "irregular" values, like 81% instead of 80%. I'd like that this ticks have cleaner values, like the ones achieved by using forceNiceScale.
One solution I tried was defining the step manually, as well as the maximum and minimum value of the y axis, but the step ignores my defined range, and calculates the ticks based on the range of the given data.

Yeah, tickAmount along with min and max would do the trick.
min: 50
max: 100,
tickAmount: 10

Thanks, that solved my issue!


[FEATURE REQUEST] - Standalone Overseer (even if forked from MT)

Is your feature request related to a problem? Please describe.
Overseer is extremely useful but many people think it should be it's own thing. I personally play SSF so I don't use the MT aspects anymore but would use Overseer.
Describe the solution you'd like
Split Overseer and MT into two separate apps.
Additional context
Not much else. Would it be intensive, I wonder?

I will move that issue to latest release. It will require massive amout of work to be done so I'm not sure if it will be done soon

Sorry, that feature requires too much amount of work so it won't be done in near future. If you want to try do it by yourself you are welcome to do it and push a pull request


Repackage LoadBalancerProperties

The class contains properties for both reactive and blocking implementations, so should be moved up from reactive package.

Done with https://github.com/spring-cloud/spring-cloud-commons/commit/c6bd75eb5db89b6107bef187f3eb402d7384437f.


[DMN Designer] Update sample on online editor

See https://issues.redhat.com/browse/KOGITO-1394

I've opened this PR in the wrong branch. Please ignore it.


Running tests on monorepo requires cleaning dist/test-out when using @angular-devkit/build-angular:jest

Command
test
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Since compiled tests are always moved to the same folder "dist/test-out", running tests from different projects in a monorepo requires cleaning the dist/test-out folder manually first. Exposing an option for the output dir, or putting the compiled specs on a folder relative to each subproject could solve the issue.
Minimal Reproduction
ng test <project 1>
ng test <project 2>
Exception or Error
No response
Your Environment
Angular CLI: 16.0.0
Node: 18.16.0
Package Manager: yarn 3.5.0
OS: darwin x64

Angular: 16.0.1
... animations, common, compiler, compiler-cli, core, forms
... localize, platform-browser, platform-browser-dynamic, router
... service-worker

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1600.0
@angular-devkit/build-angular   16.0.0
@angular-devkit/core            16.0.0
@angular-devkit/schematics      16.0.0-rc.1
@angular/cdk                    16.0.0
@angular/cli                    16.0.0
@angular/flex-layout            15.0.0-beta.42
@angular/material               16.0.0
@angular/pwa                    16.0.0
@schematics/angular             16.0.0
ng-packagr                      16.0.1
rxjs                            7.8.1
typescript                      5.0.4

Anything else relevant?
No response

Yeah, we're not quite at monorepo support yet. Eventually I'm hoping to move the test-out folder to some temporary directory which gets cleaned up afterwards so it won't even be in the project structure at all.

Could you parameterize the outPut dir as an option for the builder? Then we might be able to solve this and a workaround for https://github.com/angular/angular-cli/issues/25188 could also rely on the outDir, as long as no two files have the same name within each project.

Once we move the output outside of the project directory, then this won't be an issue at all. No need for users to define an outputDir at all.

That would be great. Thank you for your work on this. I have been using jest with Angular for a while now using the jest-preset-angular project, but it's been a nightmare so far when combined with esm, monorepos and IDE integration. Do you have a ballpark timeframe for this move of the output folder? It's preventing me from using the builder, for the time being.

This is all experimental right now, so there's a lot of work left to be done. Even if the output directory wasn't an issue, I still wouldn't recommend general usage of the builder yet.

do we need additional setup for code-coverage

Duplicate of #25308


The box "name" is already open and of type Box.

The following HiveError was thrown while handling a gesture:
The box "recentsearch" is already open and of type Box<dynamic>.

When the exception was thrown, this was the stack:
#0      HiveImpl._getBoxInternal (package:hive/src/hive_impl.dart:142:9)
#1      HiveImpl.box (package:hive/src/hive_impl.dart:157:33)

This happens when I call Hive.box('boxname') when it has already been opened.
But this should not at all happen when calling box method as it's supposed to be already open.
I am using the current github master branch code of hive.
commit link : https://github.com/hivedb/hive/commit/d7ee72537a3b7ce1284bcbf5ee5f86e617a4f6fc

This issue happens in _getBoxInternal method in the below line :
if ((lazy == null || box.lazy == lazy) && box.valueType == E) {

box.valueType != E is getting true
Basically this happens when :

we open the box by specifying the generic type first time when opening the box.
the next time when we open and don't specify the generic type , this happens.

Whenever you open a box, you should specify the type, even if it's dynamic.

Is this requirement documented?

I don't know, but it's what I've learned.

But the error shown is completely unrelated. This is something that needs fixing.

How is this a separate issue? You're saying that when you open the box you're specifying a generic type but when you're getting the box youre not. If you've specified a generic type when you opened it, you need to do it everywhere you use Hive.box(). This is stated in the documentation.

The error needs to show the same message isn't it?
In this scenario error says box is already open when we do Hive.box() which is not the case.

The error states that you have already opened the box with a different type than when youre getting it.

Well, I think it's not clear enough.
My team couldn't relate this error to the mistake of not Specifying the generic type at all.

As @Reprevise said, you must always specify box generic type.

I agree with @Reprevise I think this should be better documented. same issue here.

As @Reprevise said, you must always specify box generic type.

how do I do that?

The following HiveError was thrown while handling a gesture:
The box "recentsearch" is already open and of type Box<dynamic>.

When the exception was thrown, this was the stack:
#0      HiveImpl._getBoxInternal (package:hive/src/hive_impl.dart:142:9)
#1      HiveImpl.box (package:hive/src/hive_impl.dart:157:33)

This happens when I call Hive.box('boxname') when it has already been opened.
But this should not at all happen when calling box method as it's supposed to be already open.
I am using the current github master branch code of hive. commit link : d7ee725

I got the same problem and it was that I forgot to add a type of box in main
Before : await Hive.openBox('blog');
After : await Hive.openBox('blog');
it was dynamic then I specify it

Using hive is probably the most exhausting thing for just a simple local database.
It has a unique capability to show an error in every possible scenario.

Using hive is probably the most exhausting thing for just a simple local database.
It has a unique capability to show an error in every possible scenario.

I second this opinion.

Using hive is probably the most exhausting thing for just a simple local database.
It has a unique capability to show an error in every possible scenario.

I second this opinion.

I take this back bro. I made some problem there. Now it's working great. Infact I always use hive till now whenever I needed. It's extremely great 👍

This issue happens in _getBoxInternal method in the below line :
if ((lazy == null || box.lazy == lazy) && box.valueType == E) {

box.valueType != E is getting true
Basically this happens when :

we open the box by specifying the generic type first time when opening the box.
the next time when we open and don't specify the generic type , this happens.

Thanks a ton, saved me so much agony!


Unable to create instance with knife.

Starting from a blank CentOS 7 (PVHVM 512MB Standard Instance)
ssh -l root serverip
yum groupinstall 'Development Tools'
yum install ruby ruby-devel libxml2 libxml2-devel
gem install chef
gem install knife-rackspace
adduser knifeuser
su - knifeuser
mkdir .chef
chmod 700 .chef
cat << EOF > ~/.chef/knife.rb
knife[:rackspace_api_username] = "apiuser"
knife[:rackspace_api_key] = "apikey"
knife[:rackspace_region] = "DFW"
EOF
knife rackspace server create -VV -I 8aac6fb5-4bd3-4256-bf6e-ff8500bf60cd
To which I get the output of:
INFO: Using configuration from /home/knifeuser/.chef/knife.rb
DEBUG: version  (config)
DEBUG: version v2 (cli)
DEBUG: rackspace_api_key apikey
DEBUG: rackspace_username
DEBUG: rackspace_api_username apiuser
DEBUG: rackspace_auth_url
DEBUG: rackspace_auth_url
DEBUG: rackspace_auth_url https://identity.api.rackspacecloud.com/v2.0 (using)
DEBUG: rackspace_region DFW
DEBUG: rackspace_region DFW
DEBUG: rackspace v2
DEBUG: https_proxy  (config)
DEBUG: http_proxy  (config)
DEBUG: using proxy  (config)
DEBUG: ssl_verify_peer  (config)
Instance ID: a75bc74f-3930-4ac8-9d6f-ff48a503a5ec
Name: rs-00203265389326035
/usr/local/share/gems/gems/fog-1.27.0/lib/fog/rackspace/core.rb:145:in escape': undefined method gsub' for 2:Fixnum (NoMethodError)
from /usr/local/share/gems/gems/fog-1.27.0/lib/fog/rackspace/requests/compute_v2/get_flavor.rb:26:in get_flavor' 	from /usr/local/share/gems/gems/fog-1.27.0/lib/fog/rackspace/models/compute_v2/flavors.rb:31:in get'
from /usr/local/share/gems/gems/fog-1.27.0/lib/fog/rackspace/models/compute_v2/server.rb:327:in flavor' 	from /usr/local/share/gems/gems/knife-rackspace-0.10.0/lib/chef/knife/rackspace_server_create.rb:379:in run'
from /usr/local/share/gems/gems/chef-12.0.3/lib/chef/knife.rb:417:in block in run_with_pretty_exceptions' 	from /usr/local/share/gems/gems/chef-12.0.3/lib/chef/local_mode.rb:38:in with_server_connectivity'
from /usr/local/share/gems/gems/chef-12.0.3/lib/chef/knife.rb:416:in run_with_pretty_exceptions' 	from /usr/local/share/gems/gems/chef-12.0.3/lib/chef/knife.rb:213:in run'
from /usr/local/share/gems/gems/chef-12.0.3/lib/chef/application/knife.rb:139:in run' 	from /usr/local/share/gems/gems/chef-12.0.3/bin/knife:25:in <top (required)>'
▽
from /usr/local/bin/knife:23:in load' 	from /usr/local/bin/knife:23:in '

This is throwing an error here: https://github.com/fog/fog/blob/master/lib/fog/rackspace/core.rb#L145
It looks like fog.io updated this recently too. I'm betting the issue lies with fog not this gem. But i'd like someone else to reconfirm this.

I've begun testing it by downgrading fog then removing the newer version (so it would stop using it)
So far I've recreated the issue on:
fog versions:
1.27.0
1.26.0
1.25.0
This would be easier if I had a version number to test against. Any idea when the change came out?

Utilizing fog I am able to create a machine:

It looks like it's an issue w/ passing the flavor ID as an integer vs a string, which makes it interesting that you can do it directly in fog w/ :flavor_id => 2 rather than :flavor_id => "2".
However, before you go any further you might want to take a look at the list of supported server flavors. In case you're not aware, flavor ID "2" (for "512 MB Standard Instance") is "being phased out and should not be used for new servers". I recommend updating to the smallest general flavor: "general1-1", a "1 GB General Purpose v1".
@jjasghar actually I see this is coming from a default value in this gem, so it probably makes sense to change the default value at https://github.com/chef/knife-rackspace/blob/c05cf2161bac8a010238e7d4fe7b492d0c760989/lib/chef/knife/rackspace_server_create.rb#L49 from 2 to "general1-1".

I can confirm:
The following commands get past the error above using rackspace-knife(0.10.0) and fog(1.27.0):
[wlightning@test-management-server ~]$ knife rackspace server create -f 2 -I 8aac6fb5-4bd3-4256-bf6e-ff8500bf60cd
Instance ID: 7de55efa-3a5e-4777-8c15-d7d8dff0f4d1
Name: ..removed..
Flavor: 512MB Standard Instance
Image: CentOS 6 (PVHVM)
Metadata: []
ConfigDrive: false
RackConnect Wait: no
ServiceLevel Wait: no
.................................................Metadata: []
Waiting server
Public DNS Name: ..removed...
Public IP Address: ..removed..
Private IP Address: ..removed..
Password: ..removed..
Metadata: []
Waiting for sshddone
Connecting to ..removed..
ERROR: Errno::ENOENT: No such file or directory - /etc/chef/validation.pem
[wlightning@test-management-server ~]$
This appears to be purely a problem with the default behavior. I also successfully passed a -f "2". This purely seems to be an issue when no favor id is passed.


[Event Request] codeunit 5343 "CRM Sales Order to Sales Order", procedure CopyBillToInformationIfNotEmpty

Can you please add
    [IntegrationEvent(false, false)]
    local procedure OnBeforeCopyBillToInformationIfNotEmpty(CRMSalesorder: Record "CRM Salesorder"; var SalesHeader: Record "Sales Header"; var IsHandled: Boolean)
    begin
    end;

to codeunit 5343 "CRM Sales Order to Sales Order"
    local procedure CopyBillToInformationIfNotEmpty(CRMSalesorder: Record "CRM Salesorder"; var SalesHeader: Record "Sales Header")
    var
        IsHandled: Boolean;
    begin
        OnBeforeCopyBillToInformationIfNotEmpty(CRMSalesorder, SalesHeader, IsHandled);
        if IsHandled then
            exit;

        // If the Bill-To fields in CRM are all empty, then let NAV keep its standard behavior (takes Bill-To from the Customer information)
        if ((CRMSalesorder.BillTo_Line1 = '') and
            (CRMSalesorder.BillTo_Line2 = '') and
            (CRMSalesorder.BillTo_City = '') and
            (CRMSalesorder.BillTo_PostalCode = '') and
            (CRMSalesorder.BillTo_Country = '') and
            (CRMSalesorder.BillTo_StateOrProvince = ''))
        then
            exit;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10963.


docs/install: reference correct stack

https://github.com/StackStorm/st2workroom/blob/master/stacks/st2.yaml#L21 has no stack st2express, but does have st2.
Fixes the following first install issue:
[logikal] git clone https://github.com/StackStorm/st2workroom.git st2workroom
cd st2workroom
vagrant up st2expressCloning into 'st2workroom'...
remote: Counting objects: 6520, done.
remote: Compressing objects: 100% (49/49), done.
remote: Total 6520 (delta 18), reused 0 (delta 0), pack-reused 6471
Receiving objects: 100% (6520/6520), 14.98 MiB | 1.10 MiB/s, done.
Resolving deltas: 100% (3880/3880), done.
Checking connectivity... done.
[logikal] cd st2workroom
[st2workroom] vagrant up st2express                                                                                master
Installing the 'dotenv' plugin. This can take a few minutes...
Installed the plugin 'dotenv (2.0.2)'!
Installing the 'deep_merge' plugin. This can take a few minutes...
Installed the plugin 'deep_merge (1.0.1)'!
The machine with the name 'st2express' was not found configured for
this Vagrant environment.
vagrant up st2express  27.32s user 4.50s system 48% cpu 1:06.07 total

Thank you so much! We're right in the middle of doc review, but his is great to reduce confusion.


Fix/use hatchling as build system

What is the current behavior?

Issue URL:
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (1bc4041) to head (789c070).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #235   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines           55        55
=========================================
  Hits            55        55

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


Add file list

add file list & wget command
addresses issue #1

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


Support Windows

increase cmake path length to 500 in win32
set Catch2 dependency as public for qtgql dependants

TODO: fix testutils.cpp bad initializers
resolves #227

Would you mind to add windows-latest here:
https://github.com/omortie/qtgql/blob/fbbd3f194c2a3c14bf17b884f45e5d61fa42a42f/.github/workflows/scripts/create_testcases_matrix.py#L23

it seems the problem that many test cases in windows are failing is the different behavior of argparse in Windows and Linux
because in Windows the processed argparse of a JSON array object will remove double quotes " from the test case names here: https://github.com/qtgql/qtgql/blob/e74752240e1d5440c5522d35bf116c8ca11f2716/tests/test_codegen/generate.py#L27 and json.loads() will not able to process the results in args.testcases_names_json
I'm investigating the ways to pass the test case names to the generate method in windows without problem

Thanks for the update :upside_down_face:

OK from what I can see you moved the tests folder to tests/gen but you deleted all the previous tests so you need to restore them and move them to tests/gen the test generator should replace the schema address automatically


Option for single pull request per update

From @zach-taylor on June 20, 2017 18:37
We use deppbot (https://www.deppbot.com) at the moment and we like the idea of a single pull request for each update rather than a pull request for each gem update. The main reason we like this is we have a lot of dependencies, and it can get noisy and slow to perform a build for each one and the way our branch protections are set up.
Thanks for the consideration!
Copied from original issue: dependabot/feedback#11

Thanks Zach. We might add a setting for this at some point, but it's not on our roadmap right now - we prefer the multiple PRs approach as it makes them easier to test and merge. That said, we can see a world where we open a single PR for several dependency updates we know are backwards compatible.
In the meantime, Deppbot are the guys to use if you want a single PR with many Ruby updates in it.

From @zach-taylor on June 21, 2017 18:55
Sounds good 👍 I figured I would put it out there and explain our reasoning.

From @ad-m on October 1, 2017 12:19
In my opinion it is not a problem that many changes will be under one pull request. Many CIs test each commit separately, so you can continue to accurately identify the defective dependency.

Quick update: we'd still love to do this (group non-breaking updates), but it's going to need us to have lots of data to do it well (we need to know which changes are breaking). One for the future - don't worry, it won't fall off our roadmap.

From @shusson on November 20, 2017 5:59
@greysteil any news on this? I would like to be able to bump with one PR, if it breaks, bump with multiple PRs. The hope being that the original PR doesn't break often and thus we get the best of both flows.

From @elamperti on November 5, 2017 2:32
+1 to this request. Having a weekly/monthly update of all my npm packages would allow me to check quickly if they break something. If that's the case, I can analyze it and fix it manually, but otherwise it's faster than the time and resource consuming alternative of bumping versions for each package. I want to focus on the code, not on updates, and one PR per package creates a lot of noise -at least in my small repos-.

We're still a way off, but should get closer in the next few weeks. Basically, I need to do a lot of refactoring because everywhere in the update process it expects to be dealing with a single dependency. I should be able to get to that pretty soon, and this isn't the only thing waiting on it. Once that's done, the changes to make this possible are a lot more straightforward.
In the meantime, I continue to recommend Deppbot if you want a single PR with many Ruby updates in it 🙂 .

From @styfle on December 9, 2017 17:1
This was pretty painful for my Electron project to build the PR and then build the build master after merging. https://github.com/styfle/magnemite
I think the big problem is that after accepting 1 PR, all the other PRs need to be rebased and rebuilt again. So for 6 deps that need to be updated, you have to wait for 21 builds to complete. At 5 minutes per build, thats almost 2 hours. And you are spending that time clicking waiting and watching CI so you can click Merge on the next PR.
I did fix a bug with travis-ci that was building the PR twice, once for the branch and once for the PR. But this is the default behavior so I'm probably not the only one with that configuration.

+1 for this feature. I would much prefer a single PR per week to group all the updates. It's easy enough to diagnose this manually if something is wrong. Why make the case that works 99% of time harder? I could always be a config option off by default.

+1

+1
This would be a very nice addition to dependabot's arsenal.

@greysteil my team is looking for grouping of PRs of the same dependency. Often times we may leave a dependabot PR sitting because we need to do some investigation and we'll leave comments on the PR. But a newer version of the dependency comes out which makes dependabot close the original PR losing all of our comments. Is this related to this issue or should a new one be created?

I don't work on Dependabot anymore, but @feelepxyz and the team may be able to help.

my team is looking for grouping of PRs of the same dependency. Often times we may leave a dependabot PR sitting because we need to do some investigation and we'll leave comments on the PR. But a newer version of the dependency comes out which makes dependabot close the original PR losing all of our comments. Is this related to this issue or should a new one be created?

This could probably be achieved if we support configurable grouping. Currently, we open a new PR for every update because we rely on things like the branch name to be unique per version update and this would no longer apply once we start updating an existing PR with newer versions.
Support for grouping updates is high up our list: https://github.com/dependabot/feedback/issues/5
We're currently busy integrating Dependabot natively in GitHub so we've paused on feature work for the next few months while we complete this work.

+1 for this feature

Would be super cool dependabot would group updated dependencies from the same scope into a single PR. Often, these scoped packages are published as a set and need to be updated as such.

+1

Can we at least reopen this issue since it's evidently not completed?

Leaving this here in case anyone else ends up here from Google, like I did, disappointed to see it closed.
There is a similar open issue which should help address this: https://github.com/dependabot/dependabot-core/issues/1190 and currently looks like it is on the  Q2 2023 roadmap https://github.com/github/roadmap/issues/148

Most of the feature requests I see here are covered under the various tickets tracking grouped updates:
https://github.com/dependabot/dependabot-core/labels/F%3A grouped-updates 🧑🤝🧑
So I'm going to leave closed as a duplicate of one or more of those linked issues.

Looks like #1190 is the one to watch instead.

+1 for this feature request.

+1

Please use +1 reactions if you want to +1 something, that's what they are here for.
I'm assuming dozens of people are subscribed to threads like this, so you're sending everyone a notification containing a "+1" comment, which is... not exactly useful.

Thanks @The-Compiler !
We even have it documented as a policy we endorse in this repo: https://github.com/dependabot/dependabot-core#no-1-comments


[hotfix] Added missing /client/ path

Added the missing /client/ path to the favicon & logo in the default env
config.

LGTM

good catch

thanks :)


Add extraction of headings to reffy

part of #346

Not enabled for now on purpose, I suppose?

no, that was an oversight :) fixed, but that means a (simple) conflict to resolve


Database download

Currently the database download is a process inside the nexflow pipeline but it is actually also included in the dockerfile as part of container build.
We would need to decide which one is the best option and just go for one of them.

I talked to @apeltzer yesterday, and he told me, that we shall leave it to the user:
Either specify a path to the database (which already works for now) or create a nextflow process to do that. But what we not want is that the database is part of the docker image. That would blow it up. Compare to reference genomes for other sequencing pipelines.

@ggabernet As far as I got it from main.nf the option to download the database within the workflow is already there. So the issue is solved?

I just discussed with @ggabernet that we need to test this. I will do that.

Running into
`Pulling Singularity image docker://nfcore/bcellmagic [cache /sfs/7/workspace/ws/afaaj01-bcellmagic-0/work/singularity/nfcore-bcellmagic.img]
ERROR ~ No such variable: ch_igblast_db_for_process_igblast_mix
-- Check script 'main.nf' at line: 553 or see '.nextflow.log' file for more details
`
I will take a closer look.

Started working on this as well, I think I solved the issue of the error you were having. Running tests now

Finished successfully! New PR to nf-core will solve this issue :)

Will be solved by Pull Request #31. I will close this issue when the PR is merged.


arch/risc-v/bl602 : add pwm onshot watchdog driver.

Summary
arch/risc-v/bl602 : add pwm onshot watchdog driver
Impact
N/A
Testing
config:

→ System Type → BL602 Peripheral Support → TIMER1
→ Device Drivers → Timer Driver Support → PWM Driver Support → (5)Number of Output Channels Per Timer
→ Device Drivers → Timer Driver Support → Oneshot timer drive
→ Device Drivers → Timer Driver Support → Watchdog Timer Support
→ RTOS Features → Performance Monitoring → Enable CPU load monitoring → Use external clock
→ Application Configuration → Examples → Watchdog Timer example
→ Application Configuration → Examples → Pulse width modulation (PWM) example
result:

@liangzhanggb  @xiaoxiang781216  I will review and test this evening. First glace there are some things that need to be moved.  The hardware/.h file should only have register information. I see some structs defined there that should be moved to the driver header in the next level up.

@liangzhanggb  @xiaoxiang781216  I will review and test this evening. First glace there are some things that need to be moved.  The hardware/.h file should only have register information. I see some structs defined there that should be moved to the driver header in the next level up.

@liangzhanggb @xiaoxiang781216 I will review and test this evening. First glace there are some things that need to be moved. The hardware/.h file should only have register information. I see some structs defined there that should be moved to the driver header in the next level up.

I have fixed it, Thank you very much

@liangzhanggb @xiaoxiang781216 I will review and test this evening. First glace there are some things that need to be moved. The hardware/.h file should only have register information. I see some structs defined there that should be moved to the driver header in the next level up.

I have fixed it, Thank you very much

@liangzhanggb One more minor fix that I think will also get the build to pass.  I verified your latest changes and they all look good.  Thanks.
diff --git a/arch/risc-v/src/bl602/Kconfig b/arch/risc-v/src/bl602/Kconfig
index 9542d0be9c..0c819f95c8 100644
--- a/arch/risc-v/src/bl602/Kconfig
+++ b/arch/risc-v/src/bl602/Kconfig
@@ -13,6 +13,7 @@ config BL602_HAVE_UART0
        select ARCH_HAVE_UART0
        select UART0_SERIALDRIVER
        select ARCH_HAVE_SERIAL_TERMIOS
+       select ARCH_HAVE_PWM_MULTICHAN

 config BL602_UART0
        bool
@@ -37,6 +38,5 @@ config BL602_TIMER1

 config BL602_PWM0
        bool "PWM0"
-       select ARCH_HAVE_PWM_MULTICHAN

 endmenu
diff --git a/boards/risc-v/bl602/bl602evb/configs/timer/defconfig b/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
index 63ef88f96a..94d63251ff 100644
--- a/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
+++ b/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
@@ -55,6 +55,7 @@ CONFIG_ONESHOT=y
 CONFIG_PREALLOC_TIMERS=0
 CONFIG_PTHREAD_STACK_DEFAULT=8192
 CONFIG_PWM=y
+CONFIG_PWM_MULTICHAN=y
 CONFIG_PWM_NCHANNELS=2
 CONFIG_RAM_SIZE=134217728
 CONFIG_RAM_START=0xc0800000

@liangzhanggb One more minor fix that I think will also get the build to pass.  I verified your latest changes and they all look good.  Thanks.
diff --git a/arch/risc-v/src/bl602/Kconfig b/arch/risc-v/src/bl602/Kconfig
index 9542d0be9c..0c819f95c8 100644
--- a/arch/risc-v/src/bl602/Kconfig
+++ b/arch/risc-v/src/bl602/Kconfig
@@ -13,6 +13,7 @@ config BL602_HAVE_UART0
        select ARCH_HAVE_UART0
        select UART0_SERIALDRIVER
        select ARCH_HAVE_SERIAL_TERMIOS
+       select ARCH_HAVE_PWM_MULTICHAN

 config BL602_UART0
        bool
@@ -37,6 +38,5 @@ config BL602_TIMER1

 config BL602_PWM0
        bool "PWM0"
-       select ARCH_HAVE_PWM_MULTICHAN

 endmenu
diff --git a/boards/risc-v/bl602/bl602evb/configs/timer/defconfig b/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
index 63ef88f96a..94d63251ff 100644
--- a/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
+++ b/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
@@ -55,6 +55,7 @@ CONFIG_ONESHOT=y
 CONFIG_PREALLOC_TIMERS=0
 CONFIG_PTHREAD_STACK_DEFAULT=8192
 CONFIG_PWM=y
+CONFIG_PWM_MULTICHAN=y
 CONFIG_PWM_NCHANNELS=2
 CONFIG_RAM_SIZE=134217728
 CONFIG_RAM_START=0xc0800000

@liangzhanggb One more minor fix that I think will also get the build to pass. I verified your latest changes and they all look good. Thanks.
diff --git a/arch/risc-v/src/bl602/Kconfig b/arch/risc-v/src/bl602/Kconfig
index 9542d0be9c..0c819f95c8 100644
--- a/arch/risc-v/src/bl602/Kconfig
+++ b/arch/risc-v/src/bl602/Kconfig
@@ -13,6 +13,7 @@ config BL602_HAVE_UART0
        select ARCH_HAVE_UART0
        select UART0_SERIALDRIVER
        select ARCH_HAVE_SERIAL_TERMIOS
+       select ARCH_HAVE_PWM_MULTICHAN

 config BL602_UART0
        bool
@@ -37,6 +38,5 @@ config BL602_TIMER1

 config BL602_PWM0
        bool "PWM0"
-       select ARCH_HAVE_PWM_MULTICHAN

 endmenu
diff --git a/boards/risc-v/bl602/bl602evb/configs/timer/defconfig b/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
index 63ef88f96a..94d63251ff 100644
--- a/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
+++ b/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
@@ -55,6 +55,7 @@ CONFIG_ONESHOT=y
 CONFIG_PREALLOC_TIMERS=0
 CONFIG_PTHREAD_STACK_DEFAULT=8192
 CONFIG_PWM=y
+CONFIG_PWM_MULTICHAN=y
 CONFIG_PWM_NCHANNELS=2
 CONFIG_RAM_SIZE=134217728
 CONFIG_RAM_START=0xc0800000

I have fixed it, Thank you very much

@liangzhanggb One more minor fix that I think will also get the build to pass. I verified your latest changes and they all look good. Thanks.
diff --git a/arch/risc-v/src/bl602/Kconfig b/arch/risc-v/src/bl602/Kconfig
index 9542d0be9c..0c819f95c8 100644
--- a/arch/risc-v/src/bl602/Kconfig
+++ b/arch/risc-v/src/bl602/Kconfig
@@ -13,6 +13,7 @@ config BL602_HAVE_UART0
        select ARCH_HAVE_UART0
        select UART0_SERIALDRIVER
        select ARCH_HAVE_SERIAL_TERMIOS
+       select ARCH_HAVE_PWM_MULTICHAN

 config BL602_UART0
        bool
@@ -37,6 +38,5 @@ config BL602_TIMER1

 config BL602_PWM0
        bool "PWM0"
-       select ARCH_HAVE_PWM_MULTICHAN

 endmenu
diff --git a/boards/risc-v/bl602/bl602evb/configs/timer/defconfig b/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
index 63ef88f96a..94d63251ff 100644
--- a/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
+++ b/boards/risc-v/bl602/bl602evb/configs/timer/defconfig
@@ -55,6 +55,7 @@ CONFIG_ONESHOT=y
 CONFIG_PREALLOC_TIMERS=0
 CONFIG_PTHREAD_STACK_DEFAULT=8192
 CONFIG_PWM=y
+CONFIG_PWM_MULTICHAN=y
 CONFIG_PWM_NCHANNELS=2
 CONFIG_RAM_SIZE=134217728
 CONFIG_RAM_START=0xc0800000

I have fixed it, Thank you very much

LGTM I see a couple other unrelated things for this port in the Kconfigs that I can clean in another PR.

LGTM I see a couple other unrelated things for this port in the Kconfigs that I can clean in another PR.


No error message shown when attempting to register while offline

JIRA: https://openedx.atlassian.net/browse/MOB-1653

@hanningni please review

:+1:


ERROR Jasper Not Running

Hello,
I've used the code of jasper-client and put it in /home/pi/jasper/client/diagnose.py and tried to run jasper however I got this error:
pi@raspberrypi:~ $ /home/pi/jasper/jasper.py

        JASPER - THE TALKING COMPUTER           *

(c) 2015 Shubhro Saha, Charlie Marsh & Jan Holthuis *

Traceback (most recent call last):
File "/home/pi/jasper/jasper.py", line 137, in
if not args.no_network_check and not diagnose.check_network_connection():
AttributeError: 'module' object has no attribute 'check_network_connection'

The work on Jasper - specifically making it work as-is and refactoring to Python 3 is being conducted at https://github.com/aplawson/jasper-client -- including a tutorial on how to build it and/or deploy it with a custom Raspbian ISO image. //adam


Suggestion: Use the keyword "readonly" as an accessor for classes

Use case: we need a class Counter which provides a property count that is read from outside the class and written from inside the class.
In JavaScript, if we want to use a class instead of a closure, we have to make a public property:
/**
 * Read-only access to the property "count", please
 */
class Counter {
	constructor() {
		this.count = 0
	}
	inc() {
		++this.count
	}
}
let counter = new Counter()
counter.inc()
console.log(counter.count) // 1
counter.count = 10 // No error but it doesn't respect the JSDoc

In TypeScript, we could proceed in the same way. But it is tempting to use an accessor private:
class Counter {
	private count = 0
	public inc() {
		++this.count
	}
	public getCount() {
		return this.count
	}
}
let counter = new Counter()
counter.inc()
console.log(counter.getCount()) // 1
counter.count = 10 // Error: Property 'count' is private and only accessible within class 'Counter'

However, it is verbose, less readable, less performant .
I suggest the following syntax:
class Counter {
	readonly count = 0
	public inc() {
		++this.count
	}
}
let counter = new Counter()
counter.inc()
console.log(counter.count) // 1
counter.count = 10 // Error: Property 'count' is readonly and can be only mutated within class 'Counter'

It is the same solution as in JavaScript, but with a better control to access (and less JSDoc).

readonly has already been implemented for class properties, though it has different semantics from what you're proposing see #12 and #6532


Segments not updated after updating contact list

Hi,
I'm currently on "@sendgrid/client": "^7.1.1" and using the following snippet to add contacts to an existing contact list:
const request = { body: data, method: 'PUT', url: '/v3/marketing/contacts' };
return await sendgrid.request(request);

I'm getting back a 200, along with a job_id and after a few minutes the contact list is updated.
My issue is that a segment tied to said contact list is not being updated after the contact list has been updated. These new contacts meet the criteria specified under the segment.
I have this segment tied to a marketing campaign, which is not really sending out relevant emails due to an out-of-date segment.

Hello @camilobern,
Thanks for submitting a GitHub issue! We are very sorry that you are running into this problem. In order to better serve you, as this does not present itself as a library specific issue, we would like to ask that you reach out to our support team at https://support.sendgrid.com.
Thank you!
SendGrid DX Team

Did you manage to solve this? @camilobern
I am experiencing the same issue.

@crisguitar I found that the time-frame for segments to reflect the latest list of contacts was not consistent. Some times it's updated almost immediate;y after the list of contacts is updated, others it would take an hour.

I've been waiting 6 hours and my segments are still not updating ⏳

I am having this issue as well. It makes segment based automation basically a useless feature. I don't want my users to have to wait multiple hours to receive a welcome email.

Thanks for submitting a GitHub issue! We are very sorry that you are running into this problem. In order to better serve you, as this does not present itself as a library specific issue, we would like to ask that you reach out to our support team at support.sendgrid.com.

I'm having the same issue. It doesn't look like there is a solution, except to not use the Segment feature.


API call on TX receive

Ability to fire off event on TX received and send TX to a specific user provided endpoint.

You can now add a custom URL to APICallURL that will send a JSON post of a TX when it is received in the wallet allowing a 3rd part application to process the tx.


Show backend used in benchmarks.

Closes #178

I should note that tflite backend uses strategy "use CPU if specified delegate failed", and we use iOS settings for Windows, so now we can see CoreML labels in the benchmark list in Windows builds.
It's not the fault of this PR, but maybe we should do something about it in the future. Probably, create an issue to create a separate config for Windows.

I should note that tflite backend uses strategy "use CPU if specified delegate failed", and we use iOS settings for Windows, so now we can see CoreML labels in the benchmark list in Windows builds.
It's not the fault of this PR, but maybe we should do something about it in the future. Probably, create an issue to create a separate config for Windows.

I created a new issue for this https://github.com/mlcommons/mobile_app_open/issues/203#issue-1097907108


Feature request

Can you include support for PPTP , ike2/3 and SNMP?

added support for SNMP with v1.6.0, still working on IKE and PPTP

brutespray will not support IKE and PPTP


Prevent vim9script errors in older vim versions

Using the plugin with an older version of vim, 8.2 without patches in my case, gives these errors:
line    1:
E683: File name missing or invalid pattern
line    3:
E488: Trailing characters: # File: fileselect.vim
line    4:
E488: Trailing characters: # Author: Yegappan Lakshmanan (yegappan AT yahoo DOT com)
line    5:
E488: Trailing characters: # Version: 1.2
line    6:
E488: Trailing characters: # Last Modified: Jan 4, 2021
etc...
Moving this  check to the top of plugin\fileselect.vim fixes the issue:
" Need Vim 8.2.2261 and higher
if !has('patch-8.2.2261')
  finish
endif

Standard documentation (:help vim9-mix) suggest to use:
if !has('vim9script')
   finish
endif

But this was included in patch 8.2.2529 and would bump up the minimal required version mentioned in README.

Thanks for reporting the issue. Committed d20310d043fd37e7eb1721953c9fec7cb11015f8 to address this.


[TypedClient] Upgrade from 8.7.0 -> 8.7.1. Indices Exists Do() method missing.

Hello, I am moving project codebase from olivere/elastic driver (on Elasticsearch 7.x)
to go-elasticsearch typedapi (on Elasticsearch 8.7).
After upgrading go-elasticsearch to 8.7.1 compilation fails on:
tc/index.go:61:3: TC.Indices.Exists(idx.Name).Do undefined (type *"github.com/elastic/go-elasticsearch/v8/typedapi/indices/exists".Exists has no field or method Do)
Here is relevant file (8.7.1) without Do method:
https://github.com/elastic/go-elasticsearch/blob/v8.7.1/typedapi/indices/exists/exists.go
and one with Do method (8.7.0):
https://github.com/elastic/go-elasticsearch/blob/v8.7.0/typedapi/indices/exists/exists.go#L153

Hi @practical-coder,
The presence of the Do method in 8.7.0 was a bug, it didn't return anything since this endpoint doesn't provide its answer within the body but rather in the status code.
Rather you should use IsSuccess which will give you a true bool in case of success, or you could use Perform and retrieve the HttpResponse directly.

Ok, thank you for clarification @Anaethelion !
I've started using typedapi since 8.7.0 and used Do method in other indices management endpoints,
so I've done it in similar way using Exists endpoint.


Cant activate IntelliCode for python

Extension error messages:
Sorry, something went wrong activating IntelliCode support for Python. Please check the "Python" and "VS IntelliCode" output windows for details.
Acquiring model 'intellisense-members' for python
Querying IntelliCode service for available models.
Cached model is up to date.
vs-intellicode-python was passed a model: {"modelPath":"c:\Users\DD\.vscode\extensions\visualstudioexptteam.vscodeintellicode-1.1.5\IntellicodeModels\88630A1B4C077752096ECAA576B92C7D3FA1_AFE724E3F06E46849C99F748E987227C","modelId":"88630A1B4C077752096ECAA576B92C7D3FA1","outputId":"AFE724E3F06E46849C99F748E987227C"}.
Error while activating Python: {}
Version: 1.33.1 (user setup)
Commit: 51b0b28134d51361cf996d2f0a1c698247aeabd8
Date: 2019-04-11T08:27:14.102Z
Electron: 3.1.6
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.17134

Try downgrade this extension by uninstalling it first and then choosing "Install Another Version..." Don't forget to disable auto updating extensions.

This is an issue with the Python extension that was recently updated (Microsoft/vscode-python#5437). The Python extension owners are rolling out a fix, but in the mean time, you can work around it by installing an older version of the Python extension temporarily. The IntelliCode activation error for python files is a side-effect from the Python extension failing to load.


⚠ Bofrost.be has degraded performance

In f821ac0, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 865 ms

Resolved: Bofrost.be performance has improved in 7ddffdd after 6 minutes.


Skill - SlowBurn

Add +1 avoid and hit each turn up to a maxium of 15.

The conflict for this PR is failed to solve, you need to reput one for this skill


fix: Try fix e2e epochs in CI

Attempts two fixes at e2e epochs test. First, it increases the L1 block time, to account for general CI slowness. Second, it adds more retries to the L1 gas utils getTx, since the e2e epochs test works using the tx delayer, which artificially introduces a delay between a tx being sent and it being available in anvil, so it triggered a timeout in the utils.

@ludamad it also runs fine locally. Maybe new CI puts less pressure on processes?

The pressure should really be the same, I've redone this in ci2.5 as an experiment

Ah darn also fails in the CI redo

pushed an experiment with a bigger runner for this

Undid experiment and merging as it passed


create_node returns true(boolean) instead of created node id.

If I call create_node with closed node, it successfully create new node.
However it returns true value not new node id.
It only happens if I call this method with closed node which is not loaded children data yet.
One more weird behavior is it returns node id since next calls. Only first call of create_node with closed node returns true.
Should I always open the node every time if I want to create new node under of it?
I tried to search similar issue, but I couldn't. Please let me know if it is duplicated.

This is normal and expected behavior - there is no way for jstree to return the node ID as the parent should first be loaded and opened - which I guess in your config is an async operation. In this case - listen for the create_node.jstree event, or use the callback parameter to obtain the new ID.


Changelog: update for 2.3.0

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's contributing guide (中文 | English | Español).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

That's because you program a lot and you don't sleep much.
UUUU
2.3.2 now.
Looks like it wasn't just one thing.
:grimacing:

Yeah in 2.3.1 there's another regression😭


Dialog jumps on theme change unless dragged before

Tested browsers/devices(OK/FAIL):

OK: -
FAIL:

Only first time:

FF
IE11

Every time I change theme:

Chrome / Opera
Edge

URL (minimal example if possible): https://jsbin.com/kefiriy/edit?js,output
Steps to reproduce the problem:

Open the above example page
Open the dialog
Toggle theme without having the dialog dragged before.

What is the expected result?
Dialog stays still while theme changes.
What happens instead?
Dialog jumps away and back distractingly.
Any other information? (attach screenshot if possible)
This doesn't happen when the dialog was dragged before.

@boghyon , thank you for the provided example. The issue is logged for internal investigation via ticket #1770375548. Updates will be posted in this GitHub issue as well.

Hi,
When the theme is changed, a lot of css styles are changed and the browsers need time to update them. This behavior is quite obvious in complex applications - when the theme is changed almost all controls get displaced for some time.
Unfortunately, there isn't much we can do here. Everything is up to the browsers.
Best,
Teo


Ansible for setting up easy-mode apps

This is addressed in the arborworkflows.readthedocs.org documentation.


Create repository of MRSK example configurations

I'm loving MRSK so far, and the configuration docs are solid, but I'm having trouble finding any full application config to use as examples that I can learn from.
I'm thinking of something similar to:

https://github.com/kubernetes/examples
https://github.com/ContainerSolutions/kubernetes-examples

Initial search for existing MRSK configs on GitHub: https://github.com/search?q=path:/deploy.yml$/+/^servers:/+/^image:/+(NOT+repo:mrsked/mrsk)+(NOT+is:fork)&type=code

There is also this great guide from @huksley for deploying next.js https://medium.com/@ruslanfg/deploying-nextjs-using-mrsk-819aa9eb83ba

thanks @Zeko369  here also some example configurations https://gist.github.com/huksley/70c8615f41c7871700c1dafbf518654c

There is also my article about deploying Rails on AWS with GitHub Actions: https://jetrockets.com/blog/how-to-use-basecamp-s-mrsk-with-aws-and-github

HA Redis with Sentinel support here: https://github.com/mrsked/mrsk/discussions/382

Add these to your package.json file and you will save some time ...
"scripts": {
    "logs": "mrsk app logs --follow",
    "console": "mrsk app exec -i 'bin/rails console'",
    "migrate": "mrsk app exec -i 'bin/rails db:migrate'",
    "release": "mrsk lock release",
    "bash": "mrsk app exec -i bash",
}

$: yarn run console
❯ yarn run console
yarn run v1.22.19
$ mrsk app exec -i 'bin/rails console'
Get most recent version available as an image...
Launching interactive command with version latest via SSH from new container on
Loading production environment (Rails 7.0.6)
irb(main):001:0>

If you need a time series database, QuestDB is really cool and pretty easy to use:
accessories:
  questdb:
    image: questdb/questdb:7.2.1
    options:
      publish:
        - "9000:9000"
        - "9003:9003"
        - "9009:9009"
        - "8812:8812"
    host: <IP_ADDRESS>
    add-host: <IP_ADDRESS>
    directories:
      - questdb:/var/lib/questdb

$: mrsk accessory boot questdb

A simple hourly PostgreSQL Backup solution using pg_dump
Hourly backups
pg_backup:
    image: prodrigestivill/postgres-backup-local
    host: <IP_ADDRESS>
    port: "<IP_ADDRESS>:5400:5400"
    add-host: <IP_ADDRESS>
    directories:
      - pg_backup:/backups
    env:
      clear:
        SCHEDULE: '@hourly'
        POSTGRES_BACKUP_ALL: "false"
        POSTGRES_HOST: <%= ENV["DB_HOST"] %>
        POSTGRES_DB: <%= ENV["DB_NAME"] %>
        POSTGRES_USER: <%= ENV["DB_USER"] %>
        POSTGRES_PASSWORD: <%= ENV["DB_PASSWORD"] %>
        POSTGRES_EXTRA_OPTS: '-Z6 --superuser=postgres/your-user --no-owner --schema=public --blobs'
        BACKUP_KEEP_DAYS: 7
        BACKUP_KEEP_WEEKS: 4
        BACKUP_KEEP_MONTHS: 1
        HEALTHCHECK_PORT: 5401

Restore to a new PSQL host:
$: PRIMARY_POSTGRES_PASSWORD=your_pg_password
$: zcat /root/[your-container-directory]/backups/last/[backup-file].sql.gz | PGPASSWORD=$PRIMARY_POSTGRES_PASSWORD psql --host=$DB_HOST --port=$DB_PORT --username=$DB_USER --dbname=$DB_NAME

Manual Backup:
Add this line to your package.json file
"scripts": {
    "start_backup": "mrsk accessory exec pg_backup -i /backup.sh",
    "view_backups": "ssh<EMAIL_ADDRESS>ls -lsh /root/[your-container-directory]/backups/last/"
}

$: yarn run start_backup # start backup
$: yarn run view_backups # view all backups inside the latest backups directory


Modules not resetting if boss resets

Modules are not resetting if boss gets reset by running out of its range. Eg. A/S Ranks and FATE bosses.
This can lead to problems if the boss is doing an attack right after being pulled (eg. S Rank Tarchia) or if multiple modules are suddenly active at the same time.
Adding a raw update check for boss is in combat unloads the module entirely and does not just reset it, so you need to run out of range and back for it to get loaded again.

Fixed recently.


User Account Login

Description
Allow the user to login to their account using their email and password as credentials. User credentials would be checked for authenticity. This will ensure only the authorised user is accessing their account, improving security.
Acceptance Criteria

[ ] User can enter their email and password credentials easily.
[ ] Credentials checked for authenticity when submitted.
[ ] Only authorised users are able to access their account.

Implementation details
To be decided after initial project skeleton set-up. Note that a "Forgot Password?" feature may be developed in future (Issue #23).
Testing notes
n/a
Reviewers
@cdra334

We are now using google accounts for authentication, and other issues cover by other authentication, so I'm going to close this one.


incorrect typing for vite defineConfig method

Describe the bug
The vite docs describe this form of creating the config file:
export default ({ command, mode }) => {
  if (command === 'serve') {
    return {
      // serve specific config
    }
  } else {
    return {
      // build specific config
    }
  }
}

They also mention the defineConfig helper.
However the defineConfig method takes a object of UserConfigExport type:
UserConfigExport = UserConfig | Promise<UserConfig> | UserConfigFn;

the UserConfig object doesnt have any field for command.
thus the typescript file which uses defineConfig helper for above example wouldnt compile.
Reproduction
create a function like this  and you will get a compile time error
export default defineConfig((cfg: UserConfig) =>({
if (cfg.command == 'serve') { }
})

System Info
Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:
using vite 2.3.3 with typescript 4.2.4

Used package manager:
Logs

Before submitting the issue, please make sure you do the following

[ x] Read the Contributing Guidelines.
[ x] Read the docs.
[ x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[ x] Provide a description in this issue that describes the bug.
[x ] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[ x] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

Seems you typed it wrongly
You need to use UserConfigExport instead of UserConfig
Please close the issue if this solved it for you 🙂

You don't need to manually annotate the cfg type.
The whole function is of the UserConfigFn type, whose first argument is already typed as ConfigEnv:
https://github.com/vitejs/vite/blob/d1d7106424953ba32a78d68ab62a1e557d6ad19f/packages/vite/src/node/config.ts#L47-L50


[release/0.9] Backport networking commits

These are direct ports of the following commits without any additional changes from main to release/0.12 :
https://github.com/microsoft/hcsshim/commit/c79a6310e6f15a893430063ca76f44dcb7711426 : https://github.com/microsoft/hcsshim/pull/2106 ---> direct cherry-pick
https://github.com/microsoft/hcsshim/commit/8beabacfc2d21767a07c20f8dd5f9f3932dbf305 : https://github.com/microsoft/hcsshim/pull/2139 ---> This commit had merge conflicts that were fixed.
https://github.com/microsoft/hcsshim/commit/0f7d8de9948c3ebfd45911dc55b451d8902d48b2 : https://github.com/microsoft/hcsshim/pull/2085 ---> This commit had merge conflicts that were fixed.

/cc @kiashok
/cc @debj1t

unrelated test failures


Add hfss.mesh.assign_initial_mesh_manually() function for detailed mesh settings for the initial mesh.

Description of the feature
Create counterpart to the assign_initial_mesh_from_slider() method.
Easy implementation via the initial_mesh_settings.props dictionary. Similar to the assign_surface_mesh_manually but only for the initial mesh settings.
It would just complement the function overview for the hfss mesh.
Steps for implementing the feature
something like:
def assign_initial_mesh_manually(args):

mesh_props = hfss.initial_mesh_settings.props

mesh_props[...] = ...

Useful links and references
@Samuelopez-ansys

@FgPy93 Please check this pull request:
https://github.com/ansys/pyaedt/pull/5556
You will be able to assign initial mesh with the manual settings. But, for now, there is no a pythonic way to edit these properties, you can get the initial mesh:
app.mesh.initial_mesh_settings
Remind to save the project to load the updated properties.


Support into iterator

Removes the Sized bound, it is no longer necessary, since https://github.com/facebook/winterfell/pull/239 .
The change above allows for impl<T: Serializable> Serializable for &T.
The change above allows to change write_many to be generic and receive a IntoIterator without breaking the serialization for [T; C]
The removal of the Sized bound also makes the trait object save, and allows for type erasure.

@irakliyk yes! The motivation for this was the BTreeMap serialization. This also allows that to be serialized with no copying (whereas now one has to copy the map to a vec and then serialize that)


TikTok "Failed to parse JSON"

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2021.12.27. (update instructions)
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Russia
Description
Seemed Tiktok parsing has been broken from yesterday. Same happens for previous version 2021.12.25
Verbose log
Z:\Programs\cli_programs>yt-dlp --verbose https://www.tiktok.com/@nkozyavina/video/7035717794452753666
[debug] Command-line config: ['--verbose', 'https://www.tiktok.com/@nkozyavina/video/7035717794452753666']
[debug] Encodings: locale cp1251, fs utf-8, out cp1251 (No ANSI), err utf-8, pref cp1251
[debug] yt-dlp version 2021.12.27 [6223f67] (win_exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] exe versions: ffmpeg 2021-12-09-git-b9f4c1231f-full_build-www.gyan.dev (setts), ffprobe 2021-12-09-git-b9f4c1231f-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [TikTok] Extracting URL: https://www.tiktok.com/@nkozyavina/video/7035717794452753666
WARNING: [TikTok]<PHONE_NUMBER>452753666: Failed to parse JSON  (caused by JSONDecodeError('Expecting value: line 1 column 1 (char 0)')); please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you
are using the latest version; type  yt-dlp -U  to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.; Retrying with webpage
ERROR: [TikTok]<PHONE_NUMBER>452753666: Unable to extract json_string; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; type  yt-dlp -U  to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; type  yt-dlp -U  to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
  File "yt_dlp\extractor\common.py", line 599, in extract
  File "yt_dlp\extractor\tiktok.py", line 368, in _real_extract
  File "yt_dlp\extractor\common.py", line 1173, in _search_regex

Duplicate of #2282, #2282, #2250, #2250, #2246, #2246, #2243, #2183, #2183, #2165, #2159, #2159, #2150, #2150, #2135, #2133
Is it so hard to search before opening an issue!!


Allow clients to deploy multiple services

Currently, XMPP only works for the default service.

https://ocd.appscale.com:8080/job/Daily Build/3849/
https://ocd.appscale.com:8080/job/Autoscale-test-manual-triggers/678/
https://ocd.appscale.com:8080/job/Tools-test/2376/

https://ocd.appscale.com:8080/job/Daily Build/3880/
https://ocd.appscale.com:8080/job/Tools-test/2385/
https://ocd.appscale.com:8080/job/Migration/1924/
https://ocd.appscale.com:8080/job/Autoscale-test-manual-triggers/689/


Delete phpunit.xml

Coverage remained the same at 100.0% when pulling 084fbcb64ad336e9df1f370226ed3f6435891205 on iamsidofficial:master into 760bde40ab18d8a76eb674d0ea64fa3ddac49404 on Brunty:master.

Coverage remained the same at 100.0% when pulling 084fbcb64ad336e9df1f370226ed3f6435891205 on iamsidofficial:master into 760bde40ab18d8a76eb674d0ea64fa3ddac49404 on Brunty:master.

Coverage remained the same at 100.0% when pulling 084fbcb64ad336e9df1f370226ed3f6435891205 on iamsidofficial:master into 760bde40ab18d8a76eb674d0ea64fa3ddac49404 on Brunty:master.

Thank you for this PR! It was fixed in https://github.com/Brunty/cigar/pull/38 which was put in a little earlier.
Thank you for doing so as well though, much appreciated :)


NFT withdrawal stuck in processing

User: 0xbeb127e9008139078ed905e20fcbc4feaebb4116
uid: 0xd3240ca94e90dedf0b89cb4f67033e8d3fb1c749
NFT: https://soonaverse.com/nft/0x4ba5a24f59dd68024fd7b8f85ab13bbc92759af0

Thanks, waiting for solution before withdrawing again..


Read/write chunk/buffer size is hard-coded.  Make it configurable.

Read/write chunk/buffer size is hard-coded.  Make it configurable.

This has been merged.  Closing this issue.


refresh tree after adding

fixe #206

This doesn't fix the issue. Read the comments in the issue. All toastr's get closes, but it should only close the error ones.
The bug doesn't occur anymore, because it doesn't switch to the file anymore.
It's hard to fix anyway because the toastr's dont have a type. (thanks ngx-toastr).


Improve editor UI

The editor UI currently has buttons for users to enter mathematical functions (like sin, cos, and more), that are rendered as buttons and contain text that is typeset using MathJax. This leads to a poor UX when users click on them and then use the arrow keys to navigate, as this brings the typeset button into focus.
We could replace them with SVGs or images so that this doesn't happen anymore. This will involve editing the editor HTML described in the src/modules/ui.js and replacing all text enclosed between '$' with SVGs, images, or something similar.

I want to take this issue. Can you please assign it to me?

Sure, if you need any help with the internal working of the API or anything else feel free to ask.

@hrushikeshrv thanks a lot. I will open a pull request soon

@hrushikeshrv I have not found any consistent icon set on the internet. So, I think I need to create it myself in photoshop. Can I do it?

Yeah sure. If you go to the example and take a look at the buttons in question (under the Functions tab), you'll see that we need to replace the buttons for the following functions -

double integral
limit
square root
nth root
subscript
superscript
subscript & superscript
fraction
all the trigonometric functions

These buttons are described in the editor HTML on line 438 and lines 447 to 462. If you can create SVGs for these functions and replace the '$ \func $' string with them, that would work. SVGs would be better than images to keep our payload size low.
But try to find inbuilt HTML character codes/ASCII math char codes before designing an SVG. I've just found that the double integral sign can be replaced by the hex code ∬ or ∬ so we could replace the '$ \iint{} $' on line 438 with ∬ and achieve the same result (∬). I doubt we can do the same with the other buttons, though.

@hrushikeshrv ok got it. But why changes are not showing in my local environment? I have opened basic-example.html file in live-server

Which changes do you mean? If you mean the double integral sign, I haven't changed the code yet. I just found the hex code for it by googling it and tried it once by inspecting the editor HTML in my browser.

@hrushikeshrv I mean I have cloned the repository and changed the double integral sign by myself. But I did not see any changes.Changes are not reflecting in my browser.Can you please explain the process of setting the dev environment.I think ,I have made mistake in setting up the dev environment

@srijit2002 that's because the examples use a different build of the library (check the linked script in the head). For the purpose of development, remove the script that uses the example build and link the expression-backend, controller, and ui scripts from the src folder separately and in that order.

@hrushikeshrv thanks for your clear explanations.
To resolve the issues I have done the following steps-

I have made all the svg icons and put those in a separate folder called assets in the root of the directory.
And after that I have put all of those icons inside it and used them in ui.js using the image tag.

I will open a pull request on 1st october :-)
Do you have any suggestions or do I need to make any changes to the file structure? Please let me know :-)

Sounds good. Have you used/tested it yet?
I think one change we may need to make is to minify the SVG source of the SVGs you created and hard code it into the editor UI, but we can do that once you open a pull request and I review it.


Improvements and corrections in the Portuguese (BR) translation + gender neutralization

I improved and corrected the Brazilian Portuguese (pt-BR) translation for clarity, cultural relevance, and inclusivity by applying gender-neutral language.
Let me know your thoughts on these updates.

@fthiesen, @Lekler, and @sergiospagnuolo , if you have time to provide input on these suggested changes, I'd appreciate it!  Thanks pt-BR team! :)  Valeu!


Upgrade focus

Switch to versioned url

Thank you for the contribution. It was merged directly as 4665ffdfdb6df7e3da1b963753bb67575da606eb to keep commit history cleaner. Your contribution is still credited to you.


validate futr_df rows in predict

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


Unable to install using the docs

I am trying to install spinnaker all in one setup following the instructions. However my install script fails with the below error.
When i looked further, it seems that CRDS folder doesn't exist when applying the spinnaker operator.
./scripts/install.sh -o -P $Public_IP
[INFO ] Using OSS Spinnaker
[INFO ] Using OSS Spinnaker
[INFO ] Running minnaker setup for Linux
[INFO ] Cloning repo: https://github.com/armory/spinnaker-kustomize-patches#minnaker into /home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc
[WARN ] /home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc exists already.  FOLDER CONTENTS WILL GET OVERWRITTEN!
[WARN ] PROCEEDING in 3 secs... (ctrl-C to cancel; use -B option to specify a different directory)
Cloning into '/home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc'...
remote: Enumerating objects: 1964, done.
remote: Counting objects: 100% (650/650), done.
remote: Compressing objects: 100% (274/274), done.
remote: Total 1964 (delta 400), reused 548 (delta 356), pack-reused 1314
Receiving objects: 100% (1964/1964), 549.72 KiB | 9.82 MiB/s, done.
Resolving deltas: 100% (1116/1116), done.
[INFO ] Installing yq
[INFO ] Installing jq
[INFO ] Trying to detect endpoint
[INFO ] Using provided public IP A.B.C.D
[INFO ] Generating password [/home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc/secrets/spinnaker_password]:
*****RANDOM***Paswword****
[INFO ] Updating spinsvc templates with new endpoint: A.B.C.D
[INFO ] Creating spin_endpoint helper function
#!/bin/bash
#echo "$(kubectl get spinsvc spinnaker -n spinnaker -ojsonpath='{.spec.spinnakerConfig.config.security.uiSecurity.overrideBaseUrl}')"
echo "$(yq e '.spec.spinnakerConfig.config.security.uiSecurity.overrideBaseUrl' BASE_DIR/expose/patch-urls.yml)"
[[ -f BASE_DIR/secrets/spinnaker_password ]] && echo "username: 'admin'"
[[ -f BASE_DIR/secrets/spinnaker_password ]] && echo "password: '$(cat BASE_DIR/secrets/spinnaker_password)'"
[INFO ] --- Installing K3s ---
[INFO]  Using v1.19.7+k3s1 as release
[INFO]  Downloading hash https://github.com/k3s-io/k3s/releases/download/v1.19.7+k3s1/sha256sum-amd64.txt
[INFO]  Skipping binary downloaded, installed k3s matches hash
[INFO]  Skipping installation of SELinux RPM
[INFO]  Skipping /usr/local/bin/kubectl symlink to k3s, already exists
[INFO]  Skipping /usr/local/bin/crictl symlink to k3s, already exists
[INFO]  Skipping /usr/local/bin/ctr symlink to k3s, already exists
[INFO]  Creating killall script /usr/local/bin/k3s-killall.sh
[INFO]  Creating uninstall script /usr/local/bin/k3s-uninstall.sh
[INFO]  env: Creating environment file /etc/systemd/system/k3s.service.env
[INFO]  systemd: Creating service file /etc/systemd/system/k3s.service
[INFO]  systemd: Enabling k3s unit
Created symlink /etc/systemd/system/multi-user.target.wants/k3s.service → /etc/systemd/system/k3s.service.
[INFO]  No change detected so skipping service start
[INFO ]  --- END K3s ---
[INFO ] Setting Kubernetes context to Spinnaker namespace
Context "default" modified.
[INFO ] Spinnaker Operator Version:
[INFO ] Spinnaker flavor: oss
[INFO ] Changing spinnaker flavor...Done
[INFO ] Resolved operator namespace: spinnaker-operator
[INFO ] Deploying oss operator...
[INFO ] Downloading operator from https://github.com/armory/spinnaker-operator/releases/download/v/manifests.tgz
[KUBE ] kubectl apply -f /home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc/operator/deploy/crds/
[ERROR] Error executing command:
error: the path "/home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc/operator/deploy/crds/" does not exist[KUBE ] kubectl apply -f https://engineering.armory.io/manifests/pacrd-1.0.1.yaml -n spinnaker
[ERROR] Error executing command:
Warning: apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
Error from server (NotFound): error when creating "https://engineering.armory.io/manifests/pacrd-1.0.1.yaml": namespaces "spinnaker" not found
Error from server (NotFound): error when creating "https://engineering.armory.io/manifests/pacrd-1.0.1.yaml": namespaces "spinnaker" not found
Error from server (NotFound): error when creating "https://engineering.armory.io/manifests/pacrd-1.0.1.yaml": namespaces "spinnaker" not found
Error from server (NotFound): error when creating "https://engineering.armory.io/manifests/pacrd-1.0.1.yaml": namespaces "spinnaker" not found
Error from server (NotFound): error when creating "https://engineering.armory.io/manifests/pacrd-1.0.1.yaml": namespaces "spinnaker" not found
ubuntu@ip-172-31-41-23:~/spinnaker/minnaker-0.1.3$ ls -lrth /home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc/operator/deploy/crds
ls: cannot access '/home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc/operator/deploy/crds': No such file or directory
ubuntu@ip-172-31-41-23:~/spinnaker/minnaker-0.1.3$
ubuntu@ip-172-31-41-23:~/spinnaker/minnaker-0.1.3$ cd /home/ubuntu/spinnaker/minnaker-0.1.3/spinsvc/operator
ubuntu@ip-172-31-41-23:~/spinnaker/minnaker-0.1.3/spinsvc/operator$ ls
halyard-local.yml  kustomization.yml  patch-config.yaml  patch-validations.yaml

What am i missing here? Or is this documentation too old to be ignored.

Intentionally redacted the Public IP and password from the above O/p

Same for me.

Same issue encountered today.

There is an error here: https://github.com/armory/spinnaker-kustomize-patches/blob/minnaker/deploy.sh#L90
Right now, you can specify the version directly, eg: SPIN_OP_VERSION=1.2.5 ./scripts/install.sh -o -P $PRIVATE_IP
Use the needed version instead of 1.2.5

./scripts/install.sh -o -P $PRIVATE_IP

I did by using version 1.2.5 and it should work but i get
[INFO ] Cloning repo: https://github.com/armory/spinnaker-kustomize-patches#minnaker into /home/testadmin/minnaker-0.1.3/spinsvc
[WARN ] /home/testadmin/minnaker-0.1.3/spinsvc exists already.  FOLDER CONTENTS WILL GET OVERWRITTEN!
[WARN ] PROCEEDING in 3 secs... (ctrl-C to cancel; use -B option to specify a different directory)
Cloning into '/home/testadmin/minnaker-0.1.3/spinsvc'...
remote: Enumerating objects: 2429, done.
remote: Counting objects: 100% (605/605), done.
remote: Compressing objects: 100% (224/224), done.
remote: Total 2429 (delta 454), reused 409 (delta 377), pack-reused 1824
Receiving objects: 100% (2429/2429), 599.07 KiB | 8.44 MiB/s, done.
Resolving deltas: 100% (1432/1432), done.
Every 2.0s: kubectl get pods,spinsvc -n spinnaker                                                                                hostname: Tue Jun 20 08:32:28 2023

NAME                                            READY   STATUS             RESTARTS   AGE
pod/redis-cf4c79fc5-mrvrb                       1/1     Running            0          5m38s
pod/pacrd-controller-manager-64759b7f7d-8b7mz   2/2     Running            0          5m37s
pod/opa-deployment-78648c74b9-9cn2l             2/2     Running            0          5m38s
pod/mysql-0                                     1/1     Running            0          5m38s
pod/minio-0                                     0/1     CrashLoopBackOff   5          5m38s

NAME                                      VERSION   LASTCONFIGURED   STATUS   SERVICES   URL
spinnakerservice.spinnaker.io/spinnaker


Serenity/JS 3.1.0 - Wait.for(Duration.ofSeconds(6)) fails with a TimeOutExpiredError after 5 seconds

Looks like there is a regression with the Wait implementation when using WebdriverIO (with Mocha).
I can reproduce this when I raise the timeout here from 2_500 ms to 6_000 ms.
From ./integration/webdriverio-web run npm run test.
I would expect the test to continue and then pass after 6 seconds. Instead it fails before the set timeout, after 5 seconds with
[test:acceptance] [0-0] TimeoutExpiredError in "Wait for pauses the actor flow for the length of an explicitly-set duration"
[test:acceptance] TimeoutExpiredError: Timeout of 5s has expired
[test:acceptance]     at /workspace/serenity-js/packages/core/src/screenplay/time/models/Scheduler.ts:13:98
[test:acceptance] [0-0] FAILED in chrome -<EMAIL_ADDRESS>
Looks like setting an interactionTimeout: Duration.ofSeconds(15) in the serenityconfig of wdio.conf.ts has no effect, either. Test fails after 5 seconds with the TimeOutExpiredError.

Beautiful catch, @viper3400, I can reproduce it.
There are two issues here, actually.
The first one is that WebdriverIOFrameworkAdapter doesn't pass the interactionTimeout property to Serenity.configure. That's why increasing the timeout has no effect and you're still seeing 5 seconds instead of 15.
The second one seems to be an issue with Wait.for(..). At the moment, if the timeout set with Wait.for(..) is longer than the max interactionTimeout, Wait.for(..) gets terminated when the interactionTimeout expires. On the one hand, this might be desired as you get to control all the max timeouts with interactionTimeout, but on the other hand, it feels unintuitive - if you tell the test to wait for 6 seconds, you'd expect it to wait for 6 seconds and perhaps not care much about the interactionTimeout


ignore .env if not a file

.env should only be loaded when an actual file
it was previously fixed by https://github.com/compose-spec/compose-go/pull/124
however it took into regression
(again) fix https://github.com/docker/compose-cli/issues/1575

see https://github.com/compose-spec/compose-go/pull/383


LDAP groups using "memberUid"

Hello,
Kanboard really is a nice program! I was wondering though: would it be possible to look for LDAP group memberships in groups (attribute memberUid) instead of users?

+1

It's not implemented, can you explain why you need that?

Well... The "MemberOf" attribute requires an overlay for OpenLDAP, which means I should modify my LDAP structure (which follows RFC2307 - posixGroup with memberUid). Maybe it's doable though.
But since the groups would have to be "groupOfNames" and "posixGroup", I'd have to maintain 2 attributes ("member" and "memberUid") to keep retro-compatibility with other programs.
I was afraid of modifying my LDAP structure, but if it must come to this, I can try.

Yeah its a different LDAP schema structure that is fairly widely used when not interfacing with an AD LDAP implementation.

Ok, I see. If I understand correctly, this is the default LDAP schema with a standard OpenLDAP installation.
I will see how hard is it to make it work.

I can confirm it's a different (opposite) way of describing membership.
I'm also using posixGroup with memberUid attributes (each groups nodes stores it's member list, referring to them using their uid).
Usually applications that supports both offers a configuration settings to use either one or the other way.
My 2¢.

Latest Kanboard now supports ActiveDirectory, OpenLDAP with memberOf, groupOfNames and Posix Groups.
There are 4 different examples on the website to help you to configure that: https://kanboard.net/documentation/ldap-configuration-examples


circt: add shared libs and split llvm

Description of changes
This changes adds circt's shared libs and splits the build of llvm.
This fixes the previous attempt #259725 where it was hard to split the output of circt and LLVM.
It now builds LLVM first using the submodule source from circt, and then using that result, builds circt.
The only downside is that the circt tests are not passing, because of the lack of llvm-lit. I may come back to this issue later.
cc: @SharzyL @NickCao @sequencer

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

https://github.com/chipsalliance/chisel/blob/main/nix/llvm-lit.nix
packaged llvm-lit we can upstream it as well?

@sequencer
Thanks for the information!
I'll check if I can add that to the existing LLVM build, or add it as a new package.

I updated the PR.
It now can test circt with llvm-lit.

@SharzyL Thanks for the review. I've updated the code.

@NickCao
Updated the PR.


Suggestion: show top and installed LLM for current api provider

When selecting for instance Ollama it would be great to see a few suggestions plus a list of what I have installed already for quick switching.

I added a llm selector option to the latest version.

cool! having this in the settings as well would be nice.

cool! having this in the settings as well would be nice.

Unfortunately it's not possible unless the list is statically defined at compile time. This is why I added it to the sidebar at runtime.


[BUG] Dialyzer error for my app, using PromEx

Describe the bug
My use PromEx, otp_app: :my_app is causing the following dialyzer error:
lib/drug_mule/prom_ex.ex:50:no_return
Function init/1 has no local return.

To Reproduce
Steps to reproduce the behavior:

Create a new phoenix app
Follow PromEx setup instructions

Expected behavior
no dialyzer warnings
Environment

Elixir version: 1.11.2
Erlang/OTP version:
Grafana version:
Prometheus version:

Additional context
Add any other context about the problem here.

Same here.
Erlang/OTP 23 [erts-11.1.6] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1]
Elixir 1.11.3 (compiled with Erlang/OTP 23)

Thanks for opening up the issue @djthread and thanks for verifying @stefanluptak ! I was able to root cause the issue and will have a fix out along with a new version in the coming days :).

Just published version 0.1.14-beta to Hex that includes the fix for this as well as a LiveView plugin and Grafana dashboard. Feel free to reopen the issue if the dialyzer error does not go away :).


Suggestion or Feedback : allow running ABAP cleaner from reviewer point of view

Dear Team,
My suggestion to allow running ABAP cleaner if we want to review the code without making any changes in the display mode.
I am not able to use the ABAP Cleaner in interactive mode as it is locked by some other user in the transport so cannot run the tool to check if there are any improvements like spelling mistakes or negation conditions used or any identation.
Please provide your views if it is possible or there is any other option to execute the tool to display the changes in the display mode even if the object is locked under some transport.
Thanks for the valuable tool which helps to improve coding.
Regards,
Gurmukh Singh

Hi Gurmukh, you can simply use the read-only preview to do this:

Hi Gurmukh Singh,
I guess the menu "Show Read-Only Preview" menu that ConjuringCoffee pointed out is exactly what you looked for – it was indeed originally proposed in #114 for the purpose of code reviews. Really a great idea, I'm also using it all the time! So I think I can close this issue.
Kind regards,
Jörg-Michael


LogTrans Implementation

Dear Authors,
Thank you for your great work and sharing the well-sorted codebase!
Regarding LogTrans, it seems that the authors didn't release their offical codes. Would like to know what implementation did you use and whether are you planning to release the implementation.
Thanks.

Sorry, we did not implement LogTrans. The result of that model is copied from Fedformer paper and Autoformer paper.


Prefix route variables with operation name when generating single file

This resolves #52 reported by @CTBT
The changes here will prefix the route variables with a unique operation name so generating a single file output will look something like this:
@contentType = application/json

#######################################
### Request: GET /pet/{petId}
### Summary: Find pet by ID
### Description: Returns a single pet
#######################################

### Path Parameter: ID of pet to return
@GetPetById_petId = 0

GET https://petstore3.swagger.io//api/v3/pet/{{GetPetById_petId}}
Content-Type: {{contentType}}

##################################
### Request: DELETE /pet/{petId}
### Summary: Deletes a pet
##################################

### Path Parameter: Pet id to delete
@DeletePet_petId = 0

DELETE https://petstore3.swagger.io//api/v3/pet/{{DeletePet_petId}}
Content-Type: {{contentType}}

Awesome, thanks.


New command New-DbaLogShippingPrimarySecondary

To test this function you need to have run the prerequisite function "New-DbaLogShippingPrimaryDatabase" to make sure the primary database is registered.
Use the following query to see if the values are written
SELECT * FROM msdb.dbo.log_shipping_primary_secondaries
The result should look something like the printscreen below:


Error while inspecting an item

Hey! Am getting this error when I try to inspect an item.

The latest build here should fix this: CS2Interface V<IP_ADDRESS>


Missing public setters for private instance variables includes and excludes in ExtraDirectoryParameters for gradle plugin

Environment:

Jib version: 3.2.1
Build tool: Gradle, 7.4.2
OS: N/A

Description of the issue:
Public setters are not available for private instance variables includes and excludes in com.google.cloud.tools.jib.gradle.ExtraDirectoryParameters.
See doc: path
When used in gradle with groovy dsl, it is possible to assign values but it breaks the intended access level.
Using Kotlin-dsl, it is not possible to assign the values (as intended) which breaks the desired functionality.
Expected behavior:
Proper public setters should be defined.

It sounds like you are no longer required to use set() with Gradle 8.2: https://blog.gradle.org/simpler-kotlin-dsl-property-assignment

As of Gradle 8.2, it is now possible to use the = operator to assign values to Property types as a direct alternative to the set(T) method in Kotlin DSL scripts.


Revert library name change

Libraries are locked to the name value specified by the library.properties metadata file in the release at the time of
the Library Manager submission.
Any release with a different name is rejected by the Arduino Library Manager indexer.
References:

https://github.com/arduino/Arduino/wiki/Library-Manager-FAQ#why-arent-releases-of-my-library-being-picked-up-by-library-manager
https://github.com/arduino/Arduino/wiki/Library-Manager-FAQ#how-can-i-change-my-librarys-name

Thank you.


Share A Single connection to the X11 Server/Display Object

Fixes #188

This extends #186

This sounds like a good change - I'd just like to understand it a bit more before merging.
Is the implication that we were using multiple X connections (to the same display) before?  Do you suspect that this is related to the BadWindow errors we've been seeing for ages, or is this simply another bad X practice that may be causing different errors?
Is there some general principle we should be learning from this in how we interact with X?  Are there other places in the codebase we should be worrying about this?

Is the implication that we were using multiple X connections (to the same display) before?

Yes. We initiate a new connection literally everywhere you can find a withDefaultCtx in the code right now.

Do you suspect that this is related to the BadWindow errors we've been seeing for ages, or is this simply another bad X practice that may be causing different errors?

I have a good understanding of why the bad window errors occur:
A window can exist when some type of update is triggered, but not exist later when we request information about that window.  One window is closed, triggering an update, which makes us fetch the list of windows, which WILL include the second of the two windows that closes nearly simultaneously. By the time we request window about this second window it does not exist which leads to the BadWindow error.

Is there some general principle we should be learning from this in how we interact with X?

Well, I think maybe what it indicates to me is that its time to take a look replacing the Xlib based x11 library that we are currently using with https://github.com/aslatter/xhb . Xlib is, IMO, a steaming pile of shit, and the way it handles errors is nearly impossible to deal with, especially when combined with the additional issues of dealing with the FFI with C. Its crazy that we can fully understand #105 and simply not be able to do anything about it because Xlib has such a terrible error handling system.

Are there other places in the codebase we should be worrying about this?

I didn't get rid of uses of withDefaultCtx in WorkspaceSwitcher because it would have taken a pretty huge amount of work (Look at how much refactoring I had to do in WorkspaceHUD), and I really think that WorkspaceHUD should eventually replace it (since it has complete feature parity with WorkspaceSwitcher). @teleshoes claimed that he still experiences bugs with WorkspaceHUD, and its hard for me to claim that they don't exist when I get crashes frequently (whether I use HUD or Switcher).
I think that almost all of the bugs are X11 related, but they are mostly very subtle and its not clear how to fix them. Again, I think that the answer might be to look at https://github.com/aslatter/xhb

This sounds like a good change - I'd just like to understand it a bit more before merging.

I've been using it for a while now, but it would definitely benefit from a bit of testing before merge. It can sometimes be hard for me to tell if the crashes I'm getting are new or not, because some of the crashes cause spurious, unrelated core dumps because they happen asynchronously.
@sakshamsharma @teleshoes @escherdragon any help testing would be appreciated.

@travitch ping?

This isn't available on cabal yet, I take it?

@djvs I imagine we will do another version bump for whatever makes it into this request. Probably 0.5.0 given the significance of changes on master.
Not sure what @travitch thinks.

@djvs its not really that hard to install taffybar from source though. Thats what I would recommend anyway.

Wow, loving the new version so far.

Still getting this error from git taffybar though:
taffybar-linux-x86_64: user error (openDisplay)

@djvs Really? Did you use --force-reinstalls when you ran cabal install? Are you sure you are using the newest version.

I'm using the pager, if that's the same as the WorkspaceSwitcher.  But no, I didn't do --force-reinstalls, so I'll report back if I see another crash after doing so.

@djvs no using the pager is the problem 100%, can you just trying using
taffyPagerHUDLegacy :: PagerConfig -> IO Widget
taffyPagerHUDLegacy cfg = do
  pgr <- pagerNew cfg
  whud <- buildWorkspaceHUD (hudFromPagerConfig cfg) pgr
  los <- layoutSwitcherNew pgr
  wnd <- windowSwitcherNew pgr
  sp1 <- separator cfg
  sp2 <- separator cfg
  box <- hBoxNew False 0

  boxPackStart box whud PackNatural 0
  boxPackStart box sp1 PackNatural 0
  boxPackStart box los PackNatural 0
  boxPackStart box sp2 PackNatural 0
  boxPackStart box wnd PackNatural 0

  widgetShowAll box
  return (toWidget box)

instead of taffyPagerNew

you'll need to import
import System.Taffybar.WorkspaceHUD

After patching up all those unqualified imports:

taffyPagerHUDLegacy :: PagerConfig -> IO Gtk.Widget
taffyPagerHUDLegacy cfg = do
  pgr <- pagerNew cfg
  whud <- buildWorkspaceHUD (hudFromPagerConfig cfg) pgr
  los <- layoutSwitcherNew pgr
  wnd <- windowSwitcherNew pgr
  sp1 <- separator cfg
  sp2 <- separator cfg
  box <- Gtk.hBoxNew False 0

  Gtk.boxPackStart box whud Gtk.PackNatural 0
  Gtk.boxPackStart box sp1 Gtk.PackNatural 0
  Gtk.boxPackStart box los Gtk.PackNatural 0
  Gtk.boxPackStart box sp2 Gtk.PackNatural 0
  Gtk.boxPackStart box wnd Gtk.PackNatural 0

  Gtk.widgetShowAll box
  return (Gtk.toWidget box)

Left wondering what "separator" is?

Left wondering what "separator" is?

It's a field in the config. I think its string/widget that is placed between the main widgets. Not really important.

Well, it compiles without those 4 lines.  Tried to copy the separator definition from here:
https://hackage.haskell.org/package/taffybar-0.3.0/src/src/System/Taffybar/TaffyPager.hs
which did work except for some type annotation issue.  Going without them for now.  Thanks!

Well, it compiles without those 4 lines. Tried to copy the separator definition from here:
https://hackage.haskell.org/package/taffybar-0.3.0/src/src/System/Taffybar/TaffyPager.hs
which did work except for some type annotation issue. Going without them for now. Thanks!
Delete branch

@djvs Eventually you should just use taffyPagerHUDNew,
but that will require you to come up with the HUDConfig for your workspaceHUD (see
https://github.com/travitch/taffybar/blob/788463f266ec74b277b9683348b2a6f5138c4224/src/System/Taffybar/WorkspaceHUD.hs#L127)
I would recommend you try adding Icons with workspaceHUD, they're pretty awsome.

https://gist.github.com/anonymous/a70f261bcf02dc82ee043e1122adc88a
My current taffybar.hs - there are icons next to my workspaces already:

Pretty nice.


setup which works for fpm

finally just created approximately appropriate setup args
please correct if I did something wrong
Fix #435

Hello @theromis. Thank you for the pull request. This project moved away from defining the setup in setup.py and instead keeps everything in setup.cfg. Do you know why FPM is having issues reading setup.cfg?
Also, for this PR to be accepted you need to ensure that the syntax issues are addressed.

Run pip install --upgrade pip setuptools wheel in a fresh virtualenv before installing this. I've made the conscious decision of not supporting ancient versions of those and that's probably what you're hitting.

@jleclanche  thank you, this solution works