As a Developer I would like to Update Terragrunt HCL files as part of spk infra update

As a:
Developer
I want:
to update Terragrunt HCL files based on args in a provided template folder (Leaf Deployments)
So that:
Terragrunt can update the configuration of Leaf deployments infrastructure.
Describe the solution you'd like:
spk infra update should provide an argument (e.g. --template hcl-file / or var) that will pass the path to the leaf deployment definition to either update the hcl file or a var within the hcl file
Additional context:
spk infra update --template "~/discover-service/cluster-west" --vars "[cluster-name=new-cluster-west-v2]"

[ ] Create spk infra update decorator
[ ] Validate init has been successfully ran
[ ] Pass a local template path and switch hcl files with a provided
[ ] Pass a local template path and a variable map to switch vars in HCL

Does this require updates to documentation?:
Yes
Tests Ticket: #696

Additional context:
Unit Tests - Invalid Vars mapping, No template found, Multiple var names
Documentation Update

spk infra update --template "~/discover-service/cluster-west" --vars "[cluster-name=new-cluster-west-v2]"

@NathanielRose what would it look like if someone wanted to update several vars in one command?

Create spk infra update decorator

Does this encapsulate all the implementation or just front end args? Meaning spk infra update will accept args but be a no-op. If the latter then that seems like your first place for unit tests.

spk infra update --template "~/discover-service/cluster-west" --vars "[cluster-name=new-cluster-west-v2]"

@NathanielRose what would it look like if someone wanted to update several vars in one command?

@andrebriggs the --vars arg is passed an array of variables. So something like spk infra update --template "~/discover-service/cluster-west" --vars "[cluster-name=new-cluster-west-v2, vnet=mynewvnet]"
Alternatively we can simply switch out the hcl file with one that you pass.  spk infra update --template "~/discover-service/cluster-west" --hcl "path/to/new/terragrunt.hcl"

Create spk infra update decorator

Does this encapsulate all the implementation or just front end args? Meaning spk infra update will accept args but be a no-op. If the latter then that seems like your first place for unit tests.

no-op and agreed!

@andrebriggs the --vars arg is passed an array of variables. So something like spk infra update --template "~/discover-service/cluster-west" --vars "[cluster-name=new-cluster-west-v2, vnet=mynewvnet]"

@NathanielRose are we expecting that clients will write this manually on the command like or script this with automation?
Also how many work days do you think effort - M is?

@andrebriggs the --vars arg is passed an array of variables. So something like spk infra update --template "~/discover-service/cluster-west" --vars "[cluster-name=new-cluster-west-v2, vnet=mynewvnet]"

@NathanielRose are we expecting that clients will write this manually on the command like or script this with automation?

For quick variable changes, they may want to use args, for an extensive amount of vars to be replaced on 1 template they can switch the .hcl file. For iteration 1 they would have to handle each update command for that respective template through a script. Down the road, we want to add that scaling mechanism.

Also how many workdays do you think effort - M is?
M= 3-4 Days

@NathanielRose still valid?

@andrebriggs Since we're handling upgrading templates through spk generate, I dont think this is needed atm. Will put on-hold then close if folks feel otherwise.

closing to do change in design


Port fix from devel to 3.7.

This is about hotbackup and blockages due to the fact that a leader
transaction might have already started but the follower transaction
can no longer start because of the hotbackup holdTransactions. The fix
is to only hold leader transactions.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/11417/

Test run is green.

This is good to go.


Covers

thread state update and restore (mode, base-priority, swap-return value)
callee-saved registers' preservation and restore
extension for floating-point support

Currently rebased on top of #17456

@pizi-nordic what do you think about this functional test for the arm swap routine? Any early feedback is appreciated

Have a look at https://github.com/zephyrproject-rtos/zephyr/pull/17456, it should fix the compiler warning.

IMHO what is missing is a high level description of the idea behind the test.

IMHO what is missing is a high level description of the idea behind the test.

Do you think I sould add a README? Or an intro paragraph in the  test source file?
We do have such things in some tests, but it is not a formal requirement

IMHO what is missing is a high level description of the idea behind the test.

Do you think I sould add a README? Or an intro paragraph in the test source file?
We do have such things in some tests, but it is not a formal requirement

Just comment in the file. It might be also in emphasized comment in function like this:
/*
 * STEP 3: Verify if all registers were preserved
 */

/* Verify callee-saved registers */
/* Verify FPU registers */
/* Verify BASEPRI */
..

IMHO what is missing is a high level description of the idea behind the test.

Do you think I sould add a README? Or an intro paragraph in the test source file?
We do have such things in some tests, but it is not a formal requirement

Just comment in the file. It might be also in emphasized comment in function like this:
/*
 * STEP 3: Verify if all registers were preserved
 */

/* Verify callee-saved registers */
/* Verify FPU registers */
/* Verify BASEPRI */
..

I already added a README description,    @pizi-nordic , please, take a look.

@pizi-nordic @SebastianBoe could you do another (hopefully) quick review round?

Place consider adding irq_lock() around all code except places where you should be pre-empted.

Done, as well, thanks @pizi-nordic for reviewing.


(MCO-775) Ping application should include discovery_timeout

This is a much more limited alternative to https://github.com/puppetlabs/marionette-collective/pull/397

still seems to break the #1 rule for me - do what the user says.  If he gives --timeout 10, it should take 10 seconds.  If not, sure mess around and adjust it to what make sense given the environment.
So I'd look at limiting this to cases where the user did not specify a timeout at all. There's no proper way to figure this out but when not given it would be 5 in the options, thats what RPC::Client also uses for this

👍

I added a new commit to only include discovery timeout if the timeout is the default. If that seems better I'll squash it down.

CLA signed by all contributors.


Fix the components of Spring resttemplate is incorrect

Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues
#1575

Coverage decreased (-0.002%) to 13.022% when pulling b627677345f7f2ad0de8d242e17aa048d4d5b3f7 on ascrutae:features/spring-commons into e697385b5380c775b02ab46d6d5cefe2c2775c18 on apache:master.


Modin's Series.sort_values does not validate types of arguments, when pandas do

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Any
Modin version (modin.__version__): dd91a78ad3f4b8e3e569215e9c8e540ad099d4a8
Python version: 3.7.10
Code we can use to reproduce:

from modin.pandas.test.test_series import create_test_series

md_df, pd_df = create_test_series([1, 2, 3, 4])

try:
    # `sort_values` takes only bools as `ascending` parameter
    repr(md_df.sort_values(ascending=None))
except Exception as e:
    print(f"Modin risen the following exception:\n\t{type(e)}: {e}")
else:
    print("Modin didn't raise any exceptions")

try:
    # `sort_values` takes only bools as `ascending` parameter
    repr(pd_df.sort_values(ascending=None))
except Exception as e:
    print(f"Pandas risen the following exception:\n\t{type(e)}: {e}")
else:
    print("Pandas didn't raise any exceptions")

Output:
Modin didn't raise any exception
Pandas risen the following exception:
        <class 'ValueError'>: ascending must be boolean

Describe the problem
Although the validation is not performed the Modin properly handles ascending=None parameter by sorting values in descending order, so this issue is just about exceptions concordance.

This bug no longer exists on master - marking as resolved!


Same result from Array#hash if it contains to equal elements

I'm currently tinkering around with Hash#hash and noticed this bug in Array#hash:
[2, 2].hash == [1, 1].hash
# => true

[2, 2].hash will return 100019 which is the starting point of the hash calculation, because (2.hash * 207269) ^ (2.hash * 207269) will eliminate each other. My first thought of fixing this is multiplying with a random prime number instead of a fixed one. Would this work?

Whoops! @TheGrizzlyDev and I worked on that, but that mistake is mine. The recursive array specs are mind bending, but I feel confident adding a random number won't fix it. 😕
I will try to take another look at this soon.

I feel confident adding a random number won't fix it. confused
I feel like we've been here already ahhahaha. A similar but less naive approach would be to just shift at each iteration :)

@TheGrizzlyDev you're a genius.  (That worked.)

Ahahahaha thanks! I've spent enough time on it to think in hashes

Thanks for fixing this so fast! 😄 Sadly, I think this is not enough yet. Firstly it does not work with nested arrays:
[[1, 2], [3, 2]].hash == [[1, 3], [2, 3]]
# => true

The two 2s and the 3s are eliminating each other.
Secondly, as nat_int_t is "only" 64 bit, it will not work if we shift more than 64 times (when the array is bigger than 64):
def generate_array(i)
  [].tap do |out|
    66.times { out << i }
  end
end

generate_array(2).hash == generate_array(3).hash
# => true

Bummer. Well I'll have to take another stab at it later. If you have ideas, by all means, go for it. 😉

this keeps getting better and better ahahhahah. How about we take a page from DES/AES which have the same issue with using multiple rounds and having just a single key (in this case our prime number)?
We could:

select a big enough prime number (or any number big enough that if taken in smaller bits would still have a somewhat reasonable distribution of 0s and 1s and at least with more bits than nat_int_t)
for each item we get the hash of, we xor it with a prime number AND then xor it with a subkey of the aforementioned number of 64 bits in length
we do a circular shift (not sure that's the name of it, but basically a shift where a digit 'leaving' on one side would just 'come back' from the other) and then repeat the process for the next item?

PS: I hope you find more issues if this fixes the above, absolutely loving it :D

PPS: the above as an issue with periodicity, as in after a full-rotation of the key value would start cancelling each other out again. A simple solution to that would just be having another key, used the same way and with the same property but with a different length so that collision only happen after N (length of first key) * M (length of the second key) iterations. Long enough keys would cover any practical purpose. Again, let me know if this also misses some scenarios as it is so fun :)

Naive question: What speaks against adding the hashes instead of xoring them? We would not have the problem with eliminating entries than. However, I see overflow being a problem maybe?

Try it and see! 😉 (I honestly don't remember why I chose xor -- probably Stack Overflow told me to lol 😆)

When overflowing you'd end up just having 1s in the rightmost part of the hash, therefore making collisions more likely. It'd work and be simple but would not be as effective I guess as the range of collisions would just be on the lower end of the bit array... I wouldn't mind that to be honest depending on how and where hash is used. @seven1m you probably know that better, but I reckon it wouldn't be too bad just for hashes :)

How about subtractions though? Ordering is implicitly considered as in non-commutative as addition is, it doesn't cancel out the previous value but rather "adds" to it unlike xor and wouldn't overflow in the same way as additions which would lead to a less effective distribution of 0s and 1s

OK, I took another stab at it. The main problem was that I was calling Array::eql and treating it like a bool (forgot to call is_truthy()), so that was half the battle. I also decided to xor the array index and added MOAR PRIMES lol. :joy:
Anyway, let me know if y'all find any more weird cases on this one @richardboehme @TheGrizzlyDev. <3

this implementation should fail with repeating values and since xor is commutative and indexes are values so 'close' to each other they will end up cancelling each other out far too often. The second one is really easy to demonstrate:
00 XOR
01 XOR
10 XOR
11 =
00 => so with a given periodicity, xor-ring the index will cancel the final result of the whole operation out, so it pretty much provides no value in terms of avoiding collisions (hence why I suggested circular shifting before - not sure circular shifting is an actual name or I just came up with a term to explain what I am imagining, in which case please let me know hahahaha)
The first issue though is a bit harder, so I will make an example script to prove it :)

Created a script to demonstrate this since I am too lazy to prove it otherwise :D
found_hashes = []
hashes = (0..100).map do |length|
    current_hash = (0..length).to_a.map { |x| 0 }.hash
    found_index = found_hashes.find_index current_hash
    if found_index
        p "Found a repetition with length = #{length} and the previous length = #{found_index} => #{current_hash}"
    else
        found_hashes << current_hash
    end
end

with 101 tests there were only 61 unique values and the greater this number goes the higher collisions should get, in theory getting closer and closer to 50% of collisions

I'm leaving this open for a few days until @TheGrizzlyDev decides to stop poking at it and moves on. :laughing: :laughing:

about that, done some preliminary testing and found out that:

this implementation of hash collides far less than the MRI one :D
we can improve a lot just by shifting the amount of digits we shift (my theory was right on how that affects the output, but the guess was really bad ahahahaha) and this should be easy enough :^)
the implementation I just pushed of Array::repeated_combination is unbearably slow, especially when compared with MRI's one, so we should try to improve that as well to get a better test bench (getting repeated_transpositions would probably help as well)

I'll go a bit deeper tomorrow and maybe try to improve it all, but there's quite a few quick wins here and this has been quite fun :^)

There's so much in way of performance optimization we can do, first and foremost will be a method cache. But we'll have plenty of time for that... 😉


@numba.jit not handling integer parameters properly

Hi, I don't know what the problem can be, but I'm getting different results for the following code
@numba.jit
def paint(r, t):
    drawn = 0

    while (r+1)**2 - r**2 <= t:
        drawn += 1
        t -= (r+1)**2 - r**2
        r += 2

    return drawn

If my script contains only one call to
paint(10000000000000000,<PHONE_NUMBER>000000000)
I get 49 as the result, which is the right answer.
Unfortunately, If my script calls paint with small integers as parameters, like:
paint(1, 10)
paint(10000000000000000,<PHONE_NUMBER>000000000)
The second call returns 51 as the result.
Strangely, if I change the second line of the while loop to
t -= 2*r + 1

The function behaves well for both cases.
I'm using:
Python 2.7.6 :: Continuum Analytics, Inc.
conda 3.4.1
numba 0.13.1

This is now correctly handled as the handling of integer arguments has changed (see NBEP 1: Changes in integer typing).


Restrict Mentor registration to only approved emails

Description
as title states

I can take this one once the lock gallery is merged

I can take this one once the lock gallery is merged


Can’t sign in using external URL

Go to settings page (only place can sign in with external URL given #57)
Select ‘WordPress or Micropub weblog
Tap ‘Sign in’ next to External Weblog
Enter URL
Complete IndieAuth flow
Redirected back to Sunlit

‘Micro.blog hosted weblog’ is still selected. Cannot post to external website.

Thanks @paulrobertlloyd. If you close the settings screen and open it again, does it show your external blog info?

@manton Nope. It resolutely stays stuck on ‘Micro.blog hosted weblog’ 😕

I had this issue this morning too.

@paulrobertlloyd @smithtimmytim What kind of blog software are you using?

Should be fixed in the next beta.


Feat: Add bridge-test feature flag for movement-bridge lib test functions

Summary

A new bridge-test feature flag is applied to test functions in the movement-bridge crate.

Changelog

Define feature flag in protocol-units/bridge/chains/movement/Cargo.toml
Add feature flag dependency declaration into project root Cargo.toml
Add #[cfg(feature = "bridge-tests")] annotation to three test functions in protocol-units/bridge/chains/movement/src/lib.rs

Testing
Passes tests, eg:
rust_backtrace=1 cargo test --test eth_movement --features bridge-tests -- --nocapture --test-threads=1

Outstanding issues

There's a CI/CD MCR check failure happening on globally when merging PRs. Curious as to whether it affects this PR.
In the example test command and others similarly structured, the --features option must be placed before the -- so it's considered a Cargo option.

As agreed with @andygolay we will close this and re-implement the feature flagging due to large changes in main.


Update card UI

Description
Hi @raj03kumar, I am a contributor in GSSOC'23,
I want to work on Improving layout of card.
I will add some margins and padding between title, desc, and button
please assign me this task under label gssoc'23
Screenshots

Additional information
No response

I want to update a card UI my design looks like this

on hovering the image

if you like my design please assign this issue to me under gssoc 2023. I want to work on this.


fix: revert using cookies package to fix cookie headers handling

PR #327 introduced a significant increase of vendors bundle size, as reported in issue #330.
The root cause was the replacement of the cookie package with the cookies one, while fixing a severe bug in Set-Cookie headers handling for the SSR language redirection cookie.
This PR reverts to using the cookie package but performs manually the Set-Cookie header handling, exactly the way the cookies package does (i.e. no previous cookie, a single cookie or a list of cookies).
Hopefully, this one will do the trick!

LGTM

I've squashed this with your previous change locally to see changes relative to pre-cookies-package change and got this simple diff:
diff --git a/src/plugins/main.js b/src/plugins/main.js
index d916954..35f3b34 100644
--- a/src/plugins/main.js
+++ b/src/plugins/main.js
@@ -66,11 +66,18 @@ export default async (context) => {
         path: '/'
       })
     } else if (res) {
+      let headers = res.getHeader('Set-Cookie') || []
+      if (typeof headers == 'string') {
+        headers = [headers]
+      }
+
       const redirectCookie = Cookie.serialize(cookieKey, locale, {
         expires: new Date(date.setDate(date.getDate() + 365)),
         path: '/'
       })
-      res.setHeader('Set-Cookie', redirectCookie)
+      headers.push(redirectCookie)
+
+      res.setHeader('Set-Cookie', headers)
     }
   }

Makes it easier to evaluate changes.

Awesome! Thank you for the PR @rclement! And thank you for posting the diff @rchl! LGTM.


ISSUE:

Describe the bug
package not working after installing the latest version ... all sensors unavailable
Environment (please complete the following information):

OS: [e.g. HassOS/Raspbian/CentOS] Ubuntu
Type of device: [e.g. RPi3/NUC/Synology] Intel NUC
Home Assistant version: [e.g. 0.105.5]  2021.10.7
Hassio/Docker/Core? Core
Component version: [e.g. 0.1.2] 0.3.4-b5

Logs
Logger: homeassistant.setup
Source: setup.py:178
First occurred: 6:10:08 AM (1 occurrences)
Last logged: 6:10:08 AM
Setup failed for mail_and_packages: Unable to import component: cannot import name 'ENTITY_CATEGORY_DIAGNOSTIC' from 'homeassistant.const' (/srv/homeassistant_3.8/lib/python3.8/site-packages/homeassistant/const.py)
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Please add emails in plain/text format if possible and applicable.

rolled back to 3.3-2. works fine

Per the b4 release notes:
Home Assistant version 2021.11.x or higher required due to utilizing Entity Descriptions


Move read me info to dashtag-solo

dashtag-solo is the new home for a ready to deploy application to heroku, etc. We need to move the read me on dashtag there so it stays relevant

Just finished it https://github.com/anirudh-eka/dashtag_solo


Issues with the reportback-items API endpoint on staging servers

Depending on which staging url we direct the request to, the response is either a 503 error or slow enough (~40s) to timeout the connection on the client-side (ex: mobile app).
staging.dosomething
Request: GET http://staging.dosomething.org/api/v1/reportback-items.json
Response:
Error 503 backend read error

backend read error

Guru Mediation:

Details: cache-iad2123-IAD<PHONE_NUMBER> 365296685

Varnish cache server

staging.beta.dosomething
Request: GET http://staging.beta.dosomething.org/api/v1/reportback-items.json
Response: The response is correct. The response time though is ~40s which currently times out connections on the mobile app.
More notes

Both http://staging.beta.dosomething.org/api/v1/campaigns.json and http://staging.dosomething.org/api/v1/campaigns.json work and respond quickly
Part of the /reportback-items endpoint's response is a user object like...

"user": {
  "id": "55bcf222469c64a31f8f5555",
  "first_name": "Ansley",
  "last_name": "Something",
  "photo": "<some url on AWS or null>",
  "country": "<some country code or null>"
}

If I'm remembering correctly, Phoenix may be making requests back to Northstar to get this information... oh, yep, pretty sure it is. Not sure if that has anything to do with the issues: https://github.com/DoSomething/phoenix/blob/dev/lib/modules/dosomething/dosomething_reportback/includes/ReportbackItem.php#L119

Also worth mentioning, the image uri's that come back in the staging.beta response return 404s.
ex:
Response from the staging.beta reportback-items request...
{
  "id": "22682",
  ...
  "media": {
    "uri": "http://staging.beta.dosomething.org/sites/default/files/styles/480x480/public/reportbacks/2930/uid_1835306-nid_2930-0.jpg?itok=-Vrp73yB",
    "type": "image"
  },

And then we pull this out to try to get the photo to display:
http://staging.beta.dosomething.org/sites/default/files/styles/480x480/public/reportbacks/2930/uid_1835306-nid_2930-0.jpg?itok=-Vrp73yB

@jonuy Not sure why the images are 404'ing, but the API responses are back to normal now that the DB switch is back to normal on staging.
I'll investigate the image 404's a bit more.

@sheyd It's probably not worth investigating the 404 anymore. After switching the DB back, the uri's in the responses when querying now look like they're good.

Roger.


Router Regex not working with controllers

Describe the bug
The problem seems to be in the router. My code is like this doesn't work. Error 404.
$routes->group('admin', ['namespace' => '\Admin\Controllers\Auth'], function ($routes){
    $routes->add('(.*)/(.*)', '$1/$2');
});

According to the guide, it should be possible to pass the controller and method name from URL during this. I tried and it works to pass it as method argument and display it in the controller.
It works like this to redirect everything from /admin/* to some defined controller in another namesapce defined like below.
$routes->group('admin', ['namespace' => '\Admin\Controllers\Auth'], function ($routes){
    $routes->add('(.*)', 'myController');
});

This doesn't worka as well. Error 404
$routes->group('admin', ['namespace' => '\Admin\Controllers\Auth'], function ($routes){
    $routes->add('(.*)/(.*)', '$1::$2');
});

Since it doesn't make log when accessing not existing controller. I am assuming it tries to pass segments as arguments to not defined controller all the time and can not pass it as controller name.
CodeIgniter 4 version
4.0.0 rc-2 - develop branch
Affected module(s)

Router

Expected behaviour, and steps to reproduce if appropriate
With this code, I expect to pass the first segment after "admin" as controller name and second segment as the method with the defined namespace. This should work according to the user guide.
$routes->group('admin', ['namespace' => '\Admin\Controllers\Auth'], function ($routes){
    $routes->add('(.*)/(.*)', '$1/$2');
});

Context

OS: Windows 8.1 x64
Apache 2.4.41
PHP 7.3.9

First off - thanks for filling out such a thorough bug report. That is much appreciated!
However, we recently revised the user guide to not show that anymore. Not only does it not work, it's not a good idea. See this issue for more details on why.
It sounds like you're wanting the CI3 style of auto-routing based on controller/method. Good news is - that still works, you just have to make sure your namespacing is correct for the controllers in your app/Controllers directory.

First off - thanks for filling out such a thorough bug report. That is much appreciated!
However, we recently revised the user guide to not show that anymore. Not only does it not work, it's not a good idea. See this issue for more details on why.
It sounds like you're wanting the CI3 style of auto-routing based on controller/method. Good news is - that still works, you just have to make sure your namespacing is correct for the controllers in your app/Controllers directory.

Thank You very much for the explanation but I cannot make it work anyway.
In my setup I have two namespaces.

Admin - my shared 'app' that should have access controllers from any app
App - my primary namespace accessing from index.php

if I want to make my app access controllers for Admin namespace it simply doesn't work. If I set properly controller in directory admin/Controllers it will throw 404 error and I have it in config autoload to include Admin namespace as well so I tried to do it this way.

Just a thought,
maybe you missed setting \Admin\Controllers\Auth  path in \App\Config\Autoload.php, that is provided Admin is under App.
Like so:
$psr4 = [
	'Config'         => APPPATH . 'Config',
	APP_NAMESPACE    => APPPATH,  // For custom namespace
	'App'            => APPPATH,  // To ensure filters, etc still found,
	'Myth\Auth'      => APPPATH . 'ThirdParty/myth/auth/src',
        'Admin\Controllers\Auth' => APPPATH . 'directory_where_the_namespace_is',
		];

Just a thought,
maybe you missed setting \Admin\Controllers\Auth path in \App\Config\Autoload.php, that is provided Admin is under App.
Like so:
$psr4 = [
	'Config'         => APPPATH . 'Config',
	APP_NAMESPACE    => APPPATH,  // For custom namespace
	'App'            => APPPATH,  // To ensure filters, etc still found,
	'Myth\Auth'      => APPPATH . 'ThirdParty/myth/auth/src',
        'Admin\Controllers\Auth' => APPPATH . 'directory_where_the_namespace_is',
		];

        $psr4 = [
            'Config'            => APPPATH . 'Config',
            'Admin'             => ADMINPATH,
            'Admin\Auth'        => ADMINPATH. 'Auth',
            APP_NAMESPACE       => APPPATH,                // For custom namespace
            'App'               => APPPATH,                // To ensure filters, etc still found,
        ];

nope, this is set properly :) According to user guide this is not possible now. Auto-discovery is only for Events, Registrars, Route files, Services and controllers ave to be specified all the time that doesn't make sense. This restriction I do not understand why is here.

I understand you set a namespace of Admin\Auth in autoload and in the Controller you're using Admin\Controllers\Auth (which I guess will not be found).


Can't support electron version 16.0.0

Can't support electron version 16.0.0

I'm having the same issue

Hello, brother, do you have no permission to open it

open it,you will be ok

@augustnmonteiro


consoles: exclude iowait and steal from CPU Utilisation

'iowait' and 'steal' indicate specific idle/wait states, which shouldn't
be counted into CPU Utilisation. Also see
https://github.com/prometheus-operator/kube-prometheus/pull/796 and
https://github.com/kubernetes-monitoring/kubernetes-mixin/pull/667.
Per the iostat man page:
%idle
Show the percentage of time that the CPU or CPUs were idle and the
system did not have an outstanding disk I/O request.
%iowait
Show the percentage of time that the CPU or CPUs were idle during
which the system had an outstanding disk I/O request.
%steal
Show the percentage of time spent in involuntary wait by the
virtual CPU or CPUs while the hypervisor was servicing another
virtual processor.
Signed-off-by: Julian Wiedmann<EMAIL_ADDRESS>

... cleaned up the consoles/node.html change to avoid the open-coded sum.

cc @SuperQ WDYT here? In general I want to know when my CPU is busy in IOWait

@roidelapluie @SuperQ any opinions on this? Keeping this is sync with the K8s mixin seems good. Maybe a separate graph for iowait would be better than including it in the CPU utilization?

The iowait data shouldn't be counted against "utilization" IIRC. But I'm not sure if steal is accounted for in idle or not. It's a question of figuring out what's the total system CPU seconds per second.

Discussed again at the bug-scrub.
@SuperQ seems like this is low impact, and has been merged in the mixin, so please go ahead and merge unless you feel more work is needed.

oh wow, blast from the past :slightly_smiling_face:. Ty! My former colleagues at Big Blue should be happy.
(@jschintag :wave:)


PMC6368841 KEGG analysis

https://github.com/markziemann/enrichment_recipe/blob/8a068145cb39f4b44828d3838f06a810e4d2ffb4/manuscript/manuscript.Rmd#L243
From the paper methods section, they stated that KEGG was done with clusterProfiler - should we reclassify this study?

Thank you for checking them again...
You are right, I rechecked with my lab notebook results and DAVID results seem to be consistent. I think this can be put into the 3 - good results.

I know both Clusterprofiler and DAVID will give different results, but then how can we know which tool would be the best to use? We can maybe ask why clusterprofiler was preferred over DAVID?

Sorry, but I think we should shift this one to 3 (high) as if I just consider GO terms - they were similar and DAVID was just used for GO and not KEGG.
Yes, they have mixed the tools in the figures - so I can definitely add those lines.
For column B: A. Only GO Terms were the similar as the paper. The FDR values are inconsistent. However, the paper should have clearly mentioned the different tools under figure captions to bring more clarity of the terms obtained by respective tools like DAVID and ClusterProfiler. B. No
For column C: 3 (high). GO results from the original manuscript are somewhat similar to reproduced analysis in DAVID. The FDR values however are inconsistent. The list of genes as input into DAVID analysis is unclear. The paper should have mentioned that both up and down-regulated proteins were considered for DAVID analysis.
I have added some more points like why were two different tools used for GO and KEGG when both can be done in DAVID or Clusterprofiler...


Topics: update CTV to cover 4+ years of discussion

Alternative to #1811
The current text for the CTV page entirely dates from May 2020.  This complete rewrite of the page provides additional details about it, both from the original proposal and from the subsequent years of discussion.

CC @jeremyrubin

bitcoin clique probably worth a mention https://eprint.iacr.org/2024/025
CTV Miniscript support and application languages Sapio and Minsc probably worth a mention.
@shesek's wallet was one of my favorite vault demos, especially in showing that there are many different models for security that might be desirable.
https://docs.google.com/presentation/d/1B59CdMIXW-wSW6CaLSgo7y4kvgrEwVgfY14IW2XV_MA/edit#slide=id.g1235f9ffb79_0_73
https://github.com/shesek/plebfi2022-social-recovery

Made edits for all line feedback (or left a comment explaining why I didn't change it).  Leaving this comment unaddressed, as it mostly seems to include things that we haven't discussed in Optech before (except Sapio, which is linked in the Optech Mentions section).  I do think that mentioning that stuff would be useful, but I don't have the bandwidth to read and analyze it all right now.

Made edits for all line feedback (or left a comment explaining why I didn't change it). Leaving this comment unaddressed, as it mostly seems to include things that we haven't discussed in Optech before (except Sapio, which is linked in the Optech Mentions section). I do think that mentioning that stuff would be useful, but I don't have the bandwidth to read and analyze it all right now.

@JeremyRubin do you want to take a crack at summarizing this remaining piece in this PR now? Or we can merge what we have (assume you agree with the latest updates from @harding) now and augment in a future PR?


Allow base-4.13 and time-1.9

Allows the versions bundled with GHC 8.8.1-alpha1. Library, tests and benchmarks run fine.

I couldn't install QuickCheck with GHC 8.8.1-alpha2 because splitmix doesn't support GHC 8.8.1. It seems this PR fixes the problem.
Blocking https://github.com/agda/agda/issues/3725.

use --allow-newer. I'm not making release until GHC-8.8 is a thing.

Superseded by https://github.com/phadej/splitmix/pull/20 and released in https://hackage.haskell.org/package/splitmix-0.0.3


Adding Wilson's school to the list

Adding Wilson's School to the list.
More details about our school's computing facilities given in the URL below.
https://www.wilsons.school/curriculum/dept/computing/

@damalcheruvu Thank you for your patience while waiting for a review and processing your request. Pull request merged.


How to create auto increment id field?

I want to define an auto increment ID field, just like the primary id in mysql.
Is that possible?
If not, how can I simulate this pattern? I need to have a unique ID of each row in table.

You can use the system column called rowid. it is a bigint datatype column that starts at 0.
But unloading and reloading the table data a different rowid could be assigned to the previous row content

Understood, thanks.


chore: update clap 4 argument parser

Motivation
update to clap 4
ref https://epage.github.io/blog/2022/09/clap4/
fix breaking changes,

mostly renames like ArgEnum -> ValueEnum,
renamed functions for min/multiple values.
Exact field names for requires
rm deprecated attributes like AppSettings

https://github.com/clap-rs/clap/blob/master/CHANGELOG.md

Solution

added <bin> <subcmd> --help tests for all subcommands, so there are at least not panics like we saw after the last upgrade


set settings in variant

Hi,
From the documentation I was under the impression that one could set settings on each variant (the ones that are passed to cmake as -D). Like this:
{
  "variant": {
    "label": "Debug",
    "keywordSettings": {
      "buildType": "debug"
    },
    "settings": {
          "OPT1" : "1",
          "OP2" : "XYZ"
      },
    "description": "Emit debug information without performing optimizations"
  },

  "activeEnvironments": [
    "Visual C++ 14.0 - amd64_arm"
  ]
}

Right now I'm able to pass those by adding "cmake.configureSettings" to the settings.json under vscode directory.
I find it very useful that settings per variant exist.
Appreciate your explanation on the above and keep up the good work with this awesome extension. :)

That appears to be the .cmaketools.json. That file is generated automatically, and shouldn't be edited. Have you taken a look at the documentation on cmake-variants.json?

You are correct on the file! I completely missed the diferences in the file name...
I now have a cmake-variants.json file like the one bellow
{
    "buildType":
    {
        "default$": "debug",
        "description$": "the build type to use",
        "debug":
        {
            "oneWordSummary$": "Debug",
            "description$": "Emit debug information without performing optimizations",
            "buildType": "Debug",
 	    "settings": {
          	"OPT1" : "1",
          	"OP2" : "XYZ"
      	     },
        },
        "release":
        {
            "oneWordSummary$": "Release",
            "description$": "Enable optimizations, omit debug info",
            "buildType": "Release"
        },

        "minsize":
        {
            "oneWordSummary$": "MinSizeRel",
            "description$": "Optimize for smallest binary size",
            "buildType": "MinSizeRel"
        },

        "reldeb":
        {
            "oneWordSummary$": "RelWithDebInfo",
            "description$": "Perform optimizations AND include debugging information",
            "buildType": "RelWithDebInfo"
        }
}

but the result is the same... I can't get those settings to be passed to cmake. Only if I add "cmake.configureSettings" to the settings.json.
I'm I missing something else again?
Thanks!

In your example, those options will only be passed when you have selected the Debug build variant, so they won't be passed with Release/Debug/MinSizeRel. You can put then into a new variant setting, like so:
{
  "buildType": {
    "default$": "debug",
    "description$": "the build type to use",
    "debug": {
      "oneWordSummary$": "Debug",
      "description$": "Emit debug information without performing optimizations",
      "buildType": "Debug"
    },
    "release": {
      "oneWordSummary$": "Release",
      "description$": "Enable optimizations, omit debug info",
      "buildType": "Release"
    },
    "minsize": {
      "oneWordSummary$": "MinSizeRel",
      "description$": "Optimize for smallest binary size",
      "buildType": "MinSizeRel"
    },
    "reldeb": {
      "oneWordSummary$": "RelWithDebInfo",
      "description$": "Perform optimizations AND include debugging information",
      "buildType": "RelWithDebInfo"
    }
  },
  "foo": {
    "default$": "bar",
    "description$": "The build settings",
    "bar": {
      "oneWordSummary$": "Primary",
      "description$": "Primary build Settings",
      "settings": {
        "OPT1": "1",
        "OP2": "XYZ"
      }
    },
    "baz": {
      "oneWordSummary$": "Secondary",
      "description$": "Secondary build Settings",
      "settings": {
        "OPT1": "2",
        "OP2": "ABC"
      }
    }
  }
}

This will cause there to be eight variants: Debug + Primary, Debug + Secondary, Release + Primary, Release + Secondary, etc.
Note that you won't see the options you specify in the command line for CMake that is in the output window, but they are still passed to CMake using the InitializeCache.cmake file, which is generated by CMake Tools each time CMake is rerun.

That makes sense.
Another issue: I can't see any other options on the drop down except the usual 4 (Debug, Release, MinSizeRel and RelWithDebInfo)
I've copied exactly the code above into the cmake-variants.json files.
Is there anything else that needs to be done to (refresh?) the options that show when the cmake-variants.json content changes?

Where did you place the cmake-variants.json file? I need to clarify in the documentation: It must be placed in the root workspace directory, not in the .vscode directory.

Yes, I had if on the .vscode directory. Just moved it to the root and now it works as expected!
👍  for updating the documentation and clarifying this.


darwin: problem with chmod from coreutils-8.25

Issue description
i try to install a modified package for boot (pkgs/development/tools/build-managers/boot).
I want it to use JDK 8 instead of 7.
But when it comes to building boot it fails with the following error:
installing ‘boot-2.5.2’
these derivations will be built:
  /nix/store/jalm2jiygcgb1qxdmvvrab20gxmwfn7c-boot-2.5.2.drv
building path(s) ‘/nix/store/34a74ckhbba5k97wsbyn7phs6ficj9j4-boot-2.5.2’
mkdir: created directory '/nix/store/34a74ckhbba5k97wsbyn7phs6ficj9j4-boot-2.5.2'
mkdir: created directory '/nix/store/34a74ckhbba5k97wsbyn7phs6ficj9j4-boot-2.5.2/bin'
'/nix/store/0592q8kpdgg2wygmmv4qbgxxkmqcaq20-boot.sh' -> '/nix/store/34a74ckhbba5k97wsbyn7phs6ficj9j4-boot-2.5.2/bin/boot'
dyld: lazy symbol binding failed: Symbol not found: _fchmodat
  Referenced from: /nix/store/mhygbibbr31b6dqwxy1iqaj7g4lz03zj-coreutils-8.25/bin/chmod
  Expected in: /usr/lib/libSystem.B.dylib

dyld: Symbol not found: _fchmodat
  Referenced from: /nix/store/mhygbibbr31b6dqwxy1iqaj7g4lz03zj-coreutils-8.25/bin/chmod
  Expected in: /usr/lib/libSystem.B.dylib

/nix/store/s18jyv965qzrvvl6wjs73642pbfr59fg-builder.sh: line 7: 26002 Trace/BPT trap: 5       chmod -v 755 $boot_bin
builder for ‘/nix/store/jalm2jiygcgb1qxdmvvrab20gxmwfn7c-boot-2.5.2.drv’ failed with exit code 133
error: build of ‘/nix/store/jalm2jiygcgb1qxdmvvrab20gxmwfn7c-boot-2.5.2.drv’ failed

Steps to reproduce
I add this to my ~/.nixpkgs/config.nix
{
  packageOverrides = pkgs: rec {
    boot = pkgs.boot.override {
      jdk = pkgs.openjdk8;
    };
  };
}

and run nix-env --install boot
when i use otool -L to see the linking information of the named chmod cmd i get the following output:
$ otool -L /nix/store/mhygbibbr31b6dqwxy1iqaj7g4lz03zj-coreutils-8.25/bin/chmod
/nix/store/mhygbibbr31b6dqwxy1iqaj7g4lz03zj-coreutils-8.25/bin/chmod:
        /nix/store/n9pqsflc5k0fj518k1gxkj3g2dyxvbcg-gmp-6.1.1/lib/libgmp.10.dylib (compatibility version 14.0.0, current version 14.1.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1226.10.1)

Technical details

System: OSX 10.9.5
Nix version: nix-env (Nix) 1.11.3
Nixpkgs version: "16.09pre90369.202d9e2"

Yeah, I just recently killed the compatibility with 10.9 because it was a pain to maintain and Mac users are infamous for keeping their machines constantly up to date. See https://github.com/NixOS/nixpkgs/commit/948b7f23bbc42a87e66a473c5c4543cfa5a03da7 for the offending commit and part of the motivation. Might be worth creating a ticket saying "Nix on Darwin no longer supports 10.9" or posting to the mailing list. Basically, as long as I'm the only one maintaining the stdenv on Darwin, with limited time I can't really invest in keeping compatibility with old versions of OSX. If someone wants to pick that up, or present evidence that I'm cutting out a ton of users by doing this, then that'd be helpful.

https://github.com/NixOS/nixpkgs/issues/18472

Let's keep just the other issue, right?

Yeah, I think that's right. This is (to some extent) expected behavior, for better or for worse.

Well, it depends on someone stepping up in that issue.


Runtime error in invariant manager

Hi Daniel,
Using the last version of Kind2 in develop branch I get this error very often today.
kind2 v1.1.0-283-g270c506c

Runtime error in invariant manager: Not_found

Do you know what may cause it?
Thanks

Hi Hamza,
I don't know the cause. Unfortunately, the error message is very generic.
Can you send me a Lustre model to reproduce the issue?
Thanks.

We don't really understand why but looking at running processes, lots (like a very very large set) of z3 processes are still running. We also have kind2 processes active while the main one is finished. May be when we have too much os these processes we obtain this fork issue. The solution is to kill the remaining  z3 and kind2.

#582 should fix this issue

Hi Daniel
I got the   same  error:  Runtime error in invariant manager: Not_found
Please find attached the Lustre model.
Best regards
Modem_allCycles_integer.lus.zip

Hello,
Could you please share some details about your environment?
Operating System, Kind 2 version (kind2 --version), SMT solver and version.
Thanks,
Daniel

Hi  Daniel
I'am using the  Docker framework. I tested the two versions of kind2 using:
docker pull kind2/kind2:dev
docker pull kind2/kind2
Also tested  on  two OS:
-Ubuntu 16.04.7 LTS

and suse  18

i7 machine with 8 GB of RAM
The docker version is :
Client:
Version:           18.09.7
API version:       1.39
Go version:        go1.10.4
Git commit:        2d0083d
Built:             Fri Aug 16 14:19:38 2019
OS/Arch:           linux/amd64
Experimental:      false
Server:
Engine:
Version:          18.09.7
API version:      1.39 (minimum version 1.12)
Go version:       go1.10.4
Git commit:       2d0083d
Built:            Thu Aug 15 15:12:41 2019
OS/Arch:          linux/amd64
Experimental:     false
Best regards
Imed  E.  BENNOUR
Prof. Ecole Nationale d’Ingénieurs de Sousse (ENISo)
Le lun. 14 sept. 2020 à 17:47, Daniel Larraz<EMAIL_ADDRESS>a
écrit :

Hello,
Could you please share some details about your environment?
Operating System, Kind 2 version (kind2 --version), SMT solver and
version.
Thanks,
Daniel
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kind2-mc/kind2/issues/558#issuecomment-692179020, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANY45R7ZRPFRLUUHPOBQ7ITSFZCJFANCNFSM4GI65LXA
.

Hello
After some tests, I found out that the error occurs only with the Docker
use.
Best regards
Imed  E.  BENNOUR
Ecole Nationale d’Ingénieurs de Sousse (ENISo)
Le lun. 14 sept. 2020 à 19:17, Imed Bennour (ENISo) <
<EMAIL_ADDRESS>a écrit :

Hi  Daniel
I'am using the  Docker framework. I tested the two versions of kind2 using:
docker pull kind2/kind2:dev
docker pull kind2/kind2
Also tested  on  two OS:
-Ubuntu 16.04.7 LTS

and suse  18

i7 machine with 8 GB of RAM
The docker version is :
Client:
Version:           18.09.7
API version:       1.39
Go version:        go1.10.4
Git commit:        2d0083d
Built:             Fri Aug 16 14:19:38 2019
OS/Arch:           linux/amd64
Experimental:      false
Server:
Engine:
Version:          18.09.7
API version:      1.39 (minimum version 1.12)
Go version:       go1.10.4
Git commit:       2d0083d
Built:            Thu Aug 15 15:12:41 2019
OS/Arch:          linux/amd64
Experimental:     false
Best regards
Imed  E.  BENNOUR
Prof. Ecole Nationale d’Ingénieurs de Sousse (ENISo)
Le lun. 14 sept. 2020 à 17:47, Daniel Larraz<EMAIL_ADDRESS>a
écrit :

Hello,
Could you please share some details about your environment?
Operating System, Kind 2 version (kind2 --version), SMT solver and
version.
Thanks,
Daniel
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kind2-mc/kind2/issues/558#issuecomment-692179020,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANY45R7ZRPFRLUUHPOBQ7ITSFZCJFANCNFSM4GI65LXA
.


Widget Modal window not sizing properly

Widget windows for github and udeploy widgets are not sizing propertly in HygieaCloud2.1 branch

Code fixed in the branch hygieiacloud2.1


Clean up after ourselves

Remove in/ folder and clean up intermediate files in out/ folder once the ort commands have been run

Typescript code should do the clean up.

During https://github.com/NLeSC/tortellini-on-rsd/actions/runs/896534020 a warning
You are running out of disk space. The runner will stop working when the machine runs out of disk space. Free space left: 46 MB got generated.

My suggestion on how to do this is to do it like @sverhoeven did it in his bash script https://github.com/tortellini-tools/action/blob/475e573aa7be486ad97daa8a3ed7a2136daf3801/ort/run.sh#L27
That way, we don;t have to deal with permissions

We should keep, per repo, the html and gitlab json file as they are needed as output and tests


3- and N-body expectation values for MPS

Not sure how hard would this be, but it would be nice to have methods to calculate 3- and higher-order body expectation values in a matrix product state.

Hi @nquesada, thanks for the issue! This would be a relatively straightforward generalization. Do you have any particular application in mind?

Hi @mganahl ! It is useful to have higher order correlators to study to what extent a many body state can be described in a Gaussian formalism where higher order correlation functions break into sums of products of two body ones.
If you think this could be a good first issue I would be happy to try to implement it, although likely I will have questions along the way :).

@nquesada a PR is highly welcome! I can help you along the way if neccessary


[BUG] FFlags dont work

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

Acknowledgement of preliminary instructions

[x]  I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[x]  I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[x]  I am using the latest version of Bloxstrap.
[ ]  I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
every time i click export JSON it says this
Bloxstrap Log
N/A

tried opening a notepad and clicking CTRL + V on your keyboard?

And what exactly is the issue here? It is exporting the flags in json format to your clipboard.

And what exactly is the issue here?
good question

I am going to kill myself


⚠ ClamAV has degraded performance

In f2cecc2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 80 ms

Resolved: ClamAV performance has improved in 6932235.


Powershell signing

Hi Guys,
Thanks for your awesome work on Red Teaming with Powershell. If I may suggest a onliner for desactivating the powershell Execution policy "issue", when the "powershell.exe -exec bypass" doesn't work.
function Disable-ExecutionPolicy {
($ctx = $executioncontext.gettype().getfield("_context","nonpublic,instance").getvalue($executioncontext)).gettype().getfield("_authorizationManager","nonpublic,instance").setvalue($ctx, (new-object System.Management.Automation.AuthorizationManager "Microsoft.PowerShell"))
}

src: http://www.nivot.org/blog/post/2012/02/10/Bypassing-Restricted-Execution-Policy-in-Code-or-in-Script
I've just tested it in an assignement, and it seems to work out very well. Hope that will help. Cheers !

Hey, thanks for reaching out with the cool trick!
I'm trying to get my head around how exactly to use this operationally- it doesn't seem like it makes sense to integrate into any of the scripts themselves, as we'll hit the catch-22 of the code not being able to run to determine if needs to disable the policy. In the PowerShell integration in Cobalt Strike and our weaponization with Empire we use the IEX download cradle and stay off of disk, so execution policy never even enters into our thinking as a script is not loaded off of disk. All that said, not trying to be dismissive, just get an idea of how you would envision this trick being used in these projects.

Hey, sorry for the late response.
This specific assignement had endpoints with SEP installed on, along with a tightly managed GPOs, and a proxified connectivity : IEX was out of use as Github and pastbin being filtred, even a SimpleHTTPServer within the internal network hosting the scripts is unreachable. The only "external access" was USB.
This post wasn't intented to propose a pull request, but only a suggestion when execution off disk is not possible, with a cool unprivileged powershell trick :)
Hope this helps anyhow o/

@hi-t-ch I had the same thing on a recent engagement and that trick did indeed work to get round the Import-Module issue. That said, an IEX cradle worked without a problem so it never "became" a problem, but I can vouch for the above :)

Ouh ! Forgot to close this "issue". My bad. Thanks for the confirmation @stufus o/


Add an OEP-2 compliant openedx.yaml file

This adds an openedx.yaml file, as described by OEP-2:
http://open-edx-proposals.readthedocs.io/en/latest/oeps/oep-0002.html
The data in this file was transformed from the contents of
edx/repo-tools-data:repos.yaml

@@             master       #812   diff @@
==========================================
  Files           485        485
  Lines         19839      19839
  Methods           0          0
  Messages          0          0
  Branches       2821       2821
==========================================
- Hits           3710       3698    -12
- Misses        15699      15713    +14
+ Partials        430        428     -2

Review #812 on Codecov

Powered by Codecov. Last update 29ba087...25255ae

@edx/edx-mobile-push: This one is yours.

@BenjiLee: Ok to merge?

Rebased, waiting on CI


WIP PROOF Sharded Syncer on top of PR #2939

Summary
PROOF PR to run tests of the sharded Syncer on top of the non-yet-merged PR #2939
Related issue(s)
Real PR is https://github.com/kcp-dev/kcp/pull/2675

/hold
because not expected to be merged


Fix for --hpmimage support.

This patch adds support for missing hpm image support for run tool.
By using this one can install full Firmware HPM flash with the help
of op-outofband-firmware-update-bvt.xml
./run --machines op-machines-example.xml --machine firestone3 --hpmimage ~/firmware.hpm --suite op-outofband-firmware-update-bvt.xml
Signed-off-by: Pridhiviraj Paidipeddi<EMAIL_ADDRESS>

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


This script does not work with Python 3.5

It is not possible to test if the code is compatible with Python 3.5 because this setup only works with Python 2.7
For instance, fetch_gae_sdk.py does import StringIO which is not valid on Python 3.5.
https://github.com/GoogleCloudPlatform/continuous-deployment-demo/issues/8

+1 This repository has a number of python 3.x compatibility issues.
ie. urllib2 , StringIO and so on.
@google-admin Please close the repo or note the compatibility requirements

The source on the master branch has been updated for python 3. Thanks for reporting!


feature: travis doesn't run document-only changed commit

Signed-off-by: Wei Fu<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
When the PR only changes the doc file, we should not run the integration test because it will take us so long to wait.
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Describe how you did it
Use TRAVIS_COMMIT_RANGE to make diff.
Ⅳ. Describe how to verify it
NONE
Ⅴ. Special notes for reviews

Codecov Report

Merging #1412 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1412      +/-   ##
==========================================
- Coverage   38.59%   38.58%   -0.02%
==========================================
  Files         248      248
  Lines       16529    16529
==========================================
- Hits         6380     6378       -2
- Misses       9329     9330       +1
- Partials      820      821       +1

Impacted Files
Coverage Δ

ctrd/watch.go
71.87% <0%> (-6.25%)
:arrow_down:

ctrd/container.go
47.28% <0%> (-0.78%)
:arrow_down:

daemon/mgr/network.go
68.3% <0%> (-0.5%)
:arrow_down:

daemon/mgr/container.go
49.61% <0%> (+0.17%)
:arrow_up:

daemon/logger/jsonfile/utils.go
71.66% <0%> (+1.66%)
:arrow_up:

apis/server/utils.go
66.66% <0%> (+5.55%)
:arrow_up:

LGTM


Add Spacing Props

I love rems for text and sizing grid items, but I think px is still valid for spacing. Users can increase the font size and make the text bigger, but I don't think they intend to increase the spacing. If a width is defined to contain text, then it makes sense to tie it to rem, hence the "sizes" props that already exist.
If I want to add a little spacing around a user's, avatar, I don't want that to increase with the font size. If spacing uses rem, I think it needlessly wastes space on the screen when the user increases the font size by a lot. I am big fan of "density" also being configurable by the user. I've seen email clients that allow this.
An example of somewhere I have recently seen this done well is Radix Themes' Spacing (I don't agree with their typography using px, I'm only mentioning their spacing page). They suggest density options at 90%, 95%, 100%, 105%, 110%.
https://github.com/radix-ui/themes/blob/main/packages/radix-ui-themes/src/styles/tokens/space.css
I know the openprops website says, "No px here, relative units all the way." Used in this way, and considering density is a variable, you could consider predefined spacing props as "relative".
If you don't think this is a good idea for OpenProps, then I think more examples of using spacing on the website could be useful.

good call here, I see this as a couple of tasks that are worthwhile:

remove that "no px here" statement
offer a set of size props that are px based

there's no reason that pixels should be avoided so much, and if this lib wants to offer grab-n-go prop packs, there should definitely be a pixel based set.


Added AddressType type to ptrace::linux + peek/poke user fix

This was added to the BSD ptrace API and probably should have been added to the linux API to make it easier to write code for both platforms without the user having to reexport the types to their own crate.
I went to use the latest nix myself and found that having this added would improve usability for nix users.
Could potentially add a type for what they return (BSDs: c_int, linux: c_long). Data input might be trickier as linux is *mut c_void and BSD just takes data as c_int.

@asomers do you have time to have a look at this?

So the PTRACE_PEEKUSER mistake crept in in PR #949 ?  Then you should fix it.  It isn't really a regression from the users' point of view, because there hasn't been a release since then.  I think this PR is fine, but you do need to update the existing CHANGELOG entry for 949 with the additional PR link.

Yeah there's no tests for it so I didn't realise I'd made a mistake. Unfortunately it's pretty hard to test as it relies on knowledge of the user struct and what's happening in the test executable - unless you can think of a better way?

No.  I'm not even clear on what PTRACE_PEEK_USER does.  Is it redundant with PTRACE_GETREGS?

I think it might be, PTRACE_PEEK_USER reads a single word from a given offset into the user struct, from looking at the man page it seems PTRACE_GETREGS reads the entire user struct not just one word.

Fixed (I hope)


Delete, update and simplify some FE code

This PR no logic modify.

Build failed, has unused imports in FE code


Ames House Price Prediction

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
I've added the corresponding notebook and have also updated the README.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
This PR is with reference to issue #7144.

@all-contributors bot please add @Manasi2001 for code


Ultimate vision (for 1.0.0?)

Ideally we would have JSON Schema (+ Hypertext?) representation of the API. This could be manually created using existing documentation. There are tools to help with this; still, I think this would be rather time consuming, error prone, and a very manual task. Lot's of checking and testing would might be needed. Ideally Mailgun could provide the schema via an API route (ie. https://pi.mailgun.net/schema) for consumption. Using this it would be fairly easy to create a self-generating / updating library with support for all of the api, and it would catch any future changes and additions. Heroku uses this mechanism for their node client.

TypeScript would be nice

@bojand maybe it will be a good idea to create an issue named help wanted and duplicate there and information about maintainer search from README?
Also, you can mark some issues with the help-wanted label.
It is much easier to just approve other people PRs than developing on your own. ;)


fix: Bug when go to def on jar using v32 pi_zip

Fixes: https://github.com/Olical/conjure/issues/279

Tested on v31 and v32 and is working fine! :)


create pkg-config support

Solved in commit 02ef21907e0a0ef7049c672681ecd47b26ec04a8.

However, later need to find a system independent way of installing .pc files


New version: Jackett.Jackett version 0.22.277

Pull request has been created with WinGet Releaser :rocket:
Microsoft Reviewers: Open in CodeFlow

/azp run

https://github.com/microsoft/winget-pkgs/issues/142854


Update 8.11.md

Add release notes regarding the same site policy of cookies.

Hi @jds-mendix ! Your release notes have a reference for a future section in docs. Will you add this section as a separate PR before the release or shall I delete the reference for now?


(multiple) closed captions refinement 1/2

This is part 1 of 2.
Part 1: gave bloodhounds commander a name
Part 2: will be about standardized formatting in all campaigns
I translated this change into all languages which already had translated captions.
For these languages I made a retranslation which should be checked:

italian:  @MakinDay please have a look
japanese: @Deerlord0 please have a look
koreana: @BRAIN-IS-BACK please have a look
russian: @Blueberryy please have a look
schinese: @zhongqingsong please have a look (maybe you could take a look at the tchinese changes too? I don't know)

For these I do not know anyone who could approve them, but since the change is pretty small, it's probably fine. I double checked with different translation engines.

czech
danish
french
hungarian
portuguese
tchinese
thai
turkish
ukrainian

These did not have caption translations, so it's just the new english lines:

dutch
finnish
latam
norwegian
polish
romanian
spanish
swedish
vietnamese

Hmm, Lt. Williams... he seems much cooler just existing as a nameless "bloodhounds commander".

Looks good for Japanese.

Or @ywgATustcbbs maybe you could have a look at the chinese version(s)?

Or @ModdedMarionette? I'm really not sure whom to ask when it's about the chinese versions.


Add Best Practice MXP011 - Nested List Views

MXP011 is part of Performance Bot in Studio Pro.
This will be released with Studio Pro 9.9.
This PR also adds the MXPnnn codes to the headings, as discussed already with Amir.
TODO:

[ ] rebase when MXP9 and MXP10 have been merged
[ ] add the MXPnnn codes for those [closes AA-3229]

MXP9:
https://github.com/mendix/docs/pull/3847
MXP10:
https://github.com/mendix/docs/pull/3889
closes AA-3224

 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.

@mrseanryan , could you please sign CLA?


Fix debug failure from fixUnusedIdentifier

Fixes #23942
Instead of replacing the entire arrow function node, just replace the parameter with the string "()". This avoids reformatting the body, which can contain invalid emitted nodes like " (with no closing quote) that trigger formatter assertions. (When we emit it we also emit the closing quote, but we're still using the original positions which end after the opening quote.)

This avoids reformatting the body, which can contain invalid emitted nodes like " (with no closing quote) that trigger formatter assertions.

I think we need a more general solution here. I am fine with taking this change, but we should not keep going down this wake-a-mole path, we either say we run the formatter as a best-effort attempt, or we will fix the formatting scanner so that it does not assert on invalid nodes.

I think the problem in this case is that we create a new node with old nodes as children. When we emit the new nodes we create new emitted text which emits the string literal correctly, but the formatter looks at the old source file's text which only has the opening quote.

Would cloning the nodes solve the issue?

Well, then we should always be cloning them and their comments. I think that is a simpler invariant to assert, do not repurpose nodes from the existing tree, you can only add new nodes, and that is recursive..


Fetchlog from landscape

Finally it is here! Have jobsub_fetchlog download log archives from landscape (using token) instead of from condor.

Added --condor flag to use condor_transfer_data instead
Landscape doesn't support zip archives currently. We could add it, but do we really need to support zip? Could also untar and rezip on the client... :raised_eyebrow:
Set LeaveJobInQueue to False so completed jobs leave queue immediately, but not for DAGMan jobs, since those logs are not currently transferred to dCache/landscape.
https://github.com/marcmengel/jobsub_lite_config/pull/8

Fixes: #352

Understood.  Do you want to move it back to draft, or keep it as is while you make those changes, if there aren't that many?
FWIW, I was fine with the error handling as you had it, but if you see a deficiency, we'll hold off on merging.

From: Kevin Retzke @.>
Sent: Monday, April 3, 2023 10:47 AM
To: marcmengel/jobsub_lite @.>
Cc: Shreyas Bhat @.>; Review requested @.>
Subject: Re: [marcmengel/jobsub_lite] Fetchlog from landscape (PR #359)
I'd like to improve the error handling a bit before we merge this, notably dealing with responses from landscape.
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_marcmengel_jobsub-5Flite_pull_359-23issuecomment-2D1494565370&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=9KiqjvYREicTD__c-Knj2w&m=uC87374r4gDlQOB88lAbBn7W-8tJZYaduFRFmlyBmyo9SCLdbigBPVZy_vdeReS9&s=DBtXeQjMsA3qbuYme_w8ab0CRAkXPhr2G581EFR_baw&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AEWIQ3ZSZP5ZYRXFW5W3QPDW7LWIPANCNFSM6AAAAAAWPBS34Q&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=9KiqjvYREicTD__c-Knj2w&m=uC87374r4gDlQOB88lAbBn7W-8tJZYaduFRFmlyBmyo9SCLdbigBPVZy_vdeReS9&s=mg2RoWjHm6lM5Qm6BdI0g9E34YuQNuQEpx2XPaNVD_4&e=.
You are receiving this because your review was requested.Message ID: @.***>

I guess I just wanted to include the response text, so pretty small change. Should be GTG now.

LGTM.


Shave 13 bytes off the gzipped size

This isn't the PR I referred to in #9; just couldn't resist channeling my inner Jason Miller first. With these changes the minified size of the default loader drops 11% to 563 bytes (from 633); the gzipped size drops 3.4% to 369 bytes (from 382).

I really should add tests to this project now :D
Thanks you very much! Any byte saved is appreciated!


Could Have Features

Could Have Features
Could Have Feature, features that could improve the product, that are in the plan but have not been put in the works.
Could Have 🥳:

Allow users to upload their own documents to be indexed and used with the platform for information retrieval.
Integration to email. (e.g. user can get a response via email instead of accessing the web app directly)
Loading bar while generating response

Added issues 2 & 3


Split helm install into core and observability

Summary
Create a separate helm chart for observability
Details and comments
Will add CI for building the observability helm chart after this merges

Good catch. I've updated the readmes for now, will do the deployment doc with the CI update (after this merges)


Drop test, start and end selectors from schema

We offer too many choices to select the timestamps and while these are optional in the schema it's likely that users can just add them to the upload query anyway.
Let's do the upload in an opinionated way, requiring the parameters and dropping these from the schema.
@johnaohara @willr3 Thoughts?

If we register an application via a webhook and that application receives a notification about a new run, how would that downstream application be able to obtain the start/end timestamps for that run?  Is that data included in the Notification?

Webhooks receive the Run JSON, that has already the timestamps extracted. I don't see much relevance here.

After our separate conversation I realise that this is a different issue of schema definition rather removing the use of selectors. I would vote for dropping selectors in schema and making the UX simpler.

Implemented.


meta-qt6 not available in kb_recipes.json

QT6 is available in the BlackDuck knowledge base, and the Qt company has a meta-layer for it (meta-qt6). We cannot let import_yocto_bm map it in the Blackduck project as it does not keep a kb recipe for it.

Hi Paul.
Thanks for reaching out.
Meta-qt6 is not a recipe provided by Openembedded, and consequently cannot be imported as a Yocto dependency using either Detect or the import_yocto_bm utility.
I would recommend that you investigate the new beta script I have been developing to improve the Yocto scan process adding Signature scanning.
You can find this at https://github.com/matthewb66/bd_scan_yocto
The readme provides full explanation of the operation of the script.
Please provide feedback on the script as I want to move it from being Beta.
Thanks
Matthew Brady
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Paul Geurts @.>
Sent: Monday, December 4, 2023 8:00:39 AM
To: blackducksoftware/import_yocto_bm @.>
Cc: Subscribed @.***>
Subject: [blackducksoftware/import_yocto_bm] meta-qt6 not available in kb_recipes.json (Issue #19)
QT6 is available in the BlackDuck knowledge base, and the Qt company has a meta-layer for it (meta-qt6). We cannot let import_yocto_bm map it in the Blackduck project as it does not keep a kb recipe for it.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/blackducksoftware/import_yocto_bm/issues/19__;!!A4F2R9G_pg!e-WtVCIHiQaVGMUklx54322FsxZrk8a_OuEeDWQjxz1oUT72fkXCLIuJbaGdq5dMaw2e6vtNavD4yirAQqiqbVA$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AED5THBNK3BOCAZSRTZUQADYHXCPPAVCNFSM6AAAAABAF6CF4KVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAZDGOBQG44TCNQ__;!!A4F2R9G_pg!e-WtVCIHiQaVGMUklx54322FsxZrk8a_OuEeDWQjxz1oUT72fkXCLIuJbaGdq5dMaw2e6vtNavD4yirAT1FtXJU$.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hi Matthew,
Thank you for the feedback. I think meta-qt6 not being supported as it is not openembedded is strange, as meta-qt5 is supported by the tool. I've asked our Synopsis FAE for support on it but I am not sure whether this tool is even supported.
The beta script you mention might be worth looking into. Thanks for the pointer. We will find out whether your tool is matching our expectations and whether it fits our environment.
br, Paul


add tslib

For your convenient - adding tslib library solve the issue until it will be fixed in the library

closing this PR as it was just for demonstration purpsoes


Terminal should provide API to notify data cleaned by control sequences

I noticed that the terminal no sends Operating System Commands (OSC) like setting the window title. Right now the task runner filters them but this gets more and more complex and I am actually not an expert with this. So I would like to request that the terminal provides API to notify data without control sequences. Currently I used these two regular expressions to filter them:
	private static ANSI_CONTROL_SEQUENCE: RegExp = /\x1b[[()#;?]*(?:\d{1,4}(?:;\d{0,4})*)?[0-9A-ORZcf-nqry=><]/g;
	private static OPERATING_SYSTEM_COMMAND_SEQUENCE: RegExp = /\x1b[\]](?:.*)(?:\x07|\x1b\\)/g;

There are two unknowns to me right now with this:

Various control sequences can do a whole lot of stuff, so where is the best place to gather this data? It might be worth seeing if adding listeners to addChar and lineFeed will give what you need.
How do this efficiently. Because a lot of data can stream through the terminal, we should probably only enable this when it's explicitly requested (ie. only fire the event if something is listening). It should probably also be batched.

PR welcome upstream.

From a problem matcher point of view it would be enough sending the line. We could sent it when a linefeed is added. Then there is still the problem of a line content changing. We once discussed the idea of a line identifier to signal that. I agree that sending single characters might not be a smart idea in terms of performance.

If it's just on line feed then many characters could go missing, eg. "abc"  "def"  would result in "def" only. Maybe this is not something we care about, but it could lead to some obscure bugs if people relied on this new API.

This is a good example why something like a line model would be helpful: https://github.com/Microsoft/vscode/issues/22065#issuecomment-305076675.
After looking into the issue it looks like that pester constantly 'repaints' the whole screen hence the terminal sending the same line onData over and over again. For me it is very hard to tell whether I have seen that line and whether I have processed it already (the same content). What would be cool if the terminal could provide a line model where is line has an id (number) and changes are signaled on that model instead of sending raw data.

@dbaeumer I actually added an ID concept to the buffer when doing the selection stuff (https://github.com/sourcelair/xterm.js/pull/670/commits/65256c8751831b8f4c7bbc8386953ce042b12897) but I ended up not needing it and pulling it out, instead the selection listens to when the buffer is trimmed and adjusts the indexes of its own model.
It really didn't feel very good adding it as it introduced much more complexity to the already complex circular list and added the overhead of an addition object for every entry - which is a lot when you need to create tens of thousands of these things in an instant. There were other issues that come up with this like IDs being interleaved due to programs having the ability to delete/insert lines so there's no way I could think of to optimize it nicely, leading to a slower, more memory hungry terminal.
It would probably be easier and more performant to just deduplicate these errors on your end. For example you could record a hash for current problems and check it before adding new problems, or perhaps even better, record the raw terminal line of the error so you don't need to parse it again for duplicate problems.

I implemented the deduplicate yesterday but it feels like a hack on my side :-) I need to create a unique key for something that doesn't necessarily have one. I still think having a line bassed model comparable to what the editor exposes would be a nice thing. Me giving the raw data is comparable to the editor signaling raw cursor moves and text changes. Instead I get a nice model with text edit events. And I am not interested in color information like I am not interested in this on an editor buffer.

Just for the record. I can not filter on the line itself from the terminal without knowing if I have parsed that line already. Matchers are multiline aware and there could be two equal lines in one output on purpose since they difference is made by a line above. Me knowing this is implementing a line model on top of it :-)

@dbaeumer isn't the point of programs rewriting lines that the lines do change and that they may need to be reparsed?
Here's one idea that might work for both of us: exposing some data stream you can listen to as in the original comment, but only ever pushing data if it's on the last line in the terminal. Would this cover your use case? For the PS case https://github.com/Microsoft/vscode/issues/22065#issuecomment-305076675 you would get multiple lines for the last line but maybe this could be more clever somehow in recording when the cursor moves and only reporting the line a single time (not by using IDs but by only reporting once after a lineFeed call inserts a new line.

@Tyriar yes, it would be cool to be able to reparse a line when it changes. As I tried to outline in https://github.com/Microsoft/vscode/issues/29840#issuecomment-313638565 this needs to have some sort of model to do it right. I can't simply decide things on the content of a line. So instead me implementing a line model on top of ANSI control sequence characters I was somehow hoping that the terminal could expose its. I my simple mind the terminal works like this: program -> Output with control characters -> line model -> view model (does the wrapping, could do folding, ...) -> DOM.
I think the last line suggestion will work for parsing problems. However when exposing this as API in the extension host I think it might be a little to limited.

@dbaeumer currently there is only the line model, no view model. I did a big write up on introducing one to properly support reflowing lines here https://github.com/sourcelair/xterm.js/pull/644#issuecomment-298058556
This has been a pretty long discussion across a few issues but to clarify where we are now, here are my concerns:

Extra memory overhead in the buffer: I'm currently in the process of trying to push down the terminal's memory usage as I need to store more data against each character and memory usage right now is approaching unacceptable limits imo https://github.com/sourcelair/xterm.js/issues/484#issuecomment-312708021
Here is a snapshot of a 160x24 terminal with 5000 scrollback filled which is not out of the ordinary, I typically have two of these running just in the window with Microsoft/vscode opened:

Extra processing when data/lines are added to the buffer: back in December I made a big effort improving the terminal's performance https://github.com/Microsoft/vscode/issues/17875 by attacking it in a number of ways. The main take aways from this experience was that trying to write a high performance terminal in JavaScript/DOM is very hard because processing and displaying data is so expensive. I can now safely say that xterm.js is the fastest JS terminal out there by a long shot and I want us to maintain this status. A proper view model that can track lines that changes would amount to a significant amount of processing in the worst case which would be generally unnecessary for other use cases.

Some possible solutions:

Introduce a new event that you can listen to that fires when a line feed is added to the last line, ie. only fire an event once per line and ignore lines being updated or lines being inserted.
Pros: It doesn't involve storing any more data or doing anymore processing, other that pulling the string from the buffer.
Cons: it probably isn't sufficient to cover all use cases.

Add an ID to each buffer line (https://github.com/sourcelair/xterm.js/commit/65256c8751831b8f4c7bbc8386953ce042b12897) and fire an event when there is a line feed against a line or the cursor moves away from it (specifics here would need to be figured out).
Pros: It's a more complete solution and covers more use cases. We need an addition JS object and number associated with each line, this probably doesn't add up to that much data as it's on the line rather than the char.
Cons: Additional complexity within the CircularList data structure. I'm not so sure just yet when exactly we should fire these events. The IDs will be out of order in some cases which may be confusing to consumers. These events may still fire when the line did not change (eg. if the program/script erased and rewrote the line).

Introduce a proper view model that throws events only when a line changes.
Pros: I think this is the ideal solution from your perspective.
Cons: All the cons of number 2. The amount of string comparisons needed here is out of the question in the worst case with my current thinking, unless this was disabled by default and generally did not run on a lot of data.

Commentary on the solutions:

It doesn't look like this is sufficient for problem matchers
This could work, but I'm not sure it helps much with the issue in https://github.com/Microsoft/vscode/issues/22065#issuecomment-305076675
This behind an option on terminal initialization is probably the best way to go, but it's also the most difficult and time consuming to implement. Work on this is blocked on the following active PRs: https://github.com/sourcelair/xterm.js/pull/756 & https://github.com/sourcelair/xterm.js/pull/717, and it should be done in a way that accommodates future line reflowing https://github.com/sourcelair/xterm.js/pull/644#issuecomment-298058556

@Tyriar thanks a lot for the detailed explanation. Having solution 2 will for sure make my life easier and will not make things worse on my end. Regarding option 3: I am not interested in the view model since I need lines unwrapped. The line model is what would work best for me assuming that the line mode doesn't have any \n in there to make lines visually wrap .

@dbaeumer option 3 including reflow https://github.com/sourcelair/xterm.js/pull/644#issuecomment-298058556 is required in order to get unwrapped lines
Not sure when I'll have time to work on this, pretty swamped atm. It might be a good opportunity to collaborate if this is high priority for you?

@Tyriar I will be out on vacation until 2. August. I think we need to address this sooner than later since we can't assume that problems always fit onto one line.
An interims solution could be as follows: can we introduce a custom control sequence (e.g. a operating system control sequence) that describes the new lines inserted to wrap the text. I could then filter them on my end since I would be able to distinguish them from normal newline. Would this be something you can add in July then I would catchup with it for the July build ?

@dbaeumer I actually expect you to get the unwrapped lines currently, xterm.js just wraps the lines when they are going to overflow https://github.com/sourcelair/xterm.js/blob/2ebc926f4e438c77c4da702b14f82f46424120a8/src/InputHandler.ts#L55, that's the only wrapping it knows about.
On Windows though I'm not sure rows are ever marked to be wrapped (not 100% sure), at least on PowerShell.


Remove bqn mode

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.05 Release Notes (or backporting 22.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.

Now it is!

Breaks emacsPackages.ligo-mode on aarch64-darwin.
source> unpacking source archive /build/archive.tar.gz?sha=d1073474efc9e0a020a4bcdf5e0c12a217265a3a
error: hash mismatch in fixed-output derivation '/nix/store/3kzk0rbsxjyx1y2dz8narp8bq88jr5n0-source.drv':
         specified: sha256-qlLBgKeabDi0UZNOll4PgXquterQN2zHxoY52AQ980Y=
            got:    sha256-wz9DF9mqi8WUt1Ebd+ueUTA314rKkdbjmoWF8cKuS8I=
error: 1 dependencies of derivation '/nix/store/npavsm83j6wpaqbmbqp95w43gxs39knx-emacs-ligo-mode-20230302.1616.drv' failed to build

I will revert this.


Litewallet Feature: Refactor the Login View

Value of feature
5 points

Feature Name
Value(Ł)

Refactor Login View
2Ł

Is your feature request related to a problem? Please describe.
The current app will not have a recent Litecoin price anywhere.  Currently it is hidden.
Also, users like the idea of sharing a QR code
Describe the solution you'd like
Add a recent Litecoin price in the login view and a QR button in the keypad (pinpad) in the login so users can easily show an address
Describe alternatives you've considered
Do nothing
Proposed Login Views

Current
V1
V2
??

???

@mosadialiou its important that we refactor for both platforms (Android and iOS)

@mosadialiou its important that we refactor for both platforms (Android and iOS)

@mosadialiou its important that we refactor for both platforms (Android and iOS) so that the layout is the same.

I don't like the idea to add the recent LTC price on this view.

@mosadialiou its important that we refactor for both platforms (Android and iOS) so that the layout is the same.

I don't like the idea to add the recent LTC price on this view.

@mosadialiou What about the other elements?  Then login view send button (in Android) is a security risk because someone can send LTC without having logging in and it shows their balance....so that needs to be removed (why it was removed from iOS)
But the show QR is helpful. What are your thoughts on that?

@mosadialiou What about the other elements?  Then login view send button (in Android) is a security risk because someone can send LTC without having logging in and it shows their balance....so that needs to be removed (why it was removed from iOS)
But the show QR is helpful. What are your thoughts on that?

@kcw-grunt I don't follow this:

Then login view send button (in Android) is a security risk because someone can send LTC without having logging in and it shows their balance....so that needs to be removed (why it was removed from iOS)

Here is the login view on Android (what is bad here?)

@kcw-grunt I don't follow this:

Then login view send button (in Android) is a security risk because someone can send LTC without having logging in and it shows their balance....so that needs to be removed (why it was removed from iOS)

Here is the login view on Android (what is bad here?)

Thanks @mosadialiou .

This one of the options but we never agreed on the final design. Note it shows v1, v2 , ???
We should discuss a little on the final location of the QR Code and more imagery to show users what it does.

OK sound good!

cc: @antematijaca
I'm on the fence on this. I wondered what you thought?
We need to have a button to open a QR code for users to show to payers, but I'm not sure where to put it.  Or, how big etc.

I would go with V1 over V2 for placement. V2 has the QR code icon located within the keypad and it's not logical. I would expect this to be an override function like face ID, or QR code scanner. Not something unrelated to PIN number.
Maybe even place it to the right side of the logo to avoid any confusion that this is a logo or a menu. I think that size like this could work.
Some text below could also come in handy. Like "Show address" or "Share address"
One more thing - current LTC price could maybe show like "1 LTC = $218.74" because some people could see it as their current balance.

@antematijaca I think you are right.  We are having a big problem with the balance in the Android in that the devices are of various resolutions and sizes and different screen sizes and users can change the fiat which causes alot of issues with space.
Do you think you can mock a better option?

@kcw-grunt sure, I'll give it a try tomorrow and paste a couple of ideas.

I've managed to sketch some ideas, feel free to comment!
There are some changes to keypad are and if it doesn't make sense, just ignore it. I removed the text below numbers and also added "Forgot your seed phrase..." below. I've seen that in current app so wasn't sure if this is going to be included in an update.
login_v1.png!
login_v2.png!
login_v3.png

Dang @antematijaca v1 is it!!
Price Changes

Spending weeks working in the help center it is a sad reality that the daily price really confuses people. Some people actually think that when the download Litewallet, they get free Litecoin (seeing the price)....this is not a joke
The other problem is the Android screen sizes mean the daily price is a big problem in the header.  Many fiats need the extra space placing this in the login fixes the issue by removing the price in the header.
Forgot Password

I like this too because it clears the space between the PIN
Next steps
Adding to the next release

Closing with the new layout.
Android: https://github.com/litecoin-foundation/loafwallet-android/issues/242
iOS: https://github.com/litecoin-foundation/loafwallet-ios/issues/248


Implement leftover Slave API methods

The ROS Slave API docs aren't very informative about what is what in there, so we'll need to look further into certain details regarding "getBusStats" and "getBusInfo".

I do know some people who know some ROS dev's/interned at OSRF for a summer. Let me see if I can get you some more up to date info. I just thought about that

That would be great! I've posted a question on ROS Answers to see if anyone could give me any clues. There were questions before, but nobody answered them.

No luck

Too bad. Still greatly appreciated. As mentioned in my question, even officially distributed client libraries just put a "TODO" there, so I did the same for now, and I've put it so it returns an error code that signals failure to handle the command. I'll probably end up solving this by analyzing the C++ and Python code, but honestly these 2 commands provide no actual functionality, so I'm in no hurry in doing so.

According to my friend they think that that API is deprecated and has been replaced with NodeHandle
http://wiki.ros.org/roscpp/Overview/NodeHandles

I'm not sure how that relates to these two API methods. But it they're deprecating those things, then great. This node handle thing seems interesting by itself. I've made something similar for parameters, but this could be interesting for the ROS interface in general. But... this is a typical singleton and I hate that - one should be allowed to run 20 ROS nodes within one binary if they want to (even if that makes them incompatible with roslaunch).


Fix issue 86 "make lint does not check app files"

Fix Makefile to include files in apps directory
Minor fixes to app files that failed linting.

Current coverage is 100.00%

Merging #88 into master will not affect coverage as of 214b12a

@@            master     #88   diff @@
======================================
  Files           29      29
  Stmts          952     952
  Branches         0       0
  Methods
======================================
  Hit            952     952
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 214b12a

Powered by Codecov. Updated on successful CI builds.


[CDNJS] Host library on cdnjs

Someone can help with this issue?

@PeterDaveHello @PeterBot helped me with that, so soon our library would be available on @cdnjs

@jsdelivr adds library maximum 3day, if much of PR (50+ in day). I not understanding why @cdnjs takes much of time? it's interesting? maybe @cdnjs needs help?

@cdnjs one of bad cdn i ever see. They delay merging PR insteadof helping. I just love @jsdelivr

Sorry to make you feel that but we have tons of issues to handle so it's reasonable to have delay and we already tried the best to handle that, @cdnjs as you can see has almost 3k libraries to maintain and have more than 700 issues waiting on the list, that's why it'll be slower, hope this make your understood, it's a fully free and open source project, feel free to point out the problem or fix it.


Naming test files with the issue may be giving a hint to GPT

Realized this while showing an example to someone else, the filename was a big hint of what the problem was. The filename is part of the compiler diagnostic so GPT knows, and it could be giving it a hint. In real life of course, the filename will be something unrelated to the problem.
I'll make a small script in the test directory to copy the test file to some temporary file and run CWhy on that, this way the filename is some random string and does not give any indication as to what the problem is.

It would be a good experiment to randomize all the file names in the error message (and even variables and functions) to quantify the impact of names.

I believe this is all fixed with the anonymizer script so we can close the issue, correct?

Correct. Maybe we can try what you suggested above at some point: randomizing all file names, event standard library ones.

True, at least as a test.


💖 Improve Master CSS packages with SOLID

Description
Due to the previous intensive iterative product features, some structures and responsibilities were not distinguished enough.
We intend to improve this problem once and for all to avoid more side effects in the future.

@master/css core, initialize config
@master/css-extractor statically extract class names from different files
@master/css-extractor.vite
@master/css-extractor.webpack
@master/css-validator validate class names
@master/css-renderer methods
@master/eslint-config-css
...

Usage
import { readConfig } from '@master/css-shared'

import CSSExtractor from '@master/css-extractor'
import CSSValidator from '@master/css-validator'

import { CSSExtractorPlugin } from<EMAIL_ADDRESS>import { CSSExtractorPlugin } from<EMAIL_ADDRESS>
// no main export
import { isClassValid, reportErrors, createValidRules, ... } from '@master/css-validator'

🎉 Released in https://github.com/master-co/css/releases/tag/v2.0.0-beta.160


Update dependencies

Coverage remained the same at 85.562% when pulling 3f6503f2b68657cf0f74a67668b55dd495dd23e9 on rel_branch into e250784c76503ef66a75778a2b7e22270a68d1e0 on master.


[OSPRH-5019] Consider OpenStackClient pod status when calculating CR's ready state

We should consider the state of the OpenStackClient's associated pod's state when setting the OpenStackClientReadyCondition on the OpenStackClient resource.  This fixes a problem where the parent OpenStackControlPlane resource mistakenly assumed that everything was fine when in fact the OpenStackClient pod was failing.

recheck


Update README.md

Fixed typo

Coverage remained the same at 100.0% when pulling 5c9a84a8c998baf8a53e1300ba1dc729c38345ca on berniedurfee-ge:master into 39971a16a01a0aeaa3289767d2e2c6c1ee0e1ab7 on johnbellone:master.


Fix CPU_SET and CPU_ISSET

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

The Travis failures all seem to come from broken stdlib installations.

@bors: r+ afa77dfa0e2a821e38c99e239b5c3a10d9251162
Thanks!

:hourglass: Testing commit afa77df with merge b484e6f...

:sunny: Test successful - status-appveyor, travis


add holiday 2019-05-02 2019-05-03 for XSHG

2019-05-02 and 2019-05-03 are labor's day holidays(addition holiday only this year) in china, not trading day

Thanks for catching this! Fixed in #79.


some problems about dependency

When I installed the dependencies I had a problem with the linux kernel version not matching. So could you tell me the version number of the Linux kernel you are using with this work, please.

I'm currently using kernel version 5.13.0-39-generic with Ubuntu 20.04.4 LTS. Hope this helps!
-Alex


python310Packages.panflute: 2.1.3 -> 2.1.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.panflute is: "A Pythonic alternative to John MacFarlane's pandocfilters, with extra helper functions"
meta.homepage for python310Packages.panflute is: "http://scorreia.com/software/panflute"
meta.changelog for python310Packages.panflute is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 2.1.5 with grep in /nix/store/b0bcm6x1p43c232hpyhnavbgcp6p4p5w-python3.10-panflute-2.1.5
found 2.1.5 in filename of file in /nix/store/b0bcm6x1p43c232hpyhnavbgcp6p4p5w-python3.10-panflute-2.1.5

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

8 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
pandoc-acro
pandoc-include
python310Packages.panflute
python39Packages.panflute

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/b0bcm6x1p43c232hpyhnavbgcp6p4p5w-python3.10-panflute-2.1.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.panflute https://github.com/r-ryantm/nixpkgs/archive/69cb613852db5ec56250d664ee1a98a0dbd8af18.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/b0bcm6x1p43c232hpyhnavbgcp6p4p5w-python3.10-panflute-2.1.5
ls -la /nix/store/b0bcm6x1p43c232hpyhnavbgcp6p4p5w-python3.10-panflute-2.1.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

pandoc-acro
pandoc-include
python310Packages.panflute
python39Packages.panflute

Maintainer pings
cc @Synthetica9 for testing.

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

4 packages built:

pandoc-acro
pandoc-include
python310Packages.panflute
python39Packages.panflute


forward looking list of systems seminars for Ken to use on web site

Orran mentioned Ari Trachtenberg has a spreadsheet of these going forward - if possible we should get it to update the site farther in advance.

@kkrudolph can you please put the link to the spreadsheet here so others may use it?  Thanks!

@msdisme https://docs.google.com/document/d/1IPtE4UgqGD8eAg04t3wn31NZibB0_j0IGRQNEKypFW4/edit?usp=sharing


Dependent on Java to execute?

It appears that java must be installed to run the released binary. Normally I'd expect a release to act as a standalone binary that doesn't require java to run, especially something which I need to put into a pipeline step.
If it must use java, this would be helpful to document in the README.
It could also be helpful to create an image that could be used readily in a CICD pipeline (instead of downloading the binary).

Good point. Will add that. Thanks for pointing this out.


use node-grok from npm

fixes #213
we no longer have to depend on our own fork containing a fix and can
correctly rely on node-grok from npm

@demmer we can finally use an official npm build here.

+1